Closed
Bug 1433488
Opened 7 years ago
Closed 4 years ago
tweak ship-it v2 to `push` instead of `promote` for all Firefox/Devediton based on version number
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mtabara, Unassigned)
References
Details
Attachments
(3 files)
No description provided.
Reporter | ||
Comment 1•7 years ago
|
||
Work of this is being done in bug 1431799
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Updated•7 years ago
|
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 2•7 years ago
|
||
I didn't fix this in bug 1431799; let's fix this now.
Assignee: mtabara → aki
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
I patched bm83 dev rr3, so our next maple fx beta > b2 should auto-push.
Comment hidden (mozreview-request) |
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8953064 [details]
bug 1433488 - auto-push firefox betas > b2.
https://reviewboard.mozilla.org/r/222336/#review228266
Attachment #8953064 -
Flags: review?(rail) → review+
Comment hidden (mozreview-request) |
Comment 8•7 years ago
|
||
This is blocked by beetmover:
- beetmover gets its phase from the currently running relpro flavor
- the candidates beetmover task is generally filtered out from the push_firefox relpro flavor, because we already ran promote_firefox
- when we run push_firefox instead of promote_firefox, we have candidates beetmover tasks living with cdns beetmover tasks in the push_firefox graph
- beetmover candidates dies with the wrong payload, since we key off the phase for the payload.
Backing out from maple/dev-rr3 for now, until we can resolve this issue.
Updated•7 years ago
|
Attachment #8953089 -
Flags: review?(bhearsum)
Comment 9•7 years ago
|
||
mozreview-review |
Comment on attachment 8953089 [details]
bug 1433488 - all firefox+devedition relpro actions need partial updates.
https://reviewboard.mozilla.org/r/222370/#review228292
Attachment #8953089 -
Flags: review+
Comment hidden (mozreview-request) |
Comment 11•7 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #8)
> This is blocked by beetmover:
>
> - beetmover gets its phase from the currently running relpro flavor
> - the candidates beetmover task is generally filtered out from the
> push_firefox relpro flavor, because we already ran promote_firefox
> - when we run push_firefox instead of promote_firefox, we have candidates
> beetmover tasks living with cdns beetmover tasks in the push_firefox graph
> - beetmover candidates dies with the wrong payload, since we key off the
> phase for the payload.
>
> Backing out from maple/dev-rr3 for now, until we can resolve this issue.
I may have fixed this... testing on maple; submitted to review.
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8953102 [details]
bug 1433488 - require explicit shipping-phase in balrog+beetmover.
https://reviewboard.mozilla.org/r/222386/#review228304
Attachment #8953102 -
Flags: review?(bhearsum) → review+
Comment 13•7 years ago
|
||
(In reply to Aki Sasaki [:aki] from comment #11)
> (In reply to Aki Sasaki [:aki] from comment #8)
> > This is blocked by beetmover:
> >
> > - beetmover gets its phase from the currently running relpro flavor
> > - the candidates beetmover task is generally filtered out from the
> > push_firefox relpro flavor, because we already ran promote_firefox
> > - when we run push_firefox instead of promote_firefox, we have candidates
> > beetmover tasks living with cdns beetmover tasks in the push_firefox graph
> > - beetmover candidates dies with the wrong payload, since we key off the
> > phase for the payload.
> >
> > Backing out from maple/dev-rr3 for now, until we can resolve this issue.
>
> I may have fixed this... testing on maple; submitted to review.
I think there's still some magic in scriptworker.py that changes behavior based on phase. Let's back this entire bug out for now.
Reporter | ||
Updated•6 years ago
|
Updated•6 years ago
|
Summary: tweak RR3 to `push` instead of `promote` for all Firefox/Devediton based on version number → tweak ship-it v2 to `push` instead of `promote` for all Firefox/Devediton based on version number
Comment 14•6 years ago
|
||
I haven't been working on this since comment 13. Not sure how high of a priority or want this is?
We were blocked by shipping-phase needing to be explicitly defined in the beetmover and balrog tasks. This may have already happened.
Assignee: aki → nobody
Comment 15•4 years ago
|
||
We must have solved this somewhere elsewhere to get automated betas, right ? We go straight to ship_firefox flavor for that.
Comment 16•4 years ago
|
||
+1.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 4 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•