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 nodeoutofservicevolumedetach e2e test #26947
add nodeoutofservicevolumedetach e2e test #26947
Conversation
Welcome @sonasingh46! |
Hi @sonasingh46. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
testgrid-dashboards: google-gce, google-gci, provider-gcp-periodics | ||
testgrid-tab-name: nongraceful-node-shutdown | ||
testgrid-num-failures-to-alert: '6' | ||
testgrid-alert-email: release-team@kubernetes.io |
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.
release-team@kubernetes.io
wouldn't need to be alerted for failures of this. Maybe some cloud provider team?
52bdad1
to
72c610f
Compare
/assign @andrewsykim |
/assign @cheftako |
/assign @msau42 |
/assign @dims |
@@ -445,6 +445,51 @@ presubmits: | |||
privileged: true | |||
|
|||
periodics: | |||
- interval: 168h # one week |
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.
Can we add this to the existing alpha features job?
- name: pull-kubernetes-e2e-gce-alpha-features |
You may also need to add the "ginkgo-parallel=1" flag since the test is disruptive.
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.
@msau42 This job "pull-kubernetes-e2e-gce-alpha-features" skips all sig-storage tests.
https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/111373/pull-kubernetes-e2e-gce-alpha-features/1572235414100512768/
The non-graceful node shutdown test is added under e2e/storage so it is also skipped:
Kubernetes e2e suite: [It] [sig-storage] [Feature:NodeOutOfServiceVolumeDetach] [Disruptive] [LinuxOnly] NonGracefulNodeShutdown [NonGracefulNodeShutdown] pod that uses a persistent volume via gce pd driver should get immediately rescheduled to a different node after non graceful node shutdown
Reason: skipped
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.
I don't see anything in the skip setting that would cause that test to be skipped:
---test_args=--ginkgo.focus=\[Feature:(GRPCContainerProbe|InPlacePodVerticalScaling|ProbeTerminationGracePeriod|APIServerTracing|StorageVersionAPI|PodPreset|StatefulSetMinReadySeconds|CustomResourceValidationExpressions|ProxyTerminatingEndpoints)\]|Networking --ginkgo.skip=\[Feature:(SCTPConnectivity|Volumes|Networking-Performance)\]|IPv6|csi-hostpath-v0
With the alpha-features test suite, you need an explicit focus to opt-in the test case.
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.
Ok, we'll remove this job and just add the new test to the "pull-kubernetes-e2e-gce-alpha-features" job.
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.
@xing-yang @msau42 -- Opened a new PR #27600
Will close this once that gets merged. PTAL
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim, sonasingh46 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 |
72c610f
to
c4cf121
Compare
c4cf121
to
b0dfee2
Compare
Signed-off-by: Ashutosh Kumar <sonasingh46@gmail.com>
b0dfee2
to
4a054dc
Compare
/lgtm |
@sonasingh46: Updated the
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Signed-off-by: Ashutosh Kumar sonasingh46@gmail.com
Add e2e test for NodeOutOfServiceVolumeDetach feature.
Ref: kubernetes/kubernetes#108486