[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 08/11] rbd: Revert -blockdev and
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 08/11] rbd: Revert -blockdev and -drive parameter auth-supported |
Date: |
Tue, 28 Mar 2017 10:24:04 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 03/27/2017 08:26 AM, Markus Armbruster wrote:
>> This reverts half of commit 0a55679. We're having second thoughts on
>> the QAPI schema (and thus the external interface), and haven't reached
>> consensus, yet. Issues include:
>>
>
>> Let's avoid painting ourselves into a corner now, and revert the
>> feature for 2.9.
>
> There may still be some tweaks to improve the commit message and/or code
> comments to clarify things that tripped me up until later in the
> subthread, but now that I understand the difference between pseudo-file
> format (where the key-value pair backdoor still works for libvirt's
> usage of -drive file=rbd:...) and QemuOpts format (-drive driver=rbd,...
> which didn't really exist in 2.8, and where we don't want to bake in
> something we don't like in 2.9), I agree with the move.
I'm inserting ...
>> Note that users can still configure authentication methods with a
>> configuration file. They probably do that anyway if they use Ceph
>> outside QEMU as well.
... this hint right here:
Further note that this doesn't affect use of key "auth-supported" in
-drive file=rbd:...:key=value.
Good enough?
>> qemu_rbd_array_opts()'s parameter @type now must be RBD_MON_HOST,
>> which is silly. This will be cleaned up shortly.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> block/rbd.c | 31 +++----------------------------
>> qapi/block-core.json | 24 ------------------------
>> 2 files changed, 3 insertions(+), 52 deletions(-)
>>
>
> Reviewed-by: Eric Blake <address@hidden>
Thanks!
- Re: [Qemu-devel] [PATCH RFC v3 for-2.9 02/11] rbd: Fix to cleanly reject -drive without pool or image, (continued)
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 08/11] rbd: Revert -blockdev and -drive parameter auth-supported, Eric Blake, 2017/03/27
Re: [Qemu-devel] [PATCH RFC v3 for-2.9 08/11] rbd: Revert -blockdev and -drive parameter auth-supported, Jeff Cody, 2017/03/27