ovn-kubernetes / libovsdb / 22687128641

48%

main: 48%

Build Type

Pull #459

github

Committed by booxter

Commit Message

client: fix SetOption() to check connected flag instead of rpcClient

TestOpsWaitForReconnect was failing because after Disconnect() is called,
the connected flag is set to false immediately, but rpcClient is only set
to nil later by handleDisconnectNotification(). This creates a window where
SetOption() would check rpcClient != nil and fail even though the client
is disconnected.

Fix by checking the connected flag instead of rpcClient in SetOption().
This is the correct check since connected accurately reflects the client
state from the caller's perspective.

Fixes TestOpsWaitForReconnect integration test.

Pull Request Pull Request #459: client: fix send on closed channel panic during disconnect

Run Details

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

8 existing lines in 1 file now uncovered.

6166 of 12803 relevant lines covered (48.16%)

171.81 hits per line