Closed
Bug 522739
Opened 15 years ago
Closed 14 years ago
Clean up talos buildid
Categories
(Release Engineering :: General, enhancement, P5)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: catlee, Unassigned)
References
Details
(Whiteboard: [talos])
Attachments
(1 file)
|
1.66 KB,
patch
|
catlee
:
review+
bhearsum
:
checked-in+
|
Details | Diff | Splinter Review |
In buildbotcustom.steps.talos.MozillaUpdateConfig, we set the buildid to something calculated on the change's timestamp. According to Alice, this isn't required any more, as Talos calculates the buildid from the build it downloads. We should remove this code from buildbotcustom to avoid confusion.
|
||
Comment 1•15 years ago
|
||
Cool. Tastes like future.
Severity: normal → enhancement
Component: Release Engineering → Release Engineering: Future
|
||
Comment 2•14 years ago
|
||
Mass move of bugs from Release Engineering:Future -> Release Engineering. See http://coop.deadsquid.com/2010/02/kiss-the-future-goodbye/ for more details.
Component: Release Engineering: Future → Release Engineering
Priority: -- → P5
|
||
Updated•14 years ago
|
Whiteboard: [talos]
|
|
||
Comment 3•14 years ago
|
||
I need a nap after that hefty patch.
Attachment #457790 -
Flags: review?(catlee)
|
Reporter | |
Updated•14 years ago
|
Attachment #457790 -
Flags: review?(catlee) → review+
|
||
Comment 5•14 years ago
|
||
Comment on attachment 457790 [details] [diff] [review] remove buildid from talos.MozillaUpdateConfig default landing: changeset: 833:0adc58bfc380 0.8.0 landing: changeset: 837:24c6ebc1410f
Attachment #457790 -
Flags: checked-in+
|
|
||
Comment 6•14 years ago
|
||
all done here
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
Assignee | |
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•