| *** ndipanov has quit IRC | 00:00 | |
| *** mriedem has joined #openstack-cinder | 00:01 | |
| *** haomaiwa_ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:01 | |
| *** baojg has quit IRC | 00:01 | |
| *** yangyapeng has quit IRC | 00:03 | |
| *** takedakn has joined #openstack-cinder | 00:06 | |
| *** dims has joined #openstack-cinder | 00:08 | |
| *** mylu has quit IRC | 00:08 | |
| *** mylu has joined #openstack-cinder | 00:10 | |
| *** garthb has quit IRC | 00:13 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from api.v1 unit tests https://review.openstack.org/292550 | 00:16 |
|---|---|---|
| *** gouthamr has joined #openstack-cinder | 00:22 | |
| *** openstack has joined #openstack-cinder | 00:25 | |
| *** Suyash has quit IRC | 00:26 | |
| *** mylu has quit IRC | 00:27 | |
| *** gouthamr has quit IRC | 00:27 | |
| *** mylu has joined #openstack-cinder | 00:27 | |
| *** mtanino has quit IRC | 00:28 | |
| *** takedakn has quit IRC | 00:29 | |
| *** IlyaG has quit IRC | 00:29 | |
| *** mylu has quit IRC | 00:35 | |
| *** erlon has quit IRC | 00:37 | |
| *** eharney has quit IRC | 00:37 | |
| *** mylu has joined #openstack-cinder | 00:39 | |
| *** mylu has quit IRC | 00:40 | |
| *** mylu has joined #openstack-cinder | 00:45 | |
| *** dims has joined #openstack-cinder | 00:45 | |
| *** laughterwym has joined #openstack-cinder | 00:46 | |
| *** mylu has quit IRC | 00:46 | |
| *** smoriya_ has joined #openstack-cinder | 00:48 | |
| *** mylu has joined #openstack-cinder | 00:48 | |
| *** mylu has quit IRC | 00:50 | |
| *** afazekas has quit IRC | 00:53 | |
| *** dave-mccowan has joined #openstack-cinder | 00:57 | |
| *** afazekas has joined #openstack-cinder | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-cinder | 01:01 | |
| *** daneyon has quit IRC | 01:04 | |
| *** david-lyle has joined #openstack-cinder | 01:04 | |
| *** dims has quit IRC | 01:07 | |
| *** yangyapeng has joined #openstack-cinder | 01:10 | |
| *** dims has joined #openstack-cinder | 01:12 | |
| *** mylu has joined #openstack-cinder | 01:14 | |
| *** mylu has quit IRC | 01:15 | |
| *** cbinx has quit IRC | 01:16 | |
| *** EinstCrazy has joined #openstack-cinder | 01:21 | |
| *** yangyapeng has quit IRC | 01:22 | |
| *** bardia has quit IRC | 01:22 | |
| *** haomaiwang has quit IRC | 01:23 | |
| *** gouthamr_ is now known as gouthamr | 01:23 | |
| *** boris-42 has quit IRC | 01:24 | |
| *** yangyapeng has joined #openstack-cinder | 01:24 | |
| *** asselin_ has joined #openstack-cinder | 01:24 | |
| *** afazekas has quit IRC | 01:25 | |
| *** asselin_ has quit IRC | 01:28 | |
| *** afazekas has joined #openstack-cinder | 01:28 | |
| *** cknight has joined #openstack-cinder | 01:29 | |
| *** dims has quit IRC | 01:29 | |
| *** dims has joined #openstack-cinder | 01:29 | |
| *** ebalduf has quit IRC | 01:32 | |
| *** yangds has joined #openstack-cinder | 01:32 | |
| sheel | hemna:hi there | 01:32 |
| *** akshai has quit IRC | 01:33 | |
| *** esker has joined #openstack-cinder | 01:37 | |
| openstackgerrit | dongsheng.yang@easystack.cn proposed openstack/python-cinderclient: Print a more readable output of upload-to-image https://review.openstack.org/292840 | 01:40 |
| *** panda has quit IRC | 01:40 | |
| *** panda has joined #openstack-cinder | 01:40 | |
| *** IlyaG has joined #openstack-cinder | 01:41 | |
| *** mylu has joined #openstack-cinder | 01:42 | |
| *** mriedem has quit IRC | 01:42 | |
| *** mylu has quit IRC | 01:44 | |
| *** kaisers1 has joined #openstack-cinder | 01:44 | |
| *** kaisers has quit IRC | 01:45 | |
| *** asselin_ has joined #openstack-cinder | 01:50 | |
| *** dims has quit IRC | 01:52 | |
| openstackgerrit | dongsheng.yang@easystack.cn proposed openstack/python-cinderclient: Cleanup for Replication v2: remove 'replication-reenable'. https://review.openstack.org/292332 | 01:54 |
| openstackgerrit | dongsheng.yang@easystack.cn proposed openstack/python-cinderclient: Cleanup for Replication v2: remove 'replication-promote' https://review.openstack.org/292331 | 01:54 |
| *** dims has joined #openstack-cinder | 01:56 | |
| *** Lickitysplitted_ has quit IRC | 01:56 | |
| *** IlyaG has quit IRC | 01:57 | |
| *** baojg has joined #openstack-cinder | 01:59 | |
| *** Lickitysplitted has joined #openstack-cinder | 01:59 | |
| *** haomaiwang has joined #openstack-cinder | 02:00 | |
| *** haomaiwang has quit IRC | 02:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:01 | |
| *** dims has quit IRC | 02:03 | |
| *** apoorvad has quit IRC | 02:03 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 02:05 |
| *** dims has joined #openstack-cinder | 02:05 | |
| *** akshai has joined #openstack-cinder | 02:05 | |
| *** akshai_ has joined #openstack-cinder | 02:07 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Check the QoS status before we use https://review.openstack.org/271601 | 02:08 |
| *** mylu has joined #openstack-cinder | 02:08 | |
| *** mylu has quit IRC | 02:09 | |
| *** akshai has quit IRC | 02:10 | |
| *** Lee1092 has joined #openstack-cinder | 02:11 | |
| *** baojg has quit IRC | 02:17 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Check when attach hypermetro volume https://review.openstack.org/283964 | 02:18 |
| *** mylu has joined #openstack-cinder | 02:20 | |
| *** baojg has joined #openstack-cinder | 02:20 | |
| *** laughterwym has quit IRC | 02:20 | |
| *** mylu has quit IRC | 02:21 | |
| *** mylu has joined #openstack-cinder | 02:25 | |
| *** mylu has quit IRC | 02:27 | |
| *** mylu has joined #openstack-cinder | 02:27 | |
| *** mylu has quit IRC | 02:29 | |
| *** mylu has joined #openstack-cinder | 02:34 | |
| *** RustShen has joined #openstack-cinder | 02:34 | |
| *** laughterwym has joined #openstack-cinder | 02:35 | |
| *** mylu has quit IRC | 02:35 | |
| *** mylu has joined #openstack-cinder | 02:35 | |
| *** houming has joined #openstack-cinder | 02:36 | |
| *** mylu has quit IRC | 02:40 | |
| *** RustShen has quit IRC | 02:46 | |
| *** mylu has joined #openstack-cinder | 02:49 | |
| *** gcb has quit IRC | 02:51 | |
| *** krtaylor has quit IRC | 02:58 | |
| *** mylu has quit IRC | 02:59 | |
| *** haomaiwa_ has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** mylu has joined #openstack-cinder | 03:03 | |
| *** gcb has joined #openstack-cinder | 03:04 | |
| *** gouthamr has quit IRC | 03:06 | |
| openstackgerrit | Merged openstack/cinder-specs: Minor format fixes for Cinder Mitaka specs https://review.openstack.org/293069 | 03:06 |
| *** baojg has quit IRC | 03:08 | |
| *** krtaylor has joined #openstack-cinder | 03:10 | |
| *** baojg has joined #openstack-cinder | 03:10 | |
| *** gcb has quit IRC | 03:11 | |
| *** esker has quit IRC | 03:11 | |
| *** mylu has quit IRC | 03:13 | |
| *** houming has quit IRC | 03:14 | |
| *** houming_ has joined #openstack-cinder | 03:14 | |
| *** mylu has joined #openstack-cinder | 03:16 | |
| *** dave-mccowan has quit IRC | 03:17 | |
| *** mylu has quit IRC | 03:18 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 03:24 |
| *** gcb has joined #openstack-cinder | 03:26 | |
| *** mylu has joined #openstack-cinder | 03:27 | |
| *** mylu has quit IRC | 03:29 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Check when attach hypermetro volume https://review.openstack.org/283964 | 03:32 |
| *** dims has quit IRC | 03:37 | |
| *** baojg has quit IRC | 03:42 | |
| *** baojg has joined #openstack-cinder | 03:43 | |
| *** shyama has joined #openstack-cinder | 03:43 | |
| *** laughterwym has quit IRC | 03:45 | |
| *** aswadr has joined #openstack-cinder | 03:47 | |
| *** laughterwym has joined #openstack-cinder | 03:50 | |
| *** akshai_ has quit IRC | 03:52 | |
| *** cbinx has joined #openstack-cinder | 03:52 | |
| *** aswadr_ has joined #openstack-cinder | 03:53 | |
| openstackgerrit | Rust Shen proposed openstack/cinder: Translating msgs https://review.openstack.org/293242 | 03:57 |
| *** haomaiwang has quit IRC | 04:01 | |
| *** esker has joined #openstack-cinder | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** cknight1 has joined #openstack-cinder | 04:03 | |
| openstackgerrit | Merged openstack/cinder: Revert "VNX: Set timeout for naviseccli" https://review.openstack.org/292673 | 04:03 |
| *** cknight has quit IRC | 04:04 | |
| *** knikolla has quit IRC | 04:06 | |
| *** esker has quit IRC | 04:06 | |
| *** links has joined #openstack-cinder | 04:10 | |
| *** laughterwym has quit IRC | 04:14 | |
| *** laughterwym has joined #openstack-cinder | 04:15 | |
| *** laughterwym has quit IRC | 04:19 | |
| *** [1]Thelo has joined #openstack-cinder | 04:34 | |
| *** akshai has joined #openstack-cinder | 04:36 | |
| openstackgerrit | zhangguoqing proposed openstack/cinder: Fix assignment errors https://review.openstack.org/292832 | 04:37 |
| *** Thelo has quit IRC | 04:37 | |
| *** [1]Thelo is now known as Thelo | 04:37 | |
| *** cknight1 has quit IRC | 04:38 | |
| *** laughterwym has joined #openstack-cinder | 04:43 | |
| *** laughterwym has quit IRC | 04:43 | |
| *** laughterwym has joined #openstack-cinder | 04:43 | |
| *** mylu has joined #openstack-cinder | 04:44 | |
| *** baojg has quit IRC | 04:45 | |
| *** mylu has quit IRC | 04:45 | |
| *** gcb has quit IRC | 04:46 | |
| *** sgotliv has joined #openstack-cinder | 04:57 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| openstackgerrit | NGUYEN TUONG THANH proposed openstack/python-cinderclient: Fixes typo in cinderclient shell. https://review.openstack.org/293249 | 05:01 |
| *** sgotliv has quit IRC | 05:03 | |
| *** akshai_ has joined #openstack-cinder | 05:04 | |
| *** gcb has joined #openstack-cinder | 05:05 | |
| *** mylu has joined #openstack-cinder | 05:06 | |
| *** akshai has quit IRC | 05:07 | |
| *** boris-42 has joined #openstack-cinder | 05:13 | |
| *** bardia has joined #openstack-cinder | 05:16 | |
| *** Julien-zte has joined #openstack-cinder | 05:18 | |
| openstackgerrit | Merged openstack/cinder: Huawei: Implement v2.1 replication https://review.openstack.org/292037 | 05:18 |
| *** sgotliv has joined #openstack-cinder | 05:30 | |
| *** esker has joined #openstack-cinder | 05:33 | |
| *** esker has quit IRC | 05:38 | |
| *** sgotliv has quit IRC | 05:40 | |
| *** Poornima has joined #openstack-cinder | 05:40 | |
| *** panda has quit IRC | 05:40 | |
| *** panda has joined #openstack-cinder | 05:41 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/os-brick: Ensure that lun_id is integer before use '&' operand https://review.openstack.org/291801 | 05:41 |
| openstackgerrit | Merged openstack/cinder: LeftHand: Add default SSH timeout and key values https://review.openstack.org/291956 | 05:43 |
| *** akshai_ has quit IRC | 05:44 | |
| *** david-lyle has quit IRC | 05:45 | |
| *** david-lyle has joined #openstack-cinder | 05:45 | |
| *** mylu has quit IRC | 05:46 | |
| *** deepakcs has joined #openstack-cinder | 05:48 | |
| *** laughterwym has quit IRC | 05:49 | |
| *** mylu has joined #openstack-cinder | 05:52 | |
| openstackgerrit | Merged openstack/cinder: Pass correct source_id to _handle_bootable_volume_glance_meta() https://review.openstack.org/293173 | 05:52 |
| *** yangyape_ has joined #openstack-cinder | 05:56 | |
| *** mylu has quit IRC | 05:57 | |
| *** liverpooler has quit IRC | 05:59 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Scrub residual device before connect volume https://review.openstack.org/293259 | 05:59 |
| *** yangyapeng has quit IRC | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** laughterwym has joined #openstack-cinder | 06:03 | |
| *** shyama has quit IRC | 06:04 | |
| *** laughterwym has quit IRC | 06:05 | |
| *** yangyapeng has joined #openstack-cinder | 06:07 | |
| *** laughterwym has joined #openstack-cinder | 06:07 | |
| *** Julien-zte has quit IRC | 06:09 | |
| *** yangyape_ has quit IRC | 06:10 | |
| *** sgotliv has joined #openstack-cinder | 06:18 | |
| *** aswadr has quit IRC | 06:18 | |
| openstackgerrit | NGUYEN TUONG THANH proposed openstack/python-cinderclient: Fixes typo in cinderclient shell. https://review.openstack.org/293249 | 06:20 |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Retry on failure of starting migration https://review.openstack.org/258294 | 06:27 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers https://review.openstack.org/283029 | 06:28 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 06:31 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn https://review.openstack.org/271592 | 06:35 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn https://review.openstack.org/271592 | 06:43 |
| *** ChubYann has quit IRC | 06:45 | |
| *** esker has joined #openstack-cinder | 06:49 | |
| *** ishant has joined #openstack-cinder | 06:50 | |
| *** esker has quit IRC | 06:54 | |
| *** asselin_ has quit IRC | 06:57 | |
| *** bjornar has quit IRC | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** laughterwym has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** laughterwym has joined #openstack-cinder | 07:01 | |
| *** laughterwym has quit IRC | 07:01 | |
| *** yinweiishere has joined #openstack-cinder | 07:02 | |
| *** laughterwym has joined #openstack-cinder | 07:03 | |
| *** laughterwym has quit IRC | 07:07 | |
| *** mjura has joined #openstack-cinder | 07:07 | |
| *** arecknag has joined #openstack-cinder | 07:10 | |
| *** bardia has quit IRC | 07:11 | |
| *** shyama has joined #openstack-cinder | 07:15 | |
| *** lpetrut has joined #openstack-cinder | 07:15 | |
| *** tesseract has joined #openstack-cinder | 07:16 | |
| *** tesseract is now known as Guest64967 | 07:17 | |
| *** nkrinner has joined #openstack-cinder | 07:17 | |
| *** laughterwym has joined #openstack-cinder | 07:18 | |
| *** Guest64967 has quit IRC | 07:21 | |
| *** AJaeger has joined #openstack-cinder | 07:23 | |
| *** esker has joined #openstack-cinder | 07:23 | |
| AJaeger | https://review.openstack.org/292732 is the translation import and has one +A but due to zuul problems did not merge, could somebody +A again, please? | 07:24 |
| *** laughterwym has quit IRC | 07:24 | |
| *** esker has quit IRC | 07:28 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Check volume_id consistent when creating backup https://review.openstack.org/293285 | 07:29 |
| *** Julien-zte has joined #openstack-cinder | 07:35 | |
| *** gcb has quit IRC | 07:36 | |
| *** tesseract- has joined #openstack-cinder | 07:36 | |
| *** sgotliv has quit IRC | 07:38 | |
| *** sgotliv has joined #openstack-cinder | 07:42 | |
| *** chlong has quit IRC | 07:42 | |
| *** yangds has quit IRC | 07:43 | |
| *** yangds has joined #openstack-cinder | 07:43 | |
| *** gcb has joined #openstack-cinder | 07:49 | |
| *** liverpooler has joined #openstack-cinder | 07:49 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Scrub residual device before connect volume https://review.openstack.org/293259 | 07:52 |
| *** klkumar has joined #openstack-cinder | 07:54 | |
| *** sgotliv has quit IRC | 07:55 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Enhance migration start verification https://review.openstack.org/271143 | 07:55 |
| openstackgerrit | Peter Wang proposed openstack/cinder: Fix race condition when toggling SP https://review.openstack.org/262135 | 07:55 |
| *** jwcroppe has quit IRC | 07:57 | |
| *** belmoreira has joined #openstack-cinder | 07:58 | |
| *** jwcroppe has joined #openstack-cinder | 07:58 | |
| *** Julien-zte has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** jwcroppe has quit IRC | 08:02 | |
| *** lpetrut has quit IRC | 08:04 | |
| *** laughterwym has joined #openstack-cinder | 08:11 | |
| *** liverpooler has quit IRC | 08:14 | |
| *** liverpooler has joined #openstack-cinder | 08:14 | |
| sheel | DuncanT: hey there | 08:15 |
| sheel | DuncanT: need your opinion on one fix | 08:16 |
| sheel | DuncanT: please let me know when you have spare time for discussion | 08:16 |
| sheel | DuncanT: Reference : https://bugs.launchpad.net/cinder/+bug/1555938 | 08:17 |
| openstack | Launchpad bug 1555938 in Cinder "Create snapshot, create volume from snapshot or clone a volume still can succeed even the cinder-volume service is disabled" [Low,Confirmed] - Assigned to aditi sharma (adi-sky17) | 08:17 |
| *** permalac has joined #openstack-cinder | 08:20 | |
| *** kbyrne has joined #openstack-cinder | 08:22 | |
| *** EinstCrazy has quit IRC | 08:23 | |
| *** yangyapeng has quit IRC | 08:23 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder-specs: Enhance error messages for API user https://review.openstack.org/293306 | 08:24 |
| *** yangyapeng has joined #openstack-cinder | 08:26 | |
| lixiaoy1 | shee: DuncanT: how about check service state here: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L792 | 08:27 |
| lixiaoy1 | sheel: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L792 | 08:28 |
| *** EinstCrazy has joined #openstack-cinder | 08:30 | |
| *** shyama has quit IRC | 08:32 | |
| *** ig0r_ has quit IRC | 08:39 | |
| *** andymaier has joined #openstack-cinder | 08:45 | |
| *** AJaeger has left #openstack-cinder | 08:49 | |
| openstackgerrit | wanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly https://review.openstack.org/232436 | 08:53 |
| sheel | lixiaoy1: hi | 08:54 |
| sheel | lixiaoy1: checking service status seems ok | 08:54 |
| *** dalgaaf has joined #openstack-cinder | 08:55 | |
| *** yangyapeng has quit IRC | 08:56 | |
| sheel | lixiaoy1: need to dig more | 08:58 |
| *** aswadr has joined #openstack-cinder | 08:58 | |
| sheel | lixiaoy1: for snapshot creation... | 08:58 |
| lixiaoy1 | sheel: I add my comments in the bug, and seems more problems about disabled service | 08:58 |
| sheel | lixiaoy1: seems you are proposing volume status check for all operations | 08:59 |
| sheel | lixiaoy1: is it? | 08:59 |
| lixiaoy1 | sheel: for example, backup service talked with volume service through volume.rpcapi, this is not through sheduler. Should it return successful result? | 08:59 |
| lixiaoy1 | sheel: if the volume service is disabled. | 09:00 |
| sheel | lixiaoy1: it should not | 09:00 |
| sheel | lixiaoy1: then scheduler related fix will be failure | 09:00 |
| sheel | lixiaoy1: for the cases you mentioned | 09:00 |
| *** haomaiwang has quit IRC | 09:01 | |
| sheel | lixiaoy1: but checking status at volume service level seems heavy | 09:01 |
| sheel | lixiaoy1: what you say? | 09:01 |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| lixiaoy1 | sheel: yes. | 09:01 |
| sheel | lixiaoy1: means for each operation this will happen.. may be if possible to bound this check for some problematic cases only then it will be good to follow | 09:01 |
| sheel | checking status in volume service(for only problematic cases which do not pass through scheduler) | 09:02 |
| sheel | lixiaoy1: nice suggestion | 09:02 |
| lixiaoy1 | sheel: if we add check in scheduler, this prevents any calls through scheduler. but can't prevent the direct call through volume.rpcapi | 09:03 |
| sheel | lixiaoy1: exactly, agreed... | 09:04 |
| lixiaoy1 | sheel: checking status in volume service, not sure how this is done. if adding in functions, seems not good | 09:04 |
| sheel | lixiaoy1: some decorator should work for specific functionality where flow does not pass through scheduler | 09:05 |
| lixiaoy1 | sheel: it is better that you propose this problem in today's meeting | 09:05 |
| sheel | lixiaoy1: yes, sounds good | 09:05 |
| lixiaoy1 | sheel: I agree. decorator | 09:05 |
| lixiaoy1 | sheel: maybe we can have a decision what kinds of operations/functions should return failure when service is disabled in the meeting | 09:06 |
| lixiaoy1 | sheel: I am a little confused | 09:06 |
| sheel | lixiaoy1: i was checking code for manager | 09:07 |
| lixiaoy1 | sheel: good luck! I am leaving office now, and will check the meeting minutes tomorrow. :) The time is too late for me. | 09:08 |
| sheel | lixiaoy1: oh sure | 09:08 |
| sheel | lixiaoy1: will share with you | 09:08 |
| sheel | lixiaoy1: good day | 09:09 |
| lixiaoy1 | sheel: thanks. good day | 09:09 |
| sheel | lixiaoy1: thank you | 09:09 |
| *** wanghao has quit IRC | 09:09 | |
| lixiaoy1 | sheel: my pleasure | 09:09 |
| *** wanghao has joined #openstack-cinder | 09:09 | |
| *** salv-orlando has joined #openstack-cinder | 09:11 | |
| *** aswadr has left #openstack-cinder | 09:12 | |
| *** esker has joined #openstack-cinder | 09:12 | |
| *** esker has quit IRC | 09:16 | |
| sheel | :) | 09:18 |
| *** shyama has joined #openstack-cinder | 09:18 | |
| *** sgotliv has joined #openstack-cinder | 09:19 | |
| *** anshul has joined #openstack-cinder | 09:20 | |
| *** bjornar has joined #openstack-cinder | 09:22 | |
| *** ndipanov has joined #openstack-cinder | 09:22 | |
| *** jistr has joined #openstack-cinder | 09:24 | |
| *** jordanP has joined #openstack-cinder | 09:25 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:27 | |
| *** esker has joined #openstack-cinder | 09:32 | |
| *** ankit_ag has joined #openstack-cinder | 09:33 | |
| DuncanT | sheel: I think that the side effects of changing that behaviour are too risky to merge anything before the RC now (which is probably going to be cut today) | 09:33 |
| sheel | DuncanT: hi | 09:33 |
| sheel | DuncanT: yes, we should point this in newton | 09:34 |
| sheel | DuncanT: for now this was for discussion only :) | 09:34 |
| sheel | DuncanT: what you think about adding decorator to check service status for some specific operations which do not pass through scheduler | 09:34 |
| DuncanT | sheel: Need to figure out how this interacts with A/A | 09:35 |
| sheel | DuncanT: ohk, I have kept it for today's discussion in cinder meeting | 09:35 |
| DuncanT | sheel: Maybe we need a separate control and work queue to take to c-vol, then a disable c-vol service can just disconnect its work queue, and rabbit takes care of things? | 09:36 |
| sheel | DuncanT: seem quick and easy fix... | 09:36 |
| DuncanT | I suspect most of today's time will be spent on the release - we need people to focus on beating on the RC and working through bugs to figure out which really need fixing before release | 09:36 |
| DuncanT | sheel: I wouldn't say quick and easy, but we ought to look at our queues at some point from a security prospective anyway, and this could feed into that as a requirement | 09:37 |
| *** mjura has quit IRC | 09:37 | |
| *** cbinx has quit IRC | 09:38 | |
| sheel | DuncanT: :) right... i rephrase... optimized fix | 09:38 |
| sheel | DuncanT: I understand release pressure for today, anything i can help ? | 09:39 |
| *** esker has quit IRC | 09:39 | |
| sheel | (I removed discussion from today's meeting will discuss later on) | 09:39 |
| *** panda has quit IRC | 09:40 | |
| *** panda has joined #openstack-cinder | 09:41 | |
| DuncanT | sheel: I'm trying to fill out https://etherpad.openstack.org/p/cinder-mataka-release-final-push - if you want to start going through pending reviews and pull out any that look high priority and ready and add them to that list, it would be useful | 09:41 |
| sheel | DuncanT: sure I will do... | 09:44 |
| *** dims has joined #openstack-cinder | 09:51 | |
| *** shyama has quit IRC | 09:52 | |
| *** openstackgerrit has quit IRC | 09:53 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:53 | |
| *** openstackgerrit has quit IRC | 09:53 | |
| *** openstackgerrit_ has joined #openstack-cinder | 09:54 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:54 | |
| zigo | What's the purpose of python-brick-cinderclient-ext? | 09:55 |
| *** openstackgerrit_ has joined #openstack-cinder | 09:55 | |
| *** openstackgerrit has quit IRC | 09:55 | |
| *** openstackgerrit has joined #openstack-cinder | 09:56 | |
| dulek | zigo: It's to allow any other service to use Cinder like Nova does. | 09:59 |
| dulek | zigo: Attach/detach APIs aren't exposed in python-cinderclient. | 09:59 |
| zigo | dulek: Why not adding attach/detach API to cinderclient then? | 09:59 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| dulek | zigo: We certainly don't want the user or admin to mess with that. attach and detach flows are de-facto 3 API calls each, that need to be called in a well-defined way. | 10:03 |
| zigo | ok | 10:03 |
| *** sdague has joined #openstack-cinder | 10:03 | |
| dulek | zigo: You're probably asking in context of packaging? e0ne will be able to help you to asses if it's worth to do that. | 10:04 |
| dulek | zigo: He should be here in an hour or so. | 10:04 |
| zigo | dulek: Correct, I'm trying to see if I need to do it or not. | 10:04 |
| dulek | zigo: I doubt this will be a dependency for any package now. But certainly sooner or later someone will want to use it in his project. | 10:06 |
| dulek | I | 10:06 |
| dulek | I'll defer to e0ne's opinion on that. | 10:07 |
| *** tpsilva has joined #openstack-cinder | 10:07 | |
| zigo | dulek: Right, so I may skip it for Mitaka, and start working on it for Newton. | 10:07 |
| *** mylu has joined #openstack-cinder | 10:08 | |
| *** lpetrut has joined #openstack-cinder | 10:09 | |
| *** shyama has joined #openstack-cinder | 10:11 | |
| *** sdague has quit IRC | 10:11 | |
| *** sdague has joined #openstack-cinder | 10:12 | |
| *** mylu has quit IRC | 10:13 | |
| *** cdelatte has quit IRC | 10:19 | |
| sheel | DuncanT: done for pending reviews since last 10 days | 10:19 |
| DuncanT | sheel: That's great, thanks! I'm working through them slowly | 10:20 |
| sheel | DuncanT: you are welcome.. | 10:20 |
| *** yangyapeng has joined #openstack-cinder | 10:23 | |
| *** yangyapeng has quit IRC | 10:23 | |
| *** EinstCrazy has quit IRC | 10:34 | |
| *** gcb has quit IRC | 10:40 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Scrub residual device before connect volume https://review.openstack.org/293259 | 10:41 |
| *** mkrcmari__ has quit IRC | 10:42 | |
| *** lprice has joined #openstack-cinder | 10:47 | |
| *** lprice1 has quit IRC | 10:47 | |
| *** gcb has joined #openstack-cinder | 10:52 | |
| *** dims has quit IRC | 10:53 | |
| *** ociuhandu has quit IRC | 10:55 | |
| *** salv-orlando has quit IRC | 10:56 | |
| *** pece has joined #openstack-cinder | 10:59 | |
| *** dims has joined #openstack-cinder | 10:59 | |
| *** yhayashi has quit IRC | 10:59 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** esker has joined #openstack-cinder | 11:04 | |
| *** esker has quit IRC | 11:08 | |
| *** iceyao has joined #openstack-cinder | 11:11 | |
| *** mkrcmari__ has joined #openstack-cinder | 11:11 | |
| *** merooney has joined #openstack-cinder | 11:12 | |
| *** klkumar has quit IRC | 11:15 | |
| *** merooney has quit IRC | 11:17 | |
| *** openstackgerrit has quit IRC | 11:18 | |
| *** openstackgerrit has joined #openstack-cinder | 11:19 | |
| *** avishay has joined #openstack-cinder | 11:19 | |
| *** merooney has joined #openstack-cinder | 11:22 | |
| *** haomaiwang has quit IRC | 11:24 | |
| *** klkumar has joined #openstack-cinder | 11:29 | |
| *** avishay has quit IRC | 11:30 | |
| *** avishay has joined #openstack-cinder | 11:30 | |
| dulek | I've just installed Cinder in venv and it complained about PyMySQL missing. I've installed it manually and it worked. | 11:31 |
| dulek | Don't we miss that from requirements? | 11:31 |
| dulek | Hm, it's in global-requirements, but Nova doesn't have it in requirements.txt either. | 11:32 |
| *** robby_ has joined #openstack-cinder | 11:32 | |
| *** iceyao has quit IRC | 11:32 | |
| *** Lee1092 has quit IRC | 11:32 | |
| dulek | I guess it's not a hard dependency as we | 11:32 |
| dulek | as we allow many DB backends. Am I correct? | 11:32 |
| *** shyama_ has joined #openstack-cinder | 11:42 | |
| *** shyama has quit IRC | 11:43 | |
| *** ishant has quit IRC | 11:43 | |
| *** erlon has joined #openstack-cinder | 11:44 | |
| *** cdelatte has joined #openstack-cinder | 11:44 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add devref on rolling upgrades https://review.openstack.org/279186 | 11:45 |
| *** cdelatte has quit IRC | 11:46 | |
| *** shyama_ has quit IRC | 11:46 | |
| *** cdelatte has joined #openstack-cinder | 11:48 | |
| *** dave-mccowan has joined #openstack-cinder | 11:52 | |
| *** Lee1092 has joined #openstack-cinder | 11:52 | |
| openstackgerrit | xiexs proposed openstack/cinder: Make dict.keys() PY3 compatible https://review.openstack.org/293387 | 11:52 |
| openstackgerrit | Merged openstack/cinder: Huawei: Check when attach hypermetro volume https://review.openstack.org/283964 | 11:53 |
| *** yangyapeng has joined #openstack-cinder | 11:56 | |
| *** mylu has joined #openstack-cinder | 11:56 | |
| *** salv-orlando has joined #openstack-cinder | 11:57 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Make dict.keys() PY3 compatible https://review.openstack.org/293388 | 11:59 |
| *** merooney has quit IRC | 12:00 | |
| *** raildo-afk is now known as raildo | 12:00 | |
| *** pece has quit IRC | 12:01 | |
| *** robby_ has quit IRC | 12:02 | |
| *** mylu has quit IRC | 12:03 | |
| *** merooney has joined #openstack-cinder | 12:06 | |
| *** yangds has quit IRC | 12:07 | |
| openstackgerrit | Merged openstack/cinder: Huawei: Check the QoS status before we use https://review.openstack.org/271601 | 12:07 |
| *** crose has joined #openstack-cinder | 12:08 | |
| *** pece has joined #openstack-cinder | 12:09 | |
| *** pece has quit IRC | 12:10 | |
| *** pece has joined #openstack-cinder | 12:13 | |
| *** haomaiwang has joined #openstack-cinder | 12:16 | |
| *** zhanghao has joined #openstack-cinder | 12:17 | |
| *** haomaiwang has quit IRC | 12:20 | |
| *** EinstCrazy has joined #openstack-cinder | 12:24 | |
| *** ociuhandu has joined #openstack-cinder | 12:25 | |
| *** rodrigods has quit IRC | 12:26 | |
| *** JoseMello has joined #openstack-cinder | 12:27 | |
| *** salv-orlando has quit IRC | 12:28 | |
| *** akshai has joined #openstack-cinder | 12:29 | |
| DuncanT | dulek: I think we only have hard dependencies on options other than the default (lvm, rabbit, mysql) | 12:31 |
| *** mylu has joined #openstack-cinder | 12:32 | |
| *** gouthamr has joined #openstack-cinder | 12:33 | |
| *** boris-42 has quit IRC | 12:34 | |
| *** mylu_ has joined #openstack-cinder | 12:35 | |
| *** diablo_rojo has joined #openstack-cinder | 12:35 | |
| *** crose has quit IRC | 12:36 | |
| *** mylu has quit IRC | 12:38 | |
| *** timcl has joined #openstack-cinder | 12:39 | |
| *** mylu_ has quit IRC | 12:40 | |
| *** merooney has quit IRC | 12:45 | |
| *** cknight has joined #openstack-cinder | 12:49 | |
| *** merooney has joined #openstack-cinder | 12:51 | |
| *** zhanghao1 has joined #openstack-cinder | 12:52 | |
| *** zhanghao has quit IRC | 12:53 | |
| *** eharney has joined #openstack-cinder | 12:57 | |
| *** smoriya_ has quit IRC | 12:59 | |
| *** rcernin has joined #openstack-cinder | 13:03 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: CloudByte: resize volume if cloned image is larger https://review.openstack.org/293412 | 13:05 |
| *** jwcroppe has joined #openstack-cinder | 13:08 | |
| *** kaisers1 has quit IRC | 13:09 | |
| *** rlrossit has joined #openstack-cinder | 13:10 | |
| *** e0ne has joined #openstack-cinder | 13:10 | |
| *** kro_sinus_hell is now known as krot_sinus_purga | 13:15 | |
| *** krot_sinus_purga is now known as kro_sniffles | 13:15 | |
| *** smcginnis_away is now known as smcginnis | 13:15 | |
| *** merooney has quit IRC | 13:17 | |
| *** iceyao has joined #openstack-cinder | 13:17 | |
| *** kaisers has joined #openstack-cinder | 13:19 | |
| *** gcb has quit IRC | 13:19 | |
| *** xyang1 has joined #openstack-cinder | 13:19 | |
| scottda_vacation | ildikov: Hi | 13:21 |
| *** yangds has joined #openstack-cinder | 13:24 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration https://review.openstack.org/286234 | 13:24 |
| *** ig0r_ has joined #openstack-cinder | 13:24 | |
| *** Poornima has quit IRC | 13:24 | |
| *** merooney has joined #openstack-cinder | 13:25 | |
| *** kaisers has quit IRC | 13:26 | |
| *** e0ne has quit IRC | 13:27 | |
| *** kaisers has joined #openstack-cinder | 13:28 | |
| *** martyturner has joined #openstack-cinder | 13:28 | |
| *** mriedem has joined #openstack-cinder | 13:28 | |
| *** salv-orlando has joined #openstack-cinder | 13:29 | |
| *** haomaiwang has joined #openstack-cinder | 13:29 | |
| *** gcb has joined #openstack-cinder | 13:33 | |
| *** deepakcs has quit IRC | 13:35 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Fix compatibility mode of backup jobs scheduling https://review.openstack.org/293435 | 13:36 |
| *** dustins has joined #openstack-cinder | 13:37 | |
| *** ishant has joined #openstack-cinder | 13:37 | |
| dulek | tbarron, lixiaoy1: I was retesting interoperability of services today and this issue came up ^. | 13:38 |
| smcginnis | dulek: Good catch! | 13:39 |
| dulek | smcginnis, DuncanT: Any chance of including that in RC-1? Should be low risk as it affects only backward-compat-mode of backup scheduling. | 13:40 |
| *** salv-orlando has quit IRC | 13:40 | |
| smcginnis | dulek: Yeah, looking at it now. | 13:40 |
| dulek | smcginnis: Thanks. :) | 13:40 |
| *** panda has quit IRC | 13:40 | |
| sheel | eharney: hi | 13:40 |
| smcginnis | dulek: Looks small enough. I think the risk is lower than the benefit for getting that in today. | 13:40 |
| *** panda has joined #openstack-cinder | 13:41 | |
| sheel | eharney: regarding your -2 on duplicate dictionary element patch | 13:41 |
| *** edmondsw has joined #openstack-cinder | 13:41 | |
| dulek | smcginnis: You mean not to target that for RC-1? Don't quite catch what you mean. | 13:42 |
| tbarron | dulek: glad you are testing this! | 13:42 |
| smcginnis | dulek: I do mean target it to RC-1. | 13:42 |
| smcginnis | dulek: I just mean when it gets down to the end it's about balancing risk. | 13:43 |
| smcginnis | dulek: If the risk is high and the benefit is low, then definitely no. | 13:43 |
| *** jgregor has joined #openstack-cinder | 13:43 | |
| smcginnis | dulek: But in this case the risk is low and the benefit is high, so definitely yes. | 13:43 |
| DuncanT | I've put a +2 on that patch, I think it's worth having in | 13:43 |
| smcginnis | dulek: Sorry for being confusing. ;) | 13:43 |
| dulek | smcginnis: Thanks, just had trouble understanding your wording. | 13:43 |
| dulek | smcginnis: There's one more thing - status of this one: https://blueprints.launchpad.net/cinder/+spec/online-schema-upgrades | 13:44 |
| smcginnis | dulek: That happens to folks often. | 13:44 |
| smcginnis | dulek: Oh, still work on that? | 13:44 |
| dulek | smcginnis: Somehow we've merged the unit test in RC-1. I wasn't really expecting that, but from my perspective this is like accepting the guidelines. | 13:44 |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: CloudByte: resize volume if cloned image is larger https://review.openstack.org/293412 | 13:44 |
| dulek | smcginnis: I've already marked that as completed - after unit test got merged. | 13:45 |
| dulek | smcginnis: But it's targeted for Newton, so we should probably retarget it for Mitaka? | 13:45 |
| smcginnis | dulek: The whiteboard has some other patches linked to that. | 13:45 |
| smcginnis | dulek: Looking through now trying to figure out what's what. | 13:46 |
| *** porrua has joined #openstack-cinder | 13:46 | |
| dulek | smcginnis: I think all of these should have Related-Blueprint, not Implements-Blueprint. | 13:46 |
| smcginnis | dulek: OK, I'll mark it as Mitaka. That's what you were saying, right? | 13:47 |
| dulek | smcginnis: Exactly. :) | 13:47 |
| dulek | Yay, we've got one more blueprint completed for this cycle! :) | 13:47 |
| smcginnis | :) | 13:47 |
| scottda_vacation | ildikov: Ping rescinded. I just wanted to check on the status of a cinder-nova meeting to discuss api fixes we thought about scheduling for next week. I'm offline now.... | 13:49 |
| smcginnis | scottda_vacation: You're doing vacation wrong. | 13:49 |
| openstackgerrit | Merged openstack/cinder-specs: Update online-schema-upgrades spec https://review.openstack.org/264744 | 13:50 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 13:51 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 13:51 |
| *** akerr has joined #openstack-cinder | 13:52 | |
| *** merooney has quit IRC | 13:52 | |
| *** mtanino has joined #openstack-cinder | 13:54 | |
| *** lprice1 has joined #openstack-cinder | 13:54 | |
| *** mriedem is now known as mriedem_meeting | 13:54 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Cleaning PEP 203 and dict duplicate values https://review.openstack.org/288296 | 13:55 |
| *** houming_ has quit IRC | 13:56 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Cleaned duplicate dictionary values https://review.openstack.org/288296 | 13:56 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Record and check LUN wwn https://review.openstack.org/271592 | 13:56 |
| *** ankit_ag has quit IRC | 13:56 | |
| *** lprice has quit IRC | 13:56 | |
| *** martyturner has quit IRC | 13:57 | |
| *** merooney has joined #openstack-cinder | 13:58 | |
| *** merooney has quit IRC | 13:58 | |
| *** merooney has joined #openstack-cinder | 13:58 | |
| *** baumann has joined #openstack-cinder | 13:59 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** martyturner has joined #openstack-cinder | 14:01 | |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| *** edtubill has joined #openstack-cinder | 14:02 | |
| *** daneyon has joined #openstack-cinder | 14:03 | |
| *** timcl has quit IRC | 14:04 | |
| *** mgagne has quit IRC | 14:07 | |
| *** mgagne has joined #openstack-cinder | 14:07 | |
| *** timcl has joined #openstack-cinder | 14:08 | |
| *** knikolla has joined #openstack-cinder | 14:13 | |
| *** daneyon has quit IRC | 14:15 | |
| *** salv-orlando has joined #openstack-cinder | 14:15 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum https://review.openstack.org/290885 | 14:30 |
| *** aswadr_ has quit IRC | 14:30 | |
| *** baojg has joined #openstack-cinder | 14:31 | |
| openstackgerrit | Vivek Dhayaal proposed openstack/cinder: Removed RPC topic config options https://review.openstack.org/288398 | 14:34 |
| *** [1]Thelo has joined #openstack-cinder | 14:35 | |
| *** iceyao has quit IRC | 14:36 | |
| smcginnis | Reminder to US folks - meeting is an hour later now due to daylight savings time. | 14:37 |
| *** Thelo has quit IRC | 14:37 | |
| *** [1]Thelo is now known as Thelo | 14:37 | |
| *** wanghao has quit IRC | 14:40 | |
| *** wanghao has joined #openstack-cinder | 14:41 | |
| *** asselin_ has joined #openstack-cinder | 14:44 | |
| *** daneyon has joined #openstack-cinder | 14:46 | |
| *** asselin__ has joined #openstack-cinder | 14:47 | |
| *** asselin_ has quit IRC | 14:48 | |
| diablo_rojo | smcginnis: Thanks :) I was oober confused yesterday with the cp meeting.. stupid daylight savings time | 14:49 |
| *** merooney has quit IRC | 14:51 | |
| smcginnis | diablo_rojo: ;) | 14:55 |
| *** merooney has joined #openstack-cinder | 14:56 | |
| *** mriedem_meeting is now known as mriedem | 14:56 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add SnapshotStatus enum field https://review.openstack.org/284825 | 14:59 |
| *** yangds has quit IRC | 15:00 | |
| *** boichev has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** timcl has quit IRC | 15:02 | |
| *** jungleboyj has joined #openstack-cinder | 15:05 | |
| *** garthb has joined #openstack-cinder | 15:05 | |
| *** asselin__ has quit IRC | 15:08 | |
| *** timcl has joined #openstack-cinder | 15:09 | |
| *** vivekdhayaal has joined #openstack-cinder | 15:09 | |
| *** edtubill has quit IRC | 15:13 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Fix formatting in vol/snap delete API messages https://review.openstack.org/293492 | 15:13 |
| hemna | friggin DST | 15:14 |
| *** david-lyle has quit IRC | 15:15 | |
| *** david-lyle has joined #openstack-cinder | 15:15 | |
| smcginnis | hemna: ;) | 15:15 |
| bswartz | We need permanent DST -- get rid of "standard" time | 15:15 |
| Swanson | bswartz, yes! | 15:16 |
| *** ociuhandu has quit IRC | 15:16 | |
| *** knikolla has quit IRC | 15:17 | |
| *** liverpooler has quit IRC | 15:17 | |
| *** knikolla has joined #openstack-cinder | 15:22 | |
| *** ociuhandu has joined #openstack-cinder | 15:23 | |
| smcginnis | Just causes trouble. | 15:24 |
| jungleboyj | smcginnis: Is there a wiki that explains how we do the new documentation process? | 15:25 |
| Swanson | smcginnis, Pushes rush hour into the daylight where people seem capable of driving. | 15:25 |
| *** arecknag has quit IRC | 15:25 | |
| * jungleboyj likes living where there is no rush hour. :-) | 15:25 | |
| Swanson | jungleboyj, me, too. And once DST hits the roads are clear by the time I go home. | 15:26 |
| smcginnis | Swanson: Yeah, that's what happens when you don't come in to work until midday. | 15:26 |
| bswartz | lol | 15:26 |
| Swanson | smcginnis, One of the advantages, yes. | 15:27 |
| * bswartz remembers the days when he could work 2pm - midnight | 15:27 | |
| smcginnis | Service industry jobs changed me into a morning person. Reluctantly. | 15:28 |
| smcginnis | Now if I'm not out the door before 8 I feel like I've wasted time. | 15:28 |
| smcginnis | It's horrible. | 15:28 |
| Swanson | At a previous job we had 6am conference calls without using a conference number. so you had to be in the conference room. Literally thought it was going to kill me. | 15:29 |
| *** vivekdhayaal has quit IRC | 15:30 | |
| *** vivekd2 has joined #openstack-cinder | 15:30 | |
| *** david-lyle_ has joined #openstack-cinder | 15:34 | |
| *** david-lyle has quit IRC | 15:34 | |
| *** vivekd2 has quit IRC | 15:35 | |
| *** anshul has quit IRC | 15:37 | |
| *** jroll has quit IRC | 15:41 | |
| *** jroll has joined #openstack-cinder | 15:41 | |
| *** jroll has quit IRC | 15:41 | |
| *** jroll has joined #openstack-cinder | 15:41 | |
| *** vivekdhayaal has joined #openstack-cinder | 15:41 | |
| *** shyama has joined #openstack-cinder | 15:42 | |
| *** kprabhu has joined #openstack-cinder | 15:45 | |
| *** bjornar has quit IRC | 15:47 | |
| *** mylu has joined #openstack-cinder | 15:47 | |
| *** apoorvad has joined #openstack-cinder | 15:52 | |
| bswartz | Swanson: I doubt you had many issues with traffic at least | 15:54 |
| *** kprabhu has quit IRC | 15:54 | |
| bswartz | now my daughter has to be on schoolbus at 7:05, so we became morning people | 15:54 |
| Swanson | bswartz, I'd get in the car (so far as I know), time would warp and I'd be in the work parking lot. What happened between those two events and what damaged I caused is anyones guess. | 15:55 |
| diablo_rojo | During high school I had to be on the bus at 5:45 AM | 15:55 |
| Swanson | I took an open study hall at 8am so I could show up at school an hour later. was bliss. | 15:56 |
| Swanson | (For the 2 quarters I had the credits to do it.) | 15:56 |
| hemna | my son has to be at the bus stop at 6:55am | 15:56 |
| hemna | I had to become a morning person | 15:56 |
| bswartz | hemna diablo_rojo: >_< | 15:57 |
| diablo_rojo | No matter how hard I try I can't do it. | 15:57 |
| diablo_rojo | I'm just better at staying up late than waking up early. | 15:57 |
| bswartz | diablo_rojo: you can always try doing both... | 15:58 |
| bswartz | that assumes you consider sanity optional though | 15:58 |
| diablo_rojo | bswartz: I definitely have done that before..thats how I ended up drinking Monster to function and redbull on top of that if I needed to actually have energy. | 15:59 |
| *** laughterwym has quit IRC | 15:59 | |
| *** laughterwym has joined #openstack-cinder | 15:59 | |
| bswartz | smcginnis: meeting today? | 16:00 |
| smcginnis | Yep, waiting for storyboard. | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| *** martyturner has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** e0ne has joined #openstack-cinder | 16:01 | |
| *** klkumar has quit IRC | 16:01 | |
| *** merooney has quit IRC | 16:01 | |
| *** cknight has quit IRC | 16:02 | |
| *** baojg has quit IRC | 16:03 | |
| *** laughterwym has quit IRC | 16:03 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR fix create_cloned_volume for larger size https://review.openstack.org/291850 | 16:04 |
| *** martyturner has joined #openstack-cinder | 16:04 | |
| *** avishay has quit IRC | 16:06 | |
| *** crose has joined #openstack-cinder | 16:06 | |
| *** dustins has quit IRC | 16:07 | |
| *** IlyaG has joined #openstack-cinder | 16:10 | |
| *** EinstCrazy has quit IRC | 16:11 | |
| *** belmoreira has quit IRC | 16:11 | |
| *** kfarr has joined #openstack-cinder | 16:17 | |
| *** adrianofr_ has joined #openstack-cinder | 16:18 | |
| *** david-lyle_ is now known as david-lyle | 16:20 | |
| *** IlyaG has quit IRC | 16:22 | |
| *** ebalduf has joined #openstack-cinder | 16:22 | |
| *** IlyaG has joined #openstack-cinder | 16:23 | |
| *** crose has quit IRC | 16:28 | |
| *** cknight has joined #openstack-cinder | 16:30 | |
| *** crose has joined #openstack-cinder | 16:31 | |
| *** timcl has quit IRC | 16:31 | |
| *** bardia has joined #openstack-cinder | 16:32 | |
| *** crose has quit IRC | 16:34 | |
| *** laughterwym has joined #openstack-cinder | 16:36 | |
| *** chris_morrell has joined #openstack-cinder | 16:38 | |
| *** links has quit IRC | 16:38 | |
| *** chris_morrell has quit IRC | 16:42 | |
| *** chris_morrell has joined #openstack-cinder | 16:44 | |
| *** laughterwym has quit IRC | 16:44 | |
| *** chris_morrell has quit IRC | 16:45 | |
| *** e0ne has quit IRC | 16:47 | |
| *** e0ne_ has joined #openstack-cinder | 16:47 | |
| smcginnis | e0ne_: Yeah, but stable branches on clients are kind of just a reference. | 16:48 |
| *** timcl has joined #openstack-cinder | 16:49 | |
| smcginnis | e0ne_: We don't ever backport and release a new stable release. | 16:49 |
| smcginnis | Just always move forward. | 16:49 |
| smcginnis | At least so far. | 16:49 |
| *** vincent_hou has joined #openstack-cinder | 16:49 | |
| Swanson | The use of unknown, infinite or unavailable (in my case) going to be a problem for anyone else? | 16:49 |
| *** e0ne has joined #openstack-cinder | 16:50 | |
| *** yuriy_n17 has quit IRC | 16:50 | |
| *** e0ne_ has quit IRC | 16:50 | |
| erlon | smcginnis: hi, Sean, I have a question, ... | 16:51 |
| smcginnis | erlon: Sure, what's up? | 16:51 |
| *** yuriy_n17 has joined #openstack-cinder | 16:51 | |
| smcginnis | Swanson: I thought there were still a couple out there that did that. | 16:51 |
| Swanson | smcginnis, There are. | 16:51 |
| erlon | smcginnis: similar to JayXu's, we need to re-factor the Hitachi driver, are thinking about the best way to make it easier to reviewers | 16:52 |
| *** merooney has joined #openstack-cinder | 16:52 | |
| smcginnis | erlon: If you can do it in smaller steps, I think that helps. | 16:53 |
| smcginnis | erlon: It will be easier to review and easier to get through. | 16:53 |
| erlon | smcginnis: do you think its ok to send the re-factored driver with a new name, and then re-name it to the old name? | 16:53 |
| *** mylu has quit IRC | 16:53 | |
| smcginnis | erlon: You can certainly go the route proposed for VNX and do a whole new driver. | 16:53 |
| *** e0ne has quit IRC | 16:53 | |
| smcginnis | erlon: But I think that might be overkill. | 16:53 |
| fernnest | smcginnis, I'm working w/ HItachi and we have the same new driver issue as VNX | 16:53 |
| smcginnis | erlon: And add extra work. | 16:54 |
| fernnest | smcginnis, how's the best way to Tee this up for Newton? | 16:54 |
| erlon | smcginnis: for the size of the proposed changes I think that that could be a solution | 16:54 |
| smcginnis | erlon: So instead of deprecating the existing and moving to a new one, do side-by-side, then switch before the release? | 16:54 |
| *** nkrinner has quit IRC | 16:54 | |
| DuncanT | A chain of small changes is usually better for all sorts of reasons | 16:55 |
| erlon | fernnest: HBSD driver? | 16:55 |
| smcginnis | fernnest: Are you working with erlon? Or are there multiple Hitachi drivers changing here?? | 16:55 |
| fernnest | smcginnis,erlon: yes, we are considering putting in new file names and doing a rename at thend. | 16:55 |
| erlon | smcginnis: yes | 16:55 |
| smcginnis | erlon: I would agree with DuncanT. Some smaller patches to evolve the driver are probably better overall. | 16:55 |
| smcginnis | erlon: But if you want to do the new driver and switch, I suppose that's not a horrible idea. | 16:55 |
| fernnest | smcginnis,erlon: we are looking to break it up into 1000line chunks. | 16:56 |
| * smcginnis shudders | 16:56 | |
| erlon | smcginnis: the problem is that the work to split the paches in smaller steps are very big | 16:56 |
| smcginnis | fernnest: Hopefully you can get it well below 1000 lines each. | 16:56 |
| smcginnis | erlon: I'm not sure why that would be. | 16:56 |
| fernnest | erlon,smcginnis: the fewer lines the more reviews required. | 16:57 |
| ildikov | scottda_vacation: hey, sorry for the delay, I sent out the mail now, hopefully the time slot will work and then I will announce it on the ML as well | 16:57 |
| *** sgotliv has quit IRC | 16:57 | |
| openstackgerrit | Merged openstack/cinder: Fix compatibility mode of backup jobs scheduling https://review.openstack.org/293435 | 16:58 |
| smcginnis | fernnest: But the easier the reviews. | 16:58 |
| erlon | smcginnis: the problem is that we already have a driver put in place, with al the changes needed. And it is very different from the one is upstream right now, so, to have a patch that get the actual, in smallers steps (without breaking the CI) shown to be a nightmare | 16:58 |
| DuncanT | erlon: The review work of going through and comparing two drivers with massive changes is shown to be a nightmare, repeatedly | 16:59 |
| *** vivekd2 has joined #openstack-cinder | 16:59 | |
| *** vivekdhayaal has quit IRC | 16:59 | |
| fernnest | erlon, we have the same problem, we were thinking about using new file names and renaming at the end. | 16:59 |
| DuncanT | erlon: If it is hard for us to review, then it's equally hard for you to review, so the chances of bugs is high | 16:59 |
| *** raildo is now known as raildo-afk | 17:00 | |
| DuncanT | erlon: The coverage of cinder upstream testing is very, very low, not nearly enough to have an confidence | 17:00 |
| *** IlyaG has quit IRC | 17:00 | |
| *** e0ne has joined #openstack-cinder | 17:00 | |
| erlon | DuncanT: so, I think that the fernnest approach will be the best in our case as well, a new driver, with different name | 17:01 |
| *** haomaiwang has quit IRC | 17:01 | |
| erlon | DuncanT: and then rename it | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| DuncanT | erlon: It might be 'best' for you, but it sucks for the cinder community | 17:01 |
| fernnest | erlon,smcginnis:would it help if i wrote the proposal up. | 17:01 |
| *** IlyaG has joined #openstack-cinder | 17:01 | |
| *** vincent_hou has quit IRC | 17:02 | |
| smcginnis | fernnest: I think a spec would be overkill. | 17:02 |
| *** zhanghao1 has quit IRC | 17:02 | |
| smcginnis | fernnest, erlon: Still not seeing why changing the existing - without breaking CI - is a problem. | 17:02 |
| *** zhanghao has joined #openstack-cinder | 17:02 | |
| erlon | DuncanT: at least you wouldn't have to compare with the old one, right? as you said? | 17:03 |
| *** cbinx has joined #openstack-cinder | 17:03 | |
| DuncanT | erlon: Yes, we would, since we need to know all of the gotchas during an upgrade | 17:03 |
| erlon | DuncanT: hmm | 17:03 |
| DuncanT | erlon: People come and blame 'cinder' for vendor driver issues, whether that is correct or not | 17:03 |
| smcginnis | DuncanT: Good point, that makes it riskier than a new driver. | 17:03 |
| fernnest | smcginnis: absent doing one big patchset, the changes will have dependendcies in code that isn't in the patchset | 17:04 |
| smcginnis | fernnest: Why? | 17:04 |
| *** kprabhu has joined #openstack-cinder | 17:04 | |
| smcginnis | fernnest: That doesn't make sense to me. | 17:04 |
| *** merooney has quit IRC | 17:05 | |
| fernnest | smcginnis: e.g. if I change api in file1.py and the calls to in file2.py, file3.py, file4.py, I will need to submit all files. | 17:05 |
| DuncanT | smcginnis: They're moving a bunch of the code into a pip library, same as e.g. HP do | 17:05 |
| DuncanT | smcginnis: At least according to the meeting notes | 17:05 |
| smcginnis | fernnest: You don't have to do one file at a time. Do one concept at a time across multiple files. | 17:06 |
| smcginnis | It's not just a matter of taking one big patch and chopping it up. | 17:06 |
| DuncanT | fernnest: The point isn't to change one file per patch, it's to change one thing at a time | 17:06 |
| smcginnis | Evolve the driver to what you need. | 17:06 |
| smcginnis | That's what refactoring is. | 17:06 |
| erlon | smcginnis: we have, code A, and the refactored code B. To get from A to B, we would need the series of patches, of smaller patches, that could need to break the structure of the code so, A -> p1 -> p2 ... pn -> B, so make all paches to work would require lot of effort and in my POV it would take the same effort from community to review | 17:06 |
| *** lpetrut has quit IRC | 17:07 | |
| DuncanT | erlon: From a review PoV, what you're doing is saying "Have a massive amount of work, or else just merge a bunch of new and different bugs" | 17:11 |
| DuncanT | erlon: It is definitely not a healthy way of contributing to an opensource project, and many opensource projects are pushing back against it for good reasons - the kernel being an example of such a project that just syas a flat no to such things | 17:12 |
| *** jistr has quit IRC | 17:12 | |
| DuncanT | *say a flat no | 17:12 |
| erlon | DuncanT: wich if, exist, would still exist in the splitted paches :) | 17:13 |
| smcginnis | erlon: I think you're looking at this the wrong way. | 17:13 |
| *** merooney has joined #openstack-cinder | 17:13 | |
| DuncanT | erlon: Yes, but they are much, much easier to spot in the smaller patches | 17:13 |
| DuncanT | erlon: Having to only reason about one change at a time makes thinking about the ramifications much easier | 17:14 |
| erlon | DuncanT: mhm, I see | 17:14 |
| DuncanT | erlon: To turn it around another way, by not breaking it up, you pretty much make it impossible for reviewers to see deep, none-cosmetic issues, so you're throwing away a chance to have your design looked at properly by a team of very competent engineers | 17:16 |
| *** vincent_hou has joined #openstack-cinder | 17:21 | |
| *** merooney has quit IRC | 17:23 | |
| smcginnis | DuncanT: Look good to you? https://review.openstack.org/#/c/282230/ | 17:26 |
| smcginnis | tbarron: You're pretty familiar here too: https://review.openstack.org/#/c/282230/ | 17:26 |
| tbarron | smcginnis: looking | 17:26 |
| smcginnis | tbarron: Thank you! | 17:26 |
| DuncanT | smcginnis: Oh, yeah, I starting looking at that and then got distracted | 17:27 |
| smcginnis | DuncanT: I know the feeling. | 17:27 |
| *** sage has quit IRC | 17:27 | |
| DuncanT | smcginnis: The name was my biggest issue, it should be update_backup_host, not update_host, IMO - we're likely to want something to update volume hosts at some point | 17:28 |
| smcginnis | DuncanT: Hmm, true. | 17:28 |
| DuncanT | (e.g. when going from single backend to multibacked, all your host names change) | 17:28 |
| *** lpetrut has joined #openstack-cinder | 17:28 | |
| tbarron | DuncanT: +1 | 17:28 |
| smcginnis | Think lisa would mind if we update that on her? | 17:28 |
| DuncanT | smcginnis: Not at all | 17:28 |
| smcginnis | Anyone want to grab that? Or I can. | 17:29 |
| *** sage has joined #openstack-cinder | 17:29 | |
| tbarron | smcginnis: she won't mind. | 17:29 |
| DuncanT | smcginnis: Go for it. I'll merge it afterwards | 17:29 |
| smcginnis | Or tbarron. Then DuncanT and I can review it. | 17:29 |
| DuncanT | (I'm cooking dinner now in the background, it's 7:30 pm here) | 17:29 |
| tbarron | smcginnis: ok, I'll do it. | 17:29 |
| smcginnis | tbarron: Thanks | 17:30 |
| openstackgerrit | Merged openstack/cinder: Cleaned duplicate dictionary values https://review.openstack.org/288296 | 17:30 |
| *** e0ne has quit IRC | 17:30 | |
| *** Suyash has joined #openstack-cinder | 17:30 | |
| *** IlyaG has quit IRC | 17:30 | |
| *** IlyaG has joined #openstack-cinder | 17:31 | |
| *** shyama has quit IRC | 17:33 | |
| *** shyama has joined #openstack-cinder | 17:34 | |
| *** IlyaG has quit IRC | 17:34 | |
| *** ndipanov has quit IRC | 17:36 | |
| *** johnthetubaguy_ has joined #openstack-cinder | 17:39 | |
| *** panda has quit IRC | 17:40 | |
| *** panda has joined #openstack-cinder | 17:41 | |
| *** mylu has joined #openstack-cinder | 17:41 | |
| *** johnthetubaguy has quit IRC | 17:41 | |
| *** johnthetubaguy_ is now known as johnthetubaguy | 17:41 | |
| *** lpetrut has quit IRC | 17:43 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Add update_host for backup in cinder-manager https://review.openstack.org/282230 | 17:43 |
| tbarron | smcginnis: DuncanT: ^^^ | 17:43 |
| hemna | localhost | 17:44 |
| hemna | err | 17:44 |
| DuncanT | tbarron: Looks good to me, thanks | 17:45 |
| *** bardia has quit IRC | 17:45 | |
| tbarron | DuncanT: enjoy dinner | 17:45 |
| DuncanT | :-) Thanks | 17:45 |
| *** sdague has quit IRC | 17:46 | |
| *** avishay has joined #openstack-cinder | 17:47 | |
| *** vivekd2 has quit IRC | 17:49 | |
| *** rlrossit has quit IRC | 17:52 | |
| *** vincent_hou has quit IRC | 17:52 | |
| *** sdague has joined #openstack-cinder | 17:52 | |
| *** rcernin has quit IRC | 17:53 | |
| *** salv-orl_ has joined #openstack-cinder | 17:54 | |
| *** salv-orlando has quit IRC | 17:56 | |
| openstackgerrit | Joel Coffman proposed openstack/cinder-specs: Add volume encryption metadata spec https://review.openstack.org/176050 | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| DuncanT | We are *not* calling something else metadata | 18:04 |
| *** lpetrut has joined #openstack-cinder | 18:05 | |
| *** vincent_hou has joined #openstack-cinder | 18:11 | |
| *** rlrossit has joined #openstack-cinder | 18:14 | |
| *** martyturner has quit IRC | 18:14 | |
| openstackgerrit | Merged openstack/python-cinderclient: Cleanup for Replication v2: remove 'replication-promote' https://review.openstack.org/292331 | 18:15 |
| *** vincent_hou_ has joined #openstack-cinder | 18:16 | |
| ildikov | hemna: hi | 18:16 |
| *** vincent_hou has quit IRC | 18:16 | |
| *** vincent_hou_ is now known as vincent_hou | 18:16 | |
| openstackgerrit | Merged openstack/python-cinderclient: Cleanup for Replication v2: remove 'replication-reenable'. https://review.openstack.org/292332 | 18:17 |
| patrickeast | huh... so we removed the replication v1 calls? ^^ | 18:18 |
| * patrickeast might be a tad late to the party | 18:18 | |
| *** tesseract- has quit IRC | 18:19 | |
| openstackgerrit | Merged openstack/cinder: Add devref on rolling upgrades https://review.openstack.org/279186 | 18:20 |
| DuncanT | patrickeast: No, we removed the V2 calls | 18:20 |
| patrickeast | DuncanT: yea, but those two that just merged were original replication api's | 18:22 |
| patrickeast | DuncanT: in the review they cite https://review.openstack.org/#/c/110115/ | 18:22 |
| smcginnis | patrickeast: Shoot, that's why they were still there. | 18:22 |
| patrickeast | ronen didn't do any v2 | 18:22 |
| patrickeast | its confusing since they are replication apis on the cinder v2 volume api... | 18:22 |
| smcginnis | Definitely confusing to me. Guess we need to revert that now. | 18:23 |
| patrickeast | i think we might have pulled them from the api in cinder anyway, so at least the client would be in sync | 18:23 |
| patrickeast | fwiw | 18:23 |
| DuncanT | smcginnis: I thought we were keeping the V1 stuff in the client for now for people using a new client on an old system? | 18:23 |
| smcginnis | DuncanT: Yes, we're supposed to. | 18:23 |
| smcginnis | DuncanT: Which is why those two calls were still there. | 18:23 |
| *** dustins has joined #openstack-cinder | 18:23 | |
| smcginnis | I totally missed that they were v1 APIs. | 18:23 |
| smcginnis | So the one repl v2 cleanup patch was complete. | 18:24 |
| smcginnis | These two were left on purpose to support repl v1 clients. | 18:24 |
| DuncanT | smcginnis: When you revert, might be worth adding a patch saying what they are and why they're there.... hopefully nothing collides at the REST level | 18:24 |
| smcginnis | DuncanT: I think REST API we are safe. This is just confusing having two replication sets of calls. | 18:25 |
| *** ishant has quit IRC | 18:25 | |
| *** akshai has quit IRC | 18:25 | |
| DuncanT | smcginnis: That's not so bad then | 18:25 |
| DuncanT | smcginnis: I should probably have thought ot check that before now I guess... oh well | 18:26 |
| smcginnis | DuncanT: I missed it too. | 18:27 |
| smcginnis | On the plus side, I've finally caught up on all my email from yesterday. | 18:27 |
| smcginnis | Hopefully that means I'll be more focused now. | 18:27 |
| smcginnis | After this meeting... | 18:27 |
| *** akshai has joined #openstack-cinder | 18:33 | |
| *** mriedem has quit IRC | 18:35 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR fix create_cloned_volume for larger size https://review.openstack.org/291850 | 18:37 |
| *** mriedem has joined #openstack-cinder | 18:38 | |
| *** bardia has joined #openstack-cinder | 18:39 | |
| *** jordanP has quit IRC | 18:40 | |
| *** laughterwym has joined #openstack-cinder | 18:41 | |
| *** mylu has quit IRC | 18:41 | |
| *** laughterwym has quit IRC | 18:46 | |
| *** rlrossit has quit IRC | 18:46 | |
| avishay | jgriffith: hey man, when you get a chance, would appreciate your feedback - https://review.openstack.org/#/c/285296/ | 18:46 |
| avishay | jgriffith: maybe better to reply on the spec - https://review.openstack.org/#/c/282829/ | 18:46 |
| *** mylu has joined #openstack-cinder | 18:53 | |
| *** ebalduf has quit IRC | 18:53 | |
| *** amit2131 has quit IRC | 18:55 | |
| *** amit213 has joined #openstack-cinder | 18:55 | |
| *** pece has quit IRC | 18:56 | |
| *** rlrossit_ has joined #openstack-cinder | 18:57 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:01 | |
| *** harlowja has joined #openstack-cinder | 19:04 | |
| *** mkrcmari__ has quit IRC | 19:05 | |
| *** wanghao has quit IRC | 19:05 | |
| vincent_hou | smcginnis: Hi Sean. There is still one follow-up fix for storwize replication v2.1 from me: https://review.openstack.org/#/c/292570/ | 19:05 |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: Revert "Cleanup for Replication v2: remove 'replication-reenable'." https://review.openstack.org/293658 | 19:05 |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: Revert "Cleanup for Replication v2: remove 'replication-promote'" https://review.openstack.org/293659 | 19:06 |
| smcginnis | vincent_hou: Thanks, I'll take a look now. | 19:06 |
| *** wanghao has joined #openstack-cinder | 19:06 | |
| *** salv-orl_ has quit IRC | 19:15 | |
| *** ircuser-1 has quit IRC | 19:19 | |
| *** gouthamr has quit IRC | 19:23 | |
| *** merooney has joined #openstack-cinder | 19:24 | |
| *** pece has joined #openstack-cinder | 19:25 | |
| Swanson | is provider_id in snapshots something I set or is that the provider_id of the associated volume? | 19:26 |
| patrickeast | its something the driver can set | 19:27 |
| *** kro_sniffles is now known as kro_mnckn_doc | 19:27 | |
| Swanson | hmmm... I think it is going to be the associated volume's internal id. | 19:27 |
| *** avishay has quit IRC | 19:29 | |
| *** akshai has quit IRC | 19:30 | |
| Swanson | patrickeast, no way to go from a snapshot object in the driver to a volume object without directly calling the db, right? | 19:30 |
| patrickeast | Swanson: i don't think so (or at least that was the case last time I looked when we started implementing v2 replication months ago) | 19:31 |
| patrickeast | we had wanted to be able to look at the volumes provider_* fields in the snapshot methods and couldn't | 19:32 |
| patrickeast | well | 19:32 |
| Swanson | patrickeast, ah | 19:32 |
| patrickeast | couldn't without db calls | 19:32 |
| Swanson | patrickeast, I'll just use the provider_id to hold the id for the volume and do a search for the snapshot. This works for me. Takes care of some uncertainty, actually. | 19:34 |
| *** akerr has quit IRC | 19:34 | |
| *** ociuhandu has quit IRC | 19:35 | |
| *** ChubYann has joined #openstack-cinder | 19:36 | |
| *** mylu has quit IRC | 19:36 | |
| *** akshai has joined #openstack-cinder | 19:39 | |
| *** laughterwym has joined #openstack-cinder | 19:40 | |
| *** gouthamr has joined #openstack-cinder | 19:40 | |
| *** mylu has joined #openstack-cinder | 19:41 | |
| *** laughterwym has quit IRC | 19:44 | |
| *** gouthamr_ has joined #openstack-cinder | 19:44 | |
| *** e0ne has joined #openstack-cinder | 19:46 | |
| *** harlowja has quit IRC | 19:46 | |
| *** gouthamr has quit IRC | 19:48 | |
| *** ig0r_ has quit IRC | 19:55 | |
| *** ociuhandu has joined #openstack-cinder | 19:56 | |
| *** ericksonsantos has joined #openstack-cinder | 19:58 | |
| *** merooney has quit IRC | 20:00 | |
| *** haomaiwa_ has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-cinder | 20:01 | |
| *** salv-orlando has joined #openstack-cinder | 20:01 | |
| *** timcl has quit IRC | 20:02 | |
| *** vincent_hou has quit IRC | 20:04 | |
| *** vincent_hou has joined #openstack-cinder | 20:06 | |
| *** geguileo has quit IRC | 20:08 | |
| *** sgotliv has joined #openstack-cinder | 20:10 | |
| *** IlyaG has joined #openstack-cinder | 20:12 | |
| smcginnis | If any cores have a moment: https://review.openstack.org/#/c/293658/ | 20:13 |
| *** martyturner has joined #openstack-cinder | 20:15 | |
| *** IlyaG has quit IRC | 20:16 | |
| *** IlyaG has joined #openstack-cinder | 20:16 | |
| smcginnis | patrickeast, xyang1: Thanks! | 20:16 |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Small fix for NexentaEdge jsonrpc https://review.openstack.org/291966 | 20:18 |
| xyang1 | smcginnis: welcome | 20:18 |
| *** mylu has quit IRC | 20:19 | |
| *** mylu has joined #openstack-cinder | 20:20 | |
| *** geguileo has joined #openstack-cinder | 20:20 | |
| *** IlyaG has quit IRC | 20:21 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Reserve 5 migrations for Mitaka backports https://review.openstack.org/293699 | 20:25 |
| *** vincent_hou_ has joined #openstack-cinder | 20:26 | |
| *** vincent_hou has quit IRC | 20:26 | |
| *** sbezverk has quit IRC | 20:26 | |
| *** vincent_hou_ is now known as vincent_hou | 20:26 | |
| *** dencaval has quit IRC | 20:27 | |
| *** IlyaG has joined #openstack-cinder | 20:27 | |
| *** IlyaG has quit IRC | 20:32 | |
| *** diogogmt has joined #openstack-cinder | 20:33 | |
| *** ircuser-1 has joined #openstack-cinder | 20:33 | |
| *** vincent_hou_ has joined #openstack-cinder | 20:39 | |
| *** vincent_hou has quit IRC | 20:40 | |
| openstackgerrit | Merged openstack/cinder: Add update_host for backup in cinder-manager https://review.openstack.org/282230 | 20:40 |
| *** vincent_hou_ is now known as vincent_hou | 20:40 | |
| *** andymaier has quit IRC | 20:41 | |
| *** Lee1092 has quit IRC | 20:42 | |
| *** IlyaG has joined #openstack-cinder | 20:43 | |
| *** lpetrut has quit IRC | 20:44 | |
| *** sgotliv has quit IRC | 20:44 | |
| *** cknight has quit IRC | 20:48 | |
| *** abhi has quit IRC | 20:48 | |
| openstackgerrit | Merged openstack/cinder: Add scheduler RPC API v2.0 https://review.openstack.org/281307 | 20:49 |
| *** IlyaG has quit IRC | 20:51 | |
| *** mylu has quit IRC | 20:53 | |
| *** vincent_hou has quit IRC | 20:58 | |
| *** e0ne has quit IRC | 20:58 | |
| *** yangyapeng has quit IRC | 20:58 | |
| *** yangyapeng has joined #openstack-cinder | 20:59 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 21:01 | |
| *** angela-s has joined #openstack-cinder | 21:02 | |
| *** dustins has quit IRC | 21:04 | |
| *** yangyapeng has quit IRC | 21:04 | |
| *** yangyapeng has joined #openstack-cinder | 21:06 | |
| *** DericHorn-HP has joined #openstack-cinder | 21:12 | |
| angela-s | hi doc team. I'm having an issue running autohelp-wrapper update where i'm getting an error 'cannot locate liberasurecode.so.1' | 21:12 |
| *** DericHorn-HP has quit IRC | 21:12 | |
| angela-s | oops, wrong channel. sorry. | 21:12 |
| *** rlrossit_ has quit IRC | 21:17 | |
| *** kro_mnckn_doc has quit IRC | 21:19 | |
| *** anteaya has quit IRC | 21:19 | |
| patrickeast | xyang1: hey, am i remembering correctly that you created some consistency group tempest tests at one point in time? | 21:20 |
| *** jungleboyj has quit IRC | 21:20 | |
| *** diablo_rojo has quit IRC | 21:22 | |
| smcginnis | patrickeast: https://review.openstack.org/#/c/252213/ | 21:24 |
| patrickeast | ah ha | 21:24 |
| patrickeast | smcginnis: perfect, thanks! | 21:24 |
| patrickeast | i'm going to steal those and try to put together a tempest plugin for any 3rd party ci's that want to test cgs | 21:25 |
| smcginnis | patrickeast: This might help: https://review.openstack.org/#/c/274471/ | 21:25 |
| smcginnis | patrickeast: I haven't had a chance to figure out why, but they "plugin" tests work for me locally. | 21:25 |
| smcginnis | patrickeast: But none of the CIs seem to pick it up. | 21:26 |
| openstackgerrit | Merged openstack/cinder: Add backup RPC API v2.0 https://review.openstack.org/283072 | 21:26 |
| patrickeast | smcginnis: hmm that is odd | 21:26 |
| openstackgerrit | Merged openstack/cinder: Add volume RPC API v2.0 https://review.openstack.org/281317 | 21:26 |
| patrickeast | smcginnis: i haven't looked into how it works with cinder, but for the one i have been playing with its a separate pip install that exposes the entry points... in theory thats all you need for tempest to find them | 21:27 |
| smcginnis | patrickeast: Ah, I see. So not an in-tree additional test, but something separate to add on if desired? | 21:27 |
| smcginnis | patrickeast: Not a bad idea too. | 21:27 |
| patrickeast | smcginnis: yea for things like cg's where not everyone supports it | 21:27 |
| patrickeast | since there was a lot of complaining about adding tempest tests for those | 21:28 |
| smcginnis | patrickeast: Yeah | 21:28 |
| smcginnis | hemna: What's the recheck command for HP again? | 21:28 |
| smcginnis | hemna: https://review.openstack.org/#/c/291850/ | 21:28 |
| patrickeast | run-HPE Storage CI | 21:29 |
| *** porrua has quit IRC | 21:29 | |
| smcginnis | patrickeast: Thanks! | 21:29 |
| *** eharney has quit IRC | 21:30 | |
| *** abhi has joined #openstack-cinder | 21:30 | |
| kmartin | smcginnis, yeah what patrickeast said ^^ thanks guys | 21:31 |
| openstackgerrit | Merged openstack/cinder: EMC VMAX - SSl connection is not picking up values https://review.openstack.org/280933 | 21:31 |
| *** abhi has quit IRC | 21:31 | |
| kmartin | smcginnis, we'll have results in 1 hr 34 min :) | 21:32 |
| *** abhi has joined #openstack-cinder | 21:32 | |
| *** abhi has quit IRC | 21:32 | |
| *** baumann has left #openstack-cinder | 21:32 | |
| *** knikolla has quit IRC | 21:34 | |
| *** e0ne has joined #openstack-cinder | 21:37 | |
| smcginnis | kmartin: Wind the crank faster! | 21:40 |
| *** e0ne has quit IRC | 21:40 | |
| kmartin | no kidding | 21:40 |
| Swanson | rejoin_stack.sh disappear? | 21:40 |
| hemna | smcginnis, run-HPE Storage CI | 21:40 |
| *** panda has quit IRC | 21:40 | |
| *** laughterwym has joined #openstack-cinder | 21:41 | |
| *** mdenny has quit IRC | 21:41 | |
| *** panda has joined #openstack-cinder | 21:41 | |
| smcginnis | Swanson: Weird! | 21:41 |
| hemna | wth | 21:42 |
| smcginnis | Removed 2 days ago. | 21:42 |
| smcginnis | "This command has never completely worked to restart DevStack." | 21:43 |
| smcginnis | Hmm, I never had issues. | 21:43 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: add support for discard https://review.openstack.org/293726 | 21:43 |
| openstackgerrit | Merged openstack/cinder: Fix formatting in vol/snap delete API messages https://review.openstack.org/293492 | 21:44 |
| openstackgerrit | Merged openstack/python-cinderclient: Revert "Cleanup for Replication v2: remove 'replication-reenable'." https://review.openstack.org/293658 | 21:44 |
| Swanson | smcginnis, yup. | 21:45 |
| *** laughterwym has quit IRC | 21:46 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-cinderclient: Revert "Cleanup for Replication v2: remove 'replication-promote'" https://review.openstack.org/293659 | 21:48 |
| *** permalac has quit IRC | 21:50 | |
| openstackgerrit | Merged openstack/cinder: Emit notifications for volume retype https://review.openstack.org/291977 | 21:50 |
| patrickeast | jbernard_: hey, quick question about https://review.openstack.org/293726 | 21:52 |
| patrickeast | jbernard_: any reason you can't use the shared one? https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L247-L252 | 21:53 |
| *** kfarr has quit IRC | 21:53 | |
| *** mriedem has quit IRC | 21:57 | |
| *** jgregor has quit IRC | 21:57 | |
| smcginnis | patrickeast: Mind taking a look at another one? https://review.openstack.org/#/c/293699/ | 21:58 |
| patrickeast | smcginnis: done | 21:59 |
| smcginnis | patrickeast: Thank you! | 21:59 |
| patrickeast | was waiting for jenkins | 21:59 |
| patrickeast | i need a like paging service for reviews i've looked at, but was waiting for some ci to post on them before i post | 22:00 |
| *** haomaiwa_ has quit IRC | 22:01 | |
| *** zhanghao1 has joined #openstack-cinder | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| smcginnis | patrickeast: Hah, no kidding. | 22:02 |
| smcginnis | patrickeast: You'd probably end up throwing it out the window after a couple days though. | 22:02 |
| patrickeast | haha | 22:02 |
| smcginnis | patrickeast: That could get really annoying. | 22:02 |
| patrickeast | smcginnis: very true | 22:02 |
| *** zhanghao has quit IRC | 22:04 | |
| hemna | hrmm | 22:09 |
| hemna | smcginnis, our CI in general seems borked now | 22:09 |
| hemna | perfect timing | 22:09 |
| smcginnis | hemna: Yeah, perfect. | 22:09 |
| smcginnis | hemna: Any idea what's up? | 22:09 |
| hemna | I'm trying to see what's up | 22:09 |
| hemna | our local zuul says we have 3 patches running and they are all still ok | 22:09 |
| smcginnis | hemna: Bandersnatch died? :) | 22:09 |
| *** arch-nemesis has quit IRC | 22:12 | |
| hemna | smcginnis, our log server filled up | 22:13 |
| hemna | 100% used | 22:14 |
| *** mattymo_ has joined #openstack-cinder | 22:15 | |
| *** yinweiishere has quit IRC | 22:15 | |
| *** SergeyLukjanov2 has joined #openstack-cinder | 22:15 | |
| *** yinweiishere has joined #openstack-cinder | 22:15 | |
| *** SergeyLukjanov has quit IRC | 22:16 | |
| *** mattymo has quit IRC | 22:16 | |
| *** SergeyLukjanov2 is now known as SergeyLukjanov | 22:16 | |
| *** dims_ has joined #openstack-cinder | 22:16 | |
| *** tpsilva has quit IRC | 22:17 | |
| *** erlon has quit IRC | 22:17 | |
| smcginnis | hemna: Doh! | 22:17 |
| smcginnis | hemna: Are you keeping more than 30 days? | 22:17 |
| *** dims has quit IRC | 22:18 | |
| hemna | I don't think so | 22:20 |
| hemna | 38G | 22:20 |
| hemna | used | 22:20 |
| smcginnis | In logs? Dang! | 22:20 |
| hemna | yup | 22:20 |
| hemna | I'm going to just have to nuke some | 22:20 |
| smcginnis | gzip'd | 22:20 |
| hemna | all text files, gziped! | 22:20 |
| smcginnis | Holy crap | 22:20 |
| smcginnis | hemna: Well, gotta run, but I'll be back online later. | 22:21 |
| hemna | I nuked 1 dir | 22:21 |
| smcginnis | hemna: If you're able to get a successful pass on that patch I can send it through. | 22:21 |
| hemna | 1G | 22:21 |
| hemna | hopefully that's enough | 22:21 |
| smcginnis | 1 dir == /etc ? | 22:21 |
| smcginnis | :P | 22:21 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: update the do_setup and helpers for the failover https://review.openstack.org/292570 | 22:24 |
| *** daneyon has quit IRC | 22:25 | |
| *** martyturner has quit IRC | 22:26 | |
| *** Thelo has quit IRC | 22:30 | |
| *** chlong has joined #openstack-cinder | 22:32 | |
| *** dalgaaf has quit IRC | 22:38 | |
| *** JoseMello has quit IRC | 22:39 | |
| *** geguileo has quit IRC | 22:39 | |
| *** laughterwym has joined #openstack-cinder | 22:41 | |
| *** edmondsw has quit IRC | 22:43 | |
| *** laughterwym has quit IRC | 22:45 | |
| *** IlyaG has joined #openstack-cinder | 22:48 | |
| *** xyang1 has quit IRC | 22:52 | |
| *** cknight has joined #openstack-cinder | 22:54 | |
| *** geguileo has joined #openstack-cinder | 22:55 | |
| openstackgerrit | Merged openstack/cinder: Reserve 5 migrations for Mitaka backports https://review.openstack.org/293699 | 22:57 |
| *** mriedem has joined #openstack-cinder | 22:57 | |
| *** jwcroppe has quit IRC | 22:59 | |
| *** jwcroppe has joined #openstack-cinder | 22:59 | |
| *** merooney has joined #openstack-cinder | 23:00 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 23:01 | |
| *** kprabhu has quit IRC | 23:03 | |
| *** jwcroppe has quit IRC | 23:04 | |
| hemna | smcginnis, reporting success now | 23:05 |
| *** krotscheck has joined #openstack-cinder | 23:11 | |
| *** sdague has quit IRC | 23:14 | |
| *** laughterwym has joined #openstack-cinder | 23:20 | |
| *** anteaya has joined #openstack-cinder | 23:21 | |
| *** akshai has quit IRC | 23:24 | |
| *** mriedem1 has joined #openstack-cinder | 23:35 | |
| *** mriedem has quit IRC | 23:37 | |
| *** Suyash has quit IRC | 23:37 | |
| *** diogogmt has quit IRC | 23:38 | |
| *** zhanghao1 has quit IRC | 23:41 | |
| *** chlong has quit IRC | 23:41 | |
| *** laughterwym has quit IRC | 23:43 | |
| *** pece has quit IRC | 23:44 | |
| *** jwcroppe has joined #openstack-cinder | 23:44 | |
| *** mylu has joined #openstack-cinder | 23:44 | |
| *** yhayashi has joined #openstack-cinder | 23:49 | |
| *** mylu has quit IRC | 23:53 | |
| *** salv-orl_ has joined #openstack-cinder | 23:54 | |
| *** cknight has quit IRC | 23:55 | |
| *** salv-orlando has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!