| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-24 | |||
| 18:56:44 | mriedem | could we not have red hat cores approving red hat changes? | |
| 19:01:22 | melwitt | mriedem: sorry about that. I have asked red hatters not to approve red hat changes that I've +2ed so I think this was just an oversight. downgrading my vote to +1 | |
| 19:01:45 | melwitt | er, I guess that doesn't help because I'm not the approver | |
| 19:02:16 | dansmith | yeah, it won't help :( | |
| 19:02:19 | dansmith | a -2 will, but.. | |
| 19:02:39 | dansmith | it will also reset the gate if it's already there | |
| 19:02:59 | mriedem | we do'nt need to pull it out, | |
| 19:03:07 | mriedem | i'm not going to review it in depth at this point | |
| 19:03:38 | mriedem | the commit message obviously shows there was thought put into it | |
| 19:03:46 | dansmith | lol | |
| 19:04:08 | artom | It's that suave British accent, isn't it? | |
| 19:04:23 | mriedem | who wrote the commit message, lee or booth? | |
| 19:04:57 | mriedem | anywho, | |
| 19:05:00 | mriedem | btw, | |
| 19:05:10 | mriedem | the live migration job runs with ceph i think, | |
| 19:05:21 | mriedem | but i guess we don't enable volume-backed live migratoin do we, | |
| 19:05:34 | mriedem | and even if we did, we don't have a tempest test for encrypted volume-backed live migration to run with ceph | |
| 19:05:42 | mriedem | but we do have new enough libvirt and qemu in the pike UCA to test this | |
| 19:06:02 | mriedem | heh https://review.openstack.org/#/c/536177/ | |
| 19:09:30 | melwitt | mriedem: I have a patch up to enable it but it looks like it's hitting legit problems https://review.openstack.org/#/c/528104/ | |
| 19:10:23 | mriedem | melwitt: is there supposed to be at least even a release note anywhere in this series? | |
| 19:10:32 | mriedem | because given the commit message, it seems pretty complicated | |
| 19:10:53 | mriedem | or some kind of advertisement anywhere about 'hey you can do this thing now with libvirt and encrypted volumes' | |
| 19:10:53 | openstackgerrit | Merged openstack/osc-placement master: CLI for resource classes (v1.2) https://review.openstack.org/511182 | |
| 19:14:55 | melwitt | mriedem: I think there's no release note because it's automatic, that is, operators don't have to do anything, it will use native luks decryption if available. but now that you mention it, it might be helpful to mention that it will use native decryption if qemu and libvirt version combination support it | |
| 19:15:41 | mriedem | right, i mean, unless this is totally rock solid, and shit starts randomly failing, operators are probably going to want to know that something new is going on here | |
| 19:16:57 | melwitt | yeah, I agree with that | |
| 19:17:24 | artom | Heh, we talked about this in the spec: https://review.openstack.org/#/c/490824/5/specs/queens/approved/libvirt-qemu-native-luks.rst@98 | |
| 19:17:31 | artom | And then promptly forgot about it :( | |
| 19:20:11 | dhellmann | mriedem : is https://review.openstack.org/516385 of any interest or should I just abandon that? | |
| 19:23:29 | mriedem | dhellmann: it is of the utmost interest | |
| 19:23:55 | dhellmann | mriedem : ok. I'll leave it open then until someone has a minute to review it. | |
| 19:24:01 | mriedem | dhellmann: already did | |
| 19:24:02 | mriedem | :) | |
| 19:24:04 | dhellmann | :-) | |
| 19:24:05 | dhellmann | thanks | |
| 19:24:34 | mriedem | melwitt: didn't lee say he was going to be out a bit this week? do you think you could take a stab at a release note? | |
| 19:24:38 | mriedem | since you know the change | |
| 19:24:43 | mriedem | as a patch on top of course | |
| 19:24:44 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add unit test for non-placement resize https://review.openstack.org/537614 | |
| 19:24:53 | mriedem | edleafe: thanks for that | |
| 19:25:04 | edleafe | mriedem: np | |
| 19:25:22 | melwitt | mriedem: sure, will do | |
| 19:25:28 | mriedem | melwitt: thanks | |
| 19:25:35 | mriedem | then i guess we can close that one out for queens | |
| 19:26:40 | openstackgerrit | Merged openstack/osc-placement master: Address review comments from allocations patch https://review.openstack.org/536871 | |
| 19:27:09 | openstackgerrit | Merged openstack/osc-placement master: Usage docs and initial release note for osc-placement https://review.openstack.org/536858 | |
| 19:28:05 | mriedem | the cinder gate fix has been promoted btw | |
| 19:28:13 | mriedem | so hopefully stuff starts merging again soon | |
| 19:35:35 | melwitt | that's super news ++ | |
| 19:38:39 | openstackgerrit | Mark Goddard proposed openstack/nova master: Unplug all VIFs from ironic nodes during tear down https://review.openstack.org/537626 | |
| 19:49:55 | dansmith | melwitt: mriedem tssurya: do we need to have a cells meeting today? I've got nothing new to talk abotu | |
| 19:50:43 | archit | msg amodi identify titanic123 | |
| 19:51:11 | dansmith | whoopsie :) | |
| 19:51:13 | archit | lol | |
| 19:51:20 | archit | yep .:( | |
| 19:51:27 | melwitt | dansmith: I don't have anything new either | |
| 19:51:28 | efried | swhy I always do those in the status channel | |
| 19:51:35 | mriedem | dansmith: there are no major new bugs that i'm aware of, | |
| 19:51:40 | mriedem | and the resize + alternate hosts change is approved | |
| 19:51:49 | efried | Is this cinder bug going to affect all patches? | |
| 19:51:58 | mriedem | efried: anything that runs tempest api tests | |
| 19:52:01 | mriedem | so all dsvm jobs | |
| 19:52:23 | efried | so all my nova patches | |
| 19:52:50 | mriedem | it's #13 in the gate right now | |
| 19:53:03 | mriedem | dansmith: melwitt: so only other thing would be ptg topics, but that can wait until after FF | |
| 19:53:15 | dansmith | cool | |
| 19:53:45 | mriedem | tssurya: tssurya_: do you have any status updates from cern's migration to cells v2? | |
| 19:54:41 | tssurya_ | mriedem : actually belmiro was going to participate in the cells meeting today, not sure though; but yea we are going to be doing in at the end of Feb | |
| 19:54:57 | dansmith | drat! :) | |
| 19:55:14 | mriedem | tssurya_: doing it in prod? but you're all clear in pre-prod testing? | |
| 19:55:21 | tssurya_ | migrating to Ocata during the last days of this month | |
| 19:55:29 | dansmith | tssurya: can you find out if he's going to be around? | |
| 19:55:33 | tssurya_ | and we will have to move to Pike in a few weeks | |
| 19:55:35 | mriedem | ok; would be good to know as soon as possible if there are blockers *before* rocky | |
| 19:55:35 | tssurya_ | yes in prod | |
| 19:55:52 | mriedem | because i think we still plan on dropping cellsv1 and nova-net in rocky | |
| 19:56:11 | tssurya_ | dansmith : yea will ping him and see if he will be joining | |
| 19:56:17 | dansmith | tssurya_: okay thanks | |
| 19:56:22 | tssurya_ | mriedem : yea we have half out cloud on neutron | |
| 19:56:32 | tssurya_ | and trying to scale it to move out ot nova-net | |
| 19:56:36 | tssurya_ | out of * | |
| 19:56:47 | tssurya_ | our * | |
| 20:01:45 | mriedem | this needs another +2 - https://review.openstack.org/#/c/533210/ - moves the nova-tox-functional job in-tree so we can define our own files blacklist, so we don't have gate issues again with notification samples only patches not getting test | |
| 20:01:46 | mriedem | *tested | |
| 20:11:20 | openstackgerrit | sean mooney proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | |
| 20:11:20 | openstackgerrit | sean mooney proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | |
| 20:11:21 | openstackgerrit | sean mooney proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | |
| 20:11:21 | openstackgerrit | sean mooney proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/507481 | |
| 20:16:32 | sean-k-mooney | stephenfin: if you have time tomorow rodoflos nic feature based schduling patch series is now all rebased with versioned objects https://review.openstack.org/#/q/topic:bp/enable-sriov-nic-features+(status:Open) | |
| 20:24:20 | openstackgerrit | melanie witt proposed openstack/nova master: Add release note for QEMU native LUKS decryption https://review.openstack.org/537642 | |
| 20:32:17 | mriedem | melwitt: some questions in that reno | |
| 20:32:56 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624 | |
| 20:32:56 | openstackgerrit | Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625 | |
| 20:32:57 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | |
| 20:32:57 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 20:32:58 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 20:32:58 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 20:32:59 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: New-style _set_inventory_for_provider https://review.openstack.org/537648 | |
| 20:46:59 | dansmith | tssurya_: any word from belmiro? | |