Skip to content

Commit

Permalink
chart: set default value for nginx.service (#6518)
Browse files Browse the repository at this point in the history
* chart: set default value for nginx.service

* chart: don't let nginx create LoadBalancers by default

Co-authored-by: Rob Kaufman <rob@notch8.com>

---------

Co-authored-by: Rob Kaufman <rob@notch8.com>
  • Loading branch information
dunn and orangewolf authored Dec 8, 2023
1 parent 64f1a6f commit 5a1ba05
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions chart/hyrax/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -284,6 +284,14 @@ nginx:
image:
repository: bitnami/nginx
tag: 1.23.0-debian-11-r1
# this needs to be set since templates/ingress.yaml tries to
# evaluate .Values.nginx.service.port, and it needs to at least evaluate to
# nil, rather than throw an error (which it does if `service` itself is nil).
# you can change to NodePort or LoadBalancer if direct access is needed.
# ClusterIP is used as we assume an ingress is in place.
service:
type: ClusterIP
port: 80
# The set up below does malicious bot / ip blocking and mounts
# volumes to allow nginx to server assets and other public directory items
# extraVolumes:
Expand Down

0 comments on commit 5a1ba05

Please sign in to comment.