[GitHub] [superset] craig-rueda commented on a change in pull request #18642: feat(Helm): Redis with password supported in helm charts and redis chart version updated

[GitBox <gi...@apache.org>]

craig-rueda commented on a change in pull request #18642:
URL: https://github.com/apache/superset/pull/18642#discussion_r803235325



##########
File path: helm/superset/templates/_helpers.tpl
##########
@@ -89,6 +89,21 @@ WTF_CSRF_ENABLED = True
 WTF_CSRF_EXEMPT_LIST = []
 # A CSRF token that expires in 1 year
 WTF_CSRF_TIME_LIMIT = 60 * 60 * 24 * 365
+{{- if .Values.supersetNode.connections.redis_password }}
+class CeleryConfig(object):

Review comment:
       I thought the same thing, but drying that up will be a little tricky and will likely make the code less readable in this case.




-- 
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