-
Notifications
You must be signed in to change notification settings - Fork 11
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
docs: update deployment docs #2107
base: master
Are you sure you want to change the base?
Conversation
docs/hyperchains_setup.md
Outdated
docker run -it --rm \ | ||
-p 3013:3013 \ | ||
-p 4000:4000 \ | ||
-e AETERNITY_CONFIG=/home/aeternity/aeternity.yaml \ | ||
-v ${PWD}/${NAME}/nodeConfig/aeternity.yaml:/home/aeternity/aeternity.yaml \ | ||
-v ${PWD}/${NAME}/nodeConfig/${NAME}_accounts.json:/home/aeternity/node/local/rel/aeternity/data/mnesia/${NAME}_accounts.json \ | ||
-v ${PWD}/${NAME}/nodeConfig/${NAME}_contracts.json:/home/aeternity/node/local/rel/aeternity/data/mnesia/${NAME}_contracts.json \ | ||
-v ${PWD}/data/mnesia:/home/aeternity/node/local/rel/aeternity/data/mnesia \ | ||
-v ${PWD}/data/mdw.db:/home/aeternity/node/local/rel/aeternity/data/mdw.db \ | ||
-v ${PWD}/${NAME}/nodeConfig/aeternity.yaml:/home/aeternity/.aeternity/aeternity/aeternity.yaml \ | ||
-v ${PWD}/${NAME}/nodeConfig/${NAME}_accounts.json:/home/aeternity/node/data/mnesia/${NAME}_accounts.json \ | ||
-v ${PWD}/${NAME}/nodeConfig/${NAME}_contracts.json:/home/aeternity/node/data/mnesia/${NAME}_contracts.json \ | ||
-v ${PWD}/data/mnesia:/home/aeternity/node/data/mnesia \ | ||
-v ${PWD}/data/mdw.db:/home/aeternity/node/data/mdw.db \ |
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.
The change works but as this is for persistent running, to work across restarts, it could run with a named contained (without the -rm
). It would help a lot on the restart to avoid mapping all volumes again, specially to avoid conflict on the configs that was chosen for starting and should not be changed.
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.
The changes worked fine on latest. Only would remove the --rm
and use a named container.
docs/middleware_setup.md
Outdated
@@ -73,16 +73,16 @@ If you want to use a database snapshot: | |||
Start the container with the following command: | |||
|
|||
``` | |||
docker run -d \ | |||
docker run -it --rm \ |
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.
here as well
No description provided.