doc: mgr/dashboard.rst: mention ceph.conf and ceph mgr services #20961
Conversation
| @@ -74,6 +89,9 @@ will be stored in the MON's configuration database:: | |||
| The password will be stored in the configuration database in encrypted form | |||
There was a problem hiding this comment.
Clearly, the command two lines before this comment is wrong. But I'm not sure what the correct version is. Maybe this?
ceph tell mgr.\* set-login-credentials <username> <password>
?
There was a problem hiding this comment.
It's ceph mgr dashboard set-login-credentials <username> <password> - added a commit to fix.
There was a problem hiding this comment.
@smithfarm somehow I don't seem to see this last commit in the current PR. Did you push it?
doc/mgr/dashboard.rst
Outdated
| In order to be able to log in, you need to define a username and password, which | ||
| will be stored in the MON's configuration database:: | ||
|
|
||
| ceph dashboard set-login-credentials <username> <password> | ||
| ceph mgr dashboard set-login-credentials <username> <password> |
There was a problem hiding this comment.
please remove the mgr part. The command was already correct!
smithfarm
force-pushed
the
wip-dashboard-rst
branch
from
62c8975
to
f783808
Compare
|
@rjfd Removed the defective commit. I was confused by http://tracker.ceph.com/issues/23404 and http://tracker.ceph.com/issues/23406 |
Also break down the "Configuration" section into three separate sub-sections for clarity. Signed-off-by: Nathan Cutler <ncutler@suse.com>
Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm
force-pushed
the
wip-dashboard-rst
branch
from
f783808
to
b775260
Compare
|
@LenzGr I hadn't merged this yet because I wanted to mention that |
|
jenkins render docs |
|
Doc render available at http://docs.ceph.com/ceph-prs/20961/ |
doc/mgr/dashboard.rst
Outdated
| @@ -74,6 +95,9 @@ will be stored in the MON's configuration database:: | |||
| The password will be stored in the configuration database in encrypted form | |||
| using ``bcrypt``. This is a global setting that applies to all dashboard instances. | |||
|
|
|||
| Accessing the dashboard | |||
| ^^^^^^^^^^^^^^^^^^^^^^^ | |||
|
|
|||
| You can now access the dashboard using your (JavaScript-enabled) web browser, by | |||
| pointing it to the selected TCP port and any of the host names or IP addresses | |||
There was a problem hiding this comment.
Just found one minor room for improvement: could you put "TCP port" after "host names or IP addresses", so it reads as follows: "...pointing it to any of the host names or IP addresses and the selected TCP port..." (this would spare me from opening a separate PR in order to fix that). Thanks!
There was a problem hiding this comment.
@LenzGr OK, added a commit to address this.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
Also break down the "Configuration" section into three separate sub-sections
for clarity.
Signed-off-by: Nathan Cutler ncutler@suse.com