| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-26 | |||
| 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 | mriedem | we're <2 months from FF | |
| 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: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 | efried | I don't know what kind of recipes you use, that have two 4s. | |
| 18:36:57 | dansmith | file renames should be their own deal | |
| 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 | |
| 18:52:12 | efried | superseded | |
| 18:52:13 | efried | let me play | |
| 18:52:20 | mriedem | superseded is like duping a bug | |
| 18:52:26 | mriedem | not the same thing as obsolete | |
| 18:52:49 | mriedem | for example, at one point we had at least 5 or 6 blueprints for adding volume type to the server create bdm interface, | |
| 18:52:58 | mriedem | eventually i picked one and just marked all the rest as superseded by that one | |
| 18:53:56 | efried | meh, it shows up on the report either way. Superseded seems appropriate to me. | |
| 18:57:10 | efried | mdbooth_, mriedem: Local disk serial numbers appears to have a bunch of merged code but also a bunch of abandoned code. What should we do with that? | |
| 18:57:27 | efried | https://blueprints.launchpad.net/nova/+spec/local-disk-serial-numbers | |
| 18:57:36 | mriedem | when you supersede a blueprint don't you have to mark it superseded by something else? | |
| 18:57:44 | mriedem | otherwise it's not really superseded | |
| 18:57:46 | efried | apparently not :) | |
| 18:57:51 | efried | I just commented in the whiteboard | |
| 18:57:52 | mriedem | :/ | |
| 18:58:47 | mriedem | https://blueprints.launchpad.net/nova/+spec/rmd-base-enablement is still marked for the train series goal | |
| 18:59:11 | mriedem | i'm not really the person to ask about local disk serial numberes | |
| 18:59:13 | mriedem | *numbers | |
| 18:59:14 | efried | yes, all three are | |
| 18:59:18 | mriedem | it's been deferred a couple of times | |
| 18:59:38 | efried | It's not wrong imo to keep it under train since that's where it was proposed and superseded. | |
| 18:59:42 | mriedem | efried: so you're going to leave the rmd ones as marked for train? i guess what i'm looking for is eventually https://blueprints.launchpad.net/nova/train will just show approved blueprint | |
| 19:00:04 | efried | what would be neat is if there was some way to search for release *plus* status | |