Skip to content

Fix airflowctl dags state crash from an incomplete backport - #71095

Merged
Lee-W merged 1 commit into
apache:airflow-ctl/v0-1-testfrom
astronomer:ctl-v0-1-fix-dags-state-crash
Aug 5, 2026
Merged

Fix airflowctl dags state crash from an incomplete backport#71095
Lee-W merged 1 commit into
apache:airflow-ctl/v0-1-testfrom
astronomer:ctl-v0-1-fix-dags-state-crash

Conversation

@Lee-W

@Lee-W Lee-W commented Aug 4, 2026

Copy link
Copy Markdown
Member

The dags state backport (#69915, #70352) brought the caller and the tests but dropped suppress_error_log from DagRunOperations.list(), which main has. dag_command.py:147 passes it, so the command raised:

TypeError: DagRunOperations.list() got an unexpected keyword argument 'suppress_error_log'

Restore the keyword-only parameter and forward it to the client as the airflowctl_suppress_error_log extension, matching the sibling get().

The broken caller only exists on this test branch, so no released airflowctl is affected and no newsfragment is needed.

Split out of #70724 so that PR stays workflow-only.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Opus 5

Generated-by: Claude Opus 5 following the guidelines

The dags state backport (apache#69915, apache#70352) brought the caller and the tests
but dropped suppress_error_log from DagRunOperations.list(), which main
has. dag_command.py:147 passes it, so the command raised:

    TypeError: DagRunOperations.list() got an unexpected keyword
    argument 'suppress_error_log'

Restore the keyword-only parameter and forward it to the client as the
airflowctl_suppress_error_log extension, matching the sibling get().

The broken caller only exists on this test branch, so no released
airflowctl is affected and no newsfragment is needed.
@Lee-W
Lee-W marked this pull request as ready for review August 5, 2026 08:43

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I double checked with the upstream main.

@Lee-W
Lee-W merged commit b5f71b0 into apache:airflow-ctl/v0-1-test Aug 5, 2026
2 checks passed
@Lee-W
Lee-W deleted the ctl-v0-1-fix-dags-state-crash branch August 5, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants