You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/01/09 22:43:34 UTC

[GitHub] [airflow] SamWheating commented on a change in pull request #20733: Add Audit Log View to Dag View

SamWheating commented on a change in pull request #20733:
URL: https://github.com/apache/airflow/pull/20733#discussion_r780841362



##########
File path: airflow/config_templates/default_airflow.cfg
##########
@@ -658,6 +658,18 @@ auto_refresh_interval = 3
 # Boolean for displaying warning for publicly viewable deployment
 warn_deployment_exposure = True
 
+# Number of dags to look back for dag audit log view
+audit_trail_query_days = 31
+
+# Types of events to ignore for dag audit log view
+# E.g. excluded_events = success,failed,skipped,landing_times
+excluded_events =

Review comment:
       Is it worth validating that `included_events` and `excluded_events` have no overlap? I know that its a weird edge case / user Error, but I think this would lead to invalid queries when rendering the view.




-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org