Re: [PATCH] Rbd: add listwatchers command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



My thoughts, 'rbd info' is intended to describe
the STATIC, intrinsic properties of an image, while
the watchers information are dynamic, something like
'rbd showmapped', what is your opinion?

Cheers,
Li Wang

On 2015/1/22 9:35, Sage Weil wrote:
On Wed, 21 Jan 2015, Li Wang wrote:
Currently, RBD does not provide an easy way to
consult who opened a specied image, this complicates
the cloud maintenance, sometimes the administrator
found that a RBD image could not be deleted with
an error 'image has watchers', but no further
information available. The RADOS has a command to
show the watchers of an object, but the commands
at RADOS level are not supposed to be exposed to a
cloud administrator, and it is not clear for a
non-expert administrator to how to associate an
opened RBD with a corresponding watched object.
This motivates us to add an RBD command to list the
information of the clients who opened a RBD image to
facilitate the cloud maintenance.

Perhaps this should just be rolled into the existing 'rbd info' command.
More people will see it and there's no additional cost over what info is
already doing...

sage






The patches are at:
https://github.com/ceph/ceph/pull/3435

Yunchuan Wen (3):
   rbd: add listwatchers command to list the watchers of an image
   test: add rbd listwatchers to src/test/cli/rbd/help.t
   man: add rbd listwatchers to doc/man/8/rbd.rst

  doc/man/8/rbd.rst       |  3 +++
  src/rbd.cc              | 63 +++++++++++++++++++++++++++++++++++++++++++++++--
  src/test/cli/rbd/help.t |  1 +
  3 files changed, 65 insertions(+), 2 deletions(-)

--
1.9.1


--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux