You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/08/19 18:27:35 UTC

[GitHub] [superset] cccs-tom opened a new issue #16359: [Helm Chart] Allow extraVolumes and extraVolumeMounts to be specified in values.

cccs-tom opened a new issue #16359:
URL: https://github.com/apache/superset/issues/16359


   **Is your feature request related to a problem? Please describe.**
   When using Superset as a subchart, it is impossible (known Helm limitation) to "inject" configuration or Python (or other) files using `.Files.Get` or similar. We would like to be able to define our own ConfigMaps / Secrets / other volumes and mount them into the pods. Other charts we use include `extraVolumes` and `extraVolumeMounts` in their templates for this purpose. I would like to add those to the Superset chart.
   
   **Describe the solution you'd like**
   What we have currently implemented (which I would gladly contribute) are `extraVolumes` and `extraVolumeMounts` objects at the root of the Superset values. Those are applied to the main containers of all 3 deployments as well as the init-job. The values looks like this:
   ```yaml
   extraVolumes:
     volume1:
       configMap:
         name: '{{ template "superset.fullname" . }}-custom-config'
     volume2:
       secret:
         secretName: my-secret
   extraVolumeMounts:
     volume1:
       mountPath: /mnt/volume1
     volume2:
       mountPath: /mnt/volume2
   ```
   The key for each object becomes the 'name' field (so the volume and volumeMount keys must match)
   
   **Describe alternatives you've considered**
   The above fits our use case well, but other use cases may not require all volumes to be mounted everywhere. One possible alternative would be to have `extraVolumes` and `extraVolumeMounts` objects under `supersetNode`, `supersetWorker`, etc. In our specific use case, it would mean some repetition and additional maintenance, but that would be an acceptable solution nevertheless.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org