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.

Sign up for GitHub

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

Jump to bottom

scripts: add script for containers and netmap entries comparison #399

Merged
merged 1 commit into from
Jun 10, 2024
Merged

scripts: add script for containers and netmap entries comparison #399

merged 1 commit into from
Jun 10, 2024

Conversation

AnnaShaleva
Copy link
Member

May be used to check data consistency between different chain versions.

@AnnaShaleva AnnaShaleva force-pushed the compare-fscontent branch 2 times, most recently from 4cf8fba to ab5082a Compare May 21, 2024 17:57
@AnnaShaleva AnnaShaleva added this to the v0.20.0 milestone May 21, 2024
@AnnaShaleva AnnaShaleva mentioned this pull request May 21, 2024
Closed
roman-khimov
roman-khimov previously approved these changes May 21, 2024
View reviewed changes
@AnnaShaleva AnnaShaleva mentioned this pull request May 27, 2024
Closed
@AnnaShaleva AnnaShaleva requested a review from carpawell June 3, 2024 09:18
cthulhu-rider
cthulhu-rider previously approved these changes Jun 3, 2024
View reviewed changes
scripts/compare-fscontent/compare-fscontent.go Outdated Show resolved Hide resolved
scripts/compare-fscontent/compare-fscontent.go Outdated Show resolved Hide resolved
@AnnaShaleva AnnaShaleva dismissed stale reviews from cthulhu-rider and roman-khimov via a85cc0a June 3, 2024 10:00
@AnnaShaleva AnnaShaleva force-pushed the compare-fscontent branch 2 times, most recently from a85cc0a to 0317a89 Compare June 3, 2024 10:28
@AnnaShaleva
scripts: add script for containers and netmap entries comparison
9b4d6b9 
May be used to check data consistency between different chain versions.

Signed-off-by: Anna Shaleva <[email protected]>
@AnnaShaleva
Copy link
Member Author

Here's an updated example of the resulting behaviour:

anna@kiwi:~/Documents/GitProjects/nspcc-dev/neofs-contract$ go run scripts/compare-fscontent/compare-fscontent.go --ignore-height https://rpc1.morph.fs.neo.org:40341 https://rpc1.morph.t5.fs.neo.org:51331
RPC https://rpc1.morph.fs.neo.org:40341 height: 5793115
RPC https://rpc1.morph.t5.fs.neo.org:51331 height: 3674001
number of containers mismatch: 175 vs 208; number of netmap entries mismatch: 5 vs 4
exit status 1

@AnnaShaleva
Copy link
Member Author

Can we merge it? It's ready and no more issues left.

@roman-khimov roman-khimov merged commit d50c8e0 into master Jun 10, 2024
10 checks passed
@roman-khimov roman-khimov deleted the compare-fscontent branch June 10, 2024 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cthulhu-rider cthulhu-rider cthulhu-rider approved these changes

@carpawell carpawell carpawell approved these changes

@roman-khimov roman-khimov Awaiting requested review from roman-khimov roman-khimov is a code owner

@AliceInHunterland AliceInHunterland Awaiting requested review from AliceInHunterland

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.20.0
Development

Successfully merging this pull request may close these issues.
4 participants
@AnnaShaleva @roman-khimov @carpawell @cthulhu-rider