kaxil commented on a change in pull request #15296:
URL: https://github.com/apache/airflow/pull/15296#discussion_r610242636



##########
File path: chart/values.yaml
##########
@@ -567,6 +567,9 @@ pgbouncer:
   # Maximum clients that can connect to pgbouncer (higher = more file 
descriptors)
   maxClientConn: 100
 
+  # supply name of existing secret with pgbouncer.ini and users.txt defined
+  pgbouncerConfigSecretName: ~

Review comment:
       ```suggestion
     configSecretName: ~
   ```
   
   Since it is already in `pgbouncer` key -- adding `pgbouncer` is a bit 
redundant. Can you also give an example of an existing config with 
`pgbouncer.ini` and `users.txt`




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


Reply via email to