Posted	Nick	Remark
#openstack-nova - 2020-09-14
11:30:46	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
11:34:12	openstackgerrit	Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175
12:14:02	openstackgerrit	Merged openstack/nova master: Update compute rpc version alias for victoria https://review.opendev.org/751355
12:14:10	openstackgerrit	Merged openstack/nova master: doc: mark the max microversion for victoria https://review.opendev.org/751342
13:05:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/749312
13:05:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316
13:05:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315
15:05:11	openstackgerrit	Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175
15:09:25	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/748453
15:13:59	stephenfin	elod: Hey, could I ask you to stick https://review.opendev.org/#/q/topic:bug/1879878+status:open+branch:stable/ussuri to your review queue, please?
16:02:54	openstackgerrit	Stephen Finucane proposed openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947
16:02:55	openstackgerrit	Stephen Finucane proposed openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949
16:02:55	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948
16:02:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Expand SR-IOV live migration tests with NUMA https://review.opendev.org/749360
16:02:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950
16:11:52	elod	stephenfin: sure, I might arrive to those tomorrow, but I'll review them :]
16:20:25	gmann	stephenfin: gibi dansmith can you review this which will help to add ceph job on devstack gate too - https://review.opendev.org/#/c/750078/
16:33:00	melwitt	gmann: I got that one
16:33:18	gmann	melwitt: stephenfin thanks
16:55:36	dansmith	sorry, was on a call
18:12:09	openstackgerrit	Merged openstack/nova master: Run voting jobs in gate pipeline also https://review.opendev.org/750078
18:23:33	tosky	lyarwood: I left a comment in the nova-evacuate review, but I may not have been clear enough, so in case please ask
18:29:19	lyarwood	tosky: yeah understood, was hoping I wouldn't have to port this now but I'll give it a go this evening
19:31:53	openstackgerrit	Merged openstack/nova master: Remove write_to_file. https://review.opendev.org/645087
19:32:00	openstackgerrit	Merged openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947
19:32:08	openstackgerrit	Merged openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948
21:12:47	openstackgerrit	Adam Harwell proposed openstack/nova master: Add scheduler filter for routed_segments https://review.opendev.org/709280
21:40:46	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
22:03:39	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
22:53:06	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
23:05:06	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747361
#openstack-nova - 2020-09-15
02:07:27	openstackgerrit	Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175
07:39:18	openstackgerrit	Federico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain https://review.opendev.org/750652
08:03:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315
08:03:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316
08:08:57	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
08:16:14	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
08:34:25	bauzas	whoops my manners
08:34:31	bauzas	good morning Nova
08:34:54	lyarwood	Morning
08:44:47	yingjisun	I found thousands of "2020-09-14 19:56:17.152 1869 INFO nova.metadata.wsgi.server [-] 192.168.131.34,192.168.131.68 "GET /openstack/latest/vendor_data.json HTTP/1.1" status: 200 len: 128 time: 0.0072370"
08:45:33	yingjisun	is there a periodic task that query the metadata ?
08:48:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: test_evacuate: Wait until subnode is down before starting tests https://review.opendev.org/751065
09:07:03	bauzas	yingjisun: well, I'd expect cloud-init to fetch this vendordata
09:07:16	bauzas	so, litterally every instance which is booting
09:08:07	yingjisun	this is the strange point. I see thousands of such log with 15 minutes. I do not think there are so much instance booting
09:08:18	bauzas	sean-k-mooney: stephenfin: I honestly think we shouldn't deprecate stuff post-FF
09:08:23	bauzas	sean-k-mooney: stephenfin: https://review.opendev.org/#/c/745605/
09:09:06	lyarwood	bauzas: For what reason?
09:09:27	stephenfin	It seemed super low risk
09:09:33	yingjisun	I only see 68 "Starting instance ..." in the log.
09:09:45	bauzas	lyarwood: because we have something called "Feature Freeze"
09:09:58	bauzas	even with low risk deprecations
09:10:10	bauzas	that's maybe me who is a bit conservative
09:10:12	stephenfin	yingjisun: https://docs.openstack.org/nova/latest/admin/vendordata.html
09:10:18	bauzas	but is it really a thing ?
09:10:26	stephenfin	yingjisun: The vendordata feature provides a way to pass vendor or deployment-specific information to instances. This can be accessed by users using the metadata service or with config drives.
09:10:28	lyarwood	sorry I don't see how feature freeze is even related to deprecations
09:10:40	bauzas	lyarwood: you're possibly right
09:10:45	bauzas	I just raise a concern
09:10:55	lyarwood	ff is for new things, deprecations are just to highlight removal in a future release
09:10:56	bauzas	in theory, RCs are there for focusing on bugds
09:11:13	bauzas	but I'm pedantic
09:11:15	lyarwood	yeah true I just didn't map deprecations to adding risk
09:11:22	lyarwood	removing stuff maybe
09:11:45	lyarwood	the only reason I'm still going through stephenfin's xenapi removal stuff is because the bulk has already landed
09:11:53	lyarwood	so we are only removing stuff that's dead anyway now
09:12:41	stephenfin	yingjisun: I suspect you have either something like cloud-init on the guests indexing the metadata service
09:12:48	stephenfin	s/either //
09:14:03	bauzas	stephenfin: that's my expectations
09:14:16	stephenfin	lyarwood, bauzas: Yeah, it would be good to discuss as a team whether we want to proceed with that remove-xenapi series and close it out this cycle, or wait until Wallaby
09:14:17	bauzas	that's what I explained
09:14:24	stephenfin	bauzas: oh, so you did /o\
09:14:29	bauzas	it's cloud-init thingies
09:14:30	stephenfin	sorry, I missed that
09:14:35	bauzas	so litterally a guest question
09:14:38	stephenfin	yuuup
09:14:48	lyarwood	stephenfin: yeah I'd like to remove the already dead stuff before RC if we can
09:15:05	lyarwood	or did I miss rc already being cut
09:15:12	bauzas	no
09:15:16	bauzas	next week
09:15:23	bauzas	and https://review.opendev.org/#/c/751045/1 is -W because of this
09:15:44	bauzas	I'd certainly prefer if we would stop merging things that would touch nova features
09:15:52	lyarwood	k, if we have a week I'd say push ahead removing dead code but would be good to bring up on Thursday if it's still going on
09:16:10	lyarwood	bauzas: FWIW the bulk of the removal has already landed so listing it there is valid now
09:16:21	bauzas	lyarwood: this is
09:16:34	bauzas	my concern was more about any other deprecation or removal we were about to provide
09:17:06	lyarwood	kk what's the deadline for the prelude? after rc?
09:17:32	lyarwood	IOW can we just set a deadline of rc for deprecations and respin the prelude?
09:17:44	bauzas	lyarwood: no, prelude has to merge before we branch
09:17:52	stephenfin	it has to merge before the branch
09:17:54	stephenfin	yeah
09:17:58	bauzas	but again, the whole point of RCs is to focus on bugfixes
09:18:06	stephenfin	otherwise it's not included in the 22.0.0 tag
09:18:12	lyarwood	yeah understood
09:18:23	lyarwood	but we aren't at rc yet soooooo
09:18:42	lyarwood	I'd think deprecations are fine until we land the prelude and branch?
09:18:49	bauzas	and us pushing things down the line after FF makes me a bit sad since we all communicate on the glorious ideal of whight knights only caring about bugs for the next 2 weeks
09:19:04	bauzas	call me an idealist
09:19:32	lyarwood	isn't that post rc?

Earlier   Later