-
Notifications
You must be signed in to change notification settings - Fork 4
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
Chaosnet #188
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is a beta staker program for stakers willing to go the extra mile with monitoring, share their logs with the dev team, and allow to more carefully monitor the bootstrapping network. As the network matures, the beta program will be ended.
dimpar
reviewed
Sep 19, 2022
It does not hurt to add beta operators after deactivating chaosnet but for the consistency of events, we should require adding beta operators is possible only on the active chaosnet. This way, we are sure all beta operators actually participated in the chaosnet.
dimpar
reviewed
Sep 21, 2022
No more comments |
dimpar
previously approved these changes
Sep 21, 2022
We need to remember to update the #186 branch once this PR is merged. But this would cause additional work needed for the testnet with registering the beta operators. |
nkuba
reviewed
Sep 22, 2022
Made it clear that operator once added can not be removed.
nkuba
approved these changes
Sep 22, 2022
dimpar
approved these changes
Sep 22, 2022
This was referenced Sep 22, 2022
nkuba
added a commit
to keep-network/keep-core
that referenced
this pull request
Sep 23, 2022
Chaosnet: update ecdsa contracts See #3297 Updating `sortition-pools` and `random-beacon` dependencies to the ones with Chaosnet support from keep-network/sortition-pools#188. Updated deployment scripts for ecdsa contracts to transfer the chaosnet owner to a named account. Deactivated chaosnet in unit tests to avoid problems with adding operators to the pool
This was referenced Sep 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a beta staker program for stakers willing to go the extra mile with
monitoring, share their logs with the dev team, and allow to more carefully
monitor the bootstrapping network. As the network matures, the beta program will
be ended.