Closed
Bug 939821
Opened 11 years ago
Closed 11 years ago
wrapper scripts for vcs-sync, gaia-bumper need timeouts for hg
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: mozilla)
References
Details
Attachments
(1 file)
872 bytes,
patch
|
catlee
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
The vcssync{1,2} wrapper scripts have a
(hg --cwd mozharness pull -u)
The gaia-bumper wrapper script has a
/usr/local/bin/hgtool.py -b production https://hg.mozilla.org/build/mozharness mozharness
Neither of these has a timeout... and when hg.m.o goes away, the clone will hang forever.
There's a bash timeout trick here:
http://stackoverflow.com/questions/687948/timeout-a-command-in-bash-without-unnecessary-delay
but it exits non-zero either way (timeout will cause hg to exit non-zero; no-timeout will cause kill to exit non-zero) so the script will fail at this point if it's using set -e. We need a better solution.
Assignee | ||
Comment 1•11 years ago
|
||
Ooh, http://stackoverflow.com/a/4581821
timeout TIMEOUT <slow command>
Assignee | ||
Comment 2•11 years ago
|
||
Assignee: nobody → aki
Attachment #8333883 -
Flags: review?(catlee)
Comment 3•11 years ago
|
||
Comment on attachment 8333883 [details] [diff] [review]
gaia-bumper-timeout
Review of attachment 8333883 [details] [diff] [review]:
-----------------------------------------------------------------
TIL about the timeout command!
Attachment #8333883 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 8333883 [details] [diff] [review]
gaia-bumper-timeout
https://hg.mozilla.org/build/puppet/rev/043150c92101
Attachment #8333883 -
Flags: checked-in+
Assignee | ||
Comment 5•11 years ago
|
||
Not merged yet, but I'll resolve anyway.
I already tweaked the non-puppeted scripts for vcssync.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
in production
Updated•7 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•