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

   Previously this limited pod length to 63 but that's the limit for label 
values not pod ids.  I assume that's just an atavism.  Meanwhile, a UUID was 
previously used which takes up 32 characters, much more space than necessary.  
We use 8 random alphanum chars now which should be plenty.
   
   Example:
   
   Calling with 
   
   ```
   
PodGenerator.make_unique_pod_id("this-is-my-semi-long-but-not-too-long-pod-id")
   ```
   
   We would previously get this:
   
   ```
   this-is-my-semi-long-but-not-t-c27ae4725b2049d6bf953e9f1f3d14c1
   ```
   
   And now we'd get this:
   ```
   this-is-my-semi-long-but-not-too-long-pod-id-l4hiiqxh
   ```
   


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