[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 3/4] qmp: add monitor command to
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 3/4] qmp: add monitor command to add/remove a child |
Date: |
Thu, 08 Oct 2015 10:29:53 +0200 |
User-agent: |
Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu) |
On Thu 08 Oct 2015 08:15:25 AM CEST, Markus Armbruster wrote:
>> For the second point, you should also consider how useful this
>> feature is to management tools. Just being able to remove and attach
>> children from a quorum node seems very useful on its own. I don't see
>> why we should wait for having support for other block drivers; also,
>> for most block drivers there is no meaningful way of adding or
>> removing children as nicely as that is possible for quorum.
>
> Okay, this is an argument I might be able to buy.
Note that if we want to make this interface stable there's one use case
missing: there's currently no way to change the vote threshold.
This is maybe not so important for the COLO use case, but for the
general case of adding and removing children from a quorum node having
the possibility to change the threshold makes a lot of sense.
That would probably require a its own API ('quorum-set-threshold' or
something like that) so I don't think it has an effect on these
child-add and child-del commands, but I wanted to mention it here anyway
in case someone sees something that I'm overlooking.
Berto
- Re: [Qemu-block] [PATCH v5 3/4] qmp: add monitor command to add/remove a child, Max Reitz, 2015/10/08
- Re: [Qemu-block] [Qemu-devel] [PATCH v5 3/4] qmp: add monitor command to add/remove a child, Markus Armbruster, 2015/10/09
- [Qemu-block] Dynamic reconfiguration (was: qmp: add monitor command to add/remove a child), Kevin Wolf, 2015/10/08
- Re: [Qemu-block] Dynamic reconfiguration (was: qmp: add monitor command to add/remove a child), Kevin Wolf, 2015/10/09
- Re: [Qemu-block] [Qemu-devel] Dynamic reconfiguration, Markus Armbruster, 2015/10/21
- Re: [Qemu-block] [Qemu-devel] Dynamic reconfiguration, Wen Congyang, 2015/10/25
- Re: [Qemu-block] [Qemu-devel] Dynamic reconfiguration, Markus Armbruster, 2015/10/26
- Re: [Qemu-block] [Qemu-devel] Dynamic reconfiguration, Wen Congyang, 2015/10/26
Re: [Qemu-block] [Qemu-devel] [PATCH v5 3/4] qmp: add monitor command to add/remove a child, Max Reitz, 2015/10/10