zanecodes commented on code in PR #55820: URL: https://github.com/apache/airflow/pull/55820#discussion_r2446165777
########## chart/values.yaml: ########## @@ -2853,9 +2853,13 @@ cleanup: successfulJobsHistoryLimit: ~ # Configuration for postgresql subchart -# Not recommended for production +# Uses bitnamilegacy images to avoid Bitnami licensing restrictions +# Not recommended for production - use external database instead postgresql: enabled: true + image: + repository: postgres Review Comment: That's reasonable in the long term, but as mentioned, this temporary fix doesn't work at all since it uses a non-Bitnami `postgres` image that is not compatible with the Bitnami PostgreSQL Helm chart. I would argue that this change should not have been merged, since it does not fix the issue and confuses users. A better temporary fix would have been to use the `bitnamilegacy/postgresql` image as it seems was originally intended. -- 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]
