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 1870339: Sets volumeMode as block for RBD+RWX #6559
Conversation
@a2batic: This pull request references Bugzilla bug 1870339, 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. |
/assign @rawagner |
frontend/packages/console-app/src/components/modals/restore-pvc/restore-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
The Ideal way to figure out the weather its block or filesystem will be to get the PVC data & see the access mode. Let's not hardcode it. |
@a2batic: This pull request references Bugzilla bug 1870339, which is valid. 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. |
frontend/packages/console-app/src/components/modals/clone/_clone-pvc-modal.scss
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/restore-pvc/restore-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/console-app/src/components/modals/restore-pvc/restore-pvc-modal.tsx
Outdated
Show resolved
Hide resolved
/test frontend |
1 similar comment
/test frontend |
/assign @spadgett |
/test frontend |
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/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
@a2batic: This pull request references Bugzilla bug 1870339, which is valid. 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. |
// should set block only for RBD + RWX | ||
if ( | ||
_.endsWith(snapshotClassResource?.driver, cephRBDProvisionerSuffix) && | ||
accessModes[0] === 'ReadWriteMany' |
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'm not as familiar with the PVC API, but we need to check if any of the array items are ReadWriteMany
? Or that there is only one item in the array?
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.
@spadgett yes a PVC can have only 1 access mode at a time https://kubernetes.io/docs/concepts/storage/persistent-volumes/#access-modes
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.
Hi, maybe I missed it, but I don't see where the link says there can only be one. (I'm also unsure why it would be an array in the API in that 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.
As per my understanding: There can be multiple-access modes for a PV but when the PV is mounted using PVC there will be only one access mode that will be used. So we can assume that only one value will exist at a time. (I.e only fits for Static PV creation)
It's the same reason we have radio buttons for access mode while PVC creation.
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.
Hi @spadgett @cloudbehl , the link says the Important! A volume can only be mounted using one access mode at a time, even if it supports many.
But, PVC CR lets you create PVC with multiple access modes, so +1, we should handle it. Updated the PR to check in the array. Sorry for the confusion.
Signed-off-by: Kanika Murarka <kmurarka@redhat.com>
/test e2e-gcp-console |
1 similar comment
/test e2e-gcp-console |
@spadgett addressed comments, please review. |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a2batic, cloudbehl, 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 |
@a2batic: All pull requests linked via external trackers have merged: Bugzilla bug 1870339 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: Kanika Murarka kmurarka@redhat.com