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
Add e2e test for kubectl interactive delete #122653
Add e2e test for kubectl interactive delete #122653
Conversation
/triage accepted |
5bf21eb
to
2af961e
Compare
2af961e
to
7faa8bb
Compare
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.
/lgtm
/approve
e2ekubectl.RunKubectlOrDieInput(ns, deploymentYaml, "apply", "-f", "-") | ||
|
||
ginkgo.By("verifying the deployment is created and running") | ||
err := wait.PollUntilContextTimeout(ctx, 2*time.Second, 30*time.Second, true, func(ctx context.Context) (done bool, err error) { |
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.
Nit, you could use WaitForDeploymentComplete(c, deploy)
from k8s.io/kubernetes/test/e2e/framework/deployment
, instead.
LGTM label has been added. Git tree hash: 5c8f6cacb8e08bd76734fe018ba861ae47eced9b
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ardaguclu, soltysh The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds e2e test for interactive flag in kubectl delete command as promised
in the KEP to promote it to GA.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: