Project

General

Profile

Actions

Feature #24232

open

Add new command ceph mon status

Added by Vikhyat Umrao almost 6 years ago. Updated 21 days ago.

Status:
Fix Under Review
Priority:
Normal
Assignee:
-
Category:
Administration/Usability
Target version:
-
% Done:

0%

Source:
Community (user)
Tags:
Backport:
Reviewed:
Affected Versions:
Component(RADOS):
Monitor
Pull request ID:

Description

Add new command ceph mon status
For more information please check - https://tracker.ceph.com/issues/24217

Changed the tracker title and description. For more information check comments.

Actions #1

Updated by Joao Eduardo Luis almost 6 years ago

  • Project changed from Ceph to RADOS
  • Category set to Administration/Usability
  • Component(RADOS) Monitor added
Actions #2

Updated by Joao Eduardo Luis almost 6 years ago

copy/pasting from the PR opened to address this issue (https://github.com/ceph/ceph/pull/22202):

I think this is mixing two different concepts. As I see it, status provides information on the, well,
status of services; stat ought to provide something similar to stat(2) - which in our case has been
assumed to be status of the maps.
Now, I do think this is a bit confusing. However, I do not think the solution should be changing <svc>
stat to <svc> status.

Take osd stat and mon stat: the first is basically printing a summary of the osdmap; the second also
prints a summary of the monmap, but additionally provides information on quorum and election epoch.
In this case, I think mon stat would be providing more information than it should -- quorum and election
epochs refer to the service, and are ephemeral information, not related to the map.

In a nutshell, I'd rather have osd status, mon status, rgw status and whatnot, providing the status of
the current service, with information that is relevant to know about their current status, and have something
else (probably not named stat to reduce the confusion) to show map infos -- for instance,
ceph osd map get --summary or something (don't particularly like this either tbh).
Actions #3

Updated by Vikhyat Umrao almost 6 years ago

Sorry for the confusion, I did not check that we have ceph osd stat and ceph mon stat has the same purpose. I wanted a command which can give all monitor daemon status. Let me update the tracker. I completely agree with @jecluis. But for now, I think we do not have to rename this command we should get a new command - ceph mon status.

Hence changing the tracker title.

Actions #4

Updated by Vikhyat Umrao almost 6 years ago

  • Subject changed from Rename ceph mon stat to ceph mon status to Add new command ceph mon status
  • Description updated (diff)
Actions #5

Updated by Joao Eduardo Luis almost 6 years ago

added a card to the backlog: https://trello.com/c/PTgwBpmx

Actions #7

Updated by Nathan Cutler over 5 years ago

  • Status changed from New to Fix Under Review
Actions #8

Updated by Konstantin Shalygin 21 days ago

  • Source set to Community (user)
  • Pull request ID set to 23525
Actions

Also available in: Atom PDF