<...snip...>
>
> Had a chance to try your patch.
>
> # cat testCaps.py
> import libvirt
> conn = libvirt.openReadOnly('qemu:///system')
> fc = conn.nodeDeviceLookupByName('scsi_host1')
> caps = fc.listCaps();
> print "The capabilities are", caps
> nCaps = fc.numOfCaps();
> print "The number of Caps : ", nCaps
>
> # ./run python testCaps.py
> The capabilities are ['scsi_host', 'fc_host', 'vports']
> The number of Caps : 3
>
Excellent - thanks... I "assume" prior to my patch the result would
just be 'scsi_host' and 1 for number of caps...
> Given the above output(also virsh behaviour), I am thinking my patch should
> actually be checking the string "vports" instead of "vport_ops".
>
> I am sending you the v2 with that change. Feel free to change the commit
> message as appropriate.
>
I can edit your existing patch in my git branch to change "vport_ops" to
"vports" - that's not a problem...
John
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list