| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-14 | |||
| 21:08:37 | artom | sean-k-mooney, you're probably off by now, but ^^ is proof that we suck | |
| 21:20:35 | sean-k-mooney | artom: not really its not ment to be updated | |
| 21:20:35 | sean-k-mooney | artom: not really its not ment to be updated | |
| 21:21:23 | sean-k-mooney | simply not updating the mtu wehn we regenerate the xml is not a vaild option | |
| 21:21:23 | sean-k-mooney | simply not updating the mtu wehn we regenerate the xml is not a vaild option | |
| 21:21:39 | sean-k-mooney | well its an option but its still incorrect | |
| 21:21:39 | sean-k-mooney | well its an option but its still incorrect | |
| 21:22:47 | sean-k-mooney | we discussi that approch in the ptg and i tought we rejected it | |
| 21:22:47 | sean-k-mooney | we discussi that approch in the ptg and i tought we rejected it | |
| 21:23:10 | sean-k-mooney | since the vm will not have working network connectivy if we do thi | |
| 21:23:10 | sean-k-mooney | since the vm will not have working network connectivy if we do thi | |
| 21:24:17 | sean-k-mooney | geneve and vxlan for that matter do not allow packet fragmentation so if the guest had a larger mtu before and they send a full mtu sized packet it will be dropped | |
| 21:24:17 | sean-k-mooney | geneve and vxlan for that matter do not allow packet fragmentation so if the guest had a larger mtu before and they send a full mtu sized packet it will be dropped | |
| 21:25:01 | sean-k-mooney | if it had a smaller mtu before the it will be able to transmit fine but i belive the recive path will be broken although it might work | |
| 21:25:01 | sean-k-mooney | if it had a smaller mtu before the it will be able to transmit fine but i belive the recive path will be broken although it might work | |
| 21:26:04 | artom | sean-k-mooney, true - I can change the fix's approach | |
| 21:26:04 | artom | sean-k-mooney, true - I can change the fix's approach | |
| 21:26:18 | artom | But the point is, we were still happily trying to update the XML with the new MTU | |
| 21:26:18 | artom | But the point is, we were still happily trying to update the XML with the new MTU | |
| 21:26:41 | artom | Despite specifically having code in there with comments saying we can't do that | |
| 21:26:41 | artom | Despite specifically having code in there with comments saying we can't do that | |
| 21:26:48 | artom | We missed the case of an MTU already being set | |
| 21:26:48 | artom | We missed the case of an MTU already being set | |
| 21:26:54 | sean-k-mooney | that was because of a differnt upgrade issue | |
| 21:26:54 | sean-k-mooney | that was because of a differnt upgrade issue | |
| 21:27:05 | sean-k-mooney | that was for when we first added mtu support | |
| 21:27:05 | sean-k-mooney | that was for when we first added mtu support | |
| 21:27:13 | sean-k-mooney | for jumbo frames | |
| 21:27:13 | sean-k-mooney | for jumbo frames | |
| 21:27:27 | artom | Ah, yeah, I don't know the history | |
| 21:27:27 | artom | Ah, yeah, I don't know the history | |
| 21:28:18 | artom | Anyways, I'm just happy I got my head around it | |
| 21:28:18 | artom | Anyways, I'm just happy I got my head around it | |
| 21:28:52 | artom | I could never reconcile the "we just need a better error message" with the error that libvirt was throwing. We were clearly doing *something* wrong | |
| 21:28:52 | artom | I could never reconcile the "we just need a better error message" with the error that libvirt was throwing. We were clearly doing *something* wrong | |
| 21:29:47 | sean-k-mooney | as i said in the downstream bz the neutron extinon is broken by desing | |
| 21:29:47 | sean-k-mooney | as i said in the downstream bz the neutron extinon is broken by desing | |
| 21:29:55 | sean-k-mooney | and im suprised we support in our down stream product | |
| 21:29:55 | sean-k-mooney | and im suprised we support in our down stream product | |
| 21:30:17 | sean-k-mooney | when writeble mtus was first intoduced it was for just one out of tree network backend | |
| 21:30:17 | sean-k-mooney | when writeble mtus was first intoduced it was for just one out of tree network backend | |
| 21:30:28 | sean-k-mooney | and no integration was added with nova | |
| 21:30:28 | sean-k-mooney | and no integration was added with nova | |
| 21:30:31 | artom | Right, I get that part | |
| 21:30:31 | artom | Right, I get that part | |
| 21:30:47 | artom | Neutron added a thing that Nova was never modified to support | |
| 21:30:47 | artom | Neutron added a thing that Nova was never modified to support | |
| 21:30:58 | artom | Well... Yeah, I see where you're coming from | |
| 21:30:58 | artom | Well... Yeah, I see where you're coming from | |
| 21:31:21 | artom | It's not a bug if Neutron unilaterally changed the data on us | |
| 21:31:21 | artom | It's not a bug if Neutron unilaterally changed the data on us | |
| 21:31:31 | sean-k-mooney | yes and that was not intended to work with ovn or ml2/ovs at the time | |
| 21:31:31 | sean-k-mooney | yes and that was not intended to work with ovn or ml2/ovs at the time | |
| 21:31:43 | sean-k-mooney | at somepoint it started to be used with those i guesss | |
| 21:31:43 | sean-k-mooney | at somepoint it started to be used with those i guesss | |
| 21:32:14 | artom | Yeah | |
| 21:32:14 | artom | Yeah | |
| 21:32:26 | artom | Anyways, sorry for the noise :) As I said, just happy I groked it | |
| 21:32:26 | artom | Anyways, sorry for the noise :) As I said, just happy I groked it | |
| 21:32:33 | artom | *Fully* groked it | |
| 21:32:33 | artom | *Fully* groked it | |
| 21:32:40 | artom | * artom is halfway through his beer | |
| 21:33:11 | sean-k-mooney | no worries | |
| 21:33:11 | sean-k-mooney | no worries | |
| 21:34:07 | sean-k-mooney | https://bugs.launchpad.net/neutron/+bug/1671634 | |
| 21:34:09 | openstack | Launchpad bug 1671634 in neutron "[RFE] Allow to set MTU for networks" [Wishlist,Fix released] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | |
| 21:34:09 | openstack | Launchpad bug 1671634 in neutron "[RFE] Allow to set MTU for networks" [Wishlist,Fix released] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | |
| 21:34:09 | sean-k-mooney | https://bugs.launchpad.net/neutron/+bug/1671634 | |
| 21:34:14 | sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/483518 | |
| 21:34:14 | sean-k-mooney | https://review.opendev.org/c/openstack/neutron/+/483518 | |
| 21:34:20 | sean-k-mooney | this is when it was intoduced | |
| 21:34:20 | sean-k-mooney | this is when it was intoduced | |
| 21:37:13 | sean-k-mooney | artom: and this is where i added the mtu https://review.opendev.org/c/openstack/nova/+/553072 to fix https://bugs.launchpad.net/nova/+bug/1747496 | |
| 21:37:15 | openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Medium,In progress] - Assigned to sean mooney (sean-k-mooney) | |
| 21:37:15 | openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Medium,In progress] - Assigned to sean mooney (sean-k-mooney) | |
| 21:37:15 | sean-k-mooney | artom: and this is where i added the mtu https://review.opendev.org/c/openstack/nova/+/553072 to fix https://bugs.launchpad.net/nova/+bug/1747496 | |
| 21:39:46 | sean-k-mooney | artom: anyway enjoy your weekend i was loging in to start backing up file before i resintall my home server | |
| 21:39:46 | sean-k-mooney | artom: anyway enjoy your weekend i was loging in to start backing up file before i resintall my home server | |
| 21:40:14 | artom | sean-k-mooney, o/ you too man | |
| 21:40:14 | artom | sean-k-mooney, o/ you too man | |
| #openstack-nova - 2021-05-15 | |||
| 04:05:13 | openstackgerrit | Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928 | |
| 06:16:50 | openstackgerrit | Merged openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297 | |
| 06:16:50 | openstackgerrit | Merged openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297 | |
| 07:18:35 | openstackgerrit | melanie witt proposed openstack/nova stable/wallaby: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791577 | |
| 07:18:35 | openstackgerrit | melanie witt proposed openstack/nova stable/wallaby: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791577 | |
| 13:24:07 | zigo | Hi. Does https://review.opendev.org/c/openstack/nova/+/764246 also solves (live) migration of VMs that are using LUKS encryption? | |
| 13:24:07 | zigo | Hi. Does https://review.opendev.org/c/openstack/nova/+/764246 also solves (live) migration of VMs that are using LUKS encryption? | |
| 17:49:21 | openstackgerrit | Merged openstack/nova master: Stop leaking ceph df cmd in RBD utils https://review.opendev.org/c/openstack/nova/+/789374 | |
| 17:49:21 | openstackgerrit | Merged openstack/nova master: Stop leaking ceph df cmd in RBD utils https://review.opendev.org/c/openstack/nova/+/789374 | |
| #openstack-nova - 2021-05-16 | |||
| 11:02:56 | openstackgerrit | norman shen proposed openstack/nova-specs master: Speed up server details https://review.opendev.org/c/openstack/nova-specs/+/791620 | |
| 11:15:42 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 12:36:40 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 12:56:04 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 13:02:06 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 13:18:08 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 13:21:41 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 13:46:43 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 13:58:30 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 14:11:46 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |
| 14:17:57 | openstackgerrit | norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348 | |