Skip to content

azure: Change all S3 URLs in our CI to use subdomains #61168

New issue

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

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

Already on GitHub? # to your account

Closed
pietroalbini opened this issue May 25, 2019 · 0 comments · Fixed by #61284
Closed

azure: Change all S3 URLs in our CI to use subdomains #61168

pietroalbini opened this issue May 25, 2019 · 0 comments · Fixed by #61284
Labels
T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@pietroalbini
Copy link
Member

Old S3 URLs are used all across our CI scripts and they're going to be deprecated. We should switch all of s3.amazonaws.com/bucket to bucket.s3.amazonaws.com.

@pietroalbini pietroalbini added T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. azure-evaluation labels May 25, 2019
@alexcrichton alexcrichton changed the title Change all S3 URLs in our CI to use subdomains azure: Change all S3 URLs in our CI to use subdomains May 25, 2019
alexcrichton added a commit to alexcrichton/rust that referenced this issue May 29, 2019
Ensure that they're all forwards-compatible with AWS updates happening
next year by ensuring the bucket name shows up in the domain name.

Closes rust-lang#61168
Centril added a commit to Centril/rust that referenced this issue May 30, 2019
Update all s3 URLs used on CI with subdomains

Ensure that they're all forwards-compatible with AWS updates happening
next year by ensuring the bucket name shows up in the domain name.

Closes rust-lang#61168
Centril added a commit to Centril/rust that referenced this issue May 30, 2019
Update all s3 URLs used on CI with subdomains

Ensure that they're all forwards-compatible with AWS updates happening
next year by ensuring the bucket name shows up in the domain name.

Closes rust-lang#61168
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant