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
update rbd volume #46304
update rbd volume #46304
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
/sig storage |
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.
Thanks. This needs a couple of tweaks.
@@ -530,7 +530,7 @@ please install corresponding CSI drivers. | |||
* [`portworxVolume`](/docs/concepts/storage/volumes/#portworxvolume) - Portworx volume | |||
(**deprecated** starting v1.25) | |||
* [`rbd`](/docs/concepts/storage/volumes/#rbd) - Rados Block Device (RBD) volume | |||
(**deprecated** starting v1.28, no migration plan, support will be removed in a future release) | |||
(**deprecated** starting v1.28, no migration plan, support was removed in v1.31) |
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.
This needs to move to come after the text:
Older versions of Kubernetes also supported the following in-tree PersistentVolume types:
contents of an `rbd` volume are preserved and the volume is unmounted. This | ||
means that a RBD volume can be pre-populated with data, and that data can be | ||
shared between pods. | ||
### rbd (removed) |
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.
### rbd (removed) | |
### rbd (removed) {#rbd} |
@sftim thanks for your review. updated. |
/lgtm |
LGTM label has been added. Git tree hash: 0b3dea24d5f7509f18bdb2017d75f33f8ad13341
|
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.
Just one more thing, I think.
* `storageos` - StorageOS volume. | ||
(**not available** starting v1.25) | ||
* [`rbd`](/docs/concepts/storage/volumes/#rbd) - Rados Block Device (RBD) volume | ||
(**not available** starting v1.31) |
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.
Alphabetical order please
Actually, this should be in alphabetical order, but already isn't. Can fix that in another PR.
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.
@sftim thanks. I will update it after kubernetes/kubernetes#124544 is merged.
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.
Thanks
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sftim 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 |
/lgtm . Thanks! |
kubernetes/kubernetes#124546 removed CephRBD volume plugin and its csi migration support
preview: https://deploy-preview-46304--kubernetes-io-main-staging.netlify.app/docs/concepts/storage/volumes/#rbd-removed