[release-1.31] [cinder-csi-plugin] Don't report topology capability when --with-topology=False
This is a manual backport of #2862 (master) and #2871 (release-1.32). As noted in the first commit, the merge conflict was trivial and due to the absence of #2734, which added a log into the Manila CSI Driver.
NONE
These failures are due to the version of golangci-lint we use. I've proposed https://github.com/kubernetes/cloud-provider-openstack/pull/2874 to fix this. We'll need to merge that first.
These failures are due to the version of golangci-lint we use. I've proposed #2874 to fix this. We'll need to merge that first.
That's merged now, so I've rebased and force pushed this.
Jobs are failing because the stable/2023.2 branch was recently deleted upstream. Looks like that's also borked on master. This is going to be fun :see_no_evil:
Waiting on https://github.com/kubernetes/cloud-provider-openstack/pull/2742
/test openstack-cloud-csi-manila-e2e-test-release-131
/test openstack-cloud-csi-cinder-e2e-test-release-131
/test openstack-cloud-csi-manila-e2e-test-release-131 /test openstack-cloud-csi-cinder-e2e-test-release-131
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: kayrus
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [kayrus]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment