SchSeba / sriov-network-operator-1 / 22141382091

63%

master: 63%

Build Type

push

github

Committed by SchSeba

Commit Message

fix(daemon): include host status fields in updateSyncState skip check

The optimization to skip unnecessary API calls in updateSyncState was
only checking SyncStatus, LastSyncError, and Conditions. This caused
updates to Status.Interfaces, Status.Bridges, and Status.System to be
skipped when those fields changed but the other fields remained the same.

This broke ExternallyManaged policy validation because when VFs were
manually configured on the host, the daemon would discover the new
NumVfs value but skip updating the SriovNetworkNodeState status. The
webhook would then see stale NumVfs=0 and reject the policy.

Add equality checks for Interfaces, Bridges, and System fields to
ensure host status changes are properly persisted to the API.

Signed-off-by: Cursor AI <noreply@cursor.com>

Run Details

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

460 existing lines in 9 files now uncovered.

9647 of 15262 relevant lines covered (63.21%)

0.7 hits per line