riccardo-salamanna opened a new pull request #21606:
URL: https://github.com/apache/airflow/pull/21606


   I noticed that when deploying airflow with pgbouncer using 
resultBackendSecretName and metadataSecretName, the pgbouncer-stats-secrets 
defaults to the values predefined credentials expressed in 
data.metadataConnection.user and data.metadataConnection.password, breaking the 
deployment
   ```
   data:
     connection: {{ urlJoin (dict "scheme" "postgresql" "userinfo" (printf 
"%s:%s" (.Values.data.metadataConnection.user | urlquery) 
(.Values.data.metadataConnection.pass | urlquery) ) "host" (printf 
"127.0.0.1:%s" (.Values.ports.pgbouncer | toString)) "path" "/pgbouncer" 
"query" (printf "sslmode=%s" (.Values.pgbouncer.metricsExporterSidecar.sslmode 
| toString ))) | b64enc | quote }}
   {{- end }}
   ```
   i added the possibility to use an external stats secret for pgbouncer 
exporter and it does worj, i don't know if it is the preferred solution but i 
mainly wanted to raise the issue. thanks


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to