eBay / go-ovn

A Go library for OVN Northbound/Southbound DB access using native OVSDB protocol
Apache License 2.0
108 stars 59 forks source link

Add back tranmutex #153

Open trozet opened 3 years ago

trozet commented 3 years ago

Need a RWMutex to protect nil client usage during reconnect. Also includes a workaround for https://bugzilla.redhat.com/show_bug.cgi?id=2000375

trozet commented 3 years ago

@girishmg @hzhou8 @tssurya @dcbw

hzhou8 commented 3 years ago

Hi @trozet, the problem looks clear but I wonder if this is the right way to fix. There can be different reasons for a transaction to fail - it can be temporary. Are we going to blindly reconnect regardless of the reason?