| *** diogogmt has quit IRC | 00:00 | |
| *** yangyapeng has quit IRC | 00:03 | |
| *** alonma has quit IRC | 00:03 | |
| *** thumpba has joined #openstack-cinder | 00:04 | |
| *** alonma has joined #openstack-cinder | 00:06 | |
| *** alonma has quit IRC | 00:10 | |
| *** mtanino has quit IRC | 00:11 | |
| *** jungleboyj has joined #openstack-cinder | 00:11 | |
| *** thumpba has quit IRC | 00:12 | |
| *** alonma has joined #openstack-cinder | 00:16 | |
| *** jwcroppe_ has joined #openstack-cinder | 00:20 | |
| *** alonma has quit IRC | 00:20 | |
| *** jwcroppe has quit IRC | 00:22 | |
| *** alonma has joined #openstack-cinder | 00:23 | |
| *** laughterwym has joined #openstack-cinder | 00:25 | |
| *** laughterwym has quit IRC | 00:26 | |
| *** laughterwym has joined #openstack-cinder | 00:26 | |
| *** alonma has quit IRC | 00:27 | |
| *** jwcroppe has joined #openstack-cinder | 00:28 | |
| *** alonma has joined #openstack-cinder | 00:30 | |
| *** jwcroppe_ has quit IRC | 00:31 | |
| *** akerr_ is now known as akerr_away | 00:33 | |
| *** akerr_away is now known as akerr_ | 00:33 | |
| *** kaisers_ has joined #openstack-cinder | 00:34 | |
| *** alonma has quit IRC | 00:35 | |
| *** alonma has joined #openstack-cinder | 00:37 | |
| *** kaisers_ has quit IRC | 00:38 | |
| *** Suyi_ has quit IRC | 00:40 | |
| *** karthik__ has quit IRC | 00:42 | |
| *** alonma has quit IRC | 00:42 | |
| *** alonma has joined #openstack-cinder | 00:46 | |
| *** alonma has quit IRC | 00:50 | |
| *** alonma has joined #openstack-cinder | 00:53 | |
| *** jungleboyj has quit IRC | 00:54 | |
| *** karthik__ has joined #openstack-cinder | 00:56 | |
| *** alonma has quit IRC | 00:57 | |
| *** sdake has quit IRC | 00:57 | |
| *** sdake has joined #openstack-cinder | 00:59 | |
| *** Guest45378 has quit IRC | 00:59 | |
| *** alonma has joined #openstack-cinder | 01:00 | |
| *** wilson has joined #openstack-cinder | 01:01 | |
| *** wilson is now known as Guest15865 | 01:02 | |
| *** alonma has quit IRC | 01:04 | |
| *** karthik__ has quit IRC | 01:07 | |
| *** nikeshm has quit IRC | 01:08 | |
| *** diogogmt has joined #openstack-cinder | 01:09 | |
| openstackgerrit | zheng yin proposed openstack/python-cinderclient: remove raise "e" https://review.openstack.org/350000 | 01:11 |
|---|---|---|
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage https://review.openstack.org/349086 | 01:11 |
| *** Apoorva has quit IRC | 01:12 | |
| *** yangyapeng has joined #openstack-cinder | 01:16 | |
| *** iceyao has joined #openstack-cinder | 01:18 | |
| *** mriedem has quit IRC | 01:19 | |
| *** karthik__ has joined #openstack-cinder | 01:19 | |
| Guy | Hi everybody, sorry for disturbing, can I get some reviews on https://review.openstack.org/#/c/347100/ and https://review.openstack.org/#/c/349318/2 please. Thank you in advance. | 01:20 |
| *** akshai has joined #openstack-cinder | 01:23 | |
| *** Apoorva has joined #openstack-cinder | 01:23 | |
| *** abhitechie has joined #openstack-cinder | 01:27 | |
| *** delattec has quit IRC | 01:28 | |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** tangchen has quit IRC | 01:33 | |
| *** alonma has joined #openstack-cinder | 01:33 | |
| *** akshai has quit IRC | 01:35 | |
| *** abhitechie has quit IRC | 01:37 | |
| *** alonma has quit IRC | 01:37 | |
| *** tangchen has joined #openstack-cinder | 01:39 | |
| *** Apoorva_ has joined #openstack-cinder | 01:40 | |
| *** alonma has joined #openstack-cinder | 01:40 | |
| openstackgerrit | zhangyanxian proposed openstack/os-brick: Update the home-page info with the developer documentation https://review.openstack.org/350337 | 01:41 |
| openstackgerrit | zhangyanxian proposed openstack/os-brick: Update the home-page info with the developer documentation https://review.openstack.org/350337 | 01:41 |
| *** sdake has quit IRC | 01:44 | |
| *** Apoorva has quit IRC | 01:44 | |
| *** EinstCrazy has joined #openstack-cinder | 01:44 | |
| *** Apoorva_ has quit IRC | 01:44 | |
| *** alonma has quit IRC | 01:44 | |
| *** alonma has joined #openstack-cinder | 01:47 | |
| *** alonma has quit IRC | 01:52 | |
| *** karthik__ has quit IRC | 01:53 | |
| *** alonma has joined #openstack-cinder | 01:54 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 01:55 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add encryptor attach/detach in utils https://review.openstack.org/341914 | 01:55 |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 01:57 |
| *** alonma has quit IRC | 01:59 | |
| *** yangyape_ has joined #openstack-cinder | 02:03 | |
| *** yangyapeng has quit IRC | 02:04 | |
| *** alonma has joined #openstack-cinder | 02:08 | |
| *** bardia has quit IRC | 02:12 | |
| *** alonma has quit IRC | 02:13 | |
| *** akerr_ has quit IRC | 02:16 | |
| *** alonma has joined #openstack-cinder | 02:16 | |
| *** coolsvap has joined #openstack-cinder | 02:19 | |
| *** alonma has quit IRC | 02:21 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 02:22 |
| *** kaisers_ has joined #openstack-cinder | 02:23 | |
| *** gouthamr has joined #openstack-cinder | 02:24 | |
| *** gouthamr_ has joined #openstack-cinder | 02:27 | |
| *** kaisers_ has quit IRC | 02:27 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 02:29 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 02:29 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 02:29 |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups https://review.openstack.org/350350 | 02:29 |
| *** gouthamr has quit IRC | 02:30 | |
| *** alonma has joined #openstack-cinder | 02:32 | |
| *** yangyapeng has joined #openstack-cinder | 02:34 | |
| *** alonma has quit IRC | 02:37 | |
| *** yangyape_ has quit IRC | 02:37 | |
| *** markvoelker has joined #openstack-cinder | 02:39 | |
| *** alonma has joined #openstack-cinder | 02:39 | |
| *** dave-mccowan has joined #openstack-cinder | 02:41 | |
| *** Guest15865 has quit IRC | 02:41 | |
| *** wilson has joined #openstack-cinder | 02:41 | |
| *** wilson is now known as Guest23591 | 02:42 | |
| *** alonma has quit IRC | 02:44 | |
| *** alonma has joined #openstack-cinder | 02:46 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix ImportError of fake_driver https://review.openstack.org/350357 | 02:47 |
| *** daneyon has quit IRC | 02:48 | |
| *** alonma has quit IRC | 02:51 | |
| *** alonma has joined #openstack-cinder | 02:54 | |
| openstackgerrit | Merged openstack/cinder: Docs: Correct i18n information https://review.openstack.org/346477 | 02:57 |
| *** alonma has quit IRC | 02:58 | |
| *** alonma has joined #openstack-cinder | 03:01 | |
| *** Srinu has joined #openstack-cinder | 03:05 | |
| *** alonma has quit IRC | 03:05 | |
| Srinu | patrickeast:hi | 03:06 |
| *** dave-mccowan has quit IRC | 03:08 | |
| *** alonma has joined #openstack-cinder | 03:08 | |
| openstackgerrit | Merged openstack/cinder: Fix documentation and remove dead code https://review.openstack.org/349089 | 03:09 |
| openstackgerrit | Merged openstack/cinder: ScaleIO ignore rename of external volumes https://review.openstack.org/343606 | 03:09 |
| *** alonma has quit IRC | 03:12 | |
| *** iceyao_ has joined #openstack-cinder | 03:12 | |
| *** Guest23591 has quit IRC | 03:14 | |
| *** wilson_liu has joined #openstack-cinder | 03:15 | |
| *** iceyao has quit IRC | 03:16 | |
| *** alonma has joined #openstack-cinder | 03:17 | |
| wilson_liu | hi, I have a question about the concurrency of message cinder-volume recieved from rabbitmq | 03:17 |
| wilson_liu | Do we have any configuration to config the concurrency? | 03:18 |
| *** alonma has quit IRC | 03:21 | |
| *** cknight has joined #openstack-cinder | 03:26 | |
| *** Nakato has quit IRC | 03:27 | |
| *** gouthamr_ has quit IRC | 03:29 | |
| *** Nakato has joined #openstack-cinder | 03:32 | |
| *** alonma has joined #openstack-cinder | 03:36 | |
| *** pradipm has joined #openstack-cinder | 03:38 | |
| *** alonma has quit IRC | 03:40 | |
| *** pradipm has quit IRC | 03:42 | |
| *** alonma has joined #openstack-cinder | 03:43 | |
| *** alonma has quit IRC | 03:47 | |
| *** nkrinner_afk is now known as nkrinner | 03:47 | |
| *** alonma has joined #openstack-cinder | 03:50 | |
| *** sdake has joined #openstack-cinder | 03:50 | |
| *** abhitechie has joined #openstack-cinder | 03:53 | |
| *** sdake_ has joined #openstack-cinder | 03:54 | |
| *** alonma has quit IRC | 03:54 | |
| *** markvoelker has quit IRC | 03:55 | |
| *** alonma has joined #openstack-cinder | 03:57 | |
| *** sdake has quit IRC | 03:58 | |
| *** bardia has joined #openstack-cinder | 04:01 | |
| *** alonma has quit IRC | 04:02 | |
| *** karthikp_ has quit IRC | 04:04 | |
| *** alonma has joined #openstack-cinder | 04:04 | |
| *** sdake_ has quit IRC | 04:05 | |
| *** alonma has quit IRC | 04:09 | |
| *** links has joined #openstack-cinder | 04:10 | |
| *** alonma has joined #openstack-cinder | 04:11 | |
| *** alonma has quit IRC | 04:16 | |
| *** jdillaman has quit IRC | 04:18 | |
| *** alonma has joined #openstack-cinder | 04:19 | |
| *** nikeshm has joined #openstack-cinder | 04:22 | |
| nikeshm | DuncanT: ping | 04:22 |
| nikeshm | sorry | 04:22 |
| nikeshm | hi | 04:22 |
| *** kaisers_ has joined #openstack-cinder | 04:23 | |
| *** alonma has quit IRC | 04:23 | |
| *** kaisers_ has quit IRC | 04:23 | |
| *** alonma has joined #openstack-cinder | 04:26 | |
| nikeshm | DuncanT: we deleted 'kaminario_nodedup_substring' option in https://review.openstack.org/346540 to decide from extra-specs about creating dedup/nodedup volumes, but in reviews we were suggested to deprecate this option in current release and delete it in Ocata release. | 04:27 |
| nikeshm | DuncanT: Please let me know what should i do for your comment on https://review.openstack.org/#/c/349019 | 04:29 |
| *** alonma has quit IRC | 04:31 | |
| *** markvoelker has joined #openstack-cinder | 04:33 | |
| *** alonma has joined #openstack-cinder | 04:33 | |
| *** alonma has quit IRC | 04:38 | |
| *** markvoelker has quit IRC | 04:39 | |
| *** alonma has joined #openstack-cinder | 04:41 | |
| *** alonma has quit IRC | 04:45 | |
| *** alonma has joined #openstack-cinder | 04:48 | |
| *** alonma has quit IRC | 04:52 | |
| *** alonma has joined #openstack-cinder | 04:55 | |
| *** lkuchlan has joined #openstack-cinder | 04:56 | |
| *** alonma has quit IRC | 05:00 | |
| *** pdeore has joined #openstack-cinder | 05:01 | |
| *** julim has quit IRC | 05:02 | |
| *** alonma has joined #openstack-cinder | 05:02 | |
| *** bardia has quit IRC | 05:02 | |
| openstackgerrit | Merged openstack/cinder: Huawei: Fixed problems caused by password expiration https://review.openstack.org/305633 | 05:04 |
| *** lkuchlan has quit IRC | 05:04 | |
| *** lkuchlan has joined #openstack-cinder | 05:04 | |
| *** cknight has quit IRC | 05:06 | |
| *** chris_morrell has joined #openstack-cinder | 05:06 | |
| *** alonma has quit IRC | 05:07 | |
| *** chris_morrell has quit IRC | 05:07 | |
| *** karthikp_ has joined #openstack-cinder | 05:09 | |
| *** gcb has quit IRC | 05:11 | |
| *** gcb has joined #openstack-cinder | 05:12 | |
| *** pbandark has joined #openstack-cinder | 05:12 | |
| *** alonma has joined #openstack-cinder | 05:12 | |
| *** e0ne has joined #openstack-cinder | 05:12 | |
| *** e0ne has quit IRC | 05:15 | |
| *** alonma has quit IRC | 05:17 | |
| openstackgerrit | Vijay Ladani proposed openstack/cinder: Handle snapshot delete conflict in 3par driver https://review.openstack.org/342722 | 05:18 |
| *** alonma has joined #openstack-cinder | 05:20 | |
| *** Poornima has joined #openstack-cinder | 05:22 | |
| *** alonma has quit IRC | 05:24 | |
| *** alonma has joined #openstack-cinder | 05:27 | |
| *** markvoelker has joined #openstack-cinder | 05:28 | |
| *** liverpooler has joined #openstack-cinder | 05:28 | |
| *** rcernin has joined #openstack-cinder | 05:28 | |
| *** liverpooler has quit IRC | 05:30 | |
| *** liverpooler has joined #openstack-cinder | 05:31 | |
| *** alonma has quit IRC | 05:31 | |
| *** markvoelker has quit IRC | 05:33 | |
| *** alonma has joined #openstack-cinder | 05:34 | |
| *** alonma has quit IRC | 05:38 | |
| *** alonma has joined #openstack-cinder | 05:41 | |
| *** yangyape_ has joined #openstack-cinder | 05:42 | |
| *** yangyapeng has quit IRC | 05:42 | |
| *** ChubYann has quit IRC | 05:42 | |
| *** jdillaman has joined #openstack-cinder | 05:44 | |
| *** alonma has quit IRC | 05:46 | |
| *** alonma has joined #openstack-cinder | 05:48 | |
| *** alonma has quit IRC | 05:53 | |
| *** alonma has joined #openstack-cinder | 05:56 | |
| *** julim has joined #openstack-cinder | 05:57 | |
| *** alonma has quit IRC | 06:00 | |
| *** alonma has joined #openstack-cinder | 06:03 | |
| *** mjura has joined #openstack-cinder | 06:04 | |
| *** sdake has joined #openstack-cinder | 06:07 | |
| *** sdake has quit IRC | 06:07 | |
| *** sdake has joined #openstack-cinder | 06:07 | |
| *** alonma has quit IRC | 06:07 | |
| *** alonma has joined #openstack-cinder | 06:10 | |
| *** mjura has quit IRC | 06:12 | |
| *** mjura has joined #openstack-cinder | 06:13 | |
| *** iceyao_ has quit IRC | 06:13 | |
| *** iceyao has joined #openstack-cinder | 06:13 | |
| *** alonma has quit IRC | 06:14 | |
| DuncanT | nikeshm: Replied to review. Just updating the commit message with an explanation should be fine. Thanks. | 06:15 |
| *** alonma has joined #openstack-cinder | 06:19 | |
| *** lpetrut has joined #openstack-cinder | 06:19 | |
| *** alonmarx has joined #openstack-cinder | 06:22 | |
| *** markvoelker has joined #openstack-cinder | 06:22 | |
| *** alonma has quit IRC | 06:23 | |
| *** sdake has quit IRC | 06:23 | |
| *** alonma has joined #openstack-cinder | 06:26 | |
| *** markvoelker has quit IRC | 06:26 | |
| *** lkuchlan has quit IRC | 06:29 | |
| *** lkuchlan has joined #openstack-cinder | 06:30 | |
| *** alonma has quit IRC | 06:31 | |
| *** alonma has joined #openstack-cinder | 06:33 | |
| nikeshm | DuncanT: hi, actually i tried to remove kaminario_nodedup_substring in Change I43630dd6f7b9c1d18f59be12626b66f798de2822 but added it back in same change | 06:34 |
| nikeshm | DuncanT: i am not readding it | 06:34 |
| nikeshm | in https://review.openstack.org/#/c/349019/10 | 06:34 |
| *** Digvijay has joined #openstack-cinder | 06:36 | |
| nikeshm | so bit confuse | 06:36 |
| *** pgadiya has joined #openstack-cinder | 06:36 | |
| *** alonma has quit IRC | 06:38 | |
| *** vbala has joined #openstack-cinder | 06:38 | |
| *** arecknag has joined #openstack-cinder | 06:39 | |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder: Remove unused register_models() method https://review.openstack.org/350393 | 06:40 |
| *** alonma has joined #openstack-cinder | 06:41 | |
| *** belmoreira has joined #openstack-cinder | 06:42 | |
| *** alonma has quit IRC | 06:45 | |
| *** tesseract- has joined #openstack-cinder | 06:45 | |
| DuncanT | nikeshm: Sorry, I messed up rebasing the patch. I'll update the review. You need to use the actual deprecation mechanism for the deprecation so that the standard deprecation warnings come out. Basically follow Vipin's comments in https://review.openstack.org/#/c/346540/16/cinder/volume/drivers/kaminario/kaminario_common.py (fix the import order too please). | 06:47 |
| *** alonma has joined #openstack-cinder | 06:48 | |
| *** alonma has quit IRC | 06:52 | |
| *** alonma has joined #openstack-cinder | 06:55 | |
| *** tangchen has quit IRC | 07:00 | |
| nikeshm | DuncanT: thanks, no need to say sorry, you are always helpful | 07:00 |
| *** alonma has quit IRC | 07:00 | |
| *** alonma has joined #openstack-cinder | 07:02 | |
| *** karthikp_ has quit IRC | 07:03 | |
| *** alonma has quit IRC | 07:07 | |
| nikeshm | DuncanT vbala: changes suggested are small, shall i change it it retype patch | 07:07 |
| nikeshm | itself | 07:08 |
| *** alonma has joined #openstack-cinder | 07:09 | |
| DuncanT | Yes, that's probably easiest. | 07:12 |
| *** alonma has quit IRC | 07:14 | |
| vbala | nikeshm: I'm fine with that | 07:14 |
| *** markvoelker has joined #openstack-cinder | 07:16 | |
| *** e0ne has joined #openstack-cinder | 07:16 | |
| openstackgerrit | Merged openstack/cinder: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 07:19 |
| *** alonma has joined #openstack-cinder | 07:19 | |
| *** markvoelker has quit IRC | 07:21 | |
| *** pcaruana has joined #openstack-cinder | 07:21 | |
| nikeshm | vbala: http://paste.openstack.org/show/547713/ is this OK? | 07:21 |
| nikeshm | vbala: Ocata --> 10.0.0 | 07:23 |
| nikeshm | vbala: is this to change in option tooo | 07:23 |
| nikeshm | http://paste.openstack.org/show/547713/ | 07:23 |
| *** alonma has quit IRC | 07:23 | |
| *** links has quit IRC | 07:23 | |
| vbala | nikeshm: I think you can keep the original help text and add the reason for deprecation as the value of deprecated_reason | 07:24 |
| vbala | See https://github.com/openstack/nova/blob/be59c19c969acf6b25b0711f0ebfb26aaed0a171/nova/conf/libvirt.py#L91 | 07:24 |
| nikeshm | vbala: what about Ocata --> 10.0.0 | 07:25 |
| nikeshm | vbala: i saw using Mitaka | 07:25 |
| nikeshm | in deprecated | 07:25 |
| nikeshm | message | 07:25 |
| vbala | nikeshm: No need to add the version in the help text, I think you can change the version in the info log | 07:26 |
| *** alonma has joined #openstack-cinder | 07:26 | |
| nikeshm | vbala: but how it useful to use version in info and name in text | 07:29 |
| nikeshm | any issue if we keep name in text | 07:29 |
| nikeshm | i mean Ocata | 07:29 |
| nikeshm | in info | 07:29 |
| *** ekarlso has quit IRC | 07:29 | |
| nikeshm | name in help message i mean | 07:30 |
| nikeshm | let me correct myself | 07:31 |
| *** alonma has quit IRC | 07:31 | |
| nikeshm | vbala: how is it useful to use 10.0.0 in info and Ocata in option help? | 07:32 |
| nikeshm | any issue if i use Ocata in info | 07:32 |
| *** alonma has joined #openstack-cinder | 07:34 | |
| *** lpetrut has quit IRC | 07:36 | |
| vbala | nikeshm: i think there is no need to mention 'ocata' or the version in the help text because it is already mentioned in the logs. But it's ok for me if you want to mention it in both places. | 07:36 |
| *** links has joined #openstack-cinder | 07:37 | |
| *** ekarlso has joined #openstack-cinder | 07:37 | |
| *** claudiub has joined #openstack-cinder | 07:37 | |
| vbala | We started using the version instead of the release name (Ocata) in the code, I don't whether it is changed now. | 07:38 |
| *** alonma has quit IRC | 07:38 | |
| *** alonma has joined #openstack-cinder | 07:41 | |
| *** ebalduf has quit IRC | 07:41 | |
| nikeshm | vbala: ok i will change it to 10.0.0 in info and will use "next release" in help | 07:43 |
| *** alonma has quit IRC | 07:45 | |
| *** lpetrut has joined #openstack-cinder | 07:46 | |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 07:48 |
| *** alonma has joined #openstack-cinder | 07:48 | |
| *** alonma has quit IRC | 07:52 | |
| *** alonma has joined #openstack-cinder | 07:55 | |
| *** alonma has quit IRC | 07:57 | |
| *** alonma has joined #openstack-cinder | 07:57 | |
| *** yangyapeng has joined #openstack-cinder | 07:59 | |
| *** jistr is now known as jistr|training | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** yangyap__ has joined #openstack-cinder | 08:01 | |
| *** zzzeek has joined #openstack-cinder | 08:02 | |
| *** yangyape_ has quit IRC | 08:03 | |
| *** yangyapeng has quit IRC | 08:04 | |
| *** lpetrut has quit IRC | 08:04 | |
| *** yangyap__ has quit IRC | 08:05 | |
| *** coolsvap is now known as coolsvap_ | 08:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** markvoelker has joined #openstack-cinder | 08:10 | |
| *** abhinavtechie has joined #openstack-cinder | 08:11 | |
| *** abhitechie has quit IRC | 08:12 | |
| *** markvoelker has quit IRC | 08:14 | |
| *** alonmarx has quit IRC | 08:21 | |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 08:31 |
| *** gb21 has joined #openstack-cinder | 08:40 | |
| *** lkuchlan has quit IRC | 08:42 | |
| *** yangyapeng has joined #openstack-cinder | 08:42 | |
| *** laughterwym has quit IRC | 08:44 | |
| *** gb21 has quit IRC | 08:48 | |
| *** gb21 has joined #openstack-cinder | 08:48 | |
| *** jistr|training is now known as jistr | 08:49 | |
| *** Digvijay has quit IRC | 08:58 | |
| *** Digvijay has joined #openstack-cinder | 08:59 | |
| *** amitkqed has quit IRC | 09:04 | |
| *** markvoelker has joined #openstack-cinder | 09:04 | |
| *** amitkqed has joined #openstack-cinder | 09:04 | |
| *** gb21 has quit IRC | 09:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/348667 | 09:06 |
| *** lkuchlan has joined #openstack-cinder | 09:08 | |
| *** nikeshm has quit IRC | 09:08 | |
| *** Srinu has quit IRC | 09:08 | |
| *** markvoelker has quit IRC | 09:09 | |
| *** gb21 has joined #openstack-cinder | 09:10 | |
| *** lkuchlan has quit IRC | 09:13 | |
| *** EinstCrazy has quit IRC | 09:15 | |
| *** EinstCrazy has joined #openstack-cinder | 09:16 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 09:17 |
| openstackgerrit | LisaLi proposed openstack/cinder: Update attach_volume() with versionedobjects https://review.openstack.org/268608 | 09:17 |
| *** tangchen has joined #openstack-cinder | 09:24 | |
| *** gb21 has quit IRC | 09:25 | |
| *** lpetrut has joined #openstack-cinder | 09:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/350475 | 09:31 |
| *** alonma has quit IRC | 09:33 | |
| *** smile-luobin_ has quit IRC | 09:33 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 09:34 |
| *** nikeshm has joined #openstack-cinder | 09:34 | |
| *** smile-luobin has joined #openstack-cinder | 09:34 | |
| nikeshm | vbala DuncanT: check this https://github.com/openstack/cinder/blob/fd2790f575c748b0a745e8b1030998a16c5906f1/cinder/volume/manager.py#L215-L220 | 09:34 |
| nikeshm | i used suppress_requests_ssl_warnings = true in cinder.conf | 09:35 |
| nikeshm | but even though warnings are coming | 09:35 |
| nikeshm | may be i have to add kaminario_suppress_requests_ssl_warnings option | 09:36 |
| flip214 | jgriffith: thingee: thank you very much for reviewing 347273! my patch queue starts with 347267, sadly, so it'll take some more reviews to get that merged, too ;/ | 09:36 |
| lpetrut | Hi, any chance someone could take a look over those two patches on os-brick adding Windows SMBFS and FC connectors? https://review.openstack.org/#/c/323781/ https://review.openstack.org/#/c/323780 | 09:40 |
| lpetrut | First one already got a +2 and our CI already voted positively for both of them. | 09:40 |
| lpetrut | Thanks | 09:40 |
| *** kambiz has joined #openstack-cinder | 09:44 | |
| *** alonma has joined #openstack-cinder | 09:47 | |
| *** laughterwym has joined #openstack-cinder | 09:50 | |
| *** alonma has quit IRC | 09:52 | |
| *** laughterwym has quit IRC | 09:54 | |
| *** nikeshm has quit IRC | 09:58 | |
| *** Jaison has joined #openstack-cinder | 09:58 | |
| *** markvoelker has joined #openstack-cinder | 09:58 | |
| *** links has quit IRC | 09:59 | |
| *** alonma has joined #openstack-cinder | 10:01 | |
| *** markvoelker has quit IRC | 10:04 | |
| *** alonma has quit IRC | 10:06 | |
| *** alonma has joined #openstack-cinder | 10:08 | |
| *** amit213 has quit IRC | 10:10 | |
| *** amit213 has joined #openstack-cinder | 10:12 | |
| *** ManishD has joined #openstack-cinder | 10:12 | |
| *** bhagyashris has joined #openstack-cinder | 10:12 | |
| *** alonma has quit IRC | 10:13 | |
| *** alonma has joined #openstack-cinder | 10:14 | |
| *** serverascode has quit IRC | 10:15 | |
| ManishD | HI, I am running into one case:-- 1. Attach FC and ICSI volume to single instance (same instance) -- Successful 2. Detach ICSI volume -- Successful 3. Reattach ICSI volume -- Error :- 24 - iSCSI login failed due to authorization failure | 10:16 |
| ManishD | Please can someone explain | 10:16 |
| ManishD | assist | 10:16 |
| *** bhagyashri has quit IRC | 10:16 | |
| *** Dinesh_Bhor has quit IRC | 10:16 | |
| *** serverascode has joined #openstack-cinder | 10:16 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 10:17 | |
| openstackgerrit | Merged openstack/cinder: Add proxy config option to google backup driver https://review.openstack.org/341090 | 10:18 |
| *** gb21 has joined #openstack-cinder | 10:18 | |
| *** pckizer has quit IRC | 10:19 | |
| *** pckizer has joined #openstack-cinder | 10:19 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 10:19 |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 10:26 |
| *** yangyapeng has quit IRC | 10:26 | |
| *** yangyapeng has joined #openstack-cinder | 10:27 | |
| *** laughterwym has joined #openstack-cinder | 10:33 | |
| *** Guy has quit IRC | 10:33 | |
| *** alonma has quit IRC | 10:37 | |
| *** EinstCrazy has quit IRC | 10:39 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - QoS support for the VMAX3 https://review.openstack.org/307502 | 10:39 |
| *** EinstCrazy has joined #openstack-cinder | 10:39 | |
| *** pdeore has quit IRC | 10:41 | |
| *** EinstCrazy has quit IRC | 10:44 | |
| *** iceyao has quit IRC | 10:45 | |
| *** yangyapeng has quit IRC | 10:46 | |
| *** laughterwym has quit IRC | 10:47 | |
| *** Jaison has quit IRC | 10:49 | |
| *** nikeshm has joined #openstack-cinder | 10:50 | |
| *** links has joined #openstack-cinder | 10:53 | |
| *** sdague has joined #openstack-cinder | 10:55 | |
| openstackgerrit | å¼ ä¸¹é˜³ proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 10:56 |
| openstackgerrit | nikesh proposed openstack/cinder: Ignore requests warnings in K2 cinder driver https://review.openstack.org/349013 | 10:56 |
| *** alonma has joined #openstack-cinder | 10:57 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix FC multipath cleanup https://review.openstack.org/231135 | 10:59 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix the mocking mess https://review.openstack.org/350516 | 10:59 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix weak test_vzstorage_with_mds_list https://review.openstack.org/350517 | 10:59 |
| *** pdeore has joined #openstack-cinder | 11:00 | |
| openstackgerrit | Zhang Danyang proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 11:02 |
| *** laughterwym has joined #openstack-cinder | 11:02 | |
| *** alonma has quit IRC | 11:02 | |
| openstackgerrit | Zhang Danyang proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 11:03 |
| nikeshm | vbala DuncanT: https://review.openstack.org/349013 | 11:08 |
| *** laughterwym has quit IRC | 11:11 | |
| *** alonma has joined #openstack-cinder | 11:11 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 11:16 |
| *** alonma has quit IRC | 11:16 | |
| *** alonmarx has joined #openstack-cinder | 11:17 | |
| *** gb21 has quit IRC | 11:17 | |
| *** Digvijay has quit IRC | 11:20 | |
| openstackgerrit | Zhang Danyang proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 11:21 |
| *** gb21 has joined #openstack-cinder | 11:23 | |
| *** ManishD has quit IRC | 11:28 | |
| *** alyson_ has joined #openstack-cinder | 11:28 | |
| *** alonma has joined #openstack-cinder | 11:36 | |
| openstackgerrit | Zhang Danyang proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 11:39 |
| *** yangyapeng has joined #openstack-cinder | 11:42 | |
| *** Poornima has quit IRC | 11:43 | |
| *** coolsvap_ is now known as coolsvap | 11:45 | |
| *** tpsilva has joined #openstack-cinder | 11:48 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Cleanup upgrade-related FIXME's https://review.openstack.org/350533 | 11:51 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Pin RPC server's serializer to min obj version https://review.openstack.org/350534 | 11:51 |
| *** markvoelker has joined #openstack-cinder | 11:52 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 12:01 |
| *** Lee1092 has quit IRC | 12:03 | |
| *** alonma has quit IRC | 12:05 | |
| *** alonma has joined #openstack-cinder | 12:05 | |
| *** ducttape_ has joined #openstack-cinder | 12:06 | |
| *** alonma has quit IRC | 12:07 | |
| *** alonma has joined #openstack-cinder | 12:07 | |
| *** thurloat has joined #openstack-cinder | 12:07 | |
| *** timcl has joined #openstack-cinder | 12:08 | |
| smcginnis | e0ne: Test runs have started using the previously blocked os-brick 1.4.0 and nothing has dramatically exploded, so I've requested a 1.5.0 release to pick up the latest changes. | 12:09 |
| *** alonma has quit IRC | 12:10 | |
| *** alonma has joined #openstack-cinder | 12:11 | |
| *** pdeore has quit IRC | 12:11 | |
| *** alonmarx has quit IRC | 12:12 | |
| *** alonma_ has joined #openstack-cinder | 12:13 | |
| *** alonma_ has quit IRC | 12:13 | |
| *** alonma_ has joined #openstack-cinder | 12:13 | |
| *** alonma has quit IRC | 12:15 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/350550 | 12:16 |
| JayConroy | Could someone else have a look at this patch for me please? https://review.openstack.org/#/c/347944/ It adds support for vhd disk format for volume upload-to-image. Thanks! | 12:16 |
| *** narayrak has joined #openstack-cinder | 12:16 | |
| *** xyang1 has joined #openstack-cinder | 12:16 | |
| *** gb21 has quit IRC | 12:19 | |
| *** edmondsw has joined #openstack-cinder | 12:19 | |
| *** takedakn has joined #openstack-cinder | 12:22 | |
| openstackgerrit | Merged openstack/cinder: Don't puke when exiting driver list https://review.openstack.org/350269 | 12:23 |
| *** dulek has left #openstack-cinder | 12:25 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Use volume o.vo in get_backup_volume_temp_snapshot https://review.openstack.org/350551 | 12:25 |
| *** dulek has joined #openstack-cinder | 12:25 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Fix ImportError of fake_driver https://review.openstack.org/350552 | 12:25 |
| dulek | https://review.openstack.org/#/c/246600 - looks like no one is brave enough to hit +A here. :D | 12:26 |
| smcginnis | dulek: Hah! I'll take a look and see if I am. ;) | 12:26 |
| scottda | I ain't scared.... | 12:27 |
| smcginnis | :) | 12:27 |
| dulek | scottda: terrified? | 12:27 |
| nikeshm | smcginnis: hi | 12:27 |
| smcginnis | nikeshm: hey | 12:27 |
| scottda | ha | 12:28 |
| dulek | smcginnis: https://bugs.launchpad.net/cinder/+bug/1609208 - is that the bug you're fixing with your most recent commit? | 12:28 |
| openstack | Launchpad bug 1609208 in Cinder "No module named fake_driver" [Undecided,In progress] - Assigned to Lisa Li (lisali) | 12:28 |
| nikeshm | smcginnnis: if yu get time have a look https://review.openstack.org/#/c/348932 | 12:28 |
| nikeshm | smcginnis: one +2 there | 12:28 |
| smcginnis | nikeshm: Sure | 12:28 |
| xyang1 | dulek: I was going to check with you to see if you are working on an update | 12:29 |
| smcginnis | dulek: Doh, didn't realize she already found that and had a patch up. | 12:29 |
| smcginnis | Multiple bugs for the same issue... | 12:29 |
| *** gouthamr has joined #openstack-cinder | 12:30 | |
| nikeshm | smcginnis: also is this acceptible https://review.openstack.org/#/c/350182/4/cinder/volume/drivers/kaminario/kaminario_common.py line no 63 | 12:30 |
| nikeshm | unit tests was failing so i did | 12:31 |
| nikeshm | like that | 12:31 |
| xyang1 | dulek: I'd rather have yours merged now so I can update my patch | 12:31 |
| nikeshm | in unit tests i am not using krest | 12:31 |
| dulek | xyang1: Oh, update on the issues Szymon mentioned on previous patchset. I've missed them. | 12:31 |
| *** dulek has left #openstack-cinder | 12:32 | |
| *** dulek has joined #openstack-cinder | 12:32 | |
| dulek | xyang1: Yeah, these made sense, I'll do a follow up patch. | 12:33 |
| xyang1 | dulek: thanks. waiting for yours to merge, then I'll need to change my patch based on it | 12:34 |
| openstackgerrit | Merged openstack/python-cinderclient: Changed backup-restore to accept backup name https://review.openstack.org/344941 | 12:34 |
| smcginnis | nikeshm: I guess so. Will add to the list to look at. | 12:35 |
| *** dustins has joined #openstack-cinder | 12:36 | |
| *** sdake has joined #openstack-cinder | 12:37 | |
| *** narayrak has quit IRC | 12:41 | |
| *** akerr has joined #openstack-cinder | 12:41 | |
| *** iceyao has joined #openstack-cinder | 12:41 | |
| *** diablo_rojo has joined #openstack-cinder | 12:44 | |
| e0ne | smcginnis: good news, thanks Sean! | 12:47 |
| *** julim has quit IRC | 12:48 | |
| adrianofr | dulek: ping | 12:50 |
| dulek | adrianofr: Hi! | 12:51 |
| adrianofr | dulek: Hey :) | 12:52 |
| adrianofr | I'm fixing this https://review.openstack.org/#/c/285469/46/releasenotes/notes/hnas-drivers-refactoring-9dbe297ffecced21.yaml according to your comments | 12:52 |
| *** abhinavtechie has quit IRC | 12:52 | |
| *** narayrak has joined #openstack-cinder | 12:52 | |
| adrianofr | dulek: But what command do you use to check reno format? | 12:52 |
| dulek | adrianofr: Just run "tox -e releasenotes". :) | 12:52 |
| adrianofr | dulek: I guess this is not working properly in my env then :/ | 12:53 |
| adrianofr | dulek: it's giving me success even without changing my reno | 12:53 |
| adrianofr | that one that jenkins didn't like | 12:53 |
| claudiub | hemna: hello. if you have some time, can you take a look at the windows os-brick fc and smb connectors? They have a working ci, they already have a +2 from smcginnis. Thanks! https://review.openstack.org/#/c/323780/ https://review.openstack.org/#/c/323781/ | 12:54 |
| dulek | adrianofr: You can also check out how the note itself looks like in cinder/releasenotes/build/html/index.html | 12:54 |
| *** narayrak has quit IRC | 12:54 | |
| dulek | adrianofr: reno (app we use for release notes) works on git repos and commits, so you need to have all the changes committed to have them included in the build. | 12:54 |
| *** narayrak has joined #openstack-cinder | 12:54 | |
| dulek | adrianofr: I mean - committed to your local repo, not sent to Gerrit. | 12:55 |
| adrianofr | dulek: hmmm. Maybe this is the problem | 12:55 |
| adrianofr | I was just adding the reno in the env and running the command | 12:55 |
| dulek | adrianofr: Most certainly that's it. | 12:56 |
| adrianofr | dulek: I guess it was not considering my reno then | 12:56 |
| adrianofr | dulek: Thanks for the help :) | 12:56 |
| dulek | adrianofr: http://docs.openstack.org/developer/cinder/devref/releasenotes.html | 12:56 |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 12:56 |
| openstackgerrit | Merged openstack/cinder: DRBD: Disk-options for new resources https://review.openstack.org/347267 | 12:58 |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/350475 | 12:58 |
| *** alonma_ has quit IRC | 12:59 | |
| openstackgerrit | nikesh proposed openstack/cinder: Ignore requests warnings in K2 cinder driver https://review.openstack.org/349013 | 13:00 |
| *** alonma has joined #openstack-cinder | 13:01 | |
| *** rlrossit has joined #openstack-cinder | 13:02 | |
| *** pauloewerton has joined #openstack-cinder | 13:04 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove Scality volume driver https://review.openstack.org/348042 | 13:05 |
| *** alonma has quit IRC | 13:05 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 13:06 |
| *** mriedem has joined #openstack-cinder | 13:06 | |
| *** alonma has joined #openstack-cinder | 13:08 | |
| *** diogogmt has quit IRC | 13:09 | |
| *** dave-mccowan has joined #openstack-cinder | 13:10 | |
| *** diogogmt has joined #openstack-cinder | 13:10 | |
| *** ducttape_ has quit IRC | 13:11 | |
| *** alonma has quit IRC | 13:13 | |
| openstackgerrit | nikesh proposed openstack/cinder: Ignore requests warnings in K2 cinder driver https://review.openstack.org/349013 | 13:14 |
| *** alonma has joined #openstack-cinder | 13:16 | |
| *** porrua has joined #openstack-cinder | 13:17 | |
| *** markvoelker has quit IRC | 13:19 | |
| *** alonma has quit IRC | 13:20 | |
| *** Lee1092 has joined #openstack-cinder | 13:21 | |
| openstackgerrit | Merged openstack/os-brick: Splitting Out Connectors from connector.py https://review.openstack.org/307974 | 13:22 |
| Dinesh_Bhor | Hi all, Can someone please take a look at it: https://review.openstack.org/#/c/350393/ | 13:23 |
| *** alonma has joined #openstack-cinder | 13:23 | |
| *** pgadiya has quit IRC | 13:25 | |
| *** alonma has quit IRC | 13:27 | |
| *** jwcroppe has quit IRC | 13:28 | |
| *** takedakn has quit IRC | 13:28 | |
| *** nkrinner is now known as nkrinner_afk | 13:29 | |
| *** alonma has joined #openstack-cinder | 13:30 | |
| *** belmoreira has quit IRC | 13:31 | |
| *** diogogmt has quit IRC | 13:33 | |
| *** faiz89 has joined #openstack-cinder | 13:33 | |
| *** alonma has quit IRC | 13:35 | |
| *** asselin has joined #openstack-cinder | 13:36 | |
| *** alonma has joined #openstack-cinder | 13:41 | |
| *** jwcroppe has joined #openstack-cinder | 13:42 | |
| *** arecknag has quit IRC | 13:44 | |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 13:46 |
| *** markvoelker has joined #openstack-cinder | 13:46 | |
| *** ebalduf has joined #openstack-cinder | 13:53 | |
| *** eharney has joined #openstack-cinder | 13:54 | |
| *** akshai has joined #openstack-cinder | 13:54 | |
| openstackgerrit | Merged openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 13:55 |
| openstackgerrit | Merged openstack/cinder: Fix ImportError of fake_driver https://review.openstack.org/350357 | 13:55 |
| *** faiz89 has quit IRC | 13:58 | |
| *** merooney has joined #openstack-cinder | 13:58 | |
| *** Yogi1 has joined #openstack-cinder | 13:58 | |
| *** timcl1 has joined #openstack-cinder | 13:59 | |
| *** faiz89 has joined #openstack-cinder | 14:00 | |
| *** krot_sickleave is now known as krotscheck | 14:00 | |
| *** akshai has quit IRC | 14:00 | |
| *** tongli has joined #openstack-cinder | 14:01 | |
| *** esker has joined #openstack-cinder | 14:01 | |
| *** akshai has joined #openstack-cinder | 14:02 | |
| *** timcl has quit IRC | 14:02 | |
| *** zhipeng has joined #openstack-cinder | 14:02 | |
| *** laughterwym has joined #openstack-cinder | 14:03 | |
| *** eharney_ has joined #openstack-cinder | 14:04 | |
| *** eharney has quit IRC | 14:04 | |
| *** tongli has quit IRC | 14:04 | |
| *** tongli has joined #openstack-cinder | 14:05 | |
| *** jwcroppe has quit IRC | 14:05 | |
| *** ducttape_ has joined #openstack-cinder | 14:06 | |
| *** zhipeng has quit IRC | 14:06 | |
| *** zhipeng has joined #openstack-cinder | 14:07 | |
| *** lkuchlan has joined #openstack-cinder | 14:07 | |
| *** lkuchlan has quit IRC | 14:13 | |
| *** cdelatte has joined #openstack-cinder | 14:21 | |
| *** links has quit IRC | 14:23 | |
| *** claudiub has quit IRC | 14:24 | |
| *** dkehn_ has quit IRC | 14:25 | |
| *** chris_morrell has joined #openstack-cinder | 14:30 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 14:33 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 14:33 |
| *** dkehn_ has joined #openstack-cinder | 14:38 | |
| *** iceyao has quit IRC | 14:39 | |
| *** mtanino has joined #openstack-cinder | 14:40 | |
| *** diogogmt has joined #openstack-cinder | 14:40 | |
| *** zhipeng has quit IRC | 14:42 | |
| *** zhipeng has joined #openstack-cinder | 14:43 | |
| *** knikolla has joined #openstack-cinder | 14:48 | |
| *** jungleboyj has joined #openstack-cinder | 14:49 | |
| jungleboyj | FYI ... the GPFS CI system lost a disk so it is down while they attempt to recover it. | 14:49 |
| *** akapil has joined #openstack-cinder | 14:49 | |
| *** eharney_ is now known as eharney | 14:51 | |
| *** xinli has joined #openstack-cinder | 14:53 | |
| scottda | geguileo: I'm testing the cinderclient cluster commands and getting an error: | 14:53 |
| geguileo | scottda: What error? | 14:53 |
| scottda | https://www.irccloud.com/pastebin/LQIxUFFE/ | 14:53 |
| geguileo | scottda: I haven't tested it in a while... | 14:53 |
| scottda | geguileo: Yes, it worked previously, so some kind of regression... | 14:54 |
| geguileo | :'-( | 14:54 |
| geguileo | Ok, I'll work on it | 14:54 |
| scottda | geguileo: thanks. I -2'd the patch, since it looked ready to merge and we probably want to hold off. | 14:55 |
| geguileo | scottda: Good idea | 14:55 |
| *** karthikp_ has joined #openstack-cinder | 14:55 | |
| * geguileo really needs to start working again in the HA A/A feature | 14:55 | |
| smcginnis | jungleboyj: Darn storage. | 14:58 |
| jungleboyj | smcginnis: Oh the irony. | 14:59 |
| *** dulek_ has joined #openstack-cinder | 14:59 | |
| scottda | eharney, xyang1, gouthamr, akerr, smcginnis, cFouts, e0ne, geguileo, dulek, flip214, ntpttr patrickeast, _alastor_, DuncanT, erlon Would you like to talk about cinder testing? | 14:59 |
| eharney | hey | 14:59 |
| e0ne | hi | 14:59 |
| smcginnis | Sure, why not. :) | 14:59 |
| geguileo | o/ | 14:59 |
| akerr | hi | 14:59 |
| erlon | scottda: Hi | 15:00 |
| *** esker has quit IRC | 15:00 | |
| xyang1 | hi | 15:00 |
| patrickeast | Yo | 15:00 |
| e0ne | #link https://etherpad.openstack.org/p/Cinder-testing | 15:00 |
| cFouts | hi | 15:00 |
| scottda | What did I call this meeting? cinder-testing? | 15:00 |
| *** daneyon has joined #openstack-cinder | 15:00 | |
| scottda | #startmeeting cinder-testing | 15:01 |
| smcginnis | scottda: I think so. | 15:01 |
| openstack | Meeting started Wed Aug 3 15:01:10 2016 UTC and is due to finish in 60 minutes. The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| gouthamr | hey | 15:01 |
| openstack | The meeting name has been set to 'cinder_testing' | 15:01 |
| _alastor_ | hey | 15:01 |
| scottda | I'd hoped to go through all the patches listed in the etherpad and update stauts, etc.....I'll do that today | 15:02 |
| e0ne | I'll propose patches with fake drivers refactoring to remove duplication, devstack integration later this week | 15:03 |
| scottda | We'd discussed some stuff at the mid-cycle. Notes are here: https://etherpad.openstack.org/p/newton-cinder-midcycle-day2 | 15:04 |
| e0ne | also, I'm going to install only cinder+keystone for the functional job | 15:04 |
| scottda | e0ne: +1 | 15:04 |
| e0ne | I'm going to send mail to openstack-dev once pathces will be ready for review | 15:04 |
| eharney | keystone for the functional job? | 15:05 |
| *** nkrinner_afk has quit IRC | 15:05 | |
| e0ne | eharney: TBH, I didn't try to run cinder w/o keystone yet | 15:05 |
| eharney | that doesn't sound like it fits with what we defined functional tests as two weeks ago | 15:05 |
| e0ne | eharney: it worked few years ago, but we need to check and fix it if needed | 15:05 |
| smcginnis | eharney: Right | 15:06 |
| e0ne | eharney: I agree that usage of 'noauth' will be better for functional tests | 15:06 |
| *** mjura has quit IRC | 15:06 | |
| *** jungleboyj has quit IRC | 15:06 | |
| eharney | at the midcycle, the agreement was that functional tests will work in an environment identical to unit tests today | 15:07 |
| DuncanT | noauth is totally broken in cinder | 15:07 |
| scottda | e0ne: Is it much work to test 'noauth' before moving on with this? | 15:07 |
| *** akapil has quit IRC | 15:07 | |
| e0ne | scottda: I didn't try to test it yet:( | 15:07 |
| scottda | DuncanT: Well that answers my question... | 15:07 |
| DuncanT | The easiest way forward with that would be to write a fake keystone middleware that just returns constant values | 15:07 |
| *** guy has joined #openstack-cinder | 15:07 | |
| smcginnis | eharney: My understanding as well. Functional should be extended unit tests. | 15:08 |
| DuncanT | Functional tests will have a real db, right? | 15:08 |
| e0ne | smcginnis, eharney: I propose to do it step-by-step | 15:08 |
| eharney | that was already a departure from the previous definition of functional... now it looks like we're drifting elsewhere | 15:08 |
| e0ne | DuncanT: +1 | 15:08 |
| *** akshai has quit IRC | 15:09 | |
| e0ne | eharney: IMO, it's better to have keystone for the beginning and and functional tests with fake drivers rather than fixing noauth or fake middleware few months more | 15:09 |
| *** enriquetaso has joined #openstack-cinder | 15:10 | |
| eharney | for it to be "better" i think we need to define what these tests are supposed to do... which we did, i thought | 15:10 |
| openstackgerrit | Merged openstack/cinder: Size in tintri driver should be converted to integer https://review.openstack.org/325025 | 15:11 |
| smcginnis | We did. And there is no keystone as part of it. | 15:11 |
| *** dulek_ has quit IRC | 15:11 | |
| e0ne | #link https://github.com/openstack/cinder/blob/master/doc/source/devref/testing.rst#functional-tests | 15:11 |
| smcginnis | That does need to be updated to be more specific. | 15:11 |
| *** dulek_ has joined #openstack-cinder | 15:12 | |
| e0ne | smcginnis, eharney: what about rabbitmq? | 15:12 |
| eharney | this is where we came up with the idea of "small-scale integration" which is an environment with Cinder+rabbit+db+Keystone and nothing else | 15:12 |
| *** dulek_ has quit IRC | 15:12 | |
| geguileo | This was said in the meeting (etherpad): | 15:12 |
| geguileo | The original idea was to just have, cider, rabbit, and mysql | 15:12 |
| geguileo | Anything more than that should go into tempest | 15:12 |
| e0ne | geguileo: +1 | 15:12 |
| eharney | "original" | 15:13 |
| *** dulek_ has joined #openstack-cinder | 15:13 | |
| geguileo | I'm ok to add Keytone for a little bit as long as it's clear that it should go away | 15:13 |
| *** guy has quit IRC | 15:14 | |
| geguileo | Although I think it would be better to get it right from the start | 15:14 |
| eharney | i don't think it's clear at all at the moment | 15:14 |
| Swanson | xyang1, thanks for the review | 15:14 |
| smcginnis | No keystone. These should basically just be like unit testing, but the scope is to test more than a "unit". | 15:14 |
| xyang1 | Swanson: np | 15:14 |
| patrickeast | IMO if we add it in now it's harder to take out later... And more than likely won't happen | 15:14 |
| scottda | If noone has time for the keystone work ATM, it pushes changes to functional tests out. Maybe no next release? | 15:15 |
| patrickeast | smcginnis: +1 | 15:15 |
| smcginnis | I would even say no mysql and rabbitmq. Those don't make sense in that context. | 15:15 |
| eharney | smcginnis: right | 15:15 |
| *** leeantho has joined #openstack-cinder | 15:15 | |
| Swanson | Next time it will be 2000 1 line patches. | 15:15 |
| geguileo | I may be getting a little lost here | 15:15 |
| geguileo | Then what are we going to test? | 15:15 |
| geguileo | Are we going to fake all DB access and messaging? | 15:15 |
| e0ne | scottda: I'll take a look on it | 15:15 |
| eharney | the idea was to have unit tests be unit tests and huge run-through-ten-modules tests be in functional tests | 15:16 |
| *** julim has joined #openstack-cinder | 15:16 | |
| e0ne | scottda: I mean how to run cinder w/o keystone | 15:16 |
| smcginnis | Anything outside of Cinder should be mocked. | 15:16 |
| eharney | but they are essentially the same test setup, just with different styles of tests | 15:16 |
| smcginnis | Just like we do for unit testing - anything outside the unit we are testing should be mocked. | 15:16 |
| e0ne | smcginnis: are you talking about unit tests? | 15:16 |
| xyang1 | Swanson: that will bump yiur stats! | 15:16 |
| smcginnis | e0ne: Functional testing. | 15:16 |
| geguileo | smcginnis: So we won't be testing in functional tests anything that goes from API to Scheduler and then to the volume, right? | 15:16 |
| smcginnis | geguileo: Correct. | 15:16 |
| *** nkrinner_afk has joined #openstack-cinder | 15:17 | |
| smcginnis | Although it can go from the API level through, but not a running deployment. | 15:17 |
| e0ne | smcginnis: :( | 15:17 |
| smcginnis | Just calling the methods. | 15:17 |
| e0ne | smcginnis: it makes functional tests not useful, IMO | 15:17 |
| smcginnis | e0ne: It makes them extremely useful. | 15:17 |
| e0ne | smcginnis: it becases the same as unit + DB | 15:17 |
| smcginnis | And give us a way to get out of the mess that our current unit tests are. | 15:18 |
| eharney | so i posted a question on the etherpad about this issue as well | 15:18 |
| *** akshai has joined #openstack-cinder | 15:18 | |
| smcginnis | Unit tests should actually be unit tests - but right now they are not. | 15:18 |
| eharney | i think having the current (midcycle) functional tests be a separate job is problematic | 15:18 |
| nikeshm | smcginnis: are yu familiar with Traceback in http://54.209.116.144/19/349019/13/check/kaminario-dsvm-tempest-full-iscsi/f871021/logs/screen-c-vol.txt.gz | 15:18 |
| smcginnis | The next level up will allow us to test interaction between different modules. | 15:19 |
| nikeshm | i m getting today | 15:19 |
| *** RageLtMan has joined #openstack-cinder | 15:19 | |
| scottda | eharney: Yeah, the py3 issue makes sense as a problem/blocker | 15:19 |
| nikeshm | this in iscsi | 15:19 |
| nikeshm | job | 15:19 |
| nikeshm | when running | 15:19 |
| eharney | if we're doing this version of functional tests, they should probably be run in the same jobs that run the unit tests | 15:19 |
| smcginnis | eharney: I don't think so. | 15:19 |
| *** jungleboyj has joined #openstack-cinder | 15:19 | |
| *** akapil has joined #openstack-cinder | 15:19 | |
| eharney | smcginnis: there are at least two downsides to having them in a separate job, i'm not sure what the upsides are | 15:20 |
| nikeshm | any one familiar with Traceback in http://54.209.116.144/19/349019/13/check/kaminario-dsvm-tempest-full-iscsi/f871021/logs/screen-c-vol.txt.gz | 15:20 |
| smcginnis | It's a different scope of testing. Unit tests should be our basic level of tests run. | 15:20 |
| smcginnis | Functional expands on that, but is good to separate so they are not run for basic validation. | 15:21 |
| eharney | sure, but to do this right, we need py27 and py3 functional jobs | 15:21 |
| smcginnis | That's true. | 15:21 |
| eharney | and i'd like to be able to see coverage stats for both combined, but i guess that can be sorted out in tox.ini somehow outside of how we do jobs | 15:21 |
| jgriffith | nikeshm: looks like privsep stuff :( | 15:21 |
| *** vbala has quit IRC | 15:21 | |
| *** akapil has quit IRC | 15:22 | |
| *** akapil has joined #openstack-cinder | 15:22 | |
| eharney | so i guess we'll just make a functional-py3 job then? | 15:22 |
| *** dkehn_ has quit IRC | 15:22 | |
| e0ne | I'm afraid to ask, but... | 15:23 |
| *** sheel has joined #openstack-cinder | 15:23 | |
| e0ne | if we are so interested in functional-py3 jib, why nobody don't talk about tempest-py3 job? | 15:23 |
| jgriffith | e0ne: +1 | 15:23 |
| eharney | sounds like a good idea to me | 15:24 |
| smcginnis | Maybe "integration" testing would be a better name for these. Unit testing for units of code, integration for the interaction between these units, functional then defined as run against a running instance, | 15:24 |
| *** dave-mccowan has quit IRC | 15:24 | |
| nikeshm | jgriffith: any solution to avoid that in our CI | 15:24 |
| smcginnis | e0ne: DOn't we have that already? | 15:24 |
| geguileo | nikeshm: I think the problem is that scsi_transport_fc module is not loaded in the system | 15:24 |
| e0ne | smcginnis: maybe in an experimental queue only | 15:24 |
| eharney | smcginnis: i think that's backwards from normal terminology which is part of what jgriffith was trying to fix | 15:24 |
| e0ne | integration between modules sounds interesting | 15:24 |
| jgriffith | geguileo: indeed, but I can't figure out if it failed to load because of privsep error or just wasn't included? | 15:25 |
| e0ne | but ingetration between different cinder components should be done too | 15:25 |
| dulek_ | haypo was talking that py3 tempest is another step in his conversion efforts. | 15:25 |
| Swanson | smcginnis, integration tests are for testing against a running instance. | 15:25 |
| geguileo | jgriffith: True | 15:25 |
| smcginnis | Swanson: It seems most folks have different definitions of integration vs testing. | 15:25 |
| smcginnis | * vs functional. | 15:25 |
| jgriffith | sigh.. here we go again | 15:25 |
| e0ne | I won't hollywar ot bukeshed on what is func. testing | 15:25 |
| jgriffith | e0ne: +1 | 15:25 |
| geguileo | e0ne: I agree, but maybe those should be integration tests to differentiate them from functional | 15:26 |
| e0ne | I'll propose my patches and send links to the mailing list | 15:26 |
| smcginnis | But I thought we were being consistent with what other projects were calling the types of testing. | 15:26 |
| jgriffith | Call it other-testing AFAIAC this is rather silly | 15:26 |
| eharney | well... "Tempest - The OpenStack Integration Test Suite" already exists, so let's not redefine it to something else | 15:26 |
| smcginnis | jgriffith: +a | 15:26 |
| jgriffith | smcginnis: yes, that was the whole point and the hour long discussion in Ft Collins | 15:26 |
| xyang1 | e0ne: jgriffith have we decided on the fake driver name yet | 15:26 |
| smcginnis | As long as we differentiate the type of testing for one vs the other. | 15:26 |
| jgriffith | smcginnis: which for some reason we've decided we need to talk about and rhash again | 15:26 |
| smcginnis | jgriffith: I agree. I thought we had it all set. But apparently not. | 15:27 |
| geguileo | eharney: Sure, but it also says: "This is a set of integration tests to be run against a live OpenStack cluster." | 15:27 |
| Swanson | the jgriffith memorial test suite. | 15:27 |
| e0ne | xyang1, jgriffith: AFAIR, we desided to use FakeLoggingDriver and FakeGateDriver names | 15:27 |
| smcginnis | geguileo: Stale comments in the etherpad I believe. | 15:27 |
| xyang1 | e0ne: ok | 15:27 |
| smcginnis | https://www.youtube.com/channel/UCJ8Koy4gsISMy0qW3CWZmaQ | 15:28 |
| geguileo | smcginnis: Not really, it's from here http://docs.openstack.org/developer/tempest/overview.html | 15:28 |
| e0ne | e.g. python-heatclient funcitonal tests http://logs.openstack.org/79/345379/12/check/gate-heatclient-dsvm-functional/14d3a7e/ | 15:28 |
| e0ne | #link http://logs.openstack.org/79/345379/12/check/gate-heatclient-dsvm-functional/14d3a7e/ | 15:28 |
| geguileo | eharney: And we are talking about integration tests where we should be able to have an error generator, right? | 15:28 |
| scottda | smcginnis: How about you post a devref patch that states what you/we think are the definitions, we review and, once merged, we take that as the definitions? | 15:28 |
| e0ne | I like how Heat team implemented funcitonal tests | 15:28 |
| geguileo | scottda: +1 | 15:28 |
| eharney | e0ne: what do those tests do? | 15:28 |
| nikeshm | geguileo: jgriffith: do we need that module in iscsi driver testing | 15:29 |
| smcginnis | geguileo: OK, then back to the four levels we defined: unit tests, functional tests, integration tests, tempest. | 15:29 |
| e0ne | eharney: they setup minimal devstack with heat components only | 15:29 |
| jgriffith | scottda: I'll post it, but frankly you were there in the meeting and I specifically asked you if you understood and agreed. You did | 15:29 |
| geguileo | smcginnis: I'm fine with that | 15:29 |
| eharney | e0ne: this is what i was proposing as "small scale integration" | 15:29 |
| *** xinli has quit IRC | 15:29 | |
| jgriffith | eharney: then propose that, don't stop progress on something else that somebody has started (ie e0ne ) | 15:30 |
| scottda | jgriffith: It's not about agreement/disagreement, it's about getting it into the devref so we all know what the defs are. | 15:30 |
| eharney | jgriffith: i'm not stopping progress on anything | 15:30 |
| jgriffith | scottda: fine, devref patch coming up | 15:30 |
| jgriffith | eharney: ok | 15:30 |
| jgriffith | eharney: sorry | 15:30 |
| eharney | according to what we defined at the midcycle, the functional test environment is done and just needs tests, not more components added | 15:30 |
| *** xinli has joined #openstack-cinder | 15:30 | |
| jgriffith | eharney: +1 | 15:30 |
| smcginnis | eharney: +1 | 15:31 |
| geguileo | nikeshm: In the logs I see "Fetching connector for FibreChannelConnector get_connector_properties" before the error | 15:31 |
| e0ne | and we will have 10 functional, 3 "small scale integration" tests in the end of O release | 15:31 |
| *** timcl1 has quit IRC | 15:32 | |
| jgriffith | scottda: can you clarify what's missing from the devref for me? | 15:32 |
| e0ne | I don't want to spray attention on too many of test types | 15:32 |
| geguileo | nikeshm: http://54.209.116.144/19/349019/13/check/kaminario-dsvm-tempest-full-iscsi/f871021/logs/screen-c-vol.txt.gz#_2016-08-03_14_49_56_929 | 15:32 |
| scottda | https://www.irccloud.com/pastebin/4qBAq2bp/ | 15:32 |
| *** dkehn_ has joined #openstack-cinder | 15:32 | |
| jgriffith | scottda: http://docs.openstack.org/developer/cinder/devref/testing.html | 15:32 |
| scottda | Do we agree that functional tests run with a database? | 15:33 |
| smcginnis | That last section should be removed, IMO. | 15:33 |
| scottda | I thought that was contentious? | 15:33 |
| scottda | smcginnis: Ok, so that needs changing | 15:33 |
| xyang1 | e0ne: do you have any patch that moves the fake driver? or are we all set using the existing LoggingDriver to write the functional tests? | 15:33 |
| jgriffith | smcginnis: the non-cinder services part? | 15:33 |
| smcginnis | And we should add Integration Tests with a description of the "small scale integration" work being done now. | 15:33 |
| smcginnis | jgriffith: The "database present and may start Cinder services to accept requests" I think. | 15:33 |
| e0ne | xyang1: I don't know now | 15:33 |
| jgriffith | ok | 15:34 |
| scottda | And messageQ will run with "small scale integration", right? | 15:34 |
| scottda | We also have nothing in the devref to differentiate between in-tree tempest and upstream | 15:35 |
| eharney | in-tree tempest vs upstream tempest is not a very interesting distinction IMO | 15:35 |
| xyang1 | e0ne: ok, I'll start with the existing LoggingDriver and will change if needed | 15:36 |
| eharney | they're tests run in the same environments, it's just where the code lives | 15:36 |
| scottda | eharney: So how to people know where the tests should go? | 15:36 |
| e0ne | xyang1: ok. I'll ping you if i have any update on fake frivers | 15:36 |
| eharney | scottda: yeah, true, we do need to document that | 15:36 |
| xyang1 | e0ne: ok, thanks | 15:37 |
| *** timcl has joined #openstack-cinder | 15:37 | |
| scottda | I don't recall coming to a decision at the mid-cycle around how to decide in-tree vs. out-of-tree for tempest? | 15:38 |
| eharney | my working assumption is that in-tree is for things that we want to test that is outside of the scope of what tempest wants | 15:38 |
| eharney | (what tempest wants as a project etc) | 15:39 |
| jgriffith | smcginnis: scottda https://gist.github.com/j-griffith/063574aac688d9a383bf6fe4a50f00c9 | 15:39 |
| scottda | eharney: Yes, that seems correct. But I think we need to codify that and put that in the devref. | 15:39 |
| *** itlinux has joined #openstack-cinder | 15:39 | |
| jgriffith | does that work for the Functional section at least? | 15:39 |
| scottda | jgriffith: Doesn't that mean we need messageQ running ? | 15:40 |
| smcginnis | jgriffith: I would think even less than that. It wouldn't necessarily be all the way from API to driver. Could just be between two modules. | 15:40 |
| *** karthikp__ has joined #openstack-cinder | 15:40 | |
| jgriffith | smcginnis: ok, but just to be clear, there's no mechanism in there right now to fake those sorts of things out | 15:42 |
| jgriffith | smcginnis: it's indeed "functional" in that it uses a client and send REAL API cmds | 15:43 |
| jgriffith | s/send/sends/ | 15:43 |
| smcginnis | jgriffith: I think that's why it needs to state a smaller scope. Just unit test style tests that validate code paths between more than just a unit of code. | 15:43 |
| e0ne | jgriffith: +1 | 15:43 |
| smcginnis | They almost fit "integration" testing in my definition more than "functional". | 15:43 |
| jgriffith | I'm no longer interested in trying to drive this to concensus or to try and *force* my opinion or the methodology used in other OpenStack projects | 15:44 |
| e0ne | smcginnis: you've just described most of our "unit | 15:44 |
| e0ne | " tests | 15:44 |
| eharney | e0ne: which is why we want to move a lot of unit tests to functional tests | 15:44 |
| smcginnis | e0ne: Right, which is aproblem. | 15:44 |
| smcginnis | A lot of our unit tests right now are not unit tests. | 15:44 |
| e0ne | jgriffith: +1 :( | 15:44 |
| jgriffith | rather than bike-shed on this why don't people write/propose tests and see how things go? | 15:45 |
| *** pece has joined #openstack-cinder | 15:45 | |
| dulek_ | jgriffith: +1 | 15:45 |
| scottda | Because then we'll keep having this conversation... | 15:45 |
| smcginnis | jgriffith: I think we need at least a general consensus so it's not chaos. But we can try that. | 15:45 |
| jgriffith | scottda: not if you don't keep bringing it up | 15:45 |
| hemna | mornin | 15:45 |
| scottda | Why not just draw the lines somewhere, write it down (in the devref) and be done with it. | 15:45 |
| jgriffith | I did and we've done nothing but argue about it :) | 15:46 |
| eharney | scottda: i think we have a good idea of what to propose in the devref now, seems like a good idea | 15:46 |
| scottda | That's why I said, smcginnis put up a patch with the definitions and be done with it. | 15:46 |
| smcginnis | jgriffith: I can try proposing something to the devref if you'd prefer. Then we can bikeshed on that. :) | 15:46 |
| jgriffith | Ok | 15:46 |
| scottda | or jgriffith or whomever. I don't care what the defs are, just define them once and for all | 15:46 |
| jgriffith | smcginnis: as long as we have something to bikeshed about :) | 15:46 |
| geguileo | smcginnis: +1 | 15:46 |
| jgriffith | smcginnis: I'll pass the torch to you, because frankly I can't figure out how to appease all of this | 15:47 |
| *** laughterwym has quit IRC | 15:47 | |
| smcginnis | ;0 | 15:47 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_WIKI_NAME to all drivers https://review.openstack.org/348002 | 15:48 |
| scottda | #action smcginnis Will put up a devref patch with definitions for unit, functional, small scale integrations, in-tree vs. out-of-tree tempest | 15:49 |
| scottda | smcginnis: OK? | 15:49 |
| smcginnis | scottda: +1 | 15:49 |
| *** akapil has quit IRC | 15:49 | |
| scottda | Anyone have anything else today? Or shall be break before the Next Exciting Meeting? | 15:50 |
| scottda | #endmeeting | 15:50 |
| openstack | Meeting ended Wed Aug 3 15:50:50 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-03-15.01.html | 15:50 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-03-15.01.txt | 15:50 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-03-15.01.log.html | 15:50 |
| smcginnis | scottda: Thanks | 15:50 |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows Fibre Channel connector https://review.openstack.org/323780 | 15:57 |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows SMBFS connector https://review.openstack.org/323781 | 15:57 |
| *** locust has quit IRC | 15:57 | |
| *** JoseMello has joined #openstack-cinder | 15:57 | |
| *** akshai has quit IRC | 15:59 | |
| flip214 | meeting? | 15:59 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-brick-cinderclient-ext: Add reno for release notes management https://review.openstack.org/350662 | 15:59 |
| hemna | heh | 15:59 |
| hemna | merge conflict city in os-brick patches now | 15:59 |
| e0ne | hemna: is't refactoring! | 16:00 |
| *** baumann has joined #openstack-cinder | 16:00 | |
| hemna | yah, we finally got that one to land | 16:01 |
| *** laughterwym has joined #openstack-cinder | 16:03 | |
| *** sv7577 has joined #openstack-cinder | 16:03 | |
| *** bardia has joined #openstack-cinder | 16:03 | |
| *** bardia has quit IRC | 16:04 | |
| *** pbandark has quit IRC | 16:06 | |
| *** jistr is now known as jistr|biab | 16:06 | |
| *** Yogi1 has quit IRC | 16:09 | |
| *** bardia has joined #openstack-cinder | 16:12 | |
| *** bardia has quit IRC | 16:13 | |
| *** sv7577 has quit IRC | 16:14 | |
| *** alonma has quit IRC | 16:19 | |
| *** seldenr has joined #openstack-cinder | 16:20 | |
| *** jdurgin1 has joined #openstack-cinder | 16:21 | |
| *** jistr|biab is now known as jistr | 16:22 | |
| flip214 | DuncanT: re 347274: I was *soooo* happy that all the voting CIs finally said "okay" that I forgot to look at the DRBD output ;/ | 16:25 |
| DuncanT | flip214: Ha :-) | 16:25 |
| *** alonma has joined #openstack-cinder | 16:27 | |
| *** faiz89 has quit IRC | 16:27 | |
| *** dansmith has quit IRC | 16:27 | |
| *** Apoorva has joined #openstack-cinder | 16:27 | |
| *** zhipeng has quit IRC | 16:27 | |
| flip214 | DuncanT: the later patches in that series are okay, though, so I guess it's only a temporary problem. | 16:28 |
| *** dansmith has joined #openstack-cinder | 16:28 | |
| *** dansmith is now known as Guest20988 | 16:28 | |
| *** zhipeng has joined #openstack-cinder | 16:28 | |
| flip214 | if it's really that patch, just approve the next one too ;) | 16:28 |
| openstackgerrit | Merged openstack/cinder: Support replication in K2 https://review.openstack.org/348932 | 16:28 |
| *** Guest20988 is now known as dansmith | 16:29 | |
| *** akshai has joined #openstack-cinder | 16:30 | |
| flip214 | especially, I wouldn't know how *that* patch might be wrong... | 16:30 |
| *** alonma has quit IRC | 16:31 | |
| *** akshai_ has joined #openstack-cinder | 16:32 | |
| *** Guest6612 has joined #openstack-cinder | 16:32 | |
| *** narayrak has quit IRC | 16:33 | |
| DuncanT | flip214: Recheck is certainly an option, but if you've no way to figure out what happened, it might indicate that more logging in the CI would be useful :-) | 16:33 |
| *** sdague has quit IRC | 16:34 | |
| *** wilson_liu has quit IRC | 16:35 | |
| *** akshai has quit IRC | 16:35 | |
| flip214 | DuncanT: I can tell you more details tomorrow, if you like. (I hope ;) | 16:36 |
| DuncanT | flip214: Great, thanks. I've a natural aversion to recheck-and-hope, but I generally understand that sometimes it is the only way. Nice to know what extra info would be useful though | 16:37 |
| *** rcernin has quit IRC | 16:38 | |
| *** laughterwym has quit IRC | 16:38 | |
| erlon | xyang1: so, splinting the patch will not be a good idea | 16:38 |
| *** alonma has joined #openstack-cinder | 16:39 | |
| *** alonmarx has joined #openstack-cinder | 16:40 | |
| xyang1 | erlon: I was just concerned about CI passing. It will be easier to get things merged if you splitting them | 16:41 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add functional-py34 to tox https://review.openstack.org/350685 | 16:41 |
| *** alonma has quit IRC | 16:43 | |
| *** sdague has joined #openstack-cinder | 16:43 | |
| *** pece has quit IRC | 16:44 | |
| *** e0ne has quit IRC | 16:44 | |
| erlon | xyang1: hmm, I see, so each driver change would be part of a depended patch | 16:45 |
| *** Suyi_ has joined #openstack-cinder | 16:45 | |
| *** alonma has joined #openstack-cinder | 16:46 | |
| xyang1 | erlon: you submit the first patch for common code changes and LVM, after that is merged, you won't have dependencies | 16:46 |
| *** chris_morrell has quit IRC | 16:47 | |
| erlon | xyang1: that would be a problem as there would be cross dependencies, for example, there's a change in driver.py, and in the create_flow.py that will call the new defined function in BaseVD. If this merge first, all other drivers will be automatically broken | 16:49 |
| xyang1 | erlon: ok | 16:50 |
| erlon | xyang1: unless each driver redefines the function from the base class | 16:50 |
| *** alonma has quit IRC | 16:50 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix FC multipath cleanup https://review.openstack.org/231135 | 16:50 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix weak test_vzstorage_with_mds_list https://review.openstack.org/350517 | 16:50 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix the mocking mess https://review.openstack.org/350516 | 16:50 |
| patrickeast | anyone know what the tempest "backendX_name" tempest conf variables are for? looking at the tests it only looks for "backend_names" as a csv list of backend names | 16:50 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix FC multipath rescan https://review.openstack.org/349598 | 16:51 |
| patrickeast | haha "Fix the mocking mess" | 16:51 |
| erlon | patrickeast: multi-backend vs single backend? | 16:52 |
| erlon | I think scottda added the backend_names | 16:52 |
| patrickeast | erlon: nah it does numbered backend names https://github.com/openstack-dev/devstack/blob/master/lib/tempest#L460 | 16:52 |
| patrickeast | erlon: but the tests look for "backend_names" | 16:52 |
| *** krotscheck is now known as kro_focused | 16:53 | |
| *** Gao has joined #openstack-cinder | 16:53 | |
| patrickeast | well, i'll put up a patch to change it... hopefully it doesn't break too much | 16:53 |
| *** zhongjun_ has quit IRC | 16:53 | |
| *** Lallana has quit IRC | 16:53 | |
| *** alonma has joined #openstack-cinder | 16:53 | |
| *** zhongjun_ has joined #openstack-cinder | 16:55 | |
| erlon | patrickeast: https://review.openstack.org/#/c/325895/10/lib/tempest | 16:55 |
| *** Guest6612 has quit IRC | 16:55 | |
| *** Guest6612 has joined #openstack-cinder | 16:55 | |
| patrickeast | ooo sweet | 16:56 |
| patrickeast | less work | 16:56 |
| *** Gao has quit IRC | 16:56 | |
| *** Gao has joined #openstack-cinder | 16:56 | |
| *** smile-luobin has quit IRC | 16:56 | |
| patrickeast | dang, thats the wrong list though unless there is a tempest patch to go with it | 16:57 |
| patrickeast | scottda: got a min to chat about that patch? | 16:58 |
| *** alonma has quit IRC | 16:58 | |
| *** pcaruana has quit IRC | 17:00 | |
| *** tesseract- has quit IRC | 17:03 | |
| *** jwcroppe has joined #openstack-cinder | 17:04 | |
| *** openstackgerrit_ has joined #openstack-cinder | 17:07 | |
| *** openstackgerrit_ has quit IRC | 17:08 | |
| nikeshm | vbala: there | 17:08 |
| nikeshm | ? | 17:08 |
| *** akshai_ has quit IRC | 17:11 | |
| *** lpetrut has quit IRC | 17:11 | |
| *** karthikp_ has quit IRC | 17:11 | |
| *** karthikp__ is now known as karthikp_ | 17:11 | |
| _alastor_ | Does get_manageable_volumes have a commandline option? | 17:20 |
| *** eharney has quit IRC | 17:20 | |
| _alastor_ | Specifically a way to display that information via a CLI command | 17:20 |
| *** dave-mccowan has joined #openstack-cinder | 17:20 | |
| hemna | _alastor_, I thought that was the entire point of that api | 17:21 |
| patrickeast | oh, did that thing merge? | 17:21 |
| hemna | to dump out the available volumes on the backend | 17:21 |
| hemna | ugly API | 17:21 |
| hemna | sadness | 17:21 |
| _alastor_ | hemna: Yeah, I thought that too, but there doesn't seem to be a CLI option for it that I've found | 17:21 |
| _alastor_ | cinder --help | grep manage isn't helping | 17:22 |
| patrickeast | _alastor_: its possible there isn't a change to cinderclient for it, or its still in review | 17:22 |
| hemna | https://review.openstack.org/#/q/project:openstack/python-cinderclient | 17:22 |
| hemna | I just started looking in there for it | 17:22 |
| *** dave-mcc_ has joined #openstack-cinder | 17:22 | |
| hemna | I think avishay did the cinder change no ? | 17:23 |
| _alastor_ | Here's the cinder change: https://review.openstack.org/#/c/285296/ | 17:23 |
| _alastor_ | looks like a merge conflict | 17:23 |
| _alastor_ | https://review.openstack.org/#/c/285381 | 17:23 |
| hemna | oopsie | 17:24 |
| _alastor_ | No activity since february... | 17:24 |
| _alastor_ | Wait, may | 17:24 |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 17:25 |
| *** zhangjn has quit IRC | 17:26 | |
| *** zhangjn has joined #openstack-cinder | 17:27 | |
| *** dave-mccowan has quit IRC | 17:27 | |
| patrickeast | _alastor_: you wanna resurrect the client change? | 17:27 |
| *** e0ne has joined #openstack-cinder | 17:27 | |
| _alastor_ | patrickeast: Never done one, but it seems like something we would want in newton | 17:27 |
| hemna | patrickeast, +1 | 17:27 |
| hemna | yah | 17:27 |
| hemna | the driver API is useless w/o the cinderclient change in | 17:28 |
| patrickeast | if no one else does sometime in the next few weeks i'll implement that driver api on the pure drivers and can go poke that thing too, although i guess if we are getting close to client freeze i needs to be done soonish right? | 17:28 |
| *** e0ne has quit IRC | 17:28 | |
| patrickeast | hemna: pff i openstack via curl exclusively | 17:28 |
| hemna | patrickeast, just pull the patch and rebase it | 17:28 |
| *** alonma has joined #openstack-cinder | 17:28 | |
| _alastor_ | patrickeast: this is something my SEs want, so I have motivation to get it in | 17:28 |
| patrickeast | _alastor_: sounds good to me, it does look like it might just take a rebase and some tweaking based on the review feedback... shouldn't be too crazy | 17:30 |
| _alastor_ | patrickeast: I can just `git review -d` it, then perform the rebase and any subsequent `git review` will post to this review? | 17:30 |
| patrickeast | _alastor_: yep | 17:31 |
| hemna | _alastor_, yes | 17:31 |
| _alastor_ | patrickeast: cool | 17:31 |
| nikeshm | hi i address vipin comment on https://review.openstack.org/#/c/349019 | 17:31 |
| nikeshm | can someone please check is it correct | 17:32 |
| *** alonma has quit IRC | 17:33 | |
| *** alonma has joined #openstack-cinder | 17:35 | |
| *** lpetrut has joined #openstack-cinder | 17:37 | |
| *** alonmarx_ has joined #openstack-cinder | 17:37 | |
| *** alonma has quit IRC | 17:40 | |
| *** alonmarx has quit IRC | 17:40 | |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows Fibre Channel connector https://review.openstack.org/323780 | 17:40 |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows SMBFS connector https://review.openstack.org/323781 | 17:40 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: LeftHand add entry point tracing https://review.openstack.org/339128 | 17:42 |
| *** lkuchlan has joined #openstack-cinder | 17:42 | |
| *** alonma has joined #openstack-cinder | 17:43 | |
| *** e0ne has joined #openstack-cinder | 17:43 | |
| *** faiz89 has joined #openstack-cinder | 17:46 | |
| *** alonma has quit IRC | 17:47 | |
| *** zhipeng has quit IRC | 17:47 | |
| *** alonma has joined #openstack-cinder | 17:50 | |
| *** RageLtMan has quit IRC | 17:51 | |
| smcginnis | e0ne: Oops. :) | 17:51 |
| *** lpetrut has quit IRC | 17:52 | |
| e0ne | smcginnis: np :) | 17:52 |
| *** julim has quit IRC | 17:53 | |
| *** alonma has quit IRC | 17:54 | |
| *** diogogmt has quit IRC | 17:54 | |
| *** julim has joined #openstack-cinder | 17:56 | |
| scottda | patrickeast: My latest patch to "Add a multibackend list to tempest.conf" needs some work... | 17:56 |
| *** diogogmt has joined #openstack-cinder | 17:57 | |
| scottda | patrickeast: Sorry about the confusion | 17:57 |
| *** alonma has joined #openstack-cinder | 17:57 | |
| *** harlowja has quit IRC | 17:58 | |
| scottda | patrickeast: The change to add backend_names to tempest was done by someone else: af136a96 | 17:58 |
| *** haplo37__ has joined #openstack-cinder | 17:59 | |
| *** alonmarx_ has quit IRC | 17:59 | |
| *** dulek_ has quit IRC | 17:59 | |
| *** e0ne has quit IRC | 18:00 | |
| *** harlowja has joined #openstack-cinder | 18:00 | |
| nikeshm | hemna: any solotion of http://54.209.116.144/82/350182/5/check/kaminario-dsvm-tempest-full-iscsi/0dcadef/logs/screen-c-vol.txt.gz#_2016-08-03_15_44_24_009 | 18:01 |
| nikeshm | solution | 18:01 |
| *** alonma has quit IRC | 18:02 | |
| nikeshm | i m hitting this sometimes in CI | 18:02 |
| * hemna looks | 18:02 | |
| hemna | heh | 18:03 |
| hemna | *sigh* | 18:03 |
| hemna | that's privsep for you | 18:03 |
| hemna | it logs everything as WARNING | 18:03 |
| hemna | regardless | 18:03 |
| hemna | hate it | 18:03 |
| *** alonmarx has joined #openstack-cinder | 18:03 | |
| hemna | nikeshm, https://review.openstack.org/#/c/339275/ | 18:03 |
| *** lpetrut has joined #openstack-cinder | 18:03 | |
| hemna | nikeshm, and.....https://review.openstack.org/#/c/350415/ | 18:04 |
| *** fernnest has joined #openstack-cinder | 18:04 | |
| *** alonma has joined #openstack-cinder | 18:04 | |
| hemna | nikeshm, so that systool -c fc_host -v "error" is normal | 18:04 |
| hemna | on hosts that don't have Fibre Channel | 18:05 |
| hemna | prior to privsep, os-brick used oslo_concurrency processutils to execute commands | 18:05 |
| hemna | putils doesn't automatically log all stderr | 18:05 |
| hemna | it captures it and lets the caller deal with it. | 18:05 |
| hemna | os-brick puts a try catch around the call to systool and if there is a failure (no FC HBA's installed) it catches it and correctly ignores it | 18:06 |
| hemna | now privsep just dumps all stderr/stdout to WARNING logs. | 18:06 |
| hemna | which is bogus | 18:06 |
| *** dave-mcc_ has quit IRC | 18:08 | |
| *** alonma has quit IRC | 18:09 | |
| *** wxy__ has quit IRC | 18:10 | |
| *** alonma has joined #openstack-cinder | 18:12 | |
| *** bardia has joined #openstack-cinder | 18:12 | |
| *** wxy__ has joined #openstack-cinder | 18:13 | |
| *** alonma has quit IRC | 18:17 | |
| *** ntpttr has quit IRC | 18:17 | |
| *** alonma has joined #openstack-cinder | 18:19 | |
| *** nikeshm has quit IRC | 18:21 | |
| *** alonma has quit IRC | 18:23 | |
| *** timcl1 has joined #openstack-cinder | 18:24 | |
| *** nikeshm has joined #openstack-cinder | 18:25 | |
| *** sheel has quit IRC | 18:26 | |
| *** alonma has joined #openstack-cinder | 18:26 | |
| *** timcl has quit IRC | 18:27 | |
| nikeshm | jgriffith geguileo hemna: sorry problem was different for failure in my case http://54.209.116.144/82/350182/5/check/kaminario-dsvm-tempest-full-iscsi/0dcadef/logs/screen-c-vol.txt.gz#_2016-08-03_15_45_31_332 | 18:28 |
| nikeshm | somehow i missed | 18:28 |
| hemna | your destination volume is too small | 18:31 |
| *** alonma has quit IRC | 18:31 | |
| *** wilson_liu has joined #openstack-cinder | 18:31 | |
| *** Guest6612 has quit IRC | 18:32 | |
| *** DiaoSi has joined #openstack-cinder | 18:34 | |
| *** lprice has joined #openstack-cinder | 18:35 | |
| *** alonma has joined #openstack-cinder | 18:35 | |
| *** Gao has quit IRC | 18:38 | |
| *** angela-s has joined #openstack-cinder | 18:38 | |
| *** alonma has quit IRC | 18:39 | |
| *** alonmarx has quit IRC | 18:41 | |
| *** dustins has quit IRC | 18:41 | |
| *** alonma has joined #openstack-cinder | 18:42 | |
| *** alonmarx has joined #openstack-cinder | 18:43 | |
| johnplanow | I'm looking to add unit test coverage for manage_existing functionality per https://bugs.launchpad.net/cinder/+bug/1599140 but.. | 18:43 |
| openstack | Launchpad bug 1599140 in Cinder "manage_existing flow has poor unit test coverage" [Undecided,New] | 18:43 |
| johnplanow | I'm diving into this project for the first time and having a bit of trouble understanding how all the layers fit together. | 18:43 |
| johnplanow | Would someone mind spending 15 mins getting me pointed in the right direction? | 18:43 |
| *** dustins has joined #openstack-cinder | 18:45 | |
| *** ametts has joined #openstack-cinder | 18:45 | |
| *** julim has quit IRC | 18:46 | |
| *** alonma has quit IRC | 18:47 | |
| *** darosale has joined #openstack-cinder | 18:48 | |
| *** jdurgin1 has quit IRC | 18:48 | |
| *** alonma has joined #openstack-cinder | 18:49 | |
| scottda | johnplanow: Hi | 18:51 |
| hemna | johnplanow ! | 18:52 |
| scottda | johnplanow: michal has stated in the bug that a good example for manage_existing tests are in cinder.tests.unit.test_volume.VolumeTestCase.test_create_delete_volume | 18:52 |
| hemna | :) | 18:52 |
| hemna | back in the tech game | 18:52 |
| scottda | johnplanow: Are you wondering how the layers of cinder work together? | 18:52 |
| *** alonma has quit IRC | 18:54 | |
| nikeshm | xyang smcginnis : addressed all reviews on retype https://review.openstack.org/#/c/349019 and CI also happy, waiting fo jenkins | 18:54 |
| nikeshm | xyang1: | 18:54 |
| scottda | johnplanow: uh-oh, I see from that test there are things we no longer want in unit tests, like calling the Database... | 18:54 |
| scottda | So not a good example maybe. | 18:54 |
| *** haplo37__ has quit IRC | 18:55 | |
| *** lpetrut has quit IRC | 18:55 | |
| *** lpetrut has joined #openstack-cinder | 18:55 | |
| fernnest | smcginnis, please confirm, HPE_XP CI is back online and reporting. | 18:56 |
| *** alonmarx has quit IRC | 18:56 | |
| *** eharney has joined #openstack-cinder | 18:58 | |
| xyang1 | nikeshm: hi | 18:59 |
| xyang1 | nikeshm: ok, I'll take a look | 18:59 |
| nikeshm | xyang1: thanks | 18:59 |
| *** alonmarx has joined #openstack-cinder | 19:00 | |
| *** diogogmt has quit IRC | 19:00 | |
| *** karthik__ has joined #openstack-cinder | 19:01 | |
| diablo_rojo | xyang1: Thank you for merging the brick refactor :) That was merge conflict hell for a while, so I am happy it's over. | 19:02 |
| xyang1 | diablo_rojo: welcome:) | 19:03 |
| *** alonma has joined #openstack-cinder | 19:04 | |
| *** dave-mccowan has joined #openstack-cinder | 19:04 | |
| johnplanow | scottda: yes, wondering how the layers of cinder work together. | 19:05 |
| johnplanow | hemna: hey! long time! | 19:06 |
| *** porrua has quit IRC | 19:06 | |
| *** fifieldt has quit IRC | 19:07 | |
| *** haplo37__ has joined #openstack-cinder | 19:07 | |
| bardia | Can I get some of the cores' attention to this: https://review.openstack.org/#/c/346238 :) | 19:07 |
| johnplanow | scottda: maybe there's a better example? | 19:07 |
| *** alonma has quit IRC | 19:09 | |
| smcginnis | fernnest: Looks good. | 19:10 |
| *** timcl has joined #openstack-cinder | 19:10 | |
| smcginnis | fernnest: Hmm, that's skipping a lot of tests for core functionality (?!.*test_snapshots_actions) | 19:10 |
| smcginnis | fernnest: Can you clean up that list a little. I could see a few of those, but really are too many being skipped I think. | 19:11 |
| *** alonma has joined #openstack-cinder | 19:11 | |
| jungleboyj | xyang1: Question for you ... | 19:11 |
| *** timcl1 has quit IRC | 19:13 | |
| *** Lee1092 has quit IRC | 19:13 | |
| scottda | johnplanow: You could start with the developer's reference: http://docs.openstack.org/developer/cinder/devref/ | 19:14 |
| scottda | johnplanow: I cannot guarantee that info is all up-to-date, or complete. | 19:14 |
| *** alonma has quit IRC | 19:16 | |
| xyang1 | jungleboyj: sure | 19:17 |
| jungleboyj | xyang1: So, where do things stand with Tiramisu and Newton. Is it going to land or not? | 19:17 |
| jungleboyj | I haven't been able to follow that as closely as I would like. | 19:18 |
| *** jdurgin1 has joined #openstack-cinder | 19:18 | |
| xyang1 | jungleboyj: I am working on the groups patches first | 19:18 |
| jungleboyj | xyang1: Right. | 19:18 |
| xyang1 | jungleboyj: there are 4 patches I need to get in before Tiramisu | 19:18 |
| *** fifieldt has joined #openstack-cinder | 19:19 | |
| jungleboyj | Ok, so all the generic grouping stuff first. | 19:19 |
| *** alonma has joined #openstack-cinder | 19:19 | |
| *** alonmarx has quit IRC | 19:19 | |
| xyang1 | jungleboyj: this is the first one: https://review.openstack.org/#/c/320165/ | 19:20 |
| xyang1 | jungleboyj: I am updating the other patches now that request_spec has a ovo | 19:20 |
| xyang1 | jungleboyj: would appreciate if you could help review them | 19:21 |
| *** alonmarx has joined #openstack-cinder | 19:22 | |
| johnplanow | scottda: ok, i'll dig in a little deeper with that. thanks. | 19:22 |
| jungleboyj | xyang1: Ok. Will try to look at them. So, there is still quite a bit of work to get the generic groups in place. The tiramisu probably won't make it until Ocata then? | 19:23 |
| jungleboyj | xyang1: The driver developers here are just wondering what to plan. | 19:23 |
| *** zul_ has joined #openstack-cinder | 19:23 | |
| *** alonma has quit IRC | 19:24 | |
| xyang1 | jungleboyj: probably, we only have a month left. if all patches get merged soon, we still have hope to get Tiramisu in | 19:24 |
| jungleboyj | xyang1: Ok. Makes sense. | 19:24 |
| nikeshm | testing i am online | 19:28 |
| nikeshm | ok | 19:28 |
| *** lkuchlan has quit IRC | 19:28 | |
| *** alonma has joined #openstack-cinder | 19:29 | |
| *** nikeshm has quit IRC | 19:31 | |
| *** tesseract- has joined #openstack-cinder | 19:31 | |
| *** ametts has quit IRC | 19:32 | |
| *** alonma has quit IRC | 19:33 | |
| *** clenimar has quit IRC | 19:34 | |
| *** diogogmt has joined #openstack-cinder | 19:34 | |
| *** clenimar has joined #openstack-cinder | 19:35 | |
| *** zul_ has quit IRC | 19:35 | |
| *** gouthamr has quit IRC | 19:35 | |
| *** zul_ has joined #openstack-cinder | 19:35 | |
| *** alonma has joined #openstack-cinder | 19:36 | |
| *** tesseract- has quit IRC | 19:37 | |
| *** timcl has quit IRC | 19:39 | |
| *** alonma has quit IRC | 19:40 | |
| *** gouthamr has joined #openstack-cinder | 19:45 | |
| *** ametts has joined #openstack-cinder | 19:45 | |
| *** dulek_ has joined #openstack-cinder | 19:47 | |
| *** alonmarx has quit IRC | 19:47 | |
| *** chris_morrell has joined #openstack-cinder | 19:50 | |
| *** alonmarx has joined #openstack-cinder | 19:51 | |
| *** permalac has quit IRC | 20:02 | |
| *** permalac has joined #openstack-cinder | 20:03 | |
| openstackgerrit | Merged openstack/cinder: Remove Tegile volume driver https://review.openstack.org/348032 | 20:05 |
| *** jungleboyj has quit IRC | 20:06 | |
| *** porrua has joined #openstack-cinder | 20:09 | |
| *** lpetrut has quit IRC | 20:10 | |
| *** alonma has joined #openstack-cinder | 20:10 | |
| *** coolsvap has quit IRC | 20:11 | |
| _alastor_ | patrickeast: There seems to be some conflict over whether the name on the backend device is called an "identifier" or a "reference" | 20:14 |
| *** alonma has quit IRC | 20:15 | |
| _alastor_ | patrickeast: The implemented Cinder API says it's a "reference", but the help for all manage calls referrs to it as an "identifier", which due to the nature of python-cinderclient, means it can't find any "identifier" attribute from the api | 20:15 |
| *** chris_morrell has quit IRC | 20:16 | |
| _alastor_ | patrickeast: having fun already :) | 20:16 |
| *** merooney has quit IRC | 20:16 | |
| *** alonma has joined #openstack-cinder | 20:17 | |
| *** jungleboyj has joined #openstack-cinder | 20:18 | |
| patrickeast | _alastor_: haha, well, it can't be tooo eazy | 20:19 |
| *** alonmarx has quit IRC | 20:19 | |
| patrickeast | _alastor_: i would have it match the api given the choice | 20:19 |
| _alastor_ | patrickeast: gonna have to update the regular manage commands. Shouldn't be too hard... | 20:22 |
| patrickeast | _alastor_: easy peasy XD | 20:22 |
| *** alonma has quit IRC | 20:22 | |
| *** alonmarx has joined #openstack-cinder | 20:23 | |
| *** alonma has joined #openstack-cinder | 20:26 | |
| *** akapil has joined #openstack-cinder | 20:28 | |
| *** dustins has quit IRC | 20:30 | |
| *** porrua has quit IRC | 20:30 | |
| *** alonma has quit IRC | 20:30 | |
| fernnest | smcginnis, will do. | 20:31 |
| *** karthik__ has quit IRC | 20:33 | |
| *** alonma has joined #openstack-cinder | 20:33 | |
| *** akapil has quit IRC | 20:36 | |
| _alastor_ | patrickeast: Is the v3 blockstorage api reference not available? It's not here: http://developer.openstack.org/api-ref.html | 20:36 |
| patrickeast | _alastor_: good question... | 20:37 |
| patrickeast | scottda: ^^ you know if we have v3 docs stashed away somewhere? | 20:37 |
| *** alonma has quit IRC | 20:38 | |
| scottda | I'm not sure we ever started one...v3 originally being the same as v2 | 20:38 |
| patrickeast | yea thats what i was thinking | 20:38 |
| patrickeast | oh yea | 20:38 |
| _alastor_ | patrickeast: snapshot-manage is not even listed in the v2 api ref | 20:38 |
| patrickeast | _alastor_: well they wouldn't be very good secret api's if we documented them :p | 20:39 |
| _alastor_ | :) | 20:39 |
| patrickeast | i'm not even sure where those come from... i guess cinder/api-ref/ ?? | 20:39 |
| patrickeast | guess we need a v3 directory | 20:40 |
| *** timcl has joined #openstack-cinder | 20:40 | |
| *** chris_morrell has joined #openstack-cinder | 20:40 | |
| _alastor_ | This is starting to feel a bit like yak shaving... | 20:40 |
| patrickeast | lol | 20:40 |
| *** alonma has joined #openstack-cinder | 20:41 | |
| *** alonma has joined #openstack-cinder | 20:41 | |
| _alastor_ | side note: Maybe a shaved yak could be our new mascot? | 20:41 |
| _alastor_ | That or a stack of Jenga blocks | 20:42 |
| scottda | patrickeast: I think they are in a docs repo... | 20:42 |
| patrickeast | scottda: whats this stuff then? https://github.com/openstack/cinder/tree/master/api-ref | 20:42 |
| * patrickeast wouldn't be too surprised if its duplicated | 20:43 | |
| scottda | patrickeast: Yeah, that :) | 20:43 |
| *** akapil has joined #openstack-cinder | 20:43 | |
| *** baumann has quit IRC | 20:44 | |
| *** dustins has joined #openstack-cinder | 20:44 | |
| *** alonmarx has quit IRC | 20:45 | |
| *** alonma has quit IRC | 20:45 | |
| *** alonmarx has joined #openstack-cinder | 20:46 | |
| *** RageLtMan has joined #openstack-cinder | 20:47 | |
| *** alonma has joined #openstack-cinder | 20:48 | |
| *** ametts has quit IRC | 20:52 | |
| *** akerr has quit IRC | 20:52 | |
| *** alonma has quit IRC | 20:52 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Live Volume Autofailover https://review.openstack.org/350782 | 20:54 |
| *** alonma has joined #openstack-cinder | 20:55 | |
| *** ntpttr- is now known as ntpttr | 20:56 | |
| *** timcl has quit IRC | 20:57 | |
| *** v12aml has quit IRC | 20:57 | |
| *** jwcroppe has quit IRC | 20:58 | |
| *** alonma has quit IRC | 20:59 | |
| *** alonma has joined #openstack-cinder | 21:02 | |
| *** v12aml has joined #openstack-cinder | 21:04 | |
| *** xinli has quit IRC | 21:06 | |
| *** alonma has quit IRC | 21:07 | |
| *** dustins has quit IRC | 21:07 | |
| _alastor_ | patrickeast: figured it out. I can just look at the api view for what I need ;) | 21:08 |
| *** liverpooler has quit IRC | 21:08 | |
| *** rlrossit has quit IRC | 21:08 | |
| *** alonma has joined #openstack-cinder | 21:09 | |
| *** JoseMello has quit IRC | 21:10 | |
| *** chris_morrell has quit IRC | 21:11 | |
| *** baker has joined #openstack-cinder | 21:11 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_WIKI_NAME to all drivers https://review.openstack.org/348002 | 21:12 |
| *** pauloewerton has quit IRC | 21:13 | |
| *** lprice has quit IRC | 21:13 | |
| openstackgerrit | Matt Smith (_alastor_) proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/285381 | 21:13 |
| baker | I've been searching for a while, but have come up empty. Is it possible to override the default quota for a volume_type from -1 to 0? | 21:13 |
| *** alonma has quit IRC | 21:14 | |
| *** akapil has quit IRC | 21:14 | |
| baker | still running juno - so is_public isn't a possibility yet, unfortunately, but I only want specific tenants to be able to access a new volume type | 21:15 |
| *** fifieldt has quit IRC | 21:15 | |
| *** akapil has joined #openstack-cinder | 21:15 | |
| *** alonmarx has quit IRC | 21:16 | |
| *** alonma has joined #openstack-cinder | 21:16 | |
| *** chris_morrell has joined #openstack-cinder | 21:20 | |
| *** alonmarx has joined #openstack-cinder | 21:20 | |
| *** alonma has quit IRC | 21:21 | |
| *** faiz89 has quit IRC | 21:23 | |
| guitarzan | baker: you can set your quotas to whatever you like | 21:24 |
| *** gouthamr has quit IRC | 21:24 | |
| guitarzan | were default quotas in the config file in j? or in the db? (I don't remember) | 21:25 |
| *** diablo_rojo has quit IRC | 21:25 | |
| *** fifieldt has joined #openstack-cinder | 21:25 | |
| *** alonma has joined #openstack-cinder | 21:27 | |
| *** enriquetaso has quit IRC | 21:27 | |
| baker | I think in the DB | 21:27 |
| baker | the option is still there in the config file | 21:27 |
| baker | I see that for the standard global ones, like volume, snapshots, gigabytes | 21:28 |
| *** bardia has quit IRC | 21:28 | |
| *** jwcroppe has joined #openstack-cinder | 21:28 | |
| baker | but - if I wanted to set a default quota for a new volume_type for all tenants, I'm struggling with how to do that | 21:28 |
| baker | without having to set it on each tenant individually that is | 21:30 |
| baker | right now the quota on new volume_types for each volumes,gigabytes,snapshots is -1 | 21:31 |
| *** alonma has quit IRC | 21:31 | |
| *** jungleboyj has quit IRC | 21:33 | |
| *** angela-s has quit IRC | 21:36 | |
| *** akapil has quit IRC | 21:36 | |
| *** alonmarx has quit IRC | 21:37 | |
| *** dulek_ has quit IRC | 21:39 | |
| guitarzan | baker: you should be able to set a quota for volume_<volume_type_name> | 21:39 |
| baker | I can with the CLI - for a given tenant | 21:40 |
| baker | I think the tenant ID is required | 21:40 |
| guitarzan | set in the config file right next to you rother defaults | 21:40 |
| baker | hmm... I hadn't tried that - but it didn't look like the code would pick that up | 21:40 |
| guitarzan | if it's in the db, you can adjust the quota for the quota class "default" | 21:40 |
| baker | I was looking at the quota.py | 21:41 |
| guitarzan | I don't know if that part is juno though | 21:41 |
| baker | I was just poking around in the database, looking at the quota_classes table | 21:41 |
| baker | it has volumes,snapshots, gb's, and consisteny groups, but not any of the volume types | 21:42 |
| *** sdake has quit IRC | 21:43 | |
| baker | I just found the code in horizon that is doing it default quota updates with the cinderclient call | 21:46 |
| baker | I should be able to back-track from there | 21:46 |
| *** jwcroppe_ has joined #openstack-cinder | 21:56 | |
| *** jwcroppe has quit IRC | 21:58 | |
| *** markvoelker has quit IRC | 21:58 | |
| *** markvoelker has joined #openstack-cinder | 21:59 | |
| *** mriedem has quit IRC | 22:01 | |
| *** alonma has joined #openstack-cinder | 22:01 | |
| *** eharney has quit IRC | 22:02 | |
| *** eharney has joined #openstack-cinder | 22:03 | |
| *** alyson_ has quit IRC | 22:03 | |
| *** markvoelker has quit IRC | 22:03 | |
| hemna | anyone try retype lately that kicks off a migrate | 22:03 |
| hemna | I'm getting keystone failures | 22:04 |
| hemna | DiscoveryFailure: Could not determine a suitable URL for the plugin | 22:04 |
| smcginnis | Ruh roh | 22:05 |
| smcginnis | Haven't tried lately. | 22:05 |
| smcginnis | Just a standard retype --migration-policy on-demand? | 22:05 |
| *** darosale has quit IRC | 22:05 | |
| *** alonma has quit IRC | 22:06 | |
| hemna | yah | 22:07 |
| *** cdelatte has quit IRC | 22:08 | |
| *** ebalduf has quit IRC | 22:08 | |
| *** alonma has joined #openstack-cinder | 22:09 | |
| scottda | dang it, I've a tempest test for that stuck in review.... | 22:09 |
| *** alonma has quit IRC | 22:13 | |
| jgriffith | hemna: burn it! | 22:13 |
| *** jwcroppe has joined #openstack-cinder | 22:14 | |
| hemna | anyone else see it ? | 22:14 |
| hemna | maybe it's just me | 22:14 |
| scottda | sorry, I need a new devstack. I'll test in a while... | 22:15 |
| *** chris_morrell has quit IRC | 22:15 | |
| *** alonma has joined #openstack-cinder | 22:16 | |
| *** jwcroppe_ has quit IRC | 22:17 | |
| *** alonma has quit IRC | 22:20 | |
| *** baker has quit IRC | 22:28 | |
| *** alonma has joined #openstack-cinder | 22:30 | |
| *** karthik__ has joined #openstack-cinder | 22:34 | |
| *** alonma has quit IRC | 22:35 | |
| *** julim has joined #openstack-cinder | 22:37 | |
| *** alonma has joined #openstack-cinder | 22:38 | |
| *** ducttape_ has quit IRC | 22:41 | |
| *** alonma has quit IRC | 22:42 | |
| *** alonma has joined #openstack-cinder | 22:45 | |
| *** zul_ has quit IRC | 22:46 | |
| *** alonma has quit IRC | 22:49 | |
| *** hemna is now known as hemnafk | 22:52 | |
| *** alonma has joined #openstack-cinder | 22:52 | |
| *** johnplanow has quit IRC | 22:55 | |
| *** chris_morrell has joined #openstack-cinder | 22:56 | |
| *** xyang1 has quit IRC | 22:56 | |
| *** alonma has quit IRC | 22:57 | |
| *** alonma has joined #openstack-cinder | 22:59 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add friendly user message for incompatible microversion https://review.openstack.org/319501 | 23:00 |
| *** ducttape_ has joined #openstack-cinder | 23:02 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add API versioning to upload_to_image options https://review.openstack.org/319501 | 23:03 |
| *** sdake has joined #openstack-cinder | 23:03 | |
| *** alonma has quit IRC | 23:03 | |
| *** alonma has joined #openstack-cinder | 23:06 | |
| *** seldenr has quit IRC | 23:07 | |
| *** tpsilva has quit IRC | 23:08 | |
| *** diogogmt has quit IRC | 23:09 | |
| *** alonma has quit IRC | 23:11 | |
| *** alonma has joined #openstack-cinder | 23:14 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 23:15 |
| *** alonma has quit IRC | 23:18 | |
| *** diogogmt has joined #openstack-cinder | 23:19 | |
| *** alonma has joined #openstack-cinder | 23:21 | |
| *** adrianofr has quit IRC | 23:21 | |
| *** ducttape_ has quit IRC | 23:24 | |
| *** ducttape_ has joined #openstack-cinder | 23:25 | |
| *** leeantho has quit IRC | 23:25 | |
| *** alonma has quit IRC | 23:25 | |
| *** diogogmt has quit IRC | 23:27 | |
| *** sdague has quit IRC | 23:29 | |
| *** fernnest_ has quit IRC | 23:34 | |
| *** fernnest has quit IRC | 23:35 | |
| *** hoonetorg has quit IRC | 23:39 | |
| *** Guy has joined #openstack-cinder | 23:41 | |
| *** hoonetorg has joined #openstack-cinder | 23:44 | |
| *** lprice has joined #openstack-cinder | 23:46 | |
| Guy | DuncanT: ping | 23:47 |
| *** zhongjun_ has quit IRC | 23:47 | |
| *** jamielennox is now known as jamielennox|away | 23:48 | |
| *** zhongjun_ has joined #openstack-cinder | 23:48 | |
| *** ducttape_ has quit IRC | 23:49 | |
| *** wilson1 has joined #openstack-cinder | 23:50 | |
| *** sdake has quit IRC | 23:51 | |
| *** lprice has quit IRC | 23:52 | |
| *** wilson_liu has quit IRC | 23:52 | |
| Guy | DuncanT: Hi Duncan, in https://review.openstack.org/#/c/347103/ (this one is abandonned, the new one is https://review.openstack.org/#/c/349318/2), you said that you haven't seen anything major in the code and the tests are clear. But there is no reviews. I would like to know if you can give me some reviews about the code. Thank you in advance. | 23:54 |
| *** daneyon has quit IRC | 23:54 | |
| *** alonma has joined #openstack-cinder | 23:55 | |
| *** lprice has joined #openstack-cinder | 23:55 | |
| *** sdake has joined #openstack-cinder | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!