Skip to content

Commit

Permalink
feat(helm): Add optional apt-get install to superset_bootstrap.sh (ap…
Browse files Browse the repository at this point in the history
…ache#13678)

* add conditional apt install

* move install switch to correct command.
  • Loading branch information
danielewood authored and Allan Caetano de Oliveira committed May 21, 2021
1 parent bb5a945 commit f9436b0
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
12 changes: 9 additions & 3 deletions helm/superset/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -49,11 +49,17 @@ Create chart name and version as used by the chart label.
{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" -}}
{{- end -}}

{{- define "superset-bootstrap" }}
{{- define "superset-bootstrap" -}}
#!/bin/sh

{{ if .Values.additionalAptPackages }}
apt-get update -y \
&& apt-get install -y --no-install-recommends \
{{ range .Values.additionalAptPackages }}{{ . }} {{ end }}\
&& rm -rf /var/lib/apt/lists/*
{{ end -}}
{{ if .Values.additionalRequirements }}
pip install {{ range .Values.additionalRequirements }}{{ . }} {{ end }}

{{ end -}}
{{ end -}}

{{- define "superset-config" }}
Expand Down
4 changes: 4 additions & 0 deletions helm/superset/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ additionalRequirements:
- "psycopg2==2.8.5"
- "redis==3.2.1"

## These apt packages are applied on init of superset containers
additionalAptPackages: {}
# - nano

## The name of the secret which we will use to generate a superset_config.py file
## Note: this secret must have the key superset_config.py in it and can include other files as well
##
Expand Down

0 comments on commit f9436b0

Please sign in to comment.