Closed Bug 1491114 Opened Last year Closed Last year
Add ability to specify page
_cycles and page _timeout on the command line
47 bytes, text/x-phabricator-request
|Details | Review|
When debugging or adding a new test locally we rarely want to run the test for all 5 page cycles. Currently the only way to get around this is to modify the test's ini file (and hopefully remember not to accidentally commit that change!) Instead it would be nice if we could run ./mach raptor-test --page-cycles 1 to temporarily override the test's default. Same goes for timeout.
Assignee: nobody → ahal
Status: NEW → ASSIGNED
:ahal, do you have updates here? Do we want to support this, or adjust automatically with local/debug mode?
Sorry, was hoping to do it quickly at the work week but then hit a snag and subsequently forgot about it. I think debug mode didn't exist when I started this, but if we set the page cycle to 1 with --debug, then I agree that there aren't really any compelling use cases for adding this flag.
we don't have a debug mode, I was wondering if that would be a better vehicle for this.
I think we should add a pagetimeout cmd line argument to overide what's in the test INI - even if we add a separate debug mode. I have seen on my local machine some benchmarks timing out sometimes, that finish on time in production. Also :Gijs reported running raptor speedometer on a slower machine is timing out (and he asked if there is a cmd line arg to override).
:ahal do you have time to take this on? If not I can go ahead with a patch (I'd like to add this asap since more developers are running Raptor locally).
Yeah I probably won't have time to look at this for awhile, so please feel free to steal it from me if you like!
Ok, thanks Andrew
Assignee: ahal → rwood
Summary: Add ability to specify page cycles and timeout on the command line → Add ability to specify page_cycles and page_timeout on the command line
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/1dbaae27c584 Add ability to specify page_cycles and page_timeout on the command line; r=jmaher
You need to log in before you can comment on or make changes to this bug.