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/13 06:14:38 UTC
[GitHub] [superset] elyzov opened a new issue #16246: [Helm Chart] redis password not set for celery brokers
elyzov opened a new issue #16246:
URL: https://github.com/apache/superset/issues/16246
I've set **REDIS_PASSWORD** environment variable, but celery worker can't connect to redis, as the [password not set](https://github.com/apache/superset/blob/master/helm/superset/templates/_helpers.tpl#L82) in connection string.
### Expected results
**REDIS_PASSWORD** environment variable is used in all redis configs.
### Actual results
**REDIS_PASSWORD** environment variable is used [only in cache configuration](https://github.com/apache/superset/blob/master/helm/superset/templates/_helpers.tpl#L66).
#### How to reproduce the bug
1. Deploy redis with password.
2. Set **REDIS_PASSWORD** environment variable.
3. Deploy superset via helm chart.
4. See errors in worker logs.
### Environment
(please complete the following information):
- superset version: `1.2.0`
- helm chart version: `0.3.5`
### Checklist
Make sure to follow these steps before submitting your issue - thank you!
- [x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
- [x] I have reproduced the issue with at least the latest released version of superset.
- [x] I have checked the issue tracker for the same issue and I haven't found one similar.
--
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] elyzov edited a comment on issue #16246: [Helm Chart] redis password not set for celery brokers
Posted by GitBox <gi...@apache.org>.
elyzov edited a comment on issue #16246:
URL: https://github.com/apache/superset/issues/16246#issuecomment-898224570
I seе, I can do some workaround, but nevertheless, there is a bug in the default configuration.
--
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] nytai commented on issue #16246: [Helm Chart] redis password not set for celery brokers
Posted by GitBox <gi...@apache.org>.
nytai commented on issue #16246:
URL: https://github.com/apache/superset/issues/16246#issuecomment-898220781
You can configure celery however you want via the superset_config overrides https://superset.apache.org/docs/installation/running-on-kubernetes#superset_configpy
--
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] elyzov commented on issue #16246: [Helm Chart] redis password not set for celery brokers
Posted by GitBox <gi...@apache.org>.
elyzov commented on issue #16246:
URL: https://github.com/apache/superset/issues/16246#issuecomment-898224570
I seе, I can do some workaround, but nevertheless, there is an error in the default configuration.
--
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] ad-m commented on issue #16246: [Helm Chart] redis password not set for celery brokers
Posted by GitBox <gi...@apache.org>.
ad-m commented on issue #16246:
URL: https://github.com/apache/superset/issues/16246#issuecomment-1035795100
@elyzov could you check if that issue still is relevant after merged & released #18642 ?
--
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