| *** dannywilson has quit IRC | 00:00 | |
| *** gouthamr has joined #openstack-cinder | 00:00 | |
| vincent_hou_ | thingee: Hi Mike. If you have got a time, could you take a look at the spec https://review.openstack.org/#/c/186327/? | 00:00 |
|---|---|---|
| *** salv-orlando has joined #openstack-cinder | 00:01 | |
| *** topshare has quit IRC | 00:02 | |
| *** Guest7393 has quit IRC | 00:04 | |
| *** vincent_hou_ has quit IRC | 00:05 | |
| *** IlyaG has joined #openstack-cinder | 00:10 | |
| *** jwcroppe has quit IRC | 00:11 | |
| *** mgarza has quit IRC | 00:11 | |
| *** topshare has joined #openstack-cinder | 00:11 | |
| *** chlong has joined #openstack-cinder | 00:12 | |
| *** garthb has quit IRC | 00:14 | |
| *** salv-orlando has quit IRC | 00:14 | |
| *** topshare has quit IRC | 00:15 | |
| *** gouthamr_ has joined #openstack-cinder | 00:18 | |
| *** vokt has quit IRC | 00:19 | |
| *** gouthamr has quit IRC | 00:20 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:22 | |
| *** dims has joined #openstack-cinder | 00:23 | |
| *** jamielennox is now known as jamielennox|away | 00:24 | |
| *** tsekiyama has quit IRC | 00:26 | |
| patrickeast | thingee: hey, when you get a chance could you target https://blueprints.launchpad.net/cinder/+spec/cinder-internal-tenant and https://blueprints.launchpad.net/cinder/+spec/image-volume-cache for liberty-2? | 00:26 |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: VMEM v6000: Fix export verify routines to use proper state nodes https://review.openstack.org/165655 | 00:26 |
| *** tsekiyam_ has quit IRC | 00:27 | |
| *** Apoorva has joined #openstack-cinder | 00:27 | |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP Generic image cache. https://review.openstack.org/195795 | 00:28 |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support. https://review.openstack.org/199352 | 00:28 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Improve 3PAR driver VLUN creation and deletion https://review.openstack.org/196164 | 00:30 |
| *** mtanino has quit IRC | 00:30 | |
| thingee | patrickeast: done | 00:31 |
| patrickeast | thingee: thanks! | 00:31 |
| *** Apoorva has quit IRC | 00:32 | |
| *** jamielennox|away is now known as jamielennox | 00:35 | |
| *** leeantho has quit IRC | 00:42 | |
| *** jasondotstar has quit IRC | 00:43 | |
| *** diogogmt has quit IRC | 00:49 | |
| *** yamada-h has joined #openstack-cinder | 00:51 | |
| *** topshare has joined #openstack-cinder | 00:57 | |
| *** yamada-h has quit IRC | 00:57 | |
| *** kkeuning has joined #openstack-cinder | 01:01 | |
| *** kkeuning has quit IRC | 01:02 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:11 | |
| *** salv-orlando has joined #openstack-cinder | 01:12 | |
| *** heyun has joined #openstack-cinder | 01:15 | |
| *** Longgeek has joined #openstack-cinder | 01:15 | |
| *** yamada-h has joined #openstack-cinder | 01:17 | |
| *** heyun has quit IRC | 01:18 | |
| *** jungleboyj has joined #openstack-cinder | 01:18 | |
| *** heyun has joined #openstack-cinder | 01:19 | |
| *** davechen1 has joined #openstack-cinder | 01:19 | |
| *** Longgeek has quit IRC | 01:20 | |
| *** yamada-h has quit IRC | 01:20 | |
| *** jdurgin1 has quit IRC | 01:22 | |
| *** annegentle has joined #openstack-cinder | 01:23 | |
| *** davechen has joined #openstack-cinder | 01:23 | |
| *** yamada-h has joined #openstack-cinder | 01:24 | |
| *** dims_ has joined #openstack-cinder | 01:24 | |
| *** dims_ has quit IRC | 01:24 | |
| *** kkeuning has joined #openstack-cinder | 01:25 | |
| *** yamada-h has quit IRC | 01:25 | |
| *** davechen1 has quit IRC | 01:25 | |
| *** salv-orlando has quit IRC | 01:26 | |
| *** kkeuning has quit IRC | 01:26 | |
| *** kkeuning has joined #openstack-cinder | 01:27 | |
| *** dims has quit IRC | 01:27 | |
| *** annegentle has quit IRC | 01:27 | |
| *** kkeuning has quit IRC | 01:28 | |
| *** Apoorva has joined #openstack-cinder | 01:28 | |
| *** Apoorva has quit IRC | 01:33 | |
| *** IlyaG has quit IRC | 01:38 | |
| *** diogogmt has joined #openstack-cinder | 01:38 | |
| *** Yogi11 has joined #openstack-cinder | 01:39 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 01:44 |
| *** laughterwym_ has joined #openstack-cinder | 01:50 | |
| *** laughterwym has quit IRC | 01:53 | |
| *** asselin_ has joined #openstack-cinder | 01:55 | |
| *** kkeuning has joined #openstack-cinder | 01:57 | |
| *** akerr has joined #openstack-cinder | 01:57 | |
| *** Ctina has joined #openstack-cinder | 01:58 | |
| *** akerr_ has joined #openstack-cinder | 02:00 | |
| *** kkeuning has quit IRC | 02:00 | |
| *** jskinner has joined #openstack-cinder | 02:02 | |
| *** akerr has quit IRC | 02:03 | |
| *** jskinner has quit IRC | 02:04 | |
| *** jwcroppe has joined #openstack-cinder | 02:07 | |
| *** Yogi11 has quit IRC | 02:09 | |
| *** jwcroppe has quit IRC | 02:11 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support. https://review.openstack.org/199352 | 02:14 |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP Generic image cache. https://review.openstack.org/195795 | 02:14 |
| *** Lee1092 has joined #openstack-cinder | 02:18 | |
| *** jasondotstar has joined #openstack-cinder | 02:22 | |
| *** vincent_hou has joined #openstack-cinder | 02:22 | |
| *** mutoulbj has joined #openstack-cinder | 02:22 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 02:23 |
| *** Apoorva has joined #openstack-cinder | 02:29 | |
| *** patrickeast has quit IRC | 02:29 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/python-cinderclient: Add more details for replication https://review.openstack.org/196562 | 02:30 |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 02:30 |
| *** Apoorva has quit IRC | 02:34 | |
| *** annegentle has joined #openstack-cinder | 02:34 | |
| *** bkopilov has quit IRC | 02:36 | |
| *** gouthamr has joined #openstack-cinder | 02:37 | |
| *** annegentle has quit IRC | 02:37 | |
| *** gouthamr_ has quit IRC | 02:39 | |
| *** BharatK has joined #openstack-cinder | 02:45 | |
| *** changbl has joined #openstack-cinder | 02:48 | |
| *** hakimo_ has joined #openstack-cinder | 02:52 | |
| *** hakimo has quit IRC | 02:54 | |
| openstackgerrit | Dave Chen proposed openstack/cinder: set/unset volume image metadata https://review.openstack.org/147726 | 02:54 |
| *** kkeuning has joined #openstack-cinder | 02:55 | |
| vincent_hou | smcginnis: Hey. | 02:55 |
| openstackgerrit | LisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph https://review.openstack.org/198993 | 02:56 |
| *** rajinir has joined #openstack-cinder | 02:59 | |
| *** jasondotstar has quit IRC | 03:02 | |
| *** jwcroppe has joined #openstack-cinder | 03:08 | |
| *** salv-orlando has joined #openstack-cinder | 03:13 | |
| *** jwcroppe has quit IRC | 03:13 | |
| *** vincent_hou has quit IRC | 03:15 | |
| *** Longgeek has joined #openstack-cinder | 03:16 | |
| openstackgerrit | wanghao proposed openstack/cinder: Validate value when user update quota https://review.openstack.org/197938 | 03:18 |
| *** Longgeek has quit IRC | 03:22 | |
| *** kkeuning has quit IRC | 03:23 | |
| openstackgerrit | Lin Yang proposed openstack/python-cinderclient: Improve error message when cinder quota exceeded https://review.openstack.org/178946 | 03:27 |
| *** xyang1 has quit IRC | 03:28 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** Apoorva has joined #openstack-cinder | 03:30 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 03:31 |
| *** vincent_hou has joined #openstack-cinder | 03:31 | |
| vincent_hou | smcginnis: https://review.openstack.org/#/c/194223/ this is another important fix. | 03:32 |
| *** Apoorva has quit IRC | 03:34 | |
| *** rajinir has quit IRC | 03:35 | |
| *** annegentle has joined #openstack-cinder | 03:37 | |
| *** garthb has joined #openstack-cinder | 03:38 | |
| *** BharatK has quit IRC | 03:40 | |
| *** annegentle has quit IRC | 03:41 | |
| openstackgerrit | Lin Yang proposed openstack/python-cinderclient: Remove redundant statement and refactor https://review.openstack.org/196546 | 03:42 |
| *** davechen is now known as davechen_away | 03:45 | |
| *** patrickeast has joined #openstack-cinder | 03:45 | |
| *** patrickeast has quit IRC | 03:49 | |
| *** patrickeast has joined #openstack-cinder | 03:49 | |
| *** patrickeast has quit IRC | 03:50 | |
| *** patrickeast has joined #openstack-cinder | 03:51 | |
| *** BharatK has joined #openstack-cinder | 03:51 | |
| *** boris-42 has quit IRC | 03:52 | |
| *** Apoorva has joined #openstack-cinder | 03:55 | |
| *** bkopilov has joined #openstack-cinder | 03:57 | |
| *** _cjones_ has quit IRC | 03:57 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 04:00 |
| *** annashen has joined #openstack-cinder | 04:01 | |
| *** gouthamr has quit IRC | 04:06 | |
| *** mtreinish has quit IRC | 04:07 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: change create and delete operation https://review.openstack.org/197393 | 04:08 |
| *** r-daneel has joined #openstack-cinder | 04:13 | |
| *** simondodsley has joined #openstack-cinder | 04:14 | |
| *** jwcroppe has joined #openstack-cinder | 04:15 | |
| *** mtreinish has joined #openstack-cinder | 04:16 | |
| *** simondodsley has quit IRC | 04:19 | |
| *** david-ly_ has joined #openstack-cinder | 04:19 | |
| *** diogogmt has quit IRC | 04:20 | |
| *** sgotliv has joined #openstack-cinder | 04:21 | |
| *** david-lyle has quit IRC | 04:22 | |
| *** Apoorva has quit IRC | 04:26 | |
| *** annashen has quit IRC | 04:28 | |
| *** salv-orlando has joined #openstack-cinder | 04:29 | |
| *** akerr_ is now known as akerr_away | 04:33 | |
| *** chlong has quit IRC | 04:35 | |
| *** lpetrut has joined #openstack-cinder | 04:40 | |
| *** salv-orlando has quit IRC | 04:42 | |
| *** laughterwym__ has joined #openstack-cinder | 04:43 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:43 | |
| *** chlong has joined #openstack-cinder | 04:44 | |
| *** laughterwym_ has quit IRC | 04:46 | |
| *** vincent_hou_ has joined #openstack-cinder | 04:47 | |
| *** vincent_hou has quit IRC | 04:49 | |
| *** vincent_hou has joined #openstack-cinder | 04:49 | |
| *** chlong has quit IRC | 04:51 | |
| *** yamada-h has joined #openstack-cinder | 04:52 | |
| *** garthb has quit IRC | 04:52 | |
| *** shyama has joined #openstack-cinder | 04:52 | |
| *** changbl has quit IRC | 04:52 | |
| *** mutoulbj has quit IRC | 04:52 | |
| *** mutoulbj_ has joined #openstack-cinder | 04:53 | |
| *** vincent_hou_ has quit IRC | 04:53 | |
| *** davechen_away is now known as davechen | 04:57 | |
| *** nkrinner_afk is now known as nkrinner | 04:58 | |
| *** yamada-h_ has joined #openstack-cinder | 05:09 | |
| *** dims has joined #openstack-cinder | 05:11 | |
| *** nikeshm has joined #openstack-cinder | 05:12 | |
| *** yamada-h has quit IRC | 05:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 05:13 |
| *** lpetrut has quit IRC | 05:13 | |
| *** Longgeek has joined #openstack-cinder | 05:16 | |
| *** deepakcs has joined #openstack-cinder | 05:17 | |
| *** mutoulbj has joined #openstack-cinder | 05:18 | |
| *** chlong has joined #openstack-cinder | 05:20 | |
| *** boris-42 has joined #openstack-cinder | 05:21 | |
| *** mutoulbj_ has quit IRC | 05:22 | |
| *** annegentle has joined #openstack-cinder | 05:26 | |
| *** rakesh_mishra__ has joined #openstack-cinder | 05:26 | |
| *** Apoorva has joined #openstack-cinder | 05:27 | |
| *** shausy has joined #openstack-cinder | 05:29 | |
| *** annegentle has quit IRC | 05:30 | |
| *** Apoorva has quit IRC | 05:32 | |
| *** vincent_hou has quit IRC | 05:33 | |
| *** davechen_afk is now known as jungler | 05:37 | |
| *** dims has quit IRC | 05:40 | |
| *** rakesh_mishra__ has quit IRC | 05:47 | |
| *** rakesh_mishra__ has joined #openstack-cinder | 05:48 | |
| *** ig0r__ has joined #openstack-cinder | 05:51 | |
| *** yamada-h_ has quit IRC | 05:52 | |
| *** sgotliv has quit IRC | 05:52 | |
| *** sgotliv has joined #openstack-cinder | 05:52 | |
| *** ig0r_ has quit IRC | 05:52 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Fix doc string definitions https://review.openstack.org/193124 | 05:54 |
| *** nlevinki has joined #openstack-cinder | 05:55 | |
| *** nihilifer has joined #openstack-cinder | 05:59 | |
| *** lpetrut has joined #openstack-cinder | 06:03 | |
| *** lpetrut has quit IRC | 06:04 | |
| *** nlevinki has quit IRC | 06:05 | |
| *** rakesh_mishra__ has quit IRC | 06:08 | |
| *** rakesh_mishra__ has joined #openstack-cinder | 06:10 | |
| *** bkopilov has quit IRC | 06:19 | |
| *** patrickeast has quit IRC | 06:19 | |
| *** tobe has joined #openstack-cinder | 06:19 | |
| *** nlevinki has joined #openstack-cinder | 06:20 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 06:22 |
| *** yamada-h has joined #openstack-cinder | 06:24 | |
| *** Apoorva has joined #openstack-cinder | 06:28 | |
| *** rakesh_mishra__ has quit IRC | 06:28 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 06:28 | |
| *** salv-orlando has joined #openstack-cinder | 06:31 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 06:31 |
| *** coolsvap|away is now known as coolsvap | 06:32 | |
| *** bkopilov has joined #openstack-cinder | 06:32 | |
| *** Apoorva has quit IRC | 06:32 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 06:35 |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 06:39 |
| *** salv-orlando has quit IRC | 06:42 | |
| *** markus_z has joined #openstack-cinder | 06:46 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 06:51 |
| *** harlowja_at_home has quit IRC | 06:52 | |
| *** xiangfeiz has joined #openstack-cinder | 06:56 | |
| *** yrabl has quit IRC | 06:58 | |
| *** yrabl has joined #openstack-cinder | 06:58 | |
| *** markus_z has quit IRC | 06:58 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: change create and delete operation https://review.openstack.org/197393 | 06:59 |
| *** salv-orlando has joined #openstack-cinder | 07:03 | |
| *** yamada-h has quit IRC | 07:05 | |
| *** yamada-h has joined #openstack-cinder | 07:10 | |
| *** annegentle has joined #openstack-cinder | 07:14 | |
| *** annegentle has quit IRC | 07:19 | |
| *** yamada-h has quit IRC | 07:19 | |
| *** rakesh_mishra_ has quit IRC | 07:20 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 07:22 | |
| *** xiangfeiz has quit IRC | 07:22 | |
| *** anshul has joined #openstack-cinder | 07:24 | |
| *** e0ne has joined #openstack-cinder | 07:25 | |
| *** yamada-h has joined #openstack-cinder | 07:26 | |
| *** Apoorva has joined #openstack-cinder | 07:28 | |
| *** Apoorva has quit IRC | 07:33 | |
| *** afazekas has joined #openstack-cinder | 07:34 | |
| *** jordanP has joined #openstack-cinder | 07:35 | |
| *** rakesh_mishra_ has quit IRC | 07:36 | |
| *** jwcroppe has quit IRC | 07:37 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 07:38 |
| *** dims has joined #openstack-cinder | 07:41 | |
| *** dims_ has joined #openstack-cinder | 07:42 | |
| *** akerr_away has quit IRC | 07:43 | |
| *** jistr has joined #openstack-cinder | 07:43 | |
| *** ndipanov has joined #openstack-cinder | 07:45 | |
| *** dims has quit IRC | 07:45 | |
| *** dims_ has quit IRC | 07:46 | |
| *** tobe has quit IRC | 07:46 | |
| *** bluex-pl has joined #openstack-cinder | 07:47 | |
| *** markvoelker has quit IRC | 07:47 | |
| *** e0ne is now known as e0ne_ | 07:49 | |
| *** e0ne_ is now known as e0ne | 07:51 | |
| *** ronis has joined #openstack-cinder | 07:52 | |
| *** afazekas has quit IRC | 07:56 | |
| *** e0ne is now known as e0ne_ | 07:57 | |
| *** e0ne_ is now known as e0ne | 07:59 | |
| *** e0ne has quit IRC | 08:00 | |
| *** anshul has quit IRC | 08:00 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 08:00 |
| *** ociuhandu has joined #openstack-cinder | 08:01 | |
| *** lpetrut has joined #openstack-cinder | 08:08 | |
| *** afazekas has joined #openstack-cinder | 08:09 | |
| *** ociuhandu has quit IRC | 08:13 | |
| *** tobe has joined #openstack-cinder | 08:16 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects https://review.openstack.org/196071 | 08:17 |
| *** chlong has quit IRC | 08:21 | |
| *** tobe has quit IRC | 08:25 | |
| openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds friendly zone name support https://review.openstack.org/180518 | 08:25 |
| *** tobe has joined #openstack-cinder | 08:25 | |
| *** Apoorva has joined #openstack-cinder | 08:29 | |
| *** yamada-h has quit IRC | 08:29 | |
| *** boris-42 has quit IRC | 08:32 | |
| *** Apoorva has quit IRC | 08:33 | |
| *** belmoreira has joined #openstack-cinder | 08:38 | |
| *** tobe has quit IRC | 08:40 | |
| *** abehl has joined #openstack-cinder | 08:42 | |
| *** dims has joined #openstack-cinder | 08:43 | |
| *** lpetrut has quit IRC | 08:43 | |
| *** dims_ has joined #openstack-cinder | 08:44 | |
| *** laughterwym__ has quit IRC | 08:44 | |
| *** lpetrut has joined #openstack-cinder | 08:44 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: IBM storwize driver configuration item default valume change https://review.openstack.org/198692 | 08:45 |
| *** dims__ has joined #openstack-cinder | 08:45 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:45 |
| *** dims___ has joined #openstack-cinder | 08:46 | |
| *** lixiaoy1 has quit IRC | 08:47 | |
| *** dims has quit IRC | 08:47 | |
| *** markvoelker has joined #openstack-cinder | 08:48 | |
| *** dims_ has quit IRC | 08:49 | |
| *** dims__ has quit IRC | 08:49 | |
| *** dims___ has quit IRC | 08:50 | |
| *** markvoelker has quit IRC | 08:53 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:58 |
| *** lpetrut has quit IRC | 09:02 | |
| *** e0ne has joined #openstack-cinder | 09:03 | |
| *** annegentle has joined #openstack-cinder | 09:03 | |
| *** lpetrut has joined #openstack-cinder | 09:03 | |
| *** shausy has quit IRC | 09:07 | |
| *** shausy has joined #openstack-cinder | 09:07 | |
| *** annegentle has quit IRC | 09:08 | |
| *** r-daneel has quit IRC | 09:08 | |
| *** afazekas has quit IRC | 09:12 | |
| *** belmoreira has quit IRC | 09:12 | |
| *** markus_z has joined #openstack-cinder | 09:16 | |
| *** laughterwym has joined #openstack-cinder | 09:19 | |
| *** tobe has joined #openstack-cinder | 09:20 | |
| *** e0ne is now known as e0ne_ | 09:21 | |
| *** afazekas has joined #openstack-cinder | 09:21 | |
| *** belmoreira has joined #openstack-cinder | 09:24 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Checking the existence of volume https://review.openstack.org/174823 | 09:24 |
| *** e0ne_ is now known as e0ne | 09:26 | |
| *** Apoorva has joined #openstack-cinder | 09:30 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 09:30 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 09:30 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Service object https://review.openstack.org/160417 | 09:30 |
| *** mutoulbj_ has joined #openstack-cinder | 09:33 | |
| *** haypo has joined #openstack-cinder | 09:34 | |
| *** Apoorva has quit IRC | 09:34 | |
| *** mutoulbj has quit IRC | 09:36 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:36 | |
| *** e0ne is now known as e0ne_ | 09:37 | |
| *** nikeshm has quit IRC | 09:38 | |
| *** IanGovett has joined #openstack-cinder | 09:39 | |
| *** rakesh_mishra has quit IRC | 09:44 | |
| *** ociuhandu has joined #openstack-cinder | 09:44 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:45 | |
| *** jasondotstar has joined #openstack-cinder | 09:45 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 09:45 |
| *** nihilifer has quit IRC | 09:52 | |
| *** davechen has left #openstack-cinder | 09:53 | |
| *** aix has joined #openstack-cinder | 09:54 | |
| *** rakesh_mishra has quit IRC | 09:56 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:56 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 09:58 |
| *** e0ne_ is now known as e0ne | 10:00 | |
| *** rakesh_mishra has quit IRC | 10:02 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:03 | |
| *** rakesh_mishra has quit IRC | 10:03 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:05 | |
| *** mutoulbj_ has quit IRC | 10:09 | |
| *** e0ne is now known as e0ne_ | 10:11 | |
| *** e0ne_ is now known as e0ne | 10:13 | |
| *** zhenguo has quit IRC | 10:26 | |
| *** tobe has quit IRC | 10:29 | |
| *** afazekas has quit IRC | 10:29 | |
| *** Apoorva has joined #openstack-cinder | 10:31 | |
| *** coolsvap is now known as coolsvap|away | 10:33 | |
| *** Apoorva has quit IRC | 10:35 | |
| deepakcs | jordanP: https://review.openstack.org/#/c/192736/8/nova/virt/libvirt/driver.py , quick Q... | 10:39 |
| deepakcs | jordanP: active_disk_object.source_path <-- does this work correct when disk of of type 'network' ? I haven't tried in recent time, thus asking | 10:39 |
| *** afazekas has joined #openstack-cinder | 10:40 | |
| *** e0ne is now known as e0ne_ | 10:40 | |
| *** e0ne_ is now known as e0ne | 10:42 | |
| *** ronis has quit IRC | 10:48 | |
| *** ronis has joined #openstack-cinder | 10:48 | |
| *** markvoelker has joined #openstack-cinder | 10:49 | |
| *** annegentle has joined #openstack-cinder | 10:52 | |
| *** rakesh_mishra has quit IRC | 10:53 | |
| *** markvoelker has quit IRC | 10:54 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:54 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: change snapshot and clone operation https://review.openstack.org/198952 | 10:54 |
| *** rakesh_mishra has quit IRC | 10:54 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:54 | |
| *** annegentle has quit IRC | 10:56 | |
| *** rakesh_mishra has quit IRC | 10:58 | |
| *** rakesh_mishra has joined #openstack-cinder | 10:58 | |
| *** heyun has quit IRC | 10:58 | |
| *** topshare has quit IRC | 11:02 | |
| *** ronis has quit IRC | 11:08 | |
| jordanP | deepakcs, you tell me :) | 11:11 |
| deepakcs | jordanP: i am not sure and right now i don't have a setup to validate that. I think it won't work | 11:11 |
| deepakcs | jordanP: you need to use _get_snap_dev if active_protocol is not None (which means its a network disk) | 11:11 |
| jordanP | deepakcs, : rebase_base = _get_snap_dev(delete_info['file_to_merge'], | 11:12 |
| jordanP | active_disk_object.backing_store) | 11:12 |
| jordanP | L1924 | 11:12 |
| *** afazekas has quit IRC | 11:13 | |
| deepakcs | jordanP: what are u trying to say ? | 11:13 |
| jordanP | you mean _get_snap_dev(active_disk_object.source_path) ? | 11:13 |
| deepakcs | jordanP: yeah... i am not sure if active_disk_object.source_path is correct value when disk is of 'network' type | 11:13 |
| jordanP | deepakcs, which driver uses "network" disk ? Mine is based on files... | 11:14 |
| *** ronis has joined #openstack-cinder | 11:14 | |
| deepakcs | jordanP: rbd, gluster, sheepdog also i guess. Its been a while for me to have tried network disk config | 11:14 |
| *** mutoulbj has joined #openstack-cinder | 11:15 | |
| deepakcs | jordanP: eharney (during US time) would be good to consult on this | 11:15 |
| jordanP | so I should setup a ceph ou glusterFS to test this corner case ? | 11:15 |
| jordanP | man, I just want to fix that for cinder drivers based on RemoteFS | 11:16 |
| jordanP | I cant"t fix it for every driver on eartch ! | 11:16 |
| deepakcs | jordanP: network disk type is a supported disk type in libvirt! It so happens that ur driver doesn't utilise that | 11:17 |
| deepakcs | jordanP: i will sent a mail to eharney to check this out. | 11:17 |
| jordanP | deepakcs, I understand that. But do I have to be the QA engineer for every drivers out there ? | 11:17 |
| jordanP | it's not me who disabled that tempest test for glusterfs driver, I am sorry | 11:18 |
| deepakcs | jordanP: no, but the fix should be generic enuf, it shouldn't break other disk types, right ? OTherwise there is no point of having remotefs abstraction | 11:18 |
| jordanP | it's not going to break anything. Because it's already broken | 11:18 |
| deepakcs | jordanP: i am not speaking as glusterfs driver person but speaking in general | 11:19 |
| jordanP | the ceph CI is running fine with and without my patch. So i can assume I didnt break it. | 11:19 |
| deepakcs | jordanP: ceph doesn't use online snapshot, so it doesn't go thru that path ( i was wrong when i said rbd before) | 11:20 |
| deepakcs | jordanP: afaik gluster, sheepdog and anything else based on remotefs would use it | 11:20 |
| jordanP | deepakcs, okay. So you want me to setup a glusterFS to test that ? | 11:21 |
| deepakcs | jordanP: the reason i m not sure is bcos libvirt doesn't provide me a simple way to figure what field (source_path or some otehr field) to use when disk is network type | 11:21 |
| *** rakesh_mishra has quit IRC | 11:21 | |
| deepakcs | jordanP: it would be easier to tlak with eharney who has dealt with this in the past. thats all i m saying! | 11:21 |
| jordanP | deepakcs, okay :) no rush, I'll try to reach eharney. | 11:22 |
| deepakcs | jordanP: i will try to be there, but its late in the eve for me, in any case pls check with him | 11:22 |
| *** ronis_ has joined #openstack-cinder | 11:22 | |
| deepakcs | i will leave a comment in the patch, won't rate it | 11:23 |
| jordanP | good. Sorry I a bit nervous. But each time I have to go through sending a patch to nova, I want to : ʕノ•ᴥ•ʔノ ︵ ┻━┻ | 11:23 |
| jordanP | :) | 11:23 |
| deepakcs | jordanP: what was the last part of ur msg ? Its greek to me | 11:24 |
| jordanP | deepakcs, just some kind of ascii art | 11:24 |
| deepakcs | jordanP: can't help it, since the code to manage file and network type disks are common part in driver.py.... I didn't design it that way | 11:24 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 11:24 |
| deepakcs | jordanP: definitely not ascii :) 'art' can't be sure | 11:25 |
| jordanP | utf8 art then :D | 11:25 |
| deepakcs | jordanP: let me do this, will send a mail to eharney, Ccing you, i am hoping that would be better | 11:26 |
| jordanP | deepakcs, even better. thanks | 11:26 |
| deepakcs | jordanP: np | 11:26 |
| *** topshare has joined #openstack-cinder | 11:26 | |
| *** ronis has quit IRC | 11:26 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject https://review.openstack.org/198341 | 11:28 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 11:29 |
| *** zhenguo has joined #openstack-cinder | 11:29 | |
| *** topshare has quit IRC | 11:30 | |
| *** Apoorva has joined #openstack-cinder | 11:31 | |
| *** e0ne is now known as e0ne_ | 11:34 | |
| *** e0ne_ is now known as e0ne | 11:35 | |
| deepakcs | jordanP: mail sent, lets hope for a response | 11:36 |
| *** Apoorva has quit IRC | 11:36 | |
| jordanP | deepakcs, it can wait one day or 2. It's not critical | 11:36 |
| deepakcs | jordanP: u never know how long it can take, so we do our best to close it :) | 11:37 |
| jordanP | yep | 11:37 |
| *** e0ne has quit IRC | 11:38 | |
| *** Longgeek has quit IRC | 11:41 | |
| *** Longgeek has joined #openstack-cinder | 11:42 | |
| *** Longgeek has quit IRC | 11:42 | |
| *** Longgeek has joined #openstack-cinder | 11:42 | |
| *** Longgeek has quit IRC | 11:43 | |
| *** kaisers_ has joined #openstack-cinder | 11:46 | |
| *** dims has joined #openstack-cinder | 11:47 | |
| *** zul has quit IRC | 11:48 | |
| *** dims_ has joined #openstack-cinder | 11:48 | |
| *** kaisers_97 has joined #openstack-cinder | 11:49 | |
| *** dims__ has joined #openstack-cinder | 11:49 | |
| *** markvoelker has joined #openstack-cinder | 11:50 | |
| *** asselin_ has quit IRC | 11:51 | |
| *** asselin_ has joined #openstack-cinder | 11:52 | |
| *** dims has quit IRC | 11:52 | |
| *** dims_ has quit IRC | 11:53 | |
| *** markvoelker has quit IRC | 11:54 | |
| *** dims__ has quit IRC | 11:54 | |
| *** kaisers_ has quit IRC | 11:54 | |
| *** topshare has joined #openstack-cinder | 11:54 | |
| *** kaisers_97 has quit IRC | 11:56 | |
| *** salv-orlando has quit IRC | 11:58 | |
| *** abehl has quit IRC | 11:58 | |
| *** abehl has joined #openstack-cinder | 11:59 | |
| *** timcl has joined #openstack-cinder | 12:00 | |
| *** abehl has quit IRC | 12:09 | |
| *** afazekas has joined #openstack-cinder | 12:11 | |
| *** bswartz has quit IRC | 12:12 | |
| *** markvoelker has joined #openstack-cinder | 12:13 | |
| *** belmoreira has quit IRC | 12:17 | |
| *** abehl has joined #openstack-cinder | 12:21 | |
| *** salv-orlando has joined #openstack-cinder | 12:21 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo._utils.fileutils https://review.openstack.org/199537 | 12:22 |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 12:23 |
| *** belmoreira has joined #openstack-cinder | 12:23 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 12:23 |
| *** Longgeek has joined #openstack-cinder | 12:25 | |
| *** jasondotstar has quit IRC | 12:25 | |
| *** annegentle has joined #openstack-cinder | 12:30 | |
| *** zul has joined #openstack-cinder | 12:30 | |
| *** afazekas has quit IRC | 12:31 | |
| *** chlong has joined #openstack-cinder | 12:31 | |
| *** ganso_ has joined #openstack-cinder | 12:32 | |
| *** tellesnobrega_ has joined #openstack-cinder | 12:32 | |
| *** kkeuning has joined #openstack-cinder | 12:32 | |
| *** kkeuning has quit IRC | 12:33 | |
| *** kjnelson has joined #openstack-cinder | 12:35 | |
| *** erlon has joined #openstack-cinder | 12:39 | |
| *** tellesnobrega__ has joined #openstack-cinder | 12:39 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Attach snapshot https://review.openstack.org/198556 | 12:41 |
| *** afazekas has joined #openstack-cinder | 12:42 | |
| *** tellesnobrega_ has quit IRC | 12:42 | |
| *** jungleboyj has quit IRC | 12:42 | |
| *** tellesnobrega_ has joined #openstack-cinder | 12:42 | |
| *** tellesnobrega__ has quit IRC | 12:46 | |
| *** akerr has joined #openstack-cinder | 12:48 | |
| *** dims has joined #openstack-cinder | 12:50 | |
| *** simondodsley has joined #openstack-cinder | 12:51 | |
| *** coolsvap|away is now known as coolsvap | 12:52 | |
| *** merooney has joined #openstack-cinder | 12:53 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 12:54 |
| *** dims has quit IRC | 12:54 | |
| *** afazekas has quit IRC | 12:55 | |
| *** eharney has joined #openstack-cinder | 12:59 | |
| *** j_king has quit IRC | 12:59 | |
| *** nkrinner has quit IRC | 12:59 | |
| *** Swanson has quit IRC | 13:00 | |
| *** abehl has quit IRC | 13:00 | |
| *** mfisch has quit IRC | 13:00 | |
| *** abehl has joined #openstack-cinder | 13:00 | |
| *** Swanson has joined #openstack-cinder | 13:00 | |
| *** j_king has joined #openstack-cinder | 13:00 | |
| *** tellesnobrega__ has joined #openstack-cinder | 13:01 | |
| *** julim has joined #openstack-cinder | 13:01 | |
| *** zhipeng has joined #openstack-cinder | 13:01 | |
| *** nkrinner has joined #openstack-cinder | 13:01 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers https://review.openstack.org/197999 | 13:02 |
| *** cdelatte has joined #openstack-cinder | 13:03 | |
| *** tellesnobrega_ has quit IRC | 13:03 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Attach snapshot https://review.openstack.org/198556 | 13:04 |
| *** timcl has quit IRC | 13:04 | |
| *** tellesnobrega_ has joined #openstack-cinder | 13:04 | |
| *** jasondotstar has joined #openstack-cinder | 13:05 | |
| *** abehl has quit IRC | 13:06 | |
| *** tellesnobrega__ has quit IRC | 13:06 | |
| *** xyang1 has joined #openstack-cinder | 13:07 | |
| *** tellesnobrega_ has quit IRC | 13:09 | |
| *** tellesnobrega_ has joined #openstack-cinder | 13:09 | |
| *** xyang has joined #openstack-cinder | 13:10 | |
| *** dustins has joined #openstack-cinder | 13:12 | |
| *** deepakcs has quit IRC | 13:14 | |
| *** afazekas has joined #openstack-cinder | 13:15 | |
| *** asselin__ has joined #openstack-cinder | 13:16 | |
| *** tellesnobrega__ has joined #openstack-cinder | 13:16 | |
| *** tellesnobrega_ has quit IRC | 13:18 | |
| *** asselin_ has quit IRC | 13:19 | |
| *** changbl has joined #openstack-cinder | 13:19 | |
| *** akerr has quit IRC | 13:21 | |
| *** akerr_ has joined #openstack-cinder | 13:21 | |
| *** tellesnobrega_ has joined #openstack-cinder | 13:21 | |
| *** abehl has joined #openstack-cinder | 13:21 | |
| *** dims has joined #openstack-cinder | 13:22 | |
| *** tellesnobrega__ has quit IRC | 13:24 | |
| *** mriedem_away is now known as mriedem | 13:24 | |
| *** timcl has joined #openstack-cinder | 13:25 | |
| *** tellesnobrega__ has joined #openstack-cinder | 13:27 | |
| *** akshai has joined #openstack-cinder | 13:31 | |
| *** tellesnobrega_ has quit IRC | 13:31 | |
| *** Yogi1 has joined #openstack-cinder | 13:31 | |
| *** salv-orlando has quit IRC | 13:33 | |
| smcginnis | DuncanT: You still alive? | 13:33 |
| *** eharney has quit IRC | 13:34 | |
| *** bswartz has joined #openstack-cinder | 13:37 | |
| *** Apoorva has joined #openstack-cinder | 13:38 | |
| *** thangp has joined #openstack-cinder | 13:38 | |
| *** gouthamr has joined #openstack-cinder | 13:40 | |
| *** annegentle has quit IRC | 13:45 | |
| *** annegentle has joined #openstack-cinder | 13:45 | |
| *** rushil has joined #openstack-cinder | 13:46 | |
| *** jwcroppe has joined #openstack-cinder | 13:49 | |
| *** Apoorva has quit IRC | 13:53 | |
| *** Apoorva has joined #openstack-cinder | 13:54 | |
| *** rakesh_mishra has joined #openstack-cinder | 13:55 | |
| *** rakesh_mishra has quit IRC | 13:55 | |
| *** rakesh_mishra has joined #openstack-cinder | 13:57 | |
| *** Apoorva has quit IRC | 13:59 | |
| *** dustins has quit IRC | 13:59 | |
| *** mclaren has joined #openstack-cinder | 14:00 | |
| mclaren | Howdy. Anyone available to take a look at https://review.openstack.org/#/c/173977? Some kilo gating issues may be related to the cinder client | 14:01 |
| *** Yogi1 has quit IRC | 14:01 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 14:01 |
| *** tellesnobrega_ has joined #openstack-cinder | 14:02 | |
| *** eharney has joined #openstack-cinder | 14:02 | |
| *** annegentle has quit IRC | 14:02 | |
| *** dustins has joined #openstack-cinder | 14:03 | |
| *** bkopilov has quit IRC | 14:03 | |
| *** rushil has quit IRC | 14:04 | |
| *** mtanino has joined #openstack-cinder | 14:04 | |
| *** edmondsw has joined #openstack-cinder | 14:05 | |
| *** shyama has quit IRC | 14:05 | |
| *** tellesnobrega__ has quit IRC | 14:05 | |
| *** tellesnobrega__ has joined #openstack-cinder | 14:06 | |
| *** tellesnobrega_ has quit IRC | 14:06 | |
| *** rushil has joined #openstack-cinder | 14:06 | |
| *** shyama has joined #openstack-cinder | 14:08 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: GlusterFS backup driver https://review.openstack.org/182254 | 14:14 |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Posix backup driver https://review.openstack.org/163647 | 14:14 |
| DuncanT | smcginnis: Hi. Technically not dead yet, though I feel like I'm slowly cooking.... | 14:17 |
| *** esker has joined #openstack-cinder | 14:17 | |
| DuncanT | thingee: jgriffith: Are you able to take a look at https://review.openstack.org/#/c/173977/ to unblock the stable gate, please? | 14:17 |
| smcginnis | DuncanT: ;) | 14:18 |
| smcginnis | DuncanT: Wonder about that script you had for abandoning old reviews. | 14:18 |
| smcginnis | DuncanT: Wondering if we should run that again. | 14:18 |
| smcginnis | DuncanT: Starting to get a lot out there that haven't been touched since early Kilo. | 14:18 |
| *** BharatK has quit IRC | 14:18 | |
| DuncanT | smcginnis: I certainly can do, yeah. I wonder where I put it? | 14:18 |
| *** timcl1 has joined #openstack-cinder | 14:19 | |
| DuncanT | smcginnis: I'll go have a dig | 14:19 |
| smcginnis | DuncanT: That's always my problem with those kinds of things. :) | 14:19 |
| *** mfisch has joined #openstack-cinder | 14:19 | |
| DuncanT | smcginnis: I keep saying I should push everything to github | 14:20 |
| *** mfisch has quit IRC | 14:20 | |
| *** mfisch has joined #openstack-cinder | 14:20 | |
| smcginnis | DuncanT: Not a bad idea actually. | 14:20 |
| DuncanT | smcginnis: Unfortuately, most of my scripts are bad enough I don't want to admit to having coded them ;-) | 14:20 |
| smcginnis | git clone duncansrandomcrap | 14:21 |
| smcginnis | ;) | 14:21 |
| *** setmason has joined #openstack-cinder | 14:21 | |
| *** timcl has quit IRC | 14:21 | |
| setmason | I’m trying to get a total volume count across all tenants, I’m using cinder list —all-tenants and it seems to be stopping at 1000 volumes. Is there a more efficient method of getting this data? | 14:22 |
| *** zhipeng has quit IRC | 14:22 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Add support for backups to xiv_ds8k driver https://review.openstack.org/118298 | 14:25 |
| scottda | setmason: What version of cinderclient are you using? I think you've hit bug https://bugs.launchpad.net/cinder/+bug/1468260 | 14:26 |
| openstack | Launchpad bug 1468260 in Cinder "'cinder list' does not list more than 1000 volumes" [Undecided,Confirmed] | 14:26 |
| scottda | (for limit of 1000 volumes) | 14:26 |
| setmason | ok thanks | 14:26 |
| *** shausy has quit IRC | 14:26 | |
| scottda | Bug reports no problem with version 1.0 or 1.1, so that might be a workaround. | 14:27 |
| *** annegentle has joined #openstack-cinder | 14:27 | |
| setmason | Running Havana and v1 API | 14:28 |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: allow attaching status for multiattach volume https://review.openstack.org/172138 | 14:29 |
| scottda | That refers to cinderclient version ^^ | 14:30 |
| setmason | right. | 14:30 |
| setmason | not at system right now (on phone), will update bug with information when I get into the offfice and can run cinder --verison | 14:30 |
| scottda | cool | 14:31 |
| *** tellesnobrega__ has quit IRC | 14:32 | |
| *** rushil has quit IRC | 14:34 | |
| *** salv-orlando has joined #openstack-cinder | 14:35 | |
| thingee | DuncanT: this requirements proposal doesn't match global requirements for stable/kilo | 14:35 |
| thingee | DuncanT: keystoneclient for example | 14:36 |
| DuncanT | thingee: Ok, thanks, I'll pass that back to the people chasing it | 14:36 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume https://review.openstack.org/199590 | 14:36 |
| xyang | thingee: hi, can you please approve this blueprint and target L-2? https://blueprints.launchpad.net/cinder/+spec/non-disruptive-backup | 14:37 |
| thingee | dhellmann: is there a way to have proposal bot try again? looks like this update is out dated https://review.openstack.org/#/c/173977/3 | 14:38 |
| erlon | thingee: hi Mike | 14:39 |
| erlon | thingee: have you see my email on third-party list? | 14:39 |
| erlon | thingee: when you have a change please push that patch (https://review.openstack.org/#/c/197686/) fowardso we can enable the other CI account | 14:40 |
| *** tellesnobrega_ has joined #openstack-cinder | 14:41 | |
| *** setmason has quit IRC | 14:42 | |
| xyang | DuncanT: hi, I addressed your review comments on the non-disruptive backup patch: https://review.openstack.org/#/c/193937/ Also submitted a patch for attach snapshot: https://review.openstack.org/#/c/198556/ . Can you provide some feedback on the attach snapshot interface? | 14:43 |
| DuncanT | xyang: Certainly, looking now | 14:43 |
| xyang | DuncanT: thanks! | 14:43 |
| *** gouthamr has quit IRC | 14:44 | |
| *** nlevinki has quit IRC | 14:44 | |
| openstackgerrit | Ian Cordasco proposed openstack/python-cinderclient: Remove unnecessary check for tenant information https://review.openstack.org/199595 | 14:45 |
| *** severson_ has quit IRC | 14:46 | |
| *** ronis_ has quit IRC | 14:46 | |
| thingee | erlon: nope didn't see it. might've not been approved yet http://lists.openstack.org/pipermail/third-party-announce/2015-July/000232.html | 14:47 |
| *** salv-orlando has quit IRC | 14:47 | |
| thingee | erlon: as you can see, there is no next message. | 14:47 |
| erlon | thingee: hmm, that's moderated :/ | 14:47 |
| thingee | erlon: yea | 14:48 |
| thingee | erlon: can we get a ci run on that patch? | 14:48 |
| *** gouthamr has joined #openstack-cinder | 14:49 | |
| erlon | thingee: you mean Hitachi CI? | 14:49 |
| erlon | thingee: it run | 14:49 |
| erlon | thingee: but only the CI that was having problem (hitachi-horcm-fc) | 14:50 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: WIP: Refactoring of manager's create_volume flow https://review.openstack.org/193167 | 14:50 |
| erlon | thingee: about the fails, in general lines, we had a power failure this weekend and only manage to get it working yesterday in the end of the day. Is not 100% stable but we are working on | 14:51 |
| *** belmoreira has quit IRC | 14:53 | |
| DuncanT | xyang: Non-disruptive backup looks good now, I'll spin up a devstack and play a bit but I think I can +2 it soon | 14:53 |
| thingee | erlon: ok, we'll look for a run from the hitachi ci with this patch. I'll check back on it later today. | 14:53 |
| *** jdandrea has quit IRC | 14:54 | |
| *** jdandrea has joined #openstack-cinder | 14:55 | |
| mclaren | thingee: quick query on https://review.openstack.org/#/c/173977 ... I'm not 100% sure why cinderclient can't just pin (<=) to the kilo keystone client. What else needs to happen? Thanks | 14:55 |
| xyang | DuncanT: great! thanks! | 14:55 |
| *** rhe00_ has quit IRC | 14:55 | |
| *** Apoorva has joined #openstack-cinder | 14:56 | |
| thingee | mclaren: just like with any requirements proposal, we just match what's in global requirements https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L127 | 14:56 |
| thingee | mclaren: once the proposal bot updates the patch, we can merge it | 14:57 |
| thingee | or someone else can update it. I don't mind. | 14:57 |
| thingee | I'll just do it | 14:58 |
| *** diogogmt has joined #openstack-cinder | 14:58 | |
| *** jdandrea has quit IRC | 14:59 | |
| mclaren | thingee: ah, the current patch is stale ... I get it now -- thanks! | 15:00 |
| *** jdurgin1 has joined #openstack-cinder | 15:00 | |
| *** kaisers_ has joined #openstack-cinder | 15:01 | |
| thingee | mclaren: alright updated https://review.openstack.org/#/c/173977/ | 15:02 |
| *** bluex-pl has quit IRC | 15:02 | |
| *** dims has quit IRC | 15:04 | |
| *** coolsvap is now known as coolsvap|away | 15:04 | |
| *** dims has joined #openstack-cinder | 15:05 | |
| mclaren | thanks! | 15:06 |
| *** tellesnobrega_ has quit IRC | 15:06 | |
| *** MentalRay has joined #openstack-cinder | 15:09 | |
| *** jdandrea has joined #openstack-cinder | 15:09 | |
| *** dims has quit IRC | 15:10 | |
| *** bkopilov has joined #openstack-cinder | 15:10 | |
| *** coolsvap|away is now known as coolsvap | 15:12 | |
| *** nkrinner has quit IRC | 15:12 | |
| *** jdandrea has quit IRC | 15:12 | |
| *** kaisers_ has quit IRC | 15:13 | |
| *** mclaren has quit IRC | 15:14 | |
| *** bkopilov has quit IRC | 15:15 | |
| *** primechuck has joined #openstack-cinder | 15:15 | |
| *** rhe00_ has joined #openstack-cinder | 15:18 | |
| *** jgriffith has quit IRC | 15:20 | |
| *** liewegas has quit IRC | 15:20 | |
| *** boris-42 has joined #openstack-cinder | 15:21 | |
| *** liewegas has joined #openstack-cinder | 15:22 | |
| *** dims has joined #openstack-cinder | 15:23 | |
| *** topshare_ has joined #openstack-cinder | 15:25 | |
| *** kaisers_ has joined #openstack-cinder | 15:26 | |
| *** setmason has joined #openstack-cinder | 15:26 | |
| *** topshare has quit IRC | 15:26 | |
| *** tsekiyama has joined #openstack-cinder | 15:27 | |
| *** bkopilov has joined #openstack-cinder | 15:27 | |
| *** hemnafk is now known as hemna | 15:28 | |
| *** kaisers_ has quit IRC | 15:29 | |
| *** kaisers_ has joined #openstack-cinder | 15:31 | |
| *** asselin__ has quit IRC | 15:32 | |
| *** jgriffith has joined #openstack-cinder | 15:32 | |
| *** jdandrea has joined #openstack-cinder | 15:34 | |
| *** timcl1 has quit IRC | 15:34 | |
| *** kaisers_42 has joined #openstack-cinder | 15:35 | |
| *** kaisers_ has quit IRC | 15:36 | |
| *** pots has joined #openstack-cinder | 15:38 | |
| *** gouthamr has quit IRC | 15:39 | |
| *** sseverson has joined #openstack-cinder | 15:39 | |
| *** bswartz has quit IRC | 15:39 | |
| *** merooney has quit IRC | 15:39 | |
| *** gouthamr has joined #openstack-cinder | 15:40 | |
| *** e0ne has joined #openstack-cinder | 15:40 | |
| *** kaisers_42 has quit IRC | 15:40 | |
| openstackgerrit | hadi esiely proposed openstack/cinder: Store volume encryption metadata on each volume https://review.openstack.org/152284 | 15:40 |
| *** zul has quit IRC | 15:41 | |
| *** david-ly_ is now known as david-lyle | 15:42 | |
| *** dustins has quit IRC | 15:43 | |
| *** annegentle has quit IRC | 15:45 | |
| *** laughterwym has quit IRC | 15:45 | |
| *** timcl has joined #openstack-cinder | 15:45 | |
| *** adurbin_ has joined #openstack-cinder | 15:45 | |
| eharney | is there a meeting today? | 15:46 |
| *** annegentle has joined #openstack-cinder | 15:46 | |
| *** rhe00_ has quit IRC | 15:46 | |
| *** daneyon has joined #openstack-cinder | 15:46 | |
| e0ne | eharney: there is no agenda yet. but may be thingee would like to make some weekly anouncements | 15:46 |
| *** coolsvap is now known as coolsvap|away | 15:47 | |
| *** salv-orlando has joined #openstack-cinder | 15:47 | |
| thingee | e0ne: I don't have anything to announce this week. | 15:47 |
| e0ne | thingee: thanks :) | 15:48 |
| *** coolsvap|away is now known as coolsvap | 15:48 | |
| *** jasondotstar has quit IRC | 15:48 | |
| geguileo | So it's going to be a quick meeting :-) | 15:48 |
| *** kaisers_ has joined #openstack-cinder | 15:48 | |
| dulek | Can someone take a look on http://logs.openstack.org/90/199590/1/check/gate-cinder-python27/50c3959/console.html ? | 15:49 |
| dulek | I don't understand why it failed. | 15:49 |
| *** patrickeast has joined #openstack-cinder | 15:49 | |
| *** abehl has quit IRC | 15:49 | |
| dulek | "This test has failed because it attempted to execute commands"? | 15:49 |
| eharney | dulek: some test isn't mocking out commands, there's a script that runs at the end to detect commands that were run under sudo | 15:49 |
| *** BharatK has joined #openstack-cinder | 15:50 | |
| dulek | eharney: Okay, I'll take a look into that. Thanks! | 15:50 |
| *** daneyon has quit IRC | 15:50 | |
| eharney | jobernar ran into this same thing yesterday on a similar code path... | 15:50 |
| eharney | jbernard * | 15:50 |
| geguileo | eharney: I was thinking about jobernar problem yesterday as well :-) | 15:51 |
| jbernard | yep | 15:51 |
| jbernard | since many use vagrant, which requires passwordless-sudo, you don't catch it typically | 15:51 |
| dulek | Ah, I think I understand that - when reworking the UT I've fixed one that wasn't doing what it was supposed to do. | 15:51 |
| *** kaisers_ has quit IRC | 15:51 | |
| dulek | Now it is and commands weren't mocked. | 15:51 |
| *** yamada-h has joined #openstack-cinder | 15:51 | |
| *** yamada-h has quit IRC | 15:51 | |
| dulek | Any possiblity to run such check locally? | 15:52 |
| jbernard | dulek: i put a 'raise' before the offending command from the gate logs | 15:52 |
| jbernard | dulek: and that turns up the culprit quickly | 15:52 |
| dulek | jbernard: Cool, thanks! | 15:53 |
| *** patrickeast has quit IRC | 15:53 | |
| jbernard | dulek: np | 15:53 |
| eharney | if you run unit tests on an environment that requires passwords for sudo, it's usually pretty easy to narrow down | 15:53 |
| eharney | the test will hang and time out and tell you which test it is | 15:54 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add support for ManageableVD https://review.openstack.org/199631 | 15:54 |
| *** jistr has quit IRC | 15:56 | |
| *** salv-orlando has quit IRC | 15:56 | |
| *** kaisers_ has joined #openstack-cinder | 15:56 | |
| *** jasondotstar has joined #openstack-cinder | 15:58 | |
| *** patrickeast has joined #openstack-cinder | 15:59 | |
| *** kaisers_ has quit IRC | 15:59 | |
| *** bswartz has joined #openstack-cinder | 16:00 | |
| *** salv-orl_ has joined #openstack-cinder | 16:00 | |
| *** tellesnobrega_ has joined #openstack-cinder | 16:00 | |
| *** rhe00_ has joined #openstack-cinder | 16:01 | |
| *** jungleboyj has joined #openstack-cinder | 16:04 | |
| *** kaisers_ has joined #openstack-cinder | 16:05 | |
| *** Yogi1 has joined #openstack-cinder | 16:05 | |
| jgriffith | akerr_: did you see my comment on your trace logging patch? | 16:06 |
| jungleboyj | And we're back. | 16:06 |
| smcginnis | Quick one. :) | 16:06 |
| akerr_ | jgriffith: don't think so... but that's ameade's patch if its the one i'm thinking of. let me check | 16:07 |
| jgriffith | akerr_: ahh.. I think you're right :) | 16:07 |
| jgriffith | akerr_: in a nut-shell.... Seems like an ok idea, but I'm wondering isn't that just "debug"? | 16:07 |
| *** rushil has joined #openstack-cinder | 16:08 | |
| jgriffith | akerr_: and if we want "trace" level logging options, rather than write our own in Cinder, shouldn't we consider adding it to oslo.log as an option there instead? | 16:08 |
| akerr_ | jgriffith: like, why have the option to turn it on and off? | 16:08 |
| *** zul has joined #openstack-cinder | 16:08 | |
| jgriffith | akerr_: well, no, not really | 16:08 |
| jgriffith | akerr_: I get that | 16:08 |
| jgriffith | akerr_: but why it's not just the existing "debug" level logging | 16:08 |
| jgriffith | akerr_: or like I said, add trace level option to oslo.log | 16:09 |
| *** kaisers_ has quit IRC | 16:09 | |
| akerr_ | i thought it was using the debug level logging | 16:09 |
| *** patrickeast has quit IRC | 16:09 | |
| *** patrickeast has joined #openstack-cinder | 16:10 | |
| *** rajinir_r has joined #openstack-cinder | 16:10 | |
| jgriffith | Oh.. well shit, DuncanT merged it | 16:10 |
| *** dontalton has joined #openstack-cinder | 16:10 | |
| jgriffith | Ok, then, never mind | 16:10 |
| thingee | jgriffith: I was going to say, i think I saw that in the git logs this morning =/ | 16:10 |
| jgriffith | akerr_: ameade https://review.openstack.org/#/c/157439/15 | 16:10 |
| tbarron | jgriffith: ameade replied to you and he does want to go to oslo eventually | 16:11 |
| ameade | o? | 16:11 |
| ameade | o/ | 16:11 |
| ameade | lol | 16:11 |
| *** leeantho has joined #openstack-cinder | 16:11 | |
| *** coolsvap is now known as coolsvap|away | 16:11 | |
| rhe00_ | thingee: I am still catching up on things as I got back from vacation today. Will take a look at the CI shortly. I had some occasional trouble with nova network not giving out valid IP addresses before I left, but it should still have been reporting with a 66% success rate | 16:11 |
| *** annasort has joined #openstack-cinder | 16:12 | |
| ameade | jgriffith: there used to be the 'audit' log level that openstack added | 16:12 |
| DuncanT | Hmmm, I did what? | 16:12 |
| rhe00_ | thingee: Your emails ended up in a Openstack folder that wasn't synced to my phone so I didn't see them during my vacation. Sorry about that. | 16:12 |
| thingee | rhe00_: ok, will we already removed the driver, but we can of course readd things. Just wasn't sure if you disappeared since it has been almost 30 days. | 16:13 |
| jgriffith | DuncanT: oh.. nothing; I was just pointing out my questions in a patch to ameade | 16:13 |
| DuncanT | jgriffith: Ah, the tracing decorators? | 16:13 |
| jgriffith | DuncanT: but found it was moot as it merged this morning | 16:13 |
| ameade | eyes one this? https://review.openstack.org/#/c/199170/ | 16:13 |
| thingee | rhe00_: no thorough review or anything, just revert my removal after the CI is reporting again. | 16:13 |
| *** kaisers_ has joined #openstack-cinder | 16:13 | |
| rhe00_ | thingee: I saw that. I will get it back up and running ASAP. | 16:13 |
| *** markus_z has quit IRC | 16:15 | |
| *** dims has quit IRC | 16:15 | |
| DuncanT | ameade: audit was added for something very specific, it got misunderstood, misused then removed | 16:16 |
| e0ne | jgriffith: fyi, updated spec about wsgi: https://review.openstack.org/192683. i think it won't be merged w/o your vote:) | 16:16 |
| ameade | DuncanT: ah, figures i spose | 16:16 |
| jgriffith | silly request... can someobdy ping me on another channel or PM? | 16:16 |
| ameade | jgriffith: done | 16:16 |
| *** kaisers_ has quit IRC | 16:16 | |
| *** dims has joined #openstack-cinder | 16:16 | |
| *** merooney has joined #openstack-cinder | 16:16 | |
| jgriffith | Ok.. works, thanks | 16:16 |
| jgriffith | I completely rewrote my irssi config and thought I broke it | 16:17 |
| ameade | oh i used to use irssi, wrote a plugin at some point | 16:17 |
| smcginnis | jgriffith: Using hilight? | 16:17 |
| ameade | in perl >.< | 16:17 |
| ameade | https://github.com/ameade/queryhistory_irssi | 16:18 |
| jgriffith | smcginnis: Yeah | 16:18 |
| ameade | shameless plug | 16:18 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject https://review.openstack.org/198341 | 16:18 |
| jgriffith | ameade: cool! | 16:18 |
| xyang | thingee: XtremIO CI is reporting now. Can you please remove -2 from this patch? https://review.openstack.org/#/c/189135/ | 16:19 |
| ameade | so that i could close all those darn query windows with people | 16:19 |
| ameade | and still have context if i open them back up | 16:19 |
| *** jdurgin1 has quit IRC | 16:19 | |
| ameade | now i use irccloud.com though | 16:19 |
| *** zhenguo has quit IRC | 16:20 | |
| *** rakesh_mishra has quit IRC | 16:20 | |
| *** ronis_ has joined #openstack-cinder | 16:20 | |
| *** jordanP has quit IRC | 16:20 | |
| *** dims has quit IRC | 16:21 | |
| *** jbrc has joined #openstack-cinder | 16:21 | |
| DuncanT | I'm a fan of irccloud | 16:24 |
| *** jasondotstar has quit IRC | 16:25 | |
| winston-d | 5$ monthly, that's as much as a ting pod in digi-ocean. | 16:26 |
| jungleboyj | DuncanT: I just started using that on my iPad. Haven't really messed with it though. | 16:26 |
| DuncanT | jungleboyj: For the most part, it just works[tm] | 16:26 |
| DuncanT | jungleboyj: The client side stuff is open source, so I might replace it with something self-hosted one day, but right now, laziness is winning out | 16:27 |
| jungleboyj | DuncanT: So, it is 'is a good thing' [tm] | 16:27 |
| winston-d | does it's app supports sending notifications to your Apple watch? (i'm android user) | 16:27 |
| *** _cjones_ has joined #openstack-cinder | 16:28 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Added ABCMeta class to the InitiatorConnector https://review.openstack.org/199642 | 16:28 |
| DuncanT | winston-d: I know nothing about apple watches. Took me a while to stop it sending notifications to my pebble though | 16:28 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix NetApp cDOT driver use of Glance locations https://review.openstack.org/193597 | 16:28 |
| jungleboyj | DuncanT: Interesting, I hadn't even really looked at what all this does. Need to look closer. | 16:29 |
| jgriffith | winston-d: I'm buying you an Apple Watch for Christmas! | 16:29 |
| *** changbl has quit IRC | 16:29 | |
| jgriffith | winston-d: and you MUST wear it in shame :) | 16:29 |
| *** jaypipes has quit IRC | 16:29 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Added ABCMeta class to the InitiatorConnector https://review.openstack.org/199642 | 16:30 |
| winston-d | jgriffith: buy me with all kinds of watches if you like, i'd be happy to wear them all. | 16:31 |
| jgriffith | winston-d: LOL | 16:31 |
| jgriffith | winston-d: Yeah I realized that wasn't such a cruel joke after I said it :) | 16:32 |
| jgriffith | hemna: there's only like two methods in there | 16:32 |
| hemna | yup | 16:33 |
| jgriffith | hehe | 16:33 |
| hemna | I plan on adding a few more | 16:33 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 16:33 |
| hemna | I'm working on something that will help do live migration verification | 16:33 |
| hemna | and a new brick Connector method is going to be involved with that, so I wanted to get the ABC in first, and then add the new method next. | 16:33 |
| jgriffith | hemna: you mean like a tempest test that "runs live migration" | 16:33 |
| hemna | sorta | 16:34 |
| tbarron | winston-d: i'm also using a $5 digital ocean vm to run my irc bouncer | 16:34 |
| hemna | basically after a VM has been moved from the src to the destination, verify that all of the entries on the src are now gone. | 16:34 |
| hemna | no dangling volumes | 16:34 |
| hemna | which cause problems later. | 16:34 |
| hemna | the brick connector classes will get a new public method to ask for discovered paths for a volume. | 16:35 |
| *** jwcroppe has quit IRC | 16:35 | |
| hemna | make that call before the migration, and then the same call after migration | 16:35 |
| hemna | to verify that they are gone. | 16:35 |
| jgriffith | hemna: and if they're not? | 16:35 |
| hemna | if they aren't, then we have a dirty state | 16:36 |
| winston-d | tbarron: i also used it to get over the invisible wall we have in China. | 16:36 |
| jgriffith | hemna: well... my question is how do you fix that? | 16:36 |
| tbarron | winston-d: ++ | 16:36 |
| hemna | it depends | 16:36 |
| jgriffith | hemna: isn't that a problem with the initiator? | 16:36 |
| hemna | from the testing that we've seen, there are bugs in drivers that cause this, and also there are problems with the way nova is calling cinder that can cause it as well. | 16:36 |
| hemna | :( | 16:36 |
| hemna | and also there could be problems with the code that removes the paths | 16:37 |
| jgriffith | hemna: do you have an example of a driver with a bug? | 16:37 |
| hemna | the disconnect_volume call | 16:37 |
| hemna | we have an etherpad with our current testing results | 16:38 |
| hemna | jgriffith, https://etherpad.openstack.org/p/live-migration-results | 16:38 |
| hemna | 3par iSCSI is borked, and lvm iscsi looks borked from what we've seen. | 16:38 |
| hemna | 3par is our problem for sure and we are working on it. | 16:39 |
| Swanson | Sooo... long() doesn't exist in python3.4? | 16:39 |
| hemna | but I'm trying to get some tools in place that can help others verify live migration as well | 16:39 |
| jgriffith | hemna: odd, LVM worked in Kilo | 16:39 |
| hemna | so it's not just something that we use in house. | 16:39 |
| jgriffith | but who knows, I haven't tried it in a while | 16:39 |
| smcginnis | Swanson: I believe int is now long. | 16:39 |
| jgriffith | hemna: so I was confused I guess, you said "some drivers had bugs" I thought you identified something in code in other peoples drivers maybe | 16:40 |
| jgriffith | hemna: that's what I was curious about | 16:40 |
| *** dims has joined #openstack-cinder | 16:40 | |
| hemna | jgriffith, I have yes | 16:40 |
| hemna | it's just not on that etherpad | 16:40 |
| hemna | just haven't had time to document it | 16:40 |
| hemna | basically nova calls initialize_connection directly many times during live migration | 16:41 |
| jgriffith | hemna: oh... well maybe you should share that info so those drivers / maintainers can have a look? | 16:41 |
| smcginnis | Swanson: https://www.python.org/dev/peps/pep-0237/ | 16:41 |
| jgriffith | hemna: and also to share the info | 16:41 |
| patrickeast | somewhat related, we’ve seen issues with nova leaving behind dangling multipath connections https://bugs.launchpad.net/nova/+bug/1385798 | 16:41 |
| openstack | Launchpad bug 1385798 in OpenStack Compute (nova) "Multipath ISCSI connections left open after disconnecting volume with libvirt" [Undecided,Confirmed] - Assigned to Danny Wilson (daniel-wilson) | 16:41 |
| hemna | and some drivers do an export every time initialize_connection is called. | 16:41 |
| patrickeast | doesn’t happen with brick though | 16:41 |
| hemna | when nova simply expects initialize_connection to return the connection_info dict for a volume. | 16:41 |
| hemna | anyway | 16:41 |
| *** dontalton is now known as bitblt | 16:41 | |
| *** bitblt has quit IRC | 16:41 | |
| *** bitblt has joined #openstack-cinder | 16:41 | |
| hemna | I was going to go through the drivers today or tomorrow and document which ones do that and which don't | 16:41 |
| *** bitblt has quit IRC | 16:42 | |
| hemna | I just haven't had time yet. | 16:42 |
| *** afazekas has quit IRC | 16:42 | |
| hemna | I want to get live migration to the point where it just works for everyone. | 16:42 |
| winston-d | hemna: ++ | 16:43 |
| *** xyang has quit IRC | 16:43 | |
| winston-d | it should just works | 16:43 |
| *** xyang has joined #openstack-cinder | 16:43 | |
| hemna | and also allow us a way to verify that it's not left in a dirty state, for everyone. | 16:43 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add multiple pools support to VMAX driver https://review.openstack.org/189990 | 16:43 |
| winston-d | with conditions and terms applied | 16:43 |
| jgriffith | hemna: yeah, that would be good | 16:43 |
| jgriffith | hemna: just seems like there are only a few of use that have ever even tried it :( | 16:43 |
| jgriffith | which is too bad, and likely part of the problem | 16:43 |
| hemna | it's been the bane of my existence for a while now. | 16:43 |
| hemna | yah | 16:44 |
| jgriffith | I'm just lucky... after a couple patches to Nova it just "works" for me (and SF and LVM) | 16:44 |
| hemna | my plan is to work with asselin to see how we can get CI for LM going and add the verification. | 16:44 |
| *** primechuck has quit IRC | 16:44 | |
| hemna | we need several different scenarios tested, not just a single VM fired up with a single volume and it works migrated one way | 16:44 |
| jgriffith | single vm with single volume? | 16:45 |
| hemna | jgriffith, some drivers are lucky. lefthand is also one of them where it seems to just work. | 16:45 |
| *** xyang has quit IRC | 16:46 | |
| hemna | yah that's the simplest of cases that's easy to try. | 16:46 |
| hemna | but isn't really real world IMHO | 16:46 |
| *** xyang has joined #openstack-cinder | 16:46 | |
| hemna | we laid out our test scenarios in that etherpad, 7 of them. | 16:46 |
| jgriffith | I guess I don't follow... you mean multi-compute nodes though? | 16:46 |
| hemna | all of our tests are with multiple compute nodes | 16:47 |
| jgriffith | hemna: oh, I see... makes sense | 16:47 |
| hemna | where case 1. is you bring openstack up (multiple nodes) and you fire up 1 VM, attach a cinder volume to it, and migrate it one way. done. | 16:47 |
| *** garthb has joined #openstack-cinder | 16:47 | |
| jgriffith | I think you're missing the most important one there.... | 16:47 |
| jgriffith | boot from volume | 16:48 |
| hemna | sure | 16:48 |
| jgriffith | all the others I would think become sort of null if multi-attach ever lands and works | 16:48 |
| hemna | yah we should add boot from volume for sure | 16:48 |
| hemna | thanks for that one :) | 16:49 |
| haypo | hemna: hi | 16:50 |
| hemna | hey | 16:51 |
| haypo | hemna: we told me that you can release a new version of os-brick? | 16:51 |
| hemna | I can't, as I'm not the PTL :) | 16:51 |
| hemna | os-brick is owned by Cinder, which means only the PTL can do a release. | 16:51 |
| haypo | jgriffith: hey, what about you? :) can you release a new version of os-brick? | 16:53 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add support for ManageableVD https://review.openstack.org/199631 | 16:53 |
| smcginnis | haypo: You need to hit up thingee. | 16:53 |
| jgriffith | haypo: as hemna mentioned I believe thingee owns that now | 16:54 |
| jgriffith | haypo: I "could" but then he'd have to kill me after | 16:54 |
| e0ne | :) | 16:54 |
| thingee | haypo, jgriffith, hemna: if we're ready, I can do one today. jgriffith nor I can push these anymore since the changes to library releases. | 16:54 |
| tbarron | e0ne: thanks for the review of https://review.openstack.org/199173 | 16:54 |
| thingee | it has to be proposed | 16:55 |
| haypo | sorry, jgriffith is not the current cinder PTL? sorry. so it's thingee? | 16:55 |
| jgriffith | thingee: oh? Interesting | 16:55 |
| thingee | haypo: correct' | 16:55 |
| tbarron | e0ne: w.r.t. the question about (and I understand that it's a moot question for this cherrypick): | 16:55 |
| jgriffith | haypo: yes, thingee has been PTL for like a year | 16:55 |
| haypo | thingee: i would prefer to see https://review.openstack.org/#/c/197624/ merged into os-brick, but hemna doesn't want it :) | 16:55 |
| tbarron | e0ne: (_, export_path) = self._get_export_ip_path(share=share) | 16:55 |
| e0ne | tbarron: you're welcome | 16:55 |
| *** vokt has joined #openstack-cinder | 16:55 | |
| e0ne | tbarron: it'ls ok for cherry pick | 16:56 |
| jgriffith | thingee: so what's the new rule on the libs? | 16:56 |
| shyama | hemna: https://review.openstack.org/#/c/198400/ should this be fixed in nova rather than in cinder? | 16:56 |
| e0ne | tbarron: i don't know if any guidelines exists | 16:56 |
| tbarron | e0ne: you make the good observation that the '_' can be confused with _ from i18n lib | 16:56 |
| setmason | Is there a more efficient method of determining the number of cinder volumes you have, besides cinder list —all-tenants? I’ve got about 5000 volumes and using this command with —marker and then eventually piping into wc -l, seems a bit kludgy. | 16:56 |
| thingee | jgriffith: http://lists.openstack.org/pipermail/openstack-dev/2015-June/066346.html | 16:56 |
| tbarron | e0ne: for future reference, is there a good way in OpenStack to indicate an unused variable? | 16:57 |
| jgriffith | thingee: thanks | 16:57 |
| haypo | hemna: so you're still unable to run tests using testr? https://review.openstack.org/#/c/197624/ | 16:57 |
| e0ne | tbarron: but IMO, it would be better something like (_your_unused_variable_short_name, export_path) = self._get_export_ip_path(share=share) | 16:57 |
| haypo | hemna: it's probably not a big deal, just an annoying issue in your setup | 16:57 |
| tbarron | e0ne: yeah, I think that makes sense, will do going forwards | 16:57 |
| e0ne | tbarron: thank you, Tom | 16:57 |
| jgriffith | setmason: sadly you best option might be a DB querie at that point if you ask me | 16:58 |
| setmason | eww | 16:58 |
| tbarron | e0ne: no, thank you Ivan :-) Your review on this one was very helpful. | 16:58 |
| setmason | I dont’ want to increase osapi_max_limit… | 16:58 |
| jgriffith | setmason: we should have an admin api call for that sort of thing | 16:59 |
| haypo | thingee: i'm asking a new os-brick release to get hemna fixes for python 3. os-brick blocks four of my python 3 patches (because the current os-brick release doesn't work on python 3) | 16:59 |
| haypo | thingee: (patches for cinder) | 16:59 |
| jgriffith | setmason: if you file a bug, I'll mark it as wishlist and hack something up for it | 16:59 |
| thingee | haypo: in the middle of a meeting, so I'll need to get back to you | 16:59 |
| haypo | thingee: ok | 17:00 |
| jgriffith | setmason: probably should be something under cinder-manage | 17:00 |
| *** timcl1 has joined #openstack-cinder | 17:00 | |
| jgriffith | setmason: unless you have a global admin | 17:00 |
| hemna | meeting.... | 17:00 |
| *** hemna is now known as hemnafk | 17:00 | |
| jgriffith | setmason: somebody like DuncanT might have some suggestions/insights on solving that | 17:00 |
| *** jwcroppe has joined #openstack-cinder | 17:01 | |
| *** dustins has joined #openstack-cinder | 17:02 | |
| *** topshare_ has quit IRC | 17:02 | |
| DuncanT | setmason: There's been a suggestion before that we should have a stats API, but nobody has actually coded it (volume count & total size per backend, snapshot counts and total size per backend, backups count and total size....) | 17:02 |
| winston-d | hemnafk: | 17:02 |
| setmason | DuncanT: Thanks | 17:03 |
| DuncanT | setmason: Shouldn't be particularly hard to write, the info is all in the DB | 17:03 |
| *** timcl has quit IRC | 17:03 | |
| jgriffith | DuncanT: setmason so that's a good point... but what about "I want *all* volumes across *all* tenants" | 17:03 |
| jgriffith | DuncanT: setmason seems like that crosses the line over to cinder-manage, which could just do the db querie for you? | 17:03 |
| *** MentalRay has quit IRC | 17:03 | |
| jgriffith | DuncanT: seem kosher? | 17:03 |
| setmason | ok. will look at cinder-manage | 17:04 |
| jgriffith | DuncanT: otherwise you have the whole "run it for 'n' tenants" problem. Unless you have some uber tenant to set in the policy file on it | 17:04 |
| *** MentalRay has joined #openstack-cinder | 17:04 | |
| setmason | well it would seem that an extension to cinder list would be more appropriate | 17:04 |
| jgriffith | which I'm not quite sure how you'd do that | 17:04 |
| jgriffith | setmason: I'd agree | 17:05 |
| setmason | as we’re running this to determine cinder stats, as a provider. | 17:05 |
| *** bswartz has quit IRC | 17:05 | |
| jgriffith | setmason: assuming you have a user that accesses *all* tenants | 17:05 |
| jgriffith | setmason: then it's actually really easy | 17:05 |
| DuncanT | jgriffith: cinder-manage could do it, but it is the kind of thing that is really useful on a dashboard. admin-only by default I guess, but 'count * where' clauses shouldn't kill the db | 17:05 |
| jgriffith | setmason: we already have a call to get all for all tenants | 17:05 |
| DuncanT | jgriffith: Just getting the count, sum etc is rather lighter weight than listing 10k volumes and doing the math on the client | 17:06 |
| setmason | We’re running this as an admin, really to populate dashboards and reports. | 17:06 |
| jgriffith | DuncanT: yeah... makes sense; I'm just unclear on how to deal with multiple projects and rules around that | 17:06 |
| *** rushil has quit IRC | 17:07 | |
| DuncanT | jgriffith: I'd say these were system stats, and just ignore projects etc. Filtering can be added later | 17:07 |
| setmason | jgriffith: the cinder list —all-tenants does what I need, but it’s limited by osapi_max_limit. The workaround is to use markers every $osapi_max_limit. | 17:07 |
| jgriffith | setmason: DuncanT ok... that makes sense | 17:07 |
| jgriffith | setmason: DuncanT if nobody else wants to tackle it I'm happy to add it to my list | 17:07 |
| jgriffith | btw, isn't part of this supposed to be what ceilometer is all about :) | 17:08 |
| *** bswartz has joined #openstack-cinder | 17:08 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused arguments from c-vol's create_volume https://review.openstack.org/199590 | 17:09 |
| DuncanT | jgriffith: ceilometer would basically have to keep a complete copy of the cinder db to know this info... I don't know where the line should be drawn, but it seems like a useful sort of API to have | 17:09 |
| *** mutoulbj has quit IRC | 17:12 | |
| openstackgerrit | Ian Cordasco proposed openstack/python-cinderclient: Remove unnecessary check for tenant information https://review.openstack.org/199595 | 17:12 |
| *** daneyon has joined #openstack-cinder | 17:14 | |
| *** Apoorva has quit IRC | 17:15 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers https://review.openstack.org/197999 | 17:16 |
| *** timcl1 has quit IRC | 17:17 | |
| *** timcl has joined #openstack-cinder | 17:18 | |
| *** rushil has joined #openstack-cinder | 17:18 | |
| *** lpetrut has quit IRC | 17:18 | |
| *** tellesnobrega_ has quit IRC | 17:19 | |
| *** daneyon has quit IRC | 17:19 | |
| *** primechuck has joined #openstack-cinder | 17:19 | |
| *** ronis_ has quit IRC | 17:20 | |
| *** rushil has quit IRC | 17:21 | |
| *** rushil has joined #openstack-cinder | 17:21 | |
| *** winston-d_ has joined #openstack-cinder | 17:21 | |
| *** Apoorva has joined #openstack-cinder | 17:23 | |
| *** Apoorva has quit IRC | 17:24 | |
| *** ociuhandu has quit IRC | 17:25 | |
| *** Apoorva has joined #openstack-cinder | 17:26 | |
| *** bkopilov has quit IRC | 17:28 | |
| *** e0ne has quit IRC | 17:28 | |
| *** kjnelson has quit IRC | 17:31 | |
| *** BharatK has quit IRC | 17:32 | |
| *** kjnelson1 has joined #openstack-cinder | 17:33 | |
| *** bkopilov has joined #openstack-cinder | 17:36 | |
| *** dannywilson has joined #openstack-cinder | 17:37 | |
| *** brmj has joined #openstack-cinder | 17:37 | |
| brmj | jgriffith:ping | 17:38 |
| *** dims has quit IRC | 17:39 | |
| *** dims has joined #openstack-cinder | 17:39 | |
| brmj | jgriffith:NM | 17:40 |
| *** Apoorva has quit IRC | 17:41 | |
| *** hemnafk is now known as hemna | 17:42 | |
| *** brmj has quit IRC | 17:43 | |
| *** raunak has joined #openstack-cinder | 17:45 | |
| *** MentalRay has quit IRC | 17:46 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add support for ManageableVD https://review.openstack.org/199631 | 17:48 |
| Swanson | @#$!!$@!!!! | 17:48 |
| *** Apoorva has joined #openstack-cinder | 17:49 | |
| *** lpetrut has joined #openstack-cinder | 17:51 | |
| *** delattec has joined #openstack-cinder | 17:52 | |
| *** annashen has joined #openstack-cinder | 17:53 | |
| *** crose has quit IRC | 17:53 | |
| *** sseverson has quit IRC | 17:53 | |
| *** crose has joined #openstack-cinder | 17:53 | |
| *** bkopilov has quit IRC | 17:54 | |
| *** cdelatte has quit IRC | 17:56 | |
| *** Longgeek has quit IRC | 17:57 | |
| smcginnis | Swanson: Is your VD unmanageable? | 17:57 |
| *** Longgeek has joined #openstack-cinder | 17:57 | |
| *** Longgeek has quit IRC | 17:57 | |
| *** aix has quit IRC | 17:57 | |
| Swanson | smcginnis: It is find so long as I don't implement TransferVD. | 17:58 |
| Swanson | fine | 17:58 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Added ABCMeta class to the InitiatorConnector https://review.openstack.org/199642 | 17:58 |
| smcginnis | Swanson: You don't want to transfer your VD. | 17:58 |
| *** lpetrut has quit IRC | 17:58 | |
| *** Longgeek has joined #openstack-cinder | 17:59 | |
| *** kjnelson1 has quit IRC | 18:01 | |
| openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 18:01 |
| *** xyang has quit IRC | 18:01 | |
| *** dannywilson has quit IRC | 18:02 | |
| *** dannywilson has joined #openstack-cinder | 18:03 | |
| *** kjnelson has joined #openstack-cinder | 18:03 | |
| *** xyang has joined #openstack-cinder | 18:06 | |
| *** xyang has quit IRC | 18:07 | |
| *** xyang has joined #openstack-cinder | 18:07 | |
| *** tellesnobrega_ has joined #openstack-cinder | 18:08 | |
| *** timcl has quit IRC | 18:09 | |
| *** bkopilov has joined #openstack-cinder | 18:09 | |
| *** skylerberg has joined #openstack-cinder | 18:10 | |
| *** resker has joined #openstack-cinder | 18:12 | |
| *** bpoulos has joined #openstack-cinder | 18:16 | |
| *** esker has quit IRC | 18:16 | |
| thingee | xyang: looks like the recent emc xio runs are failing often, but I removed the -2 | 18:17 |
| *** dannywilson has quit IRC | 18:18 | |
| *** xyang1 has quit IRC | 18:19 | |
| *** ronis_ has joined #openstack-cinder | 18:20 | |
| xyang | thingee: there was another hardware issue a few days ago. that's probably why. it was just fixed. hopefully it will run more stable now. we'll keep monitoring it. thanks! | 18:22 |
| *** boris-42 has quit IRC | 18:22 | |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota: Set default values to subproject https://review.openstack.org/194406 | 18:22 |
| thingee | xyang: this is recent runs from today http://paste.openstack.org/show/355957/ | 18:24 |
| *** resker has quit IRC | 18:25 | |
| xyang | thingee: ok, we'll take a look. I thought it is fixed | 18:25 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Enable cinder-manage to delete old services from the database https://review.openstack.org/197613 | 18:29 |
| thingee | haypo: back | 18:32 |
| thingee | hemna: can you recheck https://review.openstack.org/#/c/197624/2 | 18:33 |
| hemna | ok | 18:33 |
| hemna | I had just tried it last night on a stock ubuntu 14.04. | 18:33 |
| thingee | haypo: ok, let see about getting this change in and let me double check with hemna if we need anything else in the release | 18:33 |
| hemna | I ran that on a clean system w/o an existing .testrepository | 18:34 |
| thingee | hemna: ok, I just tried it with the instruction haypo gave and it works for me | 18:34 |
| *** bswartz has quit IRC | 18:34 | |
| hemna | and if python-gdbm is required, then that has to be handled by the patch. | 18:34 |
| thingee | https://www.irccloud.com/pastebin/0l6mVs2u/ | 18:34 |
| *** jbrc has quit IRC | 18:35 | |
| thingee | haypo: ^ | 18:35 |
| *** dustins has quit IRC | 18:36 | |
| *** rushil has quit IRC | 18:38 | |
| thingee | hemna: ok I'll wait for things to be addressed there for release. | 18:39 |
| *** esker has joined #openstack-cinder | 18:39 | |
| thingee | hemna: anything else? https://review.openstack.org/#/q/project:openstack/os-brick+status:open,n,z | 18:39 |
| *** timcl has joined #openstack-cinder | 18:39 | |
| *** dustins has joined #openstack-cinder | 18:39 | |
| thingee | hemna: fyi, these connectors are low priority at this point. nova has already stopped accepting new bps | 18:39 |
| hemna | yah the new connectors are low IMHO | 18:40 |
| thingee | hemna: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule | 18:40 |
| hemna | they need shims in nova for them to be used at all anyway | 18:40 |
| thingee | hemna: right | 18:40 |
| thingee | hemna: once we have the release, can we add rbd to this https://review.openstack.org/#/c/175569/23 ? | 18:40 |
| hemna | so, I think the RBD one is special in libvirt | 18:41 |
| hemna | because libvirt itself has native RBD support | 18:41 |
| thingee | hemna: I was just talking to John Garbutt about this, and he would like things to be done in the same patch...oh ok so we don't need to update anything there | 18:41 |
| hemna | that's why I didn't touch it. also the RBD connector in os-brick isn't the same as what's in libvirt volume.py | 18:42 |
| hemna | I'm not an RBD expert though | 18:42 |
| hemna | but that was my take on it FWIW | 18:42 |
| thingee | hemna: ok, if you can reply to me in the review about that, just so others know. | 18:42 |
| thingee | jbernard: ^ | 18:42 |
| hemna | thingee, ok added my response FWIW | 18:44 |
| jbernard | ill take a look | 18:44 |
| bpoulos | i have a question about the python-cinderclient release from yesterday (1.3.0) | 18:44 |
| bpoulos | is it going to be re-released? | 18:44 |
| bpoulos | I saw there was a comment about permissions issues when pushing it, but I didn't see any follow up on the mailing list | 18:45 |
| jbernard | hemna: i didn't realize that was going to land so soon | 18:45 |
| hemna | jbernard, well that's the hope. is to finally get os-brick used in nova for L | 18:45 |
| jbernard | hemna: would it help if post an etherpad on what common pieces of the rbd connector nova could potentially use? | 18:46 |
| jbernard | hemna: there appears to be a large portion of libvirt-specific code, but not all of it | 18:46 |
| thingee | bpoulos: the permission issues is just a change to the process for pushing something to pypi. | 18:46 |
| hemna | honestly, I don't know it that well. from what I remember the rdb libvirt volume.py code relied on the native libvirt RBD support to do volume attaches | 18:47 |
| hemna | which really wouldn't work in os-brick | 18:47 |
| thingee | geguileo: is it safe for me to assume you're spearheading this https://etherpad.openstack.org/p/CinderNovaAPI ? | 18:47 |
| *** jamielennox is now known as jamielennox|away | 18:47 | |
| bpoulos | thingee: I downloaded 1.3.0 from pypi, and it looks to be out of date | 18:47 |
| hemna | since os-brick is used by cinder outside of libvirt entirely. | 18:47 |
| thingee | bpoulos: how so? | 18:47 |
| bpoulos | from somewhere between 1.1.1 and 1.2.0 | 18:47 |
| hemna | jbernard, I'm not sure how you'd go about abstracting those 2 out. | 18:47 |
| jbernard | hemna: that was my understanding as well, but i have yet to thoroughly go through it | 18:47 |
| bpoulos | thingee: it doesn't have the latest commits | 18:47 |
| hemna | doesn't mean it couldn't be done | 18:48 |
| bpoulos | thingee: everything in https://review.openstack.org/#/c/199242/1/ is missing | 18:48 |
| geguileo | thingee: No, I think that'd be hemna | 18:48 |
| hemna | but for L, I think we should punt on it. | 18:48 |
| jbernard | hemna: i will at least take a look at the review you posted | 18:48 |
| hemna | jbernard, ok great thanks | 18:48 |
| bpoulos | thingee: I downloaded it from https://pypi.python.org/pypi/python-cinderclient/1.3.0 | 18:48 |
| jbernard | hemna: i agree, getting it landed at all woudl be a huge win | 18:48 |
| jbernard | hemna: i can follow up with a secondar patch for rbd if needed | 18:48 |
| hemna | jbernard, ok great | 18:49 |
| hemna | that works for me. | 18:49 |
| jbernard | cool | 18:49 |
| bpoulos | thingee: the release notes also look like they're out of date http://docs.openstack.org/developer/python-cinderclient/ | 18:49 |
| thingee | bpoulos: whoa you're right. let me talk to the release team about this | 18:49 |
| thingee | https://github.com/openstack/python-cinderclient/commits/1.3.0 | 18:49 |
| bpoulos | thingee: thanks for looking into it! | 18:50 |
| *** patrickeast_ has joined #openstack-cinder | 18:51 | |
| thingee | hemna, geguileo are you spearheading this https://etherpad.openstack.org/p/CinderNovaAPI | 18:51 |
| geguileo | thingee: I'm not working on that | 18:52 |
| hemna | thingee, scottda | 18:52 |
| thingee | haha | 18:52 |
| thingee | scottda: Is it you? :D | 18:52 |
| hemna | thingee, scottda has done the lion's share of the work on that etherpad | 18:53 |
| thingee | hemna, scottda, I was talking to John Garbutt, and he mentioned it would be good if someone wanted to raise this in the next Nova meeting. | 18:54 |
| hemna | heh yah | 18:54 |
| *** jamielennox|away is now known as jamielennox | 18:54 | |
| thingee | hemna, scottda: I went ahead and tagged these bugs as 'volumes' so that it would get the attention of the people who are focusing on these type of issues, per John's instructions. | 18:55 |
| thingee | hemna, scottda: but I think it would be good for someone to bring it up | 18:55 |
| hemna | I'm also working with asselin to work on something that might help verify live migration issues, where volumes get left behind on the source. | 18:55 |
| *** kjnelson has quit IRC | 18:55 | |
| hemna | it's all sort of related | 18:55 |
| hemna | thingee, http://paste.openstack.org/show/356117/ | 18:56 |
| thingee | hemna: and there is no .testrepository? http://haypo-notes.readthedocs.org/en/latest/openstack.html#tox-testr-db-type-could-not-be-determined-error | 18:58 |
| jgriffith | thingee: I updated the comments in that bug FWIW: https://bugs.launchpad.net/nova/+bug/1423654 | 18:59 |
| openstack | Launchpad bug 1423654 in OpenStack Compute (nova) "Nova rescue causes LVM timeouts after moving attachments" [Medium,Confirmed] - Assigned to John Griffith (john-griffith) | 18:59 |
| hemna | running tox runs py27 first, which creates a .testrepository, and causes the py34 failure. | 18:59 |
| *** MentalRay has joined #openstack-cinder | 19:00 | |
| *** jungleboyj has quit IRC | 19:00 | |
| jgriffith | thingee: the root of the problem is that a temp lvm device is created for the rescue, and the dm mapper paths get all honked up | 19:00 |
| scottda | thingee: I can put something on the agenda for the next Nova meeting.. | 19:01 |
| scottda | I had been thinking we might talk about the Cinder side of things at the mid-cycle, and then present some type of well-formed plan to Nova after that. | 19:02 |
| hemna | thingee, ok posted another comment on that py34 patchset. I show how 1 run works, and how another run fails. both from a clean clone. | 19:07 |
| hemna | gotta get some chow....bbiab | 19:07 |
| *** hemna is now known as hemnafk | 19:08 | |
| *** e0ne has joined #openstack-cinder | 19:09 | |
| *** rajinir_r has quit IRC | 19:09 | |
| scottda | hemna: What do you think? Do we have enough consenus on Cinder changes to start involving Nova ? It seems there are still multiple ideas being considered around using the DB and '*-ing' states for mutual exclusion, or some type of DLM, not to mentino taskflow... | 19:09 |
| scottda | s/mentino/mention | 19:09 |
| *** ndipanov has quit IRC | 19:10 | |
| *** simondodsley has quit IRC | 19:13 | |
| *** bitblt has joined #openstack-cinder | 19:13 | |
| thingee | scottda: I'm fine with checking ing state | 19:13 |
| thingee | I'm not convinced with some DLM | 19:14 |
| thingee | hemnafk: ^ | 19:14 |
| *** coolsvap|away is now known as coolsvap | 19:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/cinder: Set encrypted key in connection_info during initialize https://review.openstack.org/193673 | 19:14 |
| mriedem | there go all of my beautiful +2s | 19:15 |
| *** Rockyg has joined #openstack-cinder | 19:15 | |
| *** simondodsley has joined #openstack-cinder | 19:15 | |
| thingee | mriedem: pff I already gave it a +2 | 19:15 |
| thingee | I can't believe this is still going..haha | 19:16 |
| scottda | Yes, I'm not a fan of DLM. | 19:16 |
| *** bswartz has joined #openstack-cinder | 19:16 | |
| scottda | But does the Cinder team in general have any consensus on fixing the Nova <-> Cinder API? It seems like it does not... | 19:17 |
| scottda | I figured it might take the mid-cycle to sort things out, and then go to Nova with some type of plan. | 19:18 |
| jgriffith | scottda: if you want my 2 cents (which you probably don't) I think there is some concensus on the more "understood" thing | 19:19 |
| jgriffith | scottda: there's def low hanging fruit | 19:19 |
| jgriffith | scottda: clean up of the messy reservce/attach stuff | 19:19 |
| thingee | scottda: besides the ing state checking, what are the other cases? | 19:19 |
| jgriffith | scottda: consolidate the calls in Cinder (Cinder side is a bigger mess right now IMHO) | 19:19 |
| thingee | scottda: that we need consensus on | 19:20 |
| jgriffith | scottda: and making sure status's are checked sufficiently is of course a huge win, can't see an arg there | 19:20 |
| scottda | jgriffith: certainly want your opinion... | 19:20 |
| thingee | jgriffith: can you be more specific on the consolidating? | 19:20 |
| scottda | I think the long bug list is a good starting point. | 19:20 |
| jgriffith | thingee: scottda so there's more problems on the cinder side IMO than anywhere else | 19:21 |
| jgriffith | thingee: scottda I tried to fix some of this up last year.... | 19:21 |
| jgriffith | thingee: scottda some folks use intialize_con to create targets, others use export | 19:21 |
| scottda | jgriffith: agreed that cinder fixup is the first step...I just want to start getting Nova onboard for the changes that will have to take place in Nova as well. | 19:21 |
| jgriffith | thingee: scottda there's a hodge-podge of implementation that resulted from a lack of clear understanding/documentation of the flow and what was intended to do what | 19:22 |
| jgriffith | so folks just interpretted however they interpretted it :) | 19:22 |
| thingee | jgriffith: doesn't initialize_con result in create_export anyways? | 19:22 |
| jgriffith | scottda: for some drivers :) | 19:23 |
| jgriffith | scottda: that's kind of the problem IIRC | 19:23 |
| thingee | jgriffith: this is just coming from the volume manager. | 19:23 |
| thingee | jgriffith: if I'm reading things right | 19:23 |
| *** r-daneel has joined #openstack-cinder | 19:23 | |
| jgriffith | scottda: there was a divergence in there WRT what diff drivers were doing | 19:23 |
| jgriffith | scottda: thingee yeah, but one of the problems is the inheritance structure of the drivers caused some divergence | 19:24 |
| jgriffith | scottda: thingee and some drivers over-rode the methods | 19:24 |
| jgriffith | scottda: thingee anyway... if you guys want I can dig up the details later | 19:24 |
| scottda | Well, things are pretty messy right now. I've listed a few ideas on what should be done in the etherpad under "Goal: Fixup the CinderNova API" | 19:24 |
| jgriffith | scottda: thingee but really the whole attach process on the Cinder side is not as consistent as it should be. Unless somebody fixed it all up when I wasn't looking :) | 19:25 |
| jgriffith | scottda: yeah, I'm reading that now.... | 19:25 |
| scottda | Please add/change/delete whatever...it's just my rough ideas. | 19:25 |
| thingee | jgriffith: I'm saying the manager's initialize connection method sets the order of calling create_export -> initialize_connection ... the driver can only override what those methods do, not the order. | 19:25 |
| thingee | jgriffith: maybe I'm confused here | 19:25 |
| jgriffith | scottda: you've got a lot of awesome stuff there | 19:25 |
| *** shyama has quit IRC | 19:25 | |
| scottda | just a brain dump... | 19:26 |
| bswartz | hey is the midcycle meetup at the same venue as 12 months ago? HP hosting? | 19:26 |
| scottda | bswartz: yes | 19:26 |
| scottda | https://etherpad.openstack.org/p/cinder-liberty-midcycle-meetup | 19:26 |
| bswartz | scottda: ty that's exactly what I was looking for | 19:27 |
| * bswartz notes that etherpad is all-knowing | 19:27 | |
| scottda | Too bad you cannot search for an etherpad by keywords. | 19:28 |
| thingee | bswartz: also useful https://wiki.openstack.org/wiki/Sprints#Liberty_sprints | 19:28 |
| bswartz | does google index the contents of etherpads? | 19:28 |
| jgriffith | thingee: yeah, but look at for example the 3par driver.... | 19:28 |
| jgriffith | thingee: it's intialize connection does the "create_export" stuff, and just passes when create_export is called | 19:29 |
| bswartz | thingee: ty | 19:29 |
| *** esker has quit IRC | 19:29 | |
| thingee | jgriffith: haha ok fair | 19:29 |
| thingee | jgriffith: lets note those kind of drivers and fix that? | 19:29 |
| *** esker has joined #openstack-cinder | 19:29 | |
| jgriffith | thingee: I'm not saying that's an issue | 19:29 |
| jgriffith | thingee: just saying... some drivers do it that way, others do it a different way | 19:30 |
| scottda | I'll add "audit drivers" to the etherpad... | 19:30 |
| thingee | jgriffith: help me fill in the gap...how does this relate with things being broken between nova and cinder? If nova is going to call initialize_connection...the export either happens in 3pars cases within it's initialize_connection call, or it happens later in the initiailize_connection from the manager | 19:31 |
| jgriffith | thingee: it may not matter any longer. We used to however have create-export calls outside of intialize_connection | 19:33 |
| jgriffith | thingee: and I think I very clearly said "there's nothing wrong with it" just that it was inconsistent | 19:33 |
| *** patrickeast_ has quit IRC | 19:33 | |
| jgriffith | thingee: let me scroll back and find my comment.... | 19:33 |
| thingee | jgriffith: ok... I might be more confused now trying to understand things here. | 19:33 |
| thingee | :) | 19:33 |
| *** zul has quit IRC | 19:34 | |
| scottda | thingee: I think that's a common issue. There's a bunch of different problems. I want to work on documenting some of the use cases and how things *should* work. That might help... | 19:34 |
| jgriffith | bahh... anyway, I said "not saying it's wrong" | 19:35 |
| jgriffith | I'll just keep my opinions to myself | 19:35 |
| thingee | jgriffith: I'm not upset in talking to you here. I'm genuinely trying to under here because I feel ignorant of this topic. | 19:36 |
| thingee | and you might understand things here because you have opinions | 19:37 |
| thingee | scottda: thanks | 19:37 |
| *** changbl has joined #openstack-cinder | 19:37 | |
| jgriffith | thingee: not at all, I didn't take it that way | 19:38 |
| thingee | jgriffith: well I don't want you to feel you need to keep your opinions to yourself. | 19:39 |
| jgriffith | thingee: I don't think I can be accused of keeping opinions to myself :) Even when I say I'm going to :) | 19:39 |
| jgriffith | thingee: so here's an example of what I'm looking at:https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1081 | 19:40 |
| jgriffith | in a pass that returns "None" right... so there's no model_update | 19:40 |
| jgriffith | thingee: it's stuffed in the driver.initialize_connection call | 19:42 |
| jgriffith | thingee: so I'm not sure but I *think* this may be part of why some drivers can't do live-migration | 19:42 |
| jgriffith | thingee: they never update the model info after the create | 19:42 |
| jgriffith | thingee: and IIRC I thought I heard some folks sayign that can't reuse connections/iqn's after a detach | 19:42 |
| jgriffith | thingee: so that would be a problem I think/maybe | 19:43 |
| scottda | jgriffith: Can you put that info somewhere in the etherpad? Maybe under "Random Notes"? | 19:45 |
| jgriffith | scottda: sure | 19:45 |
| scottda | thx | 19:45 |
| *** e0ne has quit IRC | 19:47 | |
| *** timcl has quit IRC | 19:50 | |
| *** lpetrut has joined #openstack-cinder | 19:50 | |
| *** daneyon has joined #openstack-cinder | 19:55 | |
| thingee | jgriffith: I see. I'm unsure what the provider_location has to do with live migration. since that's what you would usually get from the create_export return | 19:55 |
| *** lpetrut has quit IRC | 19:55 | |
| thingee | jgriffith: oh wait | 19:55 |
| thingee | jgriffith: got it reusing | 19:55 |
| jgriffith | thingee: yeah | 19:55 |
| jgriffith | thingee: but I'm not familiar with other devices enough to know if that's the problem or not | 19:56 |
| jgriffith | thingee: but it sounded like it could be | 19:56 |
| thingee | jgriffith: assume it is I suppose | 19:56 |
| jgriffith | thingee: and besides, having methods related to connectivity that just do a "pass" is kinda suspect | 19:56 |
| jgriffith | thingee: might be fine, but def seems like something to look at and make sure it's understood | 19:57 |
| *** coolsvap is now known as coolsvap|away | 19:58 | |
| *** bpoulos has left #openstack-cinder | 20:00 | |
| *** vilobhmm has joined #openstack-cinder | 20:08 | |
| *** coolsvap|away is now known as coolsvap | 20:09 | |
| *** openstackgerrit has quit IRC | 20:10 | |
| *** openstackgerrit has joined #openstack-cinder | 20:10 | |
| *** e0ne has joined #openstack-cinder | 20:11 | |
| *** kjnelson has joined #openstack-cinder | 20:11 | |
| ericksonsantos | vilobhmm, Hi :) | 20:11 |
| *** e0ne has quit IRC | 20:12 | |
| vilobhmm | ericksonsantos : reviewing your code | 20:12 |
| ericksonsantos | vilobhmm, I've uploaded a new patch set adding more tests | 20:12 |
| vilobhmm | just came back from vacation | 20:12 |
| ericksonsantos | vilobhmm, nice | 20:13 |
| *** ronis_ has quit IRC | 20:13 | |
| vilobhmm | thanks for moving the patch forward | 20:13 |
| vilobhmm | much appreciated | 20:13 |
| ericksonsantos | vilobhmm, I'll give you time to breathe haha | 20:13 |
| ericksonsantos | vilobhmm, no problem :) | 20:14 |
| vilobhmm | done…looks ok to me | 20:14 |
| ericksonsantos | vilobhmm, awesome! | 20:15 |
| *** jungleboyj has joined #openstack-cinder | 20:15 | |
| vilobhmm | smcginnis : If you can have a look at https://review.openstack.org/#/c/194406/ it will be helpful | 20:15 |
| vilobhmm | jungleboyj : ^^ | 20:16 |
| smcginnis | vilobhmm: Sure. Tied up with something right now, but have it open in a tab now and will try to get to it soon. | 20:16 |
| vilobhmm | smcginnis: whenever you are free :) no rush…thanks! | 20:16 |
| openstackgerrit | Patrick East proposed openstack/cinder: Add Cinder internal tenant support. https://review.openstack.org/199352 | 20:16 |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP Generic image cache. https://review.openstack.org/195795 | 20:16 |
| jungleboyj | vilobhmm: Looking./ | 20:17 |
| vilobhmm | jungleboyj : thanks ! | 20:17 |
| *** belmoreira has joined #openstack-cinder | 20:18 | |
| *** merooney has quit IRC | 20:20 | |
| ericksonsantos | smcginnis, jungleboyj it is nice to have your eyes on this change \o | 20:21 |
| openstackgerrit | Alex Meade proposed openstack/cinder: NetApp E-Series: Add debug tracing https://review.openstack.org/170580 | 20:21 |
| *** crose has quit IRC | 20:22 | |
| *** dustins has quit IRC | 20:23 | |
| jungleboyj | Seems relatively straight forward to me. | 20:25 |
| *** sgotliv has quit IRC | 20:26 | |
| ericksonsantos | jungleboyj, thanks! we will keep things moving! | 20:29 |
| *** daneyon has quit IRC | 20:29 | |
| *** simondodsley has quit IRC | 20:30 | |
| *** bswartz has quit IRC | 20:33 | |
| *** annasort has quit IRC | 20:38 | |
| *** jasondotstar has joined #openstack-cinder | 20:40 | |
| *** Yogi1 has quit IRC | 20:41 | |
| *** lpetrut has joined #openstack-cinder | 20:44 | |
| *** jgriffith has quit IRC | 20:45 | |
| *** rushil has joined #openstack-cinder | 20:45 | |
| xyang | eharney: hi, are you there | 20:53 |
| eharney | xyang: hi | 20:53 |
| eharney | what's up? :) | 20:53 |
| xyang | eharney: so about your comments on LVM:) | 20:53 |
| eharney | yeeeeeaah | 20:53 |
| xyang | eharney: right now LVM doesn't do attach snapshot in my patch | 20:53 |
| xyang | eharney: I can't do it because it is always local | 20:54 |
| xyang | eharney: so I tested it on my setup, but when I submit the patch, it is still a local snapshot, the path is already there, no need to attach | 20:54 |
| eharney | how do we end up needing the if "snapshot" path in create_export then? | 20:54 |
| xyang | eharney: because I tested it that way, so we could use it later | 20:55 |
| *** Lee1092 has quit IRC | 20:55 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 20:55 |
| xyang | eharney: if we use a remote volume node in the future for backup | 20:55 |
| *** annegentle has quit IRC | 20:55 | |
| xyang | eharney: then it can be used. | 20:56 |
| eharney | i see | 20:56 |
| *** akerr_ has quit IRC | 20:56 | |
| xyang | eharney: where if you think that will screw things up, maybe we need to remove those? | 20:57 |
| tbarron | xyang: eharney: when we decouple backup service from volume service we'll want to do remote attach even for lvm | 20:57 |
| xyang | eharney: somehow I need to provide a way to do attach snapshot as the default | 20:57 |
| xyang | tbarron: it that is the case, then this will be useful | 20:57 |
| eharney | tbarron: but the other half of my argument was that we could just do a volume clone instead and get the same performance on thin | 20:57 |
| tbarron | xyang: I agree | 20:57 |
| eharney | isn't the idea to do a clone if attach_snapshot isn't supported? | 20:58 |
| xyang | eharney: yes, but snapshot is quicker | 20:58 |
| eharney | i'm not sure it is if using thin lvm | 20:58 |
| eharney | they should be doing the same amount of work | 20:58 |
| xyang | eharney: is it? I don't know. usually snapshot is quicker | 20:58 |
| eharney | for thick it will be faster, but i've been wanting to move us away from thick for a long time anyway | 20:59 |
| tbarron | eharney: I don't disagree with that. whether snap or clone is faster likely depends on thin/thick IIUC | 20:59 |
| eharney | we should just give up on thick at this point now that we have thin prov accounting | 20:59 |
| eharney | but anyhow | 20:59 |
| eharney | i'm just mostly not sure that the complexity of more LVM issues (which we have been poking around at for years) is worthwhile | 21:00 |
| xyang | eharney: I'm not sure about LVM thin, but for other drivers may still want to do attach snapshot | 21:00 |
| eharney | sure, and that's fine | 21:00 |
| *** rushil has quit IRC | 21:00 | |
| xyang | eharney: well, clone is always a choice if we don't want to introduce too much complexity into LVM | 21:01 |
| eharney | i dunno, we can look at it for a bit and see what people think, but i'd lean toward just avoiding it | 21:02 |
| eharney | or at least avoid it if using thin | 21:02 |
| xyang | eharney: sure, we could change the logic if there are concerns | 21:03 |
| *** Apoorva has quit IRC | 21:04 | |
| xyang | eharney: whether to use snapshot or not can be determined by backup_use_temp_snapshot(). we can do some check there to see if it is thin | 21:04 |
| eharney | that makes sense other than maybe still causing issues for thick | 21:07 |
| *** tellesnobrega_ has quit IRC | 21:07 | |
| *** esker has quit IRC | 21:07 | |
| *** jasondotstar has quit IRC | 21:08 | |
| xyang | eharney: if you have strong concerns, I can remove it. Let's see if there are other comments. We don't need to attach snapshot right now any way as it is all local | 21:09 |
| *** thangp has quit IRC | 21:09 | |
| eharney | sure | 21:09 |
| *** lpetrut has quit IRC | 21:12 | |
| *** annasort has joined #openstack-cinder | 21:13 | |
| *** hemnafk is now known as hemna | 21:15 | |
| *** IlyaG has joined #openstack-cinder | 21:18 | |
| *** coolsvap is now known as coolsvap|away | 21:22 | |
| *** tellesnobrega_ has joined #openstack-cinder | 21:23 | |
| *** jgriffith has joined #openstack-cinder | 21:24 | |
| *** annegentle has joined #openstack-cinder | 21:25 | |
| *** tellesnobrega_ has quit IRC | 21:26 | |
| haypo | hemna: "and if python-gdbm is required, then that has to be handled by the patch." hum, i disagree. it's a bug in testrepository | 21:26 |
| hemna | running "tox" shouldn't break by your patch. and it does. | 21:27 |
| hemna | that has to get fixed or it'll be -1 from me. | 21:27 |
| *** jgriffith has quit IRC | 21:27 | |
| *** rajinir has joined #openstack-cinder | 21:27 | |
| hemna | haypo, I provided 2 example clean runs against your patch. one works, the other doesn't. | 21:28 |
| haypo | hemna: i discussed with a testrepository developer. they are moving to sqlite | 21:28 |
| haypo | hemna: it would solve “db type could not be determined” and “gdbm is missing” errors | 21:28 |
| *** e0ne has joined #openstack-cinder | 21:29 | |
| *** DericHorn-HP has joined #openstack-cinder | 21:29 | |
| hemna | haypo, whatever you have to do to make that patch work out of the box for folks. it can't break running tox. | 21:29 |
| *** fifieldt has quit IRC | 21:30 | |
| *** kjnelson has quit IRC | 21:31 | |
| haypo | i checked other projects. heat runs python 3.4 tests using testr (https://github.com/openstack/heat/blob/master/tox.ini#L69), ironic too (https://github.com/openstack/ironic/blob/master/tox.ini#L16) | 21:33 |
| haypo | hemna: testr is not widely used to run py34 tests yet because only a few tests pass on python 3.4 in most openstack projects | 21:33 |
| haypo | :-/ | 21:34 |
| *** belmoreira has quit IRC | 21:34 | |
| *** e0ne has quit IRC | 21:37 | |
| openstackgerrit | Victor Stinner proposed openstack/os-brick: Cleanup Python 3 changes https://review.openstack.org/197624 | 21:37 |
| haypo | hemna: ok, i see you problem with "tox". it's fixed by ^^ | 21:37 |
| haypo | hemna: py34 must be before py27 in the envlist of tox.ini | 21:39 |
| *** gouthamr has quit IRC | 21:40 | |
| hemna | haypo, ok I'll check it out in a bit. totally slammed right now. | 21:41 |
| *** Rockyg has quit IRC | 21:42 | |
| *** fifieldt has joined #openstack-cinder | 21:44 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Remove unnecessary export after volume migration https://review.openstack.org/194223 | 21:46 |
| *** jasondotstar has joined #openstack-cinder | 21:48 | |
| *** annegentle has quit IRC | 21:49 | |
| *** annegentle has joined #openstack-cinder | 21:49 | |
| haypo | thingee, hemna : again, https://review.openstack.org/197624 is not need for a new release of os-brick. it's a just a cleanup ;) | 21:49 |
| lifeless | haypo: you can use testr easily for py34, just give it the filenames you want to test | 21:50 |
| lifeless | haypo: tsetr run test_thing test_other test_quad | 21:51 |
| lifeless | haypo: as long as the other files can be *parsed* it will work | 21:51 |
| *** IlyaG has quit IRC | 21:51 | |
| haypo | lifeless: since all tests now pass on python 3.4, i don't want to pass any filename anymore | 21:51 |
| haypo | lifeless: just run the full test suite | 21:51 |
| lifeless | haypo: cool | 21:51 |
| *** xyang1 has joined #openstack-cinder | 21:51 | |
| haypo | lifeless: it's the purpose of my change ;) | 21:51 |
| hemna | haypo, still failing. | 21:52 |
| hemna | http://paste.openstack.org/show/356447/ | 21:52 |
| *** vokt has quit IRC | 21:53 | |
| *** jungleboyj has quit IRC | 21:53 | |
| haypo | hemna: that's because you have to install gdbm | 21:54 |
| hemna | haypo, that patch needs to account for that. | 21:55 |
| openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 21:56 |
| *** xyang has quit IRC | 21:56 | |
| haypo | hemna: you cannot install gdbm using requirements.txt or test-requirements.txt. it's part of the system, it's a package | 21:56 |
| haypo | hemna: well, gdbm is supposed to be part of the python stdlib, but ubuntu chose to split it in a different package | 21:56 |
| haypo | lifeless: ^^ | 21:56 |
| *** mriedem has quit IRC | 21:56 | |
| hemna | :(!! | 21:57 |
| hemna | that completely blows. this is a standalone library that can/will be used outside of openstack, so we can't rely on devstack installing that package just to make tox work :( | 21:57 |
| hemna | haypo, I presume other projects are going to run into this same problem, once we nuke the tox.ini whitelisted mechanism for running py34 | 21:58 |
| lifeless | hemna: yes | 21:59 |
| hemna | ouch | 21:59 |
| haypo | gdbm is installed explicitly for testr in devstack: https://github.com/openstack-dev/devstack/blob/master/files/debs/general#L20 | 21:59 |
| lifeless | hemna: its because python2 and python3 don't by default support *any* overlapping dbm format files. | 21:59 |
| haypo | i agree that it's annoying to have to install it manually | 21:59 |
| lifeless | hemna: We've been around this multiple times trying to make it Just Work. | 21:59 |
| hemna | dang | 21:59 |
| haypo | i'm using fedora, and it looks like gdbm is part of the base system, so i didn't have your issue | 21:59 |
| lifeless | hemna: what is needed is someone to work up a patch to use a different serialization method for the test timing database. | 21:59 |
| hemna | I'm going to get pinged any time an os-brick dev runs tox on their box now. :( | 22:00 |
| lifeless | hemna: -or- | 22:00 |
| *** jasondotstar has quit IRC | 22:00 | |
| lifeless | hemna: for OpenStack to stop installing testr inside the tox venv. | 22:00 |
| lifeless | hemna: its not meant to be used like that. | 22:00 |
| haypo | hemna: "I presume other projects are going to run into this same problem" yes. and as i wrote, heat & ironic already run python 3.4 tests using testr | 22:00 |
| lifeless | hemna: the problem *only* occurs because its being run with the same Python as the unittests are running under. | 22:00 |
| lifeless | hemna: but IMO testr, like tox, should be system installed. | 22:00 |
| lifeless | we already tell people to install tox | 22:01 |
| lifeless | there's no reason we can't tell them to install testr too. | 22:01 |
| hemna | yah we have to system install tox to even run it | 22:01 |
| haypo | hemna: you only have two steps to do, but only once: (1) install gdbm (2) remove .testrepository (if you already ran before), and just after that run tox -e py34 | 22:01 |
| lifeless | and it would entirely avoid this problem | 22:01 |
| hemna | that would work for me. | 22:01 |
| haypo | in practice, (2) occurs again if you manually remove .testrepository and then run tox -e py27 | 22:01 |
| hemna | haypo, the steps to make it work are fine and all, but when someone else tries to run tox for the lib, they'll complain it doesn't work (just like I've been), and then hunt me down to find out why. | 22:02 |
| haypo | hemna: i know i know... i lost many hours because of testr bugs :) | 22:02 |
| haypo | hemna: i don't want to harass you forever with testr issues. i think i will just abandon my change | 22:02 |
| lifeless | so | 22:02 |
| hemna | :( | 22:02 |
| lifeless | I'm going to argue here | 22:02 |
| lifeless | testr, with the subunit stream that gets indexed | 22:03 |
| lifeless | is part of the main openstack testing infrastructure | 22:03 |
| hemna | haypo, or can we make a change as lifeless suggests, to require testr as a system install, like tox. ? | 22:03 |
| lifeless | *not* using it is a fairly significant deviation, and cuts you off from the infra statistics gathering | 22:03 |
| lifeless | it was ok as a bootstrap thing for py34 | 22:03 |
| hemna | lifeless, +1 | 22:03 |
| lifeless | I'm obviously biased as the primary author of testr | 22:04 |
| lifeless | but | 22:04 |
| *** boris-42 has joined #openstack-cinder | 22:04 | |
| haypo | i abandonned https://review.openstack.org/#/c/197624/ | 22:04 |
| lifeless | I haven't driven any of the large adoption of it within openstack | 22:04 |
| *** annasort has quit IRC | 22:04 | |
| haypo | i don't want to spend all my nights on this silly pendantic change | 22:04 |
| hemna | I'd be ok adopting whatever the rest of openstack chooses, as long as it works and doesn't require some hidden knowledge of needing to install python-gdbm. | 22:05 |
| haypo | "I'm obviously biased as the primary author of testr" ah, really? :-D | 22:05 |
| hemna | :) | 22:05 |
| lifeless | haypo: you didn't know? Or is that sarcasm ? | 22:05 |
| haypo | lifeless: i knew it :) | 22:05 |
| hemna | so using testr outside of the venv seems like the right thing no? | 22:06 |
| haypo | lifeless, hemna: feel free to takeover my abandonned my patch, but i just want to spend any more time on it | 22:06 |
| lifeless | hemna: so the rest of openstack just accepts the current limitation and installs python-gdbm everywhere | 22:06 |
| haypo | i have far more important stuff to do ;) | 22:06 |
| hemna | lifeless, heh ok. I guess I'm just being a trouble make here. I was just hoping not to break things. | 22:07 |
| lifeless | hemna: I think your lowest investment option is to just take haypo's patch, and add a readme entry ssaying 'to test this repo, install tox and python-gdbm' | 22:07 |
| lifeless | hemna: from there, you can obviously make things better | 22:07 |
| *** IlyaG has joined #openstack-cinder | 22:07 | |
| lifeless | hemna: but at least you won't be an outlier | 22:07 |
| hemna | yah that's good | 22:07 |
| lifeless | haypo: please unabandon | 22:08 |
| *** IlyaG has quit IRC | 22:08 | |
| haypo | lifeless: i'm not going to touch this patch again :-p | 22:08 |
| hemna | ok, so haypo just update the readme and document that as lifeless described. | 22:08 |
| lifeless | haypo: noone else can unabandond | 22:08 |
| lifeless | haypo: I'll add the readme bit for hemna but it needs to be unabandoned first | 22:08 |
| hemna | haypo, please :) | 22:08 |
| openstackgerrit | Eric Harney proposed openstack/cinder: LVM Thin Provisioning auto-detect https://review.openstack.org/104653 | 22:08 |
| hemna | haypo, then we can share a beer at the next conference and move on :) | 22:09 |
| *** IlyaG_ has joined #openstack-cinder | 22:12 | |
| haypo | why did i need to unabandon my patch? | 22:12 |
| hemna | haypo, so we can just update the readme with the new requirement of installing python-gdbm | 22:13 |
| hemna | then +A the patch | 22:13 |
| hemna | since I guess the lib is no worse off than other projects wrt to py34 and needing python-gdbm manually installed. | 22:14 |
| lifeless | haypo: because gerrit won't let other people unabandon to fix it up for you | 22:15 |
| lifeless | haypo: if you abandon, they have to start over | 22:15 |
| lifeless | haypo: if you un-abandon, I can do what hemna is requesting | 22:15 |
| hemna | haypo, thank you. | 22:17 |
| haypo | hemna: i restored my change. you should now be able to take over the change id | 22:17 |
| openstackgerrit | Eric Harney proposed openstack/cinder: LVM Thin Provisioning auto-detect https://review.openstack.org/104653 | 22:18 |
| *** MentalRay has quit IRC | 22:18 | |
| *** jgriffit1 has joined #openstack-cinder | 22:20 | |
| *** jgriffit1 has quit IRC | 22:20 | |
| openstackgerrit | lifeless proposed openstack/os-brick: Cleanup Python 3 changes https://review.openstack.org/197624 | 22:20 |
| *** annegentle has quit IRC | 22:21 | |
| *** Guest69012 has joined #openstack-cinder | 22:23 | |
| *** IanGovett1 has joined #openstack-cinder | 22:26 | |
| hemna | lifeless, thanks for the help. | 22:27 |
| *** DericHorn-HP has quit IRC | 22:28 | |
| *** IanGovett has quit IRC | 22:28 | |
| *** DericHorn-HP has joined #openstack-cinder | 22:28 | |
| *** eharney has quit IRC | 22:28 | |
| haypo | hemna: i am a little bit exhausted by python 3. it takes a lot of energy to fix a million of projects and then harass people to get a release | 22:28 |
| hemna | haypo, yah understandable. | 22:29 |
| hemna | haypo, thanks a bunch for your help on this though. | 22:29 |
| hemna | sorry for being such a pest about the failures. | 22:29 |
| hemna | I really appreciate your efforts. | 22:29 |
| haypo | for Routes, it took ~3 months to get my python 3 patch merged. and now i'm waiting a release since ~20 days | 22:30 |
| haypo | for pyeclib, i sent a pull request 3 months ago, and developers replied that they will check it and then nothing. i ping them every 15 days | 22:31 |
| haypo | my patch is dead simple, i don't understand | 22:31 |
| hemna | :( | 22:31 |
| haypo | i already did that a lot of times for many other projects | 22:31 |
| *** Guest69012 has quit IRC | 22:32 | |
| *** akshai has quit IRC | 22:33 | |
| *** IlyaG_ has quit IRC | 22:37 | |
| *** eharney has joined #openstack-cinder | 22:42 | |
| *** mriedem has joined #openstack-cinder | 22:46 | |
| *** eharney has quit IRC | 22:47 | |
| *** primechuck has quit IRC | 22:47 | |
| *** edmondsw has quit IRC | 22:50 | |
| thingee | hemna, haypo, lifeless happy you all came to this conclusion...I was reading through the backlog and about to suggest just updating the readme and be done with it :P | 22:54 |
| hemna | :) | 22:54 |
| *** boris-42 has quit IRC | 22:56 | |
| *** Adriano__ has quit IRC | 22:56 | |
| *** boris-42 has joined #openstack-cinder | 22:57 | |
| *** DericHorn-HP has quit IRC | 22:58 | |
| *** Adriano__ has joined #openstack-cinder | 22:59 | |
| *** IlyaG has joined #openstack-cinder | 22:59 | |
| *** erlon has quit IRC | 23:00 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Add new Connector APIs for path validation https://review.openstack.org/199764 | 23:03 |
| openstackgerrit | Mike Perez proposed openstack/os-brick: Prep for 0.2.1 release https://review.openstack.org/199766 | 23:06 |
| openstackgerrit | Mike Perez proposed openstack/os-brick: Prep for 0.2.1 release https://review.openstack.org/199766 | 23:07 |
| *** Adriano__ has quit IRC | 23:12 | |
| *** annashen has quit IRC | 23:12 | |
| *** zhenguo has joined #openstack-cinder | 23:14 | |
| *** jgriffith has joined #openstack-cinder | 23:18 | |
| *** ganso_ has quit IRC | 23:18 | |
| *** IlyaG has quit IRC | 23:20 | |
| *** jwcroppe has quit IRC | 23:26 | |
| *** jwcroppe has joined #openstack-cinder | 23:27 | |
| *** IanGovett1 has quit IRC | 23:29 | |
| *** zul has joined #openstack-cinder | 23:29 | |
| hemna | xyang1, | 23:29 |
| hemna | xyang1, the ScaleIO brick CI is instantly failing again. :( | 23:31 |
| *** jwcroppe has quit IRC | 23:31 | |
| *** diogogmt has quit IRC | 23:38 | |
| *** topshare has joined #openstack-cinder | 23:41 | |
| xyang1 | hemna: I think the way we run the CI is not reliable. I don't know why it is different from cinder CI, but the way it is set up is different. | 23:46 |
| hemna | xyang1, ok. it failed in 5 seconds :) so it's quick at least | 23:47 |
| xyang1 | hemna: Has nothing to do with the code. | 23:47 |
| xyang1 | hemna: It has to with cherrypicking, but honestly I don't know how to explain:). Have to get another run later | 23:48 |
| xyang1 | hemna: I'll need to ask asselin what is the trick | 23:48 |
| hemna | xyang1, ok. coolio. | 23:49 |
| hemna | git.openstack.org seems borked at the moment anyway, so on that note...time to head home. | 23:50 |
| *** hemna is now known as hemnafk | 23:50 | |
| *** jwcroppe has joined #openstack-cinder | 23:55 | |
| *** jungleboyj has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!