Skip to content

CI: Increase travis timeout to 30 minutes (2).#22636

Merged
charris merged 1 commit intonumpy:mainfrom
charris:increase-travis-timeout
Nov 21, 2022
Merged

CI: Increase travis timeout to 30 minutes (2).#22636
charris merged 1 commit intonumpy:mainfrom
charris:increase-travis-timeout

Conversation

@charris
Copy link
Copy Markdown
Member

@charris charris commented Nov 21, 2022

This is an attempted fix for the Python 3.8 aarch64 builds that are timing out when merged to master.

@charris
Copy link
Copy Markdown
Member Author

charris commented Nov 21, 2022

This is a second attempt to extend the travis timeout time.

@charris charris force-pushed the increase-travis-timeout branch from 6235b6f to 8574162 Compare November 21, 2022 02:05
This is an attempted fix for the Python 3.8 aarch64 builds that are
timing out when merged to master.
@charris charris force-pushed the increase-travis-timeout branch from 8574162 to 743b1f9 Compare November 21, 2022 02:09
@charris charris changed the title CI: Increase travis timeout to 20 minutes (2). CI: Increase travis timeout to 30 minutes (2). Nov 21, 2022
@charris charris merged commit 5f0b846 into numpy:main Nov 21, 2022
@charris
Copy link
Copy Markdown
Member Author

charris commented Nov 21, 2022

Self merging again.

@charris
Copy link
Copy Markdown
Member Author

charris commented Nov 21, 2022

That didn't work either, but at least it didn't screw things up :)

@seberg
Copy link
Copy Markdown
Member

seberg commented Nov 21, 2022

That didn't work either, but at least it didn't screw things up :)

:), thanks for looking into this, Travis was sending a bit many failure mails lately.

@mattip
Copy link
Copy Markdown
Member

mattip commented Nov 21, 2022

This might be a problem with hitting some internal travis CI limits. Scipy is using cirrus ci for linux aarch64 macos arm64, and musl. The config file is here, and the CI runs are here.

@mattip
Copy link
Copy Markdown
Member

mattip commented Nov 21, 2022

Pinging @andyfaff and @rgommers since it seems they did most of the work to move the scipy builds. We might want to hit the mailing list if we decide to go for this so there is some record of why we are adding a new CI provider...

@andyfaff
Copy link
Copy Markdown
Member

andyfaff commented Nov 21, 2022

It's fairly straightforward to use cirrus-ci for CI for all those targets. The config files are a little different to other CI, in that the .cirrus.star is a starlark script that creates jobs from other yml files. I found it pretty powerful and responsive. More than happy to help any transfer.

@andyfaff
Copy link
Copy Markdown
Member

(I don't think it'll cover s390, ppc64le, etc without cross-compiling or qemu)

@rgommers
Copy link
Copy Markdown
Member

I agree Cirrus CI has been very nice.

(I don't think it'll cover s390, ppc64le, etc without cross-compiling or qemu)

If we want to keep those, then I don't think there's a need to move right now - we're basically stuck with TravisCI.

@charris
Copy link
Copy Markdown
Member Author

charris commented Nov 21, 2022

Scipy is using cirrus ci

Moving to cirrus looks like the way to go.

@charris
Copy link
Copy Markdown
Member Author

charris commented Nov 21, 2022

Note that I am going to remove Python 3.8 from NumPy 1.25, so this won't matter anymore after 1.24 is branched. But I suspect that what happened is that we were on the edge for all the test runs, and some small change pushed us over. If we cannot build 3.8 wheels for aarch64, that is how it is :) I'd still like to move to cirrus.

@mattip mattip mentioned this pull request Nov 21, 2022
6 tasks
@charris charris deleted the increase-travis-timeout branch December 29, 2022 22:18
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.

5 participants