Closed
Bug 416983
Opened 17 years ago
Closed 17 years ago
staging automation prestage should fire after slave_prestage
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bhearsum, Assigned: bhearsum)
Details
Attachments
(1 file, 1 obsolete file)
3.52 KB,
patch
|
rhelmer
:
review+
|
Details | Diff | Splinter Review |
Nick just pointed out a failure on 1.8 staging. Rsync and the linux prestage were running at the same time which caused rsync to fail out. The slave prestage steps don't take long so we should just block the main prestage on them.
Assignee | ||
Updated•17 years ago
|
Attachment #302798 -
Flags: review?(rhelmer)
Comment 1•17 years ago
|
||
Comment on attachment 302798 [details] [diff] [review]
move main prestage builder to fire after slave prestage
>Index: staging/master.cfg
>===================================================================
>RCS file: /cvsroot/mozilla/tools/buildbot-configs/automation/staging/master.cfg,v
>retrieving revision 1.24
>diff -u -r1.24 master.cfg
>--- staging/master.cfg 28 Jan 2008 17:03:56 -0000 1.24
>+++ staging/master.cfg 12 Feb 2008 13:21:09 -0000
>@@ -225,11 +225,6 @@
> command=['make', 'clean_stage'], haltOnFailure=1)
> prestageFactory.addStep(ShellCommand, description='create staging area',
> command=['make', 'stage'], haltOnFailure=1)
>-prestageFactory.addStep(ShellCommand, description='sync ftp area',
>- command=['rsync', '-av', '--delete-after',
>- '/home/ftp/',
>- 'staging-prometheus-vm:/home/ftp/'],
>- haltOnFailure=1)
Is this not needed anymore? I don't see it being moved anywhere, just removed.
Assignee | ||
Comment 2•17 years ago
|
||
Sorry, those lines shouldn't have been removed.
Attachment #302798 -
Attachment is obsolete: true
Attachment #302834 -
Flags: review?(rhelmer)
Attachment #302798 -
Flags: review?(rhelmer)
Updated•17 years ago
|
Attachment #302834 -
Flags: review?(rhelmer) → review+
Assignee | ||
Comment 3•17 years ago
|
||
Comment on attachment 302834 [details] [diff] [review]
[checked in] add back in 'sync to ftp'
Checking in staging/master.cfg;
/cvsroot/mozilla/tools/buildbot-configs/automation/staging/master.cfg,v <-- master.cfg
new revision: 1.25; previous revision: 1.24
done
Checking in staging-1.9/master.cfg;
/cvsroot/mozilla/tools/buildbot-configs/automation/staging-1.9/master.cfg,v <-- master.cfg
new revision: 1.23; previous revision: 1.22
done
Attachment #302834 -
Attachment description: add back in 'sync to ftp' → [checked in] add back in 'sync to ftp'
Assignee | ||
Comment 4•17 years ago
|
||
Alright, staging 1.8 and 1.9 master configs updated.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•