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

Status

Release Engineering
General
P5
enhancement
RESOLVED FIXED
8 years ago
4 years ago

People

(Reporter: catlee, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [talos])

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
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

8 years ago
Cool.
Tastes like future.
Severity: normal → enhancement
Component: Release Engineering → Release Engineering: Future

Comment 2

8 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
Whiteboard: [talos]

Comment 3

7 years ago
Created attachment 457790 [details] [diff] [review]
remove buildid from talos.MozillaUpdateConfig

I need a nap after that hefty patch.
Attachment #457790 - Flags: review?(catlee)
(Reporter)

Updated

7 years ago
Attachment #457790 - Flags: review?(catlee) → review+

Comment 4

7 years ago
Pretty sure about this, but still safer during a downtime.
Blocks: 579092
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+
all done here
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.