-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Updating FlexVolume doc with capabilities addition. #786
Conversation
/assign @saad-ali @chakri-nelluri |
@verult: GitHub didn't allow me to assign the following users: chakri-nelluri. Note that only kubernetes members can be assigned. 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. |
contributors/devel/flexvolume.md
Outdated
@@ -19,7 +19,8 @@ Call-outs are invoked from Controller-manager only when "--enable-controller-att | |||
### Driver invocation model: | |||
|
|||
#### Init: | |||
Initializes the driver. Called during Kubelet & Controller manager initialization. | |||
Initializes the driver. Called during Kubelet & Controller manager initialization. On success, a capabilities map is returned. | |||
Currently the only capability supported is `attach`, indicating whether the driver requires attach and detach operations. This field is *required*, although for backward-compatibility the default value is set to `true`, i.e. requires attach and detach. |
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.
Can user set it to false? and what does it mean?
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.
Updated.
2f40230
to
d4135c3
Compare
/lgtm |
Automatic merge from submit-queue |
The following PR introduces a FlexVolume API change, so the documentation needs to be updated.
kubernetes/kubernetes#47503