| *** patrickeast has joined #openstack-cinder | 00:05 | |
| *** annashen has quit IRC | 00:06 | |
| *** markvoelker has quit IRC | 00:14 | |
| *** ho has joined #openstack-cinder | 00:17 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fixed server name being retained after detach in LeftHand https://review.openstack.org/124145 | 00:19 |
|---|---|---|
| boris-42 | thingee: jgriffith Ok I replied https://review.openstack.org/#/c/145595/ on comment and Jenkins put +1 | 00:21 |
| jgriffith | boris-42: haha.. stupid quotas! | 00:22 |
| jgriffith | :) | 00:22 |
| boris-42 | jgriffith: lol | 00:22 |
| boris-42 | jgriffith: I can put quotas context=) | 00:22 |
| boris-42 | jgriffith: but creation of volumes works well | 00:23 |
| boris-42 | jgriffith: I didn't see that it is failing at all | 00:23 |
| boris-42 | jgriffith: just attaching stuff | 00:23 |
| *** annashen has joined #openstack-cinder | 00:24 | |
| jgriffith | boris-42: I'm working on the attach stuff again :( | 00:25 |
| jgriffith | boris-42: just noticed something I missed before... the exception being caught was NOT from the show | 00:26 |
| jgriffith | boris-42: the update is in that very same block | 00:26 |
| jgriffith | so... things are more plausible now | 00:26 |
| jgriffith | and it means we have a race on attach.... where there is more than one attach coming in at a time | 00:26 |
| boris-42 | jgriffith: so should I ping you next time when I see this error? | 00:26 |
| jgriffith | after I get another patch up with more debug info yes for sure | 00:27 |
| jgriffith | FWIW I already found it in one of your rally checks just a couple hours ago | 00:27 |
| boris-42 | jgriffith: if you need any help with this just ping me=) cause I am very interested in fixing it=) | 00:28 |
| jgriffith | boris-42: will do | 00:29 |
| anish | iscsi_ip_address is supposed to set what ip the target portal is exported as, correct ? | 00:30 |
| *** annashen has quit IRC | 00:32 | |
| *** Masahiro has joined #openstack-cinder | 00:40 | |
| *** _cjones_ has quit IRC | 00:40 | |
| *** Masahiro has quit IRC | 00:44 | |
| *** _cjones_ has joined #openstack-cinder | 00:46 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Remove unused variables from ensure_export() https://review.openstack.org/145381 | 00:46 |
| openstackgerrit | John Griffith proposed openstack/cinder: Add debug messaging for tgt already exists https://review.openstack.org/145985 | 00:47 |
| jgriffith | boris-42: thingee ^^ | 00:48 |
| *** nellysmitt has joined #openstack-cinder | 00:48 | |
| jgriffith | if we can merge that asap we might be able to get enough info to figure out what's going on | 00:48 |
| boris-42 | jgriffith: + we should merge my patch=) | 00:48 |
| jgriffith | boris-42: thingee I have two theories to go on right now.. neither make me happy | 00:48 |
| jgriffith | boris-42: doing it now :) | 00:49 |
| boris-42 | jgriffith: so how that message will help us? | 00:50 |
| jgriffith | boris-42: so it takes some before/after states of the system | 00:50 |
| jgriffith | boris-42: so first we do the show so we know exactly what the system had built for targets | 00:50 |
| jgriffith | that way we can see if infact we already had a target for that volume OR if we just got screwed and a target id was reused by tgtadm | 00:51 |
| jgriffith | the other message in the api well tell us if we actually sent a second attach to the same volume :( | 00:51 |
| jgriffith | or if "nova" sent a second attach to same volume | 00:51 |
| jgriffith | which means a bad race somewhere in our attach update | 00:52 |
| boris-42 | ahh | 00:52 |
| jgriffith | hmmmm.... actually | 00:52 |
| jgriffith | that would be easy to fix looking at initilize right now | 00:52 |
| jgriffith | we dont' "check" anything when that's called | 00:52 |
| *** nellysmitt has quit IRC | 00:53 | |
| jgriffith | should probably look at changing that | 00:53 |
| jgriffith | that actually might explain a few things | 00:53 |
| jgriffith | but I want to wait and see | 00:53 |
| jgriffith | the other thing we could do is just allow it and say "hey, already exists... don't bother" | 00:54 |
| jgriffith | "here's the info you're looking for" | 00:54 |
| jgriffith | which is what my patch last night aimed to do | 00:54 |
| jgriffith | BUT... I missed the fact that the damn show call was in the same try block | 00:54 |
| boris-42 | heh | 00:55 |
| jgriffith | so never go executed... and thus we failed the next call because there was no info there :( | 00:55 |
| *** _cjones_ has quit IRC | 00:55 | |
| *** scottda_ has joined #openstack-cinder | 00:56 | |
| *** scottda_ has quit IRC | 00:57 | |
| jgriffith | alright... I'm out of here for a while | 00:57 |
| jgriffith | catch ya all later | 00:57 |
| boris-42 | jgriffith: see you | 00:57 |
| *** Masahiro has joined #openstack-cinder | 00:59 | |
| *** scottda_ has joined #openstack-cinder | 01:02 | |
| *** scottda_ has quit IRC | 01:06 | |
| *** dannywilson has quit IRC | 01:09 | |
| *** tsekiyam_ has joined #openstack-cinder | 01:13 | |
| *** ebalduf has quit IRC | 01:15 | |
| *** patrickeast has quit IRC | 01:16 | |
| *** tsekiyama has quit IRC | 01:16 | |
| *** mtanino has quit IRC | 01:17 | |
| *** tsekiyam_ has quit IRC | 01:17 | |
| *** patrickeast has joined #openstack-cinder | 01:18 | |
| *** alecv has quit IRC | 01:20 | |
| *** nosnos has joined #openstack-cinder | 01:27 | |
| *** bill_az has quit IRC | 01:31 | |
| thingee | boris-42, jgriffith: catching up...errands, etc. | 01:35 |
| thingee | jgriffith: looks like smcginnis found a valid issue with the extra debug info patch for targets | 01:41 |
| thingee | jgriffith: other than that lgtm | 01:41 |
| thingee | boris-42: looks like jgriffith already took care of your patch for merge | 01:41 |
| boris-42 | thingee: yep now we should wait a hell gate queue | 01:42 |
| *** peluse has quit IRC | 01:57 | |
| *** peluse has joined #openstack-cinder | 01:58 | |
| *** leeantho has quit IRC | 01:59 | |
| *** elazro has quit IRC | 02:07 | |
| *** elazro has joined #openstack-cinder | 02:07 | |
| *** eharney_ has quit IRC | 02:12 | |
| *** jwang_ has joined #openstack-cinder | 02:14 | |
| *** jwang has quit IRC | 02:17 | |
| *** oomichi has joined #openstack-cinder | 02:18 | |
| *** mdenny has quit IRC | 02:22 | |
| *** asselin has quit IRC | 02:22 | |
| *** jgravel has quit IRC | 02:22 | |
| *** kmartin has quit IRC | 02:22 | |
| *** rhagarty has quit IRC | 02:22 | |
| *** sseverson has quit IRC | 02:22 | |
| *** rhagarty has joined #openstack-cinder | 02:23 | |
| *** jgravel has joined #openstack-cinder | 02:23 | |
| *** asselin has joined #openstack-cinder | 02:23 | |
| *** sseverson has joined #openstack-cinder | 02:23 | |
| *** mdenny has joined #openstack-cinder | 02:23 | |
| *** kmartin has joined #openstack-cinder | 02:23 | |
| *** mudassirlatif has joined #openstack-cinder | 02:26 | |
| *** jamielennox is now known as jamielennox|away | 02:30 | |
| *** zengyunling__ has joined #openstack-cinder | 02:32 | |
| *** pcaruana is now known as pcaruana|afk| | 02:34 | |
| *** scottda_ has joined #openstack-cinder | 02:34 | |
| *** haomaiwang has joined #openstack-cinder | 02:35 | |
| *** ling-yun-1 has quit IRC | 02:35 | |
| *** jamielennox|away is now known as jamielennox | 02:37 | |
| *** ling-yun-1 has joined #openstack-cinder | 02:37 | |
| *** annashen has joined #openstack-cinder | 02:37 | |
| *** zengyunling__ has quit IRC | 02:39 | |
| *** mberlin1 has joined #openstack-cinder | 02:41 | |
| *** kaisers1 has joined #openstack-cinder | 02:41 | |
| *** scottda_ has quit IRC | 02:42 | |
| *** kaisers has quit IRC | 02:43 | |
| *** mberlin has quit IRC | 02:44 | |
| *** Mandell has quit IRC | 02:44 | |
| *** ebalduf has joined #openstack-cinder | 02:46 | |
| *** scottda_ has joined #openstack-cinder | 02:47 | |
| *** nellysmitt has joined #openstack-cinder | 02:49 | |
| *** chlong has joined #openstack-cinder | 02:49 | |
| *** jamielennox is now known as jamielennox|away | 02:51 | |
| *** annashen has quit IRC | 02:51 | |
| *** ebalduf has quit IRC | 02:52 | |
| *** nellysmitt has quit IRC | 02:53 | |
| *** rhagarty has quit IRC | 02:57 | |
| *** sseverson has quit IRC | 02:57 | |
| *** kmartin has quit IRC | 02:57 | |
| *** mdenny has quit IRC | 02:57 | |
| *** jgravel has quit IRC | 02:57 | |
| *** kmartin has joined #openstack-cinder | 02:57 | |
| *** gugl2 has joined #openstack-cinder | 02:57 | |
| *** jgravel has joined #openstack-cinder | 02:57 | |
| *** rhagarty has joined #openstack-cinder | 02:57 | |
| *** mdenny has joined #openstack-cinder | 02:57 | |
| *** gugl has quit IRC | 03:00 | |
| *** jamielennox|away is now known as jamielennox | 03:01 | |
| *** jecarey has joined #openstack-cinder | 03:07 | |
| *** BharatK has joined #openstack-cinder | 03:12 | |
| *** scottda_ has quit IRC | 03:14 | |
| *** IanGovett has quit IRC | 03:15 | |
| *** annashen has joined #openstack-cinder | 03:18 | |
| *** patrickeast has quit IRC | 03:22 | |
| *** ebalduf has joined #openstack-cinder | 03:22 | |
| *** nosnos has quit IRC | 03:24 | |
| *** ebalduf has quit IRC | 03:26 | |
| *** david-lyle has joined #openstack-cinder | 03:29 | |
| *** scottda_ has joined #openstack-cinder | 03:29 | |
| *** mudassirlatif has quit IRC | 03:31 | |
| *** scottda_ has quit IRC | 03:32 | |
| *** haomaiwang has quit IRC | 03:33 | |
| *** harlowja is now known as harlowja_away | 03:38 | |
| *** scottda_ has joined #openstack-cinder | 03:44 | |
| *** annashen has quit IRC | 03:52 | |
| *** rwsu has quit IRC | 03:53 | |
| *** nosnos has joined #openstack-cinder | 03:56 | |
| *** oomichi has quit IRC | 04:01 | |
| *** scottda_ has quit IRC | 04:10 | |
| *** scottda_ has joined #openstack-cinder | 04:12 | |
| *** oomichi has joined #openstack-cinder | 04:14 | |
| *** david-lyle has quit IRC | 04:17 | |
| *** sseverson has joined #openstack-cinder | 04:23 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add debug messaging for tgt already exists https://review.openstack.org/145985 | 04:31 |
| *** mudassirlatif has joined #openstack-cinder | 04:37 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix iscsi_write_cache setting for iscsi targets https://review.openstack.org/145415 | 04:38 |
| *** annashen has joined #openstack-cinder | 04:41 | |
| *** scottda_ has quit IRC | 04:44 | |
| *** RJ_ has joined #openstack-cinder | 04:45 | |
| *** nellysmitt has joined #openstack-cinder | 04:50 | |
| *** mudassirlatif has quit IRC | 04:52 | |
| *** nellysmitt has quit IRC | 04:55 | |
| *** nshaikh has joined #openstack-cinder | 04:56 | |
| *** jwcroppe has joined #openstack-cinder | 05:00 | |
| *** Mandell has joined #openstack-cinder | 05:09 | |
| RJ_ | Any one knw the why this error ..??/ | 05:15 |
| RJ_ | http://paste.openstack.org/show/156060/ | 05:15 |
| RJ_ | while adding replication support | 05:16 |
| *** nosnos has quit IRC | 05:21 | |
| *** nosnos has joined #openstack-cinder | 05:21 | |
| *** _cjones_ has joined #openstack-cinder | 05:23 | |
| *** ebalduf has joined #openstack-cinder | 05:25 | |
| *** nosnos has quit IRC | 05:26 | |
| *** ebalduf has quit IRC | 05:29 | |
| *** nshaikh has quit IRC | 05:31 | |
| *** _cjones_ has quit IRC | 05:32 | |
| *** _cjones_ has joined #openstack-cinder | 05:32 | |
| *** Longgeek has joined #openstack-cinder | 05:40 | |
| *** _cjones_ has quit IRC | 05:40 | |
| *** jecarey has quit IRC | 05:41 | |
| *** nshaikh has joined #openstack-cinder | 05:41 | |
| *** BharatK has left #openstack-cinder | 05:45 | |
| *** BharatK has joined #openstack-cinder | 05:45 | |
| *** pradipta_away is now known as pradipta | 05:47 | |
| *** BharatK has quit IRC | 05:50 | |
| *** anshul has joined #openstack-cinder | 05:50 | |
| *** nosnos has joined #openstack-cinder | 05:53 | |
| RJ_ | Any one knw the why this error ..??/ [10:45] <RJ_> http://paste.openstack.org/show/156060/ [10:46] <RJ_> while adding replication support | 06:02 |
| *** anshul has quit IRC | 06:02 | |
| *** anshul has joined #openstack-cinder | 06:02 | |
| *** junhongl has quit IRC | 06:05 | |
| *** junhongl has joined #openstack-cinder | 06:05 | |
| *** annashen has quit IRC | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/144933 | 06:09 |
| *** ebalduf has joined #openstack-cinder | 06:26 | |
| RJ_ | Hi | 06:27 |
| RJ_ | all | 06:27 |
| RJ_ | it is necessry to have | 06:27 |
| RJ_ | different names for volume backends | 06:28 |
| *** ebalduf has quit IRC | 06:30 | |
| *** mudassirlatif has joined #openstack-cinder | 06:35 | |
| openstackgerrit | Mike Perez proposed openstack/python-cinderclient: Enable version discovery https://review.openstack.org/145613 | 06:42 |
| *** mudassirlatif has quit IRC | 06:42 | |
| *** deepakcs has joined #openstack-cinder | 06:43 | |
| *** nosnos has quit IRC | 06:48 | |
| *** nosnos has joined #openstack-cinder | 06:50 | |
| *** nellysmitt has joined #openstack-cinder | 06:51 | |
| *** nkrinner has joined #openstack-cinder | 06:53 | |
| *** nellysmitt has quit IRC | 06:54 | |
| *** thingee has quit IRC | 06:56 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Update README.rst to current state https://review.openstack.org/145519 | 06:57 |
| *** Longgeek has quit IRC | 06:58 | |
| *** Longgeek has joined #openstack-cinder | 07:11 | |
| *** EduardBiceriMate has joined #openstack-cinder | 07:12 | |
| *** karimb has joined #openstack-cinder | 07:14 | |
| *** jamielennox is now known as jamielennox|away | 07:15 | |
| *** ebalduf has joined #openstack-cinder | 07:27 | |
| *** ebalduf has quit IRC | 07:31 | |
| *** coolsvap|afk is now known as coolsvap | 07:49 | |
| *** TobiasE has joined #openstack-cinder | 07:53 | |
| *** afazekas_ has joined #openstack-cinder | 08:18 | |
| *** lpetrut has joined #openstack-cinder | 08:18 | |
| *** e0ne has joined #openstack-cinder | 08:24 | |
| *** e0ne has quit IRC | 08:27 | |
| *** Mandell has quit IRC | 08:27 | |
| *** ebalduf has joined #openstack-cinder | 08:27 | |
| *** Mandell has joined #openstack-cinder | 08:30 | |
| *** sgotliv has quit IRC | 08:31 | |
| *** ebalduf has quit IRC | 08:31 | |
| *** Mandell has quit IRC | 08:34 | |
| *** _cjones_ has joined #openstack-cinder | 08:41 | |
| *** oomichi has quit IRC | 08:41 | |
| *** _cjones_ has quit IRC | 08:46 | |
| *** lpetrut has quit IRC | 08:48 | |
| *** nellysmitt has joined #openstack-cinder | 08:55 | |
| *** dalgaaf has joined #openstack-cinder | 08:57 | |
| *** ho has quit IRC | 08:57 | |
| *** nellysmitt has quit IRC | 08:59 | |
| *** jpich has joined #openstack-cinder | 09:01 | |
| *** Masahiro has quit IRC | 09:04 | |
| *** jistr has joined #openstack-cinder | 09:05 | |
| *** lpetrut has joined #openstack-cinder | 09:05 | |
| *** Masahiro has joined #openstack-cinder | 09:05 | |
| *** jwcroppe has quit IRC | 09:07 | |
| *** deepakcs has quit IRC | 09:09 | |
| *** Longgeek_ has joined #openstack-cinder | 09:10 | |
| *** Longgeek has quit IRC | 09:11 | |
| *** nellysmitt has joined #openstack-cinder | 09:12 | |
| *** boris-42 has quit IRC | 09:13 | |
| *** lpetrut has quit IRC | 09:22 | |
| *** ebalduf has joined #openstack-cinder | 09:28 | |
| *** jordanP has joined #openstack-cinder | 09:31 | |
| *** ebalduf has quit IRC | 09:32 | |
| *** nshaikh has quit IRC | 09:35 | |
| *** ling-yun-1 has quit IRC | 09:46 | |
| *** ling-yun has joined #openstack-cinder | 09:47 | |
| *** rushiagr_away is now known as rushiagr | 09:52 | |
| *** nshaikh has joined #openstack-cinder | 10:02 | |
| *** rushiagr is now known as rushiagr_away | 10:11 | |
| *** afazekas_ is now known as afazekas | 10:11 | |
| *** deepakcs has joined #openstack-cinder | 10:15 | |
| *** rushiagr_away is now known as rushiagr | 10:18 | |
| *** Masahiro has quit IRC | 10:26 | |
| *** ebalduf has joined #openstack-cinder | 10:29 | |
| *** rushiagr is now known as rushiagr_away | 10:30 | |
| openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Support to manage block storage on Linux on System z. https://review.openstack.org/141092 | 10:30 |
| *** ebalduf has quit IRC | 10:33 | |
| *** e0ne has joined #openstack-cinder | 10:36 | |
| *** tbarron1 has joined #openstack-cinder | 10:47 | |
| *** e0ne is now known as e0ne_ | 11:03 | |
| *** e0ne_ has quit IRC | 11:13 | |
| *** alexpilotti has joined #openstack-cinder | 11:14 | |
| *** pradipta is now known as pradipta_away | 11:21 | |
| *** e0ne has joined #openstack-cinder | 11:23 | |
| *** Masahiro has joined #openstack-cinder | 11:27 | |
| *** ebalduf has joined #openstack-cinder | 11:30 | |
| *** Masahiro has quit IRC | 11:31 | |
| *** ebalduf has quit IRC | 11:34 | |
| *** nikesh_vedams has quit IRC | 11:44 | |
| *** nosnos has quit IRC | 11:54 | |
| *** erlon_ has joined #openstack-cinder | 11:57 | |
| *** e0ne is now known as e0ne_ | 11:58 | |
| *** e0ne_ has quit IRC | 12:03 | |
| *** EmilienM|afk is now known as EmilienM | 12:07 | |
| *** e0ne has joined #openstack-cinder | 12:13 | |
| *** boris-42 has joined #openstack-cinder | 12:15 | |
| *** Longgeek_ has quit IRC | 12:21 | |
| *** Mandell has joined #openstack-cinder | 12:29 | |
| *** ebalduf has joined #openstack-cinder | 12:30 | |
| *** chlong has quit IRC | 12:32 | |
| *** Mandell has quit IRC | 12:34 | |
| *** IanGovett has joined #openstack-cinder | 12:35 | |
| *** ebalduf has quit IRC | 12:35 | |
| *** erlon_ is now known as erlon | 12:39 | |
| *** anshul has quit IRC | 12:43 | |
| *** Guest83536 has joined #openstack-cinder | 12:45 | |
| *** lpetrut has joined #openstack-cinder | 12:45 | |
| *** cdelatte has joined #openstack-cinder | 12:46 | |
| *** Longgeek has joined #openstack-cinder | 12:47 | |
| *** kbyrne has quit IRC | 12:49 | |
| *** EmilienM is now known as EmilienM|afk | 12:52 | |
| *** kbyrne has joined #openstack-cinder | 12:54 | |
| *** Masahiro has joined #openstack-cinder | 12:59 | |
| *** smoriya has quit IRC | 13:00 | |
| *** Masahiro has quit IRC | 13:04 | |
| *** sgotliv has joined #openstack-cinder | 13:04 | |
| *** EmilienM|afk is now known as EmilienM | 13:09 | |
| *** deepakcs has quit IRC | 13:12 | |
| *** bswartz has quit IRC | 13:14 | |
| *** lpetrut has quit IRC | 13:25 | |
| *** ebalduf has joined #openstack-cinder | 13:31 | |
| *** ebalduf has quit IRC | 13:36 | |
| *** Guest83536 has quit IRC | 13:40 | |
| *** RJ_ has quit IRC | 13:41 | |
| *** bill_az has joined #openstack-cinder | 13:44 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix cinder manage command https://review.openstack.org/139071 | 13:52 |
| *** pcaruana|afk| has quit IRC | 13:53 | |
| *** lpetrut has joined #openstack-cinder | 13:54 | |
| *** nshaikh has quit IRC | 13:55 | |
| *** nshaikh has joined #openstack-cinder | 13:56 | |
| *** nshaikh has quit IRC | 13:57 | |
| *** tbarron1 has quit IRC | 14:00 | |
| *** kaufer has joined #openstack-cinder | 14:00 | |
| *** mberlin1 has quit IRC | 14:00 | |
| *** mberlin has joined #openstack-cinder | 14:02 | |
| *** rushiagr_away is now known as rushiagr | 14:06 | |
| *** bswartz has joined #openstack-cinder | 14:08 | |
| *** rushiagr is now known as rushiagr_away | 14:10 | |
| *** coolsvap is now known as coolsvap|afk | 14:10 | |
| *** timcl has joined #openstack-cinder | 14:12 | |
| *** Yogi11 has joined #openstack-cinder | 14:16 | |
| *** akerr has joined #openstack-cinder | 14:16 | |
| *** dustins has joined #openstack-cinder | 14:19 | |
| *** coolsvap|afk is now known as coolsvap | 14:19 | |
| *** ryanpetrello has joined #openstack-cinder | 14:20 | |
| *** e0ne is now known as e0ne_ | 14:20 | |
| *** EduardBiceriMate has quit IRC | 14:29 | |
| *** e0ne_ has quit IRC | 14:31 | |
| *** shyvenug has joined #openstack-cinder | 14:31 | |
| *** ebalduf has joined #openstack-cinder | 14:32 | |
| *** mriedem has joined #openstack-cinder | 14:32 | |
| *** ebalduf has quit IRC | 14:37 | |
| *** mwichmann has joined #openstack-cinder | 14:38 | |
| *** dulek has joined #openstack-cinder | 14:40 | |
| shyvenug | Hi I have a question on the capabilities filter used in the cinder scheduler | 14:41 |
| shyvenug | Say I have a property 'compression' which is specific to each volume. I want this to be False for one volume while the backend storage does support compression. When I add this property to the host_stat of the cinder driver the cinder scheduler does not select the given backend when I try to create a volume with compression=False | 14:45 |
| shyvenug | Is this the way the Capabilities Filter supposed to work? | 14:45 |
| *** Masahiro has joined #openstack-cinder | 14:48 | |
| *** jungleboyj has joined #openstack-cinder | 14:51 | |
| *** dannywilson has joined #openstack-cinder | 14:52 | |
| *** xyang has joined #openstack-cinder | 14:52 | |
| *** Masahiro has quit IRC | 14:52 | |
| *** pcaruana|afk| has joined #openstack-cinder | 14:53 | |
| mriedem | new gate bug https://bugs.launchpad.net/cinder/+bug/1408814 | 14:54 |
| mriedem | 78 hits in the last ~48 hours, check and gate, all failures | 14:55 |
| mriedem | could be a glance bug | 14:56 |
| *** haomaiwa_ has joined #openstack-cinder | 14:57 | |
| *** dannywilson has quit IRC | 14:59 | |
| *** akerr has quit IRC | 15:03 | |
| *** jecarey has joined #openstack-cinder | 15:08 | |
| *** thangp has joined #openstack-cinder | 15:08 | |
| *** shyvenug has quit IRC | 15:08 | |
| *** jcru has joined #openstack-cinder | 15:14 | |
| *** lpabon has joined #openstack-cinder | 15:16 | |
| *** haomaiwang has joined #openstack-cinder | 15:16 | |
| *** haomaiwa_ has quit IRC | 15:16 | |
| *** TobiasE has left #openstack-cinder | 15:16 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Use oslo.middleware https://review.openstack.org/134759 | 15:17 |
| *** akerr has joined #openstack-cinder | 15:18 | |
| *** esker has joined #openstack-cinder | 15:20 | |
| *** haomaiwang has quit IRC | 15:20 | |
| *** haomaiwang has joined #openstack-cinder | 15:21 | |
| *** haomaiwang has quit IRC | 15:21 | |
| *** IanGovett has quit IRC | 15:26 | |
| *** akerr has quit IRC | 15:26 | |
| *** bswartz has quit IRC | 15:28 | |
| *** akerr has joined #openstack-cinder | 15:30 | |
| *** ebalduf has joined #openstack-cinder | 15:33 | |
| *** bswartz has joined #openstack-cinder | 15:33 | |
| *** madskier has joined #openstack-cinder | 15:34 | |
| *** ebalduf has quit IRC | 15:37 | |
| *** humble_ has joined #openstack-cinder | 15:44 | |
| hemna | mornin | 15:45 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up QoSSpecManageApiTest setup https://review.openstack.org/146121 | 15:46 |
| *** afazekas has quit IRC | 15:50 | |
| *** bnemec is now known as beekneemech | 15:56 | |
| *** dulek has quit IRC | 16:04 | |
| *** openstackgerrit has quit IRC | 16:05 | |
| *** openstackgerrit has joined #openstack-cinder | 16:06 | |
| *** nkrinner has quit IRC | 16:07 | |
| *** lcurtis has joined #openstack-cinder | 16:13 | |
| *** fischerw has joined #openstack-cinder | 16:14 | |
| *** patrickeast has joined #openstack-cinder | 16:14 | |
| *** mtanino has joined #openstack-cinder | 16:14 | |
| *** coolsvap is now known as coolsvap|afk | 16:16 | |
| *** e0ne has joined #openstack-cinder | 16:21 | |
| *** _cjones_ has joined #openstack-cinder | 16:21 | |
| *** karimb has quit IRC | 16:22 | |
| *** mudassirlatif has joined #openstack-cinder | 16:24 | |
| *** marcusvrn has quit IRC | 16:25 | |
| *** tsekiyama has joined #openstack-cinder | 16:25 | |
| *** bswartz has quit IRC | 16:30 | |
| *** marcusvrn has joined #openstack-cinder | 16:30 | |
| *** bswartz has joined #openstack-cinder | 16:31 | |
| *** jdurgin has joined #openstack-cinder | 16:31 | |
| *** bswartz has quit IRC | 16:32 | |
| *** bswartz has joined #openstack-cinder | 16:33 | |
| *** Masahiro has joined #openstack-cinder | 16:37 | |
| *** dansmith is now known as superdan | 16:37 | |
| *** Masahiro has quit IRC | 16:41 | |
| *** jordanP has quit IRC | 16:42 | |
| *** dannywilson has joined #openstack-cinder | 16:47 | |
| *** anshul has joined #openstack-cinder | 16:48 | |
| *** dannywilson has quit IRC | 16:50 | |
| *** dannywilson has joined #openstack-cinder | 16:51 | |
| *** madskier has quit IRC | 16:53 | |
| *** rwsu has joined #openstack-cinder | 16:53 | |
| *** sgotliv has quit IRC | 16:54 | |
| *** dannywilson has quit IRC | 16:55 | |
| *** tsekiyam_ has joined #openstack-cinder | 17:00 | |
| *** Miouge has quit IRC | 17:01 | |
| *** marcusvrn has quit IRC | 17:01 | |
| *** marcusvrn1 has joined #openstack-cinder | 17:01 | |
| *** Yogi11 has quit IRC | 17:02 | |
| *** schegi has joined #openstack-cinder | 17:03 | |
| *** tsekiyama has quit IRC | 17:04 | |
| *** EmilienM is now known as EmilienM|afk | 17:04 | |
| *** schegi has quit IRC | 17:04 | |
| *** spligak has quit IRC | 17:05 | |
| *** schegi has joined #openstack-cinder | 17:05 | |
| *** tsekiyam_ has quit IRC | 17:10 | |
| *** annegentle has joined #openstack-cinder | 17:10 | |
| annegentle | this gets the weirdest doc bug ever award from me https://bugs.launchpad.net/openstack-api-site/+bug/1408828 | 17:11 |
| *** jdurgin has quit IRC | 17:11 | |
| annegentle | jgriffith: That's awesome. ^^^ | 17:11 |
| *** tsekiyama has joined #openstack-cinder | 17:11 | |
| annegentle | or perhaps I should say, wesome. | 17:11 |
| smcginnis | annegentle: +1 :) | 17:12 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Remove unused variables from ensure_export() https://review.openstack.org/145381 | 17:12 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Fix iscsi_write_cache setting for iscsi targets https://review.openstack.org/145415 | 17:12 |
| annegentle | :) | 17:12 |
| *** karimb has joined #openstack-cinder | 17:13 | |
| *** bswartz1 has joined #openstack-cinder | 17:14 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add debug messaging for tgt already exists https://review.openstack.org/145985 | 17:15 |
| *** bswartz has quit IRC | 17:16 | |
| *** beekneemech has quit IRC | 17:17 | |
| *** bnemec has joined #openstack-cinder | 17:18 | |
| *** marcusvrn1 has quit IRC | 17:18 | |
| e0ne | annegentle: ood bug :) | 17:20 |
| annegentle | e0ne: I won't take any credit for it but I sure enjoyed it :) | 17:23 |
| *** marcusvrn has joined #openstack-cinder | 17:23 | |
| *** Miouge has joined #openstack-cinder | 17:23 | |
| *** leeantho has joined #openstack-cinder | 17:24 | |
| *** e0ne is now known as e0ne_ | 17:30 | |
| *** ebalduf has joined #openstack-cinder | 17:31 | |
| *** jistr_ has joined #openstack-cinder | 17:31 | |
| *** ndipanov_ has joined #openstack-cinder | 17:31 | |
| *** ryanpetrello has quit IRC | 17:33 | |
| *** karimb has quit IRC | 17:34 | |
| *** jaypipes is now known as leakypipes | 17:34 | |
| *** jistr has quit IRC | 17:35 | |
| *** ndipanov has quit IRC | 17:35 | |
| *** ndipanov_ has quit IRC | 17:37 | |
| *** jistr_ has quit IRC | 17:37 | |
| *** Longgeek has quit IRC | 17:42 | |
| *** Longgeek has joined #openstack-cinder | 17:44 | |
| *** ryanpetrello has joined #openstack-cinder | 17:45 | |
| *** annashen has joined #openstack-cinder | 17:48 | |
| *** ndipanov_ has joined #openstack-cinder | 17:49 | |
| *** e0ne_ is now known as e0ne | 17:50 | |
| *** ndipanov_ has quit IRC | 17:50 | |
| *** dannywilson has joined #openstack-cinder | 17:51 | |
| *** dannywilson has quit IRC | 17:51 | |
| *** dannywilson has joined #openstack-cinder | 17:52 | |
| *** Longgeek has quit IRC | 17:53 | |
| *** Longgeek has joined #openstack-cinder | 17:54 | |
| *** ndipanov has joined #openstack-cinder | 17:54 | |
| *** Longgeek has quit IRC | 17:59 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix iscsi_write_cache setting for iscsi targets https://review.openstack.org/145415 | 17:59 |
| *** ndipanov has quit IRC | 17:59 | |
| hemna | jgriffith, heyas | 17:59 |
| jgriffith | mtanino: ^^ I thnk that gets us back in shape... take a look when you can | 17:59 |
| jgriffith | hemna: Yo! | 17:59 |
| hemna | jgriffith, so...the lvm driver is currently broken FYI | 17:59 |
| jgriffith | hemna: ? | 18:00 |
| jgriffith | care to elaborate? | 18:00 |
| mtanino | jgriffith: Thanks for you post again. | 18:00 |
| *** mudassirlatif has quit IRC | 18:00 | |
| jgriffith | passes tempest | 18:00 |
| hemna | jgriffith, the call to self.target_driver.ensure_export | 18:00 |
| hemna | broken | 18:00 |
| *** Longgeek has joined #openstack-cinder | 18:00 | |
| jgriffith | hemna: ummm... yeah :) | 18:00 |
| jgriffith | hemna: this one: https://review.openstack.org/#/c/145415/ ? | 18:01 |
| *** mudassirlatif has joined #openstack-cinder | 18:01 | |
| jgriffith | hemna: https://bugs.launchpad.net/cinder/+bug/1408171 | 18:01 |
| hemna | ah yah, ok cool | 18:02 |
| jgriffith | hemna: or did you find something else? | 18:02 |
| hemna | just wasn't sure if you were aware of it | 18:02 |
| jgriffith | hemna: unfortunatly yes :( | 18:02 |
| hemna | as I ran into it here testing my multi-attach after rebasing | 18:02 |
| jgriffith | hemna: yeah, lemme know if you see another problem... or if multi is still not right after this patch | 18:02 |
| *** Longgeek has quit IRC | 18:02 | |
| hemna | ok will do. | 18:02 |
| *** Yogi1 has joined #openstack-cinder | 18:02 | |
| jgriffith | hemna: that *should* put us right back where we were prior to the switch over though | 18:03 |
| hemna | I'm trying to track down a problem at db sync time (upgrade script) | 18:03 |
| *** e0ne has quit IRC | 18:03 | |
| hemna | I can't figure out how to get javelin working at all | 18:03 |
| hemna | so I'm doing some manual tests at the moment | 18:03 |
| *** Longgeek has joined #openstack-cinder | 18:04 | |
| *** Longgeek has quit IRC | 18:04 | |
| jgriffith | afraid I know nothing about Javelin so I can't help there | 18:04 |
| hemna | not many do, from what I can tell | 18:04 |
| *** annegentle has quit IRC | 18:05 | |
| *** Longgeek has joined #openstack-cinder | 18:05 | |
| *** annashen has quit IRC | 18:07 | |
| *** annashen has joined #openstack-cinder | 18:08 | |
| *** Mandell has joined #openstack-cinder | 18:13 | |
| *** Mandell has quit IRC | 18:15 | |
| *** rkj has joined #openstack-cinder | 18:15 | |
| *** jpich has quit IRC | 18:16 | |
| *** schegi has quit IRC | 18:17 | |
| *** Mandell has joined #openstack-cinder | 18:17 | |
| *** marcusvrn has quit IRC | 18:22 | |
| *** Masahiro has joined #openstack-cinder | 18:25 | |
| *** mudassirlatif has quit IRC | 18:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/145661 | 18:26 |
| *** mudassirlatif has joined #openstack-cinder | 18:27 | |
| *** mudassirlatif has quit IRC | 18:28 | |
| *** jordanP has joined #openstack-cinder | 18:28 | |
| *** Masahiro has quit IRC | 18:30 | |
| rkj | Is it possible to install Cinder on Windows? | 18:30 |
| jgriffith | rkj: nope | 18:31 |
| hemna | people use windows ? | 18:31 |
| jgriffith | hemna: I'm looking out one now | 18:31 |
| rkj | well, some specific customer need. | 18:31 |
| *** jordanP has quit IRC | 18:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/145661 | 18:38 |
| *** marcusvrn has joined #openstack-cinder | 18:39 | |
| *** Mandell has quit IRC | 18:40 | |
| *** Mandell has joined #openstack-cinder | 18:40 | |
| *** mudassirlatif has joined #openstack-cinder | 18:41 | |
| *** lpetrut has quit IRC | 18:44 | |
| *** Yogi1 has quit IRC | 18:45 | |
| *** mudassirlatif has quit IRC | 18:45 | |
| *** mudassirlatif has joined #openstack-cinder | 18:47 | |
| erlon | guys, is there a way to print debut messages in a unitary test? the log messages in the driver LOG.debug are not shown | 18:49 |
| *** kvidvans has joined #openstack-cinder | 18:50 | |
| *** dustins has quit IRC | 18:51 | |
| *** bnemec is now known as beekneemech | 18:51 | |
| *** harlowja_away is now known as harlowja | 18:52 | |
| *** humble_ has quit IRC | 18:53 | |
| -openstackstatus- NOTICE: paste.openstack.org is going offline for a database migration (duration: ~2 minutes) | 18:57 | |
| *** dannywilson has quit IRC | 18:59 | |
| *** dannywilson has joined #openstack-cinder | 19:01 | |
| *** jdurgin has joined #openstack-cinder | 19:01 | |
| *** alexpilotti has quit IRC | 19:03 | |
| *** mudassirlatif has quit IRC | 19:06 | |
| *** e0ne has joined #openstack-cinder | 19:06 | |
| *** dustins has joined #openstack-cinder | 19:07 | |
| *** dalgaaf has quit IRC | 19:07 | |
| *** xyang has quit IRC | 19:08 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Verify the instance's existance in the VMAX driver https://review.openstack.org/140913 | 19:09 |
| *** xyang has joined #openstack-cinder | 19:09 | |
| kmartin | erlon just use print(...) in the unit test if you are just debugging a test | 19:10 |
| *** Yogi11 has joined #openstack-cinder | 19:14 | |
| erlon | kmartin: havent tried that, seems to obvious to work lol | 19:15 |
| *** rkj has quit IRC | 19:18 | |
| *** EmilienM|afk is now known as EmilienM | 19:24 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix iscsi_write_cache setting for iscsi targets https://review.openstack.org/145415 | 19:25 |
| *** ebalduf has quit IRC | 19:25 | |
| *** Longgeek has quit IRC | 19:27 | |
| *** dustins has quit IRC | 19:28 | |
| *** Longgeek has joined #openstack-cinder | 19:28 | |
| *** dustins has joined #openstack-cinder | 19:30 | |
| *** bswartz1 has quit IRC | 19:32 | |
| *** Longgeek has quit IRC | 19:33 | |
| *** ebalduf has joined #openstack-cinder | 19:33 | |
| *** marcusvrn has quit IRC | 19:35 | |
| *** _cjones_ has quit IRC | 19:36 | |
| *** mwichmann has quit IRC | 19:37 | |
| *** lpetrut has joined #openstack-cinder | 19:38 | |
| *** Mandell has quit IRC | 19:39 | |
| *** mwichmann has joined #openstack-cinder | 19:40 | |
| *** _cjones_ has joined #openstack-cinder | 19:41 | |
| *** ebalduf has quit IRC | 19:49 | |
| *** e0ne has quit IRC | 19:54 | |
| *** mwichmann has quit IRC | 19:57 | |
| *** annashen has quit IRC | 19:59 | |
| *** mwichmann has joined #openstack-cinder | 20:00 | |
| *** annashen has joined #openstack-cinder | 20:03 | |
| *** thingee has joined #openstack-cinder | 20:06 | |
| thingee | annegentle_: whoa yeah weird bug | 20:13 |
| *** Masahiro has joined #openstack-cinder | 20:14 | |
| *** kbyrne has quit IRC | 20:15 | |
| *** kbyrne has joined #openstack-cinder | 20:16 | |
| *** bswartz has joined #openstack-cinder | 20:16 | |
| thingee | erlon: also ./run_tests.sh --debug | 20:16 |
| thingee | erlon: should output debug messages | 20:17 |
| thingee | err that's wrong | 20:17 |
| *** bill_az has quit IRC | 20:17 | |
| thingee | that's for breakpoints | 20:17 |
| *** Masahiro has quit IRC | 20:19 | |
| erlon | thingee: yep, also it does not run the test "No handlers could be found for logger "cinder.volume.drivers.hds.nfs" | 20:21 |
| erlon | kmartin: thingee: print() shows the stdout but only if the test fails, but better than nothing | 20:21 |
| *** dustins has quit IRC | 20:29 | |
| *** elazro has quit IRC | 20:33 | |
| *** elazro has joined #openstack-cinder | 20:33 | |
| jgriffith | erlon: as thingee suggested, I just do a breakpoint and use --debug | 20:34 |
| jgriffith | erlon: the logger message should be benign | 20:34 |
| jgriffith | erlon: then you can step throgh the code and inspect to do your debugging | 20:34 |
| jgriffith | erlon: there's another switch for print statements I think, but I don't remember and I've never used it. It's possible it doesn't work anyway :) | 20:35 |
| thingee | jgriffith: what's new with allocation stuff? | 20:35 |
| jgriffith | thingee: I got side tracked... | 20:35 |
| jgriffith | was in a car accident :( | 20:35 |
| thingee | jgriffith: holy crap | 20:35 |
| jgriffith | just getting back at it | 20:35 |
| jgriffith | yeah.... sucks | 20:35 |
| *** Yogi12 has joined #openstack-cinder | 20:36 | |
| jgriffith | nobody hurt though so that's good | 20:36 |
| thingee | jgriffith: yeah, just annoying otherwise | 20:36 |
| jgriffith | my car on the other hand..... "BOOOM" | 20:36 |
| *** akerr has quit IRC | 20:36 | |
| jgriffith | but not as bad as the other guys :) | 20:36 |
| *** Yogi11 has quit IRC | 20:36 | |
| *** lpabon has quit IRC | 20:36 | |
| *** nellysmitt has quit IRC | 20:36 | |
| thingee | jgriffith: so I was kind of unsure from discussion with harlowja if we determined we're ok from taskflow's perspective? | 20:37 |
| jgriffith | haha... yeah, so looking last night we're at least "better" than we were | 20:37 |
| harlowja | i didn't do it, ha | 20:37 |
| harlowja | oh | 20:37 |
| jgriffith | there aren't really any cases that I can see clearly that actually implement the revert | 20:37 |
| thingee | jgriffith: ok whew | 20:37 |
| jgriffith | so I'm going to move forward with my first idea | 20:37 |
| jgriffith | at least until I come up with something "better" | 20:38 |
| *** timcl has quit IRC | 20:41 | |
| jgriffith | thingee: http://imgur.com/ZKsAfxz | 20:41 |
| hemna | goosh | 20:48 |
| hemna | that looks painful | 20:48 |
| jgriffith | hemna: only painful part was when I jumped out to rip the guys head off for blowing the stop sign.... | 20:49 |
| harlowja | and u ripped his head off? | 20:49 |
| hemna | damnit | 20:49 |
| jgriffith | and slipped on the ice and almost busted my grape on the asphalt :) | 20:49 |
| hemna | doh | 20:49 |
| harlowja | lol | 20:49 |
| jgriffith | and realized... yeah, ok; sorry dude | 20:49 |
| harlowja | did u give him his head back? | 20:49 |
| jgriffith | I refrained from resorting to anger :) | 20:50 |
| jgriffith | or violence | 20:50 |
| hemna | glad you are ok though | 20:50 |
| jgriffith | lol | 20:50 |
| *** anshul has quit IRC | 20:50 | |
| jgriffith | hemna: thanks.. yeah, luckily I wasn't going very fast. That's a slow stretch and I knew it was icey | 20:51 |
| jgriffith | only 85 or so... | 20:51 |
| jgriffith | kidding | 20:51 |
| hemna | :) | 20:51 |
| *** bill_az has joined #openstack-cinder | 21:00 | |
| *** ryanpetrello_ has joined #openstack-cinder | 21:10 | |
| *** Mandell has joined #openstack-cinder | 21:10 | |
| *** ryanpetrello has quit IRC | 21:12 | |
| *** ryanpetrello_ is now known as ryanpetrello | 21:12 | |
| *** Yogi12 has quit IRC | 21:12 | |
| *** bswartz has quit IRC | 21:14 | |
| *** bswartz has joined #openstack-cinder | 21:16 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 21:18 |
| *** Yogi1 has joined #openstack-cinder | 21:19 | |
| *** tsekiyam_ has joined #openstack-cinder | 21:25 | |
| *** Yogi1 has quit IRC | 21:25 | |
| *** ling-yun-1 has joined #openstack-cinder | 21:27 | |
| *** tsekiyama has quit IRC | 21:27 | |
| *** ling-yun has quit IRC | 21:29 | |
| *** ebalduf has joined #openstack-cinder | 21:30 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add volume multi attach support https://review.openstack.org/85847 | 21:31 |
| *** krtaylor has quit IRC | 21:35 | |
| *** krtaylor has joined #openstack-cinder | 21:47 | |
| *** rmesta has joined #openstack-cinder | 21:49 | |
| *** bswartz has quit IRC | 21:55 | |
| *** tbarron has quit IRC | 21:56 | |
| *** dannywilson has quit IRC | 22:02 | |
| *** tr4sh_heap has quit IRC | 22:03 | |
| *** Masahiro has joined #openstack-cinder | 22:03 | |
| *** thingee has quit IRC | 22:03 | |
| *** thingee has joined #openstack-cinder | 22:05 | |
| *** jamielennox|away has quit IRC | 22:05 | |
| *** Masahiro has quit IRC | 22:08 | |
| *** hemna has quit IRC | 22:08 | |
| *** Mandell has quit IRC | 22:08 | |
| *** jamielennox|away has joined #openstack-cinder | 22:09 | |
| *** jamielennox|away is now known as jamielennox | 22:09 | |
| anish | jgriffith: thingee quick question. Is WIP code okay for review, or should it be put up for review only when it's all done ? | 22:10 |
| *** hemna has joined #openstack-cinder | 22:11 | |
| *** hemna has quit IRC | 22:14 | |
| jgriffith | anish: WIP code is perfectly fine, just mark it as such in gerrit | 22:18 |
| jgriffith | anish: there's a section under review where you can do that | 22:18 |
| jgriffith | anish: and some like to put an indication that it's WIP in the commit message as well | 22:19 |
| anish | excellent, thanks | 22:19 |
| *** hemna has joined #openstack-cinder | 22:20 | |
| *** Mandell has joined #openstack-cinder | 22:25 | |
| *** bswartz has joined #openstack-cinder | 22:27 | |
| *** Longgeek has joined #openstack-cinder | 22:28 | |
| *** patrickeast has quit IRC | 22:30 | |
| *** Longgeek has quit IRC | 22:32 | |
| *** jseiler has joined #openstack-cinder | 22:33 | |
| jgriffith | harlowja: ping | 22:33 |
| harlowja | jgriffith pong | 22:33 |
| *** jseiler__ has quit IRC | 22:34 | |
| harlowja | sup dawg | 22:34 |
| jgriffith | harlowja: yo.... got a minute to help with some stuff? | 22:34 |
| harlowja | sure dawg | 22:34 |
| jgriffith | harlowja: so... take a look here: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L419 | 22:34 |
| jgriffith | harlowja: this never gets issued if create fails | 22:35 |
| jgriffith | harlowja: the naive answer is "move it up before the flow call | 22:35 |
| jgriffith | harlowja: that doesn't work, because then it's performed every time there's a retry | 22:35 |
| jgriffith | (from taskflow) | 22:35 |
| jgriffith | so I'm trying to figure out a way to have a "finally" type clause after the run_flow call | 22:36 |
| jgriffith | but that's done once and only once | 22:36 |
| jgriffith | know what's I mean? | 22:36 |
| harlowja | u could just do a try finally in _run_flow() | 22:37 |
| jgriffith | harlowja: unless putting it at line 398 inside the with block | 22:37 |
| jgriffith | :) | 22:37 |
| *** kaufer has quit IRC | 22:37 | |
| *** nellysmitt has joined #openstack-cinder | 22:37 | |
| jgriffith | but isn't that internal run_flow called multiple times in the case of an exception? | 22:37 |
| *** mriedem has quit IRC | 22:38 | |
| harlowja | ah, the retry loop logic hmmm | 22:38 |
| jgriffith | harlowja: yeah, that's what I've been unsure about | 22:38 |
| harlowja | *external retry looping logic | 22:38 |
| harlowja | what about putting that into the workflow itself? | 22:39 |
| jgriffith | harlowja: I looked at that | 22:39 |
| jgriffith | harlowja: wasn't quite sure how to go about it.... that's why I pinged you :) | 22:39 |
| jgriffith | harlowja: so I'm guessing I'd add the self.stats dict to the flow_engine parameters | 22:40 |
| jgriffith | harlowja: then put the increment in the taskflow code itself right? | 22:40 |
| harlowja | ya, adding a tiny class @ https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L751 into that should work | 22:40 |
| harlowja | the whole of that 'Update volume stats' could just be a fininish tasking put at the end of that | 22:41 |
| jgriffith | Oh interesting | 22:41 |
| harlowja | perhaps dropping it into https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L679 | 22:42 |
| jgriffith | I was thinking just put it in the existing finish task class | 22:42 |
| harlowja | seems like an appropriatly named unit of work | 22:42 |
| harlowja | ya | 22:42 |
| harlowja | mind-meld complete | 22:42 |
| harlowja | lol | 22:42 |
| jgriffith | hehe... yeah, that's what I was thinking | 22:42 |
| *** nellysmitt has quit IRC | 22:42 | |
| jgriffith | so that seems like it might work.... pass in the managers stats member, move the increment and voila... done | 22:43 |
| harlowja | ya | 22:43 |
| jgriffith | Ok, I'll give it a shot and ping you for a review tonight after I get around to working on it | 22:43 |
| jgriffith | thanks for the mind-meld | 22:43 |
| harlowja | https://gist.github.com/sputnik13/523c5cbf425dc263855d is being written up, new docs by some other folk, might be useful for ya | 22:43 |
| jgriffith | harlowja: oh yeah!!! | 22:44 |
| harlowja | https://gist.github.com/sputnik13/523c5cbf425dc263855d#providing-input-data-to-tasks--- that part especially | 22:44 |
| harlowja | WIP so not complete, but hoepfully should augment the current docs | 22:44 |
| *** lpetrut has quit IRC | 22:44 | |
| jgriffith | I think that works | 22:44 |
| jgriffith | harlowja: coolio.. thanks sir | 22:44 |
| harlowja | comments though i'm sure are appreciated for the author (he's in #openstack-state-management usually) | 22:44 |
| harlowja | np | 22:45 |
| jgriffith | seems straight forward enough, we'll see once I try to implement it :) | 22:45 |
| harlowja | :) | 22:45 |
| jgriffith | this will be step 1, next I'll look how to just use the driver only for this record keeping | 22:47 |
| *** xyang has quit IRC | 22:47 | |
| *** esker has quit IRC | 22:50 | |
| *** thangp has quit IRC | 22:53 | |
| *** EmilienM is now known as EmilienM|afk | 23:01 | |
| jgriffith | harlowja: thingee ummm.... I think there's some other bugs in here | 23:02 |
| jgriffith | harlowja: thingee but maybe you guys have some insight | 23:02 |
| jgriffith | harlowja: thingee so look in cinder.volume.manager at the get_flow parameters: | 23:03 |
| harlowja | kk, looking | 23:03 |
| harlowja | what am i looking for | 23:03 |
| jgriffith | look here: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L359 | 23:04 |
| jgriffith | and now.... in the flow.manager.create_volume.... | 23:04 |
| *** scottda_ has joined #openstack-cinder | 23:04 | |
| *** scottda_ has quit IRC | 23:04 | |
| harlowja | getting out my detective eyeclass, ha | 23:04 |
| jgriffith | https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L707 | 23:05 |
| jgriffith | sorry... battery on mouse is dieing :) | 23:05 |
| jgriffith | harlowja: parameters like allow_reschedule and some others are KW args in the manager call | 23:06 |
| jgriffith | but they're positional's in teh flows.manager method | 23:06 |
| harlowja | ah, damn u had me looking for something mroe serious :-P | 23:06 |
| jgriffith | hehe | 23:06 |
| jgriffith | I guess I should look and see what it's actually doing in this case | 23:07 |
| jgriffith | I'm assuming it's just still evaluating to true and carrying on | 23:08 |
| *** scottda_ has joined #openstack-cinder | 23:08 | |
| jgriffith | harlowja: just luck that it's working | 23:09 |
| *** _cjones_ has quit IRC | 23:09 | |
| harlowja | ya, probably something to be adjusted | 23:09 |
| *** _cjones_ has joined #openstack-cinder | 23:09 | |
| jgriffith | harlowja: I'll have a look at which one makes the most sense to convert | 23:10 |
| harlowja | cools | 23:10 |
| jgriffith | harlowja: you realize I'm just bugging you today because nobody else is talking to me ;) | 23:10 |
| jgriffith | alright... gott run for a bit, be back in a bit to bug you some more ;) | 23:11 |
| harlowja | why u think i'm bugging u? | 23:11 |
| harlowja | lol | 23:11 |
| harlowja | nobody talk to me either, ha | 23:11 |
| harlowja | j/k | 23:11 |
| harlowja | its ok johnny boy | 23:11 |
| *** dannywilson has joined #openstack-cinder | 23:18 | |
| jgriffith | r7gh8ae!@ | 23:21 |
| jgriffith | DOH!!!! | 23:21 |
| jgriffith | son of a | 23:21 |
| *** scottda_ has quit IRC | 23:27 | |
| *** jeblair is now known as corvus | 23:33 | |
| *** ryanpetrello has quit IRC | 23:36 | |
| harlowja | ha | 23:37 |
| *** ryanpetrello has joined #openstack-cinder | 23:37 | |
| *** rmesta has quit IRC | 23:39 | |
| *** ryanpetrello has quit IRC | 23:42 | |
| *** pcaruana|afk| has quit IRC | 23:46 | |
| *** mriedem has joined #openstack-cinder | 23:47 | |
| *** Masahiro has joined #openstack-cinder | 23:52 | |
| *** Masahiro has quit IRC | 23:56 | |
| *** _cjones_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!