| *** yangyape_ has quit IRC | 00:02 | |
| *** garthb__ has quit IRC | 00:04 | |
| *** jwcroppe has quit IRC | 00:06 | |
| *** jwcroppe has joined #openstack-cinder | 00:07 | |
| *** mylu has quit IRC | 00:08 | |
| smcginnis | scottda: Still around? | 00:09 |
|---|---|---|
| *** mylu has joined #openstack-cinder | 00:10 | |
| *** asselin_ has joined #openstack-cinder | 00:13 | |
| *** fernnest_ has quit IRC | 00:17 | |
| *** fernnest has quit IRC | 00:18 | |
| *** asselin_ has quit IRC | 00:21 | |
| *** sdake has quit IRC | 00:21 | |
| *** haomaiwang has joined #openstack-cinder | 00:21 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Fujitsu DX: Add retry to save some failure https://review.openstack.org/297657 | 00:22 |
| *** sdake has joined #openstack-cinder | 00:25 | |
| *** haomaiwang has quit IRC | 00:26 | |
| *** laughterwym has joined #openstack-cinder | 00:27 | |
| *** leeantho has quit IRC | 00:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/307747 | 00:36 |
| *** mylu has quit IRC | 00:38 | |
| *** sdake has quit IRC | 00:38 | |
| *** sdake has joined #openstack-cinder | 00:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/302877 | 00:41 |
| *** haplo37 has joined #openstack-cinder | 00:49 | |
| *** Suyi has joined #openstack-cinder | 00:51 | |
| *** karthikp has quit IRC | 00:53 | |
| *** mtanino_ has joined #openstack-cinder | 00:54 | |
| *** cheneydc has joined #openstack-cinder | 00:55 | |
| *** mtanino has quit IRC | 00:55 | |
| *** karthikp has joined #openstack-cinder | 00:57 | |
| *** mtanino_ has quit IRC | 00:59 | |
| *** bardia has quit IRC | 01:07 | |
| *** dave-mccowan has joined #openstack-cinder | 01:07 | |
| *** sasukeh has joined #openstack-cinder | 01:13 | |
| *** yangyape_ has joined #openstack-cinder | 01:22 | |
| *** Suyi has quit IRC | 01:26 | |
| *** Suyash has quit IRC | 01:26 | |
| *** EinstCrazy has joined #openstack-cinder | 01:26 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 01:27 |
| *** yangyape_ has quit IRC | 01:29 | |
| *** mylu has joined #openstack-cinder | 01:35 | |
| *** apoorvad has joined #openstack-cinder | 01:39 | |
| *** yangxi has joined #openstack-cinder | 01:41 | |
| *** gouthamr has joined #openstack-cinder | 01:42 | |
| *** qeelee has joined #openstack-cinder | 01:43 | |
| *** sasukeh has quit IRC | 01:46 | |
| *** qeelee has quit IRC | 01:46 | |
| *** apoorvad has quit IRC | 01:52 | |
| *** mylu has quit IRC | 01:55 | |
| *** Julien-zte has joined #openstack-cinder | 01:58 | |
| *** mylu has joined #openstack-cinder | 01:58 | |
| *** EinstCra_ has joined #openstack-cinder | 02:00 | |
| *** yangxi has quit IRC | 02:00 | |
| *** apoorvad has joined #openstack-cinder | 02:02 | |
| *** EinstCrazy has quit IRC | 02:03 | |
| mriedem | smcginnis: hemna_: any idea when the privsep stuff is going to get into os-brick and released? https://review.openstack.org/#/c/277224/ | 02:04 |
| mriedem | i had rebased that to bump the minimum required version of privsep for one of the dependent changes | 02:05 |
| mriedem | otherwise it has +2s on it | 02:05 |
| mriedem | *had | 02:05 |
| *** apoorvad has quit IRC | 02:12 | |
| *** sasukeh has joined #openstack-cinder | 02:14 | |
| *** karthikp has quit IRC | 02:14 | |
| *** Lee1092 has joined #openstack-cinder | 02:14 | |
| *** karthikp has joined #openstack-cinder | 02:14 | |
| *** mriedem has quit IRC | 02:14 | |
| * patrickeast forgot i was going to look at that | 02:17 | |
| *** jdurgin has joined #openstack-cinder | 02:19 | |
| *** Julien-zte has quit IRC | 02:19 | |
| *** sdake_ has joined #openstack-cinder | 02:20 | |
| *** jwcroppe_ has joined #openstack-cinder | 02:22 | |
| *** sdake has quit IRC | 02:23 | |
| *** houming has joined #openstack-cinder | 02:24 | |
| *** jwcroppe has quit IRC | 02:25 | |
| *** xiaohui has quit IRC | 02:26 | |
| *** gouthamr has quit IRC | 02:28 | |
| *** dave-mcc_ has joined #openstack-cinder | 02:31 | |
| *** ebalduf has joined #openstack-cinder | 02:32 | |
| *** dave-mccowan has quit IRC | 02:32 | |
| *** yangxi has joined #openstack-cinder | 02:34 | |
| openstackgerrit | Merged openstack/python-cinderclient: Support api-microversions https://review.openstack.org/301941 | 02:35 |
| *** alkhodos has quit IRC | 02:35 | |
| *** sdake_ has quit IRC | 02:36 | |
| *** delattec has quit IRC | 02:40 | |
| *** sekrit is now known as CIA | 02:44 | |
| *** dave-mcc_ has quit IRC | 02:47 | |
| *** sdake has joined #openstack-cinder | 02:48 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 02:53 |
| *** iceyao has joined #openstack-cinder | 02:53 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers https://review.openstack.org/283029 | 02:56 |
| *** karthikp has quit IRC | 02:57 | |
| *** yangxi has quit IRC | 03:02 | |
| *** sgotliv_ has joined #openstack-cinder | 03:12 | |
| *** sasukeh has quit IRC | 03:14 | |
| *** jdurgin has quit IRC | 03:20 | |
| *** links has joined #openstack-cinder | 03:21 | |
| *** iceyao_ has joined #openstack-cinder | 03:28 | |
| *** yangxi has joined #openstack-cinder | 03:28 | |
| *** iceyao has quit IRC | 03:32 | |
| *** sgotliv_ has quit IRC | 03:38 | |
| *** mylu has quit IRC | 03:38 | |
| *** haplo37 has quit IRC | 03:39 | |
| *** sasukeh has joined #openstack-cinder | 03:51 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata https://review.openstack.org/147738 | 03:58 |
| *** mylu has joined #openstack-cinder | 03:59 | |
| *** mylu has quit IRC | 04:00 | |
| *** asselin_ has joined #openstack-cinder | 04:10 | |
| *** karthikp has joined #openstack-cinder | 04:10 | |
| *** ekarlso has quit IRC | 04:11 | |
| *** sgotliv has joined #openstack-cinder | 04:12 | |
| *** mylu has joined #openstack-cinder | 04:14 | |
| *** smoriya has quit IRC | 04:16 | |
| *** smoriya_afk is now known as smoriya | 04:16 | |
| *** smoriya_afk has joined #openstack-cinder | 04:17 | |
| *** ekarlso has joined #openstack-cinder | 04:25 | |
| *** apoorvad has joined #openstack-cinder | 04:31 | |
| *** markvoelker has quit IRC | 04:34 | |
| *** markvoelker has joined #openstack-cinder | 04:35 | |
| *** sdake has quit IRC | 04:36 | |
| *** apoorvad has quit IRC | 04:36 | |
| *** markvoelker has quit IRC | 04:40 | |
| *** lpetrut has joined #openstack-cinder | 04:40 | |
| *** cknight1 has quit IRC | 04:54 | |
| *** mylu has quit IRC | 04:57 | |
| *** daneyon has joined #openstack-cinder | 05:01 | |
| *** yangxi has quit IRC | 05:02 | |
| *** laughter_ has joined #openstack-cinder | 05:04 | |
| *** daneyon_ has quit IRC | 05:04 | |
| *** karthikp has quit IRC | 05:05 | |
| *** laughterwym has quit IRC | 05:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/302877 | 05:19 |
| *** gouthamr has joined #openstack-cinder | 05:19 | |
| *** laughter_ has quit IRC | 05:21 | |
| *** laughterwym has joined #openstack-cinder | 05:21 | |
| *** laughterwym has quit IRC | 05:27 | |
| *** sasukeh has quit IRC | 05:29 | |
| *** avishay has joined #openstack-cinder | 05:30 | |
| *** sasukeh has joined #openstack-cinder | 05:31 | |
| *** gouthamr has quit IRC | 05:34 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix message of quota exceed exception https://review.openstack.org/281666 | 05:34 |
| *** sasukeh has quit IRC | 05:35 | |
| openstackgerrit | LisaLi proposed openstack/cinder-specs: Retype encrypted volumes https://review.openstack.org/248593 | 05:36 |
| *** sdake_ has joined #openstack-cinder | 05:37 | |
| *** daneyon_ has joined #openstack-cinder | 05:37 | |
| openstackgerrit | aditi sharma proposed openstack/python-cinderclient: Add missing user message https://review.openstack.org/269548 | 05:39 |
| *** daneyon has quit IRC | 05:40 | |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/307747 | 05:41 |
| *** gcb has quit IRC | 05:42 | |
| *** Poornima has joined #openstack-cinder | 05:45 | |
| *** itzdilip has joined #openstack-cinder | 05:48 | |
| *** chip_ has joined #openstack-cinder | 05:49 | |
| *** laughterwym has joined #openstack-cinder | 05:51 | |
| *** chip_ has quit IRC | 05:52 | |
| *** chip_ has joined #openstack-cinder | 05:53 | |
| *** laughterwym has quit IRC | 05:56 | |
| *** ChubYann has quit IRC | 05:56 | |
| *** laughterwym has joined #openstack-cinder | 05:57 | |
| *** gcb has joined #openstack-cinder | 05:58 | |
| *** sgotliv has quit IRC | 05:58 | |
| *** harlowja_at_home has joined #openstack-cinder | 06:00 | |
| *** laughterwym has quit IRC | 06:02 | |
| *** yangxi has joined #openstack-cinder | 06:02 | |
| *** lpetrut has quit IRC | 06:03 | |
| *** sasukeh has joined #openstack-cinder | 06:07 | |
| *** mjura has joined #openstack-cinder | 06:08 | |
| *** laughterwym has joined #openstack-cinder | 06:09 | |
| *** laughterwym has quit IRC | 06:10 | |
| *** laughter_ has joined #openstack-cinder | 06:12 | |
| *** ericksonsantos has quit IRC | 06:13 | |
| *** chip_ has quit IRC | 06:16 | |
| *** rcernin has joined #openstack-cinder | 06:21 | |
| *** chip__ has joined #openstack-cinder | 06:22 | |
| *** lpetrut has joined #openstack-cinder | 06:22 | |
| *** TheBall has quit IRC | 06:22 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix of finding out available cinder volume services https://review.openstack.org/308148 | 06:25 |
| *** raildo is now known as raildo-afk | 06:25 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix of finding out available cinder volume services https://review.openstack.org/308148 | 06:26 |
| *** laughter_ has quit IRC | 06:32 | |
| *** laughterwym has joined #openstack-cinder | 06:32 | |
| *** apoorvad has joined #openstack-cinder | 06:34 | |
| *** laughterwym has quit IRC | 06:36 | |
| *** laughterwym has joined #openstack-cinder | 06:36 | |
| *** e0ne has joined #openstack-cinder | 06:38 | |
| *** apoorvad has quit IRC | 06:39 | |
| *** lpetrut has quit IRC | 06:40 | |
| *** nkrinner has joined #openstack-cinder | 06:40 | |
| *** e0ne has quit IRC | 06:43 | |
| *** e0ne has joined #openstack-cinder | 06:44 | |
| *** e0ne_ has joined #openstack-cinder | 06:45 | |
| *** sheel has joined #openstack-cinder | 06:47 | |
| *** anshul has joined #openstack-cinder | 06:47 | |
| *** anshul is now known as Guest93067 | 06:47 | |
| *** tesseract has joined #openstack-cinder | 06:47 | |
| *** e0ne has quit IRC | 06:48 | |
| *** tesseract is now known as Guest67082 | 06:48 | |
| sheel | Jenkins seems angry... | 06:49 |
| *** laughterwym has quit IRC | 06:50 | |
| *** laughterwym has joined #openstack-cinder | 06:50 | |
| *** e0ne_ has quit IRC | 06:50 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 06:54 |
| *** laughterwym has quit IRC | 06:55 | |
| *** The_Ball has joined #openstack-cinder | 06:56 | |
| *** harlowja_at_home has quit IRC | 06:59 | |
| *** permalac has joined #openstack-cinder | 07:00 | |
| *** bjornar has joined #openstack-cinder | 07:05 | |
| *** laughterwym has joined #openstack-cinder | 07:07 | |
| *** yangxi has quit IRC | 07:10 | |
| *** ebalduf has quit IRC | 07:11 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: CloudByte: resize volume if cloned image is larger https://review.openstack.org/293412 | 07:14 |
| openstackgerrit | Bhagyashri Shewale proposed openstack/python-cinderclient: Fix volume type 'is_public' flag updation https://review.openstack.org/286382 | 07:17 |
| *** asselin_ has quit IRC | 07:21 | |
| *** melwitt has quit IRC | 07:23 | |
| *** melwitt has joined #openstack-cinder | 07:25 | |
| *** melwitt is now known as Guest17033 | 07:25 | |
| *** ollie_ has joined #openstack-cinder | 07:28 | |
| *** apoorvad has joined #openstack-cinder | 07:35 | |
| *** apoorvad has quit IRC | 07:40 | |
| *** goretoxo has joined #openstack-cinder | 07:40 | |
| *** lpetrut has joined #openstack-cinder | 07:44 | |
| openstackgerrit | aditi sharma proposed openstack/python-cinderclient: Add missing user message https://review.openstack.org/269548 | 07:49 |
| *** andymaier has joined #openstack-cinder | 07:50 | |
| *** gcb has quit IRC | 07:52 | |
| *** ndipanov has joined #openstack-cinder | 07:54 | |
| *** pece has joined #openstack-cinder | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:04 | |
| *** gcb has joined #openstack-cinder | 08:04 | |
| *** yangxi has joined #openstack-cinder | 08:05 | |
| *** sgotliv 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 |
| *** ankit_ag has joined #openstack-cinder | 08:11 | |
| *** jordanP has joined #openstack-cinder | 08:14 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 08:18 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 08:18 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 08:18 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 08:18 |
| *** jistr has joined #openstack-cinder | 08:18 | |
| *** pece has quit IRC | 08:21 | |
| *** dosaboy_ is now known as dosaboy | 08:24 | |
| *** arecknag has joined #openstack-cinder | 08:27 | |
| *** gcb has quit IRC | 08:33 | |
| *** sgotliv has quit IRC | 08:36 | |
| *** laughter_ has joined #openstack-cinder | 08:38 | |
| *** laughterwym has quit IRC | 08:40 | |
| *** yhayashi has quit IRC | 08:42 | |
| *** laughterwym has joined #openstack-cinder | 08:43 | |
| *** laughter_ has quit IRC | 08:43 | |
| *** adisky has joined #openstack-cinder | 08:43 | |
| *** laughterwym has quit IRC | 08:47 | |
| *** gcb has joined #openstack-cinder | 08:50 | |
| *** iceyao_ has quit IRC | 08:53 | |
| *** iceyao has joined #openstack-cinder | 08:54 | |
| *** david-lyle has quit IRC | 08:59 | |
| *** david-lyle has joined #openstack-cinder | 09:00 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix incorrent behaviour of encrypted-volume-type https://review.openstack.org/297934 | 09:01 |
| *** Poornima has quit IRC | 09:08 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: Hitachi: extend volume if larger than snapshot https://review.openstack.org/296083 | 09:09 |
| *** Poornima has joined #openstack-cinder | 09:09 | |
| *** Poornima has quit IRC | 09:10 | |
| *** Guest93067 has quit IRC | 09:11 | |
| *** Poornima has joined #openstack-cinder | 09:12 | |
| *** iceyao has quit IRC | 09:16 | |
| *** iceyao has joined #openstack-cinder | 09:16 | |
| *** sasukeh has quit IRC | 09:18 | |
| *** Julien-zte has joined #openstack-cinder | 09:18 | |
| *** Guest93067 has joined #openstack-cinder | 09:23 | |
| *** apoorvad has joined #openstack-cinder | 09:37 | |
| *** Julien-zte has quit IRC | 09:39 | |
| *** ndipanov has quit IRC | 09:41 | |
| *** apoorvad has quit IRC | 09:42 | |
| *** Guest93067 has quit IRC | 09:42 | |
| *** ndipanov has joined #openstack-cinder | 09:43 | |
| openstackgerrit | Xu Ao proposed openstack/cinder: Delete a redundant line in the code https://review.openstack.org/308233 | 09:47 |
| *** iceyao has quit IRC | 09:49 | |
| *** iceyao has joined #openstack-cinder | 09:49 | |
| *** sgotliv has joined #openstack-cinder | 09:52 | |
| *** gcb has quit IRC | 09:56 | |
| *** laughterwym has joined #openstack-cinder | 09:58 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** laughterwym has quit IRC | 10:03 | |
| *** laughterwym has joined #openstack-cinder | 10:07 | |
| *** sdague has joined #openstack-cinder | 10:08 | |
| *** gcb has joined #openstack-cinder | 10:09 | |
| openstackgerrit | Petrut Lucian proposed openstack/os-brick: Add Windows connectors https://review.openstack.org/272522 | 10:11 |
| *** EinstCra_ has quit IRC | 10:13 | |
| *** Julien-zte has joined #openstack-cinder | 10:19 | |
| *** laughterwym has quit IRC | 10:22 | |
| *** iceyao has quit IRC | 10:25 | |
| *** bjornar has quit IRC | 10:31 | |
| *** ociuhandu has quit IRC | 10:51 | |
| *** smoriya has quit IRC | 10:54 | |
| *** smoriya_afk is now known as smoriya | 10:54 | |
| *** smoriya has quit IRC | 10:55 | |
| *** jordanP has quit IRC | 10:55 | |
| *** lprice1 has joined #openstack-cinder | 10:56 | |
| *** lprice has quit IRC | 10:56 | |
| *** LZ has joined #openstack-cinder | 10:59 | |
| *** adisky has quit IRC | 11:00 | |
| *** smoriya_afk has joined #openstack-cinder | 11:04 | |
| *** smoriya_afk is now known as smoriya | 11:04 | |
| *** smoriya has quit IRC | 11:05 | |
| *** LZ has quit IRC | 11:07 | |
| *** smoriya_afk has joined #openstack-cinder | 11:07 | |
| *** smoriya_afk is now known as smoriya | 11:07 | |
| *** cfriesen_ has joined #openstack-cinder | 11:07 | |
| *** LZ has joined #openstack-cinder | 11:08 | |
| sheel | scottda: DuncanT: Hi | 11:08 |
| sheel | its ready for review https://review.openstack.org/#/c/278605/ is ready for review after changes + Green color support from jenkins :) | 11:08 |
| *** cfriesen has quit IRC | 11:08 | |
| *** cfriesen__ has joined #openstack-cinder | 11:12 | |
| *** cfriesen_ has quit IRC | 11:12 | |
| DuncanT | sheel: +2 from me | 11:12 |
| sheel | DuncanT: yey .... | 11:13 |
| sheel | 👍 | 11:13 |
| DuncanT | sheel: I'm pretty sure Scott will be +1, he'll be on in an hour or two | 11:13 |
| sheel | DuncanT: yep... thanks | 11:13 |
| *** ociuhandu has joined #openstack-cinder | 11:14 | |
| sheel | DuncanT: https://review.openstack.org/#/c/273938/ got many +2s but still waiting for +A... | 11:15 |
| sheel | :) | 11:15 |
| DuncanT | sheel: quite possible waiting for me. I'll look now | 11:16 |
| *** yangxi has quit IRC | 11:16 | |
| sheel | DuncanT: yes... :) | 11:16 |
| DuncanT | sheel: Done. Are you coming to Austin? | 11:18 |
| sheel | DuncanT: no, its not possible this time for me... though wanted to join .. | 11:19 |
| DuncanT | :-( Ok | 11:19 |
| sheel | will catch you guys next time... :) | 11:19 |
| DuncanT | Take a look at the program and feel free to ping me if you have strong thoughts on any of the topics | 11:20 |
| sheel | ah sure, I will go through it | 11:20 |
| sheel | and update you if find something to share | 11:20 |
| sheel | any specific link you wanted to point to? | 11:21 |
| openstackgerrit | Merged openstack/cinder-specs: User facing error Summary Messages https://review.openstack.org/273938 | 11:23 |
| sheel | DuncanT: ok, got your point... :) | 11:24 |
| *** haomaiwang has joined #openstack-cinder | 11:27 | |
| *** bjornar has joined #openstack-cinder | 11:39 | |
| *** apoorvad has joined #openstack-cinder | 11:39 | |
| *** nkrinner has quit IRC | 11:44 | |
| *** apoorvad has quit IRC | 11:44 | |
| *** iceyao has joined #openstack-cinder | 11:44 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Allow getting node summary info for services https://review.openstack.org/304234 | 11:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add service node commands https://review.openstack.org/304235 | 11:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add service node cleanup command https://review.openstack.org/304236 | 11:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add Backup to cleanable resource types https://review.openstack.org/304237 | 11:46 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add service node auto-cleanup https://review.openstack.org/304238 | 11:46 |
| *** nkrinner has joined #openstack-cinder | 11:50 | |
| *** nkrinner has quit IRC | 11:55 | |
| *** laughterwym has joined #openstack-cinder | 11:55 | |
| *** nkrinner has joined #openstack-cinder | 11:56 | |
| *** stacker has joined #openstack-cinder | 11:56 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** Guest93067 has joined #openstack-cinder | 12:02 | |
| *** jaosorior has joined #openstack-cinder | 12:08 | |
| *** raildo-afk is now known as raildo | 12:10 | |
| *** cfriesen has joined #openstack-cinder | 12:11 | |
| *** cfriesen__ has quit IRC | 12:11 | |
| *** diablo_rojo has joined #openstack-cinder | 12:14 | |
| *** crose has joined #openstack-cinder | 12:14 | |
| *** cfriesen_ has joined #openstack-cinder | 12:15 | |
| *** cfriesen has quit IRC | 12:16 | |
| *** cdelatte has joined #openstack-cinder | 12:17 | |
| *** markvoelker has joined #openstack-cinder | 12:18 | |
| *** eharney has quit IRC | 12:19 | |
| *** Poornima has quit IRC | 12:30 | |
| *** laughterwym has quit IRC | 12:31 | |
| *** laughterwym has joined #openstack-cinder | 12:32 | |
| *** laughterwym has quit IRC | 12:36 | |
| *** laughterwym has joined #openstack-cinder | 12:38 | |
| *** cfriesen__ has joined #openstack-cinder | 12:38 | |
| *** cfriesen_ has quit IRC | 12:39 | |
| *** laughter_ has joined #openstack-cinder | 12:41 | |
| *** cfriesen has joined #openstack-cinder | 12:43 | |
| *** cfriesen__ has quit IRC | 12:43 | |
| *** laughterwym has quit IRC | 12:44 | |
| *** bjornar has quit IRC | 12:44 | |
| *** links has quit IRC | 12:46 | |
| *** dave-mccowan has joined #openstack-cinder | 12:46 | |
| *** porrua has joined #openstack-cinder | 12:47 | |
| *** salv-orlando has joined #openstack-cinder | 12:48 | |
| *** laughterwym has joined #openstack-cinder | 12:56 | |
| *** Poornima has joined #openstack-cinder | 12:57 | |
| *** bjornar has joined #openstack-cinder | 12:58 | |
| *** rlrossit has joined #openstack-cinder | 12:59 | |
| *** laughter_ has quit IRC | 12:59 | |
| *** ankit_ag has quit IRC | 13:01 | |
| *** Guest93067 has quit IRC | 13:01 | |
| *** Poornima has quit IRC | 13:02 | |
| *** aimeeU has joined #openstack-cinder | 13:02 | |
| *** houming has quit IRC | 13:03 | |
| *** laughterwym has quit IRC | 13:04 | |
| *** laughterwym has joined #openstack-cinder | 13:05 | |
| *** laughterwym has quit IRC | 13:05 | |
| *** laughterwym has joined #openstack-cinder | 13:05 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Check flashcopy mapping before deleting volume https://review.openstack.org/268701 | 13:07 |
| *** cfriesen_ has joined #openstack-cinder | 13:07 | |
| *** cfriesen has quit IRC | 13:08 | |
| *** cfriesen__ has joined #openstack-cinder | 13:11 | |
| *** cfriesen_ has quit IRC | 13:12 | |
| *** diablo_rojo1 has joined #openstack-cinder | 13:16 | |
| *** asselin_ has joined #openstack-cinder | 13:17 | |
| *** dansmith has quit IRC | 13:17 | |
| *** ntpttr has quit IRC | 13:18 | |
| *** Guest93067 has joined #openstack-cinder | 13:18 | |
| *** diablo_rojo has quit IRC | 13:19 | |
| *** mjura has quit IRC | 13:19 | |
| *** dansmith has joined #openstack-cinder | 13:20 | |
| *** dansmith is now known as Guest64767 | 13:20 | |
| *** jaosorior has left #openstack-cinder | 13:21 | |
| *** dustins has joined #openstack-cinder | 13:21 | |
| *** xyang1 has joined #openstack-cinder | 13:23 | |
| *** diablo_rojo1 has quit IRC | 13:23 | |
| *** diablo_rojo has joined #openstack-cinder | 13:24 | |
| *** mriedem has joined #openstack-cinder | 13:25 | |
| *** asselin_ has quit IRC | 13:26 | |
| *** akerr has joined #openstack-cinder | 13:28 | |
| *** yangyapeng has joined #openstack-cinder | 13:28 | |
| *** jordanP has joined #openstack-cinder | 13:31 | |
| *** haomaiwang has joined #openstack-cinder | 13:31 | |
| *** mjura has joined #openstack-cinder | 13:34 | |
| *** crose has quit IRC | 13:35 | |
| *** crose has joined #openstack-cinder | 13:36 | |
| *** esker has quit IRC | 13:37 | |
| *** edmondsw has joined #openstack-cinder | 13:38 | |
| *** apoorvad has joined #openstack-cinder | 13:41 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 13:43 |
| *** haomaiwang has quit IRC | 13:44 | |
| *** apoorvad has quit IRC | 13:46 | |
| *** cdelatte has quit IRC | 13:48 | |
| sheel | scottda: hi | 13:49 |
| *** ebalduf has joined #openstack-cinder | 13:49 | |
| scottda | sheel: hi | 13:50 |
| sheel | scottda: regarding patch for v3/volumes | 13:50 |
| sheel | are we goind to add only some part of code or going to include all code of v2 into v3? | 13:51 |
| sheel | https://review.openstack.org/#/c/147738/25/cinder/api/v3/volumes.py | 13:52 |
| *** links has joined #openstack-cinder | 13:52 | |
| scottda | I posted a link in you're review that shows how to only move the minimum amount of code... | 13:52 |
| sheel | scottda: yep i saw that | 13:53 |
| sheel | scottda: so should i wait for this patch to merge or can go ahead with changes on my end in v3/volumes | 13:53 |
| sheel | ? | 13:53 |
| scottda | So, that code I wrote only moves part. You don't need to wait for it to merge, but if you use the same common code is will merge better, i.e. | 13:54 |
| scottda | class VolumeController(volumes_v2.VolumeController, wsgi.Controller): | 13:54 |
| scottda | https://www.irccloud.com/pastebin/3Kr58LoV/ | 13:54 |
| scottda | etc etc | 13:54 |
| scottda | Wanghao' | 13:55 |
| scottda | Wanghao's patch changes get_volumes. You don't need that part, just what you're patch changes. | 13:56 |
| sheel | my changes are also in get_volumes | 13:56 |
| sheel | :) | 13:56 |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Newton specs to the index.rst https://review.openstack.org/308344 | 13:57 |
| sheel | so may be I can use this code and go ahead with my changes | 13:57 |
| smcginnis | scottda: Side note on that doc update patch - you responded done but looks like you forgot to push up a new patchset. | 13:57 |
| smcginnis | dulek: Probably your quickest patch ever. ;) | 13:58 |
| *** jgregor has joined #openstack-cinder | 13:58 | |
| *** crose has quit IRC | 13:59 | |
| dulek | smcginnis: Might be. :D I like to read specs rendered, that's why I've noticed. | 14:00 |
| smcginnis | dulek: Yeah, definitely. Didn't want to link it when I created the directory because I thought that would be too soon and confusing, but then forgot to go back and fix it. Thanks for catching that! | 14:00 |
| scottda | smcginnis: Yeah, I had it ready, then during 'git review' there was a merge disaster.... | 14:00 |
| scottda | smcginnis: And then I went to cook tacos | 14:01 |
| smcginnis | scottda: Hah, I get that. :) | 14:01 |
| scottda | smcginnis: So, had I known, I would have replied "Will do" instead of "Done" | 14:01 |
| scottda | smcginnis: I'll fix later. I assume there's no great hurry? | 14:01 |
| smcginnis | scottda: Eh, no rush on it, so I just wanted to point it out just in case you forgot. | 14:01 |
| scottda | smcginnis: Was that why you pinged me yesterday PM? Or something else? | 14:02 |
| smcginnis | scottda: Yeah, that was it. | 14:02 |
| scottda | smcginnis: Cool, thanks. | 14:02 |
| *** cdelatte has joined #openstack-cinder | 14:02 | |
| smcginnis | scottda: Thanks for doing it! | 14:04 |
| *** jwcroppe_ is now known as jwcroppe | 14:04 | |
| *** cknight has joined #openstack-cinder | 14:05 | |
| *** karthikp has joined #openstack-cinder | 14:07 | |
| *** ametts has joined #openstack-cinder | 14:11 | |
| *** sasukeh has joined #openstack-cinder | 14:17 | |
| *** sasukeh has quit IRC | 14:18 | |
| *** karthikp has quit IRC | 14:23 | |
| *** mjura has quit IRC | 14:25 | |
| *** avishay has quit IRC | 14:28 | |
| *** merooney has joined #openstack-cinder | 14:28 | |
| *** sasukeh has joined #openstack-cinder | 14:29 | |
| *** esker has joined #openstack-cinder | 14:29 | |
| *** esker has quit IRC | 14:29 | |
| *** esker has joined #openstack-cinder | 14:30 | |
| *** mylu has joined #openstack-cinder | 14:31 | |
| openstackgerrit | Merged openstack/cinder-specs: Add Newton specs to the index.rst https://review.openstack.org/308344 | 14:31 |
| *** arch-nemesis has joined #openstack-cinder | 14:33 | |
| *** arch-nemesis has quit IRC | 14:33 | |
| *** arch-nemesis has joined #openstack-cinder | 14:34 | |
| *** timcl has quit IRC | 14:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/308369 | 14:35 |
| *** mjura has joined #openstack-cinder | 14:38 | |
| *** jwcroppe has quit IRC | 14:39 | |
| *** jwcroppe has joined #openstack-cinder | 14:41 | |
| *** sasukeh has quit IRC | 14:41 | |
| dulek | yuriy_n17: Hi, can you elaborate why rename is needed in https://review.openstack.org/#/c/307342 ? | 14:42 |
| *** sdake_ has quit IRC | 14:42 | |
| dulek | yuriy_n17: I thought that unmanage by default will just remove a volume from Cinder without touching the backend. Why isn't it enough for RBD? | 14:43 |
| *** apoorvad has joined #openstack-cinder | 14:43 | |
| *** timcl has joined #openstack-cinder | 14:45 | |
| *** sdake_ has joined #openstack-cinder | 14:45 | |
| yuriy_n17 | dulek: Hi! When we unmanage volume for RBD that volume'll be deleted in DB but on Ceph pool the image exists. | 14:46 |
| dulek | yuriy_n17: Sure, so why rename it? Can't we just let it exist without rename? | 14:46 |
| *** apoorvad has quit IRC | 14:47 | |
| yuriy_n17 | dulek: Please look http://paste.openstack.org/show/494545/ | 14:47 |
| *** ircuser-1 has quit IRC | 14:48 | |
| yuriy_n17 | dulek: How you suggest to leave it without renaming? | 14:48 |
| dulek | yuriy_n17: Rename is done to prevent unmanaging volume two times? | 14:48 |
| *** laughterwym has quit IRC | 14:49 | |
| *** laughterwym has joined #openstack-cinder | 14:49 | |
| yuriy_n17 | dulek: Yes | 14:50 |
| dulek | yuriy_n17: Then the error you're showing isn't from the driver by from the API not being able to find the volume in the DB - because it was deleted. | 14:50 |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Add docs for running tests https://review.openstack.org/296000 | 14:50 |
| *** apoorvad has joined #openstack-cinder | 14:51 | |
| dulek | yuriy_n17: And how do you manage the renamed volume back into Cinder? | 14:51 |
| *** haomaiwang has joined #openstack-cinder | 14:52 | |
| yuriy_n17 | dulek: cinder manage <host> unmanaged-volume-.... | 14:54 |
| *** laughterwym has quit IRC | 14:54 | |
| *** apoorvad has quit IRC | 14:55 | |
| yuriy_n17 | dulek: Look at http://paste.openstack.org/show/494545/ RBD knows it. | 14:55 |
| dulek | yuriy_n17: I understand that paste. But why Cinder cares how it's called in RBD after unmanage? What problem is rename solving? | 14:56 |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient: Use six.moves.urllib.parse urlencode https://review.openstack.org/302961 | 14:56 |
| *** karthikp has joined #openstack-cinder | 14:57 | |
| *** garthb has joined #openstack-cinder | 14:58 | |
| yuriy_n17 | dulek: Well, how you propose to do it. | 14:58 |
| dulek | yuriy_n17: Leave it as it is now. I don't understand what's the problem with current implementation. | 14:59 |
| *** Julien-zte has quit IRC | 14:59 | |
| DuncanT | yuriy_n17: You can't unmanage it unless there's record in the cinder DB, so you can't unmanage twice | 14:59 |
| *** Guest64767 is now known as dansmith | 15:00 | |
| DuncanT | yuriy_n17: With or without your change, the result of calling unmanage twice on the same volume is an error | 15:00 |
| *** wxy has quit IRC | 15:00 | |
| *** gouthamr has joined #openstack-cinder | 15:01 | |
| DuncanT | scottda: Now I want tacos. All your fault. Or burritos. They'd be good too | 15:01 |
| *** baumann has joined #openstack-cinder | 15:01 | |
| smcginnis | DuncanT: How's the TexMex dining in Isreal? :D | 15:01 |
| scottda | DuncanT: Well, the good news is you're going to Austin next week | 15:01 |
| DuncanT | smcginnis: None existent near as | 15:02 |
| DuncanT | scottda: But I'm hungry now | 15:02 |
| *** mjura has quit IRC | 15:04 | |
| *** asselin_ has joined #openstack-cinder | 15:05 | |
| *** merooney has quit IRC | 15:05 | |
| *** amit213 has quit IRC | 15:06 | |
| *** amit213 has joined #openstack-cinder | 15:08 | |
| *** haomaiwang has quit IRC | 15:09 | |
| *** laughterwym has joined #openstack-cinder | 15:09 | |
| *** mylu has quit IRC | 15:10 | |
| *** mylu has joined #openstack-cinder | 15:11 | |
| *** nkrinner has quit IRC | 15:11 | |
| *** crose has joined #openstack-cinder | 15:11 | |
| *** rajinir has joined #openstack-cinder | 15:12 | |
| *** Guest93067 has quit IRC | 15:13 | |
| *** mylu has quit IRC | 15:14 | |
| *** mylu has joined #openstack-cinder | 15:15 | |
| *** cdelatte has quit IRC | 15:15 | |
| *** lpetrut has quit IRC | 15:16 | |
| dulek | Hm, no agenda for today's meeting. Had we finalized planning of Cinder Summit dinner? | 15:16 |
| *** baumann1 has joined #openstack-cinder | 15:16 | |
| *** cdelatte has joined #openstack-cinder | 15:17 | |
| *** lpetrut has joined #openstack-cinder | 15:17 | |
| openstackgerrit | Merged openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 15:18 |
| *** crose has quit IRC | 15:18 | |
| *** e0ne has joined #openstack-cinder | 15:18 | |
| *** baumann has quit IRC | 15:19 | |
| *** lprice has joined #openstack-cinder | 15:19 | |
| smcginnis | dulek: No, not yet. | 15:20 |
| smcginnis | Probably a quick meeting (knocks on formica) | 15:20 |
| *** jordanP has quit IRC | 15:21 | |
| *** lprice1 has quit IRC | 15:22 | |
| *** haomaiwang has joined #openstack-cinder | 15:22 | |
| *** mylu_ has joined #openstack-cinder | 15:22 | |
| *** mylu has quit IRC | 15:23 | |
| DuncanT | smcginnis: I'm sure I can find some points of contention if you want :-) | 15:24 |
| smcginnis | DuncanT: Hah, I have no doubt about that. :) | 15:24 |
| *** salv-orlando has quit IRC | 15:26 | |
| *** dustins has quit IRC | 15:27 | |
| *** esker has quit IRC | 15:28 | |
| *** esker has joined #openstack-cinder | 15:28 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 15:28 |
| hemna_ | mriedem, I was going to test it again, and I was hoping others would actually pull it down and test it as well (os-brick privsep patch) | 15:31 |
| *** links has quit IRC | 15:32 | |
| hemna_ | mriedem, heh nevermind. looks like it merged. | 15:35 |
| *** esker has quit IRC | 15:36 | |
| smcginnis | hemna_: Yeah, I finally got time to pull it down and run with it. | 15:36 |
| hemna_ | smcginnis, ok cool thanks for actually testing it. | 15:36 |
| hemna_ | I don't think anyone else had. | 15:36 |
| *** haomaiwang has quit IRC | 15:36 | |
| smcginnis | hemna_: Yeah, I didn't want to let something like that through without at least firing it up. ;) | 15:36 |
| hemna_ | I did test it out quite a bit before mriedem's bump | 15:36 |
| hemna_ | man that's great that landed. I have a bunch of refactoring to do now | 15:37 |
| smcginnis | hemna_: Hah | 15:37 |
| hemna_ | my lvm work needed that stuff to land | 15:37 |
| hemna_ | as it includes lots of execute calls | 15:37 |
| smcginnis | hemna_: It is good to get that in there now I think. The more time we have to work out issues the better, IMO. | 15:37 |
| hemna_ | yup | 15:38 |
| smcginnis | hemna_: The hard coded full paths were the only thing that gave me pause. | 15:38 |
| hemna_ | the emc connector ? | 15:38 |
| smcginnis | Really worried if everything we need to call will be the same between RH, Ubuntu, etc. | 15:38 |
| smcginnis | Not really that one, but the other full paths. | 15:38 |
| smcginnis | Maybe it's not an issue. | 15:38 |
| smcginnis | But now we can find out for sure. | 15:38 |
| hemna_ | I guess the only way to fix that is for the EMC driver to pass in the path in the connection_info | 15:39 |
| hemna_ | :( | 15:39 |
| hemna_ | because there is no conf for brick at all, on purpose | 15:39 |
| smcginnis | hemna_: I thought I saw other full paths in there. Maybe not an issue. | 15:39 |
| *** mriedem is now known as mriedem_haircut | 15:39 | |
| hemna_ | The ScaleIO driver can add a new cinder.conf entry that gets passed through in connection_info, that the connector can use. | 15:39 |
| xyang1 | hemna_: why only emc driver? | 15:39 |
| hemna_ | I just was commenting on the ScaleIO connector | 15:40 |
| hemna_ | that patrickeast commented on in the review. | 15:40 |
| xyang1 | hemna_: oh, ok | 15:40 |
| hemna_ | and trying to find a way to solve the path prefix issue | 15:40 |
| smcginnis | xyang1: Do you know if that package always installs to the same location, regardless of distro? | 15:41 |
| DuncanT | hemna: Is there a conf for privsep? | 15:41 |
| xyang1 | hemna_: I remember scaleio has full path, but I thought that is not the only one | 15:41 |
| smcginnis | I don't think it's the only one. | 15:41 |
| hemna_ | I'm not aware of other hard coded paths in connectors | 15:41 |
| * smcginnis takes a look | 15:41 | |
| xyang1 | smcginnis: I believe so | 15:41 |
| hemna_ | maybe scsi_id app ? | 15:41 |
| hemna_ | which is a udev app installed in /lib/udev | 15:42 |
| * hemna_ hopes that's the same place for RH, Ubuntu, etc. | 15:42 | |
| smcginnis | hemna_: That sounds familiar. I think that's the one I was concerned about. | 15:42 |
| xyang1 | hemna_: what is the review number? | 15:42 |
| hemna_ | I don't use RH so I'm not sure | 15:42 |
| hemna_ | https://review.openstack.org/#/c/277224 | 15:43 |
| *** ircuser-1 has joined #openstack-cinder | 15:43 | |
| smcginnis | xyang1: Here's the commit: https://github.com/openstack/os-brick/commit/dbf77fba1061cb4e93b3db5f8117d6ccc689f702 | 15:43 |
| *** cdelatte has quit IRC | 15:43 | |
| *** diablo_rojo has quit IRC | 15:44 | |
| *** mylu_ has quit IRC | 15:44 | |
| xyang1 | smcginnis: thanks | 15:44 |
| *** mylu has joined #openstack-cinder | 15:44 | |
| smcginnis | scsi_id might be under /sbin on RH. I should probably spin up an instance and see. | 15:45 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 15:47 |
| *** jungleboyj has joined #openstack-cinder | 15:47 | |
| openstackgerrit | Merged openstack/cinder: NexentaStor: Remove extra format string specifier https://review.openstack.org/296344 | 15:47 |
| *** mylu has quit IRC | 15:48 | |
| *** leeantho has joined #openstack-cinder | 15:49 | |
| *** diablo_rojo has joined #openstack-cinder | 15:49 | |
| *** cdelatte has joined #openstack-cinder | 15:50 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix coverage generation https://review.openstack.org/296107 | 15:52 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 15:53 |
| *** mylu has joined #openstack-cinder | 15:53 | |
| *** jgregor has left #openstack-cinder | 15:55 | |
| *** jgregor has joined #openstack-cinder | 15:55 | |
| *** xyang1 has quit IRC | 15:56 | |
| *** raildo is now known as raildo-afk | 15:56 | |
| *** mmotiani_ has joined #openstack-cinder | 15:57 | |
| *** mmotiani_ has quit IRC | 15:57 | |
| *** mylu has quit IRC | 15:57 | |
| *** salv-orlando has joined #openstack-cinder | 15:58 | |
| *** xyang1 has joined #openstack-cinder | 15:58 | |
| *** mylu has joined #openstack-cinder | 15:58 | |
| *** fernnest has joined #openstack-cinder | 15:59 | |
| *** mmotiani_ has joined #openstack-cinder | 15:59 | |
| *** cdelatte has quit IRC | 16:10 | |
| *** cdelatte has joined #openstack-cinder | 16:12 | |
| *** cdelatte has quit IRC | 16:14 | |
| *** cdelatte has joined #openstack-cinder | 16:16 | |
| *** ChubYann has joined #openstack-cinder | 16:17 | |
| *** haplo37 has joined #openstack-cinder | 16:18 | |
| *** merooney has joined #openstack-cinder | 16:18 | |
| *** apoorvad has joined #openstack-cinder | 16:21 | |
| *** apoorvad has quit IRC | 16:22 | |
| *** jistr has quit IRC | 16:22 | |
| *** apoorvad has joined #openstack-cinder | 16:22 | |
| *** gnarld_ is now known as cFouts | 16:26 | |
| scottda | dulek: Glad to hear you know about SRV. | 16:27 |
| *** erlon_ has joined #openstack-cinder | 16:29 | |
| *** arch-nemesis has quit IRC | 16:29 | |
| *** goretoxo has quit IRC | 16:31 | |
| *** cdelatte has quit IRC | 16:31 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 16:32 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 16:32 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots of in-use volumes https://review.openstack.org/276465 | 16:32 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 16:32 |
| *** merooney has quit IRC | 16:32 | |
| *** Yogi1 has joined #openstack-cinder | 16:33 | |
| *** iceyao has quit IRC | 16:33 | |
| *** dustins has joined #openstack-cinder | 16:37 | |
| *** laughterwym has quit IRC | 16:37 | |
| *** laughterwym has joined #openstack-cinder | 16:38 | |
| *** merooney has joined #openstack-cinder | 16:38 | |
| *** mylu has quit IRC | 16:40 | |
| *** raildo-afk is now known as raildo | 16:41 | |
| *** Guest8256 has quit IRC | 16:42 | |
| *** laughterwym has quit IRC | 16:42 | |
| *** wilson has joined #openstack-cinder | 16:43 | |
| *** wilson is now known as Guest47552 | 16:43 | |
| *** arch-nemesis has joined #openstack-cinder | 16:43 | |
| *** yuriy_n17 has quit IRC | 16:44 | |
| *** yuriy_n17 has joined #openstack-cinder | 16:45 | |
| *** david-nesher has joined #openstack-cinder | 16:45 | |
| *** merooney has quit IRC | 16:45 | |
| *** mriedem_haircut is now known as mriedem | 16:46 | |
| *** Guest67082 has quit IRC | 16:46 | |
| *** zul has quit IRC | 16:48 | |
| *** ivase has joined #openstack-cinder | 16:49 | |
| *** merooney has joined #openstack-cinder | 16:49 | |
| *** cdelatte has joined #openstack-cinder | 16:50 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: set_metadata throws AttributeError: id https://review.openstack.org/308475 | 16:50 |
| *** mylu has joined #openstack-cinder | 16:52 | |
| *** chuck__ has joined #openstack-cinder | 16:54 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Add docs for running tests https://review.openstack.org/296000 | 16:54 |
| thingee | smcginnis: was confused by this patch. wouldn't we want to resize if the volume is smaller than the snapshot? https://review.openstack.org/#/c/296446/2/cinder/volume/drivers/datera.py | 16:55 |
| *** andymaier has quit IRC | 16:57 | |
| patrickeast | thingee: yea that looks backwards | 16:58 |
| thingee | patrickeast: ok was just making sure I wasn't crazy :) | 16:58 |
| thingee | well crazy is another discussion | 16:59 |
| patrickeast | haha | 16:59 |
| thingee | jungleboyj: I was looking at https://review.openstack.org/#/c/307246/1 ... but not seeing that reflect in the global requirements ... http://git.openstack.org/cgit/openstack/requirements/tree/requirements.txt?h=stable/mitaka | 17:03 |
| openstackgerrit | Merged openstack/cinder: Changed Nexenta default chunksize from 16k to 32k https://review.openstack.org/281510 | 17:03 |
| jungleboyj | thingee: Hmmm, that is odd. | 17:05 |
| thingee | jungleboyj: whoops wrong file ... that's requirements not global-requirement | 17:05 |
| thingee | confusing | 17:05 |
| jungleboyj | thingee: There, what is in global-requirements matches up. :-) | 17:06 |
| thingee | patrickeast, hemna_ and another https://review.openstack.org/#/c/296371/5 | 17:08 |
| patrickeast | oooh wait | 17:09 |
| patrickeast | thingee: hang on | 17:09 |
| openstackgerrit | Merged openstack/cinder: Deprecate nas_ip and change this to nas_host https://review.openstack.org/302207 | 17:09 |
| patrickeast | thingee: i think we might be mixing something up here | 17:10 |
| openstackgerrit | Merged openstack/cinder: Hacking: Ignore tools/ for C303 https://review.openstack.org/301346 | 17:10 |
| patrickeast | thingee: the bug was that the volume we asked for (with a source id) was larger than the source snapshot, in which case those drivers are extending the clone | 17:10 |
| patrickeast | thingee: so they look at volume['size'] which is the requested one, and snapshot['volume_size'] which is the cloned volume size, and extend the cloned volume if volume['size'] is greater | 17:11 |
| thingee | patrickeast: eh? I see extend the volume by it's current size. | 17:14 |
| * thingee looks closer | 17:14 | |
| *** mtanino has joined #openstack-cinder | 17:14 | |
| patrickeast | thingee: hah, yea the naming is making this hard :( volume is the one we are requesting the driver create | 17:14 |
| patrickeast | thingee: so if the requested size is greater than the actual size (in those drivers case, the size of the source volume for the snapshot) it needs to be extended to its correct size | 17:15 |
| patrickeast | thingee: so like snapshot of a 5G volume, then do a cinder create --snapshot-id foo 10 (or whatever the command is) | 17:16 |
| patrickeast | thingee: the drivers clone the source snapshot and gets a 5G volume | 17:16 |
| patrickeast | thingee: but then decides if it needs to correct the size for the requested "volume" | 17:16 |
| thingee | patrickeast: ah I get it now | 17:16 |
| openstackgerrit | Alex Meade proposed openstack/cinder: init_host offload to default to true https://review.openstack.org/306402 | 17:17 |
| thingee | patrickeast: it seems like instead of fixing individual drivers, this should just be fixed in the taskflow volume manager stuff? | 17:18 |
| thingee | smcginnis: ^ | 17:19 |
| *** Guest17033 is now known as melwitt | 17:19 | |
| patrickeast | thingee: yea i would think so, i guess maybe its because some drivers do it all in one go | 17:19 |
| patrickeast | thingee: while some (like those and mine) do it with two calls to clone and then extend | 17:19 |
| thingee | patrickeast: would you be opposed to it being done in that layer and removing that from your driver? | 17:20 |
| patrickeast | thingee: nah that works for me | 17:20 |
| patrickeast | thingee: simplifies the driver code, always a win | 17:20 |
| patrickeast | well | 17:21 |
| patrickeast | usually almost always a win | 17:21 |
| *** haomaiwang has joined #openstack-cinder | 17:21 | |
| thingee | patrickeast: somewhere in here after the driver create volume from snapshot call http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/flows/manager/create_volume.py#n439 | 17:23 |
| patrickeast | thingee: yea that would work | 17:23 |
| patrickeast | thingee: actually reminds me of the conversation yesterday about potentially letting drivers return back a model update of what size the volumes were actually created at | 17:24 |
| patrickeast | thingee: i forget which driver it was, but they could only do like 5G granularity or something | 17:24 |
| patrickeast | thingee: same kind of thing could work here, we just peek at the new model and if the size isn't big enough do the resize call | 17:24 |
| patrickeast | s/resize/extend/ | 17:25 |
| *** merooney has quit IRC | 17:26 | |
| *** beagles has joined #openstack-cinder | 17:32 | |
| *** dustins has quit IRC | 17:33 | |
| *** merooney has joined #openstack-cinder | 17:33 | |
| beagles | is there a cinder configuration that controls how long things can stay "creating" in the absence of an accessible cinder-volume service? | 17:36 |
| beagles | I'm trying to accelerate detection of configuration failures and found a situation where the cinder environment was not fully configured so some of the services were running but cinder-volume was not | 17:37 |
| beagles | (outside of polling the host the services are running on that is ;)) | 17:37 |
| *** bardia has joined #openstack-cinder | 17:38 | |
| *** mriedem has quit IRC | 17:38 | |
| *** mriedem has joined #openstack-cinder | 17:38 | |
| *** sdake__ has joined #openstack-cinder | 17:41 | |
| *** sdake__ has quit IRC | 17:43 | |
| *** sdake_ has quit IRC | 17:43 | |
| *** Suyi has joined #openstack-cinder | 17:45 | |
| *** sdake_ has joined #openstack-cinder | 17:45 | |
| *** merooney has quit IRC | 17:45 | |
| *** Yogi1 has quit IRC | 17:51 | |
| *** sdake__ has joined #openstack-cinder | 17:53 | |
| *** sdake_ has quit IRC | 17:57 | |
| *** bjornar has quit IRC | 17:58 | |
| *** esker has joined #openstack-cinder | 17:59 | |
| patrickeast | beagles: i don't know of any sort of global setting for that, there are a bunch of different timeouts for different things (depending on what backend, what operation, etc) | 17:59 |
| patrickeast | beagles: your best bet is probably to query the service status api | 17:59 |
| beagles | patrickeast: ack thanks | 17:59 |
| guitarzan | and, it can stay inside the driver's create call for as long as the driver wants | 18:00 |
| guitarzan | there is no timeout | 18:00 |
| patrickeast | yep | 18:00 |
| *** jwcroppe_ has joined #openstack-cinder | 18:02 | |
| scottda | beagles: In fact, if c-vol is down, you're create call will go to creating and stay there forever | 18:03 |
| beagles | scottda: yeah, that's what I'm seeing | 18:03 |
| scottda | That's kinda bad. | 18:04 |
| patrickeast | mm yea | 18:04 |
| *** jwcroppe has quit IRC | 18:04 | |
| scottda | patrickeast: You going to fix that or should I? | 18:05 |
| scottda | :) | 18:05 |
| patrickeast | haha | 18:05 |
| patrickeast | scottda: if you have time go for it :D | 18:05 |
| scottda | I'll put it in the queue | 18:05 |
| *** ndipanov has quit IRC | 18:06 | |
| *** ChubYann has quit IRC | 18:07 | |
| openstackgerrit | Merged openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/302877 | 18:08 |
| *** lpetrut has quit IRC | 18:09 | |
| smcginnis | thingee, patrickeast: The issue, IIRC, is that there is not model update returned on those. So no mechanism to know if the new volume is at the requested size or not. | 18:11 |
| smcginnis | thingee, patrickeast: So no way to know if an extend is needed or not. | 18:12 |
| smcginnis | patrickeast: And like you mentioned, for some drivers it is more efficient to just create the clone at the new size right off the bat. | 18:12 |
| *** winston-d_ has joined #openstack-cinder | 18:13 | |
| jgriffith | beagles: patrickeast scottda if the service is down it should never get scheduled and go straight to error | 18:14 |
| jgriffith | beagles: patrickeast scottda once the scheduler doesn't get a host-update you should get the dreaded "no-valid host" | 18:14 |
| *** sdake__ has quit IRC | 18:14 | |
| patrickeast | jgriffith: thats what I was thinking, but it sounds like maybe its not working | 18:15 |
| jgriffith | patrickeast: hmm... beagles what backend/driver are you using? | 18:15 |
| winston-d_ | if you guys are talking about c-sch, there is a max 60s before schedule can error out down service | 18:16 |
| jgriffith | actually... scottda guess you have info on this one already | 18:16 |
| jgriffith | I'd be curious | 18:16 |
| jgriffith | winston-d_: +1 | 18:16 |
| beagles | jgriffith: it is the LVMVolumeDriver | 18:16 |
| jgriffith | beagles: ok, I'll check it out; when you say "down" do you mean you just disable the service? | 18:17 |
| jgriffith | beagles: or kill its process? | 18:17 |
| beagles | jgriffith: initially it was because there was a problem with the config on reboot so cinder-volume didn't come back up | 18:17 |
| beagles | jgriffith: I fixed that and things were going nicely | 18:17 |
| *** mylu has quit IRC | 18:18 | |
| beagles | jgriffith: but I was doing some "fail-faster" changes to our test scripts and stopped the cinder volume service with systemctl to simulate the previous situation | 18:18 |
| jgriffith | beagles: interesting | 18:18 |
| beagles | jgriffith: so that's when I started looking at what's going on. The API reports creating | 18:18 |
| winston-d_ | beagles: which API? create call? | 18:19 |
| *** garthb has quit IRC | 18:19 | |
| beagles | jgriffith: for like a good long while. the test itself times out before I get a failure. As soon as I start the volume service back up it it fails pretty quickly after that - that may be a useful data point | 18:20 |
| *** ChubYann has joined #openstack-cinder | 18:20 | |
| beagles | winston-d_: it's doing something like a "show" on the volume checking for a success/fail status | 18:20 |
| beagles | winston-d_: that's what it appears to be doing anyways from the cinder-api log | 18:20 |
| winston-d_ | ok, I need to check the code to see what state the volume is when it's scheduled. | 18:22 |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Cleanup for Volume Object https://review.openstack.org/300708 | 18:22 |
| *** mylu has joined #openstack-cinder | 18:25 | |
| *** avishay has joined #openstack-cinder | 18:27 | |
| *** mylu has quit IRC | 18:30 | |
| *** mylu has joined #openstack-cinder | 18:32 | |
| openstackgerrit | Kurt Martin proposed openstack/cinder: 3PAR: Remove metadata that tracks the instance id https://review.openstack.org/308518 | 18:34 |
| *** mylu has quit IRC | 18:35 | |
| *** haomaiwang has quit IRC | 18:35 | |
| *** mylu has joined #openstack-cinder | 18:36 | |
| *** haomaiwang has joined #openstack-cinder | 18:37 | |
| scottda | jgriffith: You are right, after 1 min the volume will go to error. I got it stuck in creating immediately after downing c-vol. | 18:37 |
| *** avishay has quit IRC | 18:38 | |
| scottda | that is, after 1 minute, and attempt to create a volume with c-vol down will result in the volume going to error. | 18:38 |
| *** mylu has quit IRC | 18:41 | |
| *** geguileo has quit IRC | 18:42 | |
| *** ociuhandu has quit IRC | 18:43 | |
| *** ociuhandu has joined #openstack-cinder | 18:43 | |
| *** mylu has joined #openstack-cinder | 18:45 | |
| *** akerr has quit IRC | 18:45 | |
| *** geguileo has joined #openstack-cinder | 18:50 | |
| *** garthb has joined #openstack-cinder | 18:51 | |
| *** e0ne has quit IRC | 18:55 | |
| *** rlrossit has quit IRC | 18:58 | |
| *** ociuhandu has quit IRC | 18:59 | |
| *** salv-orlando has quit IRC | 19:03 | |
| *** rcernin has quit IRC | 19:05 | |
| *** aimeeU has quit IRC | 19:06 | |
| *** mylu has quit IRC | 19:10 | |
| *** lpetrut has joined #openstack-cinder | 19:11 | |
| *** Suyi has quit IRC | 19:12 | |
| *** Suyi has joined #openstack-cinder | 19:12 | |
| *** sgotliv has quit IRC | 19:12 | |
| *** haomaiwang has quit IRC | 19:13 | |
| *** sheel has quit IRC | 19:15 | |
| *** ociuhandu has joined #openstack-cinder | 19:18 | |
| *** mylu has joined #openstack-cinder | 19:20 | |
| *** salv-orlando has joined #openstack-cinder | 19:26 | |
| *** haomaiwang has joined #openstack-cinder | 19:27 | |
| *** rlrossit has joined #openstack-cinder | 19:27 | |
| *** karthikp has quit IRC | 19:31 | |
| openstackgerrit | Merged openstack/os-brick: Fix coverage generation https://review.openstack.org/296107 | 19:31 |
| *** rcernin has joined #openstack-cinder | 19:32 | |
| *** Lee1092 has quit IRC | 19:32 | |
| *** e0ne has joined #openstack-cinder | 19:33 | |
| *** ametts has quit IRC | 19:33 | |
| *** haomaiwang has quit IRC | 19:33 | |
| *** akerr has joined #openstack-cinder | 19:33 | |
| *** geguileo has quit IRC | 19:36 | |
| *** haomaiwang has joined #openstack-cinder | 19:36 | |
| *** akerr_ has joined #openstack-cinder | 19:39 | |
| *** apoorvad has quit IRC | 19:39 | |
| *** apoorvad has joined #openstack-cinder | 19:40 | |
| *** akerr has quit IRC | 19:43 | |
| smcginnis | Summit Etherpads set up: https://wiki.openstack.org/wiki/Design_Summit/Newton/Etherpads#Cinder | 19:44 |
| openstackgerrit | Anthony Lee proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation https://review.openstack.org/268405 | 19:50 |
| *** gouthamr has quit IRC | 19:52 | |
| *** wilson has joined #openstack-cinder | 19:57 | |
| *** wilson is now known as Guest26399 | 19:57 | |
| *** Guest47552 has quit IRC | 19:58 | |
| akerr_ | scottda: just fyi, I was asked by some of the tempest team to squash my v3 support and user messages patches back into a single patch https://review.openstack.org/#/c/300639 | 20:00 |
| *** haomaiwang has quit IRC | 20:02 | |
| ntpttr_ | scottda: hey I see that microversion support looks like it's been added to the cinderclient, is there documentation anywhere for how to make use of it in our patches? | 20:03 |
| *** angela-s has joined #openstack-cinder | 20:07 | |
| *** geguileo has joined #openstack-cinder | 20:08 | |
| *** karthikp has joined #openstack-cinder | 20:08 | |
| *** geguileo has quit IRC | 20:13 | |
| scottda | ntpttr_: No documentation yet, but here's a patch that uses microversions in the client: https://review.openstack.org/#/c/303627 | 20:13 |
| ntpttr_ | scottda: awesome, thanks | 20:17 |
| *** ntpttr_ is now known as ntpttr | 20:17 | |
| *** haomaiwang has joined #openstack-cinder | 20:17 | |
| *** gouthamr has joined #openstack-cinder | 20:18 | |
| ntpttr | scottda: so it's as simple as wrapping the functions with the correct version that the server is expecting? | 20:19 |
| *** apoorvad has quit IRC | 20:20 | |
| scottda | ntpttr: Yes, pretty much. | 20:20 |
| *** apoorvad has joined #openstack-cinder | 20:20 | |
| ntpttr | scottda: do you know how to make the API version 3 rather than version 2? I'm working with the upload to image method which is in the v3 folder, but if I run the command I get this: | 20:23 |
| ntpttr | ERROR: API version '2.0' is not supported on 'cinderclient.v3.shell.do_upload_to_image' method. | 20:23 |
| rhagarty | xyang1, hello - time for a couple of consistency group issues? | 20:26 |
| xyang1 | rhagarty: in a meeting now | 20:26 |
| rhagarty | ok | 20:27 |
| *** bjornar has joined #openstack-cinder | 20:28 | |
| scottda | ntpttr: Is this running you're patch? I'll have a look if it is the latest code... | 20:29 |
| ntpttr | scottda: yeah, I'm guessing I need to specify version 3 with --os-volume-api-version or something? | 20:29 |
| *** timcl has quit IRC | 20:29 | |
| scottda | ntpttr: Yes. | 20:29 |
| scottda | or, actually, 3.1 since that is the version for your changes in the server | 20:30 |
| scottda | ntpttr: /v2 is still the default in the cinderclient. | 20:31 |
| *** mylu has quit IRC | 20:33 | |
| ntpttr | scottda: okay, I specified 3.1 and it's running into an attribute error in the substitution function in api_versions - obj.api_version is just giving "ERROR: api_version" | 20:34 |
| ntpttr | in methods = get_substitutions(name, obj.api_version) | 20:34 |
| scottda | ntpttr: I'll test your code and PM you.... | 20:34 |
| ntpttr | scottda: thanks, though this latest bit isn't up on gerrit yet | 20:35 |
| ntpttr | methods = get_substitutions(name, obj.api_version) | 20:35 |
| ntpttr | oops wrong clipboard | 20:35 |
| ntpttr | https://review.openstack.org/#/c/230622/ | 20:35 |
| *** haomaiwang has quit IRC | 20:36 | |
| ntpttr | the only changes I've made are to wrap the 3.1 api version in shell.py and volumes.py | 20:36 |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 20:37 |
| *** haomaiwang has joined #openstack-cinder | 20:38 | |
| *** haomaiwang has quit IRC | 20:39 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Move oslo.versionedobjects[fixtures] to test-reqs https://review.openstack.org/308581 | 20:39 |
| thingee | xyang1: happy to see you're joining the speed mentoring! | 20:39 |
| *** stacker has quit IRC | 20:40 | |
| *** jwcroppe has joined #openstack-cinder | 20:41 | |
| *** porrua has quit IRC | 20:42 | |
| xyang1 | thingee: thank you:). It sounds fun | 20:43 |
| *** jwcroppe_ has quit IRC | 20:43 | |
| xyang1 | rhagarty: hi, I am back | 20:43 |
| rhagarty | xyang1, hello - we had some QA folks do some testing of this feature from Horizon, and we ran into some issues. These issues also occur in the CLI, so I wanted to run them by you | 20:45 |
| xyang1 | rhagarty: ok, go ahead | 20:45 |
| *** rajinir has quit IRC | 20:45 | |
| rhagarty | xyang1, 4 issues... | 20:45 |
| rhagarty | 1) double or single quotes in name or description cause create CG to fail | 20:46 |
| rhagarty | 2) reverse square brackets in name (e.g. "]test[") causes create CG to fail | 20:46 |
| rhagarty | 3) > 150 characters for name and description combined causes create CG to fail | 20:46 |
| rhagarty | 4) setting description to blank (from a non-blank value) causes update CG to fail | 20:46 |
| xyang1 | rhagarty: have they tried these with volume? | 20:47 |
| smcginnis | Somebody's edge testing | 20:47 |
| xyang1 | smcginnis: indeed! | 20:47 |
| rhagarty | FYI, most of these work with update (like name) | 20:47 |
| rhagarty | you can create a volume with quotes | 20:47 |
| rhagarty | I realize these are corner cases... | 20:48 |
| xyang1 | rhagarty: hmm, I can't image this is different for volume | 20:48 |
| rhagarty | and if you wanted to ignore them, I can limmit them in Horizon | 20:48 |
| *** cknight has quit IRC | 20:48 | |
| xyang1 | rhagarty: unless some special checks were addded | 20:48 |
| xyang1 | rhagarty: that will be great if you can limit in horizon | 20:49 |
| xyang1 | rhagarty: can you check how volume names are handled? | 20:49 |
| rhagarty | the > 150 one is wierd. | 20:49 |
| xyang1 | rhagarty: they shoukd be the same | 20:49 |
| rhagarty | I create a volume with CLI using quotes | 20:50 |
| jgriffith | xyang1: they're using different validate methods :( | 20:50 |
| xyang1 | jgriffith: so volume does special checks? | 20:50 |
| jgriffith | xyang1: CG's API does too | 20:50 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Update LVM code so nova can use it https://review.openstack.org/308054 | 20:50 |
| jgriffith | validate_name_and_description | 20:50 |
| xyang1 | jgriffith: ok, let me see | 20:51 |
| jgriffith | xyang1: actually.... | 20:51 |
| jgriffith | xyang1: they should both be calling the impl at the wsgi layer I think | 20:51 |
| xyang1 | jgriffith: you are right | 20:52 |
| xyang1 | so why they behave differently | 20:52 |
| rhagarty | and note that I can change the CG name to have a quote, just not create it | 20:53 |
| *** bjornar has quit IRC | 20:53 | |
| rhagarty | ... with a quote | 20:54 |
| akerr_ | is it a volume service error or a horizon error that you get? | 20:54 |
| rhagarty | cinder error | 20:54 |
| rhagarty | state is set to error | 20:54 |
| jgriffith | xyang1: sounds like maybe the object layer is helping us out here? | 20:54 |
| akerr_ | do you have the debug logs showing why? | 20:54 |
| xyang1 | jgriffith: I'll have to check, I don't know | 20:55 |
| rhagarty | give me a sec | 20:55 |
| jgriffith | xyang1: quick glance doesn't appear so. | 20:55 |
| jgriffith | xyang1: I can try and repro here in a bit unless you want to take it | 20:55 |
| jgriffith | xyang1: should be easy enough to trace | 20:55 |
| *** mylu has joined #openstack-cinder | 20:55 | |
| xyang1 | jgriffith: if you can try, let me know what you can find out. I have not fixed my setup yet:( | 20:56 |
| *** esker has quit IRC | 20:56 | |
| *** mylu has quit IRC | 20:56 | |
| jgriffith | xyang1: hehe... we'll see if mines working now or not :) | 20:57 |
| xyang1 | jgriffith: :) | 20:58 |
| *** mylu has joined #openstack-cinder | 20:59 | |
| *** Julien-zte has joined #openstack-cinder | 21:00 | |
| *** mylu has quit IRC | 21:02 | |
| ildikov | jgriffith: hemna_: reminder, meeting on #openstack-meeting-cp | 21:02 |
| hemna_ | now? | 21:02 |
| rhagarty | xyang1, http://paste.openstack.org/show/494891/ | 21:03 |
| xyang1 | rhagarty: can you try create volume | 21:05 |
| jgriffith | xyang1: I just did.. it works | 21:06 |
| xyang1 | jgriffith: which one? | 21:06 |
| jgriffith | cinder create "jtest[" | 21:06 |
| jgriffith | xyang1: haven't managed to get policy enabled to try consis group create yet :) | 21:06 |
| xyang1 | jgriffith: did you see rhagarty's paste? | 21:07 |
| jgriffith | rhagarty: ummmm | 21:07 |
| jgriffith | rhagarty: hey dude | 21:07 |
| akerr_ | looks like its the hp backend throwing the error | 21:07 |
| jgriffith | rhagarty: not sure if you noticed but it's the HP lib that's bombing out there | 21:07 |
| jgriffith | rhagarty: NOT cinder | 21:07 |
| smcginnis | I was thinking - not all backends support some of those characters or lengths. | 21:07 |
| jgriffith | rhagarty: you need to log a bug against your HP lib :) | 21:07 |
| xyang1 | jgriffith: :) | 21:08 |
| rhagarty | ok - see that | 21:08 |
| * jgriffith 's work here is done :) | 21:08 | |
| xyang1 | smcginnis: I was thinking, if volume can take a name, group should too. I was wondering why they differ | 21:09 |
| rhagarty | thanks all... should have seen that | 21:09 |
| xyang1 | rhagarty: I'll try them after my setup is ready, won't be soon though:( | 21:10 |
| rhagarty | xyang1, ok - thanks | 21:10 |
| xyang1 | rhagarty: the combined length is odd too | 21:10 |
| *** mylu has joined #openstack-cinder | 21:10 | |
| smcginnis | xyang1: Good point. It probably should. | 21:10 |
| xyang1 | rhagarty: we are not dealing with combined length in create | 21:11 |
| *** lpetrut has quit IRC | 21:12 | |
| rhagarty | xyang1, which means it should work? | 21:15 |
| xyang1 | rhagarty: about #3, do you mean any combination of name and description, as soon as the total length greater than 150? | 21:15 |
| xyang1 | rhagarty: this is a weird one | 21:16 |
| rhagarty | xyang1, yes. And it works with "update" | 21:16 |
| *** xek has quit IRC | 21:16 | |
| xyang1 | rhagarty: and this works with volume too? | 21:16 |
| rhagarty | xyang1, will try... | 21:17 |
| akerr_ | could be a quirk in the hp interface if they're all failing at the same place in the code. maybe that endpoint has a really tiny length limit on it for the http request | 21:17 |
| xyang1 | akerr_: that's possible | 21:18 |
| xyang1 | rhagarty: can you ask QA to test all 4 cases with create volume and also check if exception is from HPE backend! | 21:19 |
| rhagarty | will do | 21:19 |
| xyang1 | rhagarty: I know you said #1 works for volume | 21:19 |
| *** cknight has joined #openstack-cinder | 21:20 | |
| Suyi | rhagarty, will do and update you | 21:20 |
| *** e0ne has quit IRC | 21:20 | |
| rhagarty | xyang1, looks like create volume also fails with name and description > 150. Works if == 150 | 21:22 |
| xyang1 | rhagarty: so that is the same as CG? | 21:23 |
| rhagarty | xyang1, hang on. may have misspoke.. | 21:24 |
| *** sbezverk has quit IRC | 21:25 | |
| rhagarty | xyang1, volume does work with > 150 | 21:25 |
| *** baumann1 has left #openstack-cinder | 21:25 | |
| rhagarty | (sorry for confusion) | 21:26 |
| xyang1 | rhagarty: can you try the exact name and description for CG? | 21:27 |
| *** sbezverk has joined #openstack-cinder | 21:27 | |
| xyang1 | rhagarty: sorry I can't try it myself now, but I can't make sense of it | 21:27 |
| rhagarty | xyang1, give me a sec... | 21:28 |
| xyang1 | rhagarty: oh, don't forget to check if i | 21:28 |
| xyang1 | rhagarty: if exception is from your backend | 21:28 |
| *** sdake_ has joined #openstack-cinder | 21:28 | |
| xyang1 | rhagarty: cinder db should not have this restriction | 21:28 |
| *** xek has joined #openstack-cinder | 21:29 | |
| openstackgerrit | Carl Pecinovsky proposed openstack/cinder: Avoid ascii code error for request body strings https://review.openstack.org/308626 | 21:31 |
| rhagarty | xyang1, http://paste.openstack.org/show/494894/ looks like our driver again | 21:33 |
| xyang1 | rhagarty: about #4, the code is written so that you have to provide a description to update | 21:33 |
| xyang1 | rhagarty: alright, thanks | 21:34 |
| rhagarty | xyang1, so you can't go from having a desc to not having one? | 21:34 |
| xyang1 | rhagarty: can you check #1 and 2? | 21:34 |
| xyang1 | rhagarty: maybe those are also restrictions on your backend | 21:34 |
| xyang1 | rhagarty: no, you have to provide a description to update to | 21:35 |
| rhagarty | xyang1, I would assume it our problem, but we will check all | 21:36 |
| xyang1 | rhagarty: ok, thanks | 21:37 |
| rhagarty | xyang1, I don't agree with the description check. Seems perfectly valid to "change" to a blank description. But if thats the case, I can add check in Horizon | 21:38 |
| xyang1 | rhagarty: so both name and description cannot be empty at update time | 21:38 |
| rhagarty | xyang1, ok - so, for update, if I supply name (which may not have changed), along with blank description (which used to have one), it should work | 21:40 |
| rhagarty | ? | 21:40 |
| *** e0ne has joined #openstack-cinder | 21:41 | |
| *** jgregor has quit IRC | 21:41 | |
| *** rlrossit has quit IRC | 21:42 | |
| xyang1 | rhagarty: no, there is check to make sure name cannot be None if you update it, and a same check for description | 21:42 |
| *** jgregor has joined #openstack-cinder | 21:42 | |
| *** jgregor has quit IRC | 21:42 | |
| rhagarty | ok | 21:42 |
| xyang1 | rhagarty: so description cannot be None if you update it | 21:42 |
| *** cdelatte has quit IRC | 21:44 | |
| *** sdake_ has quit IRC | 21:45 | |
| *** mylu has quit IRC | 21:46 | |
| *** sdake_ has joined #openstack-cinder | 21:47 | |
| rhagarty | xyang1, I think a better test would be to just make sure it changed, but I can work around it. Thanks | 21:47 |
| xyang1 | rhagarty: ok, thanks. if this is really something we need to support, I can change the code, but I am not sure | 21:48 |
| *** e0ne has quit IRC | 21:51 | |
| *** vgridnev_ has joined #openstack-cinder | 21:55 | |
| *** salv-orl_ has joined #openstack-cinder | 21:56 | |
| *** salv-orlando has quit IRC | 21:59 | |
| *** ivase has quit IRC | 22:00 | |
| *** gouthamr has quit IRC | 22:00 | |
| *** vgridnev_ has quit IRC | 22:00 | |
| *** frakt has quit IRC | 22:11 | |
| *** frakt has joined #openstack-cinder | 22:11 | |
| *** marcusvrn_ has quit IRC | 22:13 | |
| *** mylu has joined #openstack-cinder | 22:15 | |
| *** mylu has quit IRC | 22:17 | |
| *** mylu has joined #openstack-cinder | 22:24 | |
| *** edmondsw has quit IRC | 22:24 | |
| *** diablo_rojo has quit IRC | 22:29 | |
| *** jungleboyj has quit IRC | 22:31 | |
| *** arch-nemesis has quit IRC | 22:36 | |
| *** haplo37 has quit IRC | 22:38 | |
| *** xyang1 has quit IRC | 22:43 | |
| *** mriedem has quit IRC | 22:44 | |
| *** porrua has joined #openstack-cinder | 22:50 | |
| hemna_ | angela-s, ping | 22:54 |
| angela-s | hemna_: hey | 22:54 |
| hemna_ | https://bugs.launchpad.net/cinder/+bug/1486613 | 22:55 |
| openstack | Launchpad bug 1486613 in Cinder "Existing volume detach fails after modifying the zoning_mode=fabric from none" [Wishlist,Won't fix] - Assigned to Angela Smith (aallen-m) | 22:55 |
| *** porrua has quit IRC | 22:55 | |
| hemna_ | I just marked that as wont fix | 22:55 |
| hemna_ | but I have a question | 22:55 |
| angela-s | cool, thanks. | 22:55 |
| hemna_ | the problem is here that we wont' find the zone at RemoveFCZone time right? | 22:55 |
| angela-s | yes | 22:56 |
| hemna_ | so | 22:56 |
| hemna_ | what is the problem with ignoring that failure | 22:56 |
| *** krotscheck is now known as krotscheck_dcm | 22:57 | |
| hemna_ | meaning, @RemoveFCZone catches that failure, logs it and goes on | 22:57 |
| hemna_ | the detach process will progress and finish. | 22:57 |
| hemna_ | we'll have an 'orphaned' zone on the switch | 22:57 |
| *** cknight has quit IRC | 22:58 | |
| angela-s | no, the I-T will still be zoned on the FC fabric and will still remain in the active config. | 23:00 |
| hemna_ | yah, we'll have a zone on the switch still | 23:00 |
| angela-s | thus, the I-T will still be allowed to communicate | 23:00 |
| hemna_ | sure | 23:01 |
| hemna_ | we'll log the warning | 23:01 |
| hemna_ | the problem is now, you can't even detach the volume. | 23:01 |
| hemna_ | because of this | 23:01 |
| hemna_ | and I bet it leaves the volume in detaching and/or error state | 23:01 |
| hemna_ | the storage array will have removed the target export | 23:02 |
| angela-s | hmm, let me think the possibility of ignoring the lookup error and get back to you. | 23:03 |
| hemna_ | ok | 23:03 |
| *** angela-s has quit IRC | 23:04 | |
| *** Suyi has quit IRC | 23:07 | |
| *** Suyi has joined #openstack-cinder | 23:07 | |
| patrickeast | hemna_: seems like a pretty reasonable thing to ask the admin to cleanup any existing zones after they change zoning_mode to None | 23:10 |
| patrickeast | and just ignore the error in the fczm | 23:10 |
| hemna_ | patrickeast, that's kinda what I'm thinking | 23:10 |
| hemna_ | because if they went from none -> fabric | 23:11 |
| hemna_ | that means existing attachments were manually zoned. | 23:11 |
| patrickeast | yea | 23:11 |
| hemna_ | we'll log the error saying we couldn't find the zone, please ensure any manually created zones are manually removed. | 23:11 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/308369 | 23:11 |
| *** apoorvad has quit IRC | 23:12 | |
| *** sdague has quit IRC | 23:12 | |
| hemna_ | patrickeast, I updated the bug and reset it to incomplete | 23:13 |
| hemna_ | until we figure out that's a really bad thing to do. | 23:13 |
| *** The_Ball has quit IRC | 23:14 | |
| patrickeast | haha | 23:16 |
| patrickeast | hemna_: sounds good to me | 23:16 |
| *** mylu has quit IRC | 23:20 | |
| *** ozialien has joined #openstack-cinder | 23:20 | |
| *** mylu has joined #openstack-cinder | 23:20 | |
| *** salv-orl_ has quit IRC | 23:26 | |
| *** jwcroppe has quit IRC | 23:28 | |
| *** jwcroppe has joined #openstack-cinder | 23:29 | |
| *** apoorvad has joined #openstack-cinder | 23:30 | |
| *** Lickitysplitted_ has quit IRC | 23:36 | |
| *** Lickitysplitted has joined #openstack-cinder | 23:36 | |
| *** mylu has quit IRC | 23:38 | |
| *** mylu has joined #openstack-cinder | 23:38 | |
| *** sasukeh has joined #openstack-cinder | 23:42 | |
| *** The_Ball has joined #openstack-cinder | 23:43 | |
| *** karthikp has quit IRC | 23:44 | |
| *** yhayashi has joined #openstack-cinder | 23:44 | |
| *** jamielennox is now known as jamielennox|away | 23:51 | |
| *** sbezverk has quit IRC | 23:52 | |
| *** Julien-zte has quit IRC | 23:53 | |
| *** sbezverk has joined #openstack-cinder | 23:56 | |
| *** sdake__ has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!