Closed
Bug 590369
Opened 15 years ago
Closed 15 years ago
Didn't get any talos runs for 3.6.9 build1
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: armenzg)
References
Details
(Whiteboard: [talos])
Attachments
(1 file)
929 bytes,
patch
|
catlee
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
Linux sendchange:
master: production-master01.build.mozilla.org:9009
branch: mozilla-1.9.2-release-linux-talos
revision: 340d4a429fe6
comments:
user: sendchange
files: ['http://stage.mozilla.org/pub/mozilla.org/firefox/nightly/3.6.9-candidates/build1/linux-i686/en-US/firefox-3.6.9.tar.bz2']
but didn't get any talos runs showing up on
http://tinderbox.mozilla.org/showbuilds.cgi?tree=Firefox3.6-Release
nor any jobs for builders like 'Rev3 Fedora 12 release-mozilla-1.9.2 talos tp4' on the buildbot master. All platforms.
Assignee | ||
Comment 1•15 years ago
|
||
Hi Nick,
I have looked at this code during beta 3 and there was some refactoring done for schedulerDB which we followed up with a port to 0.7.10 (releases master - bug 585098).
Ben and I just looked at the following patch:
https://bug585098.bugzilla.mozilla.org/attachment.cgi?id=465231
and I think here is the code that is incorrect:
- talosBranch = "%s-release-%s" % (self.branchName, self.platform)
+ talosBranch = "%s-release-%s-talos" % (self.branchName, self.platform)
since we had fixed it to have "release-" as a prefix rather than infix:
- branch += "-release"
+ branch = "release-" + branch
Unit tests worked because the branch name was correctly constructed:
http://mxr.mozilla.org/build/source/buildbot-configs/mozilla2/release_master.py#263
Here are the branch names of the sendchanges that were used during the 3.6.9 release:
branch: mozilla-1.9.2-release-linux-talos
branch: release-mozilla-1.9.2-linux-opt-unittest
As you can see for unit tests we are following the new naming.
Catlee is this patch correct? Not that I have time to test it but I believe this should do the trick.
Attachment #469056 -
Flags: review?(catlee)
Assignee | ||
Updated•15 years ago
|
Comment 2•15 years ago
|
||
Comment on attachment 469056 [details] [diff] [review]
fix talosBranch name
Looks good.
Please land this both on the default and 0.8.0 branch.
Attachment #469056 -
Flags: review?(catlee) → review+
Assignee | ||
Comment 3•15 years ago
|
||
Comment on attachment 469056 [details] [diff] [review]
fix talosBranch name
http://hg.mozilla.org/build/buildbotcustom/rev/098ce1881309 (buildbot-0.8.0)
http://hg.mozilla.org/build/buildbotcustom/rev/be46325d78e4 (default)
Reconfigured pm to make sure the next release picks up this change.
Attachment #469056 -
Flags: checked-in+
Assignee | ||
Comment 4•15 years ago
|
||
Let's see in the next release if this gets fixed.
Assignee: nobody → armenzg
Status: NEW → RESOLVED
Closed: 15 years ago
Priority: P3 → P2
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
•