SONAR-22219 add extra volume and extra volume mounts

SonarSource · Sep 26, 2024 · a5f9697 · a5f9697
1 parent 76f19a5
commit a5f9697
Show file tree

Hide file tree

Showing 6 changed files with 26 additions and 2 deletions.
diff --git a/charts/sonarqube-dce/README.md b/charts/sonarqube-dce/README.md
@@ -387,6 +387,8 @@ The following table lists the configurable parameters of the SonarQube chart and
 | `ApplicationNodes.jwtSecret`                                     | A HS256 key encoded with base64 (_This value must be set before installing the chart, see [the documentation](https://docs.sonarsource.com/sonarqube/latest/setup-and-upgrade/deploy-on-kubernetes/cluster/)_) | `""`                                                                   |
 | `ApplicationNodes.existingJwtSecret`                             | secret that contains the `jwtSecret`                                                                                                                                                                           | `nil`                                                                  |
 | `ApplicationNodes.extraContainers`                               | Array of extra containers to run alongside                                                                                                                                                                     | `[]`                                                                   |
+| `ApplicationNodes.extraVolumes`                                  | Array of extra volumes to add to the SonarQube deployment                                                                                                                                                      | `[]`                                                                   |
+| `ApplicationNodes.extraVolumeMounts`                             | Array of extra volume mounts to add to the SonarQube deployment                                                                                                                                                | `[]`                                                                   |
 | `ApplicationNodes.hpa.enabled`                                   | Enable the HorizontalPodAutoscaler (HPA) for the app deployment                                                                                                                                                | `false`                                                                |
 | `ApplicationNodes.hpa.minReplicas`                               | Minimum number of replicas for the HPA                                                                                                                                                                         | `2`                                                                    |
 | `ApplicationNodes.hpa.maxReplicas`                               | Maximum number of replicas for the HPA                                                                                                                                                                         | `10`                                                                   |

diff --git a/charts/sonarqube/CHANGELOG.md b/charts/sonarqube/CHANGELOG.md
@@ -26,6 +26,7 @@ All changes to this chart will be documented in this file.
 * Remove socketTimeout from jdbcOverwrite.jdbcUrl's default value
 * Refactor Route to be subparameter of OpenShift
 * Make OpenShift.createSCC false by default
+* Deprecate peristence.volumes and persistence.mounts in favor or extraVolumes and extraVolumeMounts
 
 ## [10.6.0]
 * Update SonarQube to 10.6.0

diff --git a/charts/sonarqube/Chart.yaml b/charts/sonarqube/Chart.yaml
@@ -80,6 +80,8 @@ annotations:
       description: "Refactor Route to be subparameter of OpenShift"
     - kind: changed
       description: "Make OpenShift.createSCC false by default"
+    - kind: changed
+      description: "Deprecate peristence.volumes and persistence.mounts in favor or extraVolumes and extraVolumeMounts"
   artifacthub.io/containsSecurityUpdates: "false"
   artifacthub.io/images: |
     - name: sonarqube

diff --git a/charts/sonarqube/README.md b/charts/sonarqube/README.md
@@ -453,8 +453,8 @@ The following table lists the configurable parameters of the SonarQube chart and
 | `persistence.storageClass`  | Storage class to be used                                                     | `""`            |
 | `persistence.accessMode`    | Volumes access mode to be set                                                | `ReadWriteOnce` |
 | `persistence.size`          | Size of the volume                                                           | `5Gi`           |
-| `persistence.volumes`       | Specify extra volumes. Refer to ".spec.volumes" specification                | `[]`            |
-| `persistence.mounts`        | Specify extra mounts. Refer to ".spec.containers.volumeMounts" specification | `[]`            |
+| `persistence.volumes`       | (DEPRECATED) Please use extraVolumes instead                                 | `[]`            |
+| `persistence.mounts`        | (DEPRECATED) Please use extraVolumeMounts instead                            | `[]`            |
 | `persistence.uid`           | UID used for init-fs container                                               | `1000`          |
 | `persistence.guid`          | GUID used for init-fs container                                              | `0`             |
 | `emptyDir`                  | Configuration of resources for `emptyDir`                                    | `{}`            |

diff --git a/charts/sonarqube/values.schema.json b/charts/sonarqube/values.schema.json
@@ -4,6 +4,21 @@
         "replicaCount"
     ],
     "properties": {
+        "persistence": {
+            "type": "object",
+            "properties": {
+                "volumes": {
+                    "type": "array",
+                    "deprecated": true,
+                    "$comment": "(DEPRECATED) Please use `extraVolumes` instead"
+                },
+                "mounts": {
+                    "type": "array",
+                    "deprecated": true,
+                    "$comment": "(DEPRECATED) Please use `extraVolumeMounts` instead"
+                }
+            }
+        },
         "networkPolicy": {
             "type": "object",
             "properties": {

diff --git a/charts/sonarqube/values.yaml b/charts/sonarqube/values.yaml
@@ -289,6 +289,8 @@ extraInitContainers: {}
 ##   command: ['sh', '-c', 'echo Hello && sleep 3600']
 ##
 extraContainers: []
+extraVolumes: []
+extraVolumeMounts: []
 
 ## 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
@@ -468,8 +470,10 @@ persistence:
   uid: 1000
   guid: 0
 
+  ## DEPRECATED please use root level extraVolumes value
   ## Specify extra volumes. Refer to ".spec.volumes" specification : https://kubernetes.io/fr/docs/concepts/storage/volumes/
   volumes: []
+  ## DEPRECATED please use root level extraVolumeMounts value
   ## Specify extra mounts. Refer to ".spec.containers.volumeMounts" specification : https://kubernetes.io/fr/docs/concepts/storage/volumes/
   mounts: []