| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-11 | |||
| 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 | |
| 20:46:22 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/662584 | |
| 20:47:31 | sean-k-mooney | efried: is the 5 hour lag just a general issue with the gate being slow or is it related | |
| 20:48:07 | efried | sean-k-mooney: I'm sure it's just because we had a couple dozen patches in the queue (all of which failed on that issue of course) so that one got pushed. | |
| 20:51:08 | mordred | efried: ooh - and you've got the ironicclient patches too - that's so cool | |
| 20:51:37 | efried | mordred: Yes, dustinc has been working on those. I'm not sure how done he is (percentage-wise) - dustinc? | |
| 20:51:55 | sean-k-mooney | i have a ptach queue in neutron for a little over 2 and a half hours without it starting so i think the gate is slow to start jobs currently in general | |
| 20:52:21 | efried | mordred: I'm pretty sure we're going to need some feature parity for ironic in sdk before we can be all the way complete there, but fortunately dtantsur|afk knows his sdk AND his ironic :) | |