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.

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

[build] Travis uses sauce Labs limited to 5 concurrent jobs #148

Closed
ignasi35 opened this issue Apr 3, 2019 · 4 comments
Closed

[build] Travis uses sauce Labs limited to 5 concurrent jobs #148

ignasi35 opened this issue Apr 3, 2019 · 4 comments

Comments

@ignasi35
Copy link
Member

ignasi35 commented Apr 3, 2019

When migrating to travis-ci.com we increased the number of workers to up-to 13. This surfaced an issue in the build:

  • Sauce Labs limits to 5 the number of concurrent jobs so it's not uncommon to see job failures when all the Travis worker pool is available.

This is currently fixed using https://blog.travis-ci.com/2014-07-18-per-repository-concurrency-setting but that's not git tracked or explicit in .travis.yml.

@ignasi35
Copy link
Member Author

ignasi35 commented Apr 3, 2019

An option is to merge scripts into one fat script doing test, validateCode, etc...

Another option is to split the build into stages of 4 jobs each (each stage is a JDK+SCALA_VERSION combination).

@ignasi35
Copy link
Member Author

ignasi35 commented Apr 3, 2019

I don't think we need the Sauce Labs add-on on every job (I don't think it's used in validateCode, for example) so maybe setting the add-on conditionally could be useful too.

@xuwei-k
Copy link
Contributor

xuwei-k commented Oct 8, 2022

5b35f53

@mkurz
Copy link
Member

mkurz commented Oct 8, 2022

How was that sauce labs actually used? For browser testing I guess, bit I don't think we need it anymore...

@mkurz mkurz closed this as completed Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants