| *** thorst has joined #openstack-powervm | 00:22 | |
| *** thorst has quit IRC | 00:27 | |
| *** smatzek has joined #openstack-powervm | 01:19 | |
| *** smatzek has quit IRC | 01:41 | |
| *** thorst has joined #openstack-powervm | 01:59 | |
| *** thorst has quit IRC | 02:03 | |
| *** thorst has joined #openstack-powervm | 02:15 | |
| *** thorst has quit IRC | 02:16 | |
| *** thorst has joined #openstack-powervm | 02:33 | |
| *** thorst has quit IRC | 02:34 | |
| *** edmondsw has joined #openstack-powervm | 03:04 | |
| *** kylek3h has joined #openstack-powervm | 03:04 | |
| *** edmondsw has quit IRC | 03:53 | |
| *** edmondsw has joined #openstack-powervm | 03:57 | |
| *** chhavi has joined #openstack-powervm | 04:16 | |
| *** edmondsw has quit IRC | 05:02 | |
| *** chhavi has quit IRC | 05:25 | |
| *** chas has joined #openstack-powervm | 06:42 | |
| *** chhavi has joined #openstack-powervm | 06:42 | |
| *** edmondsw has joined #openstack-powervm | 07:02 | |
| *** edmondsw has quit IRC | 07:07 | |
| *** k0da has joined #openstack-powervm | 08:18 | |
| *** k0da has quit IRC | 08:51 | |
| *** k0da has joined #openstack-powervm | 08:52 | |
| *** k0da has quit IRC | 09:04 | |
| *** k0da has joined #openstack-powervm | 09:17 | |
| *** k0da has quit IRC | 09:43 | |
| *** chhavi has quit IRC | 10:08 | |
| *** edmondsw has joined #openstack-powervm | 11:04 | |
| *** edmondsw has quit IRC | 11:09 | |
| *** smatzek has joined #openstack-powervm | 11:19 | |
| *** thorst has joined #openstack-powervm | 11:25 | |
| *** edmondsw has joined #openstack-powervm | 12:29 | |
| *** esberglu has joined #openstack-powervm | 13:05 | |
| *** smatzek has quit IRC | 13:28 | |
| *** chhavi has joined #openstack-powervm | 13:47 | |
| *** kylek3h has quit IRC | 13:48 | |
| *** k0da has joined #openstack-powervm | 13:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer-powervm master: Updated from global requirements https://review.openstack.org/502634 | 13:59 |
|---|---|---|
| *** smatzek has joined #openstack-powervm | 14:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-powervm master: Updated from global requirements https://review.openstack.org/502693 | 14:05 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/502702 | 14:08 |
| *** chhavi__ has joined #openstack-powervm | 14:21 | |
| *** chhavi has quit IRC | 14:22 | |
| *** kylek3h has joined #openstack-powervm | 14:22 | |
| *** chhavi__ has quit IRC | 14:28 | |
| *** chhavi__ has joined #openstack-powervm | 14:28 | |
| *** edmondsw has quit IRC | 14:37 | |
| *** edmondsw has joined #openstack-powervm | 14:56 | |
| *** efried has joined #openstack-powervm | 14:57 | |
| *** edmondsw has quit IRC | 15:01 | |
| *** edmondsw has joined #openstack-powervm | 15:02 | |
| *** edmondsw_ has joined #openstack-powervm | 15:06 | |
| *** edmondsw has quit IRC | 15:07 | |
| *** efried has quit IRC | 15:11 | |
| *** efried has joined #openstack-powervm | 15:20 | |
| esberglu | efried: I'm finishing up the nova IT spec. In the previous spec there was some stuff about refstack validation. I'm not aware of any work that went into doing that. You know if that is still part of the plan? | 15:45 |
| esberglu | edmondsw_: ^ | 15:46 |
| efried | esberglu I don't know if we're still picking at that. thorst should answer. | 15:46 |
| efried | All, pypowervm 1.1.7 bump for nova-powervm - which is the thing we're waiting for to merge several changes - is here: https://review.openstack.org/#/c/502702/3 | 15:47 |
| esberglu | efried: edmondsw_: See 5887 when you can. Removing the discover_hosts post stack didn't work (stack is still failing to find the compute host) | 15:49 |
| efried | eff | 15:49 |
| esberglu | And it's hitting a decent chunk of CI runs (including the 1.1.7 bump above) | 15:49 |
| efried | esberglu Done. I did post a recheck on ^ already. | 15:50 |
| esberglu | efried: tx | 15:52 |
| esberglu | efried: Also that privesp change got rid of the rootwrap issue, however the same tests (snapshot) are now failing with | 16:00 |
| esberglu | Sep 11 18:24:39.136833 powervm-ci-powervm-devstacked-44144 nova-compute[2922]: ERROR oslo_messaging.rpc.server [01;35m[00mUnserializable message: ('#ERROR', ValueError('I/O operation on closed file',)) | 16:00 |
| efried | esberglu Pastebin the stack | 16:00 |
| esberglu | http://paste.openstack.org/show/620946/ | 16:01 |
| esberglu | It's coming from the | 16:02 |
| esberglu | with utils.temporary_chown(devpath), open(devpath, 'rb') as stream:f | 16:02 |
| esberglu | (minus that last f) | 16:02 |
| esberglu | in /opt/stack/nova-powervm/nova_powervm/virt/powervm/image.py | 16:02 |
| esberglu | L35 | 16:02 |
| esberglu | L45 | 16:02 |
| esberglu | https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/image.py#L45 | 16:03 |
| edmondsw_ | esberglu we don't have bandwidth to do anything with refstack right now | 16:04 |
| esberglu | edmondsw_: K I will remove that bit from the spec | 16:05 |
| edmondsw_ | tx | 16:05 |
| edmondsw_ | esberglu let's put something on the TODO list to figure out why 5887 was necessary | 16:07 |
| efried | esberglu The stack doesn't really indicate this, but I'm betting we bounced off of the rootwrap chown because they removed it. So we'll have to figure out how to do that chown via dac_admin. Or something. | 16:08 |
| esberglu | efried: Okay. I will look into it after lunch, they likely have that in place already | 16:09 |
| esberglu | hopefully | 16:09 |
| efried | esberglu Actually, hm, since that method (temporary_chown) is still in nova.utils, it'd be a little surprising if that rootwrap got whacked. | 16:10 |
| efried | esberglu Is this error consistent? | 16:11 |
| *** smatzek has quit IRC | 16:11 | |
| esberglu | efried: It's hitting all OOT runs | 16:11 |
| esberglu | So we need to fix it to get the 1.1.7 changes in | 16:12 |
| efried | lovely | 16:12 |
| esberglu | yep | 16:12 |
| edmondsw_ | efried esberglu guessing this caused it: https://github.com/openstack/nova/commit/f535e8bb9905b5632416135af5789704db6d2867 | 16:16 |
| efried | edmondsw_ See -nova | 16:17 |
| edmondsw_ | efried yep yep | 16:18 |
| edmondsw_ | I wasn't seeing how that other changed had did it... I did see an example there of how to use dac_admin now | 16:18 |
| edmondsw_ | (specifically with chown) | 16:18 |
| edmondsw_ | but you're right that they didn't address temporary_chown | 16:19 |
| *** edmondsw_ is now known as edmondsw | 16:19 | |
| edmondsw | esberglu you can see an example of how to use dac_admin with chown in nova/virt/disk/vfs/localfs.py | 16:20 |
| esberglu | edmondsw: Do you know how to change the author of the commits? My git config got messed up, but even after I fixed it and put a new patch up it isn't changing the author to me | 16:23 |
| esberglu | edmondsw: It did change the committer, might just be stuck since the first patch was uploaded with the wrong author | 16:24 |
| edmondsw | esberglu better fix test_temporary_chown in nova/tests/unit/test_utils.py when you fix the temporary_chown implementation as well... that UT should have failed when https://review.openstack.org/#/c/471972/31/etc/nova/rootwrap.d/compute.filters merged but didn't | 16:24 |
| edmondsw | esberglu author is the committer of the first patch set | 16:24 |
| edmondsw | you can't change it after that | 16:24 |
| esberglu | edmondsw: Ugh. Oh well | 16:25 |
| edmondsw | yeah, it's kind of annoying | 16:25 |
| edmondsw | pros and cons | 16:25 |
| esberglu | efried: edmondsw: thorst: The first draft for the queens spec is up | 16:26 |
| efried | esberglu Ack, will review. | 16:26 |
| edmondsw | efried esberglu we've got a couple options... 1) fix temporary_chown or 2) put back the rootwrap rule until it's actually safe to remove it | 16:27 |
| edmondsw | I'd prefer #1, but if we need a quick fix... | 16:27 |
| edmondsw | esberglu ack | 16:27 |
| mdrabe | I think you can change the author of a commit no? | 16:28 |
| esberglu | mdrabe: I manually edited the commit message with me as the author and it got overwritten with the original | 16:28 |
| efried | edmondsw Well, #1 is way more likely to fly in nova. If you're suggesting temporarily hacking it up in nova-powervm, that's different. | 16:28 |
| mdrabe | git commit --amend --author="Author Name <email@address.com>" | 16:28 |
| edmondsw | efried the latter | 16:28 |
| edmondsw | we should get a LP bug opened for #1 either way | 16:29 |
| efried | See -nova: mikal confirms it was a miss. | 16:29 |
| esberglu | mdrabe: Oh that worked thanks | 16:29 |
| efried | oh, nice | 16:30 |
| efried | Thanks mdrabe ! | 16:30 |
| edmondsw | mdrabe sweet, tx! | 16:31 |
| efried | esberglu Okay, mikal wants the LP bug, but he's gonna work the fix. | 16:32 |
| *** k0da has quit IRC | 16:34 | |
| efried | esberglu You opening that bug? | 16:36 |
| *** chas has quit IRC | 16:37 | |
| esberglu | efried: Yep | 16:37 |
| *** chas has joined #openstack-powervm | 16:37 | |
| *** edmondsw has quit IRC | 16:38 | |
| *** edmondsw has joined #openstack-powervm | 16:38 | |
| *** chas has quit IRC | 16:41 | |
| *** chas has joined #openstack-powervm | 16:42 | |
| *** edmondsw has quit IRC | 16:43 | |
| *** chas has quit IRC | 16:46 | |
| *** thorst has quit IRC | 16:47 | |
| *** esberglu is now known as esberglu_lunch | 16:52 | |
| *** edmondsw has joined #openstack-powervm | 16:56 | |
| *** edmondsw has quit IRC | 17:01 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: dac_admin_pctxt to stream block device to glance https://review.openstack.org/503078 | 17:10 |
| *** thorst has joined #openstack-powervm | 17:19 | |
| *** edmondsw has joined #openstack-powervm | 17:30 | |
| *** kylek3h has quit IRC | 17:37 | |
| *** kylek3h has joined #openstack-powervm | 17:38 | |
| *** kylek3h has quit IRC | 17:43 | |
| *** edmondsw has quit IRC | 17:46 | |
| *** edmondsw has joined #openstack-powervm | 17:47 | |
| *** edmondsw has quit IRC | 17:51 | |
| *** efried has quit IRC | 18:01 | |
| *** esberglu_lunch is now known as esberglu | 18:05 | |
| *** thorst has quit IRC | 18:06 | |
| *** edmondsw has joined #openstack-powervm | 18:18 | |
| *** chhavi__ has quit IRC | 18:36 | |
| *** chas has joined #openstack-powervm | 18:43 | |
| *** chas has quit IRC | 18:48 | |
| *** efried has joined #openstack-powervm | 19:06 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: dac_admin_pctxt to stream block device to glance https://review.openstack.org/503078 | 19:07 |
| edmondsw | esberglu I posted comments on the spec. Some easy changes. Ping me if you have questions | 19:10 |
| *** kylek3h has joined #openstack-powervm | 19:13 | |
| efried | esberglu Ahem, we don't actually need to patch https://review.openstack.org/#/c/503079/ in anywhere to have it run through our CI. We hopefully just expect the snapshot test to pass now. | 19:14 |
| *** efried has quit IRC | 19:28 | |
| edmondsw | efried take a quick look at the 2 comments I dropped on https://review.openstack.org/#/c/471926/14 | 19:30 |
| esberglu | edmondsw: ack | 19:31 |
| *** kylek3h has quit IRC | 19:38 | |
| *** kylek3h has joined #openstack-powervm | 19:46 | |
| *** kylek3h has quit IRC | 19:48 | |
| *** chas has joined #openstack-powervm | 19:51 | |
| *** kylek3h has joined #openstack-powervm | 19:54 | |
| *** efried has joined #openstack-powervm | 20:26 | |
| edmondsw | mdrabe tx for your replies.. I replied back... | 20:41 |
| *** esberglu has quit IRC | 20:58 | |
| *** esberglu has joined #openstack-powervm | 20:58 | |
| *** esberglu has quit IRC | 21:03 | |
| *** esberglu has joined #openstack-powervm | 21:10 | |
| *** kylek3h has quit IRC | 21:13 | |
| *** edmondsw has quit IRC | 21:14 | |
| *** edmondsw has joined #openstack-powervm | 21:14 | |
| *** edmondsw has quit IRC | 21:19 | |
| *** edmondsw has joined #openstack-powervm | 21:23 | |
| *** edmondsw has quit IRC | 21:27 | |
| *** edmondsw has joined #openstack-powervm | 21:27 | |
| *** edmondsw has quit IRC | 21:31 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: dac_admin_pctxt to stream block device to glance https://review.openstack.org/503078 | 21:32 |
| openstackgerrit | Merged openstack/nova-powervm master: Change NVRAM manager and Swift APIs to accept UUID https://review.openstack.org/471926 | 21:32 |
| *** edmondsw has joined #openstack-powervm | 21:34 | |
| *** thorst has joined #openstack-powervm | 21:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova-powervm master: Updated from global requirements https://review.openstack.org/502702 | 21:37 |
| *** edmondsw has quit IRC | 21:39 | |
| *** edmondsw has joined #openstack-powervm | 21:39 | |
| *** edmondsw_ has joined #openstack-powervm | 21:40 | |
| *** edmondsw has quit IRC | 21:40 | |
| *** edmondsw_ has quit IRC | 21:50 | |
| *** edmondsw has joined #openstack-powervm | 21:51 | |
| *** kylek3h has joined #openstack-powervm | 21:51 | |
| *** edmondsw has quit IRC | 21:55 | |
| *** thorst has quit IRC | 22:07 | |
| *** thorst has joined #openstack-powervm | 22:08 | |
| *** esberglu has quit IRC | 22:08 | |
| *** edmondsw has joined #openstack-powervm | 22:10 | |
| *** thorst has quit IRC | 22:12 | |
| *** kylek3h has quit IRC | 22:13 | |
| *** edmondsw has quit IRC | 22:15 | |
| *** edmondsw has joined #openstack-powervm | 22:15 | |
| *** edmondsw has quit IRC | 22:19 | |
| *** thorst has joined #openstack-powervm | 22:19 | |
| *** thorst has quit IRC | 22:31 | |
| *** esberglu has joined #openstack-powervm | 22:37 | |
| *** edmondsw has joined #openstack-powervm | 22:40 | |
| *** esberglu has quit IRC | 22:41 | |
| openstackgerrit | Eric Fried proposed openstack/nova-powervm master: WIP: dac_admin_pctxt to stream block device to glance https://review.openstack.org/503078 | 22:42 |
| efried | okay, let's try THIS ^ | 22:42 |
| *** edmondsw has quit IRC | 22:51 | |
| *** edmondsw has joined #openstack-powervm | 22:52 | |
| *** chas has quit IRC | 22:55 | |
| *** chas has joined #openstack-powervm | 22:55 | |
| *** efried has quit IRC | 22:55 | |
| *** edmondsw has quit IRC | 22:56 | |
| *** chas has quit IRC | 23:00 | |
| *** adreznec has quit IRC | 23:48 | |
| *** adreznec_ has joined #openstack-powervm | 23:48 | |
| *** adreznec_ is now known as adreznec | 23:49 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!