rom1mouret commented on PR #27111: URL: https://github.com/apache/airflow/pull/27111#issuecomment-1311134448
I am not advocating for re-opening this issue now, but I wonder if it might still lack clarity? Contrary to what one would intuitively expect, isn't the SLA relative to the *next* DAG run? The code calls `next_dagrun_info()` twice before any SLA check. The first time to calculate the end of the current interval, and the second time to calculate the end of the next interval, which could be one day in the future if the DAG is scheduled to run once a day. https://github.com/apache/airflow/blob/cc4cde987cbc073a223b531a7674856cb2847f9a/airflow/dag_processing/processor.py#L421 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
