Skip to content

CI: CircleCI seems to occasionally time out, increase the limit - #18349

Merged
charris merged 1 commit into
numpy:masterfrom
seberg:circle-ci-timeout
Feb 7, 2021
Merged

charris merged 1 commit into
numpy:masterfrom
seberg:circle-ci-timeout

Conversation

@seberg

@seberg seberg commented Feb 6, 2021 •

Copy link
Copy Markdown
Member

CircleCI seems to default to a 10 minute timeout on no output,
simply increase it to 30 minutes to ensure the build should
succeed.
This is assuming that the build time simply crawled up over 10
minutes just now.

CircleCI seems to default to a 10 minute timeout on no output,
simply increase it to 30 minutes to ensure the build should
succeed.
It seems the build does take 25+ minutes (maybe depending on the
machine it runs on). Technically, 30 minutes is probably more than
necessary, since there is a print in the middle, but that is
just a warning that could go away at some point.
@seberg
seberg force-pushed the circle-ci-timeout branch from 1b29052 to 9e9d3e5 Compare February 6, 2021 23:10
@seberg

seberg commented Feb 6, 2021

Copy link
Copy Markdown
Member Author

The build took 26 minutes on this run. It does print out in the middle, but to not rely on that too much, upped it to 30 instead of 20 minutes.

I guess we get different machines, on some it finishes in 10 minutes, on others it takes almost 30 minutes.

@seberg

seberg commented Feb 7, 2021

Copy link
Copy Markdown
Member Author

@rossbar I think we should just put this in and not worry about it for a while, but if you happen to know that there is a reason (beyond circleci maybe just using different hardware), while the build is so slow sometimes, that might be interesting.

@charris
charris merged commit 3e2406a into numpy:master Feb 7, 2021
@charris

charris commented Feb 7, 2021

Copy link
Copy Markdown
Member

Thanks Sebastian.

@seberg
seberg deleted the circle-ci-timeout branch February 7, 2021 00:48
@charris charris removed the 09 - Backport-Candidate PRs tagged should be backported label Feb 7, 2021
@charris charris removed this from the 1.20.1 release milestone Feb 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants