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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mtabara, Unassigned)

References

Details

Attachments

(3 files)

No description provided.
Blocks: 1433489
Work of this is being done in bug 1431799
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
I didn't fix this in bug 1431799; let's fix this now.
Assignee: mtabara → aki
I patched bm83 dev rr3, so our next maple fx beta > b2 should auto-push.
Attachment #8953064 - Flags: review?(rail) → review+
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.
Attachment #8953089 - Flags: review?(bhearsum)
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+
(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 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+
(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.
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

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

We must have solved this somewhere elsewhere to get automated betas, right ? We go straight to ship_firefox flavor for that.

+1.

Status: REOPENED → RESOLVED
Closed: 7 years ago4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: