Posted	Nick	Remark
#openstack-nova - 2020-06-18
21:03:24	mordred	in fact, I might just push up changes for this forging the author to say they're from you
21:03:38	mordred	efried: "surprising"
21:04:30	efried	https://review.opendev.org/508345
21:04:30	efried	https://review.opendev.org/655985
21:04:30	efried	(I haven't opened those, they're just the ones on my dashboard with 'cinder' in the title)
21:26:36	melwitt	dansmith: left comments on the bottom two rbd multi store patches. bottom one is a docstring update needed, next patch looks to be one small case of missing test coverage and a couple doc-related things
21:39:41	dansmith	melwitt: thanks
21:45:12	openstackgerrit	Dan Smith proposed openstack/nova master: Plumb image import functionality through our glance module https://review.opendev.org/731550
21:45:13	openstackgerrit	Dan Smith proposed openstack/nova master: Make libvirt able to trigger a backend image copy when needed https://review.opendev.org/656998
22:18:18	openstackgerrit	Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425
22:37:21	melwitt	gmann: is/was there a change yet to make stable/stein grenade n-v? wondering if it's safe to recheck changes yet
22:37:39	melwitt	also note that nova-live-migration is failing on stable/stein too
22:38:02	melwitt	saying virtualenv missing
22:38:20	gmann	melwitt: i am trying to get working on train with this and tomorrow I am planning for making it n-v on stein - https://review.opendev.org/#/c/736284/9
22:38:33	melwitt	ack
22:39:11	gmann	nova-live-migration is difficult one as that is legacy job, on devstack, grenade we fixed that error - https://review.opendev.org/#/c/736750/1
22:41:20	gmann	melwitt: but there are multiple updates happening on devstack/grenade and image side. so let's see how it behaves once we get all those in
22:41:42	gmann	clark is updating images also which were not present due to disk full.
#openstack-nova - 2020-06-19
00:29:53	openstackgerrit	Wenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider https://review.opendev.org/719163
03:21:28	openstackgerrit	Wenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider https://review.opendev.org/719163
04:38:05	arjunbaindur	hey, have a question about configuring PCI passthrough. We have a device that has 2 functions. On the hypervisor, the PCI address has it on same bus and same slot. When it shows up inside the VM, the PCI address has it under different slots
04:39:01	arjunbaindur	for example, suppose device has two functions at 83:00.0 and 83:00.1
04:40:20	arjunbaindur	Under the VM, the devices show up under different slots, for example 0:5.0 and 0:6.0
04:40:22	arjunbaindur	any idea why?
06:29:15	openstackgerrit	Wenping Song proposed openstack/nova master: Delete sub resource provider when delete resource provider https://review.opendev.org/719163
08:45:01	openstackgerrit	Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736957
08:51:06	openstackgerrit	Stephen Finucane proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962
08:59:25	gibi	aarents: there is a bug in the image cache disk reservation patch. https://bugs.launchpad.net/nova/+bug/1884214
08:59:25	openstack	Launchpad bug 1884214 in OpenStack Compute (nova) "reserve disk usage for image cache fails on a fresh hypervisor" [Medium,Confirmed] - Assigned to Balazs Gibizer (balazs-gibizer)
09:04:19	gibi	aarents: I will push a fix soon
09:21:46	aarents	gibi: I saw that, thank you, I was wondering if nova create _base directory only when needed at first cache() but I did not check
09:29:26	gibi	aarents: yeah I also missed that case
09:37:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Guard against missing image cache directory https://review.opendev.org/736964
09:39:21	gibi	aarents: ^^
09:41:13	aarents	gibi: checking
09:54:59	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Guard against missing image cache directory https://review.opendev.org/736964
11:30:06	openstackgerrit	Wenping Song proposed openstack/nova master: Delete resource provider in tree by top-down traversable order https://review.opendev.org/719163
11:34:17	songwenping_	Hi, gibi. Pls review my latest commit for the patch:https://review.opendev.org/719163
11:55:38	gibi	songwenping_: I will try to look at it next week
11:56:33	songwenping_	OK. Thanks.
13:05:25	openstackgerrit	Huaqiang Wang proposed openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance https://review.opendev.org/736997
13:31:57	sangeet	efried .. if the put cafile only in identity, compute fail to find it.
13:38:05	stephenfin	huaqiang: I'm about to push up a rebase of the bp/use-pcpu-and-vcpu-in-one-instance series to address comments on the bottom 10 or so patches. Is that okay with you?
14:25:22	efried	sangeet: ack.
14:27:07	dansmith	gibi: wanna circle back on these? just made some minor changes for melwitt: https://review.opendev.org/#/c/731550/4
14:39:03	dansmith	stephenfin: thoughts on this? https://review.opendev.org/#/c/736964/2/nova/virt/libvirt/imagecache.py
14:39:09	dansmith	I know it's kinda minor, but...
14:40:17	stephenfin	dansmith: makes sense to me
14:40:53	dansmith	stephenfin: okay if gibi doesn't show up here soon to object I'll just fix it for him and we can push that in
15:00:55	openstackgerrit	Dan Smith proposed openstack/nova master: WIP: Add libvirt RBD configuration doc https://review.opendev.org/731577
15:00:56	openstackgerrit	Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184
15:10:31	huaqiang	stephenfin: I am OK with your rebase. Go ahead.
15:10:42	stephenfin	sweet, thanks
15:11:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Update and correct typing information https://review.opendev.org/714694
15:11:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add typing information https://review.opendev.org/714695
15:11:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Add validation for 'cpu_realtime_mask' https://review.opendev.org/468203
15:11:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Split instance NUMA object tests https://review.opendev.org/714696
15:11:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Replace 'cpu_pinning_requested' helper https://review.opendev.org/714697
15:11:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Don't consider overhead CPUs for unpinned instances https://review.opendev.org/714698
15:11:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Remove handling of pre-Train compute nodes https://review.opendev.org/714699
15:11:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Rework 'get_realtime_constraint' https://review.opendev.org/714700
15:11:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456
15:11:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701
15:11:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Remove '_numa_fit_instance_cell_with_pinning' https://review.opendev.org/714703
15:11:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Introduce 'pcpuset' field for InstanceNUMACell https://review.opendev.org/714658
15:11:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: notifications: add the pcpuset info to instance numa cell payload https://review.opendev.org/728480
15:11:54	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267
15:11:54	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: apply mixed instance CPU policy https://review.opendev.org/728481
15:11:55	openstackgerrit	Stephen Finucane proposed openstack/nova master: Validate CPU pinning configuration for mixed instance https://review.opendev.org/716268
15:11:55	openstackgerrit	Stephen Finucane proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269
15:11:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: online data migration for cpuset of dedicated instance https://review.opendev.org/718493
15:35:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Rework 'get_realtime_constraint' https://review.opendev.org/714700
15:35:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Allow 'hw:cpu_realtime_mask' to be omitted https://review.opendev.org/737026
15:35:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456
15:35:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701
15:35:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Remove '_numa_fit_instance_cell_with_pinning' https://review.opendev.org/714703
15:35:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Introduce 'pcpuset' field for InstanceNUMACell https://review.opendev.org/714658
15:35:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: notifications: add the pcpuset info to instance numa cell payload https://review.opendev.org/728480
15:35:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267
15:35:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: apply mixed instance CPU policy https://review.opendev.org/728481
15:35:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Validate CPU pinning configuration for mixed instance https://review.opendev.org/716268
15:35:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269
15:35:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: online data migration for cpuset of dedicated instance https://review.opendev.org/718493
15:35:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: scheduler: mixed instance CPU resource translating https://review.opendev.org/728482
15:35:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: flavor: Enable 'hw:cpu_dedicated_mask' for creating a mixed instance https://review.opendev.org/714706
15:35:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: compute: bump nova-compute service version for mixed instance https://review.opendev.org/716271
15:35:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: create 'mixed' instance for realtime and shared CPUs https://review.opendev.org/728485
15:35:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt/hardware: refactor numa_get_constraints https://review.opendev.org/718294
15:35:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: Setup 'mixed' instance through 'PCPU' and 'VCPU' resource https://review.opendev.org/713355
15:52:05	artom	stephenfin, wait, are you proposing we *remove* the "must start with a ^" validation on the realtime mask?
15:52:17	artom	So we call it a mask, except when the user doesn't feel like, and then it's not a mask?
15:52:37	artom	I actually had no idea it was validated to start with ^ in 2.86
15:52:53	stephenfin	it's still a mask
15:52:55	artom	My opinion would be to keep it, and make the dedicated mask the same
15:53:02	stephenfin	it's just not an exclusion mask
15:53:21	stephenfin	it behaves sanely with this change
15:53:55	artom	I've always understood mask to mean exclusion mask
15:54:16	artom	In fact, "exclusion mask" would be a pleonasm in my book
15:54:25	dansmith	not me
15:54:31	stephenfin	that's not how taskset would work

Earlier   Later