Reimplement postrelease for S3 hosting

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: nthomas, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
*_postrelease
Issue: Removes index.html for Firefox, updates symlinks, updates bouncer aliases
Code: http://hg.mozilla.org/build/tools/file/default/scripts/release/stage-tasks.py#l378
Fix: convert symlink a redirect (x-amz-website​-redirect-location) for top level dir ?
SWAG: Unknown until more technical investigation done

Also - some partner builds get copied too. See http://hg.mozilla.org/build/tools/file/default/scripts/release/stage-tasks.py#l206 for the code for that. We may need to re-host those somewhere.
It seems pretty clear that index files can go away (they probably don't make sense in S3, and we don't have the same sort of paranoia about people downloading early as we did in the past).

We talked a bit about whether or not something like the current "latest" symlink is necessary anymore. I'm tempted to do away with it now that we have stable Bouncer entries.

Partner builds are being reworked as part of another project (no bug on file yet though), and coop says we can ignore them here.

If all of this is true, this might be a no-op? At the most, I think we'd want to do the symlink-equivalent.
Created attachment 8628861 [details] [diff] [review]
kill everything except bouncer alias updates

Not quite a no-op, we need to remove the code that does symlinks et. al. Once we're fully transitioned there's probably huge swaths of this script that we can kill.
Attachment #8628861 - Flags: review?(nthomas)
(Reporter)

Comment 3

4 years ago
Comment on attachment 8628861 [details] [diff] [review]
kill everything except bouncer alias updates

Please check that rehosting the partner builds is on Nancy's radar, or that coop in the loop with her, or we'll need to redo doSyncPartnerBundles() somehow.
Attachment #8628861 - Flags: review?(nthomas) → review+
(In reply to Nick Thomas [:nthomas] from comment #3)
> Comment on attachment 8628861 [details] [diff] [review]
> kill everything except bouncer alias updates
> 
> Please check that rehosting the partner builds is on Nancy's radar, or that
> coop in the loop with her, or we'll need to redo doSyncPartnerBundles()
> somehow.

Confirmed in bug 1181183 that coop will be handling that. I think I saw you loop Nancy in in e-mail at one point, too. So, I think the only thing left to do here is land this patch when we're ready to switch over...
(Reporter)

Comment 5

3 years ago
Status - ready to go
To land at transition: patch for tools
(Reporter)

Comment 6

3 years ago
TODO 
* add bouncer alias for thunderbird-latest-beta, and set up release config to update it in postrelease
* remove contents of {firefox,mobile,thunderbird}/releases/latest-*, replacing it with a README which talks about bouncer
* blog/newsgroup about same
(Reporter)

Comment 7

3 years ago
Comment on attachment 8628861 [details] [diff] [review]
kill everything except bouncer alias updates

Landed just now as https://hg.mozilla.org/build/tools/rev/bb6a977ee3df
Attachment #8628861 - Flags: checked-in+
(Reporter)

Comment 8

3 years ago
Bug 1223019 for some post-landing fallout.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Depends on: 1223019
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.