-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Improve petset e2es #26913
Conversation
@@ -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() { |
There was a problem hiding this comment.
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.
90053ac
to
bbe019b
Compare
@jlowdermilk PTAL I added a new commit to explicitly join new members to zookeeper master, if there is one |
Still lgtm, but please squash commits. |
GCE e2e build/test passed for commit ff23a0e. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit ff23a0e. |
Automatic merge from submit-queue |
Mainly
kubernetes/test-infra#115, #26824