Fix volume list in cinder-volume deployment

Volumes (pod-shared and cinder-conversion) should be created
regardless of ceph backend existence but those are created
conditionally.
This bug has been existing but never revealed because conditional
check for ceph backend was wrong before. After this change
https://review.opendev.org/c/openstack/openstack-helm/+/852809,
started happening.

Change-Id: Ifc90e40bc201245d3d9e2b472573948ff64e0c61
This commit is contained in:
okozachenko 2022-08-16 20:31:54 +10:00
parent d1a7abeb0c
commit 830755a3c3
3 changed files with 5 additions and 4 deletions

View File

@ -14,7 +14,7 @@ apiVersion: v1
appVersion: v1.0.0 appVersion: v1.0.0
description: OpenStack-Helm Cinder description: OpenStack-Helm Cinder
name: cinder name: cinder
version: 0.2.24 version: 0.2.25
home: https://docs.openstack.org/cinder/latest/ home: https://docs.openstack.org/cinder/latest/
icon: https://www.openstack.org/themes/openstack/images/project-mascots/Cinder/OpenStack_Project_Cinder_vertical.png icon: https://www.openstack.org/themes/openstack/images/project-mascots/Cinder/OpenStack_Project_Cinder_vertical.png
sources: sources:

View File

@ -284,13 +284,13 @@ spec:
secret: secret:
secretName: cinder-etc secretName: cinder-etc
defaultMode: 0444 defaultMode: 0444
{{- if eq "true" (include "cinder.utils.has_ceph_backend" $envAll) }}
- name: etcceph
emptyDir: {}
- name: pod-shared - name: pod-shared
emptyDir: {} emptyDir: {}
- name: cinder-conversion - name: cinder-conversion
emptyDir: {} emptyDir: {}
{{- if eq "true" (include "cinder.utils.has_ceph_backend" $envAll) }}
- name: etcceph
emptyDir: {}
- name: ceph-etc - name: ceph-etc
configMap: configMap:
name: {{ .Values.ceph_client.configmap }} name: {{ .Values.ceph_client.configmap }}

View File

@ -41,4 +41,5 @@ cinder:
- 0.2.22 Add Xena and Yoga values overrides - 0.2.22 Add Xena and Yoga values overrides
- 0.2.23 Added OCI registry authentication - 0.2.23 Added OCI registry authentication
- 0.2.24 Fix conditional check for cinder.utils.has_ceph_backend template - 0.2.24 Fix conditional check for cinder.utils.has_ceph_backend template
- 0.2.25 Remove volumes unrelated with ceph backend from conditional volume list in cinder-volume deployment
... ...