Crowiant commented on code in PR #45931:
URL: https://github.com/apache/airflow/pull/45931#discussion_r2022943159


##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -1265,6 +1265,20 @@ secrets:
       sensitive: true
       example: ~
       default: ""
+    backends_order:
+      description: |
+        Comma-separated list of secret backends. These backends will be used 
in the order they are specified.
+        Please note that the `environment_variable` and `metastore` are 
required values and cannot be removed
+        from the list. Supported values are:
+
+        * ``custom``: Custom secret backend specified in the 
``secrets[backend]`` configuration option.
+        * ``environment_variable``: Standard environment variable backend
+          
``airflow.secrets.environment_variables.EnvironmentVariablesBackend``.
+        * ``metastore``: Standard metastore backend 
``airflow.secrets.metastore.MetastoreBackend``.
+      version_added: 3.0.0
+      type: string
+      example: ~
+      default: "custom,environment_variable,metastore"

Review Comment:
   Hello @eladkal ! I don't see any info regarding the secrets section the UI 
is using right now. So maybe we could skip it for now? What do you think? 
   Regarding the naming.  As stated in the doc it is responsible for _secrets 
backends search ordering_. So maybe `backends_search_order` is more convenient?



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

Reply via email to