| *** rhagarty__ has quit IRC | 00:00 | |
| *** diablo_rojo has joined #openstack-cinder | 00:00 | |
| *** rhagarty__ has joined #openstack-cinder | 00:00 | |
| *** rhagarty__ has quit IRC | 00:00 | |
| *** rhagarty__ has joined #openstack-cinder | 00:00 | |
| *** haomaiwang has quit IRC | 00:01 | |
| *** rhagarty__ has quit IRC | 00:01 | |
| *** rhagarty__ has joined #openstack-cinder | 00:01 | |
| *** rhagarty__ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:01 | |
| *** rhagarty__ has joined #openstack-cinder | 00:01 | |
| *** rhagarty__ has quit IRC | 00:02 | |
| *** rhagarty_ has joined #openstack-cinder | 00:02 | |
| *** rhagarty_ has quit IRC | 00:02 | |
| *** rhagarty_ has joined #openstack-cinder | 00:02 | |
| *** rhagarty_ has quit IRC | 00:03 | |
| *** rhagarty_ has joined #openstack-cinder | 00:03 | |
| *** rhagarty_ has quit IRC | 00:03 | |
| *** rhagarty_ has joined #openstack-cinder | 00:03 | |
| *** rhagarty_ has quit IRC | 00:04 | |
| *** rhagarty_ has joined #openstack-cinder | 00:04 | |
| *** martyturner has quit IRC | 00:04 | |
| *** rhagarty_ has quit IRC | 00:04 | |
| *** rhagarty_ has joined #openstack-cinder | 00:04 | |
| *** rhagarty_ has quit IRC | 00:05 | |
| *** yangyapeng has quit IRC | 00:05 | |
| *** rhagarty_ has joined #openstack-cinder | 00:05 | |
| *** akshai has quit IRC | 00:05 | |
| *** rhagarty_ has quit IRC | 00:05 | |
| *** rhagarty_ has joined #openstack-cinder | 00:05 | |
| *** edmondsw has quit IRC | 00:06 | |
| *** rhagarty_ has quit IRC | 00:06 | |
| *** rhagarty_ has joined #openstack-cinder | 00:06 | |
| *** rhagarty_ has quit IRC | 00:06 | |
| *** rhagarty_ has joined #openstack-cinder | 00:06 | |
| *** rhagarty__ has joined #openstack-cinder | 00:07 | |
| *** rhagarty_ has quit IRC | 00:07 | |
| *** rhagarty__ has quit IRC | 00:07 | |
| *** rhagarty__ has joined #openstack-cinder | 00:07 | |
| *** davechen_afk has quit IRC | 00:07 | |
| *** rhagarty__ has quit IRC | 00:08 | |
| *** arch-nemesis has quit IRC | 00:13 | |
| *** diablo_rojo has quit IRC | 00:14 | |
| *** crose has quit IRC | 00:15 | |
| *** hemna is now known as hemnafk | 00:20 | |
| *** sasukeh has joined #openstack-cinder | 00:27 | |
| *** salv-orlando has joined #openstack-cinder | 00:30 | |
| *** smoriya_ has joined #openstack-cinder | 00:30 | |
| *** yhayashi has joined #openstack-cinder | 00:34 | |
| *** Julien-zte has joined #openstack-cinder | 00:37 | |
| *** tjcocozz has quit IRC | 00:37 | |
| *** bapalm has quit IRC | 00:37 | |
| *** Julien-zte has quit IRC | 00:38 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 00:38 |
|---|---|---|
| *** sheel has joined #openstack-cinder | 00:39 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:40 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 00:40 |
| *** salv-orlando has quit IRC | 00:40 | |
| *** apoorvad has quit IRC | 00:43 | |
| *** mriedem has joined #openstack-cinder | 00:44 | |
| *** wxy has joined #openstack-cinder | 00:47 | |
| *** tjcocozz has joined #openstack-cinder | 00:53 | |
| *** bapalm has joined #openstack-cinder | 00:54 | |
| *** harlowja has quit IRC | 00:55 | |
| *** gouthamr has joined #openstack-cinder | 00:57 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Report versions in cinder-manager service list https://review.openstack.org/288275 | 00:59 |
| *** gouthamr_ has joined #openstack-cinder | 00:59 | |
| *** cheneydc has joined #openstack-cinder | 01:00 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-cinder | 01:01 | |
| *** mtanino has quit IRC | 01:02 | |
| *** gouthamr has quit IRC | 01:02 | |
| openstackgerrit | zwei proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/289657 | 01:03 |
| *** jwcroppe has quit IRC | 01:04 | |
| openstackgerrit | zwei proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/289657 | 01:04 |
| *** baojg has joined #openstack-cinder | 01:06 | |
| *** sdague has joined #openstack-cinder | 01:07 | |
| *** yangyapeng has joined #openstack-cinder | 01:08 | |
| *** gcb has joined #openstack-cinder | 01:10 | |
| *** baojg has quit IRC | 01:10 | |
| *** apoorvad has joined #openstack-cinder | 01:10 | |
| openstackgerrit | zengchen proposed openstack/cinder: RPC client recreates exception instance failed https://review.openstack.org/290217 | 01:18 |
| *** Suyash has quit IRC | 01:19 | |
| openstackgerrit | wanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted https://review.openstack.org/225081 | 01:32 |
| *** haomaiwang has quit IRC | 01:32 | |
| jgriffith | smcginnis: do they not even implement extend? Guess ABC doesn't do what some thought :( | 01:33 |
| *** scohen_ has joined #openstack-cinder | 01:35 | |
| openstackgerrit | zwei proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/289657 | 01:37 |
| *** sdague has quit IRC | 01:38 | |
| *** arch-nemesis has joined #openstack-cinder | 01:38 | |
| openstackgerrit | zwei proposed openstack/python-cinderclient: Bootable filter for listening volumes from CLI https://review.openstack.org/289657 | 01:39 |
| *** scohen_ has quit IRC | 01:40 | |
| *** scohen_ has joined #openstack-cinder | 01:41 | |
| *** scohen_ is now known as scohen1 | 01:42 | |
| *** scohen1 has left #openstack-cinder | 01:43 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 01:43 |
| *** scohen_ has joined #openstack-cinder | 01:43 | |
| *** scohen_ has left #openstack-cinder | 01:43 | |
| *** arch-nemesis has quit IRC | 01:44 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Let method CONF.set_override with enfore_type=true by default https://review.openstack.org/253328 | 01:44 |
| *** sdague has joined #openstack-cinder | 01:46 | |
| *** mtanino has joined #openstack-cinder | 01:53 | |
| openstackgerrit | zengchen proposed openstack/cinder: RPC client recreates exception instance failed https://review.openstack.org/290217 | 01:55 |
| *** arch-nemesis has joined #openstack-cinder | 01:56 | |
| *** salv-orlando has joined #openstack-cinder | 01:57 | |
| openstackgerrit | Cheney proposed openstack/python-cinderclient: Parse filter item "name" correctly for snapshot-list https://review.openstack.org/290229 | 02:03 |
| *** IlyaG_ has quit IRC | 02:03 | |
| *** salv-orlando has quit IRC | 02:04 | |
| *** RustShen has joined #openstack-cinder | 02:04 | |
| openstackgerrit | zhanghao proposed openstack/os-brick: Nova and Brick remain their used iscsi sessions https://review.openstack.org/290231 | 02:04 |
| *** sdague has quit IRC | 02:05 | |
| *** Lee1092 has joined #openstack-cinder | 02:17 | |
| *** haomaiwang has joined #openstack-cinder | 02:20 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 02:21 |
| *** haomaiwang has quit IRC | 02:24 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:25 | |
| *** haomaiwa_ has quit IRC | 02:26 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:26 | |
| *** haomaiwa_ has quit IRC | 02:27 | |
| *** 20WAAIE7U has joined #openstack-cinder | 02:27 | |
| *** 20WAAIE7U has quit IRC | 02:28 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:28 | |
| *** haomaiwa_ has quit IRC | 02:29 | |
| *** haomaiwang has joined #openstack-cinder | 02:29 | |
| *** haomaiwang has quit IRC | 02:30 | |
| *** haomaiwang has joined #openstack-cinder | 02:30 | |
| *** [1]Thelo has joined #openstack-cinder | 02:30 | |
| *** haomaiwang has quit IRC | 02:31 | |
| *** haomaiwang has joined #openstack-cinder | 02:31 | |
| *** haomaiwang has quit IRC | 02:32 | |
| *** haomaiwang has joined #openstack-cinder | 02:32 | |
| *** haomaiwang has quit IRC | 02:33 | |
| *** Thelo has quit IRC | 02:33 | |
| *** [1]Thelo is now known as Thelo | 02:33 | |
| *** haomaiwang has joined #openstack-cinder | 02:33 | |
| *** haomaiwang has quit IRC | 02:34 | |
| *** yangyapeng has quit IRC | 02:44 | |
| *** cknight has joined #openstack-cinder | 02:46 | |
| *** lprice has joined #openstack-cinder | 02:48 | |
| *** lprice1 has quit IRC | 02:48 | |
| *** gcb has quit IRC | 02:52 | |
| *** jwcroppe has joined #openstack-cinder | 02:53 | |
| *** mriedem has quit IRC | 02:54 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Host selection in backup service https://review.openstack.org/290248 | 02:58 |
| *** bardia has quit IRC | 02:59 | |
| *** cknight has quit IRC | 03:07 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Run py34 tests with plain 'tox' command https://review.openstack.org/288696 | 03:07 |
| *** sasukeh has quit IRC | 03:11 | |
| *** gcb has joined #openstack-cinder | 03:13 | |
| *** haomaiwang has joined #openstack-cinder | 03:14 | |
| *** dims has quit IRC | 03:25 | |
| *** apoorvad has quit IRC | 03:27 | |
| *** markvoelker has joined #openstack-cinder | 03:30 | |
| *** ebalduf has joined #openstack-cinder | 03:31 | |
| *** salv-orlando has joined #openstack-cinder | 03:32 | |
| *** cknight has joined #openstack-cinder | 03:34 | |
| *** salv-orlando has quit IRC | 03:35 | |
| *** cknight has quit IRC | 03:37 | |
| *** haomaiwang has quit IRC | 03:40 | |
| *** haomaiw__ has joined #openstack-cinder | 03:40 | |
| *** gouthamr_ has quit IRC | 03:43 | |
| *** jwcroppe has quit IRC | 03:51 | |
| *** garthb has quit IRC | 03:52 | |
| *** jamielennox is now known as jamielennox|away | 03:54 | |
| *** sasukeh has joined #openstack-cinder | 03:59 | |
| *** links has joined #openstack-cinder | 04:03 | |
| *** shausy has joined #openstack-cinder | 04:09 | |
| *** sayalilunkad has quit IRC | 04:22 | |
| *** yangds has quit IRC | 04:30 | |
| *** sayalilunkad has joined #openstack-cinder | 04:34 | |
| jgriffith | anybody have a current count of how many drivers are in Cinder at this point? | 04:37 |
| *** fifieldt has joined #openstack-cinder | 04:47 | |
| *** sayalilunkad has quit IRC | 04:50 | |
| *** sayalilunkad has joined #openstack-cinder | 04:51 | |
| *** sasukeh has quit IRC | 04:54 | |
| *** EinstCrazy has joined #openstack-cinder | 04:57 | |
| *** lprice1 has joined #openstack-cinder | 05:01 | |
| *** lprice has quit IRC | 05:01 | |
| *** deepakcs has joined #openstack-cinder | 05:06 | |
| *** salv-orlando has joined #openstack-cinder | 05:07 | |
| DuncanT | I got 82 last time I counted for a presentation, but I wasn't being careful | 05:09 |
| *** salv-orlando has quit IRC | 05:13 | |
| *** arch-nemesis has quit IRC | 05:16 | |
| *** EinstCrazy has quit IRC | 05:28 | |
| *** bardia has joined #openstack-cinder | 05:36 | |
| *** gcb has quit IRC | 05:41 | |
| *** daneyon_ has quit IRC | 05:41 | |
| *** Poornima has joined #openstack-cinder | 05:45 | |
| *** jwcroppe has joined #openstack-cinder | 05:52 | |
| *** nkrinner has joined #openstack-cinder | 05:54 | |
| openstackgerrit | Cheney proposed openstack/python-cinderclient: Parse filter item "name" correctly for snapshot-list https://review.openstack.org/290229 | 05:58 |
| *** garthb has joined #openstack-cinder | 05:59 | |
| *** gcb has joined #openstack-cinder | 06:00 | |
| *** baojg has joined #openstack-cinder | 06:01 | |
| *** sasukeh has joined #openstack-cinder | 06:05 | |
| *** garthb has quit IRC | 06:06 | |
| *** rcernin has joined #openstack-cinder | 06:09 | |
| *** gcb has quit IRC | 06:10 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use imageutils from oslo_utils https://review.openstack.org/257328 | 06:14 |
| *** baojg has quit IRC | 06:16 | |
| *** sgotliv has joined #openstack-cinder | 06:17 | |
| *** baojg has joined #openstack-cinder | 06:19 | |
| *** laughterwym has joined #openstack-cinder | 06:29 | |
| *** asselin__ has joined #openstack-cinder | 06:29 | |
| *** asselin_ has quit IRC | 06:31 | |
| *** salv-orlando has joined #openstack-cinder | 06:31 | |
| *** laughter_ has joined #openstack-cinder | 06:32 | |
| *** baojg has quit IRC | 06:32 | |
| *** laughter_ has quit IRC | 06:32 | |
| *** laughter_ has joined #openstack-cinder | 06:34 | |
| *** laughterwym has quit IRC | 06:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/290299 | 06:35 |
| *** salv-orlando has quit IRC | 06:36 | |
| *** baojg has joined #openstack-cinder | 06:38 | |
| *** baojg has quit IRC | 06:39 | |
| *** laughter_ has quit IRC | 06:39 | |
| *** abhishekk has joined #openstack-cinder | 06:44 | |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder: Provide user friendly messages for db sync https://review.openstack.org/284156 | 06:47 |
| abhishekk | eharney: hi you around? | 06:50 |
| *** chlong has quit IRC | 06:54 | |
| *** baojg has joined #openstack-cinder | 06:59 | |
| *** baojg has quit IRC | 07:00 | |
| *** laughterwym has joined #openstack-cinder | 07:01 | |
| *** dave-mccowan has quit IRC | 07:02 | |
| openstackgerrit | javeme proposed openstack/cinder: Remove those unnecessary statements "return True" https://review.openstack.org/246805 | 07:05 |
| *** chlong has joined #openstack-cinder | 07:08 | |
| openstackgerrit | javeme proposed openstack/cinder: Encode the url parameters for XIO driver https://review.openstack.org/254157 | 07:08 |
| *** bjornar has quit IRC | 07:09 | |
| *** lpetrut has joined #openstack-cinder | 07:09 | |
| *** laughterwym has quit IRC | 07:10 | |
| *** mjura has joined #openstack-cinder | 07:12 | |
| *** laughter_ has joined #openstack-cinder | 07:13 | |
| *** sgotliv has quit IRC | 07:14 | |
| *** laughterwym has joined #openstack-cinder | 07:15 | |
| *** laughter_ has quit IRC | 07:18 | |
| openstackgerrit | Abhishek Kekane proposed openstack/cinder: volume upload-to-image fails for iso disk-format https://review.openstack.org/287606 | 07:20 |
| *** anshul has joined #openstack-cinder | 07:21 | |
| *** esker has quit IRC | 07:22 | |
| *** bardia has quit IRC | 07:23 | |
| *** nkrinner has quit IRC | 07:27 | |
| *** ildikov has quit IRC | 07:28 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Add display_name to allowed_search_options https://review.openstack.org/290312 | 07:29 |
| *** laughter_ has joined #openstack-cinder | 07:30 | |
| *** nkrinner has joined #openstack-cinder | 07:31 | |
| *** sasukeh has quit IRC | 07:32 | |
| *** laughterwym has quit IRC | 07:32 | |
| *** laughter_ has quit IRC | 07:34 | |
| *** laughterwym has joined #openstack-cinder | 07:34 | |
| *** lixiaoy1 has quit IRC | 07:36 | |
| *** laughterwym has quit IRC | 07:37 | |
| *** laughterwym has joined #openstack-cinder | 07:38 | |
| *** akolbasov has joined #openstack-cinder | 07:43 | |
| *** laughterwym has quit IRC | 07:43 | |
| *** laughterwym has joined #openstack-cinder | 07:44 | |
| *** yangyapeng has joined #openstack-cinder | 07:46 | |
| *** liverpooler has joined #openstack-cinder | 07:46 | |
| *** laughterwym has quit IRC | 07:48 | |
| openstackgerrit | Michal Jura proposed openstack/cinder: Fix failure with rbd on slow ceph clusters https://review.openstack.org/289252 | 07:49 |
| *** yangyapeng has quit IRC | 07:49 | |
| *** yangyapeng has joined #openstack-cinder | 07:49 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: rbd: Change capacity calculation from integer to float https://review.openstack.org/289394 | 07:50 |
| *** akolbasov has quit IRC | 07:52 | |
| *** ebalduf has quit IRC | 07:54 | |
| *** asselin__ has quit IRC | 07:56 | |
| *** belmoreira has joined #openstack-cinder | 07:56 | |
| *** sasukeh has joined #openstack-cinder | 07:59 | |
| *** ChubYann has quit IRC | 08:04 | |
| *** laughter_ has joined #openstack-cinder | 08:05 | |
| *** yangyapeng has quit IRC | 08:06 | |
| *** laughte__ has joined #openstack-cinder | 08:09 | |
| *** lixiaoy1 has joined #openstack-cinder | 08:09 | |
| *** e0ne has joined #openstack-cinder | 08:10 | |
| *** laughter_ has quit IRC | 08:11 | |
| *** yangyapeng has joined #openstack-cinder | 08:13 | |
| *** mjura has quit IRC | 08:13 | |
| *** lpetrut has quit IRC | 08:14 | |
| *** arecknag has joined #openstack-cinder | 08:14 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Use magic constants instead of magic numbers https://review.openstack.org/290322 | 08:14 |
| *** lpetrut has joined #openstack-cinder | 08:18 | |
| *** e0ne has quit IRC | 08:19 | |
| *** laughte__ has quit IRC | 08:20 | |
| *** laughterwym has joined #openstack-cinder | 08:21 | |
| *** laughter_ has joined #openstack-cinder | 08:22 | |
| *** laughterwym has quit IRC | 08:25 | |
| *** mjura has joined #openstack-cinder | 08:28 | |
| *** smoriya_ has quit IRC | 08:29 | |
| *** timcl has joined #openstack-cinder | 08:30 | |
| *** jwcroppe has quit IRC | 08:32 | |
| *** jwcroppe has joined #openstack-cinder | 08:32 | |
| *** timcl has quit IRC | 08:35 | |
| *** ndipanov has quit IRC | 08:35 | |
| *** zhanghao has joined #openstack-cinder | 08:38 | |
| *** sgotliv has joined #openstack-cinder | 08:38 | |
| *** lpetrut has quit IRC | 08:39 | |
| *** openstackgerrit has quit IRC | 08:47 | |
| *** openstackgerrit has joined #openstack-cinder | 08:48 | |
| *** ndipanov has joined #openstack-cinder | 08:55 | |
| *** links has quit IRC | 08:55 | |
| *** baojg has joined #openstack-cinder | 08:55 | |
| *** baojg has quit IRC | 08:59 | |
| *** ChubYann has joined #openstack-cinder | 09:02 | |
| openstackgerrit | Keiichi KII proposed openstack/os-brick: Replace _get_multipath_device_name with generic _discover_mpath_device https://review.openstack.org/290336 | 09:05 |
| openstackgerrit | Keiichi KII proposed openstack/os-brick: Replace _get_multipath_device_name with _discover_mpath_device https://review.openstack.org/290336 | 09:08 |
| *** yhayashi has quit IRC | 09:08 | |
| *** bapalm has quit IRC | 09:14 | |
| *** zhanghao has quit IRC | 09:14 | |
| *** tjcocozz has quit IRC | 09:15 | |
| *** jistr has joined #openstack-cinder | 09:19 | |
| *** ildikov has joined #openstack-cinder | 09:19 | |
| *** slunkad has joined #openstack-cinder | 09:21 | |
| *** salv-orlando has joined #openstack-cinder | 09:21 | |
| *** lixiaoy1 has left #openstack-cinder | 09:22 | |
| *** zhanghao has joined #openstack-cinder | 09:22 | |
| *** bapalm has joined #openstack-cinder | 09:27 | |
| openstackgerrit | Cheney proposed openstack/python-cinderclient: Parse filter item "name" correctly for snapshot-list https://review.openstack.org/290229 | 09:28 |
| *** salv-orlando has quit IRC | 09:29 | |
| *** zhanghao has quit IRC | 09:31 | |
| *** tjcocozz has joined #openstack-cinder | 09:31 | |
| *** jordanP has joined #openstack-cinder | 09:31 | |
| *** kambiz has quit IRC | 09:32 | |
| *** slunkad has left #openstack-cinder | 09:34 | |
| *** sayalilunkad_lap has joined #openstack-cinder | 09:35 | |
| *** kambiz has joined #openstack-cinder | 09:35 | |
| *** avishay has joined #openstack-cinder | 09:36 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods https://review.openstack.org/286598 | 09:40 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add service_nodes table and related methods https://review.openstack.org/286599 | 09:40 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add ServiceNode Versioned Object https://review.openstack.org/286600 | 09:40 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Change Job Distribution for HA A-A https://review.openstack.org/286601 | 09:40 |
| *** e0ne has joined #openstack-cinder | 09:51 | |
| *** ChubYann has quit IRC | 09:58 | |
| *** huanan_L has joined #openstack-cinder | 09:59 | |
| *** huanan has quit IRC | 09:59 | |
| *** cheneydc has quit IRC | 10:00 | |
| openstackgerrit | Sachin Yede proposed openstack/cinder: Bug id: 1514819 Message: Replaced "Openstack" with "OpenStack" in cinder-log-info.po https://review.openstack.org/290371 | 10:01 |
| *** tpsilva has joined #openstack-cinder | 10:01 | |
| *** e0ne has quit IRC | 10:01 | |
| *** baojg has joined #openstack-cinder | 10:02 | |
| *** ChubYann has joined #openstack-cinder | 10:02 | |
| *** e0ne has joined #openstack-cinder | 10:04 | |
| *** lpetrut has joined #openstack-cinder | 10:08 | |
| *** bjornar has joined #openstack-cinder | 10:09 | |
| *** laughter_ has quit IRC | 10:11 | |
| *** RustShen has quit IRC | 10:11 | |
| *** ChubYann has quit IRC | 10:12 | |
| *** lpetrut has quit IRC | 10:13 | |
| *** laughterwym has joined #openstack-cinder | 10:14 | |
| *** baojg has quit IRC | 10:14 | |
| *** laughterwym has quit IRC | 10:16 | |
| *** laughterwym has joined #openstack-cinder | 10:16 | |
| *** lpetrut has joined #openstack-cinder | 10:20 | |
| *** mvk has joined #openstack-cinder | 10:21 | |
| *** laughterwym has quit IRC | 10:22 | |
| *** laughterwym has joined #openstack-cinder | 10:23 | |
| *** laughter_ has joined #openstack-cinder | 10:24 | |
| *** laughterwym has quit IRC | 10:27 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:31 | |
| *** asselin__ has joined #openstack-cinder | 10:35 | |
| *** haomaiw__ has quit IRC | 10:45 | |
| *** salv-orlando has joined #openstack-cinder | 10:51 | |
| *** salv-orlando has quit IRC | 10:56 | |
| *** sdague has joined #openstack-cinder | 11:06 | |
| *** laughter_ has quit IRC | 11:06 | |
| *** dims has joined #openstack-cinder | 11:07 | |
| *** laughterwym has joined #openstack-cinder | 11:07 | |
| *** ildikov has quit IRC | 11:08 | |
| *** smoriya has quit IRC | 11:10 | |
| *** laughterwym has quit IRC | 11:10 | |
| *** laughterwym has joined #openstack-cinder | 11:11 | |
| *** ChubYann has joined #openstack-cinder | 11:18 | |
| *** smoriya_afk has joined #openstack-cinder | 11:19 | |
| *** takedakn has joined #openstack-cinder | 11:19 | |
| *** smoriya_afk is now known as smoriya | 11:19 | |
| *** ChubYann has quit IRC | 11:32 | |
| *** kmartin has quit IRC | 11:33 | |
| *** ociuhandu has quit IRC | 11:35 | |
| *** wxy has quit IRC | 11:39 | |
| *** ChubYann has joined #openstack-cinder | 11:41 | |
| *** yangyapeng has quit IRC | 11:42 | |
| *** chlong has quit IRC | 11:42 | |
| *** delatte has quit IRC | 11:45 | |
| *** laughter_ has joined #openstack-cinder | 11:47 | |
| *** links has joined #openstack-cinder | 11:48 | |
| *** laughterwym has quit IRC | 11:50 | |
| *** pots has quit IRC | 11:52 | |
| *** takedakn has quit IRC | 11:53 | |
| *** laughter_ has quit IRC | 11:53 | |
| *** takedakn has joined #openstack-cinder | 11:53 | |
| *** laughterwym has joined #openstack-cinder | 11:54 | |
| *** laughter_ has joined #openstack-cinder | 11:54 | |
| *** pots has joined #openstack-cinder | 11:56 | |
| *** laughterwym has quit IRC | 11:59 | |
| *** ildikov has joined #openstack-cinder | 12:04 | |
| *** avishay has quit IRC | 12:04 | |
| *** salv-orlando has joined #openstack-cinder | 12:06 | |
| *** yangds has joined #openstack-cinder | 12:10 | |
| *** chlong has joined #openstack-cinder | 12:14 | |
| *** EinstCrazy has joined #openstack-cinder | 12:15 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image https://review.openstack.org/216378 | 12:16 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 12:16 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype https://review.openstack.org/221442 | 12:16 |
| *** avishay has joined #openstack-cinder | 12:19 | |
| *** mvk has quit IRC | 12:20 | |
| *** ociuhandu has joined #openstack-cinder | 12:20 | |
| stevemar | question for cinder folks | 12:22 |
| stevemar | whats up with set_image_metadata ? | 12:23 |
| stevemar | is that the same as updating the metadata for an image via glance? | 12:23 |
| stevemar | referring to https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/volumes.py | 12:23 |
| *** lpetrut1 has joined #openstack-cinder | 12:26 | |
| *** lpetrut has quit IRC | 12:27 | |
| *** chlong has quit IRC | 12:27 | |
| *** nkrinner has quit IRC | 12:27 | |
| *** lpetrut1 is now known as lpetrut | 12:27 | |
| *** bapalm has quit IRC | 12:27 | |
| *** alaski has quit IRC | 12:27 | |
| *** takedakn has quit IRC | 12:27 | |
| *** bapalm has joined #openstack-cinder | 12:27 | |
| stevemar | is `cinder volume update --image-property` the same as `glance image update --property` ? | 12:28 |
| *** dalgaaf has joined #openstack-cinder | 12:28 | |
| stevemar | we're trying to fill the gap of missing cinder commands in OSC and were wondering if that is actually missing... https://review.openstack.org/#/c/290424/1 | 12:29 |
| *** mjura has quit IRC | 12:29 | |
| stevemar | jgriffith: smcginnis ^ | 12:29 |
| *** [1]Thelo has joined #openstack-cinder | 12:30 | |
| *** alaski has joined #openstack-cinder | 12:31 | |
| *** cdelatte has joined #openstack-cinder | 12:32 | |
| *** Thelo has quit IRC | 12:32 | |
| *** [1]Thelo is now known as Thelo | 12:32 | |
| *** nkrinner has joined #openstack-cinder | 12:40 | |
| *** chlong has joined #openstack-cinder | 12:40 | |
| *** mjura has joined #openstack-cinder | 12:44 | |
| *** Poornima has quit IRC | 12:49 | |
| *** deepakcs has quit IRC | 12:51 | |
| *** mvk has joined #openstack-cinder | 12:51 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-cinderclient: WIP: switch to keystoneauth https://review.openstack.org/290464 | 12:58 |
| *** ChubYann has quit IRC | 13:01 | |
| *** ChubYann has joined #openstack-cinder | 13:02 | |
| *** krtaylor has quit IRC | 13:02 | |
| *** dims has quit IRC | 13:05 | |
| *** dims_ has joined #openstack-cinder | 13:05 | |
| *** baojg has joined #openstack-cinder | 13:07 | |
| *** markvoelker_ has joined #openstack-cinder | 13:08 | |
| *** porrua has joined #openstack-cinder | 13:08 | |
| *** edmondsw has joined #openstack-cinder | 13:10 | |
| *** dims_ has quit IRC | 13:10 | |
| *** ankit_ag has quit IRC | 13:10 | |
| *** markvoelker has quit IRC | 13:11 | |
| DuncanT | stevemar: It's for updating the image_metadata of a bootable volume | 13:12 |
| stevemar | DuncanT: so... different ya? | 13:12 |
| *** crose has joined #openstack-cinder | 13:12 | |
| DuncanT | stevemar: The metadata is copied from glance when the volume is created | 13:12 |
| DuncanT | stevemar: but since a volume is mutable, unlike an image, you sometimes want to update it | 13:13 |
| stevemar | DuncanT: i see | 13:13 |
| DuncanT | stevemar: So similar, but different | 13:13 |
| stevemar | DuncanT: yeah, it's up there for the terrible naming award, like our tenant v project fiasco :) | 13:13 |
| stevemar | but okay, good to know it's different! | 13:13 |
| DuncanT | stevemar: Yeah, the naming sort of crept up on us | 13:14 |
| *** dims has joined #openstack-cinder | 13:14 | |
| DuncanT | stevemar: I'm considering shooting the next person who tries to call something 'metadata' in cinder | 13:15 |
| stevemar | DuncanT: ++ | 13:15 |
| stevemar | DuncanT: if you have a suggestion, let us know here: https://review.openstack.org/#/c/290424/1 -- may as well not make the same mistake twice | 13:15 |
| openstackgerrit | Merged openstack/cinder: Trim 5s+ from storwize unit tests https://review.openstack.org/289575 | 13:16 |
| *** avishay has quit IRC | 13:17 | |
| DuncanT | stevemar: stevemar Not sure what else you can call it... maybe boot_metadata rather than image_metadata... cinder already calls it image_metadata in a bunch of APIs though so hard to change | 13:17 |
| stevemar | DuncanT: boot-property? | 13:17 |
| stevemar | DuncanT: in osc-land we definitely do not use the word metadata anywhere | 13:18 |
| stevemar | we have a strict policy on that :) | 13:18 |
| DuncanT | stevemar: What do you remap the volume metadata field to then? | 13:18 |
| stevemar | DuncanT: property | 13:19 |
| DuncanT | stevemar: boot-property might be good I guess | 13:19 |
| DuncanT | stevemar: That's going to get confusing, since it is called metadata in the api output, in horizon, in python-cinderclient, in the cinder docs and in searchlight | 13:20 |
| *** yangyapeng has joined #openstack-cinder | 13:20 | |
| stevemar | DuncanT: eek... | 13:20 |
| *** markvoelker_ has quit IRC | 13:20 | |
| stevemar | sheel: not sure if you can read the scrollback, but DuncanT and i talked about your patch | 13:21 |
| sheel | stevemar: i got some | 13:21 |
| stevemar | sheel: maybe calling it --boot-property will help differentiate it from --property | 13:21 |
| *** martyturner has joined #openstack-cinder | 13:21 | |
| sheel | I think its ok | 13:21 |
| DuncanT | stevemar: So I'd pick fault with using --property, and might file a bug... OSC shouldn't be causing confusion with every other consume IMO | 13:22 |
| * stevemar shrugs | 13:22 | |
| stevemar | DuncanT: every project calls it something different | 13:22 |
| *** dave-mccowan has joined #openstack-cinder | 13:22 | |
| stevemar | we picked one and went with it | 13:22 |
| *** baojg has quit IRC | 13:22 | |
| stevemar | DuncanT: that's a battle to fight with dtroyer | 13:23 |
| sheel | DuncanT: i agree on this but osc already name it property for metadata.. | 13:24 |
| sheel | stevemar: :) | 13:24 |
| DuncanT | sheel: stevemar: Is the meaning of 'property' even consistent between projects? | 13:25 |
| DuncanT | I get the feeling you're trying to make things equivelent that really aren't | 13:25 |
| sheel | DuncanT: right | 13:25 |
| sheel | DuncanT: any suggestion on name? | 13:26 |
| sheel | stevemar: may be we have to go back and rename property as well then | 13:26 |
| sheel | stevemar: :) | 13:26 |
| DuncanT | pyton-cinderclient, the rest api, horizon, searchlight, the docs and a couple of other places already call them 'metadata' and 'image-metadata'... for better or worse, that name is baked in now | 13:27 |
| sheel | DuncanT: exactly, i am with it and same name should be used everywhere | 13:29 |
| stevemar | DuncanT: file a bug with osc if it makes you unhappy, we can mark is as wishlist for now and if folks pile on then we can slowly make the change | 13:30 |
| sheel | stevemar: but may be we have to talk with dtroyer about why we named it properry in past | 13:30 |
| sheel | stevemar: i will work to update it if all agrees | 13:30 |
| *** timcl has joined #openstack-cinder | 13:30 | |
| DuncanT | stevemar: will do. I'll ping dtroyer with the bug number once it is filed | 13:31 |
| stevemar | DuncanT: cool | 13:31 |
| stevemar | DuncanT: dtroyer will have the history for why we did it in the first place. Guessing since osc isn't so widely adopted just yet, we could change things up now before it's too late | 13:31 |
| * DuncanT wonders why python-openstackclient pulls in python-mock as an install dependecy, by and by | 13:32 | |
| stevemar | whaaaa | 13:32 |
| DuncanT | stevemar: Making them aliases for each other might actually be the best idea | 13:32 |
| stevemar | https://github.com/openstack/python-openstackclient/blob/master/requirements.txt | 13:32 |
| stevemar | DuncanT: yeah, was thinking that | 13:32 |
| stevemar | no mock there :O | 13:33 |
| *** baojg has joined #openstack-cinder | 13:33 | |
| DuncanT | stevemar: I'm just looking at the .deb - it's an indirect dependecy, so no openstack client's fault | 13:33 |
| *** akerr has joined #openstack-cinder | 13:34 | |
| DuncanT | stevemar: Just looking at the spec I started this week for volume groups, it has both metadata and properties as different things.... I'll change that before I create a monster | 13:34 |
| *** links has quit IRC | 13:35 | |
| stevemar | DuncanT: yay! | 13:37 |
| stevemar | DuncanT: yay x2 actually | 13:37 |
| DuncanT | stevemar: fortunately I haven't published it yet. I need a new name for things that add arbitrary functionality, but some from a known set and have strong semantic meanings | 13:38 |
| *** markvoelker has joined #openstack-cinder | 13:39 | |
| DuncanT | stevemar: Examples of these are 'consistency group' 'replication' 'fast snapshot' 'fast backup' and other properties a subset of volumes might have on a system | 13:40 |
| DuncanT | Features maybe? | 13:40 |
| *** tsufiev has quit IRC | 13:41 | |
| sheel | options, settings, | 13:44 |
| sheel | DuncanT: or may be atribute | 13:46 |
| DuncanT | attribute could be a good one | 13:47 |
| sheel | DuncanT: :) | 13:47 |
| *** rlrossit has joined #openstack-cinder | 13:48 | |
| *** laughter_ has quit IRC | 13:48 | |
| *** laughterwym has joined #openstack-cinder | 13:51 | |
| *** gouthamr has joined #openstack-cinder | 13:52 | |
| *** akshai has joined #openstack-cinder | 13:55 | |
| *** salv-orlando has quit IRC | 13:55 | |
| *** markvoelker has quit IRC | 13:58 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 14:07 |
| openstackgerrit | Merged openstack/cinder: Remove those unnecessary statements "return True" https://review.openstack.org/246805 | 14:07 |
| *** salv-orlando has joined #openstack-cinder | 14:07 | |
| *** jwcroppe has quit IRC | 14:10 | |
| *** salv-orlando has quit IRC | 14:14 | |
| *** fifieldt has quit IRC | 14:16 | |
| *** dustins has joined #openstack-cinder | 14:18 | |
| *** mriedem has joined #openstack-cinder | 14:19 | |
| *** mtanino has quit IRC | 14:19 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Check before delete host https://review.openstack.org/282950 | 14:20 |
| *** xyang1 has joined #openstack-cinder | 14:21 | |
| *** tsufiev has joined #openstack-cinder | 14:23 | |
| krotscheck | Heya everyone. I've got a mitaka bug up for review at https://review.openstack.org/#/c/286795/ - Are there any cores around to review this? | 14:24 |
| *** Guest40585 has joined #openstack-cinder | 14:29 | |
| *** Guest40585 has quit IRC | 14:29 | |
| openstackgerrit | AMIT KUMAR proposed openstack/cinder: Setting the value of limit argument from the request. https://review.openstack.org/290533 | 14:32 |
| *** liu has joined #openstack-cinder | 14:32 | |
| *** jwcroppe has joined #openstack-cinder | 14:32 | |
| *** laughterwym has quit IRC | 14:37 | |
| *** laughterwym has joined #openstack-cinder | 14:37 | |
| *** laughter_ has joined #openstack-cinder | 14:39 | |
| *** crose has quit IRC | 14:40 | |
| *** jwcroppe has quit IRC | 14:40 | |
| *** laughterwym has quit IRC | 14:42 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid UUID warnings for test_volume_* https://review.openstack.org/290541 | 14:42 |
| *** laughterwym has joined #openstack-cinder | 14:43 | |
| *** laughter_ has quit IRC | 14:43 | |
| *** merooney has joined #openstack-cinder | 14:46 | |
| *** laughterwym has quit IRC | 14:47 | |
| flip214 | DuncanT: may I ask you for a review of https://review.openstack.org/#/c/287807? Already got a +2... | 14:47 |
| *** mmandel has joined #openstack-cinder | 14:47 | |
| *** knikolla has joined #openstack-cinder | 14:47 | |
| *** markvoelker has joined #openstack-cinder | 14:48 | |
| e0ne | smcginnis: hi! why wo we use launchpad os-brick project only for bugs (http://bugs.launchpad.net/os-brick)? I'm looking where I should create a blueprint for it | 14:48 |
| *** igornsa_ has joined #openstack-cinder | 14:48 | |
| *** haomaiwang has joined #openstack-cinder | 14:49 | |
| e0ne | s/wo we/do we | 14:49 |
| *** ivase has joined #openstack-cinder | 14:50 | |
| *** jgregor has joined #openstack-cinder | 14:51 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Fix docstring according to function https://review.openstack.org/290549 | 14:51 |
| *** mriedem is now known as mriedem_meeting | 14:52 | |
| *** frontrunner has joined #openstack-cinder | 14:54 | |
| *** igornsa_ has quit IRC | 14:55 | |
| *** baumann has joined #openstack-cinder | 14:59 | |
| *** lprice1 has quit IRC | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 15:01 | |
| *** cknight has joined #openstack-cinder | 15:03 | |
| *** xek_ is now known as xek | 15:05 | |
| *** mtanino has joined #openstack-cinder | 15:06 | |
| *** BigWillie has joined #openstack-cinder | 15:07 | |
| *** ebalduf has joined #openstack-cinder | 15:08 | |
| *** baojg has quit IRC | 15:12 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Set expected_attrs in volume obj to what we join https://review.openstack.org/290569 | 15:12 |
| smcginnis | e0ne: We should probably track them there. | 15:13 |
| smcginnis | Not sure how to get that changed to allow it though. | 15:13 |
| smcginnis | e0ne: Digging through now to see how I enable it. | 15:13 |
| *** liu has quit IRC | 15:13 | |
| *** edtubill has joined #openstack-cinder | 15:14 | |
| anteaya | smcginnis: when you have a chance can you take a look at a patch I have up to governance regarding ptls leave of absence if they are required to take one: https://review.openstack.org/#/c/290141/2 | 15:14 |
| smcginnis | anteaya: Sure, thanks for the ping! | 15:15 |
| anteaya | thank you | 15:15 |
| *** baojg has joined #openstack-cinder | 15:15 | |
| *** merooney has quit IRC | 15:16 | |
| *** diablo_rojo has joined #openstack-cinder | 15:18 | |
| openstackgerrit | Ice Yao proposed openstack/cinder: Add judgement of backing-up volume status https://review.openstack.org/288875 | 15:18 |
| *** laughterwym has joined #openstack-cinder | 15:20 | |
| *** arch-nemesis has joined #openstack-cinder | 15:26 | |
| *** merooney has joined #openstack-cinder | 15:27 | |
| *** edtubill has quit IRC | 15:29 | |
| *** sasukeh has quit IRC | 15:30 | |
| *** baojg has quit IRC | 15:33 | |
| *** sasukeh has joined #openstack-cinder | 15:33 | |
| *** sasukeh has quit IRC | 15:34 | |
| *** sasukeh has joined #openstack-cinder | 15:34 | |
| *** lprice has joined #openstack-cinder | 15:34 | |
| *** darosale has joined #openstack-cinder | 15:35 | |
| *** edtubill has joined #openstack-cinder | 15:35 | |
| *** kprabhu has joined #openstack-cinder | 15:36 | |
| *** krtaylor has joined #openstack-cinder | 15:37 | |
| *** sasukeh has quit IRC | 15:39 | |
| *** ildikov has quit IRC | 15:40 | |
| *** mriedem_meeting is now known as mriedem | 15:40 | |
| openstackgerrit | Michal Jura proposed openstack/cinder: Put cinder.conf options in sorted order https://review.openstack.org/269503 | 15:43 |
| *** nkrinner has quit IRC | 15:46 | |
| *** sayalilunkad_lap has quit IRC | 15:48 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add SnapshotStatus enum field https://review.openstack.org/284825 | 15:48 |
| lpetrut | Hi guys, I have a question related to the oslo.versionedobjects transition. Are all the driver methods now receiving versioned objects instead of sqlalchemy models? | 15:49 |
| *** jseiler has joined #openstack-cinder | 15:49 | |
| *** sayalilunkad_lap has joined #openstack-cinder | 15:49 | |
| *** avishay has joined #openstack-cinder | 15:53 | |
| lpetrut | hmm, I guess not: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1286 | 15:53 |
| e0ne | lpetrut: looks like you found the answer to your question, didn't you? | 15:55 |
| *** kmartin has joined #openstack-cinder | 15:55 | |
| lpetrut | yep, I guess so. Is there a reason we don't pass a versioned object here? | 15:55 |
| *** EinstCrazy has quit IRC | 15:56 | |
| *** liverpooler has quit IRC | 15:56 | |
| lpetrut | basically, this causes us some issues because we don't know what to expect regarding the volume object attributes. note that the volume type extra specs will be either a dict, either a list of VolumeTypeExtraSpecs objects, depending on what the manager passes to the driver. | 15:57 |
| *** cFouts is now known as cfouts | 15:59 | |
| e0ne | lpetrut: I would like to ask it ^^ to geguileo and dulek | 15:59 |
| *** Poornima has joined #openstack-cinder | 16:00 | |
| *** haomaiwa_ has quit IRC | 16:01 | |
| geguileo | lpetrut: As you have noticed, not everything has been changed | 16:01 |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| lpetrut | geguileo: would it be safe to just pass a versioned object? | 16:02 |
| geguileo | lpetrut: We'd have to confirm, but it should be safe | 16:02 |
| *** zzzeek has quit IRC | 16:03 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove "sqlite_clean_db" option https://review.openstack.org/290618 | 16:03 |
| *** mjura has quit IRC | 16:03 | |
| *** vincent_hou has joined #openstack-cinder | 16:03 | |
| *** adrianofr_ has joined #openstack-cinder | 16:03 | |
| *** zzzeek has joined #openstack-cinder | 16:04 | |
| geguileo | lpetrut: Looks like the copy_volume_to_image has not been moved to VO yet | 16:04 |
| geguileo | lpetrut: We should move it to VO | 16:04 |
| lpetrut | geguileo: that's the only left, right? | 16:04 |
| *** gcb has joined #openstack-cinder | 16:05 | |
| geguileo | lpetrut: Probably not... lol | 16:05 |
| geguileo | lpetrut: But I haven't gone through all the APIs checking | 16:05 |
| lpetrut | geguileo: I can take a quick look to see where sqlalchemy models are still used. I may send a patch to see what the CIs say about it, if we pass VOs all the time. | 16:06 |
| geguileo | lpetrut: Ping me once you have a patch and I'll help with the reviews | 16:07 |
| lpetrut | geguileo: it would be great to have the manager consistent in what it passes to the drivers, while the drivers would not have to do extra checks on what they receive | 16:07 |
| lpetrut | geguileo: sure, thanks a lot for your help guys | 16:07 |
| geguileo | lpetrut: I agree, and we are working towards it | 16:07 |
| geguileo | lpetrut: Now with your help we'll be a step closer :-) | 16:07 |
| *** Poornima has quit IRC | 16:08 | |
| *** merooney has quit IRC | 16:11 | |
| *** cbinx has joined #openstack-cinder | 16:13 | |
| *** hemnafk is now known as hemna | 16:15 | |
| *** jwcroppe has joined #openstack-cinder | 16:15 | |
| *** Poornima has joined #openstack-cinder | 16:16 | |
| *** shausy has quit IRC | 16:16 | |
| *** gcb has quit IRC | 16:17 | |
| *** mmotiani has joined #openstack-cinder | 16:18 | |
| *** jungleboyj has joined #openstack-cinder | 16:18 | |
| Swanson | failover host is claiming "Host replication_status must be enabled or failed-over to failover." Is there anything other than replication_enabled that I need to be setting? | 16:19 |
| *** BigWillie has quit IRC | 16:20 | |
| *** arecknag has quit IRC | 16:21 | |
| *** jwcroppe has quit IRC | 16:23 | |
| *** chris_morrell has joined #openstack-cinder | 16:27 | |
| *** bjornar has quit IRC | 16:27 | |
| *** edtubill has quit IRC | 16:30 | |
| *** gcb_ has joined #openstack-cinder | 16:30 | |
| *** edtubill has joined #openstack-cinder | 16:31 | |
| *** Poornima has quit IRC | 16:31 | |
| *** Poornima_K has joined #openstack-cinder | 16:31 | |
| *** Poornima_K is now known as Poornima | 16:31 | |
| *** Poornima has quit IRC | 16:32 | |
| smcginnis | hemna: Any idea how to enabled blueprints on https://launchpad.net/os-brick | 16:32 |
| *** anshul has quit IRC | 16:32 | |
| *** dims has quit IRC | 16:33 | |
| aorourke | Swanson, that should be all you need to do. | 16:33 |
| aorourke | Swanson, but replication_enabled cannot be reported in a pool | 16:33 |
| *** laughterwym has quit IRC | 16:33 | |
| hemna | smcginnis, err, I think the project might be borked. I dunno | 16:33 |
| smcginnis | hemna: I don't see any kind of settings for that. | 16:34 |
| smcginnis | Not a big fan of launchpad. | 16:34 |
| *** sasukeh has joined #openstack-cinder | 16:35 | |
| jgriffith | aorourke: why not? | 16:35 |
| aorourke | jgriffith, it didn't work last time i tried it | 16:35 |
| aorourke | might have changed though | 16:35 |
| jgriffith | aorourke: IIRC pools are presented just like any other c-vol | 16:35 |
| jgriffith | aorourke: just the extra suffix, no? Maybe that's changed? | 16:36 |
| aorourke | jgriffith, i would have to try it again to confirm | 16:36 |
| jgriffith | aorourke: hmm... I don't have anything that does pools so I'm useless to you :) | 16:36 |
| *** chris_morrell has quit IRC | 16:37 | |
| jgriffith | hemna: did you never set up a team page? https://launchpad.net/~os-brick ? | 16:38 |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: Pass RBD order to clone call https://review.openstack.org/289987 | 16:38 |
| sheel | smcginnis: hi | 16:40 |
| *** sasukeh has quit IRC | 16:40 | |
| sheel | smcginnis: are we planning https://review.openstack.org/#/c/285293/ in mitaka? | 16:40 |
| sheel | stevemar: its related to microversioning... | 16:41 |
| sheel | sorry it was for smcginnis | 16:41 |
| sheel | smcginnis: its related to microversioning.. | 16:41 |
| *** gcb_ has quit IRC | 16:41 | |
| Swanson | aorourke, No pools. No failover joy. The host on the failover-host command is the full string from cinder service-list, right? | 16:42 |
| Swanson | Not seeing anything outside of c-api... | 16:42 |
| *** chris_morrell has joined #openstack-cinder | 16:42 | |
| patrickeast | Swanson: whats the output of service-list --withreplication ? | 16:42 |
| patrickeast | Swanson: is it showing up as enabled for replication? | 16:42 |
| aorourke | Swanson, yes. and with what patrickeast said, you can see the replicaiton states of each host | 16:43 |
| *** gcb_ has joined #openstack-cinder | 16:43 | |
| Swanson | Not-capable | 16:43 |
| Swanson | Well. There. | 16:43 |
| patrickeast | Swanson: mm thats your problem right there | 16:43 |
| _alastor_ | Hey folks, is there a maximum amount of time that Nova/Cinder will wait for a request to complete from a driver? Is that documented somewhere? | 16:44 |
| Swanson | I hate replication. | 16:44 |
| *** dims has joined #openstack-cinder | 16:44 | |
| Swanson | Okay. Well, where on earth does that get set? | 16:44 |
| patrickeast | Swanson: whats your stats update look like? | 16:44 |
| aorourke | Swanson, replication_enabled must not be set correctly then? | 16:44 |
| *** chris_morrell has quit IRC | 16:44 | |
| patrickeast | Swanson: the manager looks at the stats/capabilities on driver init | 16:44 |
| scottda | smcginnis: sheel To be clear, That fix for microversions is not currently used by anyone, but it is a dependency for sheel's bootable filter fix: https://review.openstack.org/#/c/278605/ | 16:45 |
| sheel | scottda: right.. :) | 16:45 |
| patrickeast | _alastor_: there are some commands that can time out from the rpc's, they are configurable though, and some of the like 'orchestrated' stuff from nova that calls cinder can fail with some other configurable timeouts that nova will wait for things to become available | 16:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/288344 | 16:46 |
| scottda | _alastor_: I think most/all calls from Nova down to the Cinder driver are aysnc. | 16:46 |
| patrickeast | _alastor_: not sure if there is one nice place where they are all documented | 16:46 |
| *** chris_morrell has joined #openstack-cinder | 16:46 | |
| patrickeast | scottda: +1 | 16:46 |
| *** avishay has quit IRC | 16:46 | |
| jgriffith | Swanson: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L536 | 16:46 |
| *** akshai has quit IRC | 16:46 | |
| *** yuriy_n17 has quit IRC | 16:47 | |
| scottda | _alastor_: But Nova can timeout waiting for state in Cinder to change, as patrickeast has said. | 16:47 |
| _alastor_ | patrickeast: Ok. I'm looking into implementing some retry logic in a driver and I'm trying to figure out the maximum amount of time I can take for retries before Cinder starts complaining. | 16:47 |
| *** yuriy_n17 has joined #openstack-cinder | 16:47 | |
| jgriffith | _alastor_: http://docs.openstack.org/developer/cinder/devref/attach_detach_conventions.html | 16:47 |
| Swanson | OMG paste is slow! | 16:48 |
| jgriffith | _alastor_: there are the old "nova volume-create/delete" calls in there still too but honestly the only Nova specific calls are those listed in attach/detach sequence | 16:48 |
| jgriffith | _alastor_: that doc has all of them | 16:48 |
| *** akshai has joined #openstack-cinder | 16:49 | |
| _alastor_ | jgriffith: Thanks | 16:49 |
| jgriffith | _alastor_: you're most welcome | 16:49 |
| *** e0ne has quit IRC | 16:50 | |
| *** chris_morrell has quit IRC | 16:50 | |
| *** belmoreira has quit IRC | 16:52 | |
| patrickeast | jgriffith: oh, speaking of timeouts... | 16:52 |
| Swanson | Well, this shows up first in the list http://paste.openstack.org/show/489878/ | 16:52 |
| * jgriffith runs | 16:52 | |
| *** IlyaG has joined #openstack-cinder | 16:52 | |
| *** wolsen has quit IRC | 16:52 | |
| patrickeast | jgriffith: remember that time i was like 'oh yea, failover can totally be call and we'll return the id and blah blah' | 16:52 |
| *** wolsen has joined #openstack-cinder | 16:52 | |
| jgriffith | Swanson: just trap on init int he manager and see if you're setting it or not | 16:52 |
| patrickeast | jgriffith: i miiiiight be changing my mind :( | 16:52 |
| jgriffith | patrickeast: oh suuuure!!! | 16:52 |
| jgriffith | patrickeast: see how you are! | 16:53 |
| patrickeast | haha | 16:53 |
| aorourke | jgriffith, I dumped stats in this case (https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L536) and when you have replicaiton_enabled in the pool, it does not register it | 16:53 |
| jgriffith | patrickeast: rpc timeouts on the faiover op in the driver? | 16:53 |
| patrickeast | jgriffith: so it works fine for like a reasonably small number of volumes, which was my thinking from the past replication stuff | 16:53 |
| patrickeast | jgriffith: yea | 16:53 |
| patrickeast | jgriffith: but if like the backend has thousands | 16:53 |
| *** haomaiwang has quit IRC | 16:53 | |
| patrickeast | jgriffith: in my particular case i make 1 rest api call per volume | 16:54 |
| patrickeast | jgriffith: it adds up.. | 16:54 |
| jgriffith | patrickeast: yeah, I thought of that when I looked at avishay's patch the other day :( | 16:54 |
| jgriffith | patrickeast: which sucks, because the only way to fix it is make that sync then introduce a new call to get backend-id | 16:54 |
| jgriffith | :( | 16:54 |
| patrickeast | jgriffith: granted i can always just put in the docs that if you have lots of volumes and you plan to failover just bump the rpc timeout up | 16:54 |
| jgriffith | patrickeast: yeah, but that SUCKS | 16:55 |
| *** jdurgin1 has joined #openstack-cinder | 16:55 | |
| patrickeast | jgriffith: why a new call? we save it in the db and you see it with service-list | 16:55 |
| patrickeast | jgriffith: we just wont return it | 16:55 |
| jgriffith | patrickeast: what we really need is individual rpc timeout values :) | 16:55 |
| jgriffith | patrickeast: Ahh... yeah, that would work | 16:55 |
| jgriffith | patrickeast: dErP | 16:55 |
| jgriffith | patrickeast: ok, you want to do the honors or shall I? | 16:56 |
| *** EinstCrazy has joined #openstack-cinder | 16:56 | |
| patrickeast | jgriffith: i can pick this one up if you want | 16:56 |
| patrickeast | i've got some time open | 16:56 |
| patrickeast | today* | 16:56 |
| jgriffith | patrickeast: that would be super awesome if you don't mind | 16:56 |
| * jgriffith gets easily distracted as it is | 16:57 | |
| patrickeast | haha | 16:57 |
| jgriffith | SQUIRELL!!! | 16:57 |
| jgriffith | jungleboyj: smcginnis did you guys see my post on the ML regarding test length? | 16:57 |
| jgriffith | jungleboyj: smcginnis more specifically the point that mtreinish noted about "we already have that" | 16:58 |
| jgriffith | jungleboyj: smcginnis it's just set crazy high be default. So we can just modify it. I was going to push a patch but want to wait until we get through RC's | 16:58 |
| jgriffith | incase there are weird timing issues/failures that show up in the gate after dropping it down | 16:59 |
| jgriffith | I'll push the patch as WIP for now and we can just merge after things settle | 16:59 |
| *** gcb_ has quit IRC | 16:59 | |
| dulek | lpetrut: Sorry, meeting. To access volume metadata in a consistent way you should use volume.volume_metadata. | 17:00 |
| *** timcl has quit IRC | 17:00 | |
| *** haomaiwang has joined #openstack-cinder | 17:00 | |
| jgriffith | metadata should die in a fire | 17:00 |
| lpetrut | dulek: no worries. what about volume type extra specs ? :) | 17:00 |
| * jgriffith is just sayin | 17:00 | |
| dulek | lpetrut: That will be list of SQLAlchemy objects in SQLAlchemy and list of MetadataObject in o.vo. | 17:00 |
| dulek | lpetrut: Ooooh… | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| dulek | lpetrut: I've never looked into that after thangp implemented it… | 17:01 |
| dulek | lpetrut: Awww, this sucks so much - looks like we don't have a consistent implementation between SQLA and o.vo… | 17:02 |
| lpetrut | yep | 17:02 |
| lpetrut | dulek: so, your patch fixed the issue for volume metadata (https://review.openstack.org/#/c/247763/), but the volume type extra specs are still inconsistent | 17:03 |
| *** EinstCrazy has quit IRC | 17:03 | |
| dulek | lpetrut: Yeah. | 17:03 |
| dulek | lpetrut: At this stage of the release I would say that safest idea would be to probe it in the driver to see what's inside… :( | 17:04 |
| lpetrut | so, I thought that rather than doing some checks on the driver side, it would be better to make sure we always pass VOs to the drivers | 17:04 |
| dulek | lpetrut: With a FIXME comment. :( | 17:04 |
| lpetrut | got it | 17:04 |
| dulek | lpetrut: I don't know if we won't break other drivers if we switch it suddenly. | 17:04 |
| dulek | Or is it the only place you're getting an SQLA obj? | 17:05 |
| *** vincent_hou has quit IRC | 17:05 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Modify default timeout for testr to 20 seconds https://review.openstack.org/290695 | 17:05 |
| dulek | lpetrut: It probably isn't. | 17:05 |
| Swanson | hmm... restarted the services and replication was enabled. Neato. Something blew up, tho.... | 17:06 |
| *** BigWillie has joined #openstack-cinder | 17:06 | |
| *** jistr has quit IRC | 17:06 | |
| dulek | lpetrut: https://review.openstack.org/#/c/263765/ | 17:07 |
| lpetrut | dulek: yeah, I guess there may be some leaked sqla objects in a few other places. I thought about trying to find this kind of situations, simply try to replace those with VOs, send a patch and see what the CIs say about it. But as you said, it may a bit to late for this release to make such a change | 17:07 |
| dulek | lpetrut: That's abandoned just because DuncanT did that to discourage people to put work in patches unmerge'able in this state. | 17:07 |
| *** laughterwym has joined #openstack-cinder | 17:08 | |
| *** haomaiwa_ has joined #openstack-cinder | 17:08 | |
| dulek | lpetrut: Yes, its late. I'm deeply ashamed that we have inconsistent driver interfaces, sorry. :( | 17:08 |
| lpetrut | dulek: thanks for mentioning that patch (actually chain of patches). | 17:08 |
| dulek | lpetrut: https://review.openstack.org/#/q/topic:bp/cinder-objects | 17:09 |
| *** apoorvad has joined #openstack-cinder | 17:09 | |
| *** rcernin has quit IRC | 17:09 | |
| dulek | lpetrut: Don't care if any of them is abandoned, that was purely procedural. | 17:09 |
| lpetrut | dulek: that's great. I guess I should first check for existing patches/bps before searching through the code/submitting a new one :) | 17:09 |
| lpetrut | dulek: well, I'll take care of this at the driver level for now. Thanks a lot for the info | 17:10 |
| openstackgerrit | Merged openstack/cinder: Fix failure with rbd on slow ceph clusters https://review.openstack.org/289252 | 17:11 |
| dulek | lpetrut: Thank you for finding this problem. I'll watch to sort it out in Newton. | 17:11 |
| *** laughterwym has quit IRC | 17:12 | |
| *** merooney has joined #openstack-cinder | 17:14 | |
| jungleboyj | jgriffith: So, the idea is, if your test runs more than 20 seconds it is going to fail with a timeout? | 17:20 |
| jgriffith | jungleboyj: yes | 17:20 |
| *** bjornar has joined #openstack-cinder | 17:20 | |
| jgriffith | jungleboyj: you an test it by setting that env variable to something like5 seconds | 17:21 |
| jungleboyj | Sounds like an easy enough solution. | 17:21 |
| jgriffith | jungleboyj: yeah, mtreinish pointed it out... I completely forgot that setting even existed | 17:23 |
| mtreinish | jungleboyj, jgriffith: well you don't want to get too aggresive setting that you have to account for certain amount of jitter in the run time | 17:24 |
| jgriffith | mtreinish: hehe | 17:25 |
| jungleboyj | jgriffith: Do you want us to merge this after it passes CI or hold it for a while. | 17:25 |
| jungleboyj | mtreinish: I wondered about that. | 17:25 |
| jgriffith | mtreinish: yeah, I doubled our average longest test time https://review.openstack.org/#/c/290695/1 | 17:25 |
| jgriffith | mtreinish: if that doesn't account for *jitter* we've got problems IMO :) | 17:25 |
| jgriffith | jungleboyj: I think it's safe, but I certainly don't mind if people want to hold it for a bit and do some stats gathering. Or wait until after RC's go out | 17:26 |
| jungleboyj | jgriffith: I will defer to smcginnis . :-) | 17:27 |
| Swanson | Hmmm.... getting a raise on failover-host. Also, I'm only getting replication status enabled after I restart c-vol.... | 17:27 |
| *** sayalilunkad_lap has quit IRC | 17:27 | |
| smcginnis | jgriffith: Any concerns running unit tests on a crappy machine? | 17:28 |
| smcginnis | Might be interesting to spin up a resource constrained machine and see if we run into trouble. | 17:28 |
| jgriffith | smcginnis: I can test that if you like, but frankly the only crappy machine I care about is the infra gate Instances :) | 17:28 |
| smcginnis | jgriffith: Yeah, true. I would hope most don't have problems running it | 17:29 |
| jungleboyj | :-) | 17:29 |
| smcginnis | I'm interested in seeing the gate results on that. | 17:29 |
| patrickeast | Swanson: its only enabled after restart the first time? or like every other time kind of thing? | 17:29 |
| smcginnis | I think it should be fine. | 17:29 |
| jungleboyj | I don't think we have too many test cases that are slowed by resource constraints. jgregor fixed some of those up a few weeks ago. | 17:30 |
| Swanson | patrickeast, I havent checked. I don't allow failback so I kinda have to restack between. | 17:30 |
| jgriffith | Swanson: painful, just stub in a response for failback to make your life easier :) | 17:31 |
| *** sgotliv has quit IRC | 17:31 | |
| patrickeast | Swanson: or just go edit the db | 17:32 |
| Swanson | patrickeast, jgriffith This https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L536 isn't being called until I restart c-vol. | 17:32 |
| patrickeast | Swanson: pretty easy to just change the service table | 17:32 |
| jgriffith | Swanson: do you want/need my lvm driver that implements this stuff? | 17:32 |
| jgriffith | Swanson: lemme see if I still have it | 17:32 |
| Swanson | jgriffith, I'll take it. | 17:32 |
| patrickeast | Swanson: i'll try and repro that.... looking at the code i have no idea why it would only call after restart though, is weird | 17:33 |
| jgriffith | Swanson: lemme see if I still have it | 17:33 |
| jgriffith | Swanson: hmm... I don't think I do | 17:35 |
| jgriffith | Swanson: it's EZ PZ though | 17:35 |
| *** avishay has joined #openstack-cinder | 17:35 | |
| jgriffith | just update capabilities and stub in the failover method | 17:35 |
| jgriffith | nothing to it | 17:35 |
| *** esker has joined #openstack-cinder | 17:36 | |
| jgriffith | My Nova Instance apparantly has died | 17:36 |
| *** sasukeh has joined #openstack-cinder | 17:37 | |
| *** timcl has joined #openstack-cinder | 17:37 | |
| *** esker has quit IRC | 17:38 | |
| *** sasukeh has quit IRC | 17:41 | |
| *** lpetrut has quit IRC | 17:49 | |
| geguileo | dulek: ping - https://review.openstack.org/#/c/290569/ | 17:49 |
| *** lpetrut has joined #openstack-cinder | 17:49 | |
| *** dustins_ has joined #openstack-cinder | 17:54 | |
| *** dustins has quit IRC | 17:55 | |
| *** avishay has quit IRC | 17:57 | |
| *** lpetrut has quit IRC | 17:57 | |
| *** diablo_rojo has quit IRC | 17:58 | |
| *** ChubYann has quit IRC | 17:58 | |
| *** ChubYann has joined #openstack-cinder | 17:59 | |
| *** haomaiwa_ has quit IRC | 18:01 | |
| *** 20WAAIK4T has joined #openstack-cinder | 18:01 | |
| *** merooney has quit IRC | 18:02 | |
| *** Suyash has joined #openstack-cinder | 18:02 | |
| *** laughterwym has joined #openstack-cinder | 18:03 | |
| *** vincent_hou has joined #openstack-cinder | 18:03 | |
| *** jungleboyj has quit IRC | 18:03 | |
| *** dustins_ has quit IRC | 18:03 | |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/290299 | 18:05 |
| *** akshai has quit IRC | 18:06 | |
| *** laughterwym has quit IRC | 18:07 | |
| *** meagleso has joined #openstack-cinder | 18:13 | |
| meagleso | can Liberty Cinder run without issue with a Kilo Nova? | 18:14 |
| *** akshai has joined #openstack-cinder | 18:14 | |
| dulek | geguileo: This is why I've decided to add additional joinedload: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/api.py#L1421 | 18:14 |
| dulek | geguileo: Basically I haven't seen a way to differentiate between _from_db_object for a volume_type for list and for show. | 18:15 |
| dulek | geguileo: I'll take a closer look a little later. | 18:15 |
| dulek | geguileo: Ah, and I understand your point of course. :) | 18:15 |
| *** martyturner1 has joined #openstack-cinder | 18:17 | |
| *** martyturner has quit IRC | 18:18 | |
| geguileo | dulek: Where do you want to differentiate between the show and the list? | 18:18 |
| geguileo | dulek: Oh, I think I see your point | 18:19 |
| dulek | geguileo: https://github.com/openstack/cinder/blob/master/cinder/objects/volume.py#L248-L253 | 18:19 |
| dulek | geguileo: Okay, obj_make_list seems to be able to take extra_args. | 18:20 |
| dulek | geguileo: So it should be possible. I'll post a new version as soon as I can. :) | 18:20 |
| dulek | (which may be tomorrow morning) | 18:21 |
| *** jordanP has quit IRC | 18:21 | |
| geguileo | dulek: Ok, thanks | 18:21 |
| geguileo | dulek: I appreciate the extra effort | 18:21 |
| dulek | geguileo: Thanks for being diligent, I've just looked too shallow and assumed it would be hard. | 18:22 |
| geguileo | dulek: Jaja | 18:22 |
| *** vincent_hou has quit IRC | 18:22 | |
| * dulek just got his Austin travel approved! | 18:24 | |
| *** jgregor has quit IRC | 18:25 | |
| *** akerr has quit IRC | 18:25 | |
| *** crose has joined #openstack-cinder | 18:26 | |
| *** crose has quit IRC | 18:27 | |
| *** crose has joined #openstack-cinder | 18:27 | |
| *** IlyaG has quit IRC | 18:27 | |
| *** mvk has quit IRC | 18:27 | |
| *** crose has quit IRC | 18:28 | |
| *** crose has joined #openstack-cinder | 18:28 | |
| *** IlyaG has joined #openstack-cinder | 18:28 | |
| mc_nair | dulek: nice! | 18:29 |
| *** e0ne has joined #openstack-cinder | 18:29 | |
| *** takedakn has joined #openstack-cinder | 18:29 | |
| *** takedakn has quit IRC | 18:30 | |
| mc_nair | I preordered some Franklins bbq for that Tuesday so people can try it | 18:30 |
| *** martyturner1 has quit IRC | 18:30 | |
| openstackgerrit | Merged openstack/cinder: Dell SC: create_cloned_volume didn't honor size https://review.openstack.org/290164 | 18:32 |
| *** ildikov has joined #openstack-cinder | 18:33 | |
| sheel | smcginnis: hi | 18:33 |
| *** jdurgin1 has quit IRC | 18:34 | |
| sheel | smcginnis: is there any chances of https://review.openstack.org/#/c/285293/ in mitaka? | 18:34 |
| sheel | To be clear, That fix for microversions is not currently used by anyone, but it is a dependency for my bootable filter fix: https://review.openstack.org/#/c/278605/ | 18:34 |
| *** mriedem has quit IRC | 18:36 | |
| *** mriedem has joined #openstack-cinder | 18:36 | |
| *** sasukeh has joined #openstack-cinder | 18:38 | |
| *** liewegas_ has quit IRC | 18:38 | |
| *** sage has joined #openstack-cinder | 18:39 | |
| *** sasukeh has quit IRC | 18:42 | |
| *** grumpycatt has quit IRC | 18:43 | |
| *** dims has quit IRC | 18:44 | |
| *** grumpycatt has joined #openstack-cinder | 18:45 | |
| *** jgregor has joined #openstack-cinder | 18:45 | |
| *** dims has joined #openstack-cinder | 18:46 | |
| *** meagleso has left #openstack-cinder | 18:47 | |
| *** akerr has joined #openstack-cinder | 18:49 | |
| krotscheck | Any cores around to help review this mitaka bug? https://review.openstack.org/#/c/286795/ | 18:55 |
| * krotscheck had to rebase it a few times when the gate broke | 18:55 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 18:57 |
| *** rlrossit has quit IRC | 18:58 | |
| *** lpetrut has joined #openstack-cinder | 18:58 | |
| *** jwcroppe has joined #openstack-cinder | 18:59 | |
| *** rlrossit has joined #openstack-cinder | 19:00 | |
| *** 20WAAIK4T has quit IRC | 19:01 | |
| *** markvoelker has quit IRC | 19:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:01 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 19:02 |
| *** markvoelker has joined #openstack-cinder | 19:03 | |
| *** garthb has joined #openstack-cinder | 19:05 | |
| *** liverpooler has joined #openstack-cinder | 19:06 | |
| *** dustins has joined #openstack-cinder | 19:06 | |
| *** huanan_L has quit IRC | 19:09 | |
| *** huanan has joined #openstack-cinder | 19:10 | |
| *** haomaiwa_ has quit IRC | 19:13 | |
| *** boris-42 has quit IRC | 19:14 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:14 | |
| *** markvoelker has quit IRC | 19:17 | |
| *** markvoelker has joined #openstack-cinder | 19:18 | |
| *** SergeyLukjanov has quit IRC | 19:19 | |
| *** robcresswell has joined #openstack-cinder | 19:19 | |
| *** bardia has joined #openstack-cinder | 19:20 | |
| *** SergeyLukjanov has joined #openstack-cinder | 19:21 | |
| tbarron | jgriffith: looks like test_migrations has cases that take more than 20s in gate :( | 19:22 |
| tbarron | maybe set to 30s for now? | 19:22 |
| jgriffith | tbarron: oh? | 19:22 |
| *** ociuhandu has quit IRC | 19:23 | |
| tbarron | jgriffith: at least that's the way I'm reading jenkins on 290695 | 19:23 |
| * jgriffith looks | 19:23 | |
| tbarron | I don't see them take that long running locally | 19:23 |
| jgriffith | tbarron: hmm... if that fixture timeout hits that's the exception you'd see | 19:24 |
| jgriffith | tbarron: not sure what's up here | 19:24 |
| jgriffith | tbarron: these all seem to be pymysql errors | 19:24 |
| jgriffith | tbarron: makes me wonder if there's something else going on here | 19:24 |
| *** lpetrut has quit IRC | 19:25 | |
| tbarron | jgriffith: yeah | 19:25 |
| jgriffith | tbarron: OHHH | 19:25 |
| jgriffith | tbarron: nah... there it is | 19:25 |
| *** lpetrut has joined #openstack-cinder | 19:25 | |
| jgriffith | 2016-03-09 17:13:57.295 | raise TimeoutException() | 19:25 |
| jgriffith | tbarron: wtf | 19:25 |
| *** lpetrut has quit IRC | 19:25 | |
| jgriffith | tbarron: and why would the db migration test take so long I wonder | 19:26 |
| *** lpetrut has joined #openstack-cinder | 19:26 | |
| tbarron | when I run test_migrations locally a bunch of the tests are skipped b/c DB backends are unavailable | 19:26 |
| jgriffith | tbarron: there's something else interesting going on here I suspect | 19:26 |
| tbarron | so I don't see them taking a long time. | 19:26 |
| jgriffith | tbarron: I was using samples straight out of Jenkins earlier | 19:26 |
| jgriffith | kinda surprised there would be this much variance in here | 19:27 |
| bswartz | tbarron: I've seen the same problem in Manila | 19:27 |
| bswartz | might be an infra issue | 19:27 |
| jgriffith | tbarron: bswartz http://paste.openstack.org/show/489895/ | 19:28 |
| tbarron | bswartz: slow migration tests when the mysql or postgreql are available? | 19:28 |
| jgriffith | so that hangs up... and then the timeout is hit | 19:28 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Show qos_specs_id based on policy https://review.openstack.org/241659 | 19:28 |
| jgriffith | wonder if there was an update to pymysql that broke something | 19:28 |
| tbarron | jgriffith: bswartz: yeah, may be infra/cross-project issue | 19:29 |
| jgriffith | tbarron: strange... well thanks for pointing it out | 19:29 |
| jgriffith | tbarron: I'll just let that dude sit for a bit and circle back later this week | 19:29 |
| jgriffith | tomorrow | 19:29 |
| jgriffith | whenever | 19:29 |
| jgriffith | :) | 19:29 |
| tbarron | jgriffith: sure | 19:29 |
| jgriffith | there's a lot of red going on in the gate right now it seems | 19:29 |
| *** merooney has joined #openstack-cinder | 19:30 | |
| *** dustins has quit IRC | 19:31 | |
| *** martyturner has joined #openstack-cinder | 19:31 | |
| ntpttr | jgriffith: smcginnis: sorry to bug, but if either of you have a moment would you mind giving https://review.openstack.org/#/c/241659/ a review? It fixes a problem that gives the user admin context when they don't necessarily need it and it's been hanging around for a while :) | 19:32 |
| *** laughterwym has joined #openstack-cinder | 19:32 | |
| jgriffith | ntpttr: looking | 19:32 |
| ntpttr | (it was me that added that issue in the first place a little before that though hah) | 19:33 |
| ntpttr | jgriffith: thanks | 19:33 |
| jgriffith | ntpttr: hehe... story of *MY* life :) | 19:33 |
| *** jgregor has quit IRC | 19:35 | |
| jgriffith | ntpttr: small question regarding the validate-policy method.... | 19:36 |
| jgriffith | ntpttr: I'm a bit confused... I mean; it's great you moved that into common.py | 19:36 |
| jgriffith | ntpttr: I'm trying to figure out why we had it in types and qos at all? | 19:36 |
| jgriffith | ntpttr: as opposed to wrapping the volume.api method? | 19:37 |
| *** jwcroppe has quit IRC | 19:37 | |
| *** laughterwym has quit IRC | 19:38 | |
| jgriffith | ntpttr: ahh... I see what ya did there | 19:38 |
| jgriffith | ntpttr: it's because you're only using policy to impact the view | 19:38 |
| *** sasukeh has joined #openstack-cinder | 19:38 | |
| *** dustins has joined #openstack-cinder | 19:39 | |
| *** mvk has joined #openstack-cinder | 19:39 | |
| ntpttr | jgriffith: yeah, so it needs to just be able to return true or false rather than just throw an exception | 19:40 |
| jgriffith | guitarzan: would you mind taking a look since you -1'd earlier? https://review.openstack.org/#/c/241659/14 | 19:40 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Active_backend_id wrong type https://review.openstack.org/290806 | 19:41 |
| guitarzan | jgriffith: ntpttr: sure, I noticed gerrit happily sent me an email about this | 19:42 |
| guitarzan | from nov 4th, wow | 19:42 |
| *** pots has quit IRC | 19:42 | |
| Swanson | Going to submit every patch with a subject of "Dell SC: Words long" | 19:42 |
| ntpttr | haha yeah it's been there a little while guitarzan. thanks! | 19:42 |
| *** sasukeh has quit IRC | 19:43 | |
| *** dustins has quit IRC | 19:44 | |
| *** ociuhandu has joined #openstack-cinder | 19:44 | |
| *** sdague has quit IRC | 19:45 | |
| *** pots has joined #openstack-cinder | 19:45 | |
| ntpttr | jgriffith: looks like you +2ed the adding of the validate_policy method there in one of my earliest cinder patches, for specifically viewing extra_specs | 19:47 |
| ntpttr | https://review.openstack.org/#/c/237075/6 | 19:47 |
| *** edtubill has quit IRC | 19:48 | |
| *** edtubill has joined #openstack-cinder | 19:50 | |
| *** salv-orlando has joined #openstack-cinder | 19:51 | |
| guitarzan | ntpttr: that definitely looks more like what you were after :) | 19:51 |
| *** salv-orl_ has joined #openstack-cinder | 19:53 | |
| ntpttr | guitarzan: yeah for sure, was definitely missing some needed fixes there at first. | 19:53 |
| *** diablo_rojo has joined #openstack-cinder | 19:53 | |
| *** BigWillie has quit IRC | 19:54 | |
| *** IlyaG has quit IRC | 19:54 | |
| *** sdague has joined #openstack-cinder | 19:54 | |
| *** salv-orlando has quit IRC | 19:56 | |
| *** vincent_hou has joined #openstack-cinder | 19:56 | |
| *** rlrossit has quit IRC | 19:58 | |
| ntpttr | oops looks like it's about to fail pep8 based on a little change I made. Have to push up one more patch. | 19:58 |
| *** jungleboyj has joined #openstack-cinder | 20:00 | |
| *** haomaiwa_ has quit IRC | 20:01 | |
| *** jgregor has joined #openstack-cinder | 20:01 | |
| *** baumann has left #openstack-cinder | 20:01 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Show qos_specs_id based on policy https://review.openstack.org/241659 | 20:01 |
| guitarzan | ack, was it the indentation? | 20:01 |
| *** haomaiwang has joined #openstack-cinder | 20:01 | |
| *** rhagarty has joined #openstack-cinder | 20:01 | |
| *** rlrossi__ has joined #openstack-cinder | 20:02 | |
| ntpttr | guitarzan: yeah I took out a paren because checking for the admin context was redundant, and that made it want me to move the indentation back a full four spaces | 20:02 |
| guitarzan | hmm, I think that's a subjective pep8 thing | 20:04 |
| *** yangyape_ has joined #openstack-cinder | 20:08 | |
| *** mrrowden has joined #openstack-cinder | 20:08 | |
| *** yangyapeng has quit IRC | 20:09 | |
| jgriffith | ntpttr: indeed, I did :) | 20:09 |
| *** garthb has quit IRC | 20:09 | |
| *** baumann has joined #openstack-cinder | 20:10 | |
| *** jwcroppe has joined #openstack-cinder | 20:19 | |
| *** jwcroppe has quit IRC | 20:21 | |
| *** martyturner has quit IRC | 20:28 | |
| *** martyturner has joined #openstack-cinder | 20:30 | |
| *** timcl has quit IRC | 20:31 | |
| *** timcl1 has joined #openstack-cinder | 20:31 | |
| *** ircuser-1 has quit IRC | 20:36 | |
| *** sasukeh has joined #openstack-cinder | 20:39 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Active_backend_id wrong type https://review.openstack.org/290806 | 20:42 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 20:42 |
| *** salv-orl_ has quit IRC | 20:43 | |
| *** sasukeh has quit IRC | 20:45 | |
| *** merooney has quit IRC | 20:47 | |
| diablo_rojo | dulek: Question for you :) So I am working on the patch to make volumes attach_status into a versioned object enum and I am trying to figure out how I need to bump the rpc versions. I updated volumeAttachment but do I also need to update Volume since I touched objects/volume.py? | 20:47 |
| *** tsufiev has left #openstack-cinder | 20:48 | |
| *** edtubill has quit IRC | 20:49 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1 https://review.openstack.org/287483 | 20:50 |
| *** ircuser-1 has joined #openstack-cinder | 20:51 | |
| *** IlyaG has joined #openstack-cinder | 20:52 | |
| diablo_rojo | dulek: That is assuming I need to bump the version at all when I only changed one field in volume.py and volume_attachment.py | 20:53 |
| *** merooney has joined #openstack-cinder | 20:53 | |
| *** e0ne has quit IRC | 20:53 | |
| *** sgotliv has joined #openstack-cinder | 20:57 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** Lee1092 has quit IRC | 21:02 | |
| *** sgotliv has quit IRC | 21:03 | |
| *** dims has quit IRC | 21:05 | |
| *** merooney has quit IRC | 21:06 | |
| *** mrrowden has quit IRC | 21:09 | |
| *** lcurtis has joined #openstack-cinder | 21:10 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Switch failover-host from rpc call to cast https://review.openstack.org/290857 | 21:13 |
| *** bill_az has joined #openstack-cinder | 21:16 | |
| *** porrua has quit IRC | 21:19 | |
| *** markvoelker has quit IRC | 21:20 | |
| *** sdague_ has joined #openstack-cinder | 21:24 | |
| patrickeast | jgriffith: ^ | 21:26 |
| jgriffith | patrickeast: ohhh... excellllent | 21:26 |
| *** kprabhu has quit IRC | 21:27 | |
| *** sheel has quit IRC | 21:27 | |
| patrickeast | jgriffith: i switched it over, but as i'm looking at some of the output now from the api i'm wondering if i need to make it return a 202 instead of 200 now too | 21:27 |
| jgriffith | patrickeast: :) | 21:27 |
| *** kprabhu has joined #openstack-cinder | 21:27 | |
| jgriffith | I was just commenting that we need to update cinder.api.contrib.service | 21:28 |
| jgriffith | patrickeast: other than that so far so good | 21:28 |
| *** akerr has quit IRC | 21:28 | |
| patrickeast | jgriffith: turned out to be super easy to change | 21:28 |
| jgriffith | patrickeast: so other than tht LGTM | 21:28 |
| jgriffith | patrickeast: yeah, the nice thing about Cheesecake is it has very limited ingredients | 21:29 |
| jgriffith | patrickeast: so it's not like unraveling the world to change something | 21:29 |
| patrickeast | jgriffith: yea def, its a nice change of pace for replication features :D | 21:29 |
| jgriffith | patrickeast: we'll see if lasts, once you start throwing rasberries, caramel or god forbid cherries on it :) | 21:30 |
| patrickeast | jgriffith: yea i mean... you've got all those fancy marbled cheesecakes and what not too | 21:30 |
| jgriffith | patrickeast: yeah... theyr'e all just marketing hype for the grahm cracker industry | 21:31 |
| patrickeast | lol | 21:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 21:36 |
| *** dims has joined #openstack-cinder | 21:37 | |
| *** sasukeh has joined #openstack-cinder | 21:40 | |
| *** kprabhu has quit IRC | 21:43 | |
| *** sasukeh has quit IRC | 21:44 | |
| *** cknight has quit IRC | 21:50 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Switch failover-host from rpc call to cast https://review.openstack.org/290857 | 21:51 |
| *** martyturner has quit IRC | 21:52 | |
| *** dims has quit IRC | 21:56 | |
| *** ndipanov has quit IRC | 21:57 | |
| *** timcl1 has quit IRC | 21:57 | |
| *** garthb has joined #openstack-cinder | 21:58 | |
| *** dims has joined #openstack-cinder | 21:58 | |
| *** martyturner has joined #openstack-cinder | 21:59 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Remove remaining oslo-incubator code from Cinder https://review.openstack.org/290875 | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| *** gouthamr has quit IRC | 22:02 | |
| *** eharney has quit IRC | 22:04 | |
| *** bardia has quit IRC | 22:06 | |
| *** lprice has quit IRC | 22:07 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1 https://review.openstack.org/287483 | 22:07 |
| *** baumann has left #openstack-cinder | 22:08 | |
| *** knikolla has quit IRC | 22:10 | |
| *** eharney has joined #openstack-cinder | 22:11 | |
| *** IlyaG has quit IRC | 22:16 | |
| *** sdague_ has quit IRC | 22:17 | |
| *** sdague has quit IRC | 22:17 | |
| *** david-lyle has quit IRC | 22:19 | |
| *** dave-mccowan has quit IRC | 22:20 | |
| *** david-lyle has joined #openstack-cinder | 22:20 | |
| *** jordant has joined #openstack-cinder | 22:22 | |
| *** krtaylor has quit IRC | 22:23 | |
| *** gouthamr has joined #openstack-cinder | 22:23 | |
| *** jordant has quit IRC | 22:24 | |
| *** jordant has joined #openstack-cinder | 22:25 | |
| diablo_rojo | patrickeast: Question for you. | 22:25 |
| patrickeast | diablo_rojo: sure, whats up? | 22:25 |
| *** IlyaG has joined #openstack-cinder | 22:25 | |
| diablo_rojo | patrickeast: Sean had mentioned you knew the command to get back to a local reference in git when you accidentally commit files before continuing a rebase? | 22:26 |
| Swanson | init_host isn't being called on initial startup.... | 22:26 |
| patrickeast | diablo_rojo: ah yea, 'git reflog' is the magic trick | 22:26 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Update replication to v2.1 https://review.openstack.org/287483 | 22:26 |
| patrickeast | diablo_rojo: it shows you all the points things have changed, and you can just checkout or reset back to whatever point you want | 22:27 |
| patrickeast | Swanson: oh weird | 22:27 |
| diablo_rojo | patrickeast: Thank you :) | 22:27 |
| patrickeast | Swanson: that sounds like a bug | 22:27 |
| patrickeast | diablo_rojo: np | 22:27 |
| Swanson | patrickeast, There is a LOG.info right at the beginning of init_host and I am not seeing it in my c-vol log... | 22:28 |
| *** jgregor has quit IRC | 22:28 | |
| patrickeast | Swanson: that would certainly explain why the host wouldn't have replication enabled until restarting | 22:28 |
| Swanson | Wait, there it is. | 22:29 |
| *** dims has quit IRC | 22:29 | |
| Swanson | I should just go home. | 22:29 |
| patrickeast | lol | 22:29 |
| *** [1]Thelo has joined #openstack-cinder | 22:30 | |
| Swanson | Service not found for updating replication_status. | 22:31 |
| *** mriedem has quit IRC | 22:31 | |
| *** bardia has joined #openstack-cinder | 22:32 | |
| *** crose has quit IRC | 22:32 | |
| *** Thelo has quit IRC | 22:33 | |
| *** [1]Thelo is now known as Thelo | 22:33 | |
| patrickeast | Swanson: from this place https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L528 ? | 22:33 |
| Swanson | That would be it. | 22:33 |
| patrickeast | mm thats not great | 22:34 |
| patrickeast | i like jgriffith's comment there | 22:34 |
| Swanson | Yeah, saw that. | 22:34 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum https://review.openstack.org/290885 | 22:34 |
| *** dave-mccowan has joined #openstack-cinder | 22:34 | |
| patrickeast | Swanson: oh yea, sure enough looking at my c-vol logs i have that message too | 22:35 |
| jgriffith | patrickeast: :) | 22:35 |
| patrickeast | Swanson: i must have just some way or another required restarting the service before I actually looked at the status | 22:35 |
| jgriffith | patrickeast: I was just being honest | 22:35 |
| Swanson | patrickeast, Shared pain. | 22:35 |
| *** krtaylor has joined #openstack-cinder | 22:36 | |
| patrickeast | jgriffith: Swanson: so uh... i guess that means the very first time cinder starts up the service isn't in the db yet (or with enough info) and that query fails? | 22:36 |
| *** tpsilva has quit IRC | 22:37 | |
| jgriffith | patrickeast: Swanson that shouldn't be the case.... | 22:37 |
| jgriffith | patrickeast: Swanson so what that would indicate to me is that the driver didn't init | 22:37 |
| jgriffith | patrickeast: Swanson which is odd | 22:37 |
| patrickeast | jgriffith: whys that? | 22:37 |
| patrickeast | oh wait | 22:37 |
| Swanson | jgriffith, it works tho! | 22:38 |
| jgriffith | Swanson: ? | 22:38 |
| patrickeast | jgriffith: if its calling this *in* the init call... of course it wont be init'ed | 22:38 |
| jgriffith | patrickeast: ohhhh..... hrmm | 22:38 |
| patrickeast | jgriffith: unless we set the driver status earlier | 22:38 |
| Swanson | jgriffith, I mean the driver clearly inits. Just doesn't appear to be so at the time it is being called. | 22:38 |
| * patrickeast should just go read the code | 22:38 | |
| openstackgerrit | Kurt Martin proposed openstack/cinder: LeftHand: Create cloned volume didn't honor size https://review.openstack.org/290890 | 22:39 |
| patrickeast | nvm | 22:39 |
| jgriffith | Swanson: patrickeast so I added that *after* publish_service_capabilities | 22:39 |
| patrickeast | i think we have already called self.driver.set_initialized() | 22:39 |
| jgriffith | meaning the service *should* be there | 22:39 |
| jgriffith | patrickeast: yes | 22:39 |
| *** martyturner has quit IRC | 22:40 | |
| jgriffith | which means maybe Swanson chose another driver for a reference that does bad things :) | 22:40 |
| jgriffith | just kidding Swanson | 22:40 |
| patrickeast | haha | 22:41 |
| *** sasukeh has joined #openstack-cinder | 22:41 | |
| *** vincent_hou has quit IRC | 22:41 | |
| Swanson | jgriffith, solidfire and pure. | 22:41 |
| jgriffith | patrickeast: Swanson seriously though, the reason I put that comment in there is because weve managed to send an rpc call to the right service | 22:41 |
| jgriffith | patrickeast: Swanson therefore, we MUST have an entry in the Service table no? | 22:41 |
| jgriffith | patrickeast: Swanson otherwise, "how did we even get here" | 22:42 |
| patrickeast | excellent question | 22:42 |
| patrickeast | who actually calls init_host? | 22:42 |
| Swanson | Well the next line after Driver initialization completed successfully is "Createing RPC server for service sinder-volume. | 22:42 |
| Swanson | I've already coughed up capabilities by this point... | 22:43 |
| patrickeast | oh | 22:43 |
| patrickeast | jgriffith: Swanson: take a look at service.py | 22:43 |
| jgriffith | well, there certainly could be an issue here if you try and just do it the first go around | 22:44 |
| patrickeast | we call init_host, and then do the service db stuff | 22:44 |
| jgriffith | could move that init routine | 22:44 |
| jgriffith | replication init routine that is | 22:44 |
| *** dims has joined #openstack-cinder | 22:44 | |
| jgriffith | just set it up on the first capabilities update | 22:44 |
| patrickeast | seems like that would work | 22:44 |
| *** sasukeh has quit IRC | 22:46 | |
| jgriffith | patrickeast: Swanson I'm still curious as to why it isn't working the way it is | 22:46 |
| Swanson | (Hey! Someone just closed a defect I opened 4 years ago! Nice. The product came. So too did the bug. The product left and took the bug with it. Circle of life.) | 22:46 |
| jgriffith | Swanson: HA! | 22:46 |
| *** rlrossi__ has quit IRC | 22:46 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add SnapshotStatus enum field https://review.openstack.org/284825 | 22:47 |
| jgriffith | Swanson: so this call works? bdrvpn.solidfire.com:4432 | 22:47 |
| jgriffith | Swanson: but this one doesn't? https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L524 | 22:47 |
| jgriffith | Swanson: errr... um | 22:48 |
| jgriffith | Swanson: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L258 | 22:48 |
| patrickeast | i dont think that one would have either | 22:49 |
| jgriffith | seems pretty piss-poor that we're doing things like init_capabilities etc on something that hasn't been initialized in the services table | 22:49 |
| Swanson | Neither seems to work. | 22:49 |
| patrickeast | so looking at https://github.com/openstack/cinder/blob/master/cinder/service.py#L150 which is calling init_host | 22:50 |
| Swanson | Which is to say I have both errors. | 22:50 |
| patrickeast | its still got a bunch of junk to do initializing the service after it calls it | 22:50 |
| jgriffith | well that's a kick in the shins | 22:51 |
| jgriffith | not even sure where half of all that init crap came from. Or why there's now "2" init methods | 22:52 |
| *** darosale has quit IRC | 22:52 | |
| jgriffith | patrickeast: Swanson ahh... so init_host is a secondary routine called by service.py | 22:53 |
| jgriffith | sigh | 22:53 |
| patrickeast | jgriffith: looks like we could just add an implementation of init_host_with_rpc to the volume manager | 22:53 |
| patrickeast | jgriffith: its called after we save the service object | 22:54 |
| patrickeast | jgriffith: and is just a noop right now afaik | 22:54 |
| jgriffith | patrickeast: yeah.... but I'd still like to understand why it isn't working for you guys a bit better | 22:54 |
| patrickeast | probably why it was added in the first place | 22:54 |
| jgriffith | patrickeast: because the call after manager.init_host is teh same Service_get_by_args that's failing for Swanson | 22:54 |
| jgriffith | OHHHH | 22:54 |
| jgriffith | heheehehheehehehe | 22:54 |
| patrickeast | yea but if it fails ther | 22:55 |
| patrickeast | it creates it | 22:55 |
| *** diablo_rojo has quit IRC | 22:55 | |
| jgriffith | except exception.NotFound: | 22:55 |
| jgriffith | self._create_service_ref(ctxt) | 22:55 |
| jgriffith | yup | 22:55 |
| patrickeast | haha yea | 22:55 |
| jgriffith | dang it | 22:55 |
| Swanson | Oh, nice. | 22:55 |
| *** diablo_rojo has joined #openstack-cinder | 22:55 | |
| jgriffith | beautiful! | 22:55 |
| jgriffith | so we __init__, init_host, rpc_init_host_with_rpc | 22:57 |
| patrickeast | yep | 22:57 |
| jgriffith | Just throw in another Init_capabilities call... WTF, just another method call | 22:57 |
| Swanson | Init a little, init somewhat more, init it all? | 22:57 |
| jgriffith | what could go wrong... can't imagine it causing any confusion 4 months from now | 22:57 |
| patrickeast | sure why not, everybody gets a init hook! | 22:57 |
| jgriffith | init hooks for ALL methods!!! | 22:57 |
| patrickeast | lol | 22:57 |
| jgriffith | just like the is_driver_init checks everywhere | 22:58 |
| jgriffith | :) | 23:01 |
| jgriffith | well crumbs | 23:01 |
| *** bjornar has quit IRC | 23:01 | |
| jgriffith | sorry guys | 23:01 |
| jgriffith | I should've caught that | 23:01 |
| patrickeast | hah don't feel bad... none of us knew how this worked either :p | 23:01 |
| jgriffith | LOL | 23:01 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 23:01 |
| Swanson | I'm pleased there was a rational explanation for what I was seeing. | 23:01 |
| *** jungleboyj has quit IRC | 23:01 | |
| patrickeast | so who wants to put the patch up to move it to init_host_with_rpc? | 23:01 |
| patrickeast | or add init_host_with_replication hooks? XD | 23:01 |
| *** garthb_ has joined #openstack-cinder | 23:01 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** garthb has quit IRC | 23:01 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!