-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add two more validators, providing four nodes in a devnet. #383
Conversation
i want to add a test first to this branch before merge and approval
This command makes it possible to use files located in the `./dev` directory from within the Docker containers, enabling the option of loading a custom chain spec file for testing.
@vitropy are you leading this PR? Checklist before this is ready:
|
I don't know what that means. Currently the state of this PR is blocked by needing a new chain spec file that adds validators to a new chain spec file intended for the purpose of starting a genesis block on a chain with four validators or else the ability to programmatically add validators after initialization of the development network as discussed in your team's development channel. Neither of these are things I know how to do. |
@vitropy you opened this PR... but also Frankie committed to it. What I was pointing to with the "leading this PR" request is that from prior experience on other teams, if it's unclear who is doing a task / is responsible for getting it done, then it is more inclined to slip through the cracks and get forgotten. The culture I've found useful in the past is to make it clear who the leader / main backstop person is. We can make up whatever protocols we want of course! What maybe I could have done is asked instead:
tone: not blaming, just seeking clarity, totally fine with you saying "I am handing this over", or sharing whatever your experience + culture/norms are 🦩 🍍 🐟 |
Like most things, this is a collaborative effort. The effort's current status is described in the most recent comments; I've left a status update in my most recent comment.
I think using the assignee field to communicate who is currently deemed responsible for moving the effort forward is a good practice.
See my most recent comment.
The answer to this should be in the assignee field, IMHO. Currently it's just me. I'm okay with this but need support from @entropyxyz/core-developers at the moment, and I've put this on hold until I can coordinate a time with them in between my other issue and PR assignments. If someone wants to jump in and help here, I think a sensible way to communicate that intent is to self-assign this PR.
Yes. Again, see my most recent comment. |
Created a chain type to support the docker file, I guess next steps here are to check (test this docker file) |
Yeah I know, just trying to figure out what our patterns of collaboration look like. I've been caught out before assuming "sensible process". e.g. Assignee field is a tool, AND not everyone uses it the same. I think what you've proposed is good and clear. I've added a little help by
I'll check in each week to see if there's anything that I can help with - you're assigned here but this is our lounge and want it to go well for you ❤️ |
@frankiebee can you process this <3 |
Closing this due to inactivity. |
- ${PWD}/dev:/srv/entropy/dev | ||
command: | ||
- "--chain" | ||
- "devnet-local-four-nodes" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all these need to change to devnet-local
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems healthy? I'm not sure how to assign a validator to a signing group but I don't think we did anything special to make that happen with a two-node setup or if that's required with this one. The logs do look healthy, though. Examine the logs with a command such as this:
Use this file in CI or in your workflow with a command like: