mik-laj commented on pull request #9861:
URL: https://github.com/apache/airflow/pull/9861#issuecomment-674750958


   > Btw. would you mind getting rid of this warning:
   
   Yes. Among other things, I want to solve this problem in the next change. 
   
   Basically this is now a problem that this TaskHandler stores configuration 
in two places - in the configuration file and in connection. It's not a good 
idea for two reasons:
   1. Connection is controlled by the user, so it should not contain the server 
configuration. It is very easy for a user to accidentally crash the server 
instances.
   2. When you have Airflow implementation in multiple GCP projects and you use 
Secret Backend, because you must have Secret Manager API enabled in all of 
them. Such a requirement makes it very difficult to use for some users.
   3. This is a custom way of configuring because the backend secret is fully 
configurable by the INI file.
   
   I would like to tweak it so that it is configured in a similar way to the 
StackdriverTaskHandler configuration. All configuration in ini file.


----------------------------------------------------------------
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]


Reply via email to