craig-rueda commented on a change in pull request #13917:
URL: https://github.com/apache/superset/pull/13917#discussion_r606028760
##########
File path: helm/superset/values.yaml
##########
@@ -21,15 +21,19 @@
replicaCount: 1
-## These requirements are used to build a requirements file which is then
applied on init
-## of superset containers
-additionalRequirements:
- - "psycopg2==2.8.5"
- - "redis==3.2.1"
-
-## These apt packages are applied on init of superset containers
-additionalAptPackages: {}
- # - nano
+# User ID directive
+# Note that this user must have appropriate permissions to run the bootstrap
script
+runAsUser: 1000
+
+# Install additional packages and do any other bootstrap configuration in this
script
+# For production clusters it's recommended to build own image with this step
done in CI
+bootstrapScript: |
+ #!/bin/bash
Review comment:
Understood. Can you please put it back to 0 and enable that script?
Maybe just add some verbage indicating that this isn't something one should do
in production? There's several folks that rely on that thing currently
(including me 😀).
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
