TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 100 Pull requests 4 Projects Releases 1 Activity Actions

[#1441] services/tree: Use grpc.WaitForReady option when creating client
All checks were successful
DCO action / DCO (pull_request) Successful in 1m32s
Details
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m11s
Details
Tests and linters / Run gofumpt (pull_request) Successful in 2m7s
Details
Vulncheck / Vulncheck (pull_request) Successful in 2m13s
Details
Build / Build Components (pull_request) Successful in 2m30s
Details
Tests and linters / gopls check (pull_request) Successful in 2m49s
Details
Tests and linters / Staticcheck (pull_request) Successful in 3m8s
Details
Tests and linters / Lint (pull_request) Successful in 3m49s
Details
Tests and linters / Tests (pull_request) Successful in 4m17s
Details
Tests and linters / Tests with -race (pull_request) Successful in 6m0s
Details

Browse source 
Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>
This commit is contained in:
Aleksey Savchuk 2024-10-23 14:04:37 +03:00
parent 43f7086251
commit 3914369e96
Signed by: a-savchuk
GPG key ID: 70C0A7FF6F9C4639
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -103,6 +103,7 @@ func (c *clientCache) dialTreeService(ctx context.Context, netmapAddr string) (*
tracing.NewStreamClientInterceptor(),
),
grpc.WithContextDialer(c.ds.GrpcContextDialer()),
grpc.WithDefaultCallOptions(grpc.WaitForReady(true)),
}
if !netAddr.IsTLSEnabled() {

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

@ -342,7 +342,9 @@ func (*Service) createConnection(a network.Address) (*grpc.ClientConn, error) {
metrics.NewStreamClientInterceptor(),
tracing_grpc.NewStreamClientInterceptor(),
),
grpc.WithTransportCredentials(insecure.NewCredentials()))
grpc.WithTransportCredentials(insecure.NewCredentials()),
grpc.WithDefaultCallOptions(grpc.WaitForReady(true)),
)
}
// ErrAlreadySyncing is returned when a service synchronization has already