-
Notifications
You must be signed in to change notification settings - Fork 238
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
chore: add Docker Usage Instructions to README.md #1424
base: master
Are you sure you want to change the base?
Conversation
LGTM |
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.
Nice! 💯
Just a few questions/comments to try to make this as simple as possible!
|
||
### Updating | ||
|
||
Run `docker stop avago; docker rm avago;` then start a new container with the latest version tag in your `docker run` command. |
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.
nit (or maybe &&
is more appropriate)
Run `docker stop avago; docker rm avago;` then start a new container with the latest version tag in your `docker run` command. | |
Run `docker stop avago; docker rm avago` then start a new container with the latest version tag in your `docker run` command. |
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.
I would keep ;
for idempotency. It doesn’t make an actual difference since docker stop
always returns zero, but ;
makes a clear statement: "all commands will execute no matter what."
- `AVAGO_HTTP_PORT` is set to `8080` instead of `9650` to avoid conflicts with the validator. | ||
- `AVAGO_HTTP_ALLOWED_HOSTS` and `AVAGO_HTTP_HOST` are required to allow the RPC server to be accessed from outside. You'll need to secure it with HTTPS; Caddy is recommended. | ||
|
||
RPC example uses another folder `~/.avalanchego_rpc` to avoid conflicts with the validator if you want to run both on the same machine. |
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.
curious is it common / recommended to run both on the same machine?
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.
I don’t see why not. It’s a trusted binary, so Docker-level isolation should be enough, as long as you have sufficient resources. Subnet-EVM without X and C chains doesn’t require much. Would I run it on the same machine for DFK? Probably not. But for 95% of other networks, especially on Fuji, I don’t see an issue.
Actually, that raises another question—why is it not recommended to run the same node as both a validator and a public RPC node?
Co-authored-by: Quentin McGaw <[email protected]> Signed-off-by: containerman17 <[email protected]>
Why this should be merged
The README.md currently lacks instructions on running subnet-evm using Docker. Adding a Docker section will help users set up and configure subnet-evm easily in containerized environments.
How this works
Added a new Run in Docker section to the README.md with brief instructions on configuration, data persistence, updating, networking, and example setups for both a Fuji subnet validator and RPC server.
How this was tested
I tested the examples manually.
Need to be documented?
No additional documentation is needed; all information is included in the updated README.md.
Need to update RELEASES.md?
No update to RELEASES.md is necessary since this is a documentation addition.