Skip to content

Commit

Permalink
Merge pull request ovn-org#269 from hzhou8/fix_conn_leak_at_monitor_f…
Browse files Browse the repository at this point in the history
…ailure

Fix connection leak and the cache inconsistency errors caused by it.
  • Loading branch information
dcbw authored Dec 14, 2021
2 parents ce12e45 + 83f8431 commit 4303261
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions client/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -246,6 +246,7 @@ func (o *ovsdbClient) connect(ctx context.Context, reconnect bool) error {
for id, request := range db.monitors {
err := o.monitor(ctx, MonitorCookie{DatabaseName: dbName, ID: id}, true, request)
if err != nil {
o.rpcClient.Close()
o.rpcClient = nil
return err
}
Expand Down

0 comments on commit 4303261

Please sign in to comment.