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

Improve petset e2es #26913

Merged
merged 1 commit into from
Jun 10, 2016
Merged

Conversation

bprashanth
Copy link
Contributor

Mainly

  • scale petset to 0 and wait for volume provisioner to delete pvs so we don't leak them
  • bake binaries into init[0] and cp into destination, instead of wgetting at runtime
  • dump verbose debug output on failure

kubernetes/test-infra#115, #26824

@k8s-github-robot k8s-github-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. release-note-label-needed labels Jun 6, 2016
@@ -51,6 +52,9 @@ const (
redisManifestPath = "test/e2e/testing-manifests/petset/redis"
)

// Time: 25m, slow by design.
// GCE Quota requirements: 3 pds, one per pet manifest declared above.
// GCE Api requirements: nodes and master need storage r/w permissions.
var _ = framework.KubeDescribe("PetSet", func() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Add "[Slow]" to description, since this is slow by design.

@bprashanth bprashanth force-pushed the petset_e2e branch 2 times, most recently from 90053ac to bbe019b Compare June 7, 2016 23:42
@bprashanth bprashanth added release-note-none Denotes a PR that doesn't merit a release note. and removed release-note-label-needed labels Jun 8, 2016
@j3ffml j3ffml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 8, 2016
@j3ffml j3ffml added this to the v1.3 milestone Jun 8, 2016
@j3ffml j3ffml added priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. area/test-infra labels Jun 8, 2016
@k8s-github-robot k8s-github-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jun 8, 2016
@k8s-github-robot k8s-github-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jun 8, 2016
@bprashanth
Copy link
Contributor Author

@jlowdermilk PTAL I added a new commit to explicitly join new members to zookeeper master, if there is one

@j3ffml
Copy link
Contributor

j3ffml commented Jun 8, 2016

Still lgtm, but please squash commits.

@j3ffml j3ffml added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 9, 2016
@bprashanth bprashanth added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. labels Jun 9, 2016
@bprashanth
Copy link
Contributor Author

@k8s-bot test this github issue: #26834

@k8s-bot
Copy link

k8s-bot commented Jun 9, 2016

GCE e2e build/test passed for commit ff23a0e.

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-bot
Copy link

k8s-bot commented Jun 10, 2016

GCE e2e build/test passed for commit ff23a0e.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit abc06c9 into kubernetes:master Jun 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test-infra lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. release-note-none Denotes a PR that doesn't merit a release note. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants