Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Defining default path spec in ingress if hosts.*.paths is null" #247

Merged
5 changes: 4 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,14 @@

All notable changes to this project will be documented here.

### v2.1.14
- Fix: Reverts [PR-243](https://github.com/stakater/application/pull/243), This can already be configured via `paths` [PR-247](https://github.com/stakater/application/pull/247)
aslafy-z marked this conversation as resolved.
Show resolved Hide resolved

### v2.1.13
- Fix: make deployment.image.tag and job.image.tag optional [PR-234](https://github.com/stakater/application/pull/234)

### v2.1.12
- Caused by Change Log update
- Update: Changelog updates. This release shouldnt have been made as the head commit had [`[skip ci]`](https://docs.github.com/en/actions/managing-workflow-runs/skipping-workflow-runs). [PR-248](https://github.com/stakater/application/pull/248)

### v2.1.11
- Feature: Add topologySpreadConstraints [PR-239](https://github.com/stakater/application/pull/239)
Expand Down
8 changes: 1 addition & 7 deletions application/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,7 @@ spec:
name: {{ default "http" (.servicePort) }}
{{- end }}
{{- else }}
- path: "/"
pathType: "ImplementationSpecific"
backend:
service:
name: {{ $applicationNameTpl }}
port:
name: {{ default "http" (.servicePort) }}
{{ fail "Specify paths for ingress host, check values.yaml" }}
{{- end }}
{{- end -}}
{{- if .Values.ingress.tls }}
Expand Down
3 changes: 1 addition & 2 deletions application/values-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -280,8 +280,7 @@ ingress:
# List of host addresses to be exposed by this Ingress

hosts:
- host: chart-example-1.local
- host: chart-example-2.local
- host: chart-example.local
paths:
- path: /
# pathType: ''
Expand Down
4 changes: 2 additions & 2 deletions application/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -352,8 +352,8 @@ ingress:
# List of host addresses to be exposed by this Ingress
hosts:
- host: chart-example.local
# paths:
# - path: /
paths:
- path: /
# pathType: ''
# serviceName: ''
# servicePort: ''
Expand Down
Loading