| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-16 | |||
| 13:27:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | |
| 13:27:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 13:44:42 | dansmith | jaypipes: on this: https://review.openstack.org/#/c/568613/5 | |
| 13:45:00 | jaypipes | dansmith: yes? | |
| 13:45:01 | dansmith | jaypipes: isn't not allowing reserved=total just a bug? I feel like maybe we don't need the whole microversion dance | |
| 13:45:28 | dansmith | like someone did < instead of <= in some code somewhere? | |
| 13:45:30 | jaypipes | dansmith: I feel the same way. | |
| 13:45:50 | dansmith | jaypipes: does anyone not feel that way? | |
| 13:45:53 | mriedem | lyarwood: you +2ed the ocata backport but not the pike one :) https://review.openstack.org/#/c/568369/ | |
| 13:46:15 | dansmith | my feeling is that there is no client that would depend on total=reserved returning a 400 | |
| 13:46:17 | mriedem | dansmith: i brought it up when it was being discussed, | |
| 13:46:23 | mriedem | because no one knew why the capacity check was that way | |
| 13:47:21 | mriedem | and its a behavior change in the api, so you kind of need to know when you can make that kind of request and expect it to be ok | |
| 13:47:22 | lyarwood | mriedem: appears I closed the tab before gerrit actually updated the review, noice! | |
| 13:48:01 | dansmith | mriedem: but microversions aren't for providing bug-for-bug compatibility right? that's why we don't do one to fix a 500? | |
| 13:48:22 | mriedem | this isn't a 500 | |
| 13:48:31 | dansmith | I know | |
| 13:48:51 | dansmith | but really, if we do it and get a 400, we can fall back to the same behavior as today | |
| 13:48:56 | dansmith | I dunno, this just seems overkill to me | |
| 13:49:13 | mriedem | i don't think i ever said i was going to block this when we talked about it in -placement weeks ago, | |
| 13:49:21 | mriedem | i -2ed the change this week because there is a spec up, | |
| 13:49:29 | mriedem | if people don't care about the api behavior change, then whatever | |
| 13:50:22 | dansmith | I would think that if this is a bug, with no microversion, that we'd backport the fix as .. a fix | |
| 13:50:54 | mriedem | meh - this doesn't even fix any real problem right now does it? | |
| 13:51:09 | mriedem | the actual bug was already fixed some other way is my understanding | |
| 13:51:09 | dansmith | just MHO, I'm not strongly against a microversion I guess, I just .. I dunno. | |
| 13:51:40 | dansmith | it fixes some logic that prevents nova from setting total=reserved when that should be totally legit | |
| 13:52:01 | dansmith | I dunno, I don't want to argue about it, but I definitely think total=reserved should be legit | |
| 13:52:04 | mriedem | so instead of doing ^ it deletes the node inventory right? | |
| 13:52:04 | dansmith | and should have always been legit | |
| 13:52:05 | mriedem | during cleaning | |
| 13:52:15 | dansmith | it doesn't now, | |
| 13:52:20 | dansmith | it just punts | |
| 13:52:21 | dansmith | well, wait, | |
| 13:52:30 | dansmith | it doesn't if there are no nodes, that's the failure/startup case I guess | |
| 13:52:37 | dansmith | so maybe it's still deleting inventory currently I guess | |
| 13:52:38 | mriedem | my understanding is we worked around the problem by deleting the inventory during cleaning | |
| 13:53:04 | dansmith | yeah, probably, I'm confusing the startup zero-day or ironic-is-down case, but anyway | |
| 13:53:31 | mriedem | i tend to think if this isn't needed to actually fix anything, and it's a behavior change, then it's a microversion | |
| 13:53:43 | dansmith | from the placement point of view, | |
| 13:53:54 | dansmith | I feel like it's a fix to a piece of logic that was wrong | |
| 13:54:11 | dansmith | sure, nova could continue to do the thing it does today to work around it, but | |
| 13:54:32 | dansmith | anyway, whatever, I'd rather just get it done regardless | |
| 13:55:59 | mriedem | do what y'all want, i will step aside on this one | |
| 13:57:33 | dansmith | no, it's not that big of a deal, we've got a spec, let's just roll | |
| 13:57:37 | dansmith | getting it working is the important thing | |
| 14:00:34 | openstackgerrit | Vladyslav Drok proposed openstack/nova-specs master: Allow having placement inventories with reserved value equal to total https://review.openstack.org/568613 | |
| 14:01:30 | BobBall | mriedem / naichuans: Naichuan is on PRC time but yes, the XenAPI bits of the vgpu-rocky series are definitely ready for a runway slot. | |
| 14:05:25 | dansmith | vdrok: see my latest feedback on your spec? if it's not clear, I can just tweak the wording myself so we can land that | |
| 14:05:46 | mriedem | BobBall: yeah it's in the runway now, | |
| 14:05:54 | vdrok | dansmith: yeah just noticed that, I'll add a couple sentences there, thank you | |
| 14:05:56 | mriedem | i know they are in china, just dropping pings here for later reading | |
| 14:06:03 | mriedem | BobBall: but thanks for following up | |
| 14:06:08 | dansmith | vdrok: cool | |
| 14:07:12 | BobBall | mriedem: Thanks :) I was in a nice 2 hour meeting so didn't see the message until late | |
| 14:09:00 | openstackgerrit | Vladyslav Drok proposed openstack/nova-specs master: Allow having placement inventories with reserved value equal to total https://review.openstack.org/568613 | |
| 14:12:50 | dansmith | mriedem: so, how is the virt driver going to know if it should return no inventory or reserved=total? | |
| 14:13:10 | dansmith | or I guess we'll just require the new microversion and fail if not | |
| 14:13:13 | dansmith | so nevermind | |
| 14:13:40 | mriedem | yar | |
| 14:16:25 | dansmith | mriedem: jaypipes: let's get this over with: https://review.openstack.org/#/c/568613/7 | |
| 14:17:31 | jaypipes | dansmith: I'll let mriedem do that. I'm supportive of it. | |
| 14:18:49 | mriedem | will look in a bit | |
| 14:21:32 | mriedem | random question, | |
| 14:22:25 | mriedem | on the rare days that the sun comes up here, at 6am or so, i find myself waking up early, and not wanting to get in the shower right away because everyone else is still sleeping, so i go unshowered until like mid morning, is that a normal work-from-home trajectory when summer starts? | |
| 14:22:29 | stephenfin | efried: No idea what's going on here, I'm afraid. Might be worth reaching out to infra about that https://review.openstack.org/#/c/568065/ | |
| 14:23:01 | efried | mriedem: How often do you shave? | |
| 14:23:05 | mriedem | hells bells, | |
| 14:23:08 | mriedem | you know i'm a baby face right? | |
| 14:23:16 | mriedem | twice per week tops | |
| 14:23:17 | efried | Well, I didn't want to be impolite. | |
| 14:23:41 | mriedem | i will shave once the teenage mustache is pronounced | |
| 14:24:03 | efried | Having surprisingly little trouble picturing you with peachfuzz moustache. | |
| 14:24:10 | mriedem | and to keep up appearances with the mom that's at the same bus stop | |
| 14:24:50 | stephenfin | Anyone wanna fix an annoying warning/broken test? https://review.openstack.org/#/c/568532/ | |
| 14:25:10 | mriedem | efried: i guess that's the pattern, whether or not my wife is at work in the morning and i have to go to the bus stop | |
| 14:25:10 | efried | mriedem: My perspective on showering is different. I usually only shower after I train. Which is around 8x/week these days. But some days I don't train until evening, so showering first thing in the morning isn't a thing for me. | |
| 14:25:11 | mriedem | problem solved | |
| 14:25:35 | stephenfin | efried: 8 times?? | |
| 14:26:15 | efried | stephenfin: Yeah, 2x T & Th, Saturday is my day off. | |
| 14:29:58 | mriedem | dansmith: vdrok: jaypipes: +W | |
| 14:30:11 | jaypipes | thx | |
| 14:30:29 | dansmith | where's the code patch for it? | |
| 14:30:47 | vdrok | Thank you folks for reviewing this so fast! | |
| 14:31:12 | mriedem | https://review.openstack.org/#/c/564838/ | |
| 14:36:13 | bauzas | mriedem: if you wanna know my life, I take shower before kids wake up, or then I'm done | |
| 14:36:39 | bauzas | mriedem: and given I need to taxi them to the school every morning, I guess other parents wouldn't like that | |
| 14:36:55 | bauzas | smelling is not an option | |
| 14:45:35 | openstackgerrit | Merged openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | |
| 14:45:47 | openstackgerrit | Merged openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | |
| 14:48:38 | mriedem | i forgot about this until last night, but shall we deprecate running api services with eventlet? https://review.openstack.org/#/c/549510/ | |
| 14:52:23 | openstackgerrit | Merged openstack/nova-specs master: Allow having placement inventories with reserved value equal to total https://review.openstack.org/568613 | |
| 14:56:29 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Rename spec to match blueprint https://review.openstack.org/568870 | |
| 15:00:50 | bauzas | mriedem: please, yes | |
| 15:00:51 | bauzas | +2 | |
| 15:01:11 | bauzas | we already signaled since pike, so let's send another batsignal now | |
| 15:05:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | |
| 15:15:46 | openstackgerrit | Eric Fried proposed openstack/nova master: Debug logs for allocation_candidates filters https://review.openstack.org/568712 | |
| 15:16:36 | jroll | efried: what do you train for? running or? | |
| 15:17:02 | efried | jroll: I own & run a Brazilian jiu jitsu school | |