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

Status: RESOLVED FIXED

(Testing :: Raptor, enhancement, P1)

Description

[Robert Wood [:rwood]]

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

Comment 1

[Robert Wood [:rwood]]

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

Comment 2

[Robert Wood [:rwood]]

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.

Comment 3

[Robert Wood [:rwood]]

Attachment: Bug 1603222 - Enable geckodriver debug logging by default in CI for raptor-browsertime tests

Comment 4

[Robert Wood [:rwood]]

https://treeherder.mozilla.org/#/jobs?repo=try&tier=1%2C2%2C3&revision=06058b9e66f0cbd8115b1a1661d8f169ce50eb78

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2c0887a202ad
Enable geckodriver debug logging by default in CI for raptor-browsertime tests r=tarek,stephendonner

Comment 6

[Robert Wood [:rwood]]

Created mozilla-browsertime PR to enable geckodriver verbose logging:

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

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/2c0887a202ad

Comment 8

[Robert Wood [:rwood]]

(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!

Comment 9

[Beth Rennie [:beth] (she/her)]

PR reviewed and merged.

Comment 10

[Robert Wood [:rwood]]

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

PR reviewed and merged.

Thanks Barret!