| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-11 | |||
| 16:29:46 | efried | I just meant "being on AMD isn't enough; other stars must align for SEV to be supported" | |
| 16:29:46 | efried | > It was the "(regardless of AMD-ness)" bit I didn't quite get | |
| 16:29:46 | efried | aspiers: | |
| 16:31:06 | efried | and yes, I agree with your ordering, especially since your patches are already 2/3 with one +2 and the last one very close to same, but the other ones are still awaiting your nod before I even dig into them. | |
| 16:37:27 | aspiers | efried: OK thanks, we're 100% aligned :) | |
| 16:40:26 | aspiers | efried: Oh, one extra detail regarding the above - yes the check can just look at the supports_amd_sev instance variable, but it also needs to check extra_specs for resources:MEM_ENCRYPTION_CONTEXT=1 to know that encrypted memory was requested, right? | |
| 16:40:43 | aspiers | If *both* are true then it should add all the SEV guest config tweaks | |
| 16:42:10 | efried | s/check extra_specs/check the allocation/ but otherwise, ja, genau | |
| 16:44:11 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: flatten rbd images when unshelving an instance https://review.opendev.org/457886 | |
| 16:49:06 | aspiers | efried: OK thanks, I guess that will become self-evident when I look into how to check allocations | |
| 16:49:51 | efried | aspiers: Well, I would rather you check the allocations rather than duplicating the logic you just put in your request filter to check and validate the flavor & image | |
| 16:50:12 | efried | The allocations are passed to spawn. They're just a dict. See the placement API reference for the format. | |
| 16:50:26 | aspiers | Cool, thanks! | |
| 16:50:54 | aspiers | I'm beginning to feel that this is nearing code-completion and maybe can be added to the runway queue soon | |
| 16:51:00 | efried | yup | |
| 16:51:15 | aspiers | I do want to test the whole series on real SEV hardware first though | |
| 16:51:48 | aspiers | (which I have available to me right now) | |
| 16:58:05 | efried | dustinc: sdk release 0.31.0 is in u-c now, so I'd like to respin the series to bump that and fix up that other thing we talked about yesterday. Let me know when you're clear of local edits. | |
| 17:44:22 | dustinc | efried: Hmm...might be a bit, go for it and I can deal with merging my local edits | |
| 18:03:16 | spatel | sean-k-mooney: any idea about this setting https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1724614 ? | |
| 18:03:17 | openstack | Launchpad bug 1724614 in linux (Ubuntu) "[KVM] Lower the default for halt_poll_ns to 200000 ns" [Medium,Triaged] | |
| 18:03:59 | spatel | i have default value but i heard it gives good boost to CPU if you play with this setting | |
| 18:05:24 | mriedem | efried: do we have a bug for the test_check_doubled_words py27 failures? | |
| 18:09:41 | sean-k-mooney | spatel: i have not had experience with tweaking that specific tunable in the past. | |
| 18:10:35 | spatel | even i am running in default, i heard from someone but may be its application specific. | |
| 18:10:43 | sean-k-mooney | spatel: i would proably try to find some infomation on it form nfv/hpc tuning guide but there are man paramaters that can be tweaked not all of which will help for all workloads | |
| 18:11:01 | spatel | +1 | |
| 18:11:53 | spatel | but i am seeing sometime my ksm cpu usage going high on compute node | |
| 18:12:10 | spatel | [ksmd] | |
| 18:12:29 | sean-k-mooney | you proably want to read this https://www.kernel.org/doc/Documentation/virtual/kvm/halt-polling.txt | |
| 18:17:56 | sean-k-mooney | spatel: ibm seam to be recommendign a value fo 80000 ns based on https://www.ibm.com/support/knowledgecenter/en/linuxonibm/liaag/wkvm/wkvm_c_tune_kvm.htm but this seams to be a tradeoff between powersavings and latency which translated to higher throughput at the cost of cpu efficenty | |
| 18:19:51 | spatel | it seem that setting will schedule in very less time which increase power usage | |
| 18:20:29 | spatel | anyway i am not going to change until i have solid reason to do | |
| 18:37:06 | mriedem | coreycb: is this the right change log to be looking at for ubuntu bionic py27? http://changelogs.ubuntu.com/changelogs/pool/main/p/python2.7/python2.7_2.7.15~rc1-1ubuntu0.1/changelog | |
| 18:38:34 | mriedem | efried: https://bugs.launchpad.net/nova/+bug/1832392 | |
| 18:38:35 | openstack | Launchpad bug 1832392 in OpenStack Compute (nova) "nova.tests.unit.test_hacking.HackingTestCase.test_check_doubled_words failing in py27 jobs since June 10" [High,Confirmed] | |
| 18:40:04 | efried | mriedem: Tracking it against bug/1804062 | |
| 18:40:16 | efried | mriedem: https://review.opendev.org/664622 is merging to disable the test | |
| 18:40:32 | coreycb | mriedem: that is the latest update for the bionic-security pocket but bionic-updates is more recent. you can see the versions here: https://launchpad.net/ubuntu/+source/python2.7 | |
| 18:40:42 | efried | mriedem: and stephenfin is working on a patch to pycodestyle to get closer to fixing the core issue. | |
| 18:41:54 | mriedem | coreycb: ah ok here is the june 10 change then https://launchpad.net/ubuntu/+source/python2.7/2.7.15-4ubuntu4~18.04 | |
| 18:43:41 | coreycb | mriedem: yep | |
| 18:44:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431 | |
| 18:44:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 18:45:00 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442 | |
| 18:46:27 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 18:46:28 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: [DNM] testing bug/1813789 revert resize events https://review.opendev.org/664442 | |
| 18:46:28 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Include direct-physical in compute manager events check https://review.opendev.org/664431 | |
| 18:46:35 | artom | Fscked up my rebases | |
| 18:48:02 | mriedem | ok so probably no point in rechecking anything | |
| 18:48:24 | mriedem | or working on nova for a few days... | |
| 18:49:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Refresh instance network info on deletion https://review.opendev.org/660761 | |
| 19:03:06 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 19:03:16 | aspiers | efried: there ya go ^^^ | |
| 19:03:21 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix default values in update_cell command https://review.opendev.org/662383 | |
| 19:04:26 | aspiers | artom: reflog is your friend ;-) | |
| 19:04:54 | artom | aspiers, wasn't that bad | |
| 19:05:12 | artom | If I need reflog then I've really messed up | |
| 19:05:56 | aspiers | hehe | |
| 19:06:05 | aspiers | Oh I dunno, it can be a real handy shortcut sometimes | |
| 19:07:57 | artom | I'm off to the gym, during the day, for a change, while Zuul runs | |
| 19:09:51 | efried | mriedem: Once that bypass merges, which should hopefully happen pdq (test-only patches don't run tempest/dsvm stuff, right?) you can recheck the world. | |
| 19:13:12 | mriedem | dansmith: would you see a use for a weigher config option for the vgpu/numa affinity weigher in sylvain's spec? https://review.opendev.org/#/c/650963/ | |
| 19:13:21 | mriedem | since we have config options to control all other weighers | |
| 19:13:38 | dansmith | just in general? I mean, I would expect all weighers to be controllable like that | |
| 19:13:48 | mriedem | in general yes | |
| 19:14:02 | dansmith | unless it was something SUPER airtight like the failed computes weigher | |
| 19:15:13 | dansmith | dude | |
| 19:15:20 | dansmith | you should have snipped that last line | |
| 19:15:43 | mriedem | ha | |
| 19:16:08 | mriedem | sorry | |
| 19:16:27 | mriedem | also not sure what he's proposing for the claim stuff, | |
| 19:16:52 | mriedem | normally if we were to link up the scheduler to some RT claim, we'd use the limits so that if you disable the filter we don't claim on those limits in the RT, | |
| 19:17:11 | mriedem | but with this, it sounds like the compute would always just pull the VGPU allocations and pass those onto the claim | |
| 19:19:17 | dansmith | erm | |
| 19:20:40 | dansmith | is this really adding new claims to the RT? | |
| 19:20:45 | mriedem | it says it is | |
| 19:21:05 | mriedem | https://review.opendev.org/#/c/650963/7/specs/train/approved/libvirt-vgpu-numa-affinity.rst@99 | |
| 19:21:21 | dansmith | yeah, that's the line I'm wondering about | |
| 19:29:28 | melwitt | does anyone know if the failed_builds count is exposed out of the REST API at all? | |
| 19:32:01 | mriedem | melwitt: it's not | |
| 19:32:16 | melwitt | danke | |
| 19:32:37 | mriedem | it's shoved into compute_nodes.stats which is a json blob in the db | |
| 19:33:34 | melwitt | ah ok, I was wondering about that too. I found the nova/compute/stats.py file and didn't know how it's used. thanks | |
| 19:34:14 | Nick_A | Is there a way to take online snapshots of lxd containers? | |
| 19:34:44 | mriedem | Nick_A: you should ask the lxd devs in #lxcontainers | |
| 19:34:52 | mriedem | https://github.com/openstack/nova-lxd#support-and-discussions | |
| 19:34:52 | Nick_A | thank you | |
| 20:11:04 | efried | dustinc: gonna do that now, mkay? | |
| 20:11:20 | dustinc | 👍 | |
| 20:23:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |
| 20:23:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664 | |
| 20:23:32 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028 | |
| 20:23:32 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027 | |
| 20:23:33 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691 | |
| 20:23:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690 | |
| 20:23:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023 | |
| 20:23:50 | efried | dustinc: ^ | |
| 20:44:56 | efried | f f f f. bug 1823251 hit https://review.opendev.org/#/c/664622/ after sitting in the queue for 5h. | |
| 20:44:57 | openstack | bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] https://launchpad.net/bugs/1823251 | |