add-service-port-nginx

[aprilrieger]

adds the service port to nginx, once it's enabled the hyrax chart expects those values to be set in the values file

Story

Deploy on main fails after adding the nginx service to the values file for the production deployment. Adding the service port to the values file under the nginx service will elleviate the error and have the deployment back on track in no time!

Expected Behavior Before Changes

Expected Behavior After Changes

Screenshots / Video

Notes

[orangewolf]

Do you also need serviceType?

[aprilrieger]

@orangewolf no just the port unless service.port is already defined in the values file, https://github.com/samvera/hyrax/blob/72619eaf57765419031d46ac6c83d099fa0c1b15/chart/hyrax/templates/ingress.yaml#L3C1-L3C1

{{- if .Values.ingress.enabled -}}
{{- $svcFullName := ternary (include "hyrax.nginx.host" .) (include "hyrax.fullname" .) .Values.nginx.enabled -}}
{{- $svcPort := ternary .Values.nginx.service.port .Values.service.port .Values.nginx.enabled -}}