potiuk commented on code in PR #26604: URL: https://github.com/apache/airflow/pull/26604#discussion_r978049933
########## docs/helm-chart/index.rst: ########## @@ -122,13 +122,15 @@ The command removes all the Kubernetes components associated with the chart and .. note:: Some kubernetes resources created by the chart `helm hooks <https://helm.sh/docs/topics/charts_hooks/#hook-resources-are-not-managed-with-corresponding-releases>`__ might be left in the namespace after executing ``helm uninstall``, for example, ``brokerUrlSecret`` or ``fernetKeySecret``. -Installing the Chart with Argo CD or Flux ------------------------------------------ +Installing the Chart with Argo CD, Flux or Terraform +----------------------------------------------------- -When installing the chart using Argo CD or Flux, you MUST set the two following values, or your application +When installing the chart using Argo CD, Flux, or Terraform, you MUST set the two following values, or your application will not start as the migrations will not be run: .. code-block:: yaml createUserJob.useHelmHooks: false migrateDatabaseJob.useHelmHooks: false + +This also applies if you install the chart using `--wait` in your `helm install` command. Review Comment: ```suggestion This also applies if you install the chart using ``--wait`` in your ``helm install`` command. ``` -- 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]
