Skip to content

Conversation

@kacpermuda
Copy link
Contributor

Given that new listener method on_task_instance_skipped was added in #59467, we should add listener invocation in task state api endpoint, similar to success and failure states added in #48941. Also updating listeners doc with this new method, since the implementing PR did not contain doc update.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

LGTM

@kacpermuda kacpermuda force-pushed the feat-listener-skip-api branch from 3e9d3bb to 9b3f2ce Compare January 15, 2026 13:49
@kacpermuda
Copy link
Contributor Author

Thanks, the remote logging test failure seems unrelated

@pierrejeambrun
Copy link
Member

I just restarted the failed job. lets see if it's flaky

@kacpermuda
Copy link
Contributor Author

Yup, passed now, thanks

@pierrejeambrun pierrejeambrun merged commit b77ec2c into apache:main Jan 15, 2026
243 of 244 checks passed
jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API kind:documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants