[libvirt] [PATCH v2 0/6] Allow adding mountOpts to the storage pool mount command
John Ferlan
jferlan at redhat.com
Wed Jan 9 17:31:02 UTC 2019
On 1/9/19 12:09 PM, Daniel P. Berrangé wrote:
> On Tue, Jan 08, 2019 at 12:52:20PM -0500, John Ferlan wrote:
>> v1: https://www.redhat.com/archives/libvir-list/2018-December/msg00558.html
>>
>> Kept the subject the same, but the concept has been adjusted to follow
>> issues pointed out by jtomko vis-a-vis allowing arbitrary options via XML.
>> This series adds both the NFS and the RBD adjustments that were essentially
>> in the referenced series from review.
>
> Can you give any info about what motivated this addition.
There's a bz:
https://bugzilla.redhat.com/show_bug.cgi?id=1584663
which along the way has had private comments - I probably should add the
bz to at least one of the patches, but I think because of the (probably
unnecessary) private comments in the bz, I was hesitant to do so.
I tried a different mechanism and Jano pointed out:
https://www.redhat.com/archives/libvir-list/2018-December/msg00667.html
that previous attempts were rejected due to the arbitrary text.
>
> Anything that is implemented via a separate XML namespace is
> generally considered
>
> "unsupported, you're on your own when it breaks"
>
> so if this feature is neeeded for any management application like
> oVirt, OpenStack, etc, then using a custom namespace is not going
> to be a suitable approach.
And I don't think it would be palatable to add N 'text' options to the
XML that map to the same N 'text' options in the mount command. So this
was the next best option as far as I saw it. That'd be one of those
never ending tasks to add the favorite option.
John
More information about the libvir-list
mailing list