milton0825 edited a comment on pull request #8889:
URL: https://github.com/apache/airflow/pull/8889#issuecomment-631680187


   Other use cases might be that we can replace the variable backend with other 
storage like S3 and support both read/write.
   
   It seems like `Variable` is now tightly coupled with `SecretBackend` for 
DAGs to retrieve secrets from Vault. However, I have seen Airflow users that 
use `Variable` to share data between tasks/DAGs (yes they can also use xcom but 
they still can choose to use `Variable` as long as it exists) so it's usage is 
not limited to retrieving secrets. As long as `Variable` support a set method, 
it can be confusing to the users that they thought they can set a variable 
backed by Vault, which they actually can't.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to