Merge pull request #248 from neoworx/master

Added possibility to define extra Init Containers
Oteemo · Feb 3, 2021 · d413b96 · d413b96
2 parents cbf94a8 + 3f17b60
commit d413b96
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 1 deletion.
diff --git a/charts/sonarqube/Chart.yaml b/charts/sonarqube/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 name: sonarqube
 description: SonarQube is an open sourced code quality scanning tool
-version: 9.3.1
+version: 9.4.0
 appVersion: 8.5.1-community
 keywords:
   - coverage

diff --git a/charts/sonarqube/README.md b/charts/sonarqube/README.md
@@ -97,6 +97,7 @@ The following table lists the configurable parameters of the Sonarqube chart and
 | `initContainers.image`                | Change init container image                                                                                                 | `busybox:1.32`                                   | 
 | `initContainers.securityContext`      | SecurityContext for init containers                                                                                         | `nil`                                            | 
 | `initContainers.resources`            | Resources for init containers                                                                                               | `{}`                                             | 
+| `extraInitContainers`                 | Extra init containers to e.g. download required artifacts                                                                   | `{}`                                             |
 | `caCerts.image`                       | Change init CA certificates container image                                                                                 | `adoptopenjdk/openjdk11:alpine`                  | 
 | `caCerts.secret`                      | Name of the secret containing additional CA certificates                                                                    | `nil`                                            |
 | `initSysctl.enabled`                  | Modify k8s worker to conform to system requirements                                                                         | `true`                                           | 

diff --git a/charts/sonarqube/templates/deployment.yaml b/charts/sonarqube/templates/deployment.yaml
@@ -59,6 +59,9 @@ spec:
         - name: {{ .Values.image.pullSecret }}
       {{- end }}
       initContainers:
+      {{- if .Values.extraInitContainers }}
+{{ toYaml .Values.extraInitContainers | indent 8 }}
+      {{- end }}
       {{- if .Values.caCerts }}
         - name: ca-certs
           image: {{ default "adoptopenjdk/openjdk11:alpine" .Values.caCerts.image }}

diff --git a/charts/sonarqube/values.yaml b/charts/sonarqube/values.yaml
@@ -123,6 +123,23 @@ initContainers:
   # the initContainer does not take as much resources.
   resources: {}
 
+extraInitContainers: {}
+# Extra init containers to e.g. download required artifacts
+#  - name: "prometheus-exporter-downloader"
+#    image: "busybox"
+#    imagePullPolicy: "IfNotPresent"
+#    securityContext:
+#      runAsNonRoot: true
+#    command:
+#      - wget
+#      - "-O"
+#      - "/downloads/jmx_prometheus_javaagent.jar"
+#      - ${JMX_EXPORTER_URL}
+#    volumeMounts:
+#      - name: "downloads"
+#        mountPath: "/downloads"
+
+
 ## Provide a secret containing one or more certificate files in the keys that will be added to cacerts
 ## The cacerts file will be set via SONARQUBE_WEB_JVM_OPTS and SONAR_CE_JAVAOPTS
 ##