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.

Sign up for GitHub

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

Jump to bottom

Switch to vault.centos.org #41045

Merged
merged 2 commits into from
Apr 4, 2017
Merged

Switch to vault.centos.org #41045

merged 2 commits into from
Apr 4, 2017

Conversation

TimNN
Copy link
Contributor

@TimNN TimNN commented Apr 3, 2017

Centos 5 is EOL and is no longer available from the usual mirrors, so switch to http://vault.centos.org/

r? @alexcrichton

(I changed the sed commands to apply to all *.repo files, since there were some bad urls in the other ones as well, I think).

@alexcrichton
Copy link
Member

@bors: r+ p=10000

@bors
Copy link
Contributor

bors commented Apr 3, 2017

📌 Commit 802c11c has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Apr 3, 2017

⌛ Testing commit 802c11c with merge 5677ac8...

@alexcrichton alexcrichton mentioned this pull request Apr 3, 2017
Open
@bors
Copy link
Contributor

bors commented Apr 3, 2017

💔 Test failed - status-travis

@frewsxcv
Copy link
Member

frewsxcv commented Apr 3, 2017

@bors retry

@bors
Copy link
Contributor

bors commented Apr 3, 2017

⌛ Testing commit 802c11c with merge 7a968d1...

@frewsxcv
Copy link
Member

frewsxcv commented Apr 3, 2017

@bors retry

@frewsxcv
Copy link
Member

frewsxcv commented Apr 3, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Apr 3, 2017

📌 Commit 6dc8869 has been approved by frewsxcv

@bors
Copy link
Contributor

bors commented Apr 3, 2017

⌛ Testing commit 6dc8869 with merge 2b79138...

@aidanhs
Copy link
Member

aidanhs commented Apr 3, 2017

Just to record this somewhere, dnschecker.org currently lists the following IPs (when deduplicated) for vault.centos.org:

88.208.217.170
107.158.252.35
108.61.16.227
109.72.83.61
160.10.26.25

Of these five (which I suspect are a tiny subset), two would definitely fail:

$ for f in $(cat y); do curl -sf http://$f/centos/5/os/x86_64/repodata/ >/dev/null || echo bad $f; done
bad 108.61.16.227
bad 160.10.26.25

Also, a relevant post from centos-devel about slow mirror updates last month: https://lists.centos.org/pipermail/centos-devel/2017-March/015708.html

@TimNN
Copy link
Contributor Author

TimNN commented Apr 3, 2017 •
edited
Loading

@aidanhs: I would assume that vault.centos.org doesn't use the official mirrors of the non-eol centos releases.

I believe that the ip's we are seeing for vault.centos.org are all maintained by the centos project -- http://vault.centos.org/readme.txt lists some "real" mirrors.

@bors bors mentioned this pull request Apr 3, 2017
Merged
@frewsxcv
Copy link
Member

frewsxcv commented Apr 3, 2017

@bors delegate=TimNN

Giving you review privileges in case Travis fails so you can 'bors retry'

@bors
Copy link
Contributor

bors commented Apr 3, 2017

✌️ @TimNN can now approve this pull request

@aidanhs
Copy link
Member

aidanhs commented Apr 3, 2017

@TimNN yeah good point, I stumbled across that page just after posting the comment above but they currently look like they need to catch up too. I'd lean towards working on a non-centos5 strategy after this has merged rather than moving over to a real mirror.

@bors bors mentioned this pull request Apr 3, 2017
Closed
@alexcrichton alexcrichton added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Apr 3, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Apr 3, 2017
@frewsxcv
Rollup merge of rust-lang#41048 - TimNN:centos5-vault-beta, r=alexcri…
68fcad8 
…chton

[beta] Backport rust-lang#41045: switch to vault.centos.org

r? @alexcrichton
@bors
Copy link
Contributor

bors commented Apr 4, 2017

💔 Test failed - status-travis

@mrhota
Copy link
Contributor

mrhota commented Apr 4, 2017

😞 build merely took too long and was terminated

@frewsxcv
Copy link
Member

frewsxcv commented Apr 4, 2017

@bors retry

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

@bors retry

  • ar.exe failed

@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge a9214df...

@bors
Copy link
Contributor

bors commented Apr 4, 2017

💔 Test failed - status-travis

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

@bors retry

@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge c417350...

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

(By the way, just to establish the link, the ar.exe failure is #40546 (Don't want to edit the previous comment or bo_s will pick up the retry again))

@bors
Copy link
Contributor

bors commented Apr 4, 2017

💔 Test failed - status-travis

@frewsxcv
Copy link
Member

frewsxcv commented Apr 4, 2017

@bors retry

@frewsxcv
Copy link
Member

frewsxcv commented Apr 4, 2017

It seems to consistently be dist-arm-linux causing issues here.

Here's the build log for the last fail for that machine:

https://api.travis-ci.org/jobs/218400741/log.txt?deansi=true

The build successfully finished at 9:01AM EST, but then start doing cleanup and caching and reached the three hour mark at 9:06AM EST.

@aidanhs
Copy link
Member

aidanhs commented Apr 4, 2017 •
edited
Loading

For people wanting to read the log, build starts at Tue, 04 Apr 2017 10:07:59 GMT ("1491300479230695090") and the before_cache steps finish at Tue, 04 Apr 2017 13:06:17 GMT ("1491311177686306522").

This was referenced Apr 4, 2017
Merged
Merged
@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge a5cce63...

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

@bors retry

@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge 89db846...

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

@bors retry

@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge 7a966a4...

@TimNN
Copy link
Contributor Author

TimNN commented Apr 4, 2017

@bors retry

bors added a commit that referenced this pull request Apr 4, 2017
@bors
Auto merge of #41059 - TimNN:rollup, r=TimNN
2564711 
Rollup of 1 pull requests

- Successful merges: #40998
- Failed merges: #41045
@bors
Copy link
Contributor

bors commented Apr 4, 2017

⌛ Testing commit 6dc8869 with merge 2564711...

@bors bors merged commit 6dc8869 into rust-lang:master Apr 4, 2017
@TimNN TimNN deleted the centos5-vault branch April 4, 2017 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexcrichton alexcrichton

Labels
beta-accepted Accepted for backporting to the compiler in the beta channel.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@TimNN @alexcrichton @bors @frewsxcv @aidanhs @mrhota