Earlier  
Posted Nick Remark
#openstack-nova - 2021-05-14
13:16:33 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/791478
13:16:34 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/791477
13:16:34 openstack bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,Fix released] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood)
13:16:34 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/791478
13:31:44 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Remove self.server dependency form _evacuate_server https://review.opendev.org/c/openstack/nova/+/791480
13:31:44 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Remove self.server dependency form _evacuate_server https://review.opendev.org/c/openstack/nova/+/791480
13:31:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Use _evacuate_server helper in func test https://review.opendev.org/c/openstack/nova/+/791481
13:31:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Use _evacuate_server helper in func test https://review.opendev.org/c/openstack/nova/+/791481
13:31:46 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/791482
13:31:47 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/791482
13:31:47 openstack bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,Fix released] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood)
13:31:47 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/791483
13:31:48 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/791483
13:31:48 openstack bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,Fix released] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood)
14:45:01 openstackgerrit melanie witt proposed openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297
14:45:01 openstackgerrit melanie witt proposed openstack/nova master: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791297
14:55:49 melwitt gibi, dansmith: added a reno ^ I didn't know if/how I should use fancy markup in it so I didn't
14:55:49 melwitt gibi, dansmith: added a reno ^ I didn't know if/how I should use fancy markup in it so I didn't
14:56:36 gibi melwitt: thanks, will check back soon
14:56:36 gibi melwitt: thanks, will check back soon
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

Earlier   Later