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

Manage win.rustup.rs in Terragrunt #422

Merged
merged 8 commits into from
Jun 5, 2024
Merged

Conversation

jdno
Copy link
Member

@jdno jdno commented Jun 4, 2024

win.rustup.rs is an old CloudFront distribution that is not yet managed in Terraform or Terragrunt. A new Terragrunt module has been created so that we can easily create a new staging environment.

The module copies the existing functionality, but uses a response header policy instead of a Lambda function to set the Content-Disposition header.

This is currently blocking rust-lang/rustup#3840.

@jdno jdno force-pushed the win-rustup-rs branch from 4b30b30 to 72d0233 Compare June 4, 2024 13:10
jdno added 4 commits June 4, 2024 15:40
`win.rustup.rs` is an old CloudFront distribution that is not yet
managed in Terraform or Terragrunt. A new Terragrunt module has been
created so that we can easily create a new staging environment.

The module copies the existing functionality, but uses a response header
policy instead of a Lambda function to set the `Content-Disposition`
header.
The `win.rustup.rs` distribution does not have a staging environment
right now. A new state has been created in Terragrunt to create a new
environment to test the distribution.
The existing CloudFront distribution has been imported into Terragrunt
and then updated to use the new response header policy and new Lambda
function.
@jdno jdno force-pushed the win-rustup-rs branch from 72d0233 to f3cb562 Compare June 5, 2024 09:13
@jdno jdno merged commit 20b8866 into rust-lang:master Jun 5, 2024
3 checks passed
@jdno jdno deleted the win-rustup-rs branch June 5, 2024 14:12
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.

2 participants