| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-21 | |||
| 12:11:35 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:19:05 | bauzas | cdent: sorry was afk | |
| 12:19:26 | cdent | no problem | |
| 12:19:33 | bauzas | cdent: we could yeah, totally | |
| 12:19:40 | bauzas | cdent: I just need to resurrect this one | |
| 12:20:37 | cdent | I was going through the links on the prevous rp update and wanted to make sure that anything there was still relevant before creating the new one | |
| 12:28:28 | bauzas | np | |
| 12:28:45 | openstackgerrit | Sean Dague proposed openstack/nova master: request_log addition for running under uwsgi https://review.openstack.org/485602 | |
| 12:32:31 | openstackgerrit | Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 12:44:13 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 12:46:05 | openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | |
| 12:52:57 | figleaf | leakypipes: Is https://review.openstack.org/#/c/483566/ the last one in your series? | |
| 12:53:12 | figleaf | leakypipes: if so, I can start working on adding alternates | |
| 13:13:49 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/485634 | |
| 13:13:59 | Dinesh_Bhor | stephenfin, mriedem: Hi, wanted to discuss this blueprint: https://blueprints.launchpad.net/nova/+spec/specify-keymap-on-server-boot | |
| 13:14:47 | stephenfin | Dinesh_Bhor: Are you referring to the bug I have opened to deprecate those options? | |
| 13:14:52 | Dinesh_Bhor | stephenfin, mriedem: We started work on this But it seems to contradict with this bug: https://bugs.launchpad.net/nova/+bug/1682020/ | |
| 13:14:54 | openstack | Launchpad bug 1682020 in OpenStack Compute (nova) "Remove nova default keymap option for qemu-kvm (deprecated)" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane) | |
| 13:15:03 | Dinesh_Bhor | stephenfin: yes | |
| 13:15:04 | stephenfin | Indeed you are :) | |
| 13:15:11 | stephenfin | Right, so what do you want to know? | |
| 13:15:45 | Dinesh_Bhor | stephenfin: Whether should we proceed work of that blueprint | |
| 13:15:48 | Dinesh_Bhor | ? | |
| 13:17:36 | stephenfin | Dinesh_Bhor: Personally, I wouldn't | |
| 13:17:48 | stephenfin | You've seen the comment from danpb on that bug | |
| 13:18:16 | stephenfin | We shouldn't be doing that, and it's breaking stuff for us. The blueprint sounds like it would be putting lipstick on a pig, so to speak :) | |
| 13:19:11 | openstackgerrit | Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | |
| 13:21:16 | Dinesh_Bhor | stephenfin: ohh, ohk. Understood. Could you please comment on those patches about your opinion? | |
| 13:21:52 | Dinesh_Bhor | stephenfin: patches submitted against that blueprint | |
| 13:22:26 | mriedem | keymapping huh, huawei has interest in something similar for vnc consoles i think | |
| 13:22:31 | mriedem | but i'd have to dig up the details | |
| 13:23:30 | mriedem | vdrok: a couple of small things in https://review.openstack.org/#/c/419975/ just for docs | |
| 13:24:03 | vdrok | mriedem: thx, updating | |
| 13:25:40 | stephenfin | mriedem, vdrok: Docs, you say? That's going to bring merge conflicts for doc-migration | |
| 13:25:53 | stephenfin | I assume it's too important to -2 until doc-migration is, as I've been doing of late? | |
| 13:26:31 | vdrok | stephenfin: it's about comments in code and relnote update I think, so should not affect the doc-migration | |
| 13:28:10 | stephenfin | vdrok: Yeah, it's just the support-matrix changes that would cause the issue | |
| 13:28:45 | stephenfin | vdrok: Unless mriedem disagrees, could I ask you to split that little bit out into a separate patch, and we'll merge it in after? | |
| 13:28:56 | vdrok | stephenfin: oh, ok, I'll check the doc-migration patches then, can rebase on top of the relevant one | |
| 13:29:27 | stephenfin | maybe not, because I can't be sure they'll get in before feature freeze, and I'm sure you want it in then | |
| 13:29:45 | stephenfin | two separate patches would be better, because we can merge docs after feature freeze | |
| 13:30:02 | stephenfin | vdrok: just the support-matrix doc. Release note is fine | |
| 13:30:25 | vdrok | okie | |
| 13:31:16 | stephenfin | ta :) | |
| 13:37:19 | jangutter | Would anyone have the time to give https://review.openstack.org/#/c/483459 (Netronome VIF type) a quick once-over? 62 lines of code, 290 lines of tests and 42 litres of tears. | |
| 13:46:35 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 13:46:58 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Update ironic feature matrix https://review.openstack.org/486148 | |
| 13:58:43 | stephenfin | Dinesh_Bhor: done and done | |
| 14:00:15 | stephenfin | vdrok: docs patch +2'd | |
| 14:00:24 | stephenfin | I'll leave the code one to mriedem for now | |
| 14:00:28 | vdrok | stephenfin: thanks :) | |
| 14:02:10 | Dinesh_Bhor | stephenfin: thank you so much | |
| 14:03:07 | openstackgerrit | Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 14:05:47 | jaypipes | figleaf: yes it is. please do work on alternates, that would be great | |
| 14:05:50 | jaypipes | and thank you! | |
| 14:06:22 | jaypipes | will give me time to add func tests around failures on compute nodes | |
| 14:07:06 | openstackgerrit | Sean Dague proposed openstack/nova master: request_log addition for running under uwsgi https://review.openstack.org/485602 | |
| 14:07:15 | sdague | mriedem: that should be ready to go... I hope ^^^ | |
| 14:09:28 | openstackgerrit | Felipe Monteiro proposed openstack/nova master: Update policy description for 'instance_actions' https://review.openstack.org/484609 | |
| 14:09:32 | figleaf | jaypipes: on it | |
| 14:25:27 | leakypipes | mriedem, superdan, melwitt: super important bug fix: https://review.openstack.org/#/c/485088/6 | |
| 14:25:45 | leakypipes | it addresses the transient NoValidHosts issue we've been seeing. | |
| 14:25:52 | leakypipes | in some of the functional test failures. | |
| 14:26:27 | melwitt | noice | |
| 14:35:04 | leakypipes | mriedem: btw, did you see cdent's ? to you on https://review.openstack.org/#/c/483566/? | |
| 14:35:25 | mriedem | just sat down with coffee for the first time today, so i haven't seen anything | |
| 14:35:40 | mriedem | just a lot of pings | |
| 14:38:30 | mriedem | leakypipes: it's a comment on that change | |
| 14:38:35 | mriedem | leakypipes: looking at the grenade failure | |
| 14:38:42 | mriedem | http://logs.openstack.org/66/483566/6/check/gate-grenade-dsvm-neutron-ubuntu-xenial/b0077c3/logs/new/screen-n-sch.txt.gz?level=TRACE#_2017-07-20_23_58_04_589 | |
| 14:38:46 | mriedem | we have 1 node in this job | |
| 14:39:04 | mriedem | the scheduler goes to submit an allocation and it fails because something else slipped in and changed the inventory at the same time | |
| 14:39:28 | mriedem | with concurrently running tests in a single node job, if we merge this, i think it's going to kill the gate | |
| 14:39:50 | leakypipes | gotcha | |
| 14:40:10 | mriedem | so i think that means if we have exhausted the list of filtered hosts in the filter scheduler, | |
| 14:40:23 | mriedem | we have to go back up to the scheduler manager and retry by getting a fresh set of allocation candidates | |
| 14:40:38 | mriedem | until CONF.num_retries or whatever | |
| 14:40:52 | leakypipes | mriedem: well, we could also retry the same host if we get that specific error. | |
| 14:41:51 | leakypipes | mriedem: the issue is we'd need to put somehting into the claim_resources() report client method to distinguish between 409 Conflict for concurrent update and 409 Conflict for InvalidInventory (which is returned when the capacity was exceeded by another thread and thus the same claim request would fail) | |
| 14:41:54 | mriedem | i'm happy with that, | |
| 14:42:02 | mriedem | i just wasn't sure if we could do it | |
| 14:43:04 | leakypipes | mriedem: yup. gimme about an hour. I'll add a dependent patch before that one that adds the error condition distinguishing thing | |
| 14:43:14 | leakypipes | mriedem: and then mod the patch to retry same host on concurrent update | |
| 14:43:58 | mriedem | ack | |
| 14:46:41 | figleaf | leakypipes: I have a small but significant bug in https://review.openstack.org/#/c/483566/ as long as you're fixing the 409 claim conflict | |
| 14:47:36 | figleaf | leakypipes: Also, did we agree that the number of alternates would be based on CONF.scheduler.max_attempts? | |
| 14:47:49 | leakypipes | figleaf: yeah | |
| 14:47:54 | figleaf | ok | |
| 14:51:22 | mriedem | vdrok: https://review.openstack.org/#/c/419975/18..19/doc/source/support-matrix.ini ? | |
| 14:51:35 | mriedem | you dropped that in PS19 | |
| 14:51:42 | mriedem | otherwise i'd +2 | |
| 14:53:19 | vdrok | mriedem: stephenfin asked to move it to a separate change to avoid conflict with doc migration | |
| 14:54:04 | vdrok | mriedem https://review.openstack.org/486148 | |
| 14:55:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 14:55:42 | mriedem | vdrok: ack, + | |
| 14:55:43 | mriedem | +2 | |
| 14:56:11 | mriedem | very simple +W for someone https://review.openstack.org/#/c/419975/ | |
| 14:56:19 | vdrok | Thanks! | |
| 14:56:41 | mriedem | yw | |
| 14:58:16 | melwitt | mriedem: does that one imply we also need to update the hypervisor matrix? | |