Posted	Nick	Remark
#openstack-nova - 2020-09-07
14:02:27	gibi	I assume, yes
14:02:30	bauzas	cool
14:02:36	gibi	as neutron would need to assign an ip
14:02:40	gibi	during the binding
14:02:44	bauzas	anyway, we could provide a caveat documentation if no
14:03:08	bauzas	anyway, today is the last day I'm trying to work on this
14:03:27	bauzas	gibi: sean-k-mooneyif you have changes you want to me to review, lemme know
14:03:38	bauzas	and then I'll review them tomorrow
14:03:49	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP/DNM libvirt: Start emitting DeviceRemovedEvent and DeviceRemovalFailedEvent events https://review.opendev.org/749929
14:04:31	gibi	bauzas: sriov attach is ready, sean-k-mooney is alread +1 on it and the bottom has +2s from stephenfin. series starts here https://review.opendev.org/#/c/741436
14:04:41	bauzas	gibi: ack, will look
14:04:48	gibi	thanks!
14:05:28	bauzas	gibi: now that I work on some network features, I know better the related files ;)
14:05:39	gibi	:)
14:07:27	lyarwood	kashyap: yeah that's long after tempest has stopped waiting for the volume to be detached
14:07:35	lyarwood	kashyap: let me grab some logs in pastebin
14:07:55	kashyap	lyarwood: I've got some contextual stuff here: https://kashyapc.fedorapeople.org/CirrOS_device_detach_issues/
14:16:29	lyarwood	kashyap: http://paste.openstack.org/show/797545/ - AFAICT n-cpu stops trying to detach the volume much earlier than the libvirtd logs you've posted
14:18:12	kashyap	lyarwood: Okay, I perhaps need to look further up; let me see if I can see this "Unable to detach" thing in the log
14:19:14	kashyap	lyarwood: I'm stumped - I don't see why that "Unable to detach ..." isn't captured here: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c3a/734029/2/check/devstack-platform-focal/c3ab542/controller/logs/libvirt/libvirtd_log.txt
14:19:49	kashyap	(Beaware the above log size: gzip-compressed - 8.2MB; uncompressed - 118MB)
14:20:00	lyarwood	721862 2020-09-03 19:58:35.443+0000: 65331: debug : qemuDomainDeleteDevice:128 : Detaching of device virtio-disk1 failed and no event arrived
14:20:14	lyarwood	^ kashyap I think that's what we are after
14:20:28	kashyap	lyarwood: "Huzzah", that's right
14:33:12	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Follow up for I67504a37b0fe2ae5da3cba2f3122d9d0e18b9481 https://review.opendev.org/750184
14:50:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
14:50:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: Set 'old_flavor', 'new_flavor' on source before resize https://review.opendev.org/750187
14:50:16	openstack	bug 1879878 in OpenStack Compute (nova) "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Medium,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane)
14:50:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: Expand generic reproducer for bug #1879878 https://review.opendev.org/750186
14:54:11	gmann	stephenfin: are you planning the xenapi removal for Victoria release? if so i can review your tempest patch on priority (as that will block the nova side change) otherwise after Focal migration work.
14:54:28	stephenfin	gmann: Yes, I was hoping to
14:54:45	stephenfin	I think it's in merge conflict though
14:55:42	gmann	yeah.
14:56:56	stephenfin	okay, resolved that. docstring conflict
14:58:45	stephenfin	gibi: replied at https://review.opendev.org/#/c/746945/6/nova/tests/functional/libvirt/test_pci_sriov_servers.py@a370
14:59:27	gibi	thanks, looking
15:52:44	sean-k-mooney	bauzas: technically yes but the port binding would fail
15:52:51	sean-k-mooney	ah gibi aready said that
15:52:57	bauzas	all cool then
15:55:16	sean-k-mooney	gibi so regarding https://review.opendev.org/#/c/742180/11/nova/tests/functional/regressions/test_bug_1888395.py i was thinking of using stephens seriese eventuly to enable the migration testing
15:55:48	sean-k-mooney	gibi: once the sriov migration fuctional test series merges tehn that regression test can be updated
15:55:51	gibi	sean-k-mooney: yeah that would be nice
15:56:19	gibi	I read through stephenfin's series today and I'm +2 almost all the way
15:57:20	sean-k-mooney	im not sure if i need all the patches by the way. i was hopeing to get this merged before his series to avoid conflicts on backport but im hoping both merged in victoria
15:57:39	sean-k-mooney	all the patches in stephens series that is
15:58:03	gibi	sean-k-mooney: my -1 on https://review.opendev.org/#/c/742180/ is about the question if we break SRIOV live migration if there is no multi portbinding
15:58:18	sean-k-mooney	yes so sriov live migration requires multiple port bindings
15:58:26	sean-k-mooney	it was only ment to work if the backend supported that
15:58:45	gibi	then the question will it fail cleanly?
15:59:01	sean-k-mooney	yes it will
15:59:16	sean-k-mooney	we check if multiple port bindigns is supproted i nthe conductor
15:59:21	sean-k-mooney	and fail the migration if not
16:00:07	sean-k-mooney	gibi: https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L250-L255
16:00:24	gibi	cool
16:00:27	gibi	I'm droping my -1 then
16:00:46	sean-k-mooney	:) any other concerns?
16:01:28	gibi	nope
16:01:39	sean-k-mooney	should we be worried about all those gate timeouts
16:01:54	gibi	I haven't checked the gate this afternoon
16:02:20	sean-k-mooney	im seeing sqlalcamy errors in the unit tests which are unrelated
16:02:58	gibi	I'm leaving for today...
16:03:01	gibi	o/
16:03:58	sean-k-mooney	o/
18:30:41	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/746981
18:30:41	openstackgerrit	Lee Yarwood proposed openstack/nova master: fakelibvirt: Use versionutils to set min versions found in the driver https://review.opendev.org/749707
18:30:42	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983
18:30:42	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/746982
18:30:43	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_VIDEO_MODEL_VERSIONS https://review.opendev.org/746985
18:30:43	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984
18:30:44	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_PMEM_SUPPORT https://review.opendev.org/746986
19:00:46	openstackgerrit	Merged openstack/nova stable/ussuri: Add note and daxio version to the vPMEM document https://review.opendev.org/749681
20:22:02	openstackgerrit	sean mooney proposed openstack/nova master: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/742180
20:22:02	openstack	bug 1888395 in OpenStack Compute (nova) "shared live migration of a vm with a vif is broken in train" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to sean mooney (sean-k-mooney)
20:22:02	openstackgerrit	sean mooney proposed openstack/nova master: add functional regression test for bug #1888395 https://review.opendev.org/747454
20:35:33	openstackgerrit	Merged openstack/nova stable/train: Removed the host FQDN from the exception message https://review.opendev.org/749609
20:35:40	openstackgerrit	Merged openstack/nova stable/ussuri: resolve ResourceProviderSyncFailed issue https://review.opendev.org/749668
20:35:46	openstackgerrit	Merged openstack/nova stable/ussuri: Set different VirtualDevice.key https://review.opendev.org/749418
20:35:53	openstack	bug 1889633 in OpenStack Compute (nova) train "Pinned instance with thread policy can consume VCPU" [High,In progress] https://launchpad.net/bugs/1889633 - Assigned to Stephen Finucane (stephenfinucane)
20:35:53	openstackgerrit	Merged openstack/nova stable/train: tests: Add reproducer for bug #1889633 https://review.opendev.org/748254
20:35:58	openstackgerrit	Merged openstack/nova stable/train: Add checks for volume status when rebuilding https://review.opendev.org/748558
21:16:25	openstackgerrit	sean mooney proposed openstack/nova master: use os-brick connector fixture form ServersTestBase https://review.opendev.org/750215
21:42:02	openstackgerrit	sean mooney proposed openstack/nova master: add using_multiple_port_bindings property to livemigrate_data https://review.opendev.org/750217
21:44:50	openstackgerrit	sean mooney proposed openstack/nova master: add using_multiple_port_bindings property to LiveMigrateData https://review.opendev.org/750217
22:32:49	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
#openstack-nova - 2020-09-08
00:00:20	openstackgerrit	Brin Zhang proposed openstack/nova stable/train: Add note and daxio version to the vPMEM document https://review.opendev.org/750222
00:45:40	openstackgerrit	Wenping Song proposed openstack/nova-specs master: Support vGPU management by Cyborg https://review.opendev.org/750116
01:38:20	yingjisun	buddies, I am yingjisun from VMware. Nice to meet you.
01:39:27	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
01:40:01	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
01:40:09	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821
01:40:44	gmann	gibi: fixed the reno ref link issue - https://review.opendev.org/#/c/748059/14
01:53:52	yingjisun	gibi: for the issue of "log dump isn't complete", is it the issue mentioned by Dan about missing apache logs? If yes, this issue was fixed by Hui recently.
01:54:52	yingjisun	gibi: there was an unstability issue in our testbed. Hui is working to resolve it. Hope it can be recovered quickly.
05:48:17	openstackgerrit	Brin Zhang proposed openstack/nova stable/ussuri: Reject resize operation for accelerator https://review.opendev.org/749701
07:05:53	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Undeprecate the vmwareapi driver https://review.opendev.org/742407
07:06:22	gibi	yingjisun: thanks for the info, I will check the logs and I will approve the undeprecation patch before Thursday
07:09:16	yingjisun	gibi: thank you very much. Please hold for a while. I will check if there is an env for you to check the new logs.
07:10:49	openstackgerrit	Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563
07:13:16	yingjisun	gibi, just get the latest status from my colleague. It may still take some time to repair our testbed. I will let you know once the env recovered.
07:13:29	gibi	yingjisun: OK, thanks

Earlier   Later