Closed
Bug 807376
Opened 12 years ago
Closed 12 years ago
Spidermonkey builder names are wrong
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sfink, Assigned: sfink)
Details
Attachments
(2 files)
1.60 KB,
patch
|
bhearsum
:
review+
sfink
:
checked-in+
|
Details | Diff | Splinter Review |
2.88 KB,
patch
|
bhearsum
:
review+
sfink
:
checked-in+
|
Details | Diff | Splinter Review |
bhearsum says he is seeing things like "spidermonkey_try Linux x86-64 %(branch)s leak test-rootanalysis build", which has both an un-interpolated %(branch)s as well as having "leak test-rootanalysis" instead of "spidermonkey_try-rootanalysis"
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → sphink
Assignee | ||
Comment 1•12 years ago
|
||
For generateBranchObjects, the config has been copied from PLATFORM_VARS and had all strings' interpolated stuff (which is only "%(branch)s", for ['base_name'] and ['mozconfig']) filled in according to the branch. For generateSpidermonkeyObjects, we're using the raw PLATFORM_VARS, so we need to do the substitution.
Attachment #677069 -
Flags: review?(bhearsum)
Assignee | ||
Comment 2•12 years ago
|
||
Give each spidermonkey project object a branch name so it can be used to fill in the PLATFORM's base_name value.
Attachment #677073 -
Flags: review?(bhearsum)
Updated•12 years ago
|
Attachment #677069 -
Flags: review?(bhearsum) → review+
Comment 3•12 years ago
|
||
Comment on attachment 677073 [details] [diff] [review] Give each spidermonkey object a branch name Review of attachment 677073 [details] [diff] [review]: ----------------------------------------------------------------- ::: mozilla/config.py @@ +946,5 @@ > 'macosx64-lion-debug': PLATFORM_VARS['macosx64-lion-debug'], > }, > 'hgurl': 'http://hg.mozilla.org/', > 'repo_path': 'integration/mozilla-inbound', > + 'branch': 'mozilla-central', I think this should be mozilla-inbound...r=me if you fix that upon landing.
Attachment #677073 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 4•12 years ago
|
||
(In reply to Ben Hearsum [:bhearsum] from comment #3) > Comment on attachment 677073 [details] [diff] [review] > Give each spidermonkey object a branch name > > Review of attachment 677073 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: mozilla/config.py > @@ +946,5 @@ > > 'macosx64-lion-debug': PLATFORM_VARS['macosx64-lion-debug'], > > }, > > 'hgurl': 'http://hg.mozilla.org/', > > 'repo_path': 'integration/mozilla-inbound', > > + 'branch': 'mozilla-central', > > I think this should be mozilla-inbound...r=me if you fix that upon landing. buildbot-configs/mozilla/config.py has a BRANCHES section, starting with this comment: # All branches (not in project_branches) that are to be built MUST be listed here, along with their # platforms (if different from the default set). It then lists a bunch of branches which does NOT include mozilla-inbound. From that, I inferred that mozilla-inbound is not considered a 'branch' in its own right. Then again, near the end of the file is this section: for b in BRANCHES.keys(): if b not in ('mozilla-central', 'mozilla-inbound', 'mozilla-aurora', 'try'): if 'android-noion' in BRANCHES[b]['platforms']: del BRANCHES[b]['platforms']['android-noion'] which implies that mozilla-inbound really is a branch. So which is it? Either way I go, it seems like one or the other of the above 2 chunks of code in config.py should be updated.
Comment 5•12 years ago
|
||
(In reply to Steve Fink [:sfink] from comment #4) > (In reply to Ben Hearsum [:bhearsum] from comment #3) > > Comment on attachment 677073 [details] [diff] [review] > > Give each spidermonkey object a branch name > > > > Review of attachment 677073 [details] [diff] [review]: > > ----------------------------------------------------------------- > > > > ::: mozilla/config.py > > @@ +946,5 @@ > > > 'macosx64-lion-debug': PLATFORM_VARS['macosx64-lion-debug'], > > > }, > > > 'hgurl': 'http://hg.mozilla.org/', > > > 'repo_path': 'integration/mozilla-inbound', > > > + 'branch': 'mozilla-central', > > > > I think this should be mozilla-inbound...r=me if you fix that upon landing. > > buildbot-configs/mozilla/config.py has a BRANCHES section, starting with > this comment: > > # All branches (not in project_branches) that are to be built MUST be listed > here, along with their > # platforms (if different from the default set). > It then lists a bunch of branches which does NOT include mozilla-inbound. Yeah, that's because a bunch of them live in project_branches.py, which merge in later. That comment in archaic! > which implies that mozilla-inbound really is a branch. Use mozilla-inbound. That's what it's actually building!
Assignee | ||
Updated•12 years ago
|
Attachment #677069 -
Flags: checked-in+
Assignee | ||
Comment 6•12 years ago
|
||
http://hg.mozilla.org/build/buildbotcustom/rev/6b086c80ec2b http://hg.mozilla.org/build/buildbot-configs/rev/89195fad4418
Assignee | ||
Updated•12 years ago
|
Attachment #677073 -
Flags: checked-in+
Assignee | ||
Comment 7•12 years ago
|
||
Oops, sorry, ignore the previous commit of <http://hg.mozilla.org/build/buildbot-configs/rev/89195fad4418>. It's actually for 803736. For this one: http://hg.mozilla.org/build/buildbot-configs/rev/ccbb470d04e6
Comment 8•12 years ago
|
||
In production
Comment 9•12 years ago
|
||
Haven't seen a complaint, assuming this should be FIXED now.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•