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

re-enable Jenkins to run e2e #2018

Merged
merged 2 commits into from May 8, 2018

Conversation

@jboyd01
Copy link
Contributor

commented May 3, 2018

This PR re-enables e2e in the Jenkins job. I ran jenkins tests a total of 6 times with this PR with no issues. This depends on #2010 or #2016 first being merged and then I'll need to rebase to pick up those changes to pass Jenkins.

Re-enabling e2e looks to add no more then 3-5 minutes max with prep, running the tests, and then teardown and collection of logs. Total runs of the Jenkins job comes in around 33-36 minutes.

@jboyd01

This comment has been minimized.

Copy link
Contributor Author

commented May 3, 2018

/retest

4 similar comments
@jboyd01

This comment has been minimized.

Copy link
Contributor Author

commented May 3, 2018

/retest

@jboyd01

This comment has been minimized.

Copy link
Contributor Author

commented May 3, 2018

/retest

@jboyd01

This comment has been minimized.

Copy link
Contributor Author

commented May 3, 2018

/retest

@jboyd01

This comment has been minimized.

Copy link
Contributor Author

commented May 4, 2018

/retest

@jboyd01 jboyd01 changed the title [WIP] enable Jenkins to run e2e re-enable Jenkins to run e2e May 4, 2018
@jboyd01 jboyd01 removed the do-not-merge label May 4, 2018
@jpeeler jpeeler added the LGTM1 label May 4, 2018
@pmorie pmorie requested a review from kibbles-n-bytes May 7, 2018
@jboyd01 jboyd01 force-pushed the jboyd01:enable-e2e branch from ed116bc to 63edf8e May 8, 2018
@carolynvs carolynvs added the LGTM2 label May 8, 2018
Copy link
Contributor

left a comment

LGTM

@carolynvs carolynvs merged commit 4f5ed2a into kubernetes-sigs:master May 8, 2018
3 checks passed
3 checks passed
cla/linuxfoundation jboyd01 authorized
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.