riccardo-salamanna commented on pull request #21606: URL: https://github.com/apache/airflow/pull/21606#issuecomment-1049692233
i removed the test cause i'm not thinking it might not be needed, since all i'm doing is adding a condition to the creation of the pgbouncer stats secret if i provide an external one, and there's already a test on the env for the stats.... regardless, thanks @jedcunningham for your support, i tried to run the testing for helm locally both with breeze and virtualenv, and we were not able to get them running (```subprocess.CalledProcessError: Command '['helm', 'dep', 'update', '/opt/airflow/chart']' returned non-zero exit status 1```) If you have any other instructions let me know, i would be interested in contributing to the helm chart more. 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]
