Skip to content

Commit

Permalink
Merge pull request #6 from jfv-opensource/mr-setup-certificate
Browse files Browse the repository at this point in the history
MR Allow not to use dedicated certificate
  • Loading branch information
gillesdubois authored Jan 15, 2024
2 parents 952d529 + 0aff4c0 commit e34d466
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,7 @@ Key | Description | Default
`mariadb.auth.username` | Database user to create | `""`
`mariadb.auth.password` | Password for the user | `""`
`ingress.enabled` | Enable use of ingress controllers | `true`
`ingress.tls` | Use dedicated certificates | `true`
`ingress.hosts` | OCS hosts to create application URLs | `""`
`ingress.annotations` | An array of ingress annotations | `{}`
`ingress.basicauth.enabled` | Wether to enable basic auth | `"false"`
Expand Down
4 changes: 3 additions & 1 deletion charts/ocsinventory/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,9 @@ spec:
- hosts:
{{- range .Values.ingress.hosts }}
- {{ . }}
secretName: {{ . }}-tls
{{- if $.Values.ingress.tls }}
secretName: {{ . }}-tls
{{- end}}
{{- end}}
rules:
{{- range .Values.ingress.hosts }}
Expand Down
1 change: 1 addition & 0 deletions charts/ocsinventory/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ ingress:
labels: {}
hosts:
- ""
tls: true
annotations:
kubernetes.io/ingress.class: nginx
nginx.ingress.kubernetes.io/force-ssl-redirect: "true"
Expand Down

0 comments on commit e34d466

Please sign in to comment.