| *** salv-orlando has joined #openstack-cinder | 00:06 | |
| *** erlon has quit IRC | 00:06 | |
| *** ducttape_ has quit IRC | 00:07 | |
| *** ducttape_ has joined #openstack-cinder | 00:08 | |
| *** ducttape_ has quit IRC | 00:09 | |
| *** salv-orlando has quit IRC | 00:13 | |
| *** gouthamr has quit IRC | 00:15 | |
| *** yangyapeng has quit IRC | 00:39 | |
| *** karthikp_ has joined #openstack-cinder | 00:39 | |
| *** diogogmt has quit IRC | 00:46 | |
| *** karthikp_ has quit IRC | 00:47 | |
| *** karthikp_ has joined #openstack-cinder | 00:51 | |
| *** wxy__ has quit IRC | 00:57 | |
| *** wxy__ has joined #openstack-cinder | 00:57 | |
| *** cheneydc has joined #openstack-cinder | 00:58 | |
| *** karthikp_ has quit IRC | 01:02 | |
| *** iceyao has joined #openstack-cinder | 01:02 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix of finding out available cinder volume services https://review.openstack.org/308148 | 01:02 |
|---|---|---|
| *** jamielennox is now known as jamielennox|away | 01:09 | |
| *** karthikp_ has joined #openstack-cinder | 01:09 | |
| *** salv-orlando has joined #openstack-cinder | 01:10 | |
| *** wxy has joined #openstack-cinder | 01:16 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix of finding out available cinder volume services https://review.openstack.org/308148 | 01:17 |
| *** karthikp_ has quit IRC | 01:17 | |
| *** karthikp_ has joined #openstack-cinder | 01:18 | |
| *** cheneydc has quit IRC | 01:20 | |
| *** salv-orlando has quit IRC | 01:28 | |
| *** yangyapeng has joined #openstack-cinder | 01:32 | |
| *** karthikp_ has quit IRC | 01:33 | |
| *** ducttape_ has joined #openstack-cinder | 01:40 | |
| *** Lee1092 has joined #openstack-cinder | 01:41 | |
| *** karthikp_ has joined #openstack-cinder | 01:42 | |
| *** enriquetaso has quit IRC | 01:42 | |
| *** jamielennox|away is now known as jamielennox | 01:43 | |
| *** EinstCrazy has joined #openstack-cinder | 01:44 | |
| *** ducttape_ has quit IRC | 01:45 | |
| *** karthikp_ has quit IRC | 01:45 | |
| *** karthikp_ has joined #openstack-cinder | 01:46 | |
| *** jamielennox is now known as jamielennox|away | 01:53 | |
| *** salv-orlando has joined #openstack-cinder | 01:56 | |
| *** yangyape_ has joined #openstack-cinder | 01:58 | |
| *** yangyap__ has joined #openstack-cinder | 02:00 | |
| *** EinstCra_ has joined #openstack-cinder | 02:01 | |
| *** yangyapeng has quit IRC | 02:01 | |
| *** yangyape_ has quit IRC | 02:02 | |
| *** EinstCrazy has quit IRC | 02:02 | |
| *** karthik__ has joined #openstack-cinder | 02:02 | |
| *** karthik__ has quit IRC | 02:03 | |
| *** karthikp_ has quit IRC | 02:03 | |
| *** iceyao_ has joined #openstack-cinder | 02:03 | |
| *** karthikp_ has joined #openstack-cinder | 02:03 | |
| *** iceyao__ has joined #openstack-cinder | 02:05 | |
| *** iceyao has quit IRC | 02:05 | |
| *** EinstCrazy has joined #openstack-cinder | 02:06 | |
| *** EinstCra_ has quit IRC | 02:07 | |
| *** diogogmt has joined #openstack-cinder | 02:07 | |
| *** iceyao_ has quit IRC | 02:08 | |
| *** karthikp_ has quit IRC | 02:11 | |
| *** iceyao has joined #openstack-cinder | 02:12 | |
| *** salv-orlando has quit IRC | 02:13 | |
| *** karthikp_ has joined #openstack-cinder | 02:15 | |
| *** iceyao__ has quit IRC | 02:16 | |
| openstackgerrit | Weiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate https://review.openstack.org/314559 | 02:21 |
| *** GB21 has joined #openstack-cinder | 02:25 | |
| openstackgerrit | zhufl proposed openstack/cinder: Remove unused LOG https://review.openstack.org/337406 | 02:25 |
| *** karthikp_ has quit IRC | 02:29 | |
| *** karthikp_ has joined #openstack-cinder | 02:30 | |
| *** davechen has joined #openstack-cinder | 02:35 | |
| *** salv-orlando has joined #openstack-cinder | 02:40 | |
| *** ducttape_ has joined #openstack-cinder | 02:42 | |
| *** liuke_ has joined #openstack-cinder | 02:42 | |
| *** GB21 has quit IRC | 02:43 | |
| *** jamielennox|away is now known as jamielennox | 02:43 | |
| *** liuke has quit IRC | 02:44 | |
| *** liuke_ is now known as liuke | 02:44 | |
| *** karthik__ has joined #openstack-cinder | 02:45 | |
| *** wxy__ has quit IRC | 02:46 | |
| *** wxy__ has joined #openstack-cinder | 02:46 | |
| *** karthikp_ has quit IRC | 02:48 | |
| *** ducttape_ has quit IRC | 02:48 | |
| *** salv-orlando has quit IRC | 02:58 | |
| *** karthikp_ has joined #openstack-cinder | 03:00 | |
| *** karthik__ has quit IRC | 03:03 | |
| *** rcernin has joined #openstack-cinder | 03:09 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Do not conjecture volume-id from iscsi_name https://review.openstack.org/337081 | 03:12 |
| *** jichen has joined #openstack-cinder | 03:17 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Do not conjecture volume-id from iscsi_name https://review.openstack.org/337081 | 03:22 |
| *** salv-orlando has joined #openstack-cinder | 03:24 | |
| *** ManishD has joined #openstack-cinder | 03:26 | |
| openstackgerrit | Arsen Chen proposed openstack/cinder: Add Synology DSM storage driver https://review.openstack.org/335870 | 03:39 |
| *** salv-orlando has quit IRC | 03:40 | |
| *** ducttape_ has joined #openstack-cinder | 03:44 | |
| *** ducttape_ has quit IRC | 03:48 | |
| *** davechen has quit IRC | 03:49 | |
| *** M00nr41n has joined #openstack-cinder | 03:56 | |
| *** salv-orlando has joined #openstack-cinder | 04:06 | |
| *** GB21 has joined #openstack-cinder | 04:08 | |
| openstackgerrit | zengjia proposed openstack/cinder: Add MacroSAN cinder driver https://review.openstack.org/337415 | 04:12 |
| *** yangyap__ has quit IRC | 04:12 | |
| *** EinstCrazy has quit IRC | 04:12 | |
| *** salv-orlando has quit IRC | 04:18 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo https://review.openstack.org/319040 | 04:20 |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Add filters support to get_pools https://review.openstack.org/321492 | 04:24 |
| *** M00nr41n has quit IRC | 04:28 | |
| *** adisky has joined #openstack-cinder | 04:32 | |
| *** shyama has joined #openstack-cinder | 04:34 | |
| *** GB21 has quit IRC | 04:40 | |
| *** sgotliv__ has joined #openstack-cinder | 04:40 | |
| *** savihou has quit IRC | 04:43 | |
| *** ducttape_ has joined #openstack-cinder | 04:44 | |
| *** salv-orlando has joined #openstack-cinder | 04:45 | |
| *** ducttape_ has quit IRC | 04:49 | |
| *** laughterwym has joined #openstack-cinder | 04:53 | |
| *** GB21 has joined #openstack-cinder | 04:58 | |
| *** sgotliv__ has quit IRC | 04:59 | |
| *** takedakn has joined #openstack-cinder | 04:59 | |
| *** takedakn has quit IRC | 05:02 | |
| *** salv-orlando has quit IRC | 05:05 | |
| *** sheel has joined #openstack-cinder | 05:07 | |
| *** salv-orlando has joined #openstack-cinder | 05:10 | |
| *** wxy__ has quit IRC | 05:10 | |
| *** wxy__ has joined #openstack-cinder | 05:11 | |
| *** laughterwym has quit IRC | 05:14 | |
| *** laughterwym has joined #openstack-cinder | 05:15 | |
| *** laughter_ has joined #openstack-cinder | 05:16 | |
| *** laughterwym has quit IRC | 05:19 | |
| *** shausy has joined #openstack-cinder | 05:19 | |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337423 | 05:19 |
| *** bhavik has joined #openstack-cinder | 05:21 | |
| *** ociuhandu has joined #openstack-cinder | 05:24 | |
| *** M00nr41n has joined #openstack-cinder | 05:24 | |
| *** Poornima has joined #openstack-cinder | 05:27 | |
| *** ociuhandu has quit IRC | 05:32 | |
| *** e0ne has joined #openstack-cinder | 05:32 | |
| *** laughter_ has quit IRC | 05:33 | |
| *** laughterwym has joined #openstack-cinder | 05:33 | |
| *** savihou has joined #openstack-cinder | 05:34 | |
| *** lpetrut has joined #openstack-cinder | 05:34 | |
| openstackgerrit | Merged openstack/cinder: Add __ne__ built-in function https://review.openstack.org/337042 | 05:37 |
| *** e0ne has quit IRC | 05:37 | |
| *** savihou has quit IRC | 05:40 | |
| *** savihou has joined #openstack-cinder | 05:41 | |
| *** ducttape_ has joined #openstack-cinder | 05:45 | |
| *** ducttape_ has quit IRC | 05:50 | |
| *** ChubYann has quit IRC | 05:50 | |
| *** davechen has joined #openstack-cinder | 05:54 | |
| *** maestropandy has joined #openstack-cinder | 05:54 | |
| *** ManishD has quit IRC | 05:58 | |
| *** rcernin has quit IRC | 06:01 | |
| *** yangyapeng has joined #openstack-cinder | 06:03 | |
| *** yangyapeng has quit IRC | 06:03 | |
| *** yangyapeng has joined #openstack-cinder | 06:03 | |
| *** EinstCrazy has joined #openstack-cinder | 06:04 | |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 06:07 |
| *** caoshufeng has joined #openstack-cinder | 06:07 | |
| *** sgotliv__ has joined #openstack-cinder | 06:09 | |
| *** smile-luobin_ has quit IRC | 06:12 | |
| *** GB21 has quit IRC | 06:15 | |
| *** pgadiya has joined #openstack-cinder | 06:20 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 06:23 |
| *** maestropandy has quit IRC | 06:24 | |
| *** arecknag has joined #openstack-cinder | 06:25 | |
| *** hogepodge has quit IRC | 06:25 | |
| *** hogepodge has joined #openstack-cinder | 06:27 | |
| *** maestropandy has joined #openstack-cinder | 06:28 | |
| *** pbandark has joined #openstack-cinder | 06:29 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Remove white space between print and () https://review.openstack.org/337453 | 06:31 |
| *** rcernin has joined #openstack-cinder | 06:33 | |
| *** salv-orlando has quit IRC | 06:34 | |
| *** salv-orlando has joined #openstack-cinder | 06:36 | |
| *** salv-orlando has quit IRC | 06:41 | |
| *** lpetrut has quit IRC | 06:43 | |
| *** nkrinner_afk is now known as nkrinner | 06:44 | |
| *** lpetrut has joined #openstack-cinder | 06:44 | |
| *** ducttape_ has joined #openstack-cinder | 06:46 | |
| openstackgerrit | Arsen Chen proposed openstack/cinder: Add Synology DSM storage driver https://review.openstack.org/335870 | 06:47 |
| *** ducttape_ has quit IRC | 06:50 | |
| *** lpetrut has quit IRC | 06:50 | |
| *** akapil has joined #openstack-cinder | 06:56 | |
| *** lpetrut has joined #openstack-cinder | 06:56 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: iSCSI Target: remove confusing comment/log https://review.openstack.org/337476 | 07:04 |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: iSCSI Target: remove confusing comment/log https://review.openstack.org/337476 | 07:07 |
| *** salv-orlando has joined #openstack-cinder | 07:07 | |
| *** tesseract- has joined #openstack-cinder | 07:09 | |
| *** liverpooler has joined #openstack-cinder | 07:15 | |
| *** lpetrut has quit IRC | 07:16 | |
| *** zhonghua-lee has joined #openstack-cinder | 07:16 | |
| *** iceyao_ has joined #openstack-cinder | 07:17 | |
| *** iceyao has quit IRC | 07:17 | |
| *** rhagarty__ has quit IRC | 07:17 | |
| *** rhagarty__ has joined #openstack-cinder | 07:18 | |
| *** lennyb has quit IRC | 07:19 | |
| *** salv-orlando has quit IRC | 07:19 | |
| *** lennyb has joined #openstack-cinder | 07:20 | |
| *** karthikp_ has quit IRC | 07:26 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 07:30 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Log traceback for initialize connection failure https://review.openstack.org/337500 | 07:31 |
| openstackgerrit | ningwei proposed openstack/cinder: Add cinder backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 07:31 |
| *** akapil has quit IRC | 07:37 | |
| *** geguileo has joined #openstack-cinder | 07:38 | |
| *** akapil has joined #openstack-cinder | 07:39 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Do not conjecture volume-id from iscsi_name https://review.openstack.org/337081 | 07:40 |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337423 | 07:43 |
| *** salv-orlando has joined #openstack-cinder | 07:46 | |
| *** ducttape_ has joined #openstack-cinder | 07:47 | |
| *** ManishD has joined #openstack-cinder | 07:48 | |
| *** vbala has joined #openstack-cinder | 07:49 | |
| *** ducttape_ has quit IRC | 07:51 | |
| *** chlong has quit IRC | 07:52 | |
| *** EinstCrazy has quit IRC | 07:53 | |
| *** EinstCrazy has joined #openstack-cinder | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** salv-orlando has quit IRC | 08:04 | |
| *** chlong has joined #openstack-cinder | 08:04 | |
| *** tyrola has joined #openstack-cinder | 08:06 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| openstackgerrit | Merged openstack/cinder: Add proper scheduler_hint key to dev-ref https://review.openstack.org/329455 | 08:11 |
| *** zhonghua-lee has quit IRC | 08:14 | |
| *** davechen has left #openstack-cinder | 08:15 | |
| *** zhonghua-lee has joined #openstack-cinder | 08:15 | |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337534 | 08:16 |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337423 | 08:17 |
| *** salv-orlando has joined #openstack-cinder | 08:31 | |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337534 | 08:39 |
| *** laughterwym has quit IRC | 08:42 | |
| *** laughterwym has joined #openstack-cinder | 08:42 | |
| *** laughterwym has quit IRC | 08:42 | |
| *** laughterwym has joined #openstack-cinder | 08:43 | |
| *** laughterwym has quit IRC | 08:43 | |
| *** ducttape_ has joined #openstack-cinder | 08:48 | |
| *** salv-orlando has quit IRC | 08:48 | |
| *** akapil has quit IRC | 08:48 | |
| openstackgerrit | yuyafei proposed openstack/python-cinderclient: Remove white space between print and () https://review.openstack.org/337552 | 08:48 |
| *** ducttape_ has quit IRC | 08:52 | |
| *** GB21 has joined #openstack-cinder | 08:54 | |
| openstackgerrit | Hyeock Kwon proposed openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337534 | 08:55 |
| *** lpetrut has joined #openstack-cinder | 08:58 | |
| *** EinstCrazy has quit IRC | 09:03 | |
| *** EinstCrazy has joined #openstack-cinder | 09:03 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Support microversions on inherited Controllers https://review.openstack.org/336030 | 09:06 |
| *** akapil has joined #openstack-cinder | 09:09 | |
| *** bjornar_ has joined #openstack-cinder | 09:09 | |
| *** adisky has quit IRC | 09:10 | |
| *** salv-orlando has joined #openstack-cinder | 09:15 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 09:17 |
| *** qeelee has joined #openstack-cinder | 09:19 | |
| *** EinstCra_ has joined #openstack-cinder | 09:20 | |
| *** EinstCrazy has quit IRC | 09:23 | |
| *** maestropandy has quit IRC | 09:23 | |
| openstackgerrit | liyuanzhen proposed openstack/python-cinderclient: OS_TENANT_NAME not must be required when we have OS_PROJECT_NAME https://review.openstack.org/325067 | 09:27 |
| *** savihou has quit IRC | 09:27 | |
| openstackgerrit | Jinru Yan proposed openstack/cinder: FlashSystem: Create lager volume from vol/snap https://review.openstack.org/337572 | 09:29 |
| *** salv-orlando has quit IRC | 09:30 | |
| *** maestropandy has joined #openstack-cinder | 09:31 | |
| openstackgerrit | YaoZheng proposed openstack/cinder: support volume driver set readonly https://review.openstack.org/337580 | 09:38 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Cinder client should retry with Retry-After value https://review.openstack.org/332848 | 09:41 |
| *** ducttape_ has joined #openstack-cinder | 09:48 | |
| dulek | geguileo: Hi. I think the tests added in current 331989 have little sense. I've looked through and it seems manage_existing is actually completely not unit tested. | 09:50 |
| dulek | (no wonder we're breaking this that often) | 09:50 |
| dulek | geguileo: I think that an acceptable solution would be merging patchset #2 of 331989 and filing a bug that manage_existing is not unit tested. | 09:51 |
| dulek | geguileo: What do you think? | 09:51 |
| *** yangyapeng has quit IRC | 09:51 | |
| geguileo | dulek: I'll have to look at the tests to see if they also don't make sense to me | 09:51 |
| geguileo | dulek: But if that's the case your idea seems good | 09:52 |
| *** ducttape_ has quit IRC | 09:53 | |
| dulek | geguileo: Okay, I'll see how difficult is writing better unit tests in test_volume, so hopefully I'll be able to quickly post a follow up patch with them. | 09:53 |
| *** iceyao_ has quit IRC | 09:55 | |
| *** salv-orlando has joined #openstack-cinder | 09:56 | |
| *** yangyapeng has joined #openstack-cinder | 09:56 | |
| *** akapil has quit IRC | 09:59 | |
| *** iceyao has joined #openstack-cinder | 10:00 | |
| *** savihou has joined #openstack-cinder | 10:00 | |
| *** EinstCrazy has joined #openstack-cinder | 10:02 | |
| *** akapil has joined #openstack-cinder | 10:02 | |
| *** EinstCra_ has quit IRC | 10:04 | |
| *** tyrola has quit IRC | 10:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
| *** ociuhandu has joined #openstack-cinder | 10:12 | |
| geguileo | dulek: The tests look good to me, why don't they make sense to you? | 10:15 |
| *** salv-orlando has quit IRC | 10:16 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Validate extra_specs type https://review.openstack.org/337593 | 10:25 |
| *** yangyapeng has quit IRC | 10:28 | |
| *** ntpttr has quit IRC | 10:30 | |
| *** ntpttr has joined #openstack-cinder | 10:31 | |
| *** EinstCrazy has quit IRC | 10:33 | |
| *** iceyao has quit IRC | 10:34 | |
| *** maestropandy has quit IRC | 10:36 | |
| openstackgerrit | Soffie Huang proposed openstack/cinder: FalconStor: New Cinder driver in Newton https://review.openstack.org/331579 | 10:37 |
| *** erlon has joined #openstack-cinder | 10:37 | |
| *** dims has joined #openstack-cinder | 10:38 | |
| *** akapil has quit IRC | 10:41 | |
| *** salv-orlando has joined #openstack-cinder | 10:43 | |
| openstackgerrit | Merged openstack/python-cinderclient: base.Resource not define __ne__() built-in function https://review.openstack.org/322408 | 10:47 |
| *** ducttape_ has joined #openstack-cinder | 10:49 | |
| *** ducttape_ has quit IRC | 10:54 | |
| *** krtaylor has quit IRC | 10:55 | |
| *** salv-orlando has quit IRC | 11:01 | |
| *** salv-orlando has joined #openstack-cinder | 11:04 | |
| *** salv-orl_ has joined #openstack-cinder | 11:05 | |
| dulek | geguileo: Sorry, had a meeting. So my problem with tests there is that these are checking if methods are run with correct parameters and not the actual methods. | 11:09 |
| geguileo | dulek: As I see it they are testing all individual methods | 11:09 |
| *** salv-orlando has quit IRC | 11:10 | |
| geguileo | dulek: Since this is a unittest that's what's expected | 11:10 |
| dulek | geguileo: We have test_manage_volume_flow, but it's only testing individual task. | 11:10 |
| dulek | geguileo: I'll generate coverage report and get back to you. Maybe these tests are sufficient and I'm just complaining without reason. :) | 11:11 |
| geguileo | dulek: I think what you would like is a complete test just mocking the driver to see that the whole flow runs OK | 11:12 |
| geguileo | dulek: By flow, I mean a call to the volume manager that will do the whole thing | 11:12 |
| *** GB21 has quit IRC | 11:12 | |
| geguileo | I would like that too, but I believe that is a functional test | 11:13 |
| *** cdelatte has joined #openstack-cinder | 11:17 | |
| *** jichen has quit IRC | 11:18 | |
| *** ociuhandu has quit IRC | 11:27 | |
| *** GB21 has joined #openstack-cinder | 11:32 | |
| *** akapil has joined #openstack-cinder | 11:35 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 11:39 |
| *** tpsilva has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 11:41 |
| *** yangyapeng has joined #openstack-cinder | 11:45 | |
| *** shausy has quit IRC | 11:50 | |
| *** ducttape_ has joined #openstack-cinder | 11:50 | |
| *** shausy has joined #openstack-cinder | 11:51 | |
| *** ManishD has quit IRC | 11:52 | |
| dulek | geguileo: "Coverage for cinder/volume/flows/manager/manage_existing.py : 37%" | 11:52 |
| dulek | geguileo: But yeah, that's not the fault of the patch… | 11:52 |
| geguileo | I agree that it has to be improved | 11:53 |
| geguileo | But like you said earlier we could create a bug (low hanging fruit) for that | 11:53 |
| dulek | geguileo: Doing it right now. | 11:54 |
| geguileo | Great! | 11:54 |
| *** ducttape_ has quit IRC | 11:55 | |
| *** akapil has quit IRC | 11:57 | |
| *** caoshufeng has quit IRC | 11:57 | |
| *** iceyao has joined #openstack-cinder | 12:07 | |
| *** raildo-afk is now known as raildo | 12:07 | |
| *** maestropandy has joined #openstack-cinder | 12:08 | |
| *** ducttape_ has joined #openstack-cinder | 12:09 | |
| openstackgerrit | Shlomi Avihou proposed openstack/cinder: Add backend driver for Zadara Storage VPSA https://review.openstack.org/263026 | 12:10 |
| *** chlong has quit IRC | 12:12 | |
| *** bhavik has quit IRC | 12:18 | |
| *** openstackgerrit has quit IRC | 12:19 | |
| *** bhavik has joined #openstack-cinder | 12:19 | |
| *** openstackgerrit has joined #openstack-cinder | 12:20 | |
| *** akapil has joined #openstack-cinder | 12:26 | |
| *** ducttape_ has quit IRC | 12:27 | |
| openstackgerrit | Alon Marx proposed openstack/cinder: Update xiv_ds8k driver name and description https://review.openstack.org/318694 | 12:32 |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/336168 | 12:34 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Image virtual size doesn't fit to volume size https://review.openstack.org/311152 | 12:36 |
| *** takedakn has joined #openstack-cinder | 12:37 | |
| openstackgerrit | Merged openstack/cinder: Just a few spelling modification. "occurred" https://review.openstack.org/337534 | 12:38 |
| *** ociuhandu has joined #openstack-cinder | 12:41 | |
| *** chlong has joined #openstack-cinder | 12:43 | |
| *** akapil has quit IRC | 12:47 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Cinder client should retry with Retry-After value https://review.openstack.org/332848 | 12:48 |
| *** crose has joined #openstack-cinder | 12:49 | |
| *** mdavidson has quit IRC | 12:55 | |
| *** akapil has joined #openstack-cinder | 12:58 | |
| *** Poornima has quit IRC | 13:00 | |
| *** GB21 has quit IRC | 13:02 | |
| *** crose has quit IRC | 13:04 | |
| *** julim has joined #openstack-cinder | 13:05 | |
| *** abhishekk has quit IRC | 13:07 | |
| openstackgerrit | Pablo Iranzo Gómez proposed openstack/cinder: Fix some typos https://review.openstack.org/335478 | 13:11 |
| *** enriquetaso has joined #openstack-cinder | 13:13 | |
| *** ducttape_ has joined #openstack-cinder | 13:14 | |
| *** ducttape_ has quit IRC | 13:15 | |
| *** dustins has joined #openstack-cinder | 13:17 | |
| *** pgadiya has quit IRC | 13:17 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 13:25 |
| *** sheel has quit IRC | 13:25 | |
| *** akapil has quit IRC | 13:28 | |
| *** xyang1 has joined #openstack-cinder | 13:29 | |
| *** raildo has quit IRC | 13:34 | |
| *** zhonghua-lee has quit IRC | 13:35 | |
| *** links has joined #openstack-cinder | 13:38 | |
| *** zhonghua-lee has joined #openstack-cinder | 13:39 | |
| *** eharney has joined #openstack-cinder | 13:43 | |
| *** gcb_ has joined #openstack-cinder | 13:44 | |
| *** links has quit IRC | 13:45 | |
| *** diogogmt has quit IRC | 13:47 | |
| openstackgerrit | Clenimar Filemon proposed openstack/cinder: Read domain info from context when contacting nova https://review.openstack.org/337322 | 13:48 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fix retype when storage policy is enabled https://review.openstack.org/337672 | 13:48 |
| *** diogogmt has joined #openstack-cinder | 13:49 | |
| *** dmellado has joined #openstack-cinder | 13:49 | |
| *** ducttape_ has joined #openstack-cinder | 13:50 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Use versionedObject https://review.openstack.org/335028 | 13:53 |
| *** links has joined #openstack-cinder | 13:56 | |
| *** dustins has quit IRC | 13:56 | |
| *** Julien-zte has joined #openstack-cinder | 13:56 | |
| *** links has quit IRC | 13:59 | |
| *** qeelee has quit IRC | 13:59 | |
| *** diogogmt has quit IRC | 14:02 | |
| *** geguileo has quit IRC | 14:04 | |
| *** shausy has quit IRC | 14:06 | |
| *** bhavik has quit IRC | 14:08 | |
| *** nkrinner is now known as nkrinner_afk | 14:08 | |
| *** xyang has joined #openstack-cinder | 14:09 | |
| *** dustins has joined #openstack-cinder | 14:09 | |
| *** permalac has quit IRC | 14:14 | |
| *** akapil has joined #openstack-cinder | 14:14 | |
| *** arecknag has quit IRC | 14:16 | |
| *** geguileor is now known as geguileo | 14:18 | |
| *** geguileo is now known as Guest93477 | 14:18 | |
| *** mriedem has joined #openstack-cinder | 14:20 | |
| *** dmellado has quit IRC | 14:21 | |
| *** dmellado has joined #openstack-cinder | 14:21 | |
| *** Guest93477 has quit IRC | 14:22 | |
| *** geguileo has joined #openstack-cinder | 14:23 | |
| *** sheel has joined #openstack-cinder | 14:24 | |
| *** darosale has joined #openstack-cinder | 14:29 | |
| *** dmellado has quit IRC | 14:29 | |
| *** gcb_ has quit IRC | 14:30 | |
| *** dmellado has joined #openstack-cinder | 14:30 | |
| *** darosale has quit IRC | 14:34 | |
| *** zhangjn has quit IRC | 14:36 | |
| *** dustins_ has joined #openstack-cinder | 14:36 | |
| *** takedakn has quit IRC | 14:36 | |
| *** zhangjn has joined #openstack-cinder | 14:36 | |
| *** dustins has quit IRC | 14:40 | |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Volume delete parameters https://review.openstack.org/337704 | 14:42 |
| *** xyang has quit IRC | 14:49 | |
| *** xyang has joined #openstack-cinder | 14:49 | |
| *** xinli has joined #openstack-cinder | 14:51 | |
| *** akshai has joined #openstack-cinder | 14:53 | |
| xyang | eharney: ping | 14:56 |
| *** diogogmt has joined #openstack-cinder | 14:58 | |
| openstackgerrit | Shlomi Avihou proposed openstack/cinder: Add backend driver for Zadara Storage VPSA https://review.openstack.org/263026 | 14:58 |
| *** pbandark has quit IRC | 14:59 | |
| xyang | eharney: the missing do_setup call was added and merged in stable/mitaka. can you please remove -2 from this patch so it can be backported? https://review.openstack.org/#/c/328063/ Thanks! | 15:01 |
| *** crose has joined #openstack-cinder | 15:02 | |
| eharney | xyang: i'm a little concerned about how well tested this functionality is, for something landing in a stable branch | 15:06 |
| xyang | eharney: I tested with LVM, VNX and Huawei tested it. Also this is only changing the behavior if someone change a default setting | 15:07 |
| xyang | eharney: backup_use_same_host is False by default. If it is False, the new code won’t be called | 15:08 |
| xyang | eharney: also the new code I added are copied from Liberty (or Mitaka before they were removed) | 15:09 |
| xyang | eharney: line 123-187 in https://review.openstack.org/#/c/328063/5/cinder/backup/manager.py are copied from stable/liberty | 15:10 |
| xyang | eharney: those were there earlier on in Mitaka too. they were removed later in Mitaka. so those are not completely new code | 15:11 |
| *** crose has quit IRC | 15:12 | |
| *** scottda has joined #openstack-cinder | 15:12 | |
| *** scottda is now known as scottda_vacation | 15:13 | |
| xinli | dulek: thank you very much for your code reviewing, I have several questions: 1) should I remove CONF = cgf.conf in this file, since it's not used? https://review.openstack.org/#/c/251989/26/cinder/objects/qos_specs.py@25 | 15:15 |
| xyang | eharney: see this in stable/liberty: https://github.com/openstack/cinder/blob/stable/liberty/cinder/backup/manager.py#L110 | 15:16 |
| *** Simonsen has joined #openstack-cinder | 15:25 | |
| *** Simonsen has left #openstack-cinder | 15:26 | |
| *** Julien-zte has quit IRC | 15:27 | |
| *** _ducttape_ has joined #openstack-cinder | 15:30 | |
| *** pbandark has joined #openstack-cinder | 15:31 | |
| *** ducttape_ has quit IRC | 15:31 | |
| *** openstackgerrit has quit IRC | 15:33 | |
| *** openstackgerrit has joined #openstack-cinder | 15:34 | |
| *** xyang has quit IRC | 15:38 | |
| *** maestropandy has quit IRC | 15:38 | |
| *** harlowja has joined #openstack-cinder | 15:40 | |
| *** rlrossit has joined #openstack-cinder | 15:46 | |
| *** pbandark has quit IRC | 15:46 | |
| *** _ducttape_ has quit IRC | 15:47 | |
| *** rlrossit has quit IRC | 15:48 | |
| *** ducttape_ has joined #openstack-cinder | 15:48 | |
| *** pbandark has joined #openstack-cinder | 15:49 | |
| *** dustins_ is now known as dustins | 15:50 | |
| *** leeantho has joined #openstack-cinder | 15:53 | |
| *** iceyao has quit IRC | 15:53 | |
| *** rdx_matt has joined #openstack-cinder | 15:55 | |
| xinli | geguileo: thank you very much for your code reviewing, I have follwed up all the comments. After submiting another patch, dulek: also added some comments. Could you please help to give more infor regarding this one? http://paste.openstack.org/show/526030/ | 16:00 |
| *** porrua has joined #openstack-cinder | 16:01 | |
| geguileo | xinli: Let me look | 16:01 |
| geguileo | xinli: What we mean is that in the DB we have qos_specs_id, so you should also add a lazy loadable qos_specs field | 16:03 |
| xinli | geguileo: is there any sample code I could take as reference in objects/ such as "consistencygroup.py" | 16:05 |
| geguileo | xinli: Sure, in Volume you can see volume_type | 16:06 |
| geguileo | xinli: Which is an ObjectField | 16:06 |
| geguileo | xinli: That gets loaded using volume_type_id field | 16:06 |
| xinli | geguileo: IC, I will try to fix it, this is another one, is this mean if CONF is not used, I could remove it? https://review.openstack.org/#/c/251989/26/cinder/objects/qos_specs.py@25 | 16:07 |
| geguileo | xinli: Correct, anything that's not being used can be removed from the file | 16:07 |
| geguileo | xinli: So once you remove that line you'll also have to remove the cfg import | 16:08 |
| *** porrua has quit IRC | 16:08 | |
| xinli | geguileo: sure. | 16:08 |
| *** rlrossit has joined #openstack-cinder | 16:09 | |
| thrawn01 | smcginnis: if you have a some time I would like to pick your brain on generating the support matrix from the driver interface. | 16:10 |
| * thrawn01 was reading over the irc logs from the last meeting | 16:10 | |
| *** akshai has quit IRC | 16:11 | |
| Swanson | thrawn01, I think smcginnis is in India or on his way there. Might not be a timely reply. (Or maybe he'll be all over this. Dunno.) | 16:11 |
| thrawn01 | oh ok thanks! | 16:12 |
| *** vbala has quit IRC | 16:17 | |
| *** baojg has joined #openstack-cinder | 16:17 | |
| *** aarefiev has quit IRC | 16:21 | |
| *** bjornar_ has quit IRC | 16:22 | |
| *** aarefiev has joined #openstack-cinder | 16:23 | |
| *** savihou has quit IRC | 16:25 | |
| *** Apoorva has joined #openstack-cinder | 16:25 | |
| *** mtanino has joined #openstack-cinder | 16:26 | |
| *** karthikp_ has joined #openstack-cinder | 16:27 | |
| *** Apoorva_ has joined #openstack-cinder | 16:29 | |
| *** hemnavacation is now known as hemna | 16:30 | |
| *** rcernin has quit IRC | 16:31 | |
| *** Apoorva has quit IRC | 16:33 | |
| *** tesseract- has quit IRC | 16:33 | |
| *** hogepodge has quit IRC | 16:34 | |
| *** shyama has quit IRC | 16:35 | |
| *** rdx_matt has quit IRC | 16:37 | |
| *** rdx_matt has joined #openstack-cinder | 16:37 | |
| *** Apoorva_ has quit IRC | 16:38 | |
| *** Apoorva has joined #openstack-cinder | 16:38 | |
| *** hogepodge has joined #openstack-cinder | 16:41 | |
| openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers https://review.openstack.org/317643 | 16:41 |
| *** rdx_matt has quit IRC | 16:42 | |
| *** lpetrut has quit IRC | 16:44 | |
| *** jungleboyj has joined #openstack-cinder | 16:45 | |
| *** locust has joined #openstack-cinder | 16:46 | |
| *** baojg has quit IRC | 16:46 | |
| locust | hello, where can I find the OS Images(for nodepool) that is good for cinder driver testing? | 16:47 |
| *** hogepodge has quit IRC | 16:50 | |
| *** yangyapeng has quit IRC | 16:55 | |
| *** Suyash has joined #openstack-cinder | 16:56 | |
| *** hogepodge has joined #openstack-cinder | 17:02 | |
| *** Apoorva_ has joined #openstack-cinder | 17:03 | |
| *** Apoorva has quit IRC | 17:07 | |
| *** karthikp_ has quit IRC | 17:07 | |
| *** karthikp_ has joined #openstack-cinder | 17:08 | |
| ildikov | hemna: hi | 17:08 |
| *** permalac has joined #openstack-cinder | 17:09 | |
| *** akapil has quit IRC | 17:11 | |
| *** akapil has joined #openstack-cinder | 17:12 | |
| xinli | geguileo: after adding the code to create relationship , the unit test failed, any idea how to debug the issue? http://paste.openstack.org/show/526044/ | 17:12 |
| *** pbandark has quit IRC | 17:12 | |
| hemna | hey | 17:13 |
| geguileo | xinli: I would suggest setting a trace point and debugging it | 17:14 |
| *** akapil has quit IRC | 17:14 | |
| *** wN has quit IRC | 17:15 | |
| xinli | geguileo: Ok, I will. | 17:15 |
| *** rcernin has joined #openstack-cinder | 17:17 | |
| *** hogepodge has quit IRC | 17:17 | |
| ildikov | hemna: I played a bit with removing check_attach: https://review.openstack.org/#/c/335358/ | 17:17 |
| hemna | oh as a replacement to my patch ? | 17:18 |
| ildikov | hemna: no, it is on top of yours | 17:18 |
| *** wN has joined #openstack-cinder | 17:19 | |
| *** wN has quit IRC | 17:19 | |
| *** wN has joined #openstack-cinder | 17:19 | |
| ildikov | hemna: feel free to merge it into yours if/when it looks ok | 17:19 |
| ildikov | hemna: I was just curious what happens if the function is actually deleted | 17:20 |
| ildikov | hemna: there's one place I added reserve_volume, I'm really not sure where that should be for instance... | 17:20 |
| openstackgerrit | Prashant Shetty proposed openstack/cinder: Fixed incorrect spellings https://review.openstack.org/337800 | 17:21 |
| *** cfriesen__ has joined #openstack-cinder | 17:22 | |
| cfriesen__ | hi...is this the right place to ask about os-brick? | 17:22 |
| *** mdavidson has joined #openstack-cinder | 17:23 | |
| patrickeast | cfriesen__: yea | 17:23 |
| *** hogepodge has joined #openstack-cinder | 17:23 | |
| *** lpetrut has joined #openstack-cinder | 17:24 | |
| *** karthikp_ has quit IRC | 17:25 | |
| *** zhonghua-lee has quit IRC | 17:25 | |
| *** zhonghua-lee has joined #openstack-cinder | 17:26 | |
| *** shyama has joined #openstack-cinder | 17:26 | |
| *** karthikp_ has joined #openstack-cinder | 17:26 | |
| ildikov | hemna: one interesting experience is that the only test job that fails is grenade | 17:26 |
| *** akapil has joined #openstack-cinder | 17:27 | |
| hemna | back | 17:28 |
| *** hogepodge has quit IRC | 17:28 | |
| hemna | ildikov, ok I'll try and take a look at it today | 17:28 |
| cfriesen__ | I'm on CentOS 7, using os-brick 1.1.0-1.el7. In initiator/linuxscsi.py the function get_scsi_wwn() is trying to execute 'scsi_id', but that's not in the default PATH. I note that git commit dbf77fba1 changed it to explicitly specify the full path. | 17:28 |
| hemna | I'm juggling lots of plates today | 17:29 |
| hemna | cfriesen__, that's a very old version of the lib | 17:29 |
| cfriesen__ | hemna: sure, but 1.3.0 has the same issue | 17:29 |
| hemna | the problem was that scsi_id is assumed to be defined in a rootwrap filter, which has the full path. | 17:30 |
| ildikov | hemna: sure, not urgent, I just wanted to let you know I played with it a bit and point out the interesting parts | 17:30 |
| *** akapil has quit IRC | 17:30 | |
| cfriesen__ | hemna: ah, okay. will dig into that, thanks | 17:30 |
| hemna | 1.4.0 uses privsep to execute commands and it has the full path | 17:30 |
| *** crose has joined #openstack-cinder | 17:33 | |
| *** bjornar_ has joined #openstack-cinder | 17:35 | |
| *** rajinir has joined #openstack-cinder | 17:36 | |
| *** hogepodge has joined #openstack-cinder | 17:36 | |
| *** karthik__ has joined #openstack-cinder | 17:39 | |
| *** karthikp_ has quit IRC | 17:41 | |
| *** ducttape_ has quit IRC | 17:42 | |
| *** bjornar_ has quit IRC | 17:44 | |
| *** baojg has joined #openstack-cinder | 17:47 | |
| *** baojg has quit IRC | 17:53 | |
| *** zhangjn has quit IRC | 17:54 | |
| erlon | jgriffith: ping | 17:54 |
| *** zhangjn has joined #openstack-cinder | 17:54 | |
| *** ducttape_ has joined #openstack-cinder | 17:55 | |
| *** hogepodge has quit IRC | 17:55 | |
| *** karthik__ has quit IRC | 17:55 | |
| *** cfriesen__ has left #openstack-cinder | 17:59 | |
| *** shyama has quit IRC | 18:00 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Cascade + force volume delete parameters https://review.openstack.org/337814 | 18:02 |
| *** nikeshm has joined #openstack-cinder | 18:03 | |
| *** crose has quit IRC | 18:04 | |
| *** Apoorva_ has quit IRC | 18:05 | |
| *** Apoorva has joined #openstack-cinder | 18:05 | |
| -openstackstatus- NOTICE: Job instability resulting from a block storage connectivity error on mirror.iad.rax.openstack.org has been corrected; jobs running in rax-iad should be more reliable again. | 18:13 | |
| *** ociuhandu has quit IRC | 18:15 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 18:16 |
| *** akapil has joined #openstack-cinder | 18:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/332361 | 18:19 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 18:20 |
| *** akapil has quit IRC | 18:22 | |
| *** TaylorHuang has joined #openstack-cinder | 18:23 | |
| *** hogepodge has joined #openstack-cinder | 18:26 | |
| *** TaylorH has quit IRC | 18:26 | |
| *** adrianofr has joined #openstack-cinder | 18:32 | |
| *** lpetrut has quit IRC | 18:42 | |
| *** lpetrut has joined #openstack-cinder | 18:42 | |
| *** blinky_ghost has joined #openstack-cinder | 18:49 | |
| *** baojg has joined #openstack-cinder | 18:50 | |
| blinky_ghost | hi all, I've just migrated my Openstack from Kilo to Liberty. All services work except cinder. I have this error in the api: ERROR cinder ImportError: No module named middleware.request_id. Any hint? thanks | 18:50 |
| *** lpetrut has quit IRC | 18:52 | |
| *** hogepodge has quit IRC | 18:53 | |
| *** baojg has quit IRC | 18:54 | |
| patrickeast | blinky_ghost: any more info on where that error message is coming from? stack trace? | 18:55 |
| patrickeast | blinky_ghost: my guess is this thing https://github.com/openstack/cinder/blob/01fc0e6cd93ac101343c86c2ac4dbd126d692d7c/etc/cinder/api-paste.ini#L31 | 18:55 |
| *** bjornar_ has joined #openstack-cinder | 18:55 | |
| patrickeast | maybe needed a change to api_paste between kilo and liberty | 18:55 |
| blinky_ghost | patrickeast: it comes from cinder-api | 18:55 |
| patrickeast | blinky_ghost: yea but like... from the log? from a HTTP request? | 18:56 |
| patrickeast | blinky_ghost: when you start the service? | 18:56 |
| blinky_ghost | patrickeast: yes | 18:56 |
| patrickeast | blinky_ghost: can you share more of the log where the error shows up? | 18:58 |
| *** M00nr41n has quit IRC | 18:58 | |
| blinky_ghost | patrickeast: sure a sec | 18:59 |
| *** M00nr41n has joined #openstack-cinder | 19:00 | |
| blinky_ghost | patrickeast: http://paste.openstack.org/show/526055/ | 19:00 |
| *** hogepodge has joined #openstack-cinder | 19:00 | |
| patrickeast | blinky_ghost: ok cool, can you share your api_paste.ini too? | 19:03 |
| patrickeast | my guess is you have something like: https://github.com/openstack/cinder/blob/juno-eol/etc/cinder/api-paste.ini#L24 | 19:04 |
| *** lennyb has quit IRC | 19:04 | |
| patrickeast | from the looks of it the middleware.request_id used to be in cinder, but we use oslo_middleware.request_id in liberty | 19:04 |
| blinky_ghost | patrickeast: http://paste.openstack.org/show/526058/ | 19:05 |
| patrickeast | ah yea, so line 24 on yours is more than likely the issue | 19:05 |
| patrickeast | blinky_ghost: try and make it look more like the example one from the liberty cinder tree: https://github.com/openstack/cinder/blob/01fc0e6cd93ac101343c86c2ac4dbd126d692d7c/etc/cinder/api-paste.ini#L31 | 19:06 |
| patrickeast | err https://github.com/openstack/cinder/blob/stable/liberty/etc/cinder/api-paste.ini | 19:07 |
| patrickeast | ^ thats the liberty one | 19:07 |
| blinky_ghost | patrickeast: you're the best :) | 19:10 |
| blinky_ghost | thanks | 19:10 |
| patrickeast | np | 19:10 |
| blinky_ghost | patrickeast: I only need to change this on cinder right? I didn't had any issues with the other services | 19:10 |
| patrickeast | blinky_ghost: not 100% sure, but its probably a cinder-ism | 19:10 |
| blinky_ghost | cool | 19:11 |
| *** sgotliv__ has quit IRC | 19:11 | |
| blinky_ghost | thanks a lot you saved my life :) | 19:11 |
| nikeshm | finally got the solution of slave went offline issue in my CI system, i replaced localhost with host ip in nodepool.yaml, and it worked | 19:13 |
| *** bjornar_ has quit IRC | 19:13 | |
| patrickeast | nikeshm: oh weird, any idea why that fixes it? | 19:14 |
| * patrickeast has localhost in pure's nodepool.yaml | 19:15 | |
| xinli | dulek: I made progress, I fixed 4 issues of your 7 comments, but for this one I added relationship in volume_type.py, but the error message showing: sqlalchemy.orm.exc.DetachedInstanceError: Parent instance <VolumeTypes at 0x7f06c9bcaa50> is not bound to a Session; lazy load operation of attribute 'qos_specs' cannot proceed | 19:15 |
| xinli | dulek: this is debug result, no idea what is next step? could you please give some suggestion? http://paste.openstack.org/show/526044/ | 19:16 |
| *** akapil has joined #openstack-cinder | 19:17 | |
| nikeshm | patrickeast: not sure, may be /etc/hosts is problem http://paste.openstack.org/show/526061/ | 19:18 |
| patrickeast | nikeshm: interesting, i'll have to look and see if i need to do the same on mine | 19:19 |
| *** julim has quit IRC | 19:19 | |
| *** lpetrut has joined #openstack-cinder | 19:21 | |
| *** akapil has quit IRC | 19:21 | |
| nikeshm | patrickeast: i tested on dummy tests in sandbox which were facing slave went offline previously and after this change did not face this issue, now have to tests on real cinder tests | 19:23 |
| *** lennyb has joined #openstack-cinder | 19:24 | |
| *** sheel has quit IRC | 19:25 | |
| *** hogepodge has quit IRC | 19:27 | |
| *** pbandark has joined #openstack-cinder | 19:27 | |
| *** hogepodge has joined #openstack-cinder | 19:29 | |
| *** karthikp_ has joined #openstack-cinder | 19:31 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 19:33 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 19:33 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Cascade + force volume delete parameters https://review.openstack.org/337814 | 19:40 |
| locust | hello, is here anybody who have experince in CI? | 19:41 |
| *** karthikp_ has quit IRC | 19:43 | |
| *** hogepodge has quit IRC | 19:43 | |
| *** hogepodge has joined #openstack-cinder | 19:44 | |
| patrickeast | locust: yea | 19:44 |
| patrickeast | lots of folks in here | 19:44 |
| locust | patrickeast: cool, working on CI at particular moment | 19:45 |
| *** julim has joined #openstack-cinder | 19:45 | |
| *** karthikp_ has joined #openstack-cinder | 19:45 | |
| locust | patrickeast: a have deployd it according to http://docs.openstack.org/infra/openstackci/third_party_ci.html#set-up-the-ci-server | 19:46 |
| locust | patrickeast: it is doing something, but even sure that it is something correct | 19:46 |
| patrickeast | locust: doing something is a good start | 19:47 |
| patrickeast | locust: do you have it running on the sandbox project or something? | 19:48 |
| locust | it is production deployment | 19:48 |
| locust | but at particular moment... yep , sand box ) | 19:48 |
| locust | but with big plans | 19:49 |
| locust | =) | 19:49 |
| asselin__ | locust, patrickeast double check the version of jenkins. There's a security fix with recent jenkins versions that breaks its integration with zuul. This patch can also help fix it: https://review.openstack.org/#/c/334400/ | 19:49 |
| blinky_ghost | patrickeast: Now I'm having another error when I try to boot an instance from a volume: 2016-07-05 20:49:25.438 23969 ERROR cinder.api.middleware.fault [req-a938a717-aef9-4b8f-8e89-85d8fe599fd3 57308bc934584e4a91a83da27eb51230 bcfc8e5b8fff42dabf6acdca2b2b9944 - - -] Caught error: Not enough information to determine URL. Provide either auth_url or endpoint | 19:50 |
| patrickeast | asselin__: yea :( i added a step somewhere to modify the java args for jenkins to allow that | 19:50 |
| *** baojg has joined #openstack-cinder | 19:50 | |
| blinky_ghost | patrickeast: I have to put auth_url on api-paste.ini correct? | 19:51 |
| patrickeast | blinky_ghost: mm not sure if they need to be in api-paste | 19:52 |
| patrickeast | blinky_ghost: looking at a more recent cinder thats working i don't see them in there, and things worked OK http://52.8.17.232/23/logs/etc/cinder/api-paste.ini.txt.gz | 19:53 |
| patrickeast | blinky_ghost: my guess is its the keystone section in your cinder.conf | 19:53 |
| *** baojg has quit IRC | 19:55 | |
| locust | I am thingking of what to do next | 19:55 |
| locust | patrickeast: what can I do next, and test my system? | 19:55 |
| xinli | geguileo: I want to understand some basic concept here: http://paste.openstack.org/show/526066/ | 19:55 |
| *** karthikp_ has quit IRC | 19:55 | |
| *** karthikp_ has joined #openstack-cinder | 19:56 | |
| patrickeast | locust: well, first goal would be to have your system do kind of an end-to-end run on the sandbox project, like whats shown in the project-config-example https://github.com/openstack-infra/project-config-example/blob/master/zuul/layout.yaml#L61 | 19:57 |
| patrickeast | locust: once you have it where you pick up gerrit changes, run tests, and post the logs, make sure you can do it with a tempest test run too | 19:57 |
| patrickeast | locust: after that you are all set | 19:57 |
| locust | patrickeast: is it possible to do by the end of this weak? | 19:58 |
| patrickeast | locust: maybe, depends on quite a few things | 19:58 |
| locust | gues may be you know, is it possible to put driver prototyp in the master brunch without ci | 20:01 |
| locust | ? | 20:01 |
| locust | with all cals blocked | 20:01 |
| *** sheel has joined #openstack-cinder | 20:01 | |
| locust | to extend it during next iterration? | 20:02 |
| locust | when CI is working? | 20:02 |
| *** xyang has joined #openstack-cinder | 20:03 | |
| blinky_ghost | patrickeast: you're right again thanks it works now :) | 20:04 |
| patrickeast | locust: nope, needs to have a CI before it can be merged | 20:05 |
| patrickeast | blinky_ghost: yay! | 20:05 |
| *** xyang has left #openstack-cinder | 20:07 | |
| *** Apoorva has quit IRC | 20:07 | |
| xinli | patrickeast: I have two basic qustion regarding the OVO syntax and error message interpretation, could you please take a look? http://paste.openstack.org/show/526070/ | 20:08 |
| *** Apoorva has joined #openstack-cinder | 20:09 | |
| patrickeast | xinli: sry, i'm not very knowledgable with the OVO stuff, dulek is your best bet for that but he's in a different timezone so probably won't be on for a while. | 20:10 |
| patrickeast | and/or geguileo ^ | 20:10 |
| locust | patrickeast: may be you know is it possible for driver to store any aditional info about volume inside of OpenStack? | 20:11 |
| *** karthikp_ has quit IRC | 20:11 | |
| *** karthikp_ has joined #openstack-cinder | 20:11 | |
| xinli | patrickeast: both not available, do you know what is dulek's timezone? | 20:11 |
| patrickeast | locust: yea you can stash a few things for the volume, there are a couple of provider_* fields on teh volume object that you can use | 20:11 |
| patrickeast | xinli: yea both are in europe, not sure exactly what TZ | 20:12 |
| xinli | patrickeast: thx | 20:12 |
| patrickeast | xinli: tomorrow morning (well, morning PST) before the cinder weekly meeting is a good time, they should both be around | 20:12 |
| patrickeast | locust: these ones https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L87-L90 | 20:14 |
| xinli | patrickeast: in which time and time zone? | 20:15 |
| *** akapil has joined #openstack-cinder | 20:15 | |
| patrickeast | xinli: meeting info https://wiki.openstack.org/wiki/CinderMeetings#Weekly_Cinder_team_meeting | 20:15 |
| patrickeast | xinli: try maybe an hour or so before then | 20:16 |
| *** bjornar_ has joined #openstack-cinder | 20:16 | |
| locust | it looks like I am not capable of storring arbitraty data in metadata? | 20:16 |
| xinli | patrickeast: Pacific Standard Time ? | 20:16 |
| patrickeast | xinli: the wiki says 16:00 UTC -> 9AM PST | 20:18 |
| patrickeast | locust: so... technically speaking, you can do that, but it isn't what the metadata is for | 20:21 |
| *** karthikp_ has quit IRC | 20:21 | |
| patrickeast | locust: that is user facing metadata for like tagging volumes | 20:21 |
| patrickeast | locust: which means anything backend specific is very much frowned upon being set in there by drivers | 20:22 |
| xinli | patrickeast: then it's our central time 11 AM (PST 9:00 AM). thx | 20:22 |
| patrickeast | locust: what info do you need to store? | 20:22 |
| *** akapil has quit IRC | 20:23 | |
| *** akapil has joined #openstack-cinder | 20:24 | |
| locust | patrickeast: info about additional info on the side of the storage | 20:24 |
| locust | patrickeast: tmp metadata that mus be readed | 20:25 |
| locust | and considered | 20:25 |
| locust | patrickeast: some weird enterprise magic... | 20:25 |
| patrickeast | locust: how much info is it? like a couple of values? or lots of arbitrary key-value info? Can maybe help find where you can stash it with more context around what it is and what its for | 20:27 |
| locust | few key value | 20:27 |
| locust | like I need to store 10-30 bytes | 20:28 |
| patrickeast | locust: hmm well, thats not so bad | 20:28 |
| *** enriquetaso has quit IRC | 20:28 | |
| patrickeast | locust: theoretically you could put it in a json string into provider_location or id or something | 20:28 |
| patrickeast | its not great... but might work | 20:28 |
| patrickeast | iirc you've got 255 characters to work with | 20:29 |
| locust | patrickeast: hm | 20:29 |
| locust | sounds good | 20:29 |
| *** hogepodge has quit IRC | 20:29 | |
| *** hogepodge has joined #openstack-cinder | 20:31 | |
| *** maestropandy has joined #openstack-cinder | 20:31 | |
| *** eharney has quit IRC | 20:32 | |
| *** Lee1092 has quit IRC | 20:33 | |
| locust | patrickeast: thank you | 20:35 |
| locust | patrickeast: you are from usa? | 20:36 |
| *** karthikp_ has joined #openstack-cinder | 20:37 | |
| *** akapil has quit IRC | 20:38 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor5: add extend method to NFS driver https://review.openstack.org/297880 | 20:39 |
| *** akapil has joined #openstack-cinder | 20:39 | |
| patrickeast | locust: yep | 20:41 |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Differentiate thick and thin provisioning https://review.openstack.org/332488 | 20:42 |
| *** jungleboyj has quit IRC | 20:45 | |
| *** hogepodge has quit IRC | 20:45 | |
| *** hogepodge has joined #openstack-cinder | 20:47 | |
| *** baojg has joined #openstack-cinder | 20:51 | |
| *** baojg has quit IRC | 20:55 | |
| *** karthikp_ has quit IRC | 20:58 | |
| *** blinky_ghost has quit IRC | 21:02 | |
| *** ducttape_ has quit IRC | 21:03 | |
| *** pbandark has quit IRC | 21:05 | |
| *** hogepodge has quit IRC | 21:09 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Differentiate thick and thin provisioning https://review.openstack.org/332488 | 21:09 |
| *** hogepodge has joined #openstack-cinder | 21:14 | |
| *** ducttape_ has joined #openstack-cinder | 21:16 | |
| *** akapil has quit IRC | 21:17 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Differentiate thick and thin provisioning https://review.openstack.org/332488 | 21:24 |
| *** akapil has joined #openstack-cinder | 21:26 | |
| *** gouthamr has joined #openstack-cinder | 21:28 | |
| *** jungleboyj has joined #openstack-cinder | 21:30 | |
| *** akapil has quit IRC | 21:33 | |
| *** merooney_ has joined #openstack-cinder | 21:36 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add ignore for vim python-rope dirs https://review.openstack.org/337890 | 21:37 |
| *** dustins has quit IRC | 21:37 | |
| *** merooney has quit IRC | 21:39 | |
| *** mriedem is now known as mriedem_away | 21:43 | |
| *** baojg has joined #openstack-cinder | 21:52 | |
| *** wN has quit IRC | 21:53 | |
| *** gouthamr_ has joined #openstack-cinder | 21:56 | |
| *** baojg has quit IRC | 21:56 | |
| *** rlrossit has quit IRC | 21:57 | |
| *** xyang1 has quit IRC | 21:58 | |
| *** akapil has joined #openstack-cinder | 21:59 | |
| *** akapil has quit IRC | 21:59 | |
| *** gouthamr has quit IRC | 22:00 | |
| *** Apoorva_ has joined #openstack-cinder | 22:03 | |
| *** Apoorva has quit IRC | 22:06 | |
| openstackgerrit | Merged openstack/cinder: Add logging when filtering returns nothing https://review.openstack.org/306285 | 22:11 |
| *** akapil has joined #openstack-cinder | 22:13 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Cinder client should retry with Retry-After value https://review.openstack.org/332848 | 22:13 |
| *** Apoorva has joined #openstack-cinder | 22:16 | |
| openstackgerrit | Merged openstack/cinder: Fix some typos https://review.openstack.org/335478 | 22:16 |
| *** Apoorva_ has quit IRC | 22:16 | |
| openstackgerrit | Merged openstack/cinder: Log terminate_connection exception with traceback https://review.openstack.org/336605 | 22:16 |
| openstackgerrit | Merged openstack/cinder: Remove translation from config options https://review.openstack.org/336027 | 22:16 |
| openstackgerrit | Merged openstack/cinder: Add README in release notes directory https://review.openstack.org/336393 | 22:16 |
| *** akapil has quit IRC | 22:17 | |
| *** merooney has joined #openstack-cinder | 22:18 | |
| *** lpetrut has quit IRC | 22:19 | |
| *** merooney_ has quit IRC | 22:21 | |
| *** gouthamr_ is now known as gouthamr | 22:23 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: release note for windows iSCSI https://review.openstack.org/337908 | 22:23 |
| *** Julien-zte has joined #openstack-cinder | 22:26 | |
| *** erlon has quit IRC | 22:26 | |
| *** jungleboyj has quit IRC | 22:29 | |
| *** xinli has quit IRC | 22:29 | |
| *** maestropandy has quit IRC | 22:29 | |
| *** ducttape_ has quit IRC | 22:30 | |
| *** salv-orl_ has quit IRC | 22:36 | |
| *** Julien-zte has quit IRC | 22:36 | |
| *** salv-orlando has joined #openstack-cinder | 22:37 | |
| *** karthikp_ has joined #openstack-cinder | 22:38 | |
| *** salv-orlando has quit IRC | 22:42 | |
| *** sheel has quit IRC | 22:45 | |
| *** Apoorva_ has joined #openstack-cinder | 22:47 | |
| *** Apoorva has quit IRC | 22:47 | |
| *** baojg has joined #openstack-cinder | 22:52 | |
| *** baojg has quit IRC | 22:57 | |
| *** adrianofr has quit IRC | 23:01 | |
| *** karthikp_ has quit IRC | 23:02 | |
| *** salv-orlando has joined #openstack-cinder | 23:06 | |
| *** salv-orlando has quit IRC | 23:12 | |
| *** rcernin has quit IRC | 23:18 | |
| *** Apoorva has joined #openstack-cinder | 23:24 | |
| *** Apoorva_ has quit IRC | 23:26 | |
| *** Apoorva has quit IRC | 23:29 | |
| *** Apoorva has joined #openstack-cinder | 23:29 | |
| *** sambhare has joined #openstack-cinder | 23:33 | |
| *** sambhare has quit IRC | 23:33 | |
| *** permalac has quit IRC | 23:35 | |
| *** ducttape_ has joined #openstack-cinder | 23:46 | |
| *** tpsilva has quit IRC | 23:48 | |
| *** leeantho has quit IRC | 23:52 | |
| *** baojg has joined #openstack-cinder | 23:53 | |
| *** baojg has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!