Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change log level to DEBUG when secret not found for google secret manager #27856

Merged
merged 3 commits into from
Dec 3, 2022

Conversation

lwyszomi
Copy link
Contributor

In the Airflow we serching first the connections & variables from alternative secrets backend and then in the databse, so we can have the situation when part of variables/connection we have in database and part of them in the secret backend. In this case we generating a lot of error logs that they are not errors, even if the variable/connection doesn't exist, I think, info level is enough.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@lwyszomi lwyszomi requested a review from turbaszek as a code owner November 23, 2022 11:31
@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Nov 23, 2022
@lwyszomi lwyszomi changed the title Change log level to info when secret not found for google secret manager Change log level to DEBUG when secret not found for google secret manager Nov 24, 2022
@udayarctempus
Copy link

In which version of Airflow we can see this fix ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants