bocharov commented on PR #400: URL: https://github.com/apache/kvrocks-controller/pull/400#issuecomment-5039041281
Addressed the Codecov patch-coverage feedback in 807123c. Added unit tests for the previously-uncovered branches: - `SyncClusterInfo` bounded retry/backoff — retry exhaustion and context-cancel during backoff. - `Cluster.SyncToNodes` — force-push to every node and node-level error propagation. - Reconcile probe error paths — node restoring-from-backup is skipped, an unreachable node increments its failure count, and an uninitialized (CLUSTERDOWN) node is force-pushed rather than counted as a failure. - The `/sync` handler's cluster-not-found path. `go test`/`go vet`/`-race` all clean locally. The workflow run is pending approval (fork PR) — would appreciate a committer approving it so CI + Codecov can re-evaluate. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
