sudeepgupta90 commented on code in PR #29078: URL: https://github.com/apache/airflow/pull/29078#discussion_r1083291707
########## 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: My intent was to document: - That there are 'specific' extra steps to make this helm chart work with Argo and Flux - Specifically, resolve the documentation wrt to Argo because we have been getting a few issues mentioning helm-chart doesn't work with Argo -- 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]
