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

Better handling of ginkgo args in jenkins #11291

Merged
merged 1 commit into from
Jul 15, 2015
Merged

Conversation

thockin
Copy link
Member

@thockin thockin commented Jul 15, 2015

@thockin
Copy link
Member Author

thockin commented Jul 15, 2015

@ixdy

@davidopp in lieu of quinton

@wojtek-t
Copy link
Member

Nice fix - LGTM

@k8s-bot
Copy link

k8s-bot commented Jul 15, 2015

GCE e2e build/test passed for commit 4b409b81d29794366bd65fac0a9d79f7dd4ced66.

@k8s-bot
Copy link

k8s-bot commented Jul 15, 2015

GCE e2e build/test failed for commit 38d51f3.

@davidopp
Copy link
Member

LGTM

@davidopp davidopp added lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-merge labels Jul 15, 2015
@thockin
Copy link
Member Author

thockin commented Jul 15, 2015

@k8s-bot test this please

@k8s-bot
Copy link

k8s-bot commented Jul 15, 2015

GCE e2e build/test passed for commit 38d51f3.

@ixdy
Copy link
Member

ixdy commented Jul 15, 2015

LGTM, thanks.

@erictune
Copy link
Member

I'm not merging any code until Jenkins is green for a turn.

erictune added a commit that referenced this pull request Jul 15, 2015
Better handling of ginkgo args in jenkins
@erictune erictune merged commit 8c891cf into kubernetes:master Jul 15, 2015
@thockin thockin deleted the jenkins branch July 27, 2015 15:12
erictune added a commit to erictune/kubernetes that referenced this pull request Jan 22, 2016
Was marked as flaky prior to kubernetes#11291, 6 months ago.
Code has changed many times.
No flakes in last 28 runs, and no Jenkins
history before that.
Follow up in kubernetes#19024 if you see a flake.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants