1556784 - 68 train: Rename `{promote,ship}_fennec` into `{promote,ship}_fennec_beta`

Status: RESOLVED FIXED

(Release Engineering :: Release Automation: Other, task)

Description

[Johan Lorenzo [:jlorenzo]]

We will need to spin {promote,ship}_fennec_release off mozilla-esr68. Therefore, let's rename the existing target tasks now and introduce the new ones later.

Comment 1

[Johan Lorenzo [:jlorenzo]]

Attachment: Bug 1556784 - Rename `{promote,ship}_fennec` into `{promote,ship}_fennec_beta` r=rail

Comment 2

[Johan Lorenzo [:jlorenzo]]

Attachment: [release-services/shipit] 68 train: Rename `{promote,ship}_fennec` into `{promote…,ship}_fennec_beta`

Comment 3

[Aki Sasaki [:aki] (he/him) (UTC-7)]

Could you update the braindump params files to match?

Comment 4

[Johan Lorenzo [:jlorenzo]]

Attachment: [build/braindump] taskgraph-diff: Create params files for the 68 train (Fennec sunset)

Thanks for spotting that, Aki. I ended up creating new param files, instead of changing the existing ones. The reason is: this current patch is intended to land on mozilla-beta only. So I kept the params as is, until we decide to kill Fennec on mozilla-central for good. (We haven't retired it from there, just in case we want to revert the whole "Fennec ESR" plan at the last minute)

Comment 5

[Tom Prince [:tomprince]]

Comment on attachment 9069925 [details] [diff] [review]
[build/braindump] taskgraph-diff: Create params files for the 68 train (Fennec sunset)

Review of attachment 9069925 [details] [diff] [review]:
-----------------------------------------------------------------

I think we should just update the existing files (renaming some of them as appropriate). We always have version control history if we decide to abandon fennec riding to esr68.

Comment 6

[Johan Lorenzo [:jlorenzo]]

I'm worried that some files will always break if I run them against mozilla-central, or some other against mozilla-beta. Is there a way to filter out which files (under the same folder) are 68-compatible and some should be run against 69 or 67?

Comment 7

[Aki Sasaki [:aki] (he/him) (UTC-7)]

(In reply to Johan Lorenzo [:jlorenzo] from comment #4)

Created attachment 9069925 [details] [diff] [review]
[build/braindump] taskgraph-diff: Create params files for the 68 train (Fennec sunset)

Thanks for spotting that, Aki. I ended up creating new param files, instead of changing the existing ones. The reason is: this current patch is intended to land on mozilla-beta only. So I kept the params as is, until we decide to kill Fennec on mozilla-central for good. (We haven't retired it from there, just in case we want to revert the whole "Fennec ESR" plan at the last minute)

Aha. Then, if the params files are useful for you or for the rest of the team, we can put them in a separate params dir, possibly params-fennec/ or params-fennec-68. (If not useful, let's not :) I'd like to allow for testing against 68 code if it's useful, while still allowing someone running on central to run the full set of parameters in params/.

Comment 8

[Johan Lorenzo [:jlorenzo]]

Attachment: [build/braindump] taskgraph-diff: Create params files for Fennec 68 (sunset)

I agree we don't need the desktop params, just the Fennec ones. I put just the fennec-related files under params-fennec-68, like you suggested. Thanks!

Comment 9

[Rail Aliiev [:rail]]

Comment on attachment 9069696 [details] [review]
[release-services/shipit] 68 train: Rename `{promote,ship}_fennec` into `{promote…,ship}_fennec_beta`

landed

Comment 10

[Rail Aliiev [:rail]]

Comment on attachment 9070198 [details] [diff] [review]
[build/braindump] taskgraph-diff: Create params files for Fennec 68 (sunset)

Review of attachment 9070198 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM

Comment 11

[Johan Lorenzo [:jlorenzo]]

https://hg.mozilla.org/releases/mozilla-beta/rev/243b742b3659

Comment 12

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 9070198 [details] [diff] [review]
[build/braindump] taskgraph-diff: Create params files for Fennec 68 (sunset)

Landed on default at https://hg.mozilla.org/build/braindump/rev/35e30e3bfbda7f1fa6be308951ea128d5e71d503

Please let me know, Tom, if you'd prefer another way. I'm open to alternatives solutions. I landed this patch, just to be in sync with the content of mozilla-beta.

Comment 13

[Johan Lorenzo [:jlorenzo]]

All patches landed/deployed. Closing this bug as fixed.

Please reply in this bug, if you'd like an alternative approach on the braindump patch, I'll be happy to reply back/address it.