Skip to content
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

--timeout not supported in control API in neofs-cli #1917

Closed
anikeev-yadro opened this issue Oct 17, 2022 · 1 comment · Fixed by #1919
Closed

--timeout not supported in control API in neofs-cli #1917

anikeev-yadro opened this issue Oct 17, 2022 · 1 comment · Fixed by #1919
Assignees
Labels
bug Something isn't working U0 Needs to be resolved immediately
Milestone

Comments

@anikeev-yadro
Copy link

anikeev-yadro commented Oct 17, 2022

According to #1837 --timeout option should be supported in neofs-cli

Steps to Reproduce (for bugs)

root@az:~# date; neofs-cli --endpoint 127.0.0.1:8091 -w /etc/neofs/storage/wallet.json control shards evacuate --all --verbose --timeout 3600
Mon Oct 17 14:55:49 UTC 2022
Error: unknown flag: --timeout
Usage:
  neofs-cli control shards evacuate [flags]

Flags:
      --address string    Address of wallet account
      --all               Process all shards
      --endpoint string   Remote node control address (as 'multiaddr' or '<host>:<port>')
  -g, --generate-key      Generate new private key
  -h, --help              help for evacuate
      --id strings        List of shard IDs in base58 encoding
      --no-errors         Skip invalid/unreadable objects
  -w, --wallet string     Path to the wallet or binary key

Global Flags:
  -c, --config string   Config file (default is $HOME/.config/neofs-cli/config.yaml)
  -v, --verbose         Verbose output

unknown flag: --timeout

Versions:

NeoFS Storage node
Version: v0.32.0-121-gade19077
GoVersion: go1.18.4

Your Environment

Server setup and configuration:
cloud, 4 VMs, 4 SN, 4 http qw, 4 s3 gw

Operating System and version (uname -a):
linux vedi 5.10.0-16-amd64 #1 SMP Debian 5.10.127-1 (2022-06-30) x86_64 GNU/Linux

@fyrchik
Copy link
Contributor

fyrchik commented Oct 17, 2022

We need this because evacuate, flush-cache and set-mode can be rather long.

aprasolova pushed a commit to aprasolova/neofs-node that referenced this issue Oct 19, 2022
Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working U0 Needs to be resolved immediately
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants