Closed
Bug 973545
Opened 11 years ago
Closed 11 years ago
Set --timeout 10000 on gaia-ui-test command line
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: zcampbell, Assigned: jgriffin)
References
Details
Attachments
(1 file)
|
826 bytes,
patch
|
zcampbell
:
review+
jgriffin
:
checked-in+
|
Details | Diff | Splinter Review |
To normalise it with Travis (and hence we can have equal expectations for each suite) please set "--timeout 10000" onto the command line for this suite.
|
||
Comment 1•11 years ago
|
||
(TBPL is the dashboard for displaying results; release engineering components are where the jobs themselves are configured and scheduled.)
Component: Tinderboxpushlog → General Automation
Product: Webtools → Release Engineering
QA Contact: catlee
Version: Trunk → unspecified
|
||
Comment 2•11 years ago
|
||
If both versions of automation require 10000, why isn't that the default?
Summary: Set --timeout 10000 on Gu command line → Set --timeout 10000 on gaia-ui-test command line
|
|
Reporter | |
Comment 3•11 years ago
|
||
These tests are run on device and emulator too so we don't have a default because we don't really have a default platform. The TBPL tests were getting away with it for a while with the Marionette default which is 5, but now I'd like to normalise it. It's not as easy for me to fix TBPL stuff because I'm not in the hg/releng workflow side of things which is why I hadn't done so until now.
|
|
Assignee | |
Comment 4•11 years ago
|
||
Attachment #8380964 -
Flags: review?(zcampbell)
|
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → jgriffin
Status: NEW → ASSIGNED
|
|
Reporter | |
Comment 5•11 years ago
|
||
Comment on attachment 8380964 [details] [diff] [review]
Add -timeout 10000 to gaia-ui-tests,
r+
Attachment #8380964 -
Flags: review?(zcampbell) → review+
|
|
Assignee | |
Comment 6•11 years ago
|
||
Comment on attachment 8380964 [details] [diff] [review]
Add -timeout 10000 to gaia-ui-tests,
https://hg.mozilla.org/build/mozharness/rev/cbd256df04b8
Attachment #8380964 -
Flags: checked-in+
|
|
Assignee | |
Comment 7•11 years ago
|
||
And a follow-up fix for a silly mistake: https://hg.mozilla.org/build/mozharness/rev/d950d517ec91
|
|
||
Comment 8•11 years ago
|
||
In production.
|
|
Assignee | |
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•