-
Notifications
You must be signed in to change notification settings - Fork 620
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
Enable Snapshot e2e tests #1175
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Build failed.
|
/test cloud-provider-openstack-e2e-test-csi-cinder |
@jichenjc: The specified target(s) for
Use 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. |
Build failed.
|
Hi @ramineni, is the test failure related to the new test suit? |
@lingxiankong I believe it's a technical debt we left .. there is no snapshot e2e before so this is something we need add |
@lingxiankong This PR is dependent on #974 , e2e runs on v1beta1 version , ill recheck the job once that PR is merged |
/test cloud-provider-openstack-e2e-test-csi-cinder |
@ramineni: The specified target(s) for
Use 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. |
Build failed.
|
seems this is the error ,not sure it's related to some API resource not enabled? |
574a1c5
to
f46712d
Compare
Build succeeded.
|
Build succeeded.
|
Build failed.
|
Build failed.
|
Build succeeded.
|
f46712d
to
4c8baa4
Compare
Build succeeded.
|
Build succeeded.
|
Build failed.
|
latest fail seems still related to create the the pod is created at
|
/retest |
Build succeeded.
|
Build succeeded.
|
Build failed.
|
theopenlab/openlab#603 is the thing that we are facing |
/hold see theopenlab/openlab#603 for further info |
@jichenjc Do we have any update here? |
@ramineni I retried then found CSI test is 100% failure , I I think we need get theopenlab/openlab#603 fixed |
@jichenjc Thanks. Any update from openlab on that issue, I guess they have to manually cleanup for now I suppose right? Otherwise may be we need to check with them on IRC |
I asked them and they told us it's citynetwork env and they need open issue to them as well |
/test cloud-provider-openstack-e2e-test-csi-cinder |
@ramineni: The specified target(s) for
Use 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. |
Build failed.
|
@jichenjc @lingxiankong Any idea the issue related to snapshot still exists from openlab side. I see issue is closed from openlab , so just confirming. |
theopenlab/openlab#667 is not closed yet... |
No, I meant we had some issue with snapshot , as deletion not supported or something. Can't remember exact issue. I was talking about that one? |
maybe theopenlab/openlab#603? |
Yes, theopenlab/openlab#603 was the issue we were handling before. @jichenjc the issue was closed but I didn't see any conclusion, anything we need to follow up? |
@lingxiankong reopened that issue and added some comments :) |
/test cloud-provider-openstack-e2e-test-csi-cinder |
@ramineni: The specified target(s) for
Use 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. |
Build failed.
|
This commit enables snapshot e2e tests. Also adds
snapshot-controller yamls to repo, they are required for tests
and also to fix the version supported by plugin which is 2.1.1
as of now.
What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:
Depends on theopenlab/openlab-zuul-jobs#1036
Release note: