| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-06 | |||
| 15:09:20 | stephenfin | mriedem: Excellent. I'm going to put that in the docs and mark bug 1688054 as invalid | |
| 15:09:21 | openstack | bug 1688054 in openstack-manuals "Flavors in Administrator Guide - confusing description for rxtx factor" [Medium,Confirmed] https://launchpad.net/bugs/1688054 | |
| 15:09:46 | mriedem | stephenfin: that doesn't make the bug invalid | |
| 15:09:53 | mriedem | if the docs are wrong for how it's used today | |
| 15:09:57 | sdague | esberglu: where in the code is that called? | |
| 15:10:13 | stephenfin | Yeah, that's a good point. OK, I'll clarify the intent a little better so | |
| 15:10:25 | stephenfin | ...and join the flavor/flavor2 files | |
| 15:10:28 | stephenfin | #RefactoringFTW | |
| 15:13:55 | efried | esberglu IT or OOT? SDE or traditional? Is it a snapshot operation that's being tested? | |
| 15:13:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | |
| 15:13:57 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | |
| 15:16:26 | efried | Okay, I think I answered my own questions esberglu | |
| 15:17:39 | efried | sdague: This is being done as part of the snapshot operation in our OOT driver: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L97-L100 | |
| 15:19:49 | esberglu | Sorry got pulled away for a few | |
| 15:19:56 | esberglu | efried: Yeah this is OOT snaphot | |
| 15:20:17 | efried | esberglu And presumably this is causing the test to fail. | |
| 15:20:26 | esberglu | efried: Yep | |
| 15:20:50 | efried | But the _tee_as_root doesn't raise, right? We just fail to find the disk in the next part of the code? | |
| 15:24:03 | esberglu | efried: The nova-rootwrap command raises when trying to execute that tee command | |
| 15:24:15 | efried | esberglu Oh. Boo. | |
| 15:24:34 | esberglu | efried: https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/mgmt.py#L58 | |
| 15:24:57 | esberglu | It calls that execute which goes into the rootwrap filters and should match the tee line | |
| 15:25:15 | esberglu | But sometimes it isn't matching that tee filter | |
| 15:25:22 | efried | Which rootwrap filter? | |
| 15:25:51 | esberglu | tee: CommandFilter, tee, root | |
| 15:26:07 | esberglu | in /etc/nova/rootwrap.d/compute.filters | |
| 15:26:49 | esberglu | That line should allow tee to be run as root with any parameters | |
| 15:27:24 | efried | esberglu Sorry, I mean are we using RootwrapDaemonHelper or RootwrapProcessHelper? | |
| 15:30:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314 | |
| 15:30:26 | mriedem | bauzas: here you go ^ | |
| 15:30:59 | bauzas | mriedem: roger. just in a meeting now | |
| 15:31:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for force live migration allocations https://review.openstack.org/501314 | |
| 15:31:33 | efried | esberglu Can you try setting `use_rootwrap_daemon` to True in the compute conf and see if the problem magically disappears? | |
| 15:31:57 | efried | esberglu That's in the [DEFAULT] section | |
| 15:32:01 | esberglu | efried: Yeah that isn't set currently. I can give it a try | |
| 15:32:23 | efried | esberglu Though if non-daemon is busted, that seems like it oughtta be a bug to nova. | |
| 15:34:05 | openstackgerrit | Merged openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146 | |
| 15:34:31 | sdague | efried: yeh, though the real fix would be to get rid of it entirely for privsep | |
| 15:34:39 | sdague | be aware, that mikal's patch series is doing that | |
| 15:34:44 | efried | sdague "it" which? | |
| 15:34:58 | efried | The Process version? | |
| 15:35:13 | sdague | https://review.openstack.org/#/c/489438 | |
| 15:35:20 | sdague | that's 5 patches in | |
| 15:35:44 | sdague | and it will probably take some time to land, but rootwrap for tee is removed at that point | |
| 15:36:21 | sdague | efried: the tee call at all | |
| 15:37:18 | efried | sdague Mm. And the resolution is to decorate my function with this privsep gizmo and then do a regular ol `with open(): write()` ? | |
| 15:37:27 | sdague | yep | |
| 15:37:41 | sdague | but his first patch has to land before that will work | |
| 15:38:36 | efried | esberglu ^^ FYI. Need to add that to the watchlist so we can react when it lands. | |
| 15:39:54 | esberglu | efried: Will do | |
| 15:40:27 | efried | esberglu It would seem we could transition as soon as https://review.openstack.org/#/c/459166/ is in. | |
| 15:40:51 | efried | But in the meantime, <shrug> let's see if using the rootwrap daemon helps. | |
| 15:44:06 | stephenfin | jaypipes: https://review.openstack.org/#/c/498817/ and friends have their gate +1 now, if you were idle enough :) | |
| 15:56:19 | dansmith | mriedem: yeah I had marked that as -W ahead of me writing the release note, but then I wiped it when I pushed a little fix | |
| 15:57:00 | mriedem | i don't know how the tests are even passing | |
| 15:57:12 | dansmith | mriedem: because you missed that there is a patch before this? | |
| 15:57:41 | mriedem | yes | |
| 16:00:14 | esberglu | efried: ^ 459166 just got W+1 | |
| 16:00:59 | esberglu | efried: Oh wait nvm looking at the wrong thing | |
| 16:01:25 | efried | esberglu Ya know, we could patch that guy in and make our own patch that uses it... | |
| 16:05:33 | openstackgerrit | Merged openstack/nova master: Remove unneeded attributes from context https://review.openstack.org/498615 | |
| 16:06:14 | openstackgerrit | Merged openstack/nova master: Pass config object to oslo_reports https://review.openstack.org/485575 | |
| 16:07:16 | openstackgerrit | Merged openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | |
| 16:08:17 | openstackgerrit | Merged openstack/nova master: trivial: Remove some single use function from utils https://review.openstack.org/491513 | |
| 16:18:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Split flavors docs into admin and user guides https://review.openstack.org/501342 | |
| 16:20:23 | dtantsur | dansmith: thanks, I'll check it tomorrow - day full of errands today :( | |
| 16:20:32 | openstackgerrit | Dan Smith proposed openstack/nova master: Add nova-manage db command for ironic flavor migrations https://review.openstack.org/501025 | |
| 16:20:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Add ComputeNodeList.get_by_hypervisor_type() https://review.openstack.org/501343 | |
| 16:20:33 | openstackgerrit | Dan Smith proposed openstack/nova master: Split out the core of the ironic flavor migration https://review.openstack.org/501024 | |
| 16:20:42 | dansmith | dtantsur: ack, np | |
| 16:21:20 | dtantsur | cdent: any specific schedule plans for API SIG PTG? | |
| 16:21:31 | cdent | dansmith: could you give me some early feedback on whether you think this is the right approach: https://review.openstack.org/#/c/501051/ (it’s related to post /allocations) | |
| 16:21:35 | dtantsur | cdent: I'm figuring out conflicts with https://etherpad.openstack.org/p/InteropDenver2017PTG | |
| 16:22:00 | dims | jaypipes : when is the hurricane expected? | |
| 16:22:03 | dansmith | cdent: yeah I know I need to do that, sorry | |
| 16:22:12 | dansmith | dims: weekend I think | |
| 16:22:59 | dansmith | dims: meanwhile portland is on fire and florida refuses to share the water: the | |
| 16:23:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660 | |
| 16:23:03 | dansmith | http://www.oregonlive.com/wildfires/index.ssf/2017/09/eagle_creek_fire_rages_in_colu.html | |
| 16:23:03 | cdent | dtantsur: edleafe and I talked about that yesterday and we’re going to try to cook a schedule, but as there aren’t specific topics (nobody signed up for guided reviews) that are really on the agenda, it is mostly going to be workshopping guideline improvements and whatever comes up | |
| 16:23:12 | cdent | no worries dansmith | |
| 16:23:14 | dims | dansmith : y i heard :) | |
| 16:23:42 | dtantsur | cdent: surprisingly to me, the guided review idea has not got too much traction with other ironic team members.. | |
| 16:23:49 | dims | dansmith : how far is it away from you? | |
| 16:24:21 | dansmith | dims: not far enough, but not currently a risk for us | |
| 16:24:40 | dansmith | dims: but it's crazy smoky outside and ash is falling on everything | |
| 16:25:15 | dansmith | walking outside is like getting a face full of camp fire | |
| 16:25:27 | dims | dansmith : dear lord | |
| 16:25:38 | dansmith | meanwhile, florida is all like "oh water, what are we going to do with all this water? did we mention we have a lot of water on the way?" | |
| 16:25:54 | cdent | that’s just rude | |
| 16:25:59 | dansmith | ikr | |
| 16:26:14 | mnaser | ahem im trying to troubleshoot (what i think is a nova bug).. any idea why instances can be seen in the list, but viewing them returns a 404? | |
| 16:26:32 | mnaser | ex: nova list shows them, nova show <uuid> says no server matching :X | |
| 16:26:41 | dansmith | mnaser: instance mapping missing? | |
| 16:26:45 | dims | dansmith : jaypipes : hopefully you will both make it to ptg | |
| 16:26:57 | dansmith | dims: thanks :) | |
| 16:27:24 | dansmith | mnaser: sounds like cell mappings are in place, so it can iterate and list, but no instance mapping for when it needs to go from instance uuid -> cell | |
| 16:27:35 | dansmith | mnaser: look in api database, instance_mappings table, see if there is one for the instance you're trying | |
| 16:27:38 | mnaser | i wonder why the mapping didnt go in then (customer says they recently failed builds) | |
| 16:28:29 | dansmith | mnaser: idk, should get an instance mapping once you're scheduled (or map you to cell0 if you fail) so we'd need to know what happened exactly to figure it out | |