piotrlinski commented on PR #71713:
URL: https://github.com/apache/airflow/pull/71713#issuecomment-6498347614

   Fair points. I don't have a user report or an issue for this, only my own 
reading of the code path for ephemeral KubernetesExecutor pods. You're also 
right about the second clone path: it misses the bare-clone retry and backoff 
added in #73878, so a just-issued GitHub App token being rejected would be 
mistaken for the server refusing an unadvertised SHA and silently fall back to 
the full clone.
   
   Closing this. If someone hits it on Kubernetes with a large repository, I'll 
open an issue with measurements first. A follow-up would then fetch the pinned 
commit shallowly into the existing bare repo, so it reuses the same clone, 
retry and credential handling instead of adding a second path.
   
   ---
   Drafted-by: Claude Code (Opus 5.5); reviewed by @piotrlinski before posting
   


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