Skip to content

container: make difference b/w Estimations and Reports#510

Merged
cthulhu-rider merged 1 commit intomasterfrom
fix/new-estimations-naming
Aug 26, 2025
Merged

container: make difference b/w Estimations and Reports#510
cthulhu-rider merged 1 commit intomasterfrom
fix/new-estimations-naming

Conversation

@carpawell
Copy link
Member

After 38ac6f6 there is clear difference b/w what is container's size estimation and what have been reported by nodes earlier, make it also clearer in contract's API. Refs #501.

After 38ac6f6 there is clear difference b/w
what is container's size estimation and what have been reported by nodes
earlier, make it also clearer in contract's API. Refs #501.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@carpawell carpawell self-assigned this Aug 25, 2025
@carpawell carpawell marked this pull request as ready for review August 25, 2025 12:29
@carpawell carpawell requested a review from End-rey August 25, 2025 12:29
@codecov
Copy link

codecov bot commented Aug 25, 2025

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.22%. Comparing base (c8cfdf7) to head (e5a8378).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
contracts/container/contract.go 33.33% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #510   +/-   ##
=======================================
  Coverage   62.22%   62.22%           
=======================================
  Files          23       23           
  Lines        3328     3328           
=======================================
  Hits         2071     2071           
  Misses       1257     1257           

☔ View full report in Codecov by Sentry.
📢 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.

@cthulhu-rider cthulhu-rider merged commit 4e723e6 into master Aug 26, 2025
9 of 10 checks passed
@cthulhu-rider cthulhu-rider deleted the fix/new-estimations-naming branch August 26, 2025 08:50
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 10, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 11, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 12, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 15, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 16, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 17, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 17, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 19, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
carpawell added a commit to nspcc-dev/neofs-node that referenced this pull request Sep 22, 2025
Where will not be estimations in the next contract release.
Refs nspcc-dev/neofs-contract#510.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
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.

3 participants