| *** diogogmt has quit IRC | 00:00 | |
| jgriffith | patrickeast: so the trick for me is that our names are special, we can't change them | 00:00 |
|---|---|---|
| *** kevincarr1991 has joined #openstack-cinder | 00:00 | |
| patrickeast | jgriffith: oh, that does make it harder to upgrade to a different naming scheme then | 00:02 |
| *** garthb has quit IRC | 00:04 | |
| *** kevincarr1991 has quit IRC | 00:04 | |
| jgriffith | yeah, breaks the world :( | 00:07 |
| aorourke_ | jgriffith, hey | 00:16 |
| *** tsekiyama has quit IRC | 00:17 | |
| *** setmason has joined #openstack-cinder | 00:17 | |
| *** pv_ has quit IRC | 00:18 | |
| *** tsekiyama has joined #openstack-cinder | 00:18 | |
| *** takedakn has joined #openstack-cinder | 00:21 | |
| *** ntpttr has joined #openstack-cinder | 00:21 | |
| *** ntpttr has quit IRC | 00:27 | |
| *** takedakn has quit IRC | 00:28 | |
| *** mtanino has quit IRC | 00:29 | |
| *** julim has joined #openstack-cinder | 00:30 | |
| *** tsekiyama has quit IRC | 00:36 | |
| *** julim has quit IRC | 00:41 | |
| *** dims_ has joined #openstack-cinder | 00:50 | |
| *** leeantho has quit IRC | 00:51 | |
| *** lcurtis_ has joined #openstack-cinder | 00:53 | |
| *** _cjones_ has quit IRC | 00:55 | |
| *** dims_ has quit IRC | 00:55 | |
| *** dims_ has joined #openstack-cinder | 00:58 | |
| *** Guest71307 has quit IRC | 01:02 | |
| *** kevincarr1991 has joined #openstack-cinder | 01:06 | |
| *** kevincarr1991 has quit IRC | 01:11 | |
| *** apoorvad_ has joined #openstack-cinder | 01:11 | |
| *** apoorvad has quit IRC | 01:14 | |
| *** gouthamr has quit IRC | 01:14 | |
| *** apoorvad_ has quit IRC | 01:15 | |
| *** akerr is now known as akerr_away | 01:17 | |
| *** haigang has joined #openstack-cinder | 01:22 | |
| *** akerr_away is now known as akerr | 01:22 | |
| *** haigang has quit IRC | 01:24 | |
| *** asselin_ has joined #openstack-cinder | 01:27 | |
| *** markvoelker has quit IRC | 01:28 | |
| *** dims_ has quit IRC | 01:30 | |
| *** asselin_ has quit IRC | 01:31 | |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** mriedem_away has quit IRC | 01:35 | |
| *** tsekiyama has joined #openstack-cinder | 01:37 | |
| *** fandi has quit IRC | 01:41 | |
| *** tsekiyama has quit IRC | 01:42 | |
| *** Yogi1 has joined #openstack-cinder | 01:48 | |
| *** stevemar_ has quit IRC | 01:59 | |
| *** stevemar_ has joined #openstack-cinder | 02:00 | |
| *** kevincarr1991 has joined #openstack-cinder | 02:00 | |
| *** kevincarr1991 has quit IRC | 02:05 | |
| *** Yogi1 has quit IRC | 02:05 | |
| jungleboyj | patrickeast: Ok, I see that you updated the commit. Thanks, I will take a look. | 02:06 |
| patrickeast | jungleboyj: thanks! | 02:09 |
| *** smoriya has joined #openstack-cinder | 02:12 | |
| *** stevemar_ has quit IRC | 02:41 | |
| *** stevemar_ has joined #openstack-cinder | 02:42 | |
| *** stevemar_ has quit IRC | 02:43 | |
| *** stevemar_ has joined #openstack-cinder | 02:43 | |
| *** dave-mccowan has quit IRC | 02:44 | |
| *** jamielennox has quit IRC | 02:51 | |
| *** martyturner has joined #openstack-cinder | 02:59 | |
| *** markvoelker_ has joined #openstack-cinder | 03:05 | |
| *** kevincarr1991 has joined #openstack-cinder | 03:06 | |
| *** fandi has joined #openstack-cinder | 03:10 | |
| *** kevincarr1991 has quit IRC | 03:10 | |
| *** jamielennox has joined #openstack-cinder | 03:13 | |
| *** harlowja_at_home has joined #openstack-cinder | 03:15 | |
| *** haomaiwang has joined #openstack-cinder | 03:16 | |
| *** stevemar_ has quit IRC | 03:21 | |
| *** markvoelker has joined #openstack-cinder | 03:22 | |
| *** stevemar_ has joined #openstack-cinder | 03:22 | |
| *** akerr is now known as akerr_away | 03:23 | |
| *** markvoelker_ has quit IRC | 03:25 | |
| *** markvoelker_ has joined #openstack-cinder | 03:26 | |
| *** markvoelker has quit IRC | 03:28 | |
| *** haomaiwang has quit IRC | 03:34 | |
| *** martyturner has quit IRC | 03:38 | |
| *** haomaiwang has joined #openstack-cinder | 03:42 | |
| *** DericHorn-HP has joined #openstack-cinder | 03:47 | |
| *** links has joined #openstack-cinder | 03:48 | |
| *** lcurtis_ has quit IRC | 03:50 | |
| *** DericHorn-HP has quit IRC | 03:51 | |
| *** pradipta has joined #openstack-cinder | 03:56 | |
| *** DericHorn-HP has joined #openstack-cinder | 03:57 | |
| *** kevincarr1991 has joined #openstack-cinder | 04:00 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** kevincarr1991 has quit IRC | 04:04 | |
| *** harlowja_at_home has quit IRC | 04:15 | |
| *** daneyon has joined #openstack-cinder | 04:35 | |
| *** daneyon_ has joined #openstack-cinder | 04:35 | |
| *** daneyon has quit IRC | 04:39 | |
| *** dims_ has joined #openstack-cinder | 04:46 | |
| *** DericHorn-HP has quit IRC | 04:54 | |
| *** DericHorn-HP has joined #openstack-cinder | 04:56 | |
| *** haomaiwang has quit IRC | 04:58 | |
| openstackgerrit | watanabe isao proposed openstack/cinder: Don't merge https://review.openstack.org/230880 | 05:01 |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** subscope has joined #openstack-cinder | 05:01 | |
| *** _fortis has quit IRC | 05:02 | |
| *** cbader_ has joined #openstack-cinder | 05:02 | |
| *** MIDENN_ has joined #openstack-cinder | 05:02 | |
| *** morgan has quit IRC | 05:02 | |
| *** cebruns has quit IRC | 05:03 | |
| *** mdenny has quit IRC | 05:03 | |
| *** ByPasS has quit IRC | 05:03 | |
| *** cbader has quit IRC | 05:03 | |
| *** mfisch has quit IRC | 05:03 | |
| *** cebruns has joined #openstack-cinder | 05:04 | |
| *** tristanC has quit IRC | 05:04 | |
| *** tristanC has joined #openstack-cinder | 05:05 | |
| *** mfisch has joined #openstack-cinder | 05:06 | |
| *** mfisch is now known as Guest83454 | 05:06 | |
| *** morgan has joined #openstack-cinder | 05:06 | |
| *** ByPasS has joined #openstack-cinder | 05:08 | |
| *** setmason_ has joined #openstack-cinder | 05:08 | |
| *** dsariel_ has quit IRC | 05:10 | |
| *** setmason has quit IRC | 05:11 | |
| *** setmason_ is now known as setmason | 05:11 | |
| *** _fortis has joined #openstack-cinder | 05:13 | |
| *** BharatK has joined #openstack-cinder | 05:21 | |
| *** daneyon_ has quit IRC | 05:23 | |
| *** daneyon has joined #openstack-cinder | 05:25 | |
| *** daneyon_ has joined #openstack-cinder | 05:28 | |
| *** daneyon has quit IRC | 05:30 | |
| *** pradipta has quit IRC | 05:31 | |
| *** setmason has quit IRC | 05:37 | |
| *** setmason has joined #openstack-cinder | 05:37 | |
| *** lpetrut has joined #openstack-cinder | 05:37 | |
| *** bkopilov has joined #openstack-cinder | 05:38 | |
| *** setmason_ has joined #openstack-cinder | 05:43 | |
| *** setmason has quit IRC | 05:46 | |
| *** setmason_ is now known as setmason | 05:46 | |
| *** DericHorn-HP has quit IRC | 05:47 | |
| *** edtubill has joined #openstack-cinder | 05:51 | |
| *** setmason has quit IRC | 05:53 | |
| *** setmason has joined #openstack-cinder | 05:54 | |
| *** kevincarr1991 has joined #openstack-cinder | 06:00 | |
| *** dsariel_ has joined #openstack-cinder | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| openstackgerrit | yogeshprasad proposed openstack/cinder: Retype support for CloudByte iSCSI cinder driver https://review.openstack.org/218645 | 06:02 |
| *** kevincarr1991 has quit IRC | 06:05 | |
| *** lpetrut has quit IRC | 06:07 | |
| *** e0ne has joined #openstack-cinder | 06:22 | |
| *** markvoelker_ has quit IRC | 06:24 | |
| *** dsariel_ has quit IRC | 06:27 | |
| *** e0ne has quit IRC | 06:30 | |
| *** haomaiwang has quit IRC | 06:31 | |
| *** uberjay has quit IRC | 06:33 | |
| *** markvoelker has joined #openstack-cinder | 06:35 | |
| *** uberjay has joined #openstack-cinder | 06:36 | |
| *** stevemar_ has quit IRC | 06:38 | |
| *** vlaza has joined #openstack-cinder | 06:39 | |
| *** markvoelker has quit IRC | 06:40 | |
| *** dsariel_ has joined #openstack-cinder | 06:40 | |
| *** nkrinner has joined #openstack-cinder | 06:41 | |
| *** jamielennox is now known as jamielennox|away | 06:41 | |
| *** yrabl has joined #openstack-cinder | 06:42 | |
| *** markvoelker has joined #openstack-cinder | 06:45 | |
| *** markvoelker has quit IRC | 06:50 | |
| *** jwang_ has quit IRC | 06:54 | |
| *** jwang_ has joined #openstack-cinder | 06:54 | |
| *** anshul has joined #openstack-cinder | 06:56 | |
| *** anshul has quit IRC | 06:57 | |
| *** ekarlso has quit IRC | 06:58 | |
| *** bkopilov has quit IRC | 06:58 | |
| *** anshul has joined #openstack-cinder | 06:58 | |
| *** markvoelker has joined #openstack-cinder | 07:00 | |
| *** markvoelker has quit IRC | 07:04 | |
| *** alexschm has joined #openstack-cinder | 07:05 | |
| *** markvoelker has joined #openstack-cinder | 07:14 | |
| *** ollie2 has joined #openstack-cinder | 07:17 | |
| *** markvoelker has quit IRC | 07:19 | |
| *** ekarlso has joined #openstack-cinder | 07:25 | |
| *** rushiagr_away is now known as rushiagr | 07:27 | |
| *** markvoelker has joined #openstack-cinder | 07:29 | |
| *** markvoelker has quit IRC | 07:33 | |
| *** lpetrut has joined #openstack-cinder | 07:37 | |
| *** edtubill has quit IRC | 07:37 | |
| *** lpetrut has quit IRC | 07:41 | |
| *** markvoelker has joined #openstack-cinder | 07:43 | |
| *** Anticimex has quit IRC | 07:46 | |
| *** setmason has quit IRC | 07:46 | |
| *** Anticimex has joined #openstack-cinder | 07:47 | |
| *** markvoelker has quit IRC | 07:48 | |
| *** lpetrut has joined #openstack-cinder | 07:50 | |
| *** xek_ is now known as xek | 07:53 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (again) https://review.openstack.org/201500 | 07:54 |
| *** markvoelker has joined #openstack-cinder | 07:58 | |
| *** kevincarr1991 has joined #openstack-cinder | 08:01 | |
| *** sgotliv has joined #openstack-cinder | 08:02 | |
| *** markvoelker has quit IRC | 08:02 | |
| *** kevincarr1991 has quit IRC | 08:06 | |
| *** markvoelker has joined #openstack-cinder | 08:13 | |
| *** kmartin has quit IRC | 08:13 | |
| *** rushiagr is now known as rushiagr_away | 08:13 | |
| *** kmartin has joined #openstack-cinder | 08:13 | |
| *** markvoelker has quit IRC | 08:18 | |
| *** smoriya has quit IRC | 08:24 | |
| *** jistr has joined #openstack-cinder | 08:25 | |
| *** bkopilov has joined #openstack-cinder | 08:27 | |
| *** markvoelker has joined #openstack-cinder | 08:27 | |
| *** patrickeast has quit IRC | 08:30 | |
| *** markvoelker has quit IRC | 08:32 | |
| *** patrickeast has joined #openstack-cinder | 08:32 | |
| *** rushiagr_away is now known as rushiagr | 08:38 | |
| *** markvoelker has joined #openstack-cinder | 08:42 | |
| *** markvoelker has quit IRC | 08:47 | |
| *** aix has quit IRC | 08:48 | |
| *** markvoelker has joined #openstack-cinder | 08:56 | |
| *** bkopilov has quit IRC | 08:58 | |
| *** markvoelker has quit IRC | 09:01 | |
| *** bkopilov has joined #openstack-cinder | 09:04 | |
| *** aix has joined #openstack-cinder | 09:06 | |
| *** markvoelker has joined #openstack-cinder | 09:06 | |
| *** markvoelker has quit IRC | 09:10 | |
| *** yrabl has quit IRC | 09:12 | |
| *** markvoelker has joined #openstack-cinder | 09:13 | |
| *** markvoelker has quit IRC | 09:18 | |
| *** yrabl has joined #openstack-cinder | 09:22 | |
| *** markvoelker has joined #openstack-cinder | 09:22 | |
| *** markvoelker has quit IRC | 09:26 | |
| *** markvoelker has joined #openstack-cinder | 09:32 | |
| *** akerr_away has quit IRC | 09:33 | |
| *** markvoelker has quit IRC | 09:39 | |
| *** akerr has joined #openstack-cinder | 09:41 | |
| *** deepakcs has joined #openstack-cinder | 09:42 | |
| *** aix has quit IRC | 09:45 | |
| *** markvoelker has joined #openstack-cinder | 09:46 | |
| *** markvoelker has quit IRC | 09:50 | |
| *** akerr has quit IRC | 09:52 | |
| *** haypo has joined #openstack-cinder | 09:54 | |
| *** akerr has joined #openstack-cinder | 09:55 | |
| *** IanGovett has joined #openstack-cinder | 09:57 | |
| *** aix has joined #openstack-cinder | 09:57 | |
| openstackgerrit | oliver-leahy-l proposed openstack/cinder: encryption_api_url requires a version https://review.openstack.org/230031 | 09:59 |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (again) https://review.openstack.org/201500 | 10:01 |
| *** markvoelker has joined #openstack-cinder | 10:01 | |
| *** kevincarr1991 has joined #openstack-cinder | 10:02 | |
| *** e0ne has joined #openstack-cinder | 10:03 | |
| *** markvoelker has quit IRC | 10:05 | |
| *** kevincarr1991 has quit IRC | 10:07 | |
| *** akerr has quit IRC | 10:09 | |
| *** markvoelker has joined #openstack-cinder | 10:11 | |
| *** jamielennox|away is now known as jamielennox | 10:14 | |
| *** markvoelker has quit IRC | 10:16 | |
| *** markvoelker has joined #openstack-cinder | 10:17 | |
| *** markvoelker has quit IRC | 10:22 | |
| *** dave-mccowan has joined #openstack-cinder | 10:24 | |
| *** markvoelker has joined #openstack-cinder | 10:26 | |
| *** ociuhandu has quit IRC | 10:32 | |
| *** markvoelker has quit IRC | 10:34 | |
| *** markvoelker has joined #openstack-cinder | 10:55 | |
| *** ociuhandu has joined #openstack-cinder | 10:57 | |
| *** markvoelker has quit IRC | 11:01 | |
| *** ociuhandu has quit IRC | 11:02 | |
| *** ociuhandu_ has joined #openstack-cinder | 11:02 | |
| *** daneyon_ has quit IRC | 11:03 | |
| *** markvoelker has joined #openstack-cinder | 11:05 | |
| *** BharatK has quit IRC | 11:06 | |
| *** akerr has joined #openstack-cinder | 11:10 | |
| *** markvoelker has quit IRC | 11:10 | |
| *** BharatK has joined #openstack-cinder | 11:17 | |
| *** lpetrut has quit IRC | 11:18 | |
| *** deepakcs has quit IRC | 11:19 | |
| *** markvoelker has joined #openstack-cinder | 11:19 | |
| *** markvoelker has quit IRC | 11:24 | |
| *** adrianofr has joined #openstack-cinder | 11:29 | |
| *** kevincarr1991 has joined #openstack-cinder | 11:30 | |
| *** timcl has joined #openstack-cinder | 11:33 | |
| *** markvoelker has joined #openstack-cinder | 11:34 | |
| *** kevincarr1991 has quit IRC | 11:34 | |
| *** markvoelker has quit IRC | 11:38 | |
| *** takedakn has joined #openstack-cinder | 11:41 | |
| openstackgerrit | oliver-leahy-l proposed openstack/cinder: encryption_api_url requires a version https://review.openstack.org/230031 | 11:45 |
| *** tpsilva has joined #openstack-cinder | 11:46 | |
| *** ollie2 has left #openstack-cinder | 11:48 | |
| *** markvoelker has joined #openstack-cinder | 11:48 | |
| *** ociuhandu_ has quit IRC | 11:53 | |
| *** markvoelker has quit IRC | 11:53 | |
| *** ociuhandu has joined #openstack-cinder | 11:54 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix Status-Line in HTTP response https://review.openstack.org/231438 | 11:54 |
| *** bkopilov has quit IRC | 11:59 | |
| *** markvoelker has joined #openstack-cinder | 12:03 | |
| *** akerr has quit IRC | 12:07 | |
| *** markvoelker has quit IRC | 12:07 | |
| *** markvoelker has joined #openstack-cinder | 12:14 | |
| *** gouthamr has joined #openstack-cinder | 12:18 | |
| *** markvoelker has quit IRC | 12:19 | |
| *** rushiagr is now known as rushiagr_away | 12:21 | |
| adrianofr | hemnafk: ping | 12:21 |
| *** geektoor has joined #openstack-cinder | 12:22 | |
| *** haomaiwang has joined #openstack-cinder | 12:25 | |
| *** lpetrut has joined #openstack-cinder | 12:28 | |
| *** markvoelker has joined #openstack-cinder | 12:29 | |
| *** markvoelker has quit IRC | 12:36 | |
| *** crose has joined #openstack-cinder | 12:37 | |
| *** diablo_rojo has joined #openstack-cinder | 12:37 | |
| *** tpsilva has quit IRC | 12:38 | |
| *** ekarlso has quit IRC | 12:38 | |
| *** tpsilva has joined #openstack-cinder | 12:38 | |
| *** takedakn has quit IRC | 12:39 | |
| *** markvoelker has joined #openstack-cinder | 12:44 | |
| *** edmondsw has joined #openstack-cinder | 12:45 | |
| *** akerr has joined #openstack-cinder | 12:46 | |
| *** merooney has joined #openstack-cinder | 12:48 | |
| *** markvoelker has quit IRC | 12:48 | |
| *** merooney has quit IRC | 12:49 | |
| *** jaypipes has joined #openstack-cinder | 12:50 | |
| *** julim has joined #openstack-cinder | 12:50 | |
| *** jungleboyj has quit IRC | 12:51 | |
| *** pckizer has quit IRC | 12:53 | |
| *** pckizer has joined #openstack-cinder | 12:53 | |
| *** timcl has quit IRC | 12:54 | |
| *** ekarlso has joined #openstack-cinder | 12:56 | |
| *** merooney has joined #openstack-cinder | 12:56 | |
| *** porrua has joined #openstack-cinder | 12:56 | |
| *** markvoelker has joined #openstack-cinder | 12:58 | |
| *** sgotliv has quit IRC | 12:58 | |
| *** markvoelker has quit IRC | 12:59 | |
| *** markvoelker has joined #openstack-cinder | 12:59 | |
| *** pckizer has quit IRC | 12:59 | |
| *** pckizer has joined #openstack-cinder | 12:59 | |
| *** markvoelker_ has joined #openstack-cinder | 13:01 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-cinder | 13:01 | |
| *** bswartz has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder-specs: Moving assisted-snapshot-improvements spec to mitaka https://review.openstack.org/231463 | 13:02 |
| *** markvoelker has quit IRC | 13:03 | |
| *** markvoelker has joined #openstack-cinder | 13:06 | |
| *** Yogi1 has joined #openstack-cinder | 13:08 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix Status-Line in HTTP response https://review.openstack.org/231438 | 13:08 |
| *** markvoelker_ has quit IRC | 13:10 | |
| *** setmason has joined #openstack-cinder | 13:11 | |
| *** nkrinner has quit IRC | 13:11 | |
| *** EinstCrazy has joined #openstack-cinder | 13:11 | |
| *** changbl has quit IRC | 13:13 | |
| *** nkrinner has joined #openstack-cinder | 13:14 | |
| *** EinstCrazy has quit IRC | 13:14 | |
| *** raildo-afk is now known as raildo | 13:15 | |
| *** sgotliv has joined #openstack-cinder | 13:15 | |
| *** bill_az has joined #openstack-cinder | 13:15 | |
| *** ronis has joined #openstack-cinder | 13:15 | |
| *** markvoelker has quit IRC | 13:17 | |
| *** markvoelker has joined #openstack-cinder | 13:17 | |
| *** dave-mccowan has quit IRC | 13:19 | |
| *** Guest92657 has joined #openstack-cinder | 13:22 | |
| *** Guest92657 is now known as grumpycatt | 13:23 | |
| *** thangp has joined #openstack-cinder | 13:24 | |
| *** mriedem has joined #openstack-cinder | 13:24 | |
| *** haomaiwang has quit IRC | 13:27 | |
| thangp | geguileo: ping | 13:27 |
| *** haomaiwang has joined #openstack-cinder | 13:27 | |
| thangp | geguileo: re - https://review.openstack.org/195027 | 13:28 |
| *** setmason_ has joined #openstack-cinder | 13:28 | |
| thangp | geguileo: I put up a reply... I do not agree that consistencygroup should be an extra field | 13:28 |
| *** jgregor has joined #openstack-cinder | 13:30 | |
| *** crose has quit IRC | 13:30 | |
| *** kevincarr1991 has joined #openstack-cinder | 13:31 | |
| *** setmason has quit IRC | 13:31 | |
| *** setmason_ is now known as setmason | 13:31 | |
| *** martyturner has joined #openstack-cinder | 13:32 | |
| *** bill_az_ has joined #openstack-cinder | 13:33 | |
| *** dustins has joined #openstack-cinder | 13:34 | |
| *** dave-mccowan has joined #openstack-cinder | 13:34 | |
| *** kevincarr1991 has quit IRC | 13:35 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Hacking check for opt name registration https://review.openstack.org/223375 | 13:36 |
| geguileo | thangp: pong | 13:39 |
| *** vgridnev has joined #openstack-cinder | 13:39 | |
| thangp | geguileo: hey | 13:40 |
| thangp | geguileo: i put up a reply to 195027 | 13:40 |
| geguileo | Reading it | 13:41 |
| thangp | geguileo: we have been using nova as an example of how to do things (good or bad), so it's the same for the patch | 13:41 |
| thangp | geguileo: if it's not something correct, we can ask dan | 13:42 |
| *** ronis has quit IRC | 13:42 | |
| *** ronis has joined #openstack-cinder | 13:43 | |
| geguileo | thangp: I'm looking at the instance nova object | 13:46 |
| thangp | ok | 13:46 |
| geguileo | thangp: And it's different | 13:46 |
| geguileo | thangp: The way they have their ORM objects and ours are quite different | 13:46 |
| thangp | geguileo: how so? | 13:47 |
| *** haomaiwang has quit IRC | 13:47 | |
| thangp | geguileo: maybe another example is https://github.com/openstack/nova/blob/master/nova/objects/floating_ip.py#L49 | 13:49 |
| *** mriedem has quit IRC | 13:49 | |
| thangp | geguileo: and https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L916 | 13:49 |
| *** haomaiwa_ has joined #openstack-cinder | 13:50 | |
| dulek | geguileo, thangp: As I've noted they're sometimes skipping relationships in objects and I've found at least one occurrence where they used object properties to make DB calls. | 13:51 |
| geguileo | thangp: Ok, and if you look at fixed_ip they don't allow you changing fixed_ip_id | 13:51 |
| thangp | dulek: true | 13:52 |
| dulek | geguileo, thangp: No obj_extra_fields though. Also I believe they're actually using same sqlalchemy's relationship to build relationships. | 13:52 |
| thangp | geguileo: true, and we can add it in | 13:52 |
| thangp | geguileo: my only point is that you can use fields for properties that are not part of the orm model | 13:52 |
| *** diogogmt has joined #openstack-cinder | 13:53 | |
| geguileo | thangp: True | 13:53 |
| dulek | geguileo: I agree on the non-mutable id. Good catch. | 13:54 |
| geguileo | thangp: dulek Mmmm, you guys are probably right on the consistencygroup issue | 13:55 |
| thangp | geguileo: cool, so we just need to put a catch in the save to not update consistencygroup if its changed, and some other ones you identified elsewhere | 13:56 |
| geguileo | thangp: And you are missing the snapshots field as well | 13:56 |
| geguileo | I'll update my review | 13:57 |
| thangp | geguileo: why snapshot? | 13:57 |
| geguileo | snapshots | 13:57 |
| geguileo | Plural, it's the backref | 13:57 |
| thangp | geguileo: oh | 13:58 |
| geguileo | I just catched on that now XD | 13:58 |
| dulek | geguileo: The more we get into it the worse it becomes. ;) | 13:58 |
| thangp | geguileo: to put cgsnapshot in snapshot object? | 13:58 |
| geguileo | thangp: No, there is a ref in snapshot to cgsnapshot that is called snapshots in cgsnapshot | 13:59 |
| dulek | geguileo: I guess you've had intuition that there's something wrong. ;) | 13:59 |
| geguileo | dulek: Lets call it that XD | 13:59 |
| dulek | thangp: "backref="cgsnapshots"," | 13:59 |
| thangp | geguileo: dulek: ah ok | 14:00 |
| dulek | thangp, geguileo: There probably are more mistakes like that - I'll take a look. | 14:00 |
| geguileo | dulek: Actually backref="snapshots", | 14:00 |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| *** breitz has joined #openstack-cinder | 14:01 | |
| dulek | geguileo: Ah, right. I think the one I've mentioned may also be missing. | 14:02 |
| geguileo | dulek: Probably, that would be in snapshot object | 14:02 |
| dulek | geguileo: ConsistencyGroup actually and it's missing there (that's merged). | 14:03 |
| thangp | geguileo: so we need to add cgsnapshot field in snapshot object, correct? | 14:04 |
| geguileo | thangp: I'd have to look at that | 14:04 |
| *** mriedem has joined #openstack-cinder | 14:04 | |
| thangp | geguileo: and add a list of snapshots in cgsnapshots | 14:04 |
| geguileo | thangp: That second part for sure | 14:04 |
| *** timcl has joined #openstack-cinder | 14:09 | |
| *** diogogmt has quit IRC | 14:13 | |
| mriedem | does this seem like a long time to be holding a lock in c-vol? http://logs.openstack.org/10/230510/2/gate/gate-grenade-dsvm/e796b39/logs/new/screen-c-vol.txt.gz#_2015-10-06_05_40_15_936 | 14:13 |
| mriedem | 2015-10-06 05:40:15.936 DEBUG oslo_concurrency.lockutils [req-6b402bc1-3c24-47f8-be65-609bfb98c8fd tempest-TestVolumeBootPatternV2-836916434] Lock "35fbbb48-90a3-45e2-9275-d66c9b8e9b2b-delete_snapshot" released by "cinder.volume.manager._run_flow_locked" :: held 16.103s inner /usr/local/lib/python2.7/dist-packages/oslo_concurrency/lockutils.py:265 | 14:13 |
| *** ntpttr has joined #openstack-cinder | 14:13 | |
| dulek | mriedem: Cinder had that lock for whole operation of volume creation. | 14:15 |
| dulek | mriedem: It is blocking snapshot removal while we're creating a volume from it. | 14:15 |
| mriedem | ok, i guess that lock is per-volume too | 14:16 |
| *** sgotliv has quit IRC | 14:16 | |
| *** eharney has joined #openstack-cinder | 14:17 | |
| dulek | mriedem: This is kind of a little strange construct. When we're creating a volume from snapshot or source volume, we lock the source resource with <resource-id>-delete_resource. | 14:18 |
| dulek | mriedem: Then we have decorators for that in delete_snapshot and delete_volume methods in manager. | 14:18 |
| dulek | mriedem: So we're protecting the origin resource. | 14:19 |
| smcginnis | all: Liberty RC2 is cut. Nice work. | 14:19 |
| *** Guest83454 is now known as mfisch | 14:19 | |
| *** mfisch is now known as Guest94490 | 14:20 | |
| *** jungleboyj has joined #openstack-cinder | 14:26 | |
| *** dustins has quit IRC | 14:26 | |
| *** yrabl has quit IRC | 14:27 | |
| *** stevemar_ has joined #openstack-cinder | 14:28 | |
| *** setmason has quit IRC | 14:28 | |
| *** yrabl has joined #openstack-cinder | 14:29 | |
| *** bswartz has quit IRC | 14:29 | |
| *** jaypipes has quit IRC | 14:32 | |
| *** markvoelker_ has joined #openstack-cinder | 14:32 | |
| *** kevincarr1991 has joined #openstack-cinder | 14:33 | |
| *** baumann has joined #openstack-cinder | 14:35 | |
| *** markvoelker has quit IRC | 14:36 | |
| *** dustins has joined #openstack-cinder | 14:37 | |
| *** markvoelker_ has quit IRC | 14:38 | |
| kevincarr1991 | Has anyone had issues attaching volumes using horizon? | 14:39 |
| *** vgridnev has quit IRC | 14:41 | |
| *** vgridnev has joined #openstack-cinder | 14:42 | |
| *** edtubill has joined #openstack-cinder | 14:45 | |
| *** xyang1 has joined #openstack-cinder | 14:47 | |
| *** daemontool_ has quit IRC | 14:49 | |
| *** asselin_ has joined #openstack-cinder | 14:49 | |
| haypo | hum, i got some random failures on cinder.tests.unit.test_volume.GetActiveByWindowTestCase.test_snapshot_get_active_by_window on the Python 2.7 check job :-/ | 14:50 |
| *** mc_nair has joined #openstack-cinder | 14:50 | |
| *** vlaza has quit IRC | 14:51 | |
| eharney | haypo: on patchset 230510? | 14:52 |
| haypo | eharney: https://review.openstack.org/#/c/230510/ yes | 14:53 |
| *** xyang1 has quit IRC | 14:53 | |
| eharney | haypo: i've hit this too: https://bugs.launchpad.net/cinder/+bug/1501838 | 14:53 |
| openstack | Launchpad bug 1501838 in Cinder "Tests: lazy load operation of attribute 'snapshot_metadata' cannot proceed" [Undecided,New] | 14:53 |
| haypo | eharney: 8:17 AM: "gate-cinder-python27: SUCCESS", 3:35PM "gate-cinder-python27: FAILURE" | 14:53 |
| eharney | seems like a bug related to snapshot objects | 14:53 |
| *** haomaiwang has quit IRC | 14:54 | |
| *** ronis has quit IRC | 14:54 | |
| *** haomaiwang has joined #openstack-cinder | 14:55 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 14:55 |
| *** markvoelker has joined #openstack-cinder | 14:55 | |
| *** hemnafk is now known as hemna | 14:57 | |
| *** martyturner has quit IRC | 14:57 | |
| *** diogogmt has joined #openstack-cinder | 14:58 | |
| *** martyturner has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** ollie1 has joined #openstack-cinder | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| dulek | thangp: ^ | 15:01 |
| Swanson | smcginnis: Notable RC2 fixes? Other than my driver will actually delete a cg snapshot now, of course. | 15:05 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 15:06 |
| *** david-ly_ is now known as david-lyle | 15:06 | |
| smcginnis | Swanson: https://launchpad.net/cinder/liberty/liberty-rc2 | 15:06 |
| *** aorourke_ is now known as aorourke | 15:06 | |
| *** diogogmt has quit IRC | 15:07 | |
| smcginnis | Swanson: Look for the - blueprints and 5 bugs targeted header. | 15:07 |
| smcginnis | Swanson: Those 5, plus a few more that didn't actually have bugs linked. | 15:07 |
| Swanson | smcginnis, excellent. Looks like us driver folk were a problem. | 15:08 |
| smcginnis | Swanson: Just a few driver fixes that were ready and were going to end up getting backported anyway. | 15:08 |
| *** diogogmt has joined #openstack-cinder | 15:09 | |
| thangp | dulek: looks like that test was introduced by https://github.com/openstack/cinder/commit/540bba7f8e049e0eea304ec21ed2da2d03319125 | 15:09 |
| openstackgerrit | Jacob Gregor proposed openstack/cinder: Cleanup/move code in Storwize Driver https://review.openstack.org/230117 | 15:09 |
| *** martyturner has quit IRC | 15:09 | |
| thangp | dulek: https://github.com/openstack/cinder/commit/200a65fa26415b9aa9f3ea7afc7fa2b8f2aee4b5 | 15:09 |
| jgriffith | eharney: you seen this one: http://paste.openstack.org/show/475486/ | 15:11 |
| thangp | dulek: we just need to return metadata = {} | 15:12 |
| kevincarr1991 | Has anyone installed cinder on their controller using Kilo? | 15:12 |
| eharney | jgriffith: no i haven't... interesting error though | 15:12 |
| *** dustins has quit IRC | 15:12 | |
| jgriffith | kevincarr1991: yes | 15:13 |
| thangp | dulek: it's trying to load snapshot_metadata, but there's none | 15:13 |
| *** zhangjn has joined #openstack-cinder | 15:13 | |
| dulek | thangp: And this Snapshot not bound to session stuff is because of that? | 15:14 |
| *** zhangjn has quit IRC | 15:14 | |
| kevincarr1991 | jgriffith: did you simply follow the install guide and do everything on the controller or did you have to make any special modifications | 15:14 |
| kevincarr1991 | ? | 15:14 |
| jgriffith | kevincarr1991: install guide'ish | 15:14 |
| * EmilienM | 15:14 | |
| jgriffith | kevincarr1991: I've written scripts based on install guide that I use | 15:14 |
| *** geektoor_ has joined #openstack-cinder | 15:15 | |
| thangp | dulek: usually, the session stuff is related to the context, but probably | 15:15 |
| kevincarr1991 | griffith: ahh do you have any tips on what needs to be altered for it to work properly? Right now I have a hard time attaching vm's to instances | 15:15 |
| *** zhangjn has joined #openstack-cinder | 15:16 | |
| dulek | thangp: I'll take a look tomorrow then. | 15:16 |
| *** yrabl has quit IRC | 15:16 | |
| hemna | diablo_rojo, ping | 15:16 |
| jgriffith | kevincarr1991: that's kinda broad, not sure | 15:16 |
| thangp | dulek: ok | 15:16 |
| jgriffith | kevincarr1991: have you verified you can ping your iqn's from your compute nodes? | 15:16 |
| *** daneyon has joined #openstack-cinder | 15:16 | |
| jgriffith | kevincarr1991: sounds like maybe an issue with network routing? | 15:16 |
| *** daneyon has quit IRC | 15:17 | |
| kevincarr1991 | jgriffith: I honestly do not know what iqn is | 15:17 |
| *** zhangjn has quit IRC | 15:17 | |
| dulek | thangp: One more question - https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L179 | 15:17 |
| dulek | thangp: You're sure there should be .objects? | 15:17 |
| jgriffith | kevincarr1991: what type of device? iSCSI, FC etc? | 15:17 |
| diablo_rojo | hemna: Hello :) | 15:17 |
| *** zhangjn has joined #openstack-cinder | 15:17 | |
| kevincarr1991 | jgriffith: I am going to use the default so iSCSI. I plan to use nfs eventually | 15:18 |
| *** martyturner has joined #openstack-cinder | 15:18 | |
| jgriffith | kevincarr1991: let me ask a different way :) | 15:18 |
| dulek | thangp: I'm trying to mimic this stuff in shapshots field in CGSnapshot and got a little doubt in this place. | 15:18 |
| hemna | diablo_rojo, any idea why the XIV CI has lots of 'worlddump' files from many different dates ? | 15:18 |
| *** changbl has joined #openstack-cinder | 15:18 | |
| jgriffith | kevincarr1991: what did you deploy in the setup that's not working? | 15:18 |
| *** geektoor has quit IRC | 15:18 | |
| hemna | diablo_rojo, re: http://dal05.objectstorage.softlayer.net/v1/AUTH_58396f85-2c60-47b9-aaf8-e03bc24a1a6f/IBM-XIV-CI/75/223375/12/check/ibm-xiv-iscsi/45423cd/logs/ | 15:18 |
| thangp | dulek: yeah... because it a list of objects, instead of a field - VolumeList | 15:18 |
| jgriffith | kevincarr1991: and if it's just attach that isn't working, check your n-cpu logs | 15:18 |
| hemna | diablo_rojo, looks like files since 9/1 -> 10/6 | 15:19 |
| jgriffith | kevincarr1991: look for the attach commands | 15:19 |
| thangp | dulek: if we used a object field, then we could just return VolumeList | 15:19 |
| dulek | thangp: Okay, you've convinced me. Well, that was easy, wasn't it? ;) | 15:19 |
| thangp | * VolumeAttachmentList | 15:19 |
| kevincarr1991 | jgriffith: I have deployed a basic three node setup. When I installed cinder that is the only thing that is having an issue. I will check nova to see if it has any attach commands issued. | 15:19 |
| diablo_rojo | hemna: Yeah. Jay sent out a nunch of angrygrams yesterday about the CI having issues. | 15:19 |
| thangp | dulek: you could do it either way | 15:19 |
| diablo_rojo | hemna: *bunch | 15:20 |
| diablo_rojo | hemna: We are gonna follow up today on that and make sure it's actively being dealt with. | 15:21 |
| hemna | ok, I was just curious as I was looking into the CI failures on your patch | 15:21 |
| dulek | thangp: Aww, Nova is using list objects directly like VolumeAttachmentList. Which approach is better then? | 15:21 |
| *** dsariel_ has quit IRC | 15:21 | |
| smcginnis | diablo_rojo, jungleboyj: The IBM CIs aren't looking to happy here: http://ci-watch.tintri.com/project?project=cinder&time=48+hours | 15:21 |
| diablo_rojo | hemna: Yeah seems lots of our CI tests are having issues right now unfortuantely. I don't think its specific to my patch. | 15:22 |
| jungleboyj | smcginnis: I noticed that yesterday and sent a number of angry grams yesterday. Following up today. | 15:22 |
| *** martyturner has quit IRC | 15:22 | |
| smcginnis | jungleboyj: Cool, thanks. | 15:22 |
| * smcginnis prep's removal patches | 15:22 | |
| thangp | dulek: might be better to use VolumeAttachmentList, but it makes you do more changes | 15:22 |
| smcginnis | Kidding! | 15:22 |
| dulek | thangp: What kind of changes? | 15:23 |
| thangp | dulek: you have to iterate through the objectlist differently i think | 15:23 |
| jungleboyj | smcginnis: Sorry. We are in the process of improving how things are done and have our own asselin lined up, but I am having to push people along to make it happen. | 15:24 |
| dulek | thangp: Ah, right, when *using* objects, not when writing them. Okay, that's fine, I'll stick to Nova's approach. | 15:24 |
| thangp | dulek: sure | 15:24 |
| asselin_ | jungleboyj, :) | 15:24 |
| smcginnis | jungleboyj: It doesn't look like it's just you FWIW. Our CI statuses look pretty sad right now. | 15:24 |
| dulek | thangp: Thanks, that was helpful! | 15:25 |
| smcginnis | jungleboyj: Let us know how the asselin_ cloning goes. :) | 15:25 |
| *** jdurgin1 has joined #openstack-cinder | 15:25 | |
| thangp | dulek: np | 15:25 |
| *** martyturner has joined #openstack-cinder | 15:25 | |
| hemna | diablo_rojo, no worries, we've had loads of CI issues as well. CI = pain | 15:25 |
| jungleboyj | smcginnis: I noticed that. I will keep you up to date on the asselin_ cloning. His clone will be named Isaac Beckman in Haifa. | 15:26 |
| asselin_ | jungleboyj, smcginnis btw, there is a common-ci patch now. my clones' reviews would be appreciated! https://review.openstack.org/#/c/227584/ | 15:26 |
| diablo_rojo | hemna: Particularly when it gets ignored by the people that should care :) | 15:26 |
| asselin_ | https://review.openstack.org/#/c/200330/ | 15:26 |
| *** dims_ has quit IRC | 15:27 | |
| jungleboyj | asselin_: I will pass those along. | 15:27 |
| hemna | diablo_rojo, I wouldn't know anything about that | 15:27 |
| hemna | :P | 15:27 |
| hemna | geguileo, ping | 15:27 |
| smcginnis | asselin_: Nice! | 15:27 |
| geguileo | hemna: pong | 15:27 |
| *** daneyon has joined #openstack-cinder | 15:27 | |
| *** zhangjn has quit IRC | 15:27 | |
| hemna | geguileo, any interest in a google hangout this morning to chat about the locks stuffs ? | 15:27 |
| *** zhangjn has joined #openstack-cinder | 15:28 | |
| *** zhangjn has quit IRC | 15:28 | |
| geguileo | hemna: Any possibility to do it tomorrow? | 15:28 |
| hemna | sure | 15:28 |
| hemna | I'm game for anytime. | 15:28 |
| dulek | geguileo, hemna: +1 for geguileo ask. | 15:28 |
| hemna | I was supposed to set this up last week and got busy | 15:28 |
| geguileo | hemna: Thanks | 15:29 |
| dulek | geguileo, hemna: I would like to be able to participate. Maybe tomorrow 16:00 UTC? | 15:29 |
| hemna | UTC makes my head hurts.... | 15:29 |
| * hemna is a stupid American | 15:29 | |
| scottda | dulek: Isn't that meeting time | 15:29 |
| scottda | ? | 15:29 |
| dulek | hemna: It's like tomorrow in 30 minutes. But scottda's right, so maybe after the meeting? | 15:29 |
| hemna | that's 9am PST, our cinder meeting time | 15:30 |
| *** zhangjn has joined #openstack-cinder | 15:30 | |
| geguileo | hemna: That's what I was going to say XD | 15:30 |
| dulek | hemna, geguileo: Or before the meeting? | 15:30 |
| *** alexschm has quit IRC | 15:30 | |
| jungleboyj | <russian accent> Stupid Americans </russian accent> | 15:30 |
| geguileo | I have available the hour before and after the meeting | 15:30 |
| *** zhangjn has quit IRC | 15:31 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Dell Eqlx: Support over subscription in thin provisioning https://review.openstack.org/231186 | 15:31 |
| *** apoorvad has joined #openstack-cinder | 15:31 | |
| *** apoorvad has quit IRC | 15:32 | |
| *** apoorvad has joined #openstack-cinder | 15:32 | |
| *** dims_ has joined #openstack-cinder | 15:32 | |
| hemna | I can do it at 15:00UTC tomorrow | 15:33 |
| scottda | I'd like to join as well. | 15:33 |
| hemna | jungleboyj, :) | 15:34 |
| geguileo | Ok, so we all can at 15:00 UTC? | 15:34 |
| hemna | I'm good to go | 15:34 |
| hemna | that's 8am my time | 15:34 |
| hemna | I'm usually JUST getting in to the office then | 15:35 |
| geguileo | Oh, would you prefer after the meeting then? | 15:35 |
| dulek | I'm fine with that time. | 15:35 |
| hemna | I'm ok with 17:00UTC as well | 15:35 |
| hemna | which is right after the cinder meeting | 15:35 |
| scottda | I can do 15:00 or 17:00 | 15:35 |
| hemna | lets do 17:00 | 15:35 |
| * dulek is looking for a dice with 2 walls | 15:35 | |
| hemna | :P | 15:35 |
| diablo_rojo | A dice with two walls...wouldn't that be a coin? | 15:36 |
| geguileo | dulek: I usually use a dice and split the decission on odd/even ;-) | 15:36 |
| *** vgridnev has quit IRC | 15:36 | |
| dulek | diablo_rojo: Technically there's a little probablity of landing on the edge… | 15:36 |
| *** apoorvad has quit IRC | 15:37 | |
| diablo_rojo | dulek: Fair point. I would be truly impressed if that happened. | 15:37 |
| geguileo | You can always repeat the roll | 15:37 |
| scottda | 17:00 is better for me. We could also mention it in the meeting for others who may be interested. | 15:38 |
| *** krtaylor has quit IRC | 15:38 | |
| scottda | But getting late in Europe | 15:38 |
| hemna | scottda, +1 | 15:38 |
| dulek | scottda: I think me and geguileo agreed, so it's fine. | 15:39 |
| dulek | (I'm not sure if DuncanT wouldn't like to attend) | 15:39 |
| *** zhangjn has joined #openstack-cinder | 15:40 | |
| *** links has quit IRC | 15:42 | |
| *** sgotliv has joined #openstack-cinder | 15:42 | |
| *** dustins has joined #openstack-cinder | 15:43 | |
| *** daneyon_ has joined #openstack-cinder | 15:43 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/222292 | 15:45 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 15:46 |
| *** daneyon has quit IRC | 15:46 | |
| *** jistr_ has joined #openstack-cinder | 15:48 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 15:50 |
| *** jistr has quit IRC | 15:50 | |
| *** krtaylor has joined #openstack-cinder | 15:51 | |
| *** aix has quit IRC | 15:52 | |
| *** jistr_ has quit IRC | 15:53 | |
| *** timcl has quit IRC | 15:55 | |
| *** haomaiwang has quit IRC | 15:56 | |
| *** chris_morrell has joined #openstack-cinder | 15:57 | |
| *** mtanino has joined #openstack-cinder | 15:59 | |
| *** garthb has joined #openstack-cinder | 16:00 | |
| jungleboyj | smcginnis: asselin_ It looks like the following may be why our CIs are all hosed: https://storyboard.openstack.org/#!/story/2000373 | 16:06 |
| *** jistr_ has joined #openstack-cinder | 16:06 | |
| smcginnis | jungleboyj: Hopefully that's it. | 16:07 |
| *** e0ne has quit IRC | 16:08 | |
| *** dsariel_ has joined #openstack-cinder | 16:08 | |
| *** dsariel_ has quit IRC | 16:08 | |
| jungleboyj | smcginnis: Will keep you updated. The moral of the story is that we need to get to the new 3rd Party CI code. | 16:08 |
| Swanson | jungleboyj, any idea when the 3rd party CI stuff will be ready for public consumption? Not really inclined to be an early adopter on that. | 16:11 |
| jungleboyj | Swanson: That is a question for asselin_ | 16:11 |
| jungleboyj | I think it is always a work in progress. | 16:11 |
| *** gouthamr has quit IRC | 16:12 | |
| Swanson | jungleboyj, yeah, but it would be nice if it would get to a set of consistent steps. | 16:12 |
| scottda | We support incremental backup in Kilo, but the stable/kilo python-cinderclient doesn't have the 'incremental' option.... | 16:12 |
| scottda | I've filed: https://bugs.launchpad.net/python-cinderclient/+bug/1503287 | 16:12 |
| openstack | Launchpad bug 1503287 in python-cinderclient "stable/kilo cinder supports incremental backups, client does not" [Undecided,New] | 16:12 |
| *** leeantho has joined #openstack-cinder | 16:12 | |
| scottda | How to I get this backported to stable/kilo cinderclient? Submit a patch on that branch? | 16:12 |
| jungleboyj | Swanson: Yeah, that is part of what I am hoping our CI guy can work with asselin_ on to take care of issues like that. | 16:14 |
| asselin | Swanson, I'd like to get it ready for public consumtopion asap | 16:14 |
| *** ollie1 has left #openstack-cinder | 16:14 | |
| asselin | Swanson, if you'd like to try it out, please do. ping me with any issues | 16:14 |
| asselin | Swanson, https://review.openstack.org/#/c/200330/ https://review.openstack.org/#/c/227584/ | 16:15 |
| *** nkrinner has quit IRC | 16:16 | |
| *** thangp_ has joined #openstack-cinder | 16:16 | |
| *** Guest94490 is now known as mfisch | 16:17 | |
| scottda | hemna: jungleboyj You had signed up to help with Nova bugs on this etherpad: | 16:17 |
| scottda | https://etherpad.openstack.org/p/Mitaka_Priorites_for_Nova_Volume_bugs | 16:17 |
| *** geektoor_ has quit IRC | 16:17 | |
| *** mfisch is now known as Guest62625 | 16:18 | |
| scottda | I've updated all the Priority:High bugs with some info. I'll be moving to the Medium next, as I have time. | 16:18 |
| *** thangp has quit IRC | 16:18 | |
| jungleboyj | scottda: Ok, I will put it on our backlog of things to look at. | 16:18 |
| scottda | cool | 16:19 |
| *** chris_morrell has quit IRC | 16:19 | |
| *** cfriesen has joined #openstack-cinder | 16:19 | |
| *** daneyon_ has left #openstack-cinder | 16:19 | |
| *** links has joined #openstack-cinder | 16:23 | |
| *** jistr_ has quit IRC | 16:23 | |
| Swanson | asselin, I'll take a peek | 16:25 |
| asselin | Swanson, thanks | 16:25 |
| scottda | Disregard question on Backport. Instructions are here: https://wiki.openstack.org/wiki/StableBranch#Proposing_Fixes | 16:26 |
| *** lpetrut has quit IRC | 16:26 | |
| *** ntpttr has quit IRC | 16:29 | |
| *** tsekiyama has joined #openstack-cinder | 16:29 | |
| *** mudassirlatif has joined #openstack-cinder | 16:30 | |
| *** martyturner has quit IRC | 16:30 | |
| *** rhedlind has joined #openstack-cinder | 16:32 | |
| scottda | Patch is: https://review.openstack.org/#/c/231580/ | 16:33 |
| *** _cjones_ has joined #openstack-cinder | 16:33 | |
| *** apoorvad has joined #openstack-cinder | 16:35 | |
| *** ntpttr has joined #openstack-cinder | 16:37 | |
| *** dims_ has quit IRC | 16:37 | |
| *** vgridnev has joined #openstack-cinder | 16:38 | |
| *** dims_ has joined #openstack-cinder | 16:41 | |
| *** dims_ has quit IRC | 16:41 | |
| *** anshul has quit IRC | 16:42 | |
| *** dims_ has joined #openstack-cinder | 16:42 | |
| *** jaypipes has joined #openstack-cinder | 16:43 | |
| *** dims__ has joined #openstack-cinder | 16:44 | |
| *** dims_ has quit IRC | 16:45 | |
| *** thingee has joined #openstack-cinder | 16:53 | |
| *** timcl has joined #openstack-cinder | 16:53 | |
| cfriesen | If I'm using LVM cinder volumes, where do I specify the "volume_clear" option in the cinder.conf file? I have it at the top level, but it doesn't seem to take effect. | 16:54 |
| *** jwcroppe_ has joined #openstack-cinder | 16:55 | |
| cfriesen | does it need to go under a separate subsection? | 16:55 |
| *** setmason has joined #openstack-cinder | 16:55 | |
| smcginnis | scottda: kilo is about to be security fixes only. | 16:56 |
| *** jwcroppe has quit IRC | 16:56 | |
| jgriffith | cfriesen: Make sure it's under the [DEFAULT] section header | 16:56 |
| *** timcl has quit IRC | 16:58 | |
| *** timcl has joined #openstack-cinder | 16:59 | |
| *** gouthamr has joined #openstack-cinder | 16:59 | |
| guitarzan | scottda: TIL clients have stable branches... | 17:00 |
| scottda | smcginnis: kilo for cinderclient? And what do you think about getting that patch in to support incremental backup in the kilo cinderclient? | 17:02 |
| *** earlephilhower has joined #openstack-cinder | 17:03 | |
| *** ronis has joined #openstack-cinder | 17:03 | |
| jgriffith | scottda: we don't do release version of the client | 17:03 |
| jgriffith | scottda: the whole point is the clients are supposed to *always* work | 17:04 |
| cfriesen | jgriffith: I did, I put it right near the top. Running under pdb in cinder/volume/drivers/lvm.py, function _delete_volume(), self.configuration.__dict__ shows 'config_group': 'lvm'. Does that mean anything? | 17:04 |
| jgriffith | cfriesen: yes it does | 17:04 |
| jgriffith | cfriesen: so you have a [lvm] section in your config | 17:04 |
| jgriffith | cfriesen: you can put it there as well | 17:04 |
| scottda | well, there's a stable/kilo branch for cinderclient and it's what we ship with kilo version of cinder.... | 17:04 |
| jgriffith | cfriesen: but that default should work for you | 17:04 |
| cfriesen | jgriffith: default doesn't work for sure. will try lvm | 17:05 |
| scottda | and kilo cinder has incremental backup, but cinderclient does not. | 17:05 |
| scottda | Should the client support what the server does? | 17:05 |
| cfriesen | jgriffith: I seem to recall some discussion about this on the mailing list...that the defaults didn't propagate down into the various drivers. | 17:05 |
| jgriffith | scottda: yeah, but we've never done backports for client branches that I'm aware of. That doesn't really work with pypi :) | 17:05 |
| cfriesen | jgriffith: arguably a bug, I think | 17:05 |
| jgriffith | cfriesen: correct | 17:05 |
| jgriffith | cfriesen: but it's odd, because this is a particular case that I've never had problems with in the past | 17:06 |
| scottda | OK, thanks. I'm never quite sure how the client stable branch thing works. | 17:06 |
| jgriffith | cfriesen: and IIRC it's also how the gate works | 17:06 |
| jgriffith | scottda: yeah... so we started creating the branch just to provide a marker so we knew what was in use when we released | 17:06 |
| cfriesen | jgriffith: well, we have some patches on top of a snapshot of stable/kilo....it's possible we messed something up locally. I'll see if I can test on pristine stable/kilo. | 17:06 |
| jgriffith | scottda: backporting to client isn't something I've ever come across, not to say it isn't done; just it would be new to me :) | 17:07 |
| jgriffith | scottda: also, based on backport criteria you're getting into the whole feature vs bug debate in your case :) | 17:07 |
| jgriffith | cfriesen: let me know if you do and it doesn't work | 17:07 |
| *** martyturner has joined #openstack-cinder | 17:07 | |
| scottda | hehe..yeah, I get that. Thanks for the info jgriffith | 17:08 |
| jgriffith | cfriesen: I just set it up in devstack on master here and it does still work correctly for me at least | 17:08 |
| jgriffith | scottda: :) Don't know if that was helpful or not, but hopefully it was | 17:08 |
| *** porrua has quit IRC | 17:08 | |
| *** porrua has joined #openstack-cinder | 17:08 | |
| hemna | asselin, https://review.openstack.org/231602 | 17:08 |
| asselin | hemna, ack | 17:09 |
| cfriesen | jgriffith: putting it under [lvm] works locally, will check with devstack | 17:10 |
| scottda | No, that was helpful. It just leaves things in a weird state for something like this, where the feature is in the server, but not in the client, for a particular release (not that there is a client release, but there is that branch). | 17:10 |
| scottda | A newer client will work, of course, and has the incremental backup option.... | 17:10 |
| scottda | But we don't want to ship a newer client with older server, since there are other features in the client that are not supported. | 17:11 |
| jgriffith | scottda: yeah, we REALLY screwed the pooch on Kilo and the client if you remember | 17:11 |
| jgriffith | scottda: we never released a client :( | 17:11 |
| jgriffith | scottda: ball was dropped | 17:11 |
| hemna | asselin, I think that accomplishes what I had set out to do. it rebind's the PCI device to the hosts's original driver prior to trying to do a passthrough again. | 17:11 |
| scottda | yup | 17:11 |
| hemna | hopefully that might help stabilize things | 17:11 |
| hemna | patrickeast, ^^ | 17:12 |
| *** mriedem has quit IRC | 17:13 | |
| *** chris_morrell has joined #openstack-cinder | 17:16 | |
| *** e0ne has joined #openstack-cinder | 17:16 | |
| cfriesen | jgriffith: I think I might see what's going on. You alluded to it earlier in a mailing list thread. "Any option that is defined in driver.py needs to be set in the actual enabled-backend stanza IIRC. This includes iscsi_helper, volume_clear etc." | 17:17 |
| *** chris_morrell has quit IRC | 17:17 | |
| smcginnis | scottda: Sorry, was in a meeting. | 17:17 |
| smcginnis | scottda: What jgriffith said. :) | 17:17 |
| scottda | thanks :) | 17:18 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 17:18 |
| *** timcl has quit IRC | 17:19 | |
| *** david-ly_ has joined #openstack-cinder | 17:21 | |
| *** david-lyle has quit IRC | 17:21 | |
| *** timcl has joined #openstack-cinder | 17:21 | |
| *** mriedem has joined #openstack-cinder | 17:21 | |
| *** david-ly_ is now known as david-lyle | 17:22 | |
| *** markstur has quit IRC | 17:24 | |
| *** IlyaG has joined #openstack-cinder | 17:24 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 17:25 |
| *** harlowja has quit IRC | 17:25 | |
| *** raildo is now known as raildo-afk | 17:26 | |
| *** martyturner has quit IRC | 17:26 | |
| *** dims_ has joined #openstack-cinder | 17:29 | |
| asselin | hemna, looks promising | 17:29 |
| hemna | that was basically doing the same thing as the raw host commands I was going to run | 17:30 |
| *** markstur has joined #openstack-cinder | 17:30 | |
| hemna | for whatever reason virsh wasn't showing that my PCI card was being used by a VM after I attached it | 17:30 |
| hemna | so I couldn't look for it. :( | 17:30 |
| hemna | so this seems to do the same thing | 17:31 |
| asselin | hemna, coold | 17:31 |
| *** dims__ has quit IRC | 17:32 | |
| *** IanGovett has quit IRC | 17:35 | |
| *** harlowja has joined #openstack-cinder | 17:35 | |
| *** timcl has quit IRC | 17:38 | |
| *** timcl has joined #openstack-cinder | 17:39 | |
| *** IanGovett has joined #openstack-cinder | 17:39 | |
| adrianofr | hemna: ping | 17:41 |
| *** raildo-afk is now known as raildo | 17:43 | |
| hemna | adrianofr, yes | 17:44 |
| adrianofr | hemna: Hi. HNAS CI is back and approved my change: https://review.openstack.org/#/c/228575/ | 17:45 |
| adrianofr | hemna: it'd be nice if you take a look again when you have some time | 17:46 |
| hemna | ok coolio. thanks! | 17:46 |
| *** erlon has joined #openstack-cinder | 17:47 | |
| adrianofr | hemna: thanks :) | 17:48 |
| *** porrua has quit IRC | 17:49 | |
| *** porrua has joined #openstack-cinder | 17:51 | |
| hemna | np | 17:52 |
| *** dims_ has quit IRC | 17:53 | |
| *** dave-mccowan has quit IRC | 17:54 | |
| *** rushiagr_away is now known as rushiagr | 17:56 | |
| *** IlyaG has quit IRC | 17:56 | |
| *** IlyaG has joined #openstack-cinder | 17:57 | |
| *** IlyaG has quit IRC | 17:57 | |
| *** IlyaG has joined #openstack-cinder | 17:57 | |
| *** dims_ has joined #openstack-cinder | 18:01 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Add backup_swift_auth_url to swift backup driver https://review.openstack.org/228646 | 18:02 |
| *** vgridnev has quit IRC | 18:05 | |
| *** dims_ has quit IRC | 18:05 | |
| *** rushiagr is now known as rushiagr_away | 18:06 | |
| *** dims_ has joined #openstack-cinder | 18:06 | |
| *** dave-mccowan has joined #openstack-cinder | 18:07 | |
| *** vgridnev has joined #openstack-cinder | 18:07 | |
| *** chris_morrell has joined #openstack-cinder | 18:11 | |
| *** links has quit IRC | 18:13 | |
| *** jaypipes has quit IRC | 18:14 | |
| hemna | asselin, thanks for commenting on the CloudByte patch | 18:18 |
| *** chris_morrell has quit IRC | 18:18 | |
| *** BharatK has quit IRC | 18:19 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: test_backup_swift: Set volume id per test https://review.openstack.org/231652 | 18:19 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 18:23 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Do not merge: Debugging bug 1501745 https://review.openstack.org/231122 | 18:24 |
| openstack | bug 1501745 in Cinder "gate-cinder-python34 failed" [High,Confirmed] https://launchpad.net/bugs/1501745 - Assigned to Eric Harney (eharney) | 18:24 |
| *** timcl has quit IRC | 18:26 | |
| *** angela-s has joined #openstack-cinder | 18:26 | |
| *** haigang has joined #openstack-cinder | 18:28 | |
| erlon | david-lyle: ping | 18:30 |
| david-lyle | erlon: pong | 18:30 |
| *** timcl has joined #openstack-cinder | 18:31 | |
| erlon | david-lyle: hi david, remember a day, you and John Griftith where talking about the Horizon option "Boot from image/creates a new volume"'? | 18:34 |
| erlon | https://bugs.launchpad.net/horizon/+bug/1486178 | 18:34 |
| openstack | Launchpad bug 1486178 in OpenStack Dashboard (Horizon) "Boot from image (creates a new volume) Doesn't allow specification of volume-type" [High,Fix released] - Assigned to David Lyle (david-lyle) | 18:34 |
| david-lyle | yes, I started working on that | 18:35 |
| *** pv_ has joined #openstack-cinder | 18:35 | |
| erlon | david-lyle: hmm, it says in the BP that it is resolved but I can't see anything in gerrit | 18:35 |
| pv_ | hey guys quick question | 18:35 |
| pv_ | is it important to test my driver for backwards compatibility with old openstack releases? | 18:36 |
| david-lyle | erlon: bp? | 18:36 |
| david-lyle | or bug | 18:36 |
| erlon | david-lyle: bug | 18:36 |
| erlon | pv_: yes, if you driver is in the old release | 18:36 |
| pv_ | b/c im trying with a juno devstack and i've placed my driver files in a private juno repo and my stack.sh is erroring out giving me deprecation warnings on some oslo packages | 18:36 |
| pv_ | oh | 18:36 |
| pv_ | but if its in the new one or aiming to be part of the new one then no? | 18:37 |
| pv_ | how would customers use our product if they're running an older version of openstack then? | 18:37 |
| erlon | pv_: i.e., you are changing your driver, if is already in liberty for example, you should make sure that a user upgrading to mitaka won't have problem | 18:37 |
| david-lyle | erlon don't know who marked that committed, looking | 18:37 |
| jgriffith | pv_: no | 18:38 |
| pv_ | oh ok, my driver is a new driver and im not sure which release we're aiming for right now but it doesn't exist yet | 18:38 |
| pv_ | as part of openstack | 18:38 |
| jgriffith | pv_: the upstream model is that support starts when you introduce/merge it | 18:38 |
| jgriffith | pv_: we don't allow backports of drivers to previous openstack versions | 18:39 |
| jgriffith | pv_: however you are able to have a driver that works and is made available through other channels | 18:39 |
| jgriffith | Soumitra apprantly | 18:40 |
| erlon | pv_: so, we have the same situation here. What I do, I install a version of the system, RH Juno, for example. Place my code in site-packages/cinder/..., then, bring my code up, fixing any import issues I find. Then you can generate a RPM package | 18:40 |
| *** xyang1 has joined #openstack-cinder | 18:40 | |
| pv_ | alright thanks so much | 18:41 |
| pv_ | i relayed the info onto my superiors | 18:41 |
| pv_ | :D | 18:41 |
| erlon | pv_: welcome | 18:42 |
| erlon | david-lyle: I have looked for something 'fixing' in horizon and nova, but, didn't found anything | 18:43 |
| erlon | david-lyle: what have you done so far in the fixing? | 18:43 |
| david-lyle | erlon: well I started the horizon side, but I think the issue is bigger | 18:44 |
| david-lyle | I'm not sure how to pass on the volume-type to nova | 18:44 |
| * david-lyle has to refresh mind | 18:44 | |
| erlon | david-lyle: yes, It is bigger indeed :), it goot me too | 18:45 |
| erlon | david-lyle: you have to pass it in the API, but nova os-boot does not support that volume-type | 18:46 |
| erlon | david-lyle: *got | 18:46 |
| *** baumann has quit IRC | 18:47 | |
| erlon | david-lyle: what I did so far was to change the API in nova to accept a volume-type parameter and pass that to cinder | 18:47 |
| david-lyle | well the BlockDeviceMapping doesn't even seem to have a field for it | 18:47 |
| david-lyle | which is how we pass on the other information | 18:48 |
| *** bswartz has joined #openstack-cinder | 18:49 | |
| erlon | david-lyle: yes, I had to add the field | 18:49 |
| david-lyle | ok | 18:49 |
| david-lyle | erlon: reset state | 18:51 |
| david-lyle | on the bug | 18:51 |
| david-lyle | drive-by status change | 18:51 |
| *** _cjones_ has quit IRC | 18:51 | |
| *** _cjones_ has joined #openstack-cinder | 18:52 | |
| erlon | david-lyle: ?? | 18:53 |
| david-lyle | erlon: some random person appeared to have changed the bug status with no reason | 18:53 |
| david-lyle | set it back to confirmed | 18:53 |
| david-lyle | erlon: do you have a patch proposed to nova then? | 18:54 |
| david-lyle | or do we require a change in workflow | 18:54 |
| *** setmason has quit IRC | 18:55 | |
| erlon | david-lyle: hmm, Im struggling with my internet to push it | 18:56 |
| erlon | david-lyle: https://review.openstack.org/231660 | 18:57 |
| erlon | https://www.irccloud.com/pastebin/YiNyqUQc/horizon-api%20call | 18:58 |
| erlon | david-lyle: to, call nova directly | 18:59 |
| david-lyle | erlon: I have the UI piece to populate that | 19:00 |
| hemna | 1-800-call-nova | 19:00 |
| erlon | david-lyle: great! do you have the patch proposed? | 19:01 |
| david-lyle | no, because testing proved unsuccessful :-) | 19:01 |
| david-lyle | I'll dig it up and clean it up | 19:02 |
| jungleboyj | Hello, this is Nova. | 19:02 |
| jungleboyj | hemna: Get this, my phone rang the other day, I pick it up and the person asks 'Is this OpenStack?'. I informed them that I work on OpenStack on Cinder. They just said they were looking for the OpenStack support line. | 19:03 |
| hemna | lol | 19:03 |
| david-lyle | erlon: will have to figure out object version checking too, or just provide users with false hope | 19:04 |
| david-lyle | *I will | 19:04 |
| erlon | david-lyle: Hmm, I have no idea how that works, I've changed the Object version just to follow what was being done there | 19:06 |
| Swanson | jungleboyj, I presume they meant thingee by that so you should have just coughed up his cell. | 19:06 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 19:07 |
| jungleboyj | Swanson: :-) | 19:07 |
| david-lyle | erlon: wouldn't be the first time we had an ineffective control in the UI | 19:07 |
| e0ne | patchset stats: +11, -6389 lines. I hope, tempest will pass:) | 19:08 |
| erlon | david-lyle: how can that be? there's no reference to the object version in the API call to nova | 19:09 |
| david-lyle | erlon: I find many things aren't really meant to be used | 19:10 |
| david-lyle | would just be a guessing game | 19:10 |
| erlon | david-lyle: in nova patchset? | 19:10 |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Remove extra register_opts() calls in netapp eseries https://review.openstack.org/231664 | 19:11 |
| david-lyle | no, I mean object version, if I call the API how do I know version of the object the nova I'm calling supports? | 19:11 |
| david-lyle | I can only query if the extension is supported | 19:11 |
| david-lyle | IIUC | 19:12 |
| *** setmason has joined #openstack-cinder | 19:15 | |
| scottda | david-lyle: can the nova api microversion be used to detect the newer version? | 19:16 |
| david-lyle | scottda: I don't know that the API is changing, that's part of the problem | 19:17 |
| erlon | david-lyle: I think it will call the last, it is what is happing with me, in the API request I'm not specifying anything, see the paste above | 19:17 |
| *** ociuhandu has quit IRC | 19:18 | |
| scottda | It looks like erlon 's patch is incrementing API to 1.16 | 19:18 |
| david-lyle | I think that's just the object version | 19:19 |
| scottda | sorry, you are right. | 19:20 |
| erlon | david-lyle: does the API have micro versions too? | 19:20 |
| david-lyle | erlon: I believe it should | 19:20 |
| scottda | erlon: Yes, it does. | 19:20 |
| david-lyle | not sure for extensions | 19:20 |
| david-lyle | but I would hope so | 19:20 |
| erlon | scottda: so, the change should update the API microversion too | 19:21 |
| scottda | If it is a change in the Nova API, yes. | 19:21 |
| scottda | But the Nova folks can probably give you better guidance than I. | 19:22 |
| *** e0ne has quit IRC | 19:23 | |
| erlon | scottda: hmm, how the API microversions are related to the Object versions? | 19:23 |
| *** dustins has quit IRC | 19:24 | |
| *** dustins has joined #openstack-cinder | 19:24 | |
| scottda | I don't know that they are. But if you are changing the Nova API to add support for a volume-type, that will require a bump in the microversion. | 19:24 |
| david-lyle | nested, undocumented parameter | 19:25 |
| scottda | But I'm not sure you are actually changing the Nova api, are you? | 19:25 |
| david-lyle | not sure if it's considered an API change or not | 19:25 |
| scottda | Sorry, I just looked at your review and saw the commit message. But this is only if it's an actual API change. | 19:26 |
| erlon | scottda: well, its an additional parameter. I think is an actual change | 19:29 |
| cfriesen | an additional parameter would certainly require a microversion bump in nova | 19:29 |
| *** e0ne has joined #openstack-cinder | 19:32 | |
| *** Lee1092 has quit IRC | 19:33 | |
| *** baumann has joined #openstack-cinder | 19:34 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 19:35 |
| *** IlyaG has quit IRC | 19:35 | |
| *** baumann has quit IRC | 19:38 | |
| *** tpsilva has quit IRC | 19:40 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: test_backup_swift: Set volume id per test https://review.openstack.org/231652 | 19:45 |
| openstackgerrit | Eric Harney proposed openstack/cinder: test_backup_nfs: Set volume id per test https://review.openstack.org/231674 | 19:47 |
| *** haigang has quit IRC | 19:48 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Do not merge: Debugging notification issues in tests https://review.openstack.org/231122 | 19:49 |
| *** tsekiyama has quit IRC | 19:51 | |
| erlon | david-lyle: talking to nova guys, they pointed out this change: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-volume-types,n,z | 19:53 |
| erlon | david-lyle: which also implements the volume-type in nova | 19:53 |
| david-lyle | ok, cool, but the bp is not approved :( | 19:54 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 19:55 |
| *** ronis has quit IRC | 19:55 | |
| *** tsekiyama has joined #openstack-cinder | 19:57 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 19:59 |
| erlon | david-lyle: yep | 20:00 |
| *** jaypipes has joined #openstack-cinder | 20:00 | |
| *** porrua has quit IRC | 20:05 | |
| *** tsekiyama has quit IRC | 20:08 | |
| *** DericHorn-HP has joined #openstack-cinder | 20:09 | |
| *** tsekiyama has joined #openstack-cinder | 20:09 | |
| *** chris_morrell has joined #openstack-cinder | 20:10 | |
| *** merooney has quit IRC | 20:10 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 20:12 |
| *** bswartz has quit IRC | 20:14 | |
| *** chris_morrell has quit IRC | 20:14 | |
| *** akerr has quit IRC | 20:18 | |
| *** IlyaG has joined #openstack-cinder | 20:23 | |
| *** zhangjn has quit IRC | 20:24 | |
| *** zhangjn_ has joined #openstack-cinder | 20:24 | |
| *** timcl has quit IRC | 20:29 | |
| *** IlyaG has quit IRC | 20:31 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 20:36 |
| *** jungleboyj has quit IRC | 20:38 | |
| *** mtaylor has joined #openstack-cinder | 20:39 | |
| *** wolsen has quit IRC | 20:39 | |
| *** wolsen has joined #openstack-cinder | 20:40 | |
| *** mordred has quit IRC | 20:42 | |
| *** mtaylor is now known as mordred | 20:42 | |
| *** erlon has quit IRC | 20:43 | |
| *** jgregor has quit IRC | 20:43 | |
| *** adrianofr has quit IRC | 20:43 | |
| *** jamielennox has quit IRC | 20:43 | |
| *** zigo has quit IRC | 20:43 | |
| *** aarefiev has quit IRC | 20:43 | |
| *** amit213 has quit IRC | 20:43 | |
| *** sayali has quit IRC | 20:43 | |
| *** DuncanT has quit IRC | 20:43 | |
| *** krotscheck has quit IRC | 20:43 | |
| *** serverascode has quit IRC | 20:43 | |
| *** vaidy_ril has quit IRC | 20:43 | |
| *** swamireddy has quit IRC | 20:43 | |
| *** mgagne has quit IRC | 20:43 | |
| *** Roamer` has quit IRC | 20:43 | |
| *** erlon has joined #openstack-cinder | 20:46 | |
| *** jgregor has joined #openstack-cinder | 20:46 | |
| *** adrianofr has joined #openstack-cinder | 20:46 | |
| *** jamielennox has joined #openstack-cinder | 20:46 | |
| *** zigo has joined #openstack-cinder | 20:46 | |
| *** aarefiev has joined #openstack-cinder | 20:46 | |
| *** amit213 has joined #openstack-cinder | 20:46 | |
| *** sayali has joined #openstack-cinder | 20:46 | |
| *** DuncanT has joined #openstack-cinder | 20:46 | |
| *** krotscheck has joined #openstack-cinder | 20:46 | |
| *** serverascode has joined #openstack-cinder | 20:46 | |
| *** vaidy_ril has joined #openstack-cinder | 20:46 | |
| *** swamireddy has joined #openstack-cinder | 20:46 | |
| *** mgagne has joined #openstack-cinder | 20:46 | |
| *** Roamer` has joined #openstack-cinder | 20:46 | |
| *** serverascode has quit IRC | 20:46 | |
| *** ntpttr has quit IRC | 20:47 | |
| *** DericHorn-HP has quit IRC | 20:49 | |
| *** DericHorn-HP has joined #openstack-cinder | 20:51 | |
| *** morgan has quit IRC | 20:52 | |
| *** morgan has joined #openstack-cinder | 20:52 | |
| *** serverascode has joined #openstack-cinder | 20:55 | |
| e0ne | smcginnis, thingee, jgriffith, DuncanT: hi team! I hate to ask it, but could you please to review https://review.openstack.org/218226? I must to rebase it after almost every spec merge | 20:58 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Test_backup_swift: Don't leak notifications https://review.openstack.org/231707 | 20:58 |
| smcginnis | e0ne: No problem, I'll take a look. | 20:58 |
| e0ne | smcginnis: thanks! I've fixed few typos there in the last patch | 20:58 |
| openstackgerrit | Alex O'Rourke proposed openstack/python-cinderclient: Adds v2 replication support https://review.openstack.org/231708 | 20:58 |
| smcginnis | e0ne: Funny, I reviewed it (got the green check boxes) but didn't comment on this patch. :) | 20:59 |
| e0ne | smcginnis: FYI, remove xml api patch stats: +11, -6389 :) | 20:59 |
| smcginnis | e0ne: Shoot, you're winning. I'm going to have to find more code to remove. :D | 20:59 |
| e0ne | smcginnis: :) | 21:00 |
| eharney | i'm hoping this patch i just posted ^ will help reduce occurrence of gate failures | 21:00 |
| *** DericHorn-HP has quit IRC | 21:01 | |
| *** setmason has quit IRC | 21:01 | |
| e0ne | eharney: great! | 21:01 |
| eharney | kinda have to wait and see, but i think it makes sense | 21:01 |
| *** raildo is now known as raildo-afk | 21:09 | |
| *** setmason has joined #openstack-cinder | 21:09 | |
| *** edmondsw has quit IRC | 21:14 | |
| *** changbl has quit IRC | 21:14 | |
| *** e0ne has quit IRC | 21:24 | |
| *** gouthamr has quit IRC | 21:26 | |
| *** Yogi1 has quit IRC | 21:28 | |
| *** vgridnev has quit IRC | 21:29 | |
| *** ociuhandu has joined #openstack-cinder | 21:31 | |
| jgriffith | hemna: do you know Alex's IRC nick? | 21:32 |
| jgriffith | also patrickeast around? | 21:32 |
| kevincarr1991 | I am unable to attach volumes on my instances. When I try to I get a generic error stating that nova "Failed to attach /dev/sdb" This could mean a network issue correct? | 21:33 |
| hemna | jgriffith, aorourke | 21:33 |
| *** jgregor has quit IRC | 21:33 | |
| aorourke | hemna, yes | 21:33 |
| jgriffith | aorourke: around ? :) | 21:33 |
| jgriffith | hemna: aorourke I wanted to talk about the replication stuff if you guys have a few minutes? | 21:34 |
| aorourke | aorourke, yes i have time | 21:34 |
| hemna | sup | 21:34 |
| jgriffith | aorourke: so I'm not sure my comments were making sense in the review | 21:34 |
| jgriffith | aorourke: hemna also in terms of the name/label thing | 21:34 |
| jgriffith | aorourke: hemna why not just modify the specification to have a required param in the dict that is "target_name" | 21:35 |
| hemna | works for me | 21:35 |
| hemna | or as patrickeast mentioned in his abandoned patch, 'name' | 21:35 |
| hemna | I'm fine either way | 21:36 |
| jgriffith | hemna: haven't seen that | 21:36 |
| hemna | https://review.openstack.org/#/c/231210/ | 21:36 |
| jgriffith | aorourke: I'm also not clear on why you need custom conf options for everything in the 3par driver? | 21:36 |
| jgriffith | aorourke: I mean.. if you do, then you do.. that's fine, but seems like there's some overlap | 21:36 |
| aorourke | jgriffith, is there default options for replication_mode, replication_sync_period, etc already? | 21:37 |
| jgriffith | aorourke: yes | 21:37 |
| jgriffith | aorourke: via the extra-specs setting on the volume-type | 21:37 |
| aorourke | jgriffith, ah, I must have missed that. i see now where you recommend using the extra specs | 21:37 |
| jgriffith | aorourke: but.. if you wanted to control that more gloabally via conf that's probably cool too and should be in driver.py | 21:37 |
| hemna | jgriffith, so I believe one of the entries in the conf for us, is how we map the local pool name to the remote pool name | 21:39 |
| hemna | giving the admin the ability to specify the name of the pool on the target to put the volume during creation | 21:39 |
| aorourke | hemna, that is correct. cpg_map is definitely 3par specific | 21:39 |
| hemna | just added flexibilty/complexity for us | 21:39 |
| aorourke | but the others can be consolidated into driver.py if other plan to use them as well | 21:39 |
| aorourke | jgriffith, I see in your SolidFire implementation you are also using replication_host | 21:40 |
| *** thangp_ has quit IRC | 21:41 | |
| aorourke | well, "secondary_backend", but using it for the same thing | 21:41 |
| *** ociuhandu has quit IRC | 21:42 | |
| aorourke | and pure has a "pure_replica_interval_default" which is the same as our "replication_sync_period" | 21:42 |
| *** ociuhandu has joined #openstack-cinder | 21:44 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: test_backup_swift: Set volume id per test https://review.openstack.org/231652 | 21:45 |
| *** mriedem has quit IRC | 21:45 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Test_backup_swift: Don't leak notifications https://review.openstack.org/231707 | 21:45 |
| *** harlowja has quit IRC | 21:46 | |
| *** IlyaG has joined #openstack-cinder | 21:46 | |
| openstackgerrit | Alex O'Rourke proposed openstack/python-cinderclient: Adds v2 replication support https://review.openstack.org/231708 | 21:49 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Change os-brick to use ostestr https://review.openstack.org/231735 | 21:51 |
| *** julim has quit IRC | 21:53 | |
| *** stevemar_ has quit IRC | 21:55 | |
| *** stevemar_ has joined #openstack-cinder | 21:56 | |
| *** harlowja has joined #openstack-cinder | 21:57 | |
| *** dave-mccowan has quit IRC | 21:57 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: test_backup_nfs: Set volume id per test https://review.openstack.org/231674 | 21:57 |
| *** stevemar_ has quit IRC | 21:58 | |
| jgriffith | aorourke: sorry... got pulled away | 22:00 |
| *** dustins has quit IRC | 22:01 | |
| aorourke | jgriffith, no worries | 22:01 |
| Swanson | asselin, is the project-config-example git url supposed to be a real url pointing to an example repo or and example url? | 22:01 |
| jgriffith | aorourke: not sure what you're looking at, yes I have the secondary_backend option, but as xyang1 points out that should be in the base driver for all to use | 22:02 |
| jgriffith | aorourke: I don't have the replication period thing that you mention | 22:02 |
| asselin | Swanson, it's supposed to be a real url point to a project-config repo. You can use the default provided (if the docs are correct) | 22:02 |
| aorourke | jgriffith, Pure has the sync period in their driver implementation | 22:02 |
| jgriffith | aorourke: if people need/use that my point was just that it should either be in the extra-specs or in the base driver config I think | 22:02 |
| asselin | Swanson, this is a valid project you can use: https://git.openstack.org/cgit/openstack-infra/project-config-example/ | 22:03 |
| asselin | but it'd be better to fork it first, and use that | 22:03 |
| jgriffith | aorourke: I don't see that merged? | 22:03 |
| asselin | Swanson, idea is that using this url will get you a working system: https://git.openstack.org/openstack-infra/project-config-example | 22:04 |
| Swanson | Hmmm... Getting repo not found on that. | 22:04 |
| jgriffith | aorourke: in other words, I'd ask Danny the same questions :) | 22:04 |
| aorourke | jgriffith, it's not. i was saying some of the config entries we are using for the 3par implementation are also being used in other driver implantations as well. so it would make sense to put them in driver.py for everyone to use | 22:05 |
| jgriffith | aorourke: I am also of the opinion that if you have defaults that you intend to be overridden by type settings, then you should probably just "hide" them internally | 22:05 |
| jgriffith | aorourke: ahh.. yes that :) | 22:06 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Change os-brick to use ostestr https://review.openstack.org/231735 | 22:06 |
| jgriffith | I agree with that, barring the point I made about some of them perhaps belonging in the type setting | 22:06 |
| *** stevemar_ has joined #openstack-cinder | 22:07 | |
| jgriffith | aorourke: so my whole goal of stuffing more of the stuff into the type was so we don't have this problem of all sorts of unique conf options for every driver | 22:07 |
| jgriffith | aorourke: I was hoping to just have the absolute bare minimum defined in the core code | 22:07 |
| jgriffith | aorourke: that way it would be easier/flexible for drivers as most have different semantics | 22:08 |
| aorourke | jgriffith, I see. which conf entries are you opposed putting in as defaults? | 22:08 |
| *** patrickeast has quit IRC | 22:09 | |
| *** adrianofr has quit IRC | 22:09 | |
| *** sweston has quit IRC | 22:09 | |
| *** rex_lee_ has quit IRC | 22:09 | |
| *** boris-42 has quit IRC | 22:09 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 22:11 |
| jgriffith | aorourke: to be clear, I certainly think anything is open for discussion :) | 22:12 |
| jgriffith | aorourke: but, I don't feel that replication_mode, sync_period, target_name or replication_host should be in your driver | 22:12 |
| jgriffith | aorourke: take sync vs async for example | 22:13 |
| aorourke | jgriffith, the idea I was going off of was having the entire driver instance configured to one setting. anything created with 3parfcrep (in our specific case) would use the configured settings | 22:13 |
| jgriffith | aorourke: the agreed upon design was to use capability reporting for a driver to say what it supports | 22:13 |
| jgriffith | aorourke: and then for the choice to be made via the type setting | 22:14 |
| Swanson | asselin, cgit in that url should have been my clue. | 22:14 |
| jgriffith | aorourke: sure, but then there' s one in driver FOO, and BIZ, and BAZ | 22:14 |
| jgriffith | aorourke: and there's no naming scheme | 22:14 |
| jgriffith | aorourke: and we've just duplicated a bunch of conf entries for no real reason | 22:15 |
| jgriffith | see what I mean? | 22:15 |
| asselin | Swanson, no problem...you're exposing potential issues so that's good | 22:15 |
| aorourke | jgriffith, in the sync vs. async example, how would you change what replication_type reports without having some sort of conf option? | 22:16 |
| jgriffith | aorourke: does the 3par cluster support both? | 22:16 |
| aorourke | jgriffith, yes, sync and async periodic mode | 22:16 |
| jgriffith | aorourke: I'm assumign it does thus the config option :) | 22:16 |
| jgriffith | aorourke: so the 3par driver should report "supported_replication_types = [sync, async]" | 22:17 |
| jgriffith | aorourke: and the admin chooses which are used by volume-type set replication_type=sync | 22:17 |
| jgriffith | or async | 22:17 |
| jgriffith | or one type for each | 22:17 |
| jgriffith | aorourke: it achieves the same thing, but allows it to be dynamic | 22:18 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix silent iSCSI login failures https://review.openstack.org/231740 | 22:18 |
| jgriffith | aorourke: so in my case (and others) where the same cluster can do sync or async simultaneously, we offer that flexibility still | 22:18 |
| *** edtubill has quit IRC | 22:18 | |
| jgriffith | aorourke: in the case of those that do not, then the admin just doesn't create two of them | 22:18 |
| *** apoorvad has quit IRC | 22:18 | |
| jgriffith | aorourke: does that make sense? | 22:18 |
| aorourke | jgriffith, couldn't you, in that case, have multiple volume types configured completely differently (one sync the other async) all using the same backend type? | 22:19 |
| jgriffith | <o/ | 22:19 |
| jgriffith | aorourke: yes, exactly! | 22:19 |
| *** jaypipes has quit IRC | 22:19 | |
| jgriffith | aorourke: my point is why have it there AND in the config file? | 22:19 |
| *** IlyaG has quit IRC | 22:20 | |
| *** IlyaG has joined #openstack-cinder | 22:21 | |
| jgriffith | aorourke: am I maybe not considering something? | 22:21 |
| *** cppforlife_ has quit IRC | 22:22 | |
| aorourke | jgriffith, hmm ok I believe that makes sense. I will have to explore that a little more in our implementation and get back to you. | 22:23 |
| jgriffith | aorourke: let me see if I can get some updates pushed up tonight/tomorrow that maybe will help | 22:24 |
| aorourke | jgriffith, not related to this directly, but v2 replication in general, i had just filed this an hour or so ago. https://bugs.launchpad.net/cinder/+bug/1503439 | 22:25 |
| openstack | Launchpad bug 1503439 in Cinder "Issuing v2 replication commands against a non-replicated volume type puts replication_status into an unrecoverable state" [Undecided,New] | 22:25 |
| jgriffith | aorourke: ahh.. good one :) | 22:26 |
| jgriffith | aorourke: I can fix that up, unless you already had something in mind? | 22:26 |
| aorourke | jgriffith, my thought where simply using a try catch around the volume_rpcapi call. right after it sets it to 'enabling' | 22:27 |
| aorourke | jgriffith, if it fails, set it back to what it was | 22:27 |
| jgriffith | aorourke: yeah, maybe... or the other thing is add an actual mechanism to check the type and make sure it's replicated before even trying | 22:28 |
| jgriffith | aorourke: hmm... actually, there's already a check there | 22:29 |
| aorourke | jgriffith, that is definitely an option. but if it does try because it is of replicated type and fails for a different reason, it can still get stuck in a bad state | 22:29 |
| *** gouthamr has joined #openstack-cinder | 22:29 | |
| jgriffith | aorourke: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1615 | 22:30 |
| aorourke | jgriffith, that is just to make sure you issue that command against a volume that has a replication_status of disable currently, correct? | 22:31 |
| aorourke | jgriffith, with a non-replicated type volume, replication_status is defaulted to 'disabled' | 22:32 |
| jgriffith | aorourke: if you have a volume reporting replication_status=disabled and it's not replication capabile then the bug is in your driver :) | 22:32 |
| jgriffith | aorourke: we need to fix that :) | 22:32 |
| jgriffith | aorourke: that's not the intent | 22:32 |
| jgriffith | aorourke: the intent was that the default was "None" | 22:32 |
| aorourke | jgriffith, replication_status would be 'disabled' for that volume (because it is not replicated type), but you can still issue enable_replication against it | 22:33 |
| jgriffith | aorourke: no, actually that's not how it was designed | 22:34 |
| jgriffith | aorourke: that status should never be set on a non replication capable volume | 22:34 |
| jgriffith | replication=disabled implies that it is capable but has been disabled | 22:35 |
| *** mc_nair has quit IRC | 22:35 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 22:35 |
| aorourke | jgriffith, so replication_status in the volume is defaulted at None? | 22:36 |
| jgriffith | aorourke: so that's the bug that needs fixing IMO | 22:36 |
| jgriffith | aorourke: no, it's a bug as you point out... just for different reason :) | 22:36 |
| jgriffith | so it is defaulting to "disabled" it shouldn't | 22:36 |
| jgriffith | it should be something else, like "not-supported" or None or Uninitialized or "something" | 22:37 |
| jgriffith | something clear | 22:37 |
| aorourke | jgriffith, that would solve the issue | 22:37 |
| *** gouthamr_ has joined #openstack-cinder | 22:37 | |
| *** sgotliv has quit IRC | 22:38 | |
| *** erlon has quit IRC | 22:38 | |
| jgriffith | aorourke: so part of that is from the "old" implementation ;( | 22:39 |
| aorourke | jgriffith, ahh that is unfortunate. is v1 staying in the code? | 22:40 |
| kevincarr1991 | jgriffith: we spoke earlier about not being able to attach volumes to instances. I found that nova on my compute node it throwing very generic HTTP 500 errors. Do you have any ideas in where other file locations may be for errors | 22:40 |
| jgriffith | aorourke: afraid so :( | 22:40 |
| *** IanGovett has quit IRC | 22:40 | |
| *** ociuhandu has quit IRC | 22:41 | |
| *** gouthamr has quit IRC | 22:41 | |
| jgriffith | aorourke: unless you can help me convince IBM that it's a bad thing to have in there :) | 22:41 |
| *** setmason has quit IRC | 22:41 | |
| jgriffith | aorourke: GRRRRRRR https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L474 | 22:41 |
| jgriffith | aorourke: that's kinda stupid IMO | 22:42 |
| aorourke | jgriffith, yeah i agree.....is having it defaulted it 'disabled' crucial to v1 implementation? any idea? | 22:42 |
| jgriffith | added here: https://github.com/openstack/cinder/commit/1c8f49bfe9fe3abd713e28922d5551f71228624c | 22:42 |
| jgriffith | aorourke: I'm looking at refactoring it | 22:43 |
| *** stevemar_ has quit IRC | 22:43 | |
| jgriffith | aorourke: one of the problems is that the V1 stuff is such an intertwined mix of things I'm not sure what is going to break yet :( | 22:43 |
| jgriffith | aorourke: of if I'm catching all the little corners it's hiding in | 22:43 |
| hemna | shouldn't it be None for volumes that aren't replication capable ? | 22:43 |
| *** kevincar_ has joined #openstack-cinder | 22:43 | |
| *** stevemar_ has joined #openstack-cinder | 22:43 | |
| hemna | maybe that's what you guys just said | 22:44 |
| aorourke | hemna, it makes more sense that way | 22:44 |
| hemna | heh sorry | 22:44 |
| jgriffith | aorourke: I actually filed a bug against this already once I think | 22:44 |
| aorourke | jgriffith, there is still the case where replication_enable can fail for another reason, when the volume is of replicated type, and the replication_status will get stuck as 'enabling' | 22:44 |
| jgriffith | aorourke: and suggested we use something like "unconfigured" | 22:44 |
| jgriffith | aorourke: true dat | 22:44 |
| jgriffith | aorourke: but I'm ok with that in most cases, because: | 22:44 |
| jgriffith | 1. You can return a rep status update | 22:44 |
| jgriffith | 2. if it failed for a real reason I want it to go to error | 22:45 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 22:45 |
| *** sweston has joined #openstack-cinder | 22:46 | |
| aorourke | jgriffith, I see. so instead of raising an exception you are saying return the model update anyway as an "error" state or something alike | 22:46 |
| *** stevemar_ has quit IRC | 22:46 | |
| *** kevincarr1991 has quit IRC | 22:47 | |
| jgriffith | aorourke: yeah | 22:47 |
| *** kevincar_ has quit IRC | 22:47 | |
| *** ociuhandu has joined #openstack-cinder | 22:48 | |
| jgriffith | aorourke: I already added it: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L3098 | 22:49 |
| jgriffith | aorourke: but that probably needs some tweaking | 22:49 |
| jgriffith | aorourke: there was some disagreement from folks on what I should to there | 22:50 |
| jgriffith | aorourke: I had been setting the volume to error, which didn't fly :) | 22:50 |
| jgriffith | aorourke: I should probably update the replication_status to error | 22:51 |
| *** tpeoples has joined #openstack-cinder | 22:51 | |
| *** angela-s has quit IRC | 22:52 | |
| jgriffith | aorourke: sighh... that old code really makes this kind of a mess | 22:52 |
| *** boris-42 has joined #openstack-cinder | 22:53 | |
| aorourke | jgriffith, it does...there is no easy way to tell. it should be None or Unconfigured or something | 22:53 |
| *** ociuhandu has quit IRC | 22:53 | |
| jgriffith | aorourke: I'm going to go with unconfigured... sound ok? | 22:53 |
| jgriffith | I guess I better do "not-configured" though, else I'll get a -1 for improper grammar :( | 22:54 |
| aorourke | yeah, "replication_status" -- "unconfigured" makes sense to me | 22:54 |
| *** pv_ has quit IRC | 22:54 | |
| *** jwcroppe_ is now known as jwcroppe | 22:54 | |
| *** IanGovett has joined #openstack-cinder | 22:55 | |
| aorourke | :) | 22:55 |
| *** setmason has joined #openstack-cinder | 22:56 | |
| *** IlyaG has quit IRC | 22:58 | |
| *** subscope has quit IRC | 22:59 | |
| *** apoorvad has joined #openstack-cinder | 23:01 | |
| *** adrianofr has joined #openstack-cinder | 23:01 | |
| *** mudassirlatif_ has joined #openstack-cinder | 23:02 | |
| jgriffith | aorourke: hehe https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L454 | 23:02 |
| jgriffith | :) | 23:02 |
| *** eharney has quit IRC | 23:03 | |
| jgriffith | aorourke: but here's the *good* news https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L502 | 23:04 |
| *** mudassirlatif has quit IRC | 23:05 | |
| *** mudassirlatif_ is now known as mudassirlatif | 23:05 | |
| jgriffith | aorourke: So, when a driver creates a volume, checks the type and it's replicated and everything is configured and "cool" then the driver should return that status | 23:05 |
| jgriffith | so I think that solves our problem here | 23:05 |
| jgriffith | testing now, patch coming up in a moment | 23:06 |
| *** chlong has quit IRC | 23:06 | |
| jgriffith | aorourke: thnk I'll update the doc as well to make this more clear | 23:06 |
| aorourke | jgriffith, excellent! | 23:08 |
| *** MIDENN_ has quit IRC | 23:09 | |
| *** ociuhandu has joined #openstack-cinder | 23:11 | |
| *** rex_lee_ has joined #openstack-cinder | 23:14 | |
| *** ociuhandu has quit IRC | 23:16 | |
| *** IlyaG has joined #openstack-cinder | 23:17 | |
| *** ociuhandu has joined #openstack-cinder | 23:17 | |
| *** gouthamr_ has quit IRC | 23:19 | |
| *** ociuhandu_ has joined #openstack-cinder | 23:19 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Do no use disabled as default replication_status https://review.openstack.org/231753 | 23:20 |
| *** ociuhandu has quit IRC | 23:22 | |
| *** ociuhandu_ is now known as ociuhandu | 23:22 | |
| *** patrickeast has joined #openstack-cinder | 23:29 | |
| *** uiyice has joined #openstack-cinder | 23:33 | |
| *** hemna is now known as hemnafk | 23:35 | |
| *** tsekiyama has quit IRC | 23:35 | |
| jgriffith | d[ ๏ _ ◉ ]b | 23:36 |
| *** bill_az_ has quit IRC | 23:37 | |
| *** jungleboyj has joined #openstack-cinder | 23:39 | |
| *** uiyice has quit IRC | 23:40 | |
| *** uiyice has joined #openstack-cinder | 23:40 | |
| *** cppforlife_ has joined #openstack-cinder | 23:40 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix SolidFire target composition https://review.openstack.org/231758 | 23:41 |
| openstackgerrit | John Griffith proposed openstack/cinder: Do not use disabled as default replication_status https://review.openstack.org/231753 | 23:44 |
| *** tsekiyama has joined #openstack-cinder | 23:44 | |
| *** zhangjn_ has quit IRC | 23:45 | |
| *** stevemar_ has joined #openstack-cinder | 23:50 | |
| *** takedakn has joined #openstack-cinder | 23:50 | |
| *** dave-mccowan has joined #openstack-cinder | 23:51 | |
| *** zhangjn has joined #openstack-cinder | 23:52 | |
| *** xyang1 has quit IRC | 23:52 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed) https://review.openstack.org/231201 | 23:54 |
| *** tsekiyama has quit IRC | 23:54 | |
| *** tsekiyama has joined #openstack-cinder | 23:56 | |
| *** jdurgin1 has quit IRC | 23:56 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Implement v2 replication (unmanaged) https://review.openstack.org/231245 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!