Closed embik closed 1 year ago
/cc @mjudeikis
/lgtm /approve
LGTM label has been added.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mjudeikis
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Because the kcp-front-proxy serves HTTPS by default, installing the chart without customizing the Ingress annotations will result in an error looking like
While this setting depends on the ingress controller in use, we already set
kubernetes.io/ingress.class=nginx
, so the general expectation of the default Helm values is that nginx-ingress-controller is deployed. Adding this annotation to the default set makes that implicit reference setup fully work.In addition, this PR adds a comment so that users of other ingress controllers can look up what their implementation requires for similar functionality.