Monday, 2015-11-16

*** stevemar_ has joined #openstack-cinder00:00
*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-cinder00:01
*** zhangjn has joined #openstack-cinder00:02
*** smoriya has joined #openstack-cinder00:02
*** jerrygb has quit IRC00:18
*** ircuser-1 has quit IRC00:23
*** mriedem has quit IRC00:30
*** EinstCrazy has quit IRC00:30
*** ndipanov has joined #openstack-cinder00:36
*** apoorvad has joined #openstack-cinder00:37
*** stevemar_ has quit IRC00:42
*** stevemar_ has joined #openstack-cinder00:44
*** stevemar_ has quit IRC00:49
*** haomaiwang has quit IRC01:01
*** haomaiwang has joined #openstack-cinder01:01
*** markvoelker has joined #openstack-cinder01:06
*** crose has quit IRC01:07
*** EinstCrazy has joined #openstack-cinder01:08
*** tbarron_ has quit IRC01:09
*** markvoelker has quit IRC01:10
*** tbarron_ has joined #openstack-cinder01:12
*** [1]Thelo has joined #openstack-cinder01:24
*** Thelo has quit IRC01:26
*** [1]Thelo is now known as Thelo01:26
*** haomaiwang has quit IRC01:28
*** bapalm has quit IRC01:29
*** vaidy has quit IRC01:32
*** serverascode has quit IRC01:32
*** tbarron has quit IRC01:33
*** sweston has quit IRC01:33
*** erlon has quit IRC01:33
*** ctracey has quit IRC01:33
*** mgagne has quit IRC01:33
*** swamireddy has quit IRC01:34
*** jamielennox has quit IRC01:34
*** bswartz has quit IRC01:35
*** sparr has quit IRC01:35
*** zhenguo has quit IRC01:36
*** erlon has joined #openstack-cinder01:37
*** ctracey has joined #openstack-cinder01:37
*** serverascode has joined #openstack-cinder01:37
*** tbarron has joined #openstack-cinder01:37
*** mgagne has joined #openstack-cinder01:38
*** mgagne is now known as Guest9104301:38
*** bswartz has joined #openstack-cinder01:39
*** jerrygb has joined #openstack-cinder01:39
*** sparr has joined #openstack-cinder01:40
*** sweston has joined #openstack-cinder01:40
*** jamielennox has joined #openstack-cinder01:41
*** EinstCrazy has quit IRC01:43
*** zhenguo has joined #openstack-cinder01:43
*** vaidy_ril has joined #openstack-cinder01:44
*** swamireddy has joined #openstack-cinder01:45
*** EinstCrazy has joined #openstack-cinder01:45
*** Lee1092 has joined #openstack-cinder01:50
*** EinstCrazy has quit IRC01:55
*** bapalm has joined #openstack-cinder01:56
*** EinstCrazy has joined #openstack-cinder01:56
*** lixiaoy1 has joined #openstack-cinder02:01
*** tbarron_ has quit IRC02:03
*** jerrygb has quit IRC02:04
*** bapalm has quit IRC02:04
*** dims has joined #openstack-cinder02:06
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243602:09
*** tbarron_ has joined #openstack-cinder02:17
*** dave-mccowan has joined #openstack-cinder02:21
*** jerrygb has joined #openstack-cinder02:26
*** bill_az has quit IRC02:27
*** amit213 has quit IRC02:27
*** amit213 has joined #openstack-cinder02:27
*** tbarron_ has quit IRC02:34
*** apoorvad has quit IRC02:36
*** haomaiwang has joined #openstack-cinder02:45
*** haomaiwang has quit IRC02:48
*** 20WACHN4K has joined #openstack-cinder02:49
*** crose has joined #openstack-cinder02:51
*** 20WACHN4K has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** gouthamr has quit IRC03:02
*** markvoelker has joined #openstack-cinder03:07
*** lixiaoy1_ has joined #openstack-cinder03:11
*** markvoelker has quit IRC03:11
*** crose has quit IRC03:14
*** lixiaoy1_ has quit IRC03:14
openstackgerritDeepti Ramakrishna proposed openstack/cinder: Consistency group API now returns volume type IDs  https://review.openstack.org/24562403:21
*** gcb has joined #openstack-cinder03:25
openstackgerritDeepti Ramakrishna proposed openstack/cinder: Consistency group API now returns volume type IDs  https://review.openstack.org/24562403:31
*** apoorvad has joined #openstack-cinder03:36
*** ircuser-1 has joined #openstack-cinder03:40
*** apoorvad has quit IRC03:41
*** dims has quit IRC03:47
*** dims has joined #openstack-cinder03:48
*** haomaiwang has quit IRC03:51
*** haomaiwang has joined #openstack-cinder03:51
*** crose has joined #openstack-cinder04:00
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** dims has quit IRC04:08
*** links has joined #openstack-cinder04:11
*** daneyon has joined #openstack-cinder04:21
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (managed)  https://review.openstack.org/23738704:25
openstackgerritxing-yang proposed openstack/python-cinderclient: CLI for backup snapshots  https://review.openstack.org/24563204:38
*** lixiaoy1 has quit IRC04:42
*** daneyon has quit IRC04:59
*** lixiaoy1 has joined #openstack-cinder04:59
*** pradipta has joined #openstack-cinder04:59
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** itzdilip has joined #openstack-cinder05:04
*** lixiaoy1 has left #openstack-cinder05:06
*** lixiaoy1_ has joined #openstack-cinder05:06
openstackgerritYusuke Hayashi proposed openstack/cinder: Add argment for efficient sparse image transfer  https://review.openstack.org/24563505:08
*** sileht has quit IRC05:12
*** itzdilip has quit IRC05:14
*** crose has quit IRC05:18
*** yusuke has quit IRC05:23
openstackgerritDeepti Ramakrishna proposed openstack/cinder: CG API should return volume type IDs  https://review.openstack.org/24562405:28
*** yusuke has joined #openstack-cinder05:30
*** houming-wang has joined #openstack-cinder05:38
*** dave-mccowan has quit IRC05:39
openstackgerritFelix Ma proposed openstack/cinder: Use _get_driver as utils.require_driver_initialized parameter in create_backup and restore_backup  https://review.openstack.org/24538405:44
*** zhangjn has quit IRC05:52
*** zhangjn has joined #openstack-cinder05:53
*** zhangjn has quit IRC05:53
*** zhangjn has joined #openstack-cinder05:54
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
*** houming-wang has quit IRC06:01
*** stevemar_ has joined #openstack-cinder06:11
*** anshul has joined #openstack-cinder06:11
*** lpetrut has joined #openstack-cinder06:12
*** stevemar_ has quit IRC06:14
*** ravit has joined #openstack-cinder06:15
*** houming-wang has joined #openstack-cinder06:16
*** rcernin has joined #openstack-cinder06:16
*** lpetrut has quit IRC06:17
*** shakamunyi has joined #openstack-cinder06:21
*** jamielennox is now known as jamielennox|away06:25
*** asselin has joined #openstack-cinder06:27
*** kambiz has quit IRC06:27
*** dhellmann has quit IRC06:28
*** dhellmann has joined #openstack-cinder06:28
*** asselin__ has quit IRC06:29
*** lpetrut has joined #openstack-cinder06:29
*** kambiz has joined #openstack-cinder06:30
*** asselin__ has joined #openstack-cinder06:30
*** asselin__ has quit IRC06:30
*** asselin__ has joined #openstack-cinder06:31
*** dhellmann has quit IRC06:34
*** dhellmann has joined #openstack-cinder06:34
*** asselin has quit IRC06:34
*** shakamunyi has quit IRC06:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/24564606:40
*** shakamunyi has joined #openstack-cinder06:54
*** vignesh1 has joined #openstack-cinder06:54
*** haomaiwang has quit IRC07:01
*** flip214_ is now known as flip21407:01
*** haomaiwang has joined #openstack-cinder07:01
*** jerrygb has quit IRC07:03
*** apoorvad has joined #openstack-cinder07:07
*** markvoelker has joined #openstack-cinder07:08
*** flip214 has quit IRC07:10
*** flip214 has joined #openstack-cinder07:10
*** sileht has joined #openstack-cinder07:10
*** anish__ has joined #openstack-cinder07:11
*** SamYaple_ has joined #openstack-cinder07:11
*** markvoelker has quit IRC07:13
*** anish has quit IRC07:15
*** zhenguo has quit IRC07:15
*** sweston has quit IRC07:15
*** scottda has quit IRC07:15
*** SamYaple has quit IRC07:15
*** lpetrut has quit IRC07:19
*** sgotliv_ has quit IRC07:19
*** zhenguo has joined #openstack-cinder07:19
*** scottda has joined #openstack-cinder07:19
*** sweston has joined #openstack-cinder07:20
*** martyturner has joined #openstack-cinder07:22
lixiaoy1_xyang: ping07:22
*** boris-42 has joined #openstack-cinder07:23
*** martyturner1 has joined #openstack-cinder07:23
*** lpetrut has joined #openstack-cinder07:24
*** martyturner has quit IRC07:26
*** nkrinner has joined #openstack-cinder07:33
*** ZZelle has joined #openstack-cinder07:35
*** lpetrut has quit IRC07:35
*** ZZelle has quit IRC07:36
*** alexschm has joined #openstack-cinder07:37
openstackgerritYusuke Hayashi proposed openstack/cinder: Fix comment typo in driver.py  https://review.openstack.org/24565107:38
*** belmoreira has joined #openstack-cinder07:44
openstackgerritwanghao proposed openstack/cinder-specs: Improvement to query consistency group detail  https://review.openstack.org/24299807:45
*** anish__ is now known as anish07:45
*** liverpooler has joined #openstack-cinder07:46
*** martyturner1 has quit IRC07:48
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (managed)  https://review.openstack.org/23738707:49
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** martyturner has joined #openstack-cinder08:01
*** e0ne has joined #openstack-cinder08:04
*** nkrinner has quit IRC08:10
*** setmason has quit IRC08:11
openstackgerritYusuke Hayashi proposed openstack/cinder: Fix comment typo in driver.py and doc  https://review.openstack.org/24565108:11
*** smoriya_ has joined #openstack-cinder08:13
*** nkrinner has joined #openstack-cinder08:15
*** apoorvad has quit IRC08:18
openstackgerritMerged openstack/cinder: 3PAR Fix find_existing_vluns  https://review.openstack.org/24385608:19
*** lpetrut has joined #openstack-cinder08:24
*** ankit_ag has joined #openstack-cinder08:32
*** tobe has joined #openstack-cinder08:34
*** ndipanov has quit IRC08:41
*** itzdilip has joined #openstack-cinder08:41
*** ArneWiebalck_ has joined #openstack-cinder08:46
*** itzdilip has quit IRC08:47
*** martyturner has quit IRC08:49
*** ArneWiebalck_ has left #openstack-cinder08:51
*** jordanP has joined #openstack-cinder08:51
*** ArneWiebalck_ has joined #openstack-cinder08:53
*** ArneWiebalck_ has left #openstack-cinder08:54
*** Arne has joined #openstack-cinder08:55
*** Arne has left #openstack-cinder08:56
*** wiebalck has joined #openstack-cinder08:58
*** wiebalck has left #openstack-cinder08:58
openstackgerritVictor Stinner proposed openstack/cinder: Port HP 3PAR driver to Python 3  https://review.openstack.org/24212808:59
openstackgerritVictor Stinner proposed openstack/cinder: Modify test_hpe3par to support random hash  https://review.openstack.org/24212708:59
*** haomaiwang has quit IRC09:01
*** haomaiwang has joined #openstack-cinder09:01
*** wiebalck has joined #openstack-cinder09:03
*** jerrygb has joined #openstack-cinder09:04
openstackgerritShay Halsband proposed openstack/cinder: XtremIO add support for create CG from CG src  https://review.openstack.org/24554809:05
*** sgotliv_ has joined #openstack-cinder09:06
*** yusuke has quit IRC09:06
*** jerrygb has quit IRC09:08
*** markvoelker has joined #openstack-cinder09:09
dulekDuncanT: Hi, I have a question on https://review.openstack.org/#/c/245572/09:10
*** lpetrut has quit IRC09:10
dulekDuncanT: You mention to use service catalog there. The problem I see is that I have only keystone v2 in the catalog and this method requires v3.09:11
dulekDuncanT: I wonder what's the policy here. Are we requiring Keystone V3 API explicitly? Because that's how this method was written.09:11
dulekDuncanT: There's no "project" in V2, just "tenant".09:11
DuncanTdulek: The nested quotas stuff does require V3 I believe. Not all deployers have it though, so we need a fallback path (probably no nested quotas withotu V3)09:13
*** markvoelker has quit IRC09:13
DuncanTdulek: maybe do version discovery against keystone in the catalog at startup? I don't think V3 will ever explicitly appear in the catalog now - keystone want to get rid of versions from the catalog (thought here is pushback against that)09:14
DuncanTdulek: I'm fine with adding the option to have a hard-coded URL for weird deployers, but I'd like to find the 'it just works' option for most people09:15
dulekDuncanT: Hm… So from my perspective "just works" version would be discovery from http://ip:5000 - let Keystone tell us what versions it supports.09:16
*** lpetrut has joined #openstack-cinder09:17
dulekDuncanT: This should be fine but I'm having problems with getting version discovery in keystoneclient to work…09:17
DuncanTdulek: That sounds like the right solution (except the port can change, so use the port for keystone in the catalog)09:17
DuncanTdulek: I can't help with version discovery, I've not looked at it - have you tried asking for help in the keystone channel?09:18
dulekDuncanT: I was thinking of using values from CONF.keystone_authtoken09:18
dulekDuncanT: Yeah, of course, no luck for now. ;)09:18
dulekDuncanT: Also with this nested-quotas code we've actually dropped support for non-v3 keystone in Liberty. I wonder if that was done by mistake or not.09:19
*** lpetrut has quit IRC09:19
DuncanTActually, the swift backup code already has :keystone_catalog_info:09:20
DuncanTdulek: By mistake I assume. I thought there was fallback code in there09:20
dulekDuncanT: Well, it's failing for me when i configure v2.09:21
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova  https://review.openstack.org/22412409:21
*** e0ne has quit IRC09:21
dulekDuncanT: And this keystone_catalog_info makes it even more strange. ;)09:21
*** e0ne has joined #openstack-cinder09:21
DuncanTdulek: I'd consider that a bug. Can you file it please, see if we can get confirmation it wasn't intended?09:21
dulekDuncanT: Yup, you're right, I'll try to confirm it again and file a bug.09:22
*** xek_ is now known as xek09:22
DuncanTThanks09:25
dulekDuncanT: BTW - we're not having keystone_catalog_info described in config reference. All this stuff is so messy…09:26
DuncanTdulek: :-(09:29
*** lpetrut has joined #openstack-cinder09:30
*** ravit has quit IRC09:37
*** ravit has joined #openstack-cinder09:38
*** jistr has joined #openstack-cinder09:40
*** ociuhandu has quit IRC09:40
*** yuriy_n17 has joined #openstack-cinder09:42
*** tobe has quit IRC09:44
*** tobe has joined #openstack-cinder09:45
*** houming-wang has quit IRC09:50
*** gcb has quit IRC09:51
*** subscope has joined #openstack-cinder09:52
*** alexschm has quit IRC09:57
*** zhenguo has quit IRC09:58
*** alexschm has joined #openstack-cinder09:59
*** zhangjn has quit IRC09:59
*** haomaiwang has quit IRC10:01
flip214is there some *detailed* documentation what the flow is for "nova volume-attach"? my cinder driver parts don't seem to get triggered, it's as if the request just vanishes somewhere.10:01
*** haomaiwang has joined #openstack-cinder10:01
*** lpetrut has quit IRC10:01
flip214no errors in the logs....10:01
*** pradipta has quit IRC10:02
johnthetubaguyflip214: I don't know of any10:02
johnthetubaguyflip214: the API sends an RPC call to the nova-compute node, which then does the rest, if that helps10:02
johnthetubaguythe logs will have a request-id in the message, which you can use to track things through all the different services10:03
flip214johnthetubaguy: well, that's a 3-node devstack setup, trying to get a new transport working...10:03
flip214so it's not as easy as "grep" on a single node.10:03
flip214but thanks, I'll try to find out more.10:04
*** alexschm has quit IRC10:05
johnthetubaguyflip214: go to the specific nova-compute node for the VM10:05
DuncanTI think jgriffith might have blogged about it, maybe?10:07
flip214johnthetubaguy: I am. the n-cpu service just repeats the same 20 lines every few minutes; n-api-meta is not being used; n-net just talks about "FlatDHCPManager._disassociate_stale_fixed_ips"10:07
flip214DuncanT: the page https://griffithscorner.wordpress.com/2015/07/16/volume-attach-code-flow-in-cinder/10:07
DuncanTflip214: That's the one10:08
johnthetubaguyDuncanT: nice10:08
flip214DuncanT: well, the "boot" volume gets attached normally10:08
flip214just additional ones won't work for some reason10:08
johnthetubaguyflip214: the n-api node should have seen the request and sent an RPC cast to the compute node, if you want to see the step before10:09
flip214DuncanT: that blog post just talks about cinder. no explanation about any other services....10:09
DuncanTflip214: From the nova side, assuming you've got verbose and debug turned on in the conf, you'll see an entry in nova-api and one in nova-compute10:09
DuncanTflip214: If you don't see those, it isn't your driver, since they should both happen before your driver is touched, unless there's an init problem with your driver maybe10:09
*** lpetrut has joined #openstack-cinder10:10
*** deepakcs has joined #openstack-cinder10:13
flip214DuncanT: the driver works for _for_the_initial_volume_ .... that's why I'm at a loss here.10:13
flip214but thanks, anyway!10:13
flip214I'll just have to dig through the logs...10:13
DuncanTflip214:  The initial volume is a different code flow10:13
DuncanTflip214: Have you tried adding a small LVM backend, and making sure that works? Just make it a different volume type, that should enable you to trace a 'normal' flow throught he logs10:14
flip214that's an idea, yes.10:14
*** openstackgerrit has quit IRC10:16
DuncanTflip214: Last time I added a transport, it was in the nova-volume days, I haven't looked at that bit of nova in a long while10:17
*** openstackgerrit has joined #openstack-cinder10:17
*** vignesh2 has joined #openstack-cinder10:17
*** esker has joined #openstack-cinder10:18
*** alexschm has joined #openstack-cinder10:18
*** vignesh1 has quit IRC10:20
*** alexschm has quit IRC10:24
flip214understood ;/10:25
*** ociuhandu has joined #openstack-cinder10:26
*** esker has quit IRC10:28
*** vignesh1 has joined #openstack-cinder10:30
*** ociuhandu has quit IRC10:31
*** vignesh2 has quit IRC10:32
*** ociuhandu has joined #openstack-cinder10:34
*** tobe has quit IRC10:36
*** lpetrut has quit IRC10:42
*** eduardo_ has joined #openstack-cinder10:50
*** smoriya_ is now known as zz_smoriya_10:51
*** subscope has quit IRC10:51
*** subscope has joined #openstack-cinder10:53
*** lpetrut has joined #openstack-cinder10:59
*** haomaiwang has quit IRC11:01
*** alexschm has joined #openstack-cinder11:01
*** haomaiwang has joined #openstack-cinder11:01
*** DaveJ_ has joined #openstack-cinder11:09
DaveJ_Hi guys, I'm starting to work on  a few bugs in python-cinderclient11:09
*** markvoelker has joined #openstack-cinder11:10
DaveJ_Can anyone point me to a guide on getting started, in terms of the dev workflow.  Is there a standard procedure to build and test ?11:10
e0neDaveJ_: hi. did you see this https://wiki.openstack.org/wiki/How_To_Contribute guide?11:12
*** stevemar_ has joined #openstack-cinder11:12
*** dims has joined #openstack-cinder11:12
smcginnise0ne: +111:13
*** markvoelker has quit IRC11:14
DaveJ_eOne: Thanks, I did get a look at that11:15
*** stevemar_ has quit IRC11:15
DaveJ_eOne, although I'm now looking for more specifics.  For example I see tox seems to be used in alot of the projects, is that the way to go?11:16
DaveJ_and if so, do I need to test each of the environments before submitting a fix ?11:16
smcginnisDaveJ_: It would be best practice to run tox before submitting your patch.11:17
smcginnisDaveJ_: Jenkins will end up running them (and others) as well, but you can catch issues locally before pushing up a review.11:17
DaveJ_smcginnis: Thanks, is there any guidelines on using it? I'm not overly familair, if it's just standard usage, I'm sure I can figure it out, but is there project specific targets that I should be aware off?11:18
smcginnisDaveJ_: There is a tox.ini file in the root that defines what is run. But for the most part, all you need to do is write unit tests and they will be run.11:19
DaveJ_smcginnis: Thanks.  One last question11:19
DaveJ_in the case of 'python-cinderclient'  how do I build/package it, so that I can test it out manually?  I'm assuming most pepole setup there dev environment in a way that makes this easy ?11:20
smcginnisDaveJ_: I think a common method is to install the client, then just make the site-packes directory for it be a symbolic link to your repo.11:21
smcginnisThen as you make code changes you can just run the command line to try it out.11:21
*** links has quit IRC11:21
smcginnisI'm not aware of any documentation on setting that up, so hopefully that description makes some sense to you. :)11:21
smcginnisMight not be a bad "recommended practice" guide for us to write up and include somewhere.11:22
DaveJ_smcginnis: It does make sense., its the approach I'm use to with other projects.   I'll document anything I figure out as I go.11:24
DaveJ_In terms of intalling it, I assume it's the usual 'python setup.py install'  but is there any trick to setting up symlinks back to my repo?11:24
*** [5]Thelo has joined #openstack-cinder11:24
*** Thelo has quit IRC11:27
*** [5]Thelo is now known as Thelo11:27
*** gcb has joined #openstack-cinder11:27
smcginnisDaveJ_: Sorry, at a conference and network is bad.11:32
smcginnisYes, you should be able to just run setup.py install.11:32
DaveJ_smcginnis: Thanks - I'll play around with it and see how I get on.  I think thats enough info, to let me know I'm doing something completely stupid :-)11:33
smcginnisDaveJ_: Good luck! :)11:33
e0nesmcginnis: if you'll have a time, please take a look on these tests https://review.openstack.org/#/c/158745/32/cinder/tests/unit/test_nexenta_edge.py11:36
e0nesmcginnis: IMO, they are not useful at alll but I could be wrong11:36
smcginnise0ne: Sure, I'll try to get to it soon.11:36
smcginnise0ne: Like a lot of our tests. :/11:37
e0nesmcginnis: fair enough11:37
smcginnise0ne: Maybe a pessimistic view. :)11:37
smcginnise0ne: But I think we have a lot of tests that aren't really testing anything, or not anything very useful.11:37
smcginnise0ne: But good to stop adding more that don't add value.11:38
e0nesmcginnis: almost everybody knows it but there is no time to fix these tests:(11:38
smcginnise0ne: True.11:38
smcginnisbbl11:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/24523711:44
*** xek has quit IRC11:47
*** links has joined #openstack-cinder11:55
*** links has quit IRC11:59
*** haomaiwang has quit IRC12:01
*** haomaiwa_ has joined #openstack-cinder12:01
*** jerrygb has joined #openstack-cinder12:04
*** houming-wang has joined #openstack-cinder12:08
*** jerrygb has quit IRC12:09
*** EinstCrazy has quit IRC12:14
*** sgotliv_ has quit IRC12:19
*** sgotliv_ has joined #openstack-cinder12:21
*** zhenguo has joined #openstack-cinder12:22
*** deepakcs has quit IRC12:29
*** EinstCrazy has joined #openstack-cinder12:38
*** ekarlso has quit IRC12:39
*** ekarlso has joined #openstack-cinder12:39
*** EinstCrazy has quit IRC12:39
*** EinstCrazy has joined #openstack-cinder12:39
*** markvoelker has joined #openstack-cinder12:40
*** markvoelker has quit IRC12:45
*** Yogi1 has joined #openstack-cinder12:45
*** alexschm has quit IRC12:46
*** openstack has joined #openstack-cinder12:49
*** jaypipes has joined #openstack-cinder12:51
openstackgerritEduard Biceri-Matei proposed openstack/cinder: Re-add the OpenvStorage Cinder driver  https://review.openstack.org/24117412:53
openstackgerritnandal proposed openstack/cinder: Use assertTrue/False(A) Methods  https://review.openstack.org/24348912:56
*** alexschm has joined #openstack-cinder12:59
*** Yogi1 has quit IRC13:00
*** haomaiwa_ has quit IRC13:01
*** haomaiwang has joined #openstack-cinder13:01
*** A-abc has quit IRC13:01
*** julim has joined #openstack-cinder13:06
*** dave-mccowan has joined #openstack-cinder13:08
*** markvoelker has joined #openstack-cinder13:13
openstackgerritJoel Coffman proposed openstack/cinder: Enforce volume type constraints in internal API  https://review.openstack.org/21900713:14
openstackgerritJoel Coffman proposed openstack/cinder: Clean up volume_type_update method  https://review.openstack.org/21382913:14
*** ravit has quit IRC13:16
*** ankit_ag has quit IRC13:17
*** edmondsw has joined #openstack-cinder13:28
*** subscope has quit IRC13:30
*** stevemar_ has joined #openstack-cinder13:33
openstackgerritFelix Ma proposed openstack/cinder: Fix utils.require_driver_initialized parameter issue in create_backup/restore_backup  https://review.openstack.org/24538413:35
*** alejandrito has joined #openstack-cinder13:36
*** timcl has joined #openstack-cinder13:37
*** diablo_rojo has joined #openstack-cinder13:39
*** diablo_rojo has quit IRC13:40
*** haomaiwang has quit IRC13:40
*** gouthamr has joined #openstack-cinder13:40
openstackgerritFelix Ma proposed openstack/cinder: Fix utils.require_driver_initialized issue in create_backup/restore_backup  https://review.openstack.org/24538413:42
*** bswartz has quit IRC13:45
*** subscope has joined #openstack-cinder13:46
*** boris-42 has quit IRC13:48
*** diablo_rojo has joined #openstack-cinder13:49
*** tpeoples has quit IRC13:52
*** ubuntu1 is now known as anteaya13:52
*** rushiagr_away has quit IRC13:52
*** tpeoples has joined #openstack-cinder13:52
*** Lee1092 has quit IRC13:53
*** ctracey has quit IRC13:53
*** zhiyan has quit IRC13:53
*** cppforlife_ has quit IRC13:53
*** DuncanT has quit IRC13:53
openstackgerritYusuke Hayashi proposed openstack/cinder: Add argment for efficient sparse image transfer  https://review.openstack.org/24563513:54
*** zhiyan has joined #openstack-cinder13:54
*** ctracey has joined #openstack-cinder13:55
*** cppforlife_ has joined #openstack-cinder13:55
*** Lee1092 has joined #openstack-cinder13:56
*** rushiagr_away has joined #openstack-cinder13:57
*** DuncanT has joined #openstack-cinder13:57
*** bill_az has joined #openstack-cinder13:59
*** subscope has quit IRC14:00
*** boris-42 has joined #openstack-cinder14:01
*** porrua has joined #openstack-cinder14:03
*** setmason has joined #openstack-cinder14:03
smcginnise0ne: So I looked at that patch.14:05
*** julim has quit IRC14:06
smcginnise0ne: And while I agree, I think Zohar's explanation on Nov 4th kind of makes sense.14:06
smcginnise0ne: I'm not really sure what more they can do given that it really does just pass through to the backend.14:07
smcginnisWhich isn't open, and I don't like that there is no way to verify that it is doing the right thing. :/14:07
smcginnise0ne: Any ideas?14:07
*** zhenguo has quit IRC14:08
*** dustins has joined #openstack-cinder14:08
*** ekarlso has quit IRC14:09
*** xyang1 has joined #openstack-cinder14:09
*** bswartz has joined #openstack-cinder14:14
*** haomaiwang has joined #openstack-cinder14:17
e0nesmcginnis: at least, we can check that all params are passed right to backend14:18
e0nesmcginnis: in current implementation tests like test_create_snapshot_fail will always pass14:18
*** merooney has joined #openstack-cinder14:20
smcginnise0ne: Oh, good point. That's at least something.14:24
smcginnise0ne: Can you put that recommendation in the review?14:24
e0nesmcginnis: done14:24
smcginnise0ne: I think they just need a little guidance.14:24
smcginnisThanks!14:24
e0nesmcginnis: ok, I'll  provide more detailes comments for the each of test14:25
*** akshai has joined #openstack-cinder14:26
openstackgerritzhangsong proposed openstack/cinder: Sheepdog: Optimization of error handling  https://review.openstack.org/24555214:28
*** jerrygb has joined #openstack-cinder14:29
*** jerrygb has quit IRC14:29
*** jerrygb has joined #openstack-cinder14:29
*** mriedem has joined #openstack-cinder14:30
*** cfriesen__ has joined #openstack-cinder14:31
*** ekarlso has joined #openstack-cinder14:33
*** esker has joined #openstack-cinder14:33
*** breitz has joined #openstack-cinder14:36
*** logan2 has quit IRC14:37
*** daemontool_ has joined #openstack-cinder14:38
*** esker has quit IRC14:38
openstackgerritFelix Ma proposed openstack/cinder: Check specific driver enabled in create_backup  https://review.openstack.org/24538414:39
*** resker has joined #openstack-cinder14:42
*** resker has quit IRC14:43
*** esker has joined #openstack-cinder14:43
*** bapalm has joined #openstack-cinder14:51
*** willsama has joined #openstack-cinder14:53
*** willsama has quit IRC14:53
*** bapalm has quit IRC14:53
*** esker has quit IRC14:54
*** setmason has quit IRC14:57
*** julim has joined #openstack-cinder14:59
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** lpabon has joined #openstack-cinder15:03
*** bapalm has joined #openstack-cinder15:04
openstackgerritAlon Marx proposed openstack/cinder: Remove db access from XIV/DS8K CG functions  https://review.openstack.org/24274415:05
openstackgerritMerged openstack/python-cinderclient: Update CONTRIBUTING.md to CONTRIBUTING.rst  https://review.openstack.org/24362115:08
*** mdenny has joined #openstack-cinder15:12
dulekxyang1: Hi! I'm writing the spec for live DB migrations. Can you elaborate on how you plan to fix volume_type_id issue? I would use that example to describe the idea.15:13
*** ntpttr has joined #openstack-cinder15:14
dulekxyang1: That way I would be able to use something that's real-world and you would be provided with complete solution described. :)15:14
xyang1dulek: so the live db migration will allow a column to be dropped15:14
xyang1dulek: sure:)15:14
dulekxyang1: Yes, but it spans through multiple releases. That's painful, but we can hide a lot of translation logic in versioned objects, so it shouldn't be that bad.15:15
dulekxyang1: As this is n-n relationship you're probably thinking of creating a new table for this relationships?15:16
*** crose has joined #openstack-cinder15:16
xyang1dulek: I am planning to add a table, VolumeTypeCGs, for example15:16
xyang1dulek: it should be similar to VolumeTypeProjects table15:17
*** SamYaple_ is now known as SamYaple15:17
xyang1dulek: I can put together a patch for you to review, maybe easier that way15:18
dulekxyang1: I don't think that a good idea yet.15:18
xyang1dulek: which is not a good idea, that table?15:19
dulekxyang1: No, no, writing it now. :)15:19
xyang1dulek: ok15:19
dulekxyang1: I'll just work on describing how to do that in a live way to exercise the solution Nova's using.15:19
xyang1dulek: sure, anyway, I can work on a WIP patch for you to look15:19
johnthetubaguydulek: do you notice Nova's recent change in direction around the online schema migrations?15:20
dulekxyang1: Dan Smith explained that in a very simple example. This one would be more complicated and I need to make sure that this actually work for Cinder.15:20
*** mtanino has joined #openstack-cinder15:20
johnthetubaguyah, cools, sounds like that message got through15:20
xyang1dulek: ok15:20
dulekjohnthetubaguy: Hm… Has it? ;) I'm basing on http://www.danplanet.com/blog/2015/10/07/upgrades-in-nova-database-migrations/15:21
johnthetubaguydulek: yeah, thats correct, dansmith has all the details there, there was an experimental method for the schema migrations, but we have actually dropped that from our code base now.15:22
dulekjohnthetubaguy: Okay, I'm mentioning it in the spec - it's good to know that it's gone. Thanks for awareness!15:23
dansmithjohnthetubaguy: yeah, I warned dulek about the online migration stuff being a distraction from the actual work a while ago15:24
johnthetubaguycools, I thought you had, wanted to make sure those dots had been joined15:24
*** vgridnev_ has joined #openstack-cinder15:28
*** aorourke-afk is now known as aorourke15:31
*** akerr has joined #openstack-cinder15:36
*** wiebalck_ has joined #openstack-cinder15:36
*** crose has quit IRC15:36
*** adrianofr has joined #openstack-cinder15:37
*** pots1 has joined #openstack-cinder15:37
*** setmason has joined #openstack-cinder15:38
*** houming-wang has quit IRC15:38
*** salv-orlando has joined #openstack-cinder15:39
dulekxyang1: Okay, I think I've designed it. Fun stuff, we'll be able to drop the column in O release. Or in N when we'll provide post-upgrade migration.15:41
dulekdansmith: Why exactly isn't Nova providing some kind of post-upgrade migrations to drop columns one release earlier?15:41
xyang1dulek: sure15:41
dulekxyang1: I'll try to describe that step-by-step and make sure to add you as spec reviewer. Thanks for paying attention to rolling upgrades issues.15:43
dansmithdulek: we do15:43
xyang1dulek: sounds good15:43
dansmithdulek: after we're no longer referencing a column, we land a change to remove it15:43
dulekdansmith: So you can remove a column in one release?15:44
*** thangp has joined #openstack-cinder15:45
dulekdansmith: "having to only apply database schema updates once per cycle. That means we can’t actually drop the old column until the release following the expansion."15:45
dansmithdulek: no, it's generally two away15:45
dansmithdulek: so you land the migration to add the new column and to move data to it in one release,15:45
*** vgridnev_ has quit IRC15:45
dansmiththe next release you drop the column from the model, so sqlalchemy stops referring to it15:45
dansmithin the release after that you can actually remove it from schema15:45
dansmithin the first step, however, you can be nulling out the column once you're done with it, which means the overhead for keeping it is very low15:46
dulekdansmith: SQLAlchemy models - that's the problem. Thank you, I get that now!15:46
*** vgridnev has joined #openstack-cinder15:46
dansmiththe OSM stuff in nova (that we removed for the moment) lets you accelerate the third step a bit15:46
dansmithdulek: cool15:46
*** vgridnev has quit IRC15:46
*** ntpttr1 has joined #openstack-cinder15:51
*** ntpttr has quit IRC15:51
*** mc_nair has joined #openstack-cinder15:52
*** eharney has joined #openstack-cinder15:52
*** dustins has quit IRC15:59
openstackgerritxing-yang proposed openstack/cinder: WIP: Add ConsistencyGroupVolumeTypeMapping table  https://review.openstack.org/24585415:59
*** haomaiwang has quit IRC16:01
*** haomaiwang has joined #openstack-cinder16:01
*** rhedlind has joined #openstack-cinder16:01
xyang1dulek: added a new table for your reference. wait for your live db migration spec to do the migration part16:02
*** ameade has quit IRC16:03
*** ameade has joined #openstack-cinder16:04
*** wiebalck_ has quit IRC16:05
*** jgregor has joined #openstack-cinder16:07
*** crose has joined #openstack-cinder16:08
DuncanTI'm being dumb here, trying to run tox on a new box, 'tox -e py27' aborts with 'ValueError: ('Expected version spec in', "enum34;python_version=='2.7' or python_version=='2.6'", 'at', ";python_version=='2.7' or python_version=='2.6'")' at the end of the trace. Python 2.7.6 is the default version on this system16:08
DuncanTWhat have I forgotten, please?16:09
*** leeantho has joined #openstack-cinder16:11
*** belmoreira has quit IRC16:11
smcginnisDuncanT: I think you need a newer tox?16:13
smcginnissudo pip install --upgrade --force tox16:13
DuncanTsmcginnis: A ha, that might be it, I'll upgrade, thanks16:13
smcginnisMaybe?16:13
*** leeantho_ has joined #openstack-cinder16:13
smcginnisDuncanT: np.16:13
smcginnisDuncanT: Was going to as Captain Oslo, but are you aware of any incubator code we have yet?16:14
smcginnisDuncanT: I think we just have a comment left in the i18n marker source.16:15
DuncanTsmcginnis: There was a patch for some image utils things this week, which is why I bought it up in the agenda16:15
smcginnisDuncanT: Ah, good call then. Thanks!16:15
e0neDuncanT: I'm hitting with the same issue for stable/kilo :(16:15
DuncanTLooks like there's some scheduler stuff in the tree still16:16
DuncanTe0ne: If you find a fix, let me know please?16:16
e0neDuncanT: sure16:16
smcginnise0ne, DuncanT: Maybe need old tox for kilo?16:16
e0nesmcginnis: goot idea. i'll check16:17
*** leeantho has quit IRC16:17
e0nesmcginnis: just to confirmL kilo is still open for bugfix backports, isn't it?16:18
smcginnise0ne: Actually, security only. :/16:18
smcginnise0ne: http://docs.openstack.org/releases/16:19
e0nesmcginnis: hm..16:19
e0nesmcginnis: according to https://wiki.openstack.org/wiki/StableBranch#Support_phases16:19
*** tpeoples has quit IRC16:19
e0ne"Phase II (6-12 months): Only critical bugfixes and security patches are acceptable"16:19
*** tpeoples has joined #openstack-cinder16:19
e0nein any case, it's too late for https://bugs.launchpad.net/cinder/+bug/1478737 backport :)16:20
openstackLaunchpad bug 1478737 in Cinder "Cinder upload-to-image with glance v2 fails" [Medium,Fix released] - Assigned to Tomoki Sekiyama (tsekiyama)16:20
e0ne:(16:20
e0nesmcginnis: thanks for the help. you saved my time:)16:20
smcginnise0ne: That wiki may need to be updated.16:20
jgriffithe0ne: I'm not having any errors running unit tests on Kilo with fresh venv16:20
smcginnise0ne: Hah, yeah, probably not the answer you wanted, but I guess at least it saved you some time. :)16:21
e0nejgriffith: fresh env is good:)16:21
jgriffithe0ne: although sadly it appears that the default 'n' processes is back to 1 for some reason :(16:21
*** hemnafk is now known as hemna16:21
smcginnisjgriffith: That was before you changed to testr or something. Vaguely remember you messing with that. :)16:22
jgriffithsmcginnis: yeah.. but that was16:22
jgriffithDOH16:22
jgriffithLiberty!16:22
jgriffithnot Kilo :)16:23
jgriffithThey've all blended together at this point :)16:23
smcginnis:)16:23
smcginnisNo doubt!16:23
jgriffithDerP16:23
e0neevery developer wants that users will use the latest software16:24
jgriffithe0ne: hehe... well sometimes16:25
smcginnisDuncanT: Did you try deleting .tox?16:25
DuncanTsmcginnis: Yes, I tried that. Just waiting for tox to upgrade, proxy makes pip so slow16:26
e0nejgriffith: IMO, support current-N version is always a pain. sometimes is't more painful, sometimes less...16:26
jgriffithe0ne: I hear ya.16:26
*** crose has quit IRC16:29
e0nejgriffith: I've got a next topic for holly-war: brickclient or extend a cinderclient16:29
mtaninoe0ne: Hi, I replied to your comment. If you have a chance, please let me know your thought. https://review.openstack.org/#/c/244962/16:30
smcginnise0ne: :)16:30
openstackgerritxing-yang proposed openstack/cinder-specs: Replicating a group of volumes  https://review.openstack.org/22972216:30
*** jwcroppe has joined #openstack-cinder16:30
*** jdurgin has joined #openstack-cinder16:31
*** crose has joined #openstack-cinder16:31
*** lixiaoy1_ has quit IRC16:32
e0nejgriffith: I would like to find a consensus how to deal with it in the best way16:32
e0nemtanino: I'm looking on it16:32
*** rhagarty__ has quit IRC16:33
mtaninoe0ne: Thank you.16:33
DuncanTe0ne: swords at dawn (after an all-nighter i the pub)16:33
openstackgerritMerged openstack/cinder: Downstream Fix for Genconfig  https://review.openstack.org/23220516:33
e0neDuncanT: not sure that I understood you right16:34
DuncanTe0ne: That's how best to make a decision16:34
e0neDuncanT: I prefer a dictatorship16:35
*** sgundur has joined #openstack-cinder16:35
e0neDuncanT: or random +2/-2 could works too:)16:36
DuncanTe0ne: Magic 8ball bot?16:36
e0neDuncanT: I like it16:36
*** jerrygb has quit IRC16:39
DuncanTsmcginnis: e0ne: New version of tox has got me past that error, unfortunately I now have to wait for it to rebuild the venv, which takes a while on this network16:39
*** rhagarty has joined #openstack-cinder16:39
e0nemtanino: I've posted my comment to the review16:40
*** dustins has joined #openstack-cinder16:40
mtaninoe0ne: let me check. Thanks16:40
smcginnisDuncanT: Can you go to Dunkin Donuts instead?16:41
diablo_rojo smcginnis: +116:41
smcginnisDuncanT: Or a pub with free wifi? ;)16:41
*** Yogi1 has joined #openstack-cinder16:42
diablo_rojosmcginnis: If that was a shot at Jay, he's not here to defend himself.16:42
smcginnisdiablo_rojo: Hah, no. Just know DuncanT would probably rather go to a pub than a donut shop. :)16:42
smcginnisdiablo_rojo: Though IBM sponsors dockercon and we had wifi issues this morning. :P16:43
DuncanTsmcginnis: I'm working on a remote box. Honestly, pulling everything back locally would be the sensible thing to do16:44
*** jerrygb has joined #openstack-cinder16:47
*** annasort has joined #openstack-cinder16:48
*** Yogi1 has quit IRC16:48
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377616:49
openstackgerritAlex O'Rourke proposed openstack/cinder: Remove db access from 3PAR and LH cg functions  https://review.openstack.org/23608916:49
*** xiaohui has joined #openstack-cinder16:53
openstackgerritKendall Nelson proposed openstack/os-brick: WIP: Multipath Device Action Being Parsed as Name  https://review.openstack.org/24588216:53
*** daneyon has joined #openstack-cinder16:54
*** apoorvad has joined #openstack-cinder16:54
*** gcb has quit IRC16:55
*** alexschm has quit IRC16:56
*** vignesh1 has quit IRC16:57
*** apoorvad has quit IRC17:00
*** rhagarty has quit IRC17:01
*** haomaiwang has quit IRC17:01
*** 18WAA5P6F has joined #openstack-cinder17:01
*** jaypipes has quit IRC17:02
*** xiaohui has quit IRC17:02
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Add extend_volume API to connectors  https://review.openstack.org/24373017:03
*** lprice has joined #openstack-cinder17:04
*** vgridnev has joined #openstack-cinder17:04
*** rhagarty has joined #openstack-cinder17:06
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed)  https://review.openstack.org/23120117:06
*** rhagarty has quit IRC17:08
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)  https://review.openstack.org/23124517:09
Swansonasselin_, this .. "cp /etc/puppet/modules/openstackci/contrib/hiera.yaml /etc/puppet" doesn't actually exist.  I presume single_node_ci_hiera.yaml is what I'm supposed to copy to hiera.yaml17:10
*** nkrinner has quit IRC17:11
Swansonasselin_,  And this line cp /etc/puppet/modules/openstackci/contrib/single_node_ci_data.yaml /etc/puppet/environments/common.yaml points to somethign that doesn't exist.17:13
*** logan2 has joined #openstack-cinder17:20
asselin_Swanson, those files are not merged yet...17:22
*** mriedem has quit IRC17:22
*** rhagarty has joined #openstack-cinder17:22
asselin_Swanson, they're part of this patch https://review.openstack.org/#/c/227584/17:23
asselin_b/c I was trying to simplify it17:23
Swansonasselin_, Ah.  That's the patch I'm working from.  Presumably it is installing bits outside of that patch.17:24
asselin_Swanson, yeah, go to /etc/puppet/modules/openstackci17:24
asselin_and sudo cherry-pick or checkout that patch17:24
asselin_and then the instructions should work as written17:25
*** apoorvad has joined #openstack-cinder17:26
*** e0ne has quit IRC17:27
Swansonasselin_, thanks!17:27
*** Guest91043 is now known as mgagne17:28
*** mgagne has joined #openstack-cinder17:28
*** mgagne is now known as Guest9104317:29
*** Guest91043 is now known as mgagne17:29
*** porrua has quit IRC17:30
*** sgundur has quit IRC17:38
*** jaypipes has joined #openstack-cinder17:40
*** lpetrut has quit IRC17:42
*** rcernin has quit IRC17:42
*** jordanP has quit IRC17:43
*** edtubill has joined #openstack-cinder17:58
*** IlyaG has joined #openstack-cinder17:59
*** edtubill has quit IRC17:59
*** sghanekar_ has joined #openstack-cinder18:00
*** 18WAA5P6F has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
*** edtubill has joined #openstack-cinder18:02
*** mriedem has joined #openstack-cinder18:03
*** harlowja has joined #openstack-cinder18:06
*** edtubill has quit IRC18:07
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR fix driver to work with image cache  https://review.openstack.org/23614318:08
*** edtubill has joined #openstack-cinder18:08
*** angela-s has joined #openstack-cinder18:08
*** edtubill has quit IRC18:10
*** edtubill has joined #openstack-cinder18:11
*** rcernin has joined #openstack-cinder18:12
*** eduardo_ has quit IRC18:12
*** edtubill has quit IRC18:14
*** edtubill has joined #openstack-cinder18:17
*** kfarr has joined #openstack-cinder18:17
*** porrua has joined #openstack-cinder18:20
*** lpetrut has joined #openstack-cinder18:24
*** e0ne has joined #openstack-cinder18:24
*** sgundur has joined #openstack-cinder18:24
*** jistr has quit IRC18:29
*** ociuhandu has quit IRC18:31
Swansonasselin_, I see "project_config_repo" in this but no mention of setting that up in the document.  This will run something if I just run with the project-config-example.git, right?18:41
asselin_Swanson, the docs should mention setting it up. That said, using the example will work.18:42
asselin_Swanson, "## Create an Initial 'project-config' Repository "18:43
SwansonThere it is.18:44
*** salv-orlando has quit IRC18:44
openstackgerritNate Potter proposed openstack/cinder: Show qos_specs_id based on policy  https://review.openstack.org/24165918:56
*** sgotliv_ has quit IRC18:57
*** haomaiwang has quit IRC19:01
hemnaanyone able to run the cinderclient functional tests?19:01
*** haomaiwang has joined #openstack-cinder19:01
e0nehemna: did you create a config?19:01
hemna?19:01
* hemna has never tried to run them yet.19:02
* hemna has no clue19:02
e0nehemna: you need to create conf for cinderclient func tests19:02
hemnahttp://paste.openstack.org/show/479027/19:02
hemnawhat conf?19:03
hemnawhere?19:03
e0ne1min, plz19:03
*** Guest29831 is now known as mfisch19:04
*** mfisch is now known as Guest8087519:04
e0nehemna: something like http://paste.openstack.org/show/479028/19:05
*** Guest17963 is now known as med_19:05
*** med_ has joined #openstack-cinder19:05
hemnaman paste.openstack.org is painfully slow19:05
e0nehemna: +119:06
hemnahrmm19:06
*** adelia has joined #openstack-cinder19:06
hemnathat's kinda lame to need to do that.19:06
hemnawhy not have it use the env. vars ?19:06
hemna:(19:07
e0nehemna: env vars could not work on CIs19:07
hemnawell, that's why there is also the command line options19:08
e0nehemna: we don't pass them always for bash scripts execution19:08
hemnathis just seems....odd/wrong19:08
e0nehttps://github.com/openstack/python-cinderclient/blob/6d440d2e942fa5d22d33e648c72deddaf501afb1/cinderclient/tests/functional/hooks/post_test_hook.sh#L5919:08
e0nehemna: jgriffith could know more why we do this in a such way19:09
hemnaI'm sure there is a reason why I'm wrong, but this is less than ideal IMHO.19:09
e0nehemna: afair, there were issuese with infra with passing OS_* variables19:10
e0nebut I could be wrong19:10
hemnaI'm sure there might be19:10
hemnabut I'm just saying, we don't do this for cinder client itself.19:10
hemnaand  yet CI works.19:11
e0nehemna: what do you mean?19:12
hemnaum, we don't need a conf file to run cinder client.19:12
e0newhat CI?19:12
hemnaand CI works just fine.19:12
hemnacinderclient CI19:12
hemnagate, etc19:12
e0neno, we've got such config in the Gates19:13
hemnajust to run cinderclient ?!19:13
hemnacinder create ...19:13
e0neto run functional tests19:13
hemnayou are missing my point19:13
hemnawe shouldn't need a conf file to run functional tests.19:13
*** ntpttr1 has quit IRC19:13
*** ntpttr has joined #openstack-cinder19:14
*** setmason has quit IRC19:14
e0neone moment, please19:15
*** DaveJ_ has quit IRC19:16
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient: Do not require functional_creds.conf for functional tests  https://review.openstack.org/24593319:18
e0nehemna: let's see how CI will pass or not for ^^19:19
e0neexcuse me for not goot commit message19:19
e0nehemna: https://github.com/openstack/python-cinderclient/blob/master/cinderclient/tests/functional/base.py#L32 - may be it's a root cause why conf file existts19:20
hemnasure19:21
*** lpetrut has quit IRC19:21
*** setmason has joined #openstack-cinder19:21
hemnaI think we are using 1.8 no ?19:21
hemnaadding passenv OS_AUTH_URL worked19:22
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient: Do not require functional_creds.conf for functional tests  https://review.openstack.org/24593319:22
hemnato tox.ini19:22
e0nehemna: I try to pass all OS_* variables19:22
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP LeftHand driver to now be HPE  https://review.openstack.org/24034319:23
hemnaI'm trying that locally19:24
e0neafair, we don't require any tox version19:24
hemnaminversion = 1.619:24
*** wiebalck_ has joined #openstack-cinder19:24
e0ne$ tox --version 2.1.1 imported from /usr/local/lib/python2.7/dist-packages/tox/__init__.pyc19:24
e0neit's on my env19:25
hemnahttp://paste.openstack.org/show/479029/19:25
*** edtubill has quit IRC19:25
hemnathat's with passenv = OS_*19:25
*** salv-orlando has joined #openstack-cinder19:25
hemnaos-tenant-name is None still though...hrmm19:25
hemnamanually set it and still same issue19:27
hemnaexport OS_TENANT_NAME=admin19:27
hemnasame serviceCatalog error19:27
hemnapita19:27
*** edtubill has joined #openstack-cinder19:30
e0nehttps://review.openstack.org/#/c/245855/ - Ceph CI is broken for stable/kilo branch:(19:31
e0nesmcginnis: ^^19:32
e0nehemna: works for me for admin creds19:33
hemnae0ne, all I did was source ~/devstack/accrc/admin/admin19:34
e0nehemna: I tries "source ~/devstack/openrc admin admin"19:34
e0nes/tries/tried19:34
* hemna tries19:35
eharneye0ne: i think there's been a fix or two in there over the last couple of months, some changes may need to be backported for it19:35
e0neeharney: we can fix it but kilo is for security onlu fixes now:(19:36
e0neeharney, smcginnis can we make an exception and fix ceph in kilo?19:37
eharneynot really sure what the expectation is there w.r.t. fixing CI issues, then :/19:37
e0neeharney: do you mean is't a CI issue and nothing is not needed to be changed in the cinder code?19:38
eharneye0ne: not sure, i'd have to see what bugs it's actually hitting19:38
*** vgridnev has quit IRC19:39
eharneyTBH i'm not sure that driver CI adds a lot to a branch that's security fixes only...19:40
e0nebut it will block any patch if they will be19:44
*** adelia_ has joined #openstack-cinder19:46
*** adelia has quit IRC19:49
hemnae0ne, ok that worked   source ~/devstack/openrc admin admin19:51
hemnadoes ~/devstack/accrc/admin/admin not work anymore ?19:51
e0neI never used it19:51
hemna*sigh*19:51
hemnaRan 19 tests in 63.333s (+0.625s)19:54
*** lprice has quit IRC19:54
*** lprice has joined #openstack-cinder19:54
*** wiebalck_ has quit IRC19:56
*** cpallares has quit IRC19:56
*** salv-orlando has quit IRC19:56
*** gnarld_ has joined #openstack-cinder19:57
*** cpallares has joined #openstack-cinder19:57
*** cfouts has quit IRC19:57
*** 7JTAASV77 has quit IRC19:58
*** zzzeek has quit IRC19:58
*** rcernin has quit IRC19:58
*** Lee1092 has quit IRC19:59
*** strictlyb has joined #openstack-cinder20:01
*** haomaiwang has quit IRC20:01
*** zzzeek has joined #openstack-cinder20:01
*** haomaiwang has joined #openstack-cinder20:01
e0nebb tomorrow. have a good day!20:03
*** IanGovett has joined #openstack-cinder20:03
Swansonasselin_, log server needs gcc installed apparently.  FYI for doc.20:04
*** rushil has joined #openstack-cinder20:09
*** e0ne has quit IRC20:09
openstackgerritEric Harney proposed openstack/cinder: get_all_snapshots: Fix log message typo  https://review.openstack.org/24596020:11
*** salv-orlando has joined #openstack-cinder20:14
*** lpetrut has joined #openstack-cinder20:15
*** rhagarty has quit IRC20:19
asselin__Swanson, really? it should be fixed them. Do you have the puppet output and manual fix you applied?20:20
*** liverpooler has quit IRC20:23
Swansonasselin__, http://paste.openstack.org/show/479038/20:25
Swansonasselin__, I haven't done a manual fix.  I ran apt-get install gcc and re ran the puppet apply but I'm not sure that re-ran the problem.20:25
Swansonasselin__, I can scp to it right now so I don't really care.  Going back to the main server setup again....20:26
asselin__Swanson, ok, yeah...would like to make it more robust so others won't have this issue.20:27
asselin__Swanson, what os /image is this for the log server?20:27
Swansonubuntu 14.0420:27
Swansonserver20:27
*** crose has quit IRC20:28
asselin__Swanson, i would probably install build-essentials instead of just gcc20:30
Swansonasselin__, done.20:32
Swanson(Just now)20:32
asselin__Swanson, still curious why I didn't run into this. Perhaps a new dependency was added20:32
Swansonasselin__, difference in our default packages?20:33
asselin__Swanson, https://review.openstack.org/#/c/229198/2/requirements.txt back on 9/2920:34
asselin__but I've set this up more recently than that w/o issues....20:35
*** changbl has quit IRC20:36
Swansonasselin__, If I can scp and bring up the log server web page I should be good to continue, right?20:36
asselin__Swanson, yes, but you also want os-loganalyze working20:36
asselin__Swanson, if you have time, we can fix it today20:36
Swansonasselin__, Sure.20:37
Swansonasselin__, Although I do need to run off for 15 minutes or so...20:38
asselin__sure, ping me later20:38
*** IlyaG_ has joined #openstack-cinder20:39
*** IlyaG has quit IRC20:42
*** anshul has quit IRC20:42
*** sidbbt has joined #openstack-cinder20:42
*** diablo_rojo has quit IRC20:43
asselin__Swanson, when you get a change, cherry pick this to /etc/puppet/modules/openstackci20:44
asselin__https://review.openstack.org/24597320:44
asselin__apt-remove gcc and build-essentials20:44
*** IanGovett has quit IRC20:44
asselin__pip remove os-loganalyze20:44
asselin__then run the puppet apply site.pp command again. it should install clean20:45
sidbbtsmcginnis: hello, are you here? i have a quick question. we added a blueprint for adding a new volume driver and i noticed you've approved it for mitaka-2. but the deadline for adding new volume drivers is mitaka-1, right? https://blueprints.launchpad.net/cinder/+spec/falconstor-freestor-cinder-driver20:46
openstackgerritMerged openstack/cinder: Updated from global requirements  https://review.openstack.org/24523720:49
*** vgridnev has joined #openstack-cinder20:49
openstackgerritRyan McNair proposed openstack/cinder: Support oversubscription for IBM Storwize/SVC  https://review.openstack.org/24536020:49
*** pots1 has left #openstack-cinder20:49
*** ekarlso has quit IRC20:50
*** ekarlso has joined #openstack-cinder20:50
*** diablo_rojo has joined #openstack-cinder20:50
openstackgerritMichal Dulko proposed openstack/cinder-specs: Add spec for online-schema-upgrades  https://review.openstack.org/24597620:52
mtaninosidbbt: smcginnis,  Hi, driver deadline is Mitaka-2. Please check the e-mail from Sean "Re: [openstack-dev] [cinder] Cinder Mitaka Deadlines"20:53
*** anshul has joined #openstack-cinder20:55
sidbbtmtanino, smcginnis, that's good to know. i saw the original email but missed the one where the dates were revised. thank you!20:55
mtaninosidbbt: sure.20:56
mtaninosidbbt: That's the latest one which I know.20:56
sidbbtmtanino: we were just getting ready to submit our driver. i wonder if this means we should hold off until after m-1, or should we still go ahead and submit now?20:57
mtaninosidbbt: No, it is better to post your driver as soon as possible and continue review and discussion on the tree.20:59
*** haomaiwang has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
openstackgerritKendall Nelson proposed openstack/os-brick: WIP: Multipath Device Action Being Parsed as Name  https://review.openstack.org/24588221:08
openstackgerritRyan McNair proposed openstack/cinder: Remove *_multipath_enabled flag from IBM drivers  https://review.openstack.org/24474921:08
*** raildo is now known as raildo-afk21:08
*** rhagarty has joined #openstack-cinder21:08
*** sgotliv_ has joined #openstack-cinder21:08
*** sileht has quit IRC21:09
Swansonasselin__, is build-essentials right?  Or should it be build-essential?21:12
*** merooney has quit IRC21:15
*** dustins has quit IRC21:16
*** akerr has quit IRC21:17
*** jamielennox|away is now known as jamielennox21:18
*** lpabon has quit IRC21:18
*** ZZelle_ has joined #openstack-cinder21:19
*** timcl has quit IRC21:20
*** IlyaG_ has quit IRC21:20
*** lprice has quit IRC21:20
Swansonasselin__, syntax is wrong.  Missing a ':' and ubuntu doesn't seem to know of build-essentials but I'm not sure that build-essential has gcc.21:21
*** [5]Thelo has joined #openstack-cinder21:24
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Remove db access from 3PAR and LH cg functions  https://review.openstack.org/23608921:26
*** niknakpaddywak has joined #openstack-cinder21:26
*** Thelo has quit IRC21:27
*** [5]Thelo is now known as Thelo21:27
*** sgotliv_ has quit IRC21:30
*** porrua has quit IRC21:31
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: WIP: brick add extend_volume API  https://review.openstack.org/24377621:33
*** jaypipes has quit IRC21:35
*** vgridnev has quit IRC21:35
*** bswartz has quit IRC21:37
*** markvoelker has quit IRC21:37
SwansonOkay, if I change those it still doesn't try to run os-loganalyze.  But if I jump to /opt/os-loganalyze and run sudo pip install . I get netifaces.c:1:20: fatal error: Python.h: No such file or directory21:37
Swansonasselin__, ^^21:38
SwansonSo, still some fun.21:38
*** baumann has joined #openstack-cinder21:38
Swansonasselin__, needs python-dev21:40
SwansonThat seems to have installed by hand.21:41
*** thangp has quit IRC21:42
patrickeastSwanson: trying to get a log server set up?21:47
*** rushil has quit IRC21:47
lyarwoodhemna: hey there, just saw your comment on https://review.openstack.org/#/c/238838/ I assume that's aimed at CI and not me? :)21:48
hemnalyarwood, yup :)21:49
*** rhagarty_ has joined #openstack-cinder21:49
hemnalyarwood, I can't remember the pure recheck, other than die die die now.21:49
hemnaheh21:49
hemnasorry21:49
lyarwoodhemna: haha np np just wanted to make sure I hadn't done something stupid again21:50
hemnahehe no. :)  you are fine.21:50
Swansonpatrickeast, Yeah, trying to sort through what we need to make it work automagically.  I had some gcc and python-dev issues.21:51
SwansonI think it is working now.  (It would have worked if I had installed gcc and python-dev before hand.)21:51
*** rhagarty has quit IRC21:51
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - getting iscsi ip from port in existing masking view  https://review.openstack.org/24599721:53
*** lprice has joined #openstack-cinder21:55
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - VMAX driver failing to remove zones  https://review.openstack.org/24493321:55
patrickeastSwanson: oh interesting, i had just set one up like last week for my dev env and didn't run into problems22:00
asselin__Swanson, ok will add that too.22:00
*** changbl has joined #openstack-cinder22:00
patrickeastSwanson: the instructions were basically http://paste.openstack.org/show/479047/22:00
patrickeastand everything was great22:00
*** haomaiwang has quit IRC22:01
patrickeastalthough it was with an aws 14.04 image, not sure what all came baked in22:01
*** haomaiwang has joined #openstack-cinder22:01
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Fix for last volume in VMAX3 storage group  https://review.openstack.org/24433122:01
Swansonpatrickeast, Latest version of the instructions are a bit more hands on than that.22:02
*** dustins has joined #openstack-cinder22:02
patrickeastSwanson: are you trying to only use the upstream stuff or using asselin__'s helper script like i did?22:02
Swansonpatrickeast, This ...  https://review.openstack.org/#/c/227584/22:03
Swansonpatrickeast, That is asselin__'s current setup instructions.  So no helper script.22:03
patrickeastoh, yea, thats the all upstream version22:03
patrickeastend result is, i think, basically the same22:03
Swansonpatrickeast, You're probably right.  I had the old version setup but the instructions changed so I went to do it this way.22:04
asselin__patrickeast, your review of that would be great. no script b/c I'm hoping to take some of the magic out of it22:04
patrickeastasselin__: yea i like that we show how to do it without the helpers, makes it much easier to customize later on down the road22:05
Swansonasselin__, I think that is a good choice.  And it is a nice dry run before screwing up the main CI system.22:05
patrickeasti think it depends a little bit on how much folks deal with the log servers, for my situation i set one up by itself and point jenkins at it22:06
patrickeastthen never look at it again22:06
patrickeastat least until aws fails and i lose the vm22:06
patrickeastwhereas the main ci system gets tweaks all the time, having the site.pp for it created is hugely more important22:06
asselin__patrickeast, true22:07
openstackgerritMerged openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/24564622:08
*** ekarlso has quit IRC22:11
*** ntpttr has left #openstack-cinder22:12
*** lpetrut has quit IRC22:17
*** zhonghua has joined #openstack-cinder22:18
*** IlyaG has joined #openstack-cinder22:19
smcginnismtanino: Thanks for the driver advice. Exactly right.22:19
mtaninosmcginnis: sure.22:19
smcginnishemna: Makes me smile every time I see DIE DIE DIE posted. :)22:20
hemna:)22:20
mtaninodie die die...22:20
smcginnis:)22:20
*** ekarlso has joined #openstack-cinder22:20
smcginnissidbbt: Let me know if you have any other questions for new driver submission.22:21
smcginnissidbbt: It would be good to submit a review as soon as you have something working.22:21
smcginnissidbbt: There will likely be a lot of "feedback" about your driver. :)22:21
smcginnissidbbt: So it would be good to work through it plenty of time before m-2.22:21
*** zhonghua-lee has quit IRC22:22
*** julim has quit IRC22:22
sidbbtsmcginnis: yup, thanks. we were getting ready to submit in the next few days anyway. we'll still do that, but now we have plenty of breathing room for the review, so that's good.22:24
smcginnissidbbt: Great. Just don't wait until the end of m-2 to address issues and CI and you should be in good shape. Thanks!22:24
*** IanGovett has joined #openstack-cinder22:24
*** jamielennox is now known as jamielennox|away22:27
mtaninoAhh, I found funny trap in the https://review.openstack.org/#/c/243489/22:29
smcginnismtanino: What's the issue there?22:32
mtaninosmcginnis: I found Walter's DIE DIE DIE comment on that review :)22:32
*** dave-mccowan has quit IRC22:32
mtaninoit's a trap...22:33
*** baumann has left #openstack-cinder22:33
smcginnis:)22:33
smcginnismtanino: Should kick off pure and maybe more. I want to add that to my CI too.22:33
*** daneyon has quit IRC22:34
smcginnisForget how that even came about (I think off hand comment from hemna) but I thought it would be really funny to make that the trigger for CIs.22:34
*** ZZelle_ has quit IRC22:34
smcginnisAnd patrickeast actually did it. :)22:34
*** jaypipes has joined #openstack-cinder22:34
mtaninosmcginnis: yeah, he did it bery quickly :)22:35
mtaninovery22:35
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP LeftHand driver to now be HPE  https://review.openstack.org/24034322:36
openstackgerritChris Morrell proposed openstack/cinder: Added VAG support to SolidFire.  https://review.openstack.org/24601422:36
smcginnisI think I just chokes.22:36
smcginnis*choked22:36
smcginnisAhh, no reason.22:37
*** edtubill has quit IRC22:38
*** IlyaG has quit IRC22:43
openstackgerritChris Morrell proposed openstack/cinder: Added VAG support to SolidFire.  https://review.openstack.org/24601422:46
*** IlyaG has joined #openstack-cinder22:47
*** lprice has quit IRC22:47
*** wilson1 has joined #openstack-cinder22:51
*** jwang_ has joined #openstack-cinder22:51
*** apoorvad_ has joined #openstack-cinder22:51
*** chenying_ has joined #openstack-cinder22:51
chris_morrellThink I corrected what you asked about, thanks for looking at it so fast. jgriffith said I could wait hours before someone looked things over.22:51
*** akerr has joined #openstack-cinder22:51
*** akshai_ has joined #openstack-cinder22:51
smcginnischris_morrell: That usually is the case. Just happened to see it and had some time. ;)22:52
*** toabctl has quit IRC22:52
*** leeantho__ has joined #openstack-cinder22:53
*** markstur_ has joined #openstack-cinder22:53
*** sgundur has quit IRC22:53
*** apoorvad has quit IRC22:53
*** chenying has quit IRC22:54
*** markstur has quit IRC22:54
*** mordred has quit IRC22:54
*** leeantho_ has quit IRC22:54
*** akshai has quit IRC22:54
*** wilson-1 has quit IRC22:54
*** jwang__ has quit IRC22:54
*** aarefiev has quit IRC22:54
*** sgundur has joined #openstack-cinder22:58
*** med_ has quit IRC22:58
*** Guest80875 has quit IRC22:58
chris_morrellsmcginnis: is it normal to have CI runs puke prematurely? received two emails/runs, one failed on setup, other provided an inaccessible URL.22:58
*** mfisch has joined #openstack-cinder22:58
*** mfisch is now known as Guest7350922:59
*** jgregor has quit IRC22:59
*** toabctl has joined #openstack-cinder22:59
patrickeastchris_morrell: unfortunately its pretty common for that to happen22:59
smcginnischris_morrell: We have a few right now that are not working right.22:59
smcginnischris_morrell: They do need to be fixed. But not something you need to be too worried about.22:59
chris_morrellOkay, thanks guys, will check on the review later for normal CI runs.23:00
smcginnischris_morrell: Jenkins and the Solidfire CIs are the main ones for you at this point.23:00
smcginnischris_morrell: Hopefully the rest of our CIs will become stable enough to care about in the near future.23:00
*** haomaiwang has quit IRC23:01
*** mriedem is now known as mriedem_away23:01
*** dustins has quit IRC23:01
*** akerr has quit IRC23:02
*** sgundur has quit IRC23:02
*** Lickitysplitted_ has joined #openstack-cinder23:02
*** med_ has joined #openstack-cinder23:02
* jgriffith said no such thing 23:02
jgriffith:)23:02
*** med_ is now known as Guest1205923:02
*** haomaiwa_ has joined #openstack-cinder23:02
smcginnis:)23:05
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Brick add param documentation to connectors  https://review.openstack.org/24602323:05
*** uberjay_ has joined #openstack-cinder23:06
*** aarefiev has joined #openstack-cinder23:06
*** dave-mccowan has joined #openstack-cinder23:06
*** IanGovett has quit IRC23:06
*** edmondsw has quit IRC23:06
*** ircuser-1 has quit IRC23:06
*** leseb- has quit IRC23:06
*** uberjay has quit IRC23:06
*** Lickitysplitted has quit IRC23:06
*** lifeless has quit IRC23:06
*** dobson has quit IRC23:06
*** dobson has joined #openstack-cinder23:07
*** apoorvad_ is now known as apoorvad23:07
*** mordred has joined #openstack-cinder23:07
*** leseb_ has joined #openstack-cinder23:07
*** lifeless has joined #openstack-cinder23:09
*** sidbbt has quit IRC23:10
*** alejandrito has quit IRC23:11
*** edmondsw has joined #openstack-cinder23:13
*** ircuser-1 has joined #openstack-cinder23:15
*** Guest73509 is now known as Guest8087523:16
*** zhangjn has joined #openstack-cinder23:19
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP LeftHand driver to now be HPE  https://review.openstack.org/24034323:19
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Fix for last volume in VMAX3 storage group  https://review.openstack.org/24433123:22
*** ociuhandu has joined #openstack-cinder23:22
*** zhangjn has quit IRC23:32
*** jamielennox|away is now known as jamielennox23:37
*** setmason_ has joined #openstack-cinder23:38
*** bswartz has joined #openstack-cinder23:38
*** setmason has quit IRC23:39
*** ekarlso has quit IRC23:41
*** setmason_ has quit IRC23:42
*** setmason has joined #openstack-cinder23:43
*** tpeoples is now known as tpeoples_away23:45
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR fix driver to work with image cache  https://review.openstack.org/23614323:46
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR fix driver to work with image cache  https://review.openstack.org/23614323:47
*** ekarlso has joined #openstack-cinder23:47
*** yusuke has joined #openstack-cinder23:47
*** adelia_ has quit IRC23:47
*** adelia has joined #openstack-cinder23:48
*** adelia has quit IRC23:52
*** smoriya has quit IRC23:56
*** zz_smoriya_ is now known as smoriya23:56
*** ekarlso has quit IRC23:57
*** ekarlso has joined #openstack-cinder23:57

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