Follow-up to https://github.com/ampproject/amphtml/issues/28307#issuecomment-629338702.
The problem is cherry-picks to RC on Monday or Tuesday won't be included in canary (which is "cut" on previous Friday).
So AFAICT we need two things:
- New guidance for release cops -- either use a later nightly or cherry-pick immediately into the new canary
- An automated check that cherry-pick commits to release N are included in release N+1
The problem is cherry-picks to RC on Monday or Tuesday won't be included in canary (which is "cut" on previous Friday).
Drive-by comment with terminology update for posterity:
canary branch has been replaced by the automated step of cutting the nightly branchThat aside, +1 to both your suggestions 馃槂
I prefer to modify the cherry-picking instructions instead of instructing to use a later nightly - since nightly builds are cut at ~midnight (PT) there's still going to be a small skew, so this wouldn't solve the problem for when a fix to some P0 issue is merged between midnight and Tuesday morning
As for the automated check for cherry-picks, I'd rather block this for now on @estherkim's planned work on automating cherry-picks - once that exist, we can tack-on the related check based on machine-parsable metadata that the automation provides
For now, we'll just have to be careful.
@estherkim - where are we on cherry-pick automation?
I'm writing up / thinking through the design right now. Solving this is one of its goals. Design review coming soon!
@estherkim you started working on this, correct?
Yup! Underway.
Forgot to update this - it's fixed via cherry-pick automation 馃憤
Most helpful comment
Forgot to update this - it's fixed via cherry-pick automation 馃憤