| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-06 | |||
| 15:01:14 | mriedem | efried: i'm +2 on the other ironic fix https://review.opendev.org/#/c/680542/ but we can wait until the test patch passes https://review.opendev.org/#/c/680643/ - given the state of the gate that might be sometime this weekend though, so maybe you want to +2 and one of us can approve once we have results? | |
| 15:01:58 | sean-k-mooney | donnyd: it might be because its requesting two vms getting one and timeing out getting the second? | |
| 15:02:47 | mriedem | actually that ironic patch already got test nodes, it must be nice to not be nova | |
| 15:02:49 | efried | mriedem: I was thinking along the same lines. But iiuc dtantsur followed up later and said the grenade fix wasn't actually related to this bug. We (still) don't have CI coverage that will hit it. I just proposed https://review.opendev.org/680718 in tripleo which will hopefully hit the job that originally uncovered the problem. | |
| 15:02:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: migrate: Add bw min service level check of source compute https://review.opendev.org/680395 | |
| 15:03:06 | efried | s/grenade fix/grenade failure/ | |
| 15:03:38 | mriedem | alright then | |
| 15:03:40 | mriedem | moving on | |
| 15:04:05 | efried | mriedem: but given how simple and obviously-right the fix is, and that dtantsur says it is legit, I think we might want to merge it anyway. | |
| 15:04:05 | sean-k-mooney | donnyd: ill check it at the weekend agian when the ci does not have as high a load | |
| 15:04:13 | dtantsur | efried: I'd make your tripleo patch depending on both fixes. I'm not sure which one of them is most breaking. | |
| 15:04:22 | mriedem | efried: i'm good with that | |
| 15:04:34 | mriedem | i definitely trust dtantsur here and these are simple fixes | |
| 15:04:39 | efried | dtantsur: the symptoms would be different | |
| 15:06:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: resize: Add bw min service level check of source compute https://review.opendev.org/680396 | |
| 15:06:00 | dtantsur | okie | |
| 15:08:32 | efried | dtantsur, mriedem: +A | |
| 15:08:53 | dtantsur | now let's see if you'll regret trusting me :D | |
| 15:09:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: prepare func test env for moving servers with bandwidth https://review.opendev.org/655109 | |
| 15:09:18 | efried | It's not clear that the tripleo patch will work as is; I don't understand the job chain that would get us there, or whether nova needs to be added to required_projects *somewhere* (but where??). So it may take a few iterations to figure that out. | |
| 15:10:43 | mriedem | dtantsur: please don't break my achy breaky heart | |
| 15:11:13 | dtantsur | okay | |
| 15:11:21 | mriedem | i'm assuming you get early 90s billy ray cyrus references of course | |
| 15:11:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate server with ports having resource request https://review.opendev.org/655113 | |
| 15:14:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138 | |
| 15:18:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140 | |
| 15:18:57 | gibi | mriedem, bauzas: the next 5 patches in the bw series is up-to-date. first patch: https://review.opendev.org/#/c/680394/ last up-to-date patch: https://review.opendev.org/#/c/655113 | |
| 15:19:09 | mriedem | gibi: yup, on my list for today | |
| 15:20:53 | gibi | mriedem: thanks. I still have to work on comments in th second half of the patch series and also need to do a follow up for the already merged https://review.opendev.org/#/c/656422 | |
| 15:21:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972 | |
| 15:24:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980 | |
| 15:25:11 | mriedem | stephenfin: is it not possible to deprecate an entire group of config options? | |
| 15:25:15 | mriedem | you have to do them all individually? | |
| 15:25:40 | stephenfin | you can deprecate opts but not groups, afaik | |
| 15:25:41 | stephenfin | so no | |
| 15:25:44 | mriedem | blast | |
| 15:25:49 | stephenfin | soz | |
| 15:26:12 | mriedem | is that hipster millenial for sorry? | |
| 15:26:31 | mriedem | i had to look up what a "stan" was last night | |
| 15:26:38 | stephenfin | let's say...yes | |
| 15:26:45 | cdent | you'll be cool, eventually, mriedem | |
| 15:26:54 | cdent | each day, get better and better | |
| 15:27:15 | mriedem | https://media1.tenor.com/images/fc73ef705412eeaeb838763055918587/tenor.gif?itemid=5653229 | |
| 15:27:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | |
| 15:28:09 | mriedem | oooo https://www.youtube.com/watch?v=BGrfhsxxmdE | |
| 15:28:48 | jangutter | The only "stan" I know is https://mc-stan.org | |
| 15:30:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827 | |
| 15:37:55 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019 | |
| 15:41:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate the XenAPIDriver https://review.opendev.org/680732 | |
| 15:41:38 | mriedem | efried: dansmith: gibi: bauzas: alex_xu: melwitt: stephenfin: fyi ^ | |
| 15:41:48 | dansmith | hoo boy | |
| 15:42:01 | mriedem | i'll post to the ML | |
| 15:42:12 | stephenfin | Ima +2 that so hard. | |
| 15:42:19 | stephenfin | after the ML post, of course | |
| 15:46:25 | mriedem | http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009180.html | |
| 15:47:37 | mriedem | stephenfin: since you smelled it, maybe you should deal up an etherpad for rc todo stuff like the prelude reno | |
| 15:47:47 | mriedem | i'm auto delegating efried delegating that to me | |
| 15:48:55 | stephenfin | mriedem: Sure https://etherpad.openstack.org/p/nova-train-prelude | |
| 15:51:25 | efried | thanks | |
| 15:51:39 | efried | is a prelude related to cycle highlights? | |
| 15:51:44 | mriedem | sort of, | |
| 15:51:46 | efried | cause I could use a delegate for that as well | |
| 15:51:53 | mriedem | the prelude generally feeds into highlights | |
| 15:52:10 | mriedem | the highlights have to be wordsmithed a bit for marketing speak | |
| 15:52:17 | efried | it's cruel that first draft of highlights is due the day of FF. | |
| 15:52:35 | mriedem | "the compute team leveraged the synergy of the cloud to...deprecate the xenapi driver" | |
| 15:52:45 | mriedem | efried: yup | |
| 15:52:50 | mriedem | i've complained about that before | |
| 15:53:08 | mriedem | smcginnis is good at suggestions on the highlights, | |
| 15:53:13 | mriedem | he's nearly executive level management at this point | |
| 15:53:19 | mriedem | he only speaks that langauge | |
| 15:53:19 | stephenfin | efried: I'm hoping to put together one of those internally to figure out what we need to test/document for Train/OSP16. I'll share it but it won't be 'til Wednesday week, which is probably too late? | |
| 15:53:38 | mriedem | stephenfin: FF is thursday | |
| 15:53:46 | efried | stephenfin: unless you accept that this task has been delegated to you. | |
| 15:54:02 | efried | then Thursday is the "deadline" for the "first draft", so Wednesday is a whole day early! | |
| 15:54:19 | efried | iow, it's all yours, and I don't need to vet/edit/revise it. | |
| 15:55:15 | smcginnis | efried: I'm not happy about the deadline either, but the OSF marketing folks said they needed that info earlier. :/ | |
| 15:55:47 | mriedem | need to copy/paste into a chart asap | |
| 15:55:50 | stephenfin | efried, mriedem: okay, I'll do it Monday | |
| 15:56:17 | mriedem | maybe we should put an easter egg in the highlights... | |
| 15:56:21 | mriedem | see who's paying attention | |
| 15:56:22 | efried | I'm not blaming anyone, and I'm sure there's a "good" reason. It's just like, we're already all one-armed paper hangers already. | |
| 15:56:29 | efried | mriedem: love that idea. | |
| 15:56:45 | mriedem | "nova is deprecated, all hail serverless" | |
| 16:00:16 | stephenfin | There's one non-doc patch left in the remove-consoleauth series that I thought had already merged. If anyone has bandwidth to push it through (melwitt is +2), that'd be dandy https://review.opendev.org/#/c/652970/ | |
| 16:00:29 | stephenfin | I'm just going to abandon the other two | |
| 16:02:57 | dustinc | mriedem, efried, dtantsur|afk: all caught up, thanks for your help | |
| 16:04:29 | efried | stephenfin: left a question there | |
| 16:04:41 | efried | it looks like a behavior change to me | |
| 16:05:14 | mriedem | efried: +2 on that deps update for the sighup fix https://review.opendev.org/#/c/679974/ - mnaser / dansmith fyi on that | |
| 16:05:21 | efried | thanks mriedem | |
| 16:05:23 | mriedem | definitely rc blocker | |
| 16:06:28 | dansmith | efried: I was thinking you had tested something related to sighup and decided that it was *not* properly restarting us after the fix | |
| 16:06:35 | dansmith | efried: did I misunderstand that or did something else change? | |
| 16:07:00 | efried | dansmith: that was at the point where we only had the oslo.service fix. I dug further and fixed the oslo.privsep issue. With both fixes together, shit works. | |
| 16:07:11 | dansmith | gotcha okay cool | |
| 16:07:13 | efried | links from that patch --^ | |
| 16:07:54 | dansmith | efried: yeah okay I didn't get that from the reno when I first read it, | |
| 16:08:03 | dansmith | but I didn't study both bugs and just wanted to make sure | |
| 16:08:10 | efried | ++ | |