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

Fix kubectl e2e test #8309

Merged
merged 1 commit into from May 15, 2015
Merged

Fix kubectl e2e test #8309

merged 1 commit into from May 15, 2015

Conversation

gmarek
Copy link
Contributor

@gmarek gmarek commented May 15, 2015

It was broken for a very long time, but it surfaced yesterday.

@gmarek gmarek force-pushed the fix_e2e branch 3 times, most recently from d312cc5 to de76646 Compare May 15, 2015 12:58
@fgrzadkowski
Copy link
Contributor

LGTM. Let's make sure that e2e tests pass and that either travis or shippable is green.

@wojtek-t
Copy link
Member

Shippable is green - merging.

wojtek-t added a commit that referenced this pull request May 15, 2015
@wojtek-t wojtek-t merged commit 71a09f6 into kubernetes:master May 15, 2015
@gmarek
Copy link
Contributor Author

gmarek commented May 15, 2015

"should do a rolling update of a replication controller" in update-demo test is still failing, but it was even before this fix.

@jayunit100
Copy link
Member

related: created #8325 to confirm namespaces are deleted efficiently

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.

None yet

5 participants