minnieshi commented on code in PR #28610:
URL: https://github.com/apache/airflow/pull/28610#discussion_r1057996707
##########
docs/helm-chart/production-guide.rst:
##########
@@ -34,33 +34,54 @@ can be found at :doc:`Set up a Database Backend
<apache-airflow:howto/set-up-dat
When using the helm chart, you do not need to initialize the db with
``airflow db init``
as outlined in :doc:`Set up a Database Backend
<apache-airflow:howto/set-up-database>`.
Review Comment:
somewhere it is best to say to either use this `data.metadataSecretName`, or
use 'config.database.sql_alchemy_conn', because the 1st link below refers to
the 2nd link, and the 2nd link requires us to setup the
config.database.sql_alchemy_conn section.
https://airflow.apache.org/docs/helm-chart/stable/production-guide.html#database
https://airflow.apache.org/docs/apache-airflow/2.4.1/howto/set-up-database.html#setting-up-a-postgresql-database
above was the reason caused me confused and i thought i had to configure
both parts and still not sure which one would work;
worse yet, the page:
https://airflow.apache.org/docs/apache-airflow/stable/howto/set-config.html has
other values which might serve the same as "sql_alchemy_conn_" such as
"sql_alchemy_conn_secret", "sql_alchemy_conn_cmd" etc
--
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]
