[cinder-csi-plugin]: helm-charts: use /etc/config/ for cloud.conf #2892
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[cinder-csi-plugin] helm charts: Use /etc/config/ for cloud.conf
This path is used by default in the deployment manifests for cinder-csi as well as in manifests and helm charts for occm.
It does not matter much until you reference another file via ca-file=/etc/config/cacert
in your cloud.conf and assume that to work.
So better align the volumeMount location everywhere. As there is a canonical location where most artifacts already agree on, let's use it in cinder-csi helm charts as well.
What this PR does / why we need it:
Make the cinder-csi-plugin helm charts work if we pass a secret with
cloud.confand acacertreferenced from there assuming the/etc/configpath.This works with occm (deployment manifests and helm charts) and cinder-csi deployment manifests, but not currently with the cinder-csi helm charts due to using
/etc/kubernetesthere.This PR aligns cinder-csi helm charts to also use the canonical path.
Reference:
SovereignCloudStack/cluster-stacks#188 (comment)
Release note:
(Feel free to not add this if determined not to be significant enough.)