zanecodes commented on code in PR #55820:
URL: https://github.com/apache/airflow/pull/55820#discussion_r2411019868


##########
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:
   This seems to contradict the above comment, this is not using the 
`bitnamilegacy/postgresql` image.
   
   Additionally, Bitnami heavily customizes their images, and the process for 
configuring them is rather idiosyncratic, so it's unlikely that the non-Bitnami 
`postgres` image will work correctly with the Bitnami PostgreSQL Helm chart.
   
   Has this been tested?



-- 
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