Fix wrong configFile path in glance bootstrap container.

The configFile path shouble be /etc/glance/glance-api.conf,
not default /etc/glance/glance.conf defined by helm-toolkit,
since secrets mounted in '/etc/glance' have glance-api.conf not glance.conf in it.

The wrong path '/etc/glance/glance.conf' would be a dir in bootstarp container,
and lead to all config files in /etc/glance dir unreachable.

This bug may not affect bootstrap,
but should be fixed in case the config files are needed.

Change-Id: If25966e07ca7f9a80dd0e76ff7663a945db66a23
This commit is contained in:
zhangyeda 2023-04-06 13:37:26 +08:00
parent 84a6df2e5d
commit 9f8d417b5d
3 changed files with 8 additions and 1 deletions

View File

@ -14,7 +14,7 @@ apiVersion: v1
appVersion: v1.0.0 appVersion: v1.0.0
description: OpenStack-Helm Glance description: OpenStack-Helm Glance
name: glance name: glance
version: 0.4.4 version: 0.4.5
home: https://docs.openstack.org/glance/latest/ home: https://docs.openstack.org/glance/latest/
icon: https://www.openstack.org/themes/openstack/images/project-mascots/Glance/OpenStack_Project_Glance_vertical.png icon: https://www.openstack.org/themes/openstack/images/project-mascots/Glance/OpenStack_Project_Glance_vertical.png
sources: sources:

View File

@ -39,5 +39,11 @@ volumes:
{{- if .Values.pod.tolerations.glance.enabled -}} {{- if .Values.pod.tolerations.glance.enabled -}}
{{- $_ := set $bootstrapJob "tolerationsEnabled" true -}} {{- $_ := set $bootstrapJob "tolerationsEnabled" true -}}
{{- end -}} {{- end -}}
# The configFile path shouble be /etc/glance/glance-api.conf
# not default /etc/glance/glance.conf defined by helm-toolkit,
# since secrets mounted in '/etc/glance' have glance-api.conf not glance.conf in it.
# The wrong path '/etc/glance/glance.conf' would be dir in bootstarp container,
# and lead to all config files in '/etc/glance' dir unreachable.
{{- $_ := set $bootstrapJob "configFile" "/etc/glance/glance-api.conf" -}}
{{ $bootstrapJob | include "helm-toolkit.manifests.job_bootstrap" }} {{ $bootstrapJob | include "helm-toolkit.manifests.job_bootstrap" }}
{{- end }} {{- end }}

View File

@ -38,4 +38,5 @@ glance:
- 0.4.2 Allow Ceph pools to use 1x replication - 0.4.2 Allow Ceph pools to use 1x replication
- 0.4.3 Update all Ceph images to Focal - 0.4.3 Update all Ceph images to Focal
- 0.4.4 Replace node-role.kubernetes.io/master with control-plane - 0.4.4 Replace node-role.kubernetes.io/master with control-plane
- 0.4.5 Fix wrong configFile path in glance bootstrap container.
... ...