Skip to content

ci: Upload/download from a new S3 bucket - #44617

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:download-from-us-west-1
Sep 17, 2017
Merged

ci: Upload/download from a new S3 bucket#44617
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:download-from-us-west-1

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Moving buckets from us-east-1 to us-west-1 because us-west-1 is where
rust-central-station itself runs and in general is where we have all our other
buckets.

@alexcrichton

Copy link
Copy Markdown
MemberAuthor

r? @aidanhs

@alexcrichton

Copy link
Copy Markdown
MemberAuthor

@Mark-Simulacrum

Copy link
Copy Markdown
Member

This should land as a PR to bisect-rust first, adding these URLs to the list we look at, likely duplicating all of items. That way we don't break perf.rlo (yes, we should migrate this over to rust-lang-nursery probably).

@alexcrichton

Copy link
Copy Markdown
MemberAuthor

Note that this won't break anything, replication is set up so the objects should appear in rust-lang-ci as well

@aidanhs

Copy link
Copy Markdown
Contributor

As step 1, don't we need to do a copy from the east bucket to west, so that we have all the mirrored artifacts already there?

E.g. I note I can't download https://s3-us-west-1.amazonaws.com/rust-lang-ci2/rust-ci-mirror/2017-03-17-stamp-x86_64-unknown-linux-musl so I assume the same will be true for travis.

@alexcrichton

Copy link
Copy Markdown
MemberAuthor

As step 1, don't we need to do a copy from the east bucket to west, so that we have all the mirrored artifacts already there?

I'm afraid this'll take long enough that we'd lose a build or two. I've mirrored the smaller folders like rust-ci-mirror (but botched the permissions so you couldn't download, should be fixed now), but the big folders like rustc-builds can start getting transferred once this is landed.

@aidanhs

Copy link
Copy Markdown
Contributor

@bors r+ rollup

If bors is happy then I am happy.

@bors

bors commented Sep 15, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit a7144b6 has been approved by aidanhs

@aidanhs

Copy link
Copy Markdown
Contributor

@bors r-

[00:03:32] tidy error: /checkout/src/ci/docker/scripts/sccache.sh:14: line longer than 100 chars

Moving buckets from us-east-1 to us-west-1 because us-west-1 is where
rust-central-station itself runs and in general is where we have all our other
buckets.
@alexcrichton

Copy link
Copy Markdown
MemberAuthor

@bors: r=aidanhs

@bors

bors commented Sep 16, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 5cfee9b has been approved by aidanhs

TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
…1, r=aidanhs
ci: Upload/download from a new S3 bucket
Moving buckets from us-east-1 to us-west-1 because us-west-1 is where
rust-central-station itself runs and in general is where we have all our other
buckets.
@TimNNTimNN mentioned this pull request Sep 17, 2017
TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
…1, r=aidanhs
ci: Upload/download from a new S3 bucket
Moving buckets from us-east-1 to us-west-1 because us-west-1 is where
rust-central-station itself runs and in general is where we have all our other
buckets.
@TimNNTimNN mentioned this pull request Sep 17, 2017
TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
…1, r=aidanhs
ci: Upload/download from a new S3 bucket
Moving buckets from us-east-1 to us-west-1 because us-west-1 is where
rust-central-station itself runs and in general is where we have all our other
buckets.
@TimNNTimNN mentioned this pull request Sep 17, 2017
bors added a commit that referenced this pull request Sep 17, 2017
@bors
bors merged commit 5cfee9b into rust-lang:masterSep 17, 2017
@alexcrichton
alexcrichton deleted the download-from-us-west-1 branch September 17, 2017 16:44
@alexcrichtonalexcrichton added beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. labels Sep 17, 2017
bors added a commit that referenced this pull request Sep 26, 2017
[beta] Backport accepted PRs to 1.21
Backport of:
- ~don't suggest placing `use` statements into expanded code #44215
- stabilize tcpstream_connect_timeout #44563
- stabilized iterator_for_each #44567
- travis: Move sccache to the us-west-1 region #44574
- stabilized ord_max_min #44593
- stabilized compiler_fences #44595
- ci: Upload/download from a new S3 bucket #44617
- stabilized needs_drop #44639
- Stabilized vec_splice and modified splice tracking issue #44640
- Backport libs stabilizations to 1.21 beta #44824
@dtolnaydtolnay removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Sep 26, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-acceptedAccepted for backporting to the compiler in the beta channel.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@alexcrichton@Mark-Simulacrum@aidanhs@bors@dtolnay