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

Restore Quorum documentation has incorrect cmd usage #77

Open
kelsonw1 opened this issue Sep 5, 2024 · 5 comments
Open

Restore Quorum documentation has incorrect cmd usage #77

kelsonw1 opened this issue Sep 5, 2024 · 5 comments

Comments

@kelsonw1
Copy link

kelsonw1 commented Sep 5, 2024

According to [1], it's stated to use odf mons restore-quorum c, which doesn't seem to be valid:

Management and troubleshooting tools for ODF clusters.

Usage:
  odf [command]

Available Commands:
  get         Get ODF configuration
  help        Help about any command
  purge-osd   Permanently remove an OSD from the cluster.
  set         Set ODF configuration
  subvolume   Manages subvolumes

Flags:
      --context string              Openshift context to use
  -h, --help                        help for odf
      --kubeconfig string           Openshift config path
  -n, --namespace string            Openshift namespace where the StorageCluster CR is created (default "openshift-storage")
      --operator-namespace string   Openshift namespace where the ODF operator is running

Use "odf [command] --help" for more information about a command.

Reviewing [2], we see the command should be odf restore mon-quorum <mon_id>. Is the code being updated or are the docs incorrect?

[1] https://github.com/red-hat-storage/odf-cli/blob/main/docs/mons.md#restore-quorum
[2]https://github.com/red-hat-storage/odf-cli/blob/main/cmd/odf/restore/mon_quorum.go

@subhamkrai
Copy link
Contributor

@kelsonw1 I think you were trying tool version 4.16. Is that correct?

@kelsonw1
Copy link
Author

kelsonw1 commented Sep 6, 2024

@subhamkrai I was looking at the main branch code and docs I also repulled main and have the same issue. I noticed a difference in how the command was documented vs how it was coded.

@subhamkrai
Copy link
Contributor

@kelsonw1 did you rebuild?

srai@fedora ~ (main) $ ./bin/odf 
Management and troubleshooting tools for ODF clusters.

Usage:
  odf [command]

Available Commands:
  get         Get ODF configuration
  help        Help about any command
  maintenance Perform maintenance operation on mons and OSDs deployment by scaling it down and creating a maintenance deployment.
  operator    Calls subcommands specific to various ODF operators
  purge-osd   Permanently remove an OSD from the cluster.
  restore     
  set         Set ODF configuration
  subvolume   Manages subvolumes

Flags:
      --context string              Openshift context to use
  -h, --help                        help for odf
      --kubeconfig string           Openshift config path
  -n, --namespace string            Openshift namespace where the StorageCluster CR is created (default "openshift-storage")
      --operator-namespace string   Openshift namespace where the ODF operator is running

Use "odf [command] --help" for more information about a command.

I can see above commands

@kelsonw1
Copy link
Author

kelsonw1 commented Nov 6, 2024

No, I didn't rebuild. but I pulled the main branch and saw the same issue. I haven't tested this since I opened this issue tho.

@subhamkrai
Copy link
Contributor

No, I didn't rebuild. but I pulled the main branch and saw the same issue. I haven't tested this since I opened this issue tho.

Please build again and check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants