Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

netmap: split UpdateState method for notary-enabled environment #227

Merged
merged 3 commits into from
Mar 17, 2022

Conversation

fyrchik
Copy link
Contributor

@fyrchik fyrchik commented Mar 10, 2022

Fix #225 .

Split methods similar to AddPeer/Register pair from b104a2c .

The one with Peer in name should be called by a storage node similar to AddPeer/Register.

Signed-off-by: Evgenii Stratonikov evgeniy@nspcc.ru

@alexvanin
Copy link
Contributor

The one with Peer in name should be called by a storage node similar to AddPeer/Register.

While it is better naming scheme for understanding, I am afraid it will break compatibility with older storage nodes as they continue to invoke UpdateState instead of UpdatePeerState.

netmap/netmap_contract.go Outdated Show resolved Hide resolved
netmap/netmap_contract.go Outdated Show resolved Hide resolved
netmap/netmap_contract.go Outdated Show resolved Hide resolved
… environment

Split methods similar to `AddPeer`/`Register` pair from b104a2c .

Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
Similar to `UpdateState`/`UpdateStateIR` pair.

Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
@alexvanin alexvanin merged commit f6766a1 into nspcc-dev:master Mar 17, 2022
aprasolova pushed a commit to aprasolova/neofs-node that referenced this pull request Oct 19, 2022
`Register` was renamed to `AddPeerIR` for consistency with
`UpdateState` changes in
nspcc-dev/neofs-contract#227

This is protocol breaking change for notary enabled environment.
Luckily, there is no notary enabled environments anywhere except
of neofs-dev-env, so we can do such thing. We should avoid such
changes in the future, though.

Signed-off-by: Alex Vanin <alexey@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.

IR is not able to remove SN from netmap
3 participants