XD-DENG commented on issue #4931: [AIRFLOW-4109] Added support for passing the airflow fernet key to th… URL: https://github.com/apache/airflow/pull/4931#issuecomment-474384649 Hi @stijndehaes , It should simply be `AIRFLOW__CORE__FERNET_KEY` if I'm not mistaken. In the link you shared, there is also a section sharing the same thing, but you may have missed it: https://github.com/helm/charts/tree/master/stable/airflow#airflow-configuration IMO, fernet key is nothing different from other configuration items, and we should not have that many ways to specify them. Another question: your change may be a breaking behaviour change to users who set fernet key among nodes using the method I mentioned. Yet another question: there are other credential configuration like `sql_alchemy_conn`/`broker_url`. Should we apply your logic to all of them? Personally I don't think so.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
