Posted	Nick	Remark
#openstack-nova - 2020-09-01
14:53:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Remove Agent, AgentList https://review.opendev.org/749310
14:53:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'post_interrupted_snapshot_cleanup' API https://review.opendev.org/749313
14:53:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/749312
14:53:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315
14:53:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'set_bootable' API https://review.opendev.org/749314
14:53:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove various aggregate APIs https://review.opendev.org/749317
14:53:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316
14:53:45	stephenfin	gibi: As promised. Think they're all passing unit/functional tests
14:56:32	gibi	stephenfin: thanks
14:57:01	stephenfin	gibi: As for https://review.opendev.org/#/c/730866/, okay to do a follow-up?
14:57:11	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Use local variable instead of CONF https://review.opendev.org/749319
14:58:04	stephenfin	gibi: That's the follow-up, if that approach is suitable
15:00:22	stephenfin	sean-k-mooney: still have some unaddressed nits on https://review.opendev.org/#/c/744816/4 if you respin, but I'm +2
15:11:17	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Deprecate support for non-QEMU/KVM backends https://review.opendev.org/749055
15:11:18	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Remove '[vnc] keymap', '[spice] keymap' options https://review.opendev.org/749056
15:15:37	gibi	stephenfin: sure, thanks
15:20:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/744958
15:23:42	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/748453
15:23:43	openstackgerrit	Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995
15:23:43	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use UUID as vif and network_id in vif tests https://review.opendev.org/748722
15:24:58	openstackgerrit	Balazs Gibizer proposed openstack/nova master: DNM: tracing the PCI allocation code path https://review.opendev.org/748454
15:29:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
15:29:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
15:29:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
15:31:50	stephenfin	lyarwood: Could I ask you to look through the rest of the vTPM series this week? Would love to close that out this cycle
15:35:40	sean-k-mooney	gibi: so far the latest sriov patches seam to be workign well im not done yet but no bugs so far.
16:34:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Expand SR-IOV live migration tests with NUMA https://review.opendev.org/749360
16:35:25	sean-k-mooney	gibi: here are the test results in rendered http://paste.openstack.org/show/797343/ and raw org mode form http://paste.openstack.org/show/797344/
16:37:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Use constants for libvirt version checks https://review.opendev.org/740921
16:37:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove support for Intel CMT events https://review.opendev.org/740922
16:59:00	openstackgerrit	Merged openstack/nova master: libvirt: Remove MIN_*_INTERFACE_MTU, MIN_*_TX_QUEUE_SIZE https://review.opendev.org/730866
17:14:01	openstackgerrit	Merged openstack/nova master: libvirt: Remove unused host parameters https://review.opendev.org/732725
17:20:22	sean-k-mooney	stephenfin: ill repin the os-vif patch shortly just testing one thing for sriov then i can go back to it
18:32:36	lyarwood	stephenfin: ack will do
18:52:38	lyarwood	bauzas / melwitt ; would you mind taking a look at these stable backports when you get a chance https://review.opendev.org/#/c/747975/ https://review.opendev.org/#/c/747360/
18:52:56	lyarwood	and https://review.opendev.org/#/c/742416/ sorry
19:01:24	sean-k-mooney	by the way i need to start backporting https://review.opendev.org/#/c/739131/ i noticed we still do not have it downstream in some logs i was looking at so ill start backporting it upstream tomorrow
19:03:13	lyarwood	sean-k-mooney: cool, I'll keep an eye out for it
19:04:38	openstackgerrit	sean mooney proposed openstack/os-vif master: deprecate ovs-vsctl driver and make native the default https://review.opendev.org/744816
19:05:12	sean-k-mooney	stephenfin: nits adressed ^ no other changes if we can merged that tommorrow it would make me happy.
19:42:56	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Improve CinderFixtureNewAttachFlow https://review.opendev.org/749228
19:42:57	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Robustify attachment tracking in CinderFixtureNewAttachFlow https://review.opendev.org/749229
19:42:58	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: compute: Use source_bdms to reset attachment_ids during LM rollback https://review.opendev.org/749230
19:42:59	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: compute: refactor volume bdm rollback error handling https://review.opendev.org/749231
19:43:01	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/749233
19:43:01	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Add regression tests for bug #1889108 https://review.opendev.org/749232
19:43:02	openstack	bug 1889108 in OpenStack Compute (nova) train "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
21:09:29	openstackgerrit	Artom Lifshitz proposed openstack/nova master: post live migration: don't call Neutron needlessly https://review.opendev.org/747451
21:09:29	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Revert "Handle Neutron errors in _post_live_migration()" https://review.opendev.org/747443
21:09:39	artom	lyarwood, ^^
21:09:51	artom	(Though I can no longer trade you anything, as I have nothing of value left)
#openstack-nova - 2020-09-02
01:51:32	openstackgerrit	Yingji Sun proposed openstack/nova stable/stein: Set different VirtualDevice.key https://review.opendev.org/749415
02:10:13	openstackgerrit	Yingji Sun proposed openstack/nova stable/ussuri: Set different VirtualDevice.key https://review.opendev.org/749418
04:30:12	openstackgerrit	Merged openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326
05:34:44	openstackgerrit	Brin Zhang proposed openstack/nova master: Reject resize operation for accelerator https://review.opendev.org/748560
06:00:54	brinzhang_	stephenfin: can you +A for https://review.opendev.org/#/c/748560/, reabsed because of the merged conflict but lost the +A
06:01:38	openstackgerrit	Jiri Suchomel proposed openstack/nova master: Describe '[glance]/enable_rbd_download' https://review.opendev.org/749437
06:38:59	openstackgerrit	Elancheran S proposed openstack/nova stable/train: Removes the delta file once image is extracted https://review.opendev.org/748035
06:49:20	bauzas	good morning Nova
07:15:08	gibi	bauzas: good morning
07:22:21	gibi	sean-k-mooney: thanks for the tests
07:25:45	bauzas	gibi: you can't imagine how providing the networks is difficult...
07:27:54	gibi	maybe this was the reason why mriedem didn't go down that way
07:30:23	openstackgerrit	Wenping Song proposed openstack/nova-specs master: Correct some spell errors https://review.opendev.org/749449
07:32:58	gibi	sean-k-mooney: could the 9.2 where the VFs are freed but not exists after a PF detach can be worked around by setting the host to auto allocate VFs via /etc/network/interfaces.d/ config?
08:21:51	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Remove unused vpn param from allocate_for_instance https://review.opendev.org/744933
08:21:51	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Delete non tag related device metadata during detach https://review.opendev.org/741436
08:21:52	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move equality check into LibvirtConfigGuestInterface https://review.opendev.org/744524
08:24:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/748453
08:24:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Only unplug vif after the device is detached from libvirt https://review.opendev.org/744934
08:24:24	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use UUID as vif and network_id in vif tests https://review.opendev.org/748722
08:28:07	openstackgerrit	Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995
08:30:41	openstackgerrit	Balazs Gibizer proposed openstack/nova master: DNM: tracing the PCI allocation code path https://review.opendev.org/748454
08:31:25	stephenfin	jangutter: care to look at https://review.opendev.org/#/c/744816/ ?
08:38:00	openstackgerrit	Luyao Zhong proposed openstack/nova master: Rollback when reset running live migration https://review.opendev.org/747899
08:38:00	openstackgerrit	Luyao Zhong proposed openstack/nova master: Track error migrations in resource tracker https://review.opendev.org/745281
08:38:01	openstackgerrit	Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282
09:01:49	jangutter	stephenfin: ack, looking!
09:03:57	jsuchome	gibi: lyarwood artom: hi, please take a look at https://review.opendev.org/#/c/574301/ (patch updated with stephenfin idea)+ https://review.opendev.org/#/c/749437/ (release notes)
09:04:56	gibi	jsuchome: looking...
09:09:40	gibi	jsuchome: thanks. I'm +2
09:10:47	jangutter	stephenfin, sean-k-mooney: I see https://review.opendev.org/#/c/744816/ is already in the gate. +2 from me, with a potential followup to clean up one of the nits (if it's actually a nit).
09:13:10	stephenfin	jangutter: Yup, makes sense to me
09:17:35	lyarwood	gibi: thanks!
09:18:22	lyarwood	jsuchome: https://review.opendev.org/#/c/743220/ - could you update that to use the new configurables?
09:18:47	lyarwood	jsuchome: I'm going to be pretty busy downstream this week so I'll not be able to get to it for a while
09:19:06	lyarwood	https://review.opendev.org/#/c/748212/ is the actual job sorry
09:20:03	jsuchome	lyarwood: yep; later today I hope
09:20:53	lyarwood	jsuchome: cool thanks
09:25:34	suryasingh	sean-k-mooney: melwitt dansmith lyarwood Sorry to interrupt you all. I have simple query. Does the current nova-compute service(Data-plane) support graceful shutdown ? Will the nova-compute service stop(systemctl stop openstack-nova-compute.service) wait for the ongoing operation(instance provisioning) to finish ?
09:34:36	jkulik	suryasingh, there's https://review.opendev.org/#/c/666245/ for spawning VMs at least
09:35:59	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: Unified virtual persistent memory restrictions https://review.opendev.org/749472
09:38:51	brinzhang0	alex_xu: if you are free, please review https://review.opendev.org/#/c/749472/, modify the depency of vPMEM limit in the spec, make it keep consistent with admin guide docs
09:39:25	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/749233
09:39:25	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Add regression tests for bug #1889108 https://review.opendev.org/749232
09:39:26	openstack	bug 1889108 in OpenStack Compute (nova) train "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
09:40:30	brinzhang0	alex_xu: please check the dependency package version, is it reasonable?
09:46:08	lyarwood	bauzas / elod: would you mind taking a look at https://review.opendev.org/#/c/748496/
09:47:23	suryasingh	@jkulik thanks for response and patch link. Did you tested this patch ? And is there any priority from nova-community side for this patch, if you know about it, sorry for asking.
09:56:28	jkulik	suryasingh, we're running some version of this patch in production on a queens code-base. I haven't seen any push from the nova-community for this.

Earlier   Later