rollandf / multus-cni / 28105102607

52%

master: 50%

Build Type

push

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>

Coverage Stats

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

761 existing lines in 12 files now uncovered.

2713 of 5236 relevant lines covered (51.81%)

6.75 hits per line