Apply validator resources to init containers#2557
Conversation
Signed-off-by: Shivkumar Ople <sople@nvidia.com>
73f0040 to
98cfccd
Compare
|
Thanks @Shivkumar13 , the changes look fine to me. We should be doing this for all driver daemonsets managed by gpu-operator. For example, we are still missing requests/limits for k8s-driver-manager init container in driver daemonset. There are few other init containers as well which are also missing requests/limits. |
|
@rahulait should we do that in this PR or separate ones? |
|
Lets do them in single PR if possible. Its easier to review together and keep things consistent and tied to single commit. |
|
@tariq1890 any objections to the change being done here? |
Agreed, we should be making this change in this PR. This behavioural change should applied consistently to all components managed by the operator |
Fixes #702.
This change applies the existing
spec.validator.resourcessettings to validator init containers as well as the main validator container. The affected init containers run the validator image for driver, toolkit, CUDA, and plugin checks, so using the existing validator resource field keeps the API surface unchanged while making those init containers configurable.Validation:
go test ./controllers -run TestTransformValidatorComponentWithResources -count=1go test ./controllers -run TestTransformValidator -count=1go test ./controllers -run TestTransformSandboxValidator -count=1