Tuesday, 2016-04-12

*** rdx_matt has quit IRC00:04
jgriffithanybody that decided to upgrade their CI system to use Liberty, consider adding disk_allocation_ratio var to your cinder.conf :(00:11
jgriffithThat has caused me great pain!00:11
jgriffithhmm... failed again sigh00:12
jgriffithNow I can't ssh to instances... OMG... I HATE YOU OPENSTACK00:13
*** laughterwym has joined #openstack-cinder00:26
*** mtanino_ has joined #openstack-cinder00:26
*** mtanino has quit IRC00:27
*** laughterwym has quit IRC00:31
*** qeelee has joined #openstack-cinder00:31
*** laughterwym has joined #openstack-cinder00:32
*** markvoelker has joined #openstack-cinder00:35
*** laughter_ has joined #openstack-cinder00:37
*** laughterwym has quit IRC00:37
*** leeantho_ has quit IRC00:38
*** apoorvad has quit IRC00:40
*** markvoelker has quit IRC00:41
*** daneyon__ has joined #openstack-cinder00:42
*** laughter_ has quit IRC00:42
*** jwcroppe has joined #openstack-cinder00:43
*** daneyon has quit IRC00:45
*** jwcroppe has quit IRC00:47
*** laughterwym has joined #openstack-cinder00:49
*** karthikp has joined #openstack-cinder00:52
*** laughterwym has quit IRC00:54
*** baojg has joined #openstack-cinder00:54
*** laughterwym has joined #openstack-cinder00:57
*** rdx_matt has joined #openstack-cinder01:00
*** daneyon has joined #openstack-cinder01:00
*** laughterwym has quit IRC01:01
*** laughterwym has joined #openstack-cinder01:01
*** daneyon__ has quit IRC01:04
*** rdx_matt has quit IRC01:05
*** akshai has joined #openstack-cinder01:09
*** akshai_ has joined #openstack-cinder01:10
*** yangyapeng has joined #openstack-cinder01:12
*** akshai has quit IRC01:14
*** cheneydc has joined #openstack-cinder01:18
*** yangyapeng has quit IRC01:20
*** Guest17332 has quit IRC01:28
*** yangyapeng has joined #openstack-cinder01:28
*** wilson has joined #openstack-cinder01:28
*** wilson is now known as Guest4854201:29
*** EinstCrazy has joined #openstack-cinder01:31
*** mylu has quit IRC01:47
*** mylu has joined #openstack-cinder01:49
*** gouthamr has quit IRC01:51
*** enriquetaso has quit IRC01:51
*** qeelee has quit IRC01:53
*** Cao_ has joined #openstack-cinder01:57
*** jwcroppe has joined #openstack-cinder01:57
*** Cao_ has quit IRC01:58
*** Cao_ShuFeng has joined #openstack-cinder01:58
*** salv-orlando has joined #openstack-cinder01:58
*** rdx_matt has joined #openstack-cinder02:01
*** mtanino_ has quit IRC02:02
*** cburgess has quit IRC02:05
*** baojg has quit IRC02:06
*** Cao_ShuFeng has quit IRC02:06
*** rdx_matt has quit IRC02:06
*** bapalm has quit IRC02:06
*** cburgess has joined #openstack-cinder02:07
*** salv-orlando has quit IRC02:07
*** laughterwym has quit IRC02:08
*** baojg has joined #openstack-cinder02:09
*** laughterwym has joined #openstack-cinder02:09
*** pocketprotector has quit IRC02:12
*** mattymo has quit IRC02:12
*** bapalm has joined #openstack-cinder02:13
*** liverpooler has quit IRC02:13
*** kbyrne has quit IRC02:13
*** Swanson has quit IRC02:13
*** akshai_ has quit IRC02:14
*** liverpooler has joined #openstack-cinder02:14
*** SergeyLukjanov has quit IRC02:14
*** julim has joined #openstack-cinder02:15
*** mattymo has joined #openstack-cinder02:16
*** jwcroppe has quit IRC02:16
*** Lee1092 has joined #openstack-cinder02:17
*** SergeyLukjanov has joined #openstack-cinder02:17
*** mylu has quit IRC02:18
*** Swanson has joined #openstack-cinder02:19
*** qeelee has joined #openstack-cinder02:22
*** kbyrne has joined #openstack-cinder02:24
*** Lee1092 has quit IRC02:25
*** Lee1092 has joined #openstack-cinder02:27
*** qeelee has quit IRC02:27
*** pocketprotector has joined #openstack-cinder02:27
*** houming has joined #openstack-cinder02:28
*** mylu has joined #openstack-cinder02:33
*** qeelee has joined #openstack-cinder02:33
*** karthikp has quit IRC02:34
*** karthikp has joined #openstack-cinder02:36
*** markvoelker has joined #openstack-cinder02:37
*** markvoelker has quit IRC02:42
*** rdx_matt has joined #openstack-cinder03:02
*** mylu has quit IRC03:02
*** mylu has joined #openstack-cinder03:02
*** xyang1 has quit IRC03:03
*** rdx_matt has quit IRC03:06
*** mylu has quit IRC03:12
*** qeelee has quit IRC03:18
*** mylu has joined #openstack-cinder03:19
*** links has joined #openstack-cinder03:20
*** martyturner has joined #openstack-cinder03:24
*** qeelee has joined #openstack-cinder03:26
*** martyturner has quit IRC03:28
*** mylu has quit IRC03:29
*** diablo_rojo has quit IRC03:30
*** mylu has joined #openstack-cinder03:31
*** salv-orlando has joined #openstack-cinder03:38
*** ysm has joined #openstack-cinder03:43
*** ysm has left #openstack-cinder03:43
*** mylu has quit IRC03:48
*** mylu has joined #openstack-cinder03:49
*** salv-orlando has quit IRC03:51
*** Cao_ has joined #openstack-cinder03:52
*** Cao_Shufeng has joined #openstack-cinder03:53
openstackgerritwanghao proposed openstack/cinder: VersionNotFoundForAPIMethod message is not working  https://review.openstack.org/30402703:58
*** laughterwym has quit IRC04:00
*** laughterwym has joined #openstack-cinder04:00
*** rdx_matt has joined #openstack-cinder04:03
*** qeelee has quit IRC04:04
*** laughterwym has quit IRC04:05
*** rdx_matt has quit IRC04:07
*** wilson has joined #openstack-cinder04:08
*** wilson is now known as Guest9202404:08
*** Guest48542 has quit IRC04:10
*** sheel has joined #openstack-cinder04:12
*** laughterwym has joined #openstack-cinder04:16
*** jwcroppe has joined #openstack-cinder04:19
*** laughterwym has quit IRC04:21
*** liverpooler has quit IRC04:22
*** jwcroppe has quit IRC04:24
*** Poornima has joined #openstack-cinder04:27
*** laughterwym has joined #openstack-cinder04:32
*** laughterwym has quit IRC04:36
*** markvoelker has joined #openstack-cinder04:38
*** mylu has quit IRC04:42
*** markvoelker has quit IRC04:42
*** Fdaisuke_ has joined #openstack-cinder04:42
*** Fdaisuke has quit IRC04:44
*** karthikp has quit IRC04:49
*** salv-orlando has joined #openstack-cinder04:51
*** laughterwym has joined #openstack-cinder04:53
*** EinstCrazy has quit IRC04:55
*** laughterwym has quit IRC04:58
*** yangyapeng has quit IRC04:59
*** yangyapeng has joined #openstack-cinder05:01
*** salv-orlando has quit IRC05:03
*** rdx_matt has joined #openstack-cinder05:04
*** EinstCrazy has joined #openstack-cinder05:05
*** gcb has joined #openstack-cinder05:05
*** rdx_matt has quit IRC05:08
*** laughterwym has joined #openstack-cinder05:11
*** ankit_ag has joined #openstack-cinder05:13
*** EinstCra_ has joined #openstack-cinder05:17
*** dave-mccowan has quit IRC05:19
*** EinstCrazy has quit IRC05:19
*** liverpooler has joined #openstack-cinder05:26
*** shyama has joined #openstack-cinder05:29
*** qeelee has joined #openstack-cinder05:35
*** markvoelker has joined #openstack-cinder05:38
*** markvoelker has quit IRC05:43
*** baojg has quit IRC05:45
*** baojg has joined #openstack-cinder05:46
*** sgotliv_ has quit IRC05:50
*** salv-orlando has joined #openstack-cinder05:56
*** haomaiwang has joined #openstack-cinder05:59
*** haomaiwang has quit IRC05:59
*** lprice1 has quit IRC05:59
*** rdx_matt has joined #openstack-cinder06:04
*** rdx_matt has quit IRC06:09
*** jwcroppe has joined #openstack-cinder06:22
*** laughterwym has quit IRC06:24
*** tesseract has joined #openstack-cinder06:24
*** laughterwym has joined #openstack-cinder06:24
*** tesseract is now known as Guest1511506:24
*** jwcroppe has quit IRC06:27
*** mylu has joined #openstack-cinder06:27
*** mylu has quit IRC06:32
*** bjornar has joined #openstack-cinder06:37
*** gcb has quit IRC06:39
*** salv-orlando has quit IRC06:54
*** mjura has joined #openstack-cinder06:56
*** gcb has joined #openstack-cinder06:57
*** mjura has quit IRC06:58
*** mjura has joined #openstack-cinder06:58
*** Fdaisuke has joined #openstack-cinder06:58
*** abehl has joined #openstack-cinder07:01
*** Fdaisuke_ has quit IRC07:01
*** rdx_matt has joined #openstack-cinder07:01
*** nkrinner has joined #openstack-cinder07:09
*** laughter_ has joined #openstack-cinder07:15
*** salv-orlando has joined #openstack-cinder07:15
*** laughterwym has quit IRC07:17
*** bjornar has quit IRC07:29
*** salv-orl_ has joined #openstack-cinder07:34
*** sgotliv_ has joined #openstack-cinder07:37
*** salv-orlando has quit IRC07:37
*** markvoelker has joined #openstack-cinder07:39
*** jamielennox is now known as jamielennox|away07:42
*** markvoelker has quit IRC07:44
*** ndipanov has quit IRC07:46
*** e0ne has joined #openstack-cinder07:49
*** ociuhandu has quit IRC07:51
*** ndipanov has joined #openstack-cinder07:59
*** e0ne has quit IRC08:04
*** ankit_ag has quit IRC08:19
*** baojg has quit IRC08:21
*** baojg has joined #openstack-cinder08:22
*** ankit_ag has joined #openstack-cinder08:22
*** jistr has joined #openstack-cinder08:25
*** jwcroppe has joined #openstack-cinder08:25
*** jordanP has joined #openstack-cinder08:26
*** bjornar has joined #openstack-cinder08:27
*** mylu has joined #openstack-cinder08:28
*** jwcroppe has quit IRC08:30
*** mylu has quit IRC08:32
*** e0ne has joined #openstack-cinder08:33
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Remove XML API  https://review.openstack.org/23166308:43
*** klkumar has joined #openstack-cinder08:58
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use utils.convert_str to convert HTTP header values  https://review.openstack.org/30446209:03
e0nescottda: please, take a look on the patch above ^^09:03
e0nescottda: it removes copy-paste, introduced in https://review.openstack.org/#/c/286821/409:03
*** dims has quit IRC09:04
e0nescottda: I've found convert_str function in cinder yesterday09:04
*** julim has quit IRC09:04
*** dims has joined #openstack-cinder09:05
*** julim has joined #openstack-cinder09:05
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image  https://review.openstack.org/21637809:09
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from consistency groups  https://review.openstack.org/25942909:09
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype  https://review.openstack.org/22144209:09
openstackgerritGorka Eguileor proposed openstack/cinder: Add ordering possibilities to conditional update  https://review.openstack.org/23193609:09
*** openstackgerrit has quit IRC09:17
*** openstackgerrit has joined #openstack-cinder09:18
*** arecknag has joined #openstack-cinder09:18
*** lpetrut has joined #openstack-cinder09:22
*** laughterwym has joined #openstack-cinder09:28
*** e0ne has quit IRC09:29
*** e0ne has joined #openstack-cinder09:31
*** laughter_ has quit IRC09:32
*** laughterwym has quit IRC09:33
*** laughterwym has joined #openstack-cinder09:34
*** laughterwym has quit IRC09:38
*** markvoelker has joined #openstack-cinder09:40
*** haomaiwang has joined #openstack-cinder09:42
*** haomaiwang has quit IRC09:42
*** markvoelker has quit IRC09:45
*** lpetrut has quit IRC09:45
*** lpetrut has joined #openstack-cinder09:46
openstackgerritaditi sharma proposed openstack/cinder:     Backup create does not return status  https://review.openstack.org/30448409:46
*** sdague has joined #openstack-cinder09:50
*** lpetrut has quit IRC09:51
*** lpetrut has joined #openstack-cinder09:55
*** openstackstatus has quit IRC09:57
*** openstack has joined #openstack-cinder10:00
*** cheneydc has quit IRC10:01
openstackgerritaditi sharma proposed openstack/cinder: Backup create does not return status  https://review.openstack.org/30268610:03
*** ociuhandu has joined #openstack-cinder10:06
*** salv-orl_ has quit IRC10:06
*** yangyapeng has quit IRC10:09
jordanPhi. Someone from GlusterFS here ?10:13
*** salv-orlando has joined #openstack-cinder10:15
*** Poornima has quit IRC10:15
*** baojg has quit IRC10:16
*** Poornima has joined #openstack-cinder10:18
*** Poornima has quit IRC10:18
openstackgerritaditi sharma proposed openstack/cinder: Backup create does not return status  https://review.openstack.org/30268610:19
openstackgerritaditi sharma proposed openstack/cinder: Backup create does not return status  https://review.openstack.org/30268610:20
*** zul has quit IRC10:23
*** jwcroppe has joined #openstack-cinder10:27
*** yangyapeng has joined #openstack-cinder10:29
*** jwcroppe has quit IRC10:35
*** Cao_Shufeng has quit IRC10:36
*** Cao_ has quit IRC10:36
*** zul has joined #openstack-cinder10:39
*** klkumar has quit IRC10:47
*** salv-orlando has quit IRC10:48
*** salv-orlando has joined #openstack-cinder10:48
geguileodulek: Hi, I'm working on the problems with the version pining10:51
geguileodulek: Silly question, what was the order of the node upgrades?10:51
geguileoduring rolling upgrades10:51
dulekgeguileo: It's supposed to be c-api, c-sch, c-vol, c-bak in case of Liberty->Mitaka.10:51
geguileodulek: Ok10:52
dulekgeguileo: In case of Mitaka->Newton last ones can be exchanged I think.10:52
geguileodulek: I've solved one of the problems, I'll submit patch in a little bit10:52
dulekgeguileo: And thanks for pointing to the traceback with your problems.10:52
geguileodulek: But the other one is going to be more problematic10:52
*** EinstCra_ has quit IRC10:52
dulekgeguileo: Oh, which one? The one I've commited to fix (empty service table)?10:53
*** salv-orlando has quit IRC10:53
*** yangyapeng has quit IRC10:53
geguileodulek: Probably   XD XD XD10:53
geguileodulek: What's the link to your patch?10:53
dulekgeguileo: Oh, I haven't started with that actually, so no worries. ;)10:54
geguileodulek: Better, I have to waste time (and even more to see that your solution is better  ;-)  )10:54
geguileos/have/hate10:54
geguileodulek: Why do we update the API first?10:54
geguileodulek: That's going to be problematic (this is the bug I'm trying to fix now)10:55
*** blinky_ghost has joined #openstack-cinder10:56
dulekgeguileo: So actually there shouldn't be really hard constraints on that. It just seemed logical to go topdown - as we could block on a certain microversion.10:56
dulekgeguileo: Until everything is upgraded I mean.10:57
geguileodulek: Because if the API is upgraded first, then we'll need to restart all API nodes once the upgrade is completed  :-(10:57
dulekgeguileo: But microversions got in late, so we haven't explored that possibility.10:57
dulekgeguileo: Ah yes, that's right…10:57
geguileodulek: Which I think is unnecessary10:57
dulekgeguileo: Can you explain the problem you're seeing with this approach?10:57
geguileodulek: Sure10:57
geguileodulek: The problem is that when the API starts it pins the versions on the API for the rpc client and the serializer10:58
geguileodulek: And never updates it10:58
sheele0ne: hi10:58
e0nehi10:58
geguileodulek: So even if you completed the rolling upgrade10:58
dulekgeguileo: Yeah, that's right. Moreover no SIGHUP handling was possible to be implemented easily.10:58
geguileodulek: API would not know update rpc client or serializer10:58
sheele0ne: its regarding https://blueprints.launchpad.net/cinder/+spec/remove-xml-api10:58
blinky_ghosthi all, I have a dell equallogic storage configured with cinder working fine. However I cannot import foreign volumes using "cinder manage": "ERROR cinder.volume.manager NotImplementedError: Manage existing volume not implemented". Is this some kind of bug or it's not supported in the driver? Thanks10:59
sheele0ne: I am thinking of removing doc for XML part10:59
dulekgeguileo: As on cinder.service level there's no access to instance of cinder.volume.api.API10:59
sheele0ne:  from api-site10:59
sheele0ne:  ok?10:59
geguileodulek: So I see 2 options (I haven't given too much though since I just found out the problem)10:59
e0nesheel: good idea. I've forgot about it10:59
geguileowell 310:59
geguileo1- restart API nodes after upgrades (this one sucks)10:59
sheele0ne: no issues, i will do it in a while...11:00
dulekgeguileo: Sorry, I'll read that in a moment, need to go away for few minutes.11:00
sheele0ne:  thanks for confirmation..11:00
geguileo2- Upgrade API nodes the last (I don't like because you would nave different APIs running against an upgraded cloud)11:00
geguileodulek: Ok, I'll just finish talking to the wall and you can read it when you come back  ;-)11:00
*** dave-mccowan has joined #openstack-cinder11:01
geguileo3- API detects we are doing a rolling upgrade, so prediocally checks DB to see if upgrade has completed and when it does changes RPC instances11:01
*** klkumar has joined #openstack-cinder11:02
e0nesheel: thank you for working on it11:09
sheele0ne: :)11:09
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376611:11
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376611:11
dulekgeguileo: 4 - implement SIGHUP handling for cinder-api?11:12
geguileodulek: What if cinder-api is run under apache?  Would that still work?11:17
dulekgeguileo: Good point, probably not.11:19
geguileodulek: That's why I discarded that option   :-(11:19
dulekgeguileo: But why restart is a problem actually? If you're running an available cloud, then your APIs are probably behind HAProxy.11:20
dulekgeguileo: You can just get them out of the cluster and restart one-by-one.11:21
*** tpsilva has joined #openstack-cinder11:21
geguileodulek: Well, I think it doesn't make much sense...11:22
dulekgeguileo: What am I missing? :)11:23
geguileodulek: If you upgrade your API at the end you don't need to restart them one additional time11:23
geguileodulek: right?11:23
dulekgeguileo: That's definitely right - all RPC APIs will be fine then.11:23
geguileodulek: Wait... wouldn't we have something out of sync there?11:24
geguileoIf we unpin c-sch and c-vol first, and proceed to upgrade one by one the APIs11:24
geguileoWe would have some APIs that have new endpoints and some that don't11:25
geguileoAnd if we do it before unpinning then we have API that can try to make calls to new APIs but nobody would be available to answer those calls...11:25
geguileoI think this second option would be less problematic11:26
dulekgeguileo: Yeah, that's the moment when pinning c-api to microversion comes to the picture. But wait, unpinning would probably require a restart as well…11:26
geguileoFun, fun, fun!!!11:27
dulekgeguileo: In some cases we're already returning an error for the user if he hits new API while upgrade is taking place.11:27
dulekgeguileo: e.g. cascade on volume delete.11:28
geguileoI think the best way would be to upgrade API last and unpin everything else after they have been upgraded11:28
dulekgeguileo: I think you should be able to do that even in Liberty->Mitaka upgrade.11:30
*** e0ne has quit IRC11:30
geguileodulek: Great!11:30
dulekgeguileo: As API isn't listening for anything on RPC, it should be fine.11:30
dulekgeguileo: I haven't tested it though.11:31
geguileolol11:31
geguileoWell, I'll fix the 2 bugs I've found regarding pinning and hopefully my patches will pass CI then  lol11:31
dulekWhat's the second bug then? :P11:31
dulekOne is with empty services table.11:31
geguileoIf the API is started before the other services then it also pins to None11:32
dulekgeguileo: Oooh. These are connected, but yeah.11:33
geguileodulek: Not really connected, but yeah11:33
*** Poornima has joined #openstack-cinder11:34
geguileoOne is a problem with the order of starting your services and the other is a problem in the order of operations within a component11:34
dulekgeguileo: But triggered by the same condition. ;)11:35
geguileoYes, that's why I said "but yeah", because it's all related  :-)11:35
dulekblinky_ghost: I've just looked through the code and it seems it's not implemented in the driver itself.11:39
dulekblinky_ghost: Maybe smcginnis will be able confirm when he'll be online (in 2-3 hours I think).11:39
blinky_ghostdulek: Ok thanks, smcginnis is responsible for this driver?11:40
dulekblinky_ghost: I'm not sure if he is personally, but definitely he's a Dell guy. ;)11:41
*** markvoelker has joined #openstack-cinder11:41
*** markvoelker has quit IRC11:45
*** timcl has joined #openstack-cinder11:51
openstackgerritMichal Dulko proposed openstack/cinder: Remove support for 1.x Backup RPC API  https://review.openstack.org/29769911:51
*** openstack has quit IRC12:04
*** openstack has joined #openstack-cinder12:07
dosaboyhi folks, i've had this patch up for review for a short while now - https://review.openstack.org/#/c/278629/12:07
dosaboybasically cinder ceph backup is kindof broken without that12:07
dosaboyand has been for a few releases12:07
dosaboyso would be fab to have some eyes on it12:08
*** cheneydc has joined #openstack-cinder12:12
*** markvoelker has joined #openstack-cinder12:12
*** houming has quit IRC12:12
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database  https://review.openstack.org/15396912:15
*** yangyapeng has joined #openstack-cinder12:21
openstackgerritMichal Dulko proposed openstack/cinder: Remove support for 1.x scheduler RPC API  https://review.openstack.org/29770012:22
*** jamielennox|away is now known as jamielennox12:26
*** jwcroppe has joined #openstack-cinder12:33
*** laughterwym has joined #openstack-cinder12:34
*** zhonghua-lee has quit IRC12:35
*** adrianofr_ has joined #openstack-cinder12:36
*** e0ne has joined #openstack-cinder12:36
*** cbader has quit IRC12:37
*** cbader has joined #openstack-cinder12:37
*** jwcroppe has quit IRC12:37
*** erlon has joined #openstack-cinder12:38
*** salv-orlando has joined #openstack-cinder12:39
smcginnisblinky_ghost: Just saw you comment. Yeah, it looks like the eql driver does not support managing existing yet.12:48
smcginnisblinky_ghost: IIRC, there is a blueprint out there to add it.12:48
*** rlrossit has joined #openstack-cinder12:49
*** gouthamr has joined #openstack-cinder12:50
*** klkumar has quit IRC12:51
*** links has quit IRC12:52
*** liverpooler has quit IRC12:55
*** crose has joined #openstack-cinder12:55
*** yangyapeng has quit IRC12:55
sheele0ne: here you go https://review.openstack.org/#/c/30458512:58
*** gsilvis_ is now known as gsilvis12:59
*** cdelatte has quit IRC13:00
e0nesheel: thanks, I'll +1 on it  once CI passed13:02
e0nesmcginnis: maybe, your +1 is also required for the patch above ^^13:02
smcginnise0ne: I'll take a look.13:02
e0nesmcginnis: thanks. it's about removing xml api docs13:02
smcginnisOh good!13:03
smcginnisJenkins just +1'd too.13:03
sheele0ne:  smcginnis I have added you both as reviewer  on this change13:03
sheelVerified+1 Jenkins13:03
e0ne+1'd13:05
e0nescottda: thanks for the review13:07
*** mriedem_away is now known as mriedem13:08
sheele0ne: thanks13:08
scottdae0ne: Cool. Also, we might try to track down prometheusunbound, who filed the original bug...13:08
*** EinstCrazy has joined #openstack-cinder13:09
scottdae0ne: They were running py3.x and found the issue, so they might be able to test. But he/she is not on IRC ATM.13:09
e0nescottda: good point. it will be great13:09
scottdae0ne: I also added Haypo as a reviewer13:10
*** ankit_ag has quit IRC13:13
scottdae0ne: I know you are busy, but it would be nice to get some quick feedback on the approach for /v3 endpoints in cinderclient:13:15
scottdahttps://review.openstack.org/#/c/30002813:15
e0nescottda: I'm setting up my end13:15
e0nes/end/env13:16
scottdaAfter talking with geguileo , I've moved all logic to /v3, so we can keep future changes in one place.13:16
scottdalots of files renamed, so don't look at the LOC, it's scary.13:16
*** mtanino has joined #openstack-cinder13:18
*** xyang1 has joined #openstack-cinder13:20
*** EinstCrazy has quit IRC13:21
*** yhayashi has quit IRC13:21
e0nescottda: should I somehow configure my env to test your patches?13:21
e0neor it's enouph to install updated cinderclient13:22
*** cdelatte has joined #openstack-cinder13:22
*** qeelee has quit IRC13:23
scottdae0ne: Your env will need /v3 endpoint on Cinder API13:25
scottdaThere's a patch for this in Devstack...are you using Devstack?13:25
e0nescottda: yes. but I can't find patch to devstack13:26
scottdahttps://review.openstack.org/#/c/30058513:26
e0nei'm using these search options: https://review.openstack.org/#/q/owner:%22Scott+DAngelo%22+status:open13:26
scottdaThat is owned by ameade ^^^13:27
e0nescottda: oh! you aren't an author of that patch. thanks for the link13:27
*** mylu has joined #openstack-cinder13:29
ildikovscottda: hey13:29
scottdaildikov: hi13:29
ildikovscottda: did you have any chats about multiattach yesterday finally?13:30
sheelHello All, could someone please send this to workflow: https://review.openstack.org/#/c/278605/13:30
*** mtanino has quit IRC13:30
scottdaildikov: No, I did not. I'm hoping we can discuss with jgriffith and hemna , but it did not happen yesterday. Maybe when you are around a little later you can ping them.13:31
ildikovscottda: ok, sure, will do13:31
*** raildo is now known as raildo-afk13:32
*** jungleboyj has quit IRC13:33
*** edmondsw has joined #openstack-cinder13:34
*** links has joined #openstack-cinder13:36
*** mylu has quit IRC13:37
*** mylu has joined #openstack-cinder13:38
*** Poornima has quit IRC13:39
*** akerr has joined #openstack-cinder13:40
*** knikolla has joined #openstack-cinder13:42
*** EinstCrazy has joined #openstack-cinder13:42
*** ametts has joined #openstack-cinder13:43
*** eharney has joined #openstack-cinder13:44
*** jgregor has joined #openstack-cinder13:44
*** sayalilunkad has quit IRC13:47
*** akshai has joined #openstack-cinder13:48
blinky_ghostsmcginnis: ok thanks, do you know when it will implemented? thanks13:49
*** sayalilunkad has joined #openstack-cinder13:50
*** sayalilunkad has quit IRC13:51
*** sayalilunkad has joined #openstack-cinder13:51
*** yangyapeng has joined #openstack-cinder13:53
*** jhesketh has left #openstack-cinder13:54
*** jhesketh has joined #openstack-cinder13:54
*** dustins has joined #openstack-cinder13:54
smcginnisblinky_ghost: Sorry, I don't.13:54
*** jhesketh has left #openstack-cinder13:55
smcginnisblinky_ghost: I don't maintain that one. I'll ping some folks internally though.13:55
*** crose has quit IRC13:55
smcginnisEmail sent to that team.13:58
*** sayalilunkad has quit IRC13:58
*** klkumar has joined #openstack-cinder13:59
*** yangyapeng has quit IRC14:00
*** martyturner has joined #openstack-cinder14:02
*** mriedem is now known as mriedem_meeting14:03
*** Julien-zte has joined #openstack-cinder14:04
*** klkumar has quit IRC14:05
*** diablo_rojo has joined #openstack-cinder14:07
*** enriquetaso has joined #openstack-cinder14:07
*** links has quit IRC14:08
*** cknight has joined #openstack-cinder14:10
*** mtanino has joined #openstack-cinder14:10
*** sayalilunkad has joined #openstack-cinder14:10
blinky_ghostsmcginnis: thanks. I'm using a pair of eql to deploy DR between sites and would be nice to get this working.14:14
smcginnisblinky_ghost: Yeah, makes sense, especially in that scenario.14:15
smcginnisblinky_ghost: They are looking into it now.14:15
openstackgerritMerged openstack/cinder: Fix driver.failover_host call in manager.py  https://review.openstack.org/29972314:15
smcginnisblinky_ghost: Not sure of other priorities, but hopefully this wouldn't take too long to add.14:15
blinky_ghostthanks, I think I have a solution that's using cinder lvm iscsi driver to EQL Lun but I would prefer to use the native driver of course :)14:17
*** Julien-z_ has joined #openstack-cinder14:17
openstackgerritMerged openstack/cinder: Update SolidFire driver to store cluster info  https://review.openstack.org/29974514:17
*** Julien-zte has quit IRC14:20
*** lprice has joined #openstack-cinder14:24
*** andrei_ has quit IRC14:24
*** liverpooler has joined #openstack-cinder14:25
*** yangyapeng has joined #openstack-cinder14:32
*** rhagarty__ has quit IRC14:34
*** jwcroppe has joined #openstack-cinder14:35
*** liverpooler has quit IRC14:37
*** rhagarty has joined #openstack-cinder14:39
*** tongli has joined #openstack-cinder14:40
mriedem_meetinghemna: would you be able to do a nova fishbowl session on live migration at 5:20 on wed at the summit? i don't see any cinder sessions at that time14:40
ameadeDuncanT: https://review.openstack.org/#/c/298052/ responded and i agree fyi14:40
*** jwcroppe has quit IRC14:40
smcginnisscottda: Thanks, I was hoping you would say that. :)14:41
*** haplo37 has joined #openstack-cinder14:43
*** mvk has quit IRC14:43
*** baumann has joined #openstack-cinder14:43
scottdasmcginnis: Maybe we should start a date survey? We could separate that from the location voting, perhaps, since you've listed quite a few possibilities.14:43
smcginnisscottda: Sure, I was thinking we could get some feedback on the ML, but maybe capturing in a survey would be good.14:44
smcginnisscottda: I'll give it a little time in case someone points out something I missed as to why any of those weeks wouldn't work.14:44
scottdasmcginnis: Sounds good.14:45
smcginnisscottda: Maybe Thurs/Fri I'll send out a survey.14:45
smcginnisscottda: Any conflicts for any of those weeks in FC?14:45
*** BobBall has joined #openstack-cinder14:45
scottdasmcginnis: Not that I know of. As far as rooms for the midcycle we've generally plenty, but good to book early. As far as the town there are definitely better weekends before/after, i.e. brewfests, etc.14:46
*** mylu has quit IRC14:47
smcginnisOoh, brewfests? Maybe that's relevant data to include in the decision. ;)14:47
*** raildo-afk is now known as raildo14:48
*** bjornar has quit IRC14:50
*** liverpooler has joined #openstack-cinder14:50
scottdasmcginnis June 25-26 http://downtownfortcollins.com/events/colorado-brewers-festival14:51
timclsmcginnis: I just reponded to the ML that we could host in Raleigh again.  Downside is summer in the South14:53
smcginnistimcl: Awesome, thanks!14:55
smcginnistimcl: I've had enough of the cold now, I could use a little heat! :)14:55
*** baumann1 has joined #openstack-cinder14:55
*** lprice1 has joined #openstack-cinder14:55
timclsmcginnis: :) but it isn't the heat it is the humidity :)14:55
smcginnistimcl: We have neither one here, but I'm sure the novelty would wear off quickly.14:56
*** dustins has quit IRC14:56
timclsmcginnis: yes in about 30 minutes or walking from your A/C room to the hot car...14:57
*** baumann has quit IRC14:57
smcginnis;)14:57
*** martyturner has quit IRC14:58
*** lprice has quit IRC14:59
*** ametts has quit IRC14:59
*** mylu has joined #openstack-cinder14:59
*** mvk has joined #openstack-cinder15:00
*** martyturner has joined #openstack-cinder15:01
*** dustins has joined #openstack-cinder15:01
*** edtubill has joined #openstack-cinder15:01
*** arecknag has quit IRC15:02
*** karthikp has joined #openstack-cinder15:03
openstackgerritGorka Eguileor proposed openstack/cinder: Fix service version pinning on start  https://review.openstack.org/30468515:06
openstackgerritGorka Eguileor proposed openstack/cinder: Fix RPC and Object version pinning on API  https://review.openstack.org/30468615:06
geguileodulek: ^ Those are the 2 patches15:06
*** afazekas has quit IRC15:09
*** afazekas has joined #openstack-cinder15:09
*** mjura has quit IRC15:12
*** hemna_ has joined #openstack-cinder15:13
DuncanTDublin is nice in the summer.... civilised amounts of warm, and hopefully not too much rain15:13
*** abehl has quit IRC15:15
e0neDuncanT: will we have a next meetup there?15:15
DuncanTe0ne: I will certainly put it on the ballot15:16
*** mylu has quit IRC15:16
smcginnise0ne: Just started the discussion on the ML.15:16
*** mylu has joined #openstack-cinder15:16
anteayaDublin!15:16
anteayaDuncanT: how's the dancing in Dublin?15:16
smcginnisDuncanT: I was impressed by how much tech has built up there. A lot of development since I was there in 2000.15:16
smcginnisanteaya: Hah!15:17
anteaya:)15:17
DuncanTanteaya: Of course15:17
DuncanTanteaya: The dancing is great15:17
anteayasmcginnis: let's not fool ourselves, eh?15:17
anteayaDuncanT: Dublin!!15:17
smcginnisanteaya: Gotta have priorities. ;)15:17
DuncanTanteaya: At least the swing, blues, salsa and irish dancing15:17
anteayasmcginnis: indeed15:17
*** openstackgerrit has quit IRC15:18
anteayaglad wifi made the list15:18
anteaya:)15:18
smcginnishehe15:18
anteayaDuncanT: oh, irish dancing15:18
anteayaDublin would be a good place for that15:18
DuncanTanteaya: The south west is far better for it, but there's some to be found in Dublin15:18
*** openstackgerrit has joined #openstack-cinder15:19
anteayamore than in texas I bet15:20
DuncanTWell, the best Caeli class I've found was in San Francisco, so you might be surprised15:21
*** jungleboyj has joined #openstack-cinder15:22
*** salv-orlando has quit IRC15:22
karthikpDoes anybody know to return a Oslo Volume versioned object in test casees15:22
lprice1jgriffith: Can you give me a review on https://review.openstack.org/#/c/246990/ when you get a chance?15:25
hemna_HPE folks won't make Dublin.15:27
DuncanThemna_: That might be solvable15:27
Swansonthat's why they're going15:28
hemna_we asked last time it was raised and our mgmt said no immediately.15:28
openstackgerritMerged openstack/os-brick: Updated from global requirements  https://review.openstack.org/30287315:28
SwansonThe PTL is selling Grit door to door to get enough money to go.15:29
Swanson(I am so old.)15:29
smcginnisGrit?15:29
DuncanTlprice1: Can to E-Series not do per-export secrets?15:30
*** yangyapeng has quit IRC15:32
*** logan- has quit IRC15:34
mc_nairkarthikp: are you talking about something like this - https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_volume.py#L3566?15:37
*** crose has joined #openstack-cinder15:39
lprice1DuncanT: E-Series is capable of Bi-directional CHAP, but the target chap secret is global, if that's what you're asking. It isn't capable of different secrets for each initiator (on the target side).15:39
*** logan- has joined #openstack-cinder15:40
DuncanTlprice1: basically I was looking at https://review.openstack.org/#/c/246990/ and wondering why it doesn't generate a chat secret randomly at export time like the LVM driver does, which has substantial security benefits15:40
*** dustins has quit IRC15:42
*** salv-orlando has joined #openstack-cinder15:48
*** porrua has joined #openstack-cinder15:48
*** mriedem_meeting is now known as mriedem15:49
lprice1DuncanT: Let me take a closer look at that and get back to you. If I understand correctly, then yes, I would think the driver would be capable of generating them at export time.15:50
karthikpmc_nair, i actually need to create a volume object instead of a volume dict ..15:51
karthikpmc_nair, can I use something like tests_utils.create_volume?15:51
DuncanTlprice1: I think it would be an improvement15:51
DuncanTlprice1: I've also got investigating bi-directional CHAP on my todo list... it would hopefully be useful too15:52
*** cheneydc has quit IRC15:53
bswartzdoes cinder have any kind of support for iSCSI+IPSec?15:53
*** EinstCrazy has quit IRC15:53
bswartzbecause IPSec might also improve security15:53
mriedemscottda: smcginnis: hemna: jgriffith: fyi nova is going to have a session on live migration on wed at 5:20 at the summit, there  are no cinder sessions at that time; it's not going to be all about cinder, but volumes might come up so we probably want some cinder people in the room15:54
mriedemat least hemna15:54
scottdamriedem: Sounds good. I reckon I'll be there.15:55
*** Julien-z_ has quit IRC15:57
mc_nairkarthikp: So https://github.com/openstack/cinder/blob/master/cinder/tests/unit/utils.py#L77 does appear to return a volume object. This utility is may be what you were looking for, but it does essentially what that other link does - creates a Volume object based on a dict15:57
karthikpmc_nair, perfect thanks!15:58
mc_nairkarthikp: sure thing16:00
mriedemmc_nair: any chance you could take a look at this bug? https://bugs.launchpad.net/nova/+bug/1565859 it's about libvirt volume detach timing out (failing) with storwize as the backend16:01
openstackLaunchpad bug 1565859 in OpenStack Compute (nova) "Can't detach SVC volume from an instance; guest detach device times out" [Undecided,New]16:01
*** leeantho has joined #openstack-cinder16:01
*** esker has joined #openstack-cinder16:01
mc_nairmriedem: sure - will take a look16:01
mriedemmc_nair: thanks16:02
*** jungleboyj has quit IRC16:02
jgriffithmriedem: hemna it is16:03
*** avishay has joined #openstack-cinder16:04
openstackgerritMerged openstack/python-cinderclient: Graduate to oslo.i18n and cleanup incubator usage  https://review.openstack.org/29475116:04
*** darosale has joined #openstack-cinder16:05
*** leeantho_ has joined #openstack-cinder16:06
avishaysmcginnis: hey sean, thanks for the feedback on the list manageable patch. i thought it would be useful to get feedback from some vendor representatives in tomorrow's weekly meeting. is the agenda under "tomorrow's meeting" actually for tomorrow, even though it is listed as April 6?  (the first meeting under "previous meetings" is also april 6)16:06
*** martyturner has quit IRC16:07
*** martyturner has joined #openstack-cinder16:08
ildikovjgriffith hemna_: hi. can we aim for having the multiattach discussion some time today scottda pinged us about yesterday?16:09
*** leeantho has quit IRC16:09
hemna_sure16:10
*** asselin has quit IRC16:12
*** esker has quit IRC16:12
ildikovcoolio, let's see what others say16:13
ildikovhemna_: have you progressed further with the diagrams?16:13
ildikovhemna_: I saw that the last version of the attach one is v13 IIRC16:13
ildikovon Gliffy16:13
scottdaildikov: I'm in for a discussion, but we need jgriffith ....16:14
jgriffithscottda: sure16:14
*** avishay has quit IRC16:14
jgriffithildikov: scottda hemna_ what time?16:14
jgriffithchannel etc16:14
ildikovI think we can keep it here16:15
ildikovand I can adapt regarding time16:15
scottdaI'm pretty flexible all day. Best before 16:00 MDT/ 22:00 UTC and it's already getting late for ildikov ...16:15
scottdanow? or top of the hour (11:00 MDT/17:00 UTC)?16:15
jgriffithJust let me know, I'll adjust.  Can do right now even if that works16:15
ildikovwhat I would like to know and was asked by mriedem also a short while ago, whether we know now which alternative to choose, what was the otcome of your discussion about #2 and #416:16
*** fernnest has joined #openstack-cinder16:16
ildikovI'm available now as well16:16
jgriffithildikov: honestly I they're not really different IMO16:16
ildikovhemna_?16:16
jgriffithildikov: but reserve IMO should not be changed from being a simple reserve16:17
scottdaI think #2 and #4 are close enough. Just a small detail about attachment_id being returned in 1 of 2 possible places.16:17
jgriffithildikov: I'm not going to spend my morning arguing this though :)16:17
ildikovjgriffith: that's a good news IMHO :)16:17
scottdaI also think we could punt on that detail for now....16:17
*** laughterwym has quit IRC16:17
scottdaAnd so real question is What next? A Cinder spec for this?16:17
jgriffithIf there's agreement in the approach, I'd prefer to just go and finish the code16:17
ildikovjgriffith: hmm, I thought you sorted this out with hemna_ last time :)16:17
ildikovI mean reserve16:18
jgriffithildikov: not sure what you mean?16:18
jgriffithildikov: oh, the purpose of reserve?16:18
jgriffithYeah, I don't care anymore16:18
ildikovjgriffith: that you said you don't want to argue about 'reserve' and I remember I saw your chat with hemna_ last week after the meeting here and I thought you sorted that aspect out as well16:18
*** asselin has joined #openstack-cinder16:19
jgriffithildikov: no we sorted out that hemna_ didn't see the code in my patch due to a gerritt viewing issue16:19
ildikovjgriffith: yes, sorry, I'm less fresh by now even if it's not that late  and that affects my typing "style" :)16:19
scottdaLet's step back a bit....16:20
jgriffithildikov: I'd propose I just continue with my patch16:20
scottdajgriffith: Do you think this needs a spec?16:20
jgriffithif folks don't like it or it doesn't work then cool16:20
jgriffithscottda: absolutely not16:20
ildikovscottda: yeah sure, sorry16:20
scottdaok, I'm good with that.16:20
*** nkrinner has quit IRC16:20
ildikovjgriffith: is there any API change in what you're doing?16:20
scottdaI reckon the next step would be for jgriffith to move on with the patch then...16:20
ildikovor your change will not add new visible things16:21
scottdaAnd once we see what the API changes regarding what is returned to Nova (attachment_id) we can look at what needs to change in Nova.16:21
jgriffithildikov: yes, there's an optional retrun from attach, but with the exception of my *next* patch that's completely internal between Cinder and Nova only16:21
jgriffithscottda: ildikov keep in mind that for now, the only consumer of those API calls are Nova16:21
scottdaQuestion: Does Nova need a spec for this? mriedem or ildikov ?16:22
jgriffithscottda: Nova will need something, because there will be more change there16:22
ildikovjgriffith: I asked it from a spec and microversioning perspective rather than anything technical16:22
jgriffithildikov: got ya16:22
*** dalgaaf has joined #openstack-cinder16:22
mriedemscottda: i'm not following the conversation, spec for what?16:23
jgriffithildikov: I'll leave it to you and scottda... I'm trying to stay out of microversions and specs16:23
mriedemspec for multiattach, yes, there is one16:23
ildikovjgriffith: I think it's better to capture these things in written form as well even if code talks the best to be able to track why we changed the code16:23
jgriffithmriedem: spec for changes to nova16:23
mriedemyes16:23
scottdamriedem: spec for NOva if Cinder API changes to return something that is not currently returned...16:23
ildikovjgriffith: I cannot blame you for that :)16:23
jgriffithmriedem: my suggestion is wait til I have some sort of POC code (or somebody does)16:23
mriedemwell,16:23
scottdai.e. attachment_id will not be returned during intialize_connection()16:23
mriedemcinder will be making api changes, which nova will use as part of the multiattach spec16:23
mriedemi considered it a dependency of the nova multiattach spec16:23
mriedemnot separate16:23
mriedemjgriffith: agree with you on needing cinder POC first16:24
scottdaOK, just checking on timing of Nova specs, since there are deadlines, etc.16:24
ildikovmriedem: it can be dependency, I agree but I would like to have a separate document to not mix clear multiattach things with this16:24
*** BigWillie has joined #openstack-cinder16:25
ildikovscottda: if we want multiattach in for Newton in Nova then I kind of doubt we will loose this battle due to spec deadlines16:25
hemna_there isn't a lot of difference between what I had proposed and what jgriffith has and I'm just tired of arguing so alt #4 is dead IMHO.16:25
ildikovalso I think we should be able to capture what we want to do after the summit16:26
mriedemildikov: i'm not sure what the new thing is16:26
mriedemstoring connector info in the vol attachment?16:26
*** ebalduf_ has joined #openstack-cinder16:26
mriedemildikov: i think we need to go into the summit thinking we need to come out of it knowing what we're going to do16:26
scottdaildikov: I'm not sure what you mean about losing the battle? I'm just trying to figure out what it will take for Nova to change to work with jgriffith Alt#2 changes....16:26
ildikovmriedem: storing and using that, figuring out how many other places we are using attachment_id compared to today, etc16:26
mriedemildikov: if cinder is going to change the api and nova needs to use the new api regardless of multiattach, then that's probably fine as it's own specless blueprint16:27
mriedemunless it changes the nova api behavior, then it needs a spec16:27
ildikovscottda: I meant the deadlines, as we're trying to add this to Nova for a while now16:27
ildikovmriedem: cool, that sounds good to me16:28
scottdaOK, so it sounds like we don't need a Nova spec. So....jgriffith Can please move on with the code....16:28
scottdaand whenever code is stable, *someone* can look at what Nova changes will look like.16:29
scottdaand start POC for the Nova code.16:29
ildikovscottda: we will still need to capture what we will do in Nova, but we can do it after jgriffith has a PoC and we have a better idea how that will actually work16:29
*** jwcroppe has joined #openstack-cinder16:29
ildikovI'm happy to chime in regarding the Nova PoC part16:29
scottdaildikov: Sounds like we are in sync :)16:29
jgriffithildikov: good, then I'll ask you for help :)16:30
ildikovmaybe even before having that code very stable just to at least understand the impacts16:30
ildikovjgriffith: I wanted to ask you for help, but maybe we both go to mriedem for this purpose :)16:30
scottdaOK. Anything else we need to work through?16:30
jgriffithildikov: haha.... perhaps :)16:31
scottdaAnd if we've a meeting for tomorrow, is there anything we would need to discuss there?16:31
*** xinli has quit IRC16:31
ildikovscottda: I sent out the reminder to the list16:31
*** xinli has joined #openstack-cinder16:31
openstackgerritJohn Griffith proposed openstack/cinder: Add failed_over detection to SolidFire driver  https://review.openstack.org/29974816:31
openstackgerritJohn Griffith proposed openstack/cinder: Implement Cluster Pairing for SolidFire Driver  https://review.openstack.org/29974616:31
openstackgerritJohn Griffith proposed openstack/cinder: Implement replicate_volume & failover on SolidFire  https://review.openstack.org/29974716:31
scottdaildikov: But what is the agenda?16:31
ildikovscottda: we have several things for multiattach16:31
mriedemildikov: go to me for what purpose?16:32
mriedemwriting the nova code to use the new cinder api changes?16:32
scottdaildikov: OK, just checking. We can certainly have a meeting if needed.16:32
ildikovit's also a question how much we would like to check hemna_'s diagrams for instance16:32
jgriffithmriedem: don't worry ;)  Just saying that if we got hung up on something on the nova side16:32
jgriffithmriedem: ie a tech consult etc16:32
hemna_I'm still working on live migration fwiw16:32
ildikovmriedem: answering the stupid questions of mine and jgriffith's valid ones so that we can write it16:32
scottdahemna_: Cool. Those diagrams are great, and useful for more beyond this specific issue.16:33
ildikovscottda: +116:33
hemna_scottda, thanks.16:33
hemna_I'm thinking it sure would be nice to have those for other Cinder things internally16:33
openstackgerritTom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings  https://review.openstack.org/30018916:33
openstackgerritTom Barron proposed openstack/cinder: Fix uuid warnings from test_db_api unit tests  https://review.openstack.org/30073016:33
openstackgerritTom Barron proposed openstack/cinder: Tests: Fix invalid UUIDs in EMC VNX tests  https://review.openstack.org/29311916:33
openstackgerritTom Barron proposed openstack/cinder: fix invalid uuid warnings from test_quota  https://review.openstack.org/29785416:33
openstackgerritTom Barron proposed openstack/cinder: fix invalid uuid in emc extremio unit tests  https://review.openstack.org/29784816:33
openstackgerritTom Barron proposed openstack/cinder: Fix fake uuids in NetApp ESeries unit tests  https://review.openstack.org/29736316:33
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuids in tintri unit tests  https://review.openstack.org/29737916:33
hemna_but they are time consuming to create16:33
openstackgerritTom Barron proposed openstack/cinder: fix invalid uuid warnings from test_cmd  https://review.openstack.org/29786616:33
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuids in EMC scaleio unit tests  https://review.openstack.org/29737516:33
openstackgerritTom Barron proposed openstack/cinder: Fix uuid warnings from db unit tests  https://review.openstack.org/29725516:33
openstackgerritTom Barron proposed openstack/cinder: Fix invalid uuids in sheepdog unit tests  https://review.openstack.org/29745316:33
openstackgerritTom Barron proposed openstack/cinder: Fix uuid warnings from api.contrib backup tests  https://review.openstack.org/30476716:33
openstackgerritTom Barron proposed openstack/cinder: Fix admin_actions uuid warnings in api.contrib  https://review.openstack.org/30476816:33
openstackgerritTom Barron proposed openstack/cinder: Fix CG uuid warnings in api.contrib unit tests  https://review.openstack.org/30476916:33
openstackgerritTom Barron proposed openstack/cinder: Fix qos_manage uuid warnings in api.contrib tests  https://review.openstack.org/30477016:33
openstackgerritTom Barron proposed openstack/cinder: Fix vol type/spec uuid warnings in api.contrib  https://review.openstack.org/30477116:33
openstackgerritTom Barron proposed openstack/cinder: Fix volume test uuid warnings in api.contrib  https://review.openstack.org/30477216:33
ildikovhemna_: have you started detach as well or that's still on my plate?16:34
jgriffithtbarron: dirty dog you!!16:34
mriedemildikov: ok16:34
hemna_detach is done16:34
*** mylu has quit IRC16:34
hemna_it's already up16:34
hemna_check the ether pad for the link16:34
tbarronjgriffith: :)16:34
ildikovhemna_: ok, cool, I missed it16:34
jgriffithtbarron: why get one commit for fixing UUID warnings when you can get 10 :)  Seriously, I do like having them split out... just messing with ya16:35
ildikovhemna_: it was too high on the page :)16:35
hemna_ildikov, http://www.gliffy.com/go/publish/image/10381829/L.png16:35
jgriffithtbarron: much more pleasant to review16:35
ildikovhemna_: but found it, thanks!16:35
hemna_np16:35
tbarronjgriffith: that's the idea, plus reducing the scope of merge conflicts on any particular patch16:36
hemna_live migration is quite a bit more complicated, so it's taking some time to get done16:36
ildikovhemna_: nice one16:36
*** jungleboyj has joined #openstack-cinder16:36
*** dustins has joined #openstack-cinder16:42
*** apoorvad has joined #openstack-cinder16:42
*** akshai has quit IRC16:47
*** liverpooler has quit IRC16:47
*** jwcroppe has quit IRC16:50
*** akshai has joined #openstack-cinder16:52
*** Guest15115 has quit IRC16:55
*** jwcroppe has joined #openstack-cinder16:57
*** lpetrut has quit IRC16:57
*** sjackson has joined #openstack-cinder16:57
openstackgerritEric Harney proposed openstack/cinder: Doc: Remove incorrect run_tests.sh documentation  https://review.openstack.org/27003616:58
*** salv-orlando has quit IRC16:58
*** jwcroppe has quit IRC17:01
*** akshai has quit IRC17:03
*** akshai has joined #openstack-cinder17:03
openstackgerritAlex Meade proposed openstack/python-cinderclient: WIP: Add v3 user messages  https://review.openstack.org/29905217:05
*** raildo is now known as raildo-afk17:08
*** ircuser-1 has joined #openstack-cinder17:08
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add /v3 endpoint support for cinderclient  https://review.openstack.org/30002817:14
*** jistr has quit IRC17:15
*** mriedem has left #openstack-cinder17:17
*** mriedem has joined #openstack-cinder17:17
*** jordanP has quit IRC17:17
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Support api-microversions  https://review.openstack.org/30194117:18
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add api-version to get server versions  https://review.openstack.org/30362717:20
*** andymaier has joined #openstack-cinder17:20
*** xinli has quit IRC17:21
*** xinli has joined #openstack-cinder17:21
*** dencaval has joined #openstack-cinder17:22
openstackgerritKendall Nelson proposed openstack/cinder: Add SnapshotStatus enum field  https://review.openstack.org/28482517:22
*** salv-orlando has joined #openstack-cinder17:24
*** raildo-afk is now known as raildo17:29
*** rlrossit has quit IRC17:33
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Deprecating XML config file  https://review.openstack.org/29407617:33
*** salv-orl_ has joined #openstack-cinder17:37
*** salv-orlando has quit IRC17:39
*** mriedem1 has joined #openstack-cinder17:44
*** mriedem has quit IRC17:44
*** ametts has joined #openstack-cinder17:45
*** rlrossit has joined #openstack-cinder17:47
*** lpetrut has joined #openstack-cinder17:48
*** mriedem1 is now known as mriedem17:49
*** andymaier has quit IRC17:50
*** BigWillie has quit IRC17:56
*** BigWillie has joined #openstack-cinder17:57
*** Eva-i has quit IRC17:58
*** martyturner has quit IRC17:58
*** martyturner has joined #openstack-cinder18:02
*** jungleboyj has quit IRC18:02
jgriffithxyang1: ping18:02
xyang1jgriffith: hi18:02
jgriffithxyang1: howdy18:02
xyang1jgriffith: how are you18:02
jgriffithxyang1: hey, saw your note about release notes... good idea :)18:02
jgriffithxyang1: I'm alright, thanks... you?18:03
*** ChubYann has joined #openstack-cinder18:03
xyang1jgriffith: I am good18:03
xyang1jgriffith: so are you adding release notes?18:03
jgriffithxyang1: I was wondering if you knew of a doc on how we do that correctly?18:03
jgriffithxyang1: yes, I thought I'd work on adding them right now :)18:03
xyang1jgriffith: sure, let me find out:)18:03
xyang1jgriffith: there is a command18:04
jgriffithxyang1: ahh... yay for automation18:04
*** jgregor has quit IRC18:04
jgriffithxyang1: ahh... looks like it's in the tox file now18:05
xyang1jgriffith: oh, found it?18:05
*** jgregor has joined #openstack-cinder18:06
jgriffithxyang1: found *something* :)18:06
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088518:06
xyang1jgriffith: found the doc, added to the review18:06
jgriffithxyang1: perfect, thanks!18:07
xyang1jgriffith: welcome18:07
xyang1jgriffith: I didn't realize you did not implement cheesecake in your driver in Mitaka18:07
jgriffithxyang1: nope18:08
xyang1jgriffith: I split the tiramisu spec into two, generic volume group has its own spec now.  can you review them when you get chance18:09
jgriffithxyang1: yes, I will... and thanks for doing that18:09
xyang1jgriffith: looks cleaner this way.  generic group has a lot already18:10
SwansonDaily post to see if anyone could take a look at this one ... https://review.openstack.org/#/c/300118/18:14
openstackgerritJohn Griffith proposed openstack/cinder: Implement replicate_volume & failover on SolidFire  https://review.openstack.org/29974718:14
jgriffithSwanson: for you?  Nahhh18:14
jgriffith:)18:14
Swanson:)18:14
jgriffithSwanson: dang it!!  1100 lines :(18:14
SwansonIt is huge and I think that terrifies people.18:15
SwansonBut most of the changes are uuid stuff.18:15
jgriffithSwanson: does it work?18:15
SwansonYeppers18:15
SwansonTest dude hasn't poked at it but it has passed my tests.18:15
jgriffithSwanson: what happens in an upgrade?18:16
SwansonIf provider_id isn't found I operate as I have.18:16
jgriffithSwanson: ahh.. yeah, found it18:16
openstackgerritKendall Nelson proposed openstack/cinder: Add SnapshotStatus enum field  https://review.openstack.org/28482518:16
Swansonjgriffith, replication made that something of a nightmare.18:17
*** jungleboyj has joined #openstack-cinder18:17
jgriffithSwanson: yeah, no kidding.  Just wait til some are on one backend and others on a different one :(18:17
jgriffithSwanson: and they're all still active.... train-wreck18:17
Swansonjgriffith, Yeah, I have some issues with that. As I'm currently working out failback...18:18
Swansonjgriffith, actually, that's a question.  When someone fails back a backend is there something that prevents the user from attaching to those volumes?  Can I just set myself to some kind of in process state and error out until I'm done?18:19
jgriffithSwanson: personally I think failback and it's derivatives is a bad idea18:20
jgriffithSwanson: but, it would work just like failing over works currently18:20
jgriffithSwanson: just treat it the same way we do now18:21
jgriffithSwanson: I think there was somebody else that raised a potential issue because their backend takes a long time to fail over18:21
jgriffithbut not sure18:21
swamireddyHi18:24
swamireddyI got the below err when I tried to create a volume from an image18:25
swamireddycinder.volume.manager InvalidArgument: error creating clone18:25
swamireddyI use the ceph as backend for cinder and glance18:25
*** salv-orl_ has quit IRC18:26
*** salv-orlando has joined #openstack-cinder18:27
Swansonjgriffith, Thanks for the review!18:29
jgriffithSwanson: np18:29
Swansonjgriffith, so fundamentally assume the user understands what they are doing and just go for it.18:29
jgriffithSwanson: you better check with others, I seem to not know what I'm talking about half the time :)18:29
smcginnisThat just seems like a scary thing to assume.18:30
jgriffithSwanson: see... told ya :)18:30
smcginnisBut probably what we'll have to do.18:30
smcginnis:)18:30
*** salv-orlando has quit IRC18:31
Swansonsmcginnis, jgriffith, One path leads to hopelessness and the other to total despair.  Let's hope we have the wisdom to choose wisely.18:32
smcginnisSwanson: https://www.youtube.com/watch?v=z05StkAKKF018:35
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Support api-microversions  https://review.openstack.org/30194118:35
*** mvk has quit IRC18:37
*** jdurgin1 has joined #openstack-cinder18:38
*** lpetrut has quit IRC18:39
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add api-version to get server versions  https://review.openstack.org/30362718:40
*** raildo is now known as raildo-afk18:41
*** bjornar has joined #openstack-cinder18:43
*** lpetrut has joined #openstack-cinder18:44
*** jungleboyj has quit IRC18:47
*** jwcroppe has joined #openstack-cinder18:52
*** blinky_ghost has quit IRC18:54
*** raildo-afk is now known as raildo18:56
*** ametts has quit IRC18:58
openstackgerritScott DAngelo proposed openstack/cinder: Add ability to filter by volume_glance_metadata  https://review.openstack.org/14773818:59
*** martyturner has quit IRC19:07
*** martyturner has joined #openstack-cinder19:09
*** jungleboyj has joined #openstack-cinder19:10
*** raildo is now known as raildo-afk19:14
eharneyanyone want to review this patch to add pylint to cinderclient?  https://review.openstack.org/#/c/302960/19:15
eharneyi'd like to get that up and going before we add too many changes to cinderclient in Newton19:15
*** mylu has joined #openstack-cinder19:16
*** shyama has quit IRC19:21
*** vgridnev has joined #openstack-cinder19:21
*** mylu has quit IRC19:24
*** mylu has joined #openstack-cinder19:26
*** mylu has quit IRC19:26
*** martyturner has quit IRC19:30
*** mylu has joined #openstack-cinder19:33
*** martyturner has joined #openstack-cinder19:34
*** mylu has quit IRC19:36
*** mvk has joined #openstack-cinder19:36
*** mylu has joined #openstack-cinder19:36
*** ociuhandu has quit IRC19:37
*** esker has joined #openstack-cinder19:38
*** dustins has quit IRC19:39
openstackgerritTom Barron proposed openstack/cinder: Fix qos_manage uuid warnings in api.contrib tests  https://review.openstack.org/30477019:41
*** dustins has joined #openstack-cinder19:42
*** e0ne has quit IRC19:48
*** ociuhandu has joined #openstack-cinder19:52
*** raildo-afk is now known as raildo19:53
*** timcl has quit IRC19:54
*** mylu has quit IRC19:54
*** esker has quit IRC19:55
*** salv-orlando has joined #openstack-cinder19:55
*** mylu has joined #openstack-cinder19:55
*** martyturner has quit IRC19:55
*** esker has joined #openstack-cinder19:56
*** jwcroppe_ has joined #openstack-cinder19:56
*** jwcroppe has quit IRC19:56
*** angela-s has joined #openstack-cinder19:59
*** salv-orlando has quit IRC19:59
*** cdelatte has quit IRC20:00
*** mylu has quit IRC20:01
*** tongli has quit IRC20:01
*** mylu has joined #openstack-cinder20:02
*** jbernard_ has quit IRC20:07
*** mylu has quit IRC20:07
*** akerr has quit IRC20:08
*** jbernard1 has joined #openstack-cinder20:08
*** mylu has joined #openstack-cinder20:09
*** apoorvad has quit IRC20:11
*** mylu has quit IRC20:12
*** mylu has joined #openstack-cinder20:14
openstackgerritMerged openstack/cinder: Dell SC: Updated to utilize provider_id  https://review.openstack.org/30011820:18
openstackgerritMerged openstack/cinder: Fix invalid uuids in sheepdog unit tests  https://review.openstack.org/29745320:19
openstackgerritMerged openstack/cinder: Fix fake uuids in NetApp ESeries unit tests  https://review.openstack.org/29736320:19
openstackgerritMerged openstack/cinder: fix invalid uuid warnings from test_quota  https://review.openstack.org/29785420:19
*** mylu has quit IRC20:21
*** karthikp has quit IRC20:24
*** karthikp has joined #openstack-cinder20:24
*** Guest92024 has quit IRC20:27
*** Guest92024 has joined #openstack-cinder20:27
openstackgerritDanny Al-Gaaf proposed openstack/cinder: Fix doc and source documentation errors and warning  https://review.openstack.org/29305020:28
*** mylu has joined #openstack-cinder20:31
*** Lee1092 has quit IRC20:32
*** mylu has quit IRC20:33
*** esker has quit IRC20:33
*** martyturner has joined #openstack-cinder20:34
*** baumann1 has left #openstack-cinder20:34
*** apoorvad has joined #openstack-cinder20:34
*** esker has joined #openstack-cinder20:34
*** mylu has joined #openstack-cinder20:36
*** mylu has quit IRC20:40
*** salv-orlando has joined #openstack-cinder20:41
*** cknight has quit IRC20:44
*** esker has quit IRC20:44
*** mylu has joined #openstack-cinder20:47
*** BigWillie has quit IRC20:48
*** cdelatte has joined #openstack-cinder20:48
*** mvk_ has joined #openstack-cinder20:49
*** martyturner has quit IRC20:50
*** mylu has quit IRC20:51
*** salv-orlando has quit IRC20:51
*** e0ne has joined #openstack-cinder20:52
*** salv-orlando has joined #openstack-cinder20:52
*** martyturner has joined #openstack-cinder20:52
*** mvk has quit IRC20:53
*** fernnest has quit IRC20:56
openstackgerritEric Harney proposed openstack/cinder: Tests: Define fake_constants as constants  https://review.openstack.org/30486120:58
openstackgerritMerged openstack/cinder: VMware: Support for non-default port  https://review.openstack.org/26745020:59
*** v12aml has quit IRC21:01
*** mylu has joined #openstack-cinder21:02
*** rlrossit has quit IRC21:03
*** mylu has quit IRC21:03
*** v12aml has joined #openstack-cinder21:03
*** crose has quit IRC21:05
*** mylu has joined #openstack-cinder21:05
*** ametts has joined #openstack-cinder21:06
*** raildo is now known as raildo-afk21:06
*** mylu has quit IRC21:09
*** mylu has joined #openstack-cinder21:09
*** julim has quit IRC21:11
*** sdague has quit IRC21:11
*** mylu has quit IRC21:11
*** fernnest has joined #openstack-cinder21:11
guitarzanhmm, anyone see this before? "DetachedInstanceError: Parent instance <Volume at 0x7fb1934188d0> is not bound to a Session; lazy load operation of attribute 'volume_admin_metadata' cannot proceed\n"21:14
*** mylu has joined #openstack-cinder21:18
*** esker has joined #openstack-cinder21:20
*** esker has quit IRC21:20
*** e0ne has quit IRC21:20
*** xyang1 has quit IRC21:21
*** xyang1 has joined #openstack-cinder21:21
*** hemna_ has quit IRC21:22
*** esker has joined #openstack-cinder21:24
*** e0ne has joined #openstack-cinder21:25
*** gouthamr has quit IRC21:27
*** e0ne has quit IRC21:27
*** jgregor has quit IRC21:27
eharneyguitarzan: not in particular, but i think that's a class of error related to bugs in our objects code21:28
*** gouthamr has joined #openstack-cinder21:28
*** gouthamr has quit IRC21:28
*** diablo_rojo has quit IRC21:28
*** diablo_rojo has joined #openstack-cinder21:28
guitarzaneharney: this is liberty doesn't have full objects right?21:29
*** hemna_ has joined #openstack-cinder21:29
guitarzanmissed a word in there somewhere21:29
eharneyguitarzan: right, but, still a bug21:30
guitarzansure21:30
*** BobBall_ has joined #openstack-cinder21:37
*** BobBall_ is now known as Guest3026121:37
*** sheel has quit IRC21:37
*** porrua has quit IRC21:38
*** jungleboyj has quit IRC21:38
*** BobBall has quit IRC21:39
*** mriedem has quit IRC21:40
*** diablo_rojo has quit IRC21:41
*** Guest30261 has quit IRC21:42
*** BobBall has joined #openstack-cinder21:42
*** BobBall is now known as Guest1424321:42
*** haplo37 has quit IRC21:42
*** eharney has quit IRC21:47
*** vgridnev has quit IRC21:47
*** julim has joined #openstack-cinder21:49
*** knikolla has quit IRC21:50
*** adrianofr_ has quit IRC21:50
*** mylu has quit IRC21:53
*** karthikp has quit IRC21:57
*** karthikp has joined #openstack-cinder21:57
*** dustins has quit IRC21:58
*** salv-orlando has quit IRC22:00
*** salv-orlando has joined #openstack-cinder22:01
*** IlyaG has joined #openstack-cinder22:01
*** lpetrut has quit IRC22:04
*** mylu has joined #openstack-cinder22:04
*** akshai has quit IRC22:07
*** knikolla has joined #openstack-cinder22:11
*** mylu has quit IRC22:12
*** mylu has joined #openstack-cinder22:13
*** edtubill has quit IRC22:13
*** tpsilva has quit IRC22:17
openstackgerritMerged openstack/python-cinderclient: Add pylint tox env  https://review.openstack.org/30296022:25
*** darosale has quit IRC22:26
*** jwcroppe_ has quit IRC22:31
*** jwcroppe has joined #openstack-cinder22:32
*** jwcroppe has quit IRC22:36
*** ametts has quit IRC22:37
*** karthikp has quit IRC22:38
*** bjornar has quit IRC22:41
*** martyturner has quit IRC22:45
*** angela-s has quit IRC22:46
*** ebalduf_ has quit IRC22:47
*** mylu has quit IRC22:50
*** mylu has joined #openstack-cinder22:52
*** xyang1 has quit IRC22:54
*** lcurtis has quit IRC22:57
*** mriedem has joined #openstack-cinder22:57
*** dalgaaf has quit IRC22:58
*** mylu has quit IRC23:01
*** mylu has joined #openstack-cinder23:01
*** edmondsw has quit IRC23:01
*** mtanino has quit IRC23:03
*** karthikp has joined #openstack-cinder23:07
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/30489323:07
*** salv-orlando has quit IRC23:08
*** jamielennox is now known as jamielennox|away23:09
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/30287723:13
*** jamielennox|away is now known as jamielennox23:13
*** akshai has joined #openstack-cinder23:16
*** akshai_ has joined #openstack-cinder23:17
*** Guest92024 has quit IRC23:21
*** akshai has quit IRC23:21
*** Guest92024 has joined #openstack-cinder23:22
*** akshai_ has quit IRC23:30
hogepodgesmcginnis: can I ping you on this again? https://review.openstack.org/#/c/290271/23:43
*** karthikp has quit IRC23:51
*** yhayashi has joined #openstack-cinder23:51

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