Posted	Nick	Remark
#openstack-nova - 2020-08-25
15:55:24	stephenfin	artom: TIL (or rather remembered) I have core on sdk
15:56:04	frickler	artom: maybe best continue that discussion in #openstack-sdks , also eod for me now
15:56:28	artom	frickler, ack. o/
15:57:19	frickler	just as a final comment, OSC and sdk are a single project for a small while now, so in theory there shouldn't be any gap between those
15:57:34	stephenfin	jsuchome, lyarwood: Thoughts on https://review.opendev.org/#/c/574301/35/nova/image/glance.py@225 ?
15:57:48	stephenfin	Apologies if I've missed some context on that decision (to bring back the deprecated opt)
15:59:04	artom	frickler, "in theory" :D
15:59:40	lyarwood	stephenfin: I've not heard any plans for support for other support
15:59:45	jsuchome	stephenfin: in general, I don't think there's a big chance to expand it ... but well, who knows
16:00:04	lyarwood	stephenfin: I'd prefer however that we ruled that out at a later date
16:01:20	sean-k-mooney	i could see use adding swift maybe
16:01:37	sean-k-mooney	to pull the image drectly from swrift rather then streaming it via glance
16:01:40	jsuchome	I think it's already mentioned in the blueprint btw
16:01:55	sean-k-mooney	no one has asked for that however
16:02:39	sean-k-mooney	but ya swift is the only thing i can think of that we might ever expand it for and even then im not sure it makes sense
16:02:47	sean-k-mooney	im jsut thinking off the top of my head
16:03:20	sean-k-mooney	the advantage of using the exsiting deprecated option is installer tools whoudl already support it
16:03:36	sean-k-mooney	if we use anything else that creates more work for them
16:03:56	gibi	o/
16:04:05	sean-k-mooney	o/ enjoy your evening
16:04:19	gibi	thanks
16:05:29	sean-k-mooney	which nova cores like networking and libvirt :)
16:06:10	sean-k-mooney	zuul hasnt finished running yet but i have an old patch that i want to make sure lands before m3
16:06:58	sean-k-mooney	https://review.opendev.org/#/c/602432/ but it does not need to be reviewd right away the neutron dep merged earlier today so i finally got around to rebasing it
16:11:47	openstackgerrit	Merged openstack/nova master: [goal] Prepare for job migration to Ubuntu Focal (20.04) https://review.opendev.org/738126
17:01:02	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add helpers for rebuild, cold migrate, and shelve/unshelve https://review.opendev.org/747792
17:01:02	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add emulated TPM support to Nova https://review.opendev.org/631363
17:01:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Add docs for vTPM support https://review.opendev.org/739213
17:01:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.compute.manager' https://review.opendev.org/742863
17:01:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/741995
17:01:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
17:01:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
17:01:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
17:09:17	openstackgerrit	Merged openstack/nova master: Add reno for deprecated APIs policy changes https://review.opendev.org/747784
17:22:30	gmann	gibi: reno patch is merged you can mark policy-defaults-refresh-deprecated-apis BP completed now.
18:25:08	openstackgerrit	Lee Yarwood proposed openstack/nova stable/pike: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747978
19:53:00	openstackgerrit	Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
20:18:01	openstackgerrit	Merged openstack/nova master: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747969
21:22:44	lyarwood	melwitt: thanks for the stable reviews, would you also mind looking at https://review.opendev.org/#/c/747357/ https://review.opendev.org/#/c/744553/ https://review.opendev.org/#/c/744550/ https://review.opendev.org/#/c/742706/ & https://review.opendev.org/#/c/729177/
21:34:13	melwitt	ack will do
21:38:26	lyarwood	awesome thanks :)
21:47:52	openstackgerrit	Elancheran S proposed openstack/nova stable/ussuri: Removes the delta file once image is extracted https://review.opendev.org/748034
21:48:45	openstackgerrit	Elancheran S proposed openstack/nova stable/train: Removes the delta file once image is extracted https://review.opendev.org/748035
23:29:50	openstackgerrit	Merged openstack/nova master: db: fix database migrations when name includes dash https://review.opendev.org/747164
#openstack-nova - 2020-08-26
00:54:17	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
02:51:06	openstackgerrit	Yingji Sun proposed openstack/nova master: Set different VirtualDevice.key https://review.opendev.org/713565
02:55:58	openstackgerrit	Merged openstack/nova stable/stein: Should not skip volume_size check for bdm.image_id == image_ref case https://review.opendev.org/744553
03:38:39	openstackgerrit	Merged openstack/nova stable/ussuri: compute: Do not allow rescue attempts using volume snapshot images https://review.opendev.org/742706
04:14:50	openstackgerrit	Merged openstack/nova stable/rocky: compute: Allow snapshots to be created from PAUSED volume backed instances https://review.opendev.org/729177
04:25:44	openstackgerrit	Elancheran S proposed openstack/nova stable/ussuri: Removes the delta file once image is extracted https://review.opendev.org/748034
04:51:04	openstackgerrit	Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
07:18:04	openstackgerrit	Keigo Noha proposed openstack/nova master: Change default num_retries for glance to 3 https://review.opendev.org/740389
07:35:08	gibi	gmann: cool. I marked the policy by as implemented
07:39:48	brinzhang	gibi: hi, good morning
07:40:04	gibi	brinzhang: hi
07:40:36	brinzhang	gibi: https://review.opendev.org/#/c/715326/19/nova/compute/manager.py@3654 this you mean, we should add the kwargs['accel_uuids'] to the rebuild() and _rebuild_default_impl() interfact, right?
07:41:31	brinzhang	that we should change the virt/driver.py and virt/ironic/driver.py rebuild() functions?
07:42:37	gibi	brinzhang: yes, I mean that this parameter should be part of the https://github.com/openstack/nova/blob/master/nova/virt/driver.py#L285 interface
07:43:31	brinzhang	gibi: does it need to change the irnic driver? https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1669
07:43:43	brinzhang	s/irnic/ironic/
07:43:49	gibi	yes
07:44:07	gibi	it will be a parameter that the ironic driver gets but ignores at the moment
07:46:09	brinzhang	yeah, we will add this parameter, and send a mail to the ML later ^
07:46:15	brinzhang	gibi: thanks
07:57:43	gibi	brinzhang: thanks
08:01:48	openstackgerrit	Alexandre Arents proposed openstack/nova master: Add a lock to prevent race during detach/attach of interface https://review.opendev.org/747957
08:17:35	bauzas	gibi: fwiw, I'm starting to sheperd the rest of the provider config series
08:17:43	bauzas	shepherd* even
08:17:52	gibi	bauzas: thanks.
08:18:21	bauzas	it needed me to look back at the spec, but thanks for having started to review the series
08:21:37	gibi	unfortunately the implementation changed ownwer couple of times so there are thing that needs (needed) detective work to understand the context
08:26:43	rmart04	Hey All, Apologies if this isn't the right place to ask, but are there any obvious memory limitations gotchas when using dedicated cpu policy? I'm seeing OOM killing off my instances when the host has plenty of memory left :/ (Stein / CentOS7 Packages / KVM / Kolla-Ansible)
08:27:13	openstackgerrit	Elod Illes proposed openstack/nova stable/queens: compute: Allow snapshots to be created from PAUSED volume backed instances https://review.opendev.org/729178
08:30:32	openstackgerrit	Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326
08:42:43	stephenfin	lyarwood, gibi: So I've got three series on the go rn. What would I need to pay you to get at least one of them moving? :)
08:43:01	gibi	stephenfin: the vpmem is on my menu for today
08:43:07	gibi	is that one of the three
08:43:08	gibi	?
08:43:21	stephenfin	unless you meant vTPM, no :(
08:43:27	gibi	vtpm
08:43:33	stephenfin	then yes \o/
08:43:35	gibi	cool
08:43:36	stephenfin	hurrah
08:43:43	gibi	then you can expect review on that in the coming hours
08:44:01	stephenfin	awesome. Thanks, gibi :)
08:44:22	gibi	no problemo, it is in the runway slot so it in focus :)
08:46:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.compute.manager' https://review.opendev.org/742863
08:46:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/741995
08:46:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
08:46:15	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
08:46:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
08:54:33	stephenfin	alex_xu: Can you look at https://review.opendev.org/#/c/744021/ and its predecessor today? You probably have appropriate context
08:54:49	stephenfin	alex_xu: I suspect it impacts the mixed CPU work too
08:58:47	openstackgerrit	Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326
09:01:18	stephenfin	rmart04: Using the dedicated CPU policy implies a single-node NUMA topology by default, and using a NUMA topology implies consuming memory from a single host NUMA node for each guest NUMA node
09:02:44	stephenfin	rmart04: That's a roundabout way of saying that I suspect you haven't reserved memory on per-node basis and may have allocated something like hugepages unevenly across NUMA nodes
09:05:08	stephenfin	rmart04: nova won't let you overcommit against a host NUMA node using nova instances, but it can't account for host processes or huge page allocations consuming memory on that node. You've to do that manually
09:07:03	stephenfin	rmart04: There are two config options of value here: 'reserved_host_memory_mb', which is a total amount of memory to reserve across all NUMA nodes, and 'reserved_huge_pages' which is a per-node reservation that applies when you're using explicit pagesizes
09:07:23	bauzas	stephenfin: gibi: fwiw, after the provider series, I was heading to review your vTPM series
09:07:47	stephenfin	rmart04: I suggest whacking up the first ('reserved_host_memory_mb'). It would be good to inspect idle memory consumption without instances on the host and use that value with some buffer.

Earlier   Later