Re: [Qemu-devel] [PATCH] rbd: reload ceph config for block device

[Eric Blake <eblake@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1161 bytes --]

On 06/17/2016 08:28 PM, Vaibhav Bhembre wrote:
> This patch adds ability to reload ceph configuration for an attached RBD
> block device. This is necessary for the cases where rebooting a VM and/or
> detaching-reattaching a RBD drive is not an easy option.
> 
> The reload mechanism relies on the bdrv_reopen_* calls to provide a transactional

> +++ b/qapi-schema.json
> @@ -4308,3 +4308,16 @@
>  # Since: 2.7
>  ##
>  { 'command': 'query-hotpluggable-cpus', 'returns': ['HotpluggableCPU'] }
> +
> +##
> +# @reload-rbd-config
> +#
> +# Reload the ceph config for a given RBD block device attached to the VM.
> +#
> +# @device: Name of the device.
> +#
> +# Returns: nothing on success.
> +#
> +# Since: 2.5

You've missed 2.5 by a long shot. This should be 2.7.

> +##
> +{'command': 'reload-rbd-config', 'data': { 'device': 'str' } }

Would 'node' be a better name than 'device'? Isn't this really reloading
the state of an arbitrary BDS node, regardless of whether (or even if)
there is a BB device using that BDS?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]