potiuk commented on PR #23560:
URL: https://github.com/apache/airflow/pull/23560#issuecomment-1125353155
I am very much in favour of the last proposal by @dstandish.
The Low/High/Medium is super unintuitive, and we might as well make it
explicit and generic as Daniel explained. I would even modify it slightly to
simply make it single json array with both - classes and params:
```
secrets = [
{
"backend": "airflow.smth.Backend",
"init-kwargs": {
"arg1": "",
"arg2: True,
.....
}
]
```
This is explicit, localized (args next to class), fully customizable and
much easier to reason about.
--
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]
