Skip to content

[CI] Try to stabilize Travis job#685

Merged
simon-mo merged 2 commits intoucbrise:developfrom
withsmilo:stabilize_travis_job
May 7, 2019
Merged

[CI] Try to stabilize Travis job#685
simon-mo merged 2 commits intoucbrise:developfrom
withsmilo:stabilize_travis_job

Conversation

@withsmilo
Copy link
Collaborator

There are some cases where Travis rarely fails due to an unknown error.
I think we need a little bit of waiting time to finish one test, delete the clipper cluster,
and then recreate the clipper cluster for a new test. Let's try one patch to solve this problem.

There are some cases where Travis rarely fails due to an unknown error.
I think we need a little bit of waiting time to finish one test, delete the clipper cluster,
and then recreate the clipper cluster for a new test. Let's try one patch to solve this problem.
@withsmilo withsmilo self-assigned this May 5, 2019
@withsmilo withsmilo changed the title [CITry to stabilize Travis job [CI] Try to stabilize Travis job May 5, 2019
Travis shows this message now.
'This job is running using the default Python version, which was updated to 3.6 on April 16th, 2019. You can explicitly stay on the previous version by specifying python: 2.7 in your .travis.yml.'
@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1930/
Test PASSed.

@withsmilo withsmilo requested a review from simon-mo May 5, 2019 15:08
@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1931/
Test PASSed.


language: python

python: "2.7"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why don't we specify all the versions we support? I believe we support 2.7, 3.5, and 3.6.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will actually create many builds. Just 2.7 is fine for now.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simon is right. Currently the Travis is used only for the Kubernetes integration tests.

@simon-mo simon-mo merged commit defb131 into ucbrise:develop May 7, 2019
@withsmilo withsmilo deleted the stabilize_travis_job branch May 7, 2019 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants