Re: [PATCH] storage: add RBD support to disk source pool translation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/06/2015 14:45, John Ferlan wrote:
Your cover letter indicates you didn't find any bit of documentation,
but I'll point out that the formatdomain.html.in describes the "<disk
type='volume'.../>" and how the "<source..." are described in order to
use a disk type volume..

It's a misunderstanding, I never wished to imply a lack of information
existed on how to use the feature. Actually it was the page that led
to testing type 'volume' with RBD backends and finding out it didn't
work yet. Hence supporting other pool type would not require an update
to the documentation, which was my message. Although it's true we
could mention what's not supported yet in that area.

There's no tests in this patch to "show" or "prove" that by simply
adding this code that libvirt will generate the correct qemu command in
order to find the disk and it's auth information.
[...]
That might be a good place to start to ensure you have a
way to have the domain XML recognize what it is you want and the qemu
command to include/find the disk for the domain.

Understood, working on new tests and a proper implementation.
It turns out I overlooked a lot of problems.

Curiously that bz was generated because the domain
XML didn't have the 'secrettype' defined so when formatting for a
snapshot, there was an error. (OK - so you found this already...)

Indeed. The lack of 'secrettype' prevented migrations because of this,
and your post came in timely.

Thanks John

--
Thibault

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]