| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-21 | |||
| 15:35:55 | NobodyCam | yes! | |
| 15:36:18 | tssurya | dansmith: whenever you have some time, your opinion on https://review.openstack.org/#/c/636132/ would be highly valued/needed :) thanks in advance | |
| 15:38:12 | sean-k-mooney | anyway form the sequence diagarm of the boot state transition there is definetly somthing wrong. the only time the task state shoudl be in Schduling after the conductor started setting up the disk would be a rechedule after a failed spawn i think | |
| 15:38:32 | sean-k-mooney | unfortunetly the diagram does not capture the error paths | |
| 15:44:07 | efried | stephenfin: You seem like an obvious choice for nova/docs liaison https://wiki.openstack.org/wiki/CrossProjectLiaisons#Documentation -- can I put you down? | |
| 15:44:22 | dansmith | tssurya: ack | |
| 15:44:35 | stephenfin | efried: I've already been appointed the oslo one so why not | |
| 15:45:07 | efried | stephenfin: you mean the oslo/docs liaison or the oslo/nova liaison? Cause I could totally put you down for that too. | |
| 15:45:17 | NobodyCam | thank you sean-k-mooney :) I will attempt to redeploy nova | |
| 15:45:23 | stephenfin | oslo/docs | |
| 15:45:36 | stephenfin | but sure, oslo/nova maybe makes sense too | |
| 15:45:41 | efried | k | |
| 15:47:45 | efried | Kevin_Zheng, nicolasbock: Y'all are currently down as the nova/docs liaisons https://wiki.openstack.org/wiki/CrossProjectLiaisons#Documentation -- are you wanting to continue in that capacity? | |
| 15:48:36 | mriedem | efried: i think you could replace | |
| 15:49:16 | efried | ight | |
| 15:50:38 | efried | it seems pretty clear this list isn't used much for some of the categories. Not sure how much I ought to care about it. | |
| 15:59:03 | openstackgerrit | Merged openstack/nova stable/rocky: ironic: check fresh data when sync_power_state doesn't line up https://review.openstack.org/640772 | |
| 15:59:12 | openstackgerrit | Merged openstack/nova stable/rocky: pass endpoint interface to Ironic client https://review.openstack.org/643098 | |
| 15:59:15 | efried | jackding: Did you send out a ML post for https://review.openstack.org/#/c/621646/ warning of the ComputeDriver interface change? | |
| 15:59:26 | openstackgerrit | Merged openstack/nova master: Documentation for bandwidth support https://review.openstack.org/642064 | |
| 15:59:35 | openstackgerrit | Merged openstack/nova master: Add a prelude release note for the 19.0.0 Stein GA https://review.openstack.org/644412 | |
| 15:59:49 | efried | mriedem: looks like ready to cut rocky --^ ? | |
| 16:00:49 | mriedem | efried: yup i'll propose a release | |
| 16:00:57 | efried | k | |
| 16:02:51 | sean-k-mooney | jackding: the change you ping about before still looks good to me by the way | |
| 16:02:58 | gibi | efried: do you need anything else regarding https://review.openstack.org/#/c/645084/ ? | |
| 16:03:14 | jackding | efried: no. Do I need to? it's an optional parameter that has a default value. | |
| 16:03:50 | jackding | efried: not invoked by default | |
| 16:03:54 | efried | gibi: nope, just forgot. +A now | |
| 16:04:08 | gibi | efried: coolio | |
| 16:04:17 | efried | jackding: Any time the signature changes, even for an optional kwarg, it breaks OOT drivers. | |
| 16:04:34 | efried | jackding: We're not supposed to care about them, but I still have a soft spot in my heart. | |
| 16:05:20 | efried | gibi: Should we wait for that --^ for RC1? | |
| 16:05:33 | efried | I guess so, since it's renos | |
| 16:07:43 | efried | mriedem: you see anything else we should hold rc1 for? | |
| 16:08:03 | mriedem | efried: https://review.openstack.org/#/c/642863/ | |
| 16:08:41 | gibi | efried: yeah it is fixing the renos | |
| 16:08:48 | gibi | efried: so I would wait for that | |
| 16:10:20 | mriedem | the release patch can depends-on the known stuff you want to hold up for and then just adjust the hash when they land | |
| 16:11:34 | efried | don't see a lot of value in the depends-on if the content needs to be updated anyway. Just to stop them from automerging it? | |
| 16:11:44 | mriedem | yup | |
| 16:41:37 | openstackgerrit | Merged openstack/nova master: Fix links to neutron QoS minimum bandwidth doc https://review.openstack.org/645084 | |
| 17:01:00 | mdbooth | Could this be a source of random SSH failures in the gate: https://bugzilla.redhat.com/show_bug.cgi?id=1689933 ? | |
| 17:01:02 | openstack | bugzilla.redhat.com bug 1689933 in openstack-nova "[RFE] Add a simple way to enable random number generator on all flavor/image" [Unspecified,New] - Assigned to nova-maint | |
| 17:04:46 | kashyap | bauzas: mdbooth: efried: When you get a minute, please put this one out of its misery: https://review.openstack.org/#/c/641981/ (libvirt: Use 'writeback' QEMU cache mode when 'none' is not viable) | |
| 17:05:08 | efried | kashyap: did you want that in RC1? | |
| 17:05:09 | bauzas | I'm just wrapping my head with devstack atm | |
| 17:05:22 | efried | Cause that's all we're gonna merge today. | |
| 17:05:24 | kashyap | efried: Would certainly be useful to get some initial testing | |
| 17:05:37 | mdbooth | kashyap: That's a no ;) | |
| 17:05:40 | efried | mriedem: RC1 candidate? | |
| 17:05:41 | kashyap | The sooner the better, as we'd be doing a bit of a disservice to the users | |
| 17:05:51 | kashyap | mdbooth: Hehe | |
| 17:06:38 | mdbooth | I think it's a backport candidate, though. We'll almost certainly take it downstream. It's not a regression, though, and it won't cause dataloss or any kind of error. It's just unnecessarily slow. | |
| 17:06:49 | mdbooth | s/I *don't* think/ | |
| 17:06:58 | mdbooth | Erm, I do think. | |
| 17:08:24 | sean-k-mooney | kashyap: that is not a regression so it normally would not qualify for an rc right? | |
| 17:08:28 | kashyap | mdbooth: Thanks for the review so far! Appreciate it | |
| 17:09:12 | kashyap | sean-k-mooney: Not a regression; but why inflict needless slowness? This patch can (positively) impact all new instances | |
| 17:09:23 | sean-k-mooney | kashyap: im also not sure its technically a bug. it sound more like a specless blueprint. | |
| 17:09:50 | sean-k-mooney | kashyap: well you can just set the config option too | |
| 17:11:05 | sean-k-mooney | anyway this just feels like a feature that is filed as a bug which can be worked around via a config already so im not sure if it qualifes for RC incution but i also dont think you will be breaking anything | |
| 17:11:09 | kashyap | This is fixing existing tech debt. The more it is delayed, the more I lose motivation to fix it. | |
| 17:11:41 | sean-k-mooney | sure but when i looked at this earilar i assume this was targeting train not stien | |
| 17:11:50 | kashyap | sean-k-mooney: Also, Huh? This is not a "specless blueprint". This is actually a misuse/bug. | |
| 17:12:07 | sean-k-mooney | everything working but being slow is not a bug | |
| 17:12:20 | kashyap | sean-k-mooney: True. But I won't cry into my pillow if it doesn't make it to Stein, though | |
| 17:12:50 | kashyap | Just that I'd "strongly suggest" to get this in -- also because the "other patch" (for `qemu-img convert` case) merged. | |
| 17:13:13 | sean-k-mooney | kashyap: didnt that merge before feature freeze | |
| 17:13:14 | kashyap | And it's not nice to have these splatter over two releases. | |
| 17:13:18 | kashyap | sean-k-mooney: Yep | |
| 17:13:38 | kashyap | Frankly, I was just so buried in actually digging the issue, that I lost track of all this time. Most deadlines are arbitrary in my mind. | |
| 17:14:20 | kashyap | (As in, wasn't paying attention whether it was merged before or after RC1) | |
| 17:14:38 | mriedem | efried: i do not think that writeback patch is an rc1 candidate no | |
| 17:14:40 | mriedem | it's extremely latent | |
| 17:14:50 | mriedem | damn it where is jaypipes | |
| 17:14:52 | efried | k, swhat I figured. | |
| 17:15:26 | mriedem | dansmith: edleafe: cdent: i've dumped comments on surya's 'locked_reason' spec - specific feedback on (1) the data model changes and (2) rest api payload format would be appreciated https://review.openstack.org/#/c/638629/ | |
| 17:15:37 | sean-k-mooney | kashyap: once rc1 is cut tonight it can be merged tommorow after the stable/stien branch is cut | |
| 17:15:50 | kashyap | mriedem: Yes, it is latent. | |
| 17:16:06 | sean-k-mooney | the real question is is it backportable | |
| 17:16:19 | cdent | mriedem: thanks queued | |
| 17:16:40 | kashyap | I definitely think it is backport-worthy. See mdbooth's comment earlier | |
| 17:17:19 | sean-k-mooney | kashyap: in which case it can be proposed for backport to stable/stien and the other stable branches | |
| 17:17:30 | kashyap | mriedem_grooming: I don't agree with your emphatic "no". But whatever. I have 1000 other things that need my attention | |
| 17:17:30 | sean-k-mooney | so its not really that much of an issue | |
| 17:17:52 | kashyap | sean-k-mooney: Yep, so it's fine, if it's merged tomm. | |
| 17:18:19 | sean-k-mooney | mdbooth: do you consider qemu vert type to be a bug | |
| 17:18:21 | kashyap | sean-k-mooney: But, I'd prefer it to be in Stein. I didn't "plan" this work. | |
| 17:18:45 | sean-k-mooney | mdbooth: you can say yes :) | |
| 17:18:48 | mdbooth | kashyap: I don't think it's worthy of Stein at this stage. | |
| 17:18:55 | kashyap | mdbooth: Okido... | |
| 17:18:56 | mdbooth | kashyap: Focus on the backport. | |
| 17:19:05 | kashyap | I don't know; I'll focus on dinner first | |
| 17:19:11 | mdbooth | kashyap: Good call :) | |
| 17:19:47 | kashyap | sean-k-mooney: mdbooth: But as noted, The two volume drivers patches should _also_ merge at the same time | |
| 17:19:54 | kashyap | For the "minimum possible doc" to make sense. | |
| 17:24:25 | mriedem_grooming | listen guys, i didn't say it wasn't a bug, i sait it wasn't a regression in stein, it's latent, therefore it doesn't need to be held up for rc1 which is already stressful | |
| 17:24:30 | mriedem_grooming | so we'll deal with backports | |
| 17:24:48 | mriedem_grooming | if people don't like my input on it then don't ask | |