| *** markvoelker has quit IRC | 00:02 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 00:02 |
|---|---|---|
| *** changbl has joined #openstack-cinder | 00:03 | |
| *** garthb_ has quit IRC | 00:20 | |
| *** garthb has quit IRC | 00:20 | |
| *** IlyaG_ has quit IRC | 00:23 | |
| *** salv-orlando has joined #openstack-cinder | 00:25 | |
| *** IlyaG has joined #openstack-cinder | 00:28 | |
| *** IlyaG has quit IRC | 00:28 | |
| *** salv-orlando has quit IRC | 00:29 | |
| *** mriedem has joined #openstack-cinder | 00:30 | |
| *** mriedem has quit IRC | 00:30 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:05 | |
| *** vilobhmm has quit IRC | 01:06 | |
| *** boris-42 has quit IRC | 01:10 | |
| *** zhonghua-lee has joined #openstack-cinder | 01:11 | |
| *** davechen has joined #openstack-cinder | 01:20 | |
| *** zhenguo has joined #openstack-cinder | 01:23 | |
| *** davechen1 has joined #openstack-cinder | 01:23 | |
| *** davechen has quit IRC | 01:25 | |
| *** topshare has joined #openstack-cinder | 01:28 | |
| *** p0rtal has joined #openstack-cinder | 01:29 | |
| *** salv-orlando has joined #openstack-cinder | 01:37 | |
| *** smoriya has joined #openstack-cinder | 01:37 | |
| *** boris-42 has joined #openstack-cinder | 01:38 | |
| *** Yogi1 has joined #openstack-cinder | 01:38 | |
| *** salv-orlando has quit IRC | 01:42 | |
| *** salv-orlando has joined #openstack-cinder | 01:45 | |
| *** salv-orlando has quit IRC | 01:49 | |
| *** zhenguo has quit IRC | 01:53 | |
| *** rodrigod` has joined #openstack-cinder | 01:54 | |
| *** rodrigod` is now known as rodrigods | 01:57 | |
| *** markvoelker has joined #openstack-cinder | 01:58 | |
| *** Yogi1 has quit IRC | 01:59 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 02:01 |
| *** 6JTABTZX7 has joined #openstack-cinder | 02:02 | |
| *** p0rtal has quit IRC | 02:02 | |
| *** markvoelker has quit IRC | 02:03 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 02:05 |
| *** Lee1092 has joined #openstack-cinder | 02:10 | |
| openstackgerrit | wanghao proposed openstack/cinder: Validate value when user update quota https://review.openstack.org/197938 | 02:13 |
| openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of listing detail https://review.openstack.org/202029 | 02:16 |
| *** logan2 has quit IRC | 02:18 | |
| *** dflorea has joined #openstack-cinder | 02:23 | |
| *** dflorea has quit IRC | 02:27 | |
| *** markvoelker has joined #openstack-cinder | 02:31 | |
| *** ebalduf has joined #openstack-cinder | 02:35 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add the ability to update type public status https://review.openstack.org/206788 | 02:39 |
| *** haigang has joined #openstack-cinder | 02:39 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Return multiple iSCSI portals in VNX Cinder driver https://review.openstack.org/213583 | 02:43 |
| *** haigang has quit IRC | 02:46 | |
| *** laughterwym_ has joined #openstack-cinder | 02:50 | |
| *** hakimo_ has joined #openstack-cinder | 02:52 | |
| *** laughterwym has quit IRC | 02:53 | |
| *** hakimo has quit IRC | 02:54 | |
| *** laughterwym_ has quit IRC | 02:55 | |
| *** laughterwym has joined #openstack-cinder | 02:55 | |
| *** logan2 has joined #openstack-cinder | 02:57 | |
| *** 6JTABTZX7 has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:02 | |
| lixiaoy1 | :winston-d_zZZ here? | 03:29 |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 03:47 |
| *** ebalduf has quit IRC | 03:50 | |
| *** geguileo has quit IRC | 03:52 | |
| *** salv-orlando has joined #openstack-cinder | 03:54 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 03:55 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 03:58 |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** gouthamr has quit IRC | 04:02 | |
| *** changbl has quit IRC | 04:04 | |
| *** salv-orlando has quit IRC | 04:06 | |
| *** geguileo has joined #openstack-cinder | 04:08 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 04:10 |
| *** openstack has joined #openstack-cinder | 04:18 | |
| openstackgerrit | wanghao proposed openstack/cinder: Validate value when user update quota https://review.openstack.org/197938 | 04:21 |
| *** lixiaoy1 has quit IRC | 04:24 | |
| *** zhonghua-lee has quit IRC | 04:30 | |
| *** diogogmt has quit IRC | 04:30 | |
| *** markvoelker has quit IRC | 04:31 | |
| *** zhonghua-lee has joined #openstack-cinder | 04:34 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:41 | |
| *** BharatK has joined #openstack-cinder | 04:43 | |
| *** merooney has quit IRC | 04:44 | |
| *** baojg has joined #openstack-cinder | 04:48 | |
| *** haomaiwa_ has joined #openstack-cinder | 04:49 | |
| *** haomaiwang has quit IRC | 04:49 | |
| *** deepakcs has joined #openstack-cinder | 04:57 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:02 | |
| *** shyama has joined #openstack-cinder | 05:04 | |
| *** zhonghua-lee has quit IRC | 05:10 | |
| *** daneyon_ has quit IRC | 05:13 | |
| *** daneyon has joined #openstack-cinder | 05:14 | |
| *** shausy has joined #openstack-cinder | 05:24 | |
| *** ankit_ag has joined #openstack-cinder | 05:26 | |
| *** nkrinner has joined #openstack-cinder | 05:27 | |
| *** markvoelker has joined #openstack-cinder | 05:32 | |
| *** coolsvap|away is now known as coolsvap | 05:32 | |
| *** markvoelker has quit IRC | 05:36 | |
| *** shausy has quit IRC | 05:38 | |
| *** shausy has joined #openstack-cinder | 05:39 | |
| *** edtubill has joined #openstack-cinder | 05:42 | |
| *** edtubill has left #openstack-cinder | 05:42 | |
| *** salv-orlando has joined #openstack-cinder | 05:44 | |
| *** jgregor has joined #openstack-cinder | 05:46 | |
| *** jgregor has quit IRC | 05:48 | |
| *** p0rtal has joined #openstack-cinder | 05:49 | |
| *** alexpilotti has quit IRC | 05:50 | |
| *** salv-orlando has quit IRC | 05:51 | |
| *** coolsvap is now known as coolsvap|away | 05:55 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:02 | |
| *** afazekas has quit IRC | 06:10 | |
| *** agarciam has joined #openstack-cinder | 06:11 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 06:14 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the target volume https://review.openstack.org/210237 | 06:17 |
| *** vlaza has joined #openstack-cinder | 06:18 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 06:19 |
| *** p0rtal has quit IRC | 06:21 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 06:22 |
| *** p0rtal has joined #openstack-cinder | 06:22 | |
| *** p0rtal_ has joined #openstack-cinder | 06:23 | |
| *** salv-orlando has joined #openstack-cinder | 06:23 | |
| *** p0rtal has quit IRC | 06:26 | |
| *** anshul has joined #openstack-cinder | 06:28 | |
| *** laughterwym has quit IRC | 06:29 | |
| *** baojg has quit IRC | 06:39 | |
| *** p0rtal_ has quit IRC | 06:48 | |
| *** zhenguo has joined #openstack-cinder | 06:49 | |
| *** sileht has quit IRC | 06:55 | |
| *** sileht has joined #openstack-cinder | 06:56 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** afazekas_ has joined #openstack-cinder | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:02 | |
| *** baojg has joined #openstack-cinder | 07:03 | |
| -openstackstatus- NOTICE: Gerrit is currently under very high load and may be unresponsive. infra are looking into the issue. | 07:05 | |
| *** markus_z has joined #openstack-cinder | 07:14 | |
| *** ronis has joined #openstack-cinder | 07:15 | |
| *** ronis_ has joined #openstack-cinder | 07:18 | |
| *** zhenguo has quit IRC | 07:18 | |
| *** ronis has quit IRC | 07:21 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Prevent uploading encrypted volume to images https://review.openstack.org/213616 | 07:26 |
| *** markvoelker has joined #openstack-cinder | 07:33 | |
| *** markvoelker has quit IRC | 07:37 | |
| *** zhenguo has joined #openstack-cinder | 07:39 | |
| *** haomaiwa_ has joined #openstack-cinder | 07:40 | |
| *** haomaiwang has quit IRC | 07:40 | |
| openstackgerrit | wanghao proposed openstack/cinder: Forbid creating volume from image which size is 0 https://review.openstack.org/207696 | 07:44 |
| *** IanGovett has joined #openstack-cinder | 07:50 | |
| *** haomaiwa_ has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:02 | |
| *** zhenguo has quit IRC | 08:04 | |
| *** sgotliv has joined #openstack-cinder | 08:09 | |
| openstackgerrit | YAMADA Hideki proposed openstack/cinder: Add backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 08:09 |
| *** shausy has quit IRC | 08:10 | |
| *** shausy has joined #openstack-cinder | 08:10 | |
| *** afazekas_ is now known as afazkas | 08:12 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 08:17 |
| *** jistr has joined #openstack-cinder | 08:21 | |
| *** baojg has quit IRC | 08:23 | |
| *** zhenguo has joined #openstack-cinder | 08:29 | |
| *** yrabl has joined #openstack-cinder | 08:32 | |
| *** sgotliv has quit IRC | 08:38 | |
| *** yrabl_ has joined #openstack-cinder | 08:38 | |
| *** baojg has joined #openstack-cinder | 08:39 | |
| *** haomaiwang has quit IRC | 08:43 | |
| *** haomaiwang has joined #openstack-cinder | 08:44 | |
| *** yrabl has quit IRC | 08:45 | |
| *** adiantum_ has quit IRC | 08:45 | |
| *** yrabl has joined #openstack-cinder | 08:45 | |
| *** davechen1 has left #openstack-cinder | 08:46 | |
| *** p0rtal has joined #openstack-cinder | 08:49 | |
| openstackgerrit | Lei Zhang proposed openstack/cinder: Prevent concurrent backups for same volume https://review.openstack.org/202914 | 08:50 |
| *** sgotliv has joined #openstack-cinder | 08:52 | |
| *** p0rtal has quit IRC | 08:53 | |
| *** lixiaoy1 has quit IRC | 08:54 | |
| *** aarefiev has quit IRC | 08:56 | |
| *** jordanP has joined #openstack-cinder | 08:57 | |
| *** aarefiev has joined #openstack-cinder | 08:58 | |
| *** aarefiev has quit IRC | 08:59 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 08:59 |
| *** aarefiev has joined #openstack-cinder | 08:59 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:02 | |
| *** sgotliv has quit IRC | 09:02 | |
| *** shausy has quit IRC | 09:06 | |
| tbarron | thingee: I'll probably have to carry https://review.openstack.org/#/c/203291/ to mitaka since I've been assigned other work that I have to complete in L3 and likely won't have time. | 09:09 |
| *** kbyrne has quit IRC | 09:12 | |
| *** yrabl_ has joined #openstack-cinder | 09:12 | |
| *** sgotliv has joined #openstack-cinder | 09:14 | |
| *** yrabl_ has quit IRC | 09:16 | |
| *** kbyrne has joined #openstack-cinder | 09:17 | |
| *** topshare has quit IRC | 09:22 | |
| *** zhonghua-lee has joined #openstack-cinder | 09:25 | |
| *** vincent_hou has joined #openstack-cinder | 09:33 | |
| *** markvoelker has joined #openstack-cinder | 09:33 | |
| vincent_hou | winston-d_zZZ: Hi Winston. I have resolved the issue for the 'maintenance' volumes: https://review.openstack.org/186312. | 09:35 |
| vincent_hou | smcginnis: HI, I have resolved the comments from you Thx. https://review.openstack.org/186312 | 09:36 |
| *** vincent_hou has quit IRC | 09:38 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/os-brick: Fix incorrect comments in FibreChannelConnector https://review.openstack.org/209762 | 09:38 |
| *** markvoelker has quit IRC | 09:38 | |
| *** p0rtal has joined #openstack-cinder | 09:49 | |
| *** p0rtal has quit IRC | 09:55 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** yrabl_ has joined #openstack-cinder | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:02 | |
| *** dims has joined #openstack-cinder | 10:17 | |
| -openstackstatus- NOTICE: review.openstack.org (aka gerrit) is going down for an emergency restart | 10:20 | |
| *** ChanServ changes topic to "review.openstack.org (aka gerrit) is going down for an emergency restart" | 10:20 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/os-brick: Fix incorrect comments in FibreChannelConnector https://review.openstack.org/209762 | 10:22 |
| *** baojg has quit IRC | 10:24 | |
| *** baojg has joined #openstack-cinder | 10:25 | |
| *** dims has quit IRC | 10:30 | |
| *** dims has joined #openstack-cinder | 10:31 | |
| *** dims_ has joined #openstack-cinder | 10:36 | |
| *** dims has quit IRC | 10:38 | |
| *** erlon has joined #openstack-cinder | 10:43 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:44 | |
| *** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder" | 10:46 | |
| -openstackstatus- NOTICE: Gerrit restart has resolved the issue and systems are back up and functioning | 10:47 | |
| *** baojg has quit IRC | 10:48 | |
| *** dims_ has quit IRC | 10:50 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Bump driver version https://review.openstack.org/213659 | 10:50 |
| *** dims has joined #openstack-cinder | 10:51 | |
| *** deepakcs has quit IRC | 10:53 | |
| openstackgerrit | Rajesh Tailor proposed openstack/cinder: Fix Bad indentation pylint issues https://review.openstack.org/189277 | 10:58 |
| *** dims has quit IRC | 10:59 | |
| *** daneyon has quit IRC | 10:59 | |
| *** daneyon has joined #openstack-cinder | 11:00 | |
| *** dims has joined #openstack-cinder | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** sgotliv has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:02 | |
| *** lpetrut has joined #openstack-cinder | 11:03 | |
| *** leopoldj has joined #openstack-cinder | 11:04 | |
| *** marcusvrn has joined #openstack-cinder | 11:05 | |
| *** dims has quit IRC | 11:06 | |
| *** dims has joined #openstack-cinder | 11:08 | |
| *** boris-42 has quit IRC | 11:10 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 11:11 |
| *** dims_ has joined #openstack-cinder | 11:12 | |
| *** dims has quit IRC | 11:12 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 11:13 |
| *** cdelatte has joined #openstack-cinder | 11:15 | |
| *** delattec has joined #openstack-cinder | 11:15 | |
| *** dims_ has quit IRC | 11:17 | |
| *** dims has joined #openstack-cinder | 11:22 | |
| *** sgotliv has joined #openstack-cinder | 11:23 | |
| *** sakaYK has joined #openstack-cinder | 11:23 | |
| *** sakaYK has quit IRC | 11:23 | |
| *** delattec has quit IRC | 11:26 | |
| *** cdelatte has quit IRC | 11:26 | |
| *** dims has quit IRC | 11:27 | |
| *** bluex-pl has joined #openstack-cinder | 11:28 | |
| *** bluex-pl has quit IRC | 11:28 | |
| *** bluex-pl has joined #openstack-cinder | 11:28 | |
| *** ganso_ has joined #openstack-cinder | 11:28 | |
| *** dims has joined #openstack-cinder | 11:29 | |
| *** zhonghua-lee has quit IRC | 11:29 | |
| *** smoriya has quit IRC | 11:30 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Adapt SnapshotController to view builder https://review.openstack.org/211592 | 11:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add pagination to snapshots https://review.openstack.org/195071 | 11:31 |
| *** markvoelker has joined #openstack-cinder | 11:34 | |
| *** dims_ has joined #openstack-cinder | 11:34 | |
| *** dims has quit IRC | 11:35 | |
| *** haomaiwang has quit IRC | 11:36 | |
| *** markvoelker has quit IRC | 11:39 | |
| *** dims_ has quit IRC | 11:39 | |
| *** dims has joined #openstack-cinder | 11:40 | |
| *** zhonghua-lee has joined #openstack-cinder | 11:40 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Change inventory folder hierarchy https://review.openstack.org/193095 | 11:42 |
| *** sgotliv has quit IRC | 11:42 | |
| *** leopoldj has quit IRC | 11:44 | |
| *** BharatK has quit IRC | 11:45 | |
| *** dims has quit IRC | 11:49 | |
| *** p0rtal has joined #openstack-cinder | 11:49 | |
| *** dims has joined #openstack-cinder | 11:53 | |
| *** p0rtal has quit IRC | 11:54 | |
| *** dims_ has joined #openstack-cinder | 11:56 | |
| *** sgotliv has joined #openstack-cinder | 11:57 | |
| *** dims has quit IRC | 11:58 | |
| *** chlong has quit IRC | 11:59 | |
| *** sgotliv has quit IRC | 11:59 | |
| *** sgotliv has joined #openstack-cinder | 12:00 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject https://review.openstack.org/198341 | 12:01 |
| *** dims_ has quit IRC | 12:04 | |
| *** dims has joined #openstack-cinder | 12:04 | |
| *** JoseMello has joined #openstack-cinder | 12:07 | |
| *** dims_ has joined #openstack-cinder | 12:08 | |
| *** dims has quit IRC | 12:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/210918 | 12:11 |
| *** dims has joined #openstack-cinder | 12:12 | |
| *** dims_ has quit IRC | 12:14 | |
| *** agarciam has quit IRC | 12:14 | |
| *** boichev has joined #openstack-cinder | 12:15 | |
| *** zhonghua-lee has quit IRC | 12:15 | |
| *** dims_ has joined #openstack-cinder | 12:18 | |
| *** edmondsw has joined #openstack-cinder | 12:18 | |
| *** dims has quit IRC | 12:18 | |
| *** dims has joined #openstack-cinder | 12:19 | |
| *** dims_ has quit IRC | 12:23 | |
| *** dims_ has joined #openstack-cinder | 12:26 | |
| *** dims has quit IRC | 12:26 | |
| *** dims has joined #openstack-cinder | 12:30 | |
| *** dims_ has quit IRC | 12:31 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with ConsistencyGroups https://review.openstack.org/207544 | 12:32 |
| *** dims has quit IRC | 12:35 | |
| *** dims has joined #openstack-cinder | 12:35 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete-wait-loop for CloudByte Volumes https://review.openstack.org/201059 | 12:38 |
| *** kjnelson has joined #openstack-cinder | 12:38 | |
| *** dims_ has joined #openstack-cinder | 12:38 | |
| *** dims has quit IRC | 12:40 | |
| *** edmondsw_ has joined #openstack-cinder | 12:41 | |
| *** ekarlso has quit IRC | 12:44 | |
| *** ekarlso has joined #openstack-cinder | 12:44 | |
| *** dims has joined #openstack-cinder | 12:45 | |
| *** dims_ has quit IRC | 12:45 | |
| *** edmondsw_ has quit IRC | 12:48 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers. https://review.openstack.org/197999 | 12:49 |
| *** akerr has joined #openstack-cinder | 12:53 | |
| *** xiaohui has quit IRC | 12:53 | |
| *** gouthamr has joined #openstack-cinder | 12:54 | |
| *** dims has quit IRC | 12:55 | |
| *** porrua has joined #openstack-cinder | 12:56 | |
| *** nzrf has joined #openstack-cinder | 12:57 | |
| *** raildo-afk is now known as raildo | 12:57 | |
| *** shikel has joined #openstack-cinder | 12:58 | |
| *** xiaohui has joined #openstack-cinder | 12:58 | |
| *** dims has joined #openstack-cinder | 12:58 | |
| *** eharney has joined #openstack-cinder | 12:58 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 13:03 |
| *** Adriano__ has joined #openstack-cinder | 13:03 | |
| *** xyang1 has joined #openstack-cinder | 13:05 | |
| *** lifeless has quit IRC | 13:05 | |
| *** dims has quit IRC | 13:06 | |
| *** merooney has joined #openstack-cinder | 13:08 | |
| *** dims has joined #openstack-cinder | 13:09 | |
| *** zhonghua-lee has joined #openstack-cinder | 13:12 | |
| *** haomaiwa_ has joined #openstack-cinder | 13:13 | |
| *** dustins has joined #openstack-cinder | 13:13 | |
| *** dims has quit IRC | 13:14 | |
| *** dims has joined #openstack-cinder | 13:14 | |
| *** mriedem has joined #openstack-cinder | 13:18 | |
| *** sayali has quit IRC | 13:20 | |
| *** jgregor has joined #openstack-cinder | 13:20 | |
| *** dims has quit IRC | 13:21 | |
| *** dustins has quit IRC | 13:21 | |
| *** martyturner has joined #openstack-cinder | 13:22 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 13:23 |
| *** leakypipes is now known as jaypipes | 13:25 | |
| *** sayali has joined #openstack-cinder | 13:25 | |
| *** BharatK has joined #openstack-cinder | 13:26 | |
| *** simondodsley has joined #openstack-cinder | 13:26 | |
| *** dustins has joined #openstack-cinder | 13:26 | |
| *** dims has joined #openstack-cinder | 13:27 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Port Huawei driver to use abc https://review.openstack.org/213705 | 13:28 |
| *** lifeless has joined #openstack-cinder | 13:28 | |
| *** gman-tx has quit IRC | 13:28 | |
| *** cdelatte has joined #openstack-cinder | 13:29 | |
| *** delattec has joined #openstack-cinder | 13:29 | |
| *** julim has joined #openstack-cinder | 13:31 | |
| *** thangp has joined #openstack-cinder | 13:31 | |
| *** Yogi1 has joined #openstack-cinder | 13:35 | |
| *** markvoelker has joined #openstack-cinder | 13:35 | |
| *** dims has quit IRC | 13:36 | |
| *** krtaylor has quit IRC | 13:36 | |
| *** jecarey has joined #openstack-cinder | 13:38 | |
| *** markvoelker has quit IRC | 13:39 | |
| *** diogogmt has joined #openstack-cinder | 13:41 | |
| *** dims has joined #openstack-cinder | 13:42 | |
| *** changbl has joined #openstack-cinder | 13:43 | |
| *** rmart04 has joined #openstack-cinder | 13:44 | |
| rmart04 | Afternoon! I have a couple of questions regarding OpenStack Cinder Volume service. Does anyone know how these act running multiple Cinder-Volume services in parallel? And how would that work if I plugged 2/3 of them into the same Ceph Cluster, pointed at the same RBD Pools? | 13:45 |
| *** xiaohui has quit IRC | 13:46 | |
| *** xiaohui has joined #openstack-cinder | 13:46 | |
| *** dims has quit IRC | 13:48 | |
| *** zhonghua-lee has quit IRC | 13:48 | |
| *** p0rtal has joined #openstack-cinder | 13:49 | |
| *** krtaylor has joined #openstack-cinder | 13:49 | |
| *** jungleboyj has joined #openstack-cinder | 13:49 | |
| *** mylu has joined #openstack-cinder | 13:50 | |
| *** xyang1 has quit IRC | 13:52 | |
| *** xyang1 has joined #openstack-cinder | 13:52 | |
| jbernard | rmart04: should work :) | 13:54 |
| *** p0rtal has quit IRC | 13:54 | |
| jbernard | rmart04: as long as the hostname setting in cinder.conf is unique for each cinder-volume instance | 13:54 |
| *** dims has joined #openstack-cinder | 13:54 | |
| *** diogogmt has quit IRC | 13:54 | |
| *** bswartz has quit IRC | 13:56 | |
| rmart04 | OK nice! So would each cinder volume instance end up with their own message queue, which could only be serviced by that node? eg create volume in CV1, CV2 cannot remove it? Or could either service manage the volumes the other creates? | 13:58 |
| *** zul has joined #openstack-cinder | 13:58 | |
| mriedem | cinder merged an api change which breaks a nova boot from volume scenario | 13:59 |
| mriedem | https://review.openstack.org/#/c/182994/17/cinder/api/openstack/wsgi.py | 13:59 |
| *** sakaYK has joined #openstack-cinder | 13:59 | |
| *** dims has quit IRC | 13:59 | |
| mriedem | you used to be able to create a volume from image/snapshot with 0 length name | 13:59 |
| *** breitz has joined #openstack-cinder | 14:00 | |
| mriedem | ^ requires the name to be at least 1 character | 14:00 |
| *** boris-42 has joined #openstack-cinder | 14:00 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1485198 | 14:00 |
| openstack | Launchpad bug 1485198 in OpenStack Compute (nova) "Cannot boot if bdm specifies to create a new volume from an image or a snapshot" [Undecided,In progress] - Assigned to Feodor Tersin (ftersin) | 14:00 |
| johnthetubaguy | nasty :( | 14:00 |
| mriedem | i understand the max limit of 255 chars | 14:00 |
| *** haomaiwa_ has quit IRC | 14:01 | |
| mriedem | but the min of 1 on the name is a backwards incompatible api change | 14:01 |
| jgriffith | mriedem: I agree, also name always has and should remain optional | 14:01 |
| jgriffith | Looking at patch... | 14:01 |
| *** mylu has quit IRC | 14:02 | |
| *** Yogi11 has joined #openstack-cinder | 14:02 | |
| *** haomaiwang has joined #openstack-cinder | 14:02 | |
| *** mylu has joined #openstack-cinder | 14:02 | |
| *** ftersin has joined #openstack-cinder | 14:02 | |
| mriedem | if cinder wants to enforce different rules in a v2.1 API or something, that's fine, but otherwise i think that min check has to be 0 for name so it's backwards compatible | 14:02 |
| *** frickler has joined #openstack-cinder | 14:03 | |
| jgriffith | mriedem: understood, but my point was that requiring a name is unacceptable anyway; and I believe the folks that reviewed that patch just sort of missed that nuance | 14:03 |
| *** chlong has joined #openstack-cinder | 14:04 | |
| jgriffith | mriedem: anyway, looking at a fix now | 14:04 |
| *** dims has joined #openstack-cinder | 14:04 | |
| *** Yogi1 has quit IRC | 14:04 | |
| mriedem | jgriffith: ftersin had a nova change to workaround it, but i think that's wrong, | 14:04 |
| mriedem | we can fix the min=1 in the cinder wsgi validation method for 'name' | 14:04 |
| johnthetubaguy | mriedem: certainly for the existing cinder API users | 14:04 |
| *** mylu has quit IRC | 14:04 | |
| johnthetubaguy | but I could be missing something | 14:05 |
| jgriffith | mriedem: I agree, needs fixed in Cinder | 14:05 |
| johnthetubaguy | (I guess we would use API microversions for this in Nova now, although thats kinda cheating) | 14:05 |
| mriedem | jgriffith: johnthetubaguy: ftersin: ok, i'll push up a cinder change quick | 14:05 |
| *** dustins has quit IRC | 14:06 | |
| *** bill_az has joined #openstack-cinder | 14:06 | |
| jgriffith | I already have one, just filing a bug for it now | 14:06 |
| ftersin | jgriffith: you could use 1485198 | 14:07 |
| jgriffith | ftersin: cool | 14:08 |
| *** dustins has joined #openstack-cinder | 14:08 | |
| *** mylu has joined #openstack-cinder | 14:08 | |
| *** rushil has joined #openstack-cinder | 14:08 | |
| jgriffith | TBF I think this was overlooked because folks assumed that the case that's hit here would fall under the None check | 14:09 |
| *** mylu has quit IRC | 14:10 | |
| jgriffith | Anyway.... | 14:10 |
| *** mylu has joined #openstack-cinder | 14:11 | |
| *** dims has quit IRC | 14:11 | |
| *** boichev has quit IRC | 14:11 | |
| jgriffith | running tests now... | 14:12 |
| jgriffith | ahh.. vm only has one core | 14:12 |
| openstackgerrit | John Griffith proposed openstack/cinder: Allow 0 length name https://review.openstack.org/213723 | 14:12 |
| guitarzan | jgriffith: I should've -1ed "Please add tests" :D | 14:14 |
| * jgriffith beats guitarzan over the head with a feather | 14:14 | |
| guitarzan | although if someone tries this again, I should absolutely add a blank name test | 14:15 |
| jgriffith | guitarzan: Yeah, but you know they'll just modify the test | 14:15 |
| guitarzan | true, but at least no one can claim ignorance | 14:15 |
| jgriffith | guitarzan: hack hack hack... hey, this stupid test doesn't work now; here let me fix it. | 14:15 |
| jgriffith | guitarzan: +1 | 14:16 |
| *** dims has joined #openstack-cinder | 14:16 | |
| jgriffith | guitarzan: I'll have an update in a moment with a test BTW | 14:17 |
| *** ankit_ag has quit IRC | 14:17 | |
| jgriffith | guitarzan: also the original patch better fail for a test of that case anyway :) | 14:17 |
| jgriffith | if not... then Busted!!!!!!! | 14:17 |
| *** liuxg has joined #openstack-cinder | 14:18 | |
| eharney | jgriffith: it does, i wondered that same thing and checked | 14:19 |
| *** bswartz has joined #openstack-cinder | 14:21 | |
| *** dims has quit IRC | 14:22 | |
| *** mylu has quit IRC | 14:22 | |
| *** yrabl has quit IRC | 14:22 | |
| *** mylu has joined #openstack-cinder | 14:22 | |
| jgriffith | eharney: HA! Yeah, I downloaded it to my other machine and ran it about the same time. | 14:22 |
| jgriffith | eharney: fix on the way; thanks | 14:22 |
| jgriffith | just running tests again now to make sure | 14:22 |
| jgriffith | consistency group tests fail as well | 14:24 |
| openstackgerrit | John Griffith proposed openstack/cinder: Allow 0 length name https://review.openstack.org/213723 | 14:25 |
| xyang1 | jgriffith: Does it have to do with notification again? | 14:25 |
| jgriffith | xyang1: no no... the test :_ | 14:25 |
| jgriffith | :) | 14:25 |
| jgriffith | https://review.openstack.org/#/c/213723/2/cinder/tests/unit/api/contrib/test_consistencygroups.py | 14:26 |
| xyang1 | jgriffith: Ok | 14:26 |
| jgriffith | It should fail... it's a good test :) | 14:26 |
| jgriffith | a good failure | 14:26 |
| jgriffith | you know what I mean | 14:26 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 14:27 |
| xyang1 | jgriffith: Ok, this might be for renaming | 14:27 |
| *** dflorea has joined #openstack-cinder | 14:31 | |
| *** afazkas has quit IRC | 14:31 | |
| *** mylu has quit IRC | 14:32 | |
| *** mylu has joined #openstack-cinder | 14:33 | |
| smcginnis | jgriffith: Might be worth having a unit test and putting a comment in there that it MUST allow zero length names. | 14:34 |
| *** Yogi1 has joined #openstack-cinder | 14:34 | |
| smcginnis | Otherwise I don't think many are aware there is a requirement for that. | 14:34 |
| *** Yogi11 has quit IRC | 14:36 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Allow 0 length name https://review.openstack.org/213723 | 14:37 |
| smcginnis | Awesome. | 14:38 |
| *** tsekiyama has joined #openstack-cinder | 14:38 | |
| *** liuxg has quit IRC | 14:39 | |
| *** ronis_ has quit IRC | 14:39 | |
| *** tsbot_ has joined #openstack-cinder | 14:39 | |
| *** vlaza has quit IRC | 14:40 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 14:41 |
| *** mtanino has joined #openstack-cinder | 14:42 | |
| ftersin | jgriffith: fyi, we're going to add a tempest test for that: https://review.openstack.org/#/c/213621/ | 14:42 |
| *** dflorea has quit IRC | 14:42 | |
| jgriffith | ftersin: excellent! | 14:42 |
| *** baojg has joined #openstack-cinder | 14:43 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 14:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make migration's volume source deletion async https://review.openstack.org/211257 | 14:45 |
| jordanP | ftersin, ping ? | 14:46 |
| jordanP | ftersin, how is https://review.openstack.org/#/c/213621/4 different from https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py ? | 14:46 |
| *** alexpilotti_ has joined #openstack-cinder | 14:47 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 14:47 |
| jordanP | smcginnis, ^^ | 14:48 |
| *** diogogmt has joined #openstack-cinder | 14:48 | |
| *** annegentle has joined #openstack-cinder | 14:48 | |
| smcginnis | Looking | 14:48 |
| *** dims has joined #openstack-cinder | 14:48 | |
| ftersin | jordanP: test_volume_boot_pattern boots instances from volumes. But other options are boot from a snapshot, an image (from which volume should be autocreated), a snapshot of a volume backed instance | 14:49 |
| ftersin | jordanP: the new test checks the last option | 14:50 |
| jordanP | ftersin, got it , thanks ! | 14:50 |
| *** markvoelker has joined #openstack-cinder | 14:51 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 14:51 |
| *** nkrinner has quit IRC | 14:53 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 14:53 |
| *** alexschm has joined #openstack-cinder | 14:54 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 14:54 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 14:54 |
| *** changbl has quit IRC | 14:55 | |
| *** markvoelker has quit IRC | 14:55 | |
| Swanson | asselin_: still working with your test platform scripts. I've a basic error on my system. The nodepool image-build bit fails due to a dns failure (/etc/resolv.conf in the chroot image links to air). Seen this before? | 14:56 |
| jgriffith | mtanino: thingee I think there might be some confusion on this patch https://review.openstack.org/#/c/201243/17 ; and it might be on my part | 14:56 |
| jgriffith | mtanino: I'm ok with the naming I guess, but I wanted to point out that it doesn't matter what you put in there the scheduler will filter on it | 14:56 |
| jgriffith | mtanino: thingee in other words "cinder_replication" or just "replication" it doesn't matter; either one if reported by the capabilities update will be filtered | 14:57 |
| *** dustins has quit IRC | 14:57 | |
| jgriffith | mtanino: thingee if it's added as an extra-spec k,v without scoping | 14:58 |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: Add parameter to filter pools for NetApp driver https://review.openstack.org/210588 | 14:59 |
| jgriffith | mtanino: thingee in other words, I'm just asking again why the overly verbose naming: "cinder_xxxxxx" | 14:59 |
| *** changbl has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** dustins has joined #openstack-cinder | 15:02 | |
| *** haomaiwang has joined #openstack-cinder | 15:02 | |
| *** krtaylor has quit IRC | 15:04 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: SMBFS: Fix retrieving total allocated size https://review.openstack.org/158713 | 15:04 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 15:04 |
| *** asselin has joined #openstack-cinder | 15:06 | |
| *** morgan_503 is now known as morgan_2549 | 15:06 | |
| *** esker has joined #openstack-cinder | 15:08 | |
| *** baojg has quit IRC | 15:09 | |
| *** dustins has quit IRC | 15:10 | |
| mtanino | jgriffith: good morning, thingee is not there? | 15:11 |
| *** haomaiwang has quit IRC | 15:12 | |
| jgriffith | mtanino: doesn't seem to be, at least not yet | 15:12 |
| jgriffith | mtanino: anyway; I was fine with your patch before and I'm still ok with it. I just think there might still be some confusion on how extra-specs and the filter scheduler work... but maybe there's not. | 15:13 |
| *** BharatK has quit IRC | 15:13 | |
| *** mylu has quit IRC | 15:13 | |
| jgriffith | mtanino: I think it's sort of silly to make an admin type in "cinder_" before every extra-spec key | 15:14 |
| jgriffith | but I could be completely wrong here as well | 15:14 |
| jgriffith | :) | 15:14 |
| *** markvoelker has joined #openstack-cinder | 15:15 | |
| *** krtaylor has joined #openstack-cinder | 15:16 | |
| mtanino | jgriffith: before pushing that patch, I think we need to agree that specification again as you concerned. | 15:17 |
| jgriffith | mtanino: yeah... I just don't want to hold this up | 15:17 |
| jgriffith | mtanino: I think I've raised that point a couple times but it doesn't seem to resonate | 15:17 |
| jgriffith | mtanino: so maybe I'm the only one who cares about it; once the horses are out though we can't shut the barn door :) | 15:18 |
| *** zul has quit IRC | 15:18 | |
| *** chlong has quit IRC | 15:18 | |
| mtanino | jgriffith: might be thingee didn't see your comment yet? | 15:19 |
| jgriffith | mtanino: possible | 15:19 |
| jgriffith | maybe give it a couple hours... | 15:20 |
| jgriffith | mtanino: assuming other reviewers don't have things they raise | 15:20 |
| *** mylu has joined #openstack-cinder | 15:20 | |
| *** p0rtal has joined #openstack-cinder | 15:20 | |
| *** p0rtal has quit IRC | 15:25 | |
| mtanino | jgriffith: let me confirm again. your recommendation is "bare" for Cinder well-defined keys and vendor unique key has vendor unique key like "hitachi_thinpool". right? | 15:25 |
| *** p0rtal has joined #openstack-cinder | 15:25 | |
| mtanino | s/has vendor unique key/has vendor unique prefix/ | 15:25 |
| *** dustins has joined #openstack-cinder | 15:26 | |
| mtanino | from your comment on my patch. | 15:26 |
| *** e0ne has joined #openstack-cinder | 15:26 | |
| *** crose has joined #openstack-cinder | 15:28 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 15:29 |
| *** topshare has joined #openstack-cinder | 15:30 | |
| jgriffith | mtanino: correct | 15:30 |
| *** chlong has joined #openstack-cinder | 15:32 | |
| *** BharatK has joined #openstack-cinder | 15:32 | |
| *** topshare has quit IRC | 15:33 | |
| *** alexpilotti_ is now known as alexpilotti | 15:35 | |
| *** dustins has quit IRC | 15:35 | |
| *** dannywilson has joined #openstack-cinder | 15:35 | |
| *** diogogmt_ has joined #openstack-cinder | 15:35 | |
| *** garthb has joined #openstack-cinder | 15:35 | |
| *** garthb_ has joined #openstack-cinder | 15:35 | |
| *** diogogmt has quit IRC | 15:37 | |
| *** diogogmt_ is now known as diogogmt | 15:37 | |
| *** ndipanov has joined #openstack-cinder | 15:37 | |
| *** ndipanov has quit IRC | 15:37 | |
| *** chlong has quit IRC | 15:38 | |
| *** chlong has joined #openstack-cinder | 15:40 | |
| *** _cjones_ has joined #openstack-cinder | 15:42 | |
| *** dflorea has joined #openstack-cinder | 15:43 | |
| *** p0rtal has quit IRC | 15:43 | |
| *** _cjones_ has quit IRC | 15:43 | |
| *** _cjones_ has joined #openstack-cinder | 15:44 | |
| *** akerr has quit IRC | 15:44 | |
| *** hemnafk is now known as hemna | 15:46 | |
| *** dannywilson has quit IRC | 15:46 | |
| *** mylu has quit IRC | 15:47 | |
| *** mylu has joined #openstack-cinder | 15:48 | |
| mriedem | hemna: jungleboyj: xyang1: jgriffith: i got the stable/kilo backport patch up for that failed detach thing from last week https://review.openstack.org/#/q/I11b7c45bb6570ce11e13e487cf1136ca2551036b,n,z | 15:48 |
| xyang1 | mriedem: Ok, thanks | 15:48 |
| jgriffith | mriedem: great, I'll review it now | 15:48 |
| hemna | mriedem, great | 15:48 |
| *** akerr has joined #openstack-cinder | 15:48 | |
| *** mriedem is now known as mriedem_away | 15:50 | |
| *** dustins has joined #openstack-cinder | 15:50 | |
| *** mriedem_away has quit IRC | 15:50 | |
| *** mylu has quit IRC | 15:52 | |
| *** alexschm has quit IRC | 15:52 | |
| *** lcurtis has joined #openstack-cinder | 15:56 | |
| *** skylerberg has joined #openstack-cinder | 15:58 | |
| *** hemna has quit IRC | 16:01 | |
| *** ronis_ has joined #openstack-cinder | 16:03 | |
| *** dosaboy_ has joined #openstack-cinder | 16:04 | |
| *** dosaboy_ has quit IRC | 16:04 | |
| *** dosaboy has quit IRC | 16:04 | |
| openstackgerrit | Brent Roskos proposed openstack/cinder: adds user_id to check_is_admin https://review.openstack.org/213501 | 16:06 |
| *** dosaboy has joined #openstack-cinder | 16:09 | |
| *** e0ne has quit IRC | 16:09 | |
| openstackgerrit | Brent Roskos proposed openstack/cinder: adds user_id to check_is_admin https://review.openstack.org/213501 | 16:09 |
| *** jordanP has quit IRC | 16:12 | |
| *** alexschm has joined #openstack-cinder | 16:14 | |
| *** leeantho has joined #openstack-cinder | 16:15 | |
| *** david-ly_ is now known as david-lyle | 16:15 | |
| *** leeantho_ has joined #openstack-cinder | 16:16 | |
| *** bluex-pl has quit IRC | 16:18 | |
| *** jistr has quit IRC | 16:19 | |
| jgriffith | patrickeast: ping | 16:19 |
| jgriffith | xyang1: smcginnis ping | 16:20 |
| patrickeast | jgriffith: hey | 16:20 |
| *** leeantho has quit IRC | 16:20 | |
| jgriffith | xyang1: patrickeast xyang1 I'm trying to get the final version of my replication V2 patch up today.... had a question :) | 16:20 |
| xyang1 | jgriffith: Hi | 16:20 |
| patrickeast | yay! | 16:20 |
| jgriffith | xyang1: patrickeast xyang1 I was just going through and trying to remember why I diverged from using extra-specs and moved things into the config file | 16:20 |
| *** apoorvad has joined #openstack-cinder | 16:21 | |
| jgriffith | xyang1: patrickeast smcginnis I honestly can't remember if that was a request from somebody or what :( | 16:21 |
| patrickeast | jgriffith: you mean the replication target configuration? | 16:21 |
| jgriffith | xyang1: patrickeast smcginnis but it's just as easy/reasonable I think to put it in the extra-specs | 16:21 |
| jgriffith | patrickeast: correct | 16:21 |
| jgriffith | patrickeast: xyang1 smcginnis so the spec I had: replication_devices='backend=backend_name-a'.... | 16:22 |
| smcginnis | jgriffith: I don't believe that was from any of my feedback. | 16:22 |
| jgriffith | or replication_devices='remote_device'={'key'='val'....} | 16:22 |
| patrickeast | jgriffith: not sure, imo it makes sense to be able to do either and if we have some config options we should try and get every vendor to standardize on them | 16:22 |
| smcginnis | Any reason to do that for an admin's benefit? | 16:22 |
| jgriffith | patrickeast: well, my code right now does "either" but frankly I thought that might be bad | 16:23 |
| jgriffith | patrickeast: smcginnis xyang1 the reason being is it's two support paths (actually 4) | 16:23 |
| smcginnis | Does seem to me it should be one or the other to avoid confusion. | 16:23 |
| jgriffith | smcginnis: that's what I was thinking | 16:23 |
| jgriffith | smcginnis: patrickeast xyang1 and since both are admin functions... seems they're close enough to equivalent | 16:23 |
| patrickeast | jgriffith: might be easier if its the config file from the standpoint of someone wanting to go see where things are configured to replicate to without having to go look at every volume type, right? | 16:24 |
| smcginnis | So if it's in the config file it would require a service restart to make any changes. | 16:24 |
| smcginnis | If it's in extra-specs it can be updated at any time. | 16:24 |
| patrickeast | smcginnis: good point | 16:24 |
| smcginnis | patrickeast: That is a good point too. | 16:24 |
| jgriffith | +1 to both of you... and now you see why I'm pinging you :) | 16:24 |
| patrickeast | lol | 16:24 |
| smcginnis | Easy to copy a config file between deployments. | 16:24 |
| smcginnis | jgriffith: ;) | 16:24 |
| xyang1 | jgriffith: that seems fine to me | 16:24 |
| jgriffith | xyang1: which? :) | 16:25 |
| xyang1 | jgriffith: Extra specs | 16:25 |
| smcginnis | Do we see this changing often? If not, then I would lean toward config file. | 16:25 |
| jgriffith | smcginnis: I have no idea if it will ever even get used at all | 16:25 |
| smcginnis | If this is something we want more dynamic for the end user then extra specs would provide more flexibility. | 16:25 |
| jgriffith | let alone change often | 16:25 |
| smcginnis | jgriffith: Fair point. | 16:25 |
| jgriffith | smcginnis: enduser doesn't matter though... still all admin config | 16:26 |
| apoorvad | hgs | 16:26 |
| smcginnis | Don't suppose DuncanT is around to provide a service provider perspective. | 16:26 |
| smcginnis | jgriffith: Yeah, I meant the admin end user, not the end user end user. :) | 16:26 |
| jgriffith | actually... winston-d_zZZ | 16:26 |
| jgriffith | dang | 16:26 |
| smcginnis | Or any service provider for that matter. | 16:26 |
| jgriffith | Ok... how about I propose it to use extra-spec keys and we go from there? | 16:27 |
| smcginnis | Works for me. | 16:27 |
| patrickeast | sounds good | 16:27 |
| jgriffith | It would just make the "replication-devices" a scoped key | 16:27 |
| xyang1 | jgriffith: Fine with me | 16:27 |
| *** Yogi1 has quit IRC | 16:27 | |
| smcginnis | Extra specs I think do provide a little more configuration flexibility. | 16:28 |
| *** rmart04 has quit IRC | 16:28 | |
| jgriffith | it would look like: "type=foo, extra-spec-keys = "replication=enabled,target:xxxxx....." | 16:28 |
| jgriffith | oh... and volume_backend_name=xxxx | 16:28 |
| jgriffith | The only advantage to putting it in config is then the extra-specs is just "replication_enabled=True" and if you want "volume_backend_name=xyz" I guess it doesn't really matter | 16:29 |
| *** vilobhmm has joined #openstack-cinder | 16:29 | |
| jgriffith | Ok... going extra-specs for now then | 16:29 |
| jgriffith | thanks folks! | 16:29 |
| *** daneyon has quit IRC | 16:30 | |
| *** daneyon has joined #openstack-cinder | 16:30 | |
| jgriffith | Oh... I remember why this changed!!! | 16:31 |
| jgriffith | hema... | 16:31 |
| jgriffith | who isn't here | 16:31 |
| jgriffith | had all sorts of complaints about their customers not being able to do extra-specs correctly and that it was error-prone | 16:31 |
| jgriffith | kmartin: ^^ | 16:31 |
| *** anshul has quit IRC | 16:32 | |
| jgriffith | kmartin: xyang1 patrickeast smcginnis jungleboyj Vote for your favorite: http://paste.openstack.org/show/419344/ | 16:37 |
| jgriffith | kmartin: xyang1 patrickeast smcginnis jungleboyj between extra-specs and config file | 16:37 |
| jgriffith | Note the second one both are labelled extra-specs but you can figure out which is which :) | 16:38 |
| *** Yogi1 has joined #openstack-cinder | 16:39 | |
| *** Yogi11 has joined #openstack-cinder | 16:39 | |
| *** rushil has quit IRC | 16:40 | |
| patrickeast | is there any concern about having credentials in volume types? | 16:40 |
| jgriffith | probably :) | 16:43 |
| patrickeast | jgriffith: so whats the intended flow for changing replication targets with each approach? looks like if its volume types and you have to change something it means a re-type for every volume being replicated, right? for the config file its just restart c-vol? | 16:43 |
| jgriffith | but I don't know how you get around it | 16:43 |
| *** Yogi1 has quit IRC | 16:43 | |
| patrickeast | jgriffith: well, if they were in the config file its sort of understood that it is locked down... i don't know if volume type info is the same way | 16:43 |
| jgriffith | patrickeast: not a bad point | 16:43 |
| jgriffith | patrickeast: I think that might be reason enough to start by using conf | 16:44 |
| jgriffith | patrickeast: and honestly it can be overridden if people want to | 16:44 |
| jgriffith | ok... I'm going forward with the conf file | 16:44 |
| patrickeast | jgriffith: yea true, a driver can always look for those type specs if they want to anyway | 16:45 |
| jgriffith | If vendors want to do something "special" they're always able to | 16:45 |
| jgriffith | cool | 16:45 |
| xyang1 | jgriffith: patrickeast sounds reasonable to me. This should not change that often | 16:45 |
| jgriffith | thanks!!! | 16:45 |
| *** martyturner has quit IRC | 16:45 | |
| *** earlephilhower has joined #openstack-cinder | 16:45 | |
| *** martyturner has joined #openstack-cinder | 16:45 | |
| patrickeast | gah my ci is really sucking lately | 16:48 |
| patrickeast | i stop paying attention to it for a couple weeks to actually get other tasks done and it catches on fire : / | 16:49 |
| jgriffith | patrickeast: I keep getting these random copy-image to volume failures | 16:49 |
| jgriffith | was about 1 in 20, now like 1 in 5 | 16:49 |
| patrickeast | yea i think i get some of those too | 16:49 |
| jgriffith | something changed when we moved to brick but I can't figure it out | 16:50 |
| jgriffith | at least based on the data, that's when it started, could be coincidence I suppose | 16:50 |
| *** lpetrut has quit IRC | 16:50 | |
| patrickeast | yea, its hard to tell with some many moving parts | 16:51 |
| *** sgotliv has quit IRC | 16:51 | |
| patrickeast | i'm hoping to spend some time soon to go back and deep dive on some of the failures i'm getting | 16:51 |
| jgriffith | patrickeast: let me know when you start poking at it, we should compare notes :) | 16:53 |
| patrickeast | jgriffith: will do | 16:53 |
| *** alexschm has quit IRC | 16:54 | |
| *** lpetrut has joined #openstack-cinder | 16:55 | |
| *** jecarey has quit IRC | 16:57 | |
| *** _cjones_ has quit IRC | 17:00 | |
| kmartin | jgriffith, I tend to like Form 1, where replication target is another configured cinder backend if the poles are still open | 17:00 |
| jgriffith | kmartin: sounds good, thanks! | 17:00 |
| jgriffith | kmartin: I'm a fan of that form as well | 17:00 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 17:01 |
| *** lpetrut has quit IRC | 17:03 | |
| thingee | mtanino: jgriffith the suggestion of Hitachi_ prefixed keys for vendor specific keys and the non prefix keys for well defined keys seems fine to me. | 17:04 |
| thingee | I think it’s silly to expect cinder_ prefix too | 17:04 |
| thingee | Now that I’m thinking about it. | 17:04 |
| jgriffith | thingee: cool! | 17:04 |
| jgriffith | less typing at least :) | 17:05 |
| thingee | I would still like the manager to enforce what is well defined. Not drivers. Drivers can flip true or false though. | 17:05 |
| jgriffith | 17:05 | |
| jgriffith | bahh | 17:05 |
| *** zhenguo has quit IRC | 17:06 | |
| *** zhenguo has joined #openstack-cinder | 17:06 | |
| *** zhenguo has quit IRC | 17:07 | |
| *** dflorea has quit IRC | 17:07 | |
| *** p0rtal has joined #openstack-cinder | 17:13 | |
| *** salv-orl_ has joined #openstack-cinder | 17:14 | |
| *** dflorea has joined #openstack-cinder | 17:14 | |
| jungleboyj | jgriffith and patrickeast Yeah, given the credentials concern I think using the config file is the better choice. | 17:15 |
| jungleboyj | jgriffith: patrickeast Seems like that is more secure. Obviously we have to restart to enable a change, but I don't think that is something that is going to happen often. | 17:16 |
| *** salv-orlando has quit IRC | 17:16 | |
| jungleboyj | Sorry for being late to the party. | 17:17 |
| *** p0rtal_ has joined #openstack-cinder | 17:23 | |
| *** p0rtal__ has joined #openstack-cinder | 17:26 | |
| *** rushil has joined #openstack-cinder | 17:26 | |
| *** p0rtal___ has joined #openstack-cinder | 17:27 | |
| jgriffith | jungleboyj: no worried, good feedback! | 17:27 |
| *** p0rtal has quit IRC | 17:27 | |
| jungleboyj | jgriffith: Thanks. | 17:27 |
| *** rushil has quit IRC | 17:28 | |
| *** p0rtal_ has quit IRC | 17:29 | |
| *** p0rtal has joined #openstack-cinder | 17:29 | |
| *** rushil has joined #openstack-cinder | 17:29 | |
| *** p0rtal__ has quit IRC | 17:30 | |
| *** kvidvans has joined #openstack-cinder | 17:30 | |
| jungleboyj | I don't see hemna around. Am I reading this right that the multi-attach code for Nova missed yet another release? | 17:31 |
| jungleboyj | jgriffith: smcginnis patrickeast ^^ | 17:31 |
| jgriffith | jungleboyj: yup, I believe so | 17:32 |
| patrickeast | ouch | 17:32 |
| *** dannywilson has joined #openstack-cinder | 17:32 | |
| *** _cjones_ has joined #openstack-cinder | 17:32 | |
| * jungleboyj sighs | 17:32 | |
| jungleboyj | Poor hemna | 17:32 |
| *** p0rtal___ has quit IRC | 17:32 | |
| *** dannywilson has quit IRC | 17:34 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 17:37 |
| *** markus_z has quit IRC | 17:38 | |
| *** hemna has joined #openstack-cinder | 17:44 | |
| *** xyang1 has quit IRC | 17:44 | |
| *** rhagarty__ has joined #openstack-cinder | 17:45 | |
| *** rhagarty has quit IRC | 17:46 | |
| *** rhagarty_ has quit IRC | 17:46 | |
| *** xyang1 has joined #openstack-cinder | 17:46 | |
| *** rhagarty_ has joined #openstack-cinder | 17:47 | |
| *** rhagarty__ has quit IRC | 17:50 | |
| *** DTadrzak_ has joined #openstack-cinder | 17:50 | |
| *** afazkas has joined #openstack-cinder | 17:50 | |
| *** rhagarty has joined #openstack-cinder | 17:51 | |
| *** rmetcalf has joined #openstack-cinder | 17:51 | |
| *** dims_ has joined #openstack-cinder | 17:53 | |
| *** crose has quit IRC | 17:54 | |
| DTadrzak_ | xyang1: ping | 17:55 |
| *** zul has joined #openstack-cinder | 17:56 | |
| *** dims has quit IRC | 17:57 | |
| xyang1 | DTadrzak_: Hi | 17:57 |
| *** dustins has quit IRC | 17:59 | |
| *** afazkas has quit IRC | 18:01 | |
| DTadrzak_ | xyang1: Hello. Could add this patch https://review.openstack.org/#/c/160417/ to your review queue? | 18:01 |
| *** dustins has joined #openstack-cinder | 18:01 | |
| *** ftersin has left #openstack-cinder | 18:01 | |
| xyang1 | DTadrzak_: Ok | 18:02 |
| mtanino | thingee: OK. Thank you for your confirmation. I will fix my proposed patch asap. Also do we need to post follow up patch for cinder-spec as well? | 18:03 |
| mtanino | or leave the spec as is? | 18:04 |
| *** DTadrzak_ has quit IRC | 18:05 | |
| *** IanGovett1 has joined #openstack-cinder | 18:06 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 18:06 |
| *** lpetrut has joined #openstack-cinder | 18:08 | |
| *** IanGovett has quit IRC | 18:08 | |
| *** krtaylor has quit IRC | 18:08 | |
| *** p0rtal has quit IRC | 18:09 | |
| *** p0rtal has joined #openstack-cinder | 18:10 | |
| *** rhagarty has quit IRC | 18:13 | |
| *** rhagarty_ has quit IRC | 18:13 | |
| *** rhagarty has joined #openstack-cinder | 18:13 | |
| *** rhagarty_ has joined #openstack-cinder | 18:13 | |
| *** rushil has quit IRC | 18:13 | |
| *** p0rtal has quit IRC | 18:14 | |
| *** p0rtal_ has joined #openstack-cinder | 18:14 | |
| *** IanGovett has joined #openstack-cinder | 18:16 | |
| *** IanGovett1 has quit IRC | 18:17 | |
| *** rushil has joined #openstack-cinder | 18:19 | |
| erlon | jgriffith: ping | 18:19 |
| *** krtaylor has joined #openstack-cinder | 18:21 | |
| *** gardenshed has joined #openstack-cinder | 18:22 | |
| erlon | jgriffith: John, I'm having a problem with HNAS driver, when trying to attach multiple instances at the same time, I think it is related to this implementation on os-brick _disconnect_volume_iscsi() | 18:22 |
| erlon | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L500 | 18:22 |
| erlon | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxscsi.py#L60 | 18:22 |
| erlon | I still getting orphaned iscsi sessions | 18:23 |
| erlon | hemna: ^ | 18:23 |
| Adriano__ | hemna: ping | 18:23 |
| *** afazkas has joined #openstack-cinder | 18:24 | |
| Adriano__ | hemna: we addressed your comment on https://review.openstack.org/#/c/197999/ | 18:28 |
| Adriano__ | hemna: it would be nice if you could validate it when you have some time :) | 18:28 |
| *** afazkas has quit IRC | 18:30 | |
| *** geguileo has quit IRC | 18:31 | |
| *** annegentle has quit IRC | 18:32 | |
| *** angela-s has joined #openstack-cinder | 18:33 | |
| *** mriedem has joined #openstack-cinder | 18:37 | |
| *** e0ne has joined #openstack-cinder | 18:40 | |
| *** shyama has quit IRC | 18:45 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port more unit tests to Python 3 https://review.openstack.org/213817 | 18:47 |
| jgriffith | erlon: is it that or something in multi-attach do you think? | 18:48 |
| jgriffith | erlon: I guess, what is the problem that you're seeing? | 18:48 |
| erlon | jgriffith: haven't tried multi-attach | 18:49 |
| jgriffith | erlon: ? | 18:49 |
| jgriffith | erlon: jgriffith: John, I'm having a problem with HNAS driver, when trying to attach multiple instances at the same time, I think it is related to this implementation | 18:49 |
| jgriffith | erlon: sorry... did you mean just attaching multiple instances to multiple volumes simultaneously? | 18:49 |
| erlon | jgriffith: yes | 18:50 |
| jgriffith | erlon: ahh | 18:50 |
| jgriffith | erlon: hmm... ya know, wonder if that's part of the issue I'm seeing as well | 18:50 |
| erlon | apparently, the code is not fixing the problem if the concurrency is very high | 18:50 |
| *** mylu has joined #openstack-cinder | 18:50 | |
| jgriffith | erlon: is that version released? | 18:51 |
| *** mylu has quit IRC | 18:51 | |
| erlon | jgriffith: of os-brick? | 18:51 |
| hemna | hey | 18:52 |
| hemna | what's up | 18:52 |
| *** mylu has joined #openstack-cinder | 18:52 | |
| erlon | hemna: hi, pinged you because is a os-brick related problem | 18:52 |
| jgriffith | looks like it is | 18:52 |
| hemna | that should be released afaik | 18:53 |
| *** skylerberg has quit IRC | 18:53 | |
| jgriffith | hemna: yes, problem is erlon thinks it might be causing a problem or at least not fixing the problem | 18:53 |
| hemna | maybe the wait isn't waiting/retrying long enough on super busy systems ? | 18:53 |
| hemna | I think it retries 3 times | 18:54 |
| hemna | I think we just bumped the backoff to 2 recently (when we were in CO) | 18:54 |
| erlon | jgriffith: what happens is, when you have multiple request for detach volumes, the requests wait in the lock. The request being processed clean up the system device, but, when the next request gets the lock, the first thing it does is iscsadmin --rescan, then the removed sys device is re-added | 18:54 |
| jgriffith | hemna: yeah | 18:54 |
| hemna | heh | 18:54 |
| jgriffith | erlon: yep | 18:54 |
| hemna | erlon, yah the rescan on detach is bad | 18:54 |
| hemna | erlon, I actually have a patch waiting for the final +A | 18:55 |
| hemna | erlon, https://review.openstack.org/#/c/203265/ | 18:55 |
| erlon | hemna: I increase that to 5 and still does not fixes | 18:55 |
| *** krtaylor has quit IRC | 18:55 | |
| hemna | I noticed the same problem when doing a live migration when >=2 iSCSI volumes from the same backend is attached to the vm being migrated | 18:56 |
| jgriffith | We have like 10 cores now, you'd think it'd be easier to get a +2/A | 18:56 |
| *** mylu has quit IRC | 18:57 | |
| *** gardenshed has quit IRC | 18:57 | |
| hemna | jgriffith, +1 | 18:57 |
| jgriffith | Of course it doesn't matter anyway, since it's now a lib it will be "who knows" how long before it's cut and distributed anyway | 18:57 |
| erlon | hemna: does the patch fix the problem? | 18:57 |
| hemna | that one has been waiting for a bit | 18:57 |
| hemna | erlon, it should yes | 18:57 |
| jgriffith | erlon: try applying it and see if it fixes your case | 18:58 |
| jgriffith | erlon: I've been hoping it will fix an issue I'm seeing but I'm not sure... and I'm just patiently waiting | 18:58 |
| erlon | hemna: hmm, I tried in another way, wich didn't work, I didn't know I could remove the rescan from disconnect volume | 18:58 |
| hemna | erlon, yah, just apply that patch or hack the file and remove the rescan :P | 18:58 |
| erlon | jgriffith: I will | 18:58 |
| hemna | jgriffith, thanks :) | 18:59 |
| erlon | hemna: jgriffith: thanks! | 18:59 |
| *** sakaYK has quit IRC | 19:02 | |
| *** e0ne has quit IRC | 19:03 | |
| *** annegentle has joined #openstack-cinder | 19:05 | |
| *** krtaylor has joined #openstack-cinder | 19:08 | |
| *** e0ne has joined #openstack-cinder | 19:09 | |
| *** JoseMello has quit IRC | 19:09 | |
| erlon | hemna: jgriffith: it works! | 19:13 |
| erlon | hemna: why the rescan was there BTW then? | 19:13 |
| jgriffith | EXCELLENT! | 19:13 |
| jgriffith | erlon: it was in a different location at one point | 19:14 |
| jgriffith | erlon: and used specifically to verify that a target was "gone" | 19:14 |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Correct comment to be consistent with code https://review.openstack.org/213826 | 19:14 |
| jgriffith | erlon: but the code has changed a bit, and it's now in the "wrong" place and actually causes issues rather than fixing them | 19:14 |
| erlon | jgriffith: hmm | 19:14 |
| erlon | jgriffith: it make more sense then to have it after issuing the 'echo 1 > /sys/block/.../remove', and the retrying until the BE remove the target | 19:15 |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Refactor to remove duplicate code https://review.openstack.org/213827 | 19:16 |
| erlon | which was what I tried to do, but, the nova code does not issue the remove_connection to the BE until it does not leave _disconnect_volume() | 19:17 |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Remove unused function volume_type_encryption_get https://review.openstack.org/213828 | 19:17 |
| jgriffith | Yeah, so it used to be part of the delete routine | 19:18 |
| jgriffith | erlon: delete_volume | 19:18 |
| jgriffith | we used to create targets on volume_create, and they lived until volume_delete | 19:18 |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Clean up volume_type_update method https://review.openstack.org/213829 | 19:18 |
| jgriffith | but the introduction of various target drivers that require initiator knowledge to be created required all of that be changed | 19:19 |
| erlon | jgriffith: hmm | 19:20 |
| *** gouthamr has quit IRC | 19:27 | |
| *** crose has joined #openstack-cinder | 19:30 | |
| *** mriedem1 has joined #openstack-cinder | 19:31 | |
| thingee | mtanino: there should be a follow up to spec | 19:32 |
| mtanino | thingee: ok. May I update the spec? | 19:32 |
| *** rmetcalf has quit IRC | 19:32 | |
| *** gouthamr has joined #openstack-cinder | 19:32 | |
| *** mriedem has quit IRC | 19:33 | |
| thingee | mtanino: yes | 19:33 |
| *** andreykurilin__ has joined #openstack-cinder | 19:34 | |
| mtanino | thingee: sure. will do. Also may I post a patch to implement endpoint for capabilities like " http://CINDER_API_ENDPOINT/v2/TENANT_ID/capabilities/<host>" | 19:35 |
| mtanino | ? | 19:35 |
| *** lpetrut has quit IRC | 19:36 | |
| *** dannywilson has joined #openstack-cinder | 19:37 | |
| *** lpetrut has joined #openstack-cinder | 19:39 | |
| openstackgerrit | Joel Coffman proposed openstack/cinder: Store volume encryption metadata with volumes https://review.openstack.org/213835 | 19:42 |
| *** annegentle has quit IRC | 19:42 | |
| *** annegentle has joined #openstack-cinder | 19:43 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Create CG from CG Snapshot https://review.openstack.org/195280 | 19:44 |
| *** e0ne has quit IRC | 19:45 | |
| *** e0ne has joined #openstack-cinder | 19:48 | |
| *** dims_ has quit IRC | 19:48 | |
| *** dims has joined #openstack-cinder | 19:48 | |
| *** lpetrut has quit IRC | 19:49 | |
| *** skylerberg has joined #openstack-cinder | 19:50 | |
| *** annasort has joined #openstack-cinder | 19:53 | |
| openstackgerrit | Brent Roskos proposed openstack/cinder: adds user_id to check_is_admin https://review.openstack.org/213501 | 19:56 |
| hemna | erlon, very cool! | 19:57 |
| thingee | smcginnis: appreciate you looking at volume migration status update https://review.openstack.org/186312 | 19:58 |
| thingee | smcginnis: so everything worked when trying it out? | 19:58 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers. https://review.openstack.org/197999 | 20:02 |
| *** Lee1092 has quit IRC | 20:03 | |
| *** gardenshed has joined #openstack-cinder | 20:07 | |
| *** garthb has quit IRC | 20:08 | |
| *** garthb_ has quit IRC | 20:08 | |
| *** akerr has quit IRC | 20:08 | |
| *** garthb_ has joined #openstack-cinder | 20:08 | |
| *** garthb has joined #openstack-cinder | 20:09 | |
| *** akerr has joined #openstack-cinder | 20:09 | |
| *** martyturner has quit IRC | 20:10 | |
| Adriano__ | thingee: We found a bug in the unmanage when doing some tests in this change https://review.openstack.org/197999 | 20:10 |
| Adriano__ | thingee: I've just submitted a new patch to correct it | 20:10 |
| *** ronis_ has quit IRC | 20:10 | |
| Adriano__ | thingee: just to let you know that there's nothing really new here | 20:11 |
| *** gardenshed has quit IRC | 20:12 | |
| *** lpetrut has joined #openstack-cinder | 20:13 | |
| jgriffith | Is anybody else getting prompted for login/pass trying to clone openstack/cinder? | 20:14 |
| *** daneyon has quit IRC | 20:14 | |
| *** rushil has quit IRC | 20:15 | |
| *** mylu has joined #openstack-cinder | 20:16 | |
| jgriffith | Whaaaaaaat!!!! https://www.dropbox.com/s/02pjxvsm9v66go2/Screenshot%202015-08-17%2014.16.43.png?dl=0 | 20:16 |
| hemna | whoa | 20:17 |
| simondodsley | Github Status: 16:08 Eastern Daylight TimeWe're investigating an issue affecting a small number of repositories. | 20:18 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 20:19 |
| *** crose has quit IRC | 20:20 | |
| *** akerr has quit IRC | 20:27 | |
| *** akerr has joined #openstack-cinder | 20:28 | |
| *** mylu has quit IRC | 20:33 | |
| *** mylu has joined #openstack-cinder | 20:34 | |
| *** rushil has joined #openstack-cinder | 20:34 | |
| jgriffith | simondodsley: thanks! | 20:35 |
| *** mylu_ has joined #openstack-cinder | 20:35 | |
| *** mylu has quit IRC | 20:35 | |
| jgriffith | simondodsley: yeah... folks in infra pointed out that I've just been lucky over the years using github.com instead of git.openstack.org | 20:35 |
| jgriffith | apparently I've just dodged this bullet for a really long time; but I was skeptical with the stack forge migration etc | 20:36 |
| *** rushil has quit IRC | 20:37 | |
| *** rushil has joined #openstack-cinder | 20:38 | |
| *** rushil has quit IRC | 20:38 | |
| *** lpetrut has quit IRC | 20:38 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Assisted volume migration for Oracle ZFSSA drivers https://review.openstack.org/210608 | 20:43 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding Consistency Group Support https://review.openstack.org/209578 | 20:44 |
| openstackgerrit | Patrick East proposed openstack/cinder: Implement manage/unmanage snapshot in Pure drivers https://review.openstack.org/213855 | 20:45 |
| *** raildo is now known as raildo-afk | 20:50 | |
| *** mylu_ has quit IRC | 20:52 | |
| *** dustins has quit IRC | 20:52 | |
| *** Yogi11 has quit IRC | 20:58 | |
| *** akerr has quit IRC | 20:59 | |
| *** bswartz has quit IRC | 21:00 | |
| *** IlyaG has joined #openstack-cinder | 21:06 | |
| *** IlyaG has quit IRC | 21:06 | |
| simondodsley | jgriffiths: and it's back :) | 21:06 |
| aorourke | xyang1, if you do not have create_consistencygroup_from_src implemented but you try and run cinder consisgroup-create-from-src, the CG you create goes into error state. But you can not delete that CG because it has a cgsnapshot as a dependency that needs to be delete first. Shouldn't you be allowed to delete a CG that had an error while being created? | 21:09 |
| xyang1 | aorourke: I don't understand the dependency part | 21:11 |
| aorourke | xyang1, if you create a CG from a cgsnapshot, the CG has cgsnapshot_id as a FK in the database, right? | 21:12 |
| xyang1 | aorourke: If cg depends on cgsnapshot, it should not prevent you from delete cg | 21:12 |
| aorourke | xyang1, cinder does not allow me to delete a CG until the cgsnapshot is deleted | 21:13 |
| xyang1 | aorourke: You are talking about the 1st cg | 21:14 |
| xyang1 | aorourke: You have cg1 | 21:14 |
| *** zul has quit IRC | 21:14 | |
| xyang1 | aorourke: Then you create cgsnapshot from it | 21:14 |
| aorourke | xyang1, both the original and the new cg | 21:14 |
| xyang1 | aorourke: Doesn't make sense for the new one though | 21:15 |
| xyang1 | aorourke: I can give a try | 21:15 |
| *** alexpilotti has quit IRC | 21:16 | |
| *** alexpilotti has joined #openstack-cinder | 21:16 | |
| *** julim has quit IRC | 21:17 | |
| *** simondodsley has quit IRC | 21:17 | |
| aorourke | xyang1, here is a paste of what I am trying http://paste.openstack.org/show/Ctk4e3F2asNKNaGPPzwQ/ | 21:20 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 21:20 |
| *** mylu has joined #openstack-cinder | 21:20 | |
| *** gouthamr has quit IRC | 21:21 | |
| *** mylu has quit IRC | 21:23 | |
| *** porrua has quit IRC | 21:32 | |
| anish | mriedem1: hemna: does devstack need any changes wrt this https://review.openstack.org/#/c/175569/ (or was that done/happens automatically ?) | 21:32 |
| hemna | anish, it shouldn't. nova has the requirements. | 21:34 |
| xyang1 | aorourke: Hi, it does look like there is a problem, may need to fix the db method. I'll fix it | 21:36 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: force_detach terminate_connection needs connector https://review.openstack.org/213867 | 21:36 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 21:37 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 21:37 |
| aorourke | xyang1, so you should be able to delete all CGs created from a cgsnapshot source? Regardless of if they are in error state or not? | 21:37 |
| xyang1 | aorourke: What state did you get | 21:38 |
| *** merooney has quit IRC | 21:38 | |
| xyang1 | aorourke: You should be able to delete available or error state, but not ing state | 21:38 |
| aorourke | xyang1, ok, that is what I was wondering. | 21:38 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 21:41 |
| *** gouthamr has joined #openstack-cinder | 21:42 | |
| anish | hemna: thanks ! | 21:43 |
| mtanino | jgriffith: ^^ I posted updates for get_capabilities() and related patches. | 21:44 |
| aorourke | xyang1, do you want me to create a bug for this? | 21:45 |
| xyang1 | aorourke: Up to you, I can create one when i have a fix too. The filter doesn't seem to work | 21:46 |
| *** thangp has quit IRC | 21:50 | |
| *** erlon has quit IRC | 21:53 | |
| aorourke | xyang1, I can file something on this | 21:54 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Create service_versions table https://review.openstack.org/205622 | 21:57 |
| *** rhagarty has quit IRC | 22:00 | |
| *** rhagarty_ has quit IRC | 22:00 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Create service_versions table https://review.openstack.org/205622 | 22:01 |
| *** jungleboyj has quit IRC | 22:03 | |
| *** rhagarty has joined #openstack-cinder | 22:04 | |
| *** rhagarty_ has joined #openstack-cinder | 22:04 | |
| xyang1 | aorourke: Do you have a bug number yet | 22:04 |
| aorourke | xyang1, 1485783 | 22:05 |
| xyang1 | Ok thsnkd | 22:05 |
| xyang1 | Thanks | 22:05 |
| *** jgregor has quit IRC | 22:06 | |
| *** garthb_ has quit IRC | 22:09 | |
| *** garthb has quit IRC | 22:10 | |
| *** garthb_ has joined #openstack-cinder | 22:12 | |
| *** garthb has joined #openstack-cinder | 22:12 | |
| *** garthb has quit IRC | 22:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 22:15 |
| openstackgerrit | xing-yang proposed openstack/cinder: Allow CG without snapshot to be deleted https://review.openstack.org/213892 | 22:16 |
| *** alexpilotti has quit IRC | 22:19 | |
| *** IanGovett has quit IRC | 22:23 | |
| *** marcusvrn has quit IRC | 22:23 | |
| *** dims_ has joined #openstack-cinder | 22:25 | |
| *** edmondsw has quit IRC | 22:26 | |
| *** dims has quit IRC | 22:28 | |
| *** annegentle has quit IRC | 22:30 | |
| *** zul has joined #openstack-cinder | 22:33 | |
| *** esker has quit IRC | 22:33 | |
| *** chlong has quit IRC | 22:34 | |
| *** e0ne has quit IRC | 22:41 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Allow CG without snapshot to be deleted https://review.openstack.org/213892 | 22:45 |
| *** jungleboyj has joined #openstack-cinder | 22:45 | |
| *** dims_ has quit IRC | 22:47 | |
| *** dims has joined #openstack-cinder | 22:47 | |
| *** markvoelker has quit IRC | 22:51 | |
| *** dims has quit IRC | 22:51 | |
| *** hemna is now known as hemnafk | 22:55 | |
| *** porrua has joined #openstack-cinder | 22:55 | |
| *** porrua has quit IRC | 22:57 | |
| *** zul has quit IRC | 23:04 | |
| *** salv-orl_ has quit IRC | 23:09 | |
| *** salv-orlando has joined #openstack-cinder | 23:12 | |
| *** gouthamr has quit IRC | 23:15 | |
| *** lcurtis has quit IRC | 23:17 | |
| *** sakaYK has joined #openstack-cinder | 23:20 | |
| *** tiny-hands has joined #openstack-cinder | 23:21 | |
| *** Yogi1 has joined #openstack-cinder | 23:30 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Add version columns to services table https://review.openstack.org/205622 | 23:32 |
| *** markvoelker has joined #openstack-cinder | 23:35 | |
| *** takedakn has joined #openstack-cinder | 23:35 | |
| *** salv-orlando has quit IRC | 23:36 | |
| jgriffith | thingee: eharney DuncanT smcginnis Please see my comments in this patch: https://review.openstack.org/#/c/209578/ | 23:48 |
| jgriffith | thingee: eharney DuncanT smcginnis I'll got back through it when I get home tonight; maybe I'm missing something but if the cg methods weren't there and it inherited the CG meatballs how has the driver been loading :( | 23:48 |
| jgriffith | When I saw the change I assumed it wasn't inheritting it anywhere because it was 'working' without the methods | 23:49 |
| jgriffith | Maybe I can present my pleas to the court again regarding my simplification patch :) | 23:49 |
| jgriffith | Or I"m an idiot and missing some detail in there somehwere | 23:50 |
| jgriffith | LOL | 23:50 |
| jgriffith | s/meatballs/methods/ | 23:50 |
| jgriffith | BAAHHHHH | 23:51 |
| jgriffith | See this is why I can't stand this madness | 23:51 |
| jgriffith | The base volume driver inherits and implements it with NotImplement | 23:51 |
| jgriffith | frik! | 23:51 |
| patrickeast | jgriffith: +1 | 23:53 |
| patrickeast | its still very confusing | 23:53 |
| *** akerr has joined #openstack-cinder | 23:53 | |
| patrickeast | there isn't really any validation for drivers that don't want validation and choose to only inherit directly from the basest of the base classes | 23:53 |
| patrickeast | choose to not* | 23:54 |
| *** akerr_ has joined #openstack-cinder | 23:56 | |
| *** Ctina_ has joined #openstack-cinder | 23:56 | |
| jgriffith | patrickeast: even worse, the base FC class (which I argue we shouldn't have but that's another rant :) ) inherits the VolumeDriver so everybody behind it just get's "everything" | 23:56 |
| jgriffith | bahhhh | 23:56 |
| jgriffith | it's craziness :) | 23:56 |
| patrickeast | iirc same thing for the saniscsi one | 23:57 |
| jgriffith | patrickeast: yup! | 23:57 |
| patrickeast | i haven't touched my drivers to mess with the abc stuff and everything still keeps chuggin along | 23:57 |
| patrickeast | and frankly i have no incentive to change it | 23:57 |
| jgriffith | patrickeast: right | 23:57 |
| jgriffith | patrickeast: if we're keeping it I'll do a major patch next week or so and just update everything and rip off the band-aid | 23:58 |
| *** akerr has quit IRC | 23:58 | |
| jgriffith | any who... first things first, finish replication :) | 23:58 |
| patrickeast | jgriffith: probably the best way... maybe even what should have been done initially | 23:58 |
| patrickeast | haha yea | 23:59 |
| jgriffith | few more unit tests and now the API methods | 23:59 |
| jgriffith | almost there I think | 23:59 |
| patrickeast | sweet | 23:59 |
| jgriffith | alright... time to run; catch ya later | 23:59 |
| patrickeast | i think dannywilson almost has our impl ready to start doing stuff once we've got the api's in | 23:59 |
| patrickeast | cya | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
