eladkal commented on issue #64890:
URL: https://github.com/apache/airflow/issues/64890#issuecomment-4205337816

   > Collects failed task IDs for that run
   
   This should be given in context starting Airflow 3.2 (fixed in 
https://github.com/apache/airflow/pull/61274)
   
   
   > One of the following would unblock Airflow 2 -> 3 notifier migrations 
cleanly:
   > 1. Add a supported Task SDK / callback-context helper to update 
DagRun.note from notifier code.
   >2. Expose a documented runtime client operation for patching the current 
DagRun note from callback code.
   >3. If this is intentionally unsupported, document the recommended 
replacement pattern for storing callback correlation metadata on a DagRun in 
Airflow 3.
   
   I think that the initial thought behind notes is that it was designed to be 
used by on-calls manually but I can understand how the use case is expanded 
since then. Overall I favor option 1.
   


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