Sam Stoelinga
Sam Stoelinga
Seems NAP without pathways is also impacted. I think we need a different fix. See #603
This PR should solve NAP and AXLearn support as well. Would prefer to get this merged and will check with Luke on why it wasn't working for him.
It's still needed for us to be able to run AXLearn on xpk clusters.
@mwysokin could we please prioritize this and provide recommendation on how to better configure Kueue so it's more flexible?
Thank you! PR looks great to me. Will give @nstogner a chance to review too.
Thanks for giving it a quick try. Let's keep this PR open because I do see value in exposing priorityClassName in a more straight forward way with defaulting to it...
Thanks for the follow up! It's been busy recently. Apologies for the delay. @nstogner I think this is a clean small addition for a somewhat common use case. The JSONPatch...
Thanks for your patience @liebman ! Hopefully this unblocks you for now.
Would you prefer passing a PVC or PV to the model spec? @SatyKrish please share your reasoning for preference as well.
Perfect, the PR is working for vLLM support to load models from PVC directly. Probably will be merged tomorrow.