getsentry / airflow-metrics

Metrics for airflow
Apache License 2.0
14 stars 6 forks source link

Fix log level change of blacklisted domain messages from WARNING to INFO #50

Closed DannyNemer closed 4 years ago

DannyNemer commented 4 years ago

Reverts getsentry/airflow-metrics#49, in which I accidentally changed the log level of the wrong log message. Pardon please pardon my mistake.

This PR includes the correct change, which applies the same change articulated in #49: Reduces the log level for the message "Found blacklisted domain" in patch_requests.py from WARNING to INFO because each warning log sends a message to Sentry. For my team, this becomes over 17,000 warnings every day, which is excessive for this warning that is largely unactionable.