Ah make sense to me! There is also another service case, related to service configurations.
With service type LoadBalancer:
helm install cryostat-with-svc ./charts/cryostat/ \
--set core.service.type=LoadBalancer \
--set grafana.service.type=LoadBalancer
Since its relying cloud providers, likely there are additional annotations required on the Service. For example, with AWS, I have been manually patching the Services after installing the chart:
service.beta.kubernetes.io/aws-load-balancer-nlb-target-type: ip
service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
service.beta.kubernetes.io/aws-load-balancer-type: external
Do you think it would be good to allow setting those on chart values? Currently, we can only set Service's port and type.
Maybe, not this PR scope though.
Originally posted by @tthvo in #122 (comment)