rjgoyln opened a new pull request, #72758:
URL: https://github.com/apache/airflow/pull/72758

   ## Summary
   
   A trigger whose constructor raises anything but `TypeError` while the 
triggerer re-creates it stops the whole triggerer: the exception escapes 
`create_triggers`, the runner subprocess exits, and the supervisor exits with 
it. After a restart the same trigger is picked up again, so the triggerer 
crash-loops until the task is cleared. Failures while decrypting or decoding 
the stored kwargs take the same path.
   
   - Fail only the offending trigger when its constructor raises, instead of 
stopping the runner.
   - Cover it with a test where one trigger's constructor raises and a second 
trigger still starts.
   
   A bad classpath a few lines above already fails only that trigger; the 
constructor guard now does the same, stopping at `Exception` so cancellation 
and shutdown still reach the runner loop. The `TypeError`-only guard from 
#31999 came from a readability suggestion in review, not from a reason to let 
other exceptions through.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Fable 5.1)
   
   Generated-by: Claude Code (Fable 5.1) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
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]

Reply via email to