eladkal commented on code in PR #36935:
URL: https://github.com/apache/airflow/pull/36935#discussion_r1498975866


##########
airflow/api_connexion/schemas/dag_schema.py:
##########
@@ -69,6 +69,7 @@ class Meta:
     tags = fields.List(fields.Nested(DagTagSchema), dump_only=True)
     max_active_tasks = auto_field(dump_only=True)
     max_active_runs = auto_field(dump_only=True)
+    max_consecutive_failed_dag_runs = auto_field(dump_only=True)

Review Comment:
   I wonder about the name. `max_consecutive_failed_dag_runs` might be 
interpreted as not allowing scheduling after the X failures but this is not 
really a hard limit - user can override it by enabling the dag. For the other 
max properties user can't really override them.



-- 
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

Reply via email to