Skip to content

node: use a single node-to-node connection for everything#4045

Draft
roman-khimov wants to merge 1 commit into
masterfrom
single-grpc-client
Draft

node: use a single node-to-node connection for everything#4045
roman-khimov wants to merge 1 commit into
masterfrom
single-grpc-client

Conversation

@roman-khimov

Copy link
Copy Markdown
Member

Effectively reverts 597ed18 and dea9713, mentioned problems either were solved some time ago in a different manner or were incorrectly interpreted in the first place. Currently we're all HTTP/2 with GRPC, this should be leveraged.

Effectively reverts 597ed18 and
dea9713, mentioned problems either were
solved some time ago in a different manner or were incorrectly interpreted
in the first place. Currently we're all HTTP/2 with GRPC, this should be
leveraged.

Signed-off-by: Roman Khimov <roman@nspcc.ru>
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 26.78%. Comparing base (dca8bd2) to head (ccc249e).

Files with missing lines Patch % Lines
cmd/neofs-node/object.go 0.00% 6 Missing ⚠️
cmd/neofs-node/config.go 0.00% 2 Missing ⚠️
cmd/neofs-node/reputation.go 0.00% 2 Missing ⚠️
cmd/neofs-node/netmap.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4045   +/-   ##
=======================================
  Coverage   26.77%   26.78%           
=======================================
  Files         688      688           
  Lines       46661    46646   -15     
=======================================
  Hits        12493    12493           
+ Misses      33036    33021   -15     
  Partials     1132     1132           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant