Skip to content

Make each integration test use different prefix#27255

Merged
lavalamp merged 1 commit intokubernetes:masterfrom
lavalamp:fix-docs2
Jun 12, 2016
Merged

Make each integration test use different prefix#27255
lavalamp merged 1 commit intokubernetes:masterfrom
lavalamp:fix-docs2

Conversation

@lavalamp
Copy link
Copy Markdown
Contributor

Test theory about #27179

@lavalamp
Copy link
Copy Markdown
Contributor Author

I tried this locally and it seems to work. Going to self-merge, can't make things worse than they are right now.

@lavalamp lavalamp merged commit 6573ede into kubernetes:master Jun 12, 2016
@k8s-bot
Copy link
Copy Markdown

k8s-bot commented Jun 12, 2016

GCE e2e build/test passed for commit 6e6ae45.

@jsafrane
Copy link
Copy Markdown
Member

With a second look at the code, having unique prefix in etcd does not help at all. Most integration tests use framework.DeleteAllEtcdKeys(), which remove everything from etcd. I tried to look for a way how to pass the unique prefix to it, but integration tests are started in so many different ways that I gave up:

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.

4 participants