| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-14 | |||
| 15:25:22 | gibi | melwitt: is it OK for you if I touch the reno a bit to add some formatting? | |
| 15:25:22 | gibi | melwitt: is it OK for you if I touch the reno a bit to add some formatting? | |
| 15:25:34 | melwitt | gibi: sure, go ahead | |
| 15:25:34 | melwitt | gibi: sure, go ahead | |
| 15:25:43 | gibi | OK | |
| 15:25:43 | gibi | OK | |
| 15:28:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297 | |
| 15:28:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297 | |
| 15:44:44 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix RequestLevelParams persistence handling in RequestSpec https://review.opendev.org/c/openstack/nova/+/791502 | |
| 15:44:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 15:46:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 15:46:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 15:46:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 15:48:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 15:49:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 15:50:30 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 16:32:35 | openstackgerrit | Merged openstack/os-vif master: Creating oslo.config.opts entry_points for plugins https://review.opendev.org/c/openstack/os-vif/+/789645 | |
| 16:32:35 | openstackgerrit | Merged openstack/os-vif master: Creating oslo.config.opts entry_points for plugins https://review.opendev.org/c/openstack/os-vif/+/789645 | |
| 16:32:39 | openstackgerrit | Merged openstack/os-vif master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/os-vif/+/789734 | |
| 16:32:39 | openstackgerrit | Merged openstack/os-vif master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/os-vif/+/789734 | |
| 21:03:47 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Regression test for changing the MTU during a live migration https://review.opendev.org/c/openstack/nova/+/791235 | |
| 21:08:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Regression test for changing the MTU during a live migration https://review.opendev.org/c/openstack/nova/+/791235 | |
| 21:08:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Don't update the MTU during a live migration https://review.opendev.org/c/openstack/nova/+/791553 | |
| 21:08:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Regression test for changing the MTU during a live migration https://review.opendev.org/c/openstack/nova/+/791235 | |
| 21:08:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Don't update the MTU during a live migration https://review.opendev.org/c/openstack/nova/+/791553 | |
| 21:08:37 | artom | sean-k-mooney, you're probably off by now, but ^^ is proof that we suck | |
| 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 | 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: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 | 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 | |