Tuesday, 2017-01-17

jgriffithhemna patrickeast smcginnis can you guys do me a favor when you get a chance and let me know what it is that your FC drivers are returning on that terminate connection call?00:07
jgriffithhemna patrickeast smcginnis Swanson I started an etherpad here:  https://etherpad.net/p/cinder-fcResponses-terminate_connection00:07
smcginnisjgriffith: A bunch of crap.00:07
jgriffithif you guys want to just dump some info in there that would be helpful00:08
jgriffithsmcginnis LOL00:08
jgriffithsmcginnis perfect!00:08
smcginnisjgriffith: Swanson can probably give a more accurate answer though.00:08
smcginnisOr at least more specific.00:08
jgriffithThe key in that etherpad is also "what it's used for" :)00:08
hemnajgriffith, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/hpe/hpe_3par_fc.py#L378-L39300:08
jgriffithhemna and the "what it's used for" part?00:09
smcginnisIt _should_ all be pretty consistent between drivers. It needs to be the info the FC zoning needs.00:09
hemnathe important piece there is returning initiator_target_map00:09
*** yangyapeng has quit IRC00:09
hemnathat piece is returned if and only if there are no more volumes exported from the array to the host00:09
smcginnis{'data'00:09
hemnathe FCZM then tests to see if that's in the return00:09
hemnaif it is, then it removes the zone00:09
hemnaelse the fczm does nothing00:10
smcginnis{'data': {'target_wwn': 'deadbeef', 'initiator_target_map': xxx}}00:10
jgriffithhemna if it's just used by the FZM then WTF do we have to care at the manager?00:10
jgriffithhemna or anywhere outside of that for that matter?00:11
hemnawe don't00:11
hemnathat's what I was trying to say in the meeting00:11
*** catintheroof has joined #openstack-cinder00:11
smcginnisWe could certainly take the code that's building the dict to return into a call explicitly from the driver.00:11
jgriffithhemna ahh00:11
hemnahmm00:12
jgriffithhemna so the only problem is that it's the "opposite" of the way my method works currently?00:12
hemnamaybe os-brick uses the data on disconnect_volume too00:12
smcginnishemna: Huh, I see we're returning Discard in there too. Is that right?00:12
jgriffithin other words, I currently interpret any not None as "outstanding shared-connections"00:12
hemnaah yah00:13
jgriffithwhich this would actually indicate, the opposite; the Zone is done, so the ZM may be able to clean up00:13
* patrickeast catches up00:13
hemnabrick uses target_wwn from the return00:13
patrickeastjgriffith: yea thats correct00:13
jgriffithhemna what does brick do with that?00:13
hemnait's used to discover which paths on the system to remove00:14
*** lixiaoy1 has joined #openstack-cinder00:14
jgriffithsigh00:14
patrickeasthemna: how would brick get the data currently though? we don't return it back up through the manager do we?00:14
jgriffithpatrickeast +100:14
* hemna looks00:14
jgriffithwe do not.. maybe the ZM stuff just doesn't even work anyway :)00:14
patrickeastnah, it works... but afaik it just removes the zoning00:15
patrickeastthen the initiator uses os-brick to get its own wwpn/wwns and calls os-brick00:15
patrickeasti think00:15
jgriffithhemna patrickeast so the idea was to try and make this work without having to go and modify all of the drivers, but it seems like maybe that's not going to work out00:15
hemnaso brick gets that from the bdm I think00:15
jgriffithso maybe what I'll need to do is create a special response and update *EVERYTHING* to use said response00:16
jgriffithhemna that would make sense, and be more in line with how the new code expects this to work00:16
hemnasorry guys, it's just been a while since I've messed with this00:16
jgriffithhemna that's ok, I don't mess with it at all :)00:16
patrickeastjgriffith: you do now :P00:17
jgriffithI just want to make sure that we have a defined interface and expected responses and actually stick with them00:17
hemnahttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L95800:17
jgriffithpatrickeast LOL!!  Damn it!00:17
*** jdurgin has quit IRC00:17
hemnayah, so nova passes that connection_info in to brick from the bdm00:17
patrickeasti'm game for defining a more standard response for the drivers to return from terminate_connection00:17
patrickeastor even a new driver api that just asks given some host info if its safe to disconnect or not00:18
smcginnisWhatever we end up with, let's make sure it's clearly spelled out here: https://github.com/openstack/cinder/blob/master/cinder/interface/volume_driver.py#L20300:18
jgriffithhemna so that's *good* news, because that's how I'm trying to move the new stuff to work, and in cases where that isn't there or doesn't work then use attachment-get00:18
jgriffithsmcginnis noted!00:18
hemnayah so I think the terminate_connection dict passed back is only used by the FCZM at this point00:18
smcginnispatrickeast: That could be racy to make it a separate call.00:18
smcginnispatrickeast: But that could be possible too.00:18
*** jerrygb has quit IRC00:18
hemnaand I think we wanted to nuke the decorator and just explicitly call the add/remove zone in the drivers00:18
patrickeastsmcginnis: meh, the whole thing is racy00:18
jgriffithhemna we should probably make that an inheritiance/wrapper of some sort00:19
*** david-lyle has joined #openstack-cinder00:19
smcginnispatrickeast: Hah, fair point.00:19
hemnajgriffith, heh, I've tried to enforce that a dozen times00:19
hemnavia the base FC volume driver00:19
jgriffithwe don't even have a dozen FC drivers do we :)00:19
hemnaABC crap has greatly complicated that00:19
jgriffithjust kidding00:19
hemnaso I gave up00:19
jgriffithyou can't hit me from here00:19
patrickeastlol00:19
hemnahehe00:19
jgriffithhemna hmmm... ABC, shall I rant :)00:20
jgriffithnahh00:20
hemnakill the alphabet00:20
jgriffithhemna BUT... honestly here's the thing00:20
jgriffithif somebody doesn't inherit like you say it doesn't matter00:20
jgriffithand even that by itself doesn't solve the problem00:20
hemnaif everyone would have written an FC driver using the base FC class, then we could do all of this in the parent base class00:20
hemnableh00:20
jgriffithWhat I'm getting at is just making sure the drivers all behave the same, if that means there needs to be some wrappers and indirection/helper methods that's cool00:21
hemnayah, so I will abdicate that fight at this point00:21
jgriffithhemna ahh, yeah; well that's always the case with stuff like this.  The theory is great, but reality.. not so much00:21
hemnayah I'm all for the same behavior, well at least with respect with what they are supposed to accept and return00:21
jgriffithOO programming in general.. it's. a farce :)00:21
jgriffithhemna the problem is that you may have different views on what those things are :)00:22
jgriffithanyhow00:22
hemnafrankly the fczm usage has been an evolution from the beginning and hadn't needed to change it00:22
*** sdague has quit IRC00:22
smcginnisjgriffith: Well, OO programming in a loosely type language is kind of a farce. It can work very well in other languages. :)00:22
hemnabut I'd much rather make this easier to do00:22
jgriffiththe info on that etherpad will help, I'll try and confirm that the data isn't even actually going back to the caller or being used (or if folks with FC drivers want to check that would be cool)00:23
jgriffithsmcginnis hehe.. well that's a fair point00:23
jgriffithsmcginnis like Java where you have no choice?00:23
smcginnisYou must implement this... unless you don't want to, that's cool too.00:23
jgriffithsmcginnis yeah, that's what's F'd up00:23
hemnathen you stab yourself in the eye with classpath00:24
jgriffithand I'd like to have a long talk with the person(s) that started us on the "everything's a snow flake"00:24
jgriffithoh well, at least we can try and fix this up nicely00:24
smcginnisjgriffith: +100:25
smcginnispatrickeast: This seemed to regulary affect the Pure driver from showing up in the driver list: https://review.openstack.org/42099300:26
smcginnispatrickeast: So you might have some interest in that.00:26
*** mtan_____ has joined #openstack-cinder00:28
smcginnisjgriffith: That would be a good point for that.00:29
patrickeastsmcginnis: interesting, whats the list used for?00:29
smcginnisjgriffith: We get all of that connection info there anyway.00:29
*** mtanino has quit IRC00:29
patrickeasti recall various points in time we talked about using an automated list of drivers for things... but afaik most public ones are manually maintained00:29
smcginnispatrickeast: To get the list of all drivers. Most visibly it's used for the published driver list, but I still am hoping to expand that to do additional checks and things.00:30
smcginnispatrickeast: There's also a patch somewhere to replace our driver capability wiki table with an automatically generated table.00:30
jgriffithsmcginnis hmmm?00:30
hemnaI think it includes the new CI wiki name in the output too00:30
smcginnispatrickeast: http://docs.openstack.org/developer/cinder/drivers.html00:30
smcginnisjgriffith: Was referring to what you were typing in the etherpad.00:31
jgriffithsmcginnis yeah, which part?00:31
hemnayah, that's I think how those links to the CI are that list00:31
smcginnishemna: Yep. And Unsupported now too if applicable.00:31
hemnaooh unsupported too00:31
smcginnisjgriffith: connection_is_shared00:31
hemnaw00t00:31
smcginnisjgriffith: We're querying that info anyway.00:31
jgriffithsmcginnis so I was getting at adding the connection_is_shared key, and if it's not there, or None we interpret it as false00:31
smcginnishemna: Public shaming. :D00:31
jgriffithsmcginnis oh?00:32
*** jdurgin has joined #openstack-cinder00:32
smcginnisjgriffith: That sounds good to me.00:32
hemnais 3PAR in that list yet?00:32
smcginnisjgriffith: If it's not there, then the driver isn't aware of multiattach.00:32
patrickeastsmcginnis: oh interesting, didn't even know that list was there, i always went off of https://www.openstack.org/marketplace/drivers/ https://wiki.openstack.org/wiki/CinderSupportMatrix or  http://docs.openstack.org/newton/config-reference/block-storage/volume-drivers.html00:32
* patrickeast wonders if there are too many different lists00:32
jgriffithsmcginnis right, but there are drivers like Pure that share connections still00:32
hemnaneed a list of lists00:32
smcginnispatrickeast: There are. This is one list to rule them all. ;)00:33
jgriffithbut I don't know how that works today anyway00:33
jgriffith:)00:33
smcginnispatrickeast: Assuming it actually is able to get all the drivers right.00:33
patrickeastsmcginnis: haha yea, that would be ideal00:33
smcginnispatrickeast: If you notice, that also lists all fczm drivers and backup drivers.00:33
*** jerrygb has joined #openstack-cinder00:34
smcginnishemna: While we're on the subject: https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:ci_unsupported00:35
smcginnispatrickeast, hemna: Double whammy.00:36
hemnasmcginnis, so......00:36
hemna3PAR CI hasn't been passing for quite a while.....00:36
hemnaa long while00:36
smcginnishemna: I'll run my report again. It's been a few weeks now and last time I ran it there were a few that were right on the edge.00:37
smcginnisOpenStack Ocata: The Bloodbath Release00:37
smcginnisI hope to have the CI report automatically published to here eventually: http://cinderstats-dellstorage.rhcloud.com/00:37
smcginnisFirst I should figure out why my review could job doesn't appear to be running.00:38
smcginnisCrap, and what happened to my other reports too.00:38
smcginnisHah, guess I've got some work to do there.00:38
*** jose-phillips has quit IRC00:40
hemnasmcginnis, https://goo.gl/4jKMOu00:40
hemnathat's back to october00:40
hemnaunless I ran that incorrectly00:40
smcginnishemna: OK, I'll take a look tomorrow.00:42
hemnawhat do you use to check the CI status?00:43
patrickeastjgriffith: that proposed dict format on https://etherpad.net/p/cinder-fcResponses-terminate_connection looks good to me, seems like it would be simple enough for drivers to do that, and for the manager to make reasonable assumptions if it isn't there00:43
*** david-lyle has quit IRC00:43
smcginnishemna: I've been using a modified version of this: https://github.com/openstack/third-party-ci-tools/tree/master/monitoring/lastcomment-scoreboard00:49
*** paw_ has joined #openstack-cinder00:55
*** paw_ is now known as Guest9873200:55
*** paw has quit IRC00:58
*** lgreg has quit IRC01:01
*** venkhari has joined #openstack-cinder01:06
*** csusuntao has joined #openstack-cinder01:07
*** catintheroof has quit IRC01:09
*** jerrygb has quit IRC01:13
jgriffithpatrickeast cool!  I just need to verify with you on the whole shared connection thing and how that works, but I'll get an update proposed tomorrow and let you have a look at it with the code and let me know if it works out or not01:14
*** wanghao has joined #openstack-cinder01:16
*** yangyapeng has joined #openstack-cinder01:20
*** trananhkma has quit IRC01:24
*** salv-orlando has joined #openstack-cinder01:25
*** csusuntao has quit IRC01:26
*** cknight has joined #openstack-cinder01:31
*** knomura has joined #openstack-cinder01:37
openstackgerritMerged openstack/cinder: Mark the sheepdog driver as unsupported  https://review.openstack.org/41907901:38
*** huanxuan has joined #openstack-cinder01:40
*** knomura has quit IRC01:42
*** mdnadeem has joined #openstack-cinder01:44
*** mdnadeem has quit IRC01:46
*** salv-orlando has quit IRC01:46
*** diablo_rojo has joined #openstack-cinder01:51
*** mars has joined #openstack-cinder01:54
*** mtan_____ has quit IRC01:57
*** bardia has quit IRC01:58
*** nicolasbock has quit IRC02:03
*** venkhari has quit IRC02:12
*** salv-orlando has joined #openstack-cinder02:13
*** jerrygb has joined #openstack-cinder02:14
*** jamielennox is now known as jamielennox|away02:25
*** Apoorva has quit IRC02:28
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted  https://review.openstack.org/22508102:31
*** salv-orlando has quit IRC02:32
*** gcb has joined #openstack-cinder02:36
openstackgerritMerged openstack/cinder: Prevent driver load failures from objects  https://review.openstack.org/42099302:37
openstackgerritwanghao proposed openstack/cinder: Support to query volume filter by change since/before  https://review.openstack.org/35268802:42
*** sudipto_ has joined #openstack-cinder02:52
*** sudipto has joined #openstack-cinder02:52
openstackgerritwanghongtaozz proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42103702:53
*** salv-orlando has joined #openstack-cinder03:00
*** Lee1092 has joined #openstack-cinder03:01
*** jamielennox|away is now known as jamielennox03:03
*** salv-orlando has quit IRC03:05
*** bkopilov has quit IRC03:05
*** bkopilov_ has quit IRC03:06
*** jose-phillips has joined #openstack-cinder03:10
*** jose-phillips has quit IRC03:10
*** gouthamr has quit IRC03:12
*** jose-phillips has joined #openstack-cinder03:13
*** jose-phillips has quit IRC03:14
*** jose-phillips has joined #openstack-cinder03:14
*** jose-phillips has quit IRC03:17
*** jose-phillips has joined #openstack-cinder03:18
*** jose-phillips has quit IRC03:19
*** jose-phillips has joined #openstack-cinder03:20
*** jose-phillips has quit IRC03:21
*** jose-phillips has joined #openstack-cinder03:22
*** cknight has quit IRC03:26
*** jose-phillips has quit IRC03:27
*** links has joined #openstack-cinder03:32
*** Apoorva has joined #openstack-cinder03:34
*** pbandark has joined #openstack-cinder03:43
*** mdnadeem has joined #openstack-cinder03:56
*** s-shiono has joined #openstack-cinder04:02
*** sudipto has quit IRC04:09
*** sudipto has joined #openstack-cinder04:09
*** psachin has joined #openstack-cinder04:09
*** sudipto_ has quit IRC04:10
*** sudipto_ has joined #openstack-cinder04:11
openstackgerritMerged openstack/cinder: RBD: Fix RBD replication on volumes from image source.  https://review.openstack.org/41828804:11
openstackgerritMerged openstack/cinder: Add api version history for 3.23  https://review.openstack.org/41922804:11
*** sheel has joined #openstack-cinder04:13
*** bkopilov has joined #openstack-cinder04:27
*** sudipto has quit IRC04:28
*** sudipto_ has quit IRC04:28
*** bardia has joined #openstack-cinder04:31
*** bkopilov_ has joined #openstack-cinder04:34
*** Apoorva has quit IRC04:38
*** Apoorva has joined #openstack-cinder04:47
*** Apoorva has quit IRC04:49
*** tlian has quit IRC04:50
*** salv-orlando has joined #openstack-cinder05:01
*** salv-orlando has quit IRC05:06
*** jerrygb has quit IRC05:08
*** Poornima_K has joined #openstack-cinder05:11
*** stakeda has joined #openstack-cinder05:11
*** houming has joined #openstack-cinder05:17
openstackgerritMerged openstack/cinder: Dell SC: Missing version history  https://review.openstack.org/41968605:18
*** houming has quit IRC05:20
*** houming has joined #openstack-cinder05:21
*** pgadiya has joined #openstack-cinder05:22
*** pdeore has joined #openstack-cinder05:23
*** sudipto_ has joined #openstack-cinder05:26
*** sudipto has joined #openstack-cinder05:26
*** adisky_ has joined #openstack-cinder05:31
*** david-lyle has joined #openstack-cinder05:33
*** Apoorva has joined #openstack-cinder05:37
openstackgerritwanghongtaozz proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42107905:40
*** david-lyle has quit IRC05:44
*** Poornima_K has quit IRC05:44
*** Jack_I has joined #openstack-cinder05:45
*** Poornima_K has joined #openstack-cinder05:45
*** bardia has quit IRC05:52
*** alonmarx has joined #openstack-cinder05:52
*** david-lyle has joined #openstack-cinder05:53
*** Jack_V has joined #openstack-cinder05:53
*** Jack_I has quit IRC05:57
*** Jack_Iv has joined #openstack-cinder06:00
*** Jack_Iv has quit IRC06:00
*** Jack_Iv has joined #openstack-cinder06:01
*** salv-orlando has joined #openstack-cinder06:02
*** e0ne has quit IRC06:04
*** salv-orlando has quit IRC06:07
*** Jack_Iv has quit IRC06:11
*** nkrinner_afk is now known as nkrinner06:14
*** Jack_Iv has joined #openstack-cinder06:20
*** Apoorva has quit IRC06:22
*** david-lyle has quit IRC06:24
*** Apoorva has joined #openstack-cinder06:26
*** Apoorva has quit IRC06:28
*** arne_r has joined #openstack-cinder06:30
*** sudipto_ has quit IRC06:33
*** sudipto has quit IRC06:33
*** sudipto has joined #openstack-cinder06:33
*** sudipto has quit IRC06:33
*** sudipto has joined #openstack-cinder06:34
*** sudipto_ has joined #openstack-cinder06:34
*** diablo_rojo has quit IRC06:34
*** mjura has joined #openstack-cinder06:38
*** Jack_Iv has quit IRC06:43
openstackgerritMerged openstack/cinder: CoprHD: Handle ScaleIO CI failing with version-3.5  https://review.openstack.org/41702106:47
*** Jack_Iv has joined #openstack-cinder06:51
*** houming has quit IRC06:53
*** Jack_Iv has quit IRC06:55
*** lpetrut has joined #openstack-cinder06:58
*** kongwei has joined #openstack-cinder06:59
*** houming has joined #openstack-cinder07:00
winston-1Hi team, do we have agenda for PTG published somewhere?07:03
*** lixiaoy1 has quit IRC07:04
openstackgerritKazumasa Nomura proposed openstack/cinder: Restrain the unmanage snapshot for the VSP drivers  https://review.openstack.org/42109507:04
*** kongwei has quit IRC07:05
*** houming has quit IRC07:06
*** houming has joined #openstack-cinder07:07
*** tesseract has joined #openstack-cinder07:16
*** lkuchlan_wfh has joined #openstack-cinder07:17
openstackgerritwanghongtaozz proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42110007:18
openstackgerritwanghongtaozz proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42110107:25
*** huanxuan has quit IRC07:29
openstackgerritwanghongtaozz proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42110407:30
*** ChubYann has quit IRC07:32
*** wanghao has quit IRC07:39
*** salv-orlando has joined #openstack-cinder07:40
*** wanghao has joined #openstack-cinder07:40
*** mriedem has quit IRC07:41
*** lpetrut has quit IRC07:46
*** lpetrut has joined #openstack-cinder07:53
*** dmellado has quit IRC08:02
*** geguileo has quit IRC08:03
*** dmellado has joined #openstack-cinder08:04
*** geguileo has joined #openstack-cinder08:05
*** takedakn has joined #openstack-cinder08:07
*** wanghao_ has joined #openstack-cinder08:09
*** salv-orlando has quit IRC08:09
*** wanghao has quit IRC08:12
*** lkuchlan_wfh has quit IRC08:12
*** lkuchlan has joined #openstack-cinder08:12
*** wanghao_ has quit IRC08:25
*** wanghao has joined #openstack-cinder08:25
*** Jack_Iv has joined #openstack-cinder08:45
*** markus_z has joined #openstack-cinder08:45
markus_zNeeds a second +2 https://review.openstack.org/#/c/413684/508:46
*** pgadiya_ has joined #openstack-cinder08:46
*** wanghao has quit IRC08:47
*** wanghao_ has joined #openstack-cinder08:47
*** e0ne has joined #openstack-cinder08:48
*** Jack_I has joined #openstack-cinder08:48
openstackgerritZhangHongtao proposed openstack/os-brick: Fix a wrong indentation  https://review.openstack.org/42113708:49
*** Jack_V has quit IRC08:49
*** pgadiya has quit IRC08:49
*** Jack_V has joined #openstack-cinder08:52
*** Jack_I has quit IRC08:56
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** rha has quit IRC09:03
*** markus_z has quit IRC09:04
*** Poornima_K has quit IRC09:09
*** Lehtu has quit IRC09:17
*** Jack_Iv_ has joined #openstack-cinder09:19
*** markus_z has joined #openstack-cinder09:19
*** alekseyruban has joined #openstack-cinder09:20
*** Jack_Iv has quit IRC09:21
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Implement volume replication for VMAX  https://review.openstack.org/40907909:25
*** Jack_Iv_ has quit IRC09:26
dulekIf anyone want to take a look on simple, non-conflicting patch: https://review.openstack.org/#/c/396264/09:26
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - storage group cleanup on error  https://review.openstack.org/41132909:28
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted  https://review.openstack.org/22508109:29
*** mvk has quit IRC09:30
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - rename and restructure driver  https://review.openstack.org/41192209:33
*** jordanP has joined #openstack-cinder09:38
*** wanghao_ has quit IRC09:41
*** swamireddy has quit IRC09:43
*** lkuchlan has quit IRC09:49
*** lkuchlan has joined #openstack-cinder09:49
*** lkuchlan has quit IRC09:49
*** lkuchlan has joined #openstack-cinder09:50
*** takedakn has quit IRC09:52
*** yumiriam has joined #openstack-cinder09:57
*** swamireddy has joined #openstack-cinder10:01
*** jordanP has quit IRC10:04
*** alyson_ has joined #openstack-cinder10:05
*** mvk has joined #openstack-cinder10:06
*** yangyapeng has quit IRC10:07
*** salv-orlando has joined #openstack-cinder10:10
*** s-shiono has quit IRC10:14
*** salv-orlando has quit IRC10:14
*** jose-phillips has joined #openstack-cinder10:15
*** haplo37_ has quit IRC10:19
*** haplo37_ has joined #openstack-cinder10:19
*** flip214 has quit IRC10:21
*** lixiaoy1 has joined #openstack-cinder10:23
*** flip214 has joined #openstack-cinder10:29
*** flip214 has joined #openstack-cinder10:29
*** zengyingzhe has quit IRC10:30
*** zengyingzhe has joined #openstack-cinder10:30
*** lixiaoy1 has quit IRC10:31
*** mvk has quit IRC10:32
*** tommylikehu has quit IRC10:40
*** mvk has joined #openstack-cinder10:47
*** tommylikehu has joined #openstack-cinder10:48
*** mriedem has joined #openstack-cinder11:00
*** sudipto_ has quit IRC11:02
*** sudipto has quit IRC11:02
*** wanghao has joined #openstack-cinder11:06
*** lpetrut has quit IRC11:06
*** wanghao has quit IRC11:07
*** wanghao has joined #openstack-cinder11:08
*** markus_z has quit IRC11:11
*** alekseyruban has quit IRC11:17
*** Poornima_K has joined #openstack-cinder11:18
*** ociuhandu has quit IRC11:23
*** flip214 has quit IRC11:23
*** jordanP has joined #openstack-cinder11:25
*** flip214 has joined #openstack-cinder11:25
*** flip214 has joined #openstack-cinder11:25
openstackgerritVipin Balachandran proposed openstack/cinder: Add volume to fake snapshot object  https://review.openstack.org/40735711:26
*** rha has joined #openstack-cinder11:27
*** mvk has quit IRC11:28
*** houming has quit IRC11:28
*** ntpttr has quit IRC11:31
*** houming has joined #openstack-cinder11:35
*** ntpttr has joined #openstack-cinder11:39
*** jose-phillips has quit IRC11:39
*** pdeore has quit IRC11:40
*** jose-phillips has joined #openstack-cinder11:40
*** mvk has joined #openstack-cinder11:41
*** JoseMello has joined #openstack-cinder11:43
adrianofrdulek, xyang: Hi. We have a  happy CI now  https://review.openstack.org/#/c/414544/11:50
dulekadrianofr: +W :)11:51
adrianofrdulek: Tks  :)11:52
*** pgadiya_ has quit IRC11:58
*** Poornima_K has quit IRC11:59
*** yangyapeng has joined #openstack-cinder11:59
*** cdelatte has joined #openstack-cinder11:59
*** timcl has joined #openstack-cinder11:59
*** nicolasbock has joined #openstack-cinder12:01
openstackgerritShangXiao proposed openstack/cinder: Modify the spelling mistakes  https://review.openstack.org/42123512:03
*** wanghao_ has joined #openstack-cinder12:09
*** lpetrut has joined #openstack-cinder12:10
*** wanghao has quit IRC12:12
*** links has quit IRC12:12
*** Poornima_K has joined #openstack-cinder12:13
*** jose-phillips has quit IRC12:14
*** markus_z has joined #openstack-cinder12:16
*** bkopilov_ has quit IRC12:22
*** bkopilov has quit IRC12:23
*** tesseract has quit IRC12:23
*** mkoderer has quit IRC12:26
*** catintheroof has joined #openstack-cinder12:32
*** Lee1092 has quit IRC12:36
*** wanghao_ has quit IRC12:39
*** lamt has quit IRC12:41
*** sdague has joined #openstack-cinder12:43
*** Jack_I has joined #openstack-cinder12:46
smcginniswinston-1: We just have this so far: https://etherpad.openstack.org/p/ATL-cinder-ptg-planning12:47
smcginniswinston-1: Are you able to attend?12:47
*** Jack_V has quit IRC12:48
*** mriedem has quit IRC12:48
*** houming has quit IRC12:48
*** gouthamr has joined #openstack-cinder12:49
*** Poornima_K has quit IRC12:49
openstackgerritMerged openstack/cinder: HNAS: Change snapshot names  https://review.openstack.org/41454412:51
*** gcb has quit IRC12:58
*** links has joined #openstack-cinder12:58
*** jwcroppe has joined #openstack-cinder12:59
*** links has quit IRC13:01
*** mars has quit IRC13:02
*** dave-mccowan has joined #openstack-cinder13:02
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786213:03
*** dustins has joined #openstack-cinder13:09
openstackgerritPavel Gluschak proposed openstack/cinder: vzstorage: use resize instead of grow for ploop images  https://review.openstack.org/40023513:10
*** eharney has joined #openstack-cinder13:10
*** lamt has joined #openstack-cinder13:11
*** crose has joined #openstack-cinder13:23
*** cristicalin has joined #openstack-cinder13:25
*** pdeore has joined #openstack-cinder13:39
*** Jack_Iv has joined #openstack-cinder13:40
*** sudipto has joined #openstack-cinder13:45
*** sudipto_ has joined #openstack-cinder13:45
*** markvoelker has joined #openstack-cinder13:45
*** Poornima has joined #openstack-cinder13:56
*** tommylikehu_ has joined #openstack-cinder13:58
*** mdnadeem has quit IRC14:01
*** pdeore has quit IRC14:01
*** huanxuan has joined #openstack-cinder14:05
*** asettle has joined #openstack-cinder14:06
asettleHey - wondering if someone is able/around to help me triage a bug. We've been getting a lot of bugs for manuals content with configuration content that I don't know much about.14:07
asettlehttps://bugs.launchpad.net/openstack-manuals/+bug/164456214:07
openstackLaunchpad bug 1644562 in openstack-manuals "Install and configure a storage node in Installation Guide" [Undecided,New]14:07
asettleUsing the Ubuntu install guide, the reporter was unable to create a volume. cinder-volume does not start.14:07
asettleNewton install guide.14:07
*** cristicalin has quit IRC14:08
*** mriedem has joined #openstack-cinder14:10
*** salv-orlando has joined #openstack-cinder14:10
*** liverpooler has quit IRC14:13
*** jerrygb has joined #openstack-cinder14:14
smcginnisasettle: I just did an install from that guide not long ago. Some dependency must have changed.14:14
smcginnisasettle: Appears to be from this: Unrecoverable Error: Versioned Objects in DB are capped to unknown version 1.11.14:15
smcginnisdulek: You still around? ^^14:15
asettleA successful install from that guide?14:15
smcginnisasettle: The newton Ubuntu guide on docs.o.o, right?14:15
asettleThat's the one, yep smcginnis14:16
*** ociuhandu has joined #openstack-cinder14:16
smcginnisasettle: Yeah, maybe about a month ago now, but recent enough that I know it was at least working as described recently. :)14:17
asettleOh good :) (thank god)14:17
smcginnisasettle: ;)14:18
asettleOkay, then I have absolutely no idea how this reporter ended up at this point tbh :/14:18
smcginnisLooks like oslo.versionedobjects was probable updated since I did it. 1.19.0 went out Dec 1 and 1.20.0 went out Dec 6.14:18
asettleThe reporter needs to update?14:18
smcginnisMy suspicion is either that or some other library has made a change that broke something.14:18
smcginnisasettle: Or we are not capping something that we need to.14:19
smcginnisasettle: Either that or they missed something in the setup, but my first assumption is that we broke something, not them.14:19
asettleThat's a nice assumption :P i'm a lot more ruthless than that.14:19
asettleHm, okay, so - anyway to quickly verify this? Sorry for leaning on you a lot here, I've decided to bug triage my little heart out but it gets harder the more configuration info you need to know across *all* projects.14:20
smcginnisasettle: I can only imagine!14:20
asettleHahahha *sigh* *head desk*14:21
smcginnisasettle: Well, I have a basic understanding of the versioned objects stuff, but dulek is our resident expert on all of that.14:21
smcginnisasettle: He's usually around, so he'll probably be back soon. I'm sure it will be obvious to him. ;)14:21
asettleIt's always obvious to someone :P14:21
asettleThe bug and I can wait :) thanks for having a look though smcginnis !14:22
smcginnisasettle: That's the truth.14:22
asettledulek: shoot me a ping when you're in please :)14:22
smcginnisasettle: No problem. I'll keep it open in a tab so I remember to follow up if we don't hear anything.14:22
*** tlian has joined #openstack-cinder14:23
asettle:) wonderful, thanks! Me too - I'll stare at it and hope that via osmosis cinder configuration contextual information will just absorb.14:23
asettleSomething tells me that won't work, but hey, here's trying.14:23
smcginnisasettle: Hah, good luck with that. :P14:23
asettleThank you smcginnis - I very well might need it.14:23
dulekI'm on a meeting for 25 minutes and everything collapses.14:26
*** ducttape_ has quit IRC14:26
dulekasettle: Let me read the scrollback first. :)14:26
asettledulek: it's a rollercoaster from start to finish14:26
dulekasettle: http://docs.openstack.org/developer/cinder/upgrade.html#rpc-version-pinning14:28
dulekasettle: If that's not clear enough I'll be happy to update it with your feedback.14:28
asettleAnd like magic dulek knows the things. dulek - could you explain to me what exactly you're pointing to? I know as much about cinder as I do about... Icelandic politics.14:29
*** pbandark has quit IRC14:29
dulekasettle: Basically this error happens if there are old, dead records in services, that reported older versions of either RPC APIs or objects.14:29
dulekasettle: Or - the code isn't checked out from exactly the same git ref and versions differ.14:30
asettledulek: oh! Thank you!14:31
asettleThat makes sense :)14:31
dulekasettle: There's one more mention in the 5th point here: http://docs.openstack.org/developer/cinder/upgrade.html#plan-your-upgrade14:31
dulekcinder-manage service remove is helpful if orphaned records are the cause.14:32
*** bkopilov_ has joined #openstack-cinder14:32
asettleOkay! Makes sense :)14:32
*** bkopilov has joined #openstack-cinder14:33
openstackgerritAlon Marx proposed openstack/cinder: Open the source code of ibm_storage driver  https://review.openstack.org/40495814:35
*** knomura has joined #openstack-cinder14:35
asettleThanks dulek and smcginnis :) I've updated the bug with your comments.14:35
asettleThese kind of bugs do my head in.14:36
*** Jack_Iv has quit IRC14:36
*** gcb has joined #openstack-cinder14:36
dulekasettle: Awesome. I'll think about providing better feedback for the admin when this occurs.14:36
smcginnisasettle: It sounded like they were doing a new install, but maybe it was an upgrade. Or like dulek mentioned, maybe different revs of the code.14:36
*** psachin has quit IRC14:37
duleksmcginnis: Or they've reused old database.14:37
smcginnisdulek: Yeah... whatever is going on, it doesn't appear to be a straight install through the install guide.14:38
asettle*Everyone* reports they are doing a straight install from the guide.14:38
asettleSome are easier to identify than others.14:38
smcginnisasettle: "Honestly, I didn't touch anything" (whole config is changed) :)14:39
asettleUgh. Yes. I had one bug report that was basically like "I've done *everything* you said in the guide, but here's my optional configurations, WHY DONT YOU DOCUMENT THIS"14:39
asettleIt was a very angry bug report.14:39
openstackgerritRohan Arora proposed openstack/cinder: Nosec subprocess with shell equal True from Bandit  https://review.openstack.org/42097914:40
smcginnisasettle: Oh, I love those.14:40
*** porrua has joined #openstack-cinder14:40
asettle*so* good. Also I guess the one big problem about getting some bugs in manuals is that we get a lot of troubleshooting questions.14:40
asettleI have typed "Please ask your questions in #openstack-PROJECT" more times than I can count.14:41
smcginnisasettle: I'm sure!14:41
*** cristicalin has joined #openstack-cinder14:41
asettleAnyway, cheers guys :) appreciate it!14:41
smcginnisasettle: For our storage product it was a joke for a while. *Every* bug was filed against our UI team, because obviously it was a UI problem that an error was displayed.14:41
asettleHAHAHA14:42
asettleOh dear.14:42
asettleI'm trying to think of ways to reduce some of this bug count and make it easier for everyone to triage. So far I've come up with 'kill it with fire'.14:42
smcginnisasettle: Well, any other Cinder issues come up, feel free to point them our way.14:43
smcginnisasettle: Hah!14:43
asettlesmcginnis: will you regret that promise when I suddenly start spending an inordinate amount of time here? :p14:43
asettleI'll be that docs weirdo in the corner.14:44
smcginnisasettle: haha, I might regret it a little, but I think our docs are very important, so I'll gladly put up with it. ;)14:46
asettle\o/ can I quote you on that?14:46
asettle"Docs are very important" smcginnis , 201714:46
smcginnisasettle: :)14:46
*** cristicalin has quit IRC14:47
asettleI did remove the 'our' to suit my needs.14:47
*** dustins has quit IRC14:47
smcginnisasettle: Still a fair statement. :)14:48
*** salv-orlando has quit IRC14:50
*** Guest98732 is now known as paw14:50
*** paw has quit IRC14:50
*** paw has joined #openstack-cinder14:51
*** lamt has quit IRC14:52
openstackgerritManish proposed openstack/cinder: 3PAR: Incosistency in copied and source CG  https://review.openstack.org/41933514:54
*** lamt has joined #openstack-cinder14:58
asettlesmcginnis: haha I hope so :)14:58
*** ducttape_ has joined #openstack-cinder14:59
*** alonmarx_ has joined #openstack-cinder15:02
*** swamireddy has quit IRC15:02
*** Julien-zte has joined #openstack-cinder15:03
*** rajinir has joined #openstack-cinder15:03
*** JoseMello has quit IRC15:05
*** cknight has joined #openstack-cinder15:05
*** alonmarx has quit IRC15:05
*** Poornima has quit IRC15:06
*** huanxuan has quit IRC15:06
*** swamireddy has joined #openstack-cinder15:12
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Add CG capability to groups  https://review.openstack.org/40817615:12
*** pcaruana has quit IRC15:15
*** nkrinner is now known as nkrinner_afk15:17
*** david-lyle has joined #openstack-cinder15:24
*** gcb has quit IRC15:26
*** xyang has joined #openstack-cinder15:27
*** david-lyle has quit IRC15:29
*** david-lyle has joined #openstack-cinder15:29
*** akerr has joined #openstack-cinder15:32
*** david-lyle has quit IRC15:34
*** xyang has quit IRC15:41
*** david-lyle has joined #openstack-cinder15:42
scottdaildikov: Ping15:45
*** dustins has joined #openstack-cinder15:45
ildikovscottda: pong15:45
scottdaJust checking in...it looks like that nova/volume/cinder.py patch solved the v3 endpoint issue?15:46
ildikovit works in my env15:46
ildikovI added volumev3 to the context file in Nova as it's in your patch and I don't need the other hacks now15:47
ildikovI hardcoded the microversion now for local testing15:47
ildikovmriedem added a short note to one of the patches about negotiating about the microversion rather than make it a config option15:48
scottdayeah, I saw that. I can add a placeholder microversion in that patch until we land the cinder stuff and know exactly what that will be...15:50
*** mvk has quit IRC15:51
scottdaI figure this nova patch is just for testing ATM. The main focus is to land the Cinder changes.15:51
openstackgerritRohan Arora proposed openstack/cinder: Use secure random function for AES encryption  https://review.openstack.org/41087415:51
*** xyang has joined #openstack-cinder15:55
*** david-lyle has quit IRC15:55
*** kfarr has joined #openstack-cinder15:56
*** erhudy has joined #openstack-cinder15:58
*** _ducttape_ has joined #openstack-cinder15:59
scottdasmcginnis: Do we(you) want to put something on the meeting agenda regarding our idea to sprint on HA/AA and new Cinder-nova API stuff WEd and thurs?16:00
scottda#startmeeting cinder_testing16:01
openstackMeeting started Tue Jan 17 16:01:49 2017 UTC and is due to finish in 60 minutes.  The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
openstackThe meeting name has been set to 'cinder_testing'16:01
erlonhey16:02
scottdaping eharney, xyang1, gouthamr, akerr, smcginnis, cFouts, e0ne, geguileo, dulek, flip214, ntpttr patrickeast, _alastor_, DuncanT, erlon, josemello,mdarnell, karthikp16:02
e0nehi16:02
scottdahi erlon . Now that you are hear, we know we'll have something to talk about :)16:02
xyanghi16:02
scottda#link https://etherpad.openstack.org/p/cinder-ocata-testing16:02
*** akerr has quit IRC16:02
scottdaI pointed this out last week ^^^16:02
erlonscottda: haha,16:03
*** ducttape_ has quit IRC16:03
scottdaSome tracking so that we can make sure things get *some* coverage before the release16:03
erlonscottda: nothing much new, Im returning this week from my pto16:03
scottdaBut I don't see much action on that...I'm going to put that link on the main Cinder meeting agenda16:03
erlonscottda: I notice that etherpad is very empty, may be folks should update it with details16:04
scottdayup16:04
*** tommylikehu_ has quit IRC16:04
erlonscottda: I added some bullets but still need to track the migration related bugs and try to sort then out16:05
*** ociuhandu has quit IRC16:05
scottdaOK, I added a reminder in the main Cinder agenda16:06
erlonscottda: nice16:07
*** Jack_Iv has joined #openstack-cinder16:07
scottdaI also have added something to the PTG etherpad regarding a single Testing priority for Pike release...16:07
*** sheel has quit IRC16:07
erlonscottda: people should really give some attention to testing. The AA patches are landing and they touch a lot of code16:07
*** david-lyle has joined #openstack-cinder16:07
hemnawhat's up16:08
scottdaI think if we don't have a single focus, we'll end up with the usual list of everything wrong we'd like to fix.16:08
scottdahemna: Hi.16:08
scottdaAnyone have anything about tests they'd like to talk about?16:09
erlonscottda: whast do you mean by single focus?16:09
scottdaerlon: Well, if we come up with the 10 areas we need to write tempest/functional tests for, none of them will get done....16:10
scottdaerlon: If we pick one, there is hope that we can focus and perhaps make progress.16:10
scottdaThen, we could add more if there is time.16:10
geguileoerlon: I have a fix for the manage issue16:10
scottdasmcginnis: had suggested this, and I agree.16:10
geguileoerlon: I'm going to look into the retype with migration now16:10
erlonscottda: hmm, I see16:11
scottdaBut, of course, people can work on whatever they want (or whatever their employer lets them do)16:11
openstackgerritMerged openstack/os-brick: Fix a wrong indentation  https://review.openstack.org/42113716:11
erlongeguileo: I remember I have some patches but didnt submit16:11
erlongeguileo: ill put a patch and you can give a look if theres something missing16:11
scottdageguileo: I had left some work undone on retype with migration...we hope we can end up calling Nova swap volume, to test that as well.16:11
geguileoerlon: lol, then our patches will be fighting one another16:11
erlongeguileo: haha16:12
scottdageguileo: erlon I think that using LVM will cause a Nova swap_volume if the hostname for both backends is different. Otherwise, it won't call swap_volume.16:12
geguileoscottda: I have just fixed the manage one, I have to look at the other case, as it may be related16:13
*** bswartz has quit IRC16:13
erlonscottda: I dont reacall now, but I believe swap_volume is always being called for the attached migration16:14
scottdaOK, cool. I think there is a local nova test for swap_volume, but nothing that exercises it from Cinder...16:14
scottdaand therefore not testing of anything that has Cinder call Nova, and using the novaclient, etc.16:15
markus_zscottda: Would you maybe have a look at https://review.openstack.org/#/c/413684/ ? Needs a 2nd +2.16:15
scottdaerlon: No, I don't think it will with a single-node LVM setup, since the hostname is the smae for both backends.16:15
scottdas/smae/same16:15
scottdaerlon: I thought I had a patch that worked, but there is some logic to optimize things for the case of 2 backends on the same host...I'll try to find it...16:16
scottdamarkus_z: I'll put it in my queue to have a look16:16
markus_zscottda: Thanks, that's all I can ask for :)16:16
*** breitz has joined #openstack-cinder16:17
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix sched/manager calls to filters  https://review.openstack.org/42134816:18
erlongeguileo: ^ first :)16:19
scottda#link https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L229316:19
geguileoerlon: lol, I still have to add the tests16:19
geguileoerlon: And see if it helps with the other bug16:19
scottdaerlon: That is it, I think16:19
*** diablo_rojo has joined #openstack-cinder16:19
erlonscottda: yep, theres this case, I found it because I was testing migration between pools in the same backend16:21
scottdaerlon: So, does your test patch get around this? If not, we'll go with geguileo16:21
scottda:)16:21
erlonscottda: which is the must be the same behavior you mentioned in LVM16:21
erlonscottda: yes16:21
erlonscottda: ow wait16:22
scottdaerlon: I think we really need to check the Nova logs to verify. Otherwise, we'll need an additional test. And this already takes a bit of time to run.16:22
geguileoerlon: your patch is incomplete16:23
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix retype with migration false positive success  https://review.openstack.org/42135316:23
*** mjura has quit IRC16:23
erlongeguileo: yeap, that another error16:23
erlonI split in those 216:23
scottdaerlon: You don't get to be first to post a patch if it's not complete :)16:23
erlongeguileo: scottda: the last one is handling the hostname16:24
erlonscottda: haha damn16:24
erlonscottda: geguileo: they where hanging here, fell free to -2 if you have another that fix16:25
geguileoerlon: I'll look now at reproducing the second issue and do the RCA16:26
*** lkuchlan has quit IRC16:27
erlongeguileo: ok, leme know if you have problem to reproduce it16:27
openstackgerritint32bit proposed openstack/python-cinderclient: Add snapshot-id filter to get volumes via cli  https://review.openstack.org/41651016:27
geguileoerlon: why do we need to say that None is not equal to None for host equivalency?16:27
erlongeguileo: I don't think we need to. Actually they should not be considered equal16:29
erlongeguileo: if clustering is not configured, both are None, but the hosts are actually different16:30
geguileoerlon: Why should they not be considered equal?16:30
geguileoerlon: but that should be ok16:31
geguileoerlon: clusters are equal but not the hosts16:31
erlonthe false positive happens here:https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L229516:31
*** bswartz has joined #openstack-cinder16:31
erlongeguileo: yeah but if does not have cluster_name set, it will lead to a false positive as will compair None == None16:33
geguileoerlon: I'll check it, because then the problem is somewhere else, not in that method16:33
*** dustins has quit IRC16:34
*** dustins_ has joined #openstack-cinder16:34
erlongeguileo: I was trying to understand if the cluster_name that should always have the right values or if the comparation should consider that they could be None, but didn't get to a conclusion, that's why I pinged you the last time16:35
*** arne_r has quit IRC16:36
*** Julien-zte has quit IRC16:36
*** e0ne has quit IRC16:37
*** Julien-zte has joined #openstack-cinder16:37
scottdaOk, well you two can battle it out for which patch we go with....16:37
*** Julien-zte has quit IRC16:37
scottdaBut please check the Nova swap_volume call.16:37
*** Julien-zte has joined #openstack-cinder16:38
scottdaAnything else about tests today?16:38
*** Julien-zte has quit IRC16:38
erlonscottda: hmm, I found some tests in tempest that worth to look on16:39
erlonscottda: https://review.openstack.org/#/q/project:openstack/tempest+(message:volume+OR+message:cinder)+status:open+-status:merged16:39
*** Julien-zte has joined #openstack-cinder16:39
*** Julien-zte has quit IRC16:40
erlonscottda: it seems a lot of people now and in the past have worked in some testing and just left the work16:40
*** Julien-zte has joined #openstack-cinder16:40
erlonscottda: so, some of the might be interesting to re-pick16:40
scottdaerlon: Yes. Thanks for cleaning all of those up.16:40
scottda:)16:40
*** Julien-zte has quit IRC16:40
*** Julien-zte has joined #openstack-cinder16:41
erlonscottda: haha, welcome, in advance :)16:41
erlonscottda: ill have a look16:41
*** Julien-zte has quit IRC16:41
scottdaI'm kidding, of course. But let me know if you are re-activating work on any, and I'll review.16:41
*** Julien-zte has joined #openstack-cinder16:42
*** Julien-zte has quit IRC16:42
* scottda notes his patch is at the top of the list16:42
erlonscottda: ill have a look in the list and ping you about the best ones we and do what we can afford to16:43
erlonscottda: welcome again :)16:43
openstackgerritint32bit proposed openstack/python-cinderclient: Add snapshot-id filter to get volumes via cli  https://review.openstack.org/41651016:43
erlonscottda: another point I was wondering weather we should pick some of that and move to our intree tests16:44
scottdaerlon: That could be a good idea (moving in-tree), as that may be why some languished in the review queue.16:45
smcginnis+116:46
erlonscottda: mhm, the bad side is that we loose the eyes of tempest core, but that's is hard to get anyways16:46
scottdaYes, the "hard to get" part is what I'm talking about16:47
smcginniserlon: I'm sure if we merge something especially bad, we'll get their attention. ;)16:47
erlonscottda: yeap16:47
smcginnisscottda: Right, it's lack of attention I think that has prevented us from adding some of these tests.16:47
erlonsmcginnis: haha, yeah16:47
scottdaOr it they merge something especially bad...16:47
smcginnisscottda: ;)16:48
scottdaOK, well thanks erlon . Keep us informed as to what you identify (if still in Tempest).16:48
scottdaI reckon we'll pay attention no matter what if you move them to cinder in-tree16:48
scottdaOK, anything else?16:49
xyangso regarding tempest tests for groups, should I submit them in-tree or in tempest?16:50
*** salv-orlando has joined #openstack-cinder16:51
scottdaxyang: Good question.16:51
smcginnisxyang: Hmm, maybe try in tempest, and if there is resistance there move to in-tree?16:51
xyangsure16:51
scottda+116:51
erlonscottda: sure, welcome16:51
smcginnisWe've talked about using our in-tree as kind of an incubator for tempest tests, but I think that one might be good to just go there.16:52
scottdayup16:52
*** dustins_ has quit IRC16:52
*** dustins has joined #openstack-cinder16:52
xyang@smcginnis: this one works for LVM, so should be okay I hope16:52
erlonxyang: if you are getting good feedback, and they are ready to get merged Id recommend to leave there, otherewise move then to in-tree16:52
xyangerlon: ok16:53
*** salv-orlando has quit IRC16:54
scottdaOK, if there's anything else, chime in. Otherwise, thanks everyone16:55
*** salv-orlando has joined #openstack-cinder16:55
smcginnisNothing here. Thanks scottda.16:55
scottda#endmeeting16:55
openstackMeeting ended Tue Jan 17 16:55:44 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:55
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_testing/2017/cinder_testing.2017-01-17-16.01.html16:55
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_testing/2017/cinder_testing.2017-01-17-16.01.txt16:55
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_testing/2017/cinder_testing.2017-01-17-16.01.log.html16:55
scottdasmcginnis: Did you see my question ^^ about putting the HA/AA and cinder-nova API test&review sprint on the meeting agenda?16:56
scottdasmcginnis:  I could do it, but, of course, you have more Gravitas.16:56
smcginnisscottda: Hah!16:56
smcginnisscottda: Yeah, sorry, had to bring my daughter to the ortho and just got back half way through the meeting. I did intent to bring that up.16:56
scottdacool16:57
*** stvnoyes has joined #openstack-cinder16:57
scottdaHope you daughter is ok.16:57
xyangdulek: ping16:58
smcginnisscottda: Yep, just normal check up.16:58
*** _ducttape_ has quit IRC16:59
scottdaok. So maybe orthodontist, not orthopedist   :)16:59
*** ducttape_ has joined #openstack-cinder16:59
smcginnisHah, right, orthodontist. Just the usual braces right of passage.17:00
*** markus_z has quit IRC17:00
*** _ducttape_ has joined #openstack-cinder17:01
*** oomichi has joined #openstack-cinder17:03
*** ducttape_ has quit IRC17:04
*** Apoorva has joined #openstack-cinder17:06
*** raunak has joined #openstack-cinder17:15
*** mvk has joined #openstack-cinder17:17
*** yumiriam has quit IRC17:18
*** yumiriam has joined #openstack-cinder17:18
*** sudipto has quit IRC17:24
*** sudipto_ has quit IRC17:24
*** knomura has quit IRC17:25
*** knomura has joined #openstack-cinder17:25
*** david-lyle_ has joined #openstack-cinder17:26
*** knomura has quit IRC17:30
*** porrua has quit IRC17:30
smcginnishemna: http://paste.openstack.org/show/595249/17:32
*** david-lyle_ has quit IRC17:34
*** lpetrut has quit IRC17:34
*** Jack_Iv has quit IRC17:35
*** bardia has joined #openstack-cinder17:35
*** ociuhandu has joined #openstack-cinder17:37
patrickeastsmcginnis: haha, that snapshot manage issue is like really having a hard time getting traction17:42
smcginnispatrickeast: Kind of mind boggling. :)17:43
SwansonThe simple fix for everything being hosed that got kiboshed for no adequately explored reason?17:44
smcginnisYep17:44
patrickeastthats the one!17:45
*** knomura has joined #openstack-cinder17:45
patrickeastmaybe i'll go stir up trouble in the -qa channel...17:47
patrickeastsmcginnis: are you in that one?17:47
smcginnispatrickeast: Hm, no, not currently.17:48
smcginnispatrickeast: I think this one I'm fine "delegating" for now. :D17:48
SwansonFight! Fight! Fight!17:49
patrickeastlol17:49
*** ociuhandu has quit IRC17:49
*** ntpttr has quit IRC17:50
hemnasmcginnis, where is hp in that ?17:50
patrickeastsmcginnis: i was going to add it to an agenda item on the qa-meeting but i have a conflict for the one this week :(17:51
smcginnishemna: Line 76?17:51
hemnaoh ok there it is17:51
hemna22%17:51
hemnaouch17:51
smcginnishemna: Sadly, that's not one of the worse ones.17:52
hemnadon't we have to be 50% + 1 success rate?17:52
smcginnishemna: For now I've just been looking at the last successful run and making sure it wasn't the only successful run.17:52
*** ntpttr has joined #openstack-cinder17:52
hemnathat's probably a low bar to set, but reasonable.  all things considered at this point.17:53
hemnaTintry 5%17:53
smcginnishemna: Yeah, at this point.17:53
smcginnishemna: Hopefully we can tighten that up eventually.17:53
smcginnishemna: That one was failing for a very long time, so in this case it's maybe OK since it just got back to stable.17:54
Apoorvahemna smcginnis hopefully that % will go up for Tintri. Working on making it stable.17:54
smcginnisApoorva: +1!17:54
hemnaawesome17:54
openstackgerritSean McGinnis proposed openstack/cinder: Revert "Mark Tintri volume driver as unsupported"  https://review.openstack.org/42140717:59
smcginnisApoorva: Last few comments look good. Seems stable enough and you're involved, so I'm fine reverting that one already.17:59
*** timcl has quit IRC18:00
Apoorvasmcginnis thanks!18:00
* hemna waits for CI 18:00
*** david-lyle_ has joined #openstack-cinder18:00
*** porrua has joined #openstack-cinder18:00
*** timcl has joined #openstack-cinder18:01
*** lpetrut has joined #openstack-cinder18:02
openstackgerritSzymon Wr√≥blewski proposed openstack/cinder: Refactor quota modules  https://review.openstack.org/41725718:05
*** knomura has quit IRC18:06
*** knomura has joined #openstack-cinder18:06
*** knomura has quit IRC18:11
*** ociuhandu has joined #openstack-cinder18:11
*** Jack_I has quit IRC18:11
*** ociuhandu has quit IRC18:15
*** Jack_I has joined #openstack-cinder18:17
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Storage assisted volume migration.  https://review.openstack.org/39786218:18
*** xinli has joined #openstack-cinder18:22
*** kambiz has quit IRC18:28
*** adisky_ has quit IRC18:29
*** xyang has quit IRC18:31
*** chris_morrell has joined #openstack-cinder18:34
*** chris_morrell has quit IRC18:35
openstackgerritGorka Eguileor proposed openstack/cinder: Fix volume manage  https://review.openstack.org/42143818:39
openstackgerritGorka Eguileor proposed openstack/cinder: Fix retype with migrate  https://review.openstack.org/42143918:40
geguileoerlon: scottda ^18:40
scottdaCool, I'll look on a bit18:42
*** david-lyle has quit IRC18:47
*** bardia has quit IRC18:49
*** markvoelker has quit IRC18:49
*** david-lyle_ has quit IRC18:50
*** Jack_Iv has joined #openstack-cinder18:57
*** porrua has quit IRC18:57
*** porrua has joined #openstack-cinder18:58
*** Jack_I has quit IRC18:59
*** yumiriam has quit IRC19:00
*** Jack_I has joined #openstack-cinder19:00
*** Jack_Iv has quit IRC19:01
*** xyang has joined #openstack-cinder19:01
openstackgerritAlon Marx proposed openstack/cinder: Open the source code of ibm_storage driver  https://review.openstack.org/40495819:03
*** Jack_V has joined #openstack-cinder19:05
bswartzsmcginnis: would appreciate -1 from you here: https://review.openstack.org/#/c/36974919:06
*** Jack_I has quit IRC19:06
smcginnisbswartz: Gladly19:06
*** Jack_I has joined #openstack-cinder19:09
*** Jack_Iv has joined #openstack-cinder19:10
*** Jack_V has quit IRC19:11
*** Jack_I has quit IRC19:11
*** Jack_I has joined #openstack-cinder19:12
*** Jack_Iv has quit IRC19:13
*** jose-phillips has joined #openstack-cinder19:13
*** Jack_V has joined #openstack-cinder19:14
*** Jack_I has quit IRC19:17
*** lamt has quit IRC19:27
*** lamt has joined #openstack-cinder19:28
*** lamt has quit IRC19:28
*** ChubYann has joined #openstack-cinder19:37
*** tommylikehu has quit IRC19:40
*** jordanP has quit IRC19:49
*** david-lyle has joined #openstack-cinder19:50
*** bardia_ has joined #openstack-cinder19:51
*** timcl has quit IRC19:55
*** Jack_V has quit IRC19:55
bswartzsmcginnis: ty20:08
bswartzsmcginnis: they're covering that one in the TC meeting now20:08
smcginnisbswartz: Yep, watching.20:09
*** e0ne has joined #openstack-cinder20:10
*** salv-orlando has quit IRC20:11
*** timcl has joined #openstack-cinder20:13
*** xyang has quit IRC20:18
*** xyang has joined #openstack-cinder20:20
*** mtanino has joined #openstack-cinder20:28
*** lpetrut has quit IRC20:47
*** lpetrut has joined #openstack-cinder20:49
SwansonWhat if the Cinder Back Neds are LVM? What then?20:50
smcginnisSwanson: I read that a few times and still have no idea what you're saying.20:52
SwansonI smell toast!20:52
bswartzs/Neds/Ends/20:52
* smcginnis slowly walks away20:53
smcginnisbswartz: You're trying to apply logic to something Swanson said. Down that path lies madness, trust me. :)20:53
ildikovsmcginnis: LOL20:54
bswartzI see20:54
SwansonTo illustrate smcginnis point, I did mean Neds.20:54
smcginnisbswartz: See?20:54
bswartzis it possible that Swanson is just an irc bot?20:54
smcginnisbswartz: If I didn't know him, I would think so too.20:55
smcginnisSome weird cross between a chaos monkey and an IRC bot.20:55
SwansonMy other job is as a twitter bot.20:55
bswartzalexa with fuzz testing turned on?20:55
smcginnis:)20:55
smcginnisSwanson: Ghost writer for Trumps account? :D20:56
SwansonBOTS GET SUCH ABSUE FROM THE MAINSTREAM MEDIA. ITCHY!20:57
smcginnisNailed it.20:57
*** porrua has quit IRC21:00
*** ducttape_ has joined #openstack-cinder21:01
*** jdurgin has quit IRC21:03
*** _ducttape_ has quit IRC21:04
*** alyson_ has quit IRC21:07
*** dave-mccowan has quit IRC21:09
hemnaasking for eyes...... https://review.openstack.org/#/c/404958/21:13
*** lpetrut has quit IRC21:13
* smcginnis looks for hemna's picture of eye balls...21:15
hemnahttps://goo.gl/ejiAeD21:15
smcginnislol21:17
*** ducttape_ has quit IRC21:17
*** cdelatte has quit IRC21:17
*** ducttape_ has joined #openstack-cinder21:17
*** xyang has quit IRC21:18
*** xyang has joined #openstack-cinder21:20
*** lamt has joined #openstack-cinder21:21
*** jdurgin has joined #openstack-cinder21:22
*** adrianofr has quit IRC21:29
*** xinli has quit IRC21:31
* patrickeast is confused21:32
patrickeastanyone know how our objects are lazy loading things that might be null?21:33
patrickeastlooking at this method https://github.com/openstack/cinder/blob/master/cinder/objects/snapshot.py#L21921:33
smcginnispatrickeast: It's blowing up there?21:34
patrickeastif i try and look at the snapshot["group_snapshot"]21:34
patrickeastthe db call to look it up by id is coming back with an exception, because it isn't in a group21:34
patrickeastwhich seems right21:34
patrickeastbut the object call then raises the exception instead of just returning null21:34
patrickeastor None, or whatever21:34
patrickeastwe even specify it can be None https://github.com/openstack/cinder/blob/master/cinder/objects/snapshot.py#L7021:35
smcginnisHmm, not sure on that.21:35
*** xyang has quit IRC21:36
patrickeastoh hmm it blows up for "cgsnapshot" too21:37
patrickeasti wonder if its supposed to actually support this or not21:37
*** xyang has joined #openstack-cinder21:37
smcginnisI think that nullable=True has more to do with the database than the objects.21:38
patrickeastyea21:38
*** xyang has quit IRC21:38
*** xyang has joined #openstack-cinder21:39
smcginnisBut it does seem if it's nullable, we should just be able to get back null and not blow up.21:39
patrickeastit kinda looks like those calls should be wrapped with try/catch blocks and just set the value to None21:39
smcginnispatrickeast: You may be right.21:39
patrickeastmaybe i'll put a patch together and let dulek and geguileo look at it when they are up and around21:39
patrickeastfix it for now and figure out if its the "right" way later21:40
*** xyang has quit IRC21:40
*** crose has quit IRC21:41
smcginnispatrickeast: That sounds like a decent plan. It would be good to get their take on it.21:41
patrickeastsmcginnis: oo yea, we do the same kind of thing on the volume object with the glance meta data21:45
patrickeastneato21:46
patrickeasteasy fix then21:46
smcginnispatrickeast: So we have to do that for each object?21:49
*** timcl has quit IRC21:49
*** e0ne has quit IRC21:49
patrickeastsmcginnis: yea, but i think its the minority of attributes we have to deal with like this21:49
patrickeastsmcginnis: just the ones that are links to other objects and are OK to be null21:49
smcginnispatrickeast: Hmm, I guess. Just another little detail to have to be aware of and look out for though.21:50
*** timcl has joined #openstack-cinder21:50
patrickeastsmcginnis:  yea : /21:50
*** cknight has quit IRC21:52
*** diablo_rojo has quit IRC21:54
*** gouthamr has quit IRC22:01
*** harlowja has quit IRC22:01
*** xinli has joined #openstack-cinder22:02
*** dustins has quit IRC22:10
*** alonmarx_ has quit IRC22:12
*** viks has joined #openstack-cinder22:17
*** catintheroof has quit IRC22:20
Apoorvasmcginnis  what does "36 days" signify in <last success: 2016-12-12 11:48:54 (36 days, 5:26:19 old)> from http://paste.openstack.org/show/595249/22:23
smcginnisApoorva: That it's been 36 days since the last time there was a successful run reported.22:24
Apoorvasmcginnis In that case Blockbridge EPS CI is out for 48 days and shows 100% success rate? #confused22:25
smcginnisApoorva: 100% of the times when it was reporting was successful.22:28
*** thiagolib has quit IRC22:28
*** salv-orlando has joined #openstack-cinder22:28
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Switch ManageableSnaphots & ManageableVolumes list to OVO  https://review.openstack.org/38878322:29
Apoorvasmcginnis got it. thanks22:29
*** tlian has quit IRC22:34
*** xyang has joined #openstack-cinder22:35
*** xyang has quit IRC22:36
*** kfarr has quit IRC22:38
*** david-lyle has quit IRC22:51
_alastor_I'm struggling a bit to get Cinder backed images working in Glance.  Does anyone have an example glance.conf or local.conf I could use to get my devstack instance working with it?22:56
_alastor_I keep hitting an "Unauthorized" error in glance and I've tried every username/password/auth_url combination I can think of22:57
*** xyang has joined #openstack-cinder22:57
_alastor_I've been working off this page: http://docs.openstack.org/admin-guide/blockstorage-volume-backed-image.html22:57
*** tlian has joined #openstack-cinder22:58
*** xyang has quit IRC22:59
*** david-lyle has joined #openstack-cinder23:03
*** ducttape_ has quit IRC23:07
*** david-lyle has quit IRC23:10
*** tommylikehu_ has joined #openstack-cinder23:11
*** tommylikehu_ has quit IRC23:11
*** tommylik_ has joined #openstack-cinder23:11
*** tommylik_ has quit IRC23:12
patrickeast_alastor_: i haven't tried the glance backend for a while, its totally possible it doesn't work anymore :(23:12
patrickeast_alastor_: i don't think there are any ci jobs that run with it, only the cache in cinder23:12
openstackgerritPatrick East proposed openstack/cinder: Switch to using generic groups with Pure driver  https://review.openstack.org/41685423:12
openstackgerritPatrick East proposed openstack/cinder: Allow snapshots and volumes to have Null group values  https://review.openstack.org/42157623:12
_alastor_patrickeast: Well, considering how much the SEs want this, if the feature is broken (not THAT it is, just IF it is) I'll be fixing it :)23:14
*** lamt has quit IRC23:15
_alastor_patrickeast: Do you know how to force glance to use v2 API?  Currently everything seems to be going through v1 which was deprecated in Newton23:16
patrickeast_alastor_: i think there is glance_api_version in cinder.conf that can be set23:17
*** knomura has joined #openstack-cinder23:18
_alastor_patrickeast: seems weird that the glance api version would be set by cinder....23:18
patrickeast_alastor_: i guess it depends on what you want, if you want cinder to use the v2 api when i calls glance thats the one since its configuring the glance client code in c-* services, if you want glance itself to run exposing a different version then i think it would require changes to glance configs23:20
patrickeasts/when i/when it/23:20
*** asettle has quit IRC23:20
patrickeastleft alone cinder will always use v1 i think23:20
patrickeasti remember having some issues with this a while back trying to get different v2 exposed properties on the images23:21
*** salv-orlando has quit IRC23:21
_alastor_well, I'm going to say v2 probably doesn't work.  Have a gander at that stack trace: http://paste.openstack.org/show/595294/23:22
*** knomura has quit IRC23:22
*** xyang has joined #openstack-cinder23:24
patrickeastyea that doesn't look so great23:26
*** xyang has quit IRC23:27
*** gouthamr has joined #openstack-cinder23:27
*** sdague has quit IRC23:28
*** xinli has quit IRC23:30
*** s-shiono has joined #openstack-cinder23:31
*** s-shiono_ has joined #openstack-cinder23:46
*** stakeda has quit IRC23:47
*** s-shiono has quit IRC23:48
*** dave-mccowan has joined #openstack-cinder23:51
*** rajinir has quit IRC23:56
openstackgerritRaunak Kumar proposed openstack/cinder: Fix for live migration for Nimble Storage  https://review.openstack.org/42021423:58
*** stakeda has joined #openstack-cinder23:58

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