Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-05-14
14:58:03 melwitt ty
14:58:03 melwitt ty
14:59:43 dansmith melwitt: seems alright to me.. will let gibi +W if he's around and wants to review
14:59:43 dansmith melwitt: seems alright to me.. will let gibi +W if he's around and wants to review
14:59:44 dansmith but otherwise I'd be fine with it
14:59:44 dansmith but otherwise I'd be fine with it
15:00:17 melwitt sounds cool, thanks
15:00:18 melwitt sounds cool, thanks
15:11:24 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:11:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206
15:11:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208
15:11:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix RequestLevelParams persistence handling in RequestSpec https://review.opendev.org/c/openstack/nova/+/791502
15:11:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503
15:11:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504
15:11:27 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505
15:11:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506
15:20:11 openstackgerrit Merged openstack/osc-placement master: Add functional-py39 tox target https://review.opendev.org/c/openstack/osc-placement/+/760912
15:20:11 openstackgerrit Merged openstack/osc-placement master: Add functional-py39 tox target https://review.opendev.org/c/openstack/osc-placement/+/760912
15:22:02 openstackgerrit Merged openstack/osc-placement master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/osc-placement/+/760710
15:22:02 openstackgerrit Merged openstack/osc-placement master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/osc-placement/+/760710
15:22:14 openstackgerrit Merged openstack/osc-placement master: Add py38 package metadata https://review.opendev.org/c/openstack/osc-placement/+/756985
15:22:14 openstackgerrit Merged openstack/osc-placement master: Add py38 package metadata https://review.opendev.org/c/openstack/osc-placement/+/756985
15:22:19 openstackgerrit Merged openstack/osc-placement master: Verify result for inventory set --dry-run https://review.opendev.org/c/openstack/osc-placement/+/680295
15:22:19 openstackgerrit Merged openstack/osc-placement master: Verify result for inventory set --dry-run https://review.opendev.org/c/openstack/osc-placement/+/680295
15:22:22 openstackgerrit Merged openstack/osc-placement master: Add "--resource-class" to allocation unset https://review.opendev.org/c/openstack/osc-placement/+/697620
15:22:22 openstackgerrit Merged openstack/osc-placement master: Add "--resource-class" to allocation unset https://review.opendev.org/c/openstack/osc-placement/+/697620
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

Earlier   Later