NVIDIA / gpu-operator / 38676325851

45%

main: 45%

Build Type

Pull #3081

github

Committed by tariq1890

Commit Message

fix[device-plugin] add cdi validation guardrail

This commit adds additional validation to the devicePlugin operand
transformation logic. When CDI is enabled, it is required that the
DEVICE_LIST_STRATEGY env var value includes one of
"cdi-annotations" and "cdi-cri". While the current transformation code
adds these values to DEVICE_LIST_STRATEGY when .spec.cdi.enabled is true,
the same resolved value gets overridden when the gpu-operator user sets
the DEVICE_LIST_STRATEGY env var explicitly in the devicePlugin section
of the ClusterPolicy custom resource.

This changes addresses a major limitation in the gpu-operator where
feedback from an invalid devicePlugin config  only happens at gpu pod
workload scheduling which is a significant delay. This provides a poor
UX as users waste hours debugging the containerCreate failures instead
of identifying the invalid devicePlugin config. This fix changes that by
failing at the ClusterPolicy reconcile stage.

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>

Pull Request Pull Request #3081: fix[device-plugin] add cdi validation guardrail

Coverage Stats

10 of 10 new or added lines in 1 file covered. (100.0%)

272 existing lines in 1 file now uncovered.

5869 of 13035 relevant lines covered (45.02%)

0.45 hits per line