[stable/sonarqube] Ingress path configuration for SonarQube (#13320)

* Modify Ingress template to be easier to --set

Signed-off-by: Scott Collier <collierthescott@gmail.com>

* Update README

Signed-off-by: Scott Collier <collierthescott@gmail.com>

* Updated TLS to match new Ingress format

Signed-off-by: Scott Collier <collierthescott@gmail.com>

* Fix trailing spaces

Signed-off-by: Scott Collier <collierthescott@gmail.com>

* Additional readme info

Signed-off-by: Scott Collier <collierthescott@gmail.com>

* Modified ingress template to allow path configuration.  Bumped major version to reflect incompatible API change.

Signed-off-by: Scott Collier <collierthescott@gmail.com>

stable/sonarqube/Chart.yaml

@@ -1,6 +1,6 @@
 name: sonarqube
 description: Sonarqube is an open sourced code quality scanning tool
-version: 0.16.1
+version: 1.0.0
 appVersion: 7.7
 keywords:
   - coverage

stable/sonarqube/README.md

@@ -47,6 +47,8 @@ The following table lists the configurable parameters of the Sonarqube chart and
 | `securityContext.fsGroup`                   | Group applied to mounted directories/files|  `999`                                     |
 | `ingress.enabled`                           | Flag for enabling ingress                 | false                                      |
 | `ingress.labels`                            | Ingress additional labels                 | `{}`                                       |
+| `ingress.hosts[0].name`                     | Hostname to your SonarQube installation   | `sonar.organization.com`                   |
+| `ingress.hosts[0].path`                     | Path within the URL structure             | /                                          |
 | `livenessProbe.sonarWebContext`             | SonarQube web context for livenessProbe   | /                                          |
 | `readinessProbe.sonarWebContext`            | SonarQube web context for readinessProbe  | /                                          |
 | `service.type`                              | Kubernetes service type                   | `LoadBalancer`                             |

stable/sonarqube/templates/ingress.yaml

@@ -21,15 +21,15 @@ metadata:
 {{- end }}
 spec:
   rules:
-    {{- range $host := .Values.ingress.hosts }}
-    - host: {{ $host }}
+    {{- range .Values.ingress.hosts }}
+    - host: {{ .name }}
       http:
         paths:
-          - path: /
+          - path: {{ default "/" .path }}
             backend:
               serviceName: {{ $serviceName }}
               servicePort: {{ $servicePort }}
-          - path: /*
+          - path: {{ printf "%s/*" .path }}
             backend:
               serviceName: {{ $serviceName }}
               servicePort: {{ $servicePort }}

stable/sonarqube/values.yaml

@@ -28,7 +28,11 @@ ingress:
   enabled: false
   # Used to create an Ingress record.
   hosts:
-    - sonar.organization.com
+    - name: sonar.organization.com
+      # default paths for "/" and "/*" will be added
+      path: /
+      # If a different path is defined, that path and {path}/* will be added to the ingress resource
+      # path: /sonarqube
   annotations: {}
   # kubernetes.io/ingress.class: nginx
   # kubernetes.io/tls-acme: "true"
@@ -68,14 +72,22 @@ readinessProbe:
   initialDelaySeconds: 60
   periodSeconds: 30
   failureThreshold: 6
+  # If an ingress *path* other than the root (/) is defined, it should be reflected here
+  # A trailing "/" must be included
   sonarWebContext: /
+  # sonarWebContext: /sonarqube/
 livenessProbe:
   initialDelaySeconds: 60
   periodSeconds: 30
+  # If an ingress *path* other than the root (/) is defined, it should be reflected here
+  # A trailing "/" must be included
   sonarWebContext: /
+  # sonarWebContext: /sonarqube/
 
 # Set extra env variables. Like proxy settings.
 extraEnv: {}
+  # If an ingress *path* is defined, it should be reflected here
+  # sonar.web.context: /sonarqube
 
 resources: {}
 # We usually recommend not to specify default resources and to leave this as a conscious