rjgoyln opened a new pull request, #72697:
URL: https://github.com/apache/airflow/pull/72697
## Summary
The Dataproc triggers hand a format string and its arguments to the
exception constructor the way a logging call takes them, so nothing is ever
substituted. The raised message is the repr of the argument tuple, and the
identifiers it exists to carry — the missing task instance, the reason a
cancellation failed — never reach the user:
```
('Error during cancellation handling: %s', ValueError('cluster is already
deleted'))
```
- Interpolate the seven raise sites in `triggers/dataproc.py`.
- Normalise the six `TaskInstance ... is not found` messages to one wording.
- Test the two sites reachable on Airflow 3.3+: the cluster trigger's task
state lookup and its cancellation handler.
Exception types stay as they are — narrowing `AirflowException` here would
change what a user's `except` clause catches. The remaining five sites sit
behind pre-3.3 compatibility branches that no test running against `main`
reaches.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 5)
Generated-by: Claude Code (Opus 5) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
