| *** edmondsw has quit IRC | 00:03 | |
| *** osc_khoj has joined #openstack-cinder | 00:11 | |
| *** wxy- has quit IRC | 00:13 | |
| *** lucasxu has joined #openstack-cinder | 00:24 | |
| *** zhangguoqing has joined #openstack-cinder | 00:28 | |
| *** zhangguoqing has quit IRC | 00:30 | |
| *** zhangguoqing has joined #openstack-cinder | 00:30 | |
| *** osc_khoj has quit IRC | 00:30 | |
| *** crushil has quit IRC | 00:31 | |
| *** crushil has joined #openstack-cinder | 00:32 | |
| *** zhurong has joined #openstack-cinder | 00:32 | |
| *** armax has quit IRC | 00:35 | |
| *** edmondsw has joined #openstack-cinder | 00:37 | |
| *** lucasxu has quit IRC | 00:39 | |
| *** osc_khoj has joined #openstack-cinder | 00:39 | |
| *** Shunli has joined #openstack-cinder | 00:40 | |
| *** edmondsw has quit IRC | 00:41 | |
| *** zhangguo_ has joined #openstack-cinder | 00:41 | |
| *** lucasxu has joined #openstack-cinder | 00:43 | |
| *** zhangguoqing has quit IRC | 00:44 | |
| *** wxy- has joined #openstack-cinder | 00:47 | |
| *** wanghao has joined #openstack-cinder | 00:48 | |
| *** sbezverk has joined #openstack-cinder | 00:49 | |
| *** wxy- has quit IRC | 00:52 | |
| *** lucasxu has quit IRC | 00:53 | |
| *** lucasxu has joined #openstack-cinder | 00:54 | |
| *** Nel1x has joined #openstack-cinder | 00:54 | |
| *** Rick_ has joined #openstack-cinder | 00:56 | |
| *** wanghao has quit IRC | 00:58 | |
| Rick_ | Does any core-reviewer help to review my changed https://review.openstack.org/#/c/475279/? | 00:58 |
|---|---|---|
| *** wanghao has joined #openstack-cinder | 00:58 | |
| Rick_ | It is pending some time. Please help me. | 00:59 |
| *** xyang1 has quit IRC | 01:12 | |
| *** osc_khoj has quit IRC | 01:12 | |
| *** Yogi1 has joined #openstack-cinder | 01:13 | |
| openstackgerrit | liuyamin proposed openstack/cinder master: [Api-ref] fix response status https://review.openstack.org/485389 | 01:17 |
| *** diablo_rojo has quit IRC | 01:26 | |
| *** edmondsw has joined #openstack-cinder | 01:31 | |
| *** lucasxu has quit IRC | 01:31 | |
| *** harlowja has quit IRC | 01:32 | |
| *** Apoorva_ has joined #openstack-cinder | 01:32 | |
| *** edmondsw has quit IRC | 01:35 | |
| *** Apoorva has quit IRC | 01:36 | |
| *** Apoorva_ has quit IRC | 01:37 | |
| openstackgerrit | iain MacDonnell proposed openstack/cinder master: ZFSSA iSCSI delete volume with non-existent LUN https://review.openstack.org/484956 | 01:39 |
| *** Yogi1 has quit IRC | 01:44 | |
| *** zhaochao has joined #openstack-cinder | 01:44 | |
| mriedem | double yikes http://logs.openstack.org/65/483565/4/check/gate-tempest-dsvm-py35-ubuntu-xenial/9921636/logs/screen-c-api.txt.gz?level=TRACE#_Jul_19_20_20_37_487520 | 01:49 |
| *** lhx__ has joined #openstack-cinder | 01:49 | |
| *** zhaochao has quit IRC | 01:51 | |
| *** edmondsw has joined #openstack-cinder | 01:51 | |
| *** zhaochao has joined #openstack-cinder | 01:54 | |
| *** edmondsw has quit IRC | 01:56 | |
| *** jmlowe has quit IRC | 01:58 | |
| *** Rick_ has quit IRC | 02:00 | |
| *** lhx__ has quit IRC | 02:01 | |
| *** lhx__ has joined #openstack-cinder | 02:01 | |
| *** armax has joined #openstack-cinder | 02:10 | |
| *** edmondsw has joined #openstack-cinder | 02:12 | |
| *** jwcroppe has joined #openstack-cinder | 02:13 | |
| *** swamireddy has joined #openstack-cinder | 02:14 | |
| *** bkopilov has quit IRC | 02:16 | |
| *** edmondsw has quit IRC | 02:16 | |
| *** mtanino_ has joined #openstack-cinder | 02:18 | |
| *** zsli_ has joined #openstack-cinder | 02:21 | |
| *** Shunli has quit IRC | 02:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/485403 | 02:29 |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] correct formatting errors in datera driver https://review.openstack.org/485408 | 02:34 |
| openstackgerrit | xing-yang proposed openstack/cinder master: Update group status https://review.openstack.org/484246 | 02:34 |
| openstackgerrit | Yingzhe Zeng proposed openstack/cinder master: Change Huawei driver's private data storing place https://review.openstack.org/475774 | 02:38 |
| *** crushil has quit IRC | 02:44 | |
| *** crushil has joined #openstack-cinder | 02:45 | |
| *** harlowja has joined #openstack-cinder | 02:52 | |
| *** edmondsw has joined #openstack-cinder | 02:54 | |
| *** zhangguo_ has quit IRC | 02:57 | |
| *** edmondsw has quit IRC | 02:58 | |
| *** zhangguoqing has joined #openstack-cinder | 02:59 | |
| openstackgerrit | yanghuichan proposed openstack/cinder master: Fix grammatical mistake, Changed character from "a" to "an" https://review.openstack.org/483834 | 03:04 |
| *** markvoelker has quit IRC | 03:09 | |
| *** edmondsw has joined #openstack-cinder | 03:10 | |
| openstackgerrit | Merged openstack/cinder master: VNX: ignore cg creation if group is not cg enabled https://review.openstack.org/484265 | 03:13 |
| *** edmondsw has quit IRC | 03:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/485403 | 03:21 |
| *** edmondsw has joined #openstack-cinder | 03:25 | |
| openstackgerrit | Vivek Soni proposed openstack/cinder master: 3PAR: Get host from os-brick https://review.openstack.org/482103 | 03:29 |
| *** edmondsw has quit IRC | 03:30 | |
| *** Nel1x has quit IRC | 03:33 | |
| *** gkadam has joined #openstack-cinder | 03:36 | |
| *** osc_khoj has joined #openstack-cinder | 03:40 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Do not delete group if group snapshot exists https://review.openstack.org/485432 | 03:51 |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] correct formatting errors in datera driver https://review.openstack.org/485408 | 03:54 |
| *** crushil has quit IRC | 04:02 | |
| *** crushil has joined #openstack-cinder | 04:03 | |
| *** edmondsw has joined #openstack-cinder | 04:05 | |
| *** edmondsw has quit IRC | 04:09 | |
| *** mtanino_ has quit IRC | 04:12 | |
| *** mriedem has quit IRC | 04:14 | |
| *** harlowja has quit IRC | 04:18 | |
| *** chlong_ has joined #openstack-cinder | 04:20 | |
| *** edmondsw has joined #openstack-cinder | 04:20 | |
| *** dave-mccowan has quit IRC | 04:23 | |
| *** edmondsw has quit IRC | 04:25 | |
| *** lbragstad has quit IRC | 04:25 | |
| *** arnewiebalck_ has joined #openstack-cinder | 04:28 | |
| *** harlowja has joined #openstack-cinder | 04:30 | |
| *** adisky__ has joined #openstack-cinder | 04:33 | |
| *** rcernin has joined #openstack-cinder | 04:33 | |
| *** edmondsw has joined #openstack-cinder | 04:36 | |
| *** edmondsw has quit IRC | 04:40 | |
| *** wxy- has joined #openstack-cinder | 04:51 | |
| *** nicolasbock has joined #openstack-cinder | 04:53 | |
| *** wxy- has quit IRC | 04:56 | |
| *** crushil has quit IRC | 04:58 | |
| *** crushil has joined #openstack-cinder | 04:59 | |
| openstackgerrit | Rawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI https://review.openstack.org/482642 | 05:00 |
| *** markvoelker has joined #openstack-cinder | 05:10 | |
| *** isaacb has joined #openstack-cinder | 05:16 | |
| *** diablo_rojo has joined #openstack-cinder | 05:17 | |
| *** jainarpit280392 has joined #openstack-cinder | 05:23 | |
| *** chlong_ has quit IRC | 05:26 | |
| *** harlowja has quit IRC | 05:33 | |
| *** armax has quit IRC | 05:34 | |
| *** rcernin has quit IRC | 05:36 | |
| *** osc_khoj has quit IRC | 05:42 | |
| *** edmondsw has joined #openstack-cinder | 05:43 | |
| *** gouthamr has quit IRC | 05:43 | |
| *** jwcroppe has quit IRC | 05:44 | |
| *** markvoelker has quit IRC | 05:44 | |
| *** Poornima has joined #openstack-cinder | 05:45 | |
| *** edmondsw has quit IRC | 05:47 | |
| *** Poornima has quit IRC | 05:47 | |
| *** lkuchlan has joined #openstack-cinder | 05:51 | |
| *** psachin has joined #openstack-cinder | 05:57 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix docstring errors in Dell/EMC Unity https://review.openstack.org/485449 | 05:57 |
| *** rcernin has joined #openstack-cinder | 05:59 | |
| *** edmondsw has joined #openstack-cinder | 05:59 | |
| *** markvoelker has joined #openstack-cinder | 06:01 | |
| *** edmondsw has quit IRC | 06:03 | |
| *** markvoelker has quit IRC | 06:06 | |
| *** arnewiebalck_ has quit IRC | 06:07 | |
| *** arnewiebalck_ has joined #openstack-cinder | 06:13 | |
| *** isaacb has quit IRC | 06:13 | |
| *** arnewiebalck_ has quit IRC | 06:16 | |
| *** lpetrut_ has joined #openstack-cinder | 06:18 | |
| *** isaacb has joined #openstack-cinder | 06:24 | |
| *** lkuchlan has quit IRC | 06:37 | |
| *** lhx__ has quit IRC | 06:38 | |
| *** lhx__ has joined #openstack-cinder | 06:39 | |
| *** edmondsw has joined #openstack-cinder | 06:46 | |
| *** kaisers has quit IRC | 06:46 | |
| *** edmondsw has quit IRC | 06:51 | |
| *** kaisers has joined #openstack-cinder | 06:52 | |
| *** chenying_ has joined #openstack-cinder | 06:53 | |
| openstackgerrit | Hangdong Zhang proposed openstack/python-brick-cinderclient-ext master: Update URLs in documentation https://review.openstack.org/485468 | 06:56 |
| *** chenying has quit IRC | 06:57 | |
| *** markvoelker has joined #openstack-cinder | 07:02 | |
| *** itlinux has quit IRC | 07:04 | |
| *** diablo_rojo has quit IRC | 07:07 | |
| *** lhx__ has quit IRC | 07:10 | |
| *** lhx__ has joined #openstack-cinder | 07:10 | |
| *** crushil has quit IRC | 07:12 | |
| *** crushil has joined #openstack-cinder | 07:12 | |
| *** crushil has quit IRC | 07:15 | |
| *** abhishekk has joined #openstack-cinder | 07:16 | |
| *** crushil has joined #openstack-cinder | 07:16 | |
| *** crushil has quit IRC | 07:17 | |
| *** zsli__ has joined #openstack-cinder | 07:18 | |
| *** tesseract has joined #openstack-cinder | 07:19 | |
| *** zsli_ has quit IRC | 07:21 | |
| openstackgerrit | Hangdong Zhang proposed openstack/python-cinderclient master: Update URLs in documentation https://review.openstack.org/485475 | 07:21 |
| *** chenying__ has joined #openstack-cinder | 07:26 | |
| *** zhonghua has quit IRC | 07:27 | |
| *** zhonghua has joined #openstack-cinder | 07:27 | |
| *** ChubYann has quit IRC | 07:29 | |
| *** chenying_ has quit IRC | 07:29 | |
| *** edmondsw has joined #openstack-cinder | 07:30 | |
| *** lkuchlan has joined #openstack-cinder | 07:33 | |
| *** edmondsw has quit IRC | 07:34 | |
| *** markvoelker has quit IRC | 07:36 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Implement Generic Volume Group feature https://review.openstack.org/474546 | 07:37 |
| *** lpetrut_ has quit IRC | 07:37 | |
| *** vivsoni has joined #openstack-cinder | 07:39 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - bootable volume errors https://review.openstack.org/472674 | 07:41 |
| *** vivsoni_ has quit IRC | 07:42 | |
| *** vivsoni has quit IRC | 07:43 | |
| *** aarefiev_afk is now known as aarefiev | 07:43 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - performance improvements https://review.openstack.org/483893 | 07:43 |
| *** vivsoni has joined #openstack-cinder | 07:43 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - seamless upgrade from SMI-S to REST https://review.openstack.org/482138 | 07:45 |
| *** edmondsw has joined #openstack-cinder | 07:50 | |
| *** edmondsw has quit IRC | 07:55 | |
| *** edmondsw has joined #openstack-cinder | 08:03 | |
| *** edmondsw has quit IRC | 08:07 | |
| *** nicolasbock_ has joined #openstack-cinder | 08:12 | |
| *** nicolasbock has quit IRC | 08:12 | |
| *** gcb has joined #openstack-cinder | 08:21 | |
| *** lpetrut_ has joined #openstack-cinder | 08:25 | |
| *** isaacb has quit IRC | 08:31 | |
| *** isaacb has joined #openstack-cinder | 08:32 | |
| *** markvoelker has joined #openstack-cinder | 08:33 | |
| *** edmondsw has joined #openstack-cinder | 08:39 | |
| *** lpetrut_ has quit IRC | 08:40 | |
| *** edmondsw has quit IRC | 08:43 | |
| *** edmondsw has joined #openstack-cinder | 08:51 | |
| *** edmondsw has quit IRC | 08:55 | |
| *** nicolasbock_ has quit IRC | 09:01 | |
| *** nicolasbock has joined #openstack-cinder | 09:01 | |
| *** nicolasbock has quit IRC | 09:01 | |
| *** markvoelker has quit IRC | 09:07 | |
| openstackgerrit | yanghuichan proposed openstack/cinder master: Fix grammatical mistake, Changed character from "a" to "an" https://review.openstack.org/483834 | 09:10 |
| *** dalgaaf has quit IRC | 09:10 | |
| *** hoonetorg has joined #openstack-cinder | 09:10 | |
| *** dalgaaf has joined #openstack-cinder | 09:11 | |
| openstackgerrit | Merged openstack/cinder master: Enables MySQL Cluster Support for Cinder https://review.openstack.org/433372 | 09:14 |
| *** zsli__ has quit IRC | 09:30 | |
| *** wanghao has quit IRC | 09:35 | |
| *** takedakn has joined #openstack-cinder | 09:36 | |
| *** zhangguoqing has quit IRC | 09:43 | |
| *** mvk has quit IRC | 09:43 | |
| *** zhangguoqing has joined #openstack-cinder | 09:43 | |
| *** wanghao has joined #openstack-cinder | 09:45 | |
| *** takedakn has quit IRC | 09:46 | |
| *** Poornima_K has joined #openstack-cinder | 09:46 | |
| *** Poornima_K has quit IRC | 09:50 | |
| *** wanghao has quit IRC | 09:51 | |
| *** zhangguoqing has quit IRC | 10:02 | |
| *** zhangguoqing has joined #openstack-cinder | 10:03 | |
| *** markvoelker has joined #openstack-cinder | 10:03 | |
| *** zhangguoqing has quit IRC | 10:07 | |
| *** mvk has joined #openstack-cinder | 10:10 | |
| *** edmondsw has joined #openstack-cinder | 10:24 | |
| *** edmondsw has quit IRC | 10:28 | |
| *** ociuhandu_ has quit IRC | 10:31 | |
| *** sdague has joined #openstack-cinder | 10:34 | |
| *** markvoelker has quit IRC | 10:37 | |
| *** edmondsw has joined #openstack-cinder | 10:52 | |
| *** hoonetorg has quit IRC | 10:52 | |
| *** ociuhandu_ has joined #openstack-cinder | 10:53 | |
| *** edmondsw has quit IRC | 10:56 | |
| openstackgerrit | liuyamin proposed openstack/os-brick master: Update and optimize documentation links https://review.openstack.org/485548 | 10:57 |
| *** rawanh has joined #openstack-cinder | 11:04 | |
| *** edmondsw has joined #openstack-cinder | 11:04 | |
| *** edmondsw has quit IRC | 11:09 | |
| *** markvoelker has joined #openstack-cinder | 11:11 | |
| *** Poornima_K has joined #openstack-cinder | 11:14 | |
| openstackgerrit | yanghuichan proposed openstack/cinder master: Replace six.itervalues with dict.values() in cinder https://review.openstack.org/485559 | 11:15 |
| *** dave-mccowan has joined #openstack-cinder | 11:21 | |
| *** edmondsw has joined #openstack-cinder | 11:24 | |
| *** edmondsw has quit IRC | 11:28 | |
| *** jmlowe has joined #openstack-cinder | 11:28 | |
| *** hoonetorg has joined #openstack-cinder | 11:30 | |
| *** Poornima_K has quit IRC | 11:31 | |
| *** wxy- has joined #openstack-cinder | 11:42 | |
| *** abishop has joined #openstack-cinder | 11:48 | |
| tommylikehu | hello xyang, xyang1, please take a look at this patch :https://review.openstack.org/#/c/485432/ , we talked about this before:) | 11:49 |
| *** adisky__ has quit IRC | 11:50 | |
| *** takedakn has joined #openstack-cinder | 11:52 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [WIP] [1/3] Implement policies in code, reviews are welcome https://review.openstack.org/482492 | 11:54 |
| *** csaikia has joined #openstack-cinder | 11:57 | |
| *** hoonetorg has quit IRC | 12:03 | |
| *** MVenesio has joined #openstack-cinder | 12:04 | |
| *** jmlowe has quit IRC | 12:16 | |
| *** chlong_ has joined #openstack-cinder | 12:16 | |
| *** lpetrut has joined #openstack-cinder | 12:25 | |
| *** edmondsw has joined #openstack-cinder | 12:25 | |
| *** JoseMello has joined #openstack-cinder | 12:26 | |
| *** zhaochao has quit IRC | 12:27 | |
| *** gkadam has quit IRC | 12:28 | |
| *** takedakn has quit IRC | 12:34 | |
| *** abhishekk has quit IRC | 12:35 | |
| *** catintheroof has joined #openstack-cinder | 12:36 | |
| *** markvoelker has quit IRC | 12:40 | |
| *** nicolasbock has joined #openstack-cinder | 12:50 | |
| *** nicolasbock has quit IRC | 12:51 | |
| *** nicolasbock has joined #openstack-cinder | 12:51 | |
| *** markvoelker has joined #openstack-cinder | 12:52 | |
| *** _ix has joined #openstack-cinder | 12:58 | |
| *** gcb has quit IRC | 13:02 | |
| *** gcb has joined #openstack-cinder | 13:03 | |
| *** mriedem has joined #openstack-cinder | 13:03 | |
| *** Julien-zte has quit IRC | 13:05 | |
| *** Julien-zte has joined #openstack-cinder | 13:06 | |
| *** lkuchlan has quit IRC | 13:08 | |
| *** jmlowe has joined #openstack-cinder | 13:10 | |
| *** ociuhandu_ has quit IRC | 13:11 | |
| *** xyang1 has joined #openstack-cinder | 13:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/485403 | 13:13 |
| *** lhx__ has quit IRC | 13:14 | |
| *** gouthamr has joined #openstack-cinder | 13:17 | |
| *** zhurong has quit IRC | 13:19 | |
| *** lucasxu has joined #openstack-cinder | 13:19 | |
| *** dustins has joined #openstack-cinder | 13:21 | |
| *** eharney has joined #openstack-cinder | 13:22 | |
| *** zzzeek has quit IRC | 13:23 | |
| *** lbragstad has joined #openstack-cinder | 13:24 | |
| rawanh | Hi guys | 13:24 |
| *** zzzeek has joined #openstack-cinder | 13:25 | |
| rawanh | Can I please get your feedback about my cinder NVMEoF commit https://review.openstack.org/#/c/482562/ ? | 13:25 |
| *** wxy- has quit IRC | 13:28 | |
| *** mtanino__ has joined #openstack-cinder | 13:31 | |
| rawanh | I also have os-brick and nova commits that are linked to it: | 13:38 |
| *** lhx_ has joined #openstack-cinder | 13:39 | |
| rawanh | https://review.openstack.org/#/c/482642/ | 13:39 |
| rawanh | https://review.openstack.org/#/c/482640/ | 13:39 |
| rawanh | if anyone would like to see how things are handled there | 13:39 |
| *** wxy- has joined #openstack-cinder | 13:39 | |
| *** chlong_ has quit IRC | 13:40 | |
| *** felipemonteiro has joined #openstack-cinder | 13:43 | |
| *** wxy- has quit IRC | 13:44 | |
| smcginnis | rawanh: What is the plan on CI for that driver? | 13:44 |
| *** felipemonteiro_ has joined #openstack-cinder | 13:44 | |
| rawanh | @smcginnis : we're not adding a new driver here as you see, it's just a new target Adm for LVM for now, so we'll be adding a CI for it soon | 13:45 |
| smcginnis | rawanh: Well, technically it is a new driver as it will be using a new protocol. | 13:46 |
| smcginnis | rawanh: But it will be good to see CI running. | 13:47 |
| smcginnis | rawanh: Do you have a blueprint for that too? I just noticed there is not one linked in the commit. | 13:48 |
| *** felipemonteiro has quit IRC | 13:49 | |
| hemna | rawanh, would need to see a passing CI | 13:49 |
| rawanh | @smcginnis: I see your point, regarding the spec, we're supporting one very soon too | 13:51 |
| smcginnis | rawanh: We don't need a spec written up, just a blueprint filed for tracking. | 13:52 |
| rawanh | @hemna: I see your point, so we are providing ne very soon | 13:52 |
| rawanh | @smcginnis: aha Ok | 13:52 |
| smcginnis | rawanh: Just need to create on here with some basic info: https://blueprints.launchpad.net/cinder | 13:52 |
| *** jwcroppe has joined #openstack-cinder | 13:53 | |
| *** arnewiebalck_ has joined #openstack-cinder | 13:58 | |
| *** arnewiebalck_ has quit IRC | 13:58 | |
| openstackgerrit | Chaynika Saikia proposed openstack/python-cinderclient master: Add cinder create --poll https://review.openstack.org/475919 | 13:59 |
| *** rawanh has left #openstack-cinder | 14:00 | |
| *** arnewiebalck_ has joined #openstack-cinder | 14:00 | |
| *** crushil has joined #openstack-cinder | 14:06 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Implement Generic Volume Group feature https://review.openstack.org/474546 | 14:06 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Do not delete group if group snapshot exists https://review.openstack.org/485432 | 14:12 |
| *** dillaman has joined #openstack-cinder | 14:17 | |
| *** jdillaman has quit IRC | 14:17 | |
| *** dillaman has quit IRC | 14:17 | |
| *** jdillaman has joined #openstack-cinder | 14:17 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Do not delete group if group snapshot exists https://review.openstack.org/485432 | 14:17 |
| *** jmlowe has quit IRC | 14:22 | |
| *** isaacb has quit IRC | 14:30 | |
| *** armax has joined #openstack-cinder | 14:33 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix formatting errors in Dell/EMC PS https://review.openstack.org/485687 | 14:38 |
| openstackgerrit | Merged openstack/cinder master: XIV\A9000: Added replication group support https://review.openstack.org/468738 | 14:39 |
| *** lucasxu has quit IRC | 14:39 | |
| *** arnewiebalck_ has quit IRC | 14:43 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - seamless upgrade from SMI-S to REST https://review.openstack.org/482138 | 14:46 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - bootable volume errors https://review.openstack.org/472674 | 14:49 |
| openstackgerrit | Merged openstack/cinder master: DS8K driver: specify pool and lss in extra-specs https://review.openstack.org/435857 | 14:50 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - performance improvements https://review.openstack.org/483893 | 14:51 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - seamless upgrade from SMI-S to REST https://review.openstack.org/482138 | 14:52 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - performance improvements in retype https://review.openstack.org/483933 | 14:54 |
| *** hoonetorg has joined #openstack-cinder | 14:57 | |
| *** hoonetorg has quit IRC | 14:57 | |
| *** hoonetorg has joined #openstack-cinder | 14:57 | |
| *** chlong_ has joined #openstack-cinder | 14:58 | |
| *** chhavi has joined #openstack-cinder | 14:58 | |
| *** cknight has joined #openstack-cinder | 15:00 | |
| *** Julien-zte has quit IRC | 15:05 | |
| *** Julien-zte has joined #openstack-cinder | 15:06 | |
| *** Julien-zte has quit IRC | 15:07 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder master: NetApp cDOT: Support generic groups for block https://review.openstack.org/452306 | 15:07 |
| *** Julien-zte has joined #openstack-cinder | 15:08 | |
| *** chhavi has quit IRC | 15:08 | |
| *** Julien-zte has quit IRC | 15:08 | |
| *** chhavi has joined #openstack-cinder | 15:08 | |
| *** Julien-zte has joined #openstack-cinder | 15:09 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - None connector object in a terminate_connection in Pike https://review.openstack.org/482130 | 15:10 |
| *** felipemonteiro_ has quit IRC | 15:11 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - align VMAX QOS settings with front end https://review.openstack.org/484885 | 15:12 |
| *** isaacb has joined #openstack-cinder | 15:13 | |
| *** Julien-zte has quit IRC | 15:19 | |
| *** Julien-zte has joined #openstack-cinder | 15:20 | |
| *** Julien-zte has quit IRC | 15:21 | |
| *** Julien-zte has joined #openstack-cinder | 15:21 | |
| *** Julien-zte has quit IRC | 15:21 | |
| *** lucasxu has joined #openstack-cinder | 15:21 | |
| *** Julien-zte has joined #openstack-cinder | 15:22 | |
| *** Julien-zte has quit IRC | 15:22 | |
| *** Julien-zte has joined #openstack-cinder | 15:23 | |
| *** Julien-zte has quit IRC | 15:23 | |
| *** jmlowe has joined #openstack-cinder | 15:24 | |
| *** rcernin has quit IRC | 15:25 | |
| *** hoonetorg has quit IRC | 15:27 | |
| *** armax has quit IRC | 15:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/485403 | 15:27 |
| *** armax has joined #openstack-cinder | 15:29 | |
| *** markstur has joined #openstack-cinder | 15:29 | |
| *** Apoorva has joined #openstack-cinder | 15:35 | |
| Swanson | OMG. Open source is an effort to test the infinite monkeys theory! We're through the looking glass, people! | 15:35 |
| *** Apoorva_ has joined #openstack-cinder | 15:35 | |
| *** hoonetorg has joined #openstack-cinder | 15:36 | |
| *** Apoorva has quit IRC | 15:39 | |
| *** Apoorva_ has quit IRC | 15:39 | |
| *** Apoorva has joined #openstack-cinder | 15:39 | |
| smcginnis | Swanson: Speaking of infinite monkeys, hows things? :P | 15:39 |
| *** psachin has quit IRC | 15:41 | |
| Swanson | smcginnis, I have an idea for low cost killbots. Seems to be getting more traction than my CNC Tattoo kiosk idea. | 15:43 |
| *** felipemonteiro has joined #openstack-cinder | 15:43 | |
| *** Apoorva has quit IRC | 15:44 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:44 | |
| isaacb | jungleboyj: Hi, when do the docs of the drivers suppose to land in cinder repo? | 15:46 |
| jungleboyj | isaacb: Hey there! | 15:47 |
| jungleboyj | isaacb: How do you mean? | 15:47 |
| *** Apoorva has joined #openstack-cinder | 15:47 | |
| isaacb | jungleboyj: The documentation was in openstack-manuals repo. | 15:48 |
| *** felipemonteiro has quit IRC | 15:48 | |
| isaacb | jungleboyj: I saw you started a migration process of it to cinder | 15:48 |
| *** dfisher has joined #openstack-cinder | 15:48 | |
| jungleboyj | isaacb: Right. It has been migrated. Let me get the link. | 15:49 |
| jungleboyj | isaacb: https://docs.openstack.org/cinder/latest/configuration/block-storage/volume-drivers.html | 15:50 |
| jungleboyj | So, doc/source/configuration/block-storage in the Cinder tree. | 15:51 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Pike documentation https://review.openstack.org/485724 | 15:51 |
| isaacb | jungleboyj: thanks. that's what I looked for. :-) | 15:53 |
| jungleboyj | isaacb: You are welcome. | 15:54 |
| jungleboyj | Once we are done migrating I will make things prettier and easier to find. One step at a time though. | 15:54 |
| *** Poornima has joined #openstack-cinder | 15:55 | |
| *** Poornima has quit IRC | 15:58 | |
| openstackgerrit | Rick Chen proposed openstack/cinder master: ProphetStor: Support generic group https://review.openstack.org/475279 | 16:02 |
| *** Apoorva has quit IRC | 16:02 | |
| *** Apoorva has joined #openstack-cinder | 16:03 | |
| isaacb | jungleboyj: so you suggest to wait with documentation reviews? | 16:05 |
| jungleboyj | isaacb: How do you mean? | 16:06 |
| isaacb | jungleboyj: we have some content to add to our driver documentation. should we commit the code now or wait till the migration is done? | 16:07 |
| Swanson | hemna, Congrats! Elon Musk just said that The Boring Company has verbal approval to build a hyperloop. | 16:08 |
| jungleboyj | isaacb: If it is in that part of the Cinder tree, it has been migrated and yes you can propose changes. | 16:08 |
| hemna | w00t! | 16:10 |
| hemna | about time | 16:10 |
| Swanson | Get your shovel. | 16:11 |
| *** isaacb has quit IRC | 16:11 | |
| Swanson | Some concern that he is building an underground base. | 16:11 |
| *** diablo_rojo has joined #openstack-cinder | 16:15 | |
| *** jmlowe has quit IRC | 16:15 | |
| *** chlong_ has quit IRC | 16:19 | |
| *** Poornima has joined #openstack-cinder | 16:20 | |
| *** Poornima has quit IRC | 16:20 | |
| *** itlinux has joined #openstack-cinder | 16:20 | |
| *** mvk has quit IRC | 16:21 | |
| *** itlinux has quit IRC | 16:21 | |
| *** itlinux has joined #openstack-cinder | 16:22 | |
| *** lhx_ has quit IRC | 16:26 | |
| *** tesseract has quit IRC | 16:32 | |
| *** rcernin has joined #openstack-cinder | 16:33 | |
| jungleboyj | hemna: Have a few minutes to answer some questions? | 16:36 |
| *** bbansal has joined #openstack-cinder | 16:40 | |
| *** Apoorva has quit IRC | 16:50 | |
| *** lucasxu has quit IRC | 16:51 | |
| *** lpetrut has quit IRC | 16:53 | |
| bbansal | guys, can anyone tell whom to contact in order to update about requirement consume? | 16:53 |
| jungleboyj | bbansal: What requirement? | 16:54 |
| bbansal | tooz version 1.57.4 | 16:54 |
| bbansal | cinder/cinder.egg-info/requires.txt:tooz!=1.57.4,>=1.47.0 | 16:54 |
| bbansal | according to openstack upper-constraints.txt tooz 1.57.4 shall be accepted | 16:54 |
| bbansal | and right now my CI build fails because three projects have not consumed the requirements | 16:55 |
| jungleboyj | bbansal: We have had to set that requirement because it was causing gate failures and I don't believe that the tooz bug has been resolved yet. | 16:55 |
| bbansal | cinder, nova and manila | 16:55 |
| bbansal | you should be able to poke people in that project to consume a requirements update | 16:55 |
| bbansal | this is what i was told by openstack-requirements channel | 16:55 |
| bbansal | they say cinder, nova and manila need to consume the requirements | 16:56 |
| *** lucasxu has joined #openstack-cinder | 16:58 | |
| imacdonn | I think https://review.openstack.org/#/c/447079/ is supposed to resolve that tooz-related issue ... maybe that version block can be removed now??? | 17:01 |
| bbansal | its a verison conflict on cinder, nova and manika | 17:02 |
| bbansal | manila* | 17:02 |
| bbansal | i guess manila project has posted a changeset in order to update their requirements | 17:02 |
| *** cknight has left #openstack-cinder | 17:04 | |
| *** felipemonteiro_ has quit IRC | 17:12 | |
| *** felipemonteiro has joined #openstack-cinder | 17:12 | |
| jungleboyj | imacdonn: Well, there was some discussion around that patch. It won't make things worse but it isn't guaranteed to fix the problem either. | 17:14 |
| *** tesseract has joined #openstack-cinder | 17:18 | |
| *** Apoorva has joined #openstack-cinder | 17:19 | |
| *** Apoorva has quit IRC | 17:19 | |
| *** Apoorva has joined #openstack-cinder | 17:20 | |
| *** imacdonn has quit IRC | 17:21 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder master: RBD: support driver-assisted volume migration https://review.openstack.org/256091 | 17:21 |
| *** imacdonn has joined #openstack-cinder | 17:22 | |
| *** felipemonteiro_ has joined #openstack-cinder | 17:22 | |
| *** kfarr has joined #openstack-cinder | 17:23 | |
| openstackgerrit | Rick Chen proposed openstack/cinder master: ProphetStor: Support generic group https://review.openstack.org/475279 | 17:23 |
| *** harlowja has joined #openstack-cinder | 17:23 | |
| *** felipemonteiro has quit IRC | 17:26 | |
| *** harlowja has quit IRC | 17:27 | |
| *** chhavi has quit IRC | 17:28 | |
| *** chlong_ has joined #openstack-cinder | 17:33 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Fix reset state v3 unit tests failures https://review.openstack.org/485197 | 17:36 |
| scottda | ewww..PTG is in not-beautiful part of Denver... | 17:37 |
| imacdonn | oh, hey scottda ... I add unit tests to https://review.openstack.org/#/c/484956/ ... just for you :) Could you +2 it again, please? Also looking for another +2 and +A | 17:38 |
| *** mriedem is now known as mriedem_afk | 17:39 | |
| scottda | Done. Nice charm. | 17:40 |
| imacdonn | heheh thanks | 17:40 |
| dfisher | scottda: false. all of Denver is beautiful. | 17:40 |
| scottda | ha. You live in Denver dfisher ? | 17:40 |
| dfisher | i might :) | 17:40 |
| dfisher | (i'm in a northern suburb, yes) | 17:41 |
| scottda | I like Denver. But Renaissance Denver Stapleton Hotel is in not-beautiful area. Notice I didn't come out and say ugly... | 17:41 |
| dfisher | Stapleton used to be Denver's airport. :) | 17:41 |
| dfisher | that's all cleaned out now is all modern housing/shops/whatever buzzwords you want | 17:42 |
| scottda | I know. I lived in the area for years. In Fort Collins now. | 17:42 |
| dfisher | go buffs. :P | 17:42 |
| scottda | Yeah, I was hoping for downtown location.. | 17:42 |
| scottda | ha. | 17:42 |
| scottda | You can have Boulder. | 17:42 |
| * dfisher might also be a CU alumn | 17:42 | |
| dfisher | I'll take the city, sure. the people can gtfo ;) | 17:42 |
| scottda | exactly. | 17:42 |
| openstackgerrit | Merged openstack/python-cinderclient master: Fix highest version supported by client and server https://review.openstack.org/485198 | 17:45 |
| *** csaikia has quit IRC | 17:46 | |
| *** csaikia has joined #openstack-cinder | 17:46 | |
| ildikov | smcginnis: hi :) | 17:54 |
| ildikov | smcginnis: if you're around we would need a new cinderclient release now that this patch got merged: https://review.openstack.org/485198 | 17:55 |
| ildikov | smcginnis: without that fix we don't get microversion 3.27 as supported and cannot enable the new attach flow | 17:56 |
| hemna | jungleboyj, I'm back | 17:56 |
| *** leni1 has joined #openstack-cinder | 17:57 | |
| jungleboyj | hemna: Hi. In a meeting. Hold on. | 18:02 |
| *** dustins has quit IRC | 18:02 | |
| hemna | np | 18:02 |
| imacdonn | hemna: While you're waiting:) could you review https://review.openstack.org/#/c/484956/ , if you have a min, please? | 18:05 |
| hemna | sure | 18:05 |
| imacdonn | tnx | 18:05 |
| *** harlowja has joined #openstack-cinder | 18:06 | |
| imacdonn | hemna: thanks! | 18:11 |
| hemna | np | 18:12 |
| *** eharney has quit IRC | 18:13 | |
| jungleboyj | hemna: You still around? | 18:26 |
| hemna | yup | 18:32 |
| openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/485403 | 18:34 |
| jungleboyj | hemna: Do we have a good pointer to how standalone Cinder works? | 18:35 |
| hemna | how it works? | 18:36 |
| hemna | meaning the architecture of cinder itself? | 18:36 |
| jungleboyj | For instance, how to get it installed. | 18:36 |
| hemna | I'm not sure what you mean really | 18:36 |
| jungleboyj | Also, what pieces are required to make it work. | 18:37 |
| hemna | well, there are several ways of installing cinder itself | 18:37 |
| hemna | you can use your favorite linux distro's packages | 18:37 |
| hemna | or you can use github's copy of cinder | 18:37 |
| hemna | just depends on what you are trying to do | 18:37 |
| hemna | there is also devstack | 18:38 |
| jungleboyj | Ok, so basically it is just cinder installed with a database still. | 18:39 |
| jungleboyj | It is just the client that is different. | 18:39 |
| hemna | the bare minimums | 18:40 |
| hemna | are the requirements.txt | 18:40 |
| hemna | a database | 18:40 |
| hemna | rabbitmq | 18:41 |
| hemna | and seem backend | 18:41 |
| hemna | some | 18:41 |
| diablo_rojo | jungleboyj, jgriffith kind of talks about it here: https://thenewstack.io/deploying-cinder-stand-alone-storage-service/ | 18:41 |
| hemna | diablo_rojo I did a talk on this subject in Boston | 18:41 |
| jungleboyj | diablo_rojo: Yeah, I had looked at that. | 18:42 |
| hemna | there are many different ways you can do it | 18:42 |
| hemna | that post is about cinder inside of containers | 18:42 |
| diablo_rojo | I thought it might help, my mistake. | 18:43 |
| diablo_rojo | Nevermind me :) | 18:43 |
| imacdonn | but it does hint at things like the "noauth" option to obviate keystone | 18:43 |
| hemna | what's missing is what to do with some features like create volume from image and image from volume. | 18:43 |
| hemna | that can be problematic inside a container | 18:43 |
| jungleboyj | diablo_rojo: I was using that as a starting point but I am trying to take the step to getting a standalone environment set up. | 18:45 |
| hemna | lvm inside a container sucks and is pain | 18:45 |
| *** rtjure has quit IRC | 18:47 | |
| *** rtjure has joined #openstack-cinder | 18:47 | |
| hemna | If you are trying to deploy as an exercise to learn cinder, I would deploy from GitHub or / devstack | 18:48 |
| hemna | the problem is, if you want something other than noauth, you have to deploy and configure keystone | 18:48 |
| hemna | which is a nightmare | 18:48 |
| diablo_rojo | jungleboyj, got it :) That probably won't get you that far then. Didn't know what the goal was. | 18:48 |
| *** hoonetorg has quit IRC | 18:49 | |
| jungleboyj | diablo_rojo: Yeah. Containers will happen here eventually but I need to get the standalone piece working first. | 18:50 |
| diablo_rojo | jungleboyj, cool :) Are you adding directions to the docs? | 18:51 |
| jungleboyj | diablo_rojo: I think I probably should. | 18:51 |
| *** arnewiebalck_ has joined #openstack-cinder | 18:52 | |
| diablo_rojo | jungleboyj, if you decide to do it, I can use them and see how it goes and give you feedback | 18:53 |
| jungleboyj | I would like to have a howto or reference put together when I am done. | 18:53 |
| diablo_rojo | If you want | 18:53 |
| *** ChubYann has joined #openstack-cinder | 18:53 | |
| *** boris-42____ has joined #openstack-cinder | 18:54 | |
| imacdonn | jungleboyj: so where is it that you're getting stuck ? | 18:55 |
| jungleboyj | imacdonn: No where yet, just getting started. | 18:56 |
| imacdonn | haven't tried it .. but expect you'd just follow https://docs.openstack.org/cinder/latest/install/ except skip the keystone parts, and set auth_strategy=noauth | 18:56 |
| imacdonn | skip the nova parts too, obviously | 18:57 |
| *** jdwidari has joined #openstack-cinder | 18:59 | |
| *** jdwidari has quit IRC | 18:59 | |
| *** eharney has joined #openstack-cinder | 19:01 | |
| *** mriedem_afk is now known as mriedem | 19:01 | |
| mriedem | smcginnis: you want me to propose a release now that https://review.openstack.org/#/c/485198/ is merged? | 19:10 |
| *** abishop has quit IRC | 19:17 | |
| *** isaacb has joined #openstack-cinder | 19:20 | |
| jungleboyj | mriedem: I would if it is time sensitive as smcginnis is in China right now. | 19:23 |
| jungleboyj | diablo_rojo: Sure, I will let you know. Good idea to put documentation together as I go. :-) | 19:26 |
| mriedem | oh right | 19:26 |
| jungleboyj | mriedem: I am just his secretary. ;-) | 19:26 |
| mriedem | so the thing that kind of sucks about https://review.openstack.org/#/c/485198/ | 19:26 |
| mriedem | is that it changes the API contract on that public method | 19:26 |
| mriedem | so the release should really probably be a major version bump | 19:26 |
| mriedem | but let me work my wordsmith magic in the commit message | 19:30 |
| *** jwcroppe has quit IRC | 19:31 | |
| *** dustins has joined #openstack-cinder | 19:31 | |
| mriedem | f3a1e6e is a major version bump anyway | 19:31 |
| *** jwcroppe_ has joined #openstack-cinder | 19:34 | |
| *** itlinux has quit IRC | 19:38 | |
| mriedem | jungleboyj: https://review.openstack.org/485797 | 19:38 |
| mriedem | ildikov: stvnoyes: ^ | 19:38 |
| stvnoyes | excellent | 19:39 |
| *** bbansal has quit IRC | 19:45 | |
| *** nicolasbock has quit IRC | 19:46 | |
| *** lkuchlan has joined #openstack-cinder | 19:46 | |
| *** _pewp_ has quit IRC | 19:51 | |
| *** openstack has joined #openstack-cinder | 20:00 | |
| jungleboyj | mriedem: Awesome. | 20:00 |
| mriedem | https://docs.openstack.org/releasenotes/python-cinderclient/unreleased.html | 20:01 |
| mriedem | right now you have a dev hash | 20:01 |
| mriedem | but eventually that will change to 3.0.0 | 20:01 |
| mriedem | once the tag is in the repo | 20:01 |
| mriedem | scottda: if you want a reno for the float -> str thing i can do that quick if you guys can be fast about approving it | 20:02 |
| mriedem | or, just convert the str back to a float | 20:02 |
| mriedem | it's not clear to me why a str had to be returned https://review.openstack.org/#/c/485198/ | 20:02 |
| *** Apoorva_ has joined #openstack-cinder | 20:03 | |
| *** _pewp_ has joined #openstack-cinder | 20:04 | |
| mriedem | stvnoyes: ^ was there a specific reason you felt a string had to be returned from that instead of a float like before? | 20:04 |
| mriedem | nova is already casting to a string | 20:04 |
| *** lkuchlan has quit IRC | 20:05 | |
| *** Apoorva has quit IRC | 20:06 | |
| jungleboyj | mriedem: +1 from me. | 20:09 |
| openstackgerrit | Matt Riedemann proposed openstack/python-cinderclient master: Make get_highest_client_server_version return a float again https://review.openstack.org/485805 | 20:12 |
| mriedem | jungleboyj: scottda: stvnoyes: ^ | 20:12 |
| mriedem | see what you think about that | 20:12 |
| *** isaacb has quit IRC | 20:12 | |
| mriedem | geguileo: ^ | 20:13 |
| *** gouthamr has quit IRC | 20:14 | |
| mriedem | alternatively i can just push a simple release note | 20:15 |
| *** armax has quit IRC | 20:15 | |
| *** armax has joined #openstack-cinder | 20:15 | |
| jungleboyj | mriedem: So, with your change how do you avoid the problem that geguileo was trying to fix by changing it to a string? | 20:16 |
| mriedem | the min already compares them | 20:16 |
| mriedem | get_string just converts the APIVersion object to a string, which we then cast to a float | 20:16 |
| mriedem | for the return value | 20:16 |
| mriedem | as it was before | 20:16 |
| mriedem | before it was doing this | 20:17 |
| mriedem | min(float(max_server_version), float(api_versions.MAX_VERSION)) | 20:17 |
| ildikov | mriedem: do we do further checks in Nova with it? | 20:17 |
| *** openstackgerrit has quit IRC | 20:17 | |
| scottda | mriedem: Does your fix work? i.e you tested with version 3.40? | 20:18 |
| mriedem | no http://git.openstack.org/cgit/openstack/nova/tree/nova/volume/cinder.py#n71 | 20:18 |
| mriedem | ildikov: ^ | 20:18 |
| mriedem | scottda: the test is testing it right? | 20:18 |
| jungleboyj | scottda: Just -2'd . He knows the area better so I am staying out of it. :-) | 20:18 |
| scottda | NO, I was hasty. | 20:18 |
| scottda | Didn't really look at the code, I can change that. | 20:19 |
| jungleboyj | Did you say you were tasty? ;-) | 20:19 |
| *** isaacb has joined #openstack-cinder | 20:19 | |
| ildikov | mriedem: ok, cool | 20:19 |
| jungleboyj | I am ok with the change if you are confident mriedem and scottda is ok. | 20:20 |
| mriedem | dude | 20:20 |
| mriedem | i ooze with confidence | 20:20 |
| mriedem | don't you know? | 20:20 |
| dfisher | they have a cream for that... | 20:20 |
| jungleboyj | mriedem: All too well my friend. | 20:20 |
| scottda | I should never doubt you.. | 20:20 |
| jungleboyj | scottda: Easy there. | 20:20 |
| *** bbansal has joined #openstack-cinder | 20:20 | |
| mriedem | please keep me honest here, but i think if i goofed up the change the unit test would be failing | 20:21 |
| mriedem | the only thing changing is the return type back to float | 20:21 |
| mriedem | so it's not an API change | 20:21 |
| mriedem | i'll put the old code back in before the fix and run the test again | 20:22 |
| jungleboyj | So, you get the min of max_version as a string. So it comes out as something like 3.40. | 20:22 |
| *** isaacb_ has joined #openstack-cinder | 20:22 | |
| mriedem | heh, well, the test failed | 20:22 |
| mriedem | er i mean it passed | 20:22 |
| mriedem | so maybe the test is busted | 20:23 |
| jungleboyj | Comparison doesn't fail because it is all happening as a float. | 20:23 |
| jungleboyj | Then it is returned as a float so it will be 3.4 but we are ok at that point. | 20:23 |
| jungleboyj | mriedem: Right? | 20:23 |
| *** isaacb has quit IRC | 20:25 | |
| *** markstur has quit IRC | 20:25 | |
| mriedem | the test is busted in the original change i think | 20:25 |
| mriedem | w/o my patch, i replaced the original code from https://review.openstack.org/#/c/485198/1/cinderclient/client.py and the test passes, except for needing to cast "expected" to a float | 20:26 |
| mriedem | looking at the test https://review.openstack.org/#/c/485198/1/cinderclient/tests/unit/test_client.py | 20:26 |
| mriedem | the mock_response max version is going to be '3.16' | 20:26 |
| mriedem | so when version is 3.12, the max version should be 3.16 | 20:27 |
| mriedem | when version is 3.40, the max version should be 3.40 | 20:27 |
| mriedem | oh wait, no, it'd be 3.16 | 20:27 |
| *** gouthamr has joined #openstack-cinder | 20:27 | |
| mriedem | damn this is confusing | 20:27 |
| scottda | it's the min of the 2 | 20:27 |
| scottda | the highest supported version of both client and server | 20:28 |
| mriedem | ok so max client is 3.12 and max server is 3.16, then min is 3.12, yeah | 20:28 |
| jungleboyj | mriedem: Right. | 20:28 |
| mriedem | max client is 3.40, max server is 3.16, min is 3.16 | 20:28 |
| scottda | Yeah, the test requires some gyrations to understand. Could be more explicit. | 20:29 |
| mriedem | ok, i'm not sure then why the test passes when i revert the code back to the way it was before in https://review.openstack.org/#/c/485198/1/cinderclient/client.py | 20:30 |
| mriedem | except for the type | 20:30 |
| *** _pewp_ has quit IRC | 20:31 | |
| mriedem | oooooooooooooooooooooooooooo | 20:36 |
| mriedem | now i see the issue | 20:36 |
| mriedem | the problem is not really the cast and min compare in the cinderclient code | 20:37 |
| mriedem | it's when the 3.40 float is returned from the cinderclient method, and nova casts to a str, that the 3.40 becomes 3.4 | 20:37 |
| mriedem | http://paste.openstack.org/show/616084/ | 20:37 |
| *** eharney has quit IRC | 20:38 | |
| mriedem | so the commit message in https://review.openstack.org/#/c/485198/ is actually really misleading | 20:38 |
| scottda | yeah | 20:39 |
| mriedem | ok so i'll drop my change and just push a release note | 20:39 |
| scottda | cool | 20:39 |
| jungleboyj | mriedem: Cool. | 20:39 |
| mriedem | can we change the confidence ooze to a smelly brown stuff ooze? | 20:42 |
| * scottda goes to wash his hands.... | 20:45 | |
| dfisher | purell* | 20:45 |
| *** crushil has quit IRC | 20:46 | |
| *** openstackgerrit has joined #openstack-cinder | 20:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-cinderclient master: Add release note for get_highest_client_server_version return type change https://review.openstack.org/485817 | 20:47 |
| *** jwcroppe_ has quit IRC | 20:47 | |
| * jungleboyj is suddenly queasy ... | 20:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-cinderclient master: Add release note for get_highest_client_server_version return type change https://review.openstack.org/485817 | 20:50 |
| mriedem | oops one more thing | 20:51 |
| openstackgerrit | Matt Riedemann proposed openstack/python-cinderclient master: Add release note for get_highest_client_server_version return type change https://review.openstack.org/485817 | 20:51 |
| mriedem | there we go | 20:51 |
| *** jwcroppe has joined #openstack-cinder | 20:51 | |
| mriedem | done, for real, i swear | 20:51 |
| jungleboyj | Good god man! | 20:51 |
| mriedem | hey if you would have read the release note first you would have spotted those too :) | 20:51 |
| scottda | I read it. My brane just fixs all the erors. | 20:52 |
| imacdonn | you're brian ? | 20:53 |
| jungleboyj | I read it each time and .... oh nevermind. | 20:53 |
| jungleboyj | That is one way to get 3 reviews in 2 minutes. | 20:53 |
| scottda | I think it's still missing a comma.... | 20:53 |
| jungleboyj | mriedem: Can we do it again? | 20:53 |
| jungleboyj | scottda: I agree. | 20:53 |
| scottda | too late. | 20:54 |
| jungleboyj | It wasn't really worth worrying about. | 20:54 |
| mriedem | alright i'll rev the release patch after the nova meeting | 20:54 |
| *** arnewiebalck_ has quit IRC | 20:59 | |
| *** crushil has joined #openstack-cinder | 21:00 | |
| *** lucasxu has quit IRC | 21:02 | |
| *** reveller has joined #openstack-cinder | 21:03 | |
| *** Apoorva_ has quit IRC | 21:10 | |
| *** Apoorva has joined #openstack-cinder | 21:10 | |
| *** catintheroof has quit IRC | 21:10 | |
| *** jwcroppe has quit IRC | 21:12 | |
| *** jwcroppe has joined #openstack-cinder | 21:14 | |
| *** chlong_ has quit IRC | 21:16 | |
| *** itlinux has joined #openstack-cinder | 21:19 | |
| *** mvk has joined #openstack-cinder | 21:20 | |
| *** dustins has quit IRC | 21:24 | |
| *** gouthamr has quit IRC | 21:26 | |
| *** dave-mccowan has quit IRC | 21:31 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder master: NetApp cDOT: Support generic groups for block https://review.openstack.org/452306 | 21:33 |
| *** JoseMello has quit IRC | 21:36 | |
| *** xyang1 has quit IRC | 21:56 | |
| *** jwcroppe has quit IRC | 21:58 | |
| *** felipemonteiro_ has quit IRC | 22:01 | |
| openstackgerrit | Chris M proposed openstack/cinder master: Avoid use of deprecated command from lenovo driver. https://review.openstack.org/483612 | 22:02 |
| *** cknight has joined #openstack-cinder | 22:03 | |
| *** dave-mccowan has joined #openstack-cinder | 22:06 | |
| *** kfarr has quit IRC | 22:10 | |
| *** csaikia has quit IRC | 22:18 | |
| *** dave-mccowan has quit IRC | 22:24 | |
| *** mtanino__ has quit IRC | 22:27 | |
| smcginnis | Well that was an interesting scrollback. :) | 22:28 |
| *** jmlowe has joined #openstack-cinder | 22:35 | |
| *** tesseract has quit IRC | 22:37 | |
| *** edmondsw has quit IRC | 23:04 | |
| *** bbansal has quit IRC | 23:06 | |
| *** edmondsw_ has joined #openstack-cinder | 23:07 | |
| *** gouthamr has joined #openstack-cinder | 23:08 | |
| *** bbansal has joined #openstack-cinder | 23:08 | |
| *** leni1 has quit IRC | 23:10 | |
| *** jmlowe has quit IRC | 23:10 | |
| *** edmondsw_ has quit IRC | 23:11 | |
| *** cknight has quit IRC | 23:14 | |
| *** csaikia has joined #openstack-cinder | 23:18 | |
| *** jmlowe has joined #openstack-cinder | 23:21 | |
| bbansal | after we stack devstack, and run test cases | 23:21 |
| bbansal | does cinder.conf generates automatically? | 23:21 |
| smcginnis | bbansal: Yes, based on what you put in your local.conf, devstack generates the cinder.conf file. | 23:22 |
| bbansal | okay, sean i saw your comments on the patchset | 23:23 |
| bbansal | i am testing why i dont have those in my logs | 23:23 |
| bbansal | will udpate and thanks :) | 23:23 |
| smcginnis | bbansal: OK, great. Thanks for looking into it right away. | 23:23 |
| bbansal | can I pm you ? | 23:23 |
| smcginnis | bbansal: If you need to, but most things should be discussed in the channel. | 23:24 |
| smcginnis | bbansal: But if you have something you'd like to just talk to me about, sure. | 23:24 |
| bbansal | thanks yeah its related to my patch so thought will pm | 23:25 |
| *** chhavi has joined #openstack-cinder | 23:25 | |
| bbansal | just dropped u a pm | 23:25 |
| smcginnis | We do say to run all of the volume tests: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#What_tests_do_I_use.3F | 23:26 |
| smcginnis | bbansal: But I know some need to skip specific tests. | 23:26 |
| smcginnis | bbansal: It should be a small number of test though if you need to skip. | 23:26 |
| *** dfisher has quit IRC | 23:27 | |
| bbansal | okay i will have a word with my senior enginner regarding this | 23:27 |
| bbansal | right now we r skipping 11 tests | 23:27 |
| smcginnis | bbansal: Oh, I thought it was more. But yeah, it would be good to review those to know why you need to skip them. | 23:28 |
| bbansal | sure i will check them right away | 23:28 |
| smcginnis | Appreciate it. :) | 23:28 |
| *** edmondsw has joined #openstack-cinder | 23:29 | |
| *** chhavi has quit IRC | 23:29 | |
| *** _ix has quit IRC | 23:31 | |
| *** jwcroppe has joined #openstack-cinder | 23:33 | |
| *** armax has quit IRC | 23:33 | |
| *** edmondsw has quit IRC | 23:33 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Add release note for get_highest_client_server_version return type change https://review.openstack.org/485817 | 23:35 |
| *** Apoorva_ has joined #openstack-cinder | 23:37 | |
| *** Apoorva has quit IRC | 23:41 | |
| *** _ix has joined #openstack-cinder | 23:44 | |
| *** bbansal has quit IRC | 23:51 | |
| *** oomichi has quit IRC | 23:59 | |
| openstackgerrit | Merged openstack/cinder master: ZFSSA iSCSI delete volume with non-existent LUN https://review.openstack.org/484956 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!