`proxy-add-account` should be able to parse `--alphabet-wallets` from CLI #1400

New issue

Closed

opened 2024-09-30 11:12:50 +00:00 by potyarkin · 1 comment

potyarkin commented

2024-09-30 11:12:50 +00:00

Member

Expected Behavior

frostfs-adm morph proxy-add-account should work correctly when --alphabet-wallets location is provided via CLI.

Current Behavior

frostfs-adm morph proxy-add-account does not support --alphabet-wallets:

Error: unknown flag: --alphabet-wallets

But it does not work without that parameter:

processing error: can't initialize context: can't read alphabet wallets dir: open : no such file or directory

Adding alphabet-wallets to YAML config worked as expected.

Possible Solution

Add --alphabet-wallets flag to this subcommand, similar to many other subcommands in frostfs-adm

Steps to Reproduce (for bugs)

frostfs-adm morph proxy-add-account --account $ACCOUNT --rpc-endpoint $ENDPOINT --alphabet-wallets $ALPHABET_DIR
Command above should not fail (if valid parameter values are provided)

Context

I was trying to add an account to proxy wallet without creating a YAML config file for frostfs-adm.

Regression

No.

Your Environment

Version used: frostfs-adm v0.42.9, also reproduced with current master
Server setup and configuration: n/a
Operating System and version (uname -a): n/a

## Expected Behavior `frostfs-adm morph proxy-add-account` should work correctly when `--alphabet-wallets` location is provided via CLI. ## Current Behavior `frostfs-adm morph proxy-add-account` does not support `--alphabet-wallets`: ``` Error: unknown flag: --alphabet-wallets ``` But it does not work without that parameter: ``` processing error: can't initialize context: can't read alphabet wallets dir: open : no such file or directory ``` Adding `alphabet-wallets` to YAML config worked as expected. ## Possible Solution Add `--alphabet-wallets` flag to this subcommand, similar to many other subcommands in `frostfs-adm` ## Steps to Reproduce (for bugs) - `frostfs-adm morph proxy-add-account --account $ACCOUNT --rpc-endpoint $ENDPOINT --alphabet-wallets $ALPHABET_DIR` - Command above should not fail (if valid parameter values are provided) ## Context I was trying to add an account to proxy wallet without creating a YAML config file for frostfs-adm. ## Regression No. ## Your Environment * Version used: frostfs-adm v0.42.9, also reproduced with current [master](https://git.frostfs.info/TrueCloudLab/frostfs-node/src/commit/d0ed29b3c73626f6bf881090f86bdc834d81acc1) * Server setup and configuration: n/a * Operating System and version (`uname -a`): n/a