Closed Bug 1703345 Opened 3 years ago Closed 3 years ago

Make the mozperftests use consistent --verbose options

Categories

(Testing :: mozperftest, enhancement, P2)

Default
enhancement

Tracking

(firefox89 fixed)

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: acreskey, Assigned: acreskey)

Details

Attachments

(1 file)

Most of the android perftests do not use the --verbose option, which makes debugging failures difficult.

This bug is to consistently add the --verbose option so that we can easily see all the logs (e.g. geckodriver)

Patch incoming.

I don't think we should be making this a default behaviour, that's a ton of logging that we won't need most of the time. Could we either add this temporarily while you're debugging the issue (and remove it afterwards) or can we add a taskcluster action to run a task with the --verbose flag like we do for gecko-profiling?

Ah, I didn't realize that there was a meaningful cost to the logging.

It turns out it wasn't needed for this feature because it was enabled on other tests.

Maybe I should make this bug do the opposite: remove the --verbose where it is in use?
(And we can push it enabled when we need to debug).
I wouldn't know where to start on the taskcluster action.

Severity: -- → S3
Priority: -- → P2
Summary: Make the mozperftest android tests verbose to make it easier to debug failures → Make the mozperftests use consistent --verbose options

This removes the --verbose option from a handful of perftests that had it enabled.

Pushed by acreskey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e3be78b1a5e
Make the mozperftests use consistent --verbose options r=sparky
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: