openstack/openstack-helm
Code Issues Proposed changes

fix(registry): corrects glance registry ingress

Browse Source 
This corrects an issue with glance registry ingress failing due to a
name mismatch. This also provides a proper glance registry public secret that was
missing.

Change-Id: Ibe3d5ca774365b7f4df01940884953fc0181394f
Signed-off-by: Tin Lam <tin@irrational.io>
This commit is contained in:
Tin Lam 2020-07-29 23:50:06 -05:00
parent 639f15a069
commit 9d1c46c67d
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
glance/templates/ingress-registry.yaml
View File

@ -14,7 +14,7 @@ limitations under the License.
{{- if and .Values.manifests.ingress_registry .Values.network.registry.ingress.public }} {{- if and .Values.manifests.ingress_registry .Values.network.registry.ingress.public }}
{{- $envAll := . }} {{- $envAll := . }}
{{- $ingressOpts := dict "envAll" $envAll "backendService" "registry" "backendServiceType" "image_registry" "backendPort" "g-reg" -}} {{- $ingressOpts := dict "envAll" $envAll "backendServiceType" "image_registry" "backendPort" "g-reg" -}}
{{- $secretName := $envAll.Values.secrets.tls.image_registry.api.internal -}} {{- $secretName := $envAll.Values.secrets.tls.image_registry.api.internal -}}
{{- if and .Values.manifests.certificates $secretName -}} {{- if and .Values.manifests.certificates $secretName -}}
{{- $_ := set $ingressOpts "certIssuer" .Values.endpoints.image_registry.host_fqdn_override.default.tls.issuerRef.name -}} {{- $_ := set $ingressOpts "certIssuer" .Values.endpoints.image_registry.host_fqdn_override.default.tls.issuerRef.name -}}

1
glance/values.yaml
View File

@ -574,6 +574,7 @@ secrets:
internal: glance-tls-api internal: glance-tls-api
image_registry: image_registry:
api: api:
public: glance-tls-reg-public
internal: glance-tls-reg internal: glance-tls-reg
# typically overridden by environmental # typically overridden by environmental