is_scheduled REST API parameter - #72029
Conversation
jason810496
left a comment
There was a problem hiding this comment.
Thanks for the follow-up.
Another good question I came up with is that should we consider the OnceTimetable as unscheduled or scheduled?
| dag_ids=readable_dags_filter.value, | ||
| ) | ||
| if is_scheduled is not None: | ||
| unscheduled_timetable_types = ("NullTimetable", "PartitionedAtRuntime") |
There was a problem hiding this comment.
How about maintaining the list as frozenset at model level? Though it seems API server is the only caller now, but having the them in the model level should have better maintenance and can be served as single source of truth.
There was a problem hiding this comment.
I don't feel strongly either way. Selfishly and candidly, I'd prefer this to be merged and then adjusted later since this PR is mostly for my Summit talk about using Airflow as a sort of guardrail for automated agentic coding, and it one-shot this.
I think it'd be a backward compatible change, and I'd be happy to commit to changing that in a follow-up PR. That's not a good reason to avoid this if you feel it's important, though.
There was a problem hiding this comment.
As long as we move this in a follow up PR then this works for me.
My gut reaction is that we should consider anything that doesn't have |
closes: #53741
Was generative AI tooling used to co-author this PR?
Generated-by: [Roborchestra] following the guidelines
Roborchestra is a collection of Airflow Dags that invoke AgentOperator (and other Airflow-native tools) to address open issues in Airflow.