| *** edmondsw has joined #openstack-cinder | 00:02 | |
| *** s-shiono has joined #openstack-cinder | 00:04 | |
| *** yangyapeng has joined #openstack-cinder | 00:04 | |
| *** edmondsw has quit IRC | 00:06 | |
| *** felipemonteiro_ has joined #openstack-cinder | 00:11 | |
| *** yangyapeng has quit IRC | 00:11 | |
| *** ircuser-1 has quit IRC | 00:26 | |
| *** itlinux has quit IRC | 00:29 | |
| *** sdake_ has quit IRC | 00:32 | |
| *** sdake_ has joined #openstack-cinder | 00:32 | |
| *** wanghao has joined #openstack-cinder | 00:39 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/490849 | 00:42 |
|---|---|---|
| *** zhurong has joined #openstack-cinder | 00:42 | |
| *** zhurong has quit IRC | 00:53 | |
| *** Shunli has joined #openstack-cinder | 00:58 | |
| *** yangyapeng has joined #openstack-cinder | 01:10 | |
| *** yangyapeng has quit IRC | 01:10 | |
| *** markvoelker has joined #openstack-cinder | 01:15 | |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** crushil has joined #openstack-cinder | 01:38 | |
| *** zhaochao has joined #openstack-cinder | 01:39 | |
| *** randomhack has joined #openstack-cinder | 01:41 | |
| *** randomhack has quit IRC | 01:46 | |
| *** lhx__ has joined #openstack-cinder | 01:49 | |
| *** edmondsw has joined #openstack-cinder | 01:50 | |
| *** edmondsw has quit IRC | 01:54 | |
| *** markvoelker has quit IRC | 02:02 | |
| *** markvoelker has joined #openstack-cinder | 02:02 | |
| *** markvoelker has quit IRC | 02:34 | |
| *** felipemonteiro__ has joined #openstack-cinder | 03:08 | |
| *** dave-mccowan has quit IRC | 03:09 | |
| *** felipemonteiro_ has quit IRC | 03:11 | |
| *** felipemonteiro__ has quit IRC | 03:18 | |
| *** crushil has quit IRC | 03:26 | |
| *** gkadam has joined #openstack-cinder | 03:32 | |
| *** Nel1x has quit IRC | 03:37 | |
| *** edmondsw has joined #openstack-cinder | 03:38 | |
| *** edmondsw has quit IRC | 03:42 | |
| *** links has joined #openstack-cinder | 03:49 | |
| *** nicolasbock has joined #openstack-cinder | 03:51 | |
| *** diablo_rojo has joined #openstack-cinder | 03:55 | |
| openstackgerrit | Merged openstack/cinder master: Sort cinder-manage commands https://review.openstack.org/489686 | 04:21 |
| stakeda | hi, cinder cores, could you review this small patch? https://review.openstack.org/#/c/485101/ | 04:33 |
| *** Poornima has joined #openstack-cinder | 04:33 | |
| stakeda | we would like to fix our ci before our summer holidays. thanks! | 04:33 |
| *** PsionTheory has joined #openstack-cinder | 04:37 | |
| *** david-lyle has quit IRC | 04:38 | |
| openstackgerrit | junbo.li proposed openstack/cinder master: [api-ref] Add worker cleanup for service docs api https://review.openstack.org/486290 | 04:46 |
| *** Poornima has quit IRC | 04:50 | |
| *** itlinux has joined #openstack-cinder | 04:54 | |
| *** Shunli has quit IRC | 05:06 | |
| *** Shunli has joined #openstack-cinder | 05:06 | |
| openstackgerrit | junbo.li proposed openstack/cinder master: [Api-ref] Add force parameter to volume delete https://review.openstack.org/491164 | 05:07 |
| *** pgadiya has joined #openstack-cinder | 05:21 | |
| *** lhx__ has quit IRC | 05:26 | |
| *** zhurong has joined #openstack-cinder | 05:28 | |
| *** itlinux has quit IRC | 05:32 | |
| *** psachin has joined #openstack-cinder | 05:35 | |
| *** Poornima has joined #openstack-cinder | 05:40 | |
| *** randomhack has joined #openstack-cinder | 05:42 | |
| *** randomhack has quit IRC | 05:47 | |
| *** diablo_rojo has quit IRC | 05:52 | |
| *** jiaopeng_ has joined #openstack-cinder | 05:55 | |
| *** jiaopen__ has joined #openstack-cinder | 05:56 | |
| *** jiaopeng_ has quit IRC | 05:56 | |
| *** jiaopen__ has quit IRC | 05:56 | |
| *** yangyapeng has quit IRC | 06:01 | |
| *** yangyapeng has joined #openstack-cinder | 06:02 | |
| *** lhx__ has joined #openstack-cinder | 06:12 | |
| *** yangyape_ has joined #openstack-cinder | 06:45 | |
| *** yangyapeng has quit IRC | 06:45 | |
| *** bhagyashris has joined #openstack-cinder | 07:01 | |
| *** stakeda has quit IRC | 07:06 | |
| *** nicolasbock_ has joined #openstack-cinder | 07:06 | |
| *** yangyape_ is now known as yangyapeng | 07:07 | |
| *** nicolasbock_ has quit IRC | 07:07 | |
| *** rcernin has joined #openstack-cinder | 07:08 | |
| lhx__ | geguileo, could you have a look at this bug? https://bugs.launchpad.net/cinder/+bug/1709011 | 07:10 |
| openstack | Launchpad bug 1709011 in Cinder "Support force delete backup when rados connection isn't initialized successfully." [Undecided,New] - Assigned to Hanxi Liu (hanxi-liu) | 07:10 |
| geguileo | lhx__: hi, why does it go to 'error_deleting' status? r:-?? | 07:11 |
| lhx__ | geguileo, fail_reason: Delete backup is aborted due to backup service is down | 07:12 |
| geguileo | lhx__: ok | 07:13 |
| lhx__ | geguileo, https://github.com/openstack/cinder/blob/master/cinder/backup/drivers/ceph.py#L293 | 07:13 |
| geguileo | lhx__: anything you need from me on that one? | 07:14 |
| *** edmondsw has joined #openstack-cinder | 07:14 | |
| *** nizam037 has joined #openstack-cinder | 07:16 | |
| lhx__ | geguileo, I just want to know if ceph backup driver could support force delete, so I won't implement something unuseful before that | 07:16 |
| lhx__ | ;) | 07:16 |
| geguileo | lhx__: Yes, it should support it | 07:17 |
| *** edmondsw has quit IRC | 07:18 | |
| lhx__ | geguileo, thanks for your advice :) | 07:19 |
| geguileo | lhx__: feel free to add me to the review once you are done coding | 07:19 |
| lhx__ | okay | 07:19 |
| *** tesseract has joined #openstack-cinder | 07:20 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Inspection Mechanism For Capacity Limited Host https://review.openstack.org/308869 | 08:11 |
| *** markvoelker has joined #openstack-cinder | 08:17 | |
| *** randomhack has joined #openstack-cinder | 08:18 | |
| *** markvoelker_ has joined #openstack-cinder | 08:19 | |
| *** aarefiev_afk is now known as aarefiev | 08:21 | |
| *** toabctl has joined #openstack-cinder | 08:22 | |
| toabctl | hey | 08:22 |
| toabctl | could some core merge https://review.openstack.org/#/c/488924/ please? it already has one +2 and is a simple man-page build fix. TIA | 08:22 |
| *** markvoelker has quit IRC | 08:23 | |
| *** randomhack has quit IRC | 08:24 | |
| *** s-shiono has quit IRC | 08:35 | |
| *** fnordahl_ is now known as fnordahl | 08:42 | |
| openstackgerrit | zhuangzejian proposed openstack/cinder master: Modify a little desc to satisfy the code https://review.openstack.org/472118 | 08:48 |
| *** nizam037 has quit IRC | 08:55 | |
| openstackgerrit | Guy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver https://review.openstack.org/461665 | 08:58 |
| *** edmondsw has joined #openstack-cinder | 09:02 | |
| *** edmondsw has quit IRC | 09:06 | |
| openstackgerrit | zhuangzejian proposed openstack/cinder master: Fix wrong param transfer in scheduler flows. https://review.openstack.org/491408 | 09:10 |
| lhx__ | geguileo, I saw only a few operations supporing arg '--force' in v3, compared with v2. is there any plan to add more support of that arg in v3? | 09:11 |
| lhx__ | https://github.com/openstack/python-cinderclient/blob/18381fb872e7280539e2b41ea2673bd230c89518/cinderclient/v3/shell.py#L799 | 09:11 |
| geguileo | lhx__: v3 should have more support than v2 | 09:11 |
| *** e0ne has joined #openstack-cinder | 09:12 | |
| geguileo | lhx__: it is inheriting all v2 operations and just replacing or extending some | 09:12 |
| lhx__ | geguileo, but the cli doc tells 'Volume version 2 only' | 09:14 |
| lhx__ | https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/volume-backup.html#cmdoption-volume-backup-delete-force | 09:14 |
| geguileo | lhx__: have you tried it in v3? | 09:14 |
| lhx__ | geguileo, yep, you know this arg is used for delete other than error or available, but failed when delete a error_deleting backup | 09:16 |
| openstackgerrit | zhuangzejian proposed openstack/cinder master: Fix wrong param transfer in scheduler flows. https://review.openstack.org/491408 | 09:16 |
| geguileo | lhx__: it can fail, you'll see errors in logs | 09:17 |
| lhx__ | geguileo, c-api log? | 09:19 |
| geguileo | lhx__: if the error was detected there, otherwise cinder backup log | 09:20 |
| lhx__ | geguileo, the backup log didn't throw anything error :( | 09:22 |
| geguileo | lhx__: then it'll be in the api log | 09:22 |
| lhx__ | geguileo, neither did api log | 09:23 |
| geguileo | lhx__: then the API didn't receive the request and the error was in the client side | 09:23 |
| geguileo | lhx__: look with --debug | 09:23 |
| *** lennyb has quit IRC | 09:25 | |
| lhx__ | geguileo, http://paste.openstack.org/show/617658/ | 09:25 |
| geguileo | lhx__: looks like the backend you are using doesn't support the operation | 09:26 |
| geguileo | lhx__: don't tell me you are trying it with RBD, which you already know is not supported | 09:26 |
| *** Shunli has quit IRC | 09:28 | |
| lhx__ | geguileo, haha, definitely RBD, but it occured because the cinder didn't connect to ceph client | 09:28 |
| lhx__ | https://github.com/openstack/cinder/blob/master/cinder/backup/drivers/ceph.py#L293 | 09:28 |
| geguileo | lhx__: no, it happened because the driver doesn't support force delete | 09:30 |
| *** yangyape_ has joined #openstack-cinder | 09:31 | |
| lhx__ | geguileo, because of inccorect configuration, backup was created and deleted, but the status went to 'error_deleting' | 09:31 |
| geguileo | lhx__: that doesn't matter | 09:31 |
| geguileo | lhx__: RBD driver DOES NOT support force delete | 09:31 |
| geguileo | lhx__: so it will NOT work | 09:31 |
| geguileo | lhx__: it's clear in the error message you get in the client: failed: Operation not supported: force delete. | 09:32 |
| *** yangyapeng has quit IRC | 09:32 | |
| lhx__ | geguileo, I think backup should be deleted in cinder side bacause ceph backend didn't create anything, it's a special case | 09:34 |
| geguileo | lhx__: that sounds good, but doesn't work, cinder doesn't know that nothing has been created | 09:35 |
| *** PsionTheory has quit IRC | 09:36 | |
| lhx__ | geguileo, I knew cinder know nothing, I jsut think support delete operation is better in the case of connetion failure, :) | 09:42 |
| geguileo | lhx__: then you need to add the mechanism into the ceph.py file | 09:43 |
| geguileo | I don't know what we are discussing anymore | 09:43 |
| *** dalgaaf has quit IRC | 09:47 | |
| *** sdague has joined #openstack-cinder | 09:47 | |
| *** lhx__ has quit IRC | 09:47 | |
| *** lhx__ has joined #openstack-cinder | 09:48 | |
| lhx__ | geguileo, sorry, I didn't get myself accorss, I meant when if cinder backup intailization failed and a backup has created with status of error, it should be legal to delete the backup | 09:48 |
| *** dalgaaf has joined #openstack-cinder | 09:48 | |
| geguileo | lhx__: agreed | 09:48 |
| geguileo | lhx__: but it's not implemented | 09:49 |
| lhx__ | geguileo, no need implementent in ceph :) | 09:49 |
| geguileo | lhx__: yes yo have to implement it | 09:49 |
| lhx__ | I'd like to | 09:49 |
| geguileo | lhx__: what if you request to force delete a backup that is currently in progress? | 09:50 |
| *** chhavi has joined #openstack-cinder | 09:51 | |
| lhx__ | geguileo, maybe just change the status 'error_deleting ' to 'error', no need to force delete | 09:53 |
| geguileo | lhx__: you can do that right now, you can change the status of a backup | 09:55 |
| lhx__ | geguileo, thanks :) | 09:55 |
| *** gcb has quit IRC | 10:03 | |
| *** lpetrut has joined #openstack-cinder | 10:08 | |
| *** yangyape_ has quit IRC | 10:30 | |
| *** yangyapeng has joined #openstack-cinder | 10:31 | |
| *** yangyapeng has quit IRC | 10:33 | |
| *** yangyapeng has joined #openstack-cinder | 10:35 | |
| *** lennyb has joined #openstack-cinder | 10:37 | |
| *** yangyapeng has quit IRC | 10:39 | |
| *** markvoelker has joined #openstack-cinder | 10:49 | |
| *** edmondsw has joined #openstack-cinder | 10:50 | |
| *** markvoelker_ has quit IRC | 10:53 | |
| *** markvoelker_ has joined #openstack-cinder | 10:54 | |
| *** markvoelker has quit IRC | 10:54 | |
| *** edmondsw has quit IRC | 10:54 | |
| *** Poornima has quit IRC | 10:56 | |
| *** markvoelker has joined #openstack-cinder | 10:57 | |
| *** markvoelker has quit IRC | 10:59 | |
| *** markvoelker_ has quit IRC | 11:00 | |
| *** markvoelker has joined #openstack-cinder | 11:01 | |
| *** strigazi_OFF is now known as strigazi | 11:05 | |
| *** markvoelker_ has joined #openstack-cinder | 11:12 | |
| *** pgadiya has quit IRC | 11:14 | |
| *** markvoelker has quit IRC | 11:15 | |
| *** Poornima has joined #openstack-cinder | 11:21 | |
| *** eharney has quit IRC | 11:27 | |
| *** catintheroof has joined #openstack-cinder | 11:28 | |
| *** pgadiya has joined #openstack-cinder | 11:28 | |
| *** markvoelker_ has quit IRC | 11:28 | |
| *** markvoelker has joined #openstack-cinder | 11:29 | |
| openstackgerrit | xiaoqin proposed openstack/cinder master: Add Storwize replication group support https://review.openstack.org/469394 | 11:34 |
| *** markvoelker has quit IRC | 11:34 | |
| *** yangyapeng has joined #openstack-cinder | 11:37 | |
| *** catintheroof has quit IRC | 11:38 | |
| *** yangyapeng has quit IRC | 11:39 | |
| *** yangyapeng has joined #openstack-cinder | 11:39 | |
| *** markvoelker has joined #openstack-cinder | 11:40 | |
| *** markvoelker_ has joined #openstack-cinder | 11:41 | |
| *** markvoelker_ has quit IRC | 11:42 | |
| *** markvoelker_ has joined #openstack-cinder | 11:44 | |
| *** zhurong has quit IRC | 11:44 | |
| *** markvoelker has quit IRC | 11:44 | |
| *** markvoelker has joined #openstack-cinder | 11:47 | |
| *** liverpooler has quit IRC | 11:47 | |
| *** markvoelker_ has quit IRC | 11:50 | |
| *** catintheroof has joined #openstack-cinder | 11:50 | |
| *** markvoelker_ has joined #openstack-cinder | 11:51 | |
| *** markvoelker has quit IRC | 11:52 | |
| *** markvoelker has joined #openstack-cinder | 11:53 | |
| *** baojg has joined #openstack-cinder | 11:53 | |
| *** catintheroof has quit IRC | 11:55 | |
| *** markvoelker_ has quit IRC | 11:56 | |
| *** markvoelker_ has joined #openstack-cinder | 11:56 | |
| *** markvoelker has quit IRC | 12:00 | |
| *** markvoelker has joined #openstack-cinder | 12:02 | |
| *** markvoelker_ has quit IRC | 12:03 | |
| *** nizam037 has joined #openstack-cinder | 12:07 | |
| *** lhx__ has quit IRC | 12:15 | |
| *** pgadiya has quit IRC | 12:15 | |
| *** lhx_ has joined #openstack-cinder | 12:15 | |
| *** lhx_ has quit IRC | 12:18 | |
| *** pgadiya has joined #openstack-cinder | 12:19 | |
| *** gkadam has quit IRC | 12:22 | |
| *** Poornima has quit IRC | 12:25 | |
| *** lhx_ has joined #openstack-cinder | 12:29 | |
| *** catintheroof has joined #openstack-cinder | 12:30 | |
| *** pgadiya has quit IRC | 12:33 | |
| *** gcb has joined #openstack-cinder | 12:35 | |
| *** pgadiya has joined #openstack-cinder | 12:38 | |
| *** pgadiya has quit IRC | 12:43 | |
| *** mikal has quit IRC | 12:44 | |
| *** pgadiya has joined #openstack-cinder | 12:44 | |
| *** dave-mccowan has joined #openstack-cinder | 12:49 | |
| *** dustins has joined #openstack-cinder | 12:49 | |
| *** lhx_ has quit IRC | 12:53 | |
| *** edmondsw has joined #openstack-cinder | 12:55 | |
| *** edmondsw_ has joined #openstack-cinder | 12:57 | |
| *** rawanh has joined #openstack-cinder | 12:57 | |
| *** edmondsw has quit IRC | 12:59 | |
| *** mikal has joined #openstack-cinder | 13:00 | |
| *** edmondsw_ has quit IRC | 13:01 | |
| *** edmondsw has joined #openstack-cinder | 13:02 | |
| *** liverpooler has joined #openstack-cinder | 13:06 | |
| *** edmondsw has quit IRC | 13:07 | |
| *** lucasxu has joined #openstack-cinder | 13:12 | |
| *** edmondsw has joined #openstack-cinder | 13:12 | |
| *** edmondsw_ has joined #openstack-cinder | 13:13 | |
| *** edmondsw has quit IRC | 13:13 | |
| *** lhx_ has joined #openstack-cinder | 13:17 | |
| *** nizam037 has quit IRC | 13:17 | |
| openstackgerrit | Merged openstack/cinder master: [Api-ref] Add force parameter to volume delete https://review.openstack.org/491164 | 13:23 |
| *** csaikia has joined #openstack-cinder | 13:26 | |
| *** liverpooler has quit IRC | 13:27 | |
| *** liverpooler has joined #openstack-cinder | 13:27 | |
| rawanh | Guys, Can we push this commit to be merged ? | 13:28 |
| rawanh | https://review.openstack.org/#/c/482562/ | 13:28 |
| smcginnis | rawanh: No, not right now. | 13:28 |
| smcginnis | rawanh: And you don't really need to +1 your own patch. It's kind of assumed you are OK with it since you pushed up the code. ;) | 13:28 |
| rawanh | @smcginnis: is there any missing requirement for it or things that i should do? | 13:29 |
| smcginnis | rawanh: Just need to wait until things open up for queens. | 13:30 |
| ildikov | hey. has anyone seen this error in the c-api logs nowadays: TypeError: coercing to Unicode: need string or buffer, tuple found | 13:31 |
| ildikov | it seems to come from here: /opt/stack/cinder/cinder/api/middleware/fault.py | 13:31 |
| smcginnis | ildikov: Sounds like someone not formatting a string correctly. | 13:31 |
| ildikov | and here: /opt/stack/cinder/cinder/api/openstack/wsgi.py | 13:31 |
| *** pgadiya has quit IRC | 13:32 | |
| ildikov | smcginnis: yeah, just the origin looks pretty weird | 13:32 |
| smcginnis | ildikov: If it's in the wsgi stuff, that may be a little harder to track down the source. It is pretty weird. ;) | 13:33 |
| ildikov | smcginnis: I got this when calling attachment_show and I wonder how any sort of error from there gets to api/middleware? | 13:33 |
| ildikov | smcginnis: sounds awesome, just what I wanted to hear on a Monday ;) | 13:33 |
| smcginnis | ildikov: Do you have a trace I can look at? | 13:33 |
| ildikov | https://www.irccloud.com/pastebin/RTscKcmD/attachment_show_error | 13:34 |
| rawanh | smcginnis: aha thanks :) | 13:34 |
| *** lbragstad has joined #openstack-cinder | 13:36 | |
| ildikov | smcginnis: not a nice one, sorry :/ | 13:38 |
| smcginnis | ildikov: Anything before that? It's in wsgi.Appliation.dispatch, so it's hard to easily trace that from that log. | 13:39 |
| ildikov | https://www.irccloud.com/pastebin/vvLKbwgH/ | 13:40 |
| ildikov | that's all in the logs | 13:41 |
| e0ne | smcginnis, ildikov: I'd got the similar error in the past with bug in middleware | 13:43 |
| ildikov | e0ne: do you have a bug report handy? | 13:43 |
| e0ne | smcginnis, ildikov: no:(. it was an error with unicode in HTTP headers. | 13:44 |
| e0ne | ildikov: not actually the same as you've got, but it looks similar | 13:45 |
| ildikov | e0ne: oh, I see | 13:45 |
| ildikov | e0ne: I guess it's fixed by now | 13:45 |
| e0ne | ildikov: yes, it should be | 13:46 |
| ildikov | e0ne: ok, cool | 13:46 |
| e0ne | ildikov: can you add 'raise' here https://github.com/openstack/cinder/blob/master/cinder/api/middleware/fault.py#L82 and try to reproduce the issue? | 13:48 |
| *** rawanh has quit IRC | 13:48 | |
| e0ne | ildikov: it will show you the real traceback | 13:48 |
| ildikov | e0ne: yeah, will do | 13:49 |
| ildikov | e0ne: will take some time as last week it was so hot here my laptop kept restarting, so I need to setup Devstack again... :/ | 13:49 |
| ildikov | e0ne: will paste when I get a better trace | 13:49 |
| e0ne | ildikov: :( | 13:50 |
| *** links has quit IRC | 13:50 | |
| e0ne | ildikov: we'd got about +40C in the Kharkiv last week | 13:50 |
| ildikov | e0ne: it was around 40C here too | 13:50 |
| e0ne | now, +30 looks like a great temperature :) | 13:51 |
| ildikov | way better :) | 13:52 |
| *** randomhack has joined #openstack-cinder | 13:53 | |
| smcginnis | 40! Wow, that seems really got for there. | 13:56 |
| smcginnis | bbl | 13:56 |
| *** asettle_ is now known as asettle | 13:59 | |
| *** felipemonteiro has joined #openstack-cinder | 13:59 | |
| *** felipemonteiro_ has joined #openstack-cinder | 13:59 | |
| *** mriedem has joined #openstack-cinder | 14:00 | |
| *** crushil has joined #openstack-cinder | 14:02 | |
| *** cknight has joined #openstack-cinder | 14:03 | |
| *** felipemonteiro has quit IRC | 14:03 | |
| *** imacdonn has quit IRC | 14:09 | |
| *** xyang has joined #openstack-cinder | 14:11 | |
| *** xyang has quit IRC | 14:13 | |
| *** xyang has joined #openstack-cinder | 14:15 | |
| *** jistr is now known as jistr|mtg | 14:16 | |
| *** crushil has quit IRC | 14:17 | |
| *** Yogi1 has joined #openstack-cinder | 14:20 | |
| *** crushil has joined #openstack-cinder | 14:22 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Load resource filter config file in testcase https://review.openstack.org/491504 | 14:25 |
| *** diablo_rojo has joined #openstack-cinder | 14:41 | |
| ildikov | e0ne: smcginnis: ok, I realized that it only blows up if the attachment I try to get the details of does not exist | 14:44 |
| smcginnis | ildikov: At least it's easily reproducable. | 14:44 |
| ildikov | smcginnis: yeah, I just remembered that it had issues with the normal case too | 14:45 |
| ildikov | smcginnis: but probably I pasted the wrong UUID or smth like | 14:45 |
| *** gcb_ has joined #openstack-cinder | 14:45 | |
| ildikov | smcginnis: so yeah, the error is still weird and should be fixed, but hopefully it will be easier this way | 14:45 |
| smcginnis | It really looks like a case where someone is meaning to preformat a string, and instead of doing "'stuff: %s' % val" they are doing "'stuff', val" so it ends up passing in a tuple where they expect a string. | 14:46 |
| smcginnis | The question is where. | 14:46 |
| *** gcb has quit IRC | 14:47 | |
| openstackgerrit | Tzur Eliyahu proposed openstack/cinder master: Ibm_storage - fix failover_replication. https://review.openstack.org/491509 | 14:47 |
| ildikov | smcginnis: I added a 'raise' as e0ne suggested, but what I got just looks more weird IMHO... | 14:48 |
| e0ne | ildikov, smcginnis: could you please share new logs? | 14:49 |
| ildikov | https://www.irccloud.com/pastebin/4b66rFNo/ | 14:49 |
| ildikov | e0ne: ^^ | 14:49 |
| smcginnis | ex_value is passed in to __exit__ as a tuple? | 14:51 |
| e0ne | ildikov: not such helpful I hoped :( | 14:51 |
| *** psachin has quit IRC | 14:51 | |
| e0ne | ildikov: are any other errors below this traceback? | 14:51 |
| e0ne | ildikov: btw, do you have steps to reproduce? I can try on my devstack | 14:52 |
| ildikov | e0ne: nope, that's the beginning and the end too :( | 14:52 |
| *** jistr|mtg is now known as jistr | 14:52 | |
| *** gcb_ has quit IRC | 14:52 | |
| ildikov | e0ne: cinder --os-volume-api-version 3.27 attachment-show <random UUID> | 14:53 |
| e0ne | ildikov: cool, thanks | 14:54 |
| ildikov | smcginnis: even if that happens, I have no idea how... :/ :) | 14:54 |
| ildikov | e0ne: thanks for looking into it | 14:54 |
| smcginnis | ildikov, e0ne: So it looks like to me that here: https://github.com/openstack/cinder/blob/master/cinder/api/openstack/wsgi.py#L595 | 14:55 |
| smcginnis | ex_value is not the exception string. | 14:55 |
| smcginnis | I haven't found docs yet saying what that value actually is that gets passed in. | 14:56 |
| smcginnis | Other than the "exception value" | 14:56 |
| *** jdillaman has joined #openstack-cinder | 14:57 | |
| e0ne | give me 5 mins, plz | 14:57 |
| smcginnis | Odd that we check and it does appear to be an actual exception: https://github.com/openstack/cinder/blob/master/cinder/api/openstack/wsgi.py#L604 | 14:57 |
| ildikov | smcginnis: as the attachment does not exist NotFound sounds pretty reasonable | 14:59 |
| ildikov | smcginnis: so it looks like it's not formatted ok or smth like if that's the one that we have there | 15:00 |
| *** Julien-zte has joined #openstack-cinder | 15:01 | |
| smcginnis | Looks like we are in a "with ResourceExceptionHandler:" block, we don't findn the attachment so it raises, the __exit__ handler processes and we do see it is a NotFound exception, but then something with formatting that exception fails. | 15:01 |
| *** Nil_ has quit IRC | 15:01 | |
| *** markvoelker has quit IRC | 15:03 | |
| *** crushil has quit IRC | 15:04 | |
| e0ne | smcginnis: yes, we have to add VolumeAttachmentNotFound to that list | 15:04 |
| e0ne | smcginnis: but it's not enough | 15:04 |
| *** david-lyle has joined #openstack-cinder | 15:04 | |
| smcginnis | Looks like VolumeAttachmentNotFound inherits from NotFound, so that parts OK at least. | 15:06 |
| *** markvoelker has joined #openstack-cinder | 15:06 | |
| smcginnis | This potentially could be it. It's not right at least: https://github.com/openstack/cinder/blob/d0649c40b0b42129e61177e101082e5d9a0c14b9/cinder/db/sqlalchemy/api.py#L1862 | 15:07 |
| smcginnis | Should be using % and not , | 15:07 |
| smcginnis | That sets msg to a tuple where it's meant to be a string. | 15:07 |
| *** PsionTheory has joined #openstack-cinder | 15:08 | |
| smcginnis | ildikov: Are you able to quickly test that out? | 15:08 |
| e0ne | smcginnis: looks you're tight | 15:08 |
| smcginnis | I am a little surprised six.text_type isn't able to just format that as a string representation, even though it would be wrong. | 15:09 |
| e0ne | smcginnis: you're faster than me http://paste.openstack.org/show/617690/ | 15:10 |
| smcginnis | e0ne: :) | 15:11 |
| ildikov | smcginnis: VolumeAttachmentNotFound is used as many ways as many times it's used in the code... | 15:12 |
| smcginnis | e0ne: I don't actually have a code change to push up, so you're faster than me there. ;) | 15:12 |
| smcginnis | ildikov: It looks like it's probably fine in most cases unless the code tries to do six.text_type(not_found_exception) | 15:13 |
| smcginnis | ildikov: But likely could fail in multiple locations. | 15:13 |
| ildikov | e0ne: smcginnis: is there a point to add a new message when the exception has one by default? | 15:13 |
| smcginnis | That wsgi stuff just makes things confusing and hard to find. | 15:13 |
| smcginnis | ildikov: Hmm, probably not. | 15:13 |
| e0ne | smcginnis, ildikov: need to look deeper on it | 15:14 |
| smcginnis | Well, it does add the attachment ID that was attempted to retrieve, but the caller probably already knows that. | 15:14 |
| ildikov | this is the other one: raise exception.VolumeAttachmentNotFound(filter='attachment_id = ' + attachment_id) | 15:14 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception https://review.openstack.org/491518 | 15:14 |
| ildikov | this is how the exception is defined: | 15:15 |
| ildikov | class VolumeAttachmentNotFound(NotFound): | 15:15 |
| ildikov | message = _("Volume attachment could not be found with " | 15:15 |
| ildikov | "filter: %(filter)s .") | 15:15 |
| ildikov | so the second one should be ok | 15:16 |
| smcginnis | ildikov: Looks like that case should be fine. | 15:16 |
| smcginnis | It's just that this particular one replaces the message rather than providing formatting args. | 15:16 |
| ildikov | I don't think there's a point replacing the message, but well | 15:17 |
| ildikov | if it works I will care a bit less :) | 15:17 |
| e0ne | smcginnis, ildikov: both of you are solving issues faster than I can code and test:( | 15:17 |
| smcginnis | Hah, that's a good problem to have. :) | 15:18 |
| ildikov | ok, with the fixed msg it looks much better: ERROR: Unable to find attachment with id: d36c4433-0562-445d-81c6-10186d295317 (HTTP 404) (Request-ID: req-4a6306ef-0363-4587-86fa-7433147bfdf4) | 15:20 |
| ildikov | e0ne: you have a patch up that works in my env, so you rock :) | 15:20 |
| ildikov | e0ne: with smcginnis we're just talking about fixing stuff :) | 15:20 |
| e0ne | I've added smcginnis as co-author | 15:21 |
| smcginnis | e0ne: Hah, not necessary, but thank you. :) | 15:21 |
| e0ne | I hope, no tests will fail | 15:21 |
| e0ne | smcginnis: you save me few minutes pointing on a needed like :) | 15:22 |
| smcginnis | ha | 15:22 |
| *** Swanson has joined #openstack-cinder | 15:22 | |
| *** ntpttr_laptop has joined #openstack-cinder | 15:23 | |
| ildikov | real team work :) | 15:24 |
| ildikov | smcginnis: I saw the client fix got merged for the uWSGI issue | 15:25 |
| *** bswartz has quit IRC | 15:25 | |
| ildikov | smcginnis: when could we get a client release? | 15:25 |
| smcginnis | ildikov: Another good example of team work. | 15:25 |
| e0ne | ildikov, smcginnis: do we need to backport for pike and do new release now? | 15:25 |
| ildikov | smcginnis: indeed :) | 15:25 |
| smcginnis | ildikov: We'll have to wait until after the freeze at this point, unfortunately. | 15:25 |
| ildikov | smcginnis: when is that? next week? | 15:26 |
| smcginnis | e0ne: Yeah, if that's not in stable/pike, we should get that moved. | 15:26 |
| smcginnis | ildikov: At some point after RC is cut. It might take an extra week or so because of the trailing release projects. | 15:26 |
| e0ne | smcginnis: done: https://review.openstack.org/#/c/491519/ | 15:27 |
| ildikov | smcginnis: oh, ok, I forgot about that :( | 15:27 |
| smcginnis | ildikov: But then you can bug the release manager PTL to get that going. ;) | 15:27 |
| *** bswartz has joined #openstack-cinder | 15:27 | |
| smcginnis | e0ne: Awesome. | 15:27 |
| ildikov | smcginnis: lol :) | 15:27 |
| *** waboring has joined #openstack-cinder | 15:28 | |
| *** ntpttr_laptop has quit IRC | 15:29 | |
| e0ne | smcginnis, ildikov: need to head home. I'll fix exception message later tonight it the patch won't be merged/updated | 15:33 |
| ildikov | e0ne: wasn't a blocking comment | 15:33 |
| smcginnis | Thanks e0ne, have a good one. | 15:33 |
| e0ne | thanks | 15:33 |
| ildikov | e0ne: thanks for the quick fix :) | 15:33 |
| ildikov | e0ne: and have a good evening! | 15:34 |
| *** Julien-zte has quit IRC | 15:35 | |
| *** Julien-zte has joined #openstack-cinder | 15:35 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-cinderclient master: Explicitly set 'builder' option https://review.openstack.org/457954 | 15:37 |
| openstackgerrit | Stephen Finucane proposed openstack/python-cinderclient master: doc: Remove cruft from conf.py https://review.openstack.org/457955 | 15:37 |
| openstackgerrit | Stephen Finucane proposed openstack/python-cinderclient master: Use Sphinx 1.5 warning-is-error https://review.openstack.org/457953 | 15:37 |
| *** crose has joined #openstack-cinder | 15:41 | |
| smcginnis | Spec revert for something not done in Pike: https://review.openstack.org/486921 | 15:47 |
| smcginnis | Might be good to get that out of there so no one sees it and wonders why they can't find the code. | 15:47 |
| *** lpetrut has quit IRC | 15:49 | |
| *** markvoelker has quit IRC | 15:53 | |
| *** aarefiev is now known as aarefiev_afk | 15:54 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-brick master: doc: Remove cruft from conf.py https://review.openstack.org/490878 | 15:59 |
| openstackgerrit | Stephen Finucane proposed openstack/os-brick master: doc: Restructure docs for doc-migration https://review.openstack.org/490901 | 15:59 |
| *** Apoorva has joined #openstack-cinder | 16:00 | |
| *** Apoorva has quit IRC | 16:01 | |
| openstackgerrit | Merged openstack/cinder master: vzstorage: use remotefs.VZStorageRemoteFSClient https://review.openstack.org/462563 | 16:02 |
| *** waboring has quit IRC | 16:06 | |
| *** ircuser-1 has joined #openstack-cinder | 16:06 | |
| *** aarefiev_afk has quit IRC | 16:06 | |
| *** itlinux has joined #openstack-cinder | 16:08 | |
| *** ntpttr_laptop has joined #openstack-cinder | 16:09 | |
| *** ntpttr_laptop has quit IRC | 16:09 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Clean up repl v1 volume creation from replication https://review.openstack.org/482748 | 16:09 |
| *** baojg has quit IRC | 16:10 | |
| *** Apoorva has joined #openstack-cinder | 16:13 | |
| *** yangyapeng has quit IRC | 16:13 | |
| *** yangyapeng has joined #openstack-cinder | 16:13 | |
| *** e0ne has quit IRC | 16:17 | |
| *** yangyapeng has quit IRC | 16:20 | |
| *** hemna_ has joined #openstack-cinder | 16:29 | |
| *** dustins has quit IRC | 16:33 | |
| *** ChubYann has joined #openstack-cinder | 16:34 | |
| *** markstur has joined #openstack-cinder | 16:35 | |
| *** openstackstatus has quit IRC | 16:42 | |
| *** openstackstatus has joined #openstack-cinder | 16:43 | |
| *** ChanServ sets mode: +v openstackstatus | 16:43 | |
| *** Julien-zte has quit IRC | 16:43 | |
| *** Julien-zte has joined #openstack-cinder | 16:43 | |
| *** Julien-zte has quit IRC | 16:44 | |
| *** abhitechie has joined #openstack-cinder | 16:44 | |
| *** hoonetorg has quit IRC | 16:45 | |
| *** crose has quit IRC | 16:51 | |
| *** hoonetorg has joined #openstack-cinder | 16:58 | |
| *** lhx_ has quit IRC | 17:00 | |
| *** crushil has joined #openstack-cinder | 17:02 | |
| *** hemna_ has quit IRC | 17:05 | |
| *** baojg has joined #openstack-cinder | 17:11 | |
| *** tesseract has quit IRC | 17:12 | |
| *** pcaruana has quit IRC | 17:15 | |
| *** baojg has quit IRC | 17:16 | |
| *** rcernin has quit IRC | 17:22 | |
| *** mriedem has quit IRC | 17:22 | |
| *** e0ne has joined #openstack-cinder | 17:24 | |
| *** xyang has quit IRC | 17:28 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception https://review.openstack.org/491518 | 17:29 |
| e0ne | smcginnis, ildikov: I updated patch ^^ to re-use default exception message | 17:34 |
| *** randomhack has quit IRC | 17:35 | |
| *** randomhack has joined #openstack-cinder | 17:38 | |
| *** diablo_rojo has quit IRC | 17:40 | |
| *** abhitechie has quit IRC | 17:40 | |
| *** abhinavtechie has joined #openstack-cinder | 17:40 | |
| *** dustins has joined #openstack-cinder | 17:43 | |
| *** markvoelker has joined #openstack-cinder | 17:54 | |
| *** gouthamr has joined #openstack-cinder | 17:58 | |
| *** itlinux has quit IRC | 17:59 | |
| *** randomhack has quit IRC | 18:04 | |
| *** itlinux has joined #openstack-cinder | 18:05 | |
| *** abhinavtechie has quit IRC | 18:11 | |
| *** baojg has joined #openstack-cinder | 18:12 | |
| *** lpetrut has joined #openstack-cinder | 18:12 | |
| *** yangyapeng has joined #openstack-cinder | 18:15 | |
| *** baojg has quit IRC | 18:17 | |
| *** yangyapeng has quit IRC | 18:20 | |
| *** itlinux has quit IRC | 18:21 | |
| *** itlinux has joined #openstack-cinder | 18:22 | |
| *** markvoelker has quit IRC | 18:28 | |
| *** randomhack has joined #openstack-cinder | 18:30 | |
| *** xyang has joined #openstack-cinder | 18:30 | |
| *** chhavi has quit IRC | 18:31 | |
| *** xyang has quit IRC | 18:32 | |
| *** edmondsw_ has quit IRC | 18:32 | |
| *** edmondsw has joined #openstack-cinder | 18:32 | |
| *** kfarr has joined #openstack-cinder | 18:34 | |
| *** edmondsw_ has joined #openstack-cinder | 18:35 | |
| *** edmondsw has quit IRC | 18:37 | |
| *** edmondsw_ has quit IRC | 18:39 | |
| *** lpetrut has quit IRC | 18:41 | |
| *** xyang has joined #openstack-cinder | 18:41 | |
| *** xyang has quit IRC | 18:44 | |
| *** mriedem has joined #openstack-cinder | 18:45 | |
| smcginnis | e0ne: Cool, that works. | 18:49 |
| *** nicolasbock has quit IRC | 18:57 | |
| *** diablo_rojo has joined #openstack-cinder | 19:02 | |
| *** armax has joined #openstack-cinder | 19:08 | |
| ildikov | e0ne: tested, works well, thank you! | 19:10 |
| e0ne | ildikov: thanks for testing and review | 19:10 |
| ildikov | e0ne: np :) | 19:13 |
| *** baojg has joined #openstack-cinder | 19:13 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: Make Cinder's Landing Page look like Keystone's https://review.openstack.org/491578 | 19:16 |
| jungleboyj | ^^ smcginnis hemna e0ne scottda New landing page for Cinder's documentation. Appreciate input on it. | 19:17 |
| *** baojg has quit IRC | 19:18 | |
| jungleboyj | lbragstad: Thanks for the inspiration/example. :-) | 19:18 |
| e0ne | jungleboyj: do you have a link to the keystone's landing page in touch? | 19:19 |
| lbragstad | e0ne: https://docs.openstack.org/keystone/latest/ | 19:19 |
| jungleboyj | lbragstad: :-) Thank you. | 19:19 |
| lbragstad | jungleboyj: anytime! | 19:19 |
| lbragstad | jungleboyj: fwiw - it's still wip | 19:19 |
| e0ne | jungleboyj: thank you, Jay | 19:20 |
| jungleboyj | So is our's but yours is at least readable. | 19:20 |
| lbragstad | not sure if this is specific to cinder or not - but we duplicated a lot of what was in openstack-manuals in our own project docs =/ | 19:20 |
| jungleboyj | lbragstad: Well, I have pulled over what was in openstack-docs. Still need to make sure I have it all. | 19:20 |
| lbragstad | which is why we have an Operator Documentation and an Administrator Guide | 19:20 |
| lbragstad | we're working on consolidating those into a single document tree for simplicity sake, since a lot of them over lap | 19:21 |
| lbragstad | just for future reference, a lot of the Operator Documentation is going to go away and get migrated | 19:21 |
| lbragstad | which seems to be a never ending battle of documentation shuffling | 19:22 |
| jungleboyj | Ok. I have a feeling my work on Cinder's docs will be neverending. | 19:22 |
| lbragstad | ++ | 19:22 |
| e0ne | jungleboyj: It means, you always will have a work | 19:23 |
| jungleboyj | e0ne: :-) | 19:24 |
| *** xyang1 has joined #openstack-cinder | 19:33 | |
| *** randomhack has quit IRC | 19:34 | |
| *** edmondsw has joined #openstack-cinder | 19:38 | |
| *** randomhack has joined #openstack-cinder | 19:41 | |
| *** e0ne has quit IRC | 19:43 | |
| *** crose has joined #openstack-cinder | 19:43 | |
| *** jessegler has joined #openstack-cinder | 19:48 | |
| mriedem | jungleboyj: did you get this migrated to cinder docs? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-manage-volumes.rst | 19:49 |
| mriedem | i noticed it showed up in the nova docs, which is wrong since it's mostly cinder stuff | 19:49 |
| jgriffith | What's the deal os-brick? http://paste.openstack.org/show/617709/ | 20:03 |
| jgriffith | meanwhile: jgriffith@os-1:/opt/stack/logs$ ls -la /dev/sda | 20:03 |
| jgriffith | brw-rw---- 1 root root 8, 0 Aug 7 20:00 /dev/sda | 20:03 |
| jgriffith | however it does seem that ownership of the device changes when libvirt has it and then gives it up | 20:03 |
| smcginnis | jgriffith: I was just going to say - permissions issue? | 20:03 |
| jgriffith | smcginnis but shouldn't brick be able to figure that out? | 20:04 |
| *** wanghao has quit IRC | 20:04 | |
| jgriffith | Hmm... or maybe that's my problem here? Maybe I'm trying to terminate BEFORE detaching from libvirt? | 20:04 |
| smcginnis | jgriffith: I would expect a different error actually. | 20:04 |
| *** liverpooler has quit IRC | 20:04 | |
| jgriffith | smcginnis well there's no shortage of messages in the logs :) | 20:05 |
| jgriffith | theyr'e just all pretty much useless though :( | 20:05 |
| Swanson | https://twitter.com/philyuck/status/894601208970620928 | 20:05 |
| Swanson | Not the window I thought. | 20:06 |
| smcginnis | You'd think that would give a "Permission Denied" and not a "not found" message. Timing issue by any chance? | 20:06 |
| jgriffith | Swanson although it's somewhat fitting | 20:06 |
| smcginnis | Swanson: That works for you anyway. :) | 20:06 |
| jgriffith | smcginnis yeah, I dunno; the notfound makes sense because brick it turns out isn't very sophisticated and just does an ls /dev | 20:07 |
| jgriffith | so if you don't have perms you just don't see what you're looking for | 20:07 |
| smcginnis | jgriffith: Oh, true. | 20:07 |
| jgriffith | not sure, but I think instead of doing an ls /dev and then "looking for say sda" it should probably explicitly do an inspect on sda.. but what do I know | 20:07 |
| jgriffith | parsing out ls results is kinda un-robust | 20:08 |
| jgriffith | guess it's a clue at least ;) | 20:08 |
| smcginnis | jgriffith: Would be great if there was a system interface available in python to be able to do this the right way. | 20:09 |
| jgriffith | smcginnis :) | 20:09 |
| jgriffith | Wanna join me on a new OSS project and write one? | 20:09 |
| jgriffith | surely our employers will finance us for such a thing!! | 20:09 |
| jgriffith | :) | 20:09 |
| jgriffith | Or I could down-size, move into a cabin in the woods and just do whatever I want | 20:10 |
| jgriffith | wait... internet, computers etc | 20:10 |
| jgriffith | damn | 20:10 |
| jgriffith | the good thing is that I seem to be making at least *some* kinda progress on whatever it goofy in the new attach code with shelved instances etc | 20:11 |
| smcginnis | jgriffith: That's good! | 20:11 |
| jgriffith | turns out nova really cares a lot about sequence | 20:11 |
| smcginnis | Trick is to find a cabin in the woods with a good internet uplink. :D | 20:11 |
| *** dustins has quit IRC | 20:12 | |
| jgriffith | I finally reproduced this mess manually with combinations of attach->shelve->unshelve->delete-instance | 20:12 |
| jgriffith | boom | 20:12 |
| jgriffith | smcginnis considering I couldn't even find a *house* in the country with that I'm not optimistic | 20:12 |
| jgriffith | and now that "town" has grown out to my quiet little corner it really just seems unfair | 20:13 |
| smcginnis | jgriffith: Urban sprawl in CO? I thought they had a lot of laws against that. | 20:13 |
| jgriffith | smcginnis pissh!!! | 20:13 |
| jgriffith | Laws are only for suckers that don't have money | 20:14 |
| jgriffith | you have enough money you can do whatever you want | 20:14 |
| jgriffith | as is evident by the giant f'ing golf course they built down the road | 20:14 |
| jgriffith | and all the houses to go with it | 20:14 |
| *** baojg has joined #openstack-cinder | 20:14 | |
| jgriffith | despite that fact that yes, there were laws/rules about new development and water etc | 20:14 |
| jgriffith | turns out you have enough money and there's always an exception process | 20:15 |
| ildikov | money talks, that's the only thing that hasn't changed a bit over the past few thousand years... | 20:15 |
| jgriffith | ildikov indeed... even when the form of currency changes, the rules change with it | 20:15 |
| * smcginnis pictures jgriffith in some fancy golf outfit | 20:16 | |
| jgriffith | although I guess it's just a math problem at this point... too many people in the world | 20:16 |
| jgriffith | smcginnis LOL | 20:16 |
| jgriffith | smcginnis ummmm... yeah, that one is going to have to remain pure imaginary | 20:16 |
| *** yangyapeng has joined #openstack-cinder | 20:16 | |
| *** gouthamr has quit IRC | 20:17 | |
| ildikov | jgriffith: rules don't really change, I could mention Animal Farm from Orwell as a good attempt to visualize it a bit | 20:17 |
| jgriffith | ildikov :) | 20:18 |
| ildikov | jgriffith: so, what's wrong with golf? :) | 20:18 |
| smcginnis | Boring "sport" :) | 20:19 |
| *** baojg has quit IRC | 20:19 | |
| ildikov | smcginnis: oh I'm not into it either, just got a challenge accepted feeling re the "pure imaginary" statement jgriffith has just made, so thought to figure out some details :) | 20:20 |
| jgriffith | ildikov nothing I suppose, just not really my thing. And given I live in a place where the cities and state are gobbling up water rights and taking them away from farms I have issues with golf courses | 20:20 |
| jgriffith | they use a lot of water, and I like food | 20:21 |
| *** yangyapeng has quit IRC | 20:21 | |
| jgriffith | although if they want to let me use the golf course as pasture for cattle and horses then I'm in :) | 20:21 |
| smcginnis | That would be awesome! | 20:22 |
| jgriffith | hehe! | 20:22 |
| ildikov | haha, that would be interesting :) | 20:22 |
| jgriffith | that would be really | 20:22 |
| smcginnis | Extra "hazards" | 20:22 |
| jgriffith | ha! | 20:22 |
| ildikov | lol :) | 20:22 |
| jgriffith | this could actually catch on!! | 20:23 |
| jgriffith | volume version 3.27 is not in supported versions: 1, 2, 3 | 20:23 |
| ildikov | jgriffith: as for the new attach stuff, is it purely shelve/unshelve that messes things up? | 20:23 |
| ildikov | jgriffith: what did you do again? :) | 20:24 |
| jgriffith | ildikov I can't guarantee that, would be interesting to disable those tests and see | 20:24 |
| jgriffith | I tried to rerun stack.sh with OFFLINE=True | 20:24 |
| *** gouthamr has joined #openstack-cinder | 20:24 | |
| jgriffith | turns out for some reason that's not a valid thing perhaps? | 20:24 |
| ildikov | what does that do? | 20:24 |
| jgriffith | ildikov OFFLINE? | 20:26 |
| ildikov | yep | 20:26 |
| jgriffith | the idea is you just restack as if there's no internet connection | 20:26 |
| smcginnis | Rerunning stack.sh? Yeah, that's a 50/50 chance of working. :) | 20:26 |
| jgriffith | just cleans everything and rebuild using what's in place | 20:26 |
| jgriffith | used to be a life-saver | 20:26 |
| jgriffith | especially great on airplanes | 20:26 |
| ildikov | there's no life without internet connection in this world :) | 20:26 |
| jgriffith | smcginnis yeah, looks like it pretty much sucks now | 20:26 |
| jgriffith | unfortunate | 20:27 |
| jgriffith | ildikov the other thing used to be that you could restack in about 5 minutes | 20:27 |
| ildikov | oh, that indeed sounds nice | 20:27 |
| jgriffith | instead of 20+ or whatever | 20:27 |
| jgriffith | I've gotten spoiled running cinder stuff in containers | 20:28 |
| ildikov | although for me it's not that long, when I don't re-clone all the things, it's somewhere between | 20:28 |
| jgriffith | devstack is killing me | 20:28 |
| smcginnis | ildikov: Even when your laptop is melting? | 20:28 |
| ildikov | jgriffith: I'm not picky on what env you're fixing the attach stuff in :) | 20:29 |
| ildikov | smcginnis: not that hot here anymore, however one tempest run can get it close to melt still... | 20:29 |
| ildikov | jgriffith: if I figure out how to disable tests, would that be all the shelve related ones to exclude and see whether we still have the problem present? | 20:32 |
| ildikov | jgriffith: it might tomorrow when I get there to run the whole suite again, but happy to experiment with it | 20:33 |
| jgriffith | ildikov yeah, there's only a few tests there | 20:34 |
| jgriffith | if I don't manage to get to it today I'll shoot you a note on how to do it | 20:34 |
| jgriffith | it's pretty easy | 20:34 |
| ildikov | jgriffith: sounds good, thanks | 20:35 |
| jgriffith | if nothing else you can do it the brute-force way and just delete the tests from the file :) | 20:35 |
| jgriffith | otherwise you can add skip directives in tempest.conf | 20:35 |
| ildikov | I wanted to avoid that if possible :) | 20:35 |
| jgriffith | ildikov :) | 20:35 |
| ildikov | deleting I mean :) | 20:35 |
| jgriffith | you can also put a skip decorator on them | 20:35 |
| jgriffith | yeah, I knew what you meant | 20:35 |
| ildikov | ok, I will look it up, when I start to hate my laptop again and want to torture it :) | 20:36 |
| jgriffith | oh... I know why I was getting that version thing... in my env I do export OS_VOLUME_API_VERSION=3.27 | 20:36 |
| *** gouthamr has quit IRC | 20:36 | |
| ildikov | always the small things :) | 20:37 |
| jgriffith | we should probably make things smart enough to strip off the micro-version | 20:37 |
| jgriffith | rather than erroring | 20:37 |
| ildikov | one thing at a time :) | 20:38 |
| *** randomha1k has joined #openstack-cinder | 20:40 | |
| *** randomhack has quit IRC | 20:43 | |
| *** randomha1k has quit IRC | 20:44 | |
| *** cknight has quit IRC | 20:51 | |
| *** mriedem has quit IRC | 20:56 | |
| *** Yogi1 has quit IRC | 21:07 | |
| *** crushil has quit IRC | 21:09 | |
| *** chaynikasaikia has joined #openstack-cinder | 21:13 | |
| *** cknight has joined #openstack-cinder | 21:15 | |
| *** baojg has joined #openstack-cinder | 21:15 | |
| *** lucasxu has quit IRC | 21:15 | |
| *** csaikia has quit IRC | 21:15 | |
| *** kfarr has quit IRC | 21:17 | |
| *** chaynikasaikia has quit IRC | 21:18 | |
| *** catintheroof has quit IRC | 21:20 | |
| *** cknight1 has joined #openstack-cinder | 21:20 | |
| *** cknight has quit IRC | 21:21 | |
| *** baojg has quit IRC | 21:22 | |
| *** markvoelker has joined #openstack-cinder | 21:26 | |
| *** cknight1 has quit IRC | 21:43 | |
| *** edmondsw has quit IRC | 21:45 | |
| *** edmondsw has joined #openstack-cinder | 21:45 | |
| *** crose has quit IRC | 21:47 | |
| openstackgerrit | Merged openstack/cinder master: Make Cinder's Landing Page look like Keystone's https://review.openstack.org/491578 | 21:48 |
| *** edmondsw has quit IRC | 21:50 | |
| *** csaikia has joined #openstack-cinder | 21:51 | |
| *** markvoelker has quit IRC | 22:00 | |
| *** xyang1 has quit IRC | 22:00 | |
| *** Apoorva_ has joined #openstack-cinder | 22:15 | |
| *** yangyapeng has joined #openstack-cinder | 22:17 | |
| *** Apoorva has quit IRC | 22:18 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix missing references in documentation https://review.openstack.org/491616 | 22:21 |
| *** yangyapeng has quit IRC | 22:21 | |
| *** Apoorva_ has quit IRC | 22:24 | |
| *** Apoorva has joined #openstack-cinder | 22:25 | |
| *** gouthamr has joined #openstack-cinder | 22:28 | |
| *** markvoelker has joined #openstack-cinder | 22:32 | |
| *** itlinux has quit IRC | 22:51 | |
| *** markvoelker has quit IRC | 22:54 | |
| *** edmondsw has joined #openstack-cinder | 22:54 | |
| *** markvoelker has joined #openstack-cinder | 22:54 | |
| *** PsionTheory has quit IRC | 22:58 | |
| *** edmondsw has quit IRC | 22:58 | |
| *** markvoelker has quit IRC | 22:59 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix user messages documentation https://review.openstack.org/491622 | 23:00 |
| *** ntpttr_ is now known as ntpttr | 23:05 | |
| *** chhavi has joined #openstack-cinder | 23:08 | |
| *** baojg has joined #openstack-cinder | 23:19 | |
| *** sdague has quit IRC | 23:21 | |
| *** baojg has quit IRC | 23:23 | |
| *** jessegler has quit IRC | 23:27 | |
| *** catintheroof has joined #openstack-cinder | 23:28 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org will be offline momentarily at 00:00 utc for a quick reconfiguration-related restart | 23:32 | |
| *** itlinux has joined #openstack-cinder | 23:34 | |
| *** itlinux has quit IRC | 23:37 | |
| *** stakeda has joined #openstack-cinder | 23:38 | |
| *** felipemonteiro_ has quit IRC | 23:39 | |
| *** diablo_rojo has quit IRC | 23:40 | |
| *** diablo_rojo has joined #openstack-cinder | 23:40 | |
| *** catintheroof has quit IRC | 23:50 | |
| *** crushil has joined #openstack-cinder | 23:57 | |
| openstackgerrit | Merged openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception https://review.openstack.org/491518 | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!