dstandish commented on PR #28237:
URL: https://github.com/apache/airflow/pull/28237#issuecomment-1363195912

   i am not sure it does.
   
   how does it?
   
   FTH gets the pod name from ti.hostname.  If you store IP in ti.hostname, 
then now you are in even worse situation.
   
   And note.... it was previously limited to 63, but then i lifted the limit 
(in code never released) when trying to make a "nicer pod id" -- shorter random 
suffix (8 instead of 36, allowing for more meaningful content), and allow 
hyphens for readability
   
   But then when I was digging around task handler i discoverd why this is 
necessary, until / unless we switch to labels for "find pod" or store pod name 
in TI
   
   WDYT?
   
   


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to