kubevirt / hyperconverged-cluster-operator / 30198288342

86%

Build Type

push

github

Committed by web-flow

Commit Message

handlers, netresinjector: Gate ExternalNetResourceInjection FG on NRI deployment readiness status (#4421)

* netresinjector: Re-enable deployment by default

Reverts [1] which temporarily disabled network-resources-injector
deployment by default. The underlying bug has been fixed in [2].

[1] https://github.com/kubevirt/hyperconverged-cluster-operator/pull/4377
[2] https://github.com/k8snetworkplumbingwg/network-resources-injector/pull/224

Signed-off-by: Nir Dothan <ndothan@redhat.com>
Assisted by: Claude Sonnet 4.5 <noreply@anthropic.com>

* handlers, kubevirt: Conditionally enable ExternalNetResourceInjection FG

Gate the ExternalNetResourceInjection feature gate on both the
deployNetworkResourcesInjector knob and the NetworkResourcesInjectorReady
condition being True. The condition is set by the NRI deployment handler
when all replicas are ready, and removed when the deployment is disabled.

Signed-off-by: Nir Dothan <ndothan@redhat.com>
Assisted by: Claude Sonnet 4.5 <noreply@anthropic.com>

---------

Signed-off-by: Nir Dothan <ndothan@redhat.com>

Coverage Stats

55 of 64 new or added lines in 4 files covered. (85.94%)

125 existing lines in 3 files now uncovered.

11757 of 13780 relevant lines covered (85.32%)

2.27 hits per line