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

doc: add ceph-kvstore-tool's man #17092

Merged
merged 1 commit into from Aug 29, 2017

Conversation

liuchang0812
Copy link
Contributor

Signed-off-by: liuchang0812 liuchang0812@gmail.com

@@ -0,0 +1,79 @@
orphan:
Copy link
Contributor

Choose a reason for hiding this comment

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

should be

:orphan:

Print all crc that belong to the prefix.

:command:`exists <prefix> [key]`
Query whether prefix or key is exist.
Copy link
Contributor

Choose a reason for hiding this comment

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

"exist" is a verb, not an adj.

might want to put:

Check if there is any KV pair stored with the URL encoded prefix. If key is also specified, check for the key with the prefix instead.

Synopsis
========

| **ceph-kvstore-tool** <leveldb|rocksdb|bluestore-kv> <store path> command [args...]
Copy link
Contributor

Choose a reason for hiding this comment

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

use

*command*

as it is a metavar.

===========

:program:`ceph-kvstore-tool` is a kvstore manipulation tool. It allows users to manipule
leveldb/rocksdb's data(likes OSD's omap) offline.
Copy link
Contributor

Choose a reason for hiding this comment

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

add a space before '('.

s/likes/like/

========

:program:`ceph-kvstore-tool` utility uses many commands for administration purpose
which are as follows:
Copy link
Contributor

Choose a reason for hiding this comment

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

s/administration/debugging/

@liuchang0812 liuchang0812 force-pushed the wip-man-kvstore-tool branch 3 times, most recently from a932386 to cdd0c59 Compare August 21, 2017 12:47
@ceph-jenkins
Copy link
Collaborator

make check succeeded

@liuchang0812
Copy link
Contributor Author

Using context: make check (arm64)
Could not update commit status of the Pull Request on GitHub.
java.net.SocketTimeoutException: connect timed out

@tchaikov thanks for your reviewing. updated

@liuchang0812
Copy link
Contributor Author

jenkins test this please

1 similar comment
@liuchang0812
Copy link
Contributor Author

jenkins test this please


:command:`store-copy <path> [num-keys-per-tx]`
Copy all KV pairs to another directory specified by ``path``.
[num-keys-per-tx] is the number of KV pairs coped for a transaction.

Choose a reason for hiding this comment

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

@liuchang0812 Looks copied

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thank you 👍

Signed-off-by: liuchang0812 <liuchang0812@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants