2015-01-08T00:00:53 *** takedakn has joined #openstack-cinder 2015-01-08T00:01:18 *** chlong has quit IRC 2015-01-08T00:02:05 *** chlong has joined #openstack-cinder 2015-01-08T00:03:09 *** takedakn has quit IRC 2015-01-08T00:03:32 *** takedakn has joined #openstack-cinder 2015-01-08T00:04:19 *** fischerw has quit IRC 2015-01-08T00:06:14 *** chlong has quit IRC 2015-01-08T00:18:07 John Griffith proposed openstack/cinder: Implement clone_image caching on SolidFire https://review.openstack.org/142859 2015-01-08T00:18:10 *** jaypipes has quit IRC 2015-01-08T00:18:46 *** jungleboyj has joined #openstack-cinder 2015-01-08T00:20:16 *** Masahiro has joined #openstack-cinder 2015-01-08T00:22:48 jgriffith: hey so I stated looking at some of the stub issues from the unit tests that are failing with cinder objects. 2015-01-08T00:23:01 *** scottda_ has quit IRC 2015-01-08T00:23:02 jgriffith: I noticed the first set of tests failing are using the stubs from cinder.tests.api.v2.stubs 2015-01-08T00:23:36 which is somewhat good, because it's using shared fixtures 2015-01-08T00:23:44 thingee: jgriffith smcginnis hemna Here is the fix for the _lazy in stable/juno: https://review.openstack.org/145642 2015-01-08T00:23:44 just they need to be improved 2015-01-08T00:24:59 *** Masahiro has quit IRC 2015-01-08T00:25:06 John Griffith proposed openstack/cinder: Fix drbd driver to load without 3'rd party libs https://review.openstack.org/145644 2015-01-08T00:27:12 *** IanGovett has quit IRC 2015-01-08T00:28:46 jungleboyj: thanks, lgtm once jenkins signs off 2015-01-08T00:29:38 *** chlong has joined #openstack-cinder 2015-01-08T00:31:21 thingee: Thank you! 2015-01-08T00:31:53 jungleboyj: https://review.openstack.org/#/c/145613/ 2015-01-08T00:32:03 jungleboyj: http://paste.openstack.org/raw/155906/ 2015-01-08T00:32:24 thingee, jgriffith so it turns out the allocated space code is pretty broken 2015-01-08T00:32:47 allocated_capacity_gb for scheduling 2015-01-08T00:33:30 I saw you pushed that up thingee Let me look here. 2015-01-08T00:33:36 it is tracked in the manager but if volume creation fails in the driver it doesn’t add to the allocated_capacity 2015-01-08T00:33:47 but it deletes it when you delete the volume 2015-01-08T00:34:30 *** smoriya has joined #openstack-cinder 2015-01-08T00:34:52 *** takedakn has quit IRC 2015-01-08T00:35:35 thingee: Hmmm, that is pretty simple. 2015-01-08T00:36:03 jungleboyj: suspicious 2015-01-08T00:36:24 thingee: suspicious? 2015-01-08T00:36:37 yeah simple code, probably something is wrong 2015-01-08T00:36:47 my manual testing seems to disagree, but wanted other eyes on it 2015-01-08T00:36:55 thingee: No, not at all. 2015-01-08T00:37:01 *** annashen has joined #openstack-cinder 2015-01-08T00:37:04 *** bswartz has joined #openstack-cinder 2015-01-08T00:37:25 vishy: interesting 2015-01-08T00:37:35 My eyes on it are probably not the best as the client and session code are all a mystery to me. 2015-01-08T00:38:12 My only concern is what if _get_project returns None. What heppsn then? 2015-01-08T00:38:33 *** Mandell has quit IRC 2015-01-08T00:39:10 jungleboyj: so I was talking with jamielennox about that. My understanding was that happens if a keystone plugin doesn't implement that. But apparently most are based off identity. 2015-01-08T00:39:17 so it shouldn't be a problem, but a valid concern 2015-01-08T00:39:37 most = all? 2015-01-08T00:39:42 I dunno. 2015-01-08T00:39:58 * jungleboyj really needs to figure out the keystone stuff. 2015-01-08T00:40:35 John Griffith proposed openstack/cinder: Deal with tgt already exists errors https://review.openstack.org/145608 2015-01-08T00:41:20 vishy: son of a.... 2015-01-08T00:41:42 vishy: so it's broken for a number of other reasons, but that one isn't one that I had on the list 2015-01-08T00:42:07 jgriffith: also in the icehouse code it doesn’t count available volumes to the total 2015-01-08T00:42:09 thingee: is that running, you have def _get_project(self) and self._get_project_id() 2015-01-08T00:42:14 but that has been fixed 2015-01-08T00:43:15 jamielennox: ha, apparently. 2015-01-08T00:43:16 vishy: I'll spend some time going through tomorrow and logging bugs/fixing as I go 2015-01-08T00:43:23 jamielennox: I don't know how 2015-01-08T00:43:26 vishy: unless you're already doing that :) 2015-01-08T00:43:29 jgriffith: thing is i’m not quite sure how to fix it 2015-01-08T00:43:33 my thought would be if there is no project_id returned then just don't try and append anything 2015-01-08T00:43:52 vishy: might not be able to, that's what we've been fighting about for a year now 2015-01-08T00:43:53 *** km has quit IRC 2015-01-08T00:44:06 the problem is if the driver errors is the space used or not? 2015-01-08T00:44:18 it seems like the driver should be reporting this data 2015-01-08T00:44:31 vishy: yeah, and if we fix the stats reporting they will 2015-01-08T00:44:45 vishy: so what I'm thinking is just brute force the change rather than waiting for everybody to agree 2015-01-08T00:45:01 vishy: in other words, go purely from stats reported back to mgr from driver 2015-01-08T00:45:24 jgriffith: i was thinking you could gust get it from the driver but default it to total - free if it isn’t set 2015-01-08T00:45:25 vishy: we already rely on that soley for scheduler (or should) 2015-01-08T00:45:26 thingee: what i'd love is that for every path in v2 with a URL containing the project_id you validate that project_id in URL == project_id in token then redirect the call to the same url without the project_id 2015-01-08T00:45:38 but that doesn’t work for those drivers that are passing back total as ‘unknown' 2015-01-08T00:45:39 thingee: a path to removing the project_id from cinder calls altogether 2015-01-08T00:45:47 s/calls/urls 2015-01-08T00:45:51 maybe it is just ‘unknown’ for those drivers? 2015-01-08T00:46:01 *** scottda_ has joined #openstack-cinder 2015-01-08T00:46:24 jamielennox: k 2015-01-08T00:46:53 vishy: so maybe that's a good motivation actually. 2015-01-08T00:47:00 big job, small pay off short term 2015-01-08T00:47:07 vishy: if you report unknown or inf we'll use the crappy internal guessing 2015-01-08T00:47:14 so it's in your best interest to fix it 2015-01-08T00:47:34 vishy: you around tomorrow? 2015-01-08T00:51:01 thingee: The paste link you have doesn't work. 2015-01-08T00:52:11 *** scottda_ has quit IRC 2015-01-08T00:52:53 yeah I think there might be something wrong with paste...it was working earlier 2015-01-08T00:53:10 Ok. Weird. 2015-01-08T00:55:38 jgriffith: probably 2015-01-08T00:56:55 *** Mandell has joined #openstack-cinder 2015-01-08T00:58:59 *** annashen has quit IRC 2015-01-08T01:04:39 *** dank has quit IRC 2015-01-08T01:08:18 *** alexpilotti has quit IRC 2015-01-08T01:08:33 jgriffith: Good point on the _lazy review. I need to drop off agian but I will fix that up a little later and push another patch. 2015-01-08T01:09:10 *** jdurgin has quit IRC 2015-01-08T01:15:30 wanghao proposed openstack/cinder: Implement function of import/export snapshots https://review.openstack.org/144590 2015-01-08T01:15:39 *** jungleboyj has quit IRC 2015-01-08T01:18:25 *** scottda_ has joined #openstack-cinder 2015-01-08T01:19:15 *** ebalduf has quit IRC 2015-01-08T01:20:50 Mitsuhiro Tanino proposed openstack/cinder: Remove unused variables from ensure_export() https://review.openstack.org/145381 2015-01-08T01:20:58 *** scottda_ has quit IRC 2015-01-08T01:23:52 *** tsekiyam_ has joined #openstack-cinder 2015-01-08T01:24:15 hey jgriffith, can you please approve this (again) https://review.openstack.org/#/c/145386/ …. I had approval from you and workflow +1 from someone else but had to rebase because of the oslo _lazy bug. It’s a easy to examine change regarding i18n fixes 2015-01-08T01:26:57 *** tsekiyama has quit IRC 2015-01-08T01:27:27 Anthony Lee proposed openstack/cinder: Fixed server name being retained after detach in LeftHand https://review.openstack.org/124145 2015-01-08T01:28:07 Anthony Lee proposed openstack/cinder: Remove locks from LeftHand driver https://review.openstack.org/141475 2015-01-08T01:28:26 *** tsekiyam_ has quit IRC 2015-01-08T01:28:47 *** mtanino has quit IRC 2015-01-08T01:29:53 John Griffith proposed openstack/cinder: Use cinder.utils.execute directly https://review.openstack.org/145630 2015-01-08T01:31:32 *** zhithuang has joined #openstack-cinder 2015-01-08T01:31:41 *** zhithuang is now known as winston-d_ 2015-01-08T01:33:43 *** leeantho has quit IRC 2015-01-08T01:41:37 winston-d_: ping 2015-01-08T01:54:54 *** nosnos has joined #openstack-cinder 2015-01-08T01:57:55 *** thingee has quit IRC 2015-01-08T02:02:22 *** Mandell has quit IRC 2015-01-08T02:06:13 *** elazro has quit IRC 2015-01-08T02:06:46 *** elazro has joined #openstack-cinder 2015-01-08T02:08:04 *** ryanpetrello has quit IRC 2015-01-08T02:08:45 *** haomaiwang has joined #openstack-cinder 2015-01-08T02:09:00 *** Masahiro has joined #openstack-cinder 2015-01-08T02:11:39 *** _cjones_ has quit IRC 2015-01-08T02:11:54 *** chlong has quit IRC 2015-01-08T02:14:31 *** ebalduf has joined #openstack-cinder 2015-01-08T02:15:28 *** winston-d_ has quit IRC 2015-01-08T02:15:42 *** winston-d_ has joined #openstack-cinder 2015-01-08T02:16:38 *** julim has joined #openstack-cinder 2015-01-08T02:21:06 *** ebalduf has quit IRC 2015-01-08T02:24:32 *** ebalduf has joined #openstack-cinder 2015-01-08T02:27:51 *** Mandell has joined #openstack-cinder 2015-01-08T02:28:17 OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/145661 2015-01-08T02:28:59 *** ebalduf has quit IRC 2015-01-08T02:31:06 *** winston-d_ has quit IRC 2015-01-08T02:34:13 *** thingee has joined #openstack-cinder 2015-01-08T02:36:02 Mudassir Latif proposed openstack/cinder: Add replication and PG setup methods on PureISCSIDriver https://review.openstack.org/145090 2015-01-08T02:37:06 *** ling-yun-1 has joined #openstack-cinder 2015-01-08T02:39:11 *** zengyunling__ has joined #openstack-cinder 2015-01-08T02:39:14 *** winston-d_ has joined #openstack-cinder 2015-01-08T02:40:19 *** ling-yun has quit IRC 2015-01-08T02:41:56 *** kaisers has joined #openstack-cinder 2015-01-08T02:42:02 *** fandi has quit IRC 2015-01-08T02:42:29 *** ling-yun-1 has quit IRC 2015-01-08T02:42:41 winston-d_: ping 2015-01-08T02:43:03 *** mberlin1 has joined #openstack-cinder 2015-01-08T02:43:50 *** mberlin has quit IRC 2015-01-08T02:44:02 *** kaisers1 has quit IRC 2015-01-08T02:44:49 Thang Pham proposed openstack/cinder: WIP: Cinder objects base https://review.openstack.org/131873 2015-01-08T02:45:13 Thang Pham proposed openstack/cinder: WIP: Snapshot object https://review.openstack.org/133566 2015-01-08T02:46:15 *** winston-d_ has quit IRC 2015-01-08T02:48:44 *** Mandell has quit IRC 2015-01-08T02:52:49 *** winston-d_ has joined #openstack-cinder 2015-01-08T02:53:10 thingee: pong 2015-01-08T02:53:32 *** ryanpetrello has joined #openstack-cinder 2015-01-08T02:53:44 thingee: sorry i am having a bad internet connection today. 2015-01-08T02:56:41 *** annashen has joined #openstack-cinder 2015-01-08T03:00:33 winston-d_: heh seems like a trend today :) 2015-01-08T03:00:33 so not sure if you saw vishy's earlier comments 2015-01-08T03:00:33 in #openstack-cinder 2015-01-08T03:02:27 thingee: I didn't. what is it? 2015-01-08T03:02:30 winston-d_: allocated_capacity_gb code for the scheduler appears to be broken. Since it's tracked in the manager, if the volume creation fails in the driver, it doesn't add the allocated_capacity. 2015-01-08T03:02:44 *** Mandell has joined #openstack-cinder 2015-01-08T03:02:49 thingee: is there a bug? 2015-01-08T03:03:23 winston-d_: convo is here http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2015-01-08.log 2015-01-08T03:03:31 *** lcurtis has joined #openstack-cinder 2015-01-08T03:03:44 winston-d_: search for timestamp '2015-01-08T00:32:24' 2015-01-08T03:04:31 thingee: can't believe that I didn't know that we have log in cinder channel now! 2015-01-08T03:04:36 thingee: looking, thx. 2015-01-08T03:05:04 winston-d_: oh yeah, pretty recently turned on. 2015-01-08T03:11:11 *** winston-d_ has quit IRC 2015-01-08T03:13:33 *** winston-d_ has joined #openstack-cinder 2015-01-08T03:16:10 *** Mandell has quit IRC 2015-01-08T03:18:20 *** scottda_ has joined #openstack-cinder 2015-01-08T03:20:23 *** mudassirlatif has quit IRC 2015-01-08T03:20:36 *** patrickeast has quit IRC 2015-01-08T03:21:49 Alan Jiang proposed openstack/cinder: rbd calls block eventlet threads https://review.openstack.org/145678 2015-01-08T03:22:15 *** bill_az has quit IRC 2015-01-08T03:25:12 *** ebalduf has joined #openstack-cinder 2015-01-08T03:27:10 *** Mandell has joined #openstack-cinder 2015-01-08T03:29:14 *** scottda_ has quit IRC 2015-01-08T03:30:51 *** ebalduf has quit IRC 2015-01-08T03:36:21 *** scottda_ has joined #openstack-cinder 2015-01-08T03:43:17 *** nosnos has quit IRC 2015-01-08T03:49:05 *** ryanpetrello has quit IRC 2015-01-08T03:54:03 *** _cjones_ has joined #openstack-cinder 2015-01-08T03:54:35 *** _cjones_ has joined #openstack-cinder 2015-01-08T03:55:25 *** ebalduf has joined #openstack-cinder 2015-01-08T03:55:59 *** annashen has quit IRC 2015-01-08T03:57:47 *** chlong has joined #openstack-cinder 2015-01-08T03:59:07 *** k_ has joined #openstack-cinder 2015-01-08T04:01:01 *** bkopilov has quit IRC 2015-01-08T04:02:42 *** chlong has quit IRC 2015-01-08T04:09:41 *** jungleboyj has joined #openstack-cinder 2015-01-08T04:10:34 John Griffith proposed openstack/cinder: Use cinder.utils.execute directly https://review.openstack.org/145630 2015-01-08T04:17:34 jgriffith: you around? 2015-01-08T04:18:13 *** chlong has joined #openstack-cinder 2015-01-08T04:20:01 *** _cjones_ has quit IRC 2015-01-08T04:22:07 *** fandi has joined #openstack-cinder 2015-01-08T04:23:35 *** fandi has quit IRC 2015-01-08T04:24:00 *** fandi has joined #openstack-cinder 2015-01-08T04:24:02 *** fandi has quit IRC 2015-01-08T04:26:16 *** fandi has joined #openstack-cinder 2015-01-08T04:28:53 *** annashen has joined #openstack-cinder 2015-01-08T04:30:16 *** scottda_ has quit IRC 2015-01-08T04:35:18 Mike Perez proposed openstack/python-cinderclient: Remove unused service catalog module https://review.openstack.org/145686 2015-01-08T04:36:06 *** nosnos has joined #openstack-cinder 2015-01-08T04:39:18 *** ebalduf has quit IRC 2015-01-08T04:39:35 *** lcurtis has quit IRC 2015-01-08T04:42:08 *** annashen has quit IRC 2015-01-08T04:42:54 *** ebalduf has joined #openstack-cinder 2015-01-08T04:45:12 *** rakesh_ has joined #openstack-cinder 2015-01-08T04:45:33 *** rakesh_ is now known as Guest45532 2015-01-08T04:45:57 *** Longgeek has joined #openstack-cinder 2015-01-08T04:46:18 rakesh_mishra510 2015-01-08T04:46:46 *** Guest45532 is now known as rakesh_mishra 2015-01-08T04:47:19 *** ebalduf has quit IRC 2015-01-08T04:50:12 *** david-lyle has quit IRC 2015-01-08T04:51:55 *** winston-d_ has quit IRC 2015-01-08T04:51:58 *** dannywilson has quit IRC 2015-01-08T05:00:35 *** annashen has joined #openstack-cinder 2015-01-08T05:05:53 *** winston-d_ has joined #openstack-cinder 2015-01-08T05:06:30 *** anshul has joined #openstack-cinder 2015-01-08T05:10:44 *** annashen has quit IRC 2015-01-08T05:12:48 *** _cjones_ has joined #openstack-cinder 2015-01-08T05:14:31 *** winston-d_ has quit IRC 2015-01-08T05:14:48 *** winston-d_ has joined #openstack-cinder 2015-01-08T05:17:18 *** _cjones_ has quit IRC 2015-01-08T05:19:33 *** annashen has joined #openstack-cinder 2015-01-08T05:19:51 *** annashen has quit IRC 2015-01-08T05:26:14 Alan Jiang proposed openstack/cinder: rbd calls block eventlet threads https://review.openstack.org/145678 2015-01-08T05:26:59 *** annashen has joined #openstack-cinder 2015-01-08T05:27:54 *** BharatK has joined #openstack-cinder 2015-01-08T05:30:09 *** bill_az has joined #openstack-cinder 2015-01-08T05:30:43 *** annashen has quit IRC 2015-01-08T05:32:39 *** Mandell has joined #openstack-cinder 2015-01-08T05:36:46 *** nshaikh has joined #openstack-cinder 2015-01-08T05:43:41 *** ebalduf has joined #openstack-cinder 2015-01-08T05:48:34 Mike Perez proposed openstack/python-cinderclient: Remove v1 support https://review.openstack.org/145688 2015-01-08T05:48:39 *** ebalduf has quit IRC 2015-01-08T05:52:07 *** rakesh_mishra has quit IRC 2015-01-08T05:53:34 *** winston-d_ has quit IRC 2015-01-08T05:58:36 *** rakesh_mishra has joined #openstack-cinder 2015-01-08T05:58:44 *** mudassirlatif has joined #openstack-cinder 2015-01-08T06:00:39 *** winston-d_ has joined #openstack-cinder 2015-01-08T06:04:10 OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/144933 2015-01-08T06:08:02 *** rushil has quit IRC 2015-01-08T06:12:06 *** RJ_ has joined #openstack-cinder 2015-01-08T06:14:17 *** mudassirlatif has quit IRC 2015-01-08T06:17:22 *** mudassirlatif has joined #openstack-cinder 2015-01-08T06:19:19 Trung Trinh proposed openstack/cinder: Delete volume attached to already-deleted VM https://review.openstack.org/142006 2015-01-08T06:26:01 *** mudassirlatif has quit IRC 2015-01-08T06:27:01 *** winston-d_ has quit IRC 2015-01-08T06:29:43 *** winston-d_ has joined #openstack-cinder 2015-01-08T06:33:29 *** jamielennox is now known as jamielennox|away 2015-01-08T06:33:29 *** bkopilov has joined #openstack-cinder 2015-01-08T06:41:54 *** nshaikh has quit IRC 2015-01-08T06:44:33 *** ebalduf has joined #openstack-cinder 2015-01-08T06:46:49 jgriffith: you had mentioned previously that the generic iscsi-helper code was being re-written for kilo 2015-01-08T06:47:25 is there a place to follow this so that I make sure I don't end up writing deprecated code wrt https://review.openstack.org/#/c/134398/ 2015-01-08T06:50:43 *** ebalduf has quit IRC 2015-01-08T06:52:38 *** pradipta_away is now known as pradipta 2015-01-08T06:53:17 *** nlevinki has joined #openstack-cinder 2015-01-08T06:54:38 *** mogami has joined #openstack-cinder 2015-01-08T06:56:27 anish: targets are now being created in https://github.com/openstack/cinder/tree/master/cinder/volume/targets 2015-01-08T06:56:50 example, tgt and lio being iscsi helpers 2015-01-08T06:57:32 *** nkrinner has joined #openstack-cinder 2015-01-08T06:57:42 yep, saw that 2015-01-08T06:57:45 iet seems to have gone away ? 2015-01-08T06:59:36 *** mogami has quit IRC 2015-01-08T07:00:15 Mike Perez proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 2015-01-08T07:01:17 anish: not sure about iet, sorry 2015-01-08T07:02:22 no worries, I just ask because I was working on this before the holidays, and a git pull basically broke everything 2015-01-08T07:02:30 thingee: just wanted to confirm if there are more changes coming 2015-01-08T07:02:57 *** Longgeek has quit IRC 2015-01-08T07:03:02 *** lpetrut has joined #openstack-cinder 2015-01-08T07:03:39 anish: We have bug fixes, but that should be it in terms of big move changes 2015-01-08T07:03:52 excellent ! 2015-01-08T07:03:53 *** Longgeek has joined #openstack-cinder 2015-01-08T07:04:36 *** afazekas is now known as __afazekas 2015-01-08T07:04:40 Trung Trinh proposed openstack/cinder: Delete volume attached to already-deleted VM https://review.openstack.org/142006 2015-01-08T07:04:53 *** winston-d_ has quit IRC 2015-01-08T07:06:29 *** winston-d_ has joined #openstack-cinder 2015-01-08T07:08:02 *** coolsvap|afk is now known as coolsvap 2015-01-08T07:11:15 *** e0ne has joined #openstack-cinder 2015-01-08T07:11:22 *** karimb has joined #openstack-cinder 2015-01-08T07:13:28 *** bill_az has quit IRC 2015-01-08T07:17:11 *** EduardBiceriMate has joined #openstack-cinder 2015-01-08T07:23:02 *** Mandell has quit IRC 2015-01-08T07:24:23 jgriffith: now I am ... but that won't help much, I guess. 2015-01-08T07:24:39 *** winston-d_ has quit IRC 2015-01-08T07:25:08 *** Mandell has joined #openstack-cinder 2015-01-08T07:25:50 flip214: :) 2015-01-08T07:28:13 thingee: I guess I know what john wants, already reviewing / replying. 2015-01-08T07:31:18 thingee: [x] 145644 done 2015-01-08T07:38:57 *** k4n0 has joined #openstack-cinder 2015-01-08T07:42:24 *** k4n0 has quit IRC 2015-01-08T07:45:17 Marc Koderer proposed openstack/cinder: Update README.rst to current state https://review.openstack.org/145519 2015-01-08T07:46:42 *** afazekas_ has joined #openstack-cinder 2015-01-08T07:47:08 *** chlong has quit IRC 2015-01-08T07:47:18 *** TobiasE has joined #openstack-cinder 2015-01-08T07:51:49 *** nshaikh has joined #openstack-cinder 2015-01-08T07:52:53 *** dannywilson has joined #openstack-cinder 2015-01-08T07:54:27 *** e0ne has quit IRC 2015-01-08T07:57:05 *** dannywilson has quit IRC 2015-01-08T07:59:53 *** rakesh_mishra has quit IRC 2015-01-08T08:00:05 *** rakesh_mishra has joined #openstack-cinder 2015-01-08T08:17:39 *** thingee has quit IRC 2015-01-08T08:18:48 *** sgotliv has joined #openstack-cinder 2015-01-08T08:21:21 *** delattec has joined #openstack-cinder 2015-01-08T08:21:26 *** rakesh_mishra has quit IRC 2015-01-08T08:23:23 *** cdelatte has quit IRC 2015-01-08T08:25:25 TaoBai proposed openstack/cinder: IBM Storwize driver Consistency Group Implementation https://review.openstack.org/142683 2015-01-08T08:29:33 *** ndipanov has joined #openstack-cinder 2015-01-08T08:31:04 *** pradipta is now known as pradipta_away 2015-01-08T08:31:40 *** alexpilotti has joined #openstack-cinder 2015-01-08T08:36:15 *** alexpilotti has quit IRC 2015-01-08T08:36:31 *** alexpilotti has joined #openstack-cinder 2015-01-08T08:38:37 *** Mandell has quit IRC 2015-01-08T08:40:10 *** jkraj has joined #openstack-cinder 2015-01-08T08:48:40 *** ebalduf has joined #openstack-cinder 2015-01-08T08:51:30 *** anshul has quit IRC 2015-01-08T08:53:04 *** anshul has joined #openstack-cinder 2015-01-08T08:53:22 *** ebalduf has quit IRC 2015-01-08T08:54:50 *** jpich has joined #openstack-cinder 2015-01-08T09:10:14 *** jistr has joined #openstack-cinder 2015-01-08T09:12:40 *** jordanP has joined #openstack-cinder 2015-01-08T09:13:55 Jeegn Chen proposed openstack/cinder: EMC VNX Cinder Driver Update https://review.openstack.org/141629 2015-01-08T09:17:15 *** lpetrut has quit IRC 2015-01-08T09:26:55 *** dalgaaf has joined #openstack-cinder 2015-01-08T09:28:47 *** tshefi has joined #openstack-cinder 2015-01-08T09:31:06 *** alecv has joined #openstack-cinder 2015-01-08T09:38:53 *** Guest57286 has joined #openstack-cinder 2015-01-08T09:45:08 *** nellysmitt has joined #openstack-cinder 2015-01-08T09:45:29 *** jordanP has quit IRC 2015-01-08T09:45:44 *** jordanP has joined #openstack-cinder 2015-01-08T09:49:39 *** ebalduf has joined #openstack-cinder 2015-01-08T09:51:38 *** RJ_ has left #openstack-cinder 2015-01-08T09:52:51 *** sgotliv has quit IRC 2015-01-08T09:54:39 *** ebalduf has quit IRC 2015-01-08T09:55:04 *** dulek has joined #openstack-cinder 2015-01-08T10:06:09 *** sgotliv has joined #openstack-cinder 2015-01-08T10:09:43 *** lpetrut has joined #openstack-cinder 2015-01-08T10:10:52 *** rakesh__mishra has joined #openstack-cinder 2015-01-08T10:17:18 *** TobiasE has quit IRC 2015-01-08T10:17:36 *** RJ_ has joined #openstack-cinder 2015-01-08T10:23:01 *** e0ne has joined #openstack-cinder 2015-01-08T10:27:34 *** Masahiro has quit IRC 2015-01-08T10:32:13 *** rushiagr_away is now known as rushiagr 2015-01-08T10:33:14 *** rushiagr is now known as rushiagr_away 2015-01-08T10:42:53 *** smoriya has quit IRC 2015-01-08T10:48:06 *** e0ne_ has joined #openstack-cinder 2015-01-08T10:48:50 *** e0ne has quit IRC 2015-01-08T10:50:24 *** ebalduf has joined #openstack-cinder 2015-01-08T10:55:15 *** ebalduf has quit IRC 2015-01-08T10:55:30 . 2015-01-08T10:55:41 thingee: ping 2015-01-08T10:56:13 *** haomaiwang has quit IRC 2015-01-08T10:57:44 *** karimb has quit IRC 2015-01-08T11:05:38 hi 2015-01-08T11:06:18 i am facing the error while creating the volume with type having capabilities:replication= True in key 2015-01-08T11:06:19 ..? 2015-01-08T11:07:57 http://paste.openstack.org/show/155969/ 2015-01-08T11:10:12 *** deepakcs has joined #openstack-cinder 2015-01-08T11:25:04 *** TobiasE has joined #openstack-cinder 2015-01-08T11:43:47 *** chlong has joined #openstack-cinder 2015-01-08T11:51:08 *** ebalduf has joined #openstack-cinder 2015-01-08T11:54:29 *** BharatK has quit IRC 2015-01-08T11:55:43 *** ebalduf has quit IRC 2015-01-08T12:02:24 *** Miouge has quit IRC 2015-01-08T12:03:14 *** Miouge has joined #openstack-cinder 2015-01-08T12:08:09 *** ho has quit IRC 2015-01-08T12:08:14 *** Miouge has quit IRC 2015-01-08T12:16:21 *** Masahiro has joined #openstack-cinder 2015-01-08T12:20:43 *** Masahiro has quit IRC 2015-01-08T12:24:52 git-harry proposed openstack/python-cinderclient: v2 error message grammatical error https://review.openstack.org/145767 2015-01-08T12:24:59 *** sandywalsh has quit IRC 2015-01-08T12:25:51 *** k_ has quit IRC 2015-01-08T12:26:53 *** rakesh__mishra has quit IRC 2015-01-08T12:30:57 *** nosnos has quit IRC 2015-01-08T12:35:35 *** deepakcs has quit IRC 2015-01-08T12:38:39 *** karimb has joined #openstack-cinder 2015-01-08T12:40:02 *** IanGovett has joined #openstack-cinder 2015-01-08T12:50:25 *** dulek has quit IRC 2015-01-08T12:51:56 *** ebalduf has joined #openstack-cinder 2015-01-08T12:53:10 *** dulek has joined #openstack-cinder 2015-01-08T12:54:47 *** ryanpetrello has joined #openstack-cinder 2015-01-08T12:56:49 *** ebalduf has quit IRC 2015-01-08T12:59:12 *** sgotliv_ has joined #openstack-cinder 2015-01-08T13:00:14 *** sgotliv has quit IRC 2015-01-08T13:03:21 *** Miouge has joined #openstack-cinder 2015-01-08T13:07:59 Pratik Das proposed openstack/python-cinderclient: Removed --name from cinder list as there was no NAME column in cinder volumes database. https://review.openstack.org/145774 2015-01-08T13:13:53 *** anshul has quit IRC 2015-01-08T13:16:31 *** ryanpetrello has quit IRC 2015-01-08T13:17:16 *** ryanpetrello has joined #openstack-cinder 2015-01-08T13:19:13 *** bswartz has quit IRC 2015-01-08T13:24:33 nikeshmahalka proposed openstack/cinder: Add iSCSI SCST Target support to cinder. https://review.openstack.org/145778 2015-01-08T13:26:22 *** alexpilotti has quit IRC 2015-01-08T13:29:55 *** bill_az has joined #openstack-cinder 2015-01-08T13:33:38 Anton Arefiev proposed openstack/cinder: Import only modules in volume manager https://review.openstack.org/145780 2015-01-08T13:39:20 *** nikesh_vedams has joined #openstack-cinder 2015-01-08T13:50:52 *** ryanpetrello has quit IRC 2015-01-08T13:51:41 *** e0ne_ has quit IRC 2015-01-08T13:52:36 *** kaufer has joined #openstack-cinder 2015-01-08T13:52:40 *** ebalduf has joined #openstack-cinder 2015-01-08T13:54:51 *** jcru has joined #openstack-cinder 2015-01-08T13:55:00 *** timcl has joined #openstack-cinder 2015-01-08T13:56:12 *** bswartz has joined #openstack-cinder 2015-01-08T13:56:49 *** jungleboyj has quit IRC 2015-01-08T13:56:54 *** ebalduf has quit IRC 2015-01-08T13:58:55 *** jaypipes has joined #openstack-cinder 2015-01-08T13:58:57 *** e0ne has joined #openstack-cinder 2015-01-08T14:01:23 *** nlevinki has quit IRC 2015-01-08T14:03:08 Hi,i have a small doubt,i want to create a Gerrit CI account and i already have a launchpad account for submitting patch,so for CI account can i use this launchpad account? 2015-01-08T14:04:09 i used this launchpad account for registering blueprint and submitting patch for blueprint 2015-01-08T14:04:31 nikesh_vedams: yes, you can use that same account with openstack gerrit. 2015-01-08T14:05:07 *** Masahiro has joined #openstack-cinder 2015-01-08T14:06:56 larsks: this link is confusing for me http://ci.openstack.org/third_party.html 2015-01-08T14:07:20 larsks: under Creating a Service Account 2015-01-08T14:07:56 larsks: its saying "You may already have an existing personal account in Launchpad, but you should create a new and entirely separate account for this purpose." 2015-01-08T14:08:18 nikesh_vedams: that page is about setting up a third party CI envrionment, not about submitting reviews to gerrit. Which are you trying to do? 2015-01-08T14:08:25 I may have misunderstood. 2015-01-08T14:08:44 larsks: i want to set up a CI setup 2015-01-08T14:09:04 Okay. In that case, you *would* want to create a new account specifically for the CI setup. 2015-01-08T14:09:10 *** Yogi1 has joined #openstack-cinder 2015-01-08T14:09:31 So that the results of CI testing can be distinguished from your own comments on patches. 2015-01-08T14:09:40 *** akerr has joined #openstack-cinder 2015-01-08T14:09:43 *** Masahiro has quit IRC 2015-01-08T14:11:04 larsks: ok got it,thanks 2015-01-08T14:12:17 larsks:i want to set up "sos-ci",but stuggling where to start 2015-01-08T14:12:39 larsks:https://github.com/j-griffith/sos-ci 2015-01-08T14:12:53 *** esker has joined #openstack-cinder 2015-01-08T14:13:10 *** esker has quit IRC 2015-01-08T14:14:01 nikesh_vedams: That I'm not familiar with. 2015-01-08T14:15:51 larsks:ok no probs :) 2015-01-08T14:16:11 *** nlevinki has joined #openstack-cinder 2015-01-08T14:19:34 *** ndipanov has quit IRC 2015-01-08T14:21:20 *** ndipanov has joined #openstack-cinder 2015-01-08T14:21:31 *** mriedem has joined #openstack-cinder 2015-01-08T14:25:12 Boris Pavlovic proposed openstack/cinder: Add more rally benchmarks related to Cinder https://review.openstack.org/145595 2015-01-08T14:28:01 Boris Pavlovic proposed openstack/cinder: Add more rally benchmarks related to Cinder https://review.openstack.org/145595 2015-01-08T14:30:28 *** xyang has joined #openstack-cinder 2015-01-08T14:30:56 *** alexpilotti has joined #openstack-cinder 2015-01-08T14:31:54 Steven Kaufer proposed openstack/cinder: GET volumes API sorting REST/volume/DB updates https://review.openstack.org/141915 2015-01-08T14:31:54 Steven Kaufer proposed openstack/cinder: GET volumes API sorting enhancements common utilities https://review.openstack.org/141914 2015-01-08T14:33:25 *** nshaikh has quit IRC 2015-01-08T14:35:54 *** dustins has joined #openstack-cinder 2015-01-08T14:42:02 *** thangp has joined #openstack-cinder 2015-01-08T14:53:24 *** ebalduf has joined #openstack-cinder 2015-01-08T14:53:43 *** lcurtis has joined #openstack-cinder 2015-01-08T14:58:11 *** ebalduf has quit IRC 2015-01-08T14:58:43 *** ling-yun-1 has joined #openstack-cinder 2015-01-08T15:02:13 *** zengyunling__ has quit IRC 2015-01-08T15:05:17 *** ndipanov has quit IRC 2015-01-08T15:09:32 *** Miouge has quit IRC 2015-01-08T15:09:43 *** EduardBiceriMate has quit IRC 2015-01-08T15:11:16 *** mtanino has joined #openstack-cinder 2015-01-08T15:11:42 *** ryanpetrello has joined #openstack-cinder 2015-01-08T15:12:06 *** rushil has joined #openstack-cinder 2015-01-08T15:13:56 *** akerr_ has joined #openstack-cinder 2015-01-08T15:14:05 *** lpabon has joined #openstack-cinder 2015-01-08T15:14:32 *** akerr has quit IRC 2015-01-08T15:15:11 *** vmtyler has joined #openstack-cinder 2015-01-08T15:17:14 *** ebalduf has joined #openstack-cinder 2015-01-08T15:17:41 *** nlevinki has quit IRC 2015-01-08T15:18:00 *** nlevinki has joined #openstack-cinder 2015-01-08T15:18:09 *** ndipanov has joined #openstack-cinder 2015-01-08T15:19:43 *** Longgeek has quit IRC 2015-01-08T15:21:38 *** Longgeek has joined #openstack-cinder 2015-01-08T15:24:40 *** thingee has joined #openstack-cinder 2015-01-08T15:25:34 *** tbarron has joined #openstack-cinder 2015-01-08T15:25:54 *** takedakn has joined #openstack-cinder 2015-01-08T15:27:23 *** takedakn has quit IRC 2015-01-08T15:27:44 *** takedakn has joined #openstack-cinder 2015-01-08T15:28:08 *** takedakn has quit IRC 2015-01-08T15:28:34 *** takedakn has joined #openstack-cinder 2015-01-08T15:32:39 *** Mandell has joined #openstack-cinder 2015-01-08T15:32:53 Mudassir Latif proposed openstack/cinder: Add replication and PG setup methods on PureISCSIDriver https://review.openstack.org/145090 2015-01-08T15:33:02 *** Miouge has joined #openstack-cinder 2015-01-08T15:35:14 *** tshefi has quit IRC 2015-01-08T15:35:14 *** takedakn has quit IRC 2015-01-08T15:36:48 *** jecarey has joined #openstack-cinder 2015-01-08T15:38:22 *** alecv has quit IRC 2015-01-08T15:39:04 *** vmtyler has quit IRC 2015-01-08T15:42:49 *** fischerw has joined #openstack-cinder 2015-01-08T15:43:41 *** alecv has joined #openstack-cinder 2015-01-08T15:43:42 *** Mandell has quit IRC 2015-01-08T15:47:07 *** e0ne is now known as e0ne_ 2015-01-08T15:50:29 *** lpetrut has quit IRC 2015-01-08T15:51:07 *** nlevinki has quit IRC 2015-01-08T15:53:54 *** Masahiro has joined #openstack-cinder 2015-01-08T15:54:24 *** mberlin1 has quit IRC 2015-01-08T15:55:17 *** mberlin has joined #openstack-cinder 2015-01-08T15:55:42 *** coolsvap is now known as coolsvap|afk 2015-01-08T15:57:11 *** e0ne_ has quit IRC 2015-01-08T15:58:10 *** TobiasE has quit IRC 2015-01-08T15:58:10 *** tsekiyama has joined #openstack-cinder 2015-01-08T15:58:31 *** Masahiro has quit IRC 2015-01-08T15:59:21 *** TobiasE has joined #openstack-cinder 2015-01-08T16:00:10 *** dulek has quit IRC 2015-01-08T16:00:43 *** dulek has joined #openstack-cinder 2015-01-08T16:03:14 *** RJ_ has left #openstack-cinder 2015-01-08T16:04:22 *** changbl has quit IRC 2015-01-08T16:08:56 hey thingee, I could use help triaging https://bugs.launchpad.net/openstack-api-site/+bug/1331246 -- basically what are attachments and snapshot_id in responses for GET volume for v1 API? 2015-01-08T16:09:25 *** alecv has quit IRC 2015-01-08T16:09:51 *** e0ne has joined #openstack-cinder 2015-01-08T16:15:44 *** changbl has joined #openstack-cinder 2015-01-08T16:16:11 *** nshaikh has joined #openstack-cinder 2015-01-08T16:16:15 Yuriy Nesenenko proposed openstack/cinder: PEP8(E265): block comment should start with '# ' https://review.openstack.org/145821 2015-01-08T16:18:41 *** afazekas_ has quit IRC 2015-01-08T16:19:19 *** dulek has quit IRC 2015-01-08T16:20:40 *** lpabon_ has joined #openstack-cinder 2015-01-08T16:20:44 Rushil Chugh proposed openstack/cinder: Garbage Remains when Attached Volume is Migrated with NFS Driver https://review.openstack.org/143431 2015-01-08T16:23:07 *** xyang has quit IRC 2015-01-08T16:23:26 *** alecv has joined #openstack-cinder 2015-01-08T16:23:54 jgriffith, eharney: hi! some thoughts about https://bugs.launchpad.net/cinder/+bug/1373513: i've fixed similar issue in Ubuntu upgrading lvm2 package to v2.0.111. for now, i've got errors on centos with lvm2 v2.0.100. may be it isn't cinder issue. i'm still investigating it 2015-01-08T16:26:04 *** sgotliv_ has quit IRC 2015-01-08T16:26:25 Yuriy Nesenenko proposed openstack/cinder: PEP8(E265): block comment should start with '# ' https://review.openstack.org/145821 2015-01-08T16:29:52 *** ebalduf has quit IRC 2015-01-08T16:29:58 *** Mandell has joined #openstack-cinder 2015-01-08T16:30:31 *** EduardBiceriMate has joined #openstack-cinder 2015-01-08T16:30:57 *** jkraj has quit IRC 2015-01-08T16:31:06 e0ne: have you seen anything indicating why upgrading helped? 2015-01-08T16:32:07 eharney: a very similar bug in debian https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=659762 2015-01-08T16:32:45 *** Mandell has quit IRC 2015-01-08T16:33:19 *** Mandell has joined #openstack-cinder 2015-01-08T16:34:02 *** chlong has quit IRC 2015-01-08T16:34:36 *** ebalduf has joined #openstack-cinder 2015-01-08T16:38:29 *** Mandell has quit IRC 2015-01-08T16:39:56 *** sgotliv_ has joined #openstack-cinder 2015-01-08T16:44:57 *** nellysmitt has quit IRC 2015-01-08T16:48:10 Yuriy Nesenenko proposed openstack/cinder: Fixed comments style according to the Hacking Rules https://review.openstack.org/145821 2015-01-08T16:48:19 *** lpetrut has joined #openstack-cinder 2015-01-08T16:51:16 *** lpabon_ has quit IRC 2015-01-08T16:51:23 *** jecarey has quit IRC 2015-01-08T16:53:04 *** lpetrut has quit IRC 2015-01-08T16:54:52 Navneet Singh proposed openstack/cinder: Implement Cinder Manage/Unmanage APIs in NFS driver. https://review.openstack.org/138125 2015-01-08T16:55:19 Hi We have commited our scst code. https://review.openstack.org/#/c/145778/ To have CI setup,we are following link: https://github.com/j-griffith/sos-ci 2015-01-08T16:55:54 We have a Ubuntu 14.04 server VM and we will integrate SCST changes in this VM. Now how can we use this VM for CI setup. 2015-01-08T16:56:20 We are also using this VM for commiting our scst patch 2015-01-08T16:57:45 We have differnt launchpad account for patch submission and CI account 2015-01-08T16:59:28 *** TobiasE1 has joined #openstack-cinder 2015-01-08T17:00:00 But we want to know how to setup our VM with https://github.com/j-griffith/sos-ci CI for testing our https://review.openstack.org/#/c/145778 patch 2015-01-08T17:00:13 *** patrickeast has joined #openstack-cinder 2015-01-08T17:00:43 please guide us 2015-01-08T17:00:52 *** _cjones_ has joined #openstack-cinder 2015-01-08T17:00:52 *** jecarey has joined #openstack-cinder 2015-01-08T17:01:37 *** TobiasE has quit IRC 2015-01-08T17:03:32 *** xyang has joined #openstack-cinder 2015-01-08T17:03:48 *** e0ne has quit IRC 2015-01-08T17:04:15 *** timcl has quit IRC 2015-01-08T17:05:53 *** andymaier has joined #openstack-cinder 2015-01-08T17:07:13 *** esker has joined #openstack-cinder 2015-01-08T17:08:42 Hi, any suggestion for renaming 'target_portals / target_iqns / target_luns' in iSCSI multipath patch https://review.openstack.org/#/c/134681/ ? thingee told me these need better names when the cinder-spec was approved. 2015-01-08T17:11:11 hemna: thinking more about detach my thoughts about the detach method signature might have been kinda stupid :) 2015-01-08T17:11:29 hemna: WRT multi-attach 2015-01-08T17:11:35 jgriffith, ah ok. :) I was hoping to circle back on that today 2015-01-08T17:12:00 hemna: haha... you know me well enough that if you ignore me for a day or two I may change my mind or forget :) 2015-01-08T17:12:04 :P 2015-01-08T17:12:22 now if I can just get the javelin tests to pass..... 2015-01-08T17:13:45 John Griffith proposed openstack/cinder: Implement clone_image caching on SolidFire https://review.openstack.org/142859 2015-01-08T17:15:43 *** EmilienM is now known as EmilienM|afk 2015-01-08T17:16:06 winston-d_zZZ: ping... 2015-01-08T17:16:14 doh! the dreaded zZZ 2015-01-08T17:17:59 Patrick East proposed openstack/cinder: Add support for manage/unmanage volume commands to PureISCSIDriver https://review.openstack.org/143726 2015-01-08T17:18:03 *** nshaikh has quit IRC 2015-01-08T17:19:43 jgriffith: did you see all the v1 removal patches? :) 2015-01-08T17:19:57 thingee: no... 2015-01-08T17:20:02 https://review.openstack.org/#/c/145703/ 2015-01-08T17:20:12 https://review.openstack.org/#/c/145688/ 2015-01-08T17:20:30 thingee: got'em 2015-01-08T17:20:45 thingee: only thing is do we really want to "remove" them? 2015-01-08T17:21:07 I thought that was the plan for Kilo. 2015-01-08T17:21:15 thingee: and part of the only reason I ask is that although we had it enabled in Grizzly, it wasn't default until Juno 2015-01-08T17:21:25 thingee: it was... 2015-01-08T17:21:47 jgriffith: https://github.com/openstack/cinder/blob/master/cinder/api/__init__.py#L31 2015-01-08T17:21:48 I'm just making sure we're thinking it through and not shooting ourselves in the foot 2015-01-08T17:21:53 or pissin a bunch of people off :) 2015-01-08T17:22:08 jgriffith: I agree, but we gotta rip the bandaid at some point 2015-01-08T17:22:23 thingee: yeah... and your log message is pretty clear :) 2015-01-08T17:22:47 unfortunately openstackclient, heat, tripleo, etc need to be updated to use the cinderclient itself instead of assuming volumev2 is the service type. so that discovery works 2015-01-08T17:23:07 ugh 2015-01-08T17:23:16 thingee: I'm happy to help on some of those.... 2015-01-08T17:23:21 or at least openstackclient 2015-01-08T17:23:30 jgriffith: https://github.com/Thingee/python-openstackclient/commit/4f32c6755346ab5068533bd9c5963ff60ab8e6a6 2015-01-08T17:23:43 I'm concerne about some other externals (ansible, shade etc) 2015-01-08T17:24:05 thingee: oh fine.. take the one that I was interested in helping with :) 2015-01-08T17:24:20 thingee: I want nothing to do with OOO 2015-01-08T17:24:39 *** andymaier has quit IRC 2015-01-08T17:24:44 thingee: that being said, I'll help out on anything you like, just give me orders :) 2015-01-08T17:24:45 *** timcl has joined #openstack-cinder 2015-01-08T17:24:51 jgriffith: https://github.com/openstack/heat/blob/master/heat/engine/clients/os/cinder.py#L39 2015-01-08T17:25:08 \\csim-cloud.rose.hp.com\CloudGroupShare\testing\perf\logs 2015-01-08T17:25:13 thingee: oh... so did you do all of them already? 2015-01-08T17:25:30 jgriffith: nah, that's master of heat..was pointing out the use of volumev2 2015-01-08T17:26:05 wrong channel 2015-01-08T17:26:48 *** nkrinner has quit IRC 2015-01-08T17:27:12 jgriffith, jamielennox|away: so I noticed a problem with https://review.openstack.org/#/c/145613/. 2015-01-08T17:27:44 jgriffith:hi 2015-01-08T17:27:58 nikesh_vedams: hello there 2015-01-08T17:28:14 thingee: yeah? 2015-01-08T17:28:22 jamielennox|away: besides the problems you pointed out 2015-01-08T17:28:48 jgriffith:uploaded patch,need some help :) 2015-01-08T17:28:56 in CI 2015-01-08T17:29:14 nikesh_vedams: looking 2015-01-08T17:29:21 jgriffith: so remember these https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v2/shell.py#L190 ? 2015-01-08T17:29:33 nikesh_vedams: pep8 and pylint, no big deal 2015-01-08T17:30:26 thingee: oh 2015-01-08T17:30:49 jgriffith: we want to know how to setup our VM with https://github.com/j-griffith/sos-ci CI for testing our https://review.openstack.org/#/c/145778 patch 2015-01-08T17:30:50 thingee: that's problematic 2015-01-08T17:31:35 nikesh_vedams: so sos-ci is meant to interact with an OpenStack cloud... run devstack and Temest on Instances in that Cloud 2015-01-08T17:31:45 *** jpich has quit IRC 2015-01-08T17:31:55 jgriffith: these are making my life harder for version discovery with backwards compat. If I have x.x.x.x:8776 as the publicURL and service_type 'volume'...I'll get endpoint not found if I have OS_VOLUME_API_VERSION=1 set. 2015-01-08T17:31:56 You need to deploy OpenStack and point sos-ci at that 2015-01-08T17:32:06 jgriffith: because this will set volumev2 for the service_type https://github.com/openstack/python-cinderclient/blob/master/cinderclient/shell.py#L578 2015-01-08T17:32:14 thingee: can we do a mapping trick? 2015-01-08T17:32:34 thingee: because we can't change the client 2015-01-08T17:32:46 thingee: I mean, we can't count on the client not being "older" 2015-01-08T17:34:12 jgriffith: how can we create this Openstack cloud,we have a VM with ubuntu 14.04 server and also we have a ESXi Host 2015-01-08T17:34:15 jgriffith: well so first question, we need to decide if it's worth making the cinderclient backwards compat. so either fix up https://review.openstack.org/#/c/145613/ to do some hack of if endpoint not found is raised, we fall back to service_type 'volume' 2015-01-08T17:34:17 jgriffith: or 2015-01-08T17:34:40 we assume in a major cinderclient release that v1 is gone https://review.openstack.org/#/c/145688/ 2015-01-08T17:34:43 nikesh_vedams: if you don't know how to deploy OpenStack yet you've got a very very long way to go I'm afraid 2015-01-08T17:34:53 and that the service_catalog needs to be updated before upgrading to kilo 2015-01-08T17:35:07 service type has to be volume and endpoint has to be x.x.x.x:8776 2015-01-08T17:35:10 thingee: but you can't assume people have upgraded the client... I mean you can't break their existing clients IMHO 2015-01-08T17:35:24 hmmm.... 2015-01-08T17:35:44 jgriffith: that's true. 2015-01-08T17:36:24 thingee: I mean moving everything to default to v2 is def good, and even "hiding" v1 a little 2015-01-08T17:36:47 jgriffith:so i know how to deploy it by http://docs.openstack.org/juno/install-guide/install/apt/content/ 2015-01-08T17:37:12 nikesh_vedams: you need to be looking at devstack IMHO 2015-01-08T17:39:04 *** dustins_ has joined #openstack-cinder 2015-01-08T17:39:30 jgriffith: any link will be greatly helpful :) 2015-01-08T17:39:46 nikesh_vedams: http://devstack.org 2015-01-08T17:40:27 *** jwcroppe has quit IRC 2015-01-08T17:41:02 *** jwcroppe has joined #openstack-cinder 2015-01-08T17:41:27 *** dustins has quit IRC 2015-01-08T17:41:29 jgriffith: if you have an outdated client, I guess we should make it so on the server side, you can service types volume and volumev2 point to x.x.x.x:8776? 2015-01-08T17:42:21 jgriffith: I actually haven't thought about that before, but may work...would have to test it 2015-01-08T17:43:04 *** jistr has quit IRC 2015-01-08T17:43:12 *** Masahiro has joined #openstack-cinder 2015-01-08T17:45:14 *** jwcroppe has quit IRC 2015-01-08T17:46:43 jgriffith: what a mess :) 2015-01-08T17:46:54 thingee: sigh 2015-01-08T17:47:18 I couldn't sleep for two hours last night because of this. :) 2015-01-08T17:47:29 thingee: maybe this is why nobody else has really "removed" and API version 2015-01-08T17:47:32 jgriffith thingee: sorry for silly question but what is IMHO 2015-01-08T17:47:52 nikesh_vedams: not silly... "IMHO = In My Humble Opinion" 2015-01-08T17:48:03 *** Masahiro has quit IRC 2015-01-08T17:48:15 jgriffith is very humble with his opinions :) 2015-01-08T17:48:33 hey so austin, bbq all week? 2015-01-08T17:48:34 thingee: kinda of funny that I use that term huh? 2015-01-08T17:48:35 LOL 2015-01-08T17:48:46 thingee: I'm a vegetarian 2015-01-08T17:48:49 NOT! 2015-01-08T17:48:55 *** rushiagr_away is now known as rushiagr 2015-01-08T17:48:56 jgriffith: ain't foolin' me 2015-01-08T17:49:00 :) 2015-01-08T17:49:02 *** sgotliv_ has quit IRC 2015-01-08T17:49:41 thingee: so I'm looking at some things on the allocated capacity issues vishy mentioned yesterday 2015-01-08T17:49:46 thingee: it's kind of a mess 2015-01-08T17:50:07 thingee: I'm working through some things now, I'll get a bug posted and ping you when it's up 2015-01-08T17:50:21 *** fandi has quit IRC 2015-01-08T17:51:19 jgriifith thingee : hmm then we can give Openstack colud to sos-ci by running devstack right 2015-01-08T17:51:34 nikesh_vedams: right 2015-01-08T17:51:36 jgriffith: for the creates, is it enough to notify the scheduler if we get an exception from the driver? 2015-01-08T17:51:49 thingee: that's what I'm messing with right now 2015-01-08T17:52:06 thingee: it "should" 2015-01-08T17:52:10 jgriffith: Ok, wasn't sure if it was as simple as that. I tried bugging winston-d_zZZ about it last night, but I never heard from him 2015-01-08T17:52:23 jgriiffith thingee : thanks 2015-01-08T17:52:28 nikesh_vedams: np 2015-01-08T17:52:31 *** lpetrut has joined #openstack-cinder 2015-01-08T17:52:37 Thang Pham proposed openstack/cinder-specs: Cinder objects https://review.openstack.org/130044 2015-01-08T17:52:42 well, so it's kinda funky... esp with pools in the mix 2015-01-08T17:52:59 jgriffith: yeah my head was starting to hurt when I saw the pool docstring :) 2015-01-08T17:52:59 anyway, I'm testing some things out on it. 2015-01-08T17:53:09 I'd like to move to just pulling from the driver for all of this 2015-01-08T17:53:14 I need to write a cheatsheet so I don't have to keep remembering things 2015-01-08T17:53:15 and don't even try and count ourselves 2015-01-08T17:53:23 thingee: welcome to my world :) 2015-01-08T17:53:27 jgriffith: +1 2015-01-08T17:53:49 thingee: the problem is now that we have 28+ drivers that's a real pain in my *$$ 2015-01-08T17:54:08 thingee: and looking at some of them and trying to determine how the hell they work is not fun 2015-01-08T17:54:20 thingee: so maybe a driver impl, with a fall back 2015-01-08T17:54:49 anyway.... need to waste another hour of my life in a pointless meeting, then be back to look at it again 2015-01-08T17:55:20 *** karimb has quit IRC 2015-01-08T17:57:48 *** TobiasE1 has quit IRC 2015-01-08T17:58:29 jgriffith: that's a good time to check out all social networking sites on your phone ;) 2015-01-08T18:00:01 I'm not back from holidays yet, so I think I should shut my lappy down and enjoy instead :) 2015-01-08T18:00:05 good night :) 2015-01-08T18:01:33 *** jordanP has quit IRC 2015-01-08T18:02:57 *** fandi has joined #openstack-cinder 2015-01-08T18:04:09 *** jwcroppe has joined #openstack-cinder 2015-01-08T18:04:42 jgriffith thingee :Like you said that sos-ci will run devstack and Temest on Instances in that Cloud,so will it create instance itself or we have to create instance manually? 2015-01-08T18:04:53 *** leeantho has joined #openstack-cinder 2015-01-08T18:06:03 *** rushiagr is now known as rushiagr_away 2015-01-08T18:06:56 nikesh_vedams: it will create the instance 2015-01-08T18:08:20 jgriffith thingee : ok but i think we have to manually install scst packages in those VMs 2015-01-08T18:09:03 jgriffith thingee : without scst packages tempest will fail 2015-01-08T18:09:10 *** dannywilson has joined #openstack-cinder 2015-01-08T18:10:44 jgriffith thingee : i mean if sos-ci create VM and deploy devstack in VM and try to run tempest,it will fail without scst packages 2015-01-08T18:12:26 nikesh_vedams: then you should modify sos-ci for your case. When it installs devstack it has a place to install additional packages https://github.com/j-griffith/sos-ci/blob/master/sos-ci/ansible/tasks/install_devstack.yml#L4 2015-01-08T18:12:47 assuming they are available via apt-get 2015-01-08T18:12:54 Mudassir Latif proposed openstack/cinder: Add replication and PG setup methods on PureISCSIDriver https://review.openstack.org/145090 2015-01-08T18:14:43 *** EmilienM|afk is now known as EmilienM 2015-01-08T18:18:25 *** annashen has joined #openstack-cinder 2015-01-08T18:19:29 jgriffith thingee : ohhh,we are getting scst packages through wget and then we are doing "make and make install" for installing scst packages and some other commands we are running for scst 2015-01-08T18:20:16 running 2015-01-08T18:20:40 and also we are editing /etc/rootwrap.d/ 2015-01-08T18:20:54 nikesh_vedams: Ok, so you'll need modify that file so ansible does that 2015-01-08T18:23:27 *** MIDENN_ has quit IRC 2015-01-08T18:23:34 *** mdenny has quit IRC 2015-01-08T18:24:03 *** e0ne has joined #openstack-cinder 2015-01-08T18:25:06 *** mdenny has joined #openstack-cinder 2015-01-08T18:25:30 jgriffith: ok whew, this will work if we backwards compat http://paste.openstack.org/show/156006/ 2015-01-08T18:25:35 *** leeantho has quit IRC 2015-01-08T18:25:53 *** mdenny has quit IRC 2015-01-08T18:30:17 *** bswartz has quit IRC 2015-01-08T18:33:21 *** mdenny has joined #openstack-cinder 2015-01-08T18:34:34 *** e0ne has quit IRC 2015-01-08T18:37:38 *** e0ne has joined #openstack-cinder 2015-01-08T18:38:54 *** junhongl has quit IRC 2015-01-08T18:39:00 *** lpabon has quit IRC 2015-01-08T18:39:20 *** junhongl has joined #openstack-cinder 2015-01-08T18:42:51 OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/145661 2015-01-08T18:43:24 *** sgotliv has joined #openstack-cinder 2015-01-08T18:45:40 *** nellysmitt has joined #openstack-cinder 2015-01-08T18:47:58 *** Mandell has joined #openstack-cinder 2015-01-08T18:48:07 *** _cjones_ has quit IRC 2015-01-08T18:49:03 jgriffith: now to deal with volumev2 being removed =/ 2015-01-08T18:49:57 *** _cjones_ has joined #openstack-cinder 2015-01-08T18:50:14 *** nellysmitt has quit IRC 2015-01-08T18:50:36 *** Mandell_ has joined #openstack-cinder 2015-01-08T18:51:42 *** Mandell has quit IRC 2015-01-08T18:59:52 *** jecarey has quit IRC 2015-01-08T19:01:49 thingee : one more question ,how sos-ci will take my patch and CI account information :) 2015-01-08T19:02:58 nikesh_vedams: environment variables or hard coded https://github.com/j-griffith/sos-ci/blob/master/sos-ci/os_ci.py#L17 2015-01-08T19:04:01 nikesh_vedams: the patch comes from the event queue that is being tested with 2015-01-08T19:05:00 *** jecarey has joined #openstack-cinder 2015-01-08T19:05:05 *** adrien1 has joined #openstack-cinder 2015-01-08T19:05:41 *** adrien1 is now known as adrienverge 2015-01-08T19:05:48 Hi everyone 2015-01-08T19:06:23 A new feature got merged into Cinder and I would like to have it documented in http://docs.openstack.org/trunk/config-reference/content/ 2015-01-08T19:07:01 How can I contribute? Where's the source of this config-reference? 2015-01-08T19:07:28 *** mriedem has quit IRC 2015-01-08T19:08:16 thingee: i want it to run only patches for this "https://review.openstack.org/#/c/145778/" 2015-01-08T19:10:51 nikesh_vedams: If you want to do an individual patch, I would recommend just creating a vm manually with devstack, but in the local.conf specify the CINDER_REPO and CINDER_BRANCH as explained here http://docs.openstack.org/developer/devstack/stackrc.html?highlight=_repo 2015-01-08T19:11:04 nikesh_vedams: no need for sos-ci if that's the case 2015-01-08T19:11:41 nikesh_vedams: sos-ci is for monitoring upstream events and automating the testing 2015-01-08T19:14:34 *** alexpilotti has quit IRC 2015-01-08T19:16:13 *** adrienverge has quit IRC 2015-01-08T19:18:17 thingee: but by this way how coud we provide resut of tempest log to gerrit 2015-01-08T19:18:42 *** ebalduf has quit IRC 2015-01-08T19:18:59 thingee: we must need some CI to provide results to gerrit right 2015-01-08T19:20:45 and so i was wondering how sos-ci wil take only our patch and provide resluts to gerrit 2015-01-08T19:22:29 *** mriedem has joined #openstack-cinder 2015-01-08T19:23:04 *** annashen has quit IRC 2015-01-08T19:24:39 *** annashen has joined #openstack-cinder 2015-01-08T19:24:52 *** EduardBiceriMate has quit IRC 2015-01-08T19:25:43 ameade: to add your phone go here - http://hr.netapp.com/how-do-i/ 2015-01-08T19:26:02 select Change - Phone #s 2015-01-08T19:26:10 fill out and submit 2015-01-08T19:26:22 oops wrong window 2015-01-08T19:27:26 hehe 2015-01-08T19:29:05 *** ameade_ is now known as ameade 2015-01-08T19:31:40 *** Masahiro has joined #openstack-cinder 2015-01-08T19:32:33 nikesh_vedams: if you have the ci setup, and reupload the patch, the idea is the ci would pull in the patch. 2015-01-08T19:33:01 *** _cjones_ has quit IRC 2015-01-08T19:33:31 *** jecarey has quit IRC 2015-01-08T19:33:44 *** jecarey has joined #openstack-cinder 2015-01-08T19:34:03 *** _cjones_ has joined #openstack-cinder 2015-01-08T19:34:49 *** Longgeek has quit IRC 2015-01-08T19:35:57 *** Masahiro has quit IRC 2015-01-08T19:36:27 *** Longgeek has joined #openstack-cinder 2015-01-08T19:37:38 *** dalgaaf has quit IRC 2015-01-08T19:38:43 thingee: ok but i have fear that it will take other patches too along with my patch so there should be some restrictions 2015-01-08T19:39:27 *** annashen has quit IRC 2015-01-08T19:40:14 nikesh_vedams: that's the idea of a CI. If that's not what you want, then you don't want continous integration. 2015-01-08T19:40:52 nikesh_vedams: if you want to do a one time test, start a vm manually, install whatevwer, run tempest, record the logs, upload it somewhere, post it in comments of the review. 2015-01-08T19:41:27 *** Longgeek has quit IRC 2015-01-08T19:41:42 *** e0ne is now known as e0ne_ 2015-01-08T19:42:55 thingee:For our patch what is good according to community https://review.openstack.org/#/c/145778/ 2015-01-08T19:43:14 thingee: having CI or manuall logs 2015-01-08T19:44:46 nikesh_vedams: definitely having a ci. If you test manually now, it may work now, but might not work later. 2015-01-08T19:45:14 nikesh_vedams: a CI would report if any proposed patch would break scst support 2015-01-08T19:45:28 because you are continuously testing 2015-01-08T19:47:07 vishy: so it seems to me that to start, we should increment allocated cap regardless of the result from the driver on create 2015-01-08T19:47:25 vishy: the volume's going to be there and allocated regardless 2015-01-08T19:47:35 *** timcl has quit IRC 2015-01-08T19:47:37 by "there" I mean in Cinder's DB 2015-01-08T19:48:03 jgriffith: we don't know where the driver failed though 2015-01-08T19:48:11 thingee: doesn't matter 2015-01-08T19:48:15 thingee: it's "allocated" 2015-01-08T19:48:23 thingee: in Cinder 2015-01-08T19:48:38 *** e0ne_ is now known as e0ne 2015-01-08T19:48:40 even if it's not used on the backend a request was made to allocate the space 2015-01-08T19:48:44 jgriffith thingee : thanks for giving your valueable time to us :) 2015-01-08T19:49:09 jgriffith: I get that, but I guess it doesn't seem accurate. not saying it's any better than now. 2015-01-08T19:49:19 thingee: but it is accurate 2015-01-08T19:49:26 Steven Kaufer proposed openstack/python-cinderclient: WIP Adopt CLI Sorting Argument Guidelines https://review.openstack.org/145913 2015-01-08T19:49:33 thingee: as opposed ot now which is NOT at all accurate 2015-01-08T19:49:48 thingee: so look at it like this.... 2015-01-08T19:50:05 you create a 1G volume.... acap = 1G 2015-01-08T19:50:19 you create another 1G vol, but it fails... 2015-01-08T19:50:37 You STILL have 2Gig of storage allocated from Cinder's perspective 2015-01-08T19:50:46 until you delete the volume that's in error state 2015-01-08T19:51:11 that being said.... 2015-01-08T19:51:25 the next step I'd like is to say; "hey, get this info via a call to the driver" 2015-01-08T19:51:38 and then, if that's not implemented fall back and use the local tracking 2015-01-08T19:51:39 but cinder is going to think 2 gigs is allocated, when it could've been a network issue between the cinder manager and the backend. The backend never knew about it. 2015-01-08T19:51:46 unless I'm still missing something here. 2015-01-08T19:51:49 thingee: but it IS allocated!!! 2015-01-08T19:51:58 thingee: there's a volume of 1G in the database 2015-01-08T19:52:03 thingee: you do a volume list and there it is 2015-01-08T19:52:08 that's allocated, 2015-01-08T19:52:15 maybe not "used" but it's been allocated 2015-01-08T19:52:25 thingee: see what I mean? 2015-01-08T19:52:34 Steven Kaufer proposed openstack/python-cinderclient: WIP Adopt CLI Sorting Argument Guidelines https://review.openstack.org/145913 2015-01-08T19:52:49 Yeah i understand now. I'm not sure I have a good understanding of what allocated means to the scheduler here then. 2015-01-08T19:53:27 thingee: so for scheduler that's how the weigher works and the capacity filter etc 2015-01-08T19:53:30 *** _cjones_ has quit IRC 2015-01-08T19:53:49 thingee: the thing is there's a difference between allocation and utilization 2015-01-08T19:53:59 and as long as we're clear and consistent on that I think it's "ok" 2015-01-08T19:54:14 because I was under the impression is was as best as we can guess is allocated from the backend. Unless we get to the expected state, it's not counted. 2015-01-08T19:54:31 not disagreeing, just talking through here. 2015-01-08T19:54:48 thingee: oh sure... yeah, talk is good :) 2015-01-08T19:54:58 thingee: but I'm not sure I understand your statement there? 2015-01-08T19:55:48 thingee: so this all kinda comes back to what we've grappled with for a while in terms of consistent definitions and reporting 2015-01-08T19:56:10 *** ebalduf has joined #openstack-cinder 2015-01-08T19:56:11 honestly I don't think it matters if we define blue as red and green as purple.... as long as it's consistent 2015-01-08T19:56:19 *** thangp has quit IRC 2015-01-08T19:56:24 * jgriffith doesn't know if that analogy helps or makes things worse :) 2015-01-08T19:56:27 so with what you were saying, allocated makes sense from cinder's perspective and is more accurate. I agree with that. However, it's not any better of being accurate of the backend, and that's not going to happen until we actually get reporting from the backend itself. 2015-01-08T19:56:51 am I accurate with that? :P 2015-01-08T19:56:56 thingee: well it's way better actually 2015-01-08T19:57:14 thingee: cuz right now if you have 5 volumes, create one and it goes to error you don't increment 2015-01-08T19:57:25 BUT... when you delete that volume it does decrement :( 2015-01-08T19:57:56 thingee: and it's nothing new in the storage industry to differentiate between allocated and used space 2015-01-08T19:58:04 sure 2015-01-08T19:58:11 I think I got it now 2015-01-08T19:58:46 and since Cinder is the application here we say "yes, you allocated it. Even if you didn't actually reserve any blocks on your device, you did in fact allocate N blocks from backend-foo" 2015-01-08T19:58:47 yeah seems reasonable to me. would be curious vishy's thoughts. 2015-01-08T19:59:08 thingee: vishy it's def just an interim improvement 2015-01-08T19:59:17 Patrick East proposed openstack/cinder: Add support for manage/unmanage volume commands to PureISCSIDriver https://review.openstack.org/143726 2015-01-08T19:59:58 *** jecarey has quit IRC 2015-01-08T20:00:35 thingee: there's some other weird things in here too, but that's a start anyway 2015-01-08T20:01:12 *** jecarey has joined #openstack-cinder 2015-01-08T20:08:20 thingee: hmmm.... so tricky part is if we do a revert on failure to create 2015-01-08T20:08:25 Navneet Singh proposed openstack/cinder: Implement Cinder Manage/Unmanage APIs in NFS driver. https://review.openstack.org/138125 2015-01-08T20:08:33 thingee: or I should say if "taskflow" does a revert 2015-01-08T20:09:08 probably taskflow, in case retries 2015-01-08T20:09:41 thingee: what I mean is we track in the manager, we don't know what taskflow is doing higher up :( 2015-01-08T20:10:00 thingee: in most cases you just get a volume in error state, but in some it actually reverts it 2015-01-08T20:10:23 so the volume gets nuked out of db, BUT your allocated cap isn't adjusted 2015-01-08T20:10:47 and the hooks into manager's cap reporting aren't exactly "readily" available 2015-01-08T20:11:02 hmm... guess I'll look at this some more 2015-01-08T20:13:28 *** fandi has quit IRC 2015-01-08T20:14:44 jgriffith: wait, I thought for allocated we were going to just go by what's known in the cinder db, instead of incrementing and decrementing. Would it not matter then? 2015-01-08T20:20:01 thingee: yes 2015-01-08T20:20:10 thingee: but... taskflow calls manager 2015-01-08T20:20:24 taskflow creates vol in db.... calls manager 2015-01-08T20:20:29 manager does allocation stuff 2015-01-08T20:20:34 returns to taskflow 2015-01-08T20:20:44 *** annashen has joined #openstack-cinder 2015-01-08T20:20:54 problem is that if something goes wrong, taskflow may or may not delete the volume from db 2015-01-08T20:21:07 the retries is a non-issue 2015-01-08T20:21:24 but if it reverts we've allocated and we're screwed 2015-01-08T20:21:32 but I think I know how to work that 2015-01-08T20:21:35 so never mind :) 2015-01-08T20:24:06 jgriffith: I think I might still be missing. so if you have 1G volume, alloc=1G....then you do another 1G create, fails in taskflow, the volume is not deleted by taskflow..so you have 2 x 1G volumes know to cinder. One is error the other available. If we remove out the increment/decrement code and just purely go off what's know in the cinder db. it will still be alloc 2G. Now if taskflow was to 2015-01-08T20:24:07 delete volume, we would still go off what's known in the cinder db, and be at 1G. 2015-01-08T20:24:17 *** leeantho has joined #openstack-cinder 2015-01-08T20:24:35 thingee: so that scenario is good... yes 2015-01-08T20:24:47 those are scenarios 2015-01-08T20:24:50 two* 2015-01-08T20:25:39 I'm trying to understand what the manager has to do with any of that if the scheduler is just asking the cinder db directly. there is no incrementing or decrementing 2015-01-08T20:25:42 thingee: ahh.... so you're saying just skip the update on create/delete altogether maybe and just do the periodic task of checking db and counting refs? 2015-01-08T20:25:50 yes 2015-01-08T20:26:09 thingee: so the problem is you do a create of 10 volumes at once 2015-01-08T20:26:21 thingee: your capacity info isn't updated until the next periodic 2015-01-08T20:26:24 bad things happen 2015-01-08T20:26:32 got it 2015-01-08T20:30:24 jgriffith: the other problem to is if reschedule fails in taskflow. 2015-01-08T20:30:36 just glancing at the code, might be a concern 2015-01-08T20:32:18 *** fifieldt_ has quit IRC 2015-01-08T20:33:03 jgriffith: actually, I can't find anywhere in taskflow manager create volume code where the volume is explicitly deleted from the db 2015-01-08T20:33:17 just the common.error_out_volume() 2015-01-08T20:33:34 *** fifieldt_ has joined #openstack-cinder 2015-01-08T20:34:37 *** Mandell_ has quit IRC 2015-01-08T20:37:26 thingee: just 2c from my side: afaik, our flow manager doesn't support anything except creating 2015-01-08T20:37:27 *** harlowja_away is now known as harlowja 2015-01-08T20:37:43 hope, i'm wrong 2015-01-08T20:37:51 e0ne: that's correct 2015-01-08T20:37:55 :( 2015-01-08T20:38:14 e0ne: eharney signed up to delete volume next 2015-01-08T20:38:36 thingee, eharney: great news 2015-01-08T20:38:42 e0ne: as discussed in paris, we wanted something that would be more useful to use taskflow to determine if wanted to keep using taskflow at all 2015-01-08T20:39:31 thingee: i'm looking on it and starting of state machine impl for a last few days 2015-01-08T20:39:44 *** kaufer has quit IRC 2015-01-08T20:40:13 *** jecarey has quit IRC 2015-01-08T20:41:45 *** jecarey has joined #openstack-cinder 2015-01-08T20:41:46 *** e0ne has quit IRC 2015-01-08T20:42:18 *** jecarey_ has joined #openstack-cinder 2015-01-08T20:43:10 *** bswartz has joined #openstack-cinder 2015-01-08T20:43:14 *** lpetrut has quit IRC 2015-01-08T20:44:40 *** kvidvans has quit IRC 2015-01-08T20:45:52 *** jecarey has quit IRC 2015-01-08T20:46:10 *** e0ne has joined #openstack-cinder 2015-01-08T20:46:33 *** nellysmitt has joined #openstack-cinder 2015-01-08T20:51:07 *** nellysmitt has quit IRC 2015-01-08T20:53:54 *** _cjones_ has joined #openstack-cinder 2015-01-08T20:54:15 *** david-lyle has joined #openstack-cinder 2015-01-08T20:58:26 *** lpabon has joined #openstack-cinder 2015-01-08T20:58:31 thingee: jgriffith hey guys 2015-01-08T20:58:36 thingee: jgriffith can we get this in https://review.openstack.org/#/c/145595/ 2015-01-08T20:58:44 seems like I put proper numbers now 2015-01-08T20:59:09 *** _cjones_ has quit IRC 2015-01-08T20:59:16 *** _cjones_ has joined #openstack-cinder 2015-01-08T20:59:55 *** Mandell has joined #openstack-cinder 2015-01-08T21:02:46 boris-42: yeah, once we get Jenkins coerced 2015-01-08T21:03:41 jgriffith: thanks 2015-01-08T21:04:06 *** markvoelker has joined #openstack-cinder 2015-01-08T21:06:18 *** kaufer has joined #openstack-cinder 2015-01-08T21:09:52 *** e0ne has quit IRC 2015-01-08T21:10:51 jgriffith, thingee, xyang, avishay, jungleboyj: Can you look at https://review.openstack.org/#/c/143431/ ? I have included the reasoning for the change in both the commit message and the bug. Thanks 2015-01-08T21:12:06 *** e0ne has joined #openstack-cinder 2015-01-08T21:13:43 *** mriedem has quit IRC 2015-01-08T21:15:34 *** mriedem has joined #openstack-cinder 2015-01-08T21:18:00 boris-42: sure. e0ne left you a comment tho 2015-01-08T21:18:04 *** lpetrut has joined #openstack-cinder 2015-01-08T21:18:34 *** patrickeast has quit IRC 2015-01-08T21:19:16 jgriffith: did you see my earlier commments about not finding explicit deletes in the taskflow manager create volume code? 2015-01-08T21:19:37 jgriffith: all I found was common.error_out_volume() 2015-01-08T21:20:26 *** Masahiro has joined #openstack-cinder 2015-01-08T21:20:28 thingee: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L522 2015-01-08T21:21:04 does the manager talk to the api taskflow code? I thought it was just the manager taskflow code 2015-01-08T21:21:09 jgriffith: ^ 2015-01-08T21:21:59 thingee: you tell me, you and DuncanT keep saying how easy taskflow is :) 2015-01-08T21:22:11 thingee: or at least DuncanT did, not sure if you did or not :) 2015-01-08T21:22:29 thingee: I'm trying to force that whitelist to trace through it 2015-01-08T21:22:36 :) 2015-01-08T21:23:06 maybe harlowja is around? 2015-01-08T21:23:20 jgriffith: my impression was api/taskflow was just for api code to talk to. 2015-01-08T21:23:36 thingee: that would make sense 2015-01-08T21:23:39 jgriffith what u saying, lol 2015-01-08T21:24:00 harlowja: long time no hear 2015-01-08T21:24:04 ya, happy new year 2015-01-08T21:24:11 i got a case of poison oak for xmas 2015-01-08T21:24:12 fun fun 2015-01-08T21:24:12 lol 2015-01-08T21:24:14 harlowja: is it safe to assume volume/flows/api/create_volume.py is only access from the cinder api code, not the manager? 2015-01-08T21:24:17 *** jamielennox|away is now known as jamielennox 2015-01-08T21:24:23 harlowja: revert routine on taskflow create volume 2015-01-08T21:24:36 *** patrickeast has joined #openstack-cinder 2015-01-08T21:24:39 harlowja: does taskflow ever go back and delete the db ref on a failure? 2015-01-08T21:24:41 thingee afaik ya, tried to keep them under a specific folder (api -> api) 2015-01-08T21:25:13 *** Masahiro has quit IRC 2015-01-08T21:26:34 harlowja: from what I gather, when the manager talks to taskflow, taskflow will only set the volume to error state. A volume is only deleted if something fails from the cinder api side. 2015-01-08T21:26:43 before getting to the manager/scheduler, etc. 2015-01-08T21:26:49 ya, that should be the expected case 2015-01-08T21:27:40 harlowja: thingee I'm not sure what you folks are referring to 2015-01-08T21:28:26 jgriffith: so the cinder api code, talk to volume/flows/api/create_volume.py. the manager should be just talking to volume/flows/manager/create_volume.py 2015-01-08T21:28:58 thingee: ummm... nope 2015-01-08T21:29:03 thingee: :) 2015-01-08T21:29:03 https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L147 2015-01-08T21:29:17 create volume from cinder api goes to flows/api/create_volume 2015-01-08T21:29:26 that's where the db entry is created 2015-01-08T21:29:27 :) 2015-01-08T21:30:23 ya 2015-01-08T21:30:26 which is the link I sent 2015-01-08T21:30:45 and it's also where the rever method is that does a db.volume_destroy 2015-01-08T21:30:52 simple :) 2015-01-08T21:31:00 ok maybe i'm confused to then, ha, no api workflow code should be used in other workflows 2015-01-08T21:31:06 i think thats the jist of the question 2015-01-08T21:31:20 harlowja: thingee I could be confusing everybody.... 2015-01-08T21:31:26 harlowja: thingee let's start over :) 2015-01-08T21:31:30 possibly, ha 2015-01-08T21:31:32 harlowja: thingee my question was..... 2015-01-08T21:32:08 harlowja: thingee under what circumstances can the revert method get called (here: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L511) 2015-01-08T21:32:25 when a later task fails 2015-01-08T21:32:28 *** lpabon has quit IRC 2015-01-08T21:32:33 harlowja: thingee and yes, volume create calls go through here 2015-01-08T21:32:37 which is the quota commit task, VolumeCastTask 2015-01-08T21:33:04 harlowja: ahh... so create was fine (or not) but quota update fails 2015-01-08T21:33:09 so then we revert? 2015-01-08T21:33:35 seems like it 2015-01-08T21:33:40 that would be one case of that, ya 2015-01-08T21:34:15 or if casting fails, it should get there also 2015-01-08T21:34:27 but at that point someone set 'optional_args['is_quota_committed']' 2015-01-08T21:34:41 bb 2015-01-08T21:37:44 *** jgravel has joined #openstack-cinder 2015-01-08T21:39:41 *** eharney has quit IRC 2015-01-08T21:45:27 *** Mandell_ has joined #openstack-cinder 2015-01-08T21:46:04 *** Mandell has quit IRC 2015-01-08T21:47:44 *** Tross has joined #openstack-cinder 2015-01-08T21:53:12 *** ryanpetrello has quit IRC 2015-01-08T21:54:38 *** e0ne has quit IRC 2015-01-08T21:55:44 Gloria Gu proposed openstack/python-cinderclient: Update to change name for volume type client https://review.openstack.org/141187 2015-01-08T21:58:43 *** e0ne has joined #openstack-cinder 2015-01-08T21:59:52 *** dannywilson has quit IRC 2015-01-08T22:00:27 *** dannywilson has joined #openstack-cinder 2015-01-08T22:01:22 *** patrickeast has quit IRC 2015-01-08T22:01:45 *** dannywilson has quit IRC 2015-01-08T22:01:59 *** dannywilson has joined #openstack-cinder 2015-01-08T22:02:11 *** patrickeast has joined #openstack-cinder 2015-01-08T22:02:34 *** dannywilson has quit IRC 2015-01-08T22:02:51 *** dannywilson has joined #openstack-cinder 2015-01-08T22:03:06 *** fischerw has quit IRC 2015-01-08T22:03:27 *** fischerw has joined #openstack-cinder 2015-01-08T22:05:26 *** e0ne has quit IRC 2015-01-08T22:09:57 *** fischerw has quit IRC 2015-01-08T22:10:41 *** xyang has quit IRC 2015-01-08T22:11:46 *** xyang has joined #openstack-cinder 2015-01-08T22:14:26 thingee, xyang , this is the change splitted out from the original change to have more discussion regarding whether to allow the volume type name change : https://review.openstack.org/#/c/140906/ whenever you have a change please shed some light. Thank a lot! 2015-01-08T22:14:53 thingee, xyang s/change/chance 2015-01-08T22:15:25 *** fischerw has joined #openstack-cinder 2015-01-08T22:17:48 *** EmilienM is now known as EmilienM|afk 2015-01-08T22:22:27 *** Mandell_ has quit IRC 2015-01-08T22:22:47 *** Yogi1 has quit IRC 2015-01-08T22:26:27 *** lcurtis has quit IRC 2015-01-08T22:27:32 *** markstur has quit IRC 2015-01-08T22:28:21 *** david-lyle has quit IRC 2015-01-08T22:29:21 *** mwichmann has quit IRC 2015-01-08T22:34:41 *** e0ne has joined #openstack-cinder 2015-01-08T22:35:45 *** kaufer has quit IRC 2015-01-08T22:38:13 *** jecarey_ has quit IRC 2015-01-08T22:41:37 xing-yang proposed openstack/cinder: Create Consistency Group from CG Snapshot https://review.openstack.org/145952 2015-01-08T22:43:06 *** markstur has joined #openstack-cinder 2015-01-08T22:43:59 *** delattec has quit IRC 2015-01-08T22:45:20 xing-yang proposed openstack/cinder: Create Consistency Group from CG Snapshot https://review.openstack.org/145952 2015-01-08T22:47:07 *** Mandell has joined #openstack-cinder 2015-01-08T22:47:23 *** nellysmitt has joined #openstack-cinder 2015-01-08T22:47:30 *** e0ne has quit IRC 2015-01-08T22:49:27 *** mriedem has quit IRC 2015-01-08T22:51:10 *** xyang has quit IRC 2015-01-08T22:51:49 *** nellysmitt has quit IRC 2015-01-08T22:58:15 *** annashen has quit IRC 2015-01-08T22:59:17 *** patrickeast has quit IRC 2015-01-08T22:59:30 *** dannywilson has quit IRC 2015-01-08T23:01:48 *** dannywilson has joined #openstack-cinder 2015-01-08T23:02:22 *** patrickeast has joined #openstack-cinder 2015-01-08T23:03:30 *** annashen has joined #openstack-cinder 2015-01-08T23:05:08 *** btran has quit IRC 2015-01-08T23:05:33 *** btran has joined #openstack-cinder 2015-01-08T23:09:12 *** Masahiro has joined #openstack-cinder 2015-01-08T23:09:15 *** esker has quit IRC 2015-01-08T23:13:34 *** Masahiro has quit IRC 2015-01-08T23:15:20 *** jwcroppe has quit IRC 2015-01-08T23:15:55 *** jwcroppe has joined #openstack-cinder 2015-01-08T23:20:41 *** jwcroppe has quit IRC 2015-01-08T23:21:55 *** eharney has joined #openstack-cinder 2015-01-08T23:25:02 *** eharney_ has joined #openstack-cinder 2015-01-08T23:26:35 *** eharney has quit IRC 2015-01-08T23:28:43 *** lpetrut has quit IRC 2015-01-08T23:28:49 *** dustins_ has quit IRC 2015-01-08T23:31:29 *** fischerw has quit IRC 2015-01-08T23:31:29 *** jcru has quit IRC 2015-01-08T23:33:04 *** asselin has quit IRC 2015-01-08T23:38:50 *** asselin has joined #openstack-cinder 2015-01-08T23:39:21 *** krtaylor has quit IRC 2015-01-08T23:40:01 *** akerr_ has quit IRC 2015-01-08T23:40:02 *** erlon_ has quit IRC 2015-01-08T23:42:33 Nagendra Rao Jaladanki proposed openstack/cinder-specs: Spec for Brocade Zone Driver Virtual Fabrics support https://review.openstack.org/144389 2015-01-08T23:46:35 John Griffith proposed openstack/cinder: Implement clone_image caching on SolidFire https://review.openstack.org/142859 2015-01-08T23:48:37 Anthony Lee proposed openstack/cinder: Fixed server name being retained after detach in LeftHand https://review.openstack.org/124145 2015-01-08T23:52:14 *** krtaylor has joined #openstack-cinder 2015-01-08T23:55:52 *** dannywilson has quit IRC 2015-01-08T23:56:19 *** dannywilson has joined #openstack-cinder 2015-01-08T23:59:04 *** smoriya has joined #openstack-cinder 2015-01-08T23:59:47 *** patrickeast has quit IRC