| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-11 | |||
| 08:45:52 | openstackgerrit | Yongli He proposed openstack/nova master: smartnic support https://review.opendev.org/c/openstack/nova/+/758944 | |
| 08:45:53 | openstackgerrit | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 08:48:34 | yonglihe | gibi alex_xu, sean-k-mooney, xinranwang: test cases added. | |
| 08:51:24 | yonglihe | gibi: all the problem resolved, https://review.opendev.org/c/openstack/nova/+/771363/13 | |
| 08:51:34 | gibi | reading it ... | |
| 09:42:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add note about rescuing bfv instances with the 2.87 microversion https://review.opendev.org/c/openstack/nova/+/779479 | |
| 09:43:24 | lyarwood | whops | |
| 09:47:04 | lyarwood | Type 'yes' to confirm, other to cancel: yes | |
| 09:47:05 | lyarwood | Aborting. | |
| 09:47:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add note about rescuing bfv instances with the 2.87 microversion https://review.opendev.org/c/openstack/nova/+/779479 | |
| 09:47:16 | lyarwood | >.< git-review WHY DO YOU HATE ME?! | |
| 09:48:42 | bauzas | folks, I'll resume reviewing the uefi secure boot series starting with https://review.opendev.org/c/openstack/nova/+/779302/2 | |
| 09:48:50 | bauzas | stephenfin: lyarwood: gibi: ^ | |
| 09:48:55 | stephenfin | ta | |
| 09:49:10 | gibi | ack | |
| 09:50:01 | stephenfin | lyarwood: You can probably bump your +1 to +2 on this now? https://review.opendev.org/c/openstack/nova/+/779303/ | |
| 09:50:09 | stephenfin | It failed, but not for related reasons | |
| 09:51:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add 'VDPA' to 'PciDeviceType' https://review.opendev.org/c/openstack/nova/+/777481 | |
| 09:51:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add vDPA nodedev parsing https://review.opendev.org/c/openstack/nova/+/770533 | |
| 09:51:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Move PCI host device parsing to 'host' https://review.opendev.org/c/openstack/nova/+/779851 | |
| 09:51:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add guest generation for vDPA https://review.opendev.org/c/openstack/nova/+/770532 | |
| 09:51:13 | openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Add vDPA vnic to PCI request mapping and filtering https://review.opendev.org/c/openstack/nova/+/778350 | |
| 09:51:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: add hw:mlock extra spec https://review.opendev.org/c/openstack/nova/+/778347 | |
| 09:51:27 | lyarwood | stephenfin: ack yeah sorry will do | |
| 09:52:35 | stephenfin | kashyap: Can you revisit https://review.opendev.org/c/openstack/nova/+/776681/6/nova/virt/libvirt/driver.py#5917 and see if you're happy with my response RE: the 'optional' secure boot policy? | |
| 09:52:56 | kashyap | stephenfin: Morning, will do | |
| 09:55:55 | kashyap | stephenfin: Yeah, that response is fair enough. We expect people to read the docs. We can't put a guard against every potential foot-gun | |
| 10:00:41 | stephenfin | yup, exactly | |
| 10:00:51 | kashyap | stephenfin: Also, I like the clean test in test_driver.py; thakns! | |
| 10:00:52 | stephenfin | I plan to resolve the nvram issue as a bug after feature freeze | |
| 10:01:02 | kashyap | stephenfin: Right; that's fine | |
| 10:01:05 | stephenfin | the series is already too big to bring that in, IMO | |
| 10:01:22 | kashyap | Agree | |
| 10:02:18 | kashyap | And it's a more time-taking one anyway; we shouldn't block on it for now. | |
| 10:05:23 | kashyap | Don't know where you are at this moment :-) | |
| 10:16:31 | bauzas | stephenfin: slight -1 for https://review.opendev.org/c/openstack/nova/+/779302 | |
| 10:16:48 | bauzas | I need to get my daughters from the school but please reply | |
| 10:16:53 | bauzas | and then I'll look later | |
| 10:57:58 | artom | stephenfin, gibi, with sean-k-mooney's port NUMA affinity patch on top of my socket stuff, we found an issue with PCI devices that have parents | |
| 10:58:32 | lyarwood | bauzas: stephenfin just stepped out but I replied in that change, I'm not sure what you're suggesting tbh. | |
| 10:58:50 | artom | Because I was setting 'socket' *on* the pool, it b0rked comparison for parent PFs. I've had to fix and push again: https://review.opendev.org/c/openstack/nova/+/772779 | |
| 11:00:40 | sean-k-mooney | the fix artom did was to get teh list of numa node related to the sockets the guest is on and then require the pool be in one of those numa nodes | |
| 11:01:15 | artom | Yep - that way, nothing is being set on the pool, no side effect surprises | |
| 11:01:56 | artom | It's not a radical departure from what was there before :) | |
| 11:05:06 | sean-k-mooney | hehe its litrally https://review.opendev.org/c/openstack/nova/+/772779/17..18/nova/pci/stats.py | |
| 11:05:26 | sean-k-mooney | so ya +9-14 lines | |
| 11:07:48 | lyarwood | gibi: https://review.opendev.org/c/openstack/nova/+/735947 - btw I -1'd this earlier, more for my own education and understanding as the code LGTM but the commit wasn't clear on the reasoning for this. | |
| 11:09:40 | gibi | artom: ack, looks good to me | |
| 11:10:39 | gibi | lyarwood: I have to get back to that | |
| 11:12:23 | artom | gibi, mcuh thanks! | |
| 11:12:24 | sean-k-mooney | off the top of my head i dont rememebr all the detail but the highlevle reasoning is that hyperthreading affect the entire host ans sicne cpu are part of the unname greoup these should also be. it would be invalid today to have 2 named groups one that requested HT on and the other HT off for the same host | |
| 11:12:44 | openstackgerrit | Merged openstack/nova master: apidb: Compact Rocky database migrations https://review.opendev.org/c/openstack/nova/+/759405 | |
| 11:13:45 | sean-k-mooney | i rememebr suggesting that we shoudl perhaps only ally HW_CPU_* in general in the unamed group becasue at the time i did not think there was ever a valid usecase for varing them in depently in the named groups | |
| 11:13:51 | sean-k-mooney | lyarwood:^ | |
| 11:15:11 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/735947/5/nova/api/validation/extra_specs/traits.py#26 | |
| 11:21:33 | gibi | I think I agree with sean-k-mooney here | |
| 11:21:36 | sean-k-mooney | lyarwood: actully i remember more of the context now filling it in to the patch | |
| 11:25:34 | lyarwood | k thanks I also need to drop in a min to pick up some new glasses, I'll get back to the review this afternoon | |
| 11:26:20 | sean-k-mooney | lyarwood: https://review.opendev.org/c/openstack/nova/+/735947/5//COMMIT_MSG#13 | |
| 14:01:52 | gibi | yonglihe: finshed reading the main smartnic patch I still have issues. Also I think you missed my earlier request to add a set of functional tests | |
| 14:09:04 | gibi | alex_xu: ^^ fyi | |
| 14:20:17 | gibi | is there any patch that needs my immediate attention? | |
| 14:22:42 | stephenfin | sean-k-mooney: question: is this change necessary? https://review.opendev.org/c/openstack/nova/+/770533/10/nova/virt/libvirt/host.py#1262 | |
| 14:23:10 | stephenfin | I can't figure out what uses it? I don't think we ever finished the NIC capability feature and that's the only thing that code was used for previously | |
| 14:23:29 | sean-k-mooney | the elif? | |
| 14:23:34 | sean-k-mooney | yes the elif is | |
| 14:23:49 | stephenfin | adding the 'vdpa' field to the 'capabilities' | |
| 14:23:52 | stephenfin | dic | |
| 14:23:54 | stephenfin | *dict | |
| 14:24:09 | sean-k-mooney | its used to get the path to add to the xml | |
| 14:24:11 | stephenfin | which will be saved in the 'extra_info' field of the 'PciDevice' table | |
| 14:25:02 | stephenfin | I don't think it is | |
| 14:25:04 | stephenfin | https://review.opendev.org/c/openstack/nova/+/770532/10/nova/virt/libvirt/vif.py#501 | |
| 14:25:05 | sean-k-mooney | when generating the xml for the interface we add the vdpa path not the pci addres or the parent | |
| 14:25:19 | stephenfin | that's calling the 'get_vdpa_device_path' | |
| 14:25:53 | stephenfin | which is pretty self-contained and inspect the XML from libvirt directly https://review.opendev.org/c/openstack/nova/+/770533/10/nova/virt/libvirt/host.py#1333 | |
| 14:26:02 | sean-k-mooney | ach right | |
| 14:26:16 | stephenfin | so is that something you forgot to cleanup? | |
| 14:26:26 | sean-k-mooney | both will work | |
| 14:26:41 | stephenfin | personally, I'd rather we didn't have to pass through an instance of 'Host' | |
| 14:26:46 | stephenfin | if that's an option | |
| 14:27:14 | sean-k-mooney | we do not have the pci request here | |
| 14:27:41 | sean-k-mooney | so we cant get if form the db like i planned orginally | |
| 14:27:54 | sean-k-mooney | thats why im looking it up i think | |
| 14:28:10 | sean-k-mooney | i will need to look at this in context again sorry | |
| 14:28:25 | stephenfin | okay | |
| 14:28:38 | stephenfin | I'll play around with it a little more, see if I can make sense of it :) | |
| 14:30:12 | sean-k-mooney | so we dont have the pci devices form the db in the virt driver | |
| 14:30:40 | sean-k-mooney | and at present we do not have the mapping form the neutorn port to the pci request ourside of the network_request object | |
| 14:31:09 | sean-k-mooney | so to remove passing in host we would have to pass in the network requiest to lookup the requite id then do a db lookup | |
| 14:31:15 | sean-k-mooney | that would be much more expensive | |
| 14:31:45 | stephenfin | can we pass information through as part of the binding profile, like we do for SR-IOV? | |
| 14:31:50 | sean-k-mooney | i personally have been using the vdpa path in the pci_devices table to debug this and ensure the xml is useing the correct path for the device tha was saved | |
| 14:32:01 | sean-k-mooney | stephenfin: i really dont want to do that | |
| 14:32:15 | sean-k-mooney | because i want to stop using the binding procile for the xml generation entirely soon | |
| 14:32:16 | gibi | somebody opened ~80 new bugs to nova based on fault injection. e.g. https://bugs.launchpad.net/nova/+bug/1918340 | |
| 14:32:17 | openstack | Launchpad bug 1918340 in OpenStack Compute (nova) "Fault Injection #1 - improve unit test effectiveness" [Undecided,New] | |
| 14:32:40 | stephenfin | e.g. https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L242-L246 | |
| 14:32:52 | sean-k-mooney | close them all with we do not consider bugs cause by falut injectoin to be valid? | |
| 14:33:13 | sean-k-mooney | stephenfin: ya i want to remove that use of the profile | |