[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH 13/15] qapi-schema: Rename GlusterS
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH 13/15] qapi-schema: Rename GlusterServer to SocketAddressFlat |
Date: |
Fri, 03 Mar 2017 18:05:56 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 03/02/2017 03:44 PM, Markus Armbruster wrote:
>> As its documentation says, it's not specific to Gluster. Rename it,
>> as I'm going to use it for something else.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> block/gluster.c | 38 +++++++++++++++++++-------------------
>> qapi-schema.json | 38 ++++++++++++++++++++++++++++++++++++++
>> qapi/block-core.json | 46 +---------------------------------------------
>> 3 files changed, 58 insertions(+), 64 deletions(-)
>
> Mostly mechanical once you deal with the cross-file motion. Changing
> the type name has no impact on the wire representation, so it's safe.
>
> Reviewed-by: Eric Blake <address@hidden>
>
>> +++ b/qapi-schema.json
>> @@ -4101,6 +4101,44 @@
>> 'fd': 'String' } }
>>
>> ##
>> +# @SocketAddressFlatType:
>> +#
>> +# Available SocketAddressFlat types
>> +#
>> +# @tcp: Internet address
>> +#
>> +# @unix: Unix domain socket
>> +#
>> +# Since: 2.9
>
> I probably would have listed 'since: 2.7', since the type is unchanged
> from its pre-move location...
I'd have to bump it right in the next patch :)
>> +#
>> +# Since: 2.9
>> +##
>> +{ 'union': 'SocketAddressFlat',
>
> and again
>
>
>> +++ b/qapi/block-core.json
>> @@ -2533,50 +2533,6 @@
>> '*read-pattern': 'QuorumReadPattern' } }
>>
>> ##
>> -# @GlusterTransport:
>> -#
>> -# An enumeration of Gluster transport types
>> -#
>> -# @tcp: TCP - Transmission Control Protocol
>> -#
>> -# @unix: UNIX - Unix domain socket
>> -#
>> -# Since: 2.7
>
> ...here.
[Qemu-block] [PATCH 15/15] sheepdog: Support blockdev-add, Markus Armbruster, 2017/03/02
[Qemu-block] [PATCH 09/15] sheepdog: Implement bdrv_parse_filename(), Markus Armbruster, 2017/03/02
Re: [Qemu-block] [Qemu-devel] [PATCH 00/15] block: A bunch of fixes for Sheepdog and Gluster, Eric Blake, 2017/03/02
Re: [Qemu-block] [Qemu-devel] [PATCH 00/15] block: A bunch of fixes for Sheepdog and Gluster, Peter Maydell, 2017/03/03