[libvirt] [PATCH v2 02/12] qemuDomainDiskChangeSupported: Deny changing reservations
John Ferlan
jferlan at redhat.com
Thu Mar 8 00:07:36 UTC 2018
On 03/06/2018 12:31 PM, Michal Privoznik wrote:
> On 03/02/2018 02:58 AM, John Ferlan wrote:
>>
>>
>> On 02/21/2018 01:11 PM, Michal Privoznik wrote:
>>> Couple of reasons for that:
>>>
>>> a) there's no monitor command to change path where the pr-helper
>>> connects to, or
>>> b) there's no monitor command to introduce a new pr-helper for a
>>> disk that already exists.
>>>
>>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>>> ---
>>> src/libvirt_private.syms | 1 +
>>> src/qemu/qemu_domain.c | 8 ++++++++
>>> src/util/virstoragefile.c | 18 ++++++++++++++++++
>>> src/util/virstoragefile.h | 2 ++
>>> 4 files changed, 29 insertions(+)
>>>
>>
>> [...]
>>
>>> index 8b4efc82d..de8974d66 100644
>>> --- a/src/qemu/qemu_domain.c
>>> +++ b/src/qemu/qemu_domain.c
>>> @@ -7480,6 +7480,14 @@ qemuDomainDiskChangeSupported(virDomainDiskDefPtr disk,
>>> CHECK_EQ(src->readonly, "readonly", true);
>>> CHECK_EQ(src->shared, "shared", true);
>>>
>>> + if (!virStoragePRDefIsEqual(disk->src->pr,
>>> + orig_disk->src->pr)) {
>>> + virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
>>> + _("cannot modify field '%s' of the disk"),
>>> + "reservations");
>>
>> Formatting problems above - the " should line up I think... also
>> "diskreservations" will look odd.
>
> That's not what would be written. This is what would be:
>
> error: cannot modify field 'reservations' of the disk
>
> or translated:
>
> fehler: Das Feld 'reservations' kann nicht geändert werden
>
> IIRC it was discussed in v1 too. The idea is to have 'reservations' not
> translated because it refers to the XML element name. However, the rest
> of the error message can be localized.
>
Oh right - I missed the pesky '%s'...
John
More information about the libvir-list
mailing list