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/release-notes: Luminous release notes typo fixes "ceph config-key ls"->"ceph config-key list" #16330

Merged
merged 1 commit into from Jul 14, 2017

Conversation

Projects
None yet
3 participants
@scienceluo
Contributor

scienceluo commented Jul 14, 2017

"ceph config-key ls" should be "ceph config-key list"

Signed-off-by: luo.kexue@zte.com.cn

@scienceluo scienceluo changed the title from doc: Luminous release notes typo fixes to doc/release-notes: Luminous release notes typo fixes "ceph config-key ls"->"ceph config-key list" Jul 14, 2017

@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Jul 14, 2017

locally verified:

 ./bin/ceph config-key ls

no valid command found; 10 closest matches:
config-key rm <key>
config-key exists <key>
config-key list
config-key dump
config-key get <key>
config-key put <key> {<val>}
config-key del <key>
Error EINVAL: invalid command

./bin/ceph config-key list
[
    "mgr/dashboard/x/server_addr",
    "mgr/dashboard/x/server_port",
    "mgr/restful/keys/admin",
    "mgr/restful/x/crt",
    "mgr/restful/x/key",
    "mgr/restful/x/server_addr",
    "mgr/restful/x/server_port"
]
@xiexingguo

This comment has been minimized.

Member

xiexingguo commented Jul 14, 2017

Can you drop the chmod changes?

@@ -214,7 +214,7 @@ Major Changes from Kraken
fails if clients older than the specified release are currently
connected to the cluster.
- ``ceph config-key dump`` dumps config-key entries and their
contents. (The exist ``ceph config-key ls`` only dumps the key
contents. (The exist ``ceph config-key list`` only dumps the key

This comment has been minimized.

@tchaikov

tchaikov Jul 14, 2017

Contributor

s/exist/existing/

This comment has been minimized.

@scienceluo

scienceluo Jul 14, 2017

Contributor

@tchaikov Done, thanks.

doc:fix release-notes
Signed-off-by: luo.kexue@zte.com.cn

@tchaikov tchaikov merged commit 5268573 into ceph:master Jul 14, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment