Closed Bug 1603222 Opened 2 years ago Closed 2 years ago

Enable geckodriver debug logging by default in CI for raptor-browsertime tests

Categories

(Testing :: Raptor, enhancement, P1)

Version 3
enhancement

Tracking

(firefox73 fixed)

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: rwood, Assigned: rwood)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

To have geckodriver (including marionette) debugging turned on:

  • Set this pref marionette.log.level=Trace

  • And turn on verbose (level 3) logging on the browsertime command line, i.e. change this line to '-vvv':

https://searchfox.org/mozilla-central/rev/62a130ba0ac80f75175e4b65536290b52391f116/testing/raptor/raptor/raptor.py#717

Priority: -- → P2
Assignee: nobody → rwood

Note: This also requires a patch in browsertime itself to enable the marionette trace logging (:barret provided patch on slack).

Increasing the browsertime log level to 3 (adding the -vvv) was landed in Bug 1592063. The only patch required here on the raptor framework side is setting the pref.

I will also create a pull request on the browsertime github side to get verbose logging from geckodriver.

Summary: Enable geckodriver debug logging by default in CI for raptor-browsertime android tests → Enable geckodriver debug logging by default in CI for raptor-browsertime tests
Status: NEW → ASSIGNED
Priority: P2 → P1
Keywords: leave-open
Pushed by rwood@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c0887a202ad
Enable geckodriver debug logging by default in CI for raptor-browsertime tests r=tarek,stephendonner

Created mozilla-browsertime PR to enable geckodriver verbose logging:

https://github.com/mozilla/browsertime/pull/59

(In reply to Robert Wood [:rwood] from comment #6)

Created mozilla-browsertime PR to enable geckodriver verbose logging:

https://github.com/mozilla/browsertime/pull/59

Hey :barret would you mind having a look at that pull request when you have a chance? I just took the patch you provided on slack and applied it there on the master branch. Thanks!

Flags: needinfo?(brennie)

PR reviewed and merged.

Flags: needinfo?(brennie)

(In reply to Barret Rennie [:barret] (he/him) from comment #9)

PR reviewed and merged.

Thanks Barret!

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Keywords: leave-open
Target Milestone: --- → mozilla73
You need to log in before you can comment on or make changes to this bug.