sudeepgupta90 commented on code in PR #29078: URL: https://github.com/apache/airflow/pull/29078#discussion_r1083290727
########## docs/helm-chart/index.rst: ########## @@ -140,3 +140,22 @@ will not start as the migrations will not be run: This is so these CI/CD services can perform updates without issues and preserve the immutability of Kubernetes Job manifests. This also applies if you install the chart using ``--wait`` in your ``helm install`` command. + +.. note:: + While deploying this Helm chart with Argo, you might encounter issues with ``DB Migrations`` not running automatically on upgrade. + +To ensure DB Migrations with Argo CD, you will need to: + +.. code-block::yaml + + migrateDatabaseJob: + jobAnnotations: + "argocd.argoproj.io/hook": Sync + +and, equivalent configurations in case of Flux CD Review Comment: @BasPH tbh, I haven't used Flux CD, nor could I find anything on how to achieve this in Flux CD docs. Do you think its more appropriate to remove this line? I only put it in there because the original section mentioned Flux CD in the heading. -- 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]
