Posted	Nick	Remark
#openstack-nova - 2018-07-31
19:22:30	mriedem	"Took 4.21 with stats deepcopy, took 3.85 without stats deepcopy" for 100 instances
19:22:39	mriedem	but that's totally variable at that scale
19:22:43	mriedem	and has failed a couple of times
19:23:37	efried	mriedem: Is it also going to matter that we're running "locally"?
19:24:43	mriedem	locally?
19:24:43	efried	um
19:24:56	efried	yeah, meaning no remote-ness from conductor to compute and all that.
19:25:32	mriedem	yeah for all the db stuf
19:25:47	mriedem	obviously would be better done in a devstack env
19:55:26	mriedem	i've got a recreate on that leaked node.stats thing
20:00:06	efried	woot
20:04:17	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add recreate test for bug 1784705 https://review.openstack.org/587614
20:04:17	openstack	bug 1784705 in OpenStack Compute (nova) "ResourceTracker.stats can leak across multiple ironic nodes" [High,Triaged] https://launchpad.net/bugs/1784705
20:04:18	mriedem	efried: jaypipes: dansmith: ^
20:41:35	openstackgerrit	sean mooney proposed openstack/nova master: libvirt: fix disk_bus handling for root disk https://review.openstack.org/584999
20:41:37	openstackgerrit	sean mooney proposed openstack/nova master: add regression test for bug 1759420 https://review.openstack.org/587615
20:41:37	openstack	bug 1759420 in OpenStack Compute (nova) "nova does not correctly support HW_DISK_BUS=sata or usb for kvm/qemu" [Medium,In progress] https://launchpad.net/bugs/1759420 - Assigned to sean mooney (sean-k-mooney)
21:03:36	mriedem	efried: so i think not only will i fix the leaked stats issue, but also remove the need for those deepcopy clals
21:03:37	mriedem	*calls
21:03:40	mriedem	which fixes gary's bug
21:04:07	mriedem	let the champagne flow
21:13:20	mriedem	if only i can fix these 200 or so unit tests
21:16:10	efried	mriedem: /me gets bubbles up nose, sprays
21:31:41	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Add recreate test for bug 1784705 https://review.openstack.org/587614
21:31:41	openstack	bug 1784705 in OpenStack Compute (nova) queens "ResourceTracker.stats can leak across multiple ironic nodes" [High,Triaged] https://launchpad.net/bugs/1784705
21:31:42	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636
22:10:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/587614
22:10:15	openstack	bug 1784705 in OpenStack Compute (nova) "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem)
22:10:15	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636
22:34:12	openstackgerrit	Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
23:00:35	openstackgerrit	Merged openstack/nova master: Remove unnecessary PlacementFixture setups https://review.openstack.org/587490
#openstack-nova - 2018-08-01
01:22:23	openstackgerrit	Merged openstack/nova master: Delete checking a bool opt of None condition https://review.openstack.org/390705
01:27:17	openstackgerrit	Merged openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258
01:41:41	openstackgerrit	Merged openstack/nova master: Delete orphan compute nodes before updating resources https://review.openstack.org/579922
01:42:47	openstackgerrit	Merged openstack/nova stable/queens: Fix server_group_members quota check https://review.openstack.org/581846
04:04:21	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679
04:49:43	openstackgerrit	Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
05:02:55	openstackgerrit	Merged openstack/nova master: Extract _update_to_placement method in resource tracker https://review.openstack.org/587171
05:12:42	gmann	alex_xu: hi
05:15:42	alex_xu	gmann: hi
05:16:40	gmann	alex_xu: today i do not have much for API office hour. if you have anything we can have otherwise i would like to skip today
05:16:58	alex_xu	gmann: nothing special from me
05:17:24	gmann	alex_xu: ok, let we will cancel and have next week
05:17:32	alex_xu	gmann: for the keypair quota usage, the conclude is just updating the doc I guess?
05:18:07	gmann	alex_xu: i have not checked the detail yet but i will do check tomorrow
05:18:21	alex_xu	gmann: ok, thanks
05:18:48	gmann	alex_xu: thanks
06:22:50	openstackgerrit	Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
07:08:46	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Ensure the order of AllocationRequestResources https://review.openstack.org/587700
07:22:06	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704
08:35:50	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679
08:37:35	yikun	tetsuro ^ updated, :)
08:39:31	tetsuro	yikun: Looks good to me. Thanks!
08:39:50	yikun	thanks~
08:41:25	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704
09:15:50	openstackgerrit	Pranab proposed openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378
09:30:10	openstackgerrit	Merged openstack/nova master: compute node local_gb_used include swap disks https://review.openstack.org/585928
09:33:52	openstackgerrit	Merged openstack/nova master: Define common variables for irrelevant-files https://review.openstack.org/578882
09:53:04	openstackgerrit	Merged openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434
09:59:05	kosamara	Hi bauzas, I was checking your spec numa-topology-with-rps. What's the status on this, are you actively reworking it?
09:59:36	bauzas	kosamara: yup, I'm planning to rework on it at the end of August
10:00:20	bauzas	I'll be on PTO for 3 weeks starting next Wed
10:00:31	bauzas	gibi: stephenfin: cdent: ^
10:00:57	cdent	✔
10:05:29	kosamara	bauzas: you mention RPs for PCI devices (L139 https://review.openstack.org/#/c/552924/9/specs/rocky/approved/numa-topology-with-rps.rst ). This is what I'm interested in. Nova currently doesn't create them, so I think it makes sense to do so either as part of this or as a new spec (maybe better, since your spec is already complex?)
10:05:56	gibi	bauzas: have a nice time off!
10:09:51	bauzas	kosamara: the spec is about discussing the topology, not really how to provide those devices
10:10:10	bauzas	kosamara: so, yeah maybe a new spec
10:10:30	bauzas	kosamara: at least about the upgrade between the existing and what you want
10:12:33	gibi	bauzas: FYI, we talked about the PCI device handling with kosamara in openstack-neutron as well http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2018-08-01.log.html#t2018-08-01T08:23:33
10:14:59	bauzas	gibi: ack
10:16:19	bauzas	oh, fun, GPUs look like a good driver for PCI devices :p
10:17:07	gibi	bauzas: fun indeed
10:22:54	bauzas	mdbooth: others, I'm torn with https://review.openstack.org/#/c/478330/
10:23:56	lyarwood	oh it's that time in the cycle again? :)
10:24:11	lyarwood	I'm sure this has been nack'd in the past
10:25:46	bauzas	nvm, found the spec
10:25:48	bauzas	https://blueprints.launchpad.net/nova/+spec/volume-backed-server-rebuild
10:25:58	bauzas	I'll accordingly -2 it
10:26:08	bauzas	and refer to this series
10:26:13	lyarwood	https://review.openstack.org/#/c/532407/ yeah
10:34:49	openstackgerrit	Zengzhi Wang proposed openstack/nova master: Fix using last request-id for all log when multiple request calls. https://review.openstack.org/587754
10:35:09	mdbooth	bauzas: On a call. Still want me to look?
10:35:39	bauzas	mdbooth: nope, I gently -2'd the change :)
10:35:52	mdbooth	'gently' ;)
10:36:11	bauzas	I'm French but I can be a gentleman
10:36:13	mdbooth	bauzas: You used the fluffy ban hammer
10:36:38	bauzas	mdbooth: there is one spec and already an open series
10:36:45	bauzas	I just asked the proposer to collaborate
10:37:03	bauzas	-2 just means "please stop what you're doing and discuss"
10:37:32	bauzas	it's not a "I don't like you, and for god's sake, you'll suffer pain'
10:37:45	mdbooth	bauzas: Although I recall the other discussion. I'm not in favour of the approach in this patch, tbh.
10:38:08	mdbooth	To me, rebuild on BFV means you keep the original volume and re-image it.
10:39:01	sean-k-mooney	mdbooth: is that not want was originally propsoed?
10:39:25	mdbooth	sean-k-mooney: I think that was in the other proposal I recall.
10:39:34	mdbooth	sean-k-mooney: My memory is notoriously shonky, though.
10:41:15	sean-k-mooney	sorry im thinking of something else i was vagly rembering the rescure rescure bfv changes and mixing it up with rebuild
10:41:18	bauzas	anyway, this is a behavioural change proposal, hence a spec
10:41:36	bauzas	for people wanting to get the last proposal, the spec is thrre
10:44:06	openstackgerrit	Pranab proposed openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378

Earlier   Later