Posted	Nick	Remark
#openstack-nova - 2021-05-14
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
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)

Earlier   Later