Bug #42113
open
ceph -h usage should indicate CephChoices --name= is sometime required
Added by Sage Weil over 4 years ago.
Updated over 4 years ago.
Description
orchestrator device ls [<host>...] [json|plain] [--refresh] List devices on a node
but 'bin/ceph orchestrator device ls gnit json' results in
RuntimeError: Host(s) 'json' not registered
The help should show '[--format=json|plain]' because the previous position is an 'n=N' arg.
- Translation missing: en.field_tag_list set to low-hanging-fruit
- Priority changed from High to Normal
No failures so this is normal priority?
Can't seem to assign this to myself. But I have the first version of the patch coming.
- Status changed from 12 to Fix Under Review
- Pull request ID set to 31042
@Matthew thanks for your contribution! the reason why you could not assign this ticket is that you are not listed in "Developer" of Ceph project, see https://tracker.ceph.com/projects/ceph .
- Has duplicate Bug #40801: ceph orchestrator cli alleges support for json-pretty, xml, xml-pretty formats added
Marked #40801 as duplicate. Actually #40801 was first, but got little attention.
Sebastian Wagner wrote:
Marked #40801 as duplicate. Actually #40801 was first, but got little attention.
From what I can see #40801 is slightly different. In that if you do send an '--format=non-existant-format' it will then five you format options that aren't supported, or if they are not mentioned in the 'help'.
I'll try and address both in the same patch (or is that too much scope creep?)
Matthew Oliver wrote:
I'll try and address both in the same patch (or is that too much scope creep?)
Seems the full list is more prevelent and may be coming from outside the python code. The full -f list is in the `--help`. So suddenly isn't as such an obvious fix. But if we want to add them both together, I'll continue down the rabbit hole. (as at least it helps teach me to navigate different parts of the codebase) :P
Also available in: Atom
PDF