1344222 - [tcmigration] publish to Balrog script should handle Fennec releases as well

Status: RESOLVED WONTFIX

(Release Engineering :: Release Automation: Other, enhancement, P3)

Description

[Mihai Tabara [:mtabara]⌚️GMT]

I think the main publish_balrog script[1] is generic enough to handle anything. It calls at its turn a tools script[2]. However, we need to backfill these[3] configs so that we can handle the Fennec beta/release, something like [4]

[1]: https://hg.mozilla.org/mozilla-central/file/tip/testing/mozharness/scripts/release/publish_balrog.py
[2]: https://hg.mozilla.org/build/tools/file/tip/scripts/build-promotion/balrog-release-shipper.py
[3]: https://hg.mozilla.org/mozilla-central/file/tip/testing/mozharness/configs/releases
[4]: https://hg.mozilla.org/mozilla-central/file/tip/testing/mozharness/configs/releases/updates_firefox_beta.py

Comment 1

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1344222 - add bb configs for Fennec publish to Barlog.

Review commit: https://reviewboard.mozilla.org/r/118996/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/118996/

Comment 2

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1344222 - enable publish to Balrog in Fennec releasetask graphs. r=rail

Comment 3

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1344222 - add Fennec publish to Balrog configs. a=release DONTBUILD

Review commit: https://reviewboard.mozilla.org/r/119004/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/119004/

Comment 4

[Rail Aliiev [:rail]]

Comment on attachment 8845859 [details]
Bug 1344222 - add bb configs for Fennec publish to Barlog.

https://reviewboard.mozilla.org/r/118996/#review120994

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8845861 [details]
Bug 1344222 - add Fennec publish to Balrog configs.  a=release DONTBUILD

https://reviewboard.mozilla.org/r/119004/#review121000

::: testing/mozharness/configs/releases/updates_fennec_beta.py:26
(Diff revision 1)
> +    "balrog_username": "ffxbld",
> +    "update_channels": {
> +        "beta": {
> +            "version_regex": r"^(\d+\.\d+(b\d+)?)$",
> +            "requires_mirrors": True,
> +            "patcher_config": "mozBeta-branch-patcher2.cfg",

This config is used by desktop, you shouldn't use it.

Actually, we are all wrong. We skip this step for android completely, see https://dxr.mozilla.org/build-central/source/buildbotcustom/process/release.py#1230-1242

Sorry for being a bit late with the news... :(

Comment 6

[Mihai Tabara [:mtabara]⌚️GMT]

After talking with :rail on IRC, we've decided to postpone this for a future conversation. Will close the PR in releasetasks and hold with any other related patch for fennec. I'll leave the bug opened and will revisit once we now better what's to be done here.

Comment 7

[Mihai Tabara [:mtabara]⌚️GMT]

Comment on attachment 8845860 [details] [review]
Bug 1344222 - enable publish to Balrog in Fennec releasetask graphs. r=rail

PR no longer needed.

Comment 8

[Mihai Tabara [:mtabara]⌚️GMT]

I'll leave this bug for now here pending further conversations on what to do with Balrog push.

Comment 10

[Mihai Tabara [:mtabara]⌚️GMT]

Found in triaging, I think we can close this as we don't publish Fennec via Balrog for beta/release, but only for nightly.