Posted	Nick	Remark
#openstack-nova - 2021-01-10
03:15:51	openstackgerrit	Merged openstack/nova master: Remove dead code from SchedulerReportClient https://review.opendev.org/c/openstack/nova/+/769467
06:54:27	openstackgerrit	Merged openstack/nova master: only wait for plugtime events in pre-live-migration https://review.opendev.org/c/openstack/nova/+/767368
09:50:26	openstackgerrit	Merged openstack/nova stable/ussuri: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761701
17:58:48	openstackgerrit	Ghanshyam proposed openstack/nova master: Fix ForbiddenWithAccelerators to HTTPForbidden for shelve API https://review.opendev.org/c/openstack/nova/+/770007
17:59:39	openstackgerrit	Ghanshyam proposed openstack/nova master: Fix ForbiddenWithAccelerators to HTTPForbidden for shelve API https://review.opendev.org/c/openstack/nova/+/770007
#openstack-nova - 2021-01-11
00:14:47	pfire	resized a powered off instance, cannot start instance while it is in task state resize finish, cannot confirmresize while in vmstate stopped
00:15:12	pfire	should a resize have been allowed in the first place
00:28:48	prometheanfire	got around it by manually starting the instance, then resize could be confirmed (but instance showed up as stopped), and then once instance start was issued the instance was restarted (instead of the state being updated in the db)
05:39:27	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Fix nova live migration task failed due respone value changed https://review.opendev.org/c/openstack/python-novaclient/+/769872
05:39:28	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in usage* commands https://review.opendev.org/c/openstack/python-novaclient/+/769675
05:39:28	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in quota* commands https://review.opendev.org/c/openstack/python-novaclient/+/769869
05:39:29	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: DNM/TEST: function test https://review.opendev.org/c/openstack/python-novaclient/+/770060
07:33:08	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in quota* commands https://review.opendev.org/c/openstack/python-novaclient/+/769869
07:36:33	openstackgerrit	Hemanth N proposed openstack/nova stable/train: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761725
08:14:06	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in limits commands https://review.opendev.org/c/openstack/python-novaclient/+/770065
08:19:55	gibi	good morning nova
08:26:14	chengsheng1	Please help review the code: https://review.opendev.org/c/openstack/nova/+/762330/
08:27:24	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in usage* commands https://review.opendev.org/c/openstack/python-novaclient/+/769675
08:32:18	openstackgerrit	Manik Sidana proposed openstack/nova master: Fix typo https://review.opendev.org/c/openstack/nova/+/770067
08:36:24	openstackgerrit	Manik Sidana proposed openstack/nova master: Fix typo in warning message https://review.opendev.org/c/openstack/nova/+/770067
08:38:20	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in quota* commands https://review.opendev.org/c/openstack/python-novaclient/+/769869
08:44:34	brinzhang	gibi: good morning, noVNC patch has updated, pls review again https://review.opendev.org/c/openstack/nova/+/622336
08:45:17	gibi	brinzhang: ack,
09:33:00	gibi	lyarwood: if you have time could you check a tempest failure like https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_637/764040/8/check/nova-live-migration/637f5c6/testr_results.html based on logstash we have similar failures pretty fequrently
09:33:05	gibi	http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22failed%20to%20reach%20available%20status%20(current%20detaching)%20within%20the%20required%20time%5C%22
09:43:44	lyarwood	gibi: oh fun, not this again
09:43:55	gibi	monday fun
09:44:02	lyarwood	Device detach failed for vdb: Unable to detach the device from the live config.
09:44:19	gibi	I'm not sure that it is only that test case failing with the same symptome
09:52:27	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in limits commands https://review.opendev.org/c/openstack/python-novaclient/+/770065
09:55:49	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in quota* commands https://review.opendev.org/c/openstack/python-novaclient/+/769869
09:57:11	lyarwood	gibi: melwitt wrote an ER query for this before the break in https://bugs.launchpad.net/nova/+bug/1882521
09:57:13	openstack	Launchpad bug 1882521 in OpenStack Compute (nova) ussuri "Failing device detachments on Focal" [Undecided,New]
09:58:08	lyarwood	gibi / melwitt ; looks different to the previous hit within libvirtd logs at least, we only see the device_del commands from libvirtd to QEMU and not the DEVICE_DELETED events being emitted by QEMU.
09:58:28	lyarwood	gibi / melwitt ; we can try increasing the initial wait again but that's another hackaround
09:59:22	lyarwood	gibi / melwitt ; the ultimate fix for this is waiting for libvirtd to emit events when detaching devices https://review.opendev.org/c/openstack/nova/+/749929
10:00:56	gibi	lyarwood: could I help with ^^ somehow?
10:01:29	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.88: Replace tenant* with project* in limits commands https://review.opendev.org/c/openstack/python-novaclient/+/770065
10:01:52	lyarwood	gibi: I don't have any cycles to spend on it at the moment, feel free to take a look if you do, that's only a PoC change I was using to work out how to tap into the event framework within the libvirt driver
10:02:39	gibi	lyarwood: ack, I might need more context but I will take a look at that patch
10:03:39	lyarwood	gibi: https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainEventID & https://libvirt.org/html/libvirt-libvirt-domain.html#virConnectDomainEventDeviceRemovedCallback for the libvirt specific docs
10:21:29	gibi	lyarwood: was this https://review.opendev.org/c/openstack/nova/+/749827 superseeded by https://review.opendev.org/c/openstack/nova/+/755799 ?
10:22:51	lyarwood	gibi: yeah sorry I'm not sure why I eneded up with a different change there
10:23:01	gibi	lyarwood: OK I will drop that
10:24:30	lyarwood	gibi: one thing I've just noticed, there's a 5 second gap between the first and second device_del request in libvirtd, my change was supposed to add a 10 second delay
10:25:41	gibi	I think on master we have inc_sleep_time=10
10:26:26	lyarwood	yeah I think the retry decorator only uses that for the second wait, ./me checks
10:31:32	gibi	it seem so
10:32:02	openstackgerrit	Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/c/openstack/nova/+/748226
10:32:50	openstackgerrit	Balazs Gibizer proposed openstack/nova master: WIP/DNM libvirt: Start emitting DeviceRemovedEvent and DeviceRemovalFailedEvent events https://review.opendev.org/c/openstack/nova/+/749929
10:37:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: WIP/DNM libvirt: Start emitting DeviceRemovedEvent and DeviceRemovalFailedEvent events https://review.opendev.org/c/openstack/nova/+/749929
10:40:44	openstackgerrit	Balazs Gibizer proposed openstack/nova master: WIP/DNM libvirt: Start emitting DeviceRemovedEvent and DeviceRemovalFailedEvent events https://review.opendev.org/c/openstack/nova/+/749929
11:44:24	stephenfin	gibi: Could you re +2 this, please? https://review.opendev.org/c/openstack/nova/+/769520/
11:46:22	gibi	stephenfin: oops I missed that, done noew
11:46:23	gibi	now
11:46:35	stephenfin	Also, gibi, lyarwood, sean-k-mooney: I addressed your concerns on that OSC change, I think https://review.opendev.org/c/openstack/python-openstackclient/+/762396/
11:46:59	stephenfin	gibi++ thanks
11:48:55	gibi	I'm +1 on the osc patch too
11:49:04	stephenfin	hurrah
11:55:03	gibi	gmann: I still see tempest failing with "Multiple possible networks found, use a Network ID to be more specific." on master. e.g.: https://74ab9f21dcdd2e895607-6eadae60e9a1b03c6789f42d51c17d3a.ssl.cf2.rackcdn.com/756894/5/check/nova-next/1fb33ef/testr_results.html
11:56:38	gibi	dansmith: I have two tempest failures in test_glance_direct_import_image_to_specific_stores[id-82fb131a-dd2b-11ea-aec7-340286b6c574] I tried to create a logstash signature for it but I failed
11:56:46	gibi	1) https://zuul.opendev.org/t/openstack/build/4a78ba5901474187a3bdc49cacf84b47/logs
11:56:52	gibi	2) https://zuul.opendev.org/t/openstack/build/9488788b59904e6f9bc1280f13fc0a6b/logs
11:57:09	gibi	dansmith: is this the timeout failure you mentioned on the last week's meeting?
11:57:36	sean-k-mooney	stephenfin: +1 also
12:02:11	gibi	dansmith: I also see a full nova-ceph-multistore job timeout here https://zuul.opendev.org/t/openstack/build/8807547b038b4dd983fd7e0bb4cb4b40 it might be related too
13:45:32	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Refactor ResourceRequest constructor https://review.opendev.org/c/openstack/nova/+/769720
13:46:56	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support interface attach with QoS ports https://review.opendev.org/c/openstack/nova/+/756530
13:48:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/report.py https://review.opendev.org/c/openstack/nova/+/765152
13:48:51	gibi	stephenfin: fix your comment in the qos series ^^
14:05:12	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/c/openstack/nova/+/761924
14:06:59	stephenfin	gibi: Done. Think I'm +2 on that whole series. I'll leave it to you to find another reviewer
14:07:25	stephenfin	sean-k-mooney would probably be good to have reviewing it since I think they're re-using part of it for another spec?
14:07:32	gibi	stephenfin: thanks
14:12:23	sean-k-mooney	stephenfin: am the routed networks stuff bauzas is doing?
14:12:45	stephenfin	sean-k-mooney: no, gibi's "attach ports with QoS policies" series
14:12:49	sean-k-mooney	stephenfin: the only part im reusing form that is the network info in the requst spec
14:13:06	stephenfin	oh, it's something from bauzas' series you're reusing. nvm so
14:13:12	sean-k-mooney	oh no i dont need that for anything as far as i am aware but i can review it in anycase
14:13:27	bauzas	atm working on adding UTs for the WIP patch, btw. ;)
14:13:44	sean-k-mooney	bauzas: did you see my comments on it
14:13:58	sean-k-mooney	the functional tests are asserting incorrect behavior
14:14:42	bauzas	sean-k-mooney: err, no
14:15:04	sean-k-mooney	look at the move operation tests https://review.opendev.org/c/openstack/nova/+/749068/6/nova/tests/functional/test_servers.py
14:15:21	sean-k-mooney	the test have 3 host and 2 segments
14:15:33	sean-k-mooney	the first host is not on any segemnt and the other two are on one each
14:15:58	sean-k-mooney	so the migrate live migrate and evacuate test should not pass
14:15:59	bauzas	yup
14:16:07	bauzas	what?
14:16:10	sean-k-mooney	since there is not another host on the same segment
14:16:24	sean-k-mooney	you cant change segments on any move operation
14:16:24	bauzas	hmmm
14:16:36	bauzas	then the implementation is not good
14:16:46	bauzas	I asked for it last month
14:16:54	sean-k-mooney	unfortunetly it appears so
14:17:07	bauzas	and I thought we were supporting to move to other segments if they are in the same networks
14:17:20	sean-k-mooney	no that would change the ip
14:17:36	sean-k-mooney	segment are mapped to subnets
14:17:43	bauzas	look at https://review.opendev.org/c/openstack/nova/+/749068/6/nova/scheduler/request_filter.py#316
14:17:48	sean-k-mooney	so changing a the segment changes the subnet

Earlier   Later