If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

bump staging-trunk-automation bootstrap/buildbot configs to 3.0b3

RESOLVED FIXED

Status

Release Engineering
General
P2
normal
RESOLVED FIXED
10 years ago
4 years ago

People

(Reporter: bhearsum, Assigned: rhelmer)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

10 years ago
patch coming...
(Reporter)

Comment 1

10 years ago
Created attachment 294056 [details] [diff] [review]
[checked in] bump bootstrap.cfg for 3.0b3
Attachment #294056 - Flags: review?(rhelmer)
(Reporter)

Comment 2

10 years ago
Created attachment 294152 [details] [diff] [review]
[checked in] master.cfg bump for 3.0b3
Attachment #294152 - Flags: review?(rhelmer)
(Assignee)

Updated

10 years ago
Attachment #294056 - Flags: review?(rhelmer) → review+
(Assignee)

Comment 3

10 years ago
Comment on attachment 294152 [details] [diff] [review]
[checked in] master.cfg bump for 3.0b3

>Index: staging-1.9/master.cfg
>===================================================================
>RCS file: /cvsroot/mozilla/tools/buildbot-configs/automation/staging-1.9/master.cfg,v
>retrieving revision 1.2
>diff -u -r1.2 master.cfg
>--- staging-1.9/master.cfg	11 Dec 2007 20:19:05 -0000	1.2
>+++ staging-1.9/master.cfg	21 Dec 2007 02:50:37 -0000
>@@ -224,7 +224,7 @@
> # http://weblogs.mozillazine.org/preed/2006/10/a_case_of_the_mondays_on_a_wed.html
> tagFactory.addStep(ShellCommand, description='do valid l10n checkout',
>                    command=['cvs', '-d', '/builds/cvsmirror/l10n', 'co',
>-                            '-r', 'FIREFOX_3_0b2_RELEASE',
>+                            '-r', 'FIREFOX_3_0b3_RELEASE',
>                             'l10n/de/browser/README.txt'],
>                    workdir="/tmp")

"de" isn't the first tagged locale directory now, "be" is. is this still necessary? I guess it is because "be" is the first tagged locale, skipping "af" and "ar"..

We should look into an actual fix for this, like maybe not doing partial tags of the l10n module (tag/checkout the whole thing but only build ones in shipped-locales), fixing the CVS bug, or using client.mk to do checkouts (not sure if it does the right thing in this case but it could be taught to do so).

Should we have this workaround in the production config until we have a real fix?
Attachment #294152 - Flags: review?(rhelmer) → review+
(Reporter)

Comment 4

10 years ago
> "de" isn't the first tagged locale directory now, "be" is. is this still
> necessary? I guess it is because "be" is the first tagged locale, skipping "af"
> and "ar"..
>

I chose 'de' here kindof arbitrarily. It felt like a locale that will always be in shipped-locales, that's why I chose it over others.
 
> We should look into an actual fix for this, like maybe not doing partial tags
> of the l10n module (tag/checkout the whole thing but only build ones in
> shipped-locales), fixing the CVS bug, or using client.mk to do checkouts (not
> sure if it does the right thing in this case but it could be taught to do so).
> 

Agreed.

> Should we have this workaround in the production config until we have a real
> fix?
> 

Production _should_ be OK since 'af' is working. However, it might be prudent to put this step in there.
(Reporter)

Updated

10 years ago
Priority: -- → P2
(Reporter)

Updated

10 years ago
Summary: bump staging-trunk-automation bootstrap config to 3.0b3 → bump staging-trunk-automation bootstrap/buildbot configs to 3.0b3
(Reporter)

Comment 5

10 years ago
Checking in fx-moz19-staging-bootstrap.cfg;
/cvsroot/mozilla/tools/release/configs/fx-moz19-staging-bootstrap.cfg,v  <--  fx-moz19-staging-bootstrap.cfg
new revision: 1.5; previous revision: 1.4
done
(Reporter)

Comment 6

10 years ago
Checking in staging-1.9/master.cfg;
/cvsroot/mozilla/tools/buildbot-configs/automation/staging-1.9/master.cfg,v  <--  master.cfg
new revision: 1.3; previous revision: 1.2
done
(Reporter)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Reporter)

Updated

10 years ago
Attachment #294152 - Attachment description: master.cfg bump for 3.0b3 → [checked in] master.cfg bump for 3.0b3
(Reporter)

Updated

10 years ago
Attachment #294056 - Attachment description: bump bootstrap.cfg for 3.0b3 → [checked in] bump bootstrap.cfg for 3.0b3
(Assignee)

Updated

10 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

10 years ago
Created attachment 310702 [details] [diff] [review]
oldRc should be 4, and "make stage" should honor it

Also, 1.8 master no longer had "make stage" in it; re-add this.
Assignee: bhearsum → rhelmer
Status: REOPENED → ASSIGNED
Attachment #310702 - Flags: review?(bhearsum)
(Assignee)

Comment 8

10 years ago
Comment on attachment 310702 [details] [diff] [review]
oldRc should be 4, and "make stage" should honor it

Sorry, should've been on bug 420005.
Attachment #310702 - Attachment is obsolete: true
Attachment #310702 - Flags: review?(bhearsum)
(Assignee)

Updated

10 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.