Monday, 2015-06-22

*** markvoelker has joined #openstack-cinder00:01
*** MentalRay has joined #openstack-cinder00:02
*** markvoelker has quit IRC00:05
*** Yogi1 has joined #openstack-cinder00:08
*** s-moriya has joined #openstack-cinder00:09
*** MentalRay has quit IRC00:25
*** Yogi11 has joined #openstack-cinder00:26
*** Yogi1 has quit IRC00:28
*** dimsum__ has quit IRC00:39
*** dimsum__ has joined #openstack-cinder00:41
*** annashen has joined #openstack-cinder00:48
*** dims_ has joined #openstack-cinder00:50
*** dimsum__ has quit IRC00:50
*** IanGovett has joined #openstack-cinder00:53
*** MentalRay has joined #openstack-cinder00:59
*** MentalRay has quit IRC00:59
*** salv-orlando has joined #openstack-cinder01:06
*** annashen has quit IRC01:10
*** salv-orlando has quit IRC01:11
*** annashen has joined #openstack-cinder01:17
*** vilobhmm has joined #openstack-cinder01:17
*** dimsum__ has joined #openstack-cinder01:17
*** dims_ has quit IRC01:17
*** marcusvrn has joined #openstack-cinder01:29
openstackgerritLin Yang proposed openstack/python-cinderclient: Improve error message when exceed cinder quota exceed  https://review.openstack.org/17894601:31
*** marcusvrn has quit IRC01:33
*** dims_ has joined #openstack-cinder01:40
*** dimsum__ has quit IRC01:41
*** Lee1092 has joined #openstack-cinder01:42
*** annashen has quit IRC01:46
*** annashen has joined #openstack-cinder01:49
*** markvoelker has joined #openstack-cinder01:50
*** markvoelker has quit IRC01:55
*** annashen has quit IRC02:01
*** marcusvrn has joined #openstack-cinder02:03
*** linkedinyou has quit IRC02:06
*** Yogi11 has quit IRC02:07
*** marcusvrn has quit IRC02:23
*** IanGovett has quit IRC02:53
*** vilobhmm has quit IRC03:05
*** alexpilotti has joined #openstack-cinder03:13
*** marcusvrn has joined #openstack-cinder03:17
*** salv-orlando has joined #openstack-cinder03:19
*** marcusvrn has quit IRC03:21
*** salv-orlando has quit IRC03:22
*** takedakn has joined #openstack-cinder03:24
*** sgotliv has joined #openstack-cinder03:29
*** gouthamr has joined #openstack-cinder03:32
*** takedakn has quit IRC03:33
*** takedakn has joined #openstack-cinder03:34
*** annashen has joined #openstack-cinder03:34
*** mriedem1 has joined #openstack-cinder03:37
*** mriedem has quit IRC03:38
*** markvoelker has joined #openstack-cinder03:38
*** dims_ has quit IRC03:40
*** markvoelker has quit IRC03:43
*** Longgeek has quit IRC03:48
*** Longgeek has joined #openstack-cinder03:49
*** annashen has quit IRC03:54
*** gouthamr has quit IRC03:58
*** takedakn has quit IRC04:02
*** annashen has joined #openstack-cinder04:12
*** avishay_ has joined #openstack-cinder04:13
*** laughterwym has joined #openstack-cinder04:14
*** annashen has quit IRC04:31
*** pradipta has joined #openstack-cinder04:35
*** dimsum__ has joined #openstack-cinder04:41
*** dimsum__ has quit IRC04:46
*** jwang has joined #openstack-cinder04:55
*** vilobhmm has joined #openstack-cinder04:56
*** jwang_ has quit IRC04:57
*** jwang_ has joined #openstack-cinder04:59
*** jwang has quit IRC05:01
*** salv-orlando has joined #openstack-cinder05:02
*** vilobhmm has quit IRC05:05
*** nkrinner has joined #openstack-cinder05:06
*** vilobhmm has joined #openstack-cinder05:08
*** vilobhmm has quit IRC05:10
*** vilobhmm has joined #openstack-cinder05:12
*** vilobhmm has quit IRC05:12
*** salv-orlando has quit IRC05:13
*** BharatK has joined #openstack-cinder05:13
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup  https://review.openstack.org/19393705:13
*** nihilifer has joined #openstack-cinder05:26
*** markvoelker has joined #openstack-cinder05:27
*** rushiagr_away is now known as rushiagr05:27
*** shausy has joined #openstack-cinder05:27
*** markvoelker has quit IRC05:32
*** nlevinki has joined #openstack-cinder05:33
*** shausy has quit IRC05:38
*** shausy has joined #openstack-cinder05:40
*** lpetrut has joined #openstack-cinder05:42
*** shausy has quit IRC05:45
*** nihilifer has quit IRC05:46
*** alexpilotti has quit IRC05:50
*** coolsvapz is now known as coolsvap06:02
*** deepakcs has joined #openstack-cinder06:08
*** belmoreira has joined #openstack-cinder06:10
*** avishay_ has quit IRC06:25
*** avishay_ has joined #openstack-cinder06:25
*** agarciam has joined #openstack-cinder06:26
*** pradipta has quit IRC06:34
*** ianbrown has quit IRC06:35
openstackgerritSergey Vilgelm proposed openstack/cinder: Switch to oslo.service  https://review.openstack.org/19395106:39
*** anshul has joined #openstack-cinder06:41
*** avishay_ is now known as avishay06:44
*** pradipta has joined #openstack-cinder06:46
*** Longgeek has quit IRC06:47
*** Longgeek has joined #openstack-cinder06:47
openstackgerritShay Halsband proposed openstack/cinder: XtremIO volume driver retry snapshot operation  https://review.openstack.org/19395406:48
*** lpetrut has quit IRC06:58
openstackgerritPradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector  https://review.openstack.org/18479906:58
*** yamada-h has joined #openstack-cinder07:09
*** salv-orlando has joined #openstack-cinder07:10
*** salv-orlando has quit IRC07:13
*** markvoelker has joined #openstack-cinder07:16
*** salv-orlando has joined #openstack-cinder07:17
*** markvoelker has quit IRC07:21
*** nihilifer has joined #openstack-cinder07:24
*** yamada-h has quit IRC07:25
*** markus_z has joined #openstack-cinder07:26
*** alonmarx has joined #openstack-cinder07:29
*** afazekas has joined #openstack-cinder07:31
*** sgotliv has quit IRC07:33
*** chlong has quit IRC07:39
*** ndipanov has joined #openstack-cinder07:39
*** ronis_ has joined #openstack-cinder07:46
*** nihilifer has quit IRC07:48
*** nihilifer has joined #openstack-cinder07:50
*** haomaiwa_ has quit IRC07:56
*** nihilifer has quit IRC07:59
*** nlevinki has quit IRC08:02
*** lpetrut has joined #openstack-cinder08:03
*** e0ne has joined #openstack-cinder08:05
*** e0ne has quit IRC08:11
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Change inventory folder hierarchy  https://review.openstack.org/19309508:16
*** nihilifer has joined #openstack-cinder08:17
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/19397708:20
*** belmoreira has quit IRC08:22
*** nihilifer has quit IRC08:23
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/19399608:26
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/19401908:27
*** aarefiev_home has joined #openstack-cinder08:28
openstackgerritTom Barron proposed openstack/cinder: Fix NetApp cDOT driver use of Glance image locations  https://review.openstack.org/19359708:34
*** aix has joined #openstack-cinder08:36
*** aarefiev_home has quit IRC08:39
*** yuriy_n17 has joined #openstack-cinder08:43
*** marzif has joined #openstack-cinder08:56
*** linkedinyou has joined #openstack-cinder09:02
*** markvoelker has joined #openstack-cinder09:05
*** e0ne has joined #openstack-cinder09:05
*** haypo has joined #openstack-cinder09:05
*** belmoreira has joined #openstack-cinder09:07
hayposmcginni1, 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
haypohttps://review.openstack.org/#/c/192118/ so my patch works ;)09:07
*** ociuhandu has joined #openstack-cinder09:07
hayposorry, i don't remember if you solved your issues with your local setup to get a working "tox -e py34"09:08
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy  https://review.openstack.org/19382809:08
*** markvoelker has quit IRC09:09
haypo(previously my "tox -e py34" patch has dependencies, but they are now merged ;-))09:12
*** ociuhandu has quit IRC09:19
*** e0ne is now known as e0ne_09:22
*** ociuhandu has joined #openstack-cinder09:23
*** e0ne_ is now known as e0ne09:27
*** esker has quit IRC09:42
e0nedulek: hi. could you please review https://review.openstack.org/193828?09:47
*** marcusvrn has joined #openstack-cinder09:53
*** marcusvrn has quit IRC09:58
*** aarefiev_home has joined #openstack-cinder10:01
*** Longgeek has quit IRC10:04
openstackgerritAndrey Pavlov proposed openstack/cinder: Avoid race condition at snapshot deletion stage  https://review.openstack.org/19359610:04
*** dimsum__ has joined #openstack-cinder10:14
*** pschaef has joined #openstack-cinder10:15
*** nihilifer has joined #openstack-cinder10:15
*** pschaef has quit IRC10:15
*** avishay_ has joined #openstack-cinder10:23
*** avishay has quit IRC10:23
*** ociuhandu has quit IRC10:25
*** jamielennox is now known as jamielennox|away10:25
*** nlevinki has joined #openstack-cinder10:32
*** sgotliv has joined #openstack-cinder10:33
*** nihilifer has quit IRC10:33
*** Longgeek has joined #openstack-cinder10:34
*** sgotliv has quit IRC10:35
*** sgotliv has joined #openstack-cinder10:35
*** nihilifer has joined #openstack-cinder10:37
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087310:42
*** aarefiev_home has quit IRC10:43
*** deepakcs has quit IRC10:43
*** Lee1092 has quit IRC10:43
*** rhe00_ has quit IRC10:43
*** scottda has quit IRC10:43
*** Guest11697 has quit IRC10:43
*** zigo has quit IRC10:43
*** liewegas has quit IRC10:43
*** Trozz_ has quit IRC10:43
*** jbernard has quit IRC10:43
*** Trozz_ has joined #openstack-cinder10:43
*** deepakcs has joined #openstack-cinder10:43
*** mgagne has joined #openstack-cinder10:43
*** mgagne is now known as Guest8120210:43
*** zigo has joined #openstack-cinder10:44
*** sage has joined #openstack-cinder10:44
*** rhe00_ has joined #openstack-cinder10:44
*** e0ne is now known as e0ne_10:44
*** aarefiev_home has joined #openstack-cinder10:45
*** Lee1092 has joined #openstack-cinder10:46
*** scottda has joined #openstack-cinder10:47
*** dimsum__ is now known as dims10:48
*** s-moriya has quit IRC10:50
*** dims has quit IRC10:53
*** e0ne_ is now known as e0ne10:53
*** markvoelker has joined #openstack-cinder10:53
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087310:58
*** markvoelker has quit IRC10:58
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087311:00
*** IanGovett has joined #openstack-cinder11:04
*** linkedinyou has quit IRC11:04
*** dims has joined #openstack-cinder11:05
*** nihilifer has quit IRC11:05
*** marcusvrn has joined #openstack-cinder11:07
*** yamada-h has joined #openstack-cinder11:09
*** merooney has joined #openstack-cinder11:12
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087311:14
*** yamada-h has quit IRC11:15
*** daemontool_ has joined #openstack-cinder11:16
*** marzif has quit IRC11:18
*** Longgeek has quit IRC11:18
*** cdelatte has joined #openstack-cinder11:23
*** aarefiev_home has quit IRC11:33
*** aarefiev_home has joined #openstack-cinder11:33
*** IanGovett has quit IRC11:34
*** IanGovett has joined #openstack-cinder11:35
*** merooney has quit IRC11:35
*** daemontool_ has quit IRC11:38
*** marzif has joined #openstack-cinder11:39
*** merooney has joined #openstack-cinder11:41
*** Longgeek has joined #openstack-cinder11:46
openstackgerritYuriy Nesenenko proposed openstack/cinder: Checking the existence of volume  https://review.openstack.org/17580911:51
*** deepakcs has quit IRC11:51
*** BharatK has quit IRC11:53
*** markvoelker has joined #openstack-cinder11:54
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087311:54
*** markvoelker has quit IRC11:59
*** Yogi1 has joined #openstack-cinder11:59
*** marcusvrn has quit IRC12:02
*** marcusvrn has joined #openstack-cinder12:02
*** markvoelker has joined #openstack-cinder12:03
*** BharatK has joined #openstack-cinder12:05
*** bswartz has quit IRC12:10
*** Yogi1 has quit IRC12:12
*** e0ne is now known as e0ne_12:18
*** e0ne_ is now known as e0ne12:19
*** aarefiev_home has quit IRC12:20
*** vincent_hou has joined #openstack-cinder12:21
vincent_hougeguileo: Hi.12:22
geguileovincent_hou: Hi12:22
*** edmondsw has joined #openstack-cinder12:22
vincent_hougeguileo: I saw your comments on my patch.12:22
vincent_hougeguileo: 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
geguileovincent_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_houhttps://review.openstack.org/#/c/180873/12:24
vincent_houIt seems there is no test case for LVM at all.12:24
geguileoMmmmm, it can happen, let me check12:24
*** chlong has joined #openstack-cinder12:25
geguileovincent_hou: Did you look in test_volume.py?12:25
vincent_houHA.12:26
vincent_houOK. I found the class.12:27
geguileoGreat!  :)12:27
vincent_houThank you.12:27
geguileovincent_hou: Thank you for the nice patch  :)12:27
vincent_houI was trying to get it.12:27
vincent_houNice12:27
vincent_houYou are welcome.12:28
*** chlong has quit IRC12:28
*** timcl has joined #openstack-cinder12:31
*** chlong has joined #openstack-cinder12:31
*** tobasco_ has joined #openstack-cinder12:32
*** tobasco_ is now known as tobasco12:32
tobascoanybody 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
geguileotobasco: According to this https://wiki.openstack.org/wiki/ThirdPartySystems12:35
geguileotobasco: It is currently reporting12:35
*** alexpilotti has joined #openstack-cinder12:37
*** akerr has joined #openstack-cinder12:37
*** yrabl has quit IRC12:38
openstackgerritDavid Medberry proposed openstack/cinder: Removing archaic reference to quickstart.  https://review.openstack.org/19379112:38
*** yrabl has joined #openstack-cinder12:38
*** Longgeek has quit IRC12:47
*** vincent_hou has quit IRC12:49
*** kjnelson has joined #openstack-cinder12:51
duleke0ne: Thanks for the fix, it's a shame I haven't noticed it.12:52
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887712:52
duleke0ne: I've left a comment on the patch - one issue with unit test.12:52
*** marcusvrn has quit IRC12:52
*** Zhongjun has joined #openstack-cinder12:56
mtreinishmriedem: well I guess I could have used 0.0.1, but I dunno12:57
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087312:57
*** e0ne is now known as e0ne_12:58
*** aarefiev_home has joined #openstack-cinder12:58
mtreinishmriedem: os-testr is still rough in a lot of ways, I was hoping you'd fix it in my absence12:58
*** chlong has quit IRC12:58
*** e0ne_ is now known as e0ne12:59
*** rushil has joined #openstack-cinder13:00
*** chlong has joined #openstack-cinder13:02
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087313:03
*** bswartz has joined #openstack-cinder13:05
*** chlong has quit IRC13:08
*** chlong has joined #openstack-cinder13:08
*** marcusvrn has joined #openstack-cinder13:09
*** rushiagr is now known as rushiagr_away13:10
*** dustins has joined #openstack-cinder13:11
*** coolsvap is now known as coolsvap|away13:13
*** erlon has joined #openstack-cinder13:15
*** jaypipes has joined #openstack-cinder13:15
*** Longgeek has joined #openstack-cinder13:16
*** chlong has quit IRC13:17
*** hodos has joined #openstack-cinder13:17
openstackgerritShay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth  https://review.openstack.org/18935813:17
*** Yogi1 has joined #openstack-cinder13:18
*** merooney has quit IRC13:18
tobascogeguileo: thanks!13:19
*** gouthamr has joined #openstack-cinder13:20
*** Yogi11 has joined #openstack-cinder13:21
*** rushil has quit IRC13:22
*** Yogi1 has quit IRC13:23
*** mordred has joined #openstack-cinder13:28
mordredmorning 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
mordredmy sypmtoms are taht "cinder list" and "cinder type-list" hang until they timeout13:29
*** merooney has joined #openstack-cinder13:29
*** chlong has joined #openstack-cinder13:30
mordredhttp://paste.openstack.org/show/313611/ <-- this is the end of the debug tracing from running against rax public13:30
mordredyou can see that cinderclient is nicely trying to do version autonegotiation, which is awesome13:31
mordredexcept that the urls returned are http urls13:31
mordredwhich are not valid13:31
mordredin 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 work13:31
mordred(so a forced downgrade to 1.1.1 works on both hp and rackspace)13:32
*** primechuck has joined #openstack-cinder13:32
mordredalso - explicitly specifying a version does not help13:32
*** timcl has quit IRC13:35
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy  https://review.openstack.org/19382813:35
mordredthe 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 for13:36
mordredDuncanT: ^^ if you happen to be around, might be interesting to our corporate overlords13:36
*** jungleboyj has quit IRC13:37
DuncanTmordred: Looking now13:37
DuncanTmordred: All our stuff is still using the 1.1.1 client AFAICT, so we've never noticed13:39
mordredossum13:39
mordredwell, infra's production scripts are using 1.1.1 too - so also never notied there :)13:39
*** thangp has joined #openstack-cinder13:39
mordredDuncanT: here's the trace from hp public http://paste.openstack.org/show/313664/13:39
mordredDuncanT: why it wants to talk to region-a's compute is mindboggline to me13:39
*** timcl has joined #openstack-cinder13:40
*** zigo has quit IRC13:47
*** merooney has quit IRC13:48
*** rushiagr_away is now known as rushiagr13:49
DuncanTmorded: Might be a messed up catalogue, investigating (and having my wifi crash too, just for added entertainment)13:50
smcginni1DuncanT: Jay's not here, or I would make some comment on that.13:50
DuncanTsmcginni1: lol13:50
*** smcginni1 is now known as smcginnis13:51
DuncanTsmcginnis: Remind me not to hold a mid-cycle at my house...13:51
smcginnisDuncanT: +1 ;)13:51
*** r-daneel has joined #openstack-cinder13:52
*** breitz has joined #openstack-cinder13:56
*** e0ne is now known as e0ne_13:57
*** Tross has quit IRC13:57
*** e0ne_ is now known as e0ne13:57
*** ronis_ has quit IRC14:01
*** merooney has joined #openstack-cinder14:02
DuncanTmordred: I have found at least one problem, investigating how to fix it14:03
*** zigo has joined #openstack-cinder14:05
*** lpetrut has quit IRC14:05
*** nlevinki has quit IRC14:06
mordredDuncanT: neat- finding problems is exciting14:08
*** eharney has joined #openstack-cinder14:08
*** BharatK has quit IRC14:09
DuncanTmordred: At least I'm not finding out at 4am via a pager :-)14:09
mordredDuncanT: see, this is how I show I care14:10
*** jbernard has joined #openstack-cinder14:10
*** rmesta has joined #openstack-cinder14:11
*** mriedem1 is now known as mriedem_afk14:12
*** beekneemech is now known as bnemec14:13
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087314:16
*** jecarey has joined #openstack-cinder14:17
*** rushil has joined #openstack-cinder14:19
*** timcl1 has joined #openstack-cinder14:19
*** timcl has quit IRC14:21
*** timcl1 has quit IRC14:22
*** nlevinki has joined #openstack-cinder14:22
*** aarefiev_home has quit IRC14:23
*** ameade_ is now known as ameade14:23
*** paboldin has joined #openstack-cinder14:28
paboldinjgriffith: hello John, here?14:28
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Change inventory folder hierarchy  https://review.openstack.org/19309514:28
paboldin@all, please review https://review.openstack.org/182871/14:31
DuncanTpaboldin: Why are they called iscsi_target_flags if they only apply to tgt?14:33
*** Guest78977 has joined #openstack-cinder14:33
paboldinDuncanT: 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
DuncanTpaboldin: Ok, I'll read some more14:34
*** ociuhandu has joined #openstack-cinder14:35
*** mfisch` is now known as mfisch14:36
*** mfisch is now known as Guest8229014:36
*** jungleboyj has joined #openstack-cinder14:37
dulekOpenStack Mita? Really?14:39
dulekI was hoping for Musashi.14:39
smcginnisdulek: We had an internal project named Musashi. Otherwise I would have like that one.14:40
*** rushil has quit IRC14:41
*** asselin has joined #openstack-cinder14:42
duleksmcginnis: "OpenStack Musashi. Bleeding edge of cloud technology." ;)14:42
smcginnis:)14:43
*** Longgeek has quit IRC14:43
*** marcusvrn_ has joined #openstack-cinder14:43
DuncanTpaboldin: 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 enum14:45
paboldinDuncanT: but most likely there will be the same options for the LIOadm14:46
paboldinDuncanT: https://bugs.launchpad.net/cinder/+bug/1441935/comments/6 < like here14:47
openstackLaunchpad 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-cinder14:47
*** ilyag has joined #openstack-cinder14:48
DuncanTpaboldin: Are bsoflags the only tgt flags?14:48
DuncanTpaboldin: Or are there other flags we might want to set in future?14:48
paboldinDuncanT: 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
DuncanTpaboldin: 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 patch14:50
DuncanTpaboldin: But if there are other tgt-specific flag fields, I'm less happy14:50
DuncanTpaboldin: I don't know much about tgt at all; I'm hoping you can tell me14:50
*** ilyag has quit IRC14:50
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887714:51
*** rushil has joined #openstack-cinder14:52
paboldinDuncanT: 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-cinder14:53
*** Guest78977 has quit IRC14:53
DuncanTpaboldin: Ok, I'm happy other than the missing bug to implement this config option for other initiators, so I'll approve and file that bug14:54
paboldinDuncanT: wait a second, I want to check if LIO is actually requires something alike14:54
DuncanTpaboldin: Thanks for the clarification14:54
DuncanTok14:54
DuncanTwaiting14:54
*** marzif has quit IRC14:55
*** rmesta has left #openstack-cinder14:55
*** timcl has joined #openstack-cinder14:57
*** julim has joined #openstack-cinder14:58
*** patrickeast has joined #openstack-cinder15:00
*** timcl1 has joined #openstack-cinder15:00
*** MentalRay has joined #openstack-cinder15:01
*** timcl has quit IRC15:02
mordredDuncanT: 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 help15:03
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Add more Rally scenarios to run  https://review.openstack.org/19418015:03
*** mtanino has joined #openstack-cinder15:05
*** nlevinki has quit IRC15:06
*** nkrinner has quit IRC15:07
DuncanTmordred: Thanks. I'll keep working on it anyway15:09
*** Longgeek has joined #openstack-cinder15:10
paboldinDuncanT: 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 IRC15:11
DuncanTpaboldin: Ok, it is probably ok to leave that until people find a need then15:11
DuncanTpaboldin: Thanks for looking / clarifying15:12
paboldinDuncanT: thanks for reviewing15:12
*** ronis_ has joined #openstack-cinder15:13
*** kjnelson has joined #openstack-cinder15:14
DuncanTpaboldin: Hopefully it will merge15:14
paboldinDuncanT: me too. had some not so pleasent times doing it :-/15:15
*** dannywilson has joined #openstack-cinder15:15
mkodererthingee: ping15:16
thingeemkoderer: hey15:16
mkodererthingee: hey :)15:16
*** aix has quit IRC15:16
mkodererthingee: I am again a bit confused about this ABC ML thread ;)15:17
*** daneyon has joined #openstack-cinder15:17
*** dannywil_ has joined #openstack-cinder15:17
*** dannywilson has quit IRC15:18
mkodererthingee: I think the point of cknight is a valid drawback of ABC.. but I really didn't get the point what is actually the problem15:19
*** nlevinki has joined #openstack-cinder15:19
*** agentle has joined #openstack-cinder15:19
mkodererjgriffith: ^^15:19
thingeemkoderer: 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 IRC15:20
thingeemkoderer: 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 IRC15:20
mkodererthingee: mh, so making all functions mandatory?15:21
thingeeat least the ones we can right now15:21
*** MentalRay has quit IRC15:21
thingeenot everyone is going to have consistency groups for example15:21
*** dguryanov has joined #openstack-cinder15:22
mkodererthingee: ok, this was actually one of your comments in my cinder-spec for this release15:22
mkodererthingee: following the idea to move more and more function to BaseVD sounds good15:22
mkodererbut we need to port all the drivers first ;)15:22
*** afazekas has quit IRC15:23
*** timcl has joined #openstack-cinder15:25
*** MentalRay has joined #openstack-cinder15:26
*** kjnelson1 has joined #openstack-cinder15:26
*** daneyon_ has joined #openstack-cinder15:26
*** anshul has quit IRC15:28
*** kjnelson has quit IRC15:28
*** haypo has left #openstack-cinder15:28
*** belmoreira has quit IRC15:28
*** daneyon has quit IRC15:29
*** jdurgin1 has joined #openstack-cinder15:31
*** hodos has quit IRC15:31
e0nethingee: 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-115:32
thingeee0ne: I'll need to try again, but I was not able to reproduce unit tests not passing15:33
e0nethingee: let's not mix my patches15:34
e0nethingee: i'm talking about https://bugs.launchpad.net/cinder/+bug/146716715:34
openstackLaunchpad bug 1467167 in Cinder "Can't delete backup: User does not have admin privileges" [High,In progress] - Assigned to Ivan Kolodyazhny (e0ne)15:34
thingeee0ne: ok15:34
*** sseverson has joined #openstack-cinder15:35
e0nethingee: imo, failed unit tests on a local is not a high priory issue15:35
thingeee0ne: I agree15:35
*** Guest81202 is now known as mgagne15:36
*** mgagne has joined #openstack-cinder15:36
e0nethingee: according to https://review.openstack.org/#/c/193861/. './run_tests.sh cinder.unit.objects.test_backup' fails on a master15:36
*** timcl has left #openstack-cinder15:36
*** agentle has quit IRC15:39
thangpe0ne: I put a comment in https://review.openstack.org/#/c/193861/ about why it failed15:39
*** agentle has joined #openstack-cinder15:40
e0nethangp: thanks. you're right. i'll post a reply righ now15:41
thangpe0ne: either solution will work15:41
thangpe0ne: it's just how far do we want to go15:41
*** rushil has quit IRC15:41
duleke0ne, thangp: From my perspective thangp's solution seems better.15:41
*** idevops has joined #openstack-cinder15:41
*** idevops has quit IRC15:42
openstackgerritJohn Haan proposed openstack/python-cinderclient: Adding search options to backup list  https://review.openstack.org/7823315:42
openstackgerritJohn Haan proposed openstack/python-cinderclient: Adding search options to backup list  https://review.openstack.org/7823315:45
*** harlowja_at_home has joined #openstack-cinder15:45
*** agarciam has quit IRC15:48
dulekthangp: Hi, I started to wonder a little about relationships between SQLAlchemy objects and VersionedObjects.15:48
*** aarefiev_home has joined #openstack-cinder15:48
thingeee0ne: lgtm15:48
*** Longgeek has joined #openstack-cinder15:48
thingeee0ne: thanks for taking care of that15:48
thangpdulek: how so?15:49
e0nethingee: thanks15:49
thingeethangp: https://review.openstack.org/#/c/193828/915:49
dulekthangp: For example - volume have transfer relationship. It's defined in Transfer SQLAlchemy object with backref.15:49
dulekthingee: So technically such field is available in volume object.15:50
e0nethangp, dulek: i unswered you in gerrit15:50
dulekI've meant thangp. ;) ^15:51
thangpe0ne: https://review.openstack.org/#/c/193828/9 - is context.elevated() needed for QUOTAS.commit()?15:52
*** merooney has quit IRC15:52
*** rushil has joined #openstack-cinder15:55
*** hodos has joined #openstack-cinder15:56
*** afazekas has joined #openstack-cinder15:58
*** nlevinki has quit IRC15:59
*** salv-orl_ has joined #openstack-cinder16:00
e0nethangp: no16:01
thingeehodos: where are the unit tests for nexenta ns4? https://review.openstack.org/#/c/188877/16:01
*** leeantho has joined #openstack-cinder16:01
thangpe0ne: ok, thx16:01
thingeeilyag: ^16:01
hodosthingee: wow, i must've forgot to put back in16:01
hodosthingee: let me update the review16:02
thingeekk16:02
*** salv-orlando has quit IRC16:03
*** hemna has joined #openstack-cinder16:04
*** laughterwym has quit IRC16:06
*** afazekas has quit IRC16:06
thingeee0ne: for some reason devstack isn't able to start up with https://review.openstack.org/#/c/145703/16:07
thingeee0ne: something with pip16:07
e0nethingee: :(16:07
*** hemna has quit IRC16:07
e0nethingee: i'm investigating fix for rally to support api v216:08
*** agentle has quit IRC16:08
openstackgerritJames Page proposed openstack/os-brick: Misc Python 3 compatibility fixes  https://review.openstack.org/19421616:09
*** Guest82290 is now known as mfisch16:10
*** mfisch has quit IRC16:10
*** mfisch has joined #openstack-cinder16:10
*** dtynan1 has joined #openstack-cinder16:11
thingeee0ne: https://bugs.launchpad.net/cinder/+bug/146758916:11
openstackLaunchpad bug 1467589 in Cinder "Remove Cinder V1 supprt" [Undecided,In progress] - Assigned to Mike Perez (thingee)16:11
thingeee0ne: lets track projects that need to be worked on there16:11
*** e0ne has quit IRC16:11
*** pradipta has quit IRC16:12
*** e0ne has joined #openstack-cinder16:13
*** hemna has joined #openstack-cinder16:14
*** merooney has joined #openstack-cinder16:14
*** afazekas has joined #openstack-cinder16:14
*** bswartz has quit IRC16:15
*** patrickeast has quit IRC16:16
*** asselin has quit IRC16:18
*** david-ly_ is now known as david-lyle16:18
hodosthingee: added unit tests16:19
*** Apoorva has joined #openstack-cinder16:19
*** sseverson has quit IRC16:20
*** avishay_ has quit IRC16:21
openstackgerritMitsuhiro Tanino proposed openstack/cinder: LVM: Remove unnecessary volume export after generic volume migration  https://review.openstack.org/19422316:22
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887716:22
*** agentle has joined #openstack-cinder16:23
*** Yogi11 has quit IRC16:27
*** jdurgin1 has quit IRC16:28
*** harlowja_at_home has quit IRC16:34
*** markus_z has quit IRC16:39
*** xyang1 has joined #openstack-cinder16:42
*** asselin has joined #openstack-cinder16:44
*** rwsu has joined #openstack-cinder16:45
*** daneyon has joined #openstack-cinder16:46
*** teju has joined #openstack-cinder16:46
*** daneyon_ has quit IRC16:47
*** timcl has joined #openstack-cinder16:47
openstackgerritSean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles  https://review.openstack.org/19333416:48
*** agentle has quit IRC16:49
*** afazekas has quit IRC16:49
*** agentle has joined #openstack-cinder16:50
openstackgerritJay Bryant proposed openstack/cinder: Add exception catch in report_state for DBError  https://review.openstack.org/19423216:51
*** MRay has joined #openstack-cinder16:51
*** MentalRay has quit IRC16:54
*** afazekas has joined #openstack-cinder16:54
*** daneyon_ has joined #openstack-cinder16:58
*** daneyon has quit IRC16:58
*** daneyon has joined #openstack-cinder16:59
*** vilobhmm has joined #openstack-cinder16:59
*** yamada-h has joined #openstack-cinder17:00
*** afazekas has quit IRC17:00
*** teju has left #openstack-cinder17:00
*** daneyon_ has quit IRC17:03
*** esker has joined #openstack-cinder17:03
*** patrickeast has joined #openstack-cinder17:05
*** timcl has left #openstack-cinder17:05
*** e0ne has quit IRC17:05
*** ronis_ has quit IRC17:07
*** dims has quit IRC17:09
*** dims has joined #openstack-cinder17:09
*** hodos has quit IRC17:10
*** timcl has joined #openstack-cinder17:10
*** kjnelson1 has quit IRC17:11
*** hemna has quit IRC17:14
*** hemna has joined #openstack-cinder17:15
*** daneyon has quit IRC17:18
*** daneyon has joined #openstack-cinder17:19
*** rwsu has quit IRC17:19
*** Longgeek has quit IRC17:20
*** daneyon has quit IRC17:20
*** rushil has quit IRC17:21
*** daneyon has joined #openstack-cinder17:21
*** daneyon has quit IRC17:23
*** thangp is now known as thangp_afk17:24
*** marcusvrn_ has quit IRC17:24
*** rushil has joined #openstack-cinder17:25
rlucioquestion 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 check17:26
*** aarefiev_home has quit IRC17:27
patrickeastrlucio: fwiw i’ve noticed small changes move fast and seem to get reviewed earlier17:28
*** harlowja has joined #openstack-cinder17:28
*** annashen has joined #openstack-cinder17:29
*** dustins_ has joined #openstack-cinder17:30
*** daneyon has joined #openstack-cinder17:30
rluciopatrckeast: yea, that makes sense17:30
thingeeguitarzan: I heard from mordred and DuncanT that cinder client is broken at rax as well?17:32
mordredthingee: yes17:33
DuncanTrlucio: Definitely a bunch of small changes please17:33
thingeeguitarzan, mordred, DuncanT: I'm aware of an issue with ha proxy and ssl termination with versions below Kilo.17:33
*** dustins has quit IRC17:33
mordredthingee: so- the rax issus, amongst other things, is that the version discovery is returning http endpoints17:33
mordredthingee: which are not valid17:34
mordredthingee: so, it seems that there is at least one deployment related issue17:34
mordredthingee: 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 to17:34
thingeemordred: so fix for that, is this option, https://github.com/openstack/cinder/blob/master/cinder/api/views/versions.py#L2317:34
thingeemordred: but that's kilo =/17:35
mordredyah17:35
thingeemordred: I think the discovery release is premature without this config option being out longer.17:35
mordredso - 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 workaround17:35
rlucioduncant: ok, no prob17:35
mordredI LOVE that it's doing discovery :)17:35
thingeemordred: oh me too17:36
DuncanTWhat I can't figure out is why it is trying to connect to a compute endpoint17:36
*** rwsu has joined #openstack-cinder17:36
mordredDuncanT: it only does that for me against HP17:36
mordredDuncanT: for rax, it does not seem to do that17:36
DuncanTmordred: Right. I can't figure out where it got that URL from17:37
thingeemordred: yeah DuncanT showed me the debug logs from the client's perspective. I have no clue how compute gets involved there.17:37
mordredDuncanT: :)17:37
* mordred likes to bring in the fun issues17:37
thingee:)17:37
*** paboldin has left #openstack-cinder17:37
* thingee updates release notes about public_endpoint config opt17:38
*** Zhongjun has quit IRC17:38
*** agentle has quit IRC17:41
*** agentle has joined #openstack-cinder17:41
thingeemordred, DuncanT: so the client could start checking headers from the proxy to default to previous behavior17:42
*** yamada-h has quit IRC17:43
*** rushil has quit IRC17:43
DuncanTthingee: I've got issues before that :-(17:43
thingeeDuncanT: yeah I'm not sure what happens in your case, but at least help a couple of other deployers17:44
*** reed has quit IRC17:44
*** reed has joined #openstack-cinder17:44
DuncanTStill hacking on our case17:44
thingeemorganfainberg: 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 IRC17:45
morganfainbergRefresh my memory on that header?17:46
*** timcl has joined #openstack-cinder17:46
vilobhmmDuncanT, smcginnis  : https://review.openstack.org/#/c/185704/ needs review before it runs into merge conflict again..thanks..17:46
vilobhmmwhenever you have time17:46
morganfainbergI remember saying it but my brain is context switched away from the ha proxy convo. And version discovery17:47
thingeemorganfainberg: you never mentioned the header :) ... but you mentioned there was a magical header I could check for17:47
thingeemorganfainberg: ha proxy with ssl termination17:47
morganfainbergHmm. Brain is not connecting.17:47
morganfainbergI think there is a standard proxy header that can be referenced.17:48
thingeepublic end point is https, but cinder is giving back http.17:48
morganfainbergYeah. Let me see.17:48
*** Longgeek has joined #openstack-cinder17:49
*** agentle has quit IRC17:50
*** timcl has quit IRC17:51
morganfainbergthingee: I think all the info you need should be in x-forwarded-for17:51
thingeethanks17:51
morganfainbergthingee: *think*. And that should be provided for any forwarding proxy.17:51
*** timcl has joined #openstack-cinder17:53
morganfainbergthingee: you might need to explicitly setup some haproxy headers though. Hmm. It's not clear to me atm.17:54
DuncanTOk, so it looks like maybe our bug is specifically triggered when you have a multi-region setup17:56
DuncanTSome call somewhere is ignoring OS_REGION_NAME17:56
DuncanTvilobhmm: Done17:57
thingeeDuncanT: even when explicit set by client?17:57
thingeeI guess nothing is setting the default anyways17:57
DuncanTthingee: When set in the client env17:57
vilobhmmDuncanT, smcginnis : thanks :)17:57
*** kjnelson has joined #openstack-cinder17:58
ericksonfgdsnice job, vilobhmm :)17:59
vilobhmmDuncanT : 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 :P18:01
*** anshul has joined #openstack-cinder18:01
vilobhmmericksonfgds : :) thnx18:01
thingeeDuncanT: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/shell.py#L70218:02
thingeeDuncanT: can you hack in a region_name kwarg there and let me know18:02
DuncanTOne sec18:03
*** patrickeast_ has joined #openstack-cinder18:03
smcginnisvilobhmm: It's merging. It takes a little while. :)18:03
*** patrickeast has quit IRC18:03
*** patrickeast_ is now known as patrickeast18:03
*** dustins_ has quit IRC18:03
*** dustins has joined #openstack-cinder18:04
DuncanTthingee: That's the one18:04
thingeeDuncanT: you're working again with that?18:05
vilobhmmsmcginnis : oh okay :) thanks !18:05
DuncanTthingee: It is now doing version discovery, and trying to use the internal URL18:05
DuncanTthingee: That we know about and understand18:06
thingeehmm internal one18:06
*** thangp_afk is now known as thangp18:06
DuncanTYup, I at least understand where it is getting that from though, and I can see where we give out the wrong answer18:07
*** salv-orl_ has quit IRC18:07
*** MRay has quit IRC18:09
thingeeDuncanT: weird, so os_endpoint_type should be defaulting to publicURL18:09
thingeerather endpoint_type18:10
DuncanTthingee: 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 to18:11
DuncanTthingee: So we're in the same state as bluebox et al18:11
*** e0ne has joined #openstack-cinder18:11
DuncanTthingee: We know how to fix this, but need to deploy new code to do so18:12
*** bswartz has joined #openstack-cinder18:12
thingeeDuncanT: x-forwarded-for in the headers?18:12
thingeethat comes back from the proxy?18:12
*** afazekas has joined #openstack-cinder18:12
DuncanTthingee: I can't see the headers on the server side... or do you mean client side?18:13
*** angela-s has joined #openstack-cinder18:14
thingeeDuncanT: what the client would be getting in the response18:14
DuncanTNothing18:14
DuncanTJust the normal HTTP/1.1 200 OK18:14
DuncanTContent-Type: application/json18:14
DuncanTContent-Length: 36018:14
DuncanTDate: Mon, 22 Jun 2015 18:14:11 GMT18:14
DuncanTConnection: keep-alive18:14
thingeeDuncanT: yeah my bad I meant it the other way...well if you can't get access to the logs on server side18:17
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor -os-region-name  https://review.openstack.org/19428018:18
DuncanTthingee: I will figure out how to get it. We don't log it normally18:19
hemnascottda, --os-region-name18:19
*** aarefiev_home has joined #openstack-cinder18:19
DuncanThemna: Or the env variable. It's only a typo in the bug, the problem is real (and thingee gave us the fix)18:20
thingeeDuncanT, scottda: there are multiple places get_endpoint is called in that block18:20
scottdayeah, my bad. I'll put up a new patch18:20
hemnascottda, also mention the env. var as well18:21
hemnafwiw18:21
DuncanTthingee: Yup, we'll add it to them all, thanks18:21
DuncanTAlso figure out how to fake regions in devstack18:22
*** merooney has quit IRC18:23
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor -os-region-name or ENV[OS_REGION_NAME]  https://review.openstack.org/19428018:23
*** e0ne is now known as e0ne_18:23
*** e0ne_ is now known as e0ne18:24
*** timcl has quit IRC18:24
hemna--18:24
scottdaahhhg18:25
hemna:P18:25
*** jasondot_ has joined #openstack-cinder18:25
openstackgerritScott DAngelo proposed openstack/python-cinderclient: cinderclient does not honor --os-region-name or ENV[OS_REGION_NAME]  https://review.openstack.org/19428018:26
*** belmoreira has joined #openstack-cinder18:26
*** merooney has joined #openstack-cinder18:28
*** xyang1 has quit IRC18:30
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Fix Cinder Objects unit tests  https://review.openstack.org/19386118:33
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use elevated context for backup destroy  https://review.openstack.org/19382818:33
*** avishay_ has joined #openstack-cinder18:35
*** merooney has quit IRC18:36
*** xyang1 has joined #openstack-cinder18:37
*** esker has quit IRC18:38
*** merooney has joined #openstack-cinder18:38
*** timcl has joined #openstack-cinder18:39
*** ilyag has quit IRC18:39
*** timcl1 has joined #openstack-cinder18:41
*** afazekas has quit IRC18:42
*** timcl has quit IRC18:44
*** amoturi has joined #openstack-cinder18:48
*** afazekas has joined #openstack-cinder18:49
thingeeDuncanT, scottda: just abou have this narrowed down18:50
*** jasondot_ has quit IRC18:51
*** Rockyg has joined #openstack-cinder18:51
*** agentle has joined #openstack-cinder18:58
*** salv-orlando has joined #openstack-cinder18:59
*** agentle_ has joined #openstack-cinder19:02
*** afazekas has quit IRC19:03
*** afazekas has joined #openstack-cinder19:04
*** e0ne has quit IRC19:04
*** salv-orlando has quit IRC19:04
*** e0ne has joined #openstack-cinder19:05
*** agentle has quit IRC19:05
*** MRay has joined #openstack-cinder19:06
openstackgerritSergey Vilgelm proposed openstack/cinder: Switch to oslo.service  https://review.openstack.org/19395119:07
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta driver.  https://review.openstack.org/18887719:07
*** rushiagr is now known as rushiagr_away19:08
*** Longgeek has quit IRC19:09
*** esker has joined #openstack-cinder19:11
thingeeDuncanT, scottda so from the client's perspective, not really a lot I can go off of to handle this case.19:12
DuncanT@thingee Agreed19:13
thingeeDuncanT, 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 bit19:13
thingeethe only other option is to setup middleware, which isn't any better.19:13
*** IlyaG has joined #openstack-cinder19:13
*** merooney has quit IRC19:14
thingeejamielennox|away, morganfainberg ^19:14
*** rwsu has quit IRC19:15
*** MentalRay has joined #openstack-cinder19:15
*** MRay has quit IRC19:16
thingeeluckily 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 yet19:16
*** agentle_ has quit IRC19:17
openstackgerritJon Bernard proposed openstack/os-brick: Add RBD connector  https://review.openstack.org/18617219:17
*** agentle has joined #openstack-cinder19:18
*** merooney has joined #openstack-cinder19:18
*** afazekas has quit IRC19:19
thingeeDuncanT: could also have default behavior if connection failures =/19:20
thingeeDuncanT: just to make that block even more complex19:20
thingeeI'll see if jamielennox|away has any ideas...but I think we might have to just revert.19:21
*** jasondot_ has joined #openstack-cinder19:24
*** dtynan1 has quit IRC19:25
*** dtynan has joined #openstack-cinder19:26
*** rushil has joined #openstack-cinder19:26
*** agentle has quit IRC19:27
*** annashen has quit IRC19:29
*** ociuhandu has quit IRC19:29
vilobhmmDuncanT, 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/g19:31
vilobhmm%s/values/value/g19:31
vilobhmmhttps://github.com/openstack/cinder/blob/master/cinder/quota.py#L49519:31
*** annashen has joined #openstack-cinder19:31
*** annashen has quit IRC19:32
DuncanTthingee: Timeout is so slow I don't think that would make things much better19:32
*** annashen has joined #openstack-cinder19:32
thingeeDuncanT: I was kidding on that solution. If I had to resort to that I would just revert19:32
DuncanTvilobhmm: I think -1 is unlimited, yeah19:33
vilobhmmdo we really need it19:33
vilobhmmor let the user update it depending on there deployments19:33
vilobhmmwhy can't we reset it to 0 ?19:33
DuncanTvilobhmm: 0 is not unlimitted though19:33
vilobhmmhmm19:34
DuncanTIt is 019:34
vilobhmm:P19:34
vilobhmmalrite19:34
morganfainbergYeah. jamielennox|away is the person I'd rely on first in this case.19:34
DuncanTThat's quite a behaviour change19:34
vilobhmmwill keep it the way it is…was needing this info incontext of setting default values for nested projects19:34
*** ronis has joined #openstack-cinder19:34
vilobhmmthanks DuncanT19:34
*** e0ne is now known as e0ne_19:35
*** e0ne_ is now known as e0ne19:35
DuncanTvilobhmm: NP19:36
DuncanTthingee: Are other clients using the same version discovery?19:37
*** Longgeek has joined #openstack-cinder19:37
*** avishay_ has quit IRC19:38
*** rwsu has joined #openstack-cinder19:38
*** avishay_ has joined #openstack-cinder19:38
*** rwsu has quit IRC19:40
*** rwsu has joined #openstack-cinder19:40
thingeeDuncanT: glance19:41
DuncanTthingee: Thanks19:42
*** dustins_ has joined #openstack-cinder19:42
*** dustins has quit IRC19:46
*** mgarza has joined #openstack-cinder19:47
*** merooney has quit IRC19:48
*** avishay_ has quit IRC19:52
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/19397719:52
*** agentle has joined #openstack-cinder19:52
*** annashen has quit IRC19:54
*** Lee1092 has quit IRC19:55
*** afazekas has joined #openstack-cinder19:57
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/19399619:59
*** timcl1 has left #openstack-cinder20:01
*** avishay_ has joined #openstack-cinder20:02
*** agentle has quit IRC20:02
*** MentalRay has quit IRC20:04
*** edmondsw has quit IRC20:04
*** MentalRay has joined #openstack-cinder20:06
*** markstur has joined #openstack-cinder20:07
*** avishay_ has quit IRC20:09
*** rwsu has quit IRC20:10
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/19397720:12
openstackgerritSergey Vilgelm proposed openstack/cinder: Switch to oslo.service  https://review.openstack.org/19395120:14
*** merooney has joined #openstack-cinder20:16
*** ronis has quit IRC20:17
*** anshul has quit IRC20:18
*** afazekas has quit IRC20:19
*** akerr has quit IRC20:21
*** belmoreira has quit IRC20:21
*** afazekas has joined #openstack-cinder20:21
*** rushil has quit IRC20:23
*** mriedem_afk is now known as mriedem20:24
*** amoturi has quit IRC20:27
*** rushil has joined #openstack-cinder20:28
*** Anticime1 is now known as Anticimex20:29
*** aarefiev_home has quit IRC20:30
e0nejungleboyj, smcginnis:  hi! thanks for review and attempt to merge my patch. but it depends on https://review.openstack.org/#/c/193861/20:31
jungleboyjRuh roh.  What did I do?20:32
*** dustins_ has quit IRC20:32
e0nejungleboyj: this one https://review.openstack.org/#/c/193828/ has 'Depends On'(parent) patch20:33
jungleboyjOops, didn't notice that.20:33
*** jasondot_ has quit IRC20:33
jungleboyjWell, lets take a look at that one.20:34
*** Rockyg has quit IRC20:34
e0nejungleboyj: np. it's not comfusing like -2 on parent and +2 on child patch:)20:35
jungleboyje0ne: 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
mtaninojungleboyj: nice. Wating the fix will be landed.20:36
e0nejungleboyj: thanks20:36
e0netime to sleep a bit:). bb in ~12 hours20:36
mtaninogood night :)20:36
*** marcusvrn has quit IRC20:41
*** lpetrut has quit IRC20:44
*** hodos|2 has joined #openstack-cinder20:46
openstackgerritSean McGinnis proposed openstack/cinder: Dell SC: Enable use of Storage Profiles  https://review.openstack.org/19333420:49
*** afazekas has quit IRC20:50
jungleboyje0ne: smcginnis The dependency looks ok to me.20:51
smcginnisjungleboyj: I'll take a look.20:53
*** annashen has joined #openstack-cinder20:54
ericksonfgdsvilobhmm, 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-cinder20:56
ericksonfgdsvilobhmm, otherwise, it won't be consistent, I guess.20:56
*** rajen has joined #openstack-cinder20:57
*** afazekas has joined #openstack-cinder20:58
*** jasondotstar has joined #openstack-cinder20:59
*** annashen has quit IRC21:00
*** marcusvrn_ has joined #openstack-cinder21:00
*** rushil has quit IRC21:01
*** e0ne has quit IRC21:03
scottdathingee: Do you know why setting of 'insecure' was removed from the client in this patch: https://review.openstack.org/#/c/187295/221:04
scottdaThe commit message is all about Hacking and PBR21:05
*** rushil has joined #openstack-cinder21:05
*** aix has joined #openstack-cinder21:05
*** e0ne has joined #openstack-cinder21:06
scottdaor jgriffith ^^21:06
scottda\21:06
*** e0ne has quit IRC21:09
*** rwsu has joined #openstack-cinder21:10
thingeescottda: looks like a mistake. feel free to post a patch21:11
scottdaok21:12
*** rushil has quit IRC21:14
*** annashen has joined #openstack-cinder21:15
*** Yogi1 has joined #openstack-cinder21:16
*** Yogi11 has joined #openstack-cinder21:20
*** Yogi1 has quit IRC21:21
*** IlyaG has quit IRC21:22
*** sgotliv has quit IRC21:22
*** merooney has quit IRC21:23
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/19397721:24
*** flwang has joined #openstack-cinder21:27
*** jasondotstar has quit IRC21:27
*** ociuhandu has joined #openstack-cinder21:27
*** afazekas has quit IRC21:28
*** kjnelson has quit IRC21:30
openstackgerritSergey Vilgelm proposed openstack/cinder: Switch to oslo.service  https://review.openstack.org/19395121:31
*** eharney has quit IRC21:32
scottdahmmm..it is assigned but never used, so causes a flak8 error. Perhaps it was removed for that.21:34
*** jasondot_ has joined #openstack-cinder21:37
*** thangp has quit IRC21:38
flwangvilobhmm:21:41
flwangping21:41
vilobhmmflwang : ack21:43
*** zigo has quit IRC21:43
*** MentalRay has quit IRC21:43
*** Longgeek has quit IRC21:43
*** brad[] has quit IRC21:43
*** IlyaG has joined #openstack-cinder21:47
*** zigo has joined #openstack-cinder21:49
flwangvilobhmm: hi, cool, are you working on this https://blueprints.launchpad.net/glance/+spec/glance-quota-enhancements ?21:52
*** jbernard has quit IRC21:52
openstackgerritVilobh Meshram proposed openstack/cinder: Nested Quota Driver : Set default values to subproject  https://review.openstack.org/19440621:52
vilobhmmright now there is no decision on this yet…hopefully they should have a decision in upcoming weekly meetings21:52
vilobhmmflwang : ^^21:53
flwangwho are they? BTW, i'm core member of glance team21:53
*** lan has quit IRC21:54
vilobhmmflwang : thats cool…."they" glance core team21:54
*** sparr has quit IRC21:54
flwangvilobhmm: i don't really understand why it even needs a discussion :)21:54
*** gouthamr has quit IRC21:55
flwangcurrent quota implement of glance is lack of flexible21:55
flwangwe need to refactor/change it to support configure quota by rest api instead of a static conf file21:55
vilobhmmok..and that blueprint says the same :)…lets talk on glance channel if you dont mind21:55
flwangsure21:55
*** ianbrown has joined #openstack-cinder21:56
*** mtanino has quit IRC21:57
*** jbernard has joined #openstack-cinder21:58
*** IlyaG has quit IRC21:58
*** jwcroppe_ has joined #openstack-cinder21:59
*** MentalRay has joined #openstack-cinder22:00
*** melwitt has joined #openstack-cinder22:00
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup  https://review.openstack.org/19393722:01
*** jwcroppe has quit IRC22:01
*** sparr has joined #openstack-cinder22:01
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup  https://review.openstack.org/19393722:03
*** jwcroppe_ has quit IRC22:03
*** mgarza has quit IRC22:04
*** jwcroppe has joined #openstack-cinder22:04
*** git-harry has quit IRC22:04
*** andymccr has quit IRC22:05
*** agentle has joined #openstack-cinder22:06
*** lan has joined #openstack-cinder22:06
*** git-harry has joined #openstack-cinder22:06
*** andymccr has joined #openstack-cinder22:07
*** afazekas has joined #openstack-cinder22:08
*** jwcroppe_ has joined #openstack-cinder22:10
*** Longgeek has joined #openstack-cinder22:12
*** jasondot_ has quit IRC22:12
*** jwcroppe has quit IRC22:13
*** rushiagr_away has quit IRC22:13
*** afazekas has quit IRC22:13
openstackgerritxing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver  https://review.openstack.org/18583522:13
*** zigo has quit IRC22:15
jungleboyjgeguileo: You around?22:17
*** zigo has joined #openstack-cinder22:17
*** primechuck has quit IRC22:22
openstackgerritAnish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm.  https://review.openstack.org/19345122:25
anishhemna: no tests yet, but can you give it a once over ^ ?22:26
*** rushiagr_away has joined #openstack-cinder22:28
*** chlong has quit IRC22:29
*** IlyaG has joined #openstack-cinder22:30
hemnaanish, sure22:35
hemnaanish, so this replaces the need for the ISERConnector ?22:37
hemnathe iser device paths had ip-%s-iser-.... in them22:37
openstackgerritEric Brown proposed openstack/cinder: Add bandit for security static analysis testing  https://review.openstack.org/17956822:38
*** aix has quit IRC22:41
*** aix has joined #openstack-cinder22:41
*** salv-orlando has quit IRC22:42
*** jasondotstar has joined #openstack-cinder22:43
hemnaanish, so the current ISERConnector has _get_device_path which looks for something in the form of ip-%s-iser-%s22:45
hemnayour new patch doesn't look for a device with that base22:46
*** agentle has quit IRC22:50
*** agentle has joined #openstack-cinder22:51
*** daneyon has quit IRC22:52
*** daneyon has joined #openstack-cinder22:53
*** jecarey has quit IRC22:53
*** hodos|2 has quit IRC22:54
anishhemna: I don't know if you remember this22:55
anishbut I'd bought this point up earlier22:55
anishiser in os_brick is very broken22:56
openstackgerritAnish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm.  https://review.openstack.org/19345122:56
hemnaanish, heh ok.  I've never used ISER22:56
anishso if you look at the original commit : https://review.openstack.org/#/c/146233/22/nova/virt/libvirt/volume.py22:57
*** DericHorn-HP_ has joined #openstack-cinder22:57
anishyou can see that the _get_host_device for iser was replaced22:57
anishthe iser target is no longer using the separate 'iser' is the dev/by-path22:57
anishthis part was actually done by the mellanox guys who work with iser, so should not be a problem22:58
*** dannywil_ has quit IRC22:58
anishhemna: I'm trying to get a review from them as well, so will be tested by them I hope22:59
hemnaok23:00
anishbut yes, the iser connector does get deprecated23:00
*** rwsu has quit IRC23:00
hemnaok I'll look at it a bit more tomorrow23:00
anishdoes it need to be separate when there is really no code difference needed ?23:00
anishaite, sounds good23:00
hemnaI'm trying to track down a nova -> os-brick issue I'm seeing with FC23:01
anishwokay23:01
*** rajen has quit IRC23:03
*** DericHorn-HP_ is now known as DericHorn-HP23:04
*** jdurgin has quit IRC23:04
*** ndipanov has quit IRC23:13
*** marcusvrn_ has quit IRC23:14
*** markvoelker has quit IRC23:16
openstackgerritJay Bryant proposed openstack/cinder: Add exception catch in report_state for DBError  https://review.openstack.org/19423223:17
*** DericHorn-HP has quit IRC23:21
*** jungleboyj has quit IRC23:24
*** DericHorn-HP has joined #openstack-cinder23:27
*** melwitt has quit IRC23:27
*** jasondotstar has quit IRC23:29
*** DericHorn-HP has quit IRC23:30
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087323:30
*** annashen has quit IRC23:31
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087323:32
*** MentalRay has quit IRC23:34
*** annashen has joined #openstack-cinder23:34
openstackgerritVilobh Meshram proposed openstack/cinder: Nested Quota Driver : Set default values to subproject  https://review.openstack.org/19440623:34
*** vilobhmm has quit IRC23:36
*** vilobhmm has joined #openstack-cinder23:36
*** angela-s has quit IRC23:38
*** chlong has joined #openstack-cinder23:40
*** Longgeek has quit IRC23:43
*** gouthamr has joined #openstack-cinder23:44
*** agentle has quit IRC23:45
*** jaypipes has quit IRC23:46
*** annashen has quit IRC23:46
openstackgerritAnish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm.  https://review.openstack.org/19345123:48
*** nikeshm has quit IRC23:49
openstackgerritAnish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm.  https://review.openstack.org/19345123:49
*** jasondotstar has joined #openstack-cinder23:54
*** vilobhmm has quit IRC23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!