adm: morph init error: `rpc-endpoint` and `local-dump` flags are mutually exclusive #244

New issue

Closed

opened 2023-04-12 17:09:05 +00:00 by anikeev-yadro · 2 comments

anikeev-yadro commented

2023-04-12 17:09:05 +00:00

Member

Expected Behavior

Command should initialize wallets

Current Behavior

Command returns error:

anikeev@NB-1670:~/neofs$ frostfs-adm --config frostfs-dev-env/frostfs-adm.yml morph init --alphabet-wallets frostfs-dev-env/services/ir --contracts frotfs-dev-env/vendor/contract --local-dump frostfs-dev-env/vendor/morph_chain.gz --protocol frostfs-dev-env/services/morph_chain/protocol.privnet.yml
Error: initialization error: `rpc-endpoint` and `local-dump` flags are mutually exclusive

Your Environment

devenv

## Expected Behavior Command should initialize wallets ## Current Behavior Command returns error: ``` anikeev@NB-1670:~/neofs$ frostfs-adm --config frostfs-dev-env/frostfs-adm.yml morph init --alphabet-wallets frostfs-dev-env/services/ir --contracts frotfs-dev-env/vendor/contract --local-dump frostfs-dev-env/vendor/morph_chain.gz --protocol frostfs-dev-env/services/morph_chain/protocol.privnet.yml Error: initialization error: `rpc-endpoint` and `local-dump` flags are mutually exclusive ``` ## Your Environment devenv

anikeev-yadro added the

triage

label 2023-04-12 17:09:05 +00:00

anikeev-yadro commented

2023-04-12 17:10:26 +00:00

Author

Member

Issue appeared while I testing #48

Issue appeared while I testing https://git.frostfs.info/TrueCloudLab/frostfs-node/issues/48

fyrchik added this to the v0.37.0 milestone 2023-04-13 08:28:46 +00:00

fyrchik added the

frostfs-adm

labels 2023-04-14 06:48:44 +00:00

fyrchik commented

2023-04-14 06:52:44 +00:00

Owner

So we have --local-dump which is there in init but can also be used via config in other commands. The initial vision was that we should be able to use a local dump everywhere where an endpoint is present.

I propose to remove --local-dump from the configuration and prioritize it if it is used as a CIL flag explicitly. We can avoid removing it from the configuration, but this will only complicate things.

So we have `--local-dump` which is there in `init` but can also be used via config in other commands. The initial vision was that we should be able to use a local dump _everywhere_ where an endpoint is present. I propose to remove `--local-dump` from the configuration and prioritize it if it is used as a CIL flag explicitly. We can avoid removing it from the configuration, but this will only complicate things.

fyrchik removed the

label 2023-04-14 06:52:51 +00:00

fyrchik added the

label 2023-04-14 09:11:46 +00:00