ale64bit/frostfs-node
1
0
Fork 0
forked from TrueCloudLab/frostfs-node
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#2162] services/tree: Close connection after the syncronization

Browse source 
There was a goroutine leak here.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2022-12-16 09:54:00 +03:00 committed by Anton Nikiforov
parent e1c3bdbfa6
commit 9e0decd12d
2 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -64,6 +64,7 @@ Changelog for NeoFS Node
- Prioritize internal addresses for clients (#2156) - Prioritize internal addresses for clients (#2156)
- Force object removal via control service (#2145) - Force object removal via control service (#2145)
- Synchronizing a tree now longer reports an error for a single-node container (#2154) - Synchronizing a tree now longer reports an error for a single-node container (#2154)
- Prevent leaking goroutines in the tree service (#2162)
### Removed ### Removed
- `-g` option from `neofs-cli control ...` and `neofs-cli container create` commands (#2089) - `-g` option from `neofs-cli control ...` and `neofs-cli container create` commands (#2089)

1
pkg/services/tree/sync.go
View file

@ -137,6 +137,7 @@ func (s *Service) synchronizeTree(ctx context.Context, d pilorama.CIDDescriptor,
// Failed to connect, try the next address. // Failed to connect, try the next address.
return false return false
} }
defer cc.Close()
treeClient := NewTreeServiceClient(cc) treeClient := NewTreeServiceClient(cc)
for { for {