Deploy sidechain environment with neofs-adm #212
Conversation
Essential services run chains. Between basic and essential runs adm tool can deploy environment. Signed-off-by: Alex Vanin <a.vanin@yadro.com>
These artifacts are required for environment deploy. Signed-off-by: Alex Vanin <a.vanin@yadro.com>
Makefile
Outdated
| @@ -72,6 +72,9 @@ up/basic: up/essential | |||
| .PHONY: up/essential | |||
| up/essential: get vendor/hosts | |||
| @$(foreach SVC, $(START_ESSENTIAL), $(shell docker-compose -f services/$(SVC)/docker-compose.yml up -d)) | |||
| @./vendor/neofs-adm --config services/ir/neofs-adm.yml morph init --alphabet-wallets ./services/ir --contracts vendor/contracts | |||
| @for f in $(shell ls ./services/storage/wallet*.json); do ./vendor/neofs-adm -c services/ir/neofs-adm.yml morph refill-gas --storage-wallet $${f} --gas 10.0 --alphabet-wallets services/ir; done | |||
There was a problem hiding this comment.
Can be just a glob without $(shell ls wrapping.
Also the output is 4 times Waiting for transactions to persist, could you add some logging like (Transfer GAS to wallet $${f})?
There was a problem hiding this comment.
Can you provide an example with glob, please? Not sure I understand what you mean.
There was a problem hiding this comment.
@for f in $(shell ls ./services/storage/wallet*.json); do
vs
@for f in ./services/storage/wallet*.json; do
services/ir/neofs-adm.yml
Outdated
| @@ -1,4 +1,15 @@ | |||
| rpc-endpoint: http://morph-chain.neofs.devenv:30333 | |||
| network: | |||
There was a problem hiding this comment.
neofs-adm config must be moved to repo root.
| @@ -30,7 +30,7 @@ ApplicationConfiguration: | |||
| RPC: | |||
| Enabled: true | |||
| EnableCORSWorkaround: false | |||
| MaxGasInvoke: 15 | |||
| MaxGasInvoke: 50 | |||
There was a problem hiding this comment.
Mainnet has 100. Think, we can safely mirror this.
There was a problem hiding this comment.
Good idea. Changed.
Environment deploy requires bigger GAS limit for complex contract deploy transactions. Signed-off-by: Alex Vanin <a.vanin@yadro.com>
Network section is required for environment deploy. Values are taken from neo-preodolenie. Signed-off-by: Alex Vanin <a.vanin@yadro.com>
Signed-off-by: Alex Vanin <a.vanin@yadro.com>
Signed-off-by: Alex Vanin <a.vanin@yadro.com>
Signed-off-by: Alex Vanin <a.vanin@yadro.com>
|
Btw, what about using |
It should be done in #204 |
make up/basicnow consist of three steps:Before that, scripts download neofs-adm and compiled version of contract.
❗ For CI use path variables and compile neofs-adm binary and smart contracts on a runner locally.
To deploy sidechain neofs-adm config was expand with
networksection. Feel free to suggest different defaults (used from sidechain dumps).Also
MaxGasInvokewas changed even though we decided to keep value the same in #207. Sidechain deploy is really heavy.