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

Migrate to travis-ci.com #2600

Closed
2 of 4 tasks
yurishkuro opened this issue Oct 28, 2020 · 12 comments
Closed
2 of 4 tasks

Migrate to travis-ci.com #2600

yurishkuro opened this issue Oct 28, 2020 · 12 comments
Labels

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Oct 28, 2020

Since yesterday our PRs are taking forever to run CI because there is just a single executor running across all of them.

Apparently, we need to migrate to travis-ci.com, and they did not do it automatically.

https://docs.travis-ci.com/user/migrate/open-source-repository-migration

@jpkrohling
Copy link
Contributor

I can take care of it tomorrow, if you haven't yet.

@yurishkuro
Copy link
Member Author

I migrated all repos, but we may have to go and re-define secrets, doesn't seem like they are moved during migration.

master build is now running 5 jobs: https://travis-ci.com/github/jaegertracing/jaeger/builds/195053976

@jpkrohling
Copy link
Contributor

Longer term, we should consider moving some (or all) of the jobs to GitHub Actions.

@yurishkuro
Copy link
Member Author

Maybe. Chris said GH Actions have 60 parallel runners per org, compared to 5 in Travis.

@yurishkuro
Copy link
Member Author

Update: the travis-ci.com builds are now required instead of travis-ci.org, however I see they keep failing on the unit test stage. For some reason the test command, according to travis, does not produce any output for 10min and it kills the job. I added travis_wait in #2598, yet to see if it helps (a run still fail for another odd reason), but it's far from ideal because it swallows all output, so if our tests run for 15min, we won't see anything in the logs until the test run is fully finished.

@yurishkuro
Copy link
Member Author

As far as I can tell, it keeps failing due to #2601.

@yurishkuro yurishkuro added meta-issue An tracking issue that requires work in other repos and removed needs-triage labels Oct 29, 2020
@yurishkuro
Copy link
Member Author

Changed to a tracking issue, because we need to make sure all SDK repos are able to build. And we need to copy over the secrets.

@jpkrohling
Copy link
Contributor

I can take a look, but I noticed that there's something weird already with the first repo I tried: jaeger-client-node. Did you migrate all repos already? In travis.com , I triggered a build for master, but looks like it doesn't see the right commit for "master":

image

image

@yurishkuro
Copy link
Member Author

We should try with a new commit. Some of Jaeger projects used to run on travis-ci.com in the past, so we can see the old build history. But I would've expected master build to work.

@yurishkuro
Copy link
Member Author

I think I migrated all repos, but we also need to change their branch protections to require .com builds instead of .org.

And we need to update build badges in the READMEs: https://github.com/search?q=org%3Ajaegertracing+travis-ci.org&type=code

@cwsolee
Copy link

cwsolee commented Nov 25, 2020

Just fyi, Travis post some updates here too https://blog.travis-ci.com/oss-announcement
@yurishkuro if you're requesting more credit from Travis but not getting response, let me know. We from IBM has been working with Travis quite closely since they add z & power support, I could follow up for you.

@yurishkuro
Copy link
Member Author

Superseded by #2645 - migrating to GitHub Actions.

@yurishkuro yurishkuro added wontfix and removed meta-issue An tracking issue that requires work in other repos labels Jan 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants