1236947 - [tc-gip] Retrieve and store logcat

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Johan Lorenzo [:jlorenzo]]

We currently don't have gecko logs. One simple way to get them would be to add:
> adb logcat -v threadtime -r LOG_FILE_MAX_SIZE_in_KB &> logcat.log

Comment 1

[Johan Lorenzo [:jlorenzo]]

There's a refactor of the configuration ongoing in bug 1225458. We'd get less conflicts if that bug lands first.

Comment 2

[Johan Lorenzo [:jlorenzo]]

Attachment: MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Review commit: https://reviewboard.mozilla.org/r/29699/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/29699/

Comment 3

[Johan Lorenzo [:jlorenzo]]

Attachment: MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Review commit: https://reviewboard.mozilla.org/r/29701/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/29701/

Comment 4

[Greg Arndt [:garndt]]

Comment on attachment 8704682 [details]
MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

https://reviewboard.mozilla.org/r/29699/#review26535

Comment 5

[Greg Arndt [:garndt]]

Comment on attachment 8704683 [details]
MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

https://reviewboard.mozilla.org/r/29701/#review26537

::: testing/taskcluster/tasks/phone_test.yml:62
(Diff revision 1)
> +        adb logcat

If this works, good for me!

Comment 6

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8704682 [details]
MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29699/diff/1-2/

Comment 7

[Johan Lorenzo [:jlorenzo]]

https://hg.mozilla.org/integration/b2g-inbound/rev/c99e3e31b19e74b9aee97c551b09ded9df925d2e
Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Comment 8

[Johan Lorenzo [:jlorenzo]]

https://tools.taskcluster.net/task-inspector/#PkiijXuCRAKD_acSHXyMmA/0

The tests are still executing. The logcat file is correctly created and published. It only gets the first line of the log, though.

It it possible to let a process run in the background, Greg?

Comment 9

[Greg Arndt [:garndt]]

(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #8)
> https://tools.taskcluster.net/task-inspector/#PkiijXuCRAKD_acSHXyMmA/0
> 
> The tests are still executing. The logcat file is correctly created and
> published. It only gets the first line of the log, though.
> 
> It it possible to let a process run in the background, Greg?

We have other scripts that kick off processes that run in the background like here:
https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/scripts/tester/test-linux.sh#108

Perhaps the commands being passed in could be moved to a script kept at gecko/testing/taskcluster/scripts/tester/ and then that will call things to run in the background? Just throwing an idea out there.

Comment 10

[Johan Lorenzo [:jlorenzo]]

https://hg.mozilla.org/integration/b2g-inbound/rev/6232c5cef7c66907fb08c74f2ad96d7fcdb236e8
Backout Bug 1236947 - [tc-gip] Retrieve and store logcat

Comment 11

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8704682 [details]
MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29699/diff/2-3/

Comment 12

[Johan Lorenzo [:jlorenzo]]

https://hg.mozilla.org/integration/b2g-inbound/rev/9e922c947dddfa78817cf9ce4cf37fc912e7cad9
Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Comment 13

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8704682 [details]
MozReview Request: Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/29699/diff/3-4/

Comment 14

[Johan Lorenzo [:jlorenzo]]

https://hg.mozilla.org/integration/b2g-inbound/rev/5ace53d088bc58ea3c621659c9604c56540e6045
Bug 1236947 - [tc-gip] Retrieve and store logcat r=garndt

Comment 15

[Johan Lorenzo [:jlorenzo]]

(In reply to Greg Arndt [:garndt] from comment #9)
> Perhaps the commands being passed in could be moved to a script kept at
> gecko/testing/taskcluster/scripts/tester/ and then that will call things to
> run in the background? Just throwing an idea out there.

That's not a bad idea! After looking again at the PR, I might have misordered the command line arguments. I pushed a new version with just that change. If that still doesn't work, I'll give the script a shot.

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/5ace53d088bc