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

rados: optionally support reading omap key from file #12286

Merged
merged 1 commit into from Dec 15, 2016

Conversation

dillaman
Copy link

@dillaman dillaman commented Dec 2, 2016

Fixes: http://tracker.ceph.com/issues/18123
Signed-off-by: Jason Dillaman dillaman@redhat.com

@tchaikov
Copy link
Contributor

tchaikov commented Dec 5, 2016

@dillaman could you update doc/man/8/rados.rst accordingly?

@dillaman
Copy link
Author

dillaman commented Dec 5, 2016

@tchaikov updated

Dump the hexadecimal value of key in the object map of object name.

:command:`setomapval` *name* *key* *value*
:command:`setomapval` [ --omap-key-file *file* ] *name* *key* *value*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the man page and usage text don't clarify how to set a binary value - it reads from stdin if you omit the value argument

@tchaikov tchaikov removed the needs-doc label Dec 7, 2016
Fixes: http://tracker.ceph.com/issues/18123
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
@dillaman
Copy link
Author

dillaman commented Dec 7, 2016

updated

@liewegas liewegas merged commit 61e592a into ceph:master Dec 15, 2016
@dillaman dillaman deleted the wip-18123 branch December 15, 2016 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants