| *** akerr has quit IRC | 00:01 | |
| *** akerr_ is now known as akerr_away | 00:05 | |
| *** akerr_away is now known as akerr_ | 00:09 | |
| *** annegentle has joined #openstack-cinder | 00:09 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects https://review.openstack.org/188535 | 00:09 |
|---|---|---|
| *** alexpilotti has quit IRC | 00:10 | |
| *** dims__ has joined #openstack-cinder | 00:11 | |
| *** dims_ has quit IRC | 00:12 | |
| *** vilobhmm has joined #openstack-cinder | 00:12 | |
| *** akerr has joined #openstack-cinder | 00:12 | |
| *** markvoelker has joined #openstack-cinder | 00:13 | |
| *** annegentle has quit IRC | 00:14 | |
| *** akerr_ has quit IRC | 00:15 | |
| *** dims__ has quit IRC | 00:17 | |
| *** dims_ has joined #openstack-cinder | 00:17 | |
| *** markvoelker has quit IRC | 00:18 | |
| *** merooney has joined #openstack-cinder | 00:22 | |
| *** annegentle has joined #openstack-cinder | 00:22 | |
| *** akerr_ has joined #openstack-cinder | 00:24 | |
| *** merooney has quit IRC | 00:24 | |
| *** merooney has joined #openstack-cinder | 00:24 | |
| *** akerr has quit IRC | 00:27 | |
| *** vilobhmm has quit IRC | 00:27 | |
| *** Sasi has joined #openstack-cinder | 00:39 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:42 | |
| *** tsekiyam_ has quit IRC | 00:42 | |
| *** mtanino has quit IRC | 00:43 | |
| *** tsekiyama has quit IRC | 00:45 | |
| *** raunak has quit IRC | 00:46 | |
| *** _cjones_ has quit IRC | 00:49 | |
| *** merooney has quit IRC | 00:52 | |
| *** rmesta has quit IRC | 00:52 | |
| *** merooney has joined #openstack-cinder | 00:52 | |
| *** chlong has quit IRC | 00:52 | |
| *** salv-orlando has joined #openstack-cinder | 00:54 | |
| *** chlong has joined #openstack-cinder | 00:54 | |
| *** merooney has quit IRC | 00:56 | |
| *** lpabon has joined #openstack-cinder | 01:03 | |
| *** annegentle has quit IRC | 01:06 | |
| *** merooney has joined #openstack-cinder | 01:06 | |
| *** salv-orlando has quit IRC | 01:10 | |
| *** merooney has quit IRC | 01:10 | |
| *** Sasi has quit IRC | 01:13 | |
| *** merooney has joined #openstack-cinder | 01:18 | |
| *** merooney has quit IRC | 01:19 | |
| *** tobe has joined #openstack-cinder | 01:38 | |
| *** Lee1092 has joined #openstack-cinder | 01:44 | |
| *** yamada-h has joined #openstack-cinder | 01:44 | |
| *** vilobhmm has joined #openstack-cinder | 01:46 | |
| *** yamada-h has quit IRC | 01:49 | |
| *** patrickeast has quit IRC | 01:50 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:58 | |
| *** vilobhmm has quit IRC | 02:01 | |
| *** markvoelker has joined #openstack-cinder | 02:02 | |
| tbarron | hemnafk: w.r.t. https://review.openstack.org/188591, do you think a bug or blueprint is most appropriate? I was torn between the two. | 02:02 |
| *** IlyaG has joined #openstack-cinder | 02:05 | |
| *** IlyaG has quit IRC | 02:06 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series driver: Remove caching logic https://review.openstack.org/185832 | 02:06 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Refactor class structure for FC https://review.openstack.org/185833 | 02:06 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Add Fibre Channel Support https://review.openstack.org/188649 | 02:06 |
| *** lixiaoy1 has joined #openstack-cinder | 02:06 | |
| *** markvoelker has quit IRC | 02:07 | |
| *** lixiaoy11 has quit IRC | 02:08 | |
| *** salv-orlando has joined #openstack-cinder | 02:09 | |
| *** Tross has joined #openstack-cinder | 02:14 | |
| *** lixiaoy11 has joined #openstack-cinder | 02:15 | |
| *** Tross1 has quit IRC | 02:15 | |
| *** Tross1 has joined #openstack-cinder | 02:16 | |
| *** lixiaoy1 has quit IRC | 02:17 | |
| *** salv-orlando has quit IRC | 02:17 | |
| *** Tross has quit IRC | 02:19 | |
| *** akerr_ has quit IRC | 02:22 | |
| *** akerr has joined #openstack-cinder | 02:25 | |
| *** daneyon_ has quit IRC | 02:29 | |
| smcginnis | thingee: ping | 02:43 |
| smcginnis | thingee: Review for this looks good but bp isn't approved yet: https://blueprints.launchpad.net/cinder/+spec/allow-non-admin-to-query-volume-filter-by-az | 02:44 |
| smcginnis | thingee: Don't want to push it through until it's green lighted. | 02:44 |
| smcginnis | thingee: Review - https://review.openstack.org/#/c/156034/ | 02:45 |
| *** yamada-h has joined #openstack-cinder | 02:45 | |
| *** yamada-h has quit IRC | 02:49 | |
| *** akerr has quit IRC | 02:52 | |
| *** Apoorva has quit IRC | 02:56 | |
| *** boris-42 has quit IRC | 02:58 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Conversion to volume object https://review.openstack.org/177054 | 03:03 |
| *** dims_ has quit IRC | 03:07 | |
| *** zul has quit IRC | 03:08 | |
| *** xyang1 has quit IRC | 03:15 | |
| *** lpetrut has joined #openstack-cinder | 03:18 | |
| *** zul has joined #openstack-cinder | 03:21 | |
| *** gouthamr has quit IRC | 03:26 | |
| *** gouthamr has joined #openstack-cinder | 03:27 | |
| *** gouthamr has quit IRC | 03:31 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Add Chap support for Huawei Storage https://review.openstack.org/188360 | 03:45 |
| *** markvoelker has joined #openstack-cinder | 03:50 | |
| *** Lee1092 has quit IRC | 03:51 | |
| *** salv-orlando has joined #openstack-cinder | 03:51 | |
| *** coolsvap|afk is now known as coolsvap | 03:53 | |
| *** markvoelker has quit IRC | 03:55 | |
| *** marcusvrn has joined #openstack-cinder | 03:57 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Prophetstor driver needs to return snapshot objects https://review.openstack.org/185495 | 03:59 |
| openstackgerrit | Rick Chen proposed openstack/cinder: Complete switch to snapshot objects https://review.openstack.org/163910 | 03:59 |
| openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 04:00 |
| *** salv-orlando has quit IRC | 04:01 | |
| *** yamada-h has joined #openstack-cinder | 04:05 | |
| *** dims_ has joined #openstack-cinder | 04:05 | |
| *** dims_ has quit IRC | 04:10 | |
| *** lpetrut has quit IRC | 04:21 | |
| *** pradipta has joined #openstack-cinder | 04:22 | |
| *** rushiagr_away is now known as rushiagr | 04:22 | |
| *** marcusvrn has quit IRC | 04:24 | |
| *** yamada-h has quit IRC | 04:24 | |
| *** shyama has joined #openstack-cinder | 04:26 | |
| *** rushiagr is now known as rushiagr_away | 04:27 | |
| *** Longgeek has joined #openstack-cinder | 04:30 | |
| *** Longgeek has quit IRC | 04:30 | |
| *** swamireddy1 has joined #openstack-cinder | 04:31 | |
| *** jamielennox is now known as jamielennox|away | 04:32 | |
| *** yamada-h has joined #openstack-cinder | 04:32 | |
| *** Longgeek has joined #openstack-cinder | 04:32 | |
| *** yamada-h has quit IRC | 04:36 | |
| *** rushiagr_away is now known as rushiagr | 04:40 | |
| *** kaisers1 has quit IRC | 04:44 | |
| *** kaisers1 has joined #openstack-cinder | 04:45 | |
| *** heyun has joined #openstack-cinder | 04:46 | |
| *** Longgeek_ has joined #openstack-cinder | 04:50 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 04:50 |
| *** Longgeek has quit IRC | 04:54 | |
| *** yamada-h has joined #openstack-cinder | 04:55 | |
| *** yamada-h has quit IRC | 05:00 | |
| *** BharatK has joined #openstack-cinder | 05:05 | |
| *** spradeepv has quit IRC | 05:06 | |
| *** lpetrut has joined #openstack-cinder | 05:08 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 05:35 |
| *** markvoelker has joined #openstack-cinder | 05:39 | |
| *** mitz has quit IRC | 05:41 | |
| *** tobe has quit IRC | 05:41 | |
| *** tobe has joined #openstack-cinder | 05:42 | |
| *** tobe has quit IRC | 05:44 | |
| *** markvoelker has quit IRC | 05:44 | |
| *** mdbooth has quit IRC | 05:48 | |
| *** nihilifer has joined #openstack-cinder | 05:49 | |
| *** nihilifer has quit IRC | 05:51 | |
| *** nihilifer has joined #openstack-cinder | 05:51 | |
| *** mdbooth has joined #openstack-cinder | 05:53 | |
| *** belmoreira has joined #openstack-cinder | 05:55 | |
| *** yamada-h has joined #openstack-cinder | 05:59 | |
| *** yamada-h has quit IRC | 06:04 | |
| *** vilobhmm has joined #openstack-cinder | 06:12 | |
| *** BharatK has quit IRC | 06:16 | |
| *** lpetrut has quit IRC | 06:19 | |
| *** tobe has joined #openstack-cinder | 06:19 | |
| *** agarciam has joined #openstack-cinder | 06:19 | |
| *** agarciam has quit IRC | 06:22 | |
| *** agarciam has joined #openstack-cinder | 06:23 | |
| *** agarciam has quit IRC | 06:24 | |
| *** agarciam has joined #openstack-cinder | 06:26 | |
| *** agarciam has quit IRC | 06:27 | |
| *** agarciam has joined #openstack-cinder | 06:28 | |
| *** agarciam has quit IRC | 06:28 | |
| *** BharatK has joined #openstack-cinder | 06:29 | |
| *** agarciam has joined #openstack-cinder | 06:30 | |
| *** yamada-h has joined #openstack-cinder | 06:33 | |
| *** yamada-h has quit IRC | 06:33 | |
| *** agarciam has quit IRC | 06:35 | |
| *** agarciam has joined #openstack-cinder | 06:36 | |
| *** vilobhmm has quit IRC | 06:46 | |
| *** Lee1092 has joined #openstack-cinder | 06:49 | |
| *** ociuhandu has joined #openstack-cinder | 06:50 | |
| *** yamada-h has joined #openstack-cinder | 06:50 | |
| *** yamada-h has quit IRC | 06:50 | |
| *** dims_ has joined #openstack-cinder | 06:54 | |
| *** e0ne has joined #openstack-cinder | 06:54 | |
| *** takedakn has joined #openstack-cinder | 06:56 | |
| *** yamada-h has joined #openstack-cinder | 06:56 | |
| *** yamada-h has quit IRC | 06:57 | |
| *** nikeshm has quit IRC | 06:57 | |
| *** jistr has joined #openstack-cinder | 06:57 | |
| *** dims_ has quit IRC | 06:59 | |
| *** takedakn has quit IRC | 07:00 | |
| *** takedakn has joined #openstack-cinder | 07:01 | |
| *** e0ne has quit IRC | 07:04 | |
| *** yamada-h has joined #openstack-cinder | 07:04 | |
| *** yamada-h has quit IRC | 07:04 | |
| *** takedakn has quit IRC | 07:04 | |
| *** chhavi has joined #openstack-cinder | 07:08 | |
| *** markus_z has joined #openstack-cinder | 07:15 | |
| chhavi | winston-d: Need to discuss on https://review.openstack.org/#/c/182980/ are you there | 07:15 |
| *** takedakn has joined #openstack-cinder | 07:16 | |
| *** takedakn has quit IRC | 07:17 | |
| *** takedakn has joined #openstack-cinder | 07:18 | |
| *** takedakn has quit IRC | 07:18 | |
| *** takedakn has joined #openstack-cinder | 07:19 | |
| *** geguileo has joined #openstack-cinder | 07:20 | |
| *** anshul has joined #openstack-cinder | 07:21 | |
| *** takedakn has quit IRC | 07:22 | |
| *** ronis has joined #openstack-cinder | 07:24 | |
| *** Miouge_ has joined #openstack-cinder | 07:26 | |
| *** ociuhandu has quit IRC | 07:28 | |
| *** markvoelker has joined #openstack-cinder | 07:28 | |
| *** Miouge has quit IRC | 07:29 | |
| *** Miouge_ is now known as Miouge | 07:29 | |
| *** lpetrut has joined #openstack-cinder | 07:29 | |
| *** Miouge has quit IRC | 07:31 | |
| *** markvoelker has quit IRC | 07:33 | |
| *** Miouge has joined #openstack-cinder | 07:35 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Port remote_fs driver to use new driver model https://review.openstack.org/160688 | 07:46 |
| *** jistr is now known as jistr|biab | 07:47 | |
| *** rwsu has joined #openstack-cinder | 07:48 | |
| *** coolsvap is now known as coolsvap|afk | 07:52 | |
| *** yamada-h has joined #openstack-cinder | 07:56 | |
| *** yamada-h has quit IRC | 08:01 | |
| *** Miouge has quit IRC | 08:05 | |
| *** Miouge has joined #openstack-cinder | 08:08 | |
| *** Miouge has quit IRC | 08:11 | |
| *** Miouge has joined #openstack-cinder | 08:22 | |
| *** ociuhandu has joined #openstack-cinder | 08:32 | |
| *** chlong has quit IRC | 08:34 | |
| *** rwsu has quit IRC | 08:36 | |
| *** ndipanov has joined #openstack-cinder | 08:37 | |
| *** shyama has quit IRC | 08:43 | |
| *** shyama has joined #openstack-cinder | 08:44 | |
| *** ankit_ag has joined #openstack-cinder | 08:45 | |
| *** ociuhandu has quit IRC | 08:47 | |
| *** jistr|biab is now known as jistr | 08:50 | |
| *** lennyb has quit IRC | 08:52 | |
| *** coolsvap|afk is now known as coolsvap | 08:54 | |
| *** jistr has quit IRC | 08:59 | |
| *** e0ne has joined #openstack-cinder | 09:09 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 09:15 |
| *** markvoelker has joined #openstack-cinder | 09:17 | |
| *** jistr has joined #openstack-cinder | 09:19 | |
| *** markvoelker has quit IRC | 09:22 | |
| *** afazekas_mtg has joined #openstack-cinder | 09:33 | |
| *** aix has joined #openstack-cinder | 09:34 | |
| *** chhavi has quit IRC | 09:36 | |
| *** dims_ has joined #openstack-cinder | 09:43 | |
| *** alexpilotti has joined #openstack-cinder | 09:45 | |
| *** dims__ has joined #openstack-cinder | 09:47 | |
| *** dims_ has quit IRC | 09:48 | |
| *** lixiaoy11 has quit IRC | 09:53 | |
| *** yamada-h has joined #openstack-cinder | 09:58 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support for force-delete backups https://review.openstack.org/166127 | 10:00 |
| *** yamada-h has quit IRC | 10:03 | |
| *** e0ne is now known as e0ne_ | 10:07 | |
| liuxg | https://review.openstack.org/#/c/188240/ | 10:10 |
| liuxg | hi all ,Please take some time to review our driver | 10:11 |
| *** e0ne_ is now known as e0ne | 10:12 | |
| *** mdenny has quit IRC | 10:17 | |
| *** markvoelker has joined #openstack-cinder | 10:17 | |
| *** boris-42 has joined #openstack-cinder | 10:20 | |
| *** markvoelker has quit IRC | 10:23 | |
| openstackgerrit | Peter Penchev proposed openstack/cinder: Re-add the StorPool distributed storage driver https://review.openstack.org/188725 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: move the testcase to cinder/tests/unit/ https://review.openstack.org/188726 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: pass the H402 docstring check https://review.openstack.org/188727 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: fall back to the configured template https://review.openstack.org/188728 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: return volume object in _attach_volume() https://review.openstack.org/188729 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: implement volume retyping and migrations https://review.openstack.org/188730 | 10:23 |
| openstackgerrit | Peter Penchev proposed openstack/cinder: StorPool: use the ABC metaclasses for volume drivers https://review.openstack.org/188731 | 10:23 |
| openstackgerrit | chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 10:25 |
| liuxg | <openstackgerrit> chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 10:26 |
| liuxg | <openstackgerrit> chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 10:26 |
| *** rongze has joined #openstack-cinder | 10:27 | |
| *** BharatK has quit IRC | 10:27 | |
| liuxg | @rongze | 10:27 |
| liuxg | Please to review our driver | 10:27 |
| liuxg | <openstackgerrit> chenzongliang proposed openstack/cinder: Support SmartX for Huawei volume driver https://review.openstack.org/188732 | 10:27 |
| *** BharatK has joined #openstack-cinder | 10:38 | |
| *** SergeyLukjanov has quit IRC | 10:39 | |
| *** pradipta has quit IRC | 10:43 | |
| *** rongze has quit IRC | 10:46 | |
| *** SergeyLukjanov has joined #openstack-cinder | 10:51 | |
| *** coolsvap is now known as coolsvap|afk | 10:59 | |
| *** Lee1092 has quit IRC | 11:01 | |
| *** yamada-h has joined #openstack-cinder | 11:01 | |
| *** yamada-h has quit IRC | 11:01 | |
| *** julim has joined #openstack-cinder | 11:01 | |
| *** rongze has joined #openstack-cinder | 11:02 | |
| *** e0ne is now known as e0ne_ | 11:05 | |
| *** e0ne_ is now known as e0ne | 11:07 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Port remote_fs driver to use new driver model https://review.openstack.org/160688 | 11:08 |
| flip214 | would someone please tell me whether I'm on the right track with https://review.openstack.org/188744? | 11:10 |
| *** markvoelker has joined #openstack-cinder | 11:18 | |
| *** markvoelker has quit IRC | 11:22 | |
| *** rongze has quit IRC | 11:27 | |
| openstackgerrit | Pavel Boldin proposed openstack/cinder: Add iscsi_target_flags configuration option https://review.openstack.org/182871 | 11:31 |
| *** yamada-h has joined #openstack-cinder | 11:32 | |
| *** marcusvrn has joined #openstack-cinder | 11:38 | |
| *** yamada-h has quit IRC | 11:38 | |
| *** tobe has quit IRC | 11:42 | |
| *** rwsu has joined #openstack-cinder | 11:43 | |
| *** heyun has quit IRC | 11:56 | |
| *** marcusvrn has quit IRC | 11:56 | |
| *** sgotliv has quit IRC | 12:03 | |
| *** zul has quit IRC | 12:16 | |
| *** zul has joined #openstack-cinder | 12:16 | |
| *** e0ne is now known as e0ne_ | 12:21 | |
| *** e0ne_ is now known as e0ne | 12:22 | |
| *** andymccr has joined #openstack-cinder | 12:23 | |
| *** sc has quit IRC | 12:23 | |
| *** chhavi has joined #openstack-cinder | 12:25 | |
| *** sc has joined #openstack-cinder | 12:26 | |
| *** sasi has joined #openstack-cinder | 12:31 | |
| *** sasi has left #openstack-cinder | 12:31 | |
| *** annegentle has joined #openstack-cinder | 12:32 | |
| *** nihilifer has quit IRC | 12:33 | |
| *** markvoelker has joined #openstack-cinder | 12:34 | |
| *** ndipanov is now known as ndipanoff | 12:36 | |
| *** zhenguo has quit IRC | 12:37 | |
| *** markvoelker has quit IRC | 12:38 | |
| *** ociuhandu has joined #openstack-cinder | 12:39 | |
| *** akerr has joined #openstack-cinder | 12:43 | |
| *** marcusvrn has joined #openstack-cinder | 12:45 | |
| *** bswartz has quit IRC | 12:49 | |
| *** marcusvrn has quit IRC | 12:49 | |
| *** bswartz has joined #openstack-cinder | 12:50 | |
| *** Yogi1 has joined #openstack-cinder | 12:51 | |
| *** sasi has joined #openstack-cinder | 12:51 | |
| *** merooney has joined #openstack-cinder | 13:06 | |
| *** dims__ has quit IRC | 13:09 | |
| *** dims_ has joined #openstack-cinder | 13:10 | |
| *** xyang has joined #openstack-cinder | 13:16 | |
| *** dustins has joined #openstack-cinder | 13:17 | |
| *** boris-42 has quit IRC | 13:18 | |
| *** merooney has quit IRC | 13:20 | |
| *** bswartz has quit IRC | 13:21 | |
| *** cknight has joined #openstack-cinder | 13:24 | |
| *** primechuck has joined #openstack-cinder | 13:25 | |
| *** mriedem has joined #openstack-cinder | 13:27 | |
| *** rushiagr is now known as rushiagr_away | 13:28 | |
| *** avishay has joined #openstack-cinder | 13:29 | |
| *** merooney has joined #openstack-cinder | 13:30 | |
| *** BharatK has quit IRC | 13:30 | |
| *** gouthamr has joined #openstack-cinder | 13:31 | |
| *** avishay has quit IRC | 13:34 | |
| *** thangp has joined #openstack-cinder | 13:35 | |
| *** avishay has joined #openstack-cinder | 13:36 | |
| *** gouthamr has left #openstack-cinder | 13:37 | |
| *** gouthamr has joined #openstack-cinder | 13:37 | |
| *** Lee1092 has joined #openstack-cinder | 13:37 | |
| *** rushil has joined #openstack-cinder | 13:37 | |
| *** e0ne is now known as e0ne_ | 13:39 | |
| *** merooney has quit IRC | 13:39 | |
| *** e0ne_ is now known as e0ne | 13:39 | |
| *** afazekas_mtg has quit IRC | 13:40 | |
| *** boris-42 has joined #openstack-cinder | 13:41 | |
| *** jaypipes is now known as leakypipes | 13:43 | |
| *** merooney has joined #openstack-cinder | 13:43 | |
| *** merooney has quit IRC | 13:45 | |
| *** asselin_ has joined #openstack-cinder | 13:45 | |
| *** chhavi has quit IRC | 13:46 | |
| *** asselin__ has joined #openstack-cinder | 13:47 | |
| *** rushil has quit IRC | 13:47 | |
| andymccr | Ran into the endpoint-type issue on cinderclient which was fixed a few weeks ago, is there any way to get a new release of cinderclient (it got fixed 5 days after 1.2.1) or is there a more "proper" way to go about this? | 13:49 |
| *** asselin_ has quit IRC | 13:50 | |
| *** eharney has joined #openstack-cinder | 13:55 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Handle ineffective backup compression https://review.openstack.org/188591 | 13:56 |
| *** afazekas has joined #openstack-cinder | 13:58 | |
| jgriffith | tbarron: ping | 14:01 |
| *** swamireddy1 has quit IRC | 14:02 | |
| *** markvoelker has joined #openstack-cinder | 14:05 | |
| scottda | andymccr: I believe you should ask thingee about a new release of cinderclient. It is independent of cinder/OpenStack release cycle and can be done anytime (by the PTL) | 14:07 |
| andymccr | scottda: thanks! I've seen it done that way with some of the other clients but didn't want to make assumptions :) | 14:07 |
| *** markvoelker has quit IRC | 14:09 | |
| tbarron | jgriffith: pong (just got out of a meeting) wazzup? | 14:09 |
| *** esker has joined #openstack-cinder | 14:09 | |
| jgriffith | tbarron: hey, was just looking at that bug on backup compression | 14:10 |
| jgriffith | tbarron: curious if you have any ideas on what to do with it? | 14:10 |
| tbarron | jgriffith: yeah, I have a review up, https://review.openstack.org/188591 | 14:10 |
| jgriffith | OH... excellent | 14:11 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Fix free virtual space in capacity filter https://review.openstack.org/188791 | 14:11 |
| *** rmesta has joined #openstack-cinder | 14:11 | |
| jgriffith | tbarron: so I was kinda of wondering about that | 14:11 |
| jgriffith | tbarron: so we compress it, compare, then take the lesser? | 14:11 |
| tbarron | jgriffith: as soon as it gets out of our CI, I have another one on the way. | 14:11 |
| jgriffith | tbarron: doesn't that make restore kinda "weird" | 14:12 |
| tbarron | jgriffith: yes, nothing smart in this iteration to detect incompressible data or something in the first place | 14:12 |
| avishay | anyone manage to get cinder working with noauth? i did it once i think but don't remember how i did it... | 14:12 |
| tbarron | jgriffith: the compression algo (including 'none') is already stored with each chunk | 14:12 |
| jgriffith | avishay: I wish! | 14:12 |
| tbarron | and restore looks at that already | 14:12 |
| jgriffith | avishay: please share if/when you get it to work | 14:12 |
| jgriffith | avishay: I *think* guitarzan may have got it to work at one point, but I could be completely wrong | 14:13 |
| tbarron | jgriffith: does that address your concern? | 14:13 |
| avishay | jgriffith: cinder is up but cinderclient isn't happy | 14:13 |
| jgriffith | tbarron: sure | 14:13 |
| jgriffith | tbarron: thanks | 14:13 |
| avishay | guitarzan: here? | 14:13 |
| tbarron | jgriffith: thanks for asking! | 14:13 |
| jgriffith | avishay: oh, interesting | 14:13 |
| jgriffith | avishay: hey... wanna have some fun? | 14:13 |
| jgriffith | avishay: trying downloading latest cinderclient with a Kilo distro | 14:13 |
| avishay | jgriffith: depends on how you define fun :) | 14:13 |
| jgriffith | avishay: :) | 14:14 |
| avishay | jgriffith: why? | 14:14 |
| jgriffith | broke | 14:14 |
| guitarzan | avishay: oh noauth? hmm | 14:14 |
| jgriffith | osl.conf vs oslo_config deps | 14:14 |
| guitarzan | I'm not sure how well that works anymore | 14:14 |
| avishay | jgriffith: mmm | 14:14 |
| *** esker has quit IRC | 14:14 | |
| jgriffith | avishay: looking this morning, customer seeing it | 14:15 |
| avishay | jgriffith: nice | 14:15 |
| *** esker has joined #openstack-cinder | 14:15 | |
| jgriffith | :( | 14:15 |
| avishay | guitarzan: i guess i can try curl and take it from there | 14:15 |
| jgriffith | avishay: so you got it all working with Cinder though... that's cool! | 14:15 |
| jgriffith | avishay: I suspect the discovery stuff that got added will break it really good on the client | 14:15 |
| guitarzan | avishay: I think cinder itself may still work with noauth | 14:16 |
| guitarzan | avishay: eventually I gave up and have a little fake auth service | 14:16 |
| guitarzan | glance for example, simply doesn't noauth :) | 14:16 |
| *** e0ne is now known as e0ne_ | 14:16 | |
| guitarzan | the keystoneclient stuff kind of blows up too | 14:16 |
| jgriffith | guitarzan: Oh, that's right, I remember you saying that | 14:16 |
| jgriffith | guitarzan: I wonder if it would be worth having that in Cinder tree as an option? | 14:17 |
| avishay | jgriffith: guitarzan: oh man | 14:17 |
| guitarzan | jgriffith: I'm a big fan of noauth for development | 14:17 |
| avishay | it would really suck if i had to use keystone for tests | 14:17 |
| jgriffith | avishay: +1 | 14:17 |
| guitarzan | but as the web gets more webby, it becomes lots harder to do anything useful | 14:17 |
| jgriffith | guitarzan: yeah, and actually I get requests about it A LOT | 14:17 |
| jgriffith | webby webs... nothing I hate more :) | 14:18 |
| *** rushiagr_away is now known as rushiagr | 14:18 | |
| guitarzan | yep | 14:18 |
| avishay | :) | 14:18 |
| jgriffith | avishay: we could probably log a bug for cinderclient to get noauth on track | 14:19 |
| jgriffith | avishay: don't know if it'd help, but maybe somebody else with cycles will be interested enough to make it work | 14:19 |
| jgriffith | avishay: personally I'd love to see it | 14:20 |
| avishay | jgriffith: if i can't get it to work i'll probably have to do it :) | 14:20 |
| jgriffith | avishay: I'd love to see Cinder be able to be a stand-alone service some day | 14:20 |
| jgriffith | avishay: :) | 14:20 |
| kragniz | andymccr: you're not the only one who would like a cinderclient release | 14:22 |
| tbarron | jgriffith: v3? | 14:22 |
| jgriffith | tbarron: ? | 14:23 |
| andymccr | kragniz: i imagine not! for the same reason? (endpoint-type var issues) | 14:24 |
| *** merooney has joined #openstack-cinder | 14:24 | |
| *** rushil has joined #openstack-cinder | 14:24 | |
| kragniz | nope, the pbr version in the released requirements.txt is breaking the glance_store gate | 14:24 |
| jgriffith | kragniz: is that fix in cinderclient now? | 14:25 |
| kragniz | jgriffith: yup | 14:25 |
| jgriffith | kragniz: k | 14:26 |
| kragniz | master requirements is good | 14:26 |
| *** dhellmann has joined #openstack-cinder | 14:26 | |
| *** e0ne_ has quit IRC | 14:26 | |
| tbarron | jgriffith: replication v3 :) | 14:27 |
| tbarron | jgriffith: feel free just to ignore the annoying gadfly | 14:28 |
| *** ronis has quit IRC | 14:28 | |
| *** mtanino has joined #openstack-cinder | 14:29 | |
| avishay | guitarzan: do you have an example of using curl to communicate with cinder on hand? haven't done it in a while | 14:30 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient: Prep for 1.2.2 https://review.openstack.org/188794 | 14:30 |
| jgriffith | tbarron: LOL | 14:31 |
| *** salv-orlando has joined #openstack-cinder | 14:31 | |
| jgriffith | tbarron: nah, I'm considering just leaving it broken at this point | 14:31 |
| jgriffith | tbarron: Fact that it's been nit picked for almost 6 months indicates nobody will every be happy | 14:31 |
| tbarron | jgriffith: it's a pretty thankless job | 14:31 |
| jgriffith | tbarron: particularly the circular crap: "Person A" says "Must have this"... then 3 months later, same person says "You can't do that" | 14:32 |
| jgriffith | :) | 14:32 |
| jgriffith | good times! | 14:32 |
| tbarron | jgriffith: I would like to see the decoupling of the capability from the create path in any case | 14:32 |
| *** dansmith is now known as superdan | 14:32 | |
| guitarzan | avishay: I'm not, I haven't done it in a long time either | 14:32 |
| guitarzan | avishay: er, I don't | 14:32 |
| avishay | guitarzan: ok thanks anyway | 14:32 |
| jgriffith | kragniz: andymccr https://review.openstack.org/188794 | 14:33 |
| tbarron | that was a noble goald | 14:33 |
| tbarron | goal | 14:33 |
| jgriffith | tbarron: I'm still going to put my patch up | 14:33 |
| jgriffith | FWIW | 14:33 |
| kragniz | jgriffith: awesome! | 14:33 |
| kragniz | thanks | 14:33 |
| jgriffith | kragniz: once that lands I'll push, assuming I still have permissions | 14:34 |
| kragniz | cool | 14:34 |
| tbarron | jgriffith: eventually it may be along the line of what you and thingee are doing w.r.t capabilities | 14:34 |
| andymccr | thanks jgriffith! | 14:34 |
| tbarron | { replication: True | 14:34 |
| *** ankit_ag has quit IRC | 14:34 | |
| tbarron | vendor_keys: ....} | 14:34 |
| *** mdenny has joined #openstack-cinder | 14:34 | |
| tbarron | and everyone can do it their own way | 14:35 |
| jgriffith | tbarron: EXACTLY! | 14:35 |
| jgriffith | :) | 14:35 |
| jgriffith | which is what I wanted in the first place! | 14:35 |
| tbarron | jgriffith: yeah, I got that. | 14:35 |
| tbarron | course I want vendor keys to allow pool_level replication too, and admin-only triggered failover. | 14:36 |
| *** Tross1 has quit IRC | 14:37 | |
| jgriffith | tbarron: of course | 14:37 |
| *** Tross has joined #openstack-cinder | 14:37 | |
| jgriffith | tbarron: and you should be able to have that IMO | 14:37 |
| tbarron | jgriffith: kumaiah | 14:38 |
| tbarron | kumbaiah | 14:38 |
| tbarron | :-) | 14:38 |
| jgriffith | Peace Love and Twinkies | 14:38 |
| *** Tross has quit IRC | 14:38 | |
| tbarron | jgriffith: sounds like Colorado snack food | 14:38 |
| tbarron | though a little early in the day | 14:39 |
| jgriffith | haha | 14:39 |
| *** IanGovett has joined #openstack-cinder | 14:40 | |
| jgriffith | DuncanT: you around? | 14:41 |
| *** belmoreira has quit IRC | 14:42 | |
| *** e0ne has joined #openstack-cinder | 14:45 | |
| ameade | winston-d: would you be so kind as to sponsor the netapp driver? https://etherpad.openstack.org/p/cinder-liberty-drivers | 14:45 |
| *** rajinir_r has quit IRC | 14:45 | |
| *** rajinir_r has joined #openstack-cinder | 14:46 | |
| avishay | jgriffith: guitarzan: it's working!!! | 14:53 |
| jgriffith | avishay: sweeeeet!!!! | 14:53 |
| avishay | jgriffith: guitarzan: i found my old VM image, went into ipython and looked at the history :) | 14:54 |
| jgriffith | nice | 14:54 |
| avishay | jgriffith: guitarzan: don't know how to get it to work from the command line, but works from python import | 14:54 |
| jgriffith | LOL | 14:54 |
| e0ne | jgriffith: hi. i would like to get you advice | 14:55 |
| jgriffith | e0ne: you sure about that? :) | 14:55 |
| e0ne | jgriffith: should we merge new drivers (https://review.openstack.org/#/c/156206/43/cinder/volume/drivers/ibm/flashsystem_iscsi.py) and allow to make code cleanup in the following patches or not? | 14:55 |
| jgriffith | e0ne: what's up? | 14:55 |
| guitarzan | avishay: victory! | 14:55 |
| openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 14:56 |
| jgriffith | e0ne: looking | 14:56 |
| jgriffith | e0ne: so you're primarily referring to the last comment L#435? | 14:57 |
| e0ne | jgriffith: personally, i prefer to not merge patches which require cleanip | 14:57 |
| jgriffith | e0ne: IMO I would agree with you | 14:57 |
| jgriffith | e0ne: the fact that you need a "cleanup" BP before you even merge the first version is a bad sign IMHO | 14:58 |
| e0ne | jgriffith: L#435 is only one case. i'm going to answer to other comments | 14:58 |
| *** hemnafk is now known as hemna | 14:58 | |
| e0ne | jgriffith: great comment! thanks | 14:58 |
| jgriffith | e0ne: there, I also added my 2 cents to the review | 15:03 |
| *** jungleboyj has joined #openstack-cinder | 15:03 | |
| e0ne | jgriffith: thanks for the help and advice. i very appreciate it | 15:04 |
| jgriffith | e0ne: certainly | 15:05 |
| *** Longgeek_ has quit IRC | 15:07 | |
| *** Longgeek has joined #openstack-cinder | 15:07 | |
| hemna | morning | 15:08 |
| jungleboyj | hemna: Morning. Happy Friday! | 15:08 |
| *** Miouge has quit IRC | 15:08 | |
| mtanino | jungleboyj: thanks :) | 15:09 |
| avishay | jgriffith: guitarzan: https://docs.google.com/document/d/1vX1P_b72pQXnrePRfbD0zbPM_QQlPDh9im57IMR_Yyk/edit?usp=sharing | 15:09 |
| hemna | yes it is! | 15:09 |
| avishay | jungleboyj: hemna: hello hello | 15:09 |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 15:10 |
| jungleboyj | mtanino: My pleasure. Thank you. | 15:10 |
| jungleboyj | avishay: Howdy sir! | 15:10 |
| e0ne | hemna: evening:) | 15:11 |
| guitarzan | avishay: great, cinder still works with noauth! :) | 15:13 |
| avishay | guitarzan: create and list worked ;) | 15:14 |
| guitarzan | that's great news imo | 15:14 |
| avishay | guitarzan: yep | 15:14 |
| *** cknight has quit IRC | 15:15 | |
| *** dannywilson has joined #openstack-cinder | 15:15 | |
| *** dannywilson has quit IRC | 15:17 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 15:17 |
| *** dannywilson has joined #openstack-cinder | 15:17 | |
| lpetrut | jgriffith: Hi. Could you please spare some time to take a look over this patch? It already got a +2, https://review.openstack.org/#/c/174291/ Merging this bug fix and the one depending on it is quite important for us as at the moment we have to apply them on the CI before running tests | 15:19 |
| *** rushil has quit IRC | 15:24 | |
| *** rushil has joined #openstack-cinder | 15:24 | |
| *** Miouge has joined #openstack-cinder | 15:25 | |
| *** e0ne is now known as e0ne_ | 15:29 | |
| *** e0ne_ is now known as e0ne | 15:30 | |
| *** markvoelker has joined #openstack-cinder | 15:30 | |
| *** e0ne is now known as e0ne_ | 15:30 | |
| *** e0ne_ is now known as e0ne | 15:30 | |
| *** patrickeast has joined #openstack-cinder | 15:31 | |
| *** markvoelker has quit IRC | 15:34 | |
| *** tsekiyama has joined #openstack-cinder | 15:36 | |
| *** daneyon has joined #openstack-cinder | 15:36 | |
| *** daneyon has quit IRC | 15:38 | |
| *** daneyon has joined #openstack-cinder | 15:38 | |
| hemna | jgriffith, hey man, what happened with the replication spec. I noticed it was abandoned this morning. I was going to take another look at it. | 15:38 |
| patrickeast | : ( yea i just saw that | 15:39 |
| jgriffith | hemna: patrickeast I abandoned it | 15:40 |
| jgriffith | well... obviously :) | 15:40 |
| hemna | ?! | 15:40 |
| patrickeast | jgriffith: so… like forever? | 15:40 |
| hemna | just curious why ? | 15:40 |
| jgriffith | I dunno | 15:40 |
| hemna | It looked like we were getting close to finishing it up no ? | 15:41 |
| jgriffith | Well it's been almost six months and no real progress, seems like it's not really going to work | 15:41 |
| patrickeast | i don’t want that thing to die… i’m willing to take it up and argue with the ibm folks to get it through | 15:41 |
| jgriffith | hemna: I thought so, until that last round of -1's | 15:41 |
| dannywilson | patrickeast: +1 | 15:41 |
| hemna | oh wrt to the failover ? | 15:41 |
| hemna | I forgot, but what did we agree to wrt failover in Vancouver ? | 15:41 |
| hemna | what's described in the spec? | 15:42 |
| patrickeast | i think if we scale back how much automatic failover happens, so its just that cinder replication-failover <volume-d> command and not any volume operation that hits the scheduler it would alieve a lot of their concerns | 15:42 |
| patrickeast | and as far as the enable/disable | 15:43 |
| patrickeast | well who gives a shit if v1 had it | 15:43 |
| jgriffith | patrickeast: yeah, so what I think I'll try is maybe a POC | 15:43 |
| jgriffith | patrickeast: see if that helps with some of this | 15:43 |
| patrickeast | jgriffith: that seems pretty reasonable | 15:43 |
| jgriffith | frankly the spec process just burns me out :) | 15:44 |
| hemna | so it looks like there are some good questions in the review, but there are also some comments that just don't seem reasonable to me | 15:44 |
| patrickeast | but its soooo close | 15:44 |
| *** Tross has joined #openstack-cinder | 15:44 | |
| jgriffith | I'll try a poc, then build a spec off of it | 15:45 |
| jgriffith | see if that helps | 15:45 |
| hemna | I'd rather see us get something in that works for more than 1 vendor, rather than trying to solve every replication gotcha and waiting another 5 years. | 15:45 |
| jgriffith | should be able to start working on it today, maybe have something ready by Wednesdays Cinder meeting | 15:45 |
| jgriffith | hemna: +1 | 15:45 |
| jgriffith | hemna: patrickeast honestly there's another approach as well | 15:46 |
| dannywilson | jgriffith: maybe we can help on spec while you work on POC? | 15:46 |
| *** david-lyle has quit IRC | 15:46 | |
| jgriffith | dannywilson: sure | 15:46 |
| *** david-lyle has joined #openstack-cinder | 15:46 | |
| jgriffith | part of the reason I've abandoned it now is that it's gone in circles on a couple things, and frankly I'm not even sure it's "right" anymore :) | 15:47 |
| dannywilson | jgriffith: what do you think about the suggestion from patrickeast above about scaling back on how much automatic failover happens? | 15:47 |
| *** rushil has quit IRC | 15:47 | |
| hemna | why not make it an admin thing for now? | 15:48 |
| jgriffith | dannywilson: that's actually the *easy* part | 15:48 |
| hemna | dunno, I'm not a replication expert | 15:48 |
| jgriffith | hemna: well that's what it is currently | 15:48 |
| jgriffith | hemna: so the thing is, the comments that are coming in now are mostly just pushing it back to what the current implementation already is | 15:48 |
| jgriffith | so let me work on it this afternoon | 15:48 |
| hemna | which never worked for anyone other than 1 vendor | 15:49 |
| jgriffith | see if we can get something finished up | 15:49 |
| jgriffith | hemna: yeah, but remember they have *20* customers just last week alone that started using it | 15:49 |
| hemna | I'd hate to see this fail again, but I understand how complex it is and getting a gazillion vendors to buy off on something that makes them all happy. | 15:50 |
| jgriffith | hemna: patrickeast dannywilson Just give me a couple days to try something different | 15:50 |
| jgriffith | and see if if works | 15:50 |
| hemna | we have to remember as a community that we should take small steps to get things right. | 15:50 |
| *** markus_z has quit IRC | 15:50 | |
| dannywilson | jgriffith: okay, let us know if we can help out in any way, we can also try a POC on our driver if you think that would help | 15:50 |
| smcginnis | hemna: +1 | 15:50 |
| hemna | and sometimes that means putting something into place that doesn't make everyone happy/work the first time around. | 15:50 |
| jgriffith | hemna: TBH I think part of the problem is a misunderstanding of the flexibility I'm proposing | 15:50 |
| *** leeantho has joined #openstack-cinder | 15:51 | |
| jgriffith | dannywilson: that's what I was thinking, a POC might be better than having everybody interpret the "words" in a spec | 15:51 |
| hemna | This is probably one of the most complex features for Cinder | 15:51 |
| jgriffith | I honestly think people will be fairly happy with what I'm trying to put together | 15:51 |
| hemna | so it's hard for me to wrap my head around it all in a spec. | 15:51 |
| *** sgotliv has joined #openstack-cinder | 15:52 | |
| hemna | but my own issue. | 15:52 |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 15:54 |
| jgriffith | hemna: I'd agree with you on that | 15:54 |
| jgriffith | hemna: and I think that's the problem all the way around | 15:54 |
| jgriffith | hemna: so maybe a spec WITH some POC code will help all of us | 15:54 |
| *** _cjones_ has joined #openstack-cinder | 15:55 | |
| *** sgotliv has quit IRC | 15:55 | |
| *** sgotliv has joined #openstack-cinder | 15:55 | |
| hemna | jgriffith, +1 | 15:56 |
| *** Miouge has quit IRC | 16:00 | |
| lpetrut | hemma: Related to your comment regarding the Microsoft CI skipping attach test, where you reffering to test_attach_detach_volume ? | 16:00 |
| *** jistr has quit IRC | 16:03 | |
| hemna | lpetrut, yah, both the linux and windows smbfs skip the attach test | 16:05 |
| *** shyama has quit IRC | 16:06 | |
| *** shyama has joined #openstack-cinder | 16:06 | |
| lpetrut | hemma: well, it seems that the validation.run_validation config option was not set, defaulting to False, in which case some of the tests are skipped. Thanks for bringing this up! We'll enable this. | 16:08 |
| *** IanGovett has quit IRC | 16:08 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls https://review.openstack.org/182396 | 16:09 |
| *** avishay has quit IRC | 16:13 | |
| *** merooney has quit IRC | 16:14 | |
| thingee | DuncanT: casual review friday | 16:22 |
| thingee | I'm guessing people are doing easy reviews today | 16:22 |
| *** Miouge has joined #openstack-cinder | 16:23 | |
| *** e0ne is now known as e0ne_ | 16:23 | |
| openstackgerrit | John Griffith proposed openstack/cinder-specs: Cinder move drivers/plugins to external libs https://review.openstack.org/184951 | 16:24 |
| jgriffith | eharney: maybe that's better, maybe it's not ^^ | 16:24 |
| *** e0ne_ has quit IRC | 16:24 | |
| jgriffith | patrickeast: smcginnis ^^ would alos be curious to get your thoughts on the alternative suggestion | 16:25 |
| *** sgotliv has quit IRC | 16:26 | |
| smcginnis | jgriffith: Will take a look now. | 16:26 |
| * patrickeast looking | 16:26 | |
| smcginnis | jgriffith: I actually lean somewhat toward alternative 2. | 16:27 |
| smcginnis | jgriffith: I think there's something to be said for public visibility of the driver code. | 16:28 |
| smcginnis | And public shaming. :) | 16:28 |
| jgriffith | smcginnis: yeah, I figured most people would :) | 16:28 |
| jgriffith | smcginnis: the only other question is still, "Am I the only one that cares about this at all" | 16:28 |
| jgriffith | smcginnis: which is fine | 16:28 |
| jgriffith | if I am | 16:28 |
| smcginnis | jgriffith: I don't think that's the case. | 16:29 |
| smcginnis | jgriffith: I've heard a lot of people behind this general idea. | 16:29 |
| *** jordanP has joined #openstack-cinder | 16:29 | |
| smcginnis | jgriffith: I can just say from our experience, having our driver go through the standard review process has definitely made it better. | 16:29 |
| patrickeast | smcginnis: +1 | 16:29 |
| smcginnis | jgriffith: We could have just implemented our driver in isolation, but we would have missed a lot. | 16:30 |
| patrickeast | i doesn’t need to be checked into the cinder repo, but having cinder reviewers look at our drivers has been hugely important | 16:30 |
| Swanson | +1 | 16:30 |
| smcginnis | My preference would be for a different repo with maybe some different requirements or even a different set of cores. | 16:30 |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects https://review.openstack.org/188535 | 16:30 |
| smcginnis | The biggest concern I have is someone does their own driver in isolation and it has problems. | 16:31 |
| smcginnis | Then an end user tries to use it and hits some bugs. | 16:31 |
| smcginnis | Yes, it will make that vendor look bad and it will fall on them to fix it. | 16:31 |
| smcginnis | But it will also give somewhat an impression that Cinder/OpenStack quality is subpar. | 16:32 |
| jgriffith | smcginnis: so myself, I'm not concerned about that at all... because it's that Vendors problem and not Cinders :) | 16:32 |
| jgriffith | smcginnis: but yes, I see your point | 16:32 |
| Swanson | Drivers "in the box" is a plus for cinder. | 16:32 |
| jgriffith | patrickeast: yeah, I get it | 16:32 |
| jgriffith | so I agree with everything you all are saying, that's why I modified to propose a compromise | 16:32 |
| smcginnis | jgriffith: Agreed, but I think it affects the overall impression of Cinder even if it isn't part of cinder proper. | 16:32 |
| *** jdurgin1 has quit IRC | 16:32 | |
| smcginnis | jgriffith: I appreciate that. Good for discussion. | 16:33 |
| smcginnis | jgriffith: I'm certainly open to being in the minority here too, but it's good to talk about. | 16:33 |
| *** dmz has joined #openstack-cinder | 16:33 | |
| *** merooney has joined #openstack-cinder | 16:34 | |
| *** crose has joined #openstack-cinder | 16:35 | |
| *** anshul has quit IRC | 16:36 | |
| *** yamada-h has joined #openstack-cinder | 16:37 | |
| *** breitz has quit IRC | 16:38 | |
| *** breitz has joined #openstack-cinder | 16:39 | |
| *** e0ne has joined #openstack-cinder | 16:40 | |
| *** Daviey has joined #openstack-cinder | 16:41 | |
| *** yamada-h has quit IRC | 16:42 | |
| *** Miouge has quit IRC | 16:42 | |
| *** e0ne has quit IRC | 16:43 | |
| patrickeast | jgriffith: i like the new option… but plz no subrepo | 16:43 |
| patrickeast | i worked on a project with a bunch of subrepos and i still have nightmares | 16:43 |
| patrickeast | or crazy sub-tree shenanigans | 16:44 |
| *** Miouge has joined #openstack-cinder | 16:44 | |
| *** IanGovett has joined #openstack-cinder | 16:44 | |
| smcginnis | No shenanigans allowed. :) | 16:44 |
| *** Miouge has quit IRC | 16:45 | |
| *** Miouge has joined #openstack-cinder | 16:46 | |
| *** annegentle has quit IRC | 16:46 | |
| patrickeast | my personal rule of thumb is that if you have to type git submodule or git subtree you’ve done something wrong | 16:47 |
| patrickeast | although i might be biased | 16:47 |
| *** jordanP has quit IRC | 16:48 | |
| Daviey | Hi, I'm having a problem with XtremIO driver initialisation. Anyone here familiar with EMC kit? | 16:48 |
| patrickeast | xyang: ^ | 16:49 |
| xyang | Daviey: hi, what is issue? | 16:50 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 16:51 |
| Daviey | xyang: When XtremeIO loads it tries to contact http://x/api/json/types/xms , which is 404 | 16:51 |
| xyang | Daviey: where did you get the driver and what xtremio version are you running | 16:52 |
| Daviey | xyang: current master, 3.0.3 | 16:52 |
| xyang | Daviey: can you send me logs? I'll ask the xtremio team to take a look. my email: xing.yang@emc.com | 16:53 |
| Daviey | ta | 16:53 |
| *** annegentle has joined #openstack-cinder | 16:54 | |
| *** cknight has joined #openstack-cinder | 16:55 | |
| flip214 | thingee: ping? | 16:56 |
| thingee | flip214: hi | 16:56 |
| flip214 | hi. | 16:56 |
| flip214 | from -infra: | 16:56 |
| flip214 | 18:55 < jeblair> flip214, asselin__, fungi: do we need a cinder sign-off on that? or is the cinder core team/ptl expecting this job to | 16:56 |
| *** cknight has quit IRC | 16:56 | |
| flip214 | be added?" | 16:56 |
| *** sasi has left #openstack-cinder | 16:56 | |
| *** cknight has joined #openstack-cinder | 16:56 | |
| flip214 | this is about https://review.openstack.org/#/c/188744, CI for the DRBD driver. | 16:56 |
| *** breitz has quit IRC | 16:57 | |
| *** cknight has left #openstack-cinder | 16:57 | |
| *** raunak has joined #openstack-cinder | 17:01 | |
| flip214 | thingee: apart from watching the results, do I need to submit the DRBD driver in some way to have it in the list of pending reviews? | 17:02 |
| flip214 | *results of CI tests against other commits, that is | 17:02 |
| thingee | propose the patch, I will compare what was already in merge...should be a quick merge. I will only merge once the CI passes though. | 17:03 |
| thingee | flip214: ^ | 17:03 |
| thingee | what was already merged in Kilo* | 17:03 |
| thingee | flip214: so wait to propose until that gate test is passing a few times would be my advice | 17:03 |
| flip214 | yeah, okay. | 17:04 |
| flip214 | well, what you reverted in April is quite a bit different from what I submitted, because of several review comments... | 17:04 |
| thingee | flip214: ok, just being optimistic of it being quick :) | 17:05 |
| thingee | I would like it to be quick. it's a friday afterall | 17:05 |
| thingee | DuncanT promised me casual review friday and he's not delivering | 17:06 |
| * thingee thinks he's probably already out dancing somewhere to care | 17:06 | |
| thingee | I'll bb | 17:06 |
| *** annegentle has quit IRC | 17:08 | |
| *** IanGovett has quit IRC | 17:10 | |
| *** vilobhmm has joined #openstack-cinder | 17:10 | |
| flip214 | thingee: I'll be gone soon, too, as it's already past 7pm here, and I've got to bring the kids to bed. | 17:10 |
| flip214 | I'll be back on Monday, though! | 17:11 |
| flip214 | have a nice weekend. | 17:11 |
| jgriffith | smcginnis: hemna DuncanT jungleboyj thingee https://review.openstack.org/#/c/188794/ | 17:11 |
| smcginnis | jgriffith: Small (but important) typo. | 17:12 |
| jgriffith | smcginnis: ruhroh | 17:12 |
| jgriffith | smcginnis: DOH!!!! | 17:12 |
| thingee | jgriffith: seems fine. We'll be doing the push on Monday. | 17:13 |
| thingee | After typo fix | 17:13 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient: Prep for 1.2.2 https://review.openstack.org/188794 | 17:13 |
| jgriffith | thingee: Oh | 17:13 |
| jgriffith | thingee: I was going to push it as soon as that landed | 17:14 |
| thingee | jgriffith: in the infra channel, I got a sense this isn't urgent to do on a Friday. | 17:14 |
| jgriffith | thingee: didn't know you had *plans* | 17:14 |
| jgriffith | Ok, well then never mind | 17:14 |
| thingee | Infra and I have plans :). Not just me | 17:14 |
| jgriffith | ok | 17:14 |
| thingee | Sorry | 17:14 |
| jgriffith | thingee: s'alright | 17:14 |
| *** merooney has quit IRC | 17:15 | |
| *** rushil has joined #openstack-cinder | 17:16 | |
| vilobhmm | done | 17:17 |
| vilobhmm | jgriffith : ^^ | 17:18 |
| *** merooney has joined #openstack-cinder | 17:18 | |
| *** asselin__ has quit IRC | 17:19 | |
| DuncanT | Thingee: I've got to come up with a new plan for casual review Fridays... I forgot the very important detail that Friday isn't a work day here. I'll start preparing a list Thursday I guess | 17:19 |
| *** markvoelker has joined #openstack-cinder | 17:19 | |
| DuncanT | Thingee: I was actually at the opera, not dancing, as it happens | 17:19 |
| jgriffith | vilobhmm: thanks | 17:20 |
| jgriffith | vilobhmm: apparantly it's a moot point, thingee "has plans" | 17:20 |
| openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Add Virtuozzo Storage Volume Driver https://review.openstack.org/188869 | 17:20 |
| vilobhmm | "plans" about what :P | 17:20 |
| jgriffith | vilobhmm: releasing a client version | 17:21 |
| jgriffith | vilobhmm: so that update was so I could release something as soon as it landed | 17:21 |
| jgriffith | vilobhmm: but there's already a schedule in place | 17:21 |
| *** crose has quit IRC | 17:22 | |
| *** markvoelker has quit IRC | 17:24 | |
| vilobhmm | alrite | 17:24 |
| thingee | jgriffith: I was ready to do the same commit an hour ago, but fungi requested this not happen on a Friday. Not my plans. | 17:25 |
| thingee | Exactly. | 17:25 |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 17:25 |
| *** harlowja has quit IRC | 17:27 | |
| jgriffith | thingee: Ahhh... yeah, that makes sense | 17:28 |
| jgriffith | thingee: I forgot about the "no client pushes on friday" thing | 17:28 |
| jgriffith | thingee: sorry about that, totally makes sense | 17:28 |
| jgriffith | thingee: maybe I can sort out the Kilo compat issues between now and then too | 17:28 |
| * smcginnis envisions DuncanT in a tux with opera glasses | 17:28 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Add iSCSI protocol support for IBM FlashSystem https://review.openstack.org/156206 | 17:29 |
| smcginnis | jgriffith: thingee So go ahead with https://review.openstack.org/#/c/188794/ but wait to push client? Or just wait all together? | 17:30 |
| *** xyang has quit IRC | 17:30 | |
| jgriffith | smcginnis: merging the bump in version num should be fine | 17:31 |
| jgriffith | smcginnis: thingee was just reminding me that we DONT push to pypi on Fridays :) | 17:31 |
| smcginnis | jgriffith: Done. OK, no casual pypi Fridays then. :) | 17:32 |
| thingee | :) | 17:32 |
| *** harlowja has joined #openstack-cinder | 17:32 | |
| jgriffith | smcginnis: LOL | 17:33 |
| *** rushiagr is now known as rushiagr_away | 17:40 | |
| *** lpetrut has quit IRC | 17:41 | |
| *** rushil has quit IRC | 17:42 | |
| *** krtaylor has quit IRC | 17:46 | |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 17:46 |
| openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions https://review.openstack.org/186580 | 17:47 |
| *** e0ne has joined #openstack-cinder | 17:48 | |
| smcginnis | jgriffith: thingee Before I do something wrong - is it OK to push through python-cinderclient global requirement updates. Just newer minimum version of keystoneclient. | 17:49 |
| *** rushil has joined #openstack-cinder | 17:50 | |
| *** annegentle has joined #openstack-cinder | 17:50 | |
| *** merooney has quit IRC | 17:52 | |
| *** Tross has quit IRC | 17:52 | |
| e0ne | smcginnis: why not if it passed CI? | 17:54 |
| thingee | smcginnis: seems fine to me. otherwise we'd have to do another push to not be conflicting on that as well | 17:54 |
| smcginnis | e0ne: That's what I assume, just didn't know if we had a policy on this. | 17:54 |
| thingee | it's the pypi dance | 17:54 |
| smcginnis | thingee: In general, as long as the OpenStack Proposal Bot patches look OK should those just be ninja'd through? | 17:55 |
| *** akerr has quit IRC | 17:56 | |
| * smcginnis off to last meeting of the day | 17:56 | |
| jgriffith | smcginnis: thingee wait | 17:56 |
| jgriffith | smcginnis: so that will introduce another problem | 17:57 |
| jgriffith | smcginnis: thingee that change says >=1.6.0 | 17:57 |
| jgriffith | smcginnis: thingee but Cinder kilo has: keystonemiddleware>=1.5.0,<1.6.0 | 17:58 |
| jgriffith | smcginnis: thingee not sure how those tie in? | 17:58 |
| jgriffith | smcginnis: thingee maybe the anser is in fact just "you can't run cinderclient on node" | 17:58 |
| e0ne | jgriffith: if i did't miss anything, smcginnis talks about keystoneclient, _not_ keyston middleware | 17:58 |
| jgriffith | e0ne: Yes, that's what I'm asking | 17:58 |
| jgriffith | e0ne: if that relationship matters or not | 17:59 |
| e0ne | good point | 17:59 |
| * e0ne verifying... | 17:59 | |
| *** thangp has quit IRC | 17:59 | |
| *** crose has joined #openstack-cinder | 17:59 | |
| e0ne | in kilo keystonemiddleware has: python-keystoneclient>=1.1.0 | 18:01 |
| e0ne | jgriffith, thingee: afair, keystoneclient is deprecated. should we switch to openstackclient instead? | 18:02 |
| *** nikeshm has joined #openstack-cinder | 18:02 | |
| jgriffith | e0ne: I'm staying out of that one ;) | 18:03 |
| *** rwsu has quit IRC | 18:04 | |
| *** rajinir_r has quit IRC | 18:12 | |
| *** akerr has joined #openstack-cinder | 18:12 | |
| *** earlephilhower has quit IRC | 18:14 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series driver: Remove caching logic https://review.openstack.org/185832 | 18:14 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Refactor class structure for FC https://review.openstack.org/185833 | 18:14 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Add Fibre Channel Support https://review.openstack.org/188649 | 18:14 |
| *** merooney has joined #openstack-cinder | 18:16 | |
| e0ne | jgriffith: :) | 18:18 |
| e0ne | eharney: hi. are you around? i would like to discuss some issues with ceph driver and eventlet before sending it to openstack-dev ml | 18:20 |
| *** krtaylor has joined #openstack-cinder | 18:20 | |
| *** earlephilhower has joined #openstack-cinder | 18:22 | |
| *** shyama has quit IRC | 18:23 | |
| *** kaisers1 has quit IRC | 18:35 | |
| *** mriedem is now known as mriedem_madmax | 18:35 | |
| *** mriedem_madmax has quit IRC | 18:35 | |
| *** kaisers1 has joined #openstack-cinder | 18:37 | |
| *** kaisers has quit IRC | 18:37 | |
| *** Tross has joined #openstack-cinder | 18:38 | |
| *** kaisers has joined #openstack-cinder | 18:39 | |
| *** xyang1 has joined #openstack-cinder | 18:41 | |
| thingee | jgriffith: e0ne hahah | 18:46 |
| *** harlowja has quit IRC | 18:46 | |
| jgriffith | thingee: :) | 18:47 |
| thingee | jgriffith: the cinderclient installed on nodes is a particular version though right? | 18:47 |
| jgriffith | thingee: I just updated the bug, I can't repro | 18:47 |
| jgriffith | thingee: the problem was reported as: "I'm running Kilo, WTF... you said I could set private types" | 18:48 |
| * jgriffith said.. you can just "pip install --upgrade python-cinderclient" | 18:48 | |
| thingee | \o/ | 18:48 |
| thingee | people should just be happy that it's friday | 18:48 |
| eharney | e0ne: i'm back | 18:48 |
| jgriffith | they said: "WTF, cinder --version PUUUKES now, oslo.cfg error" | 18:48 |
| thingee | I mean come on | 18:48 |
| jgriffith | ha | 18:49 |
| jgriffith | thingee: so I filed a bug... then I tried it in devstack and ... well | 18:50 |
| jgriffith | "it worked in devstack" | 18:50 |
| thingee | jgriffith: speaking of that, did you see the new shirt at the summit? | 18:50 |
| * jgriffith sends them a T-shirt | 18:50 | |
| jgriffith | I did not | 18:50 |
| e0ne | free T-shirts? | 18:50 |
| openstackgerrit | Joshua Huber proposed openstack/cinder: Add cinder volume driver for Blockbridge EPS https://review.openstack.org/178295 | 18:51 |
| e0ne | eharney: actually, i've got 2 questions | 18:51 |
| e0ne | eharney: i want to try implement retries for rbd driver on connection timeouts | 18:52 |
| e0ne | eharney: do you think it good idea? i just started testing work with timeouts so i don't know almost anything about realted errors | 18:52 |
| *** harlowja has joined #openstack-cinder | 18:53 | |
| *** ociuhandu has quit IRC | 18:53 | |
| eharney | e0ne: it seems like a reasonable idea to me | 18:53 |
| *** ctracey has quit IRC | 18:54 | |
| *** serverascode has quit IRC | 18:54 | |
| *** thingee has quit IRC | 18:54 | |
| *** wanghao has quit IRC | 18:54 | |
| *** rhefner has quit IRC | 18:54 | |
| *** scottda has quit IRC | 18:54 | |
| *** ameade has quit IRC | 18:54 | |
| *** briancurtin has quit IRC | 18:54 | |
| *** zhiyan has quit IRC | 18:54 | |
| e0ne | eharney: do you have any expirience with timeouts on your envs? they are disabled by default | 18:54 |
| eharney | e0ne: no, i don't | 18:54 |
| e0ne | eharney: thanks. one more questions | 18:56 |
| *** lpetrut has joined #openstack-cinder | 18:57 | |
| e0ne | eharney: i don't have good enough steps to reproduce, but some times we've got an issue, when cinder tries to connect to ceph for delete volume and this thread (real python thread, eventlet.tpool) hangs | 18:57 |
| *** rushil has quit IRC | 18:57 | |
| e0ne | eharney: did you see anything related? | 18:58 |
| eharney | e0ne: well, i think that's what came up in the Nova review of the threaded delete patch, that it could deadlock like that | 18:58 |
| eharney | e0ne: what's in Cinder now seems to be unsafe | 18:58 |
| *** rushil has joined #openstack-cinder | 18:59 | |
| e0ne | eharney: do you mean this https://review.openstack.org/#/c/180212/ one? | 18:59 |
| *** raunak has quit IRC | 18:59 | |
| eharney | e0ne: yes | 19:00 |
| *** angela-s has joined #openstack-cinder | 19:01 | |
| e0ne | eharney: in my env, only threads with rdb call are locked. main thread of c-vol feels good and working | 19:01 |
| eharney | e0ne: well... it sounds like that could still be related, though it's hard to say if that's the expected outcome | 19:03 |
| uberjay | hemna: hey there -- how would you feel about taking another look at https://review.openstack.org/178295 for me? rebased due to conflict in exceptions.py, but otherwise the same. | 19:03 |
| e0ne | eharney: agree, it could be related. i'm just confused that threads are blocked and python-rbd doesn't delete volumes without any messaged in cinder and ceph logs | 19:06 |
| e0ne | TBH, i didn't set debug logs for ceph | 19:07 |
| asselin | hemna: https://review.openstack.org/188557 | 19:08 |
| *** markvoelker has joined #openstack-cinder | 19:08 | |
| asselin | devstack patch to pull in dev version of os-brick | 19:08 |
| asselin | devstack-gate change is: https://review.openstack.org/#/c/188604/ | 19:09 |
| asselin | which will update devstack jobs to setup the workspace with os-brick git cloned with zuul ref | 19:09 |
| uberjay | hemna: you had -1'd me before, but I think just for requests vs. httplib & lack of utilization reporting. if there's more to it than that, i'd like to get on it asap, since the fast-approaching deadline is making me nervous! | 19:10 |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 19:10 |
| *** Lee1092 has quit IRC | 19:11 | |
| *** markvoelker has quit IRC | 19:12 | |
| *** annegentle has quit IRC | 19:14 | |
| *** rushil has quit IRC | 19:14 | |
| *** dannywilson has quit IRC | 19:16 | |
| *** Longgeek has quit IRC | 19:16 | |
| *** rushil has joined #openstack-cinder | 19:16 | |
| *** pradipta has joined #openstack-cinder | 19:19 | |
| *** rushil has quit IRC | 19:20 | |
| *** rushil has joined #openstack-cinder | 19:23 | |
| *** jnicklin has joined #openstack-cinder | 19:23 | |
| *** merooney has quit IRC | 19:27 | |
| *** rushil has quit IRC | 19:31 | |
| *** leakypipes has quit IRC | 19:34 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Remove duplicate volume.filters entry https://review.openstack.org/188899 | 19:36 |
| *** e0ne has quit IRC | 19:39 | |
| *** merooney has joined #openstack-cinder | 19:41 | |
| *** earlephilhower has quit IRC | 19:48 | |
| *** akerr has quit IRC | 19:51 | |
| *** Yogi1 has quit IRC | 19:51 | |
| *** annegentle has joined #openstack-cinder | 19:54 | |
| *** xyang has joined #openstack-cinder | 19:55 | |
| *** dims__ has joined #openstack-cinder | 19:57 | |
| *** merooney has quit IRC | 19:58 | |
| xyang | asselin: ping | 19:59 |
| hemna | uberjay, heyas | 20:00 |
| hemna | what's up | 20:00 |
| *** dims_ has quit IRC | 20:00 | |
| *** boris-42 has quit IRC | 20:02 | |
| *** rex_lee has quit IRC | 20:02 | |
| *** sweston has quit IRC | 20:02 | |
| *** annegentle has quit IRC | 20:04 | |
| *** annegentle has joined #openstack-cinder | 20:04 | |
| *** dustins has quit IRC | 20:07 | |
| uberjay | hemna: hey there | 20:08 |
| *** asselin_ has joined #openstack-cinder | 20:10 | |
| *** dims__ has quit IRC | 20:11 | |
| *** rushil has joined #openstack-cinder | 20:12 | |
| uberjay | hemna: I'm thinking our driver is ready for merge, and any additional fixes and features should go in as an additional patch. I certainly don't want to bother core devs if I don't have to, but the deadline is soon, which makes me nervous. Have any guidance? :) | 20:14 |
| hemna | uberjay, ok I'll look again | 20:14 |
| uberjay | hemna: thanks, I appreciate it! | 20:15 |
| smcginnis | uberjay: Blockbridge, right? | 20:15 |
| uberjay | smcginnis: yep, that's us :) | 20:16 |
| smcginnis | uberjay: You have some pep8 errors to fix. | 20:16 |
| smcginnis | Sorry | 20:16 |
| *** e0ne has joined #openstack-cinder | 20:16 | |
| uberjay | ack, must have snuck in. i'll fix them. | 20:16 |
| *** tellesnobrega_ has joined #openstack-cinder | 20:17 | |
| *** rushil has quit IRC | 20:17 | |
| hemna | uberjay, what's the value in providing the blockbridge_volume_query cinder.conf entry ? | 20:17 |
| hemna | why do admins need that ? | 20:17 |
| hemna | that seems like a driver internals thingy | 20:17 |
| *** ociuhandu has joined #openstack-cinder | 20:19 | |
| uberjay | the volume query is used to select a subset of storage advertised via a particular blockbridge storage service -- essentially an attribute query string. | 20:20 |
| smcginnis | uberjay: Also, is there really a need for BlockBridgeAPIException instead of just using VolumeBackupAPIException? | 20:21 |
| smcginnis | I see the code actually does both. | 20:21 |
| hemna | uberjay, why would an admin need that ? | 20:22 |
| uberjay | there's some overlap with the cinder scheduler, of course. e.g., consider some storage service that is used by cinder and non-cinder clients. the query gives the admin a way to select what resources should be consumed by cinder. | 20:22 |
| hemna | uberjay, like a pool for provisioning a volume ? | 20:23 |
| uberjay | smcginnis: (i'll take a look, thanks) | 20:23 |
| *** markvoelker has joined #openstack-cinder | 20:24 | |
| uberjay | hemna: something like that, yes. the pool of backend datastores used for provisioning is selectable by attribute. the admin may configure this on the backend, or via the cinder driver configuration. for example, you could use it to expose two different pools of storage (as different volume types), both provisioned via the same backend blockbridge service. | 20:25 |
| hemna | uberjay, heh ok | 20:26 |
| hemna | well, I'd suggest using Cinder's built in mechanism for 'pool's to do this then | 20:26 |
| hemna | instead of cinder.conf or extra specs | 20:26 |
| hemna | you'll be back dooring the scheduler's built in support for pools | 20:27 |
| hemna | which will cause you problems | 20:27 |
| hemna | a better approach is to list the pools you want to support in cinder.conf | 20:27 |
| hemna | then have your get_volume_stats report on those pools | 20:27 |
| uberjay | can you point me at some more details about storage pools? we don't really have a storage pool concept -- we just have datastores, which may or may not be tagged with a particular attribute does that make sense? I'm not sure if it maps well, but it may. | 20:27 |
| hemna | and let the cinder scheduler pick which pool to use. | 20:28 |
| *** markvoelker has quit IRC | 20:28 | |
| hemna | uberjay, https://github.com/openstack/cinder-specs/blob/master/specs/juno/pool-aware-cinder-scheduler.rst | 20:28 |
| hemna | uberjay, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/san/hp/hp_3par_common.py#L734-L758 | 20:32 |
| hemna | uberjay, that's what we do for our driver. | 20:33 |
| hemna | we have a list of 'pool's that volumes can be provisioned from and we report stats up for each of those. | 20:33 |
| hemna | the scheduler actually picts which one is used. | 20:33 |
| hemna | uberjay, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/san/hp/hp_3par_common.py#L89-L91 | 20:33 |
| hemna | comes from that ^^^ | 20:33 |
| uberjay | mm, gotcha. currently all we could do is directly expose each discrete datastore as a pool to support the pool-aware scheduler. it doesn't really express the full abilities of *our* provisioning query support, though. ah, interesting. i'm looking. | 20:34 |
| *** annashen has joined #openstack-cinder | 20:34 | |
| uberjay | ah, that's actually a great way to go. that's much better than having to define multiple driver instances to select different pools. | 20:35 |
| hemna | yup | 20:36 |
| hemna | :) | 20:36 |
| uberjay | ultimately, though, the way we select each "pool" of storage boils down to a query (for our system) | 20:36 |
| hemna | sure, just abstract into something you can render into your query that goes into the query on line 322 | 20:37 |
| hemna | meh, that made no sense | 20:37 |
| hemna | heh | 20:37 |
| uberjay | heh :) | 20:37 |
| hemna | if you have some description or name of the store/pool, then just use that as the pool name list in cinder.conf | 20:37 |
| hemna | then render that name into the query that is used as part of the _create_volume() call | 20:38 |
| *** tellesnobrega_ has quit IRC | 20:38 | |
| *** tellesnobrega_ has joined #openstack-cinder | 20:39 | |
| uberjay | sure, yes. that would work. Essentially the blockbridge_query_string *is* the pool name. | 20:42 |
| hemna | there ya go | 20:42 |
| hemna | :) | 20:42 |
| hemna | so I'd probably rename that conf entry and call it something that makes sense, but make it a list of supported 'pool' names. | 20:43 |
| hemna | and FWIW, this is a prime example of why keeping drivers in tree is helpful for driver writers. | 20:43 |
| hemna | :( | 20:43 |
| hemna | and forcing core devs to do driver reviews. | 20:44 |
| uberjay | can conf options be dicts? | 20:44 |
| hemna | not sure | 20:44 |
| hemna | can you do a CSV ? | 20:44 |
| hemna | that's what we do | 20:44 |
| hemna | foo,bar,blah,something sucky, something less sucky, awesomeness | 20:44 |
| uberjay | you use a list, presumably because your pool names map directly into some backend object name | 20:45 |
| *** e0ne has quit IRC | 20:45 | |
| smcginnis | hemna: +1 on benefits of in tree as well as pool names. :D | 20:45 |
| *** ctracey has joined #openstack-cinder | 20:46 | |
| hemna | smcginnis, but I love the idea of pip install <vendor driver> | 20:46 |
| hemna | right now we can't really fix bugs against older shipping releases. | 20:46 |
| smcginnis | I wonder if we had it in a different repo if we could do something a la cinderclient to "publish" them. | 20:46 |
| smcginnis | Then have them pulled down on demand. | 20:46 |
| smcginnis | So all in tree, but not all in source distro. | 20:47 |
| hemna | 40 subprojects of cinder | 20:47 |
| hemna | P A I N | 20:47 |
| smcginnis | 50 shades of cinder? | 20:47 |
| hemna | :P | 20:47 |
| xyang | smcginnis: hi, the ScaleIO CI name is fixed now. you can filter it out now | 20:47 |
| smcginnis | Not each a subproject. | 20:47 |
| smcginnis | xyang: Cool! | 20:47 |
| uberjay | yeah, I was reading that discussion and thinking the same. I can see how getting the core team away from the large (and growing) 3rd party drivers would be a big relief, but... quality control and such. unfortunately, people are going to come to you when a driver reaches behind and corrupts the cinder database. :( | 20:47 |
| hemna | smcginnis, so every driver is in 1 subproject repo cinder-drivers ? | 20:47 |
| smcginnis | hemna: I was just thinking on repo "cinder-drivers" and have some way to pull out each driver as a module published to be able to pip install per driver. | 20:48 |
| *** e0ne has joined #openstack-cinder | 20:48 | |
| smcginnis | hemna: No idea if that's at all feasible, just brainstorming on a Friday afternoon. :) | 20:49 |
| *** pradipta has quit IRC | 20:49 | |
| *** lpetrut has quit IRC | 20:49 | |
| *** serverascode has joined #openstack-cinder | 20:49 | |
| uberjay | run 3rd party drivers in another process/sandbox, maybe? ;) *grin* | 20:49 |
| *** ameade has joined #openstack-cinder | 20:51 | |
| *** gouthamr has quit IRC | 20:51 | |
| *** sweston has joined #openstack-cinder | 20:53 | |
| hemna | smcginnis, yah I'm not sure how you could separate each individual driver in that shared repo. | 20:54 |
| *** zhiyan has joined #openstack-cinder | 20:54 | |
| smcginnis | hemna: Script that expects a certain folder structure maybe? | 20:54 |
| hemna | I guess it's just a higher level dir that contains dirs of pip package dirs | 20:54 |
| smcginnis | /cinder-drivers/driver/* | 20:54 |
| smcginnis | for each driver in /cinder-drivers/drivers/* | 20:55 |
| smcginnis | Something along those lines. | 20:55 |
| smcginnis | I'm sure I'm vastly simplifying what would be a complex task though. | 20:55 |
| hemna | yah seems doable | 20:56 |
| smcginnis | Heading out. Have a good weekend everyone. | 20:58 |
| mtanino | smcginnis: you too! | 20:58 |
| hemna | smcginnis, l8rs! | 20:58 |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 20:59 |
| *** boris-42 has joined #openstack-cinder | 21:00 | |
| hemna | speaking of moving drivers out of tree.... | 21:00 |
| *** briancurtin has joined #openstack-cinder | 21:05 | |
| *** e0ne has quit IRC | 21:06 | |
| *** krtaylor has quit IRC | 21:08 | |
| *** ociuhandu has quit IRC | 21:09 | |
| *** tellesnobrega_ has quit IRC | 21:11 | |
| *** raunak has joined #openstack-cinder | 21:11 | |
| *** thingee has joined #openstack-cinder | 21:12 | |
| *** bswartz has joined #openstack-cinder | 21:13 | |
| *** rhefner has joined #openstack-cinder | 21:13 | |
| *** scottda has joined #openstack-cinder | 21:14 | |
| uberjay | hemna smcginnis -- thanks for your feedback. i'm taking care of the issues :) | 21:15 |
| *** wanghao has joined #openstack-cinder | 21:17 | |
| *** akerr has joined #openstack-cinder | 21:20 | |
| *** akerr_ has joined #openstack-cinder | 21:21 | |
| Swanson | xyang: jgriffith thanks for pushing through 182396. | 21:23 |
| *** akerr has quit IRC | 21:25 | |
| jgriffith | Swanson: sure | 21:28 |
| *** esker has quit IRC | 21:29 | |
| xyang | Swanson: welcome | 21:29 |
| *** rhagarty has quit IRC | 21:32 | |
| *** cbader has quit IRC | 21:32 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 21:36 |
| *** Miouge has quit IRC | 21:37 | |
| *** akerr_ has quit IRC | 21:38 | |
| *** openstack has joined #openstack-cinder | 21:43 | |
| *** rmesta has quit IRC | 21:43 | |
| *** rex_lee has joined #openstack-cinder | 21:43 | |
| *** rmesta has joined #openstack-cinder | 21:44 | |
| *** rmesta has quit IRC | 21:44 | |
| *** jnicklin has quit IRC | 21:44 | |
| *** rmesta has joined #openstack-cinder | 21:44 | |
| *** rmesta has quit IRC | 21:44 | |
| *** rmesta has joined #openstack-cinder | 21:45 | |
| *** rmesta has quit IRC | 21:45 | |
| *** rmesta has joined #openstack-cinder | 21:45 | |
| *** krtaylor has joined #openstack-cinder | 21:54 | |
| *** earlephilhower has joined #openstack-cinder | 21:54 | |
| *** primechuck has quit IRC | 21:59 | |
| *** dannywilson has joined #openstack-cinder | 22:03 | |
| *** dannywilson has quit IRC | 22:03 | |
| *** earlephilhower has quit IRC | 22:06 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 22:11 |
| *** markvoelker has joined #openstack-cinder | 22:12 | |
| *** earlephilhower has joined #openstack-cinder | 22:15 | |
| *** markvoelker has quit IRC | 22:17 | |
| *** earlephilhower has quit IRC | 22:19 | |
| *** annashen has quit IRC | 22:19 | |
| *** earlephilhower has joined #openstack-cinder | 22:19 | |
| *** jungleboyj has quit IRC | 22:21 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for alternate iscsi portals https://review.openstack.org/182760 | 22:21 |
| *** xyang has quit IRC | 22:26 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions https://review.openstack.org/186580 | 22:26 |
| *** xyang1 has quit IRC | 22:29 | |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 22:31 |
| *** daneyon has quit IRC | 22:33 | |
| *** daneyon has joined #openstack-cinder | 22:33 | |
| *** daneyon has quit IRC | 22:33 | |
| *** daneyon has joined #openstack-cinder | 22:34 | |
| hemna | tbarron, ping | 22:35 |
| *** openstackgerrit has quit IRC | 22:37 | |
| *** openstackgerrit has joined #openstack-cinder | 22:37 | |
| earlephilhower | thingee: Howdy, Can I ask a 3rd party Cinder CI question? Read your post on it 2015-May/064072.html, but unclear about CI posting its results to gerrit prior to approval of new driver. | 22:39 |
| asselin | earlephilhower, your ci shoudl post resutls prior to approval | 22:40 |
| hemna | earlephilhower, do you have a driver you want to submit to Cinder ? | 22:41 |
| *** lcurtis has quit IRC | 22:41 | |
| earlephilhower | asselin: Thx. So real Zuul-initiated gerrit comment addition, or just link to the logs from a manual started Jenkins run? I'm running your setup (well, as it was 2 weeks ago before diskbuilder change). | 22:42 |
| tbarron | hemna: pong | 22:43 |
| asselin | earlephilhower, actually, you should test your ci with openstack-dev/ci-sandbox | 22:43 |
| hemna | tbarron, so the compression patch | 22:43 |
| tbarron | a | 22:43 |
| tbarron | ya | 22:43 |
| earlephilhower | hemna: Yes, I've got the BP up along w/code. Working through the rest of the formalities now (I hope). | 22:43 |
| asselin | send me the link of the log files I can take a quick look. once you're happy witht the way it works in the sandbox you can enable in cinder. | 22:43 |
| hemna | tbarron, since this patch basically makes it possible to have blocks that aren't compressed, does the restore of the backup work correctly? | 22:44 |
| asselin | earlephilhower, be careful when posting to cinder. If you create too much noise (i.e. brokensystem not adding any value) your accound can get disabled | 22:44 |
| hemna | tbarron, dealing with compressed vs. uncompressed blocks? | 22:44 |
| tbarron | hemna: yes | 22:44 |
| tbarron | each chunk has its own metadata | 22:44 |
| tbarron | which includes the type of compression, including 'none' | 22:44 |
| tbarron | restore uses this | 22:44 |
| earlephilhower | asselin: Yes, it's been committing to the ci-sandbox for a while now. Just hooked up to openstack/cinder on silent mode and chugging on Nexenta patch. Logs at http://208.82.102.191:8000/$LOG_PATH/ (LOG_PATH is fixed for real gerrit stuff) | 22:45 |
| tbarron | all that pre-existed this change | 22:45 |
| hemna | tbarron, ah ok cool. that was my only concern with this since I was reading it and thinking about it. | 22:45 |
| tbarron | pls do look down at the restore a chunk method, or whatever that's called | 22:45 |
| asselin | earlephilhower, what do you mean 'fixed for real gerrit stuff? | 22:45 |
| tbarron | you are asking the right question and shouldn't just take my word for it | 22:45 |
| hemna | tbarron, ah ok I see the 'none', data being returned and that's getting stuffed into the metadata | 22:45 |
| earlephilhower | asselin, Got it! I saw the IBM system that went loopy.. LOG_PATH "fixed" means it comes out as "http://208.82.102.191:8000/80/185780/3/check/noop-check-communication/bfe2eb6" | 22:46 |
| earlephilhower | Just the manually started Jenkins jobs don't define LOG_PATH | 22:46 |
| earlephilhower | So it ends up with ugly URL | 22:47 |
| asselin | ok, yeah, do you have one of your driver? | 22:47 |
| earlephilhower | Yes, the "$LOG_PATH" one. It really had dollarsign-LOGPATH in the URL: http://208.82.102.191:8000/$LOG_PATH/ | 22:48 |
| earlephilhower | I didn't think "$" was legit in URLs, guess I was wrong... | 22:48 |
| asselin | earlephilhower, ok...you don't have one on silent mode? | 22:48 |
| asselin | earlephilhower, real patch being ci-ed? | 22:49 |
| *** ozialien has joined #openstack-cinder | 22:49 | |
| earlephilhower | asselin, do you mean like one initiated by gerrit and uploaded to the logserver but no gerrit review sent back? | 22:50 |
| asselin | correct | 22:50 |
| tbarron | hemna: in restore_v1 you'll see an iteration through metadata_objects. For each, we look at the compression algo | 22:51 |
| tbarron | (which may be 'none') | 22:51 |
| hemna | tbarron, ok perfect. thanks. I was just curious and concerned | 22:51 |
| tbarron | or rather, at this point None | 22:51 |
| tbarron | and if not None, we decompress | 22:52 |
| earlephilhower | Just kicked silent mode on this afternoon, and only 2 patches since have come through. Looks like about 10 minutes left on the first one. | 22:52 |
| tbarron | It was when I realized that the compression algo was saved for each chunk that I realized we could do this. | 22:52 |
| earlephilhower | asselin: logs uploading right now, actually...one sec for url | 22:53 |
| hemna | tbarron, so I presume the 'none' algorithm simply means jam this block behind the last one and continue ? | 22:53 |
| tbarron | more or less. it means write it at the designated offset and continue. | 22:54 |
| hemna | cool | 22:54 |
| hemna | makes sense. | 22:54 |
| tbarron | incrementals do a full restore (where all writes end up being continguous) then jump around on subsequent passes | 22:55 |
| tbarron | contiguous | 22:56 |
| tbarron | not "around" - always forwards, but not contiguously | 22:56 |
| earlephilhower | asselin: Logs from silent mode checkin here: http://208.82.102.191:8000/80/186580/8/silent/dsvm-tempest-hgst-cinder-driver/02708a4/ | 22:56 |
| * asselin looks | 22:57 | |
| tbarron | hemna: thanks, walt! | 22:57 |
| hemna | tbarron, np | 22:58 |
| asselin | earlephilhower, overall very nice! | 22:59 |
| asselin | earlephilhower, a couple of this. | 22:59 |
| asselin | 1. please increae your regex to 'volume' | 22:59 |
| asselin | 2. http://208.82.102.191:8000/80/186580/8/silent/dsvm-tempest-hgst-cinder-driver/02708a4/logs/testr_results.html.gz doesn't show as html | 22:59 |
| asselin | are you using my log server script? that should set it up correctly....https://github.com/rasselin/os-ext-testing/blob/master/puppet/install_log_server.sh | 23:00 |
| asselin | which will also setup log-analyze (optional, but a big plus) | 23:00 |
| asselin | in the mean time, I personally don't have issues with your ci commenting as -is | 23:01 |
| earlephilhower | asselin, thanks. #1 easy-peasy. #2, I'm gonna have to break out my apache-fu. We've got some corporate requirements for external servers so I couldn't use your script as-is. I kind of hit the bare-minimum as a start. | 23:01 |
| asselin | earlephilhower, ok, fair enough....take a look at the apache conf for clues then | 23:02 |
| *** ozialien has quit IRC | 23:02 | |
| thingee | earlephilhower: for companies with requirements poking holes in the firewall to allow access with some webserver, I get that. Just use some external web server. | 23:02 |
| thingee | that's managed elsewhere | 23:02 |
| asselin | earlephilhower, https://github.com/openstack-infra/puppet-openstackci/blob/master/templates/logs.vhost.erb | 23:03 |
| thingee | as a last resort though | 23:03 |
| hemna | asselin, jenkins is happy now. | 23:04 |
| asselin | yeah! | 23:04 |
| earlephilhower | asselin, I'll grok that link as best I can. I'm actually running on an external server already, but it's also serving some other needs so I couldn't change things too greatly. | 23:04 |
| hemna | thingee, FYI https://review.openstack.org/#/c/188557 | 23:04 |
| asselin | thingee, please...and all your ptl friends :) | 23:05 |
| * thingee wishes he had friends | 23:05 | |
| *** ozialien has joined #openstack-cinder | 23:05 | |
| * asselin will friend for +1 | 23:06 | |
| hemna | :) | 23:06 |
| earlephilhower | asselin, so I'm going to update the regex right now and see about the couple log-things. Appreciate the feedback! | 23:06 |
| asselin | you're welcome | 23:06 |
| openstackgerrit | Mikhail Khodos proposed openstack/cinder: Nexenta Edge iSCSI backend driver. https://review.openstack.org/158745 | 23:08 |
| earlephilhower | asselin, so if over the weekend on silent I don't see anything bad it's OK to move to check on Mon? Don't want to move to check right before weekend, and have to hack stuff at 3AM Sat. nite. | 23:08 |
| asselin | earlephilhower, yeah that's fine | 23:09 |
| hemna | have a good weekend guys. | 23:09 |
| *** hemna is now known as hemnafk | 23:09 | |
| *** dannywilson has joined #openstack-cinder | 23:15 | |
| *** angela-s has quit IRC | 23:16 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Revert "Adds drivers for DotHill Storage Arrays." https://review.openstack.org/188969 | 23:17 |
| *** annegentle has quit IRC | 23:18 | |
| *** annashen has joined #openstack-cinder | 23:20 | |
| *** dannywilson has quit IRC | 23:23 | |
| *** dannywilson has joined #openstack-cinder | 23:23 | |
| *** annashen has quit IRC | 23:26 | |
| *** ian_brown has quit IRC | 23:33 | |
| *** ozialien has quit IRC | 23:33 | |
| *** daneyon has quit IRC | 23:34 | |
| *** ian_brown has joined #openstack-cinder | 23:37 | |
| *** jwcroppe has joined #openstack-cinder | 23:41 | |
| *** ian_brown has quit IRC | 23:42 | |
| *** raunak has quit IRC | 23:45 | |
| *** ian_brown has joined #openstack-cinder | 23:47 | |
| mtanino | jgriffith: Thanks for that comment :) | 23:50 |
| *** _cjones_ has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!