Skip to content

Commit

Permalink
Fix ingressClassName nesting in Refinery chart
Browse files Browse the repository at this point in the history
Fixes honeycombio#422

Nest `ingressClassName` under `ingress` in the Refinery chart.

* Move `ingressClassName` under the `ingress` key in `charts/refinery/templates/ingress.yaml`
* Update the indentation to match the new structure
* Modify the condition to check for `ingress.ingressClassName` instead of `ingressClassName`

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/honeycombio/helm-charts/issues/422?shareId=XXXX-XXXX-XXXX-XXXX).
  • Loading branch information
bixu committed Jan 31, 2025
1 parent 835f65c commit c50927d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions charts/refinery/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ spec:
secretName: {{ .secretName }}
{{- end }}
{{- end }}
{{- if .Values.ingressClassName }}
ingressClassName: {{ .Values.ingressClassName }}
{{- if .Values.ingress.ingressClassName }}
ingressClassName: {{ .Values.ingress.ingressClassName }}
{{- end }}
rules:
{{- range .Values.ingress.hosts }}
Expand Down

0 comments on commit c50927d

Please sign in to comment.