Hi All,
I know Yunhong Jiang and Daniel Berrange have been involved in the following,
but I thought it worth sending to the list for visibility.
While writing code to convert the resource tracker to use the ComputeNode
object realized that the api samples used in the functional tests are not the
same as the format as the PciDevicePool object. For example:
hypervisor-pci-detail-resp.json has something like this:
"os-pci:pci_stats": [
{
"count": 5,
"extra_info": {
"key1": "value1",
"phys_function": "[[\"0x0000\", \"0x04\", \"0x00\", \"0x1\"]]"
},
"keya": "valuea",
"product_id": "1520",
"vendor_id": "8086"
}
],
My understanding from interactions with yjiang5 in the past leads me to think
that something like this is what is actually expected:
"os-pci:pci_stats": [
{
"count": 5,
"key1": "value1",
"phys_function": "[[\"0x0000\", \"0x04\", \"0x00\", \"0x1\"]]",
"keya": "valuea",
"product_id": "1520",
"vendor_id": "8086"
}
],
This is the way the PciDevicePool object expects the data structure to be and
is also the way the libvirt virt driver creates pci device information (i.e.
without the "extra_info" key). Other than that (which is actually pretty clear)
I couldn't find anything to tell me definitively if my interpretation is
correct and I don't want to change the functional tests without being sure they
are wrong. So if anyone can give some guidance here I would appreciate it.
I separated this stuff out into a patch with a couple of other minor cleanups
in preparation for the ComputeNode change, see:
https://review.openstack.org/#/c/161843
Let me know if I am on the right track,
Cheers,
Paul
Paul Murray
Nova Technical Lead, HP Cloud
+44 117 316 2527
Hewlett-Packard Limited registered Office: Cain Road, Bracknell, Berks RG12 1HN
Registered No: 690597 England. The contents of this message and any attachments
to it are confidential and may be legally privileged. If you have received this
message in error, you should delete it from your system immediately and advise
the sender. To any recipient of this message within HP, unless otherwise stated
you should consider this message and attachments as "HP CONFIDENTIAL".
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: [email protected]?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
