Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

DISPATCH-1115: make TravisCI job use a Proton release in addition to master#428

Closed
gemmellr wants to merge 1 commit into
apache:masterfrom
gemmellr:master
Closed

DISPATCH-1115: make TravisCI job use a Proton release in addition to master#428
gemmellr wants to merge 1 commit into
apache:masterfrom
gemmellr:master

Conversation

@gemmellr
Copy link
Copy Markdown
Member

There have been various cases that using only Proton master for CI caused Dispatch test failures, or lead to Dispatch master only working with proton master when this wasn't particularly desired at the time. This can cause wasted time when investigating test failures, and complicate release cycles / compatibility unnecessarily.

This change swaps to getting Proton from a checkout rather than submodule, and makes the proton version an env variable forming a matrix of sub-jobs to test different versions, such as master and last release as currently used here.

@gemmellr
Copy link
Copy Markdown
Member Author

Ignore this comment (but not the PR), testing integration.

@gemmellr
Copy link
Copy Markdown
Member Author

Ignore this comment (but not the PR), testing integration again.

@gemmellr
Copy link
Copy Markdown
Member Author

Updated to only do the coverage build and reporting tasks on one of the sub-jobs

@asfgit asfgit closed this in 181d97b Dec 21, 2018
@gemmellr
Copy link
Copy Markdown
Member Author

Pushed this as it seems to work as desired, if folks have other improvements we can add them later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant