| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-23 | |||
| 11:44:01 | sean-k-mooney | mriedem: when hybrid_plug=false and we are not useing dpdk we hit https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/ovs.py#L216-L223 | |
| 11:44:37 | sean-k-mooney | in this config os-vif only creates the bridge and the actual pluggin is done by the hypervior e.g. libvirt | |
| 11:44:55 | sean-k-mooney | so we dont get the speed up on live migrate | |
| 11:47:16 | mriedem | what speed up on live migrate? | |
| 11:47:17 | sean-k-mooney | i know how to fix this in two way and ill start working on it after i test your changes but ya. i noticed up to a 20 second network outage when using kernel ovs with ovs contrac firewall driver but none when using ip tables as a result. | |
| 11:47:58 | sean-k-mooney | ya mriedem effectivly the plug does not happen untill libvirt creates the vm on the dest. | |
| 11:48:04 | mriedem | ok | |
| 11:48:08 | mriedem | i'll take your word for it :) | |
| 11:48:14 | jaypipes | morning supernovas | |
| 11:48:32 | mriedem | hello jay | |
| 11:48:43 | sean-k-mooney | mriedem: its the same behavior as before multiple port bindings so not a regression or bug just not faster. | |
| 11:48:50 | sean-k-mooney | jaypipes: enjoy your PTO? | |
| 11:48:54 | jaypipes | mriedem: luckily not much happened or was discussed on the ML while I was away | |
| 11:49:05 | jaypipes | sean-k-mooney: very much, thank you Sean. :) | |
| 11:49:31 | mriedem | gibi: can you take a look at this rc3-related bug workaround? https://review.openstack.org/#/c/594508/ - rc3 is supposed to be today | |
| 11:49:53 | mriedem | gibi: but withhold a +W until sean-k-mooney is able to test it in his env | |
| 11:51:07 | gibi | mriedem: ack, witholding my love but reading the patch ;) | |
| 11:53:20 | sean-k-mooney | mriedem: oh i forgot RC3 was today. i need to restack but i should have the env running in 20 mins or so | |
| 11:54:36 | mriedem | sean-k-mooney: *no pressure!* | |
| 12:09:38 | mriedem | moshele: can you report a bug for https://review.openstack.org/#/c/595592/ ? | |
| 12:09:41 | mriedem | it's a regression in rocky | |
| 12:11:07 | moshele | mriedem: sure I will | |
| 12:11:09 | mriedem | melwitt: ^ | |
| 12:11:19 | mriedem | throw it on the big board donny! | |
| 12:12:05 | mriedem | it's fun | |
| 12:13:36 | mriedem | sahid: you should take a look at moshele's patch | |
| 12:13:46 | mriedem | moshele: what sets the driver_name to vhost? | |
| 12:14:54 | sahid | mriedem ah yes this bug has been reported downstream yesterday i think | |
| 12:14:54 | moshele | mriedem: the macvtap port which is virtio tap connected to VF so for that case it should be fine | |
| 12:15:00 | sahid | moshele: thanks for your patch | |
| 12:15:18 | moshele | sahid: np :) | |
| 12:15:44 | mriedem | moshele: sorry i don't understand that statement, | |
| 12:15:46 | mriedem | : libvirtError: unsupported configuration: Unknown PCI device <driver name='vhost'/> has been specified | |
| 12:15:46 | mriedem | looking at your paste | |
| 12:15:57 | mriedem | and looking at https://review.openstack.org/#/c/595592/1/nova/virt/libvirt/designer.py | |
| 12:16:06 | mriedem | i guess in this case the net_type was 'direct'? | |
| 12:16:11 | mriedem | so L120 | |
| 12:17:17 | sahid | mriedem: are you sure vfio is a valid driver? | |
| 12:17:22 | sahid | moshele: | |
| 12:17:55 | ykarel | jroll, hi | |
| 12:17:56 | moshele | shaid: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7/html/virtualization_deployment_and_administration_guide/sect-pci_devices-pci_passthrough | |
| 12:18:19 | moshele | sahid: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/7/html/virtualization_deployment_and_administration_guide/sect-pci_devices-pci_passthrough | |
| 12:19:12 | openstack | Launchpad bug 1787910 in tripleo "OVB overcloud deploy fails on nova placement errors" [Critical,Triaged] - Assigned to Marios Andreou (marios-b) | |
| 12:19:12 | ykarel | jroll, can u check https://bugs.launchpad.net/tripleo/+bug/1787910/comments/13 | |
| 12:19:16 | sahid | ok thanks, because i remeber of a bug in libvirt for vhostuser interface interface, when you basically set it to vhostuser (which is the driver used) libvirt is reaisng an error | |
| 12:19:18 | moshele | mriedem: no the net_type 'direct' is for macvtap port for port direct which is hostdev we don't set the driver name | |
| 12:20:04 | mriedem | moshele: yeah i figured it out, | |
| 12:20:09 | mriedem | the comment is busted in the code, | |
| 12:20:13 | mriedem | i left a comment about fixing it :) | |
| 12:21:16 | moshele | mriedem: the problem is with this https://github.com/openstack/nova/blob/34956bea4beb8e5ba474b42ba777eb88a5eadd76/nova/virt/libvirt/vif.py#L166-L176 | |
| 12:21:40 | moshele | mriedem: driver is not specified it put vhost | |
| 12:21:45 | openstack | Launchpad bug 1787910 in tripleo "OVB overcloud deploy fails on nova placement errors" [Critical,Triaged] - Assigned to Marios Andreou (marios-b) | |
| 12:21:45 | ykarel | mriedem, or u can check https://bugs.launchpad.net/tripleo/+bug/1787910/comments/13 | |
| 12:22:24 | mriedem | aha | |
| 12:22:26 | mriedem | moshele: ok | |
| 12:23:04 | moshele | mriedem: I send the mail because I am not sure about the fix. I know that my fix solve direct port | |
| 12:23:57 | mriedem | i'd ask jangutter but he's not around | |
| 12:24:23 | sahid | moshele: perhaps it's better to use None, lbvirt might have some kind of fallback, example using vfio but if available on host fallback to uio | |
| 12:24:24 | mriedem | moshele: vnic_type = 'direct' right? | |
| 12:24:31 | zigo | Hi there ! | |
| 12:24:45 | zigo | Any idea why I'm getting this when building the Nova Debian package? https://stretch-rocky.infomaniak.ch/job/nova/9/console | |
| 12:24:50 | zigo | (see the end of unit test run...) | |
| 12:25:03 | sahid | it's what is done for vhost, norammy by default, libvirt use vhost but fallback to qemu if vhost module is not loaded on host | |
| 12:25:06 | moshele | mriedem: yes | |
| 12:25:06 | mriedem | zigo: you'll need to put that in a paste | |
| 12:25:09 | mriedem | or give out creds | |
| 12:25:30 | zigo | Sure. | |
| 12:25:55 | zigo | mriedem: http://paste.openstack.org/show/728679/ | |
| 12:26:50 | zigo | From where is comming the MIMEAccept object? Is that nova's internal? | |
| 12:27:01 | zigo | A grep in nova shows nothing ... | |
| 12:27:03 | moshele | sahid: but this port will set driver name to vhost if it None https://github.com/openstack/nova/blob/34956bea4beb8e5ba474b42ba777eb88a5eadd76/nova/virt/libvirt/vif.py#L166-L176 | |
| 12:27:17 | mriedem | ykarel: i'll let jroll look at that b/c https://review.openstack.org/#/c/565841 was master only atm which is stein, and today is rocky rc3 so i'm focused on what we need to get done for rc3 | |
| 12:27:21 | moshele | sahid: s/port/part | |
| 12:27:36 | ykarel | mriedem, no prob, Thanks | |
| 12:27:47 | ykarel | rocky is more priority now | |
| 12:28:00 | mriedem | zigo: i'd guess the wsgi-intercept library | |
| 12:28:02 | mriedem | cdent: ^ | |
| 12:28:05 | mriedem | http://paste.openstack.org/show/728679/ | |
| 12:29:14 | mriedem | well best_matches = req.accept.acceptable_offers(types) is clearly in placement code | |
| 12:29:21 | cdent | zigo, looks webob | |
| 12:29:30 | mriedem | yeah | |
| 12:29:30 | sahid | moshele: hum... it's complicated, because if you set it to None, the LibvirtConfig will not generate the tx/rx args | |
| 12:29:42 | cdent | acceptable_offer is new in webob 1.8.x I think | |
| 12:30:02 | mriedem | sahid: but it looks like we default to vhost if rx/tx is configured https://github.com/openstack/nova/blob/34956bea4beb8e5ba474b42ba777eb88a5eadd76/nova/virt/libvirt/vif.py#L166-L176 | |
| 12:30:10 | mriedem | so driver_name will always be at least vhost? | |
| 12:30:25 | mriedem | which doesn't work for net_type == 'hostdev'? | |
| 12:30:42 | cdent | zigo: my guess is that the webob is not up to date with minimum requirements? | |
| 12:30:53 | zigo | cdent: Yeah, that should be it, thanks! | |
| 12:31:05 | mriedem | https://github.com/openstack/nova/blob/master/lower-constraints.txt#L170 fwiw | |
| 12:31:08 | mriedem | 1.8.2 | |
| 12:31:09 | zigo | Debian version is 1:1.7.3-2 | |
| 12:31:11 | cdent | zigo: I'm glad it was that easy :) | |
| 12:31:12 | zigo | Thanks guys. | |
| 12:31:14 | zigo | Will update it. | |
| 12:31:22 | sahid | yes yes, it's just i don't think we should specically set it to vfio, i think we should not set it at all for hostdev | |
| 12:31:40 | sahid | but if we do that tx/rx are then ignored | |
| 12:31:54 | jroll | mriedem: ykarel: I'm looking at this, I'm confused what is special about tripleo jobs here, as this works fine in our ironic jobs | |
| 12:32:08 | sahid | https://github.com/openstack/nova/blob/34956bea4beb8e5ba474b42ba777eb88a5eadd76/nova/virt/libvirt/config.py#L1390 | |
| 12:32:17 | jroll | oh wait, it's probably the flavors | |
| 12:32:24 | mriedem | sahid: yup | |