[GitHub] [airflow] dstandish commented on issue #28186: Pod name incorrect in RenderedTaskInstanceFields

[GitBox <gi...@apache.org>]

dstandish commented on issue #28186:
URL: https://github.com/apache/airflow/issues/28186#issuecomment-1374090763

   i guess the issue is that, RTIF gets stamped within _run_raw_task.  and by the time it gets here, we're already _within_ the pod.  so we no longer have the actual pod details.
   
   probably this pod spec does not belong in RTIF.  (it's not a templated field, after all) probably it was just a convenient / easy way to implement this feature, nowithstanding the limitations pointed out here.
   
   i suppose one thing we could do is, create the RTIF record earlier for KE (just with pod spec), then in run raw update it with the RTIFs.
   
   another option would be, put more KE info (or if you want to make an abstraction, executor info) on the TI record.  we could include spec here.  adding namespace and pod name to the TI record would also help with certain logging scenarios.
   
   another option would be add separate table like TIExecutorInfo which is 1-1 with TI and has arbitrary executor info such as this.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org