[GitHub] [airflow] marclamberti commented on pull request #8777: Add Production Helm chart support

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

marclamberti commented on pull request #8777:
URL: https://github.com/apache/airflow/pull/8777#issuecomment-633090457


   @thesuperzapper I think there is an issue in the Helm chart. 
   `
   redis:
    enabled: false
   `
   produces the following error:
   `
   spec.template.spec.containers[0].env[1].valueFrom.secretKeyRef.name: Invalid value: \"\": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
   `
   which I think is related to this:
   `
   - name: REDIS_PASSWORD
     valueFrom:
       secretKeyRef:
         name: {{ .Values.externalRedis.passwordSecret }}
         key: {{ .Values.externalRedis.passwordSecretKey }}
   `
   where passwordSecret is by default sets to "" and so the error.
   Hope it helps


----------------------------------------------------------------
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:
users@infra.apache.org