kubeflow / trainer / 29265847382

67%

Build Type

push

github

Committed by web-flow

Commit Message

feat(initializer): validate initializer Pod uses correct volume name (#3714)

* feat(runtime): validate initializer Pod uses correct volume name Add validation in the JobSet plugin's Validate() function to ensure the dataset/model initializer ReplicatedJob has a Volume named jobsetplgconsts.VolumeNameInitializer. Previously this was only a string constant with no enforcement, meaning a misconfigured runtime template could silently reference the wrong volume name. Fixes #3710

Signed-off-by: HarshPopat23 <musichk61@gmail.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: HarshPopat23 <musichk61@gmail.com>

* conversation of copilot

Signed-off-by: HarshPopat23 <musichk61@gmail.com>

* fix(initializer): validate initializer volume mounts

Signed-off-by: HarshPopat23 <musichk61@gmail.com>

* fix(initializer): simplify volume mount validation

Signed-off-by: HarshPopat23 <musichk61@gmail.com>

* fix(initializer): validate initializer volume mounts

Signed-off-by: HarshPopat23 <musichk61@gmail.com>

---------

Signed-off-by: HarshPopat23 <musichk61@gmail.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Coverage Stats

33 of 35 new or added lines in 1 file covered. (94.29%)

3 existing lines in 1 file now uncovered.

2438 of 3664 relevant lines covered (66.54%)

0.78 hits per line