[Raptor] Swap out log stream parser

RESOLVED FIXED in 2.2 S14 (12june)

Status

Firefox OS
Gaia::PerformanceTest
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Eli, Assigned: Eli)

Tracking

unspecified
2.2 S14 (12june)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
The adbkit-logcat stream parser does not handle the binary output from a Nexus 5 L device. Swap out the stream parser to something string-non-binary-based to see if we can overcome this issue.

Comment 1

3 years ago
Created attachment 8612876 [details] [review]
[gaia] eliperelman:bug-1169636 > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Attachment #8612876 - Flags: review?(rwood)

Comment 2

3 years ago
Comment on attachment 8612876 [details] [review]
[gaia] eliperelman:bug-1169636 > mozilla-b2g:master

LGTM, with one question in the PR. Applied patch, blew away gaia/node_modules, did a make raptor and ran launch test on flame-kk. R+ assuming that you (or someone) has actually tried this with the Nexus 5L.
Attachment #8612876 - Flags: review?(rwood) → review+
(Assignee)

Updated

3 years ago
Whiteboard: checkin-needed
(Assignee)

Updated

3 years ago
Keywords: checkin-needed
Whiteboard: checkin-needed

Updated

3 years ago
Keywords: checkin-needed
(Assignee)

Comment 3

3 years ago
In master: https://github.com/mozilla-b2g/gaia/commit/448614871aa79653494fada91be98c341ffa0926
Try run: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=53fd0e687b5ea9af4688c2232a5305bbf0446e31
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Comment 4

3 years ago
http://docs.taskcluster.net/tools/task-graph-inspector/#6EQxayjfRe6-Fju6fj9r_g

The pull request failed to pass integration tests. It could not be landed, please try again.
Target Milestone: --- → 2.2 S14 (12june)
You need to log in before you can comment on or make changes to this bug.