New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 611170 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

BattOr agent needs better serial logging

Project Member Reported by charliea@chromium.org, May 11 2016

Issue description

We now have an end-to-end BattOr test running on the Catapult CQ. However, we're occasionally encountering a bug where something is happening that causes that BattOr agent to crash, and we have no means of debugging what's happening.

We need to instrument the BattOr code so that it's possible to pass a command line flag and write to a log file. Then, we can upload that log file to cloud storage when something goes wrong during a Telemetry test or an end to end test in the Catapult commit queue.
 
Project Member

Comment 2 by bugdroid1@chromium.org, May 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/97feb4d0f2448ba8c3e4d0e5b5191a1ecbfb39d2

commit 97feb4d0f2448ba8c3e4d0e5b5191a1ecbfb39d2
Author: charliea <charliea@chromium.org>
Date: Fri May 13 16:34:57 2016

[battor agent] Add option to log serial communication

This should help us to identify where we're experiencing problems on the
Catapult CQ.

BUG= 611170 

Review-Url: https://codereview.chromium.org/1970243002
Cr-Commit-Position: refs/heads/master@{#393547}

[modify] https://crrev.com/97feb4d0f2448ba8c3e4d0e5b5191a1ecbfb39d2/tools/battor_agent/battor_connection_impl.cc
[modify] https://crrev.com/97feb4d0f2448ba8c3e4d0e5b5191a1ecbfb39d2/tools/battor_agent/battor_connection_impl.h

Status: Fixed (was: Started)

Sign in to add a comment