ferruzzi commented on PR #24058:
URL: https://github.com/apache/airflow/pull/24058#issuecomment-1143896551
> I think it would be an easy lift too, the biggest piece would be plugging
in the cloud paramater store of your choice (we used SSM but the Google folks
likely want to use something else)
An idea that I had initially was to have `fetch_variable` accept a list/set
of callables which would be called in order they are provided, that way we
could do something like:
```
var_priority = {
os.getenv,
_fetch_from_ssm,
}
value = utils.fetch_variable(key, var_priority, default_value)
```
Then anyone could drop in whatever parameter storage they choose to use, in
whatever priority order they desire. We decided that env vars would take
precedence on our test suite, but there's no reason for that to be a universal
truth.
--
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]
