Closed
Bug 607517
Opened 14 years ago
Closed 14 years ago
land buildbotcustom changes for talos to be able to install release builds
Categories
(Release Engineering :: General, defect, P3)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: anodelman, Unassigned)
References
Details
(Whiteboard: [talos][buildduty])
buildbotcustom patch in bug 599162.
Will require reconfigs of all talos masters.
Staged green. Low risk as will be inactive code until other patches start using the code in question.
Reporter | ||
Updated•14 years ago
|
Flags: needs-treeclosure+
Flags: needs-reconfig+
Comment 1•14 years ago
|
||
(In reply to comment #1)
> buildbotcustom patch in bug 599162.
>
> Will require reconfigs of all talos masters.
>
> Staged green. Low risk as will be inactive code until other patches start
> using the code in question.
Unclear from this comment, and the flags. Are you saying that a reconfig is enough? Or do you also want this done in a downtime?
Flags: needs-treeclosure+
Flags: needs-reconfig?
Flags: needs-reconfig+
Reporter | ||
Comment 2•14 years ago
|
||
I marked both needs-treeclosure and needs-downtime as I figured that we should be careful.
Flags: needs-treeclosure+
Flags: needs-reconfig?
Flags: needs-reconfig+
Reporter | ||
Updated•14 years ago
|
Flags: needs-treeclosure?
Flags: needs-treeclosure+
Flags: needs-reconfig?
Flags: needs-reconfig+
Comment 3•14 years ago
|
||
Alice, there is no needs-downtime flag. You set the needs-reconfig flag as well as treeclosure, which could lead to this landing as part of a non-downtime reconfig. I have cleared that flag for safety.
Flags: needs-reconfig?
Reporter | ||
Comment 4•14 years ago
|
||
If there was a doc explaining this process it would be easier to get the right settings.
Updated•14 years ago
|
Priority: -- → P3
Whiteboard: [talos][buildduty]
Reporter | ||
Comment 5•14 years ago
|
||
Any idea when this could be landed?
Comment 6•14 years ago
|
||
Let's aim for Friday.
Patch in attachment 482735 [details] [diff] [review] is not risky and only a reconfigure is needed (I don't know why it would need a CLOSED tree - let me know if I misunderstand it).
alice is this patch wanted only for "default" or for 0.7.x as well?
Flags: needs-treeclosure? → needs-reconfig+
Updated•14 years ago
|
Assignee: nobody → armenzg
Reporter | ||
Comment 7•14 years ago
|
||
Whatever is production is where this needs to be landed, I'm unsure if that means 'default' or some other branch... so, probably land everywhere?
Comment 8•14 years ago
|
||
0.7.x is used for releases only (at least for just a little longer).
I will land it both places.
Comment 9•14 years ago
|
||
The patch was quite bitrotten.
Could you please attach new patches please?
You can carry forward the r+ and we can land it without needing a treeclosure but only a reconfig.
Assignee: armenzg → nobody
Flags: needs-reconfig+
Reporter | ||
Comment 10•14 years ago
|
||
Posted updated patch.
Reporter | ||
Comment 11•14 years ago
|
||
Can this get in on the next downtime?
Updated•14 years ago
|
Flags: needs-treeclosure?
Comment 12•14 years ago
|
||
No need for a downtime. A reconfigure should do.
This is the patch:
> https://bugzilla.mozilla.org/attachment.cgi?id=490177
Flags: needs-treeclosure?
Flags: needs-treeclosure-
Flags: needs-reconfig?
Reporter | ||
Comment 13•14 years ago
|
||
This doesn't require a downtime and has been sitting for a long time - can I request that this be landed before it becomes bitrotted again?
Comment 14•14 years ago
|
||
It will get landed tomorrow morning.
Updated•14 years ago
|
Flags: needs-reconfig? → needs-reconfig+
Reporter | ||
Comment 16•14 years ago
|
||
Thanks releng!
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
•