Closed
Bug 838485
Opened 12 years ago
Closed 12 years ago
mozharness setup "rm -rf scripts" buildstep needs to set RETRY on failure
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philor, Assigned: mozilla)
References
Details
Attachments
(2 files)
925 bytes,
patch
|
armenzg
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
777 bytes,
patch
|
armenzg
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
It's hard to tell right now, since Windows on Cedar has been completely busted for a while, but it looks like the failure rate from permission denied during the rm -rf scripts step on Windows is about equal to the entire intermittent failure rate on non-mozharness Windows from all test failures. Like bug 838325, it needs to set RETRY on any non-zero result.
Assignee | ||
Updated•12 years ago
|
Summary: mozharness setup "rm -rf scripts" buildstep needs to set RETRY on failure → ScriptFactory get_script_repo_revision assumes 'hg' is in PATH
Assignee | ||
Comment 1•12 years ago
|
||
Assignee: nobody → aki
Attachment #710541 -
Flags: review?(armenzg)
Assignee | ||
Comment 2•12 years ago
|
||
https://tbpl.mozilla.org/php/getParsedLog.php?id=19477493&tree=Cedar&full=1
========= Started set props: script_repo_revision (results: 2, elapsed: 22 secs) (at 2013-02-05 20:03:08.078936) =========
'hg' 'id' '-i'
in dir C:\\talos-slave\test\scripts (timeout 1200 secs)
watching logfiles {}
argv: ['hg', 'id', '-i']
environment:
ALLUSERSPROFILE=C:\Documents and Settings\All Users
APPDATA=C:\Documents and Settings\cltbld\Application Data
BOOTMODE=BKSTD
CLIENTNAME=Console
COMMONPROGRAMFILES=C:\Program Files\Common Files
COMPUTERNAME=TALOS-R3-XP-098
COMSPEC=C:\WINDOWS\system32\cmd.exe
FP_NO_HOST_CHECK=NO
HOMEDRIVE=C:
HOMEPATH=\Documents and Settings\cltbld
LOGONSERVER=\\TALOS-R3-XP-098
NUMBER_OF_PROCESSORS=2
OS=Windows_NT
PATH=C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;c:\Program Files\gnuwin32\bin;c:\Python24\;c:\Python24\scripts;c:\Program Files\Vim\vim72
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH
PROCESSOR_ARCHITECTURE=x86
PROCESSOR_IDENTIFIER=x86 Family 6 Model 23 Stepping 10, GenuineIntel
PROCESSOR_LEVEL=6
PROCESSOR_REVISION=170a
PROGRAMFILES=C:\Program Files
PROMPT=$P$G
PWD=C:\talos-slave\test\scripts
SESSIONNAME=Console
SYSTEMDRIVE=C:
SYSTEMROOT=C:\WINDOWS
TEMP=C:\DOCUME~1\cltbld\LOCALS~1\Temp
TMP=C:\DOCUME~1\cltbld\LOCALS~1\Temp
USERDOMAIN=TALOS-R3-XP-098
USERNAME=cltbld
USERPROFILE=C:\Documents and Settings\cltbld
WINDIR=C:\WINDOWS
using PTY: False
'hg' is not recognized as an internal or external command,
operable program or batch file.
program finished with exit code 1
elapsedTime=0.110000
script_repo_revision: "'hg' is not recognized as an internal or external command,\r\noperable program or batch file."
========= Finished set props: script_repo_revision (results: 2, elapsed: 22 secs) (at 2013-02-05 20:03:30.629737) =========
Assignee | ||
Comment 3•12 years ago
|
||
Er, I suppose that's bug 838492.
Reporter | ||
Comment 4•12 years ago
|
||
Yeah, that's the red now, but if you go back to before that, you have https://tbpl.mozilla.org/php/getParsedLog.php?id=19201366&tree=Cedar and https://tbpl.mozilla.org/php/getParsedLog.php?id=19056336&tree=Cedar and https://tbpl.mozilla.org/php/getParsedLog.php?id=19018960&tree=Cedar and https://tbpl.mozilla.org/php/getParsedLog.php?id=19015241&tree=Cedar and https://tbpl.mozilla.org/php/getParsedLog.php?id=18903054&tree=Cedar.
Probably not really quite as high as the intermittent test failure rate, but still significant.
Summary: ScriptFactory get_script_repo_revision assumes 'hg' is in PATH → mozharness setup "rm -rf scripts" buildstep needs to set RETRY on failure
Assignee | ||
Comment 5•12 years ago
|
||
Using http://hg.mozilla.org/build/buildbotcustom/file/21b0f0b5e41c/process/factory.py#l462 as a model.
Attachment #710544 -
Flags: review?(armenzg)
Comment 6•12 years ago
|
||
Comment on attachment 710541 [details] [diff] [review]
stop assuming it's 'hg'
I had to use this exact patch on Friday for jetpack/mozharness mode.
Attachment #710541 -
Flags: review?(armenzg) → review+
Updated•12 years ago
|
Attachment #710544 -
Flags: review?(armenzg) → review+
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 710541 [details] [diff] [review]
stop assuming it's 'hg'
http://hg.mozilla.org/build/buildbotcustom/rev/deba5135d777
Attachment #710541 -
Flags: checked-in+
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 710544 [details] [diff] [review]
add a log_eval_func to the clobber step
http://hg.mozilla.org/build/buildbotcustom/rev/1f5fc9412a5c
Attachment #710544 -
Flags: checked-in+
Assignee | ||
Comment 9•12 years ago
|
||
In production.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•11 years ago
|
Component: General Automation → Mozharness
You need to log in
before you can comment on or make changes to this bug.
Description
•