k8snetworkplumbingwg / multus-cni / 28105106557

52%

master: 50%

Build Type

Pull #1492

github

Committed by rollandf

Commit Message

server: fix ignoreReadinessIndicator being silently skipped when no server config is set

overrideCNIConfigWithServerConfig returned the client config unchanged
whenever overrideConf was empty, bypassing the readinessindicatorfile
key-stripping logic even when ignoreReadinessIndicator=true.  This
caused STATUS and GC handlers to poll for the readiness indicator file
for up to 45 s in tests that create the server with
ignoreReadinessIndicator=true and no server-side config override.

Also remove the now-unused sets import.

Signed-off-by: Fred Rolland <frolland@nvidia.com>

Pull Request Pull Request #1492: feat: DRA resource.k8s.io/v1 integration via draclient

Coverage Stats

186 of 229 new or added lines in 5 files covered. (81.22%)

2 existing lines in 1 file now uncovered.

2713 of 5236 relevant lines covered (51.81%)

6.79 hits per line