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 1820238: Fetch latest changes from upstream #38
Conversation
This commit updates the CLI flags used to launch the controller manager. The usage of `address` is deprecated. Hence, this commit updates the flags to `bind-address` kubernetes#1258.
…ubernetes#1267) * [cinder-csi-plugin]: set cinder CSI sidecar timeouts * Update manifests/cinder-csi-plugin/cinder-csi-controllerplugin.yaml Co-authored-by: Ismail Alidzhikov <i.alidjikov@gmail.com> * Add cinder CSI timeouts to helm-charts Co-authored-by: Ismail Alidzhikov <i.alidjikov@gmail.com>
* [cinder-csi-plugin] Improve docs [Part2] This splits the docs to diff files for better readability * Update csc-tool.md * Update using-cinder-csi-plugin.md
Also ,run format because it will do grammar check as well
Signed-off-by: Sean Schneeweiss <sean.schneeweiss@daimler.com>
* Fix whitespaces and invalit dot * fix indentation and add nindent
…netes#1291) * added appendShareMetadata volume parameter * set share metadata from shareOpts when creating a new share * updated docs * docs: amended field description for appendShareMetadata
…ting (kubernetes#1272) * Add error handling for list fucntions before creating * fix comments
…sword" options (kubernetes#1311) * [occm]: add support for "tls-insecure" option * Cover trustee-id and trustee-password in gcfg
* [cinder-csi-plugin] support ignore-volume-az * Update using-cinder-csi-plugin.md
@Fedosin: This pull request references Bugzilla bug 1820238, 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. |
don't we want to run e2e-openstack job? |
/test e2e-openstack |
@Fedosin: The specified target(s) for
Use 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. |
oh... looks like we don't have any :( |
This repo is used for cinder csi and manila csi, and in the future for the external cloud provider. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EmilienM, Fedosin 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 |
@Fedosin: All pull requests linked via external trackers have merged: Bugzilla bug 1820238 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. |
This patch fetches all latest changes from upstream.
90378cf is important to fix the issue with remaining manila shares after cluster deletion.