| *** ducttape_ has quit IRC | 00:07 | |
| *** raunak has quit IRC | 00:15 | |
| *** leeantho has quit IRC | 00:19 | |
| *** esker has joined #openstack-cinder | 00:20 | |
| *** watanabe_isao has joined #openstack-cinder | 00:25 | |
| *** ducttape_ has joined #openstack-cinder | 00:28 | |
| *** Julien-zte has joined #openstack-cinder | 00:29 | |
| *** ducttape_ has quit IRC | 00:31 | |
| tonyb | smcginnis: Thanks for running with that os-brick patch. | 00:31 |
|---|---|---|
| tonyb | smcginnis: Sorry I missed details on my debugging/rationale for the fix | 00:31 |
| *** esker has quit IRC | 00:35 | |
| *** mtanino has quit IRC | 00:49 | |
| *** ducttape_ has joined #openstack-cinder | 00:49 | |
| *** ociuhandu has joined #openstack-cinder | 00:49 | |
| smcginnis | tonyb: No worries, thanks for getting it working! | 00:49 |
| *** laughterwym has joined #openstack-cinder | 00:50 | |
| *** bardia has joined #openstack-cinder | 00:51 | |
| *** chrichip has quit IRC | 00:52 | |
| *** chrichip has joined #openstack-cinder | 00:53 | |
| *** cheneydc has joined #openstack-cinder | 00:56 | |
| *** alonma has joined #openstack-cinder | 00:58 | |
| *** alonma has quit IRC | 01:02 | |
| *** ociuhandu has quit IRC | 01:08 | |
| *** ducttape_ has quit IRC | 01:15 | |
| *** alonma has joined #openstack-cinder | 01:18 | |
| *** chlong has joined #openstack-cinder | 01:18 | |
| *** ducttape_ has joined #openstack-cinder | 01:18 | |
| *** alonma has quit IRC | 01:23 | |
| *** yangyapeng has joined #openstack-cinder | 01:23 | |
| *** bardia has quit IRC | 01:24 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: Add v3 user messages with pagination https://review.openstack.org/299052 | 01:25 |
| openstackgerrit | wanghao proposed openstack/cinder: Support to query volume filter by change since/before https://review.openstack.org/352688 | 01:25 |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Support to query volume filter by change since/before https://review.openstack.org/352688 | 01:25 |
| *** cheneydc has quit IRC | 01:25 | |
| openstackgerrit | jingtao liang proposed openstack/cinder: Enable multipath flag for volume migration https://review.openstack.org/364015 | 01:27 |
| openstackgerrit | wanghao proposed openstack/cinder: Backup tenant attribute support https://review.openstack.org/258374 | 01:29 |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** salv-orlando has joined #openstack-cinder | 01:34 | |
| *** chrichip has quit IRC | 01:38 | |
| *** esker has joined #openstack-cinder | 01:38 | |
| *** EinstCrazy has joined #openstack-cinder | 01:39 | |
| *** EinstCrazy has quit IRC | 01:39 | |
| *** EinstCrazy has joined #openstack-cinder | 01:39 | |
| *** chrichip has joined #openstack-cinder | 01:39 | |
| *** enriquetaso has quit IRC | 01:41 | |
| *** ducttape_ has quit IRC | 01:42 | |
| *** diablo_rojo has quit IRC | 01:43 | |
| *** salv-orlando has quit IRC | 01:44 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap cluster related function with api_version https://review.openstack.org/362778 | 01:45 |
| *** ducttape_ has joined #openstack-cinder | 01:46 | |
| *** ducttape_ has quit IRC | 01:54 | |
| openstackgerrit | xianming.mao proposed openstack/cinder: Add programming language python 3 in classifier https://review.openstack.org/363429 | 02:01 |
| *** Apoorva has quit IRC | 02:03 | |
| *** EinstCra_ has joined #openstack-cinder | 02:04 | |
| *** sdake has quit IRC | 02:06 | |
| *** EinstCrazy has quit IRC | 02:08 | |
| *** esker is now known as esker[away] | 02:15 | |
| *** chrichip has quit IRC | 02:16 | |
| *** esker[away] has quit IRC | 02:16 | |
| *** chrichip has joined #openstack-cinder | 02:17 | |
| *** ducttape_ has joined #openstack-cinder | 02:19 | |
| *** esker has joined #openstack-cinder | 02:20 | |
| openstackgerrit | Merged openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/351275 | 02:23 |
| openstackgerrit | Merged openstack/python-cinderclient: Wrap volume_backup's update function with api_version https://review.openstack.org/362711 | 02:23 |
| openstackgerrit | Merged openstack/python-cinderclient: Wrap group type and group spec with api_version https://review.openstack.org/362735 | 02:23 |
| *** ducttape_ has quit IRC | 02:27 | |
| *** ducttape_ has joined #openstack-cinder | 02:35 | |
| *** EinstCra_ has quit IRC | 02:36 | |
| *** EinstCrazy has joined #openstack-cinder | 02:37 | |
| *** mriedem has quit IRC | 02:39 | |
| *** jamielennox|away is now known as jamielennox | 02:41 | |
| *** ducttape_ has quit IRC | 02:43 | |
| *** salv-orlando has joined #openstack-cinder | 02:43 | |
| *** gouthamr has quit IRC | 02:49 | |
| *** salv-orlando has quit IRC | 02:50 | |
| *** esker has quit IRC | 02:52 | |
| *** salv-orlando has joined #openstack-cinder | 02:52 | |
| *** chrichip has quit IRC | 02:54 | |
| *** chrichip has joined #openstack-cinder | 02:55 | |
| *** salv-orlando has quit IRC | 02:57 | |
| openstackgerrit | Merged openstack/cinder: HNAS: Add support for manage/unmanage snapshots in NFS driver https://review.openstack.org/333992 | 03:16 |
| *** raunak has joined #openstack-cinder | 03:17 | |
| openstackgerrit | Merged openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 03:18 |
| *** alonma has joined #openstack-cinder | 03:19 | |
| *** alonma has quit IRC | 03:24 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v3/ tests https://review.openstack.org/356841 | 03:27 |
| openstackgerrit | Merged openstack/cinder: XtremIO: fix the returned value of CG actions https://review.openstack.org/346831 | 03:30 |
| openstackgerrit | haobing proposed openstack/os-brick: Remove self.__dict__ for formatting strings https://review.openstack.org/364044 | 03:31 |
| *** ManishD has joined #openstack-cinder | 03:41 | |
| *** ducttape_ has joined #openstack-cinder | 03:43 | |
| *** Srinu has joined #openstack-cinder | 03:46 | |
| *** akshai_ has quit IRC | 03:48 | |
| *** raunak has quit IRC | 03:48 | |
| *** chrichip has quit IRC | 03:48 | |
| *** raunak has joined #openstack-cinder | 03:48 | |
| *** ducttape_ has quit IRC | 03:49 | |
| *** alonma has joined #openstack-cinder | 03:49 | |
| Srinu | patrickeast: hi | 03:51 |
| *** salv-orlando has joined #openstack-cinder | 03:53 | |
| *** alonma has quit IRC | 03:53 | |
| *** salv-orlando has quit IRC | 03:56 | |
| *** abhitechie has joined #openstack-cinder | 03:59 | |
| openstackgerrit | Merged openstack/python-cinderclient: Wrap cluster related function with api_version https://review.openstack.org/362778 | 04:01 |
| *** Srinu has quit IRC | 04:02 | |
| *** jamielennox is now known as jamielennox|away | 04:05 | |
| *** alonma has joined #openstack-cinder | 04:10 | |
| *** alonma has quit IRC | 04:14 | |
| *** alonma has joined #openstack-cinder | 04:30 | |
| *** cknight has joined #openstack-cinder | 04:32 | |
| *** links has joined #openstack-cinder | 04:32 | |
| *** asselin__ has joined #openstack-cinder | 04:34 | |
| *** krtaylor has quit IRC | 04:34 | |
| *** alonma has quit IRC | 04:34 | |
| *** rajinir has quit IRC | 04:35 | |
| *** netapp has quit IRC | 04:36 | |
| *** geguileo has quit IRC | 04:37 | |
| *** asselin_ has quit IRC | 04:37 | |
| *** amit213 has quit IRC | 04:37 | |
| *** scottda has quit IRC | 04:38 | |
| *** zhiyan has quit IRC | 04:38 | |
| *** fmccrthy has quit IRC | 04:38 | |
| *** pbandark has joined #openstack-cinder | 04:39 | |
| *** amit213 has joined #openstack-cinder | 04:40 | |
| *** scottda has joined #openstack-cinder | 04:40 | |
| *** zhiyan has joined #openstack-cinder | 04:40 | |
| *** fmccrthy has joined #openstack-cinder | 04:41 | |
| *** netapp has joined #openstack-cinder | 04:42 | |
| *** cknight has quit IRC | 04:44 | |
| *** geguileo has joined #openstack-cinder | 04:44 | |
| *** ducttape_ has joined #openstack-cinder | 04:45 | |
| *** ducttape_ has quit IRC | 04:50 | |
| *** alonma has joined #openstack-cinder | 04:50 | |
| *** laughterwym has quit IRC | 04:50 | |
| openstackgerrit | wanghao proposed openstack/cinder: Refactor volume status in managing vol https://review.openstack.org/333164 | 04:51 |
| *** laughterwym has joined #openstack-cinder | 04:52 | |
| *** alonma has quit IRC | 04:54 | |
| *** pcaruana has quit IRC | 04:57 | |
| *** pdeore has joined #openstack-cinder | 04:57 | |
| *** pgadiya has joined #openstack-cinder | 05:02 | |
| *** salv-orlando has joined #openstack-cinder | 05:03 | |
| *** cppforlife_ has quit IRC | 05:03 | |
| *** cargonza has quit IRC | 05:04 | |
| *** jamielennox|away is now known as jamielennox | 05:04 | |
| *** cppforlife_ has joined #openstack-cinder | 05:05 | |
| *** cargonza has joined #openstack-cinder | 05:06 | |
| *** harlowja_at_home has joined #openstack-cinder | 05:06 | |
| *** pgadiya has quit IRC | 05:09 | |
| *** salv-orlando has quit IRC | 05:10 | |
| *** alonma has joined #openstack-cinder | 05:10 | |
| *** AndyWojo has quit IRC | 05:13 | |
| *** salv-orlando has joined #openstack-cinder | 05:14 | |
| *** AndyWojo has joined #openstack-cinder | 05:14 | |
| *** alonma has quit IRC | 05:15 | |
| *** pgadiya has joined #openstack-cinder | 05:22 | |
| *** zhangguoqing has joined #openstack-cinder | 05:23 | |
| *** sdake has joined #openstack-cinder | 05:23 | |
| *** laughterwym has quit IRC | 05:26 | |
| *** raunak has quit IRC | 05:27 | |
| *** laughterwym has joined #openstack-cinder | 05:27 | |
| *** maestropandy has joined #openstack-cinder | 05:28 | |
| *** alonma has joined #openstack-cinder | 05:30 | |
| *** nkrinner_afk is now known as nkrinner | 05:32 | |
| *** alonma has quit IRC | 05:35 | |
| openstackgerrit | wanghao proposed openstack/cinder: Backup tenant attribute support https://review.openstack.org/258374 | 05:37 |
| *** asselin_ has joined #openstack-cinder | 05:38 | |
| *** asselin__ has quit IRC | 05:40 | |
| *** mjura has joined #openstack-cinder | 05:42 | |
| *** ChubYann has quit IRC | 05:42 | |
| *** wanghao_ has joined #openstack-cinder | 05:43 | |
| *** ducttape_ has joined #openstack-cinder | 05:46 | |
| *** ducttape_ has quit IRC | 05:50 | |
| *** harlowja_at_home has quit IRC | 05:51 | |
| *** alonma has joined #openstack-cinder | 05:51 | |
| *** mjura has quit IRC | 05:51 | |
| *** mjura has joined #openstack-cinder | 05:51 | |
| *** alonma has quit IRC | 05:55 | |
| *** edmondsw has joined #openstack-cinder | 05:59 | |
| *** edmondsw has quit IRC | 06:00 | |
| *** ebalduf has quit IRC | 06:00 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Make Resource class's function can be wraped by api_version https://review.openstack.org/364081 | 06:00 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Wrap GroupType class's function with api_version https://review.openstack.org/364082 | 06:00 |
| *** maestropandy has left #openstack-cinder | 06:01 | |
| *** coolsvap has joined #openstack-cinder | 06:05 | |
| *** BharatK_ has joined #openstack-cinder | 06:06 | |
| *** rcernin has quit IRC | 06:08 | |
| *** salv-orl_ has joined #openstack-cinder | 06:08 | |
| *** alonma has joined #openstack-cinder | 06:11 | |
| *** salv-orlando has quit IRC | 06:11 | |
| *** alonma has quit IRC | 06:15 | |
| *** lprice1 has joined #openstack-cinder | 06:19 | |
| *** dulek_ has joined #openstack-cinder | 06:20 | |
| *** JayConroy_ has joined #openstack-cinder | 06:20 | |
| *** dulek has quit IRC | 06:21 | |
| *** JayConroy has quit IRC | 06:21 | |
| *** cebruns__ has quit IRC | 06:21 | |
| openstackgerrit | haobing proposed openstack/python-cinderclient: Remove self.__dict__ for formatting strings https://review.openstack.org/364086 | 06:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/364087 | 06:21 |
| *** lprice has quit IRC | 06:21 | |
| *** dulek_ is now known as dulek | 06:21 | |
| *** JayConroy_ is now known as JayConroy | 06:21 | |
| *** cebruns_ has joined #openstack-cinder | 06:22 | |
| *** pcaruana has joined #openstack-cinder | 06:30 | |
| *** alonma has joined #openstack-cinder | 06:32 | |
| *** pschaef has joined #openstack-cinder | 06:32 | |
| *** arecknag has joined #openstack-cinder | 06:32 | |
| *** pschaef has quit IRC | 06:33 | |
| *** lpetrut has joined #openstack-cinder | 06:35 | |
| openstackgerrit | wanghao proposed openstack/cinder: Refactor volume status in managing vol https://review.openstack.org/333164 | 06:36 |
| *** alonma has quit IRC | 06:36 | |
| *** rcernin has joined #openstack-cinder | 06:41 | |
| *** rcernin has quit IRC | 06:45 | |
| *** rcernin has joined #openstack-cinder | 06:45 | |
| *** kaisers_ has joined #openstack-cinder | 06:46 | |
| *** ducttape_ has joined #openstack-cinder | 06:47 | |
| *** salv-orl_ has quit IRC | 06:50 | |
| *** alonma has joined #openstack-cinder | 06:52 | |
| *** ducttape_ has quit IRC | 06:52 | |
| *** gillesMo has joined #openstack-cinder | 06:54 | |
| *** alonma has quit IRC | 06:56 | |
| *** tesseract- has joined #openstack-cinder | 07:03 | |
| *** lpetrut has quit IRC | 07:17 | |
| *** jamielennox is now known as jamielennox|away | 07:17 | |
| *** BharatK_ has quit IRC | 07:18 | |
| *** cebruns_ has quit IRC | 07:21 | |
| *** winston-d has quit IRC | 07:22 | |
| *** winston-d has joined #openstack-cinder | 07:22 | |
| *** cebruns_ has joined #openstack-cinder | 07:22 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 07:24 |
| *** lpetrut has joined #openstack-cinder | 07:26 | |
| *** bkopilov has quit IRC | 07:30 | |
| *** lkuchlan has joined #openstack-cinder | 07:34 | |
| *** bkopilov has joined #openstack-cinder | 07:40 | |
| openstackgerrit | wanghao proposed openstack/cinder: Refactor volume status in managing vol https://review.openstack.org/333164 | 07:43 |
| *** sdake has quit IRC | 07:43 | |
| *** sdake has joined #openstack-cinder | 07:44 | |
| openstackgerrit | YaoZheng proposed openstack/python-cinderclient: Remove self.__dict__ for formatting strings https://review.openstack.org/364116 | 07:44 |
| *** lpetrut has quit IRC | 07:47 | |
| *** ducttape_ has joined #openstack-cinder | 07:49 | |
| *** salv-orlando has joined #openstack-cinder | 07:51 | |
| *** ducttape_ has quit IRC | 07:53 | |
| *** Deep_Tho1ght has joined #openstack-cinder | 07:53 | |
| *** liverpooler has joined #openstack-cinder | 07:54 | |
| Deep_Tho1ght | Hello, I'd like to understand, how cinder volume works? Does the cinder volumes take all the space available on a logical volume? | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** salv-orlando has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** arecknag has quit IRC | 08:08 | |
| *** arecknag has joined #openstack-cinder | 08:08 | |
| *** alonma has joined #openstack-cinder | 08:09 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-cinderclient: Add cinder tests for cinder volume create commands with parameters https://review.openstack.org/363540 | 08:09 |
| *** BharatK has joined #openstack-cinder | 08:15 | |
| *** BharatK has quit IRC | 08:16 | |
| *** BharatK has joined #openstack-cinder | 08:17 | |
| *** BharatK has quit IRC | 08:24 | |
| *** BharatK has joined #openstack-cinder | 08:28 | |
| sumit | Reported an issue "LUN ID is set to 0 for all the lefthand (VSA) volumes attached to an instance" : https://bugs.launchpad.net/cinder/+bug/1619140 | 08:28 |
| openstack | Launchpad bug 1619140 in Cinder "LUN ID is set to 0 for all the lefthand (VSA) volumes attached to an instance." [Undecided,New] | 08:28 |
| *** Julien-zte has quit IRC | 08:40 | |
| *** laughterwym has quit IRC | 08:43 | |
| *** e0ne has joined #openstack-cinder | 08:44 | |
| *** bkopilov has quit IRC | 08:45 | |
| *** houming has joined #openstack-cinder | 08:46 | |
| *** ducttape_ has joined #openstack-cinder | 08:49 | |
| openstackgerrit | JiWei proposed openstack/os-brick: Change assertTrue(isinstance()) with optimal assert https://review.openstack.org/364158 | 08:50 |
| *** dobson` has joined #openstack-cinder | 08:53 | |
| *** pck_ has joined #openstack-cinder | 08:53 | |
| *** pck_ has quit IRC | 08:54 | |
| *** pck_ has joined #openstack-cinder | 08:54 | |
| *** HenryG_ has joined #openstack-cinder | 08:54 | |
| *** ducttape_ has quit IRC | 08:54 | |
| *** lkuchlan is now known as lkuchlan_away | 08:55 | |
| *** winston-d has quit IRC | 08:58 | |
| *** dulek has quit IRC | 08:58 | |
| *** Swanson has quit IRC | 08:58 | |
| *** Fdaisuke has quit IRC | 08:58 | |
| *** lennyb has quit IRC | 08:58 | |
| *** bapalm has quit IRC | 08:58 | |
| *** dobson has quit IRC | 08:58 | |
| *** HenryG has quit IRC | 08:58 | |
| *** anteaya has quit IRC | 08:58 | |
| *** frakt has quit IRC | 08:58 | |
| *** pck has quit IRC | 08:58 | |
| *** Wi1d_ has quit IRC | 08:58 | |
| *** Anticimex has quit IRC | 08:58 | |
| *** jidar has quit IRC | 08:58 | |
| *** The_Ball has quit IRC | 08:58 | |
| *** salv-orlando has joined #openstack-cinder | 08:58 | |
| *** bapalm has joined #openstack-cinder | 08:59 | |
| *** Anticimex has joined #openstack-cinder | 09:00 | |
| *** Fdaisuke has joined #openstack-cinder | 09:00 | |
| *** salv-orlando has quit IRC | 09:01 | |
| *** winston-d has joined #openstack-cinder | 09:03 | |
| *** dulek has joined #openstack-cinder | 09:03 | |
| *** Swanson has joined #openstack-cinder | 09:03 | |
| *** lennyb has joined #openstack-cinder | 09:03 | |
| *** anteaya has joined #openstack-cinder | 09:03 | |
| *** frakt has joined #openstack-cinder | 09:03 | |
| *** Wi1d_ has joined #openstack-cinder | 09:03 | |
| *** jidar has joined #openstack-cinder | 09:03 | |
| *** The_Ball has joined #openstack-cinder | 09:03 | |
| *** mvk has joined #openstack-cinder | 09:13 | |
| *** HenryG_ is now known as HenryG | 09:17 | |
| openstackgerrit | haobing proposed openstack/python-cinderclient: Remove self.__dict__ for formatting strings https://review.openstack.org/364086 | 09:20 |
| *** BharatK has quit IRC | 09:20 | |
| *** lpetrut has joined #openstack-cinder | 09:22 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 09:28 |
| *** HenryG has quit IRC | 09:29 | |
| *** HenryG has joined #openstack-cinder | 09:29 | |
| *** abhinavtechie has joined #openstack-cinder | 09:36 | |
| *** alonma has quit IRC | 09:39 | |
| *** abhitechie has quit IRC | 09:39 | |
| *** alonma has joined #openstack-cinder | 09:42 | |
| *** salv-orlando has joined #openstack-cinder | 09:45 | |
| *** sdague has joined #openstack-cinder | 09:46 | |
| *** laughterwym has joined #openstack-cinder | 09:48 | |
| *** ducttape_ has joined #openstack-cinder | 09:50 | |
| *** laughterwym has quit IRC | 09:53 | |
| *** ducttape_ has quit IRC | 09:54 | |
| *** bhagyashri has joined #openstack-cinder | 09:55 | |
| *** sborkows has joined #openstack-cinder | 10:01 | |
| *** sdake has quit IRC | 10:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/364209 | 10:03 |
| *** ccesario has quit IRC | 10:06 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS stats reporting https://review.openstack.org/344477 | 10:07 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext: Updated from global requirements https://review.openstack.org/364212 | 10:10 |
| *** abhinavtechie has quit IRC | 10:11 | |
| *** Deep_Tho1ght has quit IRC | 10:15 | |
| *** ccesario has joined #openstack-cinder | 10:24 | |
| *** maestropandy1 has joined #openstack-cinder | 10:25 | |
| *** ntt has joined #openstack-cinder | 10:31 | |
| *** lkuchlan has joined #openstack-cinder | 10:39 | |
| *** EinstCrazy has quit IRC | 10:40 | |
| *** lkuchlan_away has quit IRC | 10:41 | |
| openstackgerrit | Taylor Huang proposed openstack/cinder: Rename specific driver option with synology prefix https://review.openstack.org/364225 | 10:42 |
| *** lprice1 has quit IRC | 10:42 | |
| *** lprice has joined #openstack-cinder | 10:42 | |
| bhagyashri | smcginnis: Hi, do you have free time? I have doubt regarding "os-force_detach" api. | 10:46 |
| *** yangyapeng has quit IRC | 10:48 | |
| *** maestropandy1 has quit IRC | 10:50 | |
| *** zhangguoqing has quit IRC | 10:50 | |
| *** ducttape_ has joined #openstack-cinder | 10:51 | |
| *** ducttape_ has quit IRC | 10:55 | |
| *** pdeore has quit IRC | 10:57 | |
| dulek | bhagyashri: smcginnis is in CST timezone - should be online in 3-4 hours. | 10:58 |
| *** salv-orlando has quit IRC | 10:58 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 11:03 |
| yuriy_n17 | geguileo: Hi! Very sorry for disturbing. Please take a look at https://review.openstack.org/#/c/300443/ whenever you have free time. Thanks a lot for advance. | 11:04 |
| yuriy_n17 | dulek: Hi! Very sorry for disturbing. Please take a look at https://review.openstack.org/#/c/300443/ whenever you have free time. Thanks a lot for advance. | 11:05 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: set_metadata throws AttributeError: id https://review.openstack.org/308475 | 11:09 |
| *** alyson_ has joined #openstack-cinder | 11:09 | |
| *** gcb has joined #openstack-cinder | 11:14 | |
| bhagyashri | dulek: Thank you. | 11:18 |
| *** zhangguoqing has joined #openstack-cinder | 11:20 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Fix CapacityFilter to accept None volume type https://review.openstack.org/364245 | 11:29 |
| *** maestropandy has joined #openstack-cinder | 11:30 | |
| *** maestropandy has left #openstack-cinder | 11:31 | |
| *** ntt has quit IRC | 11:34 | |
| openstackgerrit | Merged openstack/cinder: Deleting volume metadata keys with a single request https://review.openstack.org/328399 | 11:35 |
| openstackgerrit | Merged openstack/cinder: Stop using mox in test_wsgi, test_nexenta, test_netapp https://review.openstack.org/363382 | 11:40 |
| openstackgerrit | Merged openstack/cinder: Remove cinder/tests/unit/test_misc.py https://review.openstack.org/363677 | 11:40 |
| *** alyson_ has quit IRC | 11:42 | |
| *** ManishD has quit IRC | 11:47 | |
| *** alonma has quit IRC | 11:49 | |
| yuriy_n17 | dulek: Sorry for disturbing. What about to look at https://review.openstack.org/#/c/300443/? | 11:51 |
| *** ducttape_ has joined #openstack-cinder | 11:52 | |
| *** kongwei_ has joined #openstack-cinder | 11:53 | |
| dulek | yuriy_n17: Sure, I'll get to it in a moment. | 11:53 |
| yuriy_n17 | dulek: Thanks. | 11:54 |
| *** ducttape_ has quit IRC | 11:56 | |
| *** alyson_ has joined #openstack-cinder | 11:59 | |
| *** alonma has joined #openstack-cinder | 12:00 | |
| *** alonma has quit IRC | 12:04 | |
| *** tpsilva has joined #openstack-cinder | 12:05 | |
| *** laughterwym has joined #openstack-cinder | 12:07 | |
| *** ducttape_ has joined #openstack-cinder | 12:11 | |
| *** ducttape_ has quit IRC | 12:13 | |
| *** kongwei_ has quit IRC | 12:13 | |
| *** laughterwym has quit IRC | 12:14 | |
| *** laughterwym has joined #openstack-cinder | 12:15 | |
| scottda | ameade: Still trying to make Jenkins happy with v3 messages: https://review.openstack.org/#/c/299052 | 12:16 |
| *** BharatK has joined #openstack-cinder | 12:16 | |
| *** ducttape_ has joined #openstack-cinder | 12:17 | |
| *** kongwei_ has joined #openstack-cinder | 12:17 | |
| *** xyang has joined #openstack-cinder | 12:18 | |
| *** laughterwym has quit IRC | 12:19 | |
| *** alonma has joined #openstack-cinder | 12:20 | |
| *** BharatK has quit IRC | 12:21 | |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages with pagination https://review.openstack.org/299052 | 12:22 |
| ameade | scottda: ^, it's just that everyone is adding to the end of fakes.py >.< | 12:22 |
| scottda | ameade: I feel your pain | 12:22 |
| *** alonma has quit IRC | 12:24 | |
| scottda | yuriy_n17: I've added some feedback to 300443, if you've time to fix it. | 12:25 |
| *** mvk has quit IRC | 12:30 | |
| *** ducttape_ has quit IRC | 12:35 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix bug deleting cgsnapshot https://review.openstack.org/364272 | 12:36 |
| *** yangyapeng has joined #openstack-cinder | 12:36 | |
| *** ntt has joined #openstack-cinder | 12:39 | |
| *** alonma has joined #openstack-cinder | 12:40 | |
| *** Kimmo___ is now known as Kimmo_ | 12:44 | |
| *** alonma has quit IRC | 12:45 | |
| *** gcb has quit IRC | 12:48 | |
| *** jungleboyj has joined #openstack-cinder | 12:49 | |
| *** markvoelker has joined #openstack-cinder | 12:50 | |
| *** links has quit IRC | 12:51 | |
| *** gouthamr has joined #openstack-cinder | 12:51 | |
| scottda | xyang: More fixes from Cao to cinderclient (group types): https://review.openstack.org/#/c/364082 , depends on https://review.openstack.org/#/c/364081 | 12:51 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix exceed of dd action when copying with path https://review.openstack.org/354270 | 12:52 |
| xyang | scottda: sure, I'll take a look | 12:52 |
| openstackgerrit | Harini proposed openstack/cinder: Fix inconsistencies in "cinder-*" strings usage. https://review.openstack.org/364284 | 12:52 |
| *** ducttape_ has joined #openstack-cinder | 12:53 | |
| *** houming has quit IRC | 12:56 | |
| *** cdelatte has joined #openstack-cinder | 13:00 | |
| *** alonma has joined #openstack-cinder | 13:00 | |
| *** laughterwym has joined #openstack-cinder | 13:01 | |
| *** alonma has quit IRC | 13:02 | |
| *** alonma has joined #openstack-cinder | 13:02 | |
| *** rlrossit has joined #openstack-cinder | 13:03 | |
| *** laughterwym has quit IRC | 13:04 | |
| *** laughterwym has joined #openstack-cinder | 13:04 | |
| *** akerr has joined #openstack-cinder | 13:06 | |
| *** salv-orlando has joined #openstack-cinder | 13:07 | |
| *** markvoelker has quit IRC | 13:07 | |
| *** ducttape_ has quit IRC | 13:07 | |
| *** Julien-zte has joined #openstack-cinder | 13:08 | |
| *** akerr_ has joined #openstack-cinder | 13:08 | |
| scottda | bhagyashri: What was your question regarding os-force_detach? | 13:10 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 13:11 |
| *** laughterwym has quit IRC | 13:12 | |
| *** akerr has quit IRC | 13:12 | |
| *** jgregor has joined #openstack-cinder | 13:12 | |
| *** dustins has joined #openstack-cinder | 13:12 | |
| *** salv-orlando has quit IRC | 13:13 | |
| *** kongwei_ has quit IRC | 13:13 | |
| *** akshai has joined #openstack-cinder | 13:14 | |
| *** jgregor has quit IRC | 13:18 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 13:19 |
| bhagyashri | scottda: Hi, I want to know that is the "connector" parameter is mandatory in the request body for 'os-force-detach' ? | 13:19 |
| *** laughterwym has joined #openstack-cinder | 13:19 | |
| *** pgadiya has quit IRC | 13:19 | |
| scottda | bhagyashri: Yes, it is. And you should also know that this feature isn't well tested or supported... | 13:19 |
| scottda | bhagyashri: It "should" work if done properly, i.e. a connector is passed in. But we need to re-visit it when we've some cinder-nova api changes made in the Otaca release to make it more robust. That will mean Cinder having a better means to retrieve a connector for a volume if the caller doesn't have one. But that is in the future. | 13:22 |
| *** markvoelker has joined #openstack-cinder | 13:23 | |
| bhagyashri | scottda: yah, I can see that in api-ref it is shown as mandatory but for 'lvm' driver if we don't pass connector still the request is getting accepted on current master. | 13:23 |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix bug deleting cgsnapshot https://review.openstack.org/364272 | 13:25 |
| scottda | bhagyashri: hmmm..And that is not returning an error? | 13:25 |
| bhagyashri | scottda: yes it's not returning any error. | 13:26 |
| scottda | bhagyashri: OK. Code looks like it should throw an exception, but maybe there's a bug. Could you file a bug for that? | 13:26 |
| *** markvoelker has quit IRC | 13:28 | |
| bhagyashri | scottda: yes, I will file a bug for that and try to fix it. Thanks for your clarification. | 13:28 |
| e0ne | scottda: hi. I've got a question regarding to cinderclietn and microversions | 13:28 |
| scottda | e0ne: ? | 13:28 |
| *** rooneym has joined #openstack-cinder | 13:29 | |
| *** eharney has joined #openstack-cinder | 13:29 | |
| e0ne | scottda: please, remind me why did we decide to move v2 api code to /v3 and leave something like 'from cinderclient.v3.volume_transfers import *' in v2 | 13:29 |
| e0ne | scottda: it seams that v2 is inherited from v3 | 13:29 |
| e0ne | scottda: I'm confuded a bit | 13:30 |
| e0ne | s/confuded/confused | 13:30 |
| scottda | e0ne: Because V3 version 3.0 is identical to v2, and is the starting point. V2 should never change (although we are allowing some bugfixes to v2). All future API changes should go on v3 with a microversion. | 13:30 |
| *** faiz89 has joined #openstack-cinder | 13:31 | |
| *** baojg has joined #openstack-cinder | 13:31 | |
| e0ne | scottda: yes, but why we can't inherit v3 resources from v2? | 13:31 |
| *** ducttape_ has joined #openstack-cinder | 13:32 | |
| e0ne | scottda: I've found a lot of duplicated code between v3 and v1. does it make sense to fix it? | 13:32 |
| scottda | e0ne: We could have gone either direction, but we'd have had to make changes in the code in the v2 code with microversions for, say, 3.x. Either that or copy entire methods from v2 -> v3 just to make a change. | 13:33 |
| *** porrua has joined #openstack-cinder | 13:33 | |
| scottda | e0ne: Well, that duplicated code from v1 is a historical thing. When v2 was created, it was just copied over as far as I can see. I wasn't around back then... | 13:33 |
| e0ne | scottda: I do not understan why we use 'copy&paste' instead of inheritance | 13:34 |
| scottda | e0ne: The v1 duplication could be fixed, I guess. My instinct is not to touch it, and just let it rot to death... | 13:34 |
| scottda | e0ne: You mean no inheritance for v1? I agreee. | 13:34 |
| e0ne | scottda: I mean something like http://paste.openstack.org/show/565707/ | 13:36 |
| e0ne | scottda: here is example how we manage to do it in server-side: https://review.openstack.org/#/c/328399/19/cinder/api/v3/volume_metadata.py | 13:37 |
| *** diogogmt has quit IRC | 13:39 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext: Updated from global requirements https://review.openstack.org/364212 | 13:43 |
| *** rajinir has joined #openstack-cinder | 13:43 | |
| scottda | e0ne: It looks like we began by doing the right thing: inherit v3 resources from v2, i.e. v3/snapshots.py | 13:43 |
| scottda | e0ne: Then, when we've made changes, we do it in v3 by overriding the method, with new api microversion logic. | 13:43 |
| *** ducttape_ has quit IRC | 13:44 | |
| e0ne | scottda: exactly! | 13:44 |
| e0ne | scottda: and I'm asking: why do not to do it in client too? | 13:45 |
| *** diogogmt has joined #openstack-cinder | 13:45 | |
| scottda | e0ne: I honestly don't recall why. It may have been discussed, or maybe not... | 13:46 |
| *** mriedem has joined #openstack-cinder | 13:46 | |
| *** vbala has joined #openstack-cinder | 13:46 | |
| scottda | e0ne: I'm pretty certain I am to blame, whatever the reasons. | 13:47 |
| *** diablo_rojo has joined #openstack-cinder | 13:47 | |
| *** lprice has quit IRC | 13:48 | |
| *** openstackgerrit has quit IRC | 13:49 | |
| *** openstackgerrit has joined #openstack-cinder | 13:50 | |
| *** diogogmt has quit IRC | 13:50 | |
| scottda | e0ne: Looking at the original client patch (https://review.openstack.org/#/c/300028/) it was not discussed. I think it could be refactored though. If I needed to come up with a reason it would be "We merged the server side late in Mitaka, and the client patches went through in just a couple of weeks. More time would have allowed for better design and | 13:51 |
| scottda | review/test" | 13:51 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - Retrieve volume from snapshot https://review.openstack.org/363546 | 13:52 |
| e0ne | scottda: got it, thanks! | 13:52 |
| e0ne | scottda: I'll file a blueprint to refactor it | 13:53 |
| *** cknight has joined #openstack-cinder | 13:53 | |
| *** jiwei has joined #openstack-cinder | 13:53 | |
| *** Lee1092 has quit IRC | 13:54 | |
| *** jgregor has joined #openstack-cinder | 13:54 | |
| *** xyang1 has joined #openstack-cinder | 13:56 | |
| *** ducttape_ has joined #openstack-cinder | 13:57 | |
| *** akerr_ is now known as akerr_away | 13:58 | |
| *** eharney has quit IRC | 13:58 | |
| *** akerr_away is now known as akerr_ | 13:58 | |
| *** salv-orlando has joined #openstack-cinder | 13:59 | |
| scottda | e0ne: thanks. I'd be happy to implement it if you don't have time. Your choice. | 13:59 |
| *** akerr_ is now known as akerr_away | 13:59 | |
| *** akerr_away is now known as akerr_ | 13:59 | |
| *** yangyape_ has joined #openstack-cinder | 13:59 | |
| *** akerr_ is now known as akerr_away | 14:00 | |
| *** eharney has joined #openstack-cinder | 14:00 | |
| *** laughterwym has quit IRC | 14:00 | |
| *** yangyapeng has quit IRC | 14:03 | |
| openstackgerrit | Merged openstack/python-cinderclient: Make Resource class's function can be wraped by api_version https://review.openstack.org/364081 | 14:06 |
| openstackgerrit | Merged openstack/python-cinderclient: Wrap GroupType class's function with api_version https://review.openstack.org/364082 | 14:07 |
| openstackgerrit | Merged openstack/python-cinderclient: Remove self.__dict__ for formatting strings https://review.openstack.org/364086 | 14:09 |
| *** alyson_ has quit IRC | 14:09 | |
| *** akerr_away has quit IRC | 14:09 | |
| *** ducttape_ has quit IRC | 14:09 | |
| erlon | eharney: Hi Eric, can you give a look on this when you have a chance? I need a second core: https://review.openstack.org/#/c/344477/ | 14:09 |
| *** sandanar has joined #openstack-cinder | 14:11 | |
| *** tongli has joined #openstack-cinder | 14:15 | |
| *** ebalduf has joined #openstack-cinder | 14:16 | |
| *** vbala has quit IRC | 14:20 | |
| *** coolsvap has quit IRC | 14:22 | |
| *** rlrossit has quit IRC | 14:23 | |
| *** liverpooler has quit IRC | 14:23 | |
| *** lprice has joined #openstack-cinder | 14:24 | |
| *** mtanino has joined #openstack-cinder | 14:24 | |
| *** watanabe_isao has quit IRC | 14:25 | |
| openstackgerrit | Merged openstack/cinder: Fix CapacityFilter to accept None volume type https://review.openstack.org/364245 | 14:26 |
| *** crose has joined #openstack-cinder | 14:28 | |
| openstackgerrit | Merged openstack/cinder: Dell SC: Add exclude_domain_ip option https://review.openstack.org/359328 | 14:28 |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/364087 | 14:28 |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - Removal of iscsiadm from vmax cinder https://review.openstack.org/359908 | 14:28 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Re-use RBDImageMetadata and RBDVolumeIOWrapper from os-brick https://review.openstack.org/326999 | 14:28 |
| *** gillesMo has quit IRC | 14:29 | |
| *** baojg has quit IRC | 14:31 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add v3 user messages with pagination https://review.openstack.org/299052 | 14:32 |
| *** arecknag has quit IRC | 14:34 | |
| *** arecknag has joined #openstack-cinder | 14:35 | |
| *** eharney has quit IRC | 14:36 | |
| *** Lee1092 has joined #openstack-cinder | 14:38 | |
| *** ducttape_ has joined #openstack-cinder | 14:40 | |
| scottda | ameade: ^^^ yeah! | 14:40 |
| ameade | scottda: right on, thanks! | 14:40 |
| *** erhudy has joined #openstack-cinder | 14:49 | |
| *** sborkows has quit IRC | 14:50 | |
| *** diogogmt has joined #openstack-cinder | 14:51 | |
| *** xyang has quit IRC | 14:51 | |
| *** ChubYann has joined #openstack-cinder | 14:51 | |
| *** haplo37__ has joined #openstack-cinder | 14:52 | |
| *** yangyapeng has joined #openstack-cinder | 14:53 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 14:54 |
| *** esker has joined #openstack-cinder | 14:54 | |
| *** yangyape_ has quit IRC | 14:54 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - multi-attach failure to VMAX3 when SLO is omitted https://review.openstack.org/308952 | 14:55 |
| *** xyang has joined #openstack-cinder | 14:55 | |
| *** alonma has quit IRC | 14:56 | |
| *** krtaylor has joined #openstack-cinder | 14:58 | |
| *** mjura has quit IRC | 15:01 | |
| *** zhangguoqing has quit IRC | 15:02 | |
| *** pcaruana has quit IRC | 15:03 | |
| *** eharney has joined #openstack-cinder | 15:03 | |
| openstackgerrit | Harini proposed openstack/cinder: Fix inconsistencies in "cinder-*" strings usage. https://review.openstack.org/364284 | 15:05 |
| *** alonma has joined #openstack-cinder | 15:06 | |
| *** rcernin has quit IRC | 15:06 | |
| *** jiwei has quit IRC | 15:09 | |
| *** alonma has quit IRC | 15:10 | |
| *** nkrinner is now known as nkrinner_afk | 15:11 | |
| *** hemnafk is now known as hemna | 15:15 | |
| *** sandanar has quit IRC | 15:17 | |
| *** laughterwym has joined #openstack-cinder | 15:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/volume/drivers/*.py https://review.openstack.org/357652 | 15:19 |
| *** mvk has joined #openstack-cinder | 15:19 | |
| *** laughterwym has quit IRC | 15:23 | |
| *** leeantho has joined #openstack-cinder | 15:23 | |
| *** sdake has joined #openstack-cinder | 15:23 | |
| *** alonma has joined #openstack-cinder | 15:26 | |
| *** jordanP has joined #openstack-cinder | 15:26 | |
| *** arch-nemesis has joined #openstack-cinder | 15:28 | |
| *** alonma has quit IRC | 15:30 | |
| *** rcernin has joined #openstack-cinder | 15:31 | |
| *** arecknag has quit IRC | 15:31 | |
| *** laughterwym has joined #openstack-cinder | 15:32 | |
| *** arch-nemesis has quit IRC | 15:33 | |
| *** BharatK has joined #openstack-cinder | 15:34 | |
| *** pbandark has quit IRC | 15:34 | |
| *** laughter_ has joined #openstack-cinder | 15:34 | |
| karthikp | geguileo: Hi | 15:38 |
| *** laughterwym has quit IRC | 15:38 | |
| *** baojg has joined #openstack-cinder | 15:42 | |
| *** salv-orlando has quit IRC | 15:43 | |
| *** faiz89 has quit IRC | 15:44 | |
| *** baojg has quit IRC | 15:44 | |
| *** Yogi1 has joined #openstack-cinder | 15:45 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - failed rollback on VMAX3 when MV issue https://review.openstack.org/358026 | 15:46 |
| *** alonma has joined #openstack-cinder | 15:46 | |
| geguileo | karthikp: Hi | 15:47 |
| *** BharatK_ has joined #openstack-cinder | 15:47 | |
| *** kaisers_ has quit IRC | 15:47 | |
| *** arch-nemesis has joined #openstack-cinder | 15:47 | |
| *** BharatK has quit IRC | 15:47 | |
| *** xinli has joined #openstack-cinder | 15:48 | |
| *** enriquetaso has joined #openstack-cinder | 15:49 | |
| *** alonma has quit IRC | 15:51 | |
| *** markvoelker has joined #openstack-cinder | 15:51 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder: Fix manage existing MissingDependencies error https://review.openstack.org/361454 | 15:54 |
| *** esker has quit IRC | 15:55 | |
| karthikp | geguileo: Please could you review my patch in your free time .... https://review.openstack.org/#/c/319040. Could really use some help here | 15:56 |
| *** laughter_ has quit IRC | 15:57 | |
| *** laughterwym has joined #openstack-cinder | 15:58 | |
| *** jgregor has quit IRC | 16:01 | |
| *** laughterwym has quit IRC | 16:02 | |
| *** alkhodos has joined #openstack-cinder | 16:04 | |
| hemna | patrickeast, ping | 16:04 |
| *** fredli has joined #openstack-cinder | 16:05 | |
| *** raunak has joined #openstack-cinder | 16:06 | |
| *** alonma has joined #openstack-cinder | 16:07 | |
| *** GB21 has joined #openstack-cinder | 16:11 | |
| *** alonma has quit IRC | 16:11 | |
| geguileo | karthikp: I'll try to get to it | 16:12 |
| *** salv-orlando has joined #openstack-cinder | 16:12 | |
| *** jordanP has quit IRC | 16:12 | |
| karthikp | geguileo: sure thanks | 16:14 |
| *** chris_morrell has joined #openstack-cinder | 16:15 | |
| *** xinli has quit IRC | 16:18 | |
| *** xinli has joined #openstack-cinder | 16:18 | |
| *** lkuchlan has quit IRC | 16:21 | |
| *** e0ne has quit IRC | 16:21 | |
| *** BharatK_ has quit IRC | 16:21 | |
| *** akerr has joined #openstack-cinder | 16:21 | |
| *** akerr is now known as akerr_away | 16:21 | |
| *** prometheanfire has joined #openstack-cinder | 16:21 | |
| *** akerr_away is now known as akerr | 16:22 | |
| prometheanfire | is the new os-brick the one you want to merge into requirements (1.6.1)? | 16:22 |
| prometheanfire | if so, it breaks cinder, nova and dvsm | 16:22 |
| prometheanfire | https://review.openstack.org/364255 | 16:22 |
| *** BharatK has joined #openstack-cinder | 16:23 | |
| mriedem | https://github.com/openstack/os-brick/commit/f8e4f3cdbef614367c5b3cfcb32e1584475946f8 is the breaking change | 16:24 |
| hemna | what'd I break ? | 16:24 |
| *** coolsvap has joined #openstack-cinder | 16:24 | |
| smcginnis | mriedem: That doesn't look like it. | 16:24 |
| smcginnis | The whole 1.6.1 release was supposed to fix that failure from 1.6.0. :/ | 16:25 |
| *** ManishD has joined #openstack-cinder | 16:25 | |
| smcginnis | Not "that failure" as in password masking - "that failure" that now shows up in the logs due to unit tests trying to load connectors directly. | 16:25 |
| *** krtaylor has quit IRC | 16:25 | |
| mriedem | it breaks a unit test in nova http://logs.openstack.org/55/364255/1/check/gate-cross-nova-python27-db-ubuntu-xenial/26598da/console.html#_2016-09-01_12_08_57_633972 | 16:25 |
| hemna | smcginnis, didn't we get in the aliasing patch that fixed the connector access? | 16:26 |
| mriedem | i'm not saying it's a bad fix, it's just the thing that's breaking nova unit tests with that 1.6.1 version | 16:26 |
| mriedem | it's actually in 1.6.0 | 16:26 |
| mriedem | the u-c change is jumping from 1.5.0 to 1.6.1 | 16:26 |
| smcginnis | hemna: Yeah, that went in and 1.6.1 was released to get it out there. But the cinder failure looks like it's still failing that. | 16:26 |
| *** alonma has joined #openstack-cinder | 16:27 | |
| mriedem | cinder unit tests fail differently http://logs.openstack.org/55/364255/1/check/gate-cross-cinder-python27-db-ubuntu-xenial/056c655/console.html#_2016-09-01_12_05_43_797897 | 16:27 |
| prometheanfire | we skipped 1.6.0 because of known bugs | 16:27 |
| smcginnis | mriedem: That nova failure looks like it's on checking a logging call? http://logs.openstack.org/55/364255/1/check/gate-cross-nova-python27-db-ubuntu-xenial/26598da/console.html | 16:27 |
| mriedem | smcginnis: yeah it was checking that we masked the password | 16:28 |
| smcginnis | Known bugs being that unit tests were trying to access something that's not there anymore. | 16:28 |
| mriedem | but os-brick is probably already doing that now, so the test is failing on some assertion | 16:28 |
| hemna | so os-brick only masks the password when tracing | 16:28 |
| hemna | was nova testing that tracing ? | 16:28 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/volume/test_iscsi.py#L54 | 16:30 |
| hemna | yah looking now | 16:30 |
| mriedem | given the TODO, if os-brick is testing that now, we can probably just remove the nova unit test | 16:30 |
| *** alonma has quit IRC | 16:31 | |
| *** BharatK has quit IRC | 16:31 | |
| smcginnis | hemna: I don't get why the cinder UT is failing, but we should probably just fix that test. http://logs.openstack.org/55/364255/1/check/gate-cross-cinder-python27-db-ubuntu-xenial/056c655/console.html#_2016-09-01_12_05_43_798123 | 16:31 |
| mriedem | it seems that https://github.com/openstack/os-brick/commit/49a89c8757aad70611a00ea9e1c78f38060bc114 would have fixed the cinder fialure | 16:31 |
| mriedem | *failure | 16:31 |
| mriedem | with the FakeConnector alias | 16:31 |
| hemna | hrmm | 16:32 |
| hemna | I don't get why the nova test is failing | 16:32 |
| prometheanfire | mriedem: that'd fix the nova fail, but what about the cinder one? | 16:32 |
| hemna | the masking in utils.py only masks the @utils.trace stuff | 16:32 |
| *** zul has quit IRC | 16:32 | |
| mriedem | hemna: maybe a red herring | 16:33 |
| mriedem | the test sucks anyway b/c it relies on things in os-brick being called | 16:33 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/volume/test_iscsi.py#L70 | 16:33 |
| mriedem | it expects connector.LOG.debug to be called in there somewhere | 16:33 |
| mriedem | this is probably a relic of the move to using os-brick in nova and hacking the tests to make them work with the os-brick version at the time | 16:33 |
| hemna | which it should be here: https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/iscsi.py#L593 | 16:34 |
| hemna | I think nova shouldn't care about the internals of os-brick though | 16:34 |
| smcginnis | I strongly dislike tests asserting on a log.debug being called. | 16:34 |
| smcginnis | hemna: +1 | 16:34 |
| mriedem | well, these weren't internal to os-brick back in the day | 16:35 |
| hemna | true | 16:35 |
| mriedem | and like i said, this test was hacked to work with the integration of os-brick | 16:35 |
| hemna | probably like you said, a relic of when that code lived in nova | 16:35 |
| mriedem | i put the TODO on the test to move it to os-brick becaues apparently at that time there wasn't a unit test like this in os-brick | 16:35 |
| mriedem | anyway, i'm ok with just removing it | 16:36 |
| hemna | ok so what do we want to do? | 16:36 |
| mriedem | i'll push a change to remove it | 16:36 |
| hemna | ok | 16:36 |
| mriedem | prometheanfire: do you have a bug for the failures with 1.6.1? | 16:36 |
| *** xyang has quit IRC | 16:37 | |
| prometheanfire | mriedem: no, just noticed it failed in our cross testing | 16:37 |
| *** xyang has joined #openstack-cinder | 16:38 | |
| *** xyang has quit IRC | 16:39 | |
| *** Apoorva has joined #openstack-cinder | 16:40 | |
| mriedem | tonyb already has a bug for these https://bugs.launchpad.net/os-brick/+bug/1618697 | 16:40 |
| openstack | Launchpad bug 1618697 in os-brick "os-brick 1.6.0 refactor was a major API change" [Undecided,New] | 16:40 |
| *** xyang has joined #openstack-cinder | 16:41 | |
| smcginnis | hemna: Any ideas why the cinder one fails? I thought the fix we merged to os-brick should have addressed that. | 16:42 |
| *** xyang has quit IRC | 16:43 | |
| hemna | smcginnis, looking | 16:44 |
| hemna | so that FakeConnector got moved | 16:44 |
| *** akshai has quit IRC | 16:45 | |
| mriedem | prometheanfire: added https://review.openstack.org/#/c/364454/ to your u-c change | 16:45 |
| hemna | smcginnis, https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/fake.py | 16:45 |
| *** zul has joined #openstack-cinder | 16:45 | |
| mriedem | it's still going to fail, but at least i can move on | 16:45 |
| smcginnis | Hmm, should that have been _name = 'os_brick.initiator.connector.%s' % item.split('.')[-1] | 16:45 |
| mriedem | keep in mind that until g-r reqiures os-brick>=1.6.1 you shouldn't hack up any unit tests assuming you're using 1.6.1 | 16:46 |
| mriedem | so you might have to straddle both versions in a unit test | 16:46 |
| mriedem | yay for api changes | 16:46 |
| hemna | smcginnis, and the FakeConnector isn't in the connector_list | 16:46 |
| mriedem | just create an alias in os-brick | 16:46 |
| hemna | because it's not a real connector, and not needed in the call to collect the initiator information at attach time. | 16:46 |
| prometheanfire | mriedem: k, thanks | 16:46 |
| hemna | smcginnis, https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L77 | 16:47 |
| smcginnis | Ah, didn't notice it wasn't in the list. That explains some things. | 16:47 |
| *** alonma has joined #openstack-cinder | 16:47 | |
| *** akshai has joined #openstack-cinder | 16:47 | |
| smcginnis | Crap, gotta go to a meeting. Will be back soon. | 16:47 |
| *** xyang has joined #openstack-cinder | 16:48 | |
| hemna | ugh this sux | 16:49 |
| mriedem | hemna: easiest thing to probably do to fix the cinder test is add a conditional that checks the module for the FakeConnector in both locations | 16:51 |
| mriedem | hasattr | 16:51 |
| mriedem | and then drop that when g-r requires os-brick>=1.6.1 | 16:51 |
| *** alonma has quit IRC | 16:52 | |
| hemna | either that or add the FakeConnector to the aliasing in brick | 16:52 |
| mriedem | yeah, that will require another 1.6.2 release | 16:52 |
| hemna | and do another drop 1.6.2 | 16:52 |
| hemna | probably the most backwards compatible way | 16:52 |
| mriedem | also | 16:52 |
| mriedem | stable/newton exists for os-brick now | 16:52 |
| mriedem | so you'd have to backport that | 16:52 |
| mriedem | so you start getting into some weird versoin boundary territory between branches | 16:53 |
| hemna | yah | 16:53 |
| mriedem | personally i'd hack the cinder unit test to make it work both ways | 16:53 |
| mriedem | bump min in g-r | 16:53 |
| mriedem | and move ahead | 16:54 |
| *** markvoelker has quit IRC | 16:58 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 16:58 |
| *** wanghao_ has quit IRC | 17:00 | |
| *** tesseract- has quit IRC | 17:01 | |
| *** ducttape_ has quit IRC | 17:03 | |
| *** ducttape_ has joined #openstack-cinder | 17:04 | |
| *** alonma has joined #openstack-cinder | 17:07 | |
| *** mvk has quit IRC | 17:07 | |
| *** krtaylor has joined #openstack-cinder | 17:08 | |
| hemna | I has confused: http://paste.openstack.org/show/565746/ | 17:09 |
| *** alonma has quit IRC | 17:12 | |
| hemna | ah tox is pulling 1.5.0 | 17:12 |
| *** lpetrut has quit IRC | 17:12 | |
| *** jgregor has joined #openstack-cinder | 17:13 | |
| *** jgregor has quit IRC | 17:13 | |
| *** ducttape_ has quit IRC | 17:15 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v2/ tests https://review.openstack.org/356347 | 17:18 |
| *** lkuchlan has joined #openstack-cinder | 17:20 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix backup unit test with new os-brick https://review.openstack.org/364475 | 17:20 |
| *** pauloewerton has joined #openstack-cinder | 17:20 | |
| *** ducttape_ has joined #openstack-cinder | 17:23 | |
| xyang | hemna: does this missing FakeConnector only affecting one test? | 17:24 |
| *** lkuchlan has quit IRC | 17:25 | |
| *** dobson` has quit IRC | 17:26 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix backup unit test with new os-brick https://review.openstack.org/364475 | 17:26 |
| hemna | it's only that one test in there afaik | 17:27 |
| xyang | hemna: ok | 17:27 |
| *** ntt has quit IRC | 17:27 | |
| *** alonma has joined #openstack-cinder | 17:27 | |
| *** alonma has quit IRC | 17:32 | |
| *** e0ne has joined #openstack-cinder | 17:32 | |
| *** akshai has quit IRC | 17:33 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix backup unit test with new os-brick https://review.openstack.org/364475 | 17:34 |
| *** ManishD has quit IRC | 17:35 | |
| *** kaisers_ has joined #openstack-cinder | 17:35 | |
| *** dkehn_ has quit IRC | 17:40 | |
| *** kaisers_ has quit IRC | 17:40 | |
| patrickeast | hemna: pong | 17:42 |
| openstackgerrit | Merged openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 17:45 |
| *** dstepanenko_ has left #openstack-cinder | 17:48 | |
| *** alonma has joined #openstack-cinder | 17:48 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - Does not honor clone volume size https://review.openstack.org/345321 | 17:50 |
| hemna | patrickeast, is your CI reporting latetly? | 17:52 |
| hemna | patrickeast, https://review.openstack.org/#/c/363957/ | 17:52 |
| patrickeast | hemna: sorta, the last couple of days I've been messing with it | 17:52 |
| hemna | ok | 17:53 |
| *** alonma has quit IRC | 17:53 | |
| patrickeast | and last night it looks like zuul-merger choked | 17:53 |
| hemna | ewps | 17:53 |
| *** e0ne has quit IRC | 17:53 | |
| *** mvk has joined #openstack-cinder | 17:53 | |
| *** prometheanfire has left #openstack-cinder | 17:54 | |
| *** dkehn_ has joined #openstack-cinder | 17:54 | |
| *** lpetrut has joined #openstack-cinder | 17:55 | |
| *** dobson has joined #openstack-cinder | 17:56 | |
| *** harlowja_ has joined #openstack-cinder | 17:58 | |
| *** harlowja has quit IRC | 18:00 | |
| *** e0ne has joined #openstack-cinder | 18:01 | |
| *** salv-orl_ has joined #openstack-cinder | 18:08 | |
| *** alonma has joined #openstack-cinder | 18:08 | |
| *** diablo_rojo_phon has joined #openstack-cinder | 18:10 | |
| *** salv-orlando has quit IRC | 18:11 | |
| *** hoonetorg has quit IRC | 18:11 | |
| *** alonma has quit IRC | 18:12 | |
| *** jgregor has joined #openstack-cinder | 18:15 | |
| *** hoonetorg has joined #openstack-cinder | 18:16 | |
| smcginnis | hemna: Back. You figure it all out? :) | 18:17 |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #openstack-cinder | 18:19 | |
| *** alonma has joined #openstack-cinder | 18:20 | |
| *** fredli has quit IRC | 18:20 | |
| *** GB21 has quit IRC | 18:21 | |
| *** alonma has quit IRC | 18:24 | |
| *** mtanino__ has joined #openstack-cinder | 18:26 | |
| hemna | smcginnis, https://review.openstack.org/#/c/364475/ | 18:27 |
| *** mtanino has quit IRC | 18:27 | |
| *** mtanino has joined #openstack-cinder | 18:31 | |
| *** harlowja_ has quit IRC | 18:31 | |
| smcginnis | hemna: Thanks, looks like that should do it. | 18:31 |
| *** harlowja has joined #openstack-cinder | 18:32 | |
| *** mtanino__ has quit IRC | 18:33 | |
| smcginnis | Looks like I'm going to have to submit the release requests around 11:59 tonight. :/ | 18:34 |
| *** akshai has joined #openstack-cinder | 18:37 | |
| *** akerr is now known as akerr_away | 18:37 | |
| *** akerr_away is now known as akerr | 18:37 | |
| *** akerr is now known as akerr_away | 18:38 | |
| *** akerr_away is now known as akerr | 18:38 | |
| *** akerr is now known as akerr_away | 18:38 | |
| *** akerr_away is now known as akerr | 18:39 | |
| *** akerr is now known as akerr_away | 18:39 | |
| *** akerr_away is now known as akerr | 18:39 | |
| *** crose has quit IRC | 18:39 | |
| *** akerr is now known as akerr_away | 18:40 | |
| scottda | smcginnis If you need a break from thinking about the release, have a think on this: https://review.openstack.org/#/c/299458 | 18:40 |
| *** alonma has joined #openstack-cinder | 18:40 | |
| *** akerr_away is now known as akerr | 18:40 | |
| e0ne | smcginnis: do you mean both client and cinder releases? which timezone? | 18:41 |
| *** akerr is now known as akerr_away | 18:41 | |
| *** akerr_away is now known as akerr | 18:42 | |
| *** akerr is now known as akerr_away | 18:42 | |
| *** salv-orl_ has quit IRC | 18:42 | |
| scottda | e0ne: You might have opinions on that one ^^^ It's about backporting fix to stable cinderclient branches | 18:43 |
| *** akerr_away is now known as akerr | 18:43 | |
| *** akerr is now known as akerr_away | 18:43 | |
| e0ne | scottda: +1'ed. Mirantis uses the lastst stable branch for our distro too | 18:43 |
| scottda | e0ne: Yeah, we've use it as well. | 18:44 |
| *** akerr_away is now known as akerr | 18:44 | |
| e0ne | scottda: and thechnically, we can release stable verssions | 18:44 |
| scottda | e0ne: Yeah, I don't disagree with geguileo 's idea. We just haven't done it in the past, I don't think | 18:44 |
| *** akerr is now known as akerr_away | 18:44 | |
| *** akerr_away is now known as akerr | 18:44 | |
| e0ne | scottda: I'm going to propose this topic for design session | 18:45 |
| scottda | e0ne: Good idea. | 18:45 |
| *** alonma has quit IRC | 18:45 | |
| *** akerr is now known as akerr_away | 18:45 | |
| *** akerr_away is now known as akerr | 18:45 | |
| e0ne | scottda: IMO, if we need to release fix to cinderclient/os-brick, we have to do it | 18:45 |
| e0ne | scottda: but we must be sure, that we don't break anything | 18:45 |
| *** jdross has quit IRC | 18:45 | |
| *** akerr is now known as akerr_away | 18:46 | |
| *** akerr_away is now known as akerr | 18:46 | |
| scottda | e0ne: Yeah, I think that's why we haven't done any such thing in the past, for fear of breaking something. It's easier just not to release patch fix versions for older releases. | 18:46 |
| *** akerr is now known as akerr_away | 18:47 | |
| *** akerr_away is now known as akerr | 18:47 | |
| *** akerr is now known as akerr_away | 18:47 | |
| *** david-lyle has quit IRC | 18:49 | |
| *** david-lyle has joined #openstack-cinder | 18:50 | |
| *** jdross has joined #openstack-cinder | 18:50 | |
| smcginnis | e0ne: Releasing the "final" client and tagging Newton-3. | 18:54 |
| e0ne | smcginnis: do I have a time to fix comments for https://review.openstack.org/#/c/300443/? | 18:55 |
| scottda | e0ne: Go for it. smcginnis Won't be going to sleep anytime soon. | 18:56 |
| e0ne | scottda: :) | 18:56 |
| *** mriedem has quit IRC | 18:56 | |
| smcginnis | e0ne: Some time, but the sooner the better. :) | 18:56 |
| e0ne | my devstack env is almost ready | 18:56 |
| * smcginnis puts scottda down on his list | 18:56 | |
| smcginnis | :) | 18:56 |
| e0ne | I want to test patch before submitting | 18:56 |
| scottda | crap! | 18:56 |
| *** akshai has quit IRC | 18:57 | |
| *** alonma has joined #openstack-cinder | 19:00 | |
| *** alonma has quit IRC | 19:05 | |
| *** xyang has quit IRC | 19:09 | |
| *** akerr_away is now known as akerr | 19:11 | |
| *** akerr is now known as akerr_away | 19:12 | |
| *** xyang has joined #openstack-cinder | 19:13 | |
| *** harlowja_ has joined #openstack-cinder | 19:13 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:14 |
| *** raunak has quit IRC | 19:14 | |
| *** akerr_away is now known as akerr | 19:15 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:16 |
| e0ne | scottda: I failed to use 'start_version' key for existing param:( | 19:16 |
| *** harlowja has quit IRC | 19:17 | |
| scottda | e0ne: :) | 19:19 |
| e0ne | let me 5 minutes more | 19:19 |
| scottda | I'll be around. It's early for me. | 19:20 |
| e0ne | found one issue | 19:20 |
| *** alonma has joined #openstack-cinder | 19:21 | |
| *** alkhodos__ has joined #openstack-cinder | 19:21 | |
| alkhodos__ | Hi everyone, can any reviewers take a look at these patches? https://review.openstack.org/#/c/358474/ https://review.openstack.org/#/c/356142/ https://review.openstack.org/#/c/297880/ They been hanging there for a while... | 19:23 |
| *** salv-orlando has joined #openstack-cinder | 19:23 | |
| *** Lee1092 has quit IRC | 19:24 | |
| *** alkhodos__ has quit IRC | 19:24 | |
| *** johnplanow has joined #openstack-cinder | 19:24 | |
| *** raunak has joined #openstack-cinder | 19:24 | |
| *** raunak has quit IRC | 19:25 | |
| *** alkhodos__ has joined #openstack-cinder | 19:25 | |
| *** alonma has quit IRC | 19:25 | |
| *** chrichip has joined #openstack-cinder | 19:25 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:25 |
| scottda | e0ne: Sorry, see my comments | 19:32 |
| scottda | e0ne: My comment fix get's rid of duplicate entries: | 19:33 |
| scottda | https://www.irccloud.com/pastebin/1cGJiI7T/ | 19:33 |
| scottda | https://www.irccloud.com/pastebin/r7EMSCvD/ | 19:33 |
| scottda | e0ne: I'd fix it myself, but then there'd be noone left to review... | 19:33 |
| e0ne | scottda: bad news: looks like this path doesn't work:( | 19:35 |
| scottda | e0ne: my fix? I tested it it seems to work? | 19:35 |
| e0ne | scottda: http://paste.openstack.org/show/565876/ | 19:35 |
| scottda | e0ne: weird. Is that paste just from after your fix to the help, or is it a different cause? | 19:36 |
| e0ne | I hate last-minute fixes:( | 19:36 |
| e0ne | scottda: it's different issue | 19:36 |
| *** sdague has quit IRC | 19:37 | |
| e0ne | scottda: it doesn't delete more than 1 key. | 19:38 |
| e0ne | scottda: I'll try to debug it | 19:38 |
| e0ne | if I failed to fix it in 10-15 minutes, I would like to not block release for it | 19:38 |
| *** alonma has joined #openstack-cinder | 19:41 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:42 |
| scottda | e0ne: Yes, I agree we shouldn't block the release. If I understand, you are saying this fix wouldn't get in, right? | 19:42 |
| e0ne | scottda: it was a rebase issue, thanks for the help | 19:43 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:43 |
| e0ne | scottda: finally, looks like it works ^^ | 19:44 |
| *** tongli has quit IRC | 19:44 | |
| *** alonma has quit IRC | 19:45 | |
| scottda | It didn't work for me...I needed this patch: | 19:48 |
| scottda | https://www.irccloud.com/pastebin/rYouoVv5/ | 19:48 |
| *** mriedem has joined #openstack-cinder | 19:49 | |
| scottda | e0ne: Wait, it's my ENV that's the problem, I think... | 19:49 |
| e0ne | and unit tests failed:( | 19:49 |
| *** raunak has joined #openstack-cinder | 19:51 | |
| e0ne | scottda: microversions make code and rebases more complicated:( | 19:52 |
| scottda | Yeah, sorry about that. | 19:52 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 19:53 |
| e0ne | scottda: it's not your fault, TBH | 19:53 |
| scottda | Yeah, but I still feel bad about any complaints regarding microversions. | 19:53 |
| e0ne | scottda: maybe we should try to think how to make a code less complicated and make PoC for it | 19:54 |
| scottda | e0ne: You mean this patch, or microversion code in general? | 19:55 |
| e0ne | scottda: I mean microversion code in general | 19:55 |
| e0ne | scottda: IMO, it's too many places whe we hardcode microversion number | 19:56 |
| scottda | e0ne: OK. But I was wrong about you needing to bump the microversion in the client. It was my server that was out of date... | 19:56 |
| e0ne | scottda: :) | 19:56 |
| scottda | e0ne: Yeah, I agree. We'll have to think about it and come up with an idea for O release. | 19:56 |
| e0ne | scottda: ok, why I should not bump microversion number in my patch? | 19:57 |
| scottda | e0ne: +2'd by the way. Maybe jungleboyj or smcginnis can have another look at https://review.openstack.org/#/c/300443/ | 19:57 |
| scottda | e0ne: IT's ok to leave it in there, but it is actually dead code... | 19:57 |
| e0ne | scottda: thanks for the help, Scott! | 19:57 |
| scottda | e0ne: I put the plumbing in for automatic version discovery in the client... | 19:57 |
| smcginnis | scottda: Saw that, will take a look. | 19:57 |
| scottda | e0ne: Client will query the server, and determine the highest microversion supported by both. It would use this cinderclient MAX_VERSION for that | 19:58 |
| e0ne | scottda: please, don't tell me about dead code | 19:58 |
| scottda | e0ne: This: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/api_versions.py#L248 | 19:59 |
| scottda | e0ne: I need to just add the plumbing so client calls that and sets the microversion. It would probably not be on by default right away, but would need an option set. | 20:00 |
| scottda | e0ne: Manila has this and it is automatic. | 20:00 |
| scottda | e0ne: But Please don't remove my dead code? | 20:00 |
| scottda | s/?/! | 20:00 |
| scottda | e0ne: Thanks for working on this. Go get some sleep. | 20:01 |
| e0ne | scottda: sure, I'll let you to fix it | 20:01 |
| *** alonma has joined #openstack-cinder | 20:01 | |
| e0ne | scottda: you know this area better | 20:01 |
| e0ne | scottda: speaking about dead code: https://bugs.launchpad.net/cinder/+bug/1614236 | 20:05 |
| openstack | Launchpad bug 1614236 in Cinder "c-api should not log ERROR tracebacks for 400/409 cases" [Medium,Confirmed] - Assigned to Ivan Kolodyazhny (e0ne) | 20:05 |
| *** alonma has quit IRC | 20:05 | |
| *** ducttape_ has quit IRC | 20:07 | |
| *** ducttape_ has joined #openstack-cinder | 20:08 | |
| scottda | e0ne: Yeah, that'd be a good one to fix. | 20:09 |
| *** diablo_rojo has quit IRC | 20:11 | |
| *** jungleboyj has quit IRC | 20:11 | |
| e0ne | xyang: thanks for the review! | 20:11 |
| xyang | e0ne: welcome | 20:11 |
| *** coolsvap has quit IRC | 20:12 | |
| smcginnis | xyang: Beat me to it. :) | 20:13 |
| *** akerr has quit IRC | 20:13 | |
| xyang | @smcginnis: :) | 20:13 |
| *** Suyi_ has joined #openstack-cinder | 20:15 | |
| *** ntt has joined #openstack-cinder | 20:21 | |
| *** ccesario has quit IRC | 20:23 | |
| scottda | smcginnis: I think we've got all the support for new features/apis into the cinderclient now. Just a few wayward bug fixes out there.. | 20:23 |
| scottda | anyway, I've got to head to an on-site meeting. TTYL | 20:23 |
| *** baumann has joined #openstack-cinder | 20:23 | |
| smcginnis | scottda: Thanks! | 20:24 |
| smcginnis | Wow, only 49 patches in the check queue. Haven't seen the number that low in a while. | 20:24 |
| *** baumann has quit IRC | 20:26 | |
| hemna | everyone on vacation already ? | 20:26 |
| smcginnis | Wrapping up N-3 and heading out the door I guess. :) | 20:26 |
| hemna | https://review.openstack.org/#/c/363957/ | 20:26 |
| hemna | are we trying to get that in for N ? | 20:26 |
| smcginnis | Yeah. I'm OK if that goes in after N-3 though. | 20:27 |
| *** porrua has quit IRC | 20:27 | |
| smcginnis | Just haven't had a chance to try it out yet. | 20:27 |
| *** _ducttape_ has joined #openstack-cinder | 20:28 | |
| hemna | ok sounds good | 20:28 |
| *** xyang has quit IRC | 20:28 | |
| hemna | fwiw, our FC CI doesn't use the fczm | 20:28 |
| *** baumann has joined #openstack-cinder | 20:28 | |
| *** baumann has quit IRC | 20:28 | |
| smcginnis | Yeah, neither does mine. | 20:28 |
| *** baumann has joined #openstack-cinder | 20:29 | |
| smcginnis | patrickeast is the patron saint of fczm I think. | 20:29 |
| hemna | I think so yah | 20:29 |
| * patrickeast hides | 20:29 | |
| smcginnis | hehe | 20:29 |
| hemna | :P | 20:29 |
| patrickeast | whats up? | 20:29 |
| hemna | our CI isn't being run by my team anymore | 20:29 |
| hemna | so I have no clue if they even have an FC switch | 20:29 |
| patrickeast | ahh yea | 20:29 |
| smcginnis | patrickeast: Were just talking fczm testing. | 20:29 |
| patrickeast | haha | 20:29 |
| Swanson | Kinda need that for testing FC, I should think. | 20:29 |
| hemna | yup | 20:29 |
| smcginnis | patrickeast: Sorry, I forgot you always get pinged. Hope you weren't in the middle of something important. | 20:29 |
| *** e0ne has quit IRC | 20:30 | |
| hemna | nmp now | 20:30 |
| patrickeast | the last week or so mine hasn't been using the fczm either while i track down why the multi-node jobs are sucking so much on it | 20:30 |
| patrickeast | live migration on fc might be effed | 20:30 |
| hemna | bah | 20:30 |
| patrickeast | smcginnis: also, no worries | 20:31 |
| smcginnis | :) | 20:31 |
| patrickeast | being pinged just makes me feel important :D | 20:31 |
| smcginnis | Hah! :) | 20:31 |
| smcginnis | In that case... | 20:31 |
| * smcginnis sets reminder for midnight | 20:31 | |
| *** ducttape_ has quit IRC | 20:31 | |
| patrickeast | lol | 20:32 |
| hemna | :) | 20:32 |
| patrickeast | the bad part is if i sleep with my watch on and get pinged it will notify me and wake me up | 20:32 |
| patrickeast | need to figure out how to turn that off at night | 20:32 |
| smcginnis | I can't sleep with a watch on. That would drive me crazy. | 20:32 |
| smcginnis | ACtually, I don't even wear a watch anymore because it bothers me. | 20:32 |
| patrickeast | yea i got used to it with a little fitbit kinda thing | 20:33 |
| patrickeast | the new pebble i got is pretty small so it stays out of the way for the most part | 20:33 |
| hemna | tape it to your forehead ? | 20:33 |
| smcginnis | :D | 20:33 |
| patrickeast | haha | 20:33 |
| smcginnis | I just don't need a device telling me if I slept ok or not. I can usually tell myself. ;) | 20:33 |
| hemna | I tried it for a while | 20:34 |
| hemna | yup....didn't sleep last night. | 20:34 |
| patrickeast | haha | 20:34 |
| patrickeast | yea | 20:34 |
| *** gouthamr has quit IRC | 20:34 | |
| patrickeast | it is nice because i wont sleep through phone calls, and the like smart alarm clock is pretty neat | 20:34 |
| *** Yogi1 has quit IRC | 20:34 | |
| patrickeast | but i too am skeptical of the sleep tracking | 20:34 |
| *** _ducttape_ has quit IRC | 20:34 | |
| *** diablo_rojo_phon has quit IRC | 20:35 | |
| hemna | I dunno, it's kinda like Schrodinger's cat. | 20:35 |
| *** ducttape_ has joined #openstack-cinder | 20:35 | |
| hemna | am I sleeping like crap because I'm wearing this thing? | 20:35 |
| patrickeast | yea exactly | 20:35 |
| *** xyang has joined #openstack-cinder | 20:38 | |
| *** _ducttape_ has joined #openstack-cinder | 20:38 | |
| *** ducttape_ has quit IRC | 20:42 | |
| *** _ducttape_ has quit IRC | 20:45 | |
| *** ducttape_ has joined #openstack-cinder | 20:45 | |
| patrickeast | hemna: did you check that fczm change with a real switch? my guess is we're going to be merging it without any ci coverage | 20:47 |
| * hemna looks away.... | 20:48 | |
| patrickeast | unless the brocade one pops in | 20:48 |
| hemna | la la laaaa | 20:48 |
| patrickeast | haha | 20:48 |
| hemna | it's just code. | 20:48 |
| hemna | what can possibly go wrong ? | 20:48 |
| patrickeast | passes unit tests amiright? | 20:48 |
| hemna | yup! | 20:48 |
| hemna | that was the hardest part | 20:48 |
| hemna | setting a value on an attribute in an instance of a mock object is completely not obvious | 20:49 |
| patrickeast | i'll spin up one with a cisco switch real quick just to make sure nothing blows up | 20:49 |
| patrickeast | it would be ironic if our change to mark them as potentially broken breaks them | 20:49 |
| hemna | mock_obj.return_value.something = 'value' | 20:50 |
| hemna | seems backwards | 20:50 |
| hemna | but whatevs | 20:50 |
| openstackgerrit | Merged openstack/cinder: Fix backup unit test with new os-brick https://review.openstack.org/364475 | 20:50 |
| hemna | heh yah | 20:50 |
| patrickeast | haha yea the mocking stuff is a bit weird | 20:50 |
| patrickeast | make sense to someone i suppose | 20:50 |
| hemna | so, that patch just enables the ability to mark them as unsupported | 20:50 |
| hemna | do we want a follow up to mark cisco as unsupported? | 20:50 |
| patrickeast | eh not yet, we've got a whole process right? | 20:51 |
| hemna | I wasn't sure if we were already there with cisco or not | 20:51 |
| * patrickeast shrugs | 20:51 | |
| hemna | mark everything as unsupported..... | 20:51 |
| hemna | see who complains. | 20:51 |
| patrickeast | thats the safest way | 20:52 |
| hemna | absolutely | 20:52 |
| patrickeast | take the google approach, mark all of cinder as beta | 20:52 |
| hemna | perfect | 20:52 |
| alkhodos | sorry to interrupt, but a quick question :) should I put a DocImpact when I add a release note? | 20:52 |
| hemna | we wanted to do experimental api anyway | 20:53 |
| patrickeast | alkhodos: nah, only if there is more documentation needed elsewhere | 20:53 |
| *** chrichip has quit IRC | 20:53 | |
| alkhodos | patrickeast: k, thanks! | 20:54 |
| *** jgregor has quit IRC | 20:54 | |
| *** chrichip has joined #openstack-cinder | 20:55 | |
| hemna | https://imgflip.com/i/19w2gb | 20:55 |
| *** jgregor has joined #openstack-cinder | 20:55 | |
| hemna | core reviewers be like ^^ | 20:55 |
| patrickeast | haha | 20:55 |
| patrickeast | any idea what the lifecycle for the privsep daemon is? just noticed that the first call to attach in these tests gets stuck for a couple seconds while it spawns | 20:57 |
| patrickeast | hopefully thats only like once... ever? | 20:57 |
| hemna | it's supposed to stay up forever afaik | 20:57 |
| patrickeast | sweet | 20:58 |
| hemna | I think it recognizes when it bails and respawns | 20:58 |
| patrickeast | neato | 20:58 |
| patrickeast | guess it was thought through for that part pretty well | 20:58 |
| hemna | I hope so | 20:58 |
| hemna | it's been less than ideal so far | 20:58 |
| hemna | and the stderr stuff is so irritating to me still. | 20:59 |
| patrickeast | yea, guess its still pretty new | 20:59 |
| patrickeast | im hoping another release or so and it'll be great | 20:59 |
| tonyb | hemna, sorry I didn't notice that FakeConnector wasn't covered in my aliasing patch :( | 21:00 |
| hemna | nah it's ok | 21:02 |
| hemna | everyone missed it | 21:02 |
| *** jgregor has quit IRC | 21:03 | |
| *** baumann has quit IRC | 21:04 | |
| *** eharney has quit IRC | 21:10 | |
| *** alonma has joined #openstack-cinder | 21:12 | |
| *** kaisers_ has joined #openstack-cinder | 21:14 | |
| *** laughterwym has joined #openstack-cinder | 21:14 | |
| *** pauloewerton has quit IRC | 21:16 | |
| *** alonma has quit IRC | 21:17 | |
| *** xyang has quit IRC | 21:18 | |
| *** kaisers_ has quit IRC | 21:18 | |
| *** laughterwym has quit IRC | 21:18 | |
| *** xyang has joined #openstack-cinder | 21:20 | |
| *** rcernin has quit IRC | 21:24 | |
| *** xyang has quit IRC | 21:25 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor5: add extend method to NFS driver https://review.openstack.org/297880 | 21:25 |
| *** lpetrut has quit IRC | 21:29 | |
| *** xyang has joined #openstack-cinder | 21:29 | |
| *** dustins has quit IRC | 21:30 | |
| *** alkhodos__ has quit IRC | 21:32 | |
| *** alkhodos has quit IRC | 21:32 | |
| *** gouthamr has joined #openstack-cinder | 21:32 | |
| *** wN has quit IRC | 21:35 | |
| *** xyang has quit IRC | 21:36 | |
| *** alkhodos has joined #openstack-cinder | 21:39 | |
| *** ducttape_ has quit IRC | 21:44 | |
| *** gouthamr_ has joined #openstack-cinder | 21:44 | |
| *** ducttape_ has joined #openstack-cinder | 21:45 | |
| *** gouthamr has quit IRC | 21:47 | |
| *** xyang has joined #openstack-cinder | 21:48 | |
| *** lprice has quit IRC | 21:48 | |
| *** ntt has quit IRC | 21:52 | |
| mriedem | smcginnis: https://review.openstack.org/#/c/364454/ is merged | 21:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/364209 | 21:52 |
| smcginnis | mriedem: Thanks! | 21:52 |
| hemna | sweet | 21:53 |
| *** xyang has quit IRC | 21:59 | |
| *** haplo37__ has quit IRC | 21:59 | |
| smcginnis | hemna: http://logs.openstack.org/55/364255/2/check/gate-cross-cinder-python27-db-ubuntu-xenial/c773bf2/console.html#_2016-09-01_22_03_34_861603 | 22:05 |
| smcginnis | hemna: I think that's what I saw yesterday with a quick test. | 22:05 |
| smcginnis | hemna: Not sure what's going on there. | 22:05 |
| openstackgerrit | Merged openstack/python-cinderclient: Deleting volume metadata keys with a single request https://review.openstack.org/300443 | 22:05 |
| hemna | that would happen with an older os-brick | 22:06 |
| hemna | prior to the refactor | 22:06 |
| *** Julien-zte has quit IRC | 22:07 | |
| smcginnis | hemna: Using 1.6.1: http://logs.openstack.org/55/364255/2/check/gate-cross-cinder-python27-db-ubuntu-xenial/c773bf2/console.html#_2016-09-01_22_03_39_012926 | 22:07 |
| *** cknight has quit IRC | 22:08 | |
| *** Julien-zte has joined #openstack-cinder | 22:09 | |
| hemna | err....wtf | 22:09 |
| smcginnis | My thoughts exactly. | 22:09 |
| *** xinli has quit IRC | 22:11 | |
| smcginnis | At least Nova passes on the patch now. | 22:14 |
| *** xyang1 has quit IRC | 22:14 | |
| smcginnis | hemna: Maybe just put FakeConnector back where it was? | 22:14 |
| hemna | sec | 22:14 |
| *** krotscheck has quit IRC | 22:15 | |
| *** krotscheck has joined #openstack-cinder | 22:15 | |
| * hemna is confused | 22:16 | |
| *** erhudy has quit IRC | 22:22 | |
| alkhodos | can some of core reviewers please review this patch? https://review.openstack.org/#/c/356142/ Has +2 from xing-yang and CI reported | 22:23 |
| *** alonma has joined #openstack-cinder | 22:23 | |
| smcginnis | I'm waiting for hemna to fix all my problems, so sure. :) | 22:23 |
| hemna | witf | 22:23 |
| hemna | I'm getting odd things too | 22:23 |
| smcginnis | hemna: Just move the FakeConnector back to where it was and say f it? | 22:24 |
| hemna | *sigh* | 22:24 |
| hemna | that requires a new drop of brick | 22:24 |
| hemna | I have a different fix for that | 22:24 |
| hemna | but this one doesn't make sense to me | 22:24 |
| smcginnis | hemna: Oh, crap. OK | 22:24 |
| smcginnis | Didn't think that one through all the way. | 22:24 |
| hemna | smcginnis, http://paste.openstack.org/show/565900/ | 22:25 |
| hemna | ok I think I have it | 22:26 |
| smcginnis | hemna: Gotta import the whole path? | 22:26 |
| Swanson | Sean, this one going through CI anytime soon? https://review.openstack.org/#/c/361453/ | 22:26 |
| *** ducttape_ has quit IRC | 22:27 | |
| Swanson | smcginnis, ^^ | 22:27 |
| hemna | smcginnis, http://paste.openstack.org/show/565901/ | 22:27 |
| smcginnis | Swanson: I don't see it in the running jobs. Must be in the queue yet. | 22:27 |
| Swanson | aight | 22:27 |
| smcginnis | hemna: That bottom one works? | 22:28 |
| *** alonma has quit IRC | 22:28 | |
| hemna | yah | 22:28 |
| smcginnis | Cool, looked like it should. | 22:28 |
| hemna | I'm downgrading | 22:28 |
| hemna | going to check to make sure it works for older and newer | 22:28 |
| hemna | give me a few | 22:28 |
| smcginnis | hemna: Good call. We should get this right. | 22:28 |
| * smcginnis starts stopwatch | 22:28 | |
| smcginnis | Swanson: That ran a little after noon but all three platforms failed to stack. | 22:30 |
| hemna | http://paste.openstack.org/show/565903/ | 22:30 |
| hemna | ok that looks happy | 22:30 |
| smcginnis | hemna: Sweet! | 22:30 |
| *** ntpttr has quit IRC | 22:31 | |
| hemna | pushing up another patch in a bit | 22:31 |
| *** ntpttr has joined #openstack-cinder | 22:31 | |
| smcginnis | Any other cores around to help push that through? Otherwise in the interest of time I'll just ninja it. | 22:31 |
| smcginnis | Swanson: Queued up again. 8th in queue. | 22:32 |
| Swanson | Thanks! | 22:33 |
| smcginnis | Swanson: Bug fix, so it can still go in after today. | 22:34 |
| Swanson | Excellent. | 22:34 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Fix backup unit tests imports for os-brick https://review.openstack.org/364576 | 22:38 |
| hemna | once more with feeling | 22:38 |
| *** rooneym has quit IRC | 22:39 | |
| smcginnis | ;) | 22:39 |
| hemna | Swanson, do we have any Dell CI reporting on your patch? | 22:39 |
| hemna | https://review.openstack.org/#/c/361453/ | 22:39 |
| hemna | that guy | 22:39 |
| smcginnis | hemna: There was a problem stacking so I kicked it off again. | 22:40 |
| hemna | ok | 22:40 |
| smcginnis | hemna: I might need to rebuild the base image I'm using in the CI. | 22:40 |
| hemna | ugh dib builder not building ? | 22:40 |
| smcginnis | It always gets a little wonky around this time as all the global requirements get updated. | 22:40 |
| Swanson | hemna, what smcginnis said. | 22:41 |
| smcginnis | hemna: Still running my version of sos-ci | 22:41 |
| hemna | oh | 22:41 |
| hemna | heh | 22:41 |
| hemna | living on the edge | 22:41 |
| smcginnis | Works great except when it doesn't. ;_ | 22:41 |
| smcginnis | ;) | 22:41 |
| hemna | yah that's the same with comming ci | 22:41 |
| hemna | it's awesome, until it's not. | 22:41 |
| hemna | common even | 22:41 |
| smcginnis | I've had at least a few days of 90+% success, so that felt nice. :D | 22:42 |
| smcginnis | bbl | 22:43 |
| *** alonma has joined #openstack-cinder | 22:43 | |
| patrickeast | hemna: https://review.openstack.org/#/c/363957 look okay with cisco fczm | 22:43 |
| patrickeast | hemna: if you update that typo we can push it along | 22:44 |
| hemna | crap, what'd I do | 22:44 |
| patrickeast | or we just go with all | 22:44 |
| hemna | ah crap ok | 22:44 |
| hemna | I'm lame | 22:44 |
| hemna | so lame | 22:44 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add supported driver checks to Zone Manager https://review.openstack.org/363957 | 22:45 |
| hemna | patrickeast, thanks | 22:45 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 22:46 |
| hemna | can the 3PAR CI just work please...thanks. | 22:46 |
| *** ducttape_ has joined #openstack-cinder | 22:48 | |
| *** alonma has quit IRC | 22:48 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add dell_server_os configuration option https://review.openstack.org/358878 | 22:50 |
| *** gouthamr_ has quit IRC | 22:53 | |
| *** akerr has joined #openstack-cinder | 23:02 | |
| *** kaisers_ has joined #openstack-cinder | 23:02 | |
| *** alonma has joined #openstack-cinder | 23:04 | |
| *** akerr_ has joined #openstack-cinder | 23:04 | |
| *** kaisers_ has quit IRC | 23:07 | |
| hemna | patrickeast, thanks for the testing man. | 23:07 |
| hemna | ok off to soccer... | 23:07 |
| patrickeast | hemna: np | 23:07 |
| *** hemna is now known as hemnafk | 23:07 | |
| patrickeast | cya | 23:07 |
| *** akerr has quit IRC | 23:07 | |
| *** alonma has quit IRC | 23:08 | |
| *** gouthamr has joined #openstack-cinder | 23:10 | |
| *** salv-orlando has quit IRC | 23:10 | |
| *** markvoelker has joined #openstack-cinder | 23:10 | |
| *** alonma has joined #openstack-cinder | 23:14 | |
| *** gouthamr_ has joined #openstack-cinder | 23:14 | |
| *** gouthamr has quit IRC | 23:14 | |
| *** jungleboyj has joined #openstack-cinder | 23:15 | |
| *** alonma has quit IRC | 23:18 | |
| *** tpsilva has quit IRC | 23:24 | |
| *** chlong has quit IRC | 23:33 | |
| *** alonma has joined #openstack-cinder | 23:34 | |
| *** alonma has quit IRC | 23:38 | |
| *** Guest99973 has quit IRC | 23:43 | |
| *** akerr_ has quit IRC | 23:46 | |
| *** jungleboyj has quit IRC | 23:47 | |
| *** Julien-zte has quit IRC | 23:50 | |
| *** markvoelker has quit IRC | 23:51 | |
| *** alonma has joined #openstack-cinder | 23:54 | |
| *** markvoelker has joined #openstack-cinder | 23:56 | |
| *** xyang has joined #openstack-cinder | 23:56 | |
| *** Suyi_ has quit IRC | 23:57 | |
| *** xyang has quit IRC | 23:57 | |
| *** diogogmt has quit IRC | 23:58 | |
| *** alonma has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!