| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-25 | |||
| 15:07:27 | aspiers | efried: that makes sense - this stuff isn't obvious to me yet since I'm still very unfamiliar with the nova codebase, so thanks for the pointers | |
| 15:07:28 | efried | aspiers: But IMO looking at the allocation is a cleaner solution regardless. | |
| 15:07:44 | aspiers | efried: yup, sounds fine to me | |
| 15:08:24 | mtreinish | aspiers: unfortunately I won't be at Denver. But I save that talk to watch later, at least based on the abstract it looks like they had a similar idea to what andreaf and I were starting to do in CIML (at a high level) | |
| 15:08:39 | efried | aspiers: If you wanted to use hw:mem_encryption=true, you would have to check both the flavor and the image. So yeah, use the allocation. | |
| 15:08:40 | mtreinish | but they approched it with different ML techniques (and ones that might be better suited for the problem) | |
| 15:09:20 | aspiers | mtreinish: cool! very interested to hear your take on the talk. BTW I noticed earlier that elastic recheck / crm114 work seems to have made some v nice progress since last time I looked | |
| 15:09:42 | aspiers | efried: changing the spec tweak now | |
| 15:10:06 | aspiers | mtreinish: sorry to miss you this time though. we'll have to sort out those stestr issues online instead ;-) | |
| 15:11:55 | mtreinish | aspiers: yeah it's too bad. Luckily for you though masayukig will be there and he can help sort through those in person :P | |
| 15:12:04 | aspiers | mtreinish: true :) | |
| 15:12:23 | mtreinish | aspiers: hmm, I wasn't aware anyone had done anything with the crm114 stuff in a while. But I hadn't been watching it super closely | |
| 15:13:45 | aspiers | mtreinish: well it's a *long* time since I last more than glanced at http://status.openstack.org/elastic-recheck/ | |
| 15:14:02 | aspiers | mtreinish: maybe I just never noticed some of the nice stuff before | |
| 15:20:31 | bauzas | efried: I wasn't originally thinking to discuss about NUMA affinity for vGPUs in the PTG https://review.opendev.org/#/c/650963/ but given gibi provided some good concerns, I wondered when we could have time for eventually discussing it at the PTG | |
| 15:21:06 | bauzas | this isn't technically a placement-related spec, but I'd love to see it discussed then | |
| 15:21:20 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Update SEV work item to new approach based on MEM_ENCRYPTION_CONTEXT https://review.opendev.org/655717 | |
| 15:22:52 | bauzas | efried: so I proposed it for the x-proj session between nova/placement if you are okay with | |
| 15:25:45 | efried | bauzas: Yup, wfm. cdent is going to be organizing that chunk of time. | |
| 15:25:54 | bauzas | k | |
| 15:26:23 | dansmith | mriedem: what's your feeling on adding (i.e. requiring) tests for the virt drivers I'm adding image type flags for? | |
| 15:26:35 | dansmith | if there's nothing dynamic, then it's just a test to assert that two dicts match | |
| 15:27:47 | mriedem | thus not a very useful test, i don't have strong feelings about that | |
| 15:27:57 | mriedem | i'd say make sure it's mentioned in the commit message though... | |
| 15:28:38 | dansmith | okay | |
| 15:28:56 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: handle port allocation during migration https://review.opendev.org/655112 | |
| 15:28:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Extend NeutronFixture to handle migrations https://review.opendev.org/655114 | |
| 15:28:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: func test for migrate server with ports having resource request https://review.opendev.org/655113 | |
| 15:28:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec to server move RPC calls https://review.opendev.org/655721 | |
| 15:29:33 | efried | kashyap: secure boot topic, okay if I move it to Thursday 1400? | |
| 15:30:10 | jaypipes | stephenfin: +2 | |
| 15:30:18 | kashyap | efried: Thursday is the first day, right? | |
| 15:30:21 | efried | yes | |
| 15:30:21 | stephenfin | jaypipes: \o/ | |
| 15:30:38 | stephenfin | jaypipes: Thanks for taking the time to look through that | |
| 15:30:41 | kashyap | efried: I might need to be elsewhere. But we've discussed it also in Dublin, and people agreed it is something we want. | |
| 15:30:56 | kashyap | efried: So even if I miss it, it should be OK? | |
| 15:31:06 | efried | kashyap: If Saturday is better, I'll leave it there. Just had an opening. | |
| 15:31:20 | jaypipes | stephenfin: np, that *you* for having the energy to keep it going. | |
| 15:31:51 | efried | kashyap: But even better, if it's something we don't need to discuss, I can just strike it. | |
| 15:31:58 | kashyap | efried: You know what, maybe just put it on Thurs 14:00. I'll work it out to be there | |
| 15:32:05 | efried | okay | |
| 15:32:06 | stephenfin | jaypipes: Also, fwiw, I'd love to deprecate those options and would like to do so at some point in the future. Just need to figure out how to deprecate these things that are essentially unversioned APIs | |
| 15:32:08 | stephenfin | Baby steps | |
| 15:32:34 | kashyap | efried: It should be a quick thing. No elaborate discussion needed | |
| 15:34:46 | mriedem | stephenfin: do you know what might have changed in our tox setup such that if you specify a target that doesn't exist, it used to fail fast but now it doesn't | |
| 15:34:55 | mriedem | i.e. tox -e foo | |
| 15:35:33 | mriedem | pretty sure i've fat-fingered some target to see it pass when the actual thing would have failed a test | |
| 15:36:04 | mriedem | yeah heh i ran tox -e fast7 | |
| 15:36:23 | mriedem | and tox -e api-samles | |
| 15:36:35 | stephenfin | mriedem: I've spotted that and I don't know, unfortunately. | |
| 15:36:35 | stephenfin | mriedem: Hmm | |
| 15:36:41 | stephenfin | ERROR: unknown environment 'fast7' | |
| 15:36:41 | stephenfin | $ tox -e fast7 | |
| 15:36:52 | stephenfin | ^ I still see that on my machine (TM) | |
| 15:36:54 | mriedem | which version of tox do you have? | |
| 15:37:04 | stephenfin | 3.4.0 | |
| 15:37:04 | stephenfin | $ tox --version | |
| 15:37:50 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add new "supports_amd_sev" capability to libvirt driver https://review.opendev.org/638680 | |
| 15:40:06 | mriedem | 3.7.0 imported from /usr/local/lib/python2.7/dist-packages/tox/__init__.pyc | |
| 15:40:06 | mriedem | $ tox --version | |
| 15:40:19 | stephenfin | mriedem: https://tox.readthedocs.io/en/latest/changelog.html#v3-8-0-2019-03-27 | |
| 15:40:24 | stephenfin | "Fix missing error for tox -e unknown when tox.ini declares envlist. - by @medmunds #1160" | |
| 15:40:31 | stephenfin | Looks like a bug in 3.7.0 | |
| 15:40:41 | mriedem | ah https://github.com/tox-dev/tox/issues/1160 | |
| 15:41:19 | mriedem | Successfully installed tox-3.8.0 | |
| 15:41:20 | mriedem | ERROR: unknown environment 'foo' | |
| 15:41:20 | mriedem | osboxes@osboxes:~/git/nova$ tox -e foo | |
| 15:41:22 | mriedem | hot diggity damn | |
| 15:41:24 | mriedem | thanks stephenfin | |
| 15:41:30 | stephenfin | no problemo | |
| 15:44:10 | efried | wow | |
| 15:44:22 | efried | note to self: Ctrl+Alt+C does *not* open the calculator | |
| 15:44:50 | melwitt | mriedem: grafana shenanigans proposed here https://review.opendev.org/655591 wondering if I should add stable branch panels to it too | |
| 15:45:25 | aspiers | efried: lol | |
| 15:46:01 | mriedem | melwitt: ah it was the per-project thing that was busted, maybe that's what i talked about with corvus before | |
| 15:46:05 | mriedem | thanks for fixing that | |
| 15:50:45 | lajoskatona | mriedem: Hi, regardig https://review.opendev.org/640600 & https://review.opendev.org/640601 (tempest schema validation for nova v2.70 & 2.71) if you need help to push them I am happy to help you out | |
| 15:54:35 | mriedem | lyarwood: fyi https://graphite.opendev.org/render/?width=1231&height=593&_salt=1556207646.49&from=-8weeks&target=stats_counts.zuul.tenant.openstack.pipeline.check.project.opendev_org.openstack_nova.master.job.tempest-full-py3.FAILURE&target=stats_counts.zuul.tenant.openstack.pipeline.check.project.opendev_org.openstack_nova.master.job.devstack-plugin-ceph-tempest.FAILURE | |
| 15:54:39 | mriedem | you can get the graph from graphite | |
| 15:55:06 | mriedem | lajoskatona: go ahead, i'm not working on those | |
| 15:55:42 | mriedem | melwitt: nice catch on the tempest-full-py3 vs devstack-plugin-ceph-tempest (not py3) in the nova case, we should probably change nova to run against the devstack-plugin-ceph-tempest-py3 job instead | |
| 15:55:56 | lajoskatona | mriedem: ok, do I need any special thing to test in devstack? | |
| 15:56:21 | melwitt | mriedem: yeah, I was wondering that too | |
| 15:56:56 | mriedem | melwitt: that will throw the graph data off, but we could switch now, make your graph change depend on that and then revisit in a couple of weeks to compare stability | |
| 15:57:08 | mriedem | or just do that in a follow up, whatever | |
| 15:57:14 | mriedem | lajoskatona: i'm not sure i understand the question | |
| 16:00:23 | lajoskatona | mriedem: never mind, I try to to push, and come back if I have questions :-) | |
| 16:00:42 | mriedem | lajoskatona: ack and thanks | |
| 16:01:48 | lajoskatona | mriedem: :-) | |
| 16:11:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Add vmware driver image type capabilities https://review.opendev.org/655730 | |
| 16:11:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Add ironic driver image type capabilities https://review.opendev.org/655729 | |
| 16:11:12 | openstackgerrit | Dan Smith proposed openstack/nova master: Add zvm driver image type capabilities https://review.opendev.org/655732 | |
| 16:11:12 | openstackgerrit | Dan Smith proposed openstack/nova master: Add xenapi driver image type capabilities https://review.opendev.org/655731 | |
| 16:12:05 | dansmith | I dunno who to tag on the zvm one for review | |
| 16:12:12 | dansmith | but I'm pretty sure they only support raw | |
| 16:13:17 | melwitt | mriedem: I dunno if you saw, they did hit the eventlet monkey patching issue in ubuntu, last comment https://review.opendev.org/647310 | |
| 16:16:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Always pass HostAPI to get_availability_zones https://review.opendev.org/655583 | |
| 16:35:35 | mriedem | melwitt: yeah i saw | |
| 16:35:44 | mriedem | coreycb didn't +1 it though... | |