vincbeck commented on code in PR #29580:
URL: https://github.com/apache/airflow/pull/29580#discussion_r1113446539
##########
airflow/providers/amazon/aws/secrets/secrets_manager.py:
##########
@@ -264,14 +286,14 @@ def get_conn_uri(self, conn_id: str) -> str | None:
def get_variable(self, key: str) -> str | None:
"""
- Get Airflow Variable from Environment Variable
+ Get Airflow Variable
Review Comment:
In the context of this class, we are actually getting it from only one
location: AWS Secrets Manager. If this function returns None, then it is
fetched from other location: Environment variable then metastore but this is
done outside of this class. [See documentation
here](https://airflow.apache.org/docs/apache-airflow/1.10.10/howto/use-alternative-secrets-backend.html).
Let me know if you still think I should update the documentation
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
