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/07/09 12:33:04 UTC
[GitHub] [superset] aboutphilippe opened a new pull request #15617: Fix helm chart db connection credentials Fixes #15133
aboutphilippe opened a new pull request #15617:
URL: https://github.com/apache/superset/pull/15617
### SUMMARY
The helm chart is using hardcoded values for the db connection that are different than the postgres values.
For security reasons and to make sure the db connection use the correct password for postgres, I've modified the db connection values to refer to the postgres values.
### TESTING INSTRUCTIONS
Follow instructions on
https://superset.apache.org/docs/installation/running-on-kubernetes
with default security settings:
```
postgresql:
postgresqlPassword: **UNIQUEPASSWORD**
```
### ADDITIONAL INFORMATION
- x] Has associated issue: https://github.com/apache/superset/issues/15133
--
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
[GitHub] [superset] aboutphilippe closed pull request #15617: Fix helm chart db connection credentials Fixes #15133
Posted by GitBox <gi...@apache.org>.
aboutphilippe closed pull request #15617:
URL: https://github.com/apache/superset/pull/15617
--
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
[GitHub] [superset] aboutphilippe commented on pull request #15617: Fix helm chart db connection credentials Fixes #15133
Posted by GitBox <gi...@apache.org>.
aboutphilippe commented on pull request #15617:
URL: https://github.com/apache/superset/pull/15617#issuecomment-877167897
Unfortunately this solution is not working as helm v3 doesnt seem to support referring to other values.
(Tried adding quotations, tpl, with no success)
--
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