| *** alonma has quit IRC | 00:02 | |
| *** alonma has joined #openstack-cinder | 00:05 | |
| *** salv-orl_ has joined #openstack-cinder | 00:06 | |
| *** akerr has joined #openstack-cinder | 00:08 | |
| *** akerr has quit IRC | 00:08 | |
| *** salv-orlando has quit IRC | 00:09 | |
| *** alonma has quit IRC | 00:10 | |
| *** markvoelker has joined #openstack-cinder | 00:12 | |
| *** alonma has joined #openstack-cinder | 00:13 | |
| *** alonma has quit IRC | 00:17 | |
| *** sasukeh has joined #openstack-cinder | 00:19 | |
| *** Fdaisuke has joined #openstack-cinder | 00:23 | |
| *** cheneydc has joined #openstack-cinder | 01:09 | |
| *** salv-orl_ has quit IRC | 01:11 | |
| *** salv-orlando has joined #openstack-cinder | 01:12 | |
| *** alonma has joined #openstack-cinder | 01:13 | |
| *** alonma has quit IRC | 01:18 | |
| *** davechen has joined #openstack-cinder | 01:19 | |
| *** bardia has joined #openstack-cinder | 01:21 | |
| *** alonma has joined #openstack-cinder | 01:23 | |
| *** davechen1 has joined #openstack-cinder | 01:25 | |
| *** chlong has joined #openstack-cinder | 01:25 | |
| *** alonma has quit IRC | 01:27 | |
| *** alonma has joined #openstack-cinder | 01:29 | |
| *** davechen has quit IRC | 01:29 | |
| *** alonma has quit IRC | 01:33 | |
| *** alonma has joined #openstack-cinder | 01:35 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix image volume creation error https://review.openstack.org/263081 | 01:36 |
|---|---|---|
| *** alonma has quit IRC | 01:40 | |
| *** Lee1092 has joined #openstack-cinder | 01:41 | |
| *** alonma has joined #openstack-cinder | 01:46 | |
| *** alonma has quit IRC | 01:51 | |
| *** zhangjn has joined #openstack-cinder | 01:52 | |
| *** alonma has joined #openstack-cinder | 01:53 | |
| *** alonma has quit IRC | 01:57 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Add manage/unmanage snapshot support https://review.openstack.org/261156 | 01:59 |
| *** alonma has joined #openstack-cinder | 02:02 | |
| *** zhangjn has quit IRC | 02:04 | |
| *** alonma has quit IRC | 02:06 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination support to volume type https://review.openstack.org/261448 | 02:07 |
| *** alonma has joined #openstack-cinder | 02:11 | |
| *** haomaiwang has joined #openstack-cinder | 02:12 | |
| *** alonma has quit IRC | 02:15 | |
| *** alonma has joined #openstack-cinder | 02:20 | |
| *** alonma has quit IRC | 02:24 | |
| *** alonma has joined #openstack-cinder | 02:28 | |
| *** zhenguo has joined #openstack-cinder | 02:29 | |
| *** alonma has quit IRC | 02:33 | |
| *** haomaiw__ has joined #openstack-cinder | 02:34 | |
| *** haomaiwang has quit IRC | 02:34 | |
| *** gcb has joined #openstack-cinder | 02:37 | |
| *** houming has joined #openstack-cinder | 02:41 | |
| *** gouthamr has joined #openstack-cinder | 02:43 | |
| *** zhangjn has joined #openstack-cinder | 02:44 | |
| *** krtaylor has joined #openstack-cinder | 02:56 | |
| openstackgerrit | dongwenjuan proposed openstack/cinder-specs: Add ServiceGroup using Tooz driver https://review.openstack.org/258968 | 02:57 |
| *** haomaiw__ has quit IRC | 03:01 | |
| *** 6A4ABI07L has joined #openstack-cinder | 03:01 | |
| *** hunters1094 has joined #openstack-cinder | 03:09 | |
| hunters1094 | hi all | 03:10 |
| hunters1094 | happy new year | 03:10 |
| openstackgerrit | Peter Wang proposed openstack/cinder: Fix race condition when Toggling SP https://review.openstack.org/262135 | 03:14 |
| *** zhangjn has quit IRC | 03:15 | |
| *** gouthamr has quit IRC | 03:17 | |
| *** davechen has joined #openstack-cinder | 03:25 | |
| *** davechen1 has quit IRC | 03:27 | |
| *** hunters1094 has quit IRC | 03:30 | |
| *** coolsvap|away is now known as coolsvap | 03:33 | |
| *** hunters1094 has joined #openstack-cinder | 03:34 | |
| hunters1094 | hi all | 03:34 |
| hunters1094 | happy new year | 03:35 |
| *** lcurtis has joined #openstack-cinder | 03:35 | |
| *** davechen1 has joined #openstack-cinder | 03:37 | |
| *** davechen has quit IRC | 03:40 | |
| hunters1094 | are there anyone here? | 03:41 |
| *** links has joined #openstack-cinder | 03:54 | |
| *** 6A4ABI07L has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** zhangjn has joined #openstack-cinder | 04:07 | |
| openstackgerrit | FelixLi proposed openstack/cinder: Fix some inconsistency in docstrings https://review.openstack.org/263104 | 04:11 |
| openstackgerrit | Merged openstack/cinder: Updates consistency group for ibm svc driver https://review.openstack.org/259766 | 04:12 |
| *** alonma has joined #openstack-cinder | 04:16 | |
| *** zhangjn has quit IRC | 04:18 | |
| *** alonma has quit IRC | 04:21 | |
| *** sasukeh has quit IRC | 04:21 | |
| *** sasukeh has joined #openstack-cinder | 04:22 | |
| *** alonma has joined #openstack-cinder | 04:23 | |
| *** alonma has quit IRC | 04:27 | |
| *** alonma has joined #openstack-cinder | 04:29 | |
| *** hunters1094 has quit IRC | 04:32 | |
| *** alonma has quit IRC | 04:33 | |
| *** apoorvad has joined #openstack-cinder | 04:45 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 05:01 | |
| *** salv-orl_ has joined #openstack-cinder | 05:07 | |
| *** salv-orlando has quit IRC | 05:07 | |
| *** apoorvad has quit IRC | 05:20 | |
| *** apoorvad has joined #openstack-cinder | 05:22 | |
| *** deepakcs has joined #openstack-cinder | 05:22 | |
| *** apoorvad has quit IRC | 05:25 | |
| *** zhangjn has joined #openstack-cinder | 05:29 | |
| *** alonma has joined #openstack-cinder | 05:29 | |
| *** ankit_ag has joined #openstack-cinder | 05:31 | |
| *** alonma has quit IRC | 05:34 | |
| *** alonma has joined #openstack-cinder | 05:36 | |
| *** markvoelker has quit IRC | 05:39 | |
| *** alonma has quit IRC | 05:41 | |
| *** harlowja_at_home has joined #openstack-cinder | 05:51 | |
| *** zhangjn has quit IRC | 05:54 | |
| *** lcurtis has quit IRC | 05:54 | |
| *** itzdilip has joined #openstack-cinder | 05:56 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** zhangjn has joined #openstack-cinder | 06:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:01 | |
| *** yangyapeng has joined #openstack-cinder | 06:02 | |
| *** alonma has joined #openstack-cinder | 06:02 | |
| *** salv-orlando has joined #openstack-cinder | 06:06 | |
| *** alonma has quit IRC | 06:06 | |
| *** shyama has joined #openstack-cinder | 06:07 | |
| *** harlowja_at_home has quit IRC | 06:08 | |
| *** salv-orl_ has quit IRC | 06:09 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: update remove_export() with versionedobjects https://review.openstack.org/263127 | 06:10 |
| *** ChubYann has quit IRC | 06:13 | |
| *** shausy has joined #openstack-cinder | 06:14 | |
| *** lpetrut has joined #openstack-cinder | 06:18 | |
| *** yangyapeng has quit IRC | 06:20 | |
| *** alonma has joined #openstack-cinder | 06:20 | |
| *** alonma has quit IRC | 06:24 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei: Refactor driver for the second time https://review.openstack.org/256920 | 06:24 |
| openstackgerrit | Alan proposed openstack/cinder: syntax error, Change from "an volume" to "a volume" changed files: ./api/v2/views/volumes.py:46: """Generic, non-detailed view of an volume.""" ./db/api.py:244: """Set the given properties on an volume and update it. ./scheduler/driver.py:37: https://review.openstack.org/263136 | 06:25 |
| shyama | hi i have a question about pool aware cinder scheduler. Are there are list of cinder drivers that support pool aware? | 06:27 |
| *** alonma has joined #openstack-cinder | 06:29 | |
| *** zhonghua-lee has quit IRC | 06:29 | |
| *** zhonghua-lee has joined #openstack-cinder | 06:29 | |
| *** alonma has quit IRC | 06:33 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei: Support huawei consistency group https://review.openstack.org/260930 | 06:34 |
| *** alonma has joined #openstack-cinder | 06:36 | |
| *** markvoelker has joined #openstack-cinder | 06:40 | |
| *** alonma has quit IRC | 06:40 | |
| *** alonma has joined #openstack-cinder | 06:42 | |
| *** zhangjn has quit IRC | 06:44 | |
| *** markvoelker has quit IRC | 06:45 | |
| *** alonma has quit IRC | 06:46 | |
| *** lpetrut has quit IRC | 06:48 | |
| *** alonma has joined #openstack-cinder | 06:49 | |
| *** zhangjn has joined #openstack-cinder | 06:50 | |
| *** hunters1094 has joined #openstack-cinder | 06:52 | |
| *** alonma has quit IRC | 06:54 | |
| *** chlong has quit IRC | 06:55 | |
| *** alonma has joined #openstack-cinder | 06:56 | |
| *** isaacb has joined #openstack-cinder | 06:56 | |
| *** zhangjn has quit IRC | 06:58 | |
| *** alonma has quit IRC | 07:00 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** zhangjn has joined #openstack-cinder | 07:02 | |
| *** alonma has joined #openstack-cinder | 07:02 | |
| *** alonma has quit IRC | 07:06 | |
| *** hunters1094 has quit IRC | 07:08 | |
| *** yangyapeng has joined #openstack-cinder | 07:08 | |
| *** alonma has joined #openstack-cinder | 07:08 | |
| *** alonma has quit IRC | 07:13 | |
| *** alonma has joined #openstack-cinder | 07:15 | |
| *** zhangjn has quit IRC | 07:19 | |
| *** alonma has quit IRC | 07:19 | |
| *** alonma has joined #openstack-cinder | 07:22 | |
| *** alonma has quit IRC | 07:27 | |
| *** geguileo_phone has quit IRC | 07:28 | |
| *** zhangjn has joined #openstack-cinder | 07:29 | |
| *** belmoreira has joined #openstack-cinder | 07:29 | |
| *** alonma has joined #openstack-cinder | 07:30 | |
| *** lennyb_ has joined #openstack-cinder | 07:31 | |
| *** isaacb has quit IRC | 07:34 | |
| *** alonma has quit IRC | 07:34 | |
| *** salv-orlando has quit IRC | 07:34 | |
| *** geguileo_phone has joined #openstack-cinder | 07:35 | |
| *** salv-orlando has joined #openstack-cinder | 07:35 | |
| *** alonma has joined #openstack-cinder | 07:36 | |
| *** alonma has quit IRC | 07:41 | |
| *** salv-orlando has quit IRC | 07:42 | |
| *** salv-orlando has joined #openstack-cinder | 07:42 | |
| *** alonma has joined #openstack-cinder | 07:43 | |
| *** alonma has quit IRC | 07:48 | |
| *** jhnl has joined #openstack-cinder | 07:48 | |
| *** alonma has joined #openstack-cinder | 07:48 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei: Refactor driver for the second time https://review.openstack.org/256920 | 07:48 |
| *** isaacb has joined #openstack-cinder | 07:50 | |
| openstackgerrit | Alan proposed openstack/cinder: syntax error,Wrong usage of charater 'an',so changed from "an volume" to "a volume". https://review.openstack.org/263136 | 07:51 |
| *** nkrinner has joined #openstack-cinder | 07:51 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:01 | |
| *** markus_z has joined #openstack-cinder | 08:02 | |
| *** zhangjn has quit IRC | 08:04 | |
| *** zhangjn has joined #openstack-cinder | 08:04 | |
| *** zhangjn has quit IRC | 08:05 | |
| *** anshul has joined #openstack-cinder | 08:06 | |
| *** zhangjn has joined #openstack-cinder | 08:06 | |
| *** isaacb has quit IRC | 08:08 | |
| *** bardia has quit IRC | 08:12 | |
| *** geguileo_mobile has joined #openstack-cinder | 08:13 | |
| *** geguileo_phone has quit IRC | 08:15 | |
| *** jordanP has joined #openstack-cinder | 08:20 | |
| *** lpetrut has joined #openstack-cinder | 08:20 | |
| *** ildikov has joined #openstack-cinder | 08:21 | |
| openstackgerrit | Alan proposed openstack/cinder: Grammatical mistake in defining articles. https://review.openstack.org/263136 | 08:21 |
| *** alonma has quit IRC | 08:22 | |
| *** alonma has joined #openstack-cinder | 08:22 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 08:25 |
| flip214 | smcginnis: thanks for +1 on 256292! | 08:25 |
| *** alonma has quit IRC | 08:26 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination support to consistency group https://review.openstack.org/262387 | 08:28 |
| *** alonma has joined #openstack-cinder | 08:28 | |
| *** xek_ is now known as xek | 08:29 | |
| *** alonma has quit IRC | 08:30 | |
| *** alonma has joined #openstack-cinder | 08:31 | |
| *** salv-orlando has quit IRC | 08:34 | |
| *** salv-orlando has joined #openstack-cinder | 08:35 | |
| *** zhangjn has quit IRC | 08:36 | |
| *** liverpooler has joined #openstack-cinder | 08:36 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Fix image volume creation error https://review.openstack.org/263081 | 08:36 |
| *** Fdaisuke has quit IRC | 08:38 | |
| *** markvoelker has joined #openstack-cinder | 08:41 | |
| *** zhangjn has joined #openstack-cinder | 08:43 | |
| *** markvoelker has quit IRC | 08:45 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 08:46 |
| *** davechen1 is now known as davechen | 08:47 | |
| openstackgerrit | Alan proposed openstack/cinder: Fix grammatical mistake in defining articles. https://review.openstack.org/263136 | 08:50 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO) https://review.openstack.org/237387 | 08:59 |
| *** links has quit IRC | 09:00 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** links has joined #openstack-cinder | 09:04 | |
| *** shausy has quit IRC | 09:09 | |
| *** shausy has joined #openstack-cinder | 09:11 | |
| *** jistr has joined #openstack-cinder | 09:12 | |
| *** itzdilip has quit IRC | 09:18 | |
| *** manous has joined #openstack-cinder | 09:21 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: [WIP] Refactor Windows drivers using os-win https://review.openstack.org/243565 | 09:29 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add backup rename function https://review.openstack.org/263179 | 09:30 |
| openstackgerrit | LisaLi proposed openstack/python-cinderclient: Add backup-rename https://review.openstack.org/263184 | 09:32 |
| *** [1]Thelo has joined #openstack-cinder | 09:32 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror) https://review.openstack.org/249042 | 09:33 |
| *** alonma has quit IRC | 09:34 | |
| *** Thelo has quit IRC | 09:34 | |
| *** [1]Thelo is now known as Thelo | 09:34 | |
| *** alonma has joined #openstack-cinder | 09:35 | |
| *** alonma has quit IRC | 09:36 | |
| *** alonma has joined #openstack-cinder | 09:36 | |
| *** davechen has left #openstack-cinder | 09:37 | |
| *** lpetrut has quit IRC | 09:38 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (mirror) https://review.openstack.org/249042 | 09:46 |
| *** lpetrut has joined #openstack-cinder | 09:49 | |
| *** goodygum has joined #openstack-cinder | 09:50 | |
| *** ndipanov has joined #openstack-cinder | 09:56 | |
| *** hunters1094 has joined #openstack-cinder | 09:58 | |
| hunters1094 | hi all | 09:58 |
| hunters1094 | are there anyone here? | 09:58 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** zhangjn has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** openstackgerrit has quit IRC | 10:02 | |
| *** cheneydc has quit IRC | 10:02 | |
| *** openstackgerrit has joined #openstack-cinder | 10:03 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:03 | |
| *** coolsvap is now known as coolsvap|away | 10:04 | |
| *** geguileo_mobile has quit IRC | 10:09 | |
| *** yangyapeng has quit IRC | 10:11 | |
| *** aix has joined #openstack-cinder | 10:15 | |
| *** geguileo_phone has joined #openstack-cinder | 10:23 | |
| *** jistr has quit IRC | 10:24 | |
| *** jistr has joined #openstack-cinder | 10:30 | |
| *** jistr has quit IRC | 10:30 | |
| *** markvoelker has joined #openstack-cinder | 10:42 | |
| *** markvoelker has quit IRC | 10:46 | |
| *** hunters1094 has quit IRC | 10:49 | |
| *** alonma has quit IRC | 10:54 | |
| *** alonma has joined #openstack-cinder | 10:55 | |
| *** alonma has quit IRC | 10:59 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** alonma has joined #openstack-cinder | 11:01 | |
| *** 17WABG26P has joined #openstack-cinder | 11:01 | |
| *** alonma has quit IRC | 11:06 | |
| *** e0ne has joined #openstack-cinder | 11:06 | |
| *** alonma has joined #openstack-cinder | 11:07 | |
| *** alonma has quit IRC | 11:12 | |
| *** alonma has joined #openstack-cinder | 11:13 | |
| *** alonma has quit IRC | 11:18 | |
| *** shyama has quit IRC | 11:18 | |
| *** alonma has joined #openstack-cinder | 11:19 | |
| *** alonma has quit IRC | 11:24 | |
| *** alonma has joined #openstack-cinder | 11:26 | |
| *** alonma has quit IRC | 11:30 | |
| *** alonma has joined #openstack-cinder | 11:31 | |
| *** dims has joined #openstack-cinder | 11:40 | |
| *** dims has quit IRC | 11:40 | |
| *** dims has joined #openstack-cinder | 11:41 | |
| *** jistr has joined #openstack-cinder | 11:41 | |
| *** erlon has joined #openstack-cinder | 11:53 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257187 | 11:55 |
| *** 17WABG26P has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-cinder | 12:01 | |
| *** chlong has joined #openstack-cinder | 12:03 | |
| *** salv-orl_ has joined #openstack-cinder | 12:06 | |
| *** salv-orl_ has quit IRC | 12:07 | |
| *** salv-orl_ has joined #openstack-cinder | 12:07 | |
| *** salv-orlando has quit IRC | 12:09 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257195 | 12:09 |
| *** markvoelker has joined #openstack-cinder | 12:12 | |
| openstackgerrit | HouMing Wang proposed openstack/cinder: Python 3: Replace reduce and xrange with six.moves https://review.openstack.org/262526 | 12:17 |
| *** markvoelker has quit IRC | 12:17 | |
| *** timcl has joined #openstack-cinder | 12:26 | |
| *** arif-ali has quit IRC | 12:27 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Remove race conditions from transfer API https://review.openstack.org/257495 | 12:32 |
| *** shyama has joined #openstack-cinder | 12:32 | |
| *** links has quit IRC | 12:34 | |
| *** raildo-afk is now known as raildo | 12:36 | |
| *** shausy has quit IRC | 12:37 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Update initialize_connection to use versionedobjects https://review.openstack.org/238894 | 12:37 |
| *** lpetrut has quit IRC | 12:43 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Update replication to use versionedobjects https://review.openstack.org/238906 | 12:46 |
| *** zul has quit IRC | 12:47 | |
| *** zul has joined #openstack-cinder | 12:47 | |
| *** jistr has quit IRC | 12:50 | |
| *** houming has quit IRC | 12:53 | |
| *** timcl has quit IRC | 12:53 | |
| *** markvoelker has joined #openstack-cinder | 12:55 | |
| *** arif-ali has joined #openstack-cinder | 12:55 | |
| *** geguileo_phone has quit IRC | 12:57 | |
| *** geguileo_phone has joined #openstack-cinder | 12:59 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-cinder | 13:01 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257199 | 13:02 |
| *** jistr has joined #openstack-cinder | 13:02 | |
| *** gcb has quit IRC | 13:02 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Balanced FC port selection when zoning https://review.openstack.org/261151 | 13:03 |
| *** manous has quit IRC | 13:04 | |
| *** dosaboy_ is now known as dosaboy | 13:05 | |
| *** haomaiwang has quit IRC | 13:06 | |
| *** dustins has joined #openstack-cinder | 13:06 | |
| *** dave-mccowan has joined #openstack-cinder | 13:08 | |
| *** 17WABG31A has joined #openstack-cinder | 13:09 | |
| *** shyama has quit IRC | 13:10 | |
| *** salv-orl_ has quit IRC | 13:13 | |
| *** salv-orlando has joined #openstack-cinder | 13:14 | |
| *** manous has joined #openstack-cinder | 13:16 | |
| *** ankit_ag has quit IRC | 13:19 | |
| *** edmondsw has joined #openstack-cinder | 13:20 | |
| *** chlong has quit IRC | 13:20 | |
| *** gouthamr has joined #openstack-cinder | 13:26 | |
| *** gouthamr is now known as gouthamr_away | 13:27 | |
| *** gouthamr_away is now known as gouthamr | 13:27 | |
| *** akerr has joined #openstack-cinder | 13:27 | |
| *** adrianofr has joined #openstack-cinder | 13:27 | |
| *** 17WABG31A has quit IRC | 13:28 | |
| *** chlong has joined #openstack-cinder | 13:33 | |
| *** deepakcs has quit IRC | 13:33 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix service-list filter https://review.openstack.org/263257 | 13:34 |
| *** dslevin_ has joined #openstack-cinder | 13:41 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder-specs: Support backup import on another Storage database https://review.openstack.org/249176 | 13:43 |
| *** geguileo_phone has quit IRC | 13:45 | |
| *** geguileo_phone has joined #openstack-cinder | 13:46 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix service-list filter https://review.openstack.org/263257 | 13:47 |
| *** timcl has joined #openstack-cinder | 13:49 | |
| *** dslevin_ has quit IRC | 13:50 | |
| *** dslevin_ has joined #openstack-cinder | 13:51 | |
| *** dslevin_ has quit IRC | 13:51 | |
| *** dslevin_ has joined #openstack-cinder | 13:52 | |
| *** gouthamr is now known as gouthamr_away | 13:52 | |
| *** gouthamr_away has quit IRC | 13:53 | |
| *** dustins has quit IRC | 13:54 | |
| bluex | geguileo_phone: hi | 13:54 |
| geguileo_phone | hi | 13:55 |
| bluex | would you mind if I ressurect https://review.openstack.org/#/c/205839 ? | 13:55 |
| *** geguileo_mobile has joined #openstack-cinder | 13:55 | |
| *** links has joined #openstack-cinder | 13:56 | |
| *** gouthamr has joined #openstack-cinder | 13:57 | |
| *** geguileo_phone has quit IRC | 13:59 | |
| *** geguileo_mobile has quit IRC | 14:00 | |
| *** geguileo_phone has joined #openstack-cinder | 14:03 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Update create_snapshot to use volume object https://review.openstack.org/260618 | 14:03 |
| *** haomaiwang has joined #openstack-cinder | 14:06 | |
| *** links has quit IRC | 14:07 | |
| *** porrua has joined #openstack-cinder | 14:10 | |
| *** sasukeh has quit IRC | 14:17 | |
| *** julim has joined #openstack-cinder | 14:23 | |
| *** Yogi1 has joined #openstack-cinder | 14:23 | |
| *** mc_nair has joined #openstack-cinder | 14:23 | |
| *** alonma has quit IRC | 14:24 | |
| *** alonma has joined #openstack-cinder | 14:24 | |
| *** jgregor has joined #openstack-cinder | 14:26 | |
| *** merooney has joined #openstack-cinder | 14:28 | |
| *** sasukeh has joined #openstack-cinder | 14:28 | |
| *** alonma has quit IRC | 14:29 | |
| *** chenying has quit IRC | 14:30 | |
| *** breitz has joined #openstack-cinder | 14:30 | |
| *** alonma has joined #openstack-cinder | 14:30 | |
| *** jistr has quit IRC | 14:30 | |
| *** chenying has joined #openstack-cinder | 14:30 | |
| *** jistr has joined #openstack-cinder | 14:31 | |
| *** sasukeh has quit IRC | 14:32 | |
| *** alonma has quit IRC | 14:35 | |
| *** alonma has joined #openstack-cinder | 14:36 | |
| *** cknight has joined #openstack-cinder | 14:37 | |
| *** baumann has joined #openstack-cinder | 14:37 | |
| openstackgerrit | Ryan Rossiter proposed openstack/cinder: Add BackupStatus enum field https://review.openstack.org/256737 | 14:38 |
| *** mriedem has joined #openstack-cinder | 14:38 | |
| *** martyturner has joined #openstack-cinder | 14:40 | |
| *** eharney has joined #openstack-cinder | 14:41 | |
| *** alonma has quit IRC | 14:41 | |
| *** alonma has joined #openstack-cinder | 14:43 | |
| *** alonma has quit IRC | 14:48 | |
| *** mdenny has joined #openstack-cinder | 14:49 | |
| *** shyama has joined #openstack-cinder | 14:49 | |
| *** alonma has joined #openstack-cinder | 14:53 | |
| *** asselin_ has joined #openstack-cinder | 14:54 | |
| *** mtanino has joined #openstack-cinder | 14:55 | |
| *** kfarr has joined #openstack-cinder | 14:56 | |
| *** alonma has quit IRC | 14:58 | |
| *** belmoreira has quit IRC | 14:58 | |
| *** lpetrut has joined #openstack-cinder | 14:59 | |
| *** alonma has joined #openstack-cinder | 14:59 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 15:01 | |
| *** jwcroppe has joined #openstack-cinder | 15:04 | |
| *** alonma has quit IRC | 15:04 | |
| *** dustins has joined #openstack-cinder | 15:05 | |
| *** alonma has joined #openstack-cinder | 15:06 | |
| *** pots1 has joined #openstack-cinder | 15:06 | |
| *** diablo_rojo has joined #openstack-cinder | 15:06 | |
| *** diablo_rojo has left #openstack-cinder | 15:07 | |
| *** crose has joined #openstack-cinder | 15:10 | |
| *** alonma has quit IRC | 15:10 | |
| *** alonma has joined #openstack-cinder | 15:12 | |
| *** shyama has quit IRC | 15:13 | |
| *** shakamunyi has quit IRC | 15:15 | |
| *** barra204 has quit IRC | 15:15 | |
| *** alonma has quit IRC | 15:17 | |
| *** aorourke has joined #openstack-cinder | 15:17 | |
| *** alonma has joined #openstack-cinder | 15:18 | |
| *** thangp has joined #openstack-cinder | 15:20 | |
| jhnl | where can I find a detailed description of the API a driver should implement? | 15:21 |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Update restore_backup to use volume object https://review.openstack.org/262024 | 15:21 |
| *** alonma has quit IRC | 15:22 | |
| *** diablo_rojo has joined #openstack-cinder | 15:24 | |
| *** alonma has joined #openstack-cinder | 15:24 | |
| *** sasukeh has joined #openstack-cinder | 15:25 | |
| *** sheel has joined #openstack-cinder | 15:25 | |
| *** gcb has joined #openstack-cinder | 15:27 | |
| *** alonma has quit IRC | 15:28 | |
| *** shakamunyi has joined #openstack-cinder | 15:29 | |
| *** jhnl has quit IRC | 15:29 | |
| *** sasukeh has quit IRC | 15:30 | |
| *** mragupat has joined #openstack-cinder | 15:30 | |
| *** alonma has joined #openstack-cinder | 15:30 | |
| *** crose has quit IRC | 15:34 | |
| *** alonma has quit IRC | 15:35 | |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Scality SOFS: don't always read /proc/mounts twice https://review.openstack.org/250915 | 15:35 |
| *** alonma has joined #openstack-cinder | 15:37 | |
| *** rhagarty has joined #openstack-cinder | 15:39 | |
| *** alonma has quit IRC | 15:41 | |
| *** alonma has joined #openstack-cinder | 15:43 | |
| *** mtreinish has quit IRC | 15:46 | |
| *** mtreinish has joined #openstack-cinder | 15:48 | |
| *** alonma has quit IRC | 15:48 | |
| e0ne | jhnl: hi. here is a minimum features set for drivers http://docs.openstack.org/developer/cinder/devref/drivers.html | 15:49 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Replace locks in volume manager https://review.openstack.org/185646 | 15:49 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Start/Stop coordinator with Services https://review.openstack.org/263313 | 15:49 |
| *** edtubill has joined #openstack-cinder | 15:51 | |
| *** guitarzan has quit IRC | 15:52 | |
| *** ntpttr has joined #openstack-cinder | 15:52 | |
| *** edtubill has quit IRC | 15:53 | |
| *** lprice1 has joined #openstack-cinder | 15:55 | |
| *** guitarzan has joined #openstack-cinder | 15:55 | |
| *** lprice has quit IRC | 15:55 | |
| *** timcl has quit IRC | 15:55 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Snapshot missing field https://review.openstack.org/263317 | 15:56 |
| *** alonma has joined #openstack-cinder | 15:59 | |
| *** Me1jin has quit IRC | 15:59 | |
| *** haomaiwa_ has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** edtubill has joined #openstack-cinder | 16:02 | |
| *** jungleboyj has joined #openstack-cinder | 16:03 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Support oversubscription for IBM Storwize/SVC https://review.openstack.org/245360 | 16:03 |
| *** alonma has quit IRC | 16:04 | |
| openstackgerrit | Harshada Mangesh Kakad proposed openstack/cinder: Fixing the deprecated library function. https://review.openstack.org/262729 | 16:04 |
| *** harlowja_at_home has joined #openstack-cinder | 16:04 | |
| *** alonma has joined #openstack-cinder | 16:05 | |
| *** liverpooler has quit IRC | 16:07 | |
| *** alonma has quit IRC | 16:09 | |
| *** timcl has joined #openstack-cinder | 16:10 | |
| *** alonma has joined #openstack-cinder | 16:11 | |
| *** xyang has joined #openstack-cinder | 16:11 | |
| *** edtubill has quit IRC | 16:12 | |
| *** kmartin has joined #openstack-cinder | 16:13 | |
| *** alonma has quit IRC | 16:16 | |
| *** alonma has joined #openstack-cinder | 16:17 | |
| *** gcb has quit IRC | 16:18 | |
| *** xyang has quit IRC | 16:20 | |
| *** hemnafk is now known as hemna | 16:21 | |
| *** alonma has quit IRC | 16:21 | |
| *** savihou has joined #openstack-cinder | 16:23 | |
| *** alonma has joined #openstack-cinder | 16:23 | |
| hemna | morning | 16:24 |
| hemna | happy new year and all that | 16:25 |
| savihou | Hello Everyone, a noobs question, I'm trying to test our third party CI system connectivity to gerrit, and keep failing with my exisiting keypair, ssh will not work as well. What should be the valid status of the ssh pub key at review.openstack.org settings ? | 16:25 |
| *** aix has quit IRC | 16:26 | |
| *** xyang has joined #openstack-cinder | 16:27 | |
| *** alonma has quit IRC | 16:28 | |
| savihou | dismiss my question, username typo , (face palm) | 16:28 |
| smcginnis | savihou: Might be a good question to pose on #openstack-infra. What happens if you try to ssh in? | 16:28 |
| smcginnis | savihou: Hah, good, that's an easy one. :) | 16:28 |
| *** gouthamr is now known as gouthamr_away | 16:28 | |
| savihou | smcginnis: Thanks :) | 16:28 |
| *** harlowja_at_home has quit IRC | 16:29 | |
| *** alonma has joined #openstack-cinder | 16:29 | |
| *** jdurgin1 has joined #openstack-cinder | 16:31 | |
| *** alonma has quit IRC | 16:34 | |
| *** alonma has joined #openstack-cinder | 16:36 | |
| *** ntpttr has quit IRC | 16:36 | |
| e0ne | hemna: welcome back | 16:37 |
| hemna | e0ne, thanks. I'm here physically at least :) | 16:38 |
| e0ne | :) | 16:38 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Implements Replication V2 https://review.openstack.org/244867 | 16:38 |
| *** alonma has quit IRC | 16:40 | |
| *** gouthamr_away is now known as gouthamr | 16:44 | |
| *** gouthamr is now known as gouthamr_away | 16:44 | |
| *** alonma has joined #openstack-cinder | 16:44 | |
| *** garthb has joined #openstack-cinder | 16:46 | |
| *** sasukeh has joined #openstack-cinder | 16:46 | |
| *** e0ne has quit IRC | 16:48 | |
| *** alonma has quit IRC | 16:49 | |
| *** mragupat has quit IRC | 16:49 | |
| *** mragupat has joined #openstack-cinder | 16:50 | |
| *** sasukeh has quit IRC | 16:50 | |
| *** barra204 has joined #openstack-cinder | 16:51 | |
| *** haomaiwang has quit IRC | 16:51 | |
| *** shakamunyi has quit IRC | 16:51 | |
| *** edtubill has joined #openstack-cinder | 16:52 | |
| *** gouthamr_away has quit IRC | 16:54 | |
| *** 21WAAO7OX has joined #openstack-cinder | 16:54 | |
| scottda | morning hemna | 16:55 |
| *** alonma has joined #openstack-cinder | 16:55 | |
| scottda | and everyone else | 16:55 |
| smcginnis | :) | 16:56 |
| *** ircuser-1 has quit IRC | 16:58 | |
| jungleboyj | Happy new year all. | 16:58 |
| *** alonma has quit IRC | 16:59 | |
| *** 21WAAO7OX has quit IRC | 17:01 | |
| *** alonma has joined #openstack-cinder | 17:01 | |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| *** martyturner has quit IRC | 17:02 | |
| smcginnis | jungleboyj: Happy new year! | 17:02 |
| *** dustins has quit IRC | 17:03 | |
| *** martyturner has joined #openstack-cinder | 17:03 | |
| jungleboyj | smcginnis: Have a good break? | 17:03 |
| sheel | jungleboyj: Thanks jay, Happy new year to you too!!! | 17:04 |
| smcginnis | jungleboyj: Yeah, good time. I wish I had more down time to get things done, but I was able to forget about work for awhile. | 17:04 |
| smcginnis | jungleboyj: How about you? | 17:04 |
| jungleboyj | sheel: You are welcome. Happy New Year to you as well! | 17:04 |
| openstackgerrit | Doug Schveninger proposed openstack/cinder: Adding action to policy.json https://review.openstack.org/262992 | 17:05 |
| jungleboyj | smcginnis: The same. Lots of traveling to and from Iowa. Didn't have as much time to get stuff done in the Lake House as hoped. | 17:05 |
| *** dustins has joined #openstack-cinder | 17:05 | |
| *** alonma has quit IRC | 17:05 | |
| jungleboyj | smcginnis: Though, I had 8 days with my family which was wonderful and the boys had good time with my family too. So, very good. | 17:06 |
| *** ntpttr has joined #openstack-cinder | 17:06 | |
| *** timcl has quit IRC | 17:06 | |
| smcginnis | jungleboyj: +1 :) | 17:07 |
| *** alonma has joined #openstack-cinder | 17:08 | |
| *** markus_z has quit IRC | 17:09 | |
| *** diogogmt has joined #openstack-cinder | 17:11 | |
| *** alonma has quit IRC | 17:12 | |
| *** alonma has joined #openstack-cinder | 17:14 | |
| ildikov | hemna: Hi! Happy New Year! :) | 17:15 |
| *** bardia has joined #openstack-cinder | 17:18 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Optimize 3PAR array ID retrieval https://review.openstack.org/263355 | 17:18 |
| *** alonma has quit IRC | 17:19 | |
| *** leeantho has joined #openstack-cinder | 17:20 | |
| *** alonma has joined #openstack-cinder | 17:21 | |
| *** jistr has quit IRC | 17:23 | |
| *** gouthamr has joined #openstack-cinder | 17:25 | |
| *** xyang has quit IRC | 17:26 | |
| *** alonma has quit IRC | 17:26 | |
| *** alonma has joined #openstack-cinder | 17:28 | |
| *** xyang has joined #openstack-cinder | 17:32 | |
| *** alonma has quit IRC | 17:32 | |
| *** alonma has joined #openstack-cinder | 17:34 | |
| *** kfarr has quit IRC | 17:36 | |
| *** ntpttr has quit IRC | 17:39 | |
| *** jordanP has quit IRC | 17:39 | |
| *** alonma has quit IRC | 17:39 | |
| *** alonma has joined #openstack-cinder | 17:42 | |
| *** savihou has quit IRC | 17:43 | |
| *** lcurtis has joined #openstack-cinder | 17:44 | |
| *** alonma has quit IRC | 17:46 | |
| *** sasukeh has joined #openstack-cinder | 17:48 | |
| *** alonma has joined #openstack-cinder | 17:48 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Optimize 3PAR array ID retrieval https://review.openstack.org/263355 | 17:48 |
| *** rhagarty_ has joined #openstack-cinder | 17:50 | |
| *** rhagarty has quit IRC | 17:52 | |
| *** sasukeh has quit IRC | 17:52 | |
| *** alonma has quit IRC | 17:53 | |
| *** alonma has joined #openstack-cinder | 17:55 | |
| *** brandonmitchell has joined #openstack-cinder | 17:57 | |
| *** alonma has quit IRC | 17:59 | |
| hemna | phew | 18:00 |
| hemna | fun way to start the year! wee! | 18:00 |
| hemna | ildikov, Happy New year to you. | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** alonma has joined #openstack-cinder | 18:01 | |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: 3PAR: Adding volume checks to manage snapshot API https://review.openstack.org/260728 | 18:01 |
| Swanson | xyang, backup via snapshots requires no driver support, right? | 18:01 |
| Swanson | xyang, other than support for snapshots in the first place. | 18:02 |
| *** angela-s has joined #openstack-cinder | 18:04 | |
| jungleboyj | hemna: smcginnis eharney jgriffith Would be good to get this looked at: https://review.openstack.org/#/c/256737 We have a bunch of similar patches behind it that will be easier to get in if we can get this first one merged. | 18:04 |
| *** harlowja has quit IRC | 18:04 | |
| hemna | jungleboyj, ok I'll check it | 18:04 |
| smcginnis | jungleboyj: Looking... | 18:04 |
| jungleboyj | Thank you guys. | 18:04 |
| *** harlowja has joined #openstack-cinder | 18:04 | |
| *** lpetrut has quit IRC | 18:05 | |
| *** mragupat has quit IRC | 18:05 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: 3PAR: Adding volume checks to manage snapshot API https://review.openstack.org/260728 | 18:05 |
| *** salv-orl_ has joined #openstack-cinder | 18:06 | |
| *** alonma has quit IRC | 18:06 | |
| *** salv-orl_ has quit IRC | 18:06 | |
| *** mragupat has joined #openstack-cinder | 18:06 | |
| *** salv-orl_ has joined #openstack-cinder | 18:07 | |
| hemna | so, we need an enum for backup status ? | 18:08 |
| *** mragupat has quit IRC | 18:08 | |
| hemna | don't we already have backup status as a text field ? | 18:08 |
| *** geguileo_phone has quit IRC | 18:08 | |
| hemna | the commit message doesn't really explain why this is needed. | 18:08 |
| xyang | Swanson: hi, yes. backup snapshot should work for any driver that already has backup volume support | 18:08 |
| hemna | is there a spec on this ? | 18:08 |
| *** mragupat has joined #openstack-cinder | 18:08 | |
| *** salv-orlando has quit IRC | 18:08 | |
| *** alonma has joined #openstack-cinder | 18:09 | |
| *** timcl has joined #openstack-cinder | 18:09 | |
| *** vgridnev has joined #openstack-cinder | 18:11 | |
| Swanson | xyang, backup volume support? I did not override the base on that one. It didn't look like I needed to. Am I incorrect? | 18:11 |
| hemna | smcginnis, shouldn't there be a spec on this | 18:11 |
| hemna | this seems to be duplicating fields in the db | 18:11 |
| hemna | I can see this causing all sorts of confusion | 18:12 |
| *** alonma has quit IRC | 18:13 | |
| smcginnis | hemna: Using enums seemed to be a good thing. I didn't think it was a big enough change to need a spec, but if you think that's an issue, I certainly don't think it would hurt. | 18:13 |
| hemna | well | 18:14 |
| smcginnis | hemna: Very possible I don't understand all the implications of this. | 18:14 |
| hemna | the problem is now we'll have multiple fields in the DB and the object to store the same data. | 18:14 |
| hemna | :( | 18:14 |
| hemna | maybe I'm paranoid, but this seems like a source of future pain. | 18:14 |
| smcginnis | hemna: Upgrades are probably an issue too, aren't they? | 18:14 |
| hemna | yah could be | 18:15 |
| hemna | maybe I have this wrong, dunno. | 18:15 |
| hemna | would be nice to get some clarification on it though. | 18:15 |
| hemna | so, sorry for the -1, but I think it's worth discussing | 18:15 |
| smcginnis | hemna: Yeah, you're absolutely right. | 18:15 |
| *** rushiagr_away is now known as rushiagr | 18:15 | |
| hemna | ok I'm wrong | 18:16 |
| hemna | heh | 18:16 |
| hemna | nm | 18:16 |
| smcginnis | hemna: What part? | 18:16 |
| hemna | it's not what I thought it was | 18:16 |
| hemna | I thought the patch was going to add enum fields in the DB | 18:16 |
| hemna | to track the status | 18:16 |
| hemna | as well as the text field | 18:16 |
| *** alonma has joined #openstack-cinder | 18:17 | |
| hemna | it doesn't look like it does that | 18:17 |
| smcginnis | hemna: I was wondering where the DB part came in. ;) | 18:17 |
| hemna | ok, don't mind me......I'll go sit in the corner. | 18:17 |
| smcginnis | hemna: Still good to question! | 18:17 |
| *** jdurgin1 has quit IRC | 18:19 | |
| *** raildo is now known as raildo-afk | 18:20 | |
| *** rushiagr is now known as rushiagr_away | 18:20 | |
| manous | hello | 18:21 |
| jungleboyj | hemna: Sorry, was grabbing lunch. | 18:21 |
| *** alonma has quit IRC | 18:21 | |
| jungleboyj | hemna: Back now. | 18:21 |
| manous | i use volume to create instance | 18:22 |
| *** geguileo_phone has joined #openstack-cinder | 18:22 | |
| manous | i have this error "The volume cannot be assigned the same device name as the root device vda" | 18:22 |
| hemna | jungleboyj, no worries. false alarm | 18:22 |
| hemna | I haven't gotten my OpenStack gloves back on yet after vacation | 18:22 |
| *** e0ne has joined #openstack-cinder | 18:23 | |
| jungleboyj | hemna: No problem. No, the idea is just to make status fields easier to maintain using enum fields in the objects. | 18:23 |
| hemna | yah cool with me | 18:24 |
| hemna | I was worried it was a new DB field | 18:24 |
| hemna | but that's what I get for reading the commit message. | 18:24 |
| jungleboyj | No. It is going to take some work to get things switched over but should be better in the long run. | 18:25 |
| hemna | so we just have to be careful because Nova isn't using any enums to access status fields. | 18:26 |
| *** alonma has joined #openstack-cinder | 18:27 | |
| ildikov | hemna: :) | 18:28 |
| ildikov | hemna: have you seen the comment I added on the Nova part of that detach Cinder bug fix? | 18:29 |
| dulek | hemna: Ah, good point on Nova. Maybe we should add an module with enums to the client? That would be hard to keep in sync tough… | 18:29 |
| hemna | notmorgan, I haven't been able to do much yet today | 18:29 |
| hemna | bah | 18:30 |
| hemna | xchat | 18:30 |
| hemna | notmorgan, | 18:30 |
| hemna | !! | 18:30 |
| *** julim_ has joined #openstack-cinder | 18:30 | |
| openstack | hemna: Error: "!" is not a valid command. | 18:30 |
| hemna | I give up | 18:30 |
| dulek | :D | 18:30 |
| smcginnis | :) | 18:30 |
| hemna | evidently I can't type "no," in xchat | 18:30 |
| *** alonma has quit IRC | 18:31 | |
| hemna | The struggle is strong with this one. | 18:31 |
| jungleboyj | hemna: So, I am thinking that they would eventually be moving there. Ryan, who submitted the patch, is a Nova guy. :-) | 18:31 |
| *** openstackgerrit has quit IRC | 18:32 | |
| hemna | dulek, maybe we can eventually send the versioned objects over the wire to nova in a different api. | 18:32 |
| hemna | meh | 18:32 |
| hemna | that also sounds like pain though. | 18:32 |
| Swanson | Bloody hell. Was spoiled last week. Gate was only 21 deep last week. 190 this week. | 18:32 |
| *** julim has quit IRC | 18:32 | |
| *** openstackgerrit has joined #openstack-cinder | 18:33 | |
| hemna | everyone is back. how many patches can we cram in...now! | 18:33 |
| dulek | hemna: Nova would need to import our versioned objects. Not sure if Nova is importing cinder code anywhere? | 18:33 |
| *** alonma has joined #openstack-cinder | 18:34 | |
| e0ne | dulek: it shouldn't import cinder directly. everybody must use cinderclient | 18:34 |
| jungleboyj | hemna: Thanks for -1'ing and then merging. :-) Happy New Year. | 18:35 |
| dulek | e0ne: Sure, that's how I thought it should be. | 18:35 |
| hemna | :) | 18:35 |
| hemna | dulek, it's not | 18:35 |
| hemna | dulek, there was a short discussion a while back about adding versioned objects to brick as well, since the contract between cinder and brick as well as nova and brick should be more contractual | 18:36 |
| hemna | right now it's just dicts that are supposed to have the right things. | 18:36 |
| hemna | that's a cross project issue that we all should chew on | 18:37 |
| hemna | kinda the same problem really. | 18:37 |
| *** alonma has quit IRC | 18:38 | |
| dulek | Well, standard REST API contract applies, but this surely isn't automatically enforced. | 18:40 |
| hemna | sure | 18:40 |
| *** alonma has joined #openstack-cinder | 18:40 | |
| *** jwcroppe has quit IRC | 18:42 | |
| *** jwcroppe has joined #openstack-cinder | 18:42 | |
| *** martyturner has quit IRC | 18:43 | |
| *** ChubYann has joined #openstack-cinder | 18:44 | |
| *** martyturner has joined #openstack-cinder | 18:44 | |
| mriedem | given how often this shows up in successful runs, it maybe shouldn't be a warning, but maybe info | 18:44 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Unable%20to%20get%20Cinder%20internal%20context%2C%20will%20not%20use%20image-volume%20cache.%5C%22%20AND%20tags%3A%5C%22screen-c-vol.txt%5C%22%20AND%20voting%3A%5C%221%5C%22 | 18:44 |
| *** alonma has quit IRC | 18:45 | |
| dulek | xyang: Hi, can we talk about https://review.openstack.org/#/c/247763/ ? | 18:45 |
| *** ntpttr has joined #openstack-cinder | 18:46 | |
| xyang | dulek: sure, what's up | 18:46 |
| dulek | xyang: This is failing some EMC driver tests added before holidays and I have trouble figuring out how to proceed. | 18:46 |
| *** raildo-afk is now known as raildo | 18:46 | |
| *** alonma has joined #openstack-cinder | 18:46 | |
| dulek | xyang: Tests were introduced by https://review.openstack.org/#/c/243017/ | 18:46 |
| patrickeast | mriedem: seems like a pretty reasonable change to make, although maybe even better if it wouldn't show up at all unless the image cache is enabled, iirc for those jobs it isn't | 18:46 |
| *** jwcroppe has quit IRC | 18:46 | |
| mriedem | how easy is it to enable the image cache in devstack? | 18:47 |
| dulek | xyang: I've tracked this to line: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_emc_vnx.py#L2075 | 18:47 |
| mriedem | patrickeast: i brought it up b/c sdague pointed this out http://logs.openstack.org/05/258005/2/gate/gate-grenade-dsvm-multinode/226ccd7/logs/old/screen-c-api.txt.gz?level=DEBUG#_2016-01-04_17_15_11_308 | 18:47 |
| patrickeast | mriedem: pretty easy, just got to turn it on with a config option and setup the internal tenant | 18:47 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support https://review.openstack.org/255015 | 18:47 |
| xyang | dulek: which tests are failing? CI passed | 18:47 |
| dulek | xyang: Without my patch this line does nothing, as volume_metadata isn't an alias, so isn't actually setting anything. | 18:47 |
| dulek | xyang: It's just unit tests. | 18:47 |
| xyang | oh, let me see | 18:48 |
| *** sasukeh has joined #openstack-cinder | 18:48 | |
| dulek | xyang: Now with my patch test starts to fail - as "volume['volume_metadata'] = sth" starts to do something meaningful. | 18:48 |
| hemna | gah | 18:48 |
| hemna | back to this horrible gerrit UI | 18:48 |
| hemna | how do you do a search in the page now ? | 18:49 |
| dulek | xyang: That's why I believe there's logical mistake either in test (hopefully) or driver logic. | 18:49 |
| hemna | ctrl-f brings up some internal POS | 18:49 |
| *** ntpttr1 has joined #openstack-cinder | 18:49 | |
| dulek | hemna: Just click somewhere on top and use ctrl+f. But still this won't search the code that's not visible. | 18:49 |
| hemna | :(!! | 18:49 |
| hemna | that makes the code view patch useless. | 18:49 |
| hemna | WTF | 18:49 |
| hemna | page | 18:50 |
| smcginnis | ctrl-f or / searches the code, but doesn't work very well. :[ | 18:50 |
| *** ntpttr has quit IRC | 18:50 | |
| hemna | holy crap, I didn't realize that bug until now | 18:50 |
| dulek | hemna: Hm, you may see if setting render to "slow" doesn't make it load all the code at once. | 18:50 |
| patrickeast | i found the / works pretty well, you can put in regex stuff | 18:50 |
| *** alonma has quit IRC | 18:50 | |
| hemna | patrickeast, yah it's not working for me at all | 18:51 |
| dulek | hemna: You go to diff view settings using this little clockwork on the right. | 18:51 |
| smcginnis | dulek: Oh, didn't realize that was a setting! | 18:51 |
| smcginnis | dulek: Seems a pretty minimal memory advantage not to load all the code at once. | 18:51 |
| hemna | man | 18:51 |
| *** rlrossit has joined #openstack-cinder | 18:51 | |
| patrickeast | what browsers are you guys using? | 18:51 |
| hemna | I can change the theme, but I can't make search work. awesome. | 18:51 |
| smcginnis | patrickeast: chromium | 18:51 |
| hemna | chromium | 18:51 |
| patrickeast | hmm it seems to work fine for me with chrome | 18:52 |
| dulek | smcginnis, hemna: I'm not sure if setting slow this will help. ;) But I think I've read somewhere that it should. | 18:52 |
| patrickeast | should be pretty much the same | 18:52 |
| patrickeast | in theory | 18:52 |
| hemna | it's a POS | 18:52 |
| *** sasukeh has quit IRC | 18:53 | |
| *** gouthamr is now known as gouthamr_away | 18:53 | |
| *** boris-42 has quit IRC | 18:53 | |
| dulek | Yeah, I've tested - with slow settings browser's ctrl+f seems to search whole code on the page. | 18:54 |
| *** gouthamr_away is now known as gouthamr | 18:55 | |
| *** gouthamr is now known as gouthamr_away | 18:55 | |
| *** anshul has quit IRC | 18:56 | |
| *** gouthamr_away is now known as gouthamr | 18:58 | |
| *** gouthamr is now known as gouthamr_away | 18:59 | |
| *** alonma has joined #openstack-cinder | 18:59 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** gouthamr_away is now known as gouthamr | 19:01 | |
| xyang | dulek: can you try change this line in unit test: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_emc_vnx.py#L2076 to tmp_snap = "snap-as-vol-%s" % vol['id'] | 19:01 |
| *** haomaiwang has joined #openstack-cinder | 19:01 | |
| *** gouthamr is now known as gouthamr_away | 19:01 | |
| rlrossit | hemna: smcginnis: Just looked at my enum patch and noticed you guys were saying stuff on IRC. I checked the logs and it sounds like there's some confusion. Is there anything you need some more info/help on from me about this stuff? | 19:03 |
| *** alonma has quit IRC | 19:03 | |
| smcginnis | rlrossit: I think we worked it out. | 19:03 |
| hemna | rlrossit, nah, it's all good. I was confused. | 19:04 |
| xyang | dulek: here too: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_emc_vnx.py#L3842 | 19:04 |
| smcginnis | rlrossit: My one remaining doubt was whether there was any kind of upgrade implication from this. | 19:04 |
| dulek | xyang: This will help, I think. So with snapcopy=True your driver is supposed to use snap-as-vol as snapshot names? Should I add that fix into my patch or should we have an additional one? | 19:05 |
| sheel | hemna:just ctrl+f and type searchWord , then press n for recurring search like we do in vim | 19:06 |
| rlrossit | smcginnis: there are some, but no more than the versioned objects bring with them (basically these enums should be additive so you don't pull the rug out from anyone) | 19:06 |
| smcginnis | rlrossit: Makes sense. Thanks for the confirmation! | 19:06 |
| *** alonma has joined #openstack-cinder | 19:06 | |
| *** dslevin has quit IRC | 19:06 | |
| *** gouthamr_away is now known as gouthamr | 19:06 | |
| dulek | smcginnis, rlrossit: This actually may be a little tougher question - what if a new state is added and older service doesn't understand it? | 19:07 |
| sheel | hemna:hope it answers to your query | 19:07 |
| dulek | We should be able to backport such object - will automated ovo version bump tests detect enum change? | 19:07 |
| hemna | sheel, yah that wasn't working, and I couldn't get it to allow chrome's search to work either. | 19:08 |
| * hemna is so not a fan of the new gerrit UI. It's broken in so many ways. | 19:08 | |
| rlrossit | dulek: if you change an enum, that will change the object hash, so that will suggest a version bump | 19:08 |
| rlrossit | dulek: if you're also implying actual backport code, that gets tougher | 19:09 |
| sheel | hemna:oh, something strange.. | 19:09 |
| * rlrossit searches quickly for a nova example | 19:09 | |
| * dulek needs to test ovo version bump test fixture by himself… | 19:10 | |
| xyang | dulek: I think there are some problems in the unit tests. I don't know why yours can't pass while the snapcopy feature can pass unit tests. I'll ask the VNX guys to double check. | 19:10 |
| *** alonma has quit IRC | 19:10 | |
| CrtxReavr | VNX? | 19:11 |
| rlrossit | dulek: here's a nova example of nova having to do an additive change to an enum https://github.com/openstack/nova/commit/63dc986d854729c0f467d828d19f2997a37f0f11 | 19:11 |
| CrtxReavr | Those are still a thing? >=] | 19:11 |
| xyang | dulek: I mean to make the change in your patch | 19:11 |
| *** dslevin has joined #openstack-cinder | 19:12 | |
| dulek | xyang: Okay, I'll work on this in a moment. Thanks! :) | 19:12 |
| *** alonma has joined #openstack-cinder | 19:13 | |
| dulek | rlrossit: No backporting code… But I get the point and tested that changing an enum changes the hash, so I'm okay with this. :) | 19:13 |
| rlrossit | dulek: cool :) | 19:16 |
| *** alonma has quit IRC | 19:17 | |
| *** alonma has joined #openstack-cinder | 19:19 | |
| *** openstackstatus has quit IRC | 19:20 | |
| *** openstackstatus has joined #openstack-cinder | 19:21 | |
| *** ChanServ sets mode: +v openstackstatus | 19:21 | |
| *** alonma has quit IRC | 19:24 | |
| *** alonma has joined #openstack-cinder | 19:28 | |
| *** bardia has quit IRC | 19:30 | |
| openstackgerrit | Slade Baumann proposed openstack/cinder: Add ConsistencyGroupStatus enum field https://review.openstack.org/262054 | 19:32 |
| *** [1]Thelo has joined #openstack-cinder | 19:32 | |
| *** alonma has quit IRC | 19:32 | |
| *** Thelo has quit IRC | 19:34 | |
| *** [1]Thelo is now known as Thelo | 19:34 | |
| *** alonma has joined #openstack-cinder | 19:34 | |
| *** changbl has quit IRC | 19:35 | |
| *** edtubill has quit IRC | 19:35 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add metadata aliases to Volume object https://review.openstack.org/247763 | 19:35 |
| *** anshul has joined #openstack-cinder | 19:39 | |
| *** alonma has quit IRC | 19:39 | |
| *** alonma has joined #openstack-cinder | 19:41 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Strengthen assertFalse assertions https://review.openstack.org/263404 | 19:41 |
| *** jwcroppe has joined #openstack-cinder | 19:41 | |
| *** edtubill has joined #openstack-cinder | 19:45 | |
| *** alonma has quit IRC | 19:45 | |
| *** dims_ has joined #openstack-cinder | 19:46 | |
| *** dims has quit IRC | 19:47 | |
| *** manous has quit IRC | 19:48 | |
| *** sheel has quit IRC | 19:49 | |
| *** sheel has joined #openstack-cinder | 19:50 | |
| *** xyang has quit IRC | 19:50 | |
| sheel | dulek: hi | 19:50 |
| sheel | for https://review.openstack.org/#/c/255947/12 | 19:51 |
| dulek | sheel: Hi, what's up? | 19:52 |
| sheel | dulek: its cool there, how are you? | 19:52 |
| sheel | dulek:what else is required for merging of https://review.openstack.org/#/c/255947/12 :) | 19:52 |
| sheel | dulek: could you please check once, codeReview got +2 but still seems no hint for merging :) | 19:54 |
| dulek | sheel: Another core reviewer +2 vote besides jungleboyj's. There are some on the channel so maybe this will raise someone's attention. | 19:54 |
| *** bardia has joined #openstack-cinder | 19:55 | |
| jungleboyj | smcginnis: sheel Would like more eyes on his patch I think. ;-) | 19:55 |
| *** martyturner has quit IRC | 19:55 | |
| sheel | smcginnis : yes, please. | 19:56 |
| patrickeast | mriedem: fyi for that log message spam from the image cache i filed https://bugs.launchpad.net/cinder/+bug/1530964 and i'll put up a fix for it soon | 19:56 |
| openstack | Launchpad bug 1530964 in Cinder "image cache is logging warnings when not enabled" [Undecided,New] - Assigned to Patrick East (patrick-east) | 19:56 |
| smcginnis | sheel, jungleboyj: Sure, running to a meeting, but I have a tab open for it now. | 19:56 |
| jungleboyj | smcginnis: Thanks boss. | 19:57 |
| mriedem | patrickeast: thanks | 19:57 |
| smcginnis | Ha, no problem. | 19:57 |
| sheel | jungleboyj dulek smcginnis: thank you | 19:58 |
| eharney | i started reading it but it might be a bit, i don't think i really understand it | 19:59 |
| *** kfarr has joined #openstack-cinder | 20:00 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-cinder | 20:01 | |
| *** martyturner has joined #openstack-cinder | 20:01 | |
| *** daneyon has joined #openstack-cinder | 20:04 | |
| sheel | eharney: are you talking about current review context? | 20:05 |
| *** cdelatte has joined #openstack-cinder | 20:05 | |
| eharney | sheel: yes | 20:05 |
| sheel | eharney : ok, let me know if you need any information about bug or fix? | 20:06 |
| eharney | sheel: not yet | 20:07 |
| sheel | eharney : ok | 20:07 |
| e0ne | why do we need deleted int? | 20:08 |
| sheel | e0ne: Actually there are 2 bugs which are fixed in this | 20:08 |
| dulek | e0ne: Look at constraint on VolumeTypeProjects model. :( That's why. | 20:09 |
| e0ne | sheel: got it, thanks | 20:09 |
| sheel | e0ne : welcome :) | 20:09 |
| e0ne | sheel: one more question to make me on the same page:) | 20:10 |
| e0ne | sheel: why do we need 'elif read_deleted == 'int_no'"? | 20:10 |
| e0ne | https://review.openstack.org/#/c/255947/12/cinder/db/sqlalchemy/api.py - it looks too compltaced now | 20:10 |
| sheel | e0ne: its too handle read_deleted for integer instead of boolean | 20:12 |
| e0ne | sheel: can we use something like oslo_utils.strutils.bool_from_string there? | 20:12 |
| sheel | e0ne:I think no | 20:14 |
| e0ne | sheel: ok, it's not a blocker now | 20:14 |
| sheel | e0ne: ok | 20:16 |
| e0ne | sheel: quick question inline. I remove my -1 once I'll get unswer | 20:16 |
| e0ne | -1 is just to not skip it | 20:16 |
| sheel | I think it will need int in model query but bool_from_string will returns to bool | 20:16 |
| *** hemna is now known as hemnafk | 20:18 | |
| *** aix has joined #openstack-cinder | 20:22 | |
| e0ne | sheel: sorry, I missed something. I've just verified your patch on my env | 20:24 |
| *** dustins has quit IRC | 20:25 | |
| *** krtaylor has quit IRC | 20:26 | |
| *** dustins has joined #openstack-cinder | 20:26 | |
| *** dustins_ has joined #openstack-cinder | 20:26 | |
| *** dustins_ has quit IRC | 20:26 | |
| *** dustins has quit IRC | 20:27 | |
| *** dustins has joined #openstack-cinder | 20:28 | |
| sheel | e0ne: thank you for your time and approval.. | 20:28 |
| e0ne | sheel: one more stupid question | 20:29 |
| *** dustins has quit IRC | 20:29 | |
| sheel | e0ne: please, tell me | 20:29 |
| e0ne | sheel: you use " soft_delete" method. where is it defined? can't find both in cinder and oslo.db | 20:29 |
| *** markstur_ has joined #openstack-cinder | 20:29 | |
| sheel | e0ne: its in oslo_db | 20:30 |
| *** dustins has joined #openstack-cinder | 20:30 | |
| *** vgridnev has quit IRC | 20:30 | |
| e0ne | sheel: thanks. github search failed:( | 20:30 |
| sheel | e0ne: https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/orm.py#L28 | 20:32 |
| e0ne | thanks | 20:32 |
| sheel | e0ne:you are welcome.. | 20:32 |
| *** kmartin has quit IRC | 20:33 | |
| *** alonma has joined #openstack-cinder | 20:41 | |
| dims_ | e0ne : here's a shiny search thingy courtesy of our infra folks - http://codesearch.openstack.org/ | 20:44 |
| e0ne | dims_: yes, it'a s great tool! | 20:44 |
| smcginnis | dims_: So much nicer than github searches! | 20:44 |
| dims_ | smcginnis : e0ne : indeed. especially digging up backward compat problems | 20:45 |
| *** alonma has quit IRC | 20:46 | |
| *** alonma has joined #openstack-cinder | 20:48 | |
| *** raildo is now known as raildo-afk | 20:51 | |
| sheel | dims_:its nice | 20:51 |
| *** alonma has quit IRC | 20:52 | |
| *** PsionTheory has joined #openstack-cinder | 20:56 | |
| *** hemnafk is now known as hemna | 20:58 | |
| *** sheel has quit IRC | 21:00 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** e0ne has quit IRC | 21:01 | |
| *** martyturner has quit IRC | 21:02 | |
| openstackgerrit | Merged openstack/python-cinderclient: Set default service type to 'volumev2' https://review.openstack.org/254876 | 21:08 |
| *** martyturner has joined #openstack-cinder | 21:10 | |
| *** merooney has quit IRC | 21:12 | |
| *** diablo_rojo has quit IRC | 21:12 | |
| *** anshul has quit IRC | 21:13 | |
| *** diablo_rojo has joined #openstack-cinder | 21:14 | |
| *** kmartin has joined #openstack-cinder | 21:17 | |
| *** martyturner has left #openstack-cinder | 21:18 | |
| *** Lee1092 has quit IRC | 21:21 | |
| *** sheel has joined #openstack-cinder | 21:22 | |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Propose new admin action to set replication status https://review.openstack.org/263444 | 21:24 |
| patrickeast | gogo last minute spec | 21:24 |
| *** julim_ has quit IRC | 21:26 | |
| *** thangp has quit IRC | 21:30 | |
| smcginnis | Wow, I haven't seen a SuSE KDE desktop in a long time. | 21:33 |
| * smcginnis is feeling disoriented | 21:33 | |
| *** akerr has quit IRC | 21:34 | |
| *** timcl has quit IRC | 21:36 | |
| ildikov | hemna: did you see my earlier question? | 21:38 |
| *** Yogi1 has quit IRC | 21:39 | |
| hemna | ildikov, no sorry, it's been a busy day | 21:39 |
| hemna | what's up | 21:39 |
| ildikov | hemna: sorry to poke you with this | 21:40 |
| hemna | no problem | 21:40 |
| ildikov | hemna: so I realized that the host_name on Cinder side is not for attaching a volume to an instance | 21:40 |
| ildikov | Cinder raises an error if both instance and host_name is specified | 21:40 |
| hemna | ildikov, didn't my patch remove that ? | 21:41 |
| ildikov | I haven't checked, but the problem is that it will not pass the tests on the gate this way | 21:41 |
| hemna | https://review.openstack.org/#/c/255595/4/cinder/api/contrib/volume_actions.py | 21:42 |
| ildikov | or at least will not really be backward compatible, I mean no upgrade path, etc. | 21:42 |
| *** cknight has quit IRC | 21:42 | |
| ildikov | cool, I remembered that part, but I wonder if it's not an issue that Nova then will work only with the latest Cinder | 21:43 |
| hemna | yah I think so | 21:43 |
| *** dustins has quit IRC | 21:44 | |
| hemna | well at least multi-attach | 21:44 |
| *** ndipanov has quit IRC | 21:45 | |
| ildikov | you mean to pass it only in case of multi-attach | 21:45 |
| hemna | I don't think so | 21:47 |
| hemna | I figure lets be consistent | 21:47 |
| hemna | the host_name field is there, we just never pass it from nova | 21:47 |
| hemna | I think only cinder adds it when it attaches a volume to itself, when there is no instance_uuid | 21:47 |
| scottda | hemna: I think Ivan's patches for attach-without-nova also uses host_name | 21:48 |
| *** alonma has joined #openstack-cinder | 21:48 | |
| ildikov | but that check is already there, I wonder how we can do it in an acceptable way | 21:48 |
| hemna | scottda, yah that would make sense | 21:49 |
| hemna | ildikov, I'm not sure really | 21:49 |
| hemna | any change is going to be an issue I think | 21:50 |
| ildikov | hemna: that's kinda sad :( | 21:50 |
| hemna | :( | 21:50 |
| hemna | I don't know what else to do. | 21:50 |
| ildikov | but anyway, my bugfix patch depends on yours anyways | 21:50 |
| ildikov | so let's get yours in and then we can fix this bug on Nova side later and add docs about it until | 21:51 |
| hemna | looks like I need to push up another patch to address some small things that john pointed out. | 21:52 |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Propose new admin action to set replication status https://review.openstack.org/263444 | 21:52 |
| hemna | this whole interaction between nova and cinder just feels like a house of cards at this point. | 21:53 |
| *** alonma has quit IRC | 21:53 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Change minimum 3PAR API version for replication https://review.openstack.org/263458 | 21:53 |
| *** alonma has joined #openstack-cinder | 21:57 | |
| *** diogogmt has quit IRC | 21:57 | |
| *** mragupat_ has joined #openstack-cinder | 21:58 | |
| *** haomaiwang has quit IRC | 22:01 | |
| ildikov | hemna: yeah, I have that feeling nowadays very often | 22:01 |
| ildikov | although just simply about this multiattach feature | 22:01 |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| hemna | ok | 22:01 |
| *** mragupat has quit IRC | 22:01 | |
| ildikov | I started to read about the hypervisors and I'm not 100% sure how it will work | 22:01 |
| hemna | so I think at a minimum we should raise this in the next Cinder meeting | 22:02 |
| *** alonma has quit IRC | 22:02 | |
| ildikov | and Nova code is a bit of a mess too at least for a new comer like me | 22:02 |
| hemna | I think that's almost always the case there.... | 22:02 |
| hemna | :P | 22:02 |
| ildikov | this bug fix issue you mean? | 22:02 |
| hemna | my original nova patch was a mess too. | 22:02 |
| hemna | ildikov, yah | 22:02 |
| ildikov | I will not disagree :) | 22:02 |
| ildikov | ok, when is that meeting? | 22:03 |
| hemna | Wed. | 22:03 |
| hemna | https://wiki.openstack.org/wiki/CinderMeetings | 22:03 |
| *** brandonmitchell has left #openstack-cinder | 22:03 | |
| *** porrua has quit IRC | 22:04 | |
| ildikov | ok, that should work | 22:05 |
| *** changbl has joined #openstack-cinder | 22:06 | |
| *** gouthamr has quit IRC | 22:07 | |
| hemna | ildikov, ok added a topic https://wiki.openstack.org/wiki/CinderMeetings#Next_meeting | 22:08 |
| ildikov | thanks much! | 22:08 |
| hemna | np | 22:08 |
| *** alexpilotti_ has joined #openstack-cinder | 22:08 | |
| ildikov | Nova non-prio feature deadline is also pretty close | 22:09 |
| ildikov | I'm trying to get review attention, but it will still be tough | 22:09 |
| hemna | so, does that mean all of the nova patches have to land before that? | 22:09 |
| ildikov | do you know anyone to ask? | 22:09 |
| hemna | do you know what the deadline date is ? | 22:09 |
| ildikov | Jan 21 IIRC | 22:09 |
| hemna | ildikov, you can ask mriedem | 22:09 |
| hemna | 21st ok | 22:10 |
| hemna | that gives us a little wiggle room | 22:10 |
| ildikov | ok, will do | 22:10 |
| ildikov | I have a +2 on the first patch, the problem comes with the second one as we call out to Cinder every time for the attachment_id as it currently not stored in Nova | 22:10 |
| hemna | url ? | 22:11 |
| ildikov | if it would be stored that requires another DB round, with db_migrate I guess, many new things... | 22:12 |
| ildikov | https://review.openstack.org/#/c/193134/49 | 22:12 |
| *** akerr has joined #openstack-cinder | 22:13 | |
| *** zhonghua-lee has quit IRC | 22:13 | |
| *** merooney has joined #openstack-cinder | 22:13 | |
| *** zhonghua-lee has joined #openstack-cinder | 22:14 | |
| mriedem | so the round trip issue is in https://review.openstack.org/#/c/193134/ | 22:14 |
| mriedem | there was a spec at one point to store the volume attachment id in the nova bdm table | 22:14 |
| hemna | oh you mean get_attachment_id inside of detach ? | 22:14 |
| mriedem | b/c the attach api currently returns the volume id for both the volume id and attachment id, which is misleading | 22:14 |
| ildikov | yeah, it was moved there recently | 22:14 |
| *** akerr_ has joined #openstack-cinder | 22:14 | |
| hemna | mriedem, ? | 22:15 |
| ildikov | mriedem: TBH it's in the multiattach spec at the moment that Nova does not store the attachment_id | 22:15 |
| *** salv-orl_ has quit IRC | 22:15 | |
| ildikov | it was accepted that way | 22:15 |
| mriedem | this https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/volumes.py#L203 | 22:15 |
| hemna | that stuff has been around for ages | 22:16 |
| mriedem | right | 22:16 |
| ildikov | people were more comfortable with that version | 22:16 |
| mriedem | and that's why we nixed a spec asking to change that | 22:16 |
| mriedem | b/c it would have required a db migration and an api microversion and no one really saw the need | 22:16 |
| mriedem | since you can get the attachments from the volume api | 22:16 |
| mriedem | and we have this policy about no new api proxy | 22:17 |
| mriedem | like george bush and no new taxes | 22:17 |
| hemna | heh | 22:17 |
| *** eharney has quit IRC | 22:18 | |
| mriedem | so it would appear there is a reason to have it now | 22:18 |
| hemna | looking back at my original nova patch, I had get_volume_attachment as a callable method in cinder.py | 22:18 |
| *** diablo_rojo has quit IRC | 22:18 | |
| *** akerr has quit IRC | 22:18 | |
| ildikov | so volume_id is used there as earlier there was no attachment_id or? | 22:18 |
| mriedem | right, so john's concern in https://review.openstack.org/#/c/193134/ is having to back to get that all the time | 22:18 |
| mriedem | ildikov: nova only stores the volume id | 22:18 |
| hemna | https://review.openstack.org/#/c/85852/2/nova/compute/manager.py | 22:19 |
| hemna | my old patch | 22:19 |
| hemna | it does make a call to fetch the volume, then get the attachment id | 22:19 |
| ildikov | mriedem: yeah, sorry, I missed that it's the translation function | 22:19 |
| hemna | I didn't see a way of getting around it w/o nova db changes :( | 22:19 |
| mriedem | yeah i see the method added here https://review.openstack.org/#/c/85852/2/nova/volume/cinder.py | 22:20 |
| hemna | and honestly, I'm a bit worried about storing cinder stuffs inside of nova | 22:20 |
| hemna | it seems to be a breakdown of the contract | 22:20 |
| *** mriedem is now known as mriedem_jedi | 22:20 | |
| hemna | cinder should be the source of truth | 22:20 |
| hemna | :) | 22:20 |
| mriedem_jedi | yeah, like the attachment status snafu's | 22:21 |
| mriedem_jedi | that andrea was trying to address | 22:21 |
| ildikov | hemna: for volume info I have to agree :) | 22:21 |
| *** mriedem_jedi is now known as mriedem | 22:21 | |
| hemna | mriedem, yup. | 22:21 |
| hemna | so, that induces a round trip. | 22:21 |
| mriedem | hemna: for neutron there is the network info cache on the instance | 22:21 |
| mriedem | which is also racy | 22:21 |
| hemna | yup | 22:22 |
| mriedem | but it gets updated in a periodic task and on network operations on the instance | 22:22 |
| hemna | lol, just have nova access the cinder db directly. | 22:22 |
| hemna | problem solved. | 22:22 |
| hemna | :P | 22:22 |
| hemna | thank you, I'll be here all week. | 22:22 |
| ildikov | how much will this be a performance issue? | 22:22 |
| *** akerr_ has quit IRC | 22:23 | |
| hemna | it's 1 extra round trip at detach time. | 22:23 |
| ildikov | sorry for the dumb questions, getting late here and it's the first Monday of the year for 37 minutes still :) | 22:23 |
| hemna | I think there were cases in nova that it already had the volume object and it didn't have to do the round trip | 22:23 |
| hemna | hrmm | 22:24 |
| ildikov | you mean I should check these round trips and see whether I might already have the info | 22:24 |
| hemna | maybe I'm wrong on that one | 22:24 |
| hemna | yah | 22:24 |
| mriedem | was just reading john's comments in https://review.openstack.org/#/c/193134/32/nova/compute/manager.py | 22:25 |
| mriedem | i haven't been reviewing this series so i'm out of context here | 22:25 |
| mriedem | but i remember that other spec about wanting to store the attachment id | 22:25 |
| hemna | which comment ? | 22:25 |
| hemna | sorry just trying to follow | 22:25 |
| ildikov | there was one version of the multiattach spec or at least comments about it | 22:25 |
| *** alexpilotti_ has quit IRC | 22:26 | |
| ildikov | but even if it made it into that particular spec it did not last long I'm sure | 22:26 |
| hemna | oh I see what he's saying | 22:27 |
| hemna | in the rpc call into the manager | 22:27 |
| *** jungleboyj has quit IRC | 22:27 | |
| hemna | you might already have pulled the cinder volume object, and hence had the attachment_id to pass in the rpc call | 22:27 |
| hemna | if that's the case, then yah that'd be better | 22:27 |
| hemna | I don't know that call chain very well, so we'd have to find that out | 22:27 |
| mriedem | yeah so you get the volume from cinder in n-api and pass the attachment id over rpc to compute, | 22:29 |
| ildikov | yeah, I might need some help on that one | 22:29 |
| ildikov | also isn't RPC versioned too in some way? | 22:29 |
| hemna | mriedem, yah I think that'd work. | 22:29 |
| mriedem | if compute doesn't get the attachment id, it's pre-mitaka code and compute has to go to cinder again | 22:29 |
| mriedem | ildikov: yea rpc is versioned | 22:29 |
| mriedem | you'd be passing a new kwarg | 22:29 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Revert "Rebrand HP XP driver to now be HPE" https://review.openstack.org/263467 | 22:29 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L470 | 22:30 |
| mriedem | so we're talking about adding an attachment_id kwarg to that right? | 22:30 |
| mriedem | if the client version is < 4.7, you wouldn't send that kwarg | 22:31 |
| hemna | yah | 22:31 |
| mriedem | b/c older compute wouldn't understand | 22:31 |
| *** ircuser-1 has joined #openstack-cinder | 22:31 | |
| *** akerr has joined #openstack-cinder | 22:32 | |
| *** jgregor has quit IRC | 22:33 | |
| *** jgregor has joined #openstack-cinder | 22:33 | |
| ildikov | I assume I can find examples there how to check the version | 22:33 |
| *** jgregor has quit IRC | 22:33 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L597 | 22:34 |
| *** e0ne has joined #openstack-cinder | 22:34 | |
| mriedem | this is a better example https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L670 | 22:34 |
| mriedem | where it's checking the version and popping kwargs | 22:34 |
| ntpttr1 | I'm not sure if it's just me locally, but since I got back from holiday tox isn't working for me on the master branch of cinder, because it fails to import tooz.coordination in cinder.tests.unit.test_coordination. Is this happening with anyone else, or is there something else I have to update? | 22:35 |
| ildikov | mriedem: ok, thanks | 22:37 |
| *** baumann has left #openstack-cinder | 22:39 | |
| ildikov | mriedem: I will check the Nova API calls and then modify that patch accordingly | 22:40 |
| ildikov | I might annoy you with stupid questions on the go, but will try not to | 22:41 |
| ildikov | thanks for the pointers | 22:41 |
| *** rlrossit has quit IRC | 22:41 | |
| mriedem | ildikov: hemna: alternatively, there is always the bdm.connection_info bag of goodies that's stored in the nova db | 22:46 |
| mriedem | it's just a json blob | 22:46 |
| mriedem | unversioned | 22:46 |
| ildikov | I saw that there is, but I didn't know it's unversioned | 22:46 |
| mriedem | but it's a way to stash an attachment id on attach, and look it up on detach | 22:46 |
| mriedem | it's mostly just what nova gets back from calling os-initialize_connection | 22:47 |
| hemna | yah, that's kinda a dump | 22:47 |
| *** akerr has quit IRC | 22:48 | |
| hemna | but I believe that's populated prior to calling cinder's attach | 22:48 |
| ildikov | mriedem: there's a _local_delete function which calls the volume_api to detach in compute api: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1833 | 22:48 |
| hemna | the cinder attach is the last call in the chain (updates cinder's DB record) | 22:48 |
| *** akerr has joined #openstack-cinder | 22:49 | |
| mriedem | ildikov: _local_delete is a separate delete path | 22:49 |
| mriedem | that doesn't go through the compute manager | 22:49 |
| mriedem | in special cases | 22:49 |
| hemna | also, fwiw, the connection_info blob gets blown away during live migration time | 22:49 |
| mriedem | like if the service is down | 22:49 |
| hemna | well, recreated | 22:49 |
| mriedem | hemna: yeah, that's another bug i think that was never fixed, | 22:50 |
| mriedem | i.e. stashing off the 'old' connection info to use in case live migration fails and we need to rollback | 22:50 |
| hemna | yup | 22:50 |
| ildikov | in that case I assume we will need to retrieve the attachment id from Cinder then | 22:50 |
| mriedem | it's actually listed as security bug | 22:50 |
| hemna | huh, no kidding | 22:50 |
| hemna | makes sense I guess. | 22:50 |
| hemna | ildikov, yes | 22:50 |
| mriedem | hemna: well, not one that anyone is fixing | 22:51 |
| mriedem | 'during live migration, gain root access to the host and make live migration fail...' | 22:51 |
| mriedem | etc etc | 22:51 |
| mriedem | so there is a bug for the rollback case, but the security bug wasn't a priority | 22:51 |
| mriedem | anywho, have to run | 22:56 |
| mriedem | busy first day back | 22:56 |
| hemna | mriedem, thanks for the help! | 22:57 |
| *** mriedem is now known as mriedem_away | 22:57 | |
| *** edtubill has quit IRC | 22:58 | |
| *** mragupat_ has quit IRC | 22:59 | |
| ildikov | hemna: I will work on the RPC stuff tomorrow | 22:59 |
| ildikov | I hope that will be a good enough solution | 23:00 |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-cinder | 23:01 | |
| hemna | ok sounds good | 23:03 |
| ildikov | I also call it a day | 23:03 |
| ildikov | as it is already tomorrow here :) | 23:03 |
| hemna | doh | 23:04 |
| hemna | ok have a good evening. | 23:04 |
| ildikov | thanks for the chat, I learnt new things today too, so all good | 23:04 |
| *** NightKhaos has quit IRC | 23:04 | |
| ildikov | thanks, brushing my teeth and sleep, no other options :) | 23:04 |
| hemna | hehe. ok night! | 23:04 |
| ildikov | have a nice day | 23:04 |
| ildikov | laters :) | 23:05 |
| *** kfarr has quit IRC | 23:05 | |
| hemna | ildikov, same to you. | 23:05 |
| *** ildikov is now known as ildikov_afk | 23:05 | |
| *** alonma has joined #openstack-cinder | 23:05 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: LeftHand: Implement un/manage snapshot support https://review.openstack.org/255015 | 23:07 |
| *** alonma has quit IRC | 23:10 | |
| *** e0ne has quit IRC | 23:11 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: 3PAR: Adding volume checks to manage snapshot API https://review.openstack.org/260728 | 23:15 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Roll back reservations quota in RPC if necessary https://review.openstack.org/263473 | 23:16 |
| *** edmondsw has quit IRC | 23:17 | |
| *** akerr has quit IRC | 23:31 | |
| *** sgotliv has quit IRC | 23:33 | |
| *** boris-42 has joined #openstack-cinder | 23:33 | |
| *** akerr has joined #openstack-cinder | 23:33 | |
| *** gouthamr has joined #openstack-cinder | 23:38 | |
| *** asselin_ has quit IRC | 23:41 | |
| *** NightKhaos has joined #openstack-cinder | 23:46 | |
| *** erlon has quit IRC | 23:46 | |
| *** angela-s has quit IRC | 23:46 | |
| *** yhayashi has joined #openstack-cinder | 23:50 | |
| *** breitz has quit IRC | 23:55 | |
| *** breitz has joined #openstack-cinder | 23:56 | |
| *** smoriya_ has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!