wkhudgins92 commented on issue #8170: [AIRFLOW-6515] Change Log Levels from Info/Warn to Error URL: https://github.com/apache/airflow/pull/8170#issuecomment-611762956 @turbaszek I would love to add a regression test for this but I agree with @kaxil , it likely is not possible. You could flag certain words in an info message, like I did with the grep statements, but at the end of the day there's always a change that those keywords will appear in a valid info/debug log statement. For example, ``` self.log.info("Secret %s not found in Path: %s", secret_id, secret_path) ``` not found in path sounds like an error, or at least a warn, but in reality, in this specific context it should be a debug. The best we could do would be a flag/warning but I don't think we can build a sufficiently smart CI check to incorporate it into the official CI processes. Another alternative could be clear guidelines on logging in some kind of easy-to-access contributor documentation. We could even make it a checkbox on the PR template.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
