1365595 - update postrelease configs for DevEdition

Status: RESOLVED FIXED

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

Description

[[e-mail] (:bhearsum)]

Parts of this are in https://hg.mozilla.org/projects/jamun/rev/a770578dc025

Comment 1

[Mihai Tabara [:mtabara]⌚️GMT]

* Publish to balrog stuff is tracked already in bug 1358614
* for version bump I'll push a fix in a bit
* mark as shipped task reuses the postrelease_firefox_beta which already exists in config.py and in-tree
* bouncer aliases reuses the bouncer_firefox_devedition.py which is already there from bug 1358613

Comment 2

[Mihai Tabara [:mtabara]⌚️GMT]

Attachment: Bug 1365595 - enable version bump in devedition. a=release DONTBUILD

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

Comment 3

[Mihai Tabara [:mtabara]⌚️GMT]

Comment on attachment 8870857 [details]
Bug 1365595 - enable version bump in devedition.  a=release DONTBUILD

https://reviewboard.mozilla.org/r/142426/#review146018

::: mozilla/config.py:2434
(Diff revision 1)
>  }
>  BRANCHES['mozilla-beta']['postrelease_version_bump_config'] = {
>      "firefox": 'releases/postrelease_firefox_beta.py',
>      # configs are generic so can be reused
>      "fennec": 'releases/postrelease_firefox_beta.py',
> -    "devedition": "disabled",
> +    "devedition": 'releases/postrelease_firefox_beta.py',

Normally this should be a no-op as versiom bump will be done regardless by the beta build. However, I suppose the script will die with the current config - https://hg.mozilla.org/releases/mozilla-beta/file/tip/testing/mozharness/scripts/release/postrelease_version_bump.py#l25

We could as well at another variable in Release-Runner / config to enable/disable version bumping so that it's by default disabled for Dawn.

Comment 4

[Mihai Tabara [:mtabara]⌚️GMT]

If I got all this wrong and that config change is not needed, we can close this. We're safe with configs for all the others.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8870857 [details]
Bug 1365595 - enable version bump in devedition.  a=release DONTBUILD

https://reviewboard.mozilla.org/r/142426/#review146024

Comment 6

[Mihai Tabara [:mtabara]⌚️GMT]

So normally I'd uplift this patch to enabled version bump in devedition but ...
As I can see in the last graph of devedition, https://tools.taskcluster.net/task-group-inspector/#/5ZQ1F9PwTzeRTSoD71xGSw?_k=ay2949 the version bump task is not even scheduled (as opposed to other mark-release-as shipped or bouncer aliases) which is even better. I'm still not following how that happens. The version bump script is scheduled here[1] and that variable comes from here[2] which comes actually from config.py[3] which is set to "disabled". 

@rail: Is there a magic behind-the-scenes Jinja2 templating conditions I'm missing for "disabled" or how does that turn off that builder?

[1]: https://github.com/mozilla/releasetasks/blob/master/releasetasks/templates/desktop/release_graph.yml.tmpl#L211
[2]: https://hg.mozilla.org/build/tools/file/tip/buildfarm/release/release-runner.py#l400
[3]: https://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/config.py#l2434

Comment 7

[Rail Aliiev [:rail]]

The behavior is controlled by postrelease_version_bump_enabled,see https://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/config.py#l2471

Comment 8

[Mihai Tabara [:mtabara]⌚️GMT]

(In reply to Rail Aliiev [:rail] ⌚️ET from comment #7)
> The behavior is controlled by postrelease_version_bump_enabled,see
> https://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla/config.
> py#l2471

Doh, "postrelease_version_bump_enabled" vs "postrelease_version_bump_config". This is what happens when you let me name these variables :)

"postrelease_version_bump_enabled" skips the schedule of the builder entirely so it doesn't matter what we have in the configs for "postrelease_version_bump_config". Could be as well "disabled" or anything else.

Closing this, all done. Mea culpa for confusion.