Monday, 2017-08-07

*** edmondsw has joined #openstack-cinder00:02
*** s-shiono has joined #openstack-cinder00:04
*** yangyapeng has joined #openstack-cinder00:04
*** edmondsw has quit IRC00:06
*** felipemonteiro_ has joined #openstack-cinder00:11
*** yangyapeng has quit IRC00:11
*** ircuser-1 has quit IRC00:26
*** itlinux has quit IRC00:29
*** sdake_ has quit IRC00:32
*** sdake_ has joined #openstack-cinder00:32
*** wanghao has joined #openstack-cinder00:39
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements
*** zhurong has joined #openstack-cinder00:42
*** zhurong has quit IRC00:53
*** Shunli has joined #openstack-cinder00:58
*** yangyapeng has joined #openstack-cinder01:10
*** yangyapeng has quit IRC01:10
*** markvoelker has joined #openstack-cinder01:15
*** yangyapeng has joined #openstack-cinder01:17
*** crushil has joined #openstack-cinder01:38
*** zhaochao has joined #openstack-cinder01:39
*** randomhack has joined #openstack-cinder01:41
*** randomhack has quit IRC01:46
*** lhx__ has joined #openstack-cinder01:49
*** edmondsw has joined #openstack-cinder01:50
*** edmondsw has quit IRC01:54
*** markvoelker has quit IRC02:02
*** markvoelker has joined #openstack-cinder02:02
*** markvoelker has quit IRC02:34
*** felipemonteiro__ has joined #openstack-cinder03:08
*** dave-mccowan has quit IRC03:09
*** felipemonteiro_ has quit IRC03:11
*** felipemonteiro__ has quit IRC03:18
*** crushil has quit IRC03:26
*** gkadam has joined #openstack-cinder03:32
*** Nel1x has quit IRC03:37
*** edmondsw has joined #openstack-cinder03:38
*** edmondsw has quit IRC03:42
*** links has joined #openstack-cinder03:49
*** nicolasbock has joined #openstack-cinder03:51
*** diablo_rojo has joined #openstack-cinder03:55
openstackgerritMerged openstack/cinder master: Sort cinder-manage commands
stakedahi, cinder cores, could you review this small patch?
*** Poornima has joined #openstack-cinder04:33
stakedawe would like to fix our ci before our summer holidays. thanks!04:33
*** PsionTheory has joined #openstack-cinder04:37
*** david-lyle has quit IRC04:38 proposed openstack/cinder master: [api-ref] Add worker cleanup for service docs api
*** Poornima has quit IRC04:50
*** itlinux has joined #openstack-cinder04:54
*** Shunli has quit IRC05:06
*** Shunli has joined #openstack-cinder05:06 proposed openstack/cinder master: [Api-ref] Add force parameter to volume delete
*** pgadiya has joined #openstack-cinder05:21
*** lhx__ has quit IRC05:26
*** zhurong has joined #openstack-cinder05:28
*** itlinux has quit IRC05:32
*** psachin has joined #openstack-cinder05:35
*** Poornima has joined #openstack-cinder05:40
*** randomhack has joined #openstack-cinder05:42
*** randomhack has quit IRC05:47
*** diablo_rojo has quit IRC05:52
*** jiaopeng_ has joined #openstack-cinder05:55
*** jiaopen__ has joined #openstack-cinder05:56
*** jiaopeng_ has quit IRC05:56
*** jiaopen__ has quit IRC05:56
*** yangyapeng has quit IRC06:01
*** yangyapeng has joined #openstack-cinder06:02
*** lhx__ has joined #openstack-cinder06:12
*** yangyape_ has joined #openstack-cinder06:45
*** yangyapeng has quit IRC06:45
*** bhagyashris has joined #openstack-cinder07:01
*** stakeda has quit IRC07:06
*** nicolasbock_ has joined #openstack-cinder07:06
*** yangyape_ is now known as yangyapeng07:07
*** nicolasbock_ has quit IRC07:07
*** rcernin has joined #openstack-cinder07:08
lhx__geguileo, could you have a look at this bug?
openstackLaunchpad bug 1709011 in Cinder "Support force delete backup when rados connection isn't initialized successfully." [Undecided,New] - Assigned to Hanxi Liu (hanxi-liu)07:10
geguileolhx__: hi, why does it go to 'error_deleting' status? r:-??07:11
lhx__geguileo, fail_reason: Delete backup is aborted due to backup service is down07:12
geguileolhx__: ok07:13
geguileolhx__: anything you need from me on that one?07:14
*** edmondsw has joined #openstack-cinder07:14
*** nizam037 has joined #openstack-cinder07:16
lhx__geguileo, I just want to know if ceph backup driver could support force delete, so I won't implement something unuseful before that07:16
geguileolhx__: Yes, it should support it07:17
*** edmondsw has quit IRC07:18
lhx__geguileo, thanks for your advice :)07:19
geguileolhx__: feel free to add me to the review once you are done coding07:19
*** tesseract has joined #openstack-cinder07:20
openstackgerritwangxiyuan proposed openstack/cinder-specs master: Inspection Mechanism For Capacity Limited Host
*** markvoelker has joined #openstack-cinder08:17
*** randomhack has joined #openstack-cinder08:18
*** markvoelker_ has joined #openstack-cinder08:19
*** aarefiev_afk is now known as aarefiev08:21
*** toabctl has joined #openstack-cinder08:22
toabctlcould some core merge please? it already has one +2 and is a simple man-page build fix. TIA08:22
*** markvoelker has quit IRC08:23
*** randomhack has quit IRC08:24
*** s-shiono has quit IRC08:35
*** fnordahl_ is now known as fnordahl08:42
openstackgerritzhuangzejian proposed openstack/cinder master: Modify a little desc to satisfy the code
*** nizam037 has quit IRC08:55
openstackgerritGuy Kim proposed openstack/cinder master: Update configuration names in ITRI DISCO volume driver
*** edmondsw has joined #openstack-cinder09:02
*** edmondsw has quit IRC09:06
openstackgerritzhuangzejian proposed openstack/cinder master: Fix wrong param transfer in scheduler flows.
lhx__geguileo, I saw only a few operations supporing arg '--force' in v3, compared with v2. is there any plan to add more support of that arg in v3?09:11
geguileolhx__: v3 should have more support than v209:11
*** e0ne has joined #openstack-cinder09:12
geguileolhx__: it is inheriting all v2 operations and just replacing or extending some09:12
lhx__geguileo, but the cli doc tells 'Volume version 2 only'09:14
geguileolhx__: have you tried it in v3?09:14
lhx__geguileo, yep, you know this arg is used for delete other than error or available, but failed when delete a error_deleting backup09:16
openstackgerritzhuangzejian proposed openstack/cinder master: Fix wrong param transfer in scheduler flows.
geguileolhx__: it can fail, you'll see errors in logs09:17
lhx__geguileo, c-api log?09:19
geguileolhx__: if the error was detected there, otherwise cinder backup log09:20
lhx__geguileo, the backup log didn't throw anything error :(09:22
geguileolhx__: then it'll be in the api log09:22
lhx__geguileo, neither did api log09:23
geguileolhx__: then the API didn't receive the request and the error was in the client side09:23
geguileolhx__: look with --debug09:23
*** lennyb has quit IRC09:25
geguileolhx__: looks like the backend you are using doesn't support the operation09:26
geguileolhx__: don't tell me you are trying it with RBD, which you already know is not supported09:26
*** Shunli has quit IRC09:28
lhx__geguileo, haha, definitely RBD, but it occured because the cinder didn't connect to ceph client09:28
geguileolhx__: no, it happened because the driver doesn't support force delete09:30
*** yangyape_ has joined #openstack-cinder09:31
lhx__geguileo, because of inccorect configuration, backup was created and deleted, but the status went to 'error_deleting'09:31
geguileolhx__: that doesn't matter09:31
geguileolhx__: RBD driver DOES NOT support force delete09:31
geguileolhx__: so it will NOT work09:31
geguileolhx__: it's clear in the error message you get in the client: failed: Operation not supported: force delete.09:32
*** yangyapeng has quit IRC09:32
lhx__geguileo, I think backup should be deleted in cinder side bacause ceph backend didn't create anything, it's a special case09:34
geguileolhx__: that sounds good, but doesn't work, cinder doesn't know that nothing has been created09:35
*** PsionTheory has quit IRC09:36
lhx__geguileo, I knew cinder know nothing, I jsut think support delete operation is better in the case of connetion failure, :)09:42
geguileolhx__: then you need to add the mechanism into the file09:43
geguileoI don't know what we are discussing anymore09:43
*** dalgaaf has quit IRC09:47
*** sdague has joined #openstack-cinder09:47
*** lhx__ has quit IRC09:47
*** lhx__ has joined #openstack-cinder09:48
lhx__geguileo, sorry, I didn't get myself accorss, I meant when if cinder backup intailization failed and a backup has created with status of error, it should be legal to delete the backup09:48
*** dalgaaf has joined #openstack-cinder09:48
geguileolhx__: agreed09:48
geguileolhx__: but it's not implemented09:49
lhx__geguileo, no need implementent in ceph :)09:49
geguileolhx__: yes yo have to implement it09:49
lhx__I'd like to09:49
geguileolhx__: what if you request to force delete a backup that is currently in progress?09:50
*** chhavi has joined #openstack-cinder09:51
lhx__geguileo, maybe just change the status 'error_deleting ' to 'error', no need to force delete09:53
geguileolhx__: you can do that right now, you can change the status of a backup09:55
lhx__geguileo, thanks :)09:55
*** gcb has quit IRC10:03
*** lpetrut has joined #openstack-cinder10:08
*** yangyape_ has quit IRC10:30
*** yangyapeng has joined #openstack-cinder10:31
*** yangyapeng has quit IRC10:33
*** yangyapeng has joined #openstack-cinder10:35
*** lennyb has joined #openstack-cinder10:37
*** yangyapeng has quit IRC10:39
*** markvoelker has joined #openstack-cinder10:49
*** edmondsw has joined #openstack-cinder10:50
*** markvoelker_ has quit IRC10:53
*** markvoelker_ has joined #openstack-cinder10:54
*** markvoelker has quit IRC10:54
*** edmondsw has quit IRC10:54
*** Poornima has quit IRC10:56
*** markvoelker has joined #openstack-cinder10:57
*** markvoelker has quit IRC10:59
*** markvoelker_ has quit IRC11:00
*** markvoelker has joined #openstack-cinder11:01
*** strigazi_OFF is now known as strigazi11:05
*** markvoelker_ has joined #openstack-cinder11:12
*** pgadiya has quit IRC11:14
*** markvoelker has quit IRC11:15
*** Poornima has joined #openstack-cinder11:21
*** eharney has quit IRC11:27
*** catintheroof has joined #openstack-cinder11:28
*** pgadiya has joined #openstack-cinder11:28
*** markvoelker_ has quit IRC11:28
*** markvoelker has joined #openstack-cinder11:29
openstackgerritxiaoqin proposed openstack/cinder master: Add Storwize replication group support
*** markvoelker has quit IRC11:34
*** yangyapeng has joined #openstack-cinder11:37
*** catintheroof has quit IRC11:38
*** yangyapeng has quit IRC11:39
*** yangyapeng has joined #openstack-cinder11:39
*** markvoelker has joined #openstack-cinder11:40
*** markvoelker_ has joined #openstack-cinder11:41
*** markvoelker_ has quit IRC11:42
*** markvoelker_ has joined #openstack-cinder11:44
*** zhurong has quit IRC11:44
*** markvoelker has quit IRC11:44
*** markvoelker has joined #openstack-cinder11:47
*** liverpooler has quit IRC11:47
*** markvoelker_ has quit IRC11:50
*** catintheroof has joined #openstack-cinder11:50
*** markvoelker_ has joined #openstack-cinder11:51
*** markvoelker has quit IRC11:52
*** markvoelker has joined #openstack-cinder11:53
*** baojg has joined #openstack-cinder11:53
*** catintheroof has quit IRC11:55
*** markvoelker_ has quit IRC11:56
*** markvoelker_ has joined #openstack-cinder11:56
*** markvoelker has quit IRC12:00
*** markvoelker has joined #openstack-cinder12:02
*** markvoelker_ has quit IRC12:03
*** nizam037 has joined #openstack-cinder12:07
*** lhx__ has quit IRC12:15
*** pgadiya has quit IRC12:15
*** lhx_ has joined #openstack-cinder12:15
*** lhx_ has quit IRC12:18
*** pgadiya has joined #openstack-cinder12:19
*** gkadam has quit IRC12:22
*** Poornima has quit IRC12:25
*** lhx_ has joined #openstack-cinder12:29
*** catintheroof has joined #openstack-cinder12:30
*** pgadiya has quit IRC12:33
*** gcb has joined #openstack-cinder12:35
*** pgadiya has joined #openstack-cinder12:38
*** pgadiya has quit IRC12:43
*** mikal has quit IRC12:44
*** pgadiya has joined #openstack-cinder12:44
*** dave-mccowan has joined #openstack-cinder12:49
*** dustins has joined #openstack-cinder12:49
*** lhx_ has quit IRC12:53
*** edmondsw has joined #openstack-cinder12:55
*** edmondsw_ has joined #openstack-cinder12:57
*** rawanh has joined #openstack-cinder12:57
*** edmondsw has quit IRC12:59
*** mikal has joined #openstack-cinder13:00
*** edmondsw_ has quit IRC13:01
*** edmondsw has joined #openstack-cinder13:02
*** liverpooler has joined #openstack-cinder13:06
*** edmondsw has quit IRC13:07
*** lucasxu has joined #openstack-cinder13:12
*** edmondsw has joined #openstack-cinder13:12
*** edmondsw_ has joined #openstack-cinder13:13
*** edmondsw has quit IRC13:13
*** lhx_ has joined #openstack-cinder13:17
*** nizam037 has quit IRC13:17
openstackgerritMerged openstack/cinder master: [Api-ref] Add force parameter to volume delete
*** csaikia has joined #openstack-cinder13:26
*** liverpooler has quit IRC13:27
*** liverpooler has joined #openstack-cinder13:27
rawanhGuys, Can we push this commit to be merged ?13:28
smcginnisrawanh: No, not right now.13:28
smcginnisrawanh: And you don't really need to +1 your own patch. It's kind of assumed you are OK with it since you pushed up the code. ;)13:28
rawanh@smcginnis: is there any missing requirement for it or things that i should do?13:29
smcginnisrawanh: Just need to wait until things open up for queens.13:30
ildikovhey. has anyone seen this error in the c-api logs nowadays: TypeError: coercing to Unicode: need string or buffer, tuple found13:31
ildikovit seems to come from here: /opt/stack/cinder/cinder/api/middleware/fault.py13:31
smcginnisildikov: Sounds like someone not formatting a string correctly.13:31
ildikovand here: /opt/stack/cinder/cinder/api/openstack/wsgi.py13:31
*** pgadiya has quit IRC13:32
ildikovsmcginnis: yeah, just the origin looks pretty weird13:32
smcginnisildikov: If it's in the wsgi stuff, that may be a little harder to track down the source. It is pretty weird. ;)13:33
ildikovsmcginnis: I got this when calling attachment_show and I wonder how any sort of error from there gets to api/middleware?13:33
ildikovsmcginnis: sounds awesome, just what I wanted to hear on a Monday ;)13:33
smcginnisildikov: Do you have a trace I can look at?13:33
rawanhsmcginnis: aha thanks :)13:34
*** lbragstad has joined #openstack-cinder13:36
ildikovsmcginnis: not a nice one, sorry :/13:38
smcginnisildikov: Anything before that? It's in wsgi.Appliation.dispatch, so it's hard to easily trace that from that log.13:39
ildikovthat's all in the logs13:41
e0nesmcginnis, ildikov: I'd got the similar error in the past with bug in middleware13:43
ildikove0ne: do you have a bug report handy?13:43
e0nesmcginnis, ildikov: no:(. it was an error with unicode in HTTP headers.13:44
e0neildikov: not actually the same as you've got, but it looks similar13:45
ildikove0ne: oh, I see13:45
ildikove0ne: I guess it's fixed by now13:45
e0neildikov: yes, it should be13:46
ildikove0ne: ok, cool13:46
e0neildikov: can you add 'raise' here and try to reproduce the issue?13:48
*** rawanh has quit IRC13:48
e0neildikov: it will show you the real traceback13:48
ildikove0ne: yeah, will do13:49
ildikove0ne: will take some time as last week it was so hot here my laptop kept restarting, so I need to setup Devstack again... :/13:49
ildikove0ne: will paste when I get a better trace13:49
e0neildikov: :(13:50
*** links has quit IRC13:50
e0neildikov: we'd got about +40C in the Kharkiv last week13:50
ildikove0ne: it was around 40C here too13:50
e0nenow, +30 looks like a great temperature :)13:51
ildikovway better :)13:52
*** randomhack has joined #openstack-cinder13:53
smcginnis40! Wow, that seems really got for there.13:56
*** asettle_ is now known as asettle13:59
*** felipemonteiro has joined #openstack-cinder13:59
*** felipemonteiro_ has joined #openstack-cinder13:59
*** mriedem has joined #openstack-cinder14:00
*** crushil has joined #openstack-cinder14:02
*** cknight has joined #openstack-cinder14:03
*** felipemonteiro has quit IRC14:03
*** imacdonn has quit IRC14:09
*** xyang has joined #openstack-cinder14:11
*** xyang has quit IRC14:13
*** xyang has joined #openstack-cinder14:15
*** jistr is now known as jistr|mtg14:16
*** crushil has quit IRC14:17
*** Yogi1 has joined #openstack-cinder14:20
*** crushil has joined #openstack-cinder14:22
openstackgerritTommyLike proposed openstack/cinder master: Load resource filter config file in testcase
*** diablo_rojo has joined #openstack-cinder14:41
ildikove0ne: smcginnis: ok, I realized that it only blows up if the attachment I try to get the details of does not exist14:44
smcginnisildikov: At least it's easily reproducable.14:44
ildikovsmcginnis: yeah, I just remembered that it had issues with the normal case too14:45
ildikovsmcginnis: but probably I pasted the wrong UUID or smth like14:45
*** gcb_ has joined #openstack-cinder14:45
ildikovsmcginnis: so yeah, the error is still weird and should be fixed, but hopefully it will be easier this way14:45
smcginnisIt really looks like a case where someone is meaning to preformat a string, and instead of doing "'stuff: %s' % val" they are doing "'stuff', val" so it ends up passing in a tuple where they expect a string.14:46
smcginnisThe question is where.14:46
*** gcb has quit IRC14:47
openstackgerritTzur Eliyahu proposed openstack/cinder master: Ibm_storage - fix failover_replication.
ildikovsmcginnis: I added a 'raise' as e0ne suggested, but what I got just looks more weird IMHO...14:48
e0neildikov, smcginnis: could you please share new logs?14:49
ildikove0ne: ^^14:49
smcginnisex_value is passed in to __exit__ as a tuple?14:51
e0neildikov: not such helpful I hoped :(14:51
*** psachin has quit IRC14:51
e0neildikov: are any other errors below this traceback?14:51
e0neildikov: btw, do you have steps to reproduce? I can try on my devstack14:52
ildikove0ne: nope, that's the beginning and the end too :(14:52
*** jistr|mtg is now known as jistr14:52
*** gcb_ has quit IRC14:52
ildikove0ne: cinder --os-volume-api-version 3.27 attachment-show <random UUID>14:53
e0neildikov: cool, thanks14:54
ildikovsmcginnis: even if that happens, I have no idea how... :/ :)14:54
ildikove0ne: thanks for looking into it14:54
smcginnisildikov, e0ne: So it looks like to me that here:
smcginnisex_value is not the exception string.14:55
smcginnisI haven't found docs yet saying what that value actually is that gets passed in.14:56
smcginnisOther than the "exception value"14:56
*** jdillaman has joined #openstack-cinder14:57
e0negive me 5 mins, plz14:57
smcginnisOdd that we check and it does appear to be an actual exception:
ildikovsmcginnis: as the attachment does not exist NotFound sounds pretty reasonable14:59
ildikovsmcginnis: so it looks like it's not formatted ok or smth like if that's the one that we have there15:00
*** Julien-zte has joined #openstack-cinder15:01
smcginnisLooks like we are in a "with ResourceExceptionHandler:" block, we don't findn the attachment so it raises, the __exit__ handler processes and we do see it is a NotFound exception, but then something with formatting that exception fails.15:01
*** Nil_ has quit IRC15:01
*** markvoelker has quit IRC15:03
*** crushil has quit IRC15:04
e0nesmcginnis: yes, we have to add VolumeAttachmentNotFound to that list15:04
e0nesmcginnis: but it's not enough15:04
*** david-lyle has joined #openstack-cinder15:04
smcginnisLooks like VolumeAttachmentNotFound inherits from NotFound, so that parts OK at least.15:06
*** markvoelker has joined #openstack-cinder15:06
smcginnisThis potentially could be it. It's not right at least:
smcginnisShould be using % and not ,15:07
smcginnisThat sets msg to a tuple where it's meant to be a string.15:07
*** PsionTheory has joined #openstack-cinder15:08
smcginnisildikov: Are you able to quickly test that out?15:08
e0nesmcginnis: looks you're tight15:08
smcginnisI am a little surprised six.text_type isn't able to just format that as a string representation, even though it would be wrong.15:09
e0nesmcginnis: you're faster than me
smcginnise0ne: :)15:11
ildikovsmcginnis: VolumeAttachmentNotFound is used as many ways as many times it's used in the code...15:12
smcginnise0ne: I don't actually have a code change to push up, so you're faster than me there. ;)15:12
smcginnisildikov: It looks like it's probably fine in most cases unless the code tries to do six.text_type(not_found_exception)15:13
smcginnisildikov: But likely could fail in multiple locations.15:13
ildikove0ne: smcginnis: is there a point to add a new message when the exception has one by default?15:13
smcginnisThat wsgi stuff just makes things confusing and hard to find.15:13
smcginnisildikov: Hmm, probably not.15:13
e0nesmcginnis, ildikov: need to look deeper on it15:14
smcginnisWell, it does add the attachment ID that was attempted to retrieve, but the caller probably already knows that.15:14
ildikovthis is the other one: raise exception.VolumeAttachmentNotFound(filter='attachment_id = ' + attachment_id)15:14
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception
ildikovthis is how the exception is defined:15:15
ildikovclass VolumeAttachmentNotFound(NotFound):15:15
ildikov    message = _("Volume attachment could not be found with "15:15
ildikov                "filter: %(filter)s .")15:15
ildikovso the second one should be ok15:16
smcginnisildikov: Looks like that case should be fine.15:16
smcginnisIt's just that this particular one replaces the message rather than providing formatting args.15:16
ildikovI don't think there's a point replacing the message, but well15:17
ildikovif it works I will care a bit less :)15:17
e0nesmcginnis, ildikov: both of you  are solving issues faster than I can code and test:(15:17
smcginnisHah, that's a good problem to have. :)15:18
ildikovok, with the fixed msg it looks much better: ERROR: Unable to find attachment with id: d36c4433-0562-445d-81c6-10186d295317 (HTTP 404) (Request-ID: req-4a6306ef-0363-4587-86fa-7433147bfdf4)15:20
ildikove0ne: you have a patch up that works in my env, so you rock :)15:20
ildikove0ne: with smcginnis we're just talking about fixing stuff :)15:20
e0neI've added smcginnis as co-author15:21
smcginnise0ne: Hah, not necessary, but thank you. :)15:21
e0neI hope, no tests will fail15:21
e0nesmcginnis: you save me few minutes pointing on a needed like :)15:22
*** Swanson has joined #openstack-cinder15:22
*** ntpttr_laptop has joined #openstack-cinder15:23
ildikovreal team work :)15:24
ildikovsmcginnis: I saw the client fix got merged for the uWSGI issue15:25
*** bswartz has quit IRC15:25
ildikovsmcginnis: when could we get a client release?15:25
smcginnisildikov: Another good example of team work.15:25
e0neildikov, smcginnis: do we need to backport for pike and do new release now?15:25
ildikovsmcginnis: indeed :)15:25
smcginnisildikov: We'll have to wait until after the freeze at this point, unfortunately.15:25
ildikovsmcginnis: when is that? next week?15:26
smcginnise0ne: Yeah, if that's not in stable/pike, we should get that moved.15:26
smcginnisildikov: At some point after RC is cut. It might take an extra week or so because of the trailing release projects.15:26
e0nesmcginnis: done:
ildikovsmcginnis: oh, ok, I forgot about that :(15:27
smcginnisildikov: But then you can bug the release manager PTL to get that going. ;)15:27
*** bswartz has joined #openstack-cinder15:27
smcginnise0ne: Awesome.15:27
ildikovsmcginnis: lol :)15:27
*** waboring has joined #openstack-cinder15:28
*** ntpttr_laptop has quit IRC15:29
e0nesmcginnis, ildikov: need to head home. I'll fix exception message later tonight it the patch won't be merged/updated15:33
ildikove0ne: wasn't a blocking comment15:33
smcginnisThanks e0ne, have a good one.15:33
ildikove0ne: thanks for the quick fix :)15:33
ildikove0ne: and have a good evening!15:34
*** Julien-zte has quit IRC15:35
*** Julien-zte has joined #openstack-cinder15:35
openstackgerritStephen Finucane proposed openstack/python-cinderclient master: Explicitly set 'builder' option
openstackgerritStephen Finucane proposed openstack/python-cinderclient master: doc: Remove cruft from
openstackgerritStephen Finucane proposed openstack/python-cinderclient master: Use Sphinx 1.5 warning-is-error
*** crose has joined #openstack-cinder15:41
smcginnisSpec revert for something not done in Pike:
smcginnisMight be good to get that out of there so no one sees it and wonders why they can't find the code.15:47
*** lpetrut has quit IRC15:49
*** markvoelker has quit IRC15:53
*** aarefiev is now known as aarefiev_afk15:54
openstackgerritStephen Finucane proposed openstack/os-brick master: doc: Remove cruft from
openstackgerritStephen Finucane proposed openstack/os-brick master: doc: Restructure docs for doc-migration
*** Apoorva has joined #openstack-cinder16:00
*** Apoorva has quit IRC16:01
openstackgerritMerged openstack/cinder master: vzstorage: use remotefs.VZStorageRemoteFSClient
*** waboring has quit IRC16:06
*** ircuser-1 has joined #openstack-cinder16:06
*** aarefiev_afk has quit IRC16:06
*** itlinux has joined #openstack-cinder16:08
*** ntpttr_laptop has joined #openstack-cinder16:09
*** ntpttr_laptop has quit IRC16:09
openstackgerritSean McGinnis proposed openstack/cinder master: Clean up repl v1 volume creation from replication
*** baojg has quit IRC16:10
*** Apoorva has joined #openstack-cinder16:13
*** yangyapeng has quit IRC16:13
*** yangyapeng has joined #openstack-cinder16:13
*** e0ne has quit IRC16:17
*** yangyapeng has quit IRC16:20
*** hemna_ has joined #openstack-cinder16:29
*** dustins has quit IRC16:33
*** ChubYann has joined #openstack-cinder16:34
*** markstur has joined #openstack-cinder16:35
*** openstackstatus has quit IRC16:42
*** openstackstatus has joined #openstack-cinder16:43
*** ChanServ sets mode: +v openstackstatus16:43
*** Julien-zte has quit IRC16:43
*** Julien-zte has joined #openstack-cinder16:43
*** Julien-zte has quit IRC16:44
*** abhitechie has joined #openstack-cinder16:44
*** hoonetorg has quit IRC16:45
*** crose has quit IRC16:51
*** hoonetorg has joined #openstack-cinder16:58
*** lhx_ has quit IRC17:00
*** crushil has joined #openstack-cinder17:02
*** hemna_ has quit IRC17:05
*** baojg has joined #openstack-cinder17:11
*** tesseract has quit IRC17:12
*** pcaruana has quit IRC17:15
*** baojg has quit IRC17:16
*** rcernin has quit IRC17:22
*** mriedem has quit IRC17:22
*** e0ne has joined #openstack-cinder17:24
*** xyang has quit IRC17:28
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception
e0nesmcginnis, ildikov: I updated patch ^^ to re-use default exception message17:34
*** randomhack has quit IRC17:35
*** randomhack has joined #openstack-cinder17:38
*** diablo_rojo has quit IRC17:40
*** abhitechie has quit IRC17:40
*** abhinavtechie has joined #openstack-cinder17:40
*** dustins has joined #openstack-cinder17:43
*** markvoelker has joined #openstack-cinder17:54
*** gouthamr has joined #openstack-cinder17:58
*** itlinux has quit IRC17:59
*** randomhack has quit IRC18:04
*** itlinux has joined #openstack-cinder18:05
*** abhinavtechie has quit IRC18:11
*** baojg has joined #openstack-cinder18:12
*** lpetrut has joined #openstack-cinder18:12
*** yangyapeng has joined #openstack-cinder18:15
*** baojg has quit IRC18:17
*** yangyapeng has quit IRC18:20
*** itlinux has quit IRC18:21
*** itlinux has joined #openstack-cinder18:22
*** markvoelker has quit IRC18:28
*** randomhack has joined #openstack-cinder18:30
*** xyang has joined #openstack-cinder18:30
*** chhavi has quit IRC18:31
*** xyang has quit IRC18:32
*** edmondsw_ has quit IRC18:32
*** edmondsw has joined #openstack-cinder18:32
*** kfarr has joined #openstack-cinder18:34
*** edmondsw_ has joined #openstack-cinder18:35
*** edmondsw has quit IRC18:37
*** edmondsw_ has quit IRC18:39
*** lpetrut has quit IRC18:41
*** xyang has joined #openstack-cinder18:41
*** xyang has quit IRC18:44
*** mriedem has joined #openstack-cinder18:45
smcginnise0ne: Cool, that works.18:49
*** nicolasbock has quit IRC18:57
*** diablo_rojo has joined #openstack-cinder19:02
*** armax has joined #openstack-cinder19:08
ildikove0ne: tested, works well, thank you!19:10
e0neildikov:  thanks for testing and review19:10
ildikove0ne: np :)19:13
*** baojg has joined #openstack-cinder19:13
openstackgerritJay Bryant proposed openstack/cinder master: Make Cinder's Landing Page look like Keystone's
jungleboyj^^ smcginnis hemna e0ne scottda  New landing page for Cinder's documentation.  Appreciate input on it.19:17
*** baojg has quit IRC19:18
jungleboyjlbragstad:  Thanks for the inspiration/example.  :-)19:18
e0nejungleboyj: do you have a link to the keystone's landing page in touch?19:19
jungleboyjlbragstad:  :-)  Thank you.19:19
lbragstadjungleboyj: anytime!19:19
lbragstadjungleboyj: fwiw - it's still wip19:19
e0nejungleboyj: thank you, Jay19:20
jungleboyjSo is our's but yours is at least readable.19:20
lbragstadnot sure if this is specific to cinder or not - but we duplicated a lot of what was in openstack-manuals in our own project docs =/19:20
jungleboyjlbragstad:  Well, I have pulled over what was in openstack-docs.  Still need to make sure I have it all.19:20
lbragstadwhich is why we have an Operator Documentation and an Administrator Guide19:20
lbragstadwe're working on consolidating those into a single document tree for simplicity sake, since a lot of them over lap19:21
lbragstadjust for future reference, a lot of the Operator Documentation is going to go away and get migrated19:21
lbragstadwhich seems to be a never ending battle of documentation shuffling19:22
jungleboyjOk.  I have a feeling my work on Cinder's docs will be neverending.19:22
e0nejungleboyj: It means, you always will have a work19:23
jungleboyje0ne:  :-)19:24
*** xyang1 has joined #openstack-cinder19:33
*** randomhack has quit IRC19:34
*** edmondsw has joined #openstack-cinder19:38
*** randomhack has joined #openstack-cinder19:41
*** e0ne has quit IRC19:43
*** crose has joined #openstack-cinder19:43
*** jessegler has joined #openstack-cinder19:48
mriedemjungleboyj: did you get this migrated to cinder docs?
mriedemi noticed it showed up in the nova docs, which is wrong since it's mostly cinder stuff19:49
jgriffithWhat's the deal os-brick?
jgriffithmeanwhile:  jgriffith@os-1:/opt/stack/logs$ ls -la /dev/sda20:03
jgriffithbrw-rw---- 1 root root 8, 0 Aug  7 20:00 /dev/sda20:03
jgriffithhowever it does seem that ownership of the device changes when libvirt has it and then gives it up20:03
smcginnisjgriffith: I was just going to say - permissions issue?20:03
jgriffithsmcginnis but shouldn't brick be able to figure that out?20:04
*** wanghao has quit IRC20:04
jgriffithHmm... or maybe that's my problem here?  Maybe I'm trying to terminate BEFORE detaching from libvirt?20:04
smcginnisjgriffith: I would expect a different error actually.20:04
*** liverpooler has quit IRC20:04
jgriffithsmcginnis well there's no shortage of messages in the logs :)20:05
jgriffiththeyr'e just all pretty much useless though :(20:05
SwansonNot the window I thought.20:06
smcginnisYou'd think that would give a "Permission Denied" and not a "not found" message. Timing issue by any chance?20:06
jgriffithSwanson although it's somewhat fitting20:06
smcginnisSwanson: That works for you anyway. :)20:06
jgriffithsmcginnis yeah, I dunno; the notfound makes sense because brick it turns out isn't very sophisticated and just does an ls /dev20:07
jgriffithso if you don't have perms you just don't see what you're looking for20:07
smcginnisjgriffith: Oh, true.20:07
jgriffithnot sure, but I think instead of doing an ls /dev and then "looking for say sda" it should probably explicitly do an inspect on sda.. but what do I know20:07
jgriffithparsing out ls results is kinda un-robust20:08
jgriffithguess it's a clue at least ;)20:08
smcginnisjgriffith: Would be great if there was a system interface available in python to be able to do this the right way.20:09
jgriffithsmcginnis :)20:09
jgriffithWanna join me on a new OSS project and write one?20:09
jgriffithsurely our employers will finance us for such a thing!!20:09
jgriffithOr I could down-size, move into a cabin in the woods and just do whatever I want20:10
jgriffithwait... internet, computers etc20:10
jgriffiththe good thing is that I seem to be making at least *some* kinda progress on whatever it goofy in the new attach code with shelved instances etc20:11
smcginnisjgriffith: That's good!20:11
jgriffithturns out nova really cares a lot about sequence20:11
smcginnisTrick is to find a cabin in the woods with a good internet uplink. :D20:11
*** dustins has quit IRC20:12
jgriffithI finally reproduced this mess manually with combinations of attach->shelve->unshelve->delete-instance20:12
jgriffithsmcginnis considering I couldn't even find a *house* in the country with that I'm not optimistic20:12
jgriffithand now that "town" has grown out to my quiet little corner it really just seems unfair20:13
smcginnisjgriffith: Urban sprawl in CO? I thought they had a lot of laws against that.20:13
jgriffithsmcginnis pissh!!!20:13
jgriffithLaws are only for suckers that don't have money20:14
jgriffithyou have enough money you can do whatever you want20:14
jgriffithas is evident by the giant f'ing golf course they built down the road20:14
jgriffithand all the houses to go with it20:14
*** baojg has joined #openstack-cinder20:14
jgriffithdespite that fact that yes, there were laws/rules about new development and water etc20:14
jgriffithturns out you have enough money and there's always an exception process20:15
ildikovmoney talks, that's the only thing that hasn't changed a bit over the past few thousand years...20:15
jgriffithildikov indeed... even when the form of currency changes, the rules change with it20:15
* smcginnis pictures jgriffith in some fancy golf outfit20:16
jgriffithalthough I guess it's just a math problem at this point... too many people in the world20:16
jgriffithsmcginnis LOL20:16
jgriffithsmcginnis ummmm... yeah, that one is going to have to remain pure imaginary20:16
*** yangyapeng has joined #openstack-cinder20:16
*** gouthamr has quit IRC20:17
ildikovjgriffith: rules don't really change, I could mention Animal Farm from Orwell as a good attempt to visualize it a bit20:17
jgriffithildikov :)20:18
ildikovjgriffith: so, what's wrong with golf? :)20:18
smcginnisBoring "sport" :)20:19
*** baojg has quit IRC20:19
ildikovsmcginnis: oh I'm not into it either, just got a challenge accepted feeling re the "pure imaginary" statement jgriffith has just made, so thought to figure out some details :)20:20
jgriffithildikov nothing I suppose, just not really my thing.  And given I live in a place where the cities and state are gobbling up water rights and taking them away from farms I have issues with golf courses20:20
jgriffiththey use a lot of water, and I like food20:21
*** yangyapeng has quit IRC20:21
jgriffithalthough if they want to let me use the golf course as pasture for cattle and horses then I'm in :)20:21
smcginnisThat would be awesome!20:22
ildikovhaha, that would be interesting :)20:22
jgriffiththat would be really20:22
smcginnisExtra "hazards"20:22
ildikovlol :)20:22
jgriffiththis could actually catch on!!20:23
jgriffith volume version 3.27 is not in supported versions: 1, 2, 320:23
ildikovjgriffith: as for the new attach stuff, is it purely shelve/unshelve that messes things up?20:23
ildikovjgriffith: what did you do again? :)20:24
jgriffithildikov I can't guarantee that, would be interesting to disable those tests and see20:24
jgriffithI tried to rerun with OFFLINE=True20:24
*** gouthamr has joined #openstack-cinder20:24
jgriffithturns out for some reason that's not a valid thing perhaps?20:24
ildikovwhat does that do?20:24
jgriffithildikov OFFLINE?20:26
jgriffiththe idea is you just restack as if there's no internet connection20:26
smcginnisRerunning Yeah, that's a 50/50 chance of working. :)20:26
jgriffithjust cleans everything and rebuild using what's in place20:26
jgriffithused to be a life-saver20:26
jgriffithespecially great on airplanes20:26
ildikovthere's no life without internet connection in this world :)20:26
jgriffithsmcginnis yeah, looks like it pretty much sucks now20:26
jgriffithildikov the other thing used to be that you could restack in about 5 minutes20:27
ildikovoh, that indeed sounds nice20:27
jgriffithinstead of 20+ or whatever20:27
jgriffithI've gotten spoiled running cinder stuff in containers20:28
ildikovalthough for me it's not that long, when I don't re-clone all the things, it's somewhere between20:28
jgriffithdevstack is killing me20:28
smcginnisildikov: Even when your laptop is melting?20:28
ildikovjgriffith: I'm not picky on what env you're fixing the attach stuff in :)20:29
ildikovsmcginnis: not that hot here anymore, however one tempest run can get it close to melt still...20:29
ildikovjgriffith: if I figure out how to disable tests, would that be all the shelve related ones to exclude and see whether we still have the problem present?20:32
ildikovjgriffith: it might tomorrow when I get there to run the whole suite again, but happy to experiment with it20:33
jgriffithildikov yeah, there's only a few tests there20:34
jgriffithif I don't manage to get to it today I'll shoot you a note on how to do it20:34
jgriffithit's pretty easy20:34
ildikovjgriffith: sounds good, thanks20:35
jgriffithif nothing else you can do it the brute-force way and just delete the tests from the file :)20:35
jgriffithotherwise you can add skip directives in tempest.conf20:35
ildikovI wanted to avoid that if possible :)20:35
jgriffithildikov :)20:35
ildikovdeleting I mean :)20:35
jgriffithyou can also put a skip decorator on them20:35
jgriffithyeah, I knew what you meant20:35
ildikovok, I will look it up, when I start to hate my laptop again and want to torture it :)20:36
jgriffithoh... I know why I was getting that version thing... in my env I do export OS_VOLUME_API_VERSION=3.2720:36
*** gouthamr has quit IRC20:36
ildikovalways the small things :)20:37
jgriffithwe should probably make things smart enough to strip off the micro-version20:37
jgriffithrather than erroring20:37
ildikovone thing at a time :)20:38
*** randomha1k has joined #openstack-cinder20:40
*** randomhack has quit IRC20:43
*** randomha1k has quit IRC20:44
*** cknight has quit IRC20:51
*** mriedem has quit IRC20:56
*** Yogi1 has quit IRC21:07
*** crushil has quit IRC21:09
*** chaynikasaikia has joined #openstack-cinder21:13
*** cknight has joined #openstack-cinder21:15
*** baojg has joined #openstack-cinder21:15
*** lucasxu has quit IRC21:15
*** csaikia has quit IRC21:15
*** kfarr has quit IRC21:17
*** chaynikasaikia has quit IRC21:18
*** catintheroof has quit IRC21:20
*** cknight1 has joined #openstack-cinder21:20
*** cknight has quit IRC21:21
*** baojg has quit IRC21:22
*** markvoelker has joined #openstack-cinder21:26
*** cknight1 has quit IRC21:43
*** edmondsw has quit IRC21:45
*** edmondsw has joined #openstack-cinder21:45
*** crose has quit IRC21:47
openstackgerritMerged openstack/cinder master: Make Cinder's Landing Page look like Keystone's
*** edmondsw has quit IRC21:50
*** csaikia has joined #openstack-cinder21:51
*** markvoelker has quit IRC22:00
*** xyang1 has quit IRC22:00
*** Apoorva_ has joined #openstack-cinder22:15
*** yangyapeng has joined #openstack-cinder22:17
*** Apoorva has quit IRC22:18
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix missing references in documentation
*** yangyapeng has quit IRC22:21
*** Apoorva_ has quit IRC22:24
*** Apoorva has joined #openstack-cinder22:25
*** gouthamr has joined #openstack-cinder22:28
*** markvoelker has joined #openstack-cinder22:32
*** itlinux has quit IRC22:51
*** markvoelker has quit IRC22:54
*** edmondsw has joined #openstack-cinder22:54
*** markvoelker has joined #openstack-cinder22:54
*** PsionTheory has quit IRC22:58
*** edmondsw has quit IRC22:58
*** markvoelker has quit IRC22:59
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix user messages documentation
*** ntpttr_ is now known as ntpttr23:05
*** chhavi has joined #openstack-cinder23:08
*** baojg has joined #openstack-cinder23:19
*** sdague has quit IRC23:21
*** baojg has quit IRC23:23
*** jessegler has quit IRC23:27
*** catintheroof has joined #openstack-cinder23:28
-openstackstatus- NOTICE: The Gerrit service on will be offline momentarily at 00:00 utc for a quick reconfiguration-related restart23:32
*** itlinux has joined #openstack-cinder23:34
*** itlinux has quit IRC23:37
*** stakeda has joined #openstack-cinder23:38
*** felipemonteiro_ has quit IRC23:39
*** diablo_rojo has quit IRC23:40
*** diablo_rojo has joined #openstack-cinder23:40
*** catintheroof has quit IRC23:50
*** crushil has joined #openstack-cinder23:57
openstackgerritMerged openstack/cinder master: Fix message format for VolumeAttachmentNotFound exception

Generated by 2.15.3 by Marius Gedminas - find it at!