*** markvoelker has joined #openstack-cinder | 00:01 | |
*** MentalRay has joined #openstack-cinder | 00:02 | |
*** markvoelker has quit IRC | 00:05 | |
*** Yogi1 has joined #openstack-cinder | 00:08 | |
*** s-moriya has joined #openstack-cinder | 00:09 | |
*** MentalRay has quit IRC | 00:25 | |
*** Yogi11 has joined #openstack-cinder | 00:26 | |
*** Yogi1 has quit IRC | 00:28 | |
*** dimsum__ has quit IRC | 00:39 | |
*** dimsum__ has joined #openstack-cinder | 00:41 | |
*** annashen has joined #openstack-cinder | 00:48 | |
*** dims_ has joined #openstack-cinder | 00:50 | |
*** dimsum__ has quit IRC | 00:50 | |
*** IanGovett has joined #openstack-cinder | 00:53 | |
*** MentalRay has joined #openstack-cinder | 00:59 | |
*** MentalRay has quit IRC | 00:59 | |
*** salv-orlando has joined #openstack-cinder | 01:06 | |
*** annashen has quit IRC | 01:10 | |
*** salv-orlando has quit IRC | 01:11 | |
*** annashen has joined #openstack-cinder | 01:17 | |
*** vilobhmm has joined #openstack-cinder | 01:17 | |
*** dimsum__ has joined #openstack-cinder | 01:17 | |
*** dims_ has quit IRC | 01:17 | |
*** marcusvrn has joined #openstack-cinder | 01:29 | |
openstackgerrit | Lin Yang proposed openstack/python-cinderclient: Improve error message when exceed cinder quota exceed https://review.openstack.org/178946 | 01:31 |
---|---|---|
*** marcusvrn has quit IRC | 01:33 | |
*** dims_ has joined #openstack-cinder | 01:40 | |
*** dimsum__ has quit IRC | 01:41 | |
*** Lee1092 has joined #openstack-cinder | 01:42 | |
*** annashen has quit IRC | 01:46 | |
*** annashen has joined #openstack-cinder | 01:49 | |
*** markvoelker has joined #openstack-cinder | 01:50 | |
*** markvoelker has quit IRC | 01:55 | |
*** annashen has quit IRC | 02:01 | |
*** marcusvrn has joined #openstack-cinder | 02:03 | |
*** linkedinyou has quit IRC | 02:06 | |
*** Yogi11 has quit IRC | 02:07 | |
*** marcusvrn has quit IRC | 02:23 | |
*** IanGovett has quit IRC | 02:53 | |
*** vilobhmm has quit IRC | 03:05 | |
*** alexpilotti has joined #openstack-cinder | 03:13 | |
*** marcusvrn has joined #openstack-cinder | 03:17 | |
*** salv-orlando has joined #openstack-cinder | 03:19 | |
*** marcusvrn has quit IRC | 03:21 | |
*** salv-orlando has quit IRC | 03:22 | |
*** takedakn has joined #openstack-cinder | 03:24 | |
*** sgotliv has joined #openstack-cinder | 03:29 | |
*** gouthamr has joined #openstack-cinder | 03:32 | |
*** takedakn has quit IRC | 03:33 | |
*** takedakn has joined #openstack-cinder | 03:34 | |
*** annashen has joined #openstack-cinder | 03:34 | |
*** mriedem1 has joined #openstack-cinder | 03:37 | |
*** mriedem has quit IRC | 03:38 | |
*** markvoelker has joined #openstack-cinder | 03:38 | |
*** dims_ has quit IRC | 03:40 | |
*** markvoelker has quit IRC | 03:43 | |
*** Longgeek has quit IRC | 03:48 | |
*** Longgeek has joined #openstack-cinder | 03:49 | |
*** annashen has quit IRC | 03:54 | |
*** gouthamr has quit IRC | 03:58 | |
*** takedakn has quit IRC | 04:02 | |
*** annashen has joined #openstack-cinder | 04:12 | |
*** avishay_ has joined #openstack-cinder | 04:13 | |
*** laughterwym has joined #openstack-cinder | 04:14 | |
*** annashen has quit IRC | 04:31 | |
*** pradipta has joined #openstack-cinder | 04:35 | |
*** dimsum__ has joined #openstack-cinder | 04:41 | |
*** dimsum__ has quit IRC | 04:46 | |
*** jwang has joined #openstack-cinder | 04:55 | |
*** vilobhmm has joined #openstack-cinder | 04:56 | |
*** jwang_ has quit IRC | 04:57 | |
*** jwang_ has joined #openstack-cinder | 04:59 | |
*** jwang has quit IRC | 05:01 | |
*** salv-orlando has joined #openstack-cinder | 05:02 | |
*** vilobhmm has quit IRC | 05:05 | |
*** nkrinner has joined #openstack-cinder | 05:06 | |
*** vilobhmm has joined #openstack-cinder | 05:08 | |
*** vilobhmm has quit IRC | 05:10 | |
*** vilobhmm has joined #openstack-cinder | 05:12 | |
*** vilobhmm has quit IRC | 05:12 | |
*** salv-orlando has quit IRC | 05:13 | |
*** BharatK has joined #openstack-cinder | 05:13 | |
openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 05:13 |
*** nihilifer has joined #openstack-cinder | 05:26 | |
*** markvoelker has joined #openstack-cinder | 05:27 | |
*** rushiagr_away is now known as rushiagr | 05:27 | |
*** shausy has joined #openstack-cinder | 05:27 | |
*** markvoelker has quit IRC | 05:32 | |
*** nlevinki has joined #openstack-cinder | 05:33 | |
*** shausy has quit IRC | 05:38 | |
*** shausy has joined #openstack-cinder | 05:40 | |
*** lpetrut has joined #openstack-cinder | 05:42 | |
*** shausy has quit IRC | 05:45 | |
*** nihilifer has quit IRC | 05:46 | |
*** alexpilotti has quit IRC | 05:50 | |
*** coolsvapz is now known as coolsvap | 06:02 | |
*** deepakcs has joined #openstack-cinder | 06:08 | |
*** belmoreira has joined #openstack-cinder | 06:10 | |
*** avishay_ has quit IRC | 06:25 | |
*** avishay_ has joined #openstack-cinder | 06:25 | |
*** agarciam has joined #openstack-cinder | 06:26 | |
*** pradipta has quit IRC | 06:34 | |
*** ianbrown has quit IRC | 06:35 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to oslo.service https://review.openstack.org/193951 | 06:39 |
*** anshul has joined #openstack-cinder | 06:41 | |
*** avishay_ is now known as avishay | 06:44 | |
*** pradipta has joined #openstack-cinder | 06:46 | |
*** Longgeek has quit IRC | 06:47 | |
*** Longgeek has joined #openstack-cinder | 06:47 | |
openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO volume driver retry snapshot operation https://review.openstack.org/193954 | 06:48 |
*** lpetrut has quit IRC | 06:58 | |
openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 06:58 |
*** yamada-h has joined #openstack-cinder | 07:09 | |
*** salv-orlando has joined #openstack-cinder | 07:10 | |
*** salv-orlando has quit IRC | 07:13 | |
*** markvoelker has joined #openstack-cinder | 07:16 | |
*** salv-orlando has joined #openstack-cinder | 07:17 | |
*** markvoelker has quit IRC | 07:21 | |
*** nihilifer has joined #openstack-cinder | 07:24 | |
*** yamada-h has quit IRC | 07:25 | |
*** markus_z has joined #openstack-cinder | 07:26 | |
*** alonmarx has joined #openstack-cinder | 07:29 | |
*** afazekas has joined #openstack-cinder | 07:31 | |
*** sgotliv has quit IRC | 07:33 | |
*** chlong has quit IRC | 07:39 | |
*** ndipanov has joined #openstack-cinder | 07:39 | |
*** ronis_ has joined #openstack-cinder | 07:46 | |
*** nihilifer has quit IRC | 07:48 | |
*** nihilifer has joined #openstack-cinder | 07:50 | |
*** haomaiwa_ has quit IRC | 07:56 | |
*** nihilifer has quit IRC | 07:59 | |
*** nlevinki has quit IRC | 08:02 | |
*** lpetrut has joined #openstack-cinder | 08:03 | |
*** e0ne has joined #openstack-cinder | 08:05 | |
*** e0ne has quit IRC | 08:11 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Change inventory folder hierarchy https://review.openstack.org/193095 | 08:16 |
*** nihilifer has joined #openstack-cinder | 08:17 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/193977 | 08:20 |
*** belmoreira has quit IRC | 08:22 | |
*** nihilifer has quit IRC | 08:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/193996 | 08:26 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/194019 | 08:27 |
*** aarefiev_home has joined #openstack-cinder | 08:28 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix NetApp cDOT driver use of Glance image locations https://review.openstack.org/193597 | 08:34 |
*** aix has joined #openstack-cinder | 08:36 | |
*** aarefiev_home has quit IRC | 08:39 | |
*** yuriy_n17 has joined #openstack-cinder | 08:43 | |
*** marzif has joined #openstack-cinder | 08:56 | |
*** linkedinyou has joined #openstack-cinder | 09:02 | |
*** markvoelker has joined #openstack-cinder | 09:05 | |
*** e0ne has joined #openstack-cinder | 09:05 | |
*** haypo has joined #openstack-cinder | 09:05 | |
*** belmoreira has joined #openstack-cinder | 09:07 | |
haypo | smcginni1, jgriffith: morning. FYI i added a py34 check job to Cinder, and this check succeded with "tox: add a basic py34 venv running a single test" | 09:07 |
haypo | https://review.openstack.org/#/c/192118/ so my patch works ;) | 09:07 |
*** ociuhandu has joined #openstack-cinder | 09:07 | |
haypo | sorry, i don't remember if you solved your issues with your local setup to get a working "tox -e py34" | 09:08 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy https://review.openstack.org/193828 | 09:08 |
*** markvoelker has quit IRC | 09:09 | |
haypo | (previously my "tox -e py34" patch has dependencies, but they are now merged ;-)) | 09:12 |
*** ociuhandu has quit IRC | 09:19 | |
*** e0ne is now known as e0ne_ | 09:22 | |
*** ociuhandu has joined #openstack-cinder | 09:23 | |
*** e0ne_ is now known as e0ne | 09:27 | |
*** esker has quit IRC | 09:42 | |
e0ne | dulek: hi. could you please review https://review.openstack.org/193828? | 09:47 |
*** marcusvrn has joined #openstack-cinder | 09:53 | |
*** marcusvrn has quit IRC | 09:58 | |
*** aarefiev_home has joined #openstack-cinder | 10:01 | |
*** Longgeek has quit IRC | 10:04 | |
openstackgerrit | Andrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage https://review.openstack.org/193596 | 10:04 |
*** dimsum__ has joined #openstack-cinder | 10:14 | |
*** pschaef has joined #openstack-cinder | 10:15 | |
*** nihilifer has joined #openstack-cinder | 10:15 | |
*** pschaef has quit IRC | 10:15 | |
*** avishay_ has joined #openstack-cinder | 10:23 | |
*** avishay has quit IRC | 10:23 | |
*** ociuhandu has quit IRC | 10:25 | |
*** jamielennox is now known as jamielennox|away | 10:25 | |
*** nlevinki has joined #openstack-cinder | 10:32 | |
*** sgotliv has joined #openstack-cinder | 10:33 | |
*** nihilifer has quit IRC | 10:33 | |
*** Longgeek has joined #openstack-cinder | 10:34 | |
*** sgotliv has quit IRC | 10:35 | |
*** sgotliv has joined #openstack-cinder | 10:35 | |
*** nihilifer has joined #openstack-cinder | 10:37 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 10:42 |
*** aarefiev_home has quit IRC | 10:43 | |
*** deepakcs has quit IRC | 10:43 | |
*** Lee1092 has quit IRC | 10:43 | |
*** rhe00_ has quit IRC | 10:43 | |
*** scottda has quit IRC | 10:43 | |
*** Guest11697 has quit IRC | 10:43 | |
*** zigo has quit IRC | 10:43 | |
*** liewegas has quit IRC | 10:43 | |
*** Trozz_ has quit IRC | 10:43 | |
*** jbernard has quit IRC | 10:43 | |
*** Trozz_ has joined #openstack-cinder | 10:43 | |
*** deepakcs has joined #openstack-cinder | 10:43 | |
*** mgagne has joined #openstack-cinder | 10:43 | |
*** mgagne is now known as Guest81202 | 10:43 | |
*** zigo has joined #openstack-cinder | 10:44 | |
*** sage has joined #openstack-cinder | 10:44 | |
*** rhe00_ has joined #openstack-cinder | 10:44 | |
*** e0ne is now known as e0ne_ | 10:44 | |
*** aarefiev_home has joined #openstack-cinder | 10:45 | |
*** Lee1092 has joined #openstack-cinder | 10:46 | |
*** scottda has joined #openstack-cinder | 10:47 | |
*** dimsum__ is now known as dims | 10:48 | |
*** s-moriya has quit IRC | 10:50 | |
*** dims has quit IRC | 10:53 | |
*** e0ne_ is now known as e0ne | 10:53 | |
*** markvoelker has joined #openstack-cinder | 10:53 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 10:58 |
*** markvoelker has quit IRC | 10:58 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 11:00 |
*** IanGovett has joined #openstack-cinder | 11:04 | |
*** linkedinyou has quit IRC | 11:04 | |
*** dims has joined #openstack-cinder | 11:05 | |
*** nihilifer has quit IRC | 11:05 | |
*** marcusvrn has joined #openstack-cinder | 11:07 | |
*** yamada-h has joined #openstack-cinder | 11:09 | |
*** merooney has joined #openstack-cinder | 11:12 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 11:14 |
*** yamada-h has quit IRC | 11:15 | |
*** daemontool_ has joined #openstack-cinder | 11:16 | |
*** marzif has quit IRC | 11:18 | |
*** Longgeek has quit IRC | 11:18 | |
*** cdelatte has joined #openstack-cinder | 11:23 | |
*** aarefiev_home has quit IRC | 11:33 | |
*** aarefiev_home has joined #openstack-cinder | 11:33 | |
*** IanGovett has quit IRC | 11:34 | |
*** IanGovett has joined #openstack-cinder | 11:35 | |
*** merooney has quit IRC | 11:35 | |
*** daemontool_ has quit IRC | 11:38 | |
*** marzif has joined #openstack-cinder | 11:39 | |
*** merooney has joined #openstack-cinder | 11:41 | |
*** Longgeek has joined #openstack-cinder | 11:46 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Checking the existence of volume https://review.openstack.org/175809 | 11:51 |
*** deepakcs has quit IRC | 11:51 | |
*** BharatK has quit IRC | 11:53 | |
*** markvoelker has joined #openstack-cinder | 11:54 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 11:54 |
*** markvoelker has quit IRC | 11:59 | |
*** Yogi1 has joined #openstack-cinder | 11:59 | |
*** marcusvrn has quit IRC | 12:02 | |
*** marcusvrn has joined #openstack-cinder | 12:02 | |
*** markvoelker has joined #openstack-cinder | 12:03 | |
*** BharatK has joined #openstack-cinder | 12:05 | |
*** bswartz has quit IRC | 12:10 | |
*** Yogi1 has quit IRC | 12:12 | |
*** e0ne is now known as e0ne_ | 12:18 | |
*** e0ne_ is now known as e0ne | 12:19 | |
*** aarefiev_home has quit IRC | 12:20 | |
*** vincent_hou has joined #openstack-cinder | 12:21 | |
vincent_hou | geguileo: Hi. | 12:22 |
geguileo | vincent_hou: Hi | 12:22 |
*** edmondsw has joined #openstack-cinder | 12:22 | |
vincent_hou | geguileo: I saw your comments on my patch. | 12:22 |
vincent_hou | geguileo: I have one question for the test case. In fact I was looking for the proper place to add it, but did not find it. | 12:23 |
geguileo | vincent_hou: I've been reviewing for a few hours and while I remember your name I don't remember which one was your patch... | 12:23 |
vincent_hou | https://review.openstack.org/#/c/180873/ | 12:24 |
vincent_hou | It seems there is no test case for LVM at all. | 12:24 |
geguileo | Mmmmm, it can happen, let me check | 12:24 |
*** chlong has joined #openstack-cinder | 12:25 | |
geguileo | vincent_hou: Did you look in test_volume.py? | 12:25 |
vincent_hou | HA. | 12:26 |
vincent_hou | OK. I found the class. | 12:27 |
geguileo | Great! :) | 12:27 |
vincent_hou | Thank you. | 12:27 |
geguileo | vincent_hou: Thank you for the nice patch :) | 12:27 |
vincent_hou | I was trying to get it. | 12:27 |
vincent_hou | Nice | 12:27 |
vincent_hou | You are welcome. | 12:28 |
*** chlong has quit IRC | 12:28 | |
*** timcl has joined #openstack-cinder | 12:31 | |
*** chlong has joined #openstack-cinder | 12:31 | |
*** tobasco_ has joined #openstack-cinder | 12:32 | |
*** tobasco_ is now known as tobasco | 12:32 | |
tobasco | anybody know if the nexenta driver that was removed due to CI has made any progress in getting back or any info if it will be back for liberty? | 12:33 |
geguileo | tobasco: According to this https://wiki.openstack.org/wiki/ThirdPartySystems | 12:35 |
geguileo | tobasco: It is currently reporting | 12:35 |
*** alexpilotti has joined #openstack-cinder | 12:37 | |
*** akerr has joined #openstack-cinder | 12:37 | |
*** yrabl has quit IRC | 12:38 | |
openstackgerrit | David Medberry proposed openstack/cinder: Removing archaic reference to quickstart. https://review.openstack.org/193791 | 12:38 |
*** yrabl has joined #openstack-cinder | 12:38 | |
*** Longgeek has quit IRC | 12:47 | |
*** vincent_hou has quit IRC | 12:49 | |
*** kjnelson has joined #openstack-cinder | 12:51 | |
dulek | e0ne: Thanks for the fix, it's a shame I haven't noticed it. | 12:52 |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 12:52 |
dulek | e0ne: I've left a comment on the patch - one issue with unit test. | 12:52 |
*** marcusvrn has quit IRC | 12:52 | |
*** Zhongjun has joined #openstack-cinder | 12:56 | |
mtreinish | mriedem: well I guess I could have used 0.0.1, but I dunno | 12:57 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 12:57 |
*** e0ne is now known as e0ne_ | 12:58 | |
*** aarefiev_home has joined #openstack-cinder | 12:58 | |
mtreinish | mriedem: os-testr is still rough in a lot of ways, I was hoping you'd fix it in my absence | 12:58 |
*** chlong has quit IRC | 12:58 | |
*** e0ne_ is now known as e0ne | 12:59 | |
*** rushil has joined #openstack-cinder | 13:00 | |
*** chlong has joined #openstack-cinder | 13:02 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 13:03 |
*** bswartz has joined #openstack-cinder | 13:05 | |
*** chlong has quit IRC | 13:08 | |
*** chlong has joined #openstack-cinder | 13:08 | |
*** marcusvrn has joined #openstack-cinder | 13:09 | |
*** rushiagr is now known as rushiagr_away | 13:10 | |
*** dustins has joined #openstack-cinder | 13:11 | |
*** coolsvap is now known as coolsvap|away | 13:13 | |
*** erlon has joined #openstack-cinder | 13:15 | |
*** jaypipes has joined #openstack-cinder | 13:15 | |
*** Longgeek has joined #openstack-cinder | 13:16 | |
*** chlong has quit IRC | 13:17 | |
*** hodos has joined #openstack-cinder | 13:17 | |
openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 13:17 |
*** Yogi1 has joined #openstack-cinder | 13:18 | |
*** merooney has quit IRC | 13:18 | |
tobasco | geguileo: thanks! | 13:19 |
*** gouthamr has joined #openstack-cinder | 13:20 | |
*** Yogi11 has joined #openstack-cinder | 13:21 | |
*** rushil has quit IRC | 13:22 | |
*** Yogi1 has quit IRC | 13:23 | |
*** mordred has joined #openstack-cinder | 13:28 | |
mordred | morning lovely people ... I'm wondering if anyone has experienced cinderclient 1.2 hanging/failing when using it against hp or rackspace public clouds? | 13:28 |
mordred | my sypmtoms are taht "cinder list" and "cinder type-list" hang until they timeout | 13:29 |
*** merooney has joined #openstack-cinder | 13:29 | |
*** chlong has joined #openstack-cinder | 13:30 | |
mordred | http://paste.openstack.org/show/313611/ <-- this is the end of the debug tracing from running against rax public | 13:30 |
mordred | you can see that cinderclient is nicely trying to do version autonegotiation, which is awesome | 13:31 |
mordred | except that the urls returned are http urls | 13:31 |
mordred | which are not valid | 13:31 |
mordred | in cinderclient 1.1 the version stuff was just handled by the env var and the urls from the keystone catalog were used - those are https and those work | 13:31 |
mordred | (so a forced downgrade to 1.1.1 works on both hp and rackspace) | 13:32 |
*** primechuck has joined #openstack-cinder | 13:32 | |
mordred | also - explicitly specifying a version does not help | 13:32 |
*** timcl has quit IRC | 13:35 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy https://review.openstack.org/193828 | 13:35 |
mordred | the issue with hp is even weirder - in that it's hanging against trying to talk to the compute endpoint of a region I didn't ask for | 13:36 |
mordred | DuncanT: ^^ if you happen to be around, might be interesting to our corporate overlords | 13:36 |
*** jungleboyj has quit IRC | 13:37 | |
DuncanT | mordred: Looking now | 13:37 |
DuncanT | mordred: All our stuff is still using the 1.1.1 client AFAICT, so we've never noticed | 13:39 |
mordred | ossum | 13:39 |
mordred | well, infra's production scripts are using 1.1.1 too - so also never notied there :) | 13:39 |
*** thangp has joined #openstack-cinder | 13:39 | |
mordred | DuncanT: here's the trace from hp public http://paste.openstack.org/show/313664/ | 13:39 |
mordred | DuncanT: why it wants to talk to region-a's compute is mindboggline to me | 13:39 |
*** timcl has joined #openstack-cinder | 13:40 | |
*** zigo has quit IRC | 13:47 | |
*** merooney has quit IRC | 13:48 | |
*** rushiagr_away is now known as rushiagr | 13:49 | |
DuncanT | morded: Might be a messed up catalogue, investigating (and having my wifi crash too, just for added entertainment) | 13:50 |
smcginni1 | DuncanT: Jay's not here, or I would make some comment on that. | 13:50 |
DuncanT | smcginni1: lol | 13:50 |
*** smcginni1 is now known as smcginnis | 13:51 | |
DuncanT | smcginnis: Remind me not to hold a mid-cycle at my house... | 13:51 |
smcginnis | DuncanT: +1 ;) | 13:51 |
*** r-daneel has joined #openstack-cinder | 13:52 | |
*** breitz has joined #openstack-cinder | 13:56 | |
*** e0ne is now known as e0ne_ | 13:57 | |
*** Tross has quit IRC | 13:57 | |
*** e0ne_ is now known as e0ne | 13:57 | |
*** ronis_ has quit IRC | 14:01 | |
*** merooney has joined #openstack-cinder | 14:02 | |
DuncanT | mordred: I have found at least one problem, investigating how to fix it | 14:03 |
*** zigo has joined #openstack-cinder | 14:05 | |
*** lpetrut has quit IRC | 14:05 | |
*** nlevinki has quit IRC | 14:06 | |
mordred | DuncanT: neat- finding problems is exciting | 14:08 |
*** eharney has joined #openstack-cinder | 14:08 | |
*** BharatK has quit IRC | 14:09 | |
DuncanT | mordred: At least I'm not finding out at 4am via a pager :-) | 14:09 |
mordred | DuncanT: see, this is how I show I care | 14:10 |
*** jbernard has joined #openstack-cinder | 14:10 | |
*** rmesta has joined #openstack-cinder | 14:11 | |
*** mriedem1 is now known as mriedem_afk | 14:12 | |
*** beekneemech is now known as bnemec | 14:13 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 14:16 |
*** jecarey has joined #openstack-cinder | 14:17 | |
*** rushil has joined #openstack-cinder | 14:19 | |
*** timcl1 has joined #openstack-cinder | 14:19 | |
*** timcl has quit IRC | 14:21 | |
*** timcl1 has quit IRC | 14:22 | |
*** nlevinki has joined #openstack-cinder | 14:22 | |
*** aarefiev_home has quit IRC | 14:23 | |
*** ameade_ is now known as ameade | 14:23 | |
*** paboldin has joined #openstack-cinder | 14:28 | |
paboldin | jgriffith: hello John, here? | 14:28 |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Change inventory folder hierarchy https://review.openstack.org/193095 | 14:28 |
paboldin | @all, please review https://review.openstack.org/182871/ | 14:31 |
DuncanT | paboldin: Why are they called iscsi_target_flags if they only apply to tgt? | 14:33 |
*** Guest78977 has joined #openstack-cinder | 14:33 | |
paboldin | DuncanT: there were ongoing discussion during the review. People decided that they could be applied to some other things. This is also stated in the referenced bug discussion. | 14:33 |
DuncanT | paboldin: Ok, I'll read some more | 14:34 |
*** ociuhandu has joined #openstack-cinder | 14:35 | |
*** mfisch` is now known as mfisch | 14:36 | |
*** mfisch is now known as Guest82290 | 14:36 | |
*** jungleboyj has joined #openstack-cinder | 14:37 | |
dulek | OpenStack Mita? Really? | 14:39 |
dulek | I was hoping for Musashi. | 14:39 |
smcginnis | dulek: We had an internal project named Musashi. Otherwise I would have like that one. | 14:40 |
*** rushil has quit IRC | 14:41 | |
*** asselin has joined #openstack-cinder | 14:42 | |
dulek | smcginnis: "OpenStack Musashi. Bleeding edge of cloud technology." ;) | 14:42 |
smcginnis | :) | 14:43 |
*** Longgeek has quit IRC | 14:43 | |
*** marcusvrn_ has joined #openstack-cinder | 14:43 | |
DuncanT | paboldin: That review seems to be stuck in circles... I tend to agree with John and the others who said it should be a tgt-specific option, and constrained as an enum | 14:45 |
paboldin | DuncanT: but most likely there will be the same options for the LIOadm | 14:46 |
paboldin | DuncanT: https://bugs.launchpad.net/cinder/+bug/1441935/comments/6 < like here | 14:47 |
openstack | Launchpad bug 1441935 in Cinder "Cinder LVM-over-iSCSI should use O_DIRECT by default" [Wishlist,In progress] - Assigned to Pavel Boldin (pboldin) | 14:47 |
*** lpetrut has joined #openstack-cinder | 14:47 | |
*** ilyag has joined #openstack-cinder | 14:48 | |
DuncanT | paboldin: Are bsoflags the only tgt flags? | 14:48 |
DuncanT | paboldin: Or are there other flags we might want to set in future? | 14:48 |
paboldin | DuncanT: Yes. But the target_flags could be other in future .`target_flags' is for target-specific flags. In case of `tgtadm' it is the `bsoflags' | 14:49 |
DuncanT | paboldin: But are bsoflags are the only tgt-specific flags we might want to set? If yes, I'll raise a bug that the flag needs adding for other targets and approve this patch | 14:50 |
DuncanT | paboldin: But if there are other tgt-specific flag fields, I'm less happy | 14:50 |
DuncanT | paboldin: I don't know much about tgt at all; I'm hoping you can tell me | 14:50 |
*** ilyag has quit IRC | 14:50 | |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 14:51 |
*** rushil has joined #openstack-cinder | 14:52 | |
paboldin | DuncanT: there are bsopts as well, but we are not going to use this because it only used (according to docs) for the Ceph backing device (and we can use Ceph directly at the moment) | 14:52 |
*** ilyag has joined #openstack-cinder | 14:53 | |
*** Guest78977 has quit IRC | 14:53 | |
DuncanT | paboldin: Ok, I'm happy other than the missing bug to implement this config option for other initiators, so I'll approve and file that bug | 14:54 |
paboldin | DuncanT: wait a second, I want to check if LIO is actually requires something alike | 14:54 |
DuncanT | paboldin: Thanks for the clarification | 14:54 |
DuncanT | ok | 14:54 |
DuncanT | waiting | 14:54 |
*** marzif has quit IRC | 14:55 | |
*** rmesta has left #openstack-cinder | 14:55 | |
*** timcl has joined #openstack-cinder | 14:57 | |
*** julim has joined #openstack-cinder | 14:58 | |
*** patrickeast has joined #openstack-cinder | 15:00 | |
*** timcl1 has joined #openstack-cinder | 15:00 | |
*** MentalRay has joined #openstack-cinder | 15:01 | |
*** timcl has quit IRC | 15:02 | |
mordred | DuncanT: fwiw, I pushed up a patch to pin shade to 1.1.1 of cinderclient for now, so no pressing/urgent thing from infra/ansible front ... otoh, please let me know if there's anything I can do to help | 15:03 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run https://review.openstack.org/194180 | 15:03 |
*** mtanino has joined #openstack-cinder | 15:05 | |
*** nlevinki has quit IRC | 15:06 | |
*** nkrinner has quit IRC | 15:07 | |
DuncanT | mordred: Thanks. I'll keep working on it anyway | 15:09 |
*** Longgeek has joined #openstack-cinder | 15:10 | |
paboldin | DuncanT: well, at the moment there is no necessity for the target-specific flags for e.g. LIO. This is because RTS tool only supports BlockStorage that has no configuratiration options like that. However, other targets (like `rtslib_fb.UserBackedStorageObject') have some additional configuration options. | 15:10 |
*** kjnelson has quit IRC | 15:11 | |
DuncanT | paboldin: Ok, it is probably ok to leave that until people find a need then | 15:11 |
DuncanT | paboldin: Thanks for looking / clarifying | 15:12 |
paboldin | DuncanT: thanks for reviewing | 15:12 |
*** ronis_ has joined #openstack-cinder | 15:13 | |
*** kjnelson has joined #openstack-cinder | 15:14 | |
DuncanT | paboldin: Hopefully it will merge | 15:14 |
paboldin | DuncanT: me too. had some not so pleasent times doing it :-/ | 15:15 |
*** dannywilson has joined #openstack-cinder | 15:15 | |
mkoderer | thingee: ping | 15:16 |
thingee | mkoderer: hey | 15:16 |
mkoderer | thingee: hey :) | 15:16 |
*** aix has quit IRC | 15:16 | |
mkoderer | thingee: I am again a bit confused about this ABC ML thread ;) | 15:17 |
*** daneyon has joined #openstack-cinder | 15:17 | |
*** dannywil_ has joined #openstack-cinder | 15:17 | |
*** dannywilson has quit IRC | 15:18 | |
mkoderer | thingee: I think the point of cknight is a valid drawback of ABC.. but I really didn't get the point what is actually the problem | 15:19 |
*** nlevinki has joined #openstack-cinder | 15:19 | |
*** agentle has joined #openstack-cinder | 15:19 | |
mkoderer | jgriffith: ^^ | 15:19 |
thingee | mkoderer: I think the point jgriffith is making is he sees us just have a single abc class. not a bunch of different classes to represent features. | 15:20 |
*** timcl1 has quit IRC | 15:20 | |
thingee | mkoderer: people can use the reference implementation if they want figure out what methods need to implement for features that are not in this single class. | 15:20 |
*** Longgeek has quit IRC | 15:20 | |
mkoderer | thingee: mh, so making all functions mandatory? | 15:21 |
thingee | at least the ones we can right now | 15:21 |
*** MentalRay has quit IRC | 15:21 | |
thingee | not everyone is going to have consistency groups for example | 15:21 |
*** dguryanov has joined #openstack-cinder | 15:22 | |
mkoderer | thingee: ok, this was actually one of your comments in my cinder-spec for this release | 15:22 |
mkoderer | thingee: following the idea to move more and more function to BaseVD sounds good | 15:22 |
mkoderer | but we need to port all the drivers first ;) | 15:22 |
*** afazekas has quit IRC | 15:23 | |
*** timcl has joined #openstack-cinder | 15:25 | |
*** MentalRay has joined #openstack-cinder | 15:26 | |
*** kjnelson1 has joined #openstack-cinder | 15:26 | |
*** daneyon_ has joined #openstack-cinder | 15:26 | |
*** anshul has quit IRC | 15:28 | |
*** kjnelson has quit IRC | 15:28 | |
*** haypo has left #openstack-cinder | 15:28 | |
*** belmoreira has quit IRC | 15:28 | |
*** daneyon has quit IRC | 15:29 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
*** hodos has quit IRC | 15:31 | |
e0ne | thingee: hi. i don't know did you receives my message last weekend or not. i marked one more bug as high priority and targeted it to L-1 | 15:32 |
thingee | e0ne: I'll need to try again, but I was not able to reproduce unit tests not passing | 15:33 |
e0ne | thingee: let's not mix my patches | 15:34 |
e0ne | thingee: i'm talking about https://bugs.launchpad.net/cinder/+bug/1467167 | 15:34 |
openstack | Launchpad bug 1467167 in Cinder "Can't delete backup: User does not have admin privileges" [High,In progress] - Assigned to Ivan Kolodyazhny (e0ne) | 15:34 |
thingee | e0ne: ok | 15:34 |
*** sseverson has joined #openstack-cinder | 15:35 | |
e0ne | thingee: imo, failed unit tests on a local is not a high priory issue | 15:35 |
thingee | e0ne: I agree | 15:35 |
*** Guest81202 is now known as mgagne | 15:36 | |
*** mgagne has joined #openstack-cinder | 15:36 | |
e0ne | thingee: according to https://review.openstack.org/#/c/193861/. './run_tests.sh cinder.unit.objects.test_backup' fails on a master | 15:36 |
*** timcl has left #openstack-cinder | 15:36 | |
*** agentle has quit IRC | 15:39 | |
thangp | e0ne: I put a comment in https://review.openstack.org/#/c/193861/ about why it failed | 15:39 |
*** agentle has joined #openstack-cinder | 15:40 | |
e0ne | thangp: thanks. you're right. i'll post a reply righ now | 15:41 |
thangp | e0ne: either solution will work | 15:41 |
thangp | e0ne: it's just how far do we want to go | 15:41 |
*** rushil has quit IRC | 15:41 | |
dulek | e0ne, thangp: From my perspective thangp's solution seems better. | 15:41 |
*** idevops has joined #openstack-cinder | 15:41 | |
*** idevops has quit IRC | 15:42 | |
openstackgerrit | John Haan proposed openstack/python-cinderclient: Adding search options to backup list https://review.openstack.org/78233 | 15:42 |
openstackgerrit | John Haan proposed openstack/python-cinderclient: Adding search options to backup list https://review.openstack.org/78233 | 15:45 |
*** harlowja_at_home has joined #openstack-cinder | 15:45 | |
*** agarciam has quit IRC | 15:48 | |
dulek | thangp: Hi, I started to wonder a little about relationships between SQLAlchemy objects and VersionedObjects. | 15:48 |
*** aarefiev_home has joined #openstack-cinder | 15:48 | |
thingee | e0ne: lgtm | 15:48 |
*** Longgeek has joined #openstack-cinder | 15:48 | |
thingee | e0ne: thanks for taking care of that | 15:48 |
thangp | dulek: how so? | 15:49 |
e0ne | thingee: thanks | 15:49 |
thingee | thangp: https://review.openstack.org/#/c/193828/9 | 15:49 |
dulek | thangp: For example - volume have transfer relationship. It's defined in Transfer SQLAlchemy object with backref. | 15:49 |
dulek | thingee: So technically such field is available in volume object. | 15:50 |
e0ne | thangp, dulek: i unswered you in gerrit | 15:50 |
dulek | I've meant thangp. ;) ^ | 15:51 |
thangp | e0ne: https://review.openstack.org/#/c/193828/9 - is context.elevated() needed for QUOTAS.commit()? | 15:52 |
*** merooney has quit IRC | 15:52 | |
*** rushil has joined #openstack-cinder | 15:55 | |
*** hodos has joined #openstack-cinder | 15:56 | |
*** afazekas has joined #openstack-cinder | 15:58 | |
*** nlevinki has quit IRC | 15:59 | |
*** salv-orl_ has joined #openstack-cinder | 16:00 | |
e0ne | thangp: no | 16:01 |
thingee | hodos: where are the unit tests for nexenta ns4? https://review.openstack.org/#/c/188877/ | 16:01 |
*** leeantho has joined #openstack-cinder | 16:01 | |
thangp | e0ne: ok, thx | 16:01 |
thingee | ilyag: ^ | 16:01 |
hodos | thingee: wow, i must've forgot to put back in | 16:01 |
hodos | thingee: let me update the review | 16:02 |
thingee | kk | 16:02 |
*** salv-orlando has quit IRC | 16:03 | |
*** hemna has joined #openstack-cinder | 16:04 | |
*** laughterwym has quit IRC | 16:06 | |
*** afazekas has quit IRC | 16:06 | |
thingee | e0ne: for some reason devstack isn't able to start up with https://review.openstack.org/#/c/145703/ | 16:07 |
thingee | e0ne: something with pip | 16:07 |
e0ne | thingee: :( | 16:07 |
*** hemna has quit IRC | 16:07 | |
e0ne | thingee: i'm investigating fix for rally to support api v2 | 16:08 |
*** agentle has quit IRC | 16:08 | |
openstackgerrit | James Page proposed openstack/os-brick: Misc Python 3 compatibility fixes https://review.openstack.org/194216 | 16:09 |
*** Guest82290 is now known as mfisch | 16:10 | |
*** mfisch has quit IRC | 16:10 | |
*** mfisch has joined #openstack-cinder | 16:10 | |
*** dtynan1 has joined #openstack-cinder | 16:11 | |
thingee | e0ne: https://bugs.launchpad.net/cinder/+bug/1467589 | 16:11 |
openstack | Launchpad bug 1467589 in Cinder "Remove Cinder V1 supprt" [Undecided,In progress] - Assigned to Mike Perez (thingee) | 16:11 |
thingee | e0ne: lets track projects that need to be worked on there | 16:11 |
*** e0ne has quit IRC | 16:11 | |
*** pradipta has quit IRC | 16:12 | |
*** e0ne has joined #openstack-cinder | 16:13 | |
*** hemna has joined #openstack-cinder | 16:14 | |
*** merooney has joined #openstack-cinder | 16:14 | |
*** afazekas has joined #openstack-cinder | 16:14 | |
*** bswartz has quit IRC | 16:15 | |
*** patrickeast has quit IRC | 16:16 | |
*** asselin has quit IRC | 16:18 | |
*** david-ly_ is now known as david-lyle | 16:18 | |
hodos | thingee: added unit tests | 16:19 |
*** Apoorva has joined #openstack-cinder | 16:19 | |
*** sseverson has quit IRC | 16:20 | |
*** avishay_ has quit IRC | 16:21 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Remove unnecessary volume export after generic volume migration https://review.openstack.org/194223 | 16:22 |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 16:22 |
*** agentle has joined #openstack-cinder | 16:23 | |
*** Yogi11 has quit IRC | 16:27 | |
*** jdurgin1 has quit IRC | 16:28 | |
*** harlowja_at_home has quit IRC | 16:34 | |
*** markus_z has quit IRC | 16:39 | |
*** xyang1 has joined #openstack-cinder | 16:42 | |
*** asselin has joined #openstack-cinder | 16:44 | |
*** rwsu has joined #openstack-cinder | 16:45 | |
*** daneyon has joined #openstack-cinder | 16:46 | |
*** teju has joined #openstack-cinder | 16:46 | |
*** daneyon_ has quit IRC | 16:47 | |
*** timcl has joined #openstack-cinder | 16:47 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles https://review.openstack.org/193334 | 16:48 |
*** agentle has quit IRC | 16:49 | |
*** afazekas has quit IRC | 16:49 | |
*** agentle has joined #openstack-cinder | 16:50 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Add exception catch in report_state for DBError https://review.openstack.org/194232 | 16:51 |
*** MRay has joined #openstack-cinder | 16:51 | |
*** MentalRay has quit IRC | 16:54 | |
*** afazekas has joined #openstack-cinder | 16:54 | |
*** daneyon_ has joined #openstack-cinder | 16:58 | |
*** daneyon has quit IRC | 16:58 | |
*** daneyon has joined #openstack-cinder | 16:59 | |
*** vilobhmm has joined #openstack-cinder | 16:59 | |
*** yamada-h has joined #openstack-cinder | 17:00 | |
*** afazekas has quit IRC | 17:00 | |
*** teju has left #openstack-cinder | 17:00 | |
*** daneyon_ has quit IRC | 17:03 | |
*** esker has joined #openstack-cinder | 17:03 | |
*** patrickeast has joined #openstack-cinder | 17:05 | |
*** timcl has left #openstack-cinder | 17:05 | |
*** e0ne has quit IRC | 17:05 | |
*** ronis_ has quit IRC | 17:07 | |
*** dims has quit IRC | 17:09 | |
*** dims has joined #openstack-cinder | 17:09 | |
*** hodos has quit IRC | 17:10 | |
*** timcl has joined #openstack-cinder | 17:10 | |
*** kjnelson1 has quit IRC | 17:11 | |
*** hemna has quit IRC | 17:14 | |
*** hemna has joined #openstack-cinder | 17:15 | |
*** daneyon has quit IRC | 17:18 | |
*** daneyon has joined #openstack-cinder | 17:19 | |
*** rwsu has quit IRC | 17:19 | |
*** Longgeek has quit IRC | 17:20 | |
*** daneyon has quit IRC | 17:20 | |
*** rushil has quit IRC | 17:21 | |
*** daneyon has joined #openstack-cinder | 17:21 | |
*** daneyon has quit IRC | 17:23 | |
*** thangp is now known as thangp_afk | 17:24 | |
*** marcusvrn_ has quit IRC | 17:24 | |
*** rushil has joined #openstack-cinder | 17:25 | |
rlucio | question for core guys: i have a bunch of internal bug fixes that i would like to push up to vmem drivers in trunk, is it preferred to submit them one at time (eg several small changes) or merged all at once? | 17:26 |
rlucio | im guessing the former but just want to double check | 17:26 |
*** aarefiev_home has quit IRC | 17:27 | |
patrickeast | rlucio: fwiw i’ve noticed small changes move fast and seem to get reviewed earlier | 17:28 |
*** harlowja has joined #openstack-cinder | 17:28 | |
*** annashen has joined #openstack-cinder | 17:29 | |
*** dustins_ has joined #openstack-cinder | 17:30 | |
*** daneyon has joined #openstack-cinder | 17:30 | |
rlucio | patrckeast: yea, that makes sense | 17:30 |
thingee | guitarzan: I heard from mordred and DuncanT that cinder client is broken at rax as well? | 17:32 |
mordred | thingee: yes | 17:33 |
DuncanT | rlucio: Definitely a bunch of small changes please | 17:33 |
thingee | guitarzan, mordred, DuncanT: I'm aware of an issue with ha proxy and ssl termination with versions below Kilo. | 17:33 |
*** dustins has quit IRC | 17:33 | |
mordred | thingee: so- the rax issus, amongst other things, is that the version discovery is returning http endpoints | 17:33 |
mordred | thingee: which are not valid | 17:34 |
mordred | thingee: so, it seems that there is at least one deployment related issue | 17:34 |
mordred | thingee: it can't be worked around by specifying the api version explicitly, which is I think a bug in cinderclient that DuncanT was looking in to | 17:34 |
thingee | mordred: so fix for that, is this option, https://github.com/openstack/cinder/blob/master/cinder/api/views/versions.py#L23 | 17:34 |
thingee | mordred: but that's kilo =/ | 17:35 |
mordred | yah | 17:35 |
thingee | mordred: I think the discovery release is premature without this config option being out longer. | 17:35 |
mordred | so - 1.1.1 works fine - so I think if cinderclient just skips discovery attempts if you pass it an explicit version is a short-term workaround | 17:35 |
rlucio | duncant: ok, no prob | 17:35 |
mordred | I LOVE that it's doing discovery :) | 17:35 |
thingee | mordred: oh me too | 17:36 |
DuncanT | What I can't figure out is why it is trying to connect to a compute endpoint | 17:36 |
*** rwsu has joined #openstack-cinder | 17:36 | |
mordred | DuncanT: it only does that for me against HP | 17:36 |
mordred | DuncanT: for rax, it does not seem to do that | 17:36 |
DuncanT | mordred: Right. I can't figure out where it got that URL from | 17:37 |
thingee | mordred: yeah DuncanT showed me the debug logs from the client's perspective. I have no clue how compute gets involved there. | 17:37 |
mordred | DuncanT: :) | 17:37 |
* mordred likes to bring in the fun issues | 17:37 | |
thingee | :) | 17:37 |
*** paboldin has left #openstack-cinder | 17:37 | |
* thingee updates release notes about public_endpoint config opt | 17:38 | |
*** Zhongjun has quit IRC | 17:38 | |
*** agentle has quit IRC | 17:41 | |
*** agentle has joined #openstack-cinder | 17:41 | |
thingee | mordred, DuncanT: so the client could start checking headers from the proxy to default to previous behavior | 17:42 |
*** yamada-h has quit IRC | 17:43 | |
*** rushil has quit IRC | 17:43 | |
DuncanT | thingee: I've got issues before that :-( | 17:43 |
thingee | DuncanT: yeah I'm not sure what happens in your case, but at least help a couple of other deployers | 17:44 |
*** reed has quit IRC | 17:44 | |
*** reed has joined #openstack-cinder | 17:44 | |
DuncanT | Still hacking on our case | 17:44 |
thingee | morganfainberg: a while back we spoke to someone from blue box that mentioned the ha proxy issue with version discovery in Cinder client. Was the header you were thinking of a standard or is ha proxy specific? | 17:45 |
*** timcl has quit IRC | 17:45 | |
morganfainberg | Refresh my memory on that header? | 17:46 |
*** timcl has joined #openstack-cinder | 17:46 | |
vilobhmm | DuncanT, smcginnis : https://review.openstack.org/#/c/185704/ needs review before it runs into merge conflict again..thanks.. | 17:46 |
vilobhmm | whenever you have time | 17:46 |
morganfainberg | I remember saying it but my brain is context switched away from the ha proxy convo. And version discovery | 17:47 |
thingee | morganfainberg: you never mentioned the header :) ... but you mentioned there was a magical header I could check for | 17:47 |
thingee | morganfainberg: ha proxy with ssl termination | 17:47 |
morganfainberg | Hmm. Brain is not connecting. | 17:47 |
morganfainberg | I think there is a standard proxy header that can be referenced. | 17:48 |
thingee | public end point is https, but cinder is giving back http. | 17:48 |
morganfainberg | Yeah. Let me see. | 17:48 |
*** Longgeek has joined #openstack-cinder | 17:49 | |
*** agentle has quit IRC | 17:50 | |
*** timcl has quit IRC | 17:51 | |
morganfainberg | thingee: I think all the info you need should be in x-forwarded-for | 17:51 |
thingee | thanks | 17:51 |
morganfainberg | thingee: *think*. And that should be provided for any forwarding proxy. | 17:51 |
*** timcl has joined #openstack-cinder | 17:53 | |
morganfainberg | thingee: you might need to explicitly setup some haproxy headers though. Hmm. It's not clear to me atm. | 17:54 |
DuncanT | Ok, so it looks like maybe our bug is specifically triggered when you have a multi-region setup | 17:56 |
DuncanT | Some call somewhere is ignoring OS_REGION_NAME | 17:56 |
DuncanT | vilobhmm: Done | 17:57 |
thingee | DuncanT: even when explicit set by client? | 17:57 |
thingee | I guess nothing is setting the default anyways | 17:57 |
DuncanT | thingee: When set in the client env | 17:57 |
vilobhmm | DuncanT, smcginnis : thanks :) | 17:57 |
*** kjnelson has joined #openstack-cinder | 17:58 | |
ericksonfgds | nice job, vilobhmm :) | 17:59 |
vilobhmm | DuncanT : its not merged yet! you want me to ask thingee or any of our cinder core's to merge it….just don't want to end up in the rebase cycle :P | 18:01 |
*** anshul has joined #openstack-cinder | 18:01 | |
vilobhmm | ericksonfgds : :) thnx | 18:01 |
thingee | DuncanT: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/shell.py#L702 | 18:02 |
thingee | DuncanT: can you hack in a region_name kwarg there and let me know | 18:02 |
DuncanT | One sec | 18:03 |
*** patrickeast_ has joined #openstack-cinder | 18:03 | |
smcginnis | vilobhmm: It's merging. It takes a little while. :) | 18:03 |
*** patrickeast has quit IRC | 18:03 | |
*** patrickeast_ is now known as patrickeast | 18:03 | |
*** dustins_ has quit IRC | 18:03 | |
*** dustins has joined #openstack-cinder | 18:04 | |
DuncanT | thingee: That's the one | 18:04 |
thingee | DuncanT: you're working again with that? | 18:05 |
vilobhmm | smcginnis : oh okay :) thanks ! | 18:05 |
DuncanT | thingee: It is now doing version discovery, and trying to use the internal URL | 18:05 |
DuncanT | thingee: That we know about and understand | 18:06 |
thingee | hmm internal one | 18:06 |
*** thangp_afk is now known as thangp | 18:06 | |
DuncanT | Yup, I at least understand where it is getting that from though, and I can see where we give out the wrong answer | 18:07 |
*** salv-orl_ has quit IRC | 18:07 | |
*** MRay has quit IRC | 18:09 | |
thingee | DuncanT: weird, so os_endpoint_type should be defaulting to publicURL | 18:09 |
thingee | rather endpoint_type | 18:10 |
DuncanT | thingee: Sorry, I'm not explaining myself correctly. Our cinder version discovery URL is erroniously handing out a behind-the-proxy URL, and with your region name fix set that is where the client is trying to connect to | 18:11 |
DuncanT | thingee: So we're in the same state as bluebox et al | 18:11 |
*** e0ne has joined #openstack-cinder | 18:11 | |
DuncanT | thingee: We know how to fix this, but need to deploy new code to do so | 18:12 |
*** bswartz has joined #openstack-cinder | 18:12 | |
thingee | DuncanT: x-forwarded-for in the headers? | 18:12 |
thingee | that comes back from the proxy? | 18:12 |
*** afazekas has joined #openstack-cinder | 18:12 | |
DuncanT | thingee: I can't see the headers on the server side... or do you mean client side? | 18:13 |
*** angela-s has joined #openstack-cinder | 18:14 | |
thingee | DuncanT: what the client would be getting in the response | 18:14 |
DuncanT | Nothing | 18:14 |
DuncanT | Just the normal HTTP/1.1 200 OK | 18:14 |
DuncanT | Content-Type: application/json | 18:14 |
DuncanT | Content-Length: 360 | 18:14 |
DuncanT | Date: Mon, 22 Jun 2015 18:14:11 GMT | 18:14 |
DuncanT | Connection: keep-alive | 18:14 |
thingee | DuncanT: yeah my bad I meant it the other way...well if you can't get access to the logs on server side | 18:17 |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor -os-region-name https://review.openstack.org/194280 | 18:18 |
DuncanT | thingee: I will figure out how to get it. We don't log it normally | 18:19 |
hemna | scottda, --os-region-name | 18:19 |
*** aarefiev_home has joined #openstack-cinder | 18:19 | |
DuncanT | hemna: Or the env variable. It's only a typo in the bug, the problem is real (and thingee gave us the fix) | 18:20 |
thingee | DuncanT, scottda: there are multiple places get_endpoint is called in that block | 18:20 |
scottda | yeah, my bad. I'll put up a new patch | 18:20 |
hemna | scottda, also mention the env. var as well | 18:21 |
hemna | fwiw | 18:21 |
DuncanT | thingee: Yup, we'll add it to them all, thanks | 18:21 |
DuncanT | Also figure out how to fake regions in devstack | 18:22 |
*** merooney has quit IRC | 18:23 | |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor -os-region-name or ENV[OS_REGION_NAME] https://review.openstack.org/194280 | 18:23 |
*** e0ne is now known as e0ne_ | 18:23 | |
*** e0ne_ is now known as e0ne | 18:24 | |
*** timcl has quit IRC | 18:24 | |
hemna | -- | 18:24 |
scottda | ahhhg | 18:25 |
hemna | :P | 18:25 |
*** jasondot_ has joined #openstack-cinder | 18:25 | |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor --os-region-name or ENV[OS_REGION_NAME] https://review.openstack.org/194280 | 18:26 |
*** belmoreira has joined #openstack-cinder | 18:26 | |
*** merooney has joined #openstack-cinder | 18:28 | |
*** xyang1 has quit IRC | 18:30 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix Cinder Objects unit tests https://review.openstack.org/193861 | 18:33 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy https://review.openstack.org/193828 | 18:33 |
*** avishay_ has joined #openstack-cinder | 18:35 | |
*** merooney has quit IRC | 18:36 | |
*** xyang1 has joined #openstack-cinder | 18:37 | |
*** esker has quit IRC | 18:38 | |
*** merooney has joined #openstack-cinder | 18:38 | |
*** timcl has joined #openstack-cinder | 18:39 | |
*** ilyag has quit IRC | 18:39 | |
*** timcl1 has joined #openstack-cinder | 18:41 | |
*** afazekas has quit IRC | 18:42 | |
*** timcl has quit IRC | 18:44 | |
*** amoturi has joined #openstack-cinder | 18:48 | |
*** afazekas has joined #openstack-cinder | 18:49 | |
thingee | DuncanT, scottda: just abou have this narrowed down | 18:50 |
*** jasondot_ has quit IRC | 18:51 | |
*** Rockyg has joined #openstack-cinder | 18:51 | |
*** agentle has joined #openstack-cinder | 18:58 | |
*** salv-orlando has joined #openstack-cinder | 18:59 | |
*** agentle_ has joined #openstack-cinder | 19:02 | |
*** afazekas has quit IRC | 19:03 | |
*** afazekas has joined #openstack-cinder | 19:04 | |
*** e0ne has quit IRC | 19:04 | |
*** salv-orlando has quit IRC | 19:04 | |
*** e0ne has joined #openstack-cinder | 19:05 | |
*** agentle has quit IRC | 19:05 | |
*** MRay has joined #openstack-cinder | 19:06 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to oslo.service https://review.openstack.org/193951 | 19:07 |
openstackgerrit | Mikhail Khodos proposed openstack/cinder: Re-add Nexenta driver. https://review.openstack.org/188877 | 19:07 |
*** rushiagr is now known as rushiagr_away | 19:08 | |
*** Longgeek has quit IRC | 19:09 | |
*** esker has joined #openstack-cinder | 19:11 | |
thingee | DuncanT, scottda so from the client's perspective, not really a lot I can go off of to handle this case. | 19:12 |
DuncanT | @thingee Agreed | 19:13 |
thingee | DuncanT, scottda not sure if others have ideas, but it might be a bit premature to have this in the client release if the config option to set the public endpoint from cinder's perspective. | 19:13 |
DuncanT | @thingee We need to fix our version discovery returns. The region bit was the really puzzling bit | 19:13 |
thingee | the only other option is to setup middleware, which isn't any better. | 19:13 |
*** IlyaG has joined #openstack-cinder | 19:13 | |
*** merooney has quit IRC | 19:14 | |
thingee | jamielennox|away, morganfainberg ^ | 19:14 |
*** rwsu has quit IRC | 19:15 | |
*** MentalRay has joined #openstack-cinder | 19:15 | |
*** MRay has quit IRC | 19:16 | |
thingee | luckily I don't think documentation has been updated to have people setup cinder services without version numbers set...so might not even be known to people yet | 19:16 |
*** agentle_ has quit IRC | 19:17 | |
openstackgerrit | Jon Bernard proposed openstack/os-brick: Add RBD connector https://review.openstack.org/186172 | 19:17 |
*** agentle has joined #openstack-cinder | 19:18 | |
*** merooney has joined #openstack-cinder | 19:18 | |
*** afazekas has quit IRC | 19:19 | |
thingee | DuncanT: could also have default behavior if connection failures =/ | 19:20 |
thingee | DuncanT: just to make that block even more complex | 19:20 |
thingee | I'll see if jamielennox|away has any ideas...but I think we might have to just revert. | 19:21 |
*** jasondot_ has joined #openstack-cinder | 19:24 | |
*** dtynan1 has quit IRC | 19:25 | |
*** dtynan has joined #openstack-cinder | 19:26 | |
*** rushil has joined #openstack-cinder | 19:26 | |
*** agentle has quit IRC | 19:27 | |
*** annashen has quit IRC | 19:29 | |
*** ociuhandu has quit IRC | 19:29 | |
vilobhmm | DuncanT, jgriffith : why do we have -1 values set as default for resources for whom we don't have a flag set…does -1 imply unlimited ? Is there a particular usage of it ? | 19:31 |
vilobhmm | :%s/values/value/g | 19:31 |
vilobhmm | %s/values/value/g | 19:31 |
vilobhmm | https://github.com/openstack/cinder/blob/master/cinder/quota.py#L495 | 19:31 |
*** annashen has joined #openstack-cinder | 19:31 | |
*** annashen has quit IRC | 19:32 | |
DuncanT | thingee: Timeout is so slow I don't think that would make things much better | 19:32 |
*** annashen has joined #openstack-cinder | 19:32 | |
thingee | DuncanT: I was kidding on that solution. If I had to resort to that I would just revert | 19:32 |
DuncanT | vilobhmm: I think -1 is unlimited, yeah | 19:33 |
vilobhmm | do we really need it | 19:33 |
vilobhmm | or let the user update it depending on there deployments | 19:33 |
vilobhmm | why can't we reset it to 0 ? | 19:33 |
DuncanT | vilobhmm: 0 is not unlimitted though | 19:33 |
vilobhmm | hmm | 19:34 |
DuncanT | It is 0 | 19:34 |
vilobhmm | :P | 19:34 |
vilobhmm | alrite | 19:34 |
morganfainberg | Yeah. jamielennox|away is the person I'd rely on first in this case. | 19:34 |
DuncanT | That's quite a behaviour change | 19:34 |
vilobhmm | will keep it the way it is…was needing this info incontext of setting default values for nested projects | 19:34 |
*** ronis has joined #openstack-cinder | 19:34 | |
vilobhmm | thanks DuncanT | 19:34 |
*** e0ne is now known as e0ne_ | 19:35 | |
*** e0ne_ is now known as e0ne | 19:35 | |
DuncanT | vilobhmm: NP | 19:36 |
DuncanT | thingee: Are other clients using the same version discovery? | 19:37 |
*** Longgeek has joined #openstack-cinder | 19:37 | |
*** avishay_ has quit IRC | 19:38 | |
*** rwsu has joined #openstack-cinder | 19:38 | |
*** avishay_ has joined #openstack-cinder | 19:38 | |
*** rwsu has quit IRC | 19:40 | |
*** rwsu has joined #openstack-cinder | 19:40 | |
thingee | DuncanT: glance | 19:41 |
DuncanT | thingee: Thanks | 19:42 |
*** dustins_ has joined #openstack-cinder | 19:42 | |
*** dustins has quit IRC | 19:46 | |
*** mgarza has joined #openstack-cinder | 19:47 | |
*** merooney has quit IRC | 19:48 | |
*** avishay_ has quit IRC | 19:52 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/193977 | 19:52 |
*** agentle has joined #openstack-cinder | 19:52 | |
*** annashen has quit IRC | 19:54 | |
*** Lee1092 has quit IRC | 19:55 | |
*** afazekas has joined #openstack-cinder | 19:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/193996 | 19:59 |
*** timcl1 has left #openstack-cinder | 20:01 | |
*** avishay_ has joined #openstack-cinder | 20:02 | |
*** agentle has quit IRC | 20:02 | |
*** MentalRay has quit IRC | 20:04 | |
*** edmondsw has quit IRC | 20:04 | |
*** MentalRay has joined #openstack-cinder | 20:06 | |
*** markstur has joined #openstack-cinder | 20:07 | |
*** avishay_ has quit IRC | 20:09 | |
*** rwsu has quit IRC | 20:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/193977 | 20:12 |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to oslo.service https://review.openstack.org/193951 | 20:14 |
*** merooney has joined #openstack-cinder | 20:16 | |
*** ronis has quit IRC | 20:17 | |
*** anshul has quit IRC | 20:18 | |
*** afazekas has quit IRC | 20:19 | |
*** akerr has quit IRC | 20:21 | |
*** belmoreira has quit IRC | 20:21 | |
*** afazekas has joined #openstack-cinder | 20:21 | |
*** rushil has quit IRC | 20:23 | |
*** mriedem_afk is now known as mriedem | 20:24 | |
*** amoturi has quit IRC | 20:27 | |
*** rushil has joined #openstack-cinder | 20:28 | |
*** Anticime1 is now known as Anticimex | 20:29 | |
*** aarefiev_home has quit IRC | 20:30 | |
e0ne | jungleboyj, smcginnis: hi! thanks for review and attempt to merge my patch. but it depends on https://review.openstack.org/#/c/193861/ | 20:31 |
jungleboyj | Ruh roh. What did I do? | 20:32 |
*** dustins_ has quit IRC | 20:32 | |
e0ne | jungleboyj: this one https://review.openstack.org/#/c/193828/ has 'Depends On'(parent) patch | 20:33 |
jungleboyj | Oops, didn't notice that. | 20:33 |
*** jasondot_ has quit IRC | 20:33 | |
jungleboyj | Well, lets take a look at that one. | 20:34 |
*** Rockyg has quit IRC | 20:34 | |
e0ne | jungleboyj: np. it's not comfusing like -2 on parent and +2 on child patch:) | 20:35 |
jungleboyj | e0ne: mtanino Someone beat me to getting the oslo.service change in. I am going going to work looking at that one as well: https://review.openstack.org/#/c/193951/ | 20:35 |
mtanino | jungleboyj: nice. Wating the fix will be landed. | 20:36 |
e0ne | jungleboyj: thanks | 20:36 |
e0ne | time to sleep a bit:). bb in ~12 hours | 20:36 |
mtanino | good night :) | 20:36 |
*** marcusvrn has quit IRC | 20:41 | |
*** lpetrut has quit IRC | 20:44 | |
*** hodos|2 has joined #openstack-cinder | 20:46 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles https://review.openstack.org/193334 | 20:49 |
*** afazekas has quit IRC | 20:50 | |
jungleboyj | e0ne: smcginnis The dependency looks ok to me. | 20:51 |
smcginnis | jungleboyj: I'll take a look. | 20:53 |
*** annashen has joined #openstack-cinder | 20:54 | |
ericksonfgds | vilobhmm, I think in https://github.com/openstack/cinder/blob/master/cinder/quota.py#L495 we need to change the return to 0 instead of -1, if the current project is a subproject. | 20:55 |
*** salv-orlando has joined #openstack-cinder | 20:56 | |
ericksonfgds | vilobhmm, otherwise, it won't be consistent, I guess. | 20:56 |
*** rajen has joined #openstack-cinder | 20:57 | |
*** afazekas has joined #openstack-cinder | 20:58 | |
*** jasondotstar has joined #openstack-cinder | 20:59 | |
*** annashen has quit IRC | 21:00 | |
*** marcusvrn_ has joined #openstack-cinder | 21:00 | |
*** rushil has quit IRC | 21:01 | |
*** e0ne has quit IRC | 21:03 | |
scottda | thingee: Do you know why setting of 'insecure' was removed from the client in this patch: https://review.openstack.org/#/c/187295/2 | 21:04 |
scottda | The commit message is all about Hacking and PBR | 21:05 |
*** rushil has joined #openstack-cinder | 21:05 | |
*** aix has joined #openstack-cinder | 21:05 | |
*** e0ne has joined #openstack-cinder | 21:06 | |
scottda | or jgriffith ^^ | 21:06 |
scottda | \ | 21:06 |
*** e0ne has quit IRC | 21:09 | |
*** rwsu has joined #openstack-cinder | 21:10 | |
thingee | scottda: looks like a mistake. feel free to post a patch | 21:11 |
scottda | ok | 21:12 |
*** rushil has quit IRC | 21:14 | |
*** annashen has joined #openstack-cinder | 21:15 | |
*** Yogi1 has joined #openstack-cinder | 21:16 | |
*** Yogi11 has joined #openstack-cinder | 21:20 | |
*** Yogi1 has quit IRC | 21:21 | |
*** IlyaG has quit IRC | 21:22 | |
*** sgotliv has quit IRC | 21:22 | |
*** merooney has quit IRC | 21:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/193977 | 21:24 |
*** flwang has joined #openstack-cinder | 21:27 | |
*** jasondotstar has quit IRC | 21:27 | |
*** ociuhandu has joined #openstack-cinder | 21:27 | |
*** afazekas has quit IRC | 21:28 | |
*** kjnelson has quit IRC | 21:30 | |
openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to oslo.service https://review.openstack.org/193951 | 21:31 |
*** eharney has quit IRC | 21:32 | |
scottda | hmmm..it is assigned but never used, so causes a flak8 error. Perhaps it was removed for that. | 21:34 |
*** jasondot_ has joined #openstack-cinder | 21:37 | |
*** thangp has quit IRC | 21:38 | |
flwang | vilobhmm: | 21:41 |
flwang | ping | 21:41 |
vilobhmm | flwang : ack | 21:43 |
*** zigo has quit IRC | 21:43 | |
*** MentalRay has quit IRC | 21:43 | |
*** Longgeek has quit IRC | 21:43 | |
*** brad[] has quit IRC | 21:43 | |
*** IlyaG has joined #openstack-cinder | 21:47 | |
*** zigo has joined #openstack-cinder | 21:49 | |
flwang | vilobhmm: hi, cool, are you working on this https://blueprints.launchpad.net/glance/+spec/glance-quota-enhancements ? | 21:52 |
*** jbernard has quit IRC | 21:52 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Nested Quota Driver : Set default values to subproject https://review.openstack.org/194406 | 21:52 |
vilobhmm | right now there is no decision on this yet…hopefully they should have a decision in upcoming weekly meetings | 21:52 |
vilobhmm | flwang : ^^ | 21:53 |
flwang | who are they? BTW, i'm core member of glance team | 21:53 |
*** lan has quit IRC | 21:54 | |
vilobhmm | flwang : thats cool…."they" glance core team | 21:54 |
*** sparr has quit IRC | 21:54 | |
flwang | vilobhmm: i don't really understand why it even needs a discussion :) | 21:54 |
*** gouthamr has quit IRC | 21:55 | |
flwang | current quota implement of glance is lack of flexible | 21:55 |
flwang | we need to refactor/change it to support configure quota by rest api instead of a static conf file | 21:55 |
vilobhmm | ok..and that blueprint says the same :)…lets talk on glance channel if you dont mind | 21:55 |
flwang | sure | 21:55 |
*** ianbrown has joined #openstack-cinder | 21:56 | |
*** mtanino has quit IRC | 21:57 | |
*** jbernard has joined #openstack-cinder | 21:58 | |
*** IlyaG has quit IRC | 21:58 | |
*** jwcroppe_ has joined #openstack-cinder | 21:59 | |
*** MentalRay has joined #openstack-cinder | 22:00 | |
*** melwitt has joined #openstack-cinder | 22:00 | |
openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 22:01 |
*** jwcroppe has quit IRC | 22:01 | |
*** sparr has joined #openstack-cinder | 22:01 | |
openstackgerrit | xing-yang proposed openstack/cinder: Non-disruptive backup https://review.openstack.org/193937 | 22:03 |
*** jwcroppe_ has quit IRC | 22:03 | |
*** mgarza has quit IRC | 22:04 | |
*** jwcroppe has joined #openstack-cinder | 22:04 | |
*** git-harry has quit IRC | 22:04 | |
*** andymccr has quit IRC | 22:05 | |
*** agentle has joined #openstack-cinder | 22:06 | |
*** lan has joined #openstack-cinder | 22:06 | |
*** git-harry has joined #openstack-cinder | 22:06 | |
*** andymccr has joined #openstack-cinder | 22:07 | |
*** afazekas has joined #openstack-cinder | 22:08 | |
*** jwcroppe_ has joined #openstack-cinder | 22:10 | |
*** Longgeek has joined #openstack-cinder | 22:12 | |
*** jasondot_ has quit IRC | 22:12 | |
*** jwcroppe has quit IRC | 22:13 | |
*** rushiagr_away has quit IRC | 22:13 | |
*** afazekas has quit IRC | 22:13 | |
openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 22:13 |
*** zigo has quit IRC | 22:15 | |
jungleboyj | geguileo: You around? | 22:17 |
*** zigo has joined #openstack-cinder | 22:17 | |
*** primechuck has quit IRC | 22:22 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 22:25 |
anish | hemna: no tests yet, but can you give it a once over ^ ? | 22:26 |
*** rushiagr_away has joined #openstack-cinder | 22:28 | |
*** chlong has quit IRC | 22:29 | |
*** IlyaG has joined #openstack-cinder | 22:30 | |
hemna | anish, sure | 22:35 |
hemna | anish, so this replaces the need for the ISERConnector ? | 22:37 |
hemna | the iser device paths had ip-%s-iser-.... in them | 22:37 |
openstackgerrit | Eric Brown proposed openstack/cinder: Add bandit for security static analysis testing https://review.openstack.org/179568 | 22:38 |
*** aix has quit IRC | 22:41 | |
*** aix has joined #openstack-cinder | 22:41 | |
*** salv-orlando has quit IRC | 22:42 | |
*** jasondotstar has joined #openstack-cinder | 22:43 | |
hemna | anish, so the current ISERConnector has _get_device_path which looks for something in the form of ip-%s-iser-%s | 22:45 |
hemna | your new patch doesn't look for a device with that base | 22:46 |
*** agentle has quit IRC | 22:50 | |
*** agentle has joined #openstack-cinder | 22:51 | |
*** daneyon has quit IRC | 22:52 | |
*** daneyon has joined #openstack-cinder | 22:53 | |
*** jecarey has quit IRC | 22:53 | |
*** hodos|2 has quit IRC | 22:54 | |
anish | hemna: I don't know if you remember this | 22:55 |
anish | but I'd bought this point up earlier | 22:55 |
anish | iser in os_brick is very broken | 22:56 |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 22:56 |
hemna | anish, heh ok. I've never used ISER | 22:56 |
anish | so if you look at the original commit : https://review.openstack.org/#/c/146233/22/nova/virt/libvirt/volume.py | 22:57 |
*** DericHorn-HP_ has joined #openstack-cinder | 22:57 | |
anish | you can see that the _get_host_device for iser was replaced | 22:57 |
anish | the iser target is no longer using the separate 'iser' is the dev/by-path | 22:57 |
anish | this part was actually done by the mellanox guys who work with iser, so should not be a problem | 22:58 |
*** dannywil_ has quit IRC | 22:58 | |
anish | hemna: I'm trying to get a review from them as well, so will be tested by them I hope | 22:59 |
hemna | ok | 23:00 |
anish | but yes, the iser connector does get deprecated | 23:00 |
*** rwsu has quit IRC | 23:00 | |
hemna | ok I'll look at it a bit more tomorrow | 23:00 |
anish | does it need to be separate when there is really no code difference needed ? | 23:00 |
anish | aite, sounds good | 23:00 |
hemna | I'm trying to track down a nova -> os-brick issue I'm seeing with FC | 23:01 |
anish | wokay | 23:01 |
*** rajen has quit IRC | 23:03 | |
*** DericHorn-HP_ is now known as DericHorn-HP | 23:04 | |
*** jdurgin has quit IRC | 23:04 | |
*** ndipanov has quit IRC | 23:13 | |
*** marcusvrn_ has quit IRC | 23:14 | |
*** markvoelker has quit IRC | 23:16 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Add exception catch in report_state for DBError https://review.openstack.org/194232 | 23:17 |
*** DericHorn-HP has quit IRC | 23:21 | |
*** jungleboyj has quit IRC | 23:24 | |
*** DericHorn-HP has joined #openstack-cinder | 23:27 | |
*** melwitt has quit IRC | 23:27 | |
*** jasondotstar has quit IRC | 23:29 | |
*** DericHorn-HP has quit IRC | 23:30 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 23:30 |
*** annashen has quit IRC | 23:31 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 23:32 |
*** MentalRay has quit IRC | 23:34 | |
*** annashen has joined #openstack-cinder | 23:34 | |
openstackgerrit | Vilobh Meshram proposed openstack/cinder: Nested Quota Driver : Set default values to subproject https://review.openstack.org/194406 | 23:34 |
*** vilobhmm has quit IRC | 23:36 | |
*** vilobhmm has joined #openstack-cinder | 23:36 | |
*** angela-s has quit IRC | 23:38 | |
*** chlong has joined #openstack-cinder | 23:40 | |
*** Longgeek has quit IRC | 23:43 | |
*** gouthamr has joined #openstack-cinder | 23:44 | |
*** agentle has quit IRC | 23:45 | |
*** jaypipes has quit IRC | 23:46 | |
*** annashen has quit IRC | 23:46 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 23:48 |
*** nikeshm has quit IRC | 23:49 | |
openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 23:49 |
*** jasondotstar has joined #openstack-cinder | 23:54 | |
*** vilobhmm has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!