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

Move default travis API to travis-ci.com #1238

Merged
merged 7 commits into from Aug 20, 2021

Conversation

andre15silva
Copy link
Contributor

Fixes #1231

Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Copy link
Contributor Author

andre15silva commented Aug 13, 2021

The remaining places where travis-ci.org is used are tests that are built on top of certain travis builds only available on travis-ci.org.

After documentation is updated, this PR is ready.

@monperrus
Copy link
Contributor

Nice.

Signed-off-by: André Silva <andre15andre@hotmail.com>
This option is no longer required since the `launcherMode` option
already makes the distinction between which modes use the
`TravisMainProcess` and which use the `GithubMainProcess`

Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Copy link
Contributor Author

travis-ci.com isn't behaving as expected in https://travis-ci.com/github/repairnator/test-repairnator/requests

I'm trying to trigger https://travis-ci.org/surli/test-repairnator/builds/352395977 but in travis-ci.com. I'll try a bit later, in case it is just a temporary issue with travis.

Also, the build 564711868 is no longer available, and is used in an example in the README. That should also be replaced by another example.

@monperrus
Copy link
Contributor

monperrus commented Aug 20, 2021 via email

Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Copy link
Contributor Author

I was able to get a "build request" working right now, see app.travis-ci.com/github/repairnator/test-repairnator/requests

Thanks! Docs updated.

The only thing missing now is to update the repairnator/pipeline docker image, so that the default there is also travis-ci.com and the examples from the docs actually work.

For that we need to deploy a new version with these changes, so I say we merge this first, deploy, and then build the new docker image.

@andre15silva andre15silva marked this pull request as ready for review August 20, 2021 09:24
@monperrus monperrus merged commit c4dc012 into eclipse:master Aug 20, 2021
@monperrus
Copy link
Contributor

Very good. Merged, thanks @andre15silva.

Do you have push access on dockerhub?

@andre15silva
Copy link
Contributor Author

andre15silva commented Aug 20, 2021

repairnator-core and repairnator-pipeline snapshots deployed.

Do you have push access on dockerhub?

Yes, I have pushed the new image tagged as repairnator/pipeline:3.3 and repairnator/pipeline:latest

@monperrus
Copy link
Contributor

monperrus commented Aug 21, 2021 via email

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

Successfully merging this pull request may close these issues.

Make travis-ci.com default
2 participants