Skip to content

Commit

Permalink
Merge pull request #1042 from DSD-DBS/ingress
Browse files Browse the repository at this point in the history
feat: Make ingressClassName configurable
  • Loading branch information
MoritzWeber0 authored Sep 26, 2023
2 parents d66c41d + 10271b3 commit 035542d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ helm-deploy:
--set target=local \
--set general.port=8080 \
--set development=$(DEVELOPMENT_MODE) \
--set cluster.ingressClassName=traefik \
--set backend.k8sSessionNamespace="$(SESSION_NAMESPACE)" \
--set backend.authentication.oauth.redirectURI="http://localhost:$(PORT)/oauth2/callback" \
--set backend.authentication.oauth.endpoints.wellKnown="http://$(RELEASE)-oauth-mock:8080/default/.well-known/openid-configuration" \
Expand Down
4 changes: 1 addition & 3 deletions helm/templates/routing/nginx.ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,7 @@ metadata:
nginx.ingress.kubernetes.io/proxy-buffering: "off"
nginx.ingress.kubernetes.io/proxy-request-buffering: "off"
spec:
{{ if ne .Values.target "local" }}
ingressClassName: nginx
{{ end }}
ingressClassName: {{ .Values.cluster.ingressClassName }}
rules:
- host: {{ .Values.general.host }}
http:
Expand Down
2 changes: 2 additions & 0 deletions helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -199,6 +199,8 @@ cluster:
# It is used for the backend serviceAccount
imagePullSecret: test

ingressClassName: nginx

namespaces:
sessions:
podSecurityContext: *podSecurityContext
Expand Down

0 comments on commit 035542d

Please sign in to comment.