<div dir="ltr"><div>libvirt pool dumpxml:<br><pool type='rbd'></div><div> <name>rbd.test</name></div><div> <uuid>57e55e10-63c9-4166-861c-f330d868260c</uuid></div><div> <capacity unit='bytes'>36792725520384</capacity></div><div> <allocation unit='bytes'>760411510361</allocation></div><div> <available unit='bytes'>34510461812736</available></div><div> <source></div><div> <host name='10.10.40.1' port='6789'/></div><div> <host name='10.10.40.2' port='6789'/></div><div> <host name='10.10.40.3' port='6789'/></div><div> <name>rbd.test</name></div><div> <auth type='ceph' username='admin'></div><div> <secret uuid='f65cc5a8-b77b-4254-9030-d50a528fb456'/></div><div> </auth></div><div> </source></div><div></pool></div><div><br></div><div>with this patch i get correct disk definition using virt-install:<br>...</div><div><div> <disk type='network' device='disk'></div><div> <driver name='qemu' type='raw' cache='writeback'/></div><div> <auth username='admin'></div><div> <secret type='ceph' uuid='f65cc5a8-b77b-4254-9030-d50a528fb456'/></div><div> </auth></div><div> <source protocol='rbd' name='rbd.test/50845b54-0ed6-48e9-8d84-07b2cd2c0c4d.img'></div><div> <host name='10.10.40.1' port='6789'/></div><div> <host name='10.10.40.2' port='6789'/></div><div> <host name='10.10.40.3' port='6789'/></div><div> </source></div><div> <target dev='vda' bus='virtio'/></div><div> <alias name='virtio-disk0'/></div><div> <address type='pci' domain='0x0000' bus='0x00' slot='0x07' function='0x0'/></div><div> </disk></div></div><div>...</div><div><br></div><div>Without the patch it took only one host entry and no auth element. so the domain would fail.</div><div><br></div><div>Regards,</div><div>Rauno</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 26, 2018 at 9:30 PM, Cole Robinson <span dir="ltr"><<a href="mailto:crobinso@redhat.com" target="_blank">crobinso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 01/26/2018 05:29 AM, Rauno Väli wrote:<br>
> virt-install should copy libvirt storage pool auth element and all<br>
> source hosts to domain xml<br>
><br>
<br>
</span>Thanks for the patch! Can you give me an example pool config and<br>
matching disk config it generates? So I can add a test case for this<br>
patch. Changing the auth values is fine of course<br>
<br>
Thanks,<br>
Cole<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Regards,<div>Rauno Väli</div><div>OYE Network Ltd.</div></div></div>
</div>