| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-26 | |||
| 17:09:54 | dansmith | it just needs a pointer to the engine is all | |
| 17:11:05 | dansmith | anyway, I get it.. still meh, but I understand | |
| 17:11:13 | melwitt | you are right that this is a pretty special case | |
| 17:11:34 | melwitt | (special to have a bug like that) | |
| 17:12:26 | dansmith | I need to go through the other uses of get_engine in more detail and figure out if we maybe need some other trap to make sure that we don't get an engine with no context, even in other situations where cell targeting isn't a thing | |
| 17:12:33 | dansmith | but I don't have time for that today | |
| 17:12:42 | melwitt | ok. it caused me a lot of confusion and time spent so I thought there might be value in removing the last context manager weirdness, but if you think it's better to have than not, I can accept that | |
| 17:26:18 | aspiers | efried: found some more issues with SEV series, please bear with me. /cc bbobrov | |
| 17:26:36 | efried | ack | |
| 17:48:17 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 17:48:17 | openstackgerrit | Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680 | |
| 17:48:18 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 17:48:18 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 17:48:19 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 17:48:19 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 18:02:42 | openstackgerrit | Merged openstack/nova master: Convert nova-next to a zuul v3 job https://review.opendev.org/670196 | |
| 18:16:27 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 18:22:40 | artom | This NUMA LM func test is actually some of the most fun code I've written in a while | |
| 18:22:50 | artom | ... not sure that's a good thing :P | |
| 18:25:08 | mriedem | efried: should https://review.opendev.org/#/c/631242/ be rebased on your SDK series now? | |
| 18:25:18 | mriedem | ...or is that a bad idea b/c cyborg isn't in the sdk yet? | |
| 18:26:15 | efried | mriedem: I would like it if cyborg used sdk, but the cyborg blueprint shouldn't have to be dependent on the sdk one. | |
| 18:26:36 | efried | mriedem: If we could merge https://review.opendev.org/#/c/643664/ then yeah, I would say we should do that. | |
| 18:26:50 | efried | any cores around? dansmith? | |
| 18:27:02 | mriedem | he's out, or on the way | |
| 18:27:12 | efried | melwitt? | |
| 18:27:35 | mriedem | efried: in general I'm just wondering about what looks like stalled progress on https://review.opendev.org/#/q/topic:bp/nova-cyborg-interaction+(status:open+OR+status:merged) | |
| 18:28:15 | efried | mriedem: I think the progress is on the cyborg side for the mo https://review.opendev.org/#/c/659233/ | |
| 18:28:15 | mriedem | we're <2 months from FF | |
| 18:28:29 | mriedem | oh ok | |
| 18:29:15 | efried | mriedem: recall at ptg demo time, they were working against a PoC feature branch off of cyborg, not code merged (or even proposed) in master. So they're catching up on that now. | |
| 18:29:40 | efried | though I'll admit progress could be faster. | |
| 18:30:19 | mriedem | yar | |
| 18:30:55 | mriedem | now that we're past spec freeze for train, i'm sure you don't feel like it, but what are your thoughts on a state-of-the-state email about the status of our cycle themes? https://specs.openstack.org/openstack/nova-specs/priorities/train-priorities.html | |
| 18:31:18 | mriedem | i'm writing my monthly report which is why i'm in reporting mode :) | |
| 18:31:49 | efried | mriedem: that sounds like a friggin hoot | |
| 18:32:07 | efried | I should delegate that | |
| 18:32:12 | efried | to someone in reporting mode | |
| 18:32:14 | mriedem | man it looks like https://review.opendev.org/#/c/659233/8 should be split up | |
| 18:32:46 | efried | I agree, but it's such a garbled mess right now it would be tough to tell how. | |
| 18:34:35 | mriedem | right again | |
| 18:35:34 | dansmith | doesn't it have a recipe in the commit message? | |
| 18:35:53 | dansmith | I mean "dos2unix of a file" does not belong in a patch with anything else | |
| 18:35:56 | dansmith | I mean good lord | |
| 18:36:25 | mriedem | yeah i saw that too | |
| 18:36:57 | dansmith | file renames should be their own deal | |
| 18:36:57 | efried | I don't know what kind of recipes you use, that have two 4s. | |
| 18:37:02 | efried | must be why you're so skinny. | |
| 18:37:40 | dansmith | heh | |
| 18:38:12 | efried | Y'all should feel free to add your -1 for those reasons as well. I've done enough ranting on that patch for one day. | |
| 18:38:18 | mriedem | i just did | |
| 18:38:27 | dansmith | can I just -1 with "smh" ? | |
| 18:39:01 | mriedem | given i had to google that, i'm guessing someone from china won't get it either :) | |
| 18:39:23 | dansmith | heh, I feel like I'm 20 years older than my self, but you're like.. eighty | |
| 18:39:25 | efried | I'd rather keep the comments on the constructive side, even if brutal. | |
| 18:39:38 | efried | not much they can do with "smh" | |
| 18:39:41 | dansmith | efried: I wasn't serious of course | |
| 18:39:52 | efried | yeah | |
| 18:39:56 | efried | of course | |
| 18:39:58 | dansmith | but I did see you were like "woah, nuke this whole sumbitch" | |
| 18:40:49 | efried | well, the *probably* can't justify holding a copy of provider_tree, but they *definitely* can't justify holding a copy of os-resource-classes | |
| 18:40:58 | efried | that's just... no. | |
| 18:41:04 | dansmith | heh | |
| 18:41:16 | dansmith | yeah, that's like... well, ... smh | |
| 18:41:17 | efried | but also, yeah, get rid of 90% of the ReportClient | |
| 18:41:19 | mriedem | dansmith: i'd like you to get off my lawn | |
| 18:41:29 | dansmith | mriedem: you first | |
| 18:41:36 | mriedem | hip is broken, can't | |
| 18:41:48 | dansmith | I don't have an all-terrain cane so I'm kinda stuck | |
| 18:42:21 | efried | Oh, is *that* what "ATC" stands for on the summit badge? | |
| 18:42:32 | dansmith | heh | |
| 18:43:41 | efried | btw, zun copied in the *whole* SchedulerReportClient. | |
| 18:44:03 | efried | Chris and I both had a leetle to say about that https://review.opendev.org/#/c/586960/ | |
| 18:44:08 | mriedem | this is the point where i joke about how everyone shits on nova but then copies a bunch of the code out of nova | |
| 18:44:32 | efried | and then it doesn't work for them, so they shit on nova | |
| 18:44:49 | mriedem | i'd pay to see someone copy something wholesale that is about cells | |
| 18:44:51 | mriedem | that'd be great | |
| 18:47:09 | mriedem | i left a comment on the zun patch, pointing to watcher's start https://review.opendev.org/#/c/661245/ | |
| 18:47:15 | mriedem | which is pretty simple | |
| 18:47:39 | mriedem | they clearly copied some stuff from SchedulerReportClient but not much | |
| 18:48:20 | mriedem | efried: where do you want blueprints that aren't going to be in train moved to, as in series goal? https://blueprints.launchpad.net/nova/train | |
| 18:48:22 | mriedem | like the rmd ones? | |
| 18:48:32 | mriedem | just...undefined? | |
| 18:48:37 | efried | that's our only option, nah? | |
| 18:48:46 | efried | or do we have the power to create a new one, like 'backlog'? | |
| 18:48:48 | mriedem | yeah, unless you create a u release, but we don't know the name of that yet | |
| 18:48:58 | mriedem | there is a trunk series but we don't really use it | |
| 18:49:05 | efried | maybe we should | |
| 18:49:06 | mriedem | you could create a backlog series.... | |
| 18:49:14 | mriedem | but the 300 or so nova blueprints are already in the backlog | |
| 18:49:31 | mriedem | er, make that | |
| 18:49:32 | mriedem | 1 → 75 of 1128 results | |
| 18:49:42 | efried | it would be nicer to start a backlog afresh | |
| 18:49:56 | mriedem | shrug | |
| 18:49:58 | efried | and for a diligent PTL to refresh it every cycle, moving dead stuff to undefined | |
| 18:49:59 | mriedem | is rmd in the backlog? | |
| 18:50:09 | efried | let me go look at it | |
| 18:50:21 | efried | it's probably just dead, obsolete, but let me look real quick | |
| 18:51:34 | efried | mriedem: if we mark Definition as Obsolete, they drop off this report? | |
| 18:52:09 | mriedem | i think obsolete is as close as we get to dead yeah | |