Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-04
14:55:42 sean-k-mooney gibi: you can almost always use the gender nutral form in english if you prefer although singular they or it can annoy some
14:56:01 mriedem brinzhang: the reason for the functional test failures is inline https://review.opendev.org/#/c/673133/
14:56:06 stephenfin mriedem, gibi: I need a way to prevent people only setting '[compute] cpu_dedicated_set' on hosts that have non-pinned instances (to prevent https://review.opendev.org/#/c/674895/22/nova/virt/libvirt/driver.py@7358)
14:56:07 sean-k-mooney for some reason people dont like to be refered to as "it"
14:56:11 mriedem it's because of stubs in the api samples tests
14:56:22 gibi sean-k-mooney: thank. I try to remember to that
14:56:28 mriedem if we were using real compute services and the CinderFixture we wouldn't have that problem, but re-writing those tests is beyond this patch
14:56:43 stephenfin I'm thinking iterating through every single instance on the host in 'LibvirtDriver.init_host' is too expensive
14:57:12 gibi stephenfin: don't we already iterating every instance in init_host?
14:57:30 sean-k-mooney we do in update_resouces or whatevre its called
14:57:43 sean-k-mooney i thikn we call that as part of init
14:57:49 stephenfin gibi: If we do, I don't see it
14:57:52 sean-k-mooney we iterate over all instance to plug there interfaces
14:57:55 sean-k-mooney at the very least
14:58:01 stephenfin It all seems to be basic config opt checks
14:58:03 mriedem gibi might be thinking of ComputeManager.init_host
14:58:03 stephenfin Hmm
14:58:12 dansmith artom: check me here: https://review.opendev.org/#/c/634606/73
14:58:17 gibi yeah, I think I remember ComputeManager.init_host
14:58:18 openstackgerrit Luyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447
14:58:19 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs https://review.opendev.org/678448
14:58:20 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450
14:58:20 openstackgerrit Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449
14:58:21 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
14:58:21 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
14:58:22 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
14:58:22 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
14:58:23 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
14:58:23 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
14:58:24 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
14:58:31 dansmith artom: that's a big nasty patch, so it's a good sign that I only had a couple minor things
14:59:02 mriedem ComputeManager.init_host gets all instances https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1256
14:59:07 stephenfin Oh, ComputeManager.init_host
14:59:26 stephenfin I was thinking this would have to live in 'LibvirtDriver.init_host'
14:59:37 artom dansmith, witness me!
14:59:57 mriedem stephenfin: is cpu_dedicated_set a new config option in that series?
15:00:02 stephenfin yup
15:00:32 mriedem how do you determine if an instance is pinned or not?
15:00:38 mriedem looking at the instance.numa_topology?
15:01:01 cdent try to lift it up
15:01:05 cdent if it doesn't move...
15:01:08 stephenfin mriedem: Instance.numa_topology.cpu_pinning_requested
15:01:22 mriedem stephenfin: so none of that is libvirt specific yeah?
15:01:45 stephenfin technically no. It just won't do anything without libvirt
15:02:16 mriedem for now
15:02:27 stephenfin That 'cpu_pinning_requested' property would never be set on non-libvirt driver
15:02:30 stephenfin Correct
15:02:48 mriedem so...add a compute driver capability so a nova-compute running vsphere doesn't have to look at that for 1K+ instances managed by that host
15:03:35 mriedem if self.driver.capabilities.get('supports_some_hw_magic', False):
15:03:38 mriedem # do expensive thing
15:04:16 stephenfin so not "if CONF.compute_driver == 'libvirt'" ?
15:04:20 mriedem alternatively we change the ComputeDriver.init_host interface to pass the list of instances on that host into the virt driver
15:04:24 openstackgerrit Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449
15:04:24 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448
15:04:25 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
15:04:25 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450
15:04:26 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
15:04:26 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
15:04:27 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
15:04:27 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
15:04:28 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
15:04:28 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
15:04:29 artom stephenfin, eww
15:04:33 mriedem stephenfin: right that ^
15:04:35 artom stephenfin, I have no context, but errr
15:04:38 artom *eww
15:04:40 mriedem we generally don't ever want to look at that config to make decisions
15:04:46 stephenfin fair fair
15:04:56 mriedem *something something factory pattern*
15:05:34 mriedem anyway, maybe people would prefer to just swap those 3 lines of code in ComputeManager.init_host to pass the InstanceList to the driver and let the driver sort it out
15:05:55 mriedem either way you're going to explode and kill the start of the service if it's misconfigured correct?
15:07:32 stephenfin yup
15:07:37 luyao dansmith: comments addressed. https://review.opendev.org/#/c/678447/9 and https://review.opendev.org/#/c/678448/12
15:12:28 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448
15:12:28 openstackgerrit Luyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447
15:12:29 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450
15:12:29 openstackgerrit Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449
15:12:30 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
15:12:30 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
15:12:31 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
15:12:31 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
15:12:32 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
15:12:32 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
15:12:33 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
15:21:43 openstackgerrit Eric Fried proposed openstack/nova master: Bump min for oslo.service & .privsep to fix SIGHUP https://review.opendev.org/679974
15:22:17 openstackgerrit Adam Spiers proposed openstack/nova master: Ensure q35 machine type is used when booting with SEV https://review.opendev.org/680065
15:23:12 kashyap cfriesen: Hi, you or someone you know picking this up? -- https://review.opendev.org/#/c/621646/ ("Preserve UEFI NVRAM variable store")
15:24:19 kashyap Oh, actually zhu.boxiang has picked it up lately
15:25:20 cfriesen kashyap: yeah, was just going to say. :)
15:30:23 sean-k-mooney dansmith: i think the cross rpc migration is working. i have not looked at the logs to confim but it succeed via horizon
15:30:49 dansmith sean-k-mooney: okay, with a numa flavor or non-numa flavor?
15:31:00 sean-k-mooney with cpu pinning enabled
15:31:03 sean-k-mooney so numa
15:31:31 dansmith okay, so the behavior we expect is for it to work, not claim, and just translate the topo directly over to the other side, for better or worse, correct?
15:31:40 dansmith artom: ^
15:31:50 openstackgerrit Adam Spiers proposed openstack/nova master: Ensure q35 machine type is used when booting with SEV https://review.opendev.org/680065
15:31:58 aspiers efried, kashyap, sean-k-mooney: ^^^ this implements what we agreed yesterday
15:32:01 artom dansmith, yep, aka, what we have now
15:32:21 dansmith yeah, okay
15:32:24 kashyap aspiers: Already reading the code for a couple of mins (while on a call)

Earlier   Later