Merge pull request #325 from stgraber/main

incusd/network/ovsdb: Properly close the connections
lxc · Dec 22, 2023 · 032e040 · 032e040
2 parents 4595146 + 697cafa
commit 032e040
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/internal/server/network/ovn/ovn_nb.go b/internal/server/network/ovn/ovn_nb.go
@@ -204,6 +204,7 @@ func NewNB(s *state.State) (*NB, error) {
 	// Set finalizer to stop the monitor.
 	runtime.SetFinalizer(client, func(o *NB) {
 		_ = ovn.MonitorCancel(context.Background(), o.cookie)
+		ovn.Close()
 	})
 
 	return client, nil

diff --git a/internal/server/network/ovn/ovn_sb.go b/internal/server/network/ovn/ovn_sb.go
@@ -197,6 +197,7 @@ func NewSB(s *state.State) (*SB, error) {
 	// Set finalizer to stop the monitor.
 	runtime.SetFinalizer(client, func(o *SB) {
 		_ = ovn.MonitorCancel(context.Background(), o.cookie)
+		ovn.Close()
 	})
 
 	return client, nil

diff --git a/internal/server/network/ovs/ovs.go b/internal/server/network/ovs/ovs.go
@@ -63,6 +63,7 @@ func NewVSwitch() (*VSwitch, error) {
 	// Set finalizer to stop the monitor.
 	runtime.SetFinalizer(client, func(o *VSwitch) {
 		_ = ovs.MonitorCancel(context.Background(), o.cookie)
+		ovs.Close()
 	})
 
 	// Get the root UUID.