| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-14 | |||
| 15:55:50 | opendevreview | Artom Lifshitz proposed openstack/nova master: libvirt: remove default cputune shares value https://review.opendev.org/c/openstack/nova/+/824048 | |
| 16:00:22 | opendevreview | Dmitry Tantsur proposed openstack/nova master: Add a proper schema version to network_data.json https://review.opendev.org/c/openstack/nova/+/849881 | |
| 17:09:07 | artom | sean-k-mooney, stephenfin, any chance we can finalize the domain spec before EOD? | |
| 17:40:14 | sean-k-mooney | ill take a look at it again shortly so yes? | |
| 17:40:32 | artom | \o/ | |
| 18:07:40 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: Configurable instance domains https://review.opendev.org/c/openstack/nova-specs/+/849765 | |
| 18:15:12 | sean-k-mooney | artom: +2 on ^ stephenfin melwitt gibi incase ye are still about | |
| 18:15:55 | sean-k-mooney | artom: on other commit is something we add examples of the updatead api payload or psudo api requests | |
| 18:16:05 | sean-k-mooney | in this case i dont think you need to do that | |
| 18:16:15 | artom | Which other commit? | |
| 18:16:25 | artom | Oh, if I respin this spec? | |
| 18:16:34 | sean-k-mooney | sorry other specs | |
| 18:16:45 | artom | stephenfin didn't do it, and this is just plagiarizing his domain spec ;) | |
| 18:16:47 | sean-k-mooney | so somethinmes we add example post/put payloads | |
| 18:17:05 | sean-k-mooney | but i think thats overkill for a this | |
| 18:17:17 | sean-k-mooney | /domin/hostname | |
| 18:17:42 | sean-k-mooney | i.e his spec was hostname yours is domain | |
| 18:18:08 | sean-k-mooney | anyway i didnt see anything worth another respin but im happy to rereview if others do | |
| 18:22:17 | opendevreview | Merged openstack/nova master: Remove use of pkg_resources https://review.opendev.org/c/openstack/nova/+/740661 | |
| 18:22:19 | gibi | sean-k-mooney, artom: I'm +2+Ad the domain spec | |
| 18:22:42 | artom | That was fast | |
| 18:22:47 | gibi | I read fast :) | |
| 18:23:00 | sean-k-mooney | and the content was familar :P | |
| 18:23:03 | gibi | yeah | |
| 18:23:21 | artom | Yeah, it's not like we kept rehashing it every second tech call for the past 4 months | |
| 18:23:22 | artom | Oh wiat | |
| 18:23:26 | gibi | I could swear I read this before somewher | |
| 18:23:27 | gibi | e | |
| 18:24:20 | sean-k-mooney | artom: implementaiton wil be ready by tueseday yes | |
| 18:24:44 | artom | https://i.imgflip.com/6ms8wt.jpg | |
| 18:25:02 | gibi | artom: so will you abandon https://review.opendev.org/c/openstack/nova-specs/+/840974 ? | |
| 18:25:22 | sean-k-mooney | its not mutually exclucive but i would at least punt it to AA | |
| 18:25:29 | artom | Done | |
| 18:25:33 | sean-k-mooney | cool | |
| 18:25:44 | artom | It's not mutually exclusive, but there isn't much point to it | |
| 18:26:01 | artom | Like, what would use those per-NIC domain fields? | |
| 18:26:18 | artom | If we care about cloud-init, then it's only --domain | |
| 18:26:50 | sean-k-mooney | well linux support FQDNs per interface | |
| 18:27:14 | sean-k-mooney | so technially you chould have a script that called systemd-resolved to configure that or network manager | |
| 18:27:25 | sean-k-mooney | but its nice ot have rather then required | |
| 18:27:48 | sean-k-mooney | so for now lets focus on --domain | |
| 18:27:59 | sean-k-mooney | and if people ask for the metadata we can add that in AA | |
| 18:28:43 | opendevreview | Ghanshyam proposed openstack/nova master: Keep legacy admin behaviour in new RBAC https://review.opendev.org/c/openstack/nova/+/849209 | |
| 18:31:17 | opendevreview | Merged openstack/nova-specs master: Configurable instance domains https://review.opendev.org/c/openstack/nova-specs/+/849765 | |
| 20:31:17 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Remove unnecessary TODO https://review.opendev.org/c/openstack/nova/+/830645 | |
| 20:31:17 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Ignore LibvirtConfigObject kwargs https://review.opendev.org/c/openstack/nova/+/830644 | |
| 20:31:18 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 20:31:18 | opendevreview | Rico Lin proposed openstack/nova master: add locked_memory extra spec and image property https://review.opendev.org/c/openstack/nova/+/778347 | |
| 20:31:19 | opendevreview | Rico Lin proposed openstack/nova master: Add traits for viommu model https://review.opendev.org/c/openstack/nova/+/844507 | |
| #openstack-nova - 2022-07-15 | |||
| 05:25:47 | auniyal | Hello | |
| 05:25:56 | auniyal | please review these 3 | |
| 05:26:02 | auniyal | https://review.opendev.org/c/openstack/nova/+/848886 | |
| 05:26:16 | auniyal | https://review.opendev.org/c/openstack/nova/+/849104 | |
| 05:26:27 | auniyal | https://review.opendev.org/c/openstack/nova/+/849532 | |
| 10:03:45 | sean-k-mooney | stephenfin: if you get time can you fix the requirmetns file in https://review.opendev.org/c/openstack/nova/+/849867 then it should be good to merge | |
| 10:45:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1981813 in func env https://review.opendev.org/c/openstack/nova/+/849985 | |
| 10:45:33 | gibi | sean-k-mooney: ^^ this is the reproduction for vnic_type change bug | |
| 11:25:42 | opendevreview | Merged openstack/nova master: libvirt: remove default cputune shares value https://review.opendev.org/c/openstack/nova/+/824048 | |
| 11:34:03 | artom | gibi, I left a comment, I'm... not sure? | |
| 11:34:09 | artom | Smells like more of a unit test | |
| 11:35:15 | gibi | artom: you probably found a bug in that test | |
| 11:36:03 | gibi | it should only pass if _heal_instance_info_cache updates the vnic_type in our cache and then restart | |
| 11:36:30 | artom | My point is that the vnic change and heal cache isn't needed | |
| 11:36:38 | gibi | probably made a two broad mock | |
| 11:36:52 | artom | If you just start the server with a macvtap and mock the PCI code to return NotFound, you'll still get the exact same service startup error | |
| 11:36:58 | gibi | yeah | |
| 11:37:30 | artom | And like, I know in real life the NotFound comes from changing the vnic_type | |
| 11:37:52 | artom | OK, I think I got it. | |
| 11:38:33 | artom | No, wait. | |
| 11:41:06 | artom | The thing that Nova is doing wrong is attempting to find the wrong PCI device? Or because the device is already used by the instance, we can't find it? | |
| 11:42:27 | gibi | it is used by the instance so we cannot find it afaik | |
| 11:42:58 | gibi | or more precisely | |
| 11:43:02 | gibi | /sys/bus/pci/devices/0000:19:0a.7/net is not exists | |
| 11:43:22 | gibi | as the pci device is attached to the instance there is no netdev on the host | |
| 11:45:42 | gibi | after the vnic_type changed to macvtap the vif plug code wants to call set_vf_interface_vlan on the pci device | |
| 11:45:55 | artom | So wait, shouldn't that also happen if you just restart nova-compute with any macvtap device? | |
| 11:46:23 | gibi | hm | |
| 11:46:30 | gibi | that is a good question | |
| 11:46:33 | gibi | wait | |
| 11:46:38 | gibi | if it is consumed as a macvtap | |
| 11:46:44 | gibi | then the VF is not consumed | |
| 11:46:49 | gibi | and probably the netdev exists | |
| 11:47:02 | artom | Ah, right. | |
| 11:47:06 | artom | Sorry, creating confusion. | |
| 11:47:07 | gibi | the problem is that we consumed the VF and then looking up the VF's netdev | |
| 11:47:27 | artom | Bingo. | |
| 11:47:35 | artom | So the functional test should probably be asserting *that* | |
| 11:49:42 | gibi | so probably mocking get_ifname_by_pci_address is too much and I should inject the fault at set_vf_interface_vlan as that is the last specific call | |
| 11:49:55 | gibi | for the macvtap plug | |
| 11:49:56 | artom | Also, this is in the plug logic, right? | |
| 11:50:03 | artom | So should also happen during instance hard reboot? | |
| 11:50:18 | artom | We don't do any PCI accounting update during reboot... | |
| 11:50:22 | gibi | yes | |
| 11:50:30 | gibi | probably the plug fails at hard reboot too | |
| 11:50:45 | gibi | I can try as I have real sriov hw right now | |
| 11:52:25 | gibi | still the test will not be perfect as with mocking I can only simulate that the VF is consumed | |
| 11:52:40 | gibi | so if you move the port set before boot but keep the mock as is then it will still fail | |
| 11:52:57 | gibi | as it will still simulate that the VF is cosumed | |
| 11:53:04 | gibi | even if in reality it would not be | |
| 11:53:48 | artom | Right, there's no way around the mocking | |