kubevirt / hyperconverged-cluster-operator / 27490832997

80%

push

github

Fix 409 conflict on webhook dry-run validation of operand CRs (#4296)

The validating webhook's dry-run update of operand CRs (KubeVirt, CDI,
etc.) could fail with a 409 Conflict when the HCO reconciler was
concurrently updating the same CR. This race is more likely after an
upgrade, where the reconciler performs additional reconciliation rounds.

Add retry-on-conflict logic to the webhook's updateOperatorCr so it
re-fetches the operand and retries the dry-run update up to 3 times.
Also wrap the PatchHCO call in the RoleAggregationStrategy func-test
with Eventually to tolerate transient admission errors.


Assisted-by: Claude <noreply@anthropic.com>

Signed-off-by: Oren Cohen <ocohen@redhat.com>

15 of 21 new or added lines in 1 file covered. (71.43%)

10446 of 13009 relevant lines covered (80.3%)

2.05 hits per line