Skip to content

CI: Trim some tests from i686-gnu #59054

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

Merged
merged 1 commit into from
Mar 10, 2019
Merged

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Mar 9, 2019

This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.

@rust-highfive
Copy link
Contributor

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 9, 2019
@Centril
Copy link
Contributor

Centril commented Mar 10, 2019

r? @pietroalbini

@bors p=50

@pietroalbini
Copy link
Member

Yeah, this seems fine to me.

@bors r+ p=50

@bors
Copy link
Collaborator

bors commented Mar 10, 2019

📌 Commit 635b2c95446a54c819c4efcc1eb3123de4bcfe80 has been approved by pietroalbini

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 10, 2019
@pietroalbini
Copy link
Member

cc @rust-lang/infra

@pietroalbini
Copy link
Member

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 10, 2019
This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.
@ehuss ehuss force-pushed the ci-i686-gnu-tests branch from 635b2c9 to 609316a Compare March 10, 2019 15:23
@ehuss
Copy link
Contributor Author

ehuss commented Mar 10, 2019

My apologies on the typo. I also removed src/doc because it is an implicit dependency from linkchecker.

@pietroalbini
Copy link
Member

@bors r+ p=50

@bors
Copy link
Collaborator

bors commented Mar 10, 2019

📌 Commit 609316a has been approved by pietroalbini

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 10, 2019
@bors
Copy link
Collaborator

bors commented Mar 10, 2019

⌛ Testing commit 609316a with merge 827ca46317788c731a7dc7cd265596de48debeeb...

@rust-highfive
Copy link
Contributor

The job dist-x86_64-linux of your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
[00:01:20] 
[00:01:20] Total download size: 4.9 M
[00:01:20] Downloading Packages:
[00:01:22] --------------------------------------------------------------------------------
[00:01:22] Total                                           1.8 MB/s | 4.9 MB     00:02     
[00:01:22] warning: rpmts_HdrFromFdno: Header V3 DSA signature: NOKEY, key ID e8562897
[00:01:22] Importing GPG key 0xE8562897 "CentOS-5 Key (CentOS 5 Official Signing Key) <[email protected]>" from /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-5
[00:01:22] Running Transaction Test
[00:01:22] Finished Transaction Test
[00:01:22] Transaction Test Succeeded
[00:01:23] Running Transaction
---
[00:04:08] + hide_output make install
[00:04:08] + set +x
[00:04:30] shared.sh: line 1:   352 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"
[00:04:30] + cd ..
[00:04:30] + rm -rf openssl-1.0.2k
[00:04:30] ./build-openssl.sh: line 16:  4115 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"  (wd: /tmp/openssl-1.0.2k)
[00:04:30] + ln -nsf /etc/pki/tls/cert.pem /rustroot/ssl/
[00:04:31]  ---> f2e67aeafcd4
[00:04:31] Step 14/41 : COPY dist-x86_64-linux/build-curl.sh /tmp/
[00:04:32]  ---> 471a6412d298
[00:04:32] Step 15/41 : RUN ./build-curl.sh
[00:04:32] Step 15/41 : RUN ./build-curl.sh
[00:04:32]  ---> Running in 6a9f7cabb59b
[00:04:32] + source shared.sh
[00:04:32] + VERSION=7.51.0
[00:04:32] + curl http://cool.haxx.se/download/curl-7.51.0.tar.bz2
[00:04:34]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[00:04:34]                                  Dload  Upload   Total   Spent    Left  Speed
[00:04:35] 
  0 2509k    0  1183    0     0    776      0  0:55:11  0:00:01  0:55:10   776
  0 2509k    0  1183    0     0    776      0  0:55:11  0:00:01  0:55:10   776
 56 2509k   56 1420k    0     0   570k      0  0:00:04  0:00:02  0:00:02 1466k
100 2509k  100 2509k    0     0   896k      0  0:00:02  0:00:02 --:--:-- 1969k
[00:04:35] + mkdir curl-build
[00:04:35] + cd curl-build
[00:04:35] + hide_output ../curl-7.51.0/configure --prefix=/rustroot --with-ssl=/rustroot --disable-sspi --disable-gopher --disable-smtp --disable-smb --disable-imap --disable-pop3 --disable-tftp --disable-telnet --disable-manual --disable-dict --disable-rtsp --disable-ldaps --disable-ldap
[00:05:02] + hide_output make -j10
[00:05:02] + set +x
[00:05:16] shared.sh: line 1:    13 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"
[00:05:16] + hide_output make install
---
 79 67.8M   79 53.8M    0     0  6850k      0  0:00:10  0:00:08  0:00:02 7212k
 91 67.8M   91 61.8M    0     0  6992k      0  0:00:09  0:00:09 --:--:-- 7158k
100 67.8M  100 67.8M    0     0  7028k      0  0:00:09  0:00:09 --:--:-- 7616k
[00:09:14] + cd gcc-5.5.0
[00:09:14] + sed -i 's|ftp://gcc\.gnu\.org/|http://gcc.gnu.org/|g' ./contrib/download_prerequisites
[00:09:14] --2019-03-10 16:08:46--  http://gcc.gnu.org/pub/gcc/infrastructure/mpfr-2.4.2.tar.bz2
[00:09:14] Resolving gcc.gnu.org... 209.132.180.131
[00:09:14] Connecting to gcc.gnu.org|209.132.180.131|:80... connected.
[00:09:14] HTTP request sent, awaiting response... 200 OK
---
[01:14:52]  ---> 59c3369741b9
[01:14:52] Step 25/41 : RUN ./build-clang.sh
[01:14:52]  ---> Running in b577d2bca641
[01:14:52] + source shared.sh
[01:14:52] + LLVM=llvmorg-8.0.0-rc2
[01:14:52] + mkdir llvm-project
[01:14:52] + cd llvm-project
[01:14:52] + curl -L https://github.com/llvm/llvm-project/archive/llvmorg-8.0.0-rc2.tar.gz
[01:14:52] + tar xzf - --strip-components=1
[01:14:52]                                  Dload  Upload   Total   Spent    Left  Speed
[01:14:53] 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   136    0   136    0     0    566      0 --:--:-- --:--:-- --:--:--   657
---
[01:15:11] + cd clang-build
[01:15:11] + INC=/rustroot/include
[01:15:11] + INC=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/5.5.0/include-fixed
[01:15:11] + INC=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/5.5.0/include-fixed:/usr/include
[01:15:11] + hide_output cmake ../llvm -DCMAKE_C_COMPILER=/rustroot/bin/gcc -DCMAKE_CXX_COMPILER=/rustroot/bin/g++ -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/rustroot -DLLVM_TARGETS_TO_BUILD=X86 '-DLLVM_ENABLE_PROJECTS=clang;lld' -DC_INCLUDE_DIRS=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/5.5.0/include-fixed:/usr/include
[01:15:42] Sun Mar 10 17:15:13 UTC 2019 - building ...
[01:15:53] + hide_output make -j10
[01:15:53] + set +x
[01:16:23] Sun Mar 10 17:15:55 UTC 2019 - building ...
---
[02:44:52]  ---> 07458566a534
[02:44:52] Step 32/41 : RUN ./build-perl.sh
[02:44:52]  ---> Running in edd48f553a04
[02:44:52] + source shared.sh
[02:44:52] + curl https://www.cpan.org/src/5.0/perl-5.28.0.tar.gz
[02:44:53]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[02:44:53]                                  Dload  Upload   Total   Spent    Left  Speed
[02:44:54] 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
 30 17.0M   30 5357k    0     0  7827k      0  0:00:02 --:--:--  0:00:02 8959k
100 17.0M  100 17.0M    0     0  13.9M      0  0:00:01  0:00:01 --:--:-- 15.0M
[02:44:54] + cd perl-5.28.0
[02:44:54] + CC=gcc
[02:44:54] + CFLAGS='-I /rustroot/include -fgnu89-inline'
[02:44:54] + hide_output ./configure.gnu
[02:44:54] + set +x
[02:45:24] Sun Mar 10 18:44:56 UTC 2019 - building ...
[02:45:39] + hide_output make -j10

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@bors
Copy link
Collaborator

bors commented Mar 10, 2019

💔 Test failed - checks-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 10, 2019
@kennytm
Copy link
Member

kennytm commented Mar 10, 2019

@bors retry

What's up with the docker cache????

@bors bors removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 10, 2019
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Mar 10, 2019
@bors
Copy link
Collaborator

bors commented Mar 10, 2019

⌛ Testing commit 609316a with merge 9d71ec1...

bors added a commit that referenced this pull request Mar 10, 2019
CI: Trim some tests from i686-gnu

This removes some tests from the i686-gnu job. This job clocks in at 2hr 56min, and removing these should cut about 10 to 15 minutes, giving a little more breathing room. I suspect these don't need to be tested on every platform.
@pietroalbini
Copy link
Member

What's up with the docker cache????

This PR changed a Docker image so all of them will be rebuilt.

@bors
Copy link
Collaborator

bors commented Mar 10, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: pietroalbini
Pushing 9d71ec1 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 10, 2019
@bors bors merged commit 609316a into rust-lang:master Mar 10, 2019
@kennytm
Copy link
Member

kennytm commented Mar 11, 2019

@pietroalbini This PR only changed the Dockerfile of i686-gnu, it shouldn't affect dist-x86_64-linux. The current hash calculation is probably too conservative.

@Mark-Simulacrum
Copy link
Member

Due to dependencies on other folders, we essentially have to be as pessimistic as possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants