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
RHSTOR-4130: Cross storage class clone /restore #13549
RHSTOR-4130: Cross storage class clone /restore #13549
Conversation
Hi @TimothyAsirJeyasing. Thanks for your PR. I'm waiting for a openshift 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. |
21e954f
to
3553273
Compare
@SanjalKatiyar Please review |
/ok-to-test |
please check how |
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
Ya, just now I noticed, restore-pvc-modal.tsx has a better implementation, cc: @TimothyAsirJeyasing please check |
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
3553273
to
1024522
Compare
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
this is still missing, create button is not disabled (for cases where it should be)... |
if "clone" modal is filtering based on same provisioner + encrypted SC, then same filtering should be added for "restore" modal too... |
For restore modal, there is no option available about encryption for snapshot class resource of VolumeSnapshotClass kind. We need the operator to provide this. |
not really, don't you know which StorageClass was used for creating the snapshot ?? Once you know that, you can simply check if that StorageClass and new selected ones are both encrypted or not... |
1024522
to
16a388a
Compare
frontend/packages/console-app/src/components/modals/restore-pvc/restore-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/clone/clone-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
/retest |
/test e2e-gcp-console |
@jhadvig 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
/approve
Thanks @TimothyAsirJeyasing for this PR :)
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cyril-ui-developer, jhadvig, TimothyAsirJeyasing 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 |
QE Approver: |
Thank you for providing the screen snapshots, they are very useful for reviews. /label px-approved |
@yapei @opayne1 can you guys please take a look and add labels if that's alright ? |
/label qe-approved |
@TimothyAsirJeyasing: This pull request references RHSTOR-4130 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the epic to target the "4.16.0" version, but no target version was set. 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 openshift-eng/jira-lifecycle-plugin repository. |
/label docs-approved |
/test e2e-gcp-console |
1 similar comment
/test e2e-gcp-console |
/test e2e-gcp-console |
fbca5b1
into
openshift:master
This patch provides the following feature:
Users can now choose a storage class from the same provider when cloning or restoring.
This flexibility allows seamless transitions between storage classes with different replica counts (e.g., moving from SC with 3 replicas to 2/1 replicas).
Managed services can now clone Persistent Volumes (PVs) from slow to fast storage classes.
This enhancement ensures efficient data movement and optimisation.
Epic link: https://issues.redhat.com/browse/RHSTOR-4130