Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Purge Fastly cache after release #3819

Closed
wants to merge 1 commit into from
Closed

Conversation

jdno
Copy link
Member

@jdno jdno commented May 10, 2024

When releasing a new version of rustup, we need to purge the cached version on Fastly. This is done by calling Fastly's API using curl and purging the surrogate key that is used for rustup.

See rust-lang/simpleinfra#415 for more details.

When releasing a new version of rustup, we need to purge the cached
version on Fastly. This is done by calling Fastly's API using curl and
purging the surrogate key that is used for rustup.
@rami3l
Copy link
Member

rami3l commented May 10, 2024

@jdno Thank you! Since this commit cannot be tested (and is essentially used by t-infra exclusively), I entrust you with its correctness.

@rami3l
Copy link
Member

rami3l commented May 10, 2024

@jdno One (newbie) question: is it possible to move this script to e.g. some repo that you directly control? After all, it's a bit weird to have something in our repo that we can't run 🤔

@jdno
Copy link
Member Author

jdno commented May 10, 2024

Thanks for the quick review! I'll ask @Mark-Simulacrum to have a look as well.

Before we merge this it would be great to test it with dev-static. I've tested the snippets that I've added and am fairly confident that everything should just work™. But it'd be great to actually confirm this. 🙈

@jdno
Copy link
Member Author

jdno commented May 10, 2024

@jdno One (newbie) question: is it possible to move this script to e.g. some repo that you directly control? After all, it's a bit weird to have something in our repo that we can't run 🤔

Mark and I briefly discussed in Zulip that it would be good to replace this script with an automated release process, maybe similar to rust-lang/promote-release which runs on AWS CodeBuild and is deployed through rust-lang/simpleinfra.

We can definitely have a more concrete discussion about how the release process can be automated/improved. 🙂

@djc
Copy link
Contributor

djc commented May 12, 2024

Agreed with @rami3l that it doesn't make much sense for us to review this. A more automated release process sounds good, I guess that's something someone from the infra team would implement?

@rami3l
Copy link
Member

rami3l commented May 12, 2024

This is waiting for @Mark-Simulacrum's approval then.

jdno added a commit to jdno/rust-rustup that referenced this pull request May 28, 2024
The Rustup release process has historically been a manual process that
involved copying files from S3 to the local machine and back to S3. This
introduced a high risk of human error. When modifications to the
existing release script became necessary, the decision was made to
automate the release process (see rust-lang#3819 for details).

The documentation in the dev-guide has been updated to cover the new
release process, which is fully automated to produce `beta` releases
using GitHub Actions and the [promote-release] tooling.

[promote-release]: https://github.com/rust-lang/promote-release
jdno added a commit to jdno/rust-rustup that referenced this pull request May 28, 2024
The Rustup release process has historically been a manual process that
involved copying files from S3 to the local machine and back to S3. This
introduced a high risk of human error. When modifications to the
existing release script became necessary, the decision was made to
automate the release process (see rust-lang#3819 for details).

The documentation in the dev-guide has been updated to cover the new
release process, which is fully automated to produce `beta` releases
using GitHub Actions and the [promote-release] tooling.

[promote-release]: https://github.com/rust-lang/promote-release
@jdno
Copy link
Member Author

jdno commented May 28, 2024

We've been discussing refactoring the release process in Zulip and on the draft implementation in promote-release.

The outcome of that discussion is that we definitely want to reimplement the Python script as part of promote-release. But that requires some modifications to the current release process.

I've written a draft for that process in #3844.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants