Bug #4599
closedceph auth import -i <file> is broken
0%
Description
Digging into history, but: it's not ruled out by preprocess_command, and so never gets to
the code in handle_command; when I try to make it do so, I end up in a locked-Mutex assert
that I haven't yet untangled.
Updated by Dan Mick about 11 years ago
The mutex issue was my new code. The bug is simply that import should be reenabled.
Updated by David Zafman almost 11 years ago
As of now ./ceph auth import -i appears to work, but reading from stdin doesn't:
dzafman@ubuntu:~/ceph/src$ ./ceph auth export > saveauthc*** DEVELOPER MODE: setting PYTHONPATH and LD_LIBRARY_PATH
exported master keyring
dzafman@ubuntu:~/ceph/src$ ./ceph auth import -i saveauth
- DEVELOPER MODE: setting PYTHONPATH and LD_LIBRARY_PATH
imported keyring
dzafman@ubuntu:~/ceph/src$ cat saveauth | ./ceph auth import - DEVELOPER MODE: setting PYTHONPATH and LD_LIBRARY_PATH
<<<HANGS>>>>
Updated by Dan Mick almost 11 years ago
Did it ever work from stdin? I don't see any code in the original tool that would
do that...
Updated by Dan Mick almost 11 years ago
OK. I believe it never worked with stdin, but David points out:
1) the help could be more clear about that, and
2) if -i <file> isn't given, auth import says it imported a keyring of nothing, which is
a bit confusing
3) if stdin is supplied via pipe, ceph hangs. This may not be auth import specific, but
I don't understand it yet.
Updated by Dan Mick almost 11 years ago
- Status changed from 12 to Fix Under Review
- Assignee changed from Dan Mick to David Zafman
Updated by Dan Mick almost 11 years ago
Fix pushed to wip-4599; David, can you have a look?
Updated by Dan Mick almost 11 years ago
- Status changed from Fix Under Review to 7
- Assignee changed from David Zafman to Dan Mick