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
Bug 1877860: Volume Snapshot basic CRUD tests #6448
Bug 1877860: Volume Snapshot basic CRUD tests #6448
Conversation
/assign @dtaylor113 |
Hi @cloudbehl are this tests in addition too, and will they impact/conflict with: #6340 ? |
/hold |
7caccc3
to
53f864a
Compare
@dtaylor113 Yes these are additional test which helps generic flows like YAML based creation, edit resource & deletion of the resource. @bipuladh canceling hold as discussed. The bug is related to tests where YAML creation button was not present & the tests were not getting passed. /hold cancel |
53f864a
to
d2e69d7
Compare
@cloudbehl: This pull request references Bugzilla bug 1877860, which is invalid:
Comment 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. |
/bugzilla refresh |
@cloudbehl: This pull request references Bugzilla bug 1877860, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 3 validation(s) were run on this bug
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. |
Hi @cloudbehl, irt. the changes to Cypress integration test changes in |
apiVersion: snapshot.storage.k8s.io/v1beta1 | ||
kind: VolumeSnapshotContent | ||
metadata: | ||
name: new-snapshot-content-test |
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.
Should we have "test" in the name if this is going to be the real volume snapshot template? (Here are elsewhere in the samples.)
We try to make the examples just work if you click save with no changes, but I imagine that's not possible here.
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.
Should we have "test" in the name if this is going to be the real volume snapshot template? (Here are elsewhere in the samples.)
Right, we can call it an example/test. I'll change it.
We try to make the examples just work if you click save with no changes, but I imagine that's not possible here.
I agree, Right now volume snapshots are not adapted to all the storage providers as it's not GA'ed. When snapshot gets GA'ed we should get the default CSI hostpath driver out of the box. So for now just added an example of it as this is provided on k8s documentation as well.
...console-app/src/components/volume-snapshot/create-volume-snapshot/create-volume-snapshot.tsx
Outdated
Show resolved
Hide resolved
75574b9
to
dadab2e
Compare
@spadgett updated the PR please have a look. |
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
/retest Please review the full test history for this PR and help us cut down flakes. |
11 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/hold needs rebase |
Signed-off-by: Ankush Behl <cloudbehl@gmail.com>
dadab2e
to
d285699
Compare
/hold cancel FYI @spadgett |
/test e2e-gcp-console |
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 -thanks
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
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cloudbehl, dtaylor113, spadgett 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 |
@cloudbehl: The following test failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
/test e2e-gcp-console |
/retest Please review the full test history for this PR and help us cut down flakes. |
@cloudbehl: All pull requests linked via external trackers have merged: Bugzilla bug 1877860 has been moved to the MODIFIED state. 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: Ankush Behl cloudbehl@gmail.com