1355018 - Make post-release steps use the unified repo

Status: RESOLVED FIXED

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

Description

[Johan Lorenzo [:jlorenzo]]

Similarly to bug 1354625, version bump didn't work because we're not using the unified repo[2].


[1] https://mozilla-release-logs.s3.amazonaws.com/mozilla-beta/firefox-53.0b10/build2/firefox_mozilla-beta_version_bump-all-A2PA-KU0QtGbtlXJ1seaNw-0
[2] https://dxr.mozilla.org/mozilla-central/rev/45692c884fdd5136a64fb2f8a61a0c8183b69331/testing/mozharness/configs/releases/postrelease_firefox_beta.py#5

Comment 1

[Johan Lorenzo [:jlorenzo]]

Attachment: Bug 1355018 - Make post-release steps use the unified repo

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

Comment 2

[Johan Lorenzo [:jlorenzo]]

I'm not too sure on how to test out the patch. Is there a job I can set up?

Comment 3

[Johan Lorenzo [:jlorenzo]]

In the meantime, I manually bumped the version[1] and pushed the tags[2]

[1] https://hg.mozilla.org/releases/mozilla-beta/rev/08842b01078828dd64934259977d30db09e59f71
[2] https://hg.mozilla.org/releases/mozilla-beta/rev/4f39d509a2e8d0173aa6bff8cf8ecd9d87fac7e4

Comment 4

[Rail Aliiev [:rail]]

(In reply to Johan Lorenzo [:jlorenzo] from comment #2)
> I'm not too sure on how to test out the patch. Is there a job I can set up?

You can run this locally (on your laptop), just disable pushing and check `hg out`.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8856391 [details]
Bug 1355018 - Make post-release steps use the unified repo

https://reviewboard.mozilla.org/r/128316/#review130884

Can you test this before pushing.

Comment 6

[Aki Sasaki (not active)]

Manually bumped 53.0b12 https://hg.mozilla.org/releases/mozilla-beta/rev/5a2d561c1ea6

Comment 7

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8856391 [details]
Bug 1355018 - Make post-release steps use the unified repo

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/128316/diff/1-2/

Comment 8

[Johan Lorenzo [:jlorenzo]]

Actually, the fix is the same as in bug 1355404, we just need to specify a `clone_upstream_url`. I tested it locally. That gave: 

> 14:33:57     INFO - Running command: ['hg', '--config', 'ui.merge=internal:merge', '--config', 'extensions.robustcheckout=/tmp/toto/scripts/external_tools/robustcheckout.py', 'robustcheckout', 'https://hg.mozilla.org/releases/mozilla-beta', 'mozilla-beta', '--sharebase', '/builds/hg-shared', '--upstream', 'https://hg.mozilla.org/mozilla-unified', '--branch', 'default']
> 14:33:57     INFO - Copy/paste: hg --config ui.merge=internal:merge --config extensions.robustcheckout=/tmp/toto/scripts/external_tools/robustcheckout.py robustcheckout https://hg.mozilla.org/releases/mozilla-beta mozilla-beta --sharebase /builds/hg-shared --upstream https://hg.mozilla.org/mozilla-unified --branch default

Comment 9

[Johan Lorenzo [:jlorenzo]]

I landed the patches on beta, release and esr52, because we have several releases in these 3 channels. If everything goes fine there, I'll land it in the other branches.

https://hg.mozilla.org/releases/mozilla-beta/rev/9e169ddd1f823af7567318fcb62e95bcb87578ff
https://hg.mozilla.org/releases/mozilla-release/rev/81e6d873a1033156c523129b6d3b25b0913f2fb6
https://hg.mozilla.org/releases/mozilla-esr52/rev/d01c882f6b1c1b116edc0db2afdfd0d6a0e46fb9

Comment 10

[Johan Lorenzo [:jlorenzo]]

We managed to ship 53.0[1] and 52.1.0esr[2] with this patch. Time to make it ride the trains.

[1] https://tools.taskcluster.net/task-group-inspector/#/FH3KauqaRfuuAkelDT6Cow/pPyen0iWQweSLYGpOPk6mw?_k=s5sv6h
[2] https://tools.taskcluster.net/task-group-inspector/#/LPXWQp_1TMqtKr8a5k6-Aw/RZuKkRtVS4S4vrX_3n5GUA?_k=7ovclj

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2d53e3a676bb
Make post-release steps use the unified repo r=rail

Comment 12

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2d53e3a676bb