Skip to content

Container attributes#3733

Merged
roman-khimov merged 3 commits intomasterfrom
container-attributes
Dec 26, 2025
Merged

Container attributes#3733
roman-khimov merged 3 commits intomasterfrom
container-attributes

Conversation

@cthulhu-rider
Copy link
Contributor

@cthulhu-rider cthulhu-rider commented Dec 11, 2025

No description provided.

@cthulhu-rider cthulhu-rider force-pushed the container-attributes branch 6 times, most recently from d136d4e to 257168b Compare December 24, 2025 12:41
@codecov
Copy link

codecov bot commented Dec 24, 2025

Codecov Report

❌ Patch coverage is 1.36054% with 435 lines in your changes missing coverage. Please review.
✅ Project coverage is 25.88%. Comparing base (fafa18f) to head (0ddcf2b).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
pkg/services/container/server.go 4.80% 119 Missing ⚠️
cmd/neofs-cli/modules/container/attributes.go 0.00% 101 Missing ⚠️
...nnerring/processors/container/process_container.go 0.00% 101 Missing ⚠️
pkg/morph/event/container/notary_requests.go 0.00% 42 Missing ⚠️
pkg/innerring/processors/container/handlers.go 0.00% 20 Missing ⚠️
pkg/morph/client/container/attributes.go 0.00% 18 Missing ⚠️
pkg/innerring/processors/container/processor.go 0.00% 12 Missing ⚠️
cmd/neofs-node/container.go 0.00% 10 Missing ⚠️
cmd/neofs-cli/modules/container/root.go 0.00% 6 Missing ⚠️
pkg/innerring/processors/container/common.go 0.00% 4 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3733      +/-   ##
==========================================
- Coverage   26.13%   25.88%   -0.26%     
==========================================
  Files         658      660       +2     
  Lines       41777    42213     +436     
==========================================
+ Hits        10919    10927       +8     
- Misses      29856    30285     +429     
+ Partials     1002     1001       -1     

☔ 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 force-pushed the container-attributes branch 2 times, most recently from e0d6ff2 to 303fea6 Compare December 24, 2025 12:48
@cthulhu-rider cthulhu-rider marked this pull request as ready for review December 24, 2025 12:52
@roman-khimov
Copy link
Member

Is #3728 obsolete with this?

@cthulhu-rider cthulhu-rider force-pushed the container-attributes branch 3 times, most recently from b579875 to 87e636f Compare December 25, 2025 07:27
@roman-khimov
Copy link
Member

Conflicts now.

var tokV2 sessionv2.Token
err = tokV2.Unmarshal(v.binTokenSession)
if err == nil {
// TODO
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does @End-rey know about this TODO?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conflicting area, will definitely know

acc = notary.FakeContractAccount(mainTx.Signers[2].Account)
} else {
pubBytes, ok := vm.ParseSignatureContract(script)
pubBytes, ok := scparser.ParseSignatureContract(script)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

separate commit fixes?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's related to neo-go update

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i understand, i meant updating neo-go with fixes in a separate commit

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont need the update on its own, it came automatically

Support nspcc-dev/neofs-contract#564.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
Follow nspcc-dev/neofs-api#360.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
@roman-khimov roman-khimov merged commit e5a4d70 into master Dec 26, 2025
17 of 22 checks passed
@roman-khimov roman-khimov deleted the container-attributes branch December 26, 2025 08:48
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
When executed with `--cid` flag, printing it again makes no sense. And
given the current container mutability (via attributes, #3733),
evaluating it from the container is incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore result may be incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore it's worth keeping (ID,
container) pairs.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
When executed with `--cid` flag, printing it again makes no sense. And
given the current container mutability (via attributes, #3733),
evaluating it from the container is incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore result may be incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore it's worth keeping (ID,
container) pairs.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
When executed with `--cid` flag, printing it again makes no sense. And
given the current container mutability (via attributes, #3733),
evaluating it from the container is incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore result may be incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore it's worth keeping (ID,
container) pairs.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
When executed with `--cid` flag, printing it again makes no sense. And
given the current container mutability (via attributes, #3733),
evaluating it from the container is incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore result may be incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore it's worth keeping (ID,
container) pairs.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
When executed with `--cid` flag, printing it again makes no sense. And
given the current container mutability (via attributes, #3733),
evaluating it from the container is incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore result may be incorrect.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
Containers are mutable now (#3733), therefore it's worth keeping (ID,
container) pairs.

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
IDs can be in the dump now. If so, there is no need to calculate them.
Especially taking into account containers' mutability (#3733).

Refs #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
cthulhu-rider added a commit that referenced this pull request Jan 28, 2026
This may result in wrong ID since containers became mutable (#3733).
This also eliminates unnecessary resource consumption.

Closes #3788.

Signed-off-by: Leonard Lyubich <leonard@morphbits.io>
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