| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-11 | |||
| 16:14:30 | sean-k-mooney | stephenfin: efried is there anything i can do to help with either the PCPU in plamcenet seriese or pmem? | |
| 16:15:40 | efried | sean-k-mooney: I'm at a point where I'm willing to proxy your vote on https://review.opendev.org/#/c/678455/ -- once we've agreed whether it needs to be changed or not. | |
| 16:15:56 | efried | I'm not clear whether y'all came to an agreement above? | |
| 16:16:02 | stephenfin | sean-k-mooney: I'm on the vpmem one, so if you fancy looking at either https://review.opendev.org/#/c/674895/ | |
| 16:16:08 | stephenfin | efried: alex_xu is on it | |
| 16:16:18 | mriedem | TestInstanceNotificationSampleWithMultipleCompute.test_multiple_compute_actions just reset the gate again, | |
| 16:16:22 | mriedem | i think we have a regression | |
| 16:16:47 | stephenfin | We agreed it was better to keep the one guest NUMA node to one host NUMA node model, even if we don't do proper affinity for the VPMEM devices | |
| 16:17:12 | efried | and that's because qemu has some checks in place that don't allow us to do the slightly better thing? | |
| 16:17:13 | sean-k-mooney | stephenfin: so like pci passthoug with affinity disabled | |
| 16:17:25 | stephenfin | efried: Yup | |
| 16:17:30 | efried | okay cool. | |
| 16:17:42 | stephenfin | sean-k-mooney: Yeah, good analogy | |
| 16:18:01 | stephenfin | efried: I did have a comment on https://review.opendev.org/#/c/678448/20 though | |
| 16:18:36 | stephenfin | I'm not sure if it's worth respinning for but if we don't, we're stuck with that object name forever | |
| 16:19:32 | sean-k-mooney | whats SlugField | |
| 16:20:15 | sean-k-mooney | oh somting that is alpha numeric or has _ | |
| 16:20:27 | stephenfin | sean-k-mooney: That's the terminology I'm used to from Django. I thought it was more widespread than that, tbh | |
| 16:20:40 | stephenfin | yeah, exactly | |
| 16:20:44 | sean-k-mooney | not really but i have seen you use it before | |
| 16:21:16 | stephenfin | here we go https://en.wikipedia.org/wiki/Clean_URL#Slug | |
| 16:21:24 | efried | stephenfin: I thought slug was with hyphens, not underscores | |
| 16:21:33 | stephenfin | Good point. It is | |
| 16:21:44 | efried | oh, according to that wp article it's either. | |
| 16:22:02 | efried | but also lowercased | |
| 16:22:08 | efried | and also specific to URLs | |
| 16:22:15 | sean-k-mooney | ya this is specifcly only uppercase | |
| 16:22:27 | efried | I was beaten up about this by jaypipes at some point when I tried to name a method slugify | |
| 16:22:27 | efried | in fact | |
| 16:22:28 | sean-k-mooney | its basically what i know as CONST_CASE | |
| 16:22:40 | efried | I think it was the method now known in os-traits and os-resource-classes as normalize_name :) | |
| 16:22:41 | efried | so no | |
| 16:22:44 | efried | you can't call it a slug | |
| 16:22:53 | efried | because Jay will kick your ass | |
| 16:23:17 | efried | IMO it's definitely not worth respinning for this. | |
| 16:23:41 | efried | even if this object winds up single use | |
| 16:23:59 | stephenfin | Fair point. I was on the fence as it was | |
| 16:24:18 | sean-k-mooney | we can alwas do it later | |
| 16:24:33 | sean-k-mooney | if we have a common felid type this can jsut be an alias or inherit | |
| 16:24:41 | stephenfin | very true | |
| 16:24:47 | stephenfin | I switched to +2 anyway | |
| 16:24:51 | mriedem | https://bugs.launchpad.net/nova/+bug/1843615 | |
| 16:24:52 | openstack | Launchpad bug 1843615 in OpenStack Compute (nova) "TestInstanceNotificationSampleWithMultipleCompute.test_multiple_compute_actions intermittently failing since Sept 10, 2019" [Undecided,New] | |
| 16:25:09 | stephenfin | on account of it not being a slug and therefore probably not as reusable as I thought it would be | |
| 16:25:24 | sean-k-mooney | stephenfin: once question does the qutotat code need to be move down in the series | |
| 16:25:41 | stephenfin | sean-k-mooney: Yup, see my comment on same. I'll move it when I squash that fixup patch back | |
| 16:25:54 | stephenfin | except for the functional test. That has to wait til the end, of course | |
| 16:25:55 | sean-k-mooney | ah cool | |
| 16:26:12 | stephenfin | the squash back which will be happening any minute now | |
| 16:29:43 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 16:29:43 | openstackgerrit | Alex Xu proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 16:29:44 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 16:29:44 | openstackgerrit | Alex Xu proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 16:29:45 | openstackgerrit | Alex Xu proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | |
| 16:30:39 | alex_xu | stephenfin: sean-k-mooney ^ done \o/ | |
| 16:31:09 | alex_xu | really small change actually | |
| 16:31:14 | alex_xu | and tested | |
| 16:31:33 | efried | sean-k-mooney: Can I abandon https://review.opendev.org/#/c/681474/ ? | |
| 16:31:48 | sean-k-mooney | oh yes | |
| 16:33:18 | alex_xu | stephenfin: sean-k-mooney removed the workaround code at https://review.opendev.org/678455, then a super simple check at https://review.opendev.org/678456 | |
| 16:34:30 | sean-k-mooney | alex_xu: i expect the code is simpler overall | |
| 16:34:52 | alex_xu | sean-k-mooney: yea | |
| 16:35:14 | alex_xu | oops, I forget to remove that useless need_pin parameters | |
| 16:36:23 | sean-k-mooney | im not sure get_guest_numa is correct it proablyt should be get_guest_singel_numa | |
| 16:37:42 | efried | these sound like fupables | |
| 16:37:52 | alex_xu | sean-k-mooney: emm? | |
| 16:38:21 | sean-k-mooney | https://review.opendev.org/#/c/678455/30/nova/virt/hardware.py | |
| 16:38:36 | sean-k-mooney | it ignores the flavor and always returns a single numa node | |
| 16:38:51 | sean-k-mooney | so get_guest_numa is missleading | |
| 16:39:03 | alex_xu | sean-k-mooney: sorry, that one should be removed, noone use that method anymore | |
| 16:39:13 | sean-k-mooney | ah ok | |
| 16:39:19 | sean-k-mooney | i was looking for where it was used | |
| 16:39:28 | sean-k-mooney | that explain why i could not find it | |
| 16:40:10 | bauzas | see you folks tomorrow morning | |
| 16:41:11 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 16:41:12 | openstackgerrit | Alex Xu proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 16:41:12 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 16:41:13 | openstackgerrit | Alex Xu proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 16:41:13 | openstackgerrit | Alex Xu proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | |
| 16:42:01 | alex_xu | sean-k-mooney: actually, it is all about this little bit https://review.opendev.org/#/c/678456/32/nova/virt/hardware.py@1688 | |
| 16:43:35 | sean-k-mooney | alex_xu: ya that shoudl cause nodes to be 1 | |
| 16:44:17 | sean-k-mooney | and i think we will take this brance when creating the numa toplgoy right https://review.opendev.org/#/c/678456/32/nova/virt/hardware.py@1742 | |
| 16:45:34 | sean-k-mooney | well we will also call numa_topology = _get_numa_topology_auto(...) | |
| 16:47:15 | sean-k-mooney | ya | |
| 16:47:19 | alex_xu | sean-k-mooney: yes, it is go to the _get_nuam_toplogy_auto | |
| 16:47:23 | sean-k-mooney | the singel numa toplogy is created by https://review.opendev.org/#/c/678456/32/nova/virt/hardware.py@1514 | |
| 16:47:42 | sean-k-mooney | which is exactly what we want | |
| 16:47:53 | alex_xu | yea | |
| 16:53:15 | stephenfin | efried: What do you think about https://review.opendev.org/#/c/678455/25/nova/virt/libvirt/driver.py@1299 ? | |
| 16:54:12 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |
| 16:54:12 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021 | |
| 16:54:13 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 16:54:18 | artom | dansmith, mriedem ^^ | |
| 16:54:19 | efried | stephenfin: It is critical that a dirty pmem not be allowed to be assigned to another guest | |
| 16:55:16 | efried | sorry, you're asking whether we should continue cleaning up wait until the end to fail out? | |
| 16:55:32 | stephenfin | yup | |
| 16:55:40 | efried | this is by no means the only way we can fail out of a destroy() early | |
| 16:55:43 | efried | is it? | |
| 16:55:50 | stephenfin | That's what I'm trying to figure out | |
| 16:56:05 | stephenfin | _unplug_vifs is called with ignore_errors=True | |