1724219 - [browsertime] make browsertime command warning less scary

Status: RESOLVED FIXED

(Testing :: Performance, defect, P2)

Description

[Kimberly Sereduck :kimberlythegeek]

Browsertime is configured to display this warning when it is run, advising users to run browsertime raptor instead:

 0:01.75 [WARNING] This tool is UNSUPPORTED by the perftest team and it is NOT recommended to use for performance testing. Instead, if you are looking to perform performance tests on your patch, use `./mach raptor --browsertime`.

Add an exception so that this warning is not displayed when run with the --setup flag

Comment 1

[Dave Hunt [:davehunt] [he/him] ⌚BST]

There may be other commands we want to allow, such as --update-upstream-url. Perhaps we could simply move this message into the else block of this code: https://searchfox.org/mozilla-central/rev/00977c4e37865a92f1c15572ae4aea90e934b25b/tools/browsertime/mach_commands.py#660-666

Comment 2

[Kimberly Sereduck :kimberlythegeek]

Attachment: Bug 1724219: make browsertime command warning less scary

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/97a426816756
make browsertime command warning less scary r=perftest-reviewers,AlexandruIonescu

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/97a426816756