1660771 - cleanup in mergeduty esr-bump action as ESR78 no longer ships android stuff

Status: RESOLVED FIXED

(Release Engineering :: Release Automation, defect)

Description

[Mihai Tabara [:mtabara]⌚️GMT]

While running a simulation for ESR78 earlier today, we noticed that the task broke with

    with open(abs_path, "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: '/app/workdir/src/mobile/android/config/version-files/beta/version.txt'
exit code: 1

Given that we've EOL'ed Fennec stuff in ESR68, we should remove these version bumps in ESR78, going forward.

Comment 1

[Mihai Tabara [:mtabara]⌚️GMT]

Ah, turns out when I pushed the no-op action-task, I used the wrong behavior "bump-esr" which defaults to esr68, which naturally still has all those files. I should've used the https://searchfox.org/mozilla-central/source/taskcluster/ci/config.yml#399 instead. I'll fix our docs, no other action needed at this point.

Comment 2

[Aki Sasaki (not active)]

We might be able to stop bumping those in esr68 as well, which may simplify things.

Comment 3

[Mihai Tabara [:mtabara]⌚️GMT]

Fixed docs for this in https://github.com/mozilla/build-relengdocs/pull/41/files. Incoming patch to clean configs too.

Comment 4

[Mihai Tabara [:mtabara]⌚️GMT]

Re-opening this to fix the configs too, since we're here.

Comment 5

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1660771 - default mergeduty bump-esr to esr78. r=#releng

Comment 6

[Mihai Tabara [:mtabara]⌚️GMT]

Comment on attachment 9171909 [details]
Bug 1660771 - default mergeduty bump-esr to esr78. r=#releng

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Mergeduty configs for release automation to better fit RelEng process
• User impact if declined:
• Fix Landed on Version:
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This is release automation change to default mergeduty configs to esr78 and EOL esr68.
• String or UUID changes made by this patch:

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f682fa674cb9
default mergeduty bump-esr to esr78. r=releng-reviewers,bhearsum,jlorenzo DONTBUILD

Comment 8

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/f682fa674cb9

Comment 9

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9171909 [details]
Bug 1660771 - default mergeduty bump-esr to esr78. r=#releng

Approved for esr78.

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr78/rev/651bd1b0c8b7