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

mgr: print a more helpful error message for when users lack mgr ceph caps #15697

Merged
merged 1 commit into from Jun 20, 2017

Conversation

Projects
None yet
5 participants
@gregsfortytwo
Member

gregsfortytwo commented Jun 14, 2017

No description provided.

@xiexingguo

this really helps a lot 👍

@jcsp

This comment has been minimized.

Contributor

jcsp commented Jun 15, 2017

👍

@@ -25,6 +25,15 @@ of ``ceph status``, which should now include a mgr status line::
mgr active: $name
Client authentication
---------------------
The manager is a new daemon which requires new CephX capabilities. If you upgrade

This comment has been minimized.

@alfredodeza

alfredodeza Jun 16, 2017

Contributor

I don't think we should mention "new" here. When would it stop being new? In the next release? It would read fine with s/new//g here

This comment has been minimized.

@alfredodeza

alfredodeza Jun 16, 2017

Contributor

The use of "you" and "your" can be removed, doesn't add anything specific to what authentication is trying to do. It would read like:

The manager is a daemon which requires CephX capabilities. When upgrading
a cluster from an old version of Ceph, or when using the default install/deploy
tools, the admin client should get this capability automatically. Using custom
tooling may get ``EACCES`` errors when invoking certain ceph cluster commands.
To fix that, add a "mgr allow *" stanza to the client's cephx capabilities by
mgr: print a more helpful error message for when users lack mgr cephx…
… caps

Add some brief documentation on updating their caps and link to it.

Fixes: http://tracker.ceph.com/issues/20296

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
@gregsfortytwo

This comment has been minimized.

Member

gregsfortytwo commented Jun 17, 2017

Here you go @alfredodeza

@liewegas liewegas removed the needs-review label Jun 20, 2017

@liewegas liewegas merged commit 368ff5e into ceph:master Jun 20, 2017

4 checks passed

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