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

add n to sync node config #1050

Closed
olehnikolaiev opened this issue Feb 28, 2024 · 2 comments · Fixed by #1051, #1053, #1059 or skalenetwork/skale-node#595
Closed

add n to sync node config #1050

olehnikolaiev opened this issue Feb 28, 2024 · 2 comments · Fixed by #1051, #1053, #1059 or skalenetwork/skale-node#595
Assignees
@badrogger
Labels
bug Something isn't working epic:archive-node
Milestone
SKALE 2.3

Comments

@olehnikolaiev
Copy link
Contributor

add n - number of the nodes in schain to the wallets / ima config section. this field is needed by oracle-verifier contract to read number of the nodes in schain
@DmytroNazarenko DmytroNazarenko added the bug Something isn't working label Feb 28, 2024
@DmytroNazarenko DmytroNazarenko modified the milestones: SKALE 2.4, SKALE 2.3 Feb 28, 2024
@PolinaKiporenko PolinaKiporenko moved this to In Progress in SKALE Engineering 🚀 Feb 28, 2024
@PolinaKiporenko
Copy link

related to skalenetwork/skaled#1836

@PolinaKiporenko PolinaKiporenko mentioned this issue Feb 28, 2024
Closed
@badrogger badrogger linked a pull request Feb 28, 2024 that will close this issue
Add n to sync node config ima section #1051
Merged
@DmytroNazarenko DmytroNazarenko moved this from In Progress to Code Review in SKALE Engineering 🚀 Feb 29, 2024
@DmytroNazarenko DmytroNazarenko moved this from Code Review to Ready For Release Candidate in SKALE Engineering 🚀 Feb 29, 2024
This was linked to pull requests Feb 29, 2024
Sync node stable (hotfix for 2.3 release) #1053
Merged
Update admin to 2.11.0-sync-node.4 (2.3 hotfix) skalenetwork/skale-node#591
Merged
@badrogger badrogger linked a pull request Mar 7, 2024 that will close this issue
Fix n value in wallets ima current node section #1059
Merged
@OleksanderSalamatov
Copy link

OleksanderSalamatov commented Mar 13, 2024 •
edited
Loading

admin:2.11.0-sync-node.5

n is now present in archive node schain config and is correct value

            "wallets": {
                "ima": {
                    "n": 16
                }
            },

Can be closed

@badrogger badrogger removed a link to a pull request Mar 13, 2024
Update admin to 2.11.0-sync-node.4 (2.3 hotfix) skalenetwork/skale-node#591
Merged
@badrogger badrogger linked a pull request Mar 13, 2024 that will close this issue
Update admin to 2.11.0-sync.5 skalenetwork/skale-node#595
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@badrogger badrogger

Labels
bug Something isn't working epic:archive-node
Projects
SKALE Engineering 🚀
Status: Ready For Release Candidate
Milestone
SKALE 2.3
5 participants
@badrogger @DmytroNazarenko @olehnikolaiev @OleksanderSalamatov @PolinaKiporenko