Travis CI skip conditions
See original GitHub issueRecently the Travis CI queue has taken a bit of a beating. This has been caused by large demand. The small number of workers (appears to 164) quickly getting maxed out. Some unfortunate multi-day infrastructure issues that left jobs hanging. This has led me to wonder what improvements we can make in this area. The big wins unfortunately will require a significant amount of discussion and work. Though there are still some simple small wins that should improve on the situation.
Awhile back we added PRs ( https://github.com/conda-forge/conda-smithy/pull/87 ) ( https://github.com/conda-forge/conda-smithy/pull/94 ) which eliminated the appveyor.yaml
file when it wasn’t needed. Subsequently @pelson put forward PR ( https://github.com/conda-forge/conda-smithy/pull/95 ), which pushed a button on AppVeyor to disable builds when appveyor.yaml
wasn’t present. This saved a significant number of builds on AppVeyor and made that queue very usable for us.
We can use this same strategy with Travis CI. It also has a toggle switch that can be set via the Travis CI API as stated in the docs. Also, we can use the same logic to rename the .travis.yml
file. All of this should only be a handful of lines of code. While this isn’t a huge win, it is easy to do and does help some.
Issue Analytics
- State:
- Created 7 years ago
- Comments:6 (6 by maintainers)
So, I think there are ~25 feedstocks that have benefited from these improvements and skipping CircleCI builds via PR ( https://github.com/conda-forge/conda-smithy/pull/218 ). Again this isn’t huge when looking at the nearly 800 feedstocks present. Still it isn’t such a small number either. After all with CI usage the question is less about how many feedstocks and more about how many active feedstocks. Thus ~25 feedstocks ends up being a more significant number.
Thanks for this detail @jakirkham. FWIW, I’m available for a good few hours now to work on these issues. Ping me if you need anything.