| *** yangyapeng has quit IRC | 00:00 | |
| *** alonma has quit IRC | 00:00 | |
| *** karthik__ has quit IRC | 00:00 | |
| *** jwcroppe has quit IRC | 00:01 | |
| *** jamielennox|away is now known as jamielennox | 00:01 | |
| *** ducttape_ has joined #openstack-cinder | 00:02 | |
| *** jwcroppe has joined #openstack-cinder | 00:04 | |
| *** ducttape_ has quit IRC | 00:05 | |
| *** mriedem has joined #openstack-cinder | 00:06 | |
| *** akerr has joined #openstack-cinder | 00:06 | |
| *** zul has quit IRC | 00:07 | |
| *** alonma has joined #openstack-cinder | 00:07 | |
| *** akerr_ has joined #openstack-cinder | 00:08 | |
| *** zul has joined #openstack-cinder | 00:09 | |
| *** sdake has quit IRC | 00:10 | |
| *** ducttape_ has joined #openstack-cinder | 00:11 | |
| *** akerr has quit IRC | 00:12 | |
| *** alonma has quit IRC | 00:12 | |
| *** mtanino has quit IRC | 00:14 | |
| *** alonma has joined #openstack-cinder | 00:15 | |
| *** lprice1 has joined #openstack-cinder | 00:15 | |
| *** lprice has quit IRC | 00:17 | |
| *** daneyon has joined #openstack-cinder | 00:18 | |
| *** alonma has quit IRC | 00:20 | |
| *** alonma has joined #openstack-cinder | 00:22 | |
| *** alonma has quit IRC | 00:27 | |
| *** laughterwym has joined #openstack-cinder | 00:30 | |
| *** Suyi_ has quit IRC | 00:31 | |
| *** laughterwym has quit IRC | 00:32 | |
| *** Guy has quit IRC | 00:32 | |
| *** laughterwym has joined #openstack-cinder | 00:32 | |
| *** haplo37__ has quit IRC | 00:40 | |
| *** ducttape_ has quit IRC | 00:44 | |
| *** thurloat has quit IRC | 00:48 | |
| *** Guy has joined #openstack-cinder | 00:50 | |
| *** thurloat has joined #openstack-cinder | 00:50 | |
| *** alonma has joined #openstack-cinder | 00:56 | |
| *** alonma has quit IRC | 01:01 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 01:03 |
|---|---|---|
| *** alonma has joined #openstack-cinder | 01:04 | |
| *** alonma has quit IRC | 01:08 | |
| *** alonma has joined #openstack-cinder | 01:11 | |
| *** yangyapeng has joined #openstack-cinder | 01:13 | |
| *** alonma has quit IRC | 01:15 | |
| *** Apoorva_ has joined #openstack-cinder | 01:16 | |
| *** alonma has joined #openstack-cinder | 01:18 | |
| *** Apoorva has quit IRC | 01:18 | |
| *** gouthamr has joined #openstack-cinder | 01:19 | |
| *** QZhang has quit IRC | 01:19 | |
| *** cknight has joined #openstack-cinder | 01:19 | |
| *** Apoorva_ has quit IRC | 01:20 | |
| *** chris_morrell has quit IRC | 01:22 | |
| *** alonma has quit IRC | 01:23 | |
| *** iceyao has joined #openstack-cinder | 01:28 | |
| *** zhonghua-lee has joined #openstack-cinder | 01:33 | |
| *** jamielennox is now known as jamielennox|away | 01:34 | |
| *** sofie has joined #openstack-cinder | 01:35 | |
| *** sofie has quit IRC | 01:38 | |
| *** alonma has joined #openstack-cinder | 01:39 | |
| *** gouthamr has quit IRC | 01:41 | |
| *** gouthamr has joined #openstack-cinder | 01:42 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update detach_volume() with versionedobjects https://review.openstack.org/268609 | 01:42 |
| openstackgerrit | LisaLi proposed openstack/cinder: Update attach_volume() with versionedobjects https://review.openstack.org/268608 | 01:42 |
| *** alonma has quit IRC | 01:44 | |
| *** Lee1092 has joined #openstack-cinder | 01:46 | |
| *** alonma has joined #openstack-cinder | 01:47 | |
| *** alonma has quit IRC | 01:52 | |
| *** iceyao has quit IRC | 01:52 | |
| *** alonma has joined #openstack-cinder | 01:54 | |
| *** iceyao has joined #openstack-cinder | 01:54 | |
| *** EinstCrazy has joined #openstack-cinder | 01:55 | |
| openstackgerrit | Matt Smith (_alastor_) proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/285381 | 01:55 |
| *** iceyao has quit IRC | 01:56 | |
| *** iceyao has joined #openstack-cinder | 01:56 | |
| openstackgerrit | Matt Smith (_alastor_) proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/285381 | 01:57 |
| *** dmellado has quit IRC | 01:57 | |
| *** iceyao has quit IRC | 01:57 | |
| *** iceyao has joined #openstack-cinder | 01:58 | |
| *** alonma has quit IRC | 01:58 | |
| *** alonma has joined #openstack-cinder | 02:01 | |
| *** dmellado has joined #openstack-cinder | 02:02 | |
| *** yangyapeng has quit IRC | 02:03 | |
| *** yangyape_ has joined #openstack-cinder | 02:03 | |
| *** karthikp_ has quit IRC | 02:04 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 02:04 |
| openstackgerrit | LisaLi proposed openstack/cinder: Replace key manager with Castellan https://review.openstack.org/280492 | 02:04 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add encryptor attach/detach in utils https://review.openstack.org/341914 | 02:04 |
| openstackgerrit | LisaLi proposed openstack/cinder: Specify key algorithm and size for create_key https://review.openstack.org/251503 | 02:04 |
| *** alonma has quit IRC | 02:05 | |
| *** alonma has joined #openstack-cinder | 02:08 | |
| *** alonma has quit IRC | 02:13 | |
| *** ducttape_ has joined #openstack-cinder | 02:15 | |
| *** Guy has quit IRC | 02:16 | |
| *** huyang has joined #openstack-cinder | 02:18 | |
| *** dave-mccowan has quit IRC | 02:19 | |
| *** alonma has joined #openstack-cinder | 02:20 | |
| *** ducttape_ has quit IRC | 02:20 | |
| *** mriedem has quit IRC | 02:20 | |
| *** alonma has quit IRC | 02:24 | |
| *** markvoelker has joined #openstack-cinder | 02:26 | |
| *** DiaoSi has quit IRC | 02:26 | |
| *** Lallana has joined #openstack-cinder | 02:26 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: NetApp cDOT: Add cheesecake replication support https://review.openstack.org/346094 | 02:28 |
| *** dkehn_ has quit IRC | 02:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/350876 | 02:34 |
| *** jamielennox|away is now known as jamielennox | 02:36 | |
| *** Guy has joined #openstack-cinder | 02:40 | |
| *** dkehn_ has joined #openstack-cinder | 02:44 | |
| *** alonma has joined #openstack-cinder | 02:47 | |
| *** gouthamr has quit IRC | 02:51 | |
| *** ebalduf has joined #openstack-cinder | 02:51 | |
| *** alonma has quit IRC | 02:52 | |
| *** itlinux has quit IRC | 02:54 | |
| *** alonma has joined #openstack-cinder | 02:54 | |
| openstackgerrit | huanan proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 02:55 |
| *** itlinux has joined #openstack-cinder | 02:56 | |
| *** yangyape_ has quit IRC | 02:57 | |
| *** yangyapeng has joined #openstack-cinder | 02:58 | |
| *** alonma has quit IRC | 02:59 | |
| *** bardia has joined #openstack-cinder | 03:01 | |
| *** alonma has joined #openstack-cinder | 03:01 | |
| *** sdake has joined #openstack-cinder | 03:04 | |
| openstackgerrit | huanan proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 03:06 |
| *** alonma has quit IRC | 03:06 | |
| *** alonma has joined #openstack-cinder | 03:09 | |
| *** alonma has quit IRC | 03:14 | |
| *** alonma has joined #openstack-cinder | 03:16 | |
| *** ducttape_ has joined #openstack-cinder | 03:17 | |
| *** alonma has quit IRC | 03:20 | |
| *** coolsvap has joined #openstack-cinder | 03:22 | |
| *** ducttape_ has quit IRC | 03:22 | |
| *** sdake has quit IRC | 03:23 | |
| *** nikeshm has joined #openstack-cinder | 03:34 | |
| nikeshm | addressed all reviews on retype https://review.openstack.org/#/c/349019 CI and jenkins are happy | 03:36 |
| *** jwcroppe has quit IRC | 03:39 | |
| *** markvoelker has quit IRC | 03:46 | |
| *** yangyapeng has quit IRC | 03:47 | |
| *** yangyapeng has joined #openstack-cinder | 03:48 | |
| *** sdake has joined #openstack-cinder | 03:50 | |
| *** yangyape_ has joined #openstack-cinder | 03:53 | |
| *** yangyapeng has quit IRC | 03:53 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 03:58 |
| *** asselin_ has joined #openstack-cinder | 04:00 | |
| *** daneyon has quit IRC | 04:01 | |
| *** asselin has quit IRC | 04:02 | |
| *** cknight has quit IRC | 04:07 | |
| *** alonma has joined #openstack-cinder | 04:10 | |
| openstackgerrit | Merged openstack/cinder: Add stochastic weight handler to scheduler https://review.openstack.org/347010 | 04:12 |
| *** alonma has quit IRC | 04:14 | |
| *** julim has quit IRC | 04:16 | |
| *** ducttape_ has joined #openstack-cinder | 04:19 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 04:19 |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 04:19 |
| *** markvoelker has joined #openstack-cinder | 04:22 | |
| *** ducttape_ has quit IRC | 04:23 | |
| *** nkrinner_afk is now known as nkrinner | 04:24 | |
| *** markvoelker has quit IRC | 04:26 | |
| openstackgerrit | Merged openstack/cinder: Return vaild reason when unknown error occur. https://review.openstack.org/339313 | 04:28 |
| *** links has joined #openstack-cinder | 04:28 | |
| *** asselin has joined #openstack-cinder | 04:29 | |
| *** yangyape_ has quit IRC | 04:29 | |
| *** alonmarx has joined #openstack-cinder | 04:31 | |
| openstackgerrit | nikesh proposed openstack/cinder: Improve snapshot handling in K2 https://review.openstack.org/348984 | 04:31 |
| *** asselin_ has quit IRC | 04:31 | |
| *** lprice1 has quit IRC | 04:31 | |
| *** alonma has joined #openstack-cinder | 04:31 | |
| *** abhinavtechie has joined #openstack-cinder | 04:31 | |
| *** alonma has quit IRC | 04:35 | |
| *** sdake has quit IRC | 04:37 | |
| *** alonma has joined #openstack-cinder | 04:38 | |
| *** alonma has quit IRC | 04:42 | |
| *** alonma has joined #openstack-cinder | 04:45 | |
| *** alonma has quit IRC | 04:50 | |
| *** pdeore has joined #openstack-cinder | 04:58 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 04:59 |
| *** jraju has joined #openstack-cinder | 05:04 | |
| *** yangyapeng has joined #openstack-cinder | 05:05 | |
| *** links has quit IRC | 05:06 | |
| *** narayrak has joined #openstack-cinder | 05:06 | |
| *** pbandark has joined #openstack-cinder | 05:10 | |
| *** alonmarx has quit IRC | 05:12 | |
| *** alonma has joined #openstack-cinder | 05:12 | |
| *** rcernin has joined #openstack-cinder | 05:16 | |
| *** markvoelker has joined #openstack-cinder | 05:16 | |
| *** abhitechie has joined #openstack-cinder | 05:17 | |
| *** abhitechie has quit IRC | 05:17 | |
| *** alonma has quit IRC | 05:17 | |
| *** ducttape_ has joined #openstack-cinder | 05:19 | |
| *** alonma has joined #openstack-cinder | 05:20 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 05:20 |
| *** alonmarx has joined #openstack-cinder | 05:21 | |
| *** darosale has joined #openstack-cinder | 05:21 | |
| *** markvoelker has quit IRC | 05:22 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 05:22 |
| *** iceyao has quit IRC | 05:23 | |
| *** ducttape_ has quit IRC | 05:24 | |
| *** alonma has quit IRC | 05:25 | |
| *** yangyape_ has joined #openstack-cinder | 05:29 | |
| *** yangyapeng has quit IRC | 05:29 | |
| *** iceyao has joined #openstack-cinder | 05:29 | |
| *** alonma has joined #openstack-cinder | 05:30 | |
| *** zouyapeng has quit IRC | 05:33 | |
| *** haplo37__ has joined #openstack-cinder | 05:34 | |
| *** akerr_ is now known as akerr_away | 05:35 | |
| *** alonma has quit IRC | 05:35 | |
| *** Poornima has joined #openstack-cinder | 05:36 | |
| *** alonma has joined #openstack-cinder | 05:38 | |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup-update https://review.openstack.org/263184 | 05:39 |
| *** lkuchlan has joined #openstack-cinder | 05:40 | |
| *** haplo37__ has quit IRC | 05:42 | |
| *** alonma has quit IRC | 05:42 | |
| *** darosale has quit IRC | 05:44 | |
| *** alonma has joined #openstack-cinder | 05:45 | |
| *** lpetrut has joined #openstack-cinder | 05:46 | |
| *** alonmarx has quit IRC | 05:48 | |
| *** alonma has quit IRC | 05:49 | |
| *** alonma has joined #openstack-cinder | 05:52 | |
| *** alonmarx has joined #openstack-cinder | 05:52 | |
| *** maestropandy has joined #openstack-cinder | 05:52 | |
| openstackgerrit | naga venkata proposed openstack/cinder: Adds support for cinder backup with customized ceph cluster name https://review.openstack.org/350994 | 05:54 |
| *** alonma has quit IRC | 05:57 | |
| *** alonma has joined #openstack-cinder | 05:59 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 06:02 |
| *** alonma has quit IRC | 06:04 | |
| *** Srinu has joined #openstack-cinder | 06:04 | |
| *** pcaruana has joined #openstack-cinder | 06:10 | |
| openstackgerrit | huanan proposed openstack/cinder: Migrate volume between backends in an async way https://review.openstack.org/334923 | 06:14 |
| *** alonmarx has quit IRC | 06:14 | |
| *** alonma has joined #openstack-cinder | 06:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/351004 | 06:14 |
| openstackgerrit | naga venkata proposed openstack/os-brick: Adds support for cinder backup with customized ceph cluster name https://review.openstack.org/351005 | 06:17 |
| *** alonma has quit IRC | 06:18 | |
| *** ducttape_ has joined #openstack-cinder | 06:20 | |
| *** alonma has joined #openstack-cinder | 06:21 | |
| *** ducttape_ has quit IRC | 06:25 | |
| *** lkuchlan has quit IRC | 06:26 | |
| *** lkuchlan_ has joined #openstack-cinder | 06:26 | |
| *** alonma has quit IRC | 06:27 | |
| *** mfisch has quit IRC | 06:29 | |
| *** belmoreira has joined #openstack-cinder | 06:30 | |
| *** alonma has joined #openstack-cinder | 06:32 | |
| *** pgadiya has joined #openstack-cinder | 06:32 | |
| *** mfisch has joined #openstack-cinder | 06:34 | |
| *** mfisch has quit IRC | 06:34 | |
| *** mfisch has joined #openstack-cinder | 06:34 | |
| *** lkuchlan_ has quit IRC | 06:35 | |
| *** alonma has quit IRC | 06:36 | |
| openstackgerrit | Vijay Ladani proposed openstack/cinder: Handle snapshot delete conflict in 3par driver https://review.openstack.org/342722 | 06:36 |
| *** alonmarx has joined #openstack-cinder | 06:37 | |
| *** karthikp_ has joined #openstack-cinder | 06:38 | |
| *** alonma has joined #openstack-cinder | 06:39 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to argparse https://review.openstack.org/340129 | 06:41 |
| *** karthikp_ has quit IRC | 06:43 | |
| *** alonma has quit IRC | 06:43 | |
| *** bardia has quit IRC | 06:45 | |
| *** alonma has joined #openstack-cinder | 06:46 | |
| *** alonma has quit IRC | 06:50 | |
| *** bhagya has joined #openstack-cinder | 06:52 | |
| *** alonma has joined #openstack-cinder | 06:53 | |
| *** tesseract- has joined #openstack-cinder | 06:53 | |
| *** permalac has quit IRC | 06:54 | |
| *** Dinesh_Bhor has quit IRC | 06:55 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:55 | |
| *** arecknag has joined #openstack-cinder | 06:55 | |
| *** bhagyashris has quit IRC | 06:55 | |
| *** alonma has quit IRC | 06:57 | |
| *** lpetrut has quit IRC | 06:59 | |
| *** alonmarx has quit IRC | 07:02 | |
| *** alonmarx has joined #openstack-cinder | 07:05 | |
| *** ebalduf has quit IRC | 07:06 | |
| *** alonma has joined #openstack-cinder | 07:08 | |
| *** lpetrut has joined #openstack-cinder | 07:11 | |
| *** liverpooler has joined #openstack-cinder | 07:11 | |
| *** alonma has quit IRC | 07:12 | |
| *** Poornima has quit IRC | 07:12 | |
| *** alonma has joined #openstack-cinder | 07:15 | |
| *** alonma has quit IRC | 07:19 | |
| flip214 | DuncanT: the drbdmanage PPA package is missing the http://git.drbd.org/drbdmanage.git/commit/93efc2a92f9988810bc49fc769c1b02475245902 and http://git.drbd.org/drbdmanage.git/commit/f26452e7685597294a75d38f5ac7d2413c7ac2d6 fixes, therefore creation times out. | 07:20 |
| flip214 | We'll upgrade the PPA today, then 347274 should work too. | 07:20 |
| *** ducttape_ has joined #openstack-cinder | 07:21 | |
| *** alonma has joined #openstack-cinder | 07:22 | |
| *** Dinesh_Bhor has quit IRC | 07:22 | |
| *** bhagyashris has joined #openstack-cinder | 07:24 | |
| *** bhagya has quit IRC | 07:24 | |
| openstackgerrit | naga venkata proposed openstack/os-brick: Adds support for cinder backup with customized ceph cluster name https://review.openstack.org/351005 | 07:24 |
| *** ducttape_ has quit IRC | 07:25 | |
| openstackgerrit | naga venkata proposed openstack/cinder: Adds support for cinder backup with customized ceph cluster name https://review.openstack.org/350994 | 07:26 |
| *** alonma has quit IRC | 07:26 | |
| *** alonmarx has quit IRC | 07:30 | |
| *** lkuchlan_ has joined #openstack-cinder | 07:31 | |
| *** lpetrut has quit IRC | 07:31 | |
| *** alonma has joined #openstack-cinder | 07:32 | |
| *** openstackgerrit has quit IRC | 07:33 | |
| *** openstackgerrit has joined #openstack-cinder | 07:34 | |
| *** alonmarx has joined #openstack-cinder | 07:34 | |
| *** zouyapeng has joined #openstack-cinder | 07:36 | |
| *** alonma has quit IRC | 07:37 | |
| *** lpetrut has joined #openstack-cinder | 07:40 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 07:46 |
| *** alonma has joined #openstack-cinder | 07:47 | |
| *** alonma has quit IRC | 07:52 | |
| *** yangyapeng has joined #openstack-cinder | 07:53 | |
| *** mvk has quit IRC | 07:53 | |
| *** yangyape_ has quit IRC | 07:54 | |
| *** alonma has joined #openstack-cinder | 07:54 | |
| *** e0ne has joined #openstack-cinder | 07:55 | |
| *** e0ne has quit IRC | 07:55 | |
| *** alonmarx has quit IRC | 07:56 | |
| *** e0ne has joined #openstack-cinder | 07:56 | |
| *** alonma has quit IRC | 07:56 | |
| *** lpetrut has quit IRC | 07:56 | |
| *** alonma has joined #openstack-cinder | 07:56 | |
| *** markvoelker has joined #openstack-cinder | 07:58 | |
| *** akapil has joined #openstack-cinder | 07:59 | |
| *** alonmarx has joined #openstack-cinder | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** markvoelker has quit IRC | 08:03 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 08:04 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** yangyapeng has quit IRC | 08:13 | |
| *** yangyapeng has joined #openstack-cinder | 08:13 | |
| *** e0ne has quit IRC | 08:14 | |
| *** e0ne has joined #openstack-cinder | 08:15 | |
| *** alonmarx has quit IRC | 08:20 | |
| *** iceyao_ has joined #openstack-cinder | 08:22 | |
| *** iceyao has quit IRC | 08:22 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 08:29 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Retype encrypted volumes https://review.openstack.org/319078 | 08:30 |
| *** huanan_L has joined #openstack-cinder | 08:37 | |
| *** laughterwym has quit IRC | 08:38 | |
| *** huanan has quit IRC | 08:39 | |
| *** laughterwym has joined #openstack-cinder | 08:41 | |
| *** laughterwym has quit IRC | 08:46 | |
| *** markvoelker has joined #openstack-cinder | 08:53 | |
| *** gb21 has joined #openstack-cinder | 08:55 | |
| *** markvoelker has quit IRC | 08:57 | |
| *** e0ne has quit IRC | 09:03 | |
| *** e0ne has joined #openstack-cinder | 09:04 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-cinderclient: Tests for testing volume-create command https://review.openstack.org/343474 | 09:12 |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder: Removed a dead db function register_models() https://review.openstack.org/350393 | 09:15 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup-update https://review.openstack.org/263184 | 09:21 |
| *** ducttape_ has joined #openstack-cinder | 09:22 | |
| *** xek_ has joined #openstack-cinder | 09:23 | |
| *** xek has quit IRC | 09:24 | |
| *** dulek has quit IRC | 09:25 | |
| nikeshm | vbala: hi | 09:25 |
| *** lpetrut has joined #openstack-cinder | 09:26 | |
| *** dulek has joined #openstack-cinder | 09:27 | |
| *** ducttape_ has quit IRC | 09:27 | |
| *** gb21 has quit IRC | 09:29 | |
| *** alonma has quit IRC | 09:36 | |
| *** laughterwym has joined #openstack-cinder | 09:41 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Re-use RBDImageMetadata and RBDVolumeIOWrapper from os-brick https://review.openstack.org/326999 | 09:42 |
| *** alonma has joined #openstack-cinder | 09:44 | |
| *** mvk has joined #openstack-cinder | 09:45 | |
| *** markvoelker has joined #openstack-cinder | 09:47 | |
| *** alonma has quit IRC | 09:49 | |
| *** permalac has joined #openstack-cinder | 09:49 | |
| *** laughterwym has quit IRC | 09:50 | |
| *** alonma has joined #openstack-cinder | 09:51 | |
| *** alonma has quit IRC | 09:52 | |
| *** alonma has joined #openstack-cinder | 09:52 | |
| *** markvoelker has quit IRC | 09:52 | |
| *** akapil has quit IRC | 09:53 | |
| *** laughterwym has joined #openstack-cinder | 09:55 | |
| *** lkuchlan_ has quit IRC | 09:55 | |
| *** laughterwym has quit IRC | 10:03 | |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 10:04 |
| *** laughterwym has joined #openstack-cinder | 10:11 | |
| *** laughterwym has quit IRC | 10:15 | |
| *** alonma has quit IRC | 10:20 | |
| *** yangyapeng has quit IRC | 10:21 | |
| *** ducttape_ has joined #openstack-cinder | 10:23 | |
| *** yangyapeng has joined #openstack-cinder | 10:24 | |
| *** ducttape_ has quit IRC | 10:28 | |
| *** alonma has joined #openstack-cinder | 10:28 | |
| *** chlong has quit IRC | 10:28 | |
| *** gb21 has joined #openstack-cinder | 10:32 | |
| *** yangyapeng has quit IRC | 10:32 | |
| *** alonma has quit IRC | 10:32 | |
| *** alonma has joined #openstack-cinder | 10:35 | |
| *** alonma has quit IRC | 10:40 | |
| *** gb21 is now known as GB21 | 10:41 | |
| *** markvoelker has joined #openstack-cinder | 10:41 | |
| *** chlong has joined #openstack-cinder | 10:42 | |
| *** alonma has joined #openstack-cinder | 10:42 | |
| *** markvoelker has quit IRC | 10:46 | |
| *** EinstCrazy has quit IRC | 10:46 | |
| *** lkuchlan_ has joined #openstack-cinder | 10:46 | |
| *** alonma has quit IRC | 10:47 | |
| *** gcb has quit IRC | 10:49 | |
| *** alonma has joined #openstack-cinder | 10:49 | |
| *** kaisers has quit IRC | 10:50 | |
| *** kaisers has joined #openstack-cinder | 10:51 | |
| *** kaisers has quit IRC | 10:51 | |
| *** alonma has quit IRC | 10:54 | |
| *** akapil has joined #openstack-cinder | 10:54 | |
| *** alonma has joined #openstack-cinder | 10:57 | |
| *** kaisers has joined #openstack-cinder | 10:57 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 11:00 |
| *** alonma has quit IRC | 11:01 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent adding same OVO version twice to history https://review.openstack.org/351124 | 11:01 |
| *** gcb has joined #openstack-cinder | 11:01 | |
| *** GB21 has quit IRC | 11:01 | |
| *** GB21 has joined #openstack-cinder | 11:02 | |
| *** alonma has joined #openstack-cinder | 11:04 | |
| *** timcl has joined #openstack-cinder | 11:09 | |
| *** lkuchlan_ has quit IRC | 11:09 | |
| *** alonma has quit IRC | 11:10 | |
| *** alonma has joined #openstack-cinder | 11:11 | |
| *** sdague has joined #openstack-cinder | 11:21 | |
| *** ducttape_ has joined #openstack-cinder | 11:24 | |
| openstackgerrit | Merged openstack/cinder: Add metadata length check to volume-manage https://review.openstack.org/340727 | 11:24 |
| *** yangyapeng has joined #openstack-cinder | 11:26 | |
| *** chlong has quit IRC | 11:27 | |
| *** chlong has joined #openstack-cinder | 11:28 | |
| *** ducttape_ has quit IRC | 11:29 | |
| *** ericksonsantos has quit IRC | 11:34 | |
| *** markvoelker has joined #openstack-cinder | 11:35 | |
| *** markvoelker has quit IRC | 11:39 | |
| *** xek__ has joined #openstack-cinder | 11:41 | |
| *** xek_ has quit IRC | 11:42 | |
| *** dulek has quit IRC | 11:43 | |
| *** dulek has joined #openstack-cinder | 11:45 | |
| *** alonma has quit IRC | 11:48 | |
| *** Adriano___ has joined #openstack-cinder | 11:51 | |
| *** dave-mccowan has joined #openstack-cinder | 11:55 | |
| *** alonma has joined #openstack-cinder | 11:56 | |
| *** laughterwym has joined #openstack-cinder | 12:00 | |
| *** alonma has quit IRC | 12:00 | |
| *** gouthamr has joined #openstack-cinder | 12:01 | |
| *** eharney has quit IRC | 12:02 | |
| *** alonma has joined #openstack-cinder | 12:03 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Don't use config option sqlite_db https://review.openstack.org/351142 | 12:03 |
| *** markvoelker has joined #openstack-cinder | 12:05 | |
| *** laughterwym has quit IRC | 12:07 | |
| *** zul has quit IRC | 12:07 | |
| *** alonma has quit IRC | 12:08 | |
| *** akerr_away is now known as akerr_ | 12:08 | |
| *** laughterwym has joined #openstack-cinder | 12:09 | |
| *** alonma has joined #openstack-cinder | 12:10 | |
| *** ducttape_ has joined #openstack-cinder | 12:12 | |
| *** alonma has quit IRC | 12:12 | |
| *** alonma has joined #openstack-cinder | 12:13 | |
| *** tpsilva has joined #openstack-cinder | 12:14 | |
| *** diogogmt has joined #openstack-cinder | 12:15 | |
| *** laughterwym has quit IRC | 12:18 | |
| *** zul has joined #openstack-cinder | 12:21 | |
| *** laughterwym has joined #openstack-cinder | 12:22 | |
| *** diogogmt has quit IRC | 12:24 | |
| *** eharney has joined #openstack-cinder | 12:25 | |
| *** diogogmt has joined #openstack-cinder | 12:25 | |
| *** Srinu has quit IRC | 12:25 | |
| Dinesh_Bhor | smcginnis: Hi, Could you please take a look at it if possible? https://review.openstack.org/#/c/350393/ It has already got one +2. | 12:25 |
| *** xyang1 has joined #openstack-cinder | 12:27 | |
| *** GB21 has quit IRC | 12:30 | |
| *** ducttape_ has quit IRC | 12:31 | |
| Dinesh_Bhor | JayConroy: Thanks Jay | 12:32 |
| openstackgerrit | naga venkata proposed openstack/os-brick: Fixes backup with customized ceph cluster name https://review.openstack.org/351005 | 12:33 |
| *** laughterwym has quit IRC | 12:34 | |
| *** diogogmt has quit IRC | 12:35 | |
| *** zhipeng has joined #openstack-cinder | 12:35 | |
| *** pdeore has quit IRC | 12:36 | |
| *** laughterwym has joined #openstack-cinder | 12:38 | |
| *** pabardina has joined #openstack-cinder | 12:43 | |
| *** baojg_ has joined #openstack-cinder | 12:44 | |
| pabardina | Hello ! I would like to migrate a volume from one site to another (2 differents regions), is it possible to do with swift (as a backup driver). Someone has an idea ? I didn't find much informations... | 12:47 |
| scottda | e0ne: geguileo When you've a chance, will you look at Cao's cinderclient changes for microversion of subparser and help messages? https://review.openstack.org/#/c/340129/5 | 12:47 |
| scottda | I want some other eyes on it, to see if the general direction of this looks OK to you. | 12:47 |
| geguileo | scottda: Looks like a nice feature | 12:48 |
| e0ne | scottda: looking on it | 12:49 |
| geguileo | scottda: I'll review it tomorrow, as I've finished with reviews today and want to finally get back to the HA A/A patches | 12:49 |
| scottda | geguileo: Yes, we definitely need something like this. I don't have a problem with it as it is, I just want others to look and make sure the direction is sound, and that you don't have a different idea. | 12:49 |
| *** alonma_ has joined #openstack-cinder | 12:50 | |
| scottda | I think we want to release a cinderclient in the next couple weeks, so I figured we'd be wise to wrap up any in-flight patches as well... | 12:50 |
| geguileo | scottda: Yeah, that's what I understood when you asked :-) | 12:51 |
| *** pauloewerton has joined #openstack-cinder | 12:51 | |
| geguileo | scottda: Yes, let's try to make cinderclient as in sync with the services as possible | 12:51 |
| geguileo | scottda: If you are reviewing cinderclient patches you may want to have a look at the backup-update patch: https://review.openstack.org/#/c/263184/ | 12:53 |
| *** alonma has quit IRC | 12:53 | |
| geguileo | scottda: By the way, isn't the help supposed to hide commands that are not available if you use "os-volume-api-version"? | 12:54 |
| *** laughterwym has quit IRC | 12:56 | |
| *** gcb has quit IRC | 12:56 | |
| *** lkuchlan_ has joined #openstack-cinder | 12:57 | |
| e0ne | scottda: I don't understand use-case of --start-version and --end-version params | 12:57 |
| *** tangchen has quit IRC | 12:59 | |
| *** enriquetaso has joined #openstack-cinder | 12:59 | |
| e0ne | scottda: does it mean, that feature A in microverstion 3.10 will be supported in 3.15? | 12:59 |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 12:59 |
| geguileo | e0ne: The idea is that you may add a parameter to a command that only works for a specific API version | 13:00 |
| geguileo | e0ne: So you don't want it to show on the help if you are using a version that's prior to that onw | 13:00 |
| geguileo | For example you start allowing to update the backup name in 3.1 and backup description in 3.2 | 13:00 |
| geguileo | If you ask for help on 3.1 it should only show you the option to update name | 13:01 |
| e0ne | geguileo: I'm agree with it | 13:01 |
| e0ne | geguileo: but update the backup name feature should be supported only in 3.1, shouldn't it? | 13:02 |
| geguileo | e0ne: Well, in 3.1 onwards | 13:02 |
| geguileo | And update de description in 3.2 onwards | 13:02 |
| *** bhagya has joined #openstack-cinder | 13:02 | |
| *** abhinavtechie has quit IRC | 13:03 | |
| geguileo | But we may have arguments that are only valid until one specific version because there were removed then | 13:03 |
| geguileo | Hence the need to the stop | 13:03 |
| e0ne | geguileo: but now, we don't do like you said. e.g.: https://github.com/openstack/cinder/blob/master/cinder/api/v3/messages.py#L92 | 13:03 |
| geguileo | e0ne: Isn't that doing precisely that? | 13:04 |
| geguileo | e0ne: Only 3.5 onwards will acept pagination | 13:04 |
| *** faiz89 has joined #openstack-cinder | 13:05 | |
| geguileo | e0ne: So the list arguments for pagination in the client only make sense if we are requesting version 3.5 or higher | 13:05 |
| e0ne | geguileo: oh.. you're right | 13:06 |
| *** bhagyashris has quit IRC | 13:06 | |
| *** Dinesh_Bhor has quit IRC | 13:06 | |
| scottda | e0ne: Yes, I'm not sure we'll ever use the end_version, but it is there if we do decide to remove something. | 13:07 |
| *** zul has quit IRC | 13:07 | |
| *** laughterwym has joined #openstack-cinder | 13:07 | |
| e0ne | scottda: got it, it was my bad in understanding how it works:( | 13:08 |
| *** bhagyashris has joined #openstack-cinder | 13:08 | |
| *** rlrossit has joined #openstack-cinder | 13:11 | |
| *** sdake has joined #openstack-cinder | 13:11 | |
| *** avishay has joined #openstack-cinder | 13:12 | |
| *** bhagya has quit IRC | 13:12 | |
| *** nolwenn has joined #openstack-cinder | 13:12 | |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: Remove comma from end of policy.json https://review.openstack.org/351177 | 13:12 |
| *** markvoelker has quit IRC | 13:13 | |
| *** merooney has joined #openstack-cinder | 13:13 | |
| *** liverpooler has quit IRC | 13:14 | |
| akerr_ | Anyone else's drivers having a weird issue with the attach volume to shelved servers test ever since https://github.com/openstack/tempest/commit/3f5aa985b5de6759cf361dfecb5f042a2949aac5? It looks like its trying to issue a detach call using "iqn.fake" | 13:14 |
| nikeshm | me too | 13:17 |
| *** baojg_ has quit IRC | 13:17 | |
| nikeshm | akerr_: i am facing today | 13:18 |
| *** iceyao_ has quit IRC | 13:18 | |
| nikeshm | akerr_: http://paste.openstack.org/show/549222/ http://paste.openstack.org/show/549223/. | 13:18 |
| akerr_ | nikeshm: yea, we have something similar, our FCP drivers expect a key of 'wwpns' and one of the iscsi drivers fails because 'iqn.fake' doesn't exist | 13:19 |
| nikeshm | akerr_ : after changing bit https://review.openstack.org/#/c/349019/15..17 iscsi is passed waiting for fc | 13:20 |
| eharney | akerr_: i think this patch is working on the detach part of that issue: https://review.openstack.org/#/c/340951/ | 13:20 |
| eharney | akerr_: we were troubleshooting the same problem a few weeks ago | 13:21 |
| *** zul has joined #openstack-cinder | 13:22 | |
| akerr_ | eharney: thanks this at least explains where iqn.fake is coming from :) | 13:22 |
| *** laughterwym has quit IRC | 13:23 | |
| *** sdake has quit IRC | 13:24 | |
| *** dustins has joined #openstack-cinder | 13:24 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 13:24 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Oversubscription support https://review.openstack.org/307499 | 13:24 |
| *** porrua has joined #openstack-cinder | 13:27 | |
| *** akapil has quit IRC | 13:28 | |
| *** akapil has joined #openstack-cinder | 13:29 | |
| *** laughterwym has joined #openstack-cinder | 13:29 | |
| *** akapil has quit IRC | 13:29 | |
| *** akapil has joined #openstack-cinder | 13:29 | |
| *** ducttape_ has joined #openstack-cinder | 13:31 | |
| *** julim has joined #openstack-cinder | 13:31 | |
| *** akapil has quit IRC | 13:31 | |
| *** pabardina has left #openstack-cinder | 13:36 | |
| *** ducttape_ has quit IRC | 13:36 | |
| *** mriedem has joined #openstack-cinder | 13:39 | |
| *** Adriano___ is now known as adrianofr_ | 13:42 | |
| *** nkrinner is now known as nkrinner_afk | 13:43 | |
| *** ametts has joined #openstack-cinder | 13:45 | |
| *** laughterwym has quit IRC | 13:46 | |
| *** jraju has quit IRC | 13:46 | |
| *** zhonghua-lee has quit IRC | 13:48 | |
| *** zhonghua-lee has joined #openstack-cinder | 13:49 | |
| *** pgadiya has quit IRC | 13:51 | |
| *** laughterwym has joined #openstack-cinder | 13:51 | |
| *** cdelatte has joined #openstack-cinder | 13:51 | |
| *** markvoelker has joined #openstack-cinder | 13:51 | |
| *** sdake has joined #openstack-cinder | 13:54 | |
| *** iceyao has joined #openstack-cinder | 13:54 | |
| *** iceyao has quit IRC | 13:57 | |
| *** darosale has joined #openstack-cinder | 13:58 | |
| nikeshm | xyang1: CI is failing due to https://github.com/openstack/tempest/commit/3f5aa985b5de6759cf361dfecb5f042a2949aac5? | 13:58 |
| nikeshm | shall i comment that test for now | 13:59 |
| nikeshm | and let pass patch on retype patch and then can dig deeper on that | 13:59 |
| nikeshm | and let CI pass on retype | 14:00 |
| nikeshm | patch | 14:00 |
| *** arecknag has quit IRC | 14:00 | |
| *** maestropandy has left #openstack-cinder | 14:01 | |
| *** belmoreira has quit IRC | 14:02 | |
| *** rlrossit has quit IRC | 14:02 | |
| *** edmondsw has quit IRC | 14:02 | |
| *** dustins has quit IRC | 14:04 | |
| *** rlrossi__ has joined #openstack-cinder | 14:05 | |
| *** zul has quit IRC | 14:07 | |
| *** zul has joined #openstack-cinder | 14:07 | |
| *** dustins has joined #openstack-cinder | 14:07 | |
| *** pdeore has joined #openstack-cinder | 14:08 | |
| xyang1 | nikeshm: why is wwpns not in connector, seems that it may be a problem in the code | 14:10 |
| *** laughter_ has joined #openstack-cinder | 14:11 | |
| *** coolsvap has quit IRC | 14:11 | |
| nikeshm | <xyang1> : akerr_ also | 14:11 |
| nikeshm | faced | 14:12 |
| nikeshm | issue | 14:12 |
| akerr_ | nikeshm: yes, I've enabled TEMPEST_RUN_VALIDATION in our tests which *should* skip the problem tests for now until the patch eharney linked to is merged. We also should probably more gracefully handle missing info in our drivers (which I've opened some bugs internally about) | 14:13 |
| *** ebalduf has joined #openstack-cinder | 14:13 | |
| *** diablo_rojo has joined #openstack-cinder | 14:14 | |
| *** laughterwym has quit IRC | 14:14 | |
| *** alonma_ has quit IRC | 14:19 | |
| nikeshm | xyang1: in my case, detach was issue, connector['host'] and connector['wwpns'] was giving key error, after changing to connector.get('host', '') and connector.get('wwpns', []) it passed | 14:20 |
| xyang1 | nikeshm: ok | 14:20 |
| nikeshm | i did recheck again now | 14:20 |
| nikeshm | to pass CI | 14:21 |
| nikeshm | akerr_ : are you getting error in attach too? | 14:21 |
| akerr_ | nikeshm: i only see it in detach | 14:21 |
| openstackgerrit | naga venkata proposed openstack/cinder: Fixes backup with customized ceph cluster name https://review.openstack.org/350994 | 14:21 |
| *** yangyape_ has joined #openstack-cinder | 14:23 | |
| nikeshm | akerr_: ok in may case, since i am not raising exception in terminate_connection, so making those small changes helped me to pass those tests | 14:23 |
| akerr_ | nikeshm: xyang1: https://review.openstack.org/#/c/340951 seems to be the correct solution to the detach issues, but who knows when it'll merge | 14:23 |
| *** EinstCrazy has joined #openstack-cinder | 14:23 | |
| akerr_ | or if | 14:23 |
| *** zul has quit IRC | 14:25 | |
| xyang1 | akerr_: ok, thanks. | 14:25 |
| *** timcl has quit IRC | 14:25 | |
| *** zul has joined #openstack-cinder | 14:25 | |
| *** yangyapeng has quit IRC | 14:25 | |
| *** akerr_ is now known as akerr | 14:26 | |
| nikeshm | xyang1: do i need to report a bug for the change in mentioned https://review.openstack.org/#/c/349019/17 acccording to vipin comment | 14:28 |
| nikeshm | s/in/i | 14:29 |
| nikeshm | it is small change only | 14:29 |
| xyang1 | nikeshm: just mention that in the commit msg | 14:30 |
| *** mtanino has joined #openstack-cinder | 14:31 | |
| *** diogogmt has joined #openstack-cinder | 14:31 | |
| *** catintheroof has joined #openstack-cinder | 14:32 | |
| *** tangchen has joined #openstack-cinder | 14:32 | |
| *** jungleboyj has joined #openstack-cinder | 14:33 | |
| *** wN has quit IRC | 14:33 | |
| avishay | is there some automated way to update a repo's requirements.txt to latest oslo versions? | 14:37 |
| *** wN has joined #openstack-cinder | 14:38 | |
| *** wN has quit IRC | 14:38 | |
| *** wN has joined #openstack-cinder | 14:38 | |
| *** pdeore has quit IRC | 14:38 | |
| *** ducttape_ has joined #openstack-cinder | 14:41 | |
| avishay | cd cd | 14:43 |
| *** jwcroppe has joined #openstack-cinder | 14:45 | |
| *** jwcroppe has quit IRC | 14:45 | |
| *** ducttape_ has quit IRC | 14:45 | |
| *** jwcroppe has joined #openstack-cinder | 14:45 | |
| openstackgerrit | nikesh proposed openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 14:46 |
| *** alonma has joined #openstack-cinder | 14:46 | |
| nikeshm | xyang1 vbala: done | 14:49 |
| nikeshm | thanks | 14:49 |
| *** guy__ has joined #openstack-cinder | 14:50 | |
| *** alonma has quit IRC | 14:51 | |
| *** yangyapeng has joined #openstack-cinder | 14:53 | |
| *** alonma has joined #openstack-cinder | 14:54 | |
| *** edmondsw has joined #openstack-cinder | 14:54 | |
| *** yangyape_ has quit IRC | 14:54 | |
| *** karthikp_ has joined #openstack-cinder | 14:55 | |
| *** mdenny has joined #openstack-cinder | 14:56 | |
| guy__ | Hi everyone, I am sorry for disturbing. Can anyone review these two commits: https://review.openstack.org/#/c/349318/2 and https://review.openstack.org/#/c/347100/5, please? Thank you in advance for your help. | 14:57 |
| *** alonma has quit IRC | 14:58 | |
| *** lprice has joined #openstack-cinder | 14:59 | |
| *** timcl has joined #openstack-cinder | 15:00 | |
| thiagolib | hello someone has set up the cinder-volume to run on OpenStack mirantis pacemaker that is installed through the fuel | 15:02 |
| *** tangchen has quit IRC | 15:03 | |
| Swanson | ok | 15:03 |
| *** akshai has joined #openstack-cinder | 15:07 | |
| *** rcernin has quit IRC | 15:09 | |
| *** faiz89 has quit IRC | 15:10 | |
| *** guy__ has quit IRC | 15:11 | |
| *** daneyon_ has joined #openstack-cinder | 15:11 | |
| *** sv7577 has joined #openstack-cinder | 15:12 | |
| *** akshai has quit IRC | 15:12 | |
| *** daneyon has joined #openstack-cinder | 15:12 | |
| *** e0ne has quit IRC | 15:15 | |
| *** chris_morrell has joined #openstack-cinder | 15:15 | |
| *** daneyon_ has quit IRC | 15:16 | |
| *** EinstCrazy has quit IRC | 15:19 | |
| *** EinstCrazy has joined #openstack-cinder | 15:22 | |
| *** faiz89 has joined #openstack-cinder | 15:23 | |
| *** tangchen_ has joined #openstack-cinder | 15:23 | |
| *** catintheroof has quit IRC | 15:25 | |
| *** alonma has joined #openstack-cinder | 15:26 | |
| *** karthikp_ has quit IRC | 15:27 | |
| *** rlrossi__ has quit IRC | 15:27 | |
| *** karthikp_ has joined #openstack-cinder | 15:28 | |
| *** dustins has quit IRC | 15:28 | |
| *** rlrossit has joined #openstack-cinder | 15:30 | |
| *** leeantho has joined #openstack-cinder | 15:30 | |
| *** EinstCrazy has quit IRC | 15:30 | |
| *** alonma has quit IRC | 15:31 | |
| *** alonmarx has joined #openstack-cinder | 15:31 | |
| openstackgerrit | Jay Conroy proposed openstack/cinder: Fix volume upload-to-image for vhd disk-format https://review.openstack.org/347944 | 15:31 |
| JayConroy | New patch with added release note if anyone wants to review! https://review.openstack.org/347944 Thanks :) | 15:33 |
| *** dustins has joined #openstack-cinder | 15:35 | |
| openstackgerrit | Jay Conroy proposed openstack/cinder: Fix volume upload-to-image for vhd disk-format https://review.openstack.org/347944 | 15:36 |
| *** alonmarx has quit IRC | 15:37 | |
| *** hemnafk is now known as hemna | 15:37 | |
| sv7577 | Kindly review the volumes/summary patch https://review.openstack.org/#/c/335067/ Thanks | 15:40 |
| *** diablo_rojo has quit IRC | 15:40 | |
| *** EinstCrazy has joined #openstack-cinder | 15:42 | |
| *** diogogmt has quit IRC | 15:42 | |
| *** sv7577 has quit IRC | 15:42 | |
| *** alonmarx has joined #openstack-cinder | 15:43 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Pin RPC server's serializer to min obj version https://review.openstack.org/350534 | 15:43 |
| *** diogogmt has joined #openstack-cinder | 15:43 | |
| *** faiz89 has quit IRC | 15:45 | |
| *** crose has joined #openstack-cinder | 15:45 | |
| *** EinstCrazy has quit IRC | 15:47 | |
| *** EinstCrazy has joined #openstack-cinder | 15:48 | |
| *** lkuchlan_ has quit IRC | 15:49 | |
| *** narayrak has quit IRC | 15:49 | |
| *** tangchen_ has quit IRC | 15:51 | |
| *** karthikp_ has quit IRC | 15:55 | |
| *** alonma has joined #openstack-cinder | 15:56 | |
| *** ametts has quit IRC | 15:57 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fail manage operations if service is down https://review.openstack.org/345245 | 15:58 |
| *** alonma has quit IRC | 16:00 | |
| *** liverpooler has joined #openstack-cinder | 16:00 | |
| *** karthikp_ has joined #openstack-cinder | 16:02 | |
| *** tangchen_ has joined #openstack-cinder | 16:04 | |
| *** ametts has joined #openstack-cinder | 16:04 | |
| *** darosale has quit IRC | 16:04 | |
| *** timcl has quit IRC | 16:05 | |
| *** mdavidson has quit IRC | 16:05 | |
| *** ducttape_ has joined #openstack-cinder | 16:06 | |
| *** osnexus has quit IRC | 16:06 | |
| *** fernnest has joined #openstack-cinder | 16:06 | |
| *** alonma has joined #openstack-cinder | 16:08 | |
| geguileo | scottda: ping - cinderclient cluster commands | 16:08 |
| *** pbandark has quit IRC | 16:12 | |
| *** eharney has quit IRC | 16:13 | |
| *** alonma has quit IRC | 16:13 | |
| *** xinli has joined #openstack-cinder | 16:13 | |
| *** aorourke has joined #openstack-cinder | 16:14 | |
| *** seldenr has joined #openstack-cinder | 16:15 | |
| *** alonma has joined #openstack-cinder | 16:15 | |
| *** yangyapeng has quit IRC | 16:17 | |
| *** xinli has quit IRC | 16:17 | |
| *** yangyapeng has joined #openstack-cinder | 16:19 | |
| *** alonma has quit IRC | 16:20 | |
| scottda | geguileo: I'm afk ATM | 16:20 |
| geguileo | scottda: Ok, just wanted to let you know that the commands work (https://review.openstack.org/#/c/327692/) | 16:21 |
| geguileo | scottda: I'll update the patch with this comment | 16:21 |
| *** alonma has joined #openstack-cinder | 16:22 | |
| *** alonmarx has quit IRC | 16:22 | |
| *** e0ne has joined #openstack-cinder | 16:23 | |
| *** diablo_rojo has joined #openstack-cinder | 16:23 | |
| *** charlesr has joined #openstack-cinder | 16:25 | |
| *** timcl has joined #openstack-cinder | 16:25 | |
| *** jungleboyj has quit IRC | 16:27 | |
| *** karthikp_ has quit IRC | 16:27 | |
| mriedem | scottda: i thought at the midcycle we agreed on https://review.openstack.org/#/c/325895/ just being a passthrough to tempest.conf? | 16:27 |
| *** alonma has quit IRC | 16:27 | |
| openstackgerrit | Avishay Traeger proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/351275 | 16:27 |
| *** avishay has quit IRC | 16:28 | |
| *** alonmarx has joined #openstack-cinder | 16:28 | |
| *** Apoorva has joined #openstack-cinder | 16:29 | |
| *** cknight has joined #openstack-cinder | 16:29 | |
| *** zhipeng_ has joined #openstack-cinder | 16:30 | |
| *** alonma has joined #openstack-cinder | 16:30 | |
| *** _ducttape_ has joined #openstack-cinder | 16:30 | |
| *** zhipeng has quit IRC | 16:31 | |
| *** Apoorva has quit IRC | 16:31 | |
| *** Apoorva has joined #openstack-cinder | 16:31 | |
| *** kro_focused is now known as krotscheck | 16:32 | |
| *** EinstCra_ has joined #openstack-cinder | 16:32 | |
| *** ducttape_ has quit IRC | 16:33 | |
| *** sandanar has joined #openstack-cinder | 16:33 | |
| *** alonma has quit IRC | 16:34 | |
| openstackgerrit | Merged openstack/cinder: Remove comma from end of policy.json https://review.openstack.org/351177 | 16:35 |
| *** EinstCrazy has quit IRC | 16:35 | |
| *** alonma has joined #openstack-cinder | 16:37 | |
| swamireddy | smcginnis: hi | 16:38 |
| *** jungleboyj has joined #openstack-cinder | 16:39 | |
| *** cknight has quit IRC | 16:40 | |
| *** cknight has joined #openstack-cinder | 16:40 | |
| *** alonma has quit IRC | 16:41 | |
| *** e0ne has quit IRC | 16:43 | |
| *** cknight has quit IRC | 16:44 | |
| *** alonma has joined #openstack-cinder | 16:44 | |
| *** alonma has quit IRC | 16:49 | |
| *** _ducttape_ has quit IRC | 16:50 | |
| *** ducttape_ has joined #openstack-cinder | 16:51 | |
| *** alonma has joined #openstack-cinder | 16:51 | |
| openstackgerrit | Merged openstack/cinder: VMAX Driver - SnapVX licensing checks for VMAX3 https://review.openstack.org/322972 | 16:53 |
| *** david-lyle has quit IRC | 16:53 | |
| openstackgerrit | Merged openstack/cinder: Prevent adding same OVO version twice to history https://review.openstack.org/351124 | 16:53 |
| *** david-lyle has joined #openstack-cinder | 16:53 | |
| *** alonma has quit IRC | 16:56 | |
| scottda | mriedem: We did, and I tried that, but then the parsing would need to be done on the tempest side. | 16:56 |
| mriedem | scottda: could use some more comments about how this is used, but i +1'ed it | 16:56 |
| mriedem | but i had to look at the tempest and cinder code to see if this made sense | 16:57 |
| mriedem | which kind of sucks | 16:57 |
| *** Suyi_ has joined #openstack-cinder | 16:57 | |
| scottda | mriedem Yeah, I'll add the comments. I originally had my own patch to make these changes to devstack, but someone pointed me to this one in-flight, so I've been trying to make it all work and appropriate (and well-commented) ever since... | 16:58 |
| *** alonma has joined #openstack-cinder | 16:59 | |
| *** belmoreira has joined #openstack-cinder | 17:00 | |
| *** alonma has quit IRC | 17:03 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_WIKI_NAME to all drivers https://review.openstack.org/348002 | 17:06 |
| *** alonma has joined #openstack-cinder | 17:06 | |
| *** mvk has quit IRC | 17:08 | |
| *** laughter_ has quit IRC | 17:10 | |
| *** alonma has quit IRC | 17:10 | |
| *** laughterwym has joined #openstack-cinder | 17:11 | |
| *** permalac has quit IRC | 17:11 | |
| *** sandanar has quit IRC | 17:11 | |
| *** eharney has joined #openstack-cinder | 17:12 | |
| *** belmoreira has quit IRC | 17:13 | |
| *** alonma has joined #openstack-cinder | 17:13 | |
| *** zhipeng_ has quit IRC | 17:14 | |
| *** jdurgin1 has quit IRC | 17:14 | |
| *** zhipeng has joined #openstack-cinder | 17:15 | |
| *** laughterwym has quit IRC | 17:15 | |
| *** lpetrut has quit IRC | 17:15 | |
| *** alonma has quit IRC | 17:17 | |
| *** alonma has joined #openstack-cinder | 17:20 | |
| *** alonmarx has quit IRC | 17:20 | |
| *** karthikp_ has joined #openstack-cinder | 17:21 | |
| openstackgerrit | nikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers https://review.openstack.org/350182 | 17:21 |
| *** faiz89 has joined #openstack-cinder | 17:21 | |
| *** karthikp_ has quit IRC | 17:24 | |
| *** alonmarx has joined #openstack-cinder | 17:24 | |
| *** alonma has quit IRC | 17:25 | |
| *** alonma has joined #openstack-cinder | 17:27 | |
| *** karthikp_ has joined #openstack-cinder | 17:30 | |
| *** alonmarx has quit IRC | 17:30 | |
| *** alonma has quit IRC | 17:32 | |
| *** zul has quit IRC | 17:36 | |
| *** alonma has joined #openstack-cinder | 17:37 | |
| *** jwcroppe has quit IRC | 17:37 | |
| *** chris_morrell has quit IRC | 17:39 | |
| *** darosale has joined #openstack-cinder | 17:39 | |
| *** alonma has quit IRC | 17:42 | |
| *** jwcroppe has joined #openstack-cinder | 17:42 | |
| *** alonma has joined #openstack-cinder | 17:44 | |
| bswartz | patrickeast winston-d: I'm a Cinder ATC again! | 17:46 |
| patrickeast | woot! | 17:46 |
| *** karthikp_ has quit IRC | 17:49 | |
| *** alonma has quit IRC | 17:49 | |
| *** zul has joined #openstack-cinder | 17:49 | |
| *** alonma has joined #openstack-cinder | 17:51 | |
| *** e0ne has joined #openstack-cinder | 17:53 | |
| *** tesseract- has quit IRC | 17:53 | |
| *** xinli has joined #openstack-cinder | 17:55 | |
| *** alonma has quit IRC | 17:56 | |
| *** mvk has joined #openstack-cinder | 17:56 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 17:57 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 17:57 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 17:57 |
| *** alonma has joined #openstack-cinder | 17:59 | |
| *** darosale_ has joined #openstack-cinder | 18:00 | |
| *** darosale has quit IRC | 18:01 | |
| *** darosale_ is now known as darosale | 18:01 | |
| *** julim has quit IRC | 18:02 | |
| *** alonma has quit IRC | 18:03 | |
| *** julim has joined #openstack-cinder | 18:05 | |
| *** alonma has joined #openstack-cinder | 18:06 | |
| *** ducttape_ has quit IRC | 18:08 | |
| *** lpetrut has joined #openstack-cinder | 18:09 | |
| *** ducttape_ has joined #openstack-cinder | 18:10 | |
| *** alonma has quit IRC | 18:10 | |
| *** tpeoples has quit IRC | 18:12 | |
| *** alonma has joined #openstack-cinder | 18:13 | |
| *** alonma has quit IRC | 18:17 | |
| nikeshm | xyang1 smcginnis : +1 from jenkins and CI is happy and addressed all comments | 18:18 |
| nikeshm | https://review.openstack.org/#/c/349019/18 | 18:18 |
| *** tpsilva has quit IRC | 18:18 | |
| *** alonma has joined #openstack-cinder | 18:20 | |
| *** alonma_ has joined #openstack-cinder | 18:22 | |
| *** alonma has quit IRC | 18:25 | |
| *** karthikp_ has joined #openstack-cinder | 18:25 | |
| *** alonma_ has quit IRC | 18:26 | |
| *** bardia has joined #openstack-cinder | 18:28 | |
| *** alonma has joined #openstack-cinder | 18:29 | |
| *** alonma has quit IRC | 18:33 | |
| *** karthikp_ has quit IRC | 18:35 | |
| *** karthikp_ has joined #openstack-cinder | 18:35 | |
| *** alonma has joined #openstack-cinder | 18:38 | |
| *** jdillaman has quit IRC | 18:38 | |
| *** charlesr has quit IRC | 18:39 | |
| *** jdillaman has joined #openstack-cinder | 18:39 | |
| *** diogogmt has quit IRC | 18:42 | |
| *** diogogmt has joined #openstack-cinder | 18:42 | |
| *** alonma has quit IRC | 18:43 | |
| mriedem | so http://logs.openstack.org/95/350595/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/cb41fc0/logs/deprecations.txt.gz | 18:44 |
| mriedem | WARNING cinder.quota Deprecated: Default quota for resource: backups is set by the default quota flag: quota_backups, it is now deprecated. Please use the default quota class for default quota. | 18:45 |
| mriedem | those have been in the cinder logs as long as i've worked on openstack | 18:45 |
| mriedem | which is ~havana | 18:45 |
| mriedem | what does it take to actually fix that? wouldn't it just be changes to devstack? | 18:45 |
| *** ChubYann has joined #openstack-cinder | 18:45 | |
| scottda | mriedem: I'll have a look. Maybe the answer is to remove that deprecation Warning. | 18:46 |
| mriedem | or if that's never going to be removed, shouldn't it be undeprecated? | 18:46 |
| mriedem | yeah | 18:46 |
| scottda | mriedem: It looks like that message just went in a year ago with cinder commit e2eb94bdbbe | 18:48 |
| *** yangyapeng has quit IRC | 18:49 | |
| scottda | mriedem ahh..no, that is just the latest change to the message.... | 18:49 |
| mriedem | yeah | 18:50 |
| mriedem | stuff like this | 18:50 |
| mriedem | WARNING cinder.quota [req-17d548ab-931b-46cb-80b1-20fe8e7f7edc None] Deprecated: Default quota for resource: gigabytes_lvmdriver-1 is set by the default quota flag: quota_gigabytes_lvmdriver-1, it is now deprecated. Please use the default quota class for default quota. | 18:50 |
| mriedem | has been around forever | 18:50 |
| scottda | mriedem: I'll put up a patch to remove that deprecation warning, and see if anyone has a reason why not to remove it. | 18:51 |
| *** jwcroppe has quit IRC | 18:52 | |
| mriedem | thanks | 18:52 |
| bardia | smcginnis, eharney: Can I please get a few minutes of your time on this: https://review.openstack.org/#/c/346238 | 18:53 |
| jgriffith | mriedem: scottda um, I don't think you should remove the deprecation warning please | 18:56 |
| openstackgerrit | nikesh proposed openstack/cinder: Improve snapshot handling in K2 https://review.openstack.org/348984 | 18:56 |
| jgriffith | mriedem: scottda that was deprecated and never removed and never updated in devstack | 18:56 |
| mriedem | jgriffith: it's been deprecated forever | 18:56 |
| jgriffith | mriedem: scottda all of the above needs to happen | 18:56 |
| mriedem | so we should defecate or get off the pot | 18:56 |
| jgriffith | mriedem: yes, that's what I"m saying... | 18:56 |
| jgriffith | mriedem: removing the message is wrong, but updating devstack is *right* | 18:57 |
| scottda | jgriffith: OK, so we should change devstack first... | 18:57 |
| jgriffith | mriedem: IMO yes, and then remove the capability altogether, it's more than met the deprecation period | 18:57 |
| jgriffith | mriedem: unless that stuff just never worked, because I know there was a lot of thrashing on it | 18:58 |
| jgriffith | Oop | 18:58 |
| jgriffith | oops | 18:58 |
| jgriffith | s/mriedman/scottda/ above | 18:58 |
| scottda | jgriffith: Thanks for the info. I'll look at devstack and then file a bug... | 18:59 |
| jgriffith | scottda: yeah, let's see if that change was ever finished up or not | 18:59 |
| *** rlrossit has quit IRC | 18:59 | |
| *** jdillaman has quit IRC | 18:59 | |
| jgriffith | I'm assuming the deprecation warning wouldn't go in without the change working :) | 18:59 |
| *** xinli has quit IRC | 18:59 | |
| jgriffith | but that could be a bad assumption :) | 18:59 |
| *** rlrossit has joined #openstack-cinder | 19:00 | |
| *** EinstCra_ has quit IRC | 19:00 | |
| jay-mehta | hello folks | 19:00 |
| jay-mehta | I was devstacking master | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder Traceback (most recent call last): | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder File "/usr/local/bin/cinder-volume", line 10, in <module> | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder sys.exit(main()) | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder File "/opt/stack/cinder/cinder/cmd/volume.py", line 79, in main | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder priv_context.init(root_helper=shlex.split(utils.get_root_helper())) | 19:00 |
| jay-mehta | 2016-08-04 11:44:20.711 TRACE cinder AttributeError: 'module' object has no attribute 'init' | 19:00 |
| *** rlrossit_ has joined #openstack-cinder | 19:01 | |
| jay-mehta | This is when I try to bring up c-vol | 19:01 |
| jay-mehta | Somehow I didn't had the latest oslo_privsep installed which led to above failure | 19:02 |
| *** alonma has joined #openstack-cinder | 19:02 | |
| *** EinstCrazy has joined #openstack-cinder | 19:02 | |
| openstackgerrit | Avishay Traeger proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/351275 | 19:03 |
| *** rlrossit has quit IRC | 19:04 | |
| *** jdillaman has joined #openstack-cinder | 19:05 | |
| *** jwcroppe has joined #openstack-cinder | 19:06 | |
| *** alonma has quit IRC | 19:06 | |
| *** jwcroppe has quit IRC | 19:07 | |
| *** fifieldt has quit IRC | 19:07 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 19:08 |
| *** alonma has joined #openstack-cinder | 19:09 | |
| *** timcl has quit IRC | 19:10 | |
| *** EinstCrazy has quit IRC | 19:13 | |
| *** sdake_ has joined #openstack-cinder | 19:13 | |
| *** alonma has quit IRC | 19:13 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 19:14 |
| *** sdake has quit IRC | 19:16 | |
| *** baumann has joined #openstack-cinder | 19:18 | |
| nikeshm | hi if any core get time, please give +2 and +1 on https://review.openstack.org/#/c/349019, one +2 already there | 19:18 |
| *** fifieldt has joined #openstack-cinder | 19:18 | |
| *** alonma has joined #openstack-cinder | 19:19 | |
| jgriffith | mriedem: scottda so looking at that default quota change thing.. you were actually more on the right track | 19:19 |
| jgriffith | mriedem: scottda we did in fact update that so it's just ignoring the option and logging the warning... I had thought it was still using the old global set thing | 19:19 |
| scottda | jgriffith: Thanks for looking. So, sounds like it's ok to yank that warning? | 19:20 |
| *** haplo37__ has joined #openstack-cinder | 19:20 | |
| jgriffith | scottda: in other words, that option should probably just be removed from the conf as should the lines associated with it | 19:20 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove VolumeDriver abcs https://review.openstack.org/351346 | 19:21 |
| openstackgerrit | Eric Harney proposed openstack/cinder: RBD: Inherit from VolumeDriver instead of *VD https://review.openstack.org/351347 | 19:21 |
| jgriffith | scottda: https://gist.github.com/j-griffith/617cfd08cc9bd879e1b512c4fe3b7f39 | 19:21 |
| hemna | eharney, w00t! | 19:22 |
| scottda | jgriffith: Cool, I'm in the middle of that patch now, unless you want to do it... | 19:22 |
| *** alonma has quit IRC | 19:23 | |
| jgriffith | scottda: nope. all yours | 19:23 |
| lpetrut | hemna: Hi, we've got green results on the Windows os-brick connector patches after rebasing them. Any chance those could be merged? (https://review.openstack.org/#/c/323781/ https://review.openstack.org/#/c/323780/) | 19:23 |
| e0ne | jgriffith: hi. I've got questions about solidfire and netapp backends:) | 19:24 |
| hemna | lpetrut, I'm just now looking at a block of os-brick patches | 19:25 |
| hemna | lpetrut, I'll get to it today | 19:25 |
| jgriffith | e0ne: sure | 19:25 |
| lpetrut | hemna: that's great, thank you! | 19:25 |
| *** alonma has joined #openstack-cinder | 19:26 | |
| e0ne | jgriffith: I've got Liverty env an try to retype from SolidFire to Netapp | 19:26 |
| *** rlrossit_ has quit IRC | 19:26 | |
| jgriffith | e0ne: k | 19:26 |
| Swanson | What's the appropriate action in failover-host if the host is specified incorrectly? I'm throwing an InvalidInput which I think is leaving things in a half state. | 19:27 |
| e0ne | and solid fire drivers failes to remove original volume | 19:27 |
| jgriffith | e0ne: so you have the migrate with retype but the SF volume never gets deleted? | 19:27 |
| jgriffith | e0ne: Ohhh! Liberty | 19:27 |
| e0ne | jgriffith: yea | 19:27 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: Implement v2.1 replication https://review.openstack.org/333565 | 19:27 |
| jgriffith | e0ne: there was a bug in the Cinder code I fixed for that I think | 19:27 |
| jgriffith | e0ne: so the volume migrate stuff tends to get *confused* with all of it's magic swapping of db entries | 19:28 |
| jgriffith | e0ne: and IIRC there was a problem in general in the manager with cleanup not happening/working | 19:28 |
| e0ne | jgriffith: here is some piece logs | 19:28 |
| e0ne | jgriffith: yes. it's what i see on my env | 19:29 |
| jgriffith | e0ne: one sanity check is try migrating LVM to something (even another LVM) | 19:29 |
| jgriffith | e0ne: I'm happy to fire it up in a bit and try it on my side as well and see if I can help | 19:29 |
| jgriffith | e0ne: getting liberty deployed any more though is kinda painful :( | 19:29 |
| openstackgerrit | nikesh proposed openstack/cinder: Ignore requests warnings in K2 cinder driver https://review.openstack.org/349013 | 19:30 |
| *** alonma has quit IRC | 19:31 | |
| akerr | i think you can just checkout the stable/liberty branch of devstack and it' *should* work | 19:31 |
| *** agrebennikov has joined #openstack-cinder | 19:31 | |
| akerr | as well as devstack ever works | 19:31 |
| *** rlrossit_ has joined #openstack-cinder | 19:31 | |
| *** rlrossit_ has quit IRC | 19:31 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 19:31 |
| *** rlrossit_ has joined #openstack-cinder | 19:31 | |
| e0ne | jgriffith: are you talking about this bug https://bugs.launchpad.net/cinder/+bug/1452823? | 19:31 |
| openstack | Launchpad bug 1452823 in Cinder kilo "retype ignores extra_specs info including backend_name" [Undecided,Fix released] - Assigned to John Griffith (john-griffith) | 19:31 |
| jgriffith | e0ne: BTW, geguileo worked on this at one point as well here: https://github.com/openstack/cinder/commit/cdf5e92f10fe8d097f91f6f63a2c77f6ab2bcdb8#diff-724c53b8cd580ae533b4fb301df7c3e9 | 19:31 |
| jgriffith | e0ne: nah, that was another horrible problem :) | 19:32 |
| *** alonma has joined #openstack-cinder | 19:33 | |
| e0ne | jgriffith: netapp(nfs)=>solidfire migration works for me because nfs driver uses provider_location to delete a volume | 19:33 |
| jgriffith | e0ne: I don't know if you remember the proposal to have everything go through the scheduler or not when we were in Tokyo, but that bug you found is what prompted that opinion | 19:33 |
| agrebennikov | hey jgriffith, so regarding this issue... if it ws fixed - you guys had to completely change the approach of retyping | 19:33 |
| jgriffith | agrebennikov: ? | 19:33 |
| agrebennikov | since there is no changed inthe deletion method in solidfire driver | 19:34 |
| jgriffith | agrebennikov: which issue, there are two of them :) | 19:34 |
| agrebennikov | it just consimes the id of the volume | 19:34 |
| e0ne | jgriffith: I support the idea to call anything via scheduler | 19:34 |
| agrebennikov | oh)) | 19:34 |
| agrebennikov | the retype | 19:34 |
| *** jwcroppe has joined #openstack-cinder | 19:34 | |
| jgriffith | agrebennikov: this one: https://bugs.launchpad.net/cinder/+bug/1452823 | 19:34 |
| openstack | Launchpad bug 1452823 in Cinder kilo "retype ignores extra_specs info including backend_name" [Undecided,Fix released] - Assigned to John Griffith (john-griffith) | 19:34 |
| agrebennikov | when the violume on the solidfire is not deleted | 19:34 |
| agrebennikov | no)) jgriffith this is not related | 19:34 |
| jgriffith | agrebennikov: ok, the first one :) | 19:35 |
| jgriffith | agrebennikov: the one e0ne is asking about | 19:35 |
| agrebennikov | right | 19:35 |
| *** akerr has quit IRC | 19:35 | |
| agrebennikov | so the bug you show describes the case of the same backends | 19:35 |
| agrebennikov | we have different backends | 19:35 |
| jgriffith | agrebennikov: which I believe is related to this patch geguileo worked on: https://github.com/openstack/cinder/commit/cdf5e92f10fe8d097f91f6f63a2c77f6ab2bcdb8#diff-724c53b8cd580ae533b4fb301df7c3e9 | 19:35 |
| jgriffith | agrebennikov: and I'm looking for follow up work on it now :) | 19:35 |
| *** karthikp_ has quit IRC | 19:36 | |
| e0ne | jgriffith: that patch doesn't solve our problem | 19:36 |
| agrebennikov | jgriffith, hang on | 19:36 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Remove deprecated CONF.use_default_quota_class https://review.openstack.org/351355 | 19:37 |
| agrebennikov | it is very different issue)) | 19:37 |
| *** Apoorva has quit IRC | 19:37 | |
| agrebennikov | I mean, unrelated to the manager | 19:37 |
| e0ne | jgriffith, agrebennikov: let me describe what did we found with this issue | 19:37 |
| jgriffith | maybe it'll be easier for me to just reproduce the issue you guys are seeing | 19:37 |
| jgriffith | e0ne: that would be helpful :) | 19:37 |
| agrebennikov | jgriffith, sure - if you've got solidfire | 19:37 |
| *** alonma has quit IRC | 19:38 | |
| scottda | jay-mehta: My devstack from yesterday works. Is yours freshly cloned today? | 19:39 |
| e0ne | jgriffith: here is manager.migrate_volume_completion code: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1816-L1835 | 19:39 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 19:39 |
| *** alonma has joined #openstack-cinder | 19:40 | |
| *** haplo37__ has quit IRC | 19:41 | |
| e0ne | https://github.com/openstack/cinder/blob/stable/liberty/cinder/volume/manager.py#L1752-L1766 - the same, but liberty | 19:41 |
| *** xinli has joined #openstack-cinder | 19:41 | |
| e0ne | jgriffith: 'finish_volume_migration' method swaps volume data in the db | 19:41 |
| e0ne | jgriffith: than we call rpcapi.delete_volume(ctxt, updated_new) | 19:42 |
| jgriffith | e0ne: right.. and it does it prior to calling the delete | 19:42 |
| e0ne | updated_new will have new (fake or temp) id with old other data like name on storage, provider location, etc | 19:42 |
| e0ne | jgriffith: in delete, we just call driver.volume_delete(vol_obj) | 19:43 |
| *** Lee1092 has quit IRC | 19:43 | |
| *** karthikp_ has joined #openstack-cinder | 19:44 | |
| e0ne | jgriffith: and here is how solid fire driver works: https://github.com/openstack/cinder/blob/stable/liberty/cinder/volume/drivers/solidfire.py#L955 | 19:44 |
| hemna | lpetrut, ping | 19:44 |
| e0ne | jgriffith: it uses volume.id | 19:44 |
| lpetrut | yep | 19:44 |
| hemna | 'device_name': '\\\\.\\PhysicalDrive1' | 19:44 |
| hemna | seeing that in the logs | 19:44 |
| e0ne | jgriffith: but manager passas volume object with a new id | 19:44 |
| hemna | lpetrut, http://64.119.130.115/os-brick/323780/13/fc/Hyper-V_logs/c2-hp-u36/nova-compute.log.gz | 19:45 |
| *** crose has quit IRC | 19:45 | |
| jgriffith | e0ne: right, which is the problem with the swapping of things | 19:45 |
| hemna | I think you need more \'s | 19:45 |
| *** alonma has quit IRC | 19:45 | |
| e0ne | jgriffith: wich is not available at solid fire storage | 19:45 |
| e0ne | jgriffith: yep | 19:45 |
| jgriffith | e0ne: IIRC we had the same problem with LVM and some others. That's why that delete_volume call was moved before the update IIRC | 19:45 |
| jgriffith | but I could be wrong | 19:45 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 19:46 |
| jgriffith | e0ne: I'm happy to load this up and come up with a fix for you if it helps? Unless you've already got it figured out | 19:46 |
| lpetrut | hemna: not sure I understand what you mean. so, basically, we have '\\.\PhysicalDrive1', which is fine | 19:46 |
| hemna | well, I'm just curious why all the extra \'s | 19:46 |
| hemna | evidently nova is happy with it | 19:47 |
| hemna | but \\\\.\\ ? | 19:47 |
| hemna | really | 19:47 |
| hemna | heh | 19:47 |
| lpetrut | oh, got it, the backslashes are automatically escaped when printed | 19:47 |
| e0ne | jgriffith: I'm not sure what should we fix | 19:47 |
| *** alonma has joined #openstack-cinder | 19:47 | |
| hemna | 016-08-04 08:10:23.296 | 19:47 |
| hemna | in the log | 19:47 |
| hemna | you can see em | 19:47 |
| e0ne | driver's fix could be good in a short-term, but manager's fix sound better in a long-term perspective | 19:48 |
| lpetrut | hemna: for the same reason, we usually use raw strings when defining Windows paths, to avoid the mess of escaping strings, but when those get logged, there's not much we can do. | 19:51 |
| hemna | so the log is adding those \'s ? | 19:51 |
| lpetrut | yep | 19:51 |
| hemna | :( | 19:51 |
| hemna | ok, just one small nit on the patch | 19:52 |
| hemna | but I think it's useful | 19:52 |
| e0ne | jgriffith: if would be good if you can try retype from silidfire to something on master. I have not env with master:( | 19:52 |
| *** alonma has quit IRC | 19:52 | |
| hemna | lpetrut, after that, I think it's good to go | 19:53 |
| hemna | same w/ smbfs | 19:53 |
| lpetrut | great, thanks again for taking the time to review the patches ! updating them right now. | 19:54 |
| *** alonma has joined #openstack-cinder | 19:55 | |
| hemna | lpetrut, great! thanks for the work | 19:55 |
| *** timcl has joined #openstack-cinder | 19:56 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add cluster related commands https://review.openstack.org/327692 | 19:58 |
| *** alonma has quit IRC | 19:59 | |
| *** akerr has joined #openstack-cinder | 20:00 | |
| *** alonmarx has joined #openstack-cinder | 20:00 | |
| *** dustins has quit IRC | 20:00 | |
| lpetrut | umm, one question about the utils.trace decorator, are you guys ok with logging the connection_properties dict? I was not sure whether we should log CHAP credentials or similar data that may be considered sensitive. | 20:01 |
| e0ne | jgriffith: http://paste.openstack.org/show/549296/ - it works for me, but I'm not expert in this driver | 20:01 |
| *** bill_az has joined #openstack-cinder | 20:02 | |
| *** alonma has joined #openstack-cinder | 20:02 | |
| *** eharney has quit IRC | 20:02 | |
| *** akerr_ has joined #openstack-cinder | 20:02 | |
| *** dustins has joined #openstack-cinder | 20:02 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 20:04 |
| *** alonmarx has quit IRC | 20:04 | |
| *** ametts has quit IRC | 20:05 | |
| *** zul has quit IRC | 20:05 | |
| *** akerr has quit IRC | 20:05 | |
| *** alonma has quit IRC | 20:07 | |
| agrebennikov | jgriffith, this is exactly how it works in generic lvm driver | 20:07 |
| *** merooney has quit IRC | 20:07 | |
| agrebennikov | when the driver gets the entire volume object, but the id is incorrect (from the backend's point of view) | 20:08 |
| *** alonma has joined #openstack-cinder | 20:09 | |
| *** alonmarx has joined #openstack-cinder | 20:10 | |
| *** Apoorva has joined #openstack-cinder | 20:10 | |
| *** Apoorva has quit IRC | 20:10 | |
| *** ametts has joined #openstack-cinder | 20:10 | |
| *** Apoorva has joined #openstack-cinder | 20:10 | |
| *** jungleboyj has quit IRC | 20:11 | |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows Fibre Channel connector https://review.openstack.org/323780 | 20:12 |
| openstackgerrit | Lucian Petrut proposed openstack/os-brick: Add Windows SMBFS connector https://review.openstack.org/323781 | 20:12 |
| *** markvoelker has quit IRC | 20:13 | |
| *** porrua has quit IRC | 20:14 | |
| *** alonma has quit IRC | 20:14 | |
| openstackgerrit | Merged openstack/cinder: Improvement to query cg detail(Part 1) https://review.openstack.org/329842 | 20:15 |
| *** alonma has joined #openstack-cinder | 20:16 | |
| jgriffith | e0ne: oh... so you just added name_id as a check and got it that way | 20:17 |
| e0ne | jgriffith: yes, but I don't know how is it right | 20:17 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 20:18 |
| e0ne | jgriffith: lvm, nfs and rbd work similar | 20:18 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR add entry point debug tracing https://review.openstack.org/339116 | 20:18 |
| e0ne | jgriffith: they use name or provider location | 20:18 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: LeftHand add entry point tracing https://review.openstack.org/339128 | 20:19 |
| *** eharney has joined #openstack-cinder | 20:19 | |
| jgriffith | e0ne: name should be fine | 20:19 |
| agrebennikov | jgriffith, | 20:19 |
| agrebennikov | name = volume['name'] | 20:19 |
| agrebennikov | if is_snapshot: | 20:19 |
| agrebennikov | name = self._escape_snapshot(volume['name']) | 20:19 |
| agrebennikov | self.vg.delete(name) | 20:19 |
| agrebennikov | this is the example from lvm | 20:19 |
| e0ne | jgriffith: cool. I'll file a bug and submit patch tomorrow | 20:19 |
| jgriffith | e0ne: yeah | 20:20 |
| e0ne | agrebennikov: please, use paste to post code | 20:20 |
| jgriffith | e0ne: I'm happy to submit the patch if you have other things going on , but either way | 20:20 |
| jgriffith | totally up to you | 20:20 |
| e0ne | jgriffith: feel free to submit it :) | 20:20 |
| *** timcl has quit IRC | 20:20 | |
| jgriffith | e0ne: :) That way I can test it out on our CI a bit | 20:20 |
| e0ne | jgriffith: I don't want to do it in 11pm:) | 20:20 |
| e0ne | s/in/at 11pm | 20:21 |
| jgriffith | e0ne: and agrebennikov thanks very much for bringing it to my attention | 20:21 |
| *** alonma has quit IRC | 20:21 | |
| e0ne | jgriffith: np | 20:21 |
| agrebennikov | нц | 20:21 |
| agrebennikov | yw | 20:21 |
| *** alonma has joined #openstack-cinder | 20:23 | |
| *** jungleboyj has joined #openstack-cinder | 20:26 | |
| *** crose has joined #openstack-cinder | 20:28 | |
| *** alonma has quit IRC | 20:28 | |
| *** alonmarx has quit IRC | 20:29 | |
| *** catintheroof has joined #openstack-cinder | 20:30 | |
| jungleboyj | scottda: Ping. | 20:30 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 20:30 |
| *** alonma has joined #openstack-cinder | 20:30 | |
| *** e0ne has quit IRC | 20:31 | |
| *** e0ne has joined #openstack-cinder | 20:32 | |
| scottda | jungleboyj: hey | 20:32 |
| *** lpetrut has quit IRC | 20:33 | |
| *** pauloewerton has quit IRC | 20:33 | |
| jungleboyj | scottda: Hey, what was the process you use to get an auth token for doing curl commands? | 20:33 |
| *** alonmarx has joined #openstack-cinder | 20:34 | |
| scottda | Openstack token-update | 20:34 |
| scottda | I IRC | 20:34 |
| *** alonma has quit IRC | 20:35 | |
| *** lpetrut has joined #openstack-cinder | 20:35 | |
| *** xinli has quit IRC | 20:35 | |
| *** alonma has joined #openstack-cinder | 20:38 | |
| scottda | ohh..no, it's "openstack token issue" | 20:38 |
| scottda | jungleboyj: ^^ | 20:38 |
| jungleboyj | And then we should be able that to paste into something like this: -H "X-Auth-Token: {SHA1}4ccd82d7fdf59988d0ea61db5e56ec2d53ac2ae6" | 20:40 |
| patrickeast | yea | 20:41 |
| scottda | jungleboyj: yup, after the "X-Auth-Token:<your_token_here>" | 20:42 |
| scottda | jungleboyj: Or, if you're doing a bunch of curls, use: | 20:42 |
| *** alonma has quit IRC | 20:43 | |
| scottda | -H "X-Auth-Token: $TOKEN" | 20:43 |
| scottda | which will work if you do "export TOKEN=3993de94a94....." | 20:43 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 20:44 |
| scottda | If you save the commands you use with $TOKEN as a shell variable, you can just export the token each time you have to get a new one... | 20:44 |
| *** e0ne has quit IRC | 20:44 | |
| *** alonma has joined #openstack-cinder | 20:45 | |
| jungleboyj | Cool. Thanks. I sent that info along and will see if it works. | 20:46 |
| *** jdillaman has quit IRC | 20:46 | |
| jungleboyj | scottda: Do you use id or user_id ? | 20:47 |
| patrickeast | id is the token | 20:48 |
| *** jdillaman has joined #openstack-cinder | 20:48 | |
| *** alonma has quit IRC | 20:50 | |
| jungleboyj | With or without [SHA1] ? | 20:50 |
| scottda | jungleboyj: remove the SHA1. that's there to obscure the real token from the debug output | 20:54 |
| jungleboyj | Ah, thank you. | 20:54 |
| scottda | anytime | 20:54 |
| scottda | Unless you are on-call 24/7 for your public cloud. In that case, only during banking hours. | 20:54 |
| openstackgerrit | Merged openstack/cinder: XtremIO: Implement update_migrated_volume https://review.openstack.org/344333 | 20:55 |
| openstackgerrit | Merged openstack/cinder: Support retype in K2 cinder driver https://review.openstack.org/349019 | 20:56 |
| scottda | jungleboyj: Try this: | 20:58 |
| scottda | export TOKEN=`openstack token issue | awk 'FNR == 5 {print $4}'` | 20:58 |
| scottda | Then you can do this: | 20:58 |
| scottda | curl -g -i -X GET http://192.168.122.151:8776/v2/9dfc1992c01f43e493218e545e4ef76a/volumes/detail -H "User-Agent: python-cinderclient" -H "Accept: application/json" -H "X-Auth-Token: $TOKEN" | 20:59 |
| *** alonma has joined #openstack-cinder | 20:59 | |
| hemna | I'm so jealous of the SF driver. | 21:01 |
| hemna | I don't have any VAGs | 21:01 |
| *** lprice has quit IRC | 21:02 | |
| *** alonma has quit IRC | 21:03 | |
| patrickeast | lol | 21:05 |
| hemna | # VAG limit reached. Purge and start over. | 21:05 |
| *** alonma has joined #openstack-cinder | 21:06 | |
| smcginnis | no comment | 21:07 |
| hemna | none | 21:07 |
| hemna | smcginnis, https://bugs.launchpad.net/nova/+bug/1609984 | 21:08 |
| openstack | Launchpad bug 1609984 in OpenStack Compute (nova) "volume-detach fails for shelved instance" [Undecided,New] | 21:08 |
| hemna | sadness | 21:08 |
| *** diablo_rojo has quit IRC | 21:08 | |
| smcginnis | I had to skip that test in my CI. Non stop failures. | 21:08 |
| smcginnis | hemna: There was some talk about it this morning. It was brought up that maybe our drivers should handle that a little better. | 21:09 |
| hemna | well | 21:09 |
| hemna | we can't really | 21:09 |
| hemna | it's a bogus connector being passed in | 21:09 |
| hemna | also | 21:10 |
| hemna | there was no export on the array as well | 21:10 |
| smcginnis | hemna: Yeah, I wasn't sold on that idea. Garbage in garbage out. | 21:10 |
| hemna | because the attach can happen when an instance is shelved | 21:10 |
| *** jwcroppe has quit IRC | 21:10 | |
| hemna | and nova never calls the driver to do anything. | 21:10 |
| hemna | cinder just goes....yah..attached. | 21:10 |
| hemna | the entire process is borked | 21:10 |
| smcginnis | :/ | 21:10 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 21:11 |
| scottda | Yeah, we should have a think on how the shelved attach/detach thing should work, and make it better with the ShinyNewCinderNova API | 21:11 |
| *** alonma has quit IRC | 21:11 | |
| hemna | so | 21:11 |
| smcginnis | scottda: +1 | 21:11 |
| *** alonmarx has quit IRC | 21:11 | |
| hemna | we are seeing lots of tempest failures now because the test used to be skipped | 21:11 |
| hemna | because of some ssh reason | 21:11 |
| scottda | I think when an instance is shelved and a volume is attached, it always detaches, but keeps the state as "attached" and "in-use" in cinder... | 21:11 |
| hemna | they just removed the skip, and now cinder CI's are getting hit with it | 21:12 |
| hemna | and it'll never work as it is | 21:12 |
| scottda | Because when the instance is un-shelved, it can be on another host. | 21:12 |
| hemna | but you can attach a volume to a shelved instance too | 21:12 |
| hemna | which does nothing in cinder but call attach_volume (creating the DB attachment table record) | 21:12 |
| hemna | *sigh* | 21:12 |
| hemna | but when you detach a volume from a shelved instance, the driver's terminate_connection is called | 21:13 |
| scottda | yup. but is it really attached? Or just says "attached" in cinder db? | 21:13 |
| hemna | which will always fail | 21:13 |
| scottda | right, that ^^ | 21:13 |
| hemna | even if they fixed the bug I just filed. | 21:13 |
| hemna | completely hosed. | 21:13 |
| scottda | blahhh | 21:13 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group type and group specs https://review.openstack.org/320165 | 21:13 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 21:13 |
| hemna | leeantho, just alerted me to this when our CI started puking after the tempest test skip was removed :( | 21:13 |
| scottda | <rant> That's why cinder should not be in the business of keeping attach/detached state, which is only of interest to Nova </rant> | 21:14 |
| *** dustins has quit IRC | 21:14 | |
| *** jungleboyj has quit IRC | 21:14 | |
| hemna | now all of our CIs are failing 100% after 11:30pm last night (when the tempest skip was removed) | 21:14 |
| *** lkuchlan_ has joined #openstack-cinder | 21:14 | |
| *** alonma has joined #openstack-cinder | 21:14 | |
| *** alonmarx has joined #openstack-cinder | 21:14 | |
| hemna | I think there are several other drivers that have this same issue | 21:14 |
| smcginnis | scottda: I was holding my tongue on that one. ;) | 21:14 |
| hemna | if anyone accesses connector['host'] in terminate_connection, they are hosed. | 21:14 |
| smcginnis | hemna: We started failing over night. I'm pretty sure a few others as well. | 21:15 |
| hemna | I'm guessing it's the same issue | 21:15 |
| *** jwcroppe has joined #openstack-cinder | 21:15 | |
| *** jwcroppe has quit IRC | 21:15 | |
| hemna | this test shouldn't be run at all | 21:15 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 21:15 |
| hemna | nova's shelve/unshelve process is broken horribly | 21:15 |
| smcginnis | hemna: Revert the skip removal? | 21:15 |
| smcginnis | hemna: Got a link for it? | 21:16 |
| hemna | sec | 21:16 |
| hemna | https://github.com/openstack/tempest/commit/3f5aa985b5de6759cf361dfecb5f042a2949aac5 | 21:16 |
| scottda | BTW, Nova meeting is ongoing now, if there's anything here worth bringing up during open discussion. | 21:16 |
| hemna | https://review.openstack.org/#/c/339472/ | 21:16 |
| hemna | that needs to get reverted | 21:16 |
| hemna | I'll lurk in the meeting | 21:17 |
| scottda | I always lurk. But they know I'm not Nova, since I don't know the secret handshake.... | 21:18 |
| *** julim has quit IRC | 21:18 | |
| *** jwcroppe has joined #openstack-cinder | 21:18 | |
| hemna | I think this might have broken several 3rd party CIs | 21:18 |
| hemna | EMC | 21:18 |
| *** julim has joined #openstack-cinder | 21:18 | |
| hemna | their unmap_lun accesses connector['host'] | 21:18 |
| hemna | pewp! | 21:18 |
| *** alonma has quit IRC | 21:19 | |
| *** eharney has quit IRC | 21:19 | |
| *** alonma has joined #openstack-cinder | 21:21 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add group types and group specs https://review.openstack.org/320157 | 21:21 |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add generic volume groups https://review.openstack.org/322627 | 21:22 |
| *** julim has quit IRC | 21:23 | |
| openstackgerrit | xing-yang proposed openstack/python-cinderclient: Add support for group snapshots https://review.openstack.org/329770 | 21:23 |
| *** jwcroppe has quit IRC | 21:23 | |
| thiagolib | geguileo: hi i am trying to set the cinder - volume active / passive , and would like to know what I need to configure in cinder.conf and also how should I set the cinder - volume service to start on both nodes . | 21:25 |
| *** baumann_ has joined #openstack-cinder | 21:25 | |
| *** alonma has quit IRC | 21:26 | |
| *** ametts has quit IRC | 21:26 | |
| *** faiz89 has quit IRC | 21:27 | |
| *** dustins has joined #openstack-cinder | 21:27 | |
| *** baumann has quit IRC | 21:28 | |
| *** alonma has joined #openstack-cinder | 21:28 | |
| *** akerr_ has quit IRC | 21:30 | |
| *** xinli has joined #openstack-cinder | 21:31 | |
| hemna | #openstack-qa | 21:31 |
| hemna | err | 21:31 |
| *** akapil has joined #openstack-cinder | 21:32 | |
| *** lprice has joined #openstack-cinder | 21:32 | |
| *** alonma has quit IRC | 21:33 | |
| *** baumann_ has quit IRC | 21:36 | |
| *** baumann has joined #openstack-cinder | 21:36 | |
| *** alonma has joined #openstack-cinder | 21:39 | |
| hemna | scottda, https://review.openstack.org/#/c/257853 | 21:39 |
| hemna | that seems to fix it | 21:39 |
| hemna | at least in the short term | 21:39 |
| smcginnis | Looks promising | 21:40 |
| *** cdelatte has quit IRC | 21:41 | |
| scottda | hemna: Yeah, I was working with Andrea on this a while back....unfortunately, I think he's taking a new job in a week and I don't think he'll be working on Nova... | 21:41 |
| hemna | doh | 21:41 |
| scottda | so that patch might need a new owner. I'll figure that out and ping mriedem When I can confirm. | 21:41 |
| mriedem | hi-ya! | 21:42 |
| hemna | :) | 21:42 |
| mriedem | hemna: was your instance shelved or shelved-offloaded? | 21:42 |
| hemna | it was shelved | 21:42 |
| mriedem | shelved_offloaded takes a different path | 21:42 |
| hemna | oh sorry | 21:42 |
| hemna | SHELVED_OFFLOADED | 21:43 |
| scottda | mriedem: I was just pointing out ^^ that I think andrearosa may be moving to a new, Non-Nova job (not positive on this) and that patch might need a new owner...https://review.openstack.org/#/c/257853/6 | 21:43 |
| mriedem | scottda: yeah saw that | 21:43 |
| *** alonma has quit IRC | 21:43 | |
| mriedem | i'm trying to decide if we just get the quick fix in and handle the connector/state checking stuff in a follow up | 21:43 |
| mriedem | which is more complicated it sounds like | 21:44 |
| mriedem | apparnetly we don't actually attach the volume to the instance (initialize-connection) until after the server is unshelved | 21:44 |
| hemna | yah I just found that out too | 21:44 |
| scottda | right | 21:44 |
| hemna | during testing this | 21:44 |
| mriedem | that was definitely PaulMurray's team working on that | 21:44 |
| hemna | so we really can't call cinder terminate_connection in this case | 21:44 |
| hemna | most if not all cinder drivers will puke in that case | 21:45 |
| mriedem | well, | 21:45 |
| mriedem | the api code would check to see if bdm.connection_info['connector'[ | 21:45 |
| mriedem | if not there, we don't call terminate_connection | 21:45 |
| mriedem | which in the case of (1) old attachments or (2) shelved_offloaded instances, it won't be there | 21:45 |
| hemna | yah that's probably safe | 21:45 |
| hemna | it's better than passing a bogus connector. | 21:46 |
| *** alonma has joined #openstack-cinder | 21:46 | |
| mriedem | ok i'll update the patch quick | 21:48 |
| hemna | mriedem, thanks man | 21:49 |
| gouthamr | xyang1: https://review.openstack.org/#/c/346094/ updated after your review. Thanks! Would appreciate your re-review whenever you have time :) | 21:49 |
| xyang1 | gouthamr: sure | 21:50 |
| *** alonma has quit IRC | 21:51 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups https://review.openstack.org/350350 | 21:51 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Live Volume Autofailover https://review.openstack.org/350782 | 21:52 |
| jay-mehta | scottda, Yes, I freshly stack.sh. However I didn't pull latest devstack. That might be the reason | 21:53 |
| *** alonma has joined #openstack-cinder | 21:53 | |
| *** bill_az has quit IRC | 21:55 | |
| *** lkuchlan_ has quit IRC | 21:57 | |
| *** alonma has quit IRC | 21:58 | |
| *** alonmarx has quit IRC | 21:59 | |
| mriedem | hemna: so if you attached the volume when the instance was in shelved_offloaded state, and you didn't unshelve the instance, then when we call volume_api.detach - there aren't any issues? | 21:59 |
| mriedem | i guess that just does some cinder db cleanup work? | 21:59 |
| mriedem | and isn't a problem b/c we never actually initialized connection on the host | 21:59 |
| mriedem | just wondering why that doens't fail too | 21:59 |
| hemna | well, it worked :) | 21:59 |
| mriedem | bah | 22:00 |
| *** baumann has quit IRC | 22:00 | |
| hemna | and put the volume back in available state | 22:00 |
| mriedem | ok | 22:00 |
| mriedem | i'm assuming this is FC? | 22:00 |
| hemna | it's because the volume was never exported on the backend | 22:00 |
| hemna | and it was nothing more than a cinder DB entry | 22:00 |
| mriedem | because we never called initialize_connection right? | 22:00 |
| hemna | I tested iSCSI | 22:00 |
| hemna | yes | 22:00 |
| mriedem | ok | 22:00 |
| *** alonma has joined #openstack-cinder | 22:01 | |
| *** darosale has quit IRC | 22:01 | |
| mriedem | hemna: note we also ignore any error from cinder in the detach case there | 22:02 |
| mriedem | except Exception as exc: | 22:02 |
| mriedem | err_str = _LW("Ignoring volume cleanup failure due to %s") | 22:02 |
| mriedem | LOG.warn(err_str % exc, instance=instance) | 22:02 |
| *** alonmarx has joined #openstack-cinder | 22:02 | |
| hemna | heh yah | 22:03 |
| *** gouthamr has quit IRC | 22:05 | |
| *** xinli has quit IRC | 22:05 | |
| *** gouthamr has joined #openstack-cinder | 22:05 | |
| *** gouthamr has quit IRC | 22:05 | |
| *** alonma has quit IRC | 22:06 | |
| mriedem | ok, code change is made, just running unit tests quick, | 22:07 |
| mriedem | i'll need to write more, but you could test this out first | 22:07 |
| mriedem | if you can recreate quick | 22:07 |
| *** alonma has joined #openstack-cinder | 22:08 | |
| hemna | ok | 22:09 |
| mriedem | man, it's a good thing i was put through having to review that damn stashed connector change for like 3 weeks in jan/feb | 22:10 |
| *** karthikp_ has quit IRC | 22:10 | |
| hemna | :) | 22:10 |
| mriedem | i wouldn't have remembered that was there otherwise | 22:10 |
| mriedem | ok updated https://review.openstack.org/#/c/257853/ | 22:11 |
| *** rlrossit_ has quit IRC | 22:11 | |
| hemna | ok I'll pull that one and test the same scenario ? | 22:12 |
| mriedem | yeah | 22:12 |
| hemna | ok | 22:12 |
| *** alonma has quit IRC | 22:12 | |
| hemna | cinder never got called | 22:14 |
| hemna | and the volume is left in detaching | 22:14 |
| hemna | Ignoring volume cleanup failure due to <type 'NoneType'> can't be decoded | 22:14 |
| mriedem | hmm | 22:14 |
| mriedem | bdm.volume_id might not be set | 22:15 |
| *** alonma has joined #openstack-cinder | 22:15 | |
| mriedem | b/c we never attached | 22:15 |
| *** karthikp_ has joined #openstack-cinder | 22:16 | |
| mriedem | hmm no it should be set | 22:16 |
| mriedem | when the bdm was created | 22:16 |
| mriedem | i know what it is | 22:17 |
| mriedem | bdm.connection_info is None | 22:17 |
| hemna | I didn't see a log for "Found stashed volume connector...." | 22:17 |
| mriedem | ok, new patch set https://review.openstack.org/#/c/257853/ | 22:18 |
| hemna | I can dump the exception and see what the issue is | 22:18 |
| hemna | ok | 22:18 |
| mriedem | yeah it's bdm.connection_info was None | 22:18 |
| mriedem | and i was trying to json.loads it | 22:18 |
| *** alonma has quit IRC | 22:19 | |
| *** ducttape_ has quit IRC | 22:20 | |
| hemna | ok that worked | 22:20 |
| mriedem | \o/ | 22:20 |
| hemna | :) | 22:20 |
| mriedem | ok, i have to head home or else there will be trouble, | 22:20 |
| mriedem | but i'll add unit tests tonight | 22:21 |
| hemna | ok sounds good. thanks a bunch | 22:21 |
| mriedem | np | 22:21 |
| *** mriedem has quit IRC | 22:21 | |
| openstackgerrit | Drew Balfour proposed openstack/cinder: Change around the logic to correct the error message printed if both zfssa_initiator and zfssa_initiator_group are unset. Also clean up the logic around some other related error paths, and clean up some error messages themselves. https://review.openstack.org/351413 | 22:21 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Switch StrOpt to URIOpt for URL config options https://review.openstack.org/351414 | 22:22 |
| *** xyang1 has quit IRC | 22:22 | |
| *** alonma has joined #openstack-cinder | 22:22 | |
| *** alonmarx has quit IRC | 22:23 | |
| *** daneyon__ has joined #openstack-cinder | 22:24 | |
| *** alonma has quit IRC | 22:27 | |
| *** daneyon has quit IRC | 22:28 | |
| *** zul has joined #openstack-cinder | 22:28 | |
| *** daneyon__ has quit IRC | 22:28 | |
| openstackgerrit | Drew Balfour proposed openstack/cinder: zfssaiscsi driver initiator group option processing https://review.openstack.org/351413 | 22:28 |
| *** alonma has joined #openstack-cinder | 22:29 | |
| *** ntpttr has quit IRC | 22:31 | |
| *** zhipeng has quit IRC | 22:31 | |
| *** ntpttr has joined #openstack-cinder | 22:31 | |
| *** lpetrut has quit IRC | 22:32 | |
| *** alonma has quit IRC | 22:34 | |
| *** edmondsw has quit IRC | 22:40 | |
| *** alonma has joined #openstack-cinder | 22:41 | |
| scottda | jgriffith: the 'cinder api-version' command will only work on /v3 enpoint | 22:44 |
| scottda | jgriffith: You'll need '--os-volume-api-version 3.0' or >3.0 | 22:44 |
| *** alonma has quit IRC | 22:46 | |
| scottda | Ugghh..just noticed the the help sucks: | 22:46 |
| scottda | Display the API version information. | 22:46 |
| scottda | Should be: Display the API version on the Server | 22:46 |
| *** alonma has joined #openstack-cinder | 22:49 | |
| *** crose has quit IRC | 22:50 | |
| *** dustins has quit IRC | 22:51 | |
| *** sdague has quit IRC | 22:52 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Change api-version help to indicate server API https://review.openstack.org/351423 | 22:52 |
| *** alonma has quit IRC | 22:53 | |
| jgriffith | scottda: thanks | 22:56 |
| jgriffith | scottda: wondering if we should implement v2 versions for that? | 22:56 |
| *** alonma has joined #openstack-cinder | 22:56 | |
| *** alonma has quit IRC | 23:01 | |
| *** seldenr has quit IRC | 23:02 | |
| scottda | Yeah, we could. The endpoint is there, on the server. | 23:03 |
| *** alonma has joined #openstack-cinder | 23:03 | |
| *** alonma has quit IRC | 23:08 | |
| *** alonma has joined #openstack-cinder | 23:11 | |
| *** catintheroof has quit IRC | 23:13 | |
| *** alonma has quit IRC | 23:15 | |
| *** akapil has quit IRC | 23:18 | |
| *** alonma has joined #openstack-cinder | 23:18 | |
| *** zouyapeng has quit IRC | 23:22 | |
| *** alonma has quit IRC | 23:22 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to argparse https://review.openstack.org/340129 | 23:23 |
| *** alonma has joined #openstack-cinder | 23:25 | |
| jgriffith | scottda: i guess my point of the bug was just that "how would a user know why that didn't work" | 23:28 |
| *** markvoelker has joined #openstack-cinder | 23:28 | |
| *** alonma has quit IRC | 23:29 | |
| *** jamielennox is now known as jamielennox|away | 23:30 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to argparse https://review.openstack.org/340129 | 23:31 |
| *** alonma has joined #openstack-cinder | 23:32 | |
| *** alonma has quit IRC | 23:36 | |
| *** markvoelker_ has joined #openstack-cinder | 23:37 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 23:39 |
| *** alonma has joined #openstack-cinder | 23:40 | |
| *** markvoelker has quit IRC | 23:41 | |
| *** jamielennox|away is now known as jamielennox | 23:41 | |
| *** ducttape_ has joined #openstack-cinder | 23:41 | |
| scottda | jgriffith: yeah, see those 2 patches above from ntpttr . We should figure out a general easy to deal with the issue of unsupported features due to API version. | 23:41 |
| ntpttr | scottda: I just left a comment on https://review.openstack.org/#/c/319501/7 related to that | 23:42 |
| scottda | Yup. | 23:42 |
| ntpttr | well not related to that actually, but related to working out how to merge all those fixes | 23:42 |
| *** leeantho has quit IRC | 23:42 | |
| ntpttr | scottda: I believe that the way he's set up this exception it will deal with it pretty well if I'm not mistaken https://review.openstack.org/#/c/309283/23/cinderclient/exceptions.py | 23:43 |
| *** sdake has joined #openstack-cinder | 23:44 | |
| scottda | Yeah, maybe that's the way we want to go. I'm afk ATM ( cooking tacos) so I'll look later. | 23:45 |
| *** EinstCrazy has joined #openstack-cinder | 23:45 | |
| *** alonma has quit IRC | 23:45 | |
| *** catintheroof has joined #openstack-cinder | 23:46 | |
| *** sdake_ has quit IRC | 23:46 | |
| *** alonma has joined #openstack-cinder | 23:47 | |
| ntpttr | scottda: sounds good, enjoy the tacos | 23:48 |
| *** alonma has quit IRC | 23:52 | |
| *** haplo37__ has joined #openstack-cinder | 23:53 | |
| *** EinstCrazy has quit IRC | 23:53 | |
| *** lixiaoy1 has quit IRC | 23:54 | |
| *** ducttape_ has quit IRC | 23:56 | |
| *** guy__ has joined #openstack-cinder | 23:59 | |
| *** alonma has joined #openstack-cinder | 23:59 | |
| *** laughterwym has joined #openstack-cinder | 23:59 | |
| *** zul has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!