| *** smoriya_ has joined #openstack-cinder | 00:00 | |
| *** jamielennox|away is now known as jamielennox | 00:02 | |
| *** angela-s has quit IRC | 00:02 | |
| *** diablo_rojo has quit IRC | 00:03 | |
| *** Guest80711 is now known as med_ | 00:15 | |
| *** med_ has quit IRC | 00:15 | |
| *** med_ has joined #openstack-cinder | 00:15 | |
| *** edtubill has joined #openstack-cinder | 00:21 | |
| *** alonma has joined #openstack-cinder | 00:22 | |
| *** alonma has quit IRC | 00:26 | |
| *** lprice has joined #openstack-cinder | 00:27 | |
| *** alonma has joined #openstack-cinder | 00:29 | |
| *** lprice1 has joined #openstack-cinder | 00:30 | |
| *** lprice has quit IRC | 00:32 | |
| *** alonma has quit IRC | 00:33 | |
| *** IlyaG has joined #openstack-cinder | 00:35 | |
| *** IlyaG has quit IRC | 00:37 | |
| *** jamielennox is now known as jamielennox|away | 00:38 | |
| *** jamielennox|away is now known as jamielennox | 00:39 | |
| *** mc_nair has quit IRC | 00:40 | |
| *** thurloat_isgone is now known as thurloat | 00:42 | |
| *** zhangjn has joined #openstack-cinder | 00:42 | |
| *** alonma has joined #openstack-cinder | 00:42 | |
| *** laughterwym has joined #openstack-cinder | 00:45 | |
| *** laughterwym has quit IRC | 00:46 | |
| *** alonma has quit IRC | 00:47 | |
| *** laughterwym has joined #openstack-cinder | 00:47 | |
| *** alonma has joined #openstack-cinder | 00:48 | |
| *** EinstCrazy has joined #openstack-cinder | 00:50 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 00:52 |
|---|---|---|
| *** alonma has quit IRC | 00:53 | |
| *** edtubill has quit IRC | 00:54 | |
| *** qeelee has joined #openstack-cinder | 00:55 | |
| *** EinstCrazy has quit IRC | 00:55 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR fix driver to work with image cache https://review.openstack.org/236143 | 00:55 |
| *** cheneydc has joined #openstack-cinder | 00:57 | |
| *** mriedem_machell is now known as mriedem_afk | 00:58 | |
| *** JoseMello has quit IRC | 00:58 | |
| *** martyturner has quit IRC | 00:59 | |
| *** arch-nemesis has quit IRC | 01:02 | |
| *** rajinir has quit IRC | 01:05 | |
| *** garthb__ has quit IRC | 01:06 | |
| *** haomaiwang has joined #openstack-cinder | 01:12 | |
| *** IlyaG has joined #openstack-cinder | 01:18 | |
| *** gouthamr has joined #openstack-cinder | 01:21 | |
| *** gouthamr_ has joined #openstack-cinder | 01:23 | |
| *** davechen has joined #openstack-cinder | 01:24 | |
| *** gouthamr has quit IRC | 01:26 | |
| *** haomaiwang has quit IRC | 01:27 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:30 | |
| *** jamielennox is now known as jamielennox|away | 01:30 | |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** cknight has joined #openstack-cinder | 01:40 | |
| *** mtanino has quit IRC | 01:45 | |
| *** leeantho has quit IRC | 01:47 | |
| *** alonma has joined #openstack-cinder | 01:48 | |
| *** alonma has quit IRC | 01:53 | |
| *** alonma has joined #openstack-cinder | 01:55 | |
| *** alonma has quit IRC | 01:59 | |
| *** alonma has joined #openstack-cinder | 02:01 | |
| *** yhayashi has joined #openstack-cinder | 02:03 | |
| *** alonma has quit IRC | 02:05 | |
| *** alonma has joined #openstack-cinder | 02:07 | |
| *** salv-orl_ has joined #openstack-cinder | 02:09 | |
| *** salv-orlando has quit IRC | 02:12 | |
| *** alonma has quit IRC | 02:12 | |
| *** alonma has joined #openstack-cinder | 02:16 | |
| *** EinstCrazy has joined #openstack-cinder | 02:17 | |
| lixiaoy1 | wanghao: | 02:18 |
| lixiaoy1 | wanghao: ping | 02:18 |
| *** alonma has quit IRC | 02:21 | |
| *** alonma has joined #openstack-cinder | 02:22 | |
| *** haomaiwang has joined #openstack-cinder | 02:23 | |
| *** alonma has quit IRC | 02:27 | |
| *** jamielennox|away is now known as jamielennox | 02:31 | |
| *** xinyong has joined #openstack-cinder | 02:38 | |
| *** Thelo has quit IRC | 02:39 | |
| *** dims_ has quit IRC | 02:40 | |
| *** Apoorva has quit IRC | 02:41 | |
| openstackgerrit | yangweiwei proposed openstack/cinder: Update the method of backup-restore https://review.openstack.org/275478 | 02:43 |
| *** daneyon_ has joined #openstack-cinder | 02:51 | |
| *** mc_nair has joined #openstack-cinder | 02:53 | |
| *** alonma has joined #openstack-cinder | 02:53 | |
| *** wilson-liu has quit IRC | 02:54 | |
| *** daneyon has quit IRC | 02:54 | |
| *** bill_az has quit IRC | 02:55 | |
| *** alonma has quit IRC | 02:57 | |
| *** mc_nair has quit IRC | 02:57 | |
| *** alonma has joined #openstack-cinder | 02:59 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** alonma has quit IRC | 03:03 | |
| *** links has joined #openstack-cinder | 03:06 | |
| *** alonma has joined #openstack-cinder | 03:06 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Clean uploading volume when c-vol restarts https://review.openstack.org/275480 | 03:07 |
| *** alonma has quit IRC | 03:11 | |
| openstackgerrit | Merged openstack/cinder: Replication V2 for Pure Storage https://review.openstack.org/214825 | 03:32 |
| openstackgerrit | Merged openstack/cinder: Remove deprecated options from NFS driver https://review.openstack.org/269841 | 03:33 |
| *** alonma has joined #openstack-cinder | 03:34 | |
| *** alonma has quit IRC | 03:39 | |
| *** alonma has joined #openstack-cinder | 03:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/266163 | 03:43 |
| *** alonma has quit IRC | 03:45 | |
| *** esp_ has joined #openstack-cinder | 03:47 | |
| *** alonma has joined #openstack-cinder | 03:47 | |
| *** esp_ has quit IRC | 03:51 | |
| *** alonma has quit IRC | 03:52 | |
| *** alonma has joined #openstack-cinder | 03:53 | |
| *** qeelee has quit IRC | 03:53 | |
| *** gouthamr_ has quit IRC | 03:58 | |
| *** alonma has quit IRC | 03:58 | |
| *** alonma has joined #openstack-cinder | 04:00 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** sheel has joined #openstack-cinder | 04:02 | |
| *** laughter_ has joined #openstack-cinder | 04:04 | |
| *** alonma has quit IRC | 04:05 | |
| *** alonma has joined #openstack-cinder | 04:06 | |
| *** laughterwym has quit IRC | 04:08 | |
| *** jamielennox is now known as jamielennox|away | 04:09 | |
| *** alonma has quit IRC | 04:10 | |
| *** mc_nair has joined #openstack-cinder | 04:12 | |
| *** alonma has joined #openstack-cinder | 04:12 | |
| *** alonma has quit IRC | 04:16 | |
| *** sheel has quit IRC | 04:18 | |
| *** deepakcs has joined #openstack-cinder | 04:20 | |
| *** mriedem_afk has quit IRC | 04:22 | |
| *** shyama has joined #openstack-cinder | 04:30 | |
| *** pradipm has joined #openstack-cinder | 04:33 | |
| *** sheel has joined #openstack-cinder | 04:33 | |
| *** qeelee has joined #openstack-cinder | 04:33 | |
| *** pradipm has quit IRC | 04:33 | |
| *** mc_nair has quit IRC | 04:41 | |
| *** qeelee has quit IRC | 04:50 | |
| *** EinstCrazy has quit IRC | 04:53 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** esker has joined #openstack-cinder | 05:06 | |
| *** wanghao has quit IRC | 05:07 | |
| *** wanghao has joined #openstack-cinder | 05:07 | |
| *** bardia has quit IRC | 05:08 | |
| *** esker has quit IRC | 05:11 | |
| *** cknight has quit IRC | 05:12 | |
| openstackgerrit | Ankit Agrawal proposed openstack/cinder: NFS snapshots https://review.openstack.org/147186 | 05:13 |
| *** cknight has joined #openstack-cinder | 05:13 | |
| *** cknight has quit IRC | 05:13 | |
| *** laughter_ has quit IRC | 05:21 | |
| *** laughterwym has joined #openstack-cinder | 05:21 | |
| *** mc_nair has joined #openstack-cinder | 05:28 | |
| *** mc_nair has quit IRC | 05:32 | |
| *** sgotliv has joined #openstack-cinder | 05:43 | |
| *** vgridnev has joined #openstack-cinder | 05:43 | |
| *** qeelee has joined #openstack-cinder | 05:49 | |
| *** nkrinner has joined #openstack-cinder | 05:50 | |
| openstackgerrit | yangweiwei proposed openstack/cinder: Update the method of backup-restore https://review.openstack.org/275478 | 05:52 |
| *** mc_nair has joined #openstack-cinder | 05:56 | |
| *** mc_nair has quit IRC | 06:00 | |
| *** alonma has joined #openstack-cinder | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| openstackgerrit | Merged openstack/cinder: LeftHand: Implement v2 replication (unmanaged) https://review.openstack.org/255544 | 06:01 |
| *** qeelee has quit IRC | 06:04 | |
| *** alonma has quit IRC | 06:04 | |
| *** ChubYann has quit IRC | 06:10 | |
| *** sheel has quit IRC | 06:10 | |
| *** cfriesen has quit IRC | 06:11 | |
| *** alonma has joined #openstack-cinder | 06:12 | |
| *** alonma has quit IRC | 06:16 | |
| openstackgerrit | venkatamahesh proposed openstack/cinder: Fix the replication spelling in message https://review.openstack.org/275509 | 06:17 |
| *** sgotliv has quit IRC | 06:19 | |
| *** alonma has joined #openstack-cinder | 06:20 | |
| *** alonma has quit IRC | 06:24 | |
| *** alonma has joined #openstack-cinder | 06:26 | |
| *** jaypipes has quit IRC | 06:26 | |
| *** mc_nair has joined #openstack-cinder | 06:29 | |
| *** alonma has quit IRC | 06:30 | |
| *** zul has joined #openstack-cinder | 06:33 | |
| *** qeelee has joined #openstack-cinder | 06:33 | |
| *** mc_nair has quit IRC | 06:36 | |
| *** alonma has joined #openstack-cinder | 06:38 | |
| *** anshul has joined #openstack-cinder | 06:40 | |
| *** alonma has quit IRC | 06:43 | |
| *** avishay has joined #openstack-cinder | 06:43 | |
| *** alonma has joined #openstack-cinder | 06:44 | |
| *** alonma has quit IRC | 06:49 | |
| *** zul has quit IRC | 06:51 | |
| *** chhavi has joined #openstack-cinder | 06:51 | |
| *** alonma has joined #openstack-cinder | 06:52 | |
| *** alonma has quit IRC | 06:56 | |
| *** alonma has joined #openstack-cinder | 06:58 | |
| *** abhishekk has joined #openstack-cinder | 06:58 | |
| *** haomaiwang has quit IRC | 07:01 | |
| abhishekk | Hi all, please review this cinderclient patch along with the dependent patches, https://review.openstack.org/#/c/257170/ | 07:01 |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| abhishekk | thank you | 07:01 |
| *** alonma has quit IRC | 07:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/275529 | 07:04 |
| *** alonma has joined #openstack-cinder | 07:04 | |
| *** alonma has quit IRC | 07:08 | |
| *** alonma has joined #openstack-cinder | 07:10 | |
| *** esker has joined #openstack-cinder | 07:11 | |
| *** alonma has quit IRC | 07:14 | |
| *** IlyaG has quit IRC | 07:15 | |
| *** esker has quit IRC | 07:16 | |
| *** lpetrut has joined #openstack-cinder | 07:16 | |
| *** alonma has joined #openstack-cinder | 07:17 | |
| *** alonma has quit IRC | 07:22 | |
| *** alonma has joined #openstack-cinder | 07:24 | |
| *** laughterwym has quit IRC | 07:25 | |
| *** sandhya has joined #openstack-cinder | 07:25 | |
| *** laughterwym has joined #openstack-cinder | 07:25 | |
| sandhya | Hi, I am trying to configure HP left hand in cinder. I find hplefthandclient for doing an install of the client. But I don't find hpelefthandclient. Any pointers on this? | 07:26 |
| *** alonma has quit IRC | 07:28 | |
| *** laughterwym has quit IRC | 07:29 | |
| *** alonma has joined #openstack-cinder | 07:30 | |
| *** esker has joined #openstack-cinder | 07:32 | |
| *** alonma has quit IRC | 07:34 | |
| *** alonma has joined #openstack-cinder | 07:36 | |
| *** esker has quit IRC | 07:37 | |
| openstackgerrit | yangweiwei proposed openstack/cinder: Update the method of backup-restore https://review.openstack.org/275542 | 07:38 |
| *** alonma has quit IRC | 07:40 | |
| *** alonma has joined #openstack-cinder | 07:42 | |
| *** alonma has quit IRC | 07:46 | |
| *** belmoreira has joined #openstack-cinder | 07:47 | |
| *** alonma has joined #openstack-cinder | 07:48 | |
| *** alonma has quit IRC | 07:53 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update quota when volume type renames. https://review.openstack.org/267999 | 07:53 |
| *** esker has joined #openstack-cinder | 07:53 | |
| *** alonma has joined #openstack-cinder | 07:54 | |
| *** jwcroppe has quit IRC | 07:55 | |
| *** mvk_ has quit IRC | 07:56 | |
| *** esker has quit IRC | 07:58 | |
| *** jwcroppe has joined #openstack-cinder | 07:58 | |
| *** alonma has quit IRC | 07:59 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Update quota when volume type renames. https://review.openstack.org/267999 | 07:59 |
| *** alonma has joined #openstack-cinder | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:01 | |
| *** jwcroppe has quit IRC | 08:02 | |
| *** jwcroppe has joined #openstack-cinder | 08:04 | |
| *** alonma has quit IRC | 08:04 | |
| *** liverpooler has joined #openstack-cinder | 08:06 | |
| *** alonma has joined #openstack-cinder | 08:07 | |
| *** geguileo has joined #openstack-cinder | 08:07 | |
| *** zhangjn has quit IRC | 08:07 | |
| *** zhangjn has joined #openstack-cinder | 08:09 | |
| *** zhangjn has quit IRC | 08:10 | |
| *** salv-orlando has joined #openstack-cinder | 08:10 | |
| *** zhangjn has joined #openstack-cinder | 08:10 | |
| *** zhangjn has quit IRC | 08:11 | |
| *** laughterwym has joined #openstack-cinder | 08:11 | |
| *** zhangjn has joined #openstack-cinder | 08:11 | |
| *** zhangjn has quit IRC | 08:11 | |
| *** alonma has quit IRC | 08:12 | |
| *** zhangjn has joined #openstack-cinder | 08:12 | |
| *** zhangjn has quit IRC | 08:12 | |
| *** salv-orl_ has quit IRC | 08:12 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver https://review.openstack.org/271763 | 08:13 |
| *** lpetrut has quit IRC | 08:13 | |
| *** jistr has joined #openstack-cinder | 08:14 | |
| *** salv-orlando has quit IRC | 08:14 | |
| *** qeelee has quit IRC | 08:14 | |
| *** esker has joined #openstack-cinder | 08:14 | |
| *** salv-orlando has joined #openstack-cinder | 08:14 | |
| *** zhangjn has joined #openstack-cinder | 08:15 | |
| *** laughterwym has quit IRC | 08:15 | |
| *** zhangjn has quit IRC | 08:16 | |
| *** zhangjn has joined #openstack-cinder | 08:17 | |
| *** boris-42 has joined #openstack-cinder | 08:18 | |
| *** alonma has joined #openstack-cinder | 08:18 | |
| *** esker has quit IRC | 08:19 | |
| *** sgotliv has joined #openstack-cinder | 08:21 | |
| *** markus_z has joined #openstack-cinder | 08:21 | |
| *** alonma has quit IRC | 08:22 | |
| *** jistr is now known as jistr|mtg | 08:22 | |
| *** alonma has joined #openstack-cinder | 08:24 | |
| *** arecknag has joined #openstack-cinder | 08:25 | |
| *** sinese has joined #openstack-cinder | 08:25 | |
| *** zhangjn has quit IRC | 08:25 | |
| *** davechen has left #openstack-cinder | 08:27 | |
| *** mvk_ has joined #openstack-cinder | 08:27 | |
| *** alonma has quit IRC | 08:28 | |
| *** pots2 has quit IRC | 08:29 | |
| *** mc_nair has joined #openstack-cinder | 08:29 | |
| *** alonma has joined #openstack-cinder | 08:30 | |
| *** vgridnev has quit IRC | 08:32 | |
| *** mc_nair has quit IRC | 08:34 | |
| *** alonma has quit IRC | 08:35 | |
| *** esker has joined #openstack-cinder | 08:35 | |
| *** alonma has joined #openstack-cinder | 08:36 | |
| *** alonma has quit IRC | 08:37 | |
| *** alonma has joined #openstack-cinder | 08:38 | |
| *** zul has joined #openstack-cinder | 08:40 | |
| *** esker has quit IRC | 08:41 | |
| *** lpetrut has joined #openstack-cinder | 08:42 | |
| *** salv-orlando has quit IRC | 08:52 | |
| *** esker has joined #openstack-cinder | 08:56 | |
| *** smoriya_ has quit IRC | 08:58 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** EinstCrazy has joined #openstack-cinder | 09:01 | |
| *** esker has quit IRC | 09:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 09:01 | |
| *** laurent\ has quit IRC | 09:09 | |
| *** chhavi has quit IRC | 09:16 | |
| *** esker has joined #openstack-cinder | 09:18 | |
| *** chlong has quit IRC | 09:20 | |
| *** ankit_ag has joined #openstack-cinder | 09:21 | |
| *** alonma has quit IRC | 09:22 | |
| *** markus_z has quit IRC | 09:22 | |
| *** zhangjn has joined #openstack-cinder | 09:22 | |
| *** alonma has joined #openstack-cinder | 09:22 | |
| *** esker has quit IRC | 09:23 | |
| *** zhangjn has quit IRC | 09:24 | |
| *** jordanP has joined #openstack-cinder | 09:24 | |
| *** yhayashi has quit IRC | 09:24 | |
| *** alonma has quit IRC | 09:26 | |
| *** EinstCrazy has quit IRC | 09:29 | |
| *** EinstCrazy has joined #openstack-cinder | 09:29 | |
| *** esp_ has joined #openstack-cinder | 09:32 | |
| *** alonma has joined #openstack-cinder | 09:34 | |
| *** alonma has quit IRC | 09:35 | |
| *** alonma has joined #openstack-cinder | 09:36 | |
| *** nkrinner has quit IRC | 09:38 | |
| *** esp_ has quit IRC | 09:39 | |
| *** itzdilip has joined #openstack-cinder | 09:42 | |
| *** e0ne has joined #openstack-cinder | 09:45 | |
| *** vgridnev has joined #openstack-cinder | 09:51 | |
| *** lpetrut has quit IRC | 09:54 | |
| *** tpsilva has joined #openstack-cinder | 09:55 | |
| *** zhangjn has joined #openstack-cinder | 09:56 | |
| *** zhangjn_ has joined #openstack-cinder | 09:57 | |
| *** zhangjn_ has quit IRC | 09:58 | |
| *** esker has joined #openstack-cinder | 09:59 | |
| *** zhangjn_ has joined #openstack-cinder | 10:00 | |
| *** zhangjn_ has quit IRC | 10:00 | |
| *** haomaiwa_ has quit IRC | 10:01 | |
| *** vivekd has joined #openstack-cinder | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** jordanP has quit IRC | 10:01 | |
| *** jordanP has joined #openstack-cinder | 10:01 | |
| *** zhangjn_ has joined #openstack-cinder | 10:02 | |
| *** zhangjn_ has quit IRC | 10:02 | |
| *** cheneydc has quit IRC | 10:02 | |
| *** zhangjn_ has joined #openstack-cinder | 10:02 | |
| *** zul has quit IRC | 10:03 | |
| *** sheel has joined #openstack-cinder | 10:04 | |
| *** esker has quit IRC | 10:04 | |
| *** e0ne has quit IRC | 10:05 | |
| *** nkrinner has joined #openstack-cinder | 10:05 | |
| *** zul has joined #openstack-cinder | 10:07 | |
| *** vgridnev_ has joined #openstack-cinder | 10:13 | |
| *** vgridnev has quit IRC | 10:14 | |
| *** zhangjn_ has quit IRC | 10:15 | |
| *** zhangjn_ has joined #openstack-cinder | 10:15 | |
| *** salv-orlando has joined #openstack-cinder | 10:16 | |
| *** laurent\ has joined #openstack-cinder | 10:17 | |
| *** ociuhandu has joined #openstack-cinder | 10:18 | |
| *** salv-orlando has quit IRC | 10:21 | |
| *** ildikov has quit IRC | 10:27 | |
| sheel | DuncanT : Hi there | 10:27 |
| *** alonma has quit IRC | 10:28 | |
| *** alonma has joined #openstack-cinder | 10:29 | |
| *** alonma has quit IRC | 10:29 | |
| *** alonma has joined #openstack-cinder | 10:30 | |
| *** lpetrut has joined #openstack-cinder | 10:32 | |
| *** avishay has quit IRC | 10:32 | |
| *** esker has joined #openstack-cinder | 10:41 | |
| *** esker has quit IRC | 10:45 | |
| *** e0ne has joined #openstack-cinder | 10:51 | |
| *** vgridnev_ has quit IRC | 10:52 | |
| *** vgridnev_ has joined #openstack-cinder | 10:54 | |
| *** deepakcs has quit IRC | 10:59 | |
| *** zul has quit IRC | 11:00 | |
| *** ndipanov has joined #openstack-cinder | 11:00 | |
| *** ndipanov has quit IRC | 11:00 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** ndipanov has joined #openstack-cinder | 11:01 | |
| *** zul has joined #openstack-cinder | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** esker has joined #openstack-cinder | 11:02 | |
| *** zhangjn has quit IRC | 11:02 | |
| *** zhangjn has joined #openstack-cinder | 11:03 | |
| *** zhangjn has quit IRC | 11:05 | |
| *** chhavi has joined #openstack-cinder | 11:07 | |
| *** esker has quit IRC | 11:07 | |
| *** salv-orlando has joined #openstack-cinder | 11:07 | |
| *** zhangjn_ has quit IRC | 11:13 | |
| *** deepakcs has joined #openstack-cinder | 11:13 | |
| *** zhangjn has joined #openstack-cinder | 11:13 | |
| *** esker has joined #openstack-cinder | 11:23 | |
| *** ndipanov has quit IRC | 11:27 | |
| *** jistr|mtg has quit IRC | 11:27 | |
| *** esker has quit IRC | 11:28 | |
| *** sandhya has quit IRC | 11:31 | |
| *** sheel has quit IRC | 11:38 | |
| *** ntt has joined #openstack-cinder | 11:39 | |
| *** deepakcs has quit IRC | 11:39 | |
| ntt | Hi, it possible to run openstack-cinder-volume on a node and use lvm as backend from another server? | 11:39 |
| *** jklare has joined #openstack-cinder | 11:42 | |
| jklare | hi, anybody around? | 11:42 |
| *** boris-42 has quit IRC | 11:43 | |
| *** baojg has joined #openstack-cinder | 11:44 | |
| *** vivekd_ has joined #openstack-cinder | 11:44 | |
| *** vivekd has quit IRC | 11:45 | |
| *** vivekd_ is now known as vivekd | 11:45 | |
| *** bkumar has joined #openstack-cinder | 11:45 | |
| *** sheel has joined #openstack-cinder | 11:46 | |
| *** merooney has joined #openstack-cinder | 11:52 | |
| e0ne | jklare: hi | 11:54 |
| jklare | e0ne hi :) | 11:54 |
| e0ne | ntt: no. you have to run cinder-volume on the same node where lvm is | 11:54 |
| *** deepakcs has joined #openstack-cinder | 11:55 | |
| *** deepakcs has quit IRC | 11:55 | |
| ntt | e0ne: I'm searching for a HA solution. I already have a SAN, so I can expose a lun to the node where openstack-cinder-volume runs and use the lun as a disk for PV... but in this way I need to ensure HA for openstack-cinder-volume and the disk used as a backend. Is this the right way? | 11:56 |
| jklare | e0ne i am the ptl of the openstack-chef project and currently refactoring our cookbooks. For some reason i can not get the image service to appear on the dashboard and regarding this path (https://review.openstack.org/#/c/239091) it seems, that we would have to deploy cinderv1 and cinderv2, but that seems quite strange to me. Any comments on that? Could not find much helpful information while googling so far... | 11:57 |
| ntt | furthermore, using this method, datas will be multiple "hop" .... because if I write on one of the LV used by cinder volume, datas first will go to cinder-volume-node and then arrive on the san.... | 11:57 |
| e0ne | ntt: we don't have HA for LVM now:( | 11:58 |
| ntt | so.... if openstack-cinder-volume goes down, I need to switch services on another node and manually restart | 11:58 |
| ntt | ? | 11:58 |
| *** vivekd_ has joined #openstack-cinder | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:01 | |
| e0ne | jklare: AFAIR, it should works w/o cinder v1 | 12:02 |
| e0ne | jklare: but you need to check for all services that they use cinderv2 endpoint | 12:02 |
| jklare | e0ne mmmh.. yeah i think so too, maybe i have to dive deeper into the logs | 12:02 |
| e0ne | jklare: yea, logs will be very helpful | 12:02 |
| *** julim_ has quit IRC | 12:03 | |
| jklare | e0ne which service do you mean? shouldnt the appearance of the volumes catergory in horizon be idependent of any other service config? | 12:03 |
| *** vivekd has quit IRC | 12:04 | |
| *** vivekd_ is now known as vivekd | 12:04 | |
| jklare | e0ne the local_settings.py for horizon states ' "volume": 2 ' | 12:04 |
| *** esker has joined #openstack-cinder | 12:04 | |
| e0ne | jklare: I didn't try toget horizon working without api v1 | 12:05 |
| e0ne | jklare: do you have any errors in logs? | 12:05 |
| jklare | e0ne sadly no :( | 12:05 |
| e0ne | :( | 12:05 |
| jklare | e0ne thats why i came here to maybe figure out if its v1 is always needed to get horizon to work :) | 12:06 |
| *** yumapath has joined #openstack-cinder | 12:06 | |
| *** ildikov has joined #openstack-cinder | 12:07 | |
| david-lyle | jklare: I think v1 is required, shouldn't be, but is | 12:07 |
| jklare | david-lyle yeah, seems like that | 12:07 |
| david-lyle | hopefully won't be by m-3 | 12:07 |
| david-lyle | but there should be a bug open for that | 12:08 |
| jklare | david-lyle have not found any, but maybe i used the wrong criteria to search | 12:09 |
| *** esker has quit IRC | 12:09 | |
| *** avishay has joined #openstack-cinder | 12:10 | |
| david-lyle | jklare: https://bugs.launchpad.net/horizon/+bug/1415712 is what I was thinking of | 12:10 |
| openstack | Launchpad bug 1415712 in OpenStack Dashboard (Horizon) ""Volumes" tab doesn't show up if cinder v1 isn't registered" [High,In progress] - Assigned to Itxaka Serrano (itxakaserrano) | 12:10 |
| jklare | david-lyle thank you so much | 12:11 |
| jklare | david-lyle no idea why i did not find this | 12:11 |
| *** julim has joined #openstack-cinder | 12:11 | |
| *** links has quit IRC | 12:11 | |
| david-lyle | jklare: was tagged helpfully | 12:12 |
| david-lyle | *wasn't | 12:12 |
| david-lyle | fixed that | 12:12 |
| jklare | david-lyle ahhh, its in the horizon launchpad bugs, i was looking through the cinder one... my bad | 12:12 |
| jklare | david-lyle but yeah, its more a bug in horizon than in cinder | 12:13 |
| *** gouthamr has joined #openstack-cinder | 12:15 | |
| *** raildo-afk is now known as raildo | 12:17 | |
| *** geguileo has quit IRC | 12:20 | |
| *** zul has quit IRC | 12:24 | |
| *** aagrawal has joined #openstack-cinder | 12:28 | |
| *** baojg has quit IRC | 12:29 | |
| *** ociuhandu_ has joined #openstack-cinder | 12:30 | |
| *** yumapath has quit IRC | 12:31 | |
| *** dims has joined #openstack-cinder | 12:32 | |
| *** haomaiw__ has joined #openstack-cinder | 12:32 | |
| *** qeelee has joined #openstack-cinder | 12:33 | |
| *** ankit_ag has quit IRC | 12:34 | |
| *** haomaiwa_ has quit IRC | 12:34 | |
| *** lixiaoy1 has quit IRC | 12:34 | |
| *** shyama has quit IRC | 12:34 | |
| *** ociuhandu has quit IRC | 12:34 | |
| *** ociuhandu_ is now known as ociuhandu | 12:34 | |
| *** shyama has joined #openstack-cinder | 12:34 | |
| *** abhishekk has quit IRC | 12:36 | |
| *** zhangjn has quit IRC | 12:39 | |
| *** gouthamr has quit IRC | 12:39 | |
| -openstackstatus- NOTICE: Infra running with lower capacity now, due to a temporary problem affecting one of our nodepool providers. Please expect some delays in your jobs. Apologies for any inconvenience caused. | 12:39 | |
| *** zhangjn has joined #openstack-cinder | 12:42 | |
| *** esker has joined #openstack-cinder | 12:46 | |
| *** zhangjn has quit IRC | 12:47 | |
| *** zhangjn has joined #openstack-cinder | 12:48 | |
| *** zhangjn has quit IRC | 12:48 | |
| *** zhangjn has joined #openstack-cinder | 12:49 | |
| *** esker has quit IRC | 12:51 | |
| *** julim has quit IRC | 12:54 | |
| *** zul has joined #openstack-cinder | 12:55 | |
| *** sinese has quit IRC | 12:56 | |
| *** julim has joined #openstack-cinder | 12:57 | |
| *** sinese has joined #openstack-cinder | 12:57 | |
| *** bill_az has joined #openstack-cinder | 13:00 | |
| *** haomaiw__ has quit IRC | 13:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 13:01 | |
| *** sinese_ has joined #openstack-cinder | 13:01 | |
| *** openstackgerrit has quit IRC | 13:02 | |
| *** openstackgerrit has joined #openstack-cinder | 13:03 | |
| *** sinese has quit IRC | 13:04 | |
| *** geguileo has joined #openstack-cinder | 13:05 | |
| *** zhipeng has joined #openstack-cinder | 13:10 | |
| *** merooney has quit IRC | 13:10 | |
| *** merooney has joined #openstack-cinder | 13:10 | |
| *** nibalizer has joined #openstack-cinder | 13:12 | |
| *** jistr|mtg has joined #openstack-cinder | 13:12 | |
| *** haomaiwa_ has quit IRC | 13:16 | |
| *** alonma has quit IRC | 13:17 | |
| *** alonma has joined #openstack-cinder | 13:17 | |
| *** lprice1 has quit IRC | 13:20 | |
| *** gouthamr has joined #openstack-cinder | 13:21 | |
| *** alonma has quit IRC | 13:22 | |
| *** lprice has joined #openstack-cinder | 13:22 | |
| *** xyang1 has joined #openstack-cinder | 13:23 | |
| *** alonma has joined #openstack-cinder | 13:23 | |
| *** alonma has quit IRC | 13:28 | |
| *** timcl has joined #openstack-cinder | 13:29 | |
| *** alonma has joined #openstack-cinder | 13:29 | |
| *** laughterwym has joined #openstack-cinder | 13:30 | |
| *** gman-tx has joined #openstack-cinder | 13:31 | |
| *** gman-tx has quit IRC | 13:32 | |
| *** aagrawal has quit IRC | 13:33 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Support for consistency groups in ScaleIO driver https://review.openstack.org/269624 | 13:33 |
| *** itzdilip has quit IRC | 13:34 | |
| *** alonma has quit IRC | 13:35 | |
| *** alonma has joined #openstack-cinder | 13:35 | |
| *** sinese_ has quit IRC | 13:36 | |
| *** haomaiwang has joined #openstack-cinder | 13:36 | |
| *** itzdilip has joined #openstack-cinder | 13:38 | |
| *** edmondsw has joined #openstack-cinder | 13:38 | |
| *** amoturi has joined #openstack-cinder | 13:39 | |
| *** cdelatte has quit IRC | 13:39 | |
| *** sinese_ has joined #openstack-cinder | 13:39 | |
| *** fthiagogv has quit IRC | 13:40 | |
| *** qeelee has quit IRC | 13:42 | |
| *** zhipeng has quit IRC | 13:46 | |
| *** vivekd has quit IRC | 13:46 | |
| *** merooney has quit IRC | 13:47 | |
| *** akerr has joined #openstack-cinder | 13:48 | |
| *** baojg has joined #openstack-cinder | 13:51 | |
| *** merooney has joined #openstack-cinder | 13:52 | |
| *** cdelatte has joined #openstack-cinder | 13:55 | |
| *** abehl has joined #openstack-cinder | 13:56 | |
| *** erlon has joined #openstack-cinder | 13:57 | |
| *** isaacb has joined #openstack-cinder | 13:57 | |
| *** martyturner has joined #openstack-cinder | 13:58 | |
| *** cknight has joined #openstack-cinder | 13:59 | |
| *** martyturner has left #openstack-cinder | 14:00 | |
| *** baojg has quit IRC | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 14:01 | |
| *** itzdilip has quit IRC | 14:04 | |
| *** julim has quit IRC | 14:05 | |
| *** baojg has joined #openstack-cinder | 14:07 | |
| *** baojg has quit IRC | 14:08 | |
| *** skraynev has quit IRC | 14:08 | |
| *** baojg has joined #openstack-cinder | 14:09 | |
| *** isaacb has quit IRC | 14:09 | |
| *** mc_nair has joined #openstack-cinder | 14:11 | |
| *** baumann has joined #openstack-cinder | 14:12 | |
| *** zhangjn_ has joined #openstack-cinder | 14:15 | |
| *** rlrossit has joined #openstack-cinder | 14:16 | |
| *** jistr|mtg is now known as jistr | 14:20 | |
| *** zhangjn_ has quit IRC | 14:20 | |
| *** zhangjn_ has joined #openstack-cinder | 14:22 | |
| *** cdelatte has quit IRC | 14:22 | |
| *** krotscheck1 has joined #openstack-cinder | 14:23 | |
| *** baojg has quit IRC | 14:23 | |
| krotscheck1 | I've got a patch that's been idling in the queue for about a month. Is there some process thing I need to do to get cores to look at it? https://review.openstack.org/#/c/265409/ | 14:23 |
| *** merooney has quit IRC | 14:23 | |
| smcginnis | krotscheck1: I'll take a look. No special process. | 14:24 |
| smcginnis | krotscheck1: Usually things get looked at sooner, but I think folks have been busy lately. | 14:24 |
| krotscheck1 | smcginnis: No worries, I just want to make sure I don't get my hand slapped like I did when I asked nova to review things. | 14:25 |
| *** jwcroppe has quit IRC | 14:26 | |
| scottda | krotscheck1: We don't slap here. We use the Love and Logic method | 14:26 |
| *** vgridnev_ has quit IRC | 14:27 | |
| smcginnis | :) | 14:27 |
| *** vgridnev_ has joined #openstack-cinder | 14:27 | |
| smcginnis | And then we get out the crowbar. | 14:27 |
| * krotscheck1 throws a rainbow at scottda | 14:27 | |
| * krotscheck1 throws a rainbow crowbar at smcginnis | 14:29 | |
| *** merooney has joined #openstack-cinder | 14:30 | |
| *** bkumar has quit IRC | 14:30 | |
| *** baojg has joined #openstack-cinder | 14:32 | |
| *** baumann has quit IRC | 14:32 | |
| *** jgregor has joined #openstack-cinder | 14:34 | |
| *** skraynev has joined #openstack-cinder | 14:35 | |
| *** edtubill has joined #openstack-cinder | 14:36 | |
| *** cdelatte has joined #openstack-cinder | 14:40 | |
| *** delattec has joined #openstack-cinder | 14:42 | |
| *** merooney has quit IRC | 14:43 | |
| *** Julien-zte has joined #openstack-cinder | 14:44 | |
| *** Julien-zte has quit IRC | 14:44 | |
| *** cdelatte has quit IRC | 14:45 | |
| *** baojg has quit IRC | 14:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/275729 | 14:46 |
| *** tongli has joined #openstack-cinder | 14:47 | |
| *** julim has joined #openstack-cinder | 14:48 | |
| *** JoseMello has joined #openstack-cinder | 14:48 | |
| *** baojg has joined #openstack-cinder | 14:49 | |
| *** anteaya has quit IRC | 14:49 | |
| *** cfriesen has joined #openstack-cinder | 14:50 | |
| *** diablo_rojo has joined #openstack-cinder | 14:50 | |
| *** pots1 has joined #openstack-cinder | 14:52 | |
| *** anteaya has joined #openstack-cinder | 14:54 | |
| *** pradipm has joined #openstack-cinder | 14:54 | |
| *** mriedem has joined #openstack-cinder | 14:54 | |
| *** cknight has quit IRC | 14:56 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: WIP Fix enforcement of default child quota values https://review.openstack.org/275734 | 14:59 |
| *** zhangjn has quit IRC | 15:00 | |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** pradipm has quit IRC | 15:01 | |
| *** cknight has joined #openstack-cinder | 15:01 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext: Update typos https://review.openstack.org/266696 | 15:01 |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** pradipm has joined #openstack-cinder | 15:02 | |
| *** delatte has joined #openstack-cinder | 15:03 | |
| *** delattec has quit IRC | 15:03 | |
| *** delatte has quit IRC | 15:04 | |
| *** zhangjn has joined #openstack-cinder | 15:04 | |
| *** timcl has quit IRC | 15:06 | |
| *** timcl has joined #openstack-cinder | 15:07 | |
| DuncanT | krotscheck1: Asking here is totally reasonable, though I wouldn't recommend more often than once every few days at most | 15:09 |
| *** baumann has joined #openstack-cinder | 15:09 | |
| *** dustins has joined #openstack-cinder | 15:09 | |
| krotscheck1 | DuncanT: So what'd you say is the nag-threshold? | 15:09 |
| *** eharney has joined #openstack-cinder | 15:10 | |
| DuncanT | krotscheck1: Every day feels like nagging to me to be sure. | 15:10 |
| DuncanT | krotscheck1: A couple of times a week is fine | 15:10 |
| DuncanT | krotscheck1: I actually like people who pop up to ask, it gives me a good feeling they'll respond to feedback rather than letting code rot with a -1 | 15:11 |
| krotscheck1 | DuncanT: Yeah, I feel about the same. | 15:11 |
| DuncanT | krotscheck1: Also, there's a header used by os-profiler I think, that I can't see mentioned in the patch | 15:12 |
| *** cdelatte has joined #openstack-cinder | 15:12 | |
| DuncanT | krotscheck1: I don't remember what the header is, I vaguely remember it existing | 15:12 |
| krotscheck1 | DuncanT: Lemme look | 15:12 |
| *** jwcroppe has joined #openstack-cinder | 15:13 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: WIP Fixup nested quotas to enforce child defaults https://review.openstack.org/274825 | 15:14 |
| *** Yogi1 has joined #openstack-cinder | 15:14 | |
| krotscheck1 | DuncanT: X-Trace-Info, X-Trace-HMAC | 15:16 |
| DuncanT | krotscheck1: Any chance of a followup patch, please? Thanks | 15:16 |
| *** boris-42 has joined #openstack-cinder | 15:16 | |
| krotscheck1 | Of course, just a minute | 15:16 |
| *** kurtmartin has joined #openstack-cinder | 15:17 | |
| *** mtanino has joined #openstack-cinder | 15:19 | |
| *** delattec has joined #openstack-cinder | 15:19 | |
| openstackgerrit | Michael Krotscheck proposed openstack/cinder: Added Keystone and RequestID headers to CORS middleware https://review.openstack.org/265409 | 15:19 |
| openstackgerrit | Michael Krotscheck proposed openstack/cinder: Added osprofiler headers to cors middleware https://review.openstack.org/275751 | 15:19 |
| krotscheck1 | ARGH | 15:19 |
| krotscheck1 | WHAT | 15:19 |
| krotscheck1 | NO | 15:19 |
| * krotscheck1 did NOT want a rebase | 15:20 | |
| * krotscheck1 facepalms | 15:20 | |
| krotscheck1 | Oh well. Let's start this over again ;) | 15:20 |
| *** Yogi1 has quit IRC | 15:21 | |
| *** cdelatte has quit IRC | 15:22 | |
| *** garthb has joined #openstack-cinder | 15:22 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 15:23 |
| openstackgerrit | Jin Nan Zhang proposed openstack/python-cinderclient: add tags to .gitignore https://review.openstack.org/275754 | 15:23 |
| openstackgerrit | Jose Porrua proposed openstack/python-cinderclient: Avoid logging sensitive info in http requests. https://review.openstack.org/275755 | 15:24 |
| *** delatte has joined #openstack-cinder | 15:27 | |
| *** arecknag has quit IRC | 15:27 | |
| *** merooney has joined #openstack-cinder | 15:27 | |
| *** delattec has quit IRC | 15:28 | |
| *** delatte has quit IRC | 15:29 | |
| *** liverpooler has quit IRC | 15:31 | |
| *** esker has joined #openstack-cinder | 15:32 | |
| *** pradipm has quit IRC | 15:33 | |
| *** knikolla has joined #openstack-cinder | 15:33 | |
| *** alonma has quit IRC | 15:34 | |
| *** arch-nemesis has joined #openstack-cinder | 15:34 | |
| *** baojg has quit IRC | 15:35 | |
| *** baojg has joined #openstack-cinder | 15:35 | |
| *** cdelatte has joined #openstack-cinder | 15:37 | |
| *** delattec has joined #openstack-cinder | 15:40 | |
| *** alonma has joined #openstack-cinder | 15:40 | |
| *** cdelatte has quit IRC | 15:43 | |
| *** alonma has quit IRC | 15:44 | |
| *** porrua has joined #openstack-cinder | 15:45 | |
| *** laughterwym has quit IRC | 15:45 | |
| *** laughterwym has joined #openstack-cinder | 15:46 | |
| *** zul has quit IRC | 15:46 | |
| *** alonma has joined #openstack-cinder | 15:46 | |
| smcginnis | krotscheck1: Since that's going through again, want to just add the osprofile headers to the one and abandon that other. | 15:48 |
| smcginnis | krotscheck1: I'm actually fine either way, just thinking that might be simpler. | 15:48 |
| *** baojg has quit IRC | 15:49 | |
| *** zhangjn has quit IRC | 15:50 | |
| *** laughterwym has quit IRC | 15:50 | |
| mtanino | xyang1: Hi, Do you know Peter still have retype problem? https://bugs.launchpad.net/cinder/+bug/1536873 | 15:50 |
| openstack | Launchpad bug 1536873 in Cinder "'module' object has no attribute 'is_admin' when retyping" [High,New] - Assigned to Mitsuhiro Tanino (mitsuhiro-tanino) | 15:50 |
| *** alonma has quit IRC | 15:51 | |
| *** zhangjn has joined #openstack-cinder | 15:51 | |
| *** zhangjn has quit IRC | 15:51 | |
| *** alonma has joined #openstack-cinder | 15:52 | |
| *** salv-orl_ has joined #openstack-cinder | 15:54 | |
| xyang1 | mtanino: I need to check with him | 15:56 |
| *** baojg has joined #openstack-cinder | 15:56 | |
| mtanino | xyang1: If you have a chance, please check it. I tried to reproduced a bug, but it seems the bug was already fixed. | 15:56 |
| *** salv-orlando has quit IRC | 15:57 | |
| *** alonma has quit IRC | 15:57 | |
| *** sheel has quit IRC | 15:57 | |
| *** amoturi has quit IRC | 15:57 | |
| *** nkrinner has quit IRC | 15:57 | |
| *** amoturi has joined #openstack-cinder | 15:58 | |
| *** alonma has joined #openstack-cinder | 15:59 | |
| xyang1 | mtanino: I'll get back to you tomorrow | 15:59 |
| mtanino | xyang1: Thank you. I'll left a comment on bug from my test result. | 16:00 |
| xyang1 | mtanino: thanks | 16:00 |
| *** jgriffith_away is now known as jgriffith | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** jdurgin1 has joined #openstack-cinder | 16:02 | |
| *** vgridnev_ has quit IRC | 16:02 | |
| *** lennyb has quit IRC | 16:03 | |
| *** alonma has quit IRC | 16:03 | |
| *** thurloat is now known as thurloat_isgone | 16:03 | |
| *** timcl1 has joined #openstack-cinder | 16:04 | |
| *** esker has quit IRC | 16:05 | |
| *** alonma has joined #openstack-cinder | 16:05 | |
| *** esker has joined #openstack-cinder | 16:05 | |
| *** timcl has quit IRC | 16:07 | |
| *** alonma has quit IRC | 16:10 | |
| *** alonma has joined #openstack-cinder | 16:11 | |
| openstackgerrit | Merged openstack/python-cinderclient: CLI for backup snapshots https://review.openstack.org/245632 | 16:12 |
| *** rajinir has joined #openstack-cinder | 16:12 | |
| *** shyama has quit IRC | 16:15 | |
| *** alonma has quit IRC | 16:15 | |
| *** shyama has joined #openstack-cinder | 16:15 | |
| *** lennyb has joined #openstack-cinder | 16:16 | |
| *** daneyon has joined #openstack-cinder | 16:24 | |
| jgriffith | smcginnis: replication v2.1 start https://review.openstack.org/#/c/275797/ | 16:25 |
| smcginnis | jgriffith: Thank you!! | 16:25 |
| jgriffith | smcginnis: sure... I don't think the object versioning fix I made for the merge conflict is *right* but I'll look at it later and check with dulek if I can't figure it out | 16:26 |
| *** daneyon_ has quit IRC | 16:27 | |
| smcginnis | jgriffith: Good enough for now. At least we can see some code. | 16:27 |
| *** belmoreira has quit IRC | 16:28 | |
| openstackgerrit | Kedar Vidvans proposed openstack/cinder: Volume manage/unmanage support to ZFSSA drivers https://review.openstack.org/271462 | 16:29 |
| *** vincent_hou has joined #openstack-cinder | 16:29 | |
| *** shausy has joined #openstack-cinder | 16:34 | |
| *** anshul has quit IRC | 16:39 | |
| *** abehl has quit IRC | 16:39 | |
| *** mgarza_ has joined #openstack-cinder | 16:39 | |
| *** yumapath_ has joined #openstack-cinder | 16:40 | |
| *** mriedem is now known as mriedem_afk | 16:40 | |
| *** huanan_L has joined #openstack-cinder | 16:41 | |
| *** merooney has quit IRC | 16:42 | |
| *** julim has quit IRC | 16:42 | |
| *** huanan has quit IRC | 16:44 | |
| *** merooney has joined #openstack-cinder | 16:44 | |
| *** julim has joined #openstack-cinder | 16:44 | |
| *** ndipanov has joined #openstack-cinder | 16:48 | |
| *** aarefiev has quit IRC | 16:48 | |
| *** aarefiev has joined #openstack-cinder | 16:49 | |
| flip214 | smcginnis: yeah, it moved forward .... right into a -2. "Discussed at the nova/cinder midcycles and agreed that new rootwrap filters for nova/cinder/brick have to go through privsep in os-brick." | 16:49 |
| smcginnis | flip214: Heh, at least it got an answer I guess. :] | 16:49 |
| smcginnis | We have a few things depending on that now. | 16:50 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication https://review.openstack.org/237387 | 16:50 |
| *** merooney has quit IRC | 16:51 | |
| *** delattec has quit IRC | 16:51 | |
| *** jordanP has quit IRC | 16:52 | |
| *** jordanP has joined #openstack-cinder | 16:53 | |
| flip214 | jgriffith: I'll read the cheesecake spec, and change/redo the replication API for DRBD ... had a patch lined up, after some other (more important) changes, but I guess I can reuse quite some of that anyway. | 16:54 |
| *** sheel_ has joined #openstack-cinder | 16:54 | |
| mc_nair | smcginnis: here are the bugs where you can still currently get yourself in trouble with Nested Quotas in Liberty today - https://bugs.launchpad.net/cinder/+bug/1531502, https://bugs.launchpad.net/cinder/+bug/1537189 | 16:54 |
| openstack | Launchpad bug 1531502 in Cinder "Child project's default quota not enforced" [High,New] - Assigned to Ryan McNair (rdmcnair) | 16:54 |
| *** lpetrut has quit IRC | 16:54 | |
| openstack | Launchpad bug 1537189 in Cinder "Cinder nested quotas does not handle unlimited quotas (-1) well" [High,New] - Assigned to Ryan McNair (rdmcnair) | 16:54 |
| smcginnis | mc_nair: Thanks for working on that. | 16:56 |
| mc_nair | https://review.openstack.org/#/c/275734/ (still needs a few minor fixups) should fix the first issue and avoid the second bug going forward (because disables -1 on child) | 16:56 |
| DuncanT | sheel_: Ok, so I think patch set 5 can go in now, and it would be great if you could work with scottda to propose patch set 6 as a microversion against his new microversion code (which isn't up in the state it needs to be yet, I don't think, though he'd be able to tell you more about it) | 16:56 |
| *** EinstCrazy has quit IRC | 16:56 | |
| sheel_ | DuncanT:ok thnx | 16:57 |
| scottda | sheel_: Yes, let me know and I can help | 16:57 |
| scottda | sheel_: What is the patch set in question? | 16:57 |
| sheel_ | Scottda: thanks, its https://review.openstack.org/272232 | 16:57 |
| *** jistr has quit IRC | 16:57 | |
| *** belmoreira has joined #openstack-cinder | 16:57 | |
| scottda | cool, I'll have a look | 16:58 |
| sheel_ | scottda: would u be able to give some idea abt it today? | 16:58 |
| DuncanT | sheel_: That way you get the feature in, then we get a simple and easy-to-test change for a our first microversion, with a slightly cleaner API | 16:58 |
| sheel_ | DuncanT: yes its right way | 16:59 |
| *** e0ne has quit IRC | 16:59 | |
| scottda | sheel_: possibly, but I'm prioritizing work on the microversion patch itself. | 16:59 |
| sheel_ | scottda:please take ur time | 16:59 |
| *** IlyaG has joined #openstack-cinder | 17:00 | |
| sheel_ | just let me know whenevee u find time to discuss | 17:00 |
| scottda | ok | 17:00 |
| sheel_ | DuncanT:so i ll be releasing patchset5in a while | 17:00 |
| mc_nair | smcginnis: sure thing - was happy to have something to dig into. I'll give you some time before continuing my rant, but just lemme know when you want discuss more on the current state / where to go next | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** Apoorva has joined #openstack-cinder | 17:01 | |
| sheel_ | Duncan:please review as per time availibility | 17:01 |
| DuncanT | sheel_: I'm only on intermittently since it is getting late, but ping me a link and I'll get to it | 17:01 |
| *** 7F1AA87I3 has joined #openstack-cinder | 17:01 | |
| *** sheel_ has quit IRC | 17:01 | |
| *** sheel has joined #openstack-cinder | 17:02 | |
| *** itlinux has quit IRC | 17:02 | |
| sheel | scottda DuncanT: Thnx | 17:02 |
| *** alonma has joined #openstack-cinder | 17:04 | |
| mtanino | DuncanT: Hi, Could you revisit this metadata support patch? https://review.openstack.org/#/c/258649/ I have fixed your comment. Thanks. | 17:04 |
| mtanino | DuncanT: I'd like to push glance side too. | 17:04 |
| *** dims has quit IRC | 17:04 | |
| *** shyama has quit IRC | 17:05 | |
| *** shyama has joined #openstack-cinder | 17:05 | |
| *** dims has joined #openstack-cinder | 17:06 | |
| *** garthb has quit IRC | 17:06 | |
| *** fthiagogv has joined #openstack-cinder | 17:07 | |
| *** alonma has quit IRC | 17:08 | |
| *** alonma has joined #openstack-cinder | 17:10 | |
| *** dims has quit IRC | 17:12 | |
| *** sinese_ has quit IRC | 17:13 | |
| *** sgotliv has quit IRC | 17:14 | |
| *** alonma has quit IRC | 17:15 | |
| *** geguileo has quit IRC | 17:15 | |
| *** baojg has quit IRC | 17:16 | |
| *** baumann has quit IRC | 17:16 | |
| *** mvk_ has quit IRC | 17:16 | |
| *** leeantho has joined #openstack-cinder | 17:17 | |
| *** baumann has joined #openstack-cinder | 17:17 | |
| *** dims has joined #openstack-cinder | 17:18 | |
| *** jgriffith is now known as jgriffith_away | 17:25 | |
| *** avishay has quit IRC | 17:26 | |
| *** chhavi has quit IRC | 17:30 | |
| *** julim has quit IRC | 17:33 | |
| *** IlyaG has quit IRC | 17:33 | |
| openstackgerrit | Merged openstack/cinder: Fix the replication spelling in message https://review.openstack.org/275509 | 17:34 |
| *** anshul has joined #openstack-cinder | 17:37 | |
| *** abehl has joined #openstack-cinder | 17:37 | |
| *** jgriffith_away is now known as jgriffith | 17:39 | |
| *** merooney has joined #openstack-cinder | 17:40 | |
| *** julim has joined #openstack-cinder | 17:42 | |
| *** garthb has joined #openstack-cinder | 17:43 | |
| jgriffith | patrickeast: Swanson smcginnis hemna let me know if that looks like what you expected so far | 17:44 |
| *** belmoreira has quit IRC | 17:45 | |
| *** shyama has quit IRC | 17:45 | |
| patrickeast | jgriffith: have only taken a quick look but so far so good | 17:45 |
| smcginnis | jgriffith: Yeah, quick pass so far. Nothing big stood out. | 17:45 |
| patrickeast | jgriffith: there was one comment on the spec after it merged asking about the list replication targets api, would be easy enough to leave that one in and take out the vref part | 17:46 |
| patrickeast | jgriffith: dunno if we want to do it now though or leave it for a later iteration | 17:46 |
| patrickeast | i'm not sure how helpful it really is | 17:47 |
| *** vincent_hou has quit IRC | 17:47 | |
| jgriffith | patrickeast: I didn't see the question, but I kinda figured there wasn't much value in having it... | 17:48 |
| jgriffith | patrickeast: the info is in the config file, and the capabilities should reflect that it's succesfuly configured | 17:48 |
| jgriffith | At least I *think* :) | 17:49 |
| patrickeast | jgriffith: yep, totally agree | 17:49 |
| jgriffith | patrickeast: I'll work up the reporting for that in the driver next maybe | 17:49 |
| smcginnis | jgriffith: Only reason for having a list I can think of would be to be able to pull out that information via API for some kind of external system. | 17:50 |
| smcginnis | Otherwise, yeah, just look at the config. | 17:50 |
| jgriffith | smcginnis: maybe, but IMHO that would just morph into a list of backend_id's in this model | 17:50 |
| patrickeast | yea it could be much more simple | 17:51 |
| jgriffith | smcginnis: Probably useful... I can look at adding that in | 17:51 |
| smcginnis | jgriffith: Yeah, not saying it's needed. I'm happy wiht the way it is. ;) | 17:51 |
| jgriffith | patrickeast: yeah, honestly that whole thing was kind of a mess | 17:51 |
| patrickeast | jgriffith: hehe yea, the trick will be to avoid falling into the same trap this time too | 17:52 |
| * smcginnis crosses fingers | 17:52 | |
| * patrickeast says that after suggesting we add another api >.> | 17:52 | |
| jgriffith | patrickeast: yeah... but I'm just going to be a jerk this time around (or... more of a jerk) :) | 17:52 |
| smcginnis | Hah | 17:52 |
| patrickeast | lol | 17:52 |
| jgriffith | NO MORE SNOW FLAKES | 17:52 |
| smcginnis | But they're so purdy. | 17:52 |
| patrickeast | what if its a really beautiful snowflake?! | 17:52 |
| jgriffith | I have a blow torch | 17:53 |
| jgriffith | Melt all the snow-flakes | 17:53 |
| smcginnis | Heh, I just shovelled out from 11" of snow this morning. Kill the snowflakes! | 17:53 |
| patrickeast | haha | 17:53 |
| jgriffith | smcginnis: don't get me started!!! 2 Feet at my place yesterday | 17:53 |
| jgriffith | Burried the Jeep/Plow in a 4 foot drift a mile from home | 17:53 |
| jgriffith | then ran out of gas | 17:53 |
| smcginnis | jgriffith: Is it all melted by today? | 17:53 |
| jgriffith | haha.. not this time | 17:54 |
| patrickeast | jgriffith: for the active_backend_id should I assume that will be a new parameter for the driver __init__ or do_setup? | 17:54 |
| jgriffith | This winter has been weird/cold | 17:54 |
| smcginnis | OK, you win this round! :) | 17:54 |
| *** jordanP has quit IRC | 17:54 | |
| jgriffith | patrickeast: actually, I'm putting that in the stats update... so the driver sets it when receiving the failover call. Then it gets persisted in the DB so that yes, on init the driver can figure out where it's supposed to be pointing to | 17:54 |
| jgriffith | smcginnis: LOL... one of the few times we beat out Minnesota | 17:55 |
| jgriffith | Thankfully!! | 17:55 |
| smcginnis | We still win on being frickin' cold. | 17:55 |
| jgriffith | no doubt... cold in my book is low twenties | 17:55 |
| *** IlyaG has joined #openstack-cinder | 17:56 | |
| jgriffith | but the sun is still shining bright at least | 17:56 |
| smcginnis | Hah! | 17:56 |
| *** thurloat_isgone is now known as thurloat | 17:56 | |
| patrickeast | jgriffith: yea so for the on init part, it would be a new param? or should i plan to do some db querying? | 17:57 |
| * patrickeast might be missing something in the code | 17:57 | |
| patrickeast | just tryin to scope out the changes for our driver | 17:57 |
| jgriffith | patrickeast: yeah, just realized what you're asking. I had anticipated passing it in as an optional kwarg | 17:57 |
| patrickeast | jgriffith: ok cool | 17:57 |
| smcginnis | Swanson: Might want to pay attention if you're around. ^^ | 17:57 |
| jgriffith | patrickeast: keep the db stuff out of the driver | 17:57 |
| Swanson | What? | 17:58 |
| patrickeast | jgriffith: yea def, why i asked ;) | 17:58 |
| smcginnis | Swanson: Now's the time to hit up jgriffith with 20 questions. :D | 17:58 |
| jgriffith | patrickeast: cool, yeah good point | 17:58 |
| *** bkumar has joined #openstack-cinder | 17:58 | |
| smcginnis | Swanson: https://review.openstack.org/#/c/275797/ | 17:58 |
| *** 7F1AA87I3 has quit IRC | 18:01 | |
| *** ndipanov has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| *** mudassirlatif has joined #openstack-cinder | 18:02 | |
| *** sinese has joined #openstack-cinder | 18:04 | |
| smcginnis | jgriffith: I like this: +246, -557 | 18:05 |
| jgriffith | smcginnis: haha... we'll see how that ratio works out | 18:06 |
| smcginnis | jgriffith: It's a good start at least. | 18:06 |
| jgriffith | smcginnis: yeah, certainly the right direction | 18:06 |
| jgriffith | smcginnis: I'm planning to let others revert/remove all their old code in their drivers | 18:07 |
| jgriffith | smcginnis: starting querying on rep patches to propose reverts, but it was a bit tangled and realized it might be best to let driver maintainers do that | 18:07 |
| smcginnis | jgriffith: That's probably safer. I know HPE wanted to refactor what they had rather than revert. | 18:07 |
| smcginnis | I'm guessing most will just modify what they have. | 18:07 |
| jgriffith | smcginnis: yeah, I think that will be true for most | 18:07 |
| smcginnis | And hopefully be able to rip out a lot of code. | 18:07 |
| jgriffith | yeah, should be able to | 18:08 |
| *** merooney has quit IRC | 18:08 | |
| *** bkumar has quit IRC | 18:09 | |
| *** alonma has joined #openstack-cinder | 18:11 | |
| *** liverpooler has joined #openstack-cinder | 18:14 | |
| *** alonma has quit IRC | 18:15 | |
| *** liverpooler has quit IRC | 18:15 | |
| *** merooney has joined #openstack-cinder | 18:15 | |
| *** liverpooler has joined #openstack-cinder | 18:15 | |
| *** baojg has joined #openstack-cinder | 18:17 | |
| *** alonma has joined #openstack-cinder | 18:17 | |
| Apoorva | Hello, could you please review https://review.openstack.org/#/c/261126/ and https://review.openstack.org/#/c/268370/ | 18:18 |
| *** Apoorva is now known as apoorvad | 18:18 | |
| *** shausy has quit IRC | 18:20 | |
| *** alonma has quit IRC | 18:21 | |
| *** alonma has joined #openstack-cinder | 18:23 | |
| *** e0ne has joined #openstack-cinder | 18:24 | |
| patrickeast | jgriffith: ok had a more thorough look through the code and how things would work, i had one concern about the active backend id thing and sort of pushing the drivers to maintain state for it | 18:28 |
| patrickeast | after failover we would need to track that state in our driver which means any other c-vols (most problematic being active-active setups) may not know about it | 18:28 |
| jgriffith | patrickeast: I don't think so | 18:28 |
| *** IlyaG has quit IRC | 18:28 | |
| patrickeast | jgriffith: so like, in our driver (and i think some others) for the v2 replication we modify the volume entry in the db so every driver method we can see who to talk to | 18:28 |
| *** alonma has quit IRC | 18:28 | |
| patrickeast | jgriffith: but we don't have it with this, right? | 18:28 |
| jgriffith | patrickeast: so my take was that the service column would tell the driver which backend to access | 18:29 |
| jgriffith | patrickeast: so the "some volumes here, some there" problem just goes away | 18:29 |
| jgriffith | patrickeast: does that make sense? | 18:29 |
| *** IlyaG has joined #openstack-cinder | 18:29 | |
| jgriffith | patrickeast: because that case is kinda silly IMHO... either the backend is toast or it's not | 18:30 |
| patrickeast | jgriffith: yea that makes sense, lemmie take another stab at explaining | 18:30 |
| *** alonma has joined #openstack-cinder | 18:30 | |
| patrickeast | jgriffith: so if you have two c-vols looking for the same backend in HA kind of setup | 18:30 |
| jgriffith | slowwwly for thick headed people like me :) | 18:30 |
| jgriffith | yes... I'm with ya | 18:30 |
| patrickeast | jgriffith: one of them handles the failover and updates its reference for which backend is active now | 18:30 |
| patrickeast | jgriffith: the other one wouldn't know the failover happened, and would, until restart or something, not get the right active id | 18:30 |
| jgriffith | patrickeast: right, I think I see where you're going | 18:31 |
| *** vgridnev has joined #openstack-cinder | 18:31 | |
| jgriffith | patrickeast: so you're saying the problem is there's no guarantee that the other backend ever gets the failover command? | 18:31 |
| jgriffith | patrickeast: and as a result never updates it's internal memory version of the info | 18:31 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Bootable filter for listening volumes from CLI https://review.openstack.org/272229 | 18:32 |
| patrickeast | jgriffith: right, the older one we were basically stashing that info in the db and every driver method that would need to know could look at the volume/snapshot/whatever and do the right thing | 18:32 |
| patrickeast | jgriffith: dunno which way we want to go with this | 18:32 |
| patrickeast | jgriffith: could lean in and put some mechanism in to handle it... or maybe just update all the volumes/snaps after a failover | 18:33 |
| *** baojg has quit IRC | 18:33 | |
| jgriffith | patrickeast: thinking | 18:33 |
| *** alonma has quit IRC | 18:34 | |
| openstackgerrit | Jose Porrua proposed openstack/python-cinderclient: Avoid logging sensitive info in http requests. https://review.openstack.org/275755 | 18:35 |
| *** bkumar has joined #openstack-cinder | 18:36 | |
| *** alonma has joined #openstack-cinder | 18:36 | |
| *** alonma has quit IRC | 18:40 | |
| *** mvk_ has joined #openstack-cinder | 18:41 | |
| *** bkumar has quit IRC | 18:42 | |
| jgriffith | patrickeast: wonder if we need a reverse stats update | 18:43 |
| *** alonma has joined #openstack-cinder | 18:46 | |
| *** liverpooler has quit IRC | 18:47 | |
| eharney | does someone know what's supposed to happen currently when you SIGHUP cinder-volume? | 18:49 |
| smcginnis | eharney: I remember some patches for that, but can't recall. | 18:49 |
| patrickeast | jgriffith: yea something like that would work | 18:50 |
| smcginnis | eharney: Was SIGHUP for guru meditation reports? | 18:50 |
| eharney | smcginnis: no, that was SIGUSR1/2 | 18:50 |
| *** alonma has quit IRC | 18:50 | |
| *** lcurtis has joined #openstack-cinder | 18:50 | |
| smcginnis | eharney: Ah, that's it. | 18:50 |
| patrickeast | jgriffith: or even like calling do_setup again... although i guess that probably breaks tons of stuff | 18:50 |
| eharney | tl;dr funny stuff happens when you SIGHUP it | 18:50 |
| smcginnis | I remember it had to change from the original signal because it wasn't supported on every platform or something. | 18:51 |
| patrickeast | jgriffith: sort of like a step towards doing live config changes on a c-vol service :D | 18:51 |
| smcginnis | eharney: "Funny" stuff. | 18:51 |
| eharney | which is probably a bug | 18:51 |
| eharney | smcginnis: well, on one platform it said "Alarm clock" and died. The less funny one was where it re-inited the LVM driver and started running a second dd wipe on the same LV device | 18:51 |
| *** esp_ has joined #openstack-cinder | 18:51 | |
| *** crose has quit IRC | 18:51 | |
| *** ChubYann has joined #openstack-cinder | 18:52 | |
| smcginnis | Alarm clock? That is funny. | 18:52 |
| dims | smcginnis : we switched from SIGUSR1 to SIGUSR2 for guru meditation reports | 18:52 |
| smcginnis | dd'ing your drive, not so funny. | 18:52 |
| dims | (in oslo.reports) | 18:52 |
| smcginnis | dims: OK, thanks! | 18:52 |
| *** alonma has joined #openstack-cinder | 18:52 | |
| smcginnis | Couldnt' recall | 18:52 |
| *** jgriffith is now known as jgriffith_away | 18:53 | |
| smcginnis | Gotta leave for a work event. I'll be offline the rest of the afternoon. | 18:53 |
| smcginnis | Leave a ping if you need anything from me. Might be back on this evening. | 18:53 |
| smcginnis | Unless the work event turns into a non-work event. Hah! | 18:54 |
| *** mc_nair_ has joined #openstack-cinder | 18:55 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Use of metadata id instead of metadata name https://review.openstack.org/274589 | 18:56 |
| *** yumapath_ has quit IRC | 18:56 | |
| *** mc_nair has quit IRC | 18:56 | |
| *** mc_nair_ is now known as mc_nair | 18:56 | |
| *** alonma has quit IRC | 18:56 | |
| *** jgriffith_away is now known as jgriffith | 18:57 | |
| *** esp_ has quit IRC | 18:58 | |
| *** alonma has joined #openstack-cinder | 18:58 | |
| *** liverpooler has joined #openstack-cinder | 18:58 | |
| *** bardia has joined #openstack-cinder | 18:59 | |
| *** rlrossit has quit IRC | 18:59 | |
| *** crose has joined #openstack-cinder | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-cinder | 19:01 | |
| diablo_rojo | xyang1: Hey Xing, I was wondering if, when you had a chance, you could look at the CI on this patch: https://review.openstack.org/#/c/267085/ ? I can't see anything related to the patch in the logs as to why the patch would be failing EMX-VNX CI when it passes everyone elses. No big rush, just thought I'd ask :) | 19:01 |
| *** sheel has quit IRC | 19:02 | |
| *** alonma has quit IRC | 19:03 | |
| *** julim has quit IRC | 19:04 | |
| *** alonma has joined #openstack-cinder | 19:05 | |
| xyang1 | diablo_rojo: sure | 19:06 |
| *** mylu has joined #openstack-cinder | 19:06 | |
| *** julim has joined #openstack-cinder | 19:06 | |
| *** geguileo has joined #openstack-cinder | 19:09 | |
| *** alonma has quit IRC | 19:09 | |
| diablo_rojo | xyang1: Thank you! | 19:12 |
| xyang1 | diablo_rojo: the failure was not caused by the patch. It failed before running any tests. let's see how the rerun goes | 19:13 |
| *** alonma has joined #openstack-cinder | 19:13 | |
| diablo_rojo | xyang1: Okay cool. Thanks for looking at it so quickly. | 19:15 |
| xyang1 | diablo_rojo: np | 19:15 |
| *** esp_ has joined #openstack-cinder | 19:17 | |
| *** alonma has quit IRC | 19:18 | |
| *** jgriffith is now known as jgriffith_away | 19:18 | |
| *** esp_ has quit IRC | 19:21 | |
| *** jgriffith_away is now known as jgriffith | 19:29 | |
| *** merooney has quit IRC | 19:30 | |
| *** merooney has joined #openstack-cinder | 19:32 | |
| *** baojg has joined #openstack-cinder | 19:33 | |
| *** geguileo has quit IRC | 19:33 | |
| *** e0ne has quit IRC | 19:35 | |
| *** IlyaG has quit IRC | 19:36 | |
| *** liverpooler has quit IRC | 19:37 | |
| *** liverpooler has joined #openstack-cinder | 19:38 | |
| *** baojg has quit IRC | 19:38 | |
| *** JoseMello has quit IRC | 19:41 | |
| *** JoseMello has joined #openstack-cinder | 19:41 | |
| *** krotscheck1 is now known as krotscheck_dcm | 19:42 | |
| *** vgridnev has quit IRC | 19:43 | |
| *** boris-42 has quit IRC | 19:43 | |
| mc_nair | xyang1: hey - I was looking at your comment on https://review.openstack.org/#/c/275408/1. Looks like the change from ">=" to ">" from comments in https://review.openstack.org/#/c/185764/7/cinder/scheduler/filters/capacity_filter.py. The suggestion was to make that change in order to have thin-volumes behave like thick volumes when using max_oversubscription = 1, but it seems to have the opposite affect because it fails back to looking at | 19:47 |
| *** ociuhandu has quit IRC | 19:47 | |
| *** abehl has quit IRC | 19:48 | |
| *** anshul has quit IRC | 19:48 | |
| mc_nair | but then again I may just be totally off-base, was some tough mental gymnastics trying to figure out expected behavior :) | 19:49 |
| *** mgarza__ has joined #openstack-cinder | 19:50 | |
| *** vincent_hou has joined #openstack-cinder | 19:51 | |
| *** liverpooler has quit IRC | 19:51 | |
| *** mgarza_ has quit IRC | 19:53 | |
| *** rajinir has quit IRC | 19:55 | |
| openstackgerrit | Jose Porrua proposed openstack/python-cinderclient: Avoid logging sensitive info in http requests. https://review.openstack.org/275755 | 19:59 |
| *** haomaiwang has quit IRC | 20:01 | |
| *** 7JTAAES9V has joined #openstack-cinder | 20:01 | |
| *** rlrossit has joined #openstack-cinder | 20:01 | |
| *** vgridnev has joined #openstack-cinder | 20:02 | |
| *** merooney has quit IRC | 20:06 | |
| *** mudassirlatif_ has joined #openstack-cinder | 20:06 | |
| *** mudassirlatif has quit IRC | 20:09 | |
| *** mudassirlatif_ is now known as mudassirlatif | 20:09 | |
| *** dustins has quit IRC | 20:09 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Improve logging to debug invalid "extra_specs" entries https://review.openstack.org/275909 | 20:11 |
| *** rlrossit has quit IRC | 20:12 | |
| openstackgerrit | Alexey Khodos proposed openstack/cinder: NexentaStor5 iSCSI driver unit tests https://review.openstack.org/271537 | 20:13 |
| *** geguileo has joined #openstack-cinder | 20:13 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 20:14 |
| *** alonma has joined #openstack-cinder | 20:14 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Support name option for volume restore https://review.openstack.org/275910 | 20:15 |
| *** merooney has joined #openstack-cinder | 20:18 | |
| *** alonma has quit IRC | 20:18 | |
| *** mgarza__ has quit IRC | 20:19 | |
| xyang1 | mc_nair: that change does have problem. I just want to make sure we don't keep changing them back and force | 20:19 |
| *** rlrossit_ has joined #openstack-cinder | 20:20 | |
| mc_nair | xyang1: yea, for sure. I'm not overly familiar with that code path, but gave my take on it above. You want to discuss here or is it easiest if I just add my comment to the patch and we'll hash out there? | 20:23 |
| *** mylu has quit IRC | 20:23 | |
| *** sgotliv has joined #openstack-cinder | 20:25 | |
| *** mylu has joined #openstack-cinder | 20:26 | |
| *** lprice has quit IRC | 20:30 | |
| xyang1 | mc_nair: we can discuss here if you want | 20:30 |
| *** lprice has joined #openstack-cinder | 20:30 | |
| *** sgotliv has quit IRC | 20:36 | |
| *** gouthamr has quit IRC | 20:36 | |
| *** gouthamr has joined #openstack-cinder | 20:36 | |
| *** baojg has joined #openstack-cinder | 20:37 | |
| mc_nair | xyang1: cool. So from reading the config help, we say "A ratio lower than 1.0 will be ignored and the default value will be used instead." I haven't seen any validation that checks if we're using < 1 ratio and defaults it to 20. If we had that, I think the >1 part of the check could go away, as I think a ratio of "1" should enter the if check, because we really want to be comparing the provisioned capacity to 1 and not free capacity for | 20:39 |
| mc_nair | does that make sense to you? | 20:39 |
| *** lprice1 has joined #openstack-cinder | 20:41 | |
| *** lprice has quit IRC | 20:41 | |
| xyang1 | mc_nair: I think that help text is no longer accurate | 20:43 |
| *** thurloat is now known as thurloat_isgone | 20:43 | |
| *** baojg has quit IRC | 20:45 | |
| mc_nair | xyang1: I'd agree it's not accurate, do you know if that was intentional or not? I.e. should we be fixing the help tip or the code? My thought was we should be changing the code to be inline with the help tip | 20:45 |
| xyang1 | mc_nair: if someone sets it to negative and we use 20, that is too dramatic | 20:46 |
| *** mylu has quit IRC | 20:49 | |
| *** angela-s has joined #openstack-cinder | 20:49 | |
| *** mylu has joined #openstack-cinder | 20:49 | |
| *** crose has quit IRC | 20:50 | |
| *** apoorvad has quit IRC | 20:53 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Allow for eradicating Pure volumes on Cinder delete https://review.openstack.org/275919 | 20:53 |
| *** crose has joined #openstack-cinder | 20:55 | |
| *** dims has quit IRC | 20:56 | |
| *** 7JTAAES9V has quit IRC | 21:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 21:01 | |
| *** merooney has quit IRC | 21:03 | |
| *** vincent_hou has quit IRC | 21:05 | |
| *** edtubill has quit IRC | 21:05 | |
| *** edtubill has joined #openstack-cinder | 21:05 | |
| *** dims has joined #openstack-cinder | 21:05 | |
| *** kurtmartin has quit IRC | 21:06 | |
| *** mylu has quit IRC | 21:07 | |
| *** mylu has joined #openstack-cinder | 21:07 | |
| openstackgerrit | Jay Bryant proposed openstack/python-cinderclient: Remove debug statement https://review.openstack.org/275927 | 21:09 |
| *** raildo is now known as raildo-afk | 21:11 | |
| *** apoorvad has joined #openstack-cinder | 21:13 | |
| *** alonma has joined #openstack-cinder | 21:14 | |
| *** boris-42 has joined #openstack-cinder | 21:14 | |
| mc_nair | xyang1: ok, so then why don't we just not allow them to set < 1 ratio if we're using some other value anyway | 21:14 |
| xyang1 | mc_nair: so the check needs to be in the driver itself, not in the scheduler | 21:16 |
| *** esp_ has joined #openstack-cinder | 21:16 | |
| xyang1 | mc_nair: we can add a check in lvm driver | 21:17 |
| xyang1 | mc_nair: other driver can add the check too | 21:18 |
| *** alonma has quit IRC | 21:18 | |
| *** vgridnev has quit IRC | 21:18 | |
| *** akerr has quit IRC | 21:19 | |
| mc_nair | xyang1: not following.... how does that help us in the case that the user put max_over_subscription_ratio = 0.5? | 21:19 |
| xyang1 | mc_nair: check_for_setup_error | 21:20 |
| *** diablo_rojo has quit IRC | 21:20 | |
| xyang1 | mc_nair: handle it the same way as other invalid config | 21:20 |
| *** esp_ has quit IRC | 21:21 | |
| *** Lee1092 has quit IRC | 21:22 | |
| *** alonma has joined #openstack-cinder | 21:23 | |
| *** geguileo has quit IRC | 21:24 | |
| mc_nair | xyang1: are you just saying we would need to slide the validation of < 1 ratio somewhere that would fail early as oppose to in that part of the scheduler? | 21:24 |
| *** huanan_L has quit IRC | 21:25 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: WIP - ABC cleanup work - WIP https://review.openstack.org/272318 | 21:26 |
| *** huanan_L has joined #openstack-cinder | 21:26 | |
| *** julim has quit IRC | 21:27 | |
| xyang1 | mc_nair: the check in scheduler is still needed | 21:27 |
| xyang1 | mc_nair: but we can add an extra check in the driver | 21:27 |
| *** alonma has quit IRC | 21:28 | |
| *** alonma has joined #openstack-cinder | 21:29 | |
| *** zul has joined #openstack-cinder | 21:29 | |
| *** geguileo has joined #openstack-cinder | 21:29 | |
| *** sgotliv has joined #openstack-cinder | 21:32 | |
| *** alonma has quit IRC | 21:33 | |
| *** pratap has joined #openstack-cinder | 21:35 | |
| *** alonma has joined #openstack-cinder | 21:35 | |
| mc_nair | xyang1: the extra check that you speak of is enforcing that max_over_subscription_ratio >= 1 for a driver config, correct? | 21:38 |
| *** jgregor has quit IRC | 21:39 | |
| *** alonma has quit IRC | 21:39 | |
| *** edtubill has quit IRC | 21:39 | |
| *** zul has quit IRC | 21:40 | |
| *** thingee has quit IRC | 21:41 | |
| *** alonma has joined #openstack-cinder | 21:41 | |
| xyang1 | mc_nair: yes | 21:41 |
| *** timcl1 has quit IRC | 21:41 | |
| *** alonma has quit IRC | 21:46 | |
| *** jistr has joined #openstack-cinder | 21:46 | |
| *** jistr has quit IRC | 21:47 | |
| *** thingee has joined #openstack-cinder | 21:49 | |
| *** edtubill has joined #openstack-cinder | 21:50 | |
| *** edtubill has quit IRC | 21:50 | |
| *** mylu has quit IRC | 21:52 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP os-brick refactor get_connector_properties https://review.openstack.org/275943 | 21:52 |
| *** mylu has joined #openstack-cinder | 21:53 | |
| *** salv-orlando has joined #openstack-cinder | 21:54 | |
| *** jgregor has joined #openstack-cinder | 21:54 | |
| hemna | smcginnis, ^^ | 21:54 |
| hemna | hacking around in os-brick to see if doing that makes a bit more sense. | 21:54 |
| hemna | there is a windows connector patch in gerrit now that's ugly as can be. I'm hoping that my WIP can make it easier and cleaner to manage. | 21:55 |
| *** mylu has quit IRC | 21:56 | |
| *** mylu has joined #openstack-cinder | 21:56 | |
| *** mylu has quit IRC | 21:56 | |
| *** salv-orl_ has quit IRC | 21:56 | |
| *** edtubill has joined #openstack-cinder | 21:57 | |
| *** crose has quit IRC | 21:59 | |
| *** haomaiwa_ has quit IRC | 22:01 | |
| *** porrua has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| mc_nair | xyang1: ok. And so if we had the same (min, max) behavior we have for IntOpt in FloatOpt (which we don't currently) we could just specify that in the base driver.py class | 22:02 |
| *** alyson_ has quit IRC | 22:02 | |
| hemna | man check is backed up bad | 22:02 |
| hemna | 300 | 22:02 |
| hemna | :( | 22:02 |
| mc_nair | xyang1: but regardless, I think all that stuff of what to do with config < 1 is sort of a side discussion. The point of what to do with = 1 is still a discussion to have. From what I can tell the check should be >= 1 | 22:03 |
| xyang1 | mc_nair: I am not saying no, but I want to make sure no one will submit another patch to change it back again:) | 22:04 |
| xyang1 | mc_nair: need to ask lots of people to review it | 22:05 |
| *** PsionTheory has joined #openstack-cinder | 22:05 | |
| mc_nair | xyang1: haha ok. That seems fair. I agree about having lots of eyes on it before it merges. | 22:06 |
| xyang1 | mc_nair: at least add everyone from the previous patch | 22:06 |
| mc_nair | xyang1: gotcha. I can do that. Will add those people to the review. Anything else we should do on this? You care if the config validation we were talking about is this patch vs. follow up? | 22:08 |
| xyang1 | mc_nair: that can be a follow up | 22:08 |
| xyang1 | mc_nair: are you the author of the patch? i thought it is someone else | 22:09 |
| mc_nair | xyang1: not me. Just helping them out as it's their first patch and it's getting more complex | 22:11 |
| xyang1 | mc_nair: ok | 22:12 |
| mc_nair | xyang1: but I'll make sure the stuff we discussed gets done | 22:12 |
| xyang1 | mc_nair: thanks | 22:12 |
| mc_nair | xyang1: thanks for the chat - lemme know if you think of anything else on this | 22:12 |
| xyang1 | mc_nair: sure | 22:12 |
| *** gouthamr has quit IRC | 22:16 | |
| *** krotscheck_dcm has quit IRC | 22:17 | |
| guitarzan | this is a patch to give you the ability to not overprovision lvm thin volumes? | 22:17 |
| *** krotscheck1 has joined #openstack-cinder | 22:17 | |
| *** dims_ has joined #openstack-cinder | 22:17 | |
| *** mylu has joined #openstack-cinder | 22:18 | |
| *** sinese has quit IRC | 22:18 | |
| *** sgotliv has quit IRC | 22:21 | |
| *** dims has quit IRC | 22:21 | |
| dulek | scottda: You were asking about quota bug. I don't think you've linked the correct one. Or maybe you're observing the bug on stable/liberty? | 22:24 |
| *** baumann has quit IRC | 22:24 | |
| mc_nair | guitarzan: this is related to https://bugs.launchpad.net/cinder/+bug/1537162 | 22:24 |
| openstack | Launchpad bug 1537162 in Cinder "accounting bug for lvm with thin provisioning and max_over_subscription_ratio=1" [Undecided,In progress] - Assigned to Michael Rowden (mrrowden) | 22:24 |
| scottda | dulek: I've observed it on stable/liberty on our distro. I haven't attempted to repro on Devstack. | 22:24 |
| guitarzan | mc_nair: yeah, I'm kind of following | 22:25 |
| guitarzan | mc_nair: I just wanted to voice my support for this working correctly :) | 22:25 |
| xyang1 | mc_nair: eharney has a patch to fix the lvm thin provisioning | 22:25 |
| mc_nair | guitarzan: ah gotcha | 22:25 |
| xyang1 | mc_nair: I forgot the link | 22:25 |
| guitarzan | as in there are other reasons to want lvm thin other than over provisioning | 22:25 |
| wN | snapshots dont suck? | 22:26 |
| wN | heh | 22:26 |
| guitarzan | mostly "don't have to scrub" | 22:26 |
| openstackgerrit | Tom Barron proposed openstack/cinder: mock time.sleep in Broadcom unit test https://review.openstack.org/275952 | 22:26 |
| scottda | dulek: It's fixed with the latest Keystone policy.json | 22:26 |
| guitarzan | being the one I care about | 22:26 |
| mc_nair | scottda: quotas? bugs? no way! which one was this? | 22:27 |
| mc_nair | I'm just trying to keep them on my radar... | 22:27 |
| *** dims_ has quit IRC | 22:27 | |
| dulek | scottda: Okay! Sorry for the late response. I probably should go and catch my plane… | 22:27 |
| scottda | mc_nair: https://bugs.launchpad.net/cinder/+bug/1518513 | 22:27 |
| openstack | Launchpad bug 1518513 in Cinder "Quota update call fails in Liberty" [Medium,In progress] - Assigned to Dan Nguyen (daniel-a-nguyen) | 22:27 |
| xyang1 | mc_nair: https://review.openstack.org/#/c/266986 | 22:27 |
| *** jgregor has quit IRC | 22:27 | |
| scottda | dulek: np. Thanks | 22:27 |
| xyang1 | mc_nair: so lvm will use a different parameter | 22:28 |
| xyang1 | mc_nair: this patch you are looking at is still valid | 22:28 |
| xyang1 | mc_nair: lvm will have 1 as default instead of 20 | 22:28 |
| xyang1 | mc_nair: your followup patch should check eharney's new parameter | 22:30 |
| mc_nair | xyang1: Thanks for the link. To me it seems like that patch makes it *way* more important we land the fix for the max_over_subscription_ratio then and land https://review.openstack.org/#/c/275408/ | 22:31 |
| mc_nair | xyang1: because from what I'm getting from eharney's commit message, the intention for this is to make overprovisioning opt-in, but with the current bug in the capacity filter by setting this default to 1 we're still making it opt-in (because the bug causes this to allow over-provisioning) | 22:33 |
| mc_nair | basically right now if we wanted over-provisioning to be opt-in in LVM with that other bug we'd need to set it to default to 1.0000001 | 22:34 |
| xyang1 | mc_nair: ? | 22:35 |
| xyang1 | mc_nair: what other bug | 22:35 |
| xyang1 | mc_nair: what we discussed earlier should not be affected | 22:35 |
| mc_nair | xyang: https://bugs.launchpad.net/cinder/+bug/1537162 | 22:35 |
| openstack | Launchpad bug 1537162 in Cinder "accounting bug for lvm with thin provisioning and max_over_subscription_ratio=1" [Undecided,In progress] - Assigned to Michael Rowden (mrrowden) | 22:35 |
| mc_nair | xyang1: what we said before won't be affected, but it will affect what eharney's change does | 22:36 |
| *** alonma has joined #openstack-cinder | 22:36 | |
| mc_nair | lemme step back for a minute, it's possibly between quotas and this my brain has just melted | 22:36 |
| xyang1 | mc_nair: I don't see a conflict | 22:37 |
| *** amoturi has left #openstack-cinder | 22:38 | |
| mc_nair | xyang1: here I'm taking 5, then I'll try to articulate my concern :) | 22:38 |
| xyang1 | mc_nair: if thin and >=1, use it | 22:39 |
| xyang1 | mc_nair: if thin but invalid ratio, set it to 1 | 22:39 |
| mc_nair | xyang1: with current bug it's "if thin and > 1, use it" | 22:40 |
| mc_nair | https://github.com/openstack/cinder/blob/master/cinder/scheduler/filters/capacity_filter.py#L89 | 22:40 |
| xyang1 | i know, I am proposing a change | 22:41 |
| *** alonma has quit IRC | 22:41 | |
| mc_nair | xyang1: to what? https://review.openstack.org/#/c/275408/ already does the change we need I think | 22:42 |
| xyang1 | invalid ratio is not handled | 22:42 |
| xyang1 | still not affected by eharney's change | 22:42 |
| mc_nair | xyang1: ok - it will be good to have the invalid ratio change | 22:43 |
| *** alonma has joined #openstack-cinder | 22:43 | |
| mc_nair | xyang1: but my point is that without https://review.openstack.org/#/c/275408/, I think eharney's change doesn't do what we expect | 22:44 |
| xyang1 | mc_nair: he just changed default from 20 to 1 fir lvm | 22:45 |
| mc_nair | xyang1: because without that fix ^, max_subscription_ratio == 1 incorrectly allows for overprovisioning | 22:45 |
| xyang1 | that was agreed at the summit | 22:45 |
| mc_nair | xyang1: what was agreed at the summit, changing from 20 to 1? | 22:45 |
| xyang1 | mc_nair: for lvm | 22:46 |
| *** laughterwym has joined #openstack-cinder | 22:46 | |
| eharney | well the whole intent of changing it to 1 is to not overprovision | 22:46 |
| mc_nair | xyang1: perfect, I'm not against that change at all. The reason for that change is to disable overprovisioning by default with LVM, correct? | 22:46 |
| mc_nair | eharney: cool | 22:46 |
| *** alonma has quit IRC | 22:47 | |
| mc_nair | and that will work well once https://review.openstack.org/#/c/275408/ is in, but before that we would need to set to 1.00001 (which I'm not advocating) to default to not overprovision | 22:47 |
| eharney | and while i'm a little fuzzy on some of the details here, i suspect the folks saying that we need to fix the calculation still are correct | 22:48 |
| mc_nair | at least that's my understanding based on the code and https://bugs.launchpad.net/cinder/+bug/1537162 | 22:48 |
| openstack | Launchpad bug 1537162 in Cinder "accounting bug for lvm with thin provisioning and max_over_subscription_ratio=1" [Undecided,In progress] - Assigned to Michael Rowden (mrrowden) | 22:48 |
| *** krotscheck1 has quit IRC | 22:49 | |
| eharney | if the driver is doing thin provisioning, even when limited to a ratio of 1, the calculations need to understand to be correct, AFAICT | 22:49 |
| eharney | need to understand that* | 22:49 |
| *** alonma has joined #openstack-cinder | 22:49 | |
| mc_nair | eharney: yea - I don't think it does anything to your patch. Was just pointing out that until the fix for ^ lands, I don't think if turns off overprovisioning by default. Think it may allow for no max_ratio on overprovisioning | 22:50 |
| mc_nair | *it turns off | 22:50 |
| *** laughterwym has quit IRC | 22:50 | |
| eharney | which is not desirable behavior for any driver | 22:51 |
| *** mylu has quit IRC | 22:53 | |
| *** alonma has quit IRC | 22:54 | |
| *** mylu has joined #openstack-cinder | 22:54 | |
| mc_nair | agreed. So basically in short - we want the fix for https://review.openstack.org/#/c/275408/ to land because it helps all drivers. We probably want that fix to land before https://review.openstack.org/#/c/266986/4, because otherwise that by default would expose the max_ratio == 1 bug | 22:55 |
| mc_nair | eharney, xyang1: ^ does that make sense to you? | 22:55 |
| eharney | mc_nair: yeah, except for the part where we already merged the LVM change | 22:55 |
| mc_nair | eharney: haha ok then, new plan :) | 22:56 |
| mc_nair | I can't read | 22:56 |
| xyang1 | mc_nair: yes, we only need to worry about 275408 now | 22:56 |
| mc_nair | haha ok, cool. I missed that part of it, sorry about that | 22:57 |
| mc_nair | that was why I was talking in circles :) | 22:57 |
| mc_nair | well at least one of the reasons | 22:57 |
| *** rlrossit_ has quit IRC | 22:58 | |
| *** jwcroppe has quit IRC | 22:59 | |
| eharney | i have to run for today but i'll look at this again tomorrow | 22:59 |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 23:01 | |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/275529 | 23:02 |
| *** baojg has joined #openstack-cinder | 23:02 | |
| openstackgerrit | Merged openstack/cinder: EMC VMAX - Fix for last volume in VMAX3 storage group https://review.openstack.org/244331 | 23:05 |
| *** erlon has quit IRC | 23:06 | |
| *** tpsilva has quit IRC | 23:06 | |
| *** baojg has quit IRC | 23:07 | |
| *** xyang1 has quit IRC | 23:08 | |
| *** eharney has quit IRC | 23:09 | |
| openstackgerrit | Anthony Lee proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation https://review.openstack.org/268405 | 23:09 |
| jgriffith | patrickeast: sorry... lost my network screwing around with new VPN | 23:18 |
| *** arch-nemesis has quit IRC | 23:18 | |
| jgriffith | patrickeast: then have been busy mourning the loss of my beloved google for business :( | 23:18 |
| * jgriffith wants to jump out a window | 23:18 | |
| jgriffith | patrickeast: that's a tricky problem... because I'm not sure how we "guarantee" the message is sent to all "faked" backends | 23:19 |
| *** mylu has quit IRC | 23:21 | |
| jgriffith | patrickeast: TBF though, that's not unique to our proposal here | 23:21 |
| *** gouthamr has joined #openstack-cinder | 23:21 | |
| jgriffith | patrickeast: only thing I came up with is a periodic when things are "failed over" that continuously sends an update to the driver... but that's race prone; still may not work and is a whole can of worms | 23:22 |
| *** gouthamr_ has joined #openstack-cinder | 23:23 | |
| *** e0ne has joined #openstack-cinder | 23:24 | |
| *** mylu has joined #openstack-cinder | 23:25 | |
| patrickeast | jgriffith: yea i haven't figure out a better solution either | 23:26 |
| *** gouthamr has quit IRC | 23:26 | |
| hemna | trying to wrap my head around cheesecake | 23:26 |
| *** mylu has quit IRC | 23:26 | |
| patrickeast | jgriffith: i'm almost leaning towards a 'if it breaks just to restart your other c-vol services' documentation punt | 23:26 |
| patrickeast | s/just to/just go/ | 23:26 |
| jgriffith | patrickeast: yeah, it's an interesting case | 23:27 |
| patrickeast | hemna: sounds messy | 23:27 |
| hemna | yah it is | 23:27 |
| jgriffith | LOL | 23:27 |
| hemna | I'm just trying to figure how this works for us at all | 23:27 |
| hemna | :( | 23:27 |
| jgriffith | hemna: what are you having trouble with? | 23:27 |
| jgriffith | hemna: it's actually super easy compared to volume-level rep | 23:27 |
| hemna | heh | 23:27 |
| hemna | yah, it's supposed to be | 23:28 |
| *** geguileo has quit IRC | 23:28 | |
| hemna | and I see the API | 23:28 |
| hemna | it looks easier | 23:28 |
| jgriffith | hemna: LOL.. well, can I help? | 23:28 |
| hemna | I'm trying to figure out what to do on our backend for enable_replication | 23:28 |
| jgriffith | hemna: Ahh... ok, so | 23:28 |
| hemna | I'm not sure how to find which volumes to turn on at that point on our array. | 23:28 |
| jgriffith | hemna: there's only one real case for that IMHO | 23:28 |
| hemna | especially, since our array can have multiple openstack's pointing to it. :( | 23:28 |
| jgriffith | hemna: your backend should setup replication based on cinder.conf at init time | 23:29 |
| jgriffith | hemna: BUT, disable-rep gives an opportunity for maintenance or whatever if needed | 23:29 |
| patrickeast | jgriffith: oh speaking of the api, the one right now in the WIP is still on the volume manager, does that mean its still like a /v2/volumes/<uuid>/action kind of thing? | 23:29 |
| jgriffith | hemna: enable_replication just says... "turn that shit back on" | 23:29 |
| jgriffith | patrickeast: yeah, I'm hoping to just keep the old methods/api's where possible and modify their parameters | 23:30 |
| jgriffith | patrickeast: I don't see any value in yet anoter set of things | 23:30 |
| jgriffith | Ohhhh | 23:30 |
| jgriffith | patrickeast: No! | 23:30 |
| patrickeast | jgriffith: mm so the api still needs to be called on some volume? | 23:30 |
| jgriffith | sorry | 23:30 |
| jgriffith | patrickeast: so... that's all "host/c-vol" based now. | 23:30 |
| jgriffith | patrickeast: it all gets sucked in under services api | 23:30 |
| jgriffith | hemna: does that help at all? | 23:30 |
| hemna | chewing.... | 23:30 |
| hemna | it may.. | 23:31 |
| *** dustins has joined #openstack-cinder | 23:31 | |
| patrickeast | jgriffith: yep yep, ok cool | 23:31 |
| hemna | I'm also trying to unravel what we currently have | 23:31 |
| jgriffith | patrickeast: pheww... sorry | 23:31 |
| patrickeast | hemna: can't you just do what you had before on the per-volume basis, but for alll the volumes? | 23:31 |
| jgriffith | frik.. I have to go | 23:31 |
| hemna | kinda | 23:31 |
| patrickeast | jgriffith: oh i see part of the confusion, i meant the volume controller, not the manager | 23:32 |
| patrickeast | the manager ones all are good the way they are in the WIP | 23:32 |
| jgriffith | patrickeast: yeah, I figured that out :) | 23:32 |
| jgriffith | patrickeast: so yeah... I would move the API calls to contrib/service | 23:32 |
| patrickeast | jgriffith: gotcha | 23:33 |
| *** e0ne has quit IRC | 23:33 | |
| *** edtubill has quit IRC | 23:33 | |
| hemna | jgriffith, ok I think we have a way to make this work | 23:35 |
| jgriffith | hemna: honestly I was just looking at your code, it should be more code-removal than anything else | 23:36 |
| hemna | we do so much work/effort to setup each volume replication | 23:36 |
| jgriffith | hemna: you already have all the mechanisms to do the *things*, we just remove some of the crazy options we had | 23:36 |
| hemna | I think we can just force the admin to create the replication group name before hand, stuff it into the volume type and we are done. | 23:37 |
| jgriffith | anyway.. don't hesitate to ping me on questions. We can all figure this out together. like patrickeast 's case | 23:37 |
| hemna | instead of us dynamically creating the group on every volume. | 23:37 |
| jgriffith | hemna: hmmm | 23:37 |
| jgriffith | hemna: not sure I follow that | 23:37 |
| jgriffith | hemna: so why not just have it in the cinder.conf... then the driver sets up the group in init | 23:37 |
| jgriffith | hemna: and if a type=replicated comes in it goes to said rep-group? | 23:37 |
| hemna | yah either way | 23:38 |
| patrickeast | thats what ours will do | 23:38 |
| jgriffith | hemna: well... don't take this the wrong way... but one way is "wrong" and the other "is not" | 23:38 |
| jgriffith | :) | 23:38 |
| hemna | :P | 23:38 |
| jgriffith | hemna: better to do it the "right" way :) | 23:38 |
| jgriffith | hemna: otherwise we might as well just stick with what we have | 23:38 |
| hemna | ok | 23:39 |
| hemna | for some reason we were spinning on the need for getting the list of volumes from the manager at enable time | 23:39 |
| hemna | meh | 23:39 |
| *** geguileo has joined #openstack-cinder | 23:41 | |
| patrickeast | do we have a way to query the db for all the replicated volumes for that backend? may be pretty easy to pass em in to the failover/enable/disable commands | 23:41 |
| jgriffith | hemna: nope, not needed | 23:41 |
| patrickeast | if you guys end up needed them | 23:41 |
| jgriffith | patrickeast: yeah, they'll have the type associated with them | 23:41 |
| hemna | if we know the group name, we can find them and enable/disable | 23:42 |
| jgriffith | patrickeast: get-all-volume of type xxxx | 23:42 |
| hemna | I think we're good | 23:42 |
| jgriffith | hemna: hmm... you're scaring me | 23:42 |
| patrickeast | lol | 23:42 |
| hemna | hehe | 23:42 |
| patrickeast | jgriffith: yea that should work, although there could be multiple replicated types, right? | 23:42 |
| *** pratap has quit IRC | 23:43 | |
| jgriffith | patrickeast: yeah, but you could filter on backend-name etc | 23:43 |
| patrickeast | jgriffith: need like a join for type has replication and get all the volumes of that type on the backend | 23:43 |
| jgriffith | but.... | 23:43 |
| jgriffith | honestly the disable/enable thing is kinda... well... weird | 23:43 |
| jgriffith | In my case, I'm not planning to actually disable anything | 23:43 |
| jgriffith | I don't see any value in it | 23:43 |
| patrickeast | yea, i dont need it for my backend :D | 23:44 |
| jgriffith | if you want to stop replcating volumes, then they should be retyped | 23:44 |
| jgriffith | and then to start again... retype again | 23:44 |
| jgriffith | but if the admin wants/needs some magic tool... well, I guess that's "ok" | 23:44 |
| jgriffith | patrickeast: hemna maybe enable/disable doesn't even make sense? | 23:44 |
| *** mylu has joined #openstack-cinder | 23:44 | |
| patrickeast | i guess the only thing there is a user may not know why their volume got retyped, right? | 23:44 |
| patrickeast | jgriffith: dunno, i mean... for our simple use case it isn't really needed | 23:45 |
| hemna | if you lose a switch between the 2 arrays and want to pause it in the mean time ? | 23:45 |
| hemna | dunno, thinking outloud | 23:45 |
| patrickeast | jgriffith: but i can see where maybe they want to turn it off while the dr site is under maintenance or something | 23:45 |
| jgriffith | hemna: sure, but what's it going to matter? | 23:45 |
| patrickeast | even then i'm not sure it matters | 23:45 |
| jgriffith | hemna: I mean, it's not working anyway | 23:45 |
| hemna | I'm not sure what our array will do if the target takes a dump in the process | 23:45 |
| hemna | tbh | 23:45 |
| jgriffith | so it's just semantics/hoops to jump through | 23:46 |
| jgriffith | hemna: yeah, but my point is it's not going to matter | 23:46 |
| hemna | I'm not a replication expert hehe | 23:46 |
| jgriffith | hemna: I mean... say your switch or the remote target dumps | 23:46 |
| jgriffith | hemna: until the admin says "disable rep" it doesn't seem like it would make any difference... or offer anything that your driver couldn't figure out on it's own | 23:46 |
| jgriffith | I'm kinda leaning towards those calls being useless honestly | 23:47 |
| *** yhayashi has joined #openstack-cinder | 23:47 | |
| patrickeast | so... i could reasonably see something like they wanted to disable replication to one of the remote sites or something temporarily (in the multi-target scenario) | 23:47 |
| *** jamielennox|away is now known as jamielennox | 23:47 | |
| patrickeast | which could then make the driver not try and pick that one in failover | 23:47 |
| hemna | sure | 23:47 |
| patrickeast | even if it could reach it | 23:47 |
| patrickeast | buuut its still kind of a stretch imo | 23:47 |
| jgriffith | patrickeast: yeah, that's feasible. Although the impl I have doesn't allow that anyway :) It's all or nothing | 23:48 |
| *** mylu has quit IRC | 23:48 | |
| *** mylu has joined #openstack-cinder | 23:48 | |
| patrickeast | jgriffith: not for that it isn't, right? it only fails-over to one of the targets and we return which one | 23:48 |
| patrickeast | jgriffith: so it would be like a short-cut to temporarily disable a target instead of changing the cinder.conf/restarting c-vol | 23:48 |
| jgriffith | patrickeast: well, what I mean is that the enable/disable doesn't take any parameters | 23:48 |
| patrickeast | oooh yea | 23:49 |
| jgriffith | patrickeast: it just tells the "main" backend... "stop/start replicating" | 23:49 |
| jgriffith | that's all | 23:49 |
| jgriffith | all or nothing | 23:49 |
| hemna | so it's not multi target then ? | 23:49 |
| patrickeast | yea, got it | 23:49 |
| jgriffith | hemna: sure it is | 23:49 |
| *** edmondsw has quit IRC | 23:49 | |
| jgriffith | or can be | 23:49 |
| hemna | or it's all on for all targets | 23:49 |
| hemna | or all off for all targets | 23:49 |
| jgriffith | hemna: exactly | 23:49 |
| hemna | ah k | 23:49 |
| hemna | I suppose that's fine for now | 23:49 |
| jgriffith | hemna: replication is either enabled/possible or it's not | 23:49 |
| patrickeast | i guess in the future it could be expanded to be more specific if folks want that | 23:49 |
| hemna | simplify this shit | 23:49 |
| jgriffith | hemna: for now? | 23:50 |
| hemna | :) | 23:50 |
| *** alonma has joined #openstack-cinder | 23:50 | |
| hemna | patrickeast, yah that's what I'm thinking | 23:50 |
| jgriffith | hemna: honestly I think it's a wayy better answer than what we were trying to do | 23:50 |
| hemna | all or nothing | 23:50 |
| hemna | well it is simpler | 23:50 |
| hemna | and I think that's a good thing | 23:50 |
| patrickeast | hemna: +1 | 23:50 |
| jgriffith | and it makes sense in terms of use-cases IMHO | 23:50 |
| hemna | or it's impossible to make 80 drivers happy | 23:50 |
| patrickeast | jgriffith: so maybe back to your point earlier, would it make more sense to just not have enable/disable? | 23:51 |
| jgriffith | patrickeast: honestly I think it might | 23:51 |
| patrickeast | since we've just kind of said we dont have a good use case for it | 23:51 |
| jgriffith | patrickeast: It's sort of weird now that I think about it | 23:52 |
| jgriffith | patrickeast: I had thought some sort of maintenance mode... but an admin is going to have to be doing something on the backend for that anyway | 23:52 |
| jgriffith | so maybe it doesn't make sense right now | 23:52 |
| patrickeast | jgriffith: im on board to just pull it, not sure it really adds value with the way things would work now | 23:52 |
| jgriffith | patrickeast: I think I'll remove it... maybe ponder it a bit more on the drive home | 23:53 |
| jgriffith | but I don't see much value in it | 23:53 |
| patrickeast | later on if we do have more specific stuff and we wanted to make a more flexible maintenance mode sort of deal it would make more sense to add it back in | 23:53 |
| hemna | well, it doesn't harm anything | 23:53 |
| hemna | since it's a simple passthrough to the driver | 23:53 |
| jgriffith | hemna: yeah... I suppose but then we get into that problem of "what is expected" | 23:53 |
| hemna | kinda | 23:53 |
| jgriffith | hemna: if you put it in there then you would expect that the backend is "disabling replication" | 23:54 |
| hemna | it's up to the driver....:P | 23:54 |
| jgriffith | so then you kinda have to do that | 23:54 |
| jgriffith | hemna: oh hush :) | 23:54 |
| patrickeast | lol | 23:54 |
| hemna | :) | 23:54 |
| *** alonma has quit IRC | 23:54 | |
| *** cknight has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!