Friday, 2016-05-06

patrickeastjgriffith: cool, yea we can hammer out those details later00:00
patrickeastjgriffith: trivial to move it around00:00
jgriffithpatrickeast: crazy idea.. we could also put those things in the base driver00:00
patrickeastjgriffith: yea i was going to do that00:00
jgriffithpatrickeast: get them for free in the sub classes00:00
patrickeastjgriffith: but this puts one more layer of abstraction from driver accessing the db :p00:00
jgriffithand limit db stuff to base driver only so we don't kill ourselves with future changes00:01
jgriffithpatrickeast: yeah00:01
jgriffithpatrickeast: well.. my vote I guess is you should flush it out a bit more and see what others think00:01
jgriffithpatrickeast: or don't and see what others think :)00:01
patrickeastjgriffith: sounds like a plan00:01
patrickeastjgriffith: haha, nah i'll get it passing tests and put up a WIP review00:01
jgriffithpatrickeast: or just fix it all and merge it while we sleep :)00:01
patrickeastjgriffith: that is always an option :D00:02
*** chlong has quit IRC00:05
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Add Cheesecake APIs to policy.json file  https://review.openstack.org/31312400:06
*** yangyape_ has quit IRC00:10
*** garthb_ has quit IRC00:13
*** prometheanfire has left #openstack-cinder00:13
*** leeantho has quit IRC00:22
openstackgerritwanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata  https://review.openstack.org/14773800:24
openstackgerritwanghao proposed openstack/cinder: Backup tenant attribute support  https://review.openstack.org/25837400:25
*** salv-orlando has joined #openstack-cinder00:26
openstackgerritwanghao proposed openstack/cinder: Catch GlanceMetadataNotFound in creating cgsnp  https://review.openstack.org/30269100:29
*** laughterwym has joined #openstack-cinder00:29
*** laughter_ has joined #openstack-cinder00:31
*** salv-orlando has quit IRC00:32
*** earlephilhower has quit IRC00:33
*** laughterwym has quit IRC00:34
*** laughter_ has quit IRC00:34
*** ducttape_ has quit IRC00:35
*** itlinux has quit IRC00:36
*** laughterwym has joined #openstack-cinder00:37
*** Julien-zte has joined #openstack-cinder00:37
*** mriedem has quit IRC00:39
*** laughterwym has quit IRC00:40
*** laughterwym has joined #openstack-cinder00:40
*** akshai has joined #openstack-cinder00:40
*** akshai_ has joined #openstack-cinder00:42
*** akshai has quit IRC00:45
openstackgerritKongWei proposed openstack/cinder: Add ZTE Block Storage Driver  https://review.openstack.org/31235500:46
*** mriedem has joined #openstack-cinder00:46
*** salv-orlando has joined #openstack-cinder00:51
*** mtanino has quit IRC00:52
*** salv-orlando has quit IRC00:56
*** suyash has quit IRC00:57
*** Julien-zte has quit IRC00:58
*** iceyao has joined #openstack-cinder01:02
*** iceyao_ has joined #openstack-cinder01:06
*** ducttape_ has joined #openstack-cinder01:07
*** ducttape_ has quit IRC01:09
*** iceyao has quit IRC01:09
*** yangyapeng has joined #openstack-cinder01:12
*** enriquetaso has quit IRC01:14
*** raunak has quit IRC01:18
*** haplo37 has joined #openstack-cinder01:18
*** yangyapeng has quit IRC01:21
*** salv-orlando has joined #openstack-cinder01:22
*** nnex has joined #openstack-cinder01:24
*** Lee1092 has joined #openstack-cinder01:25
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found  https://review.openstack.org/29617601:27
*** yangxi has joined #openstack-cinder01:29
*** xyang1 has quit IRC01:29
*** nnex has quit IRC01:30
openstackgerritPatrick East proposed openstack/cinder: Move initiator_data access to helper methods  https://review.openstack.org/31320001:30
patrickeastjgriffith: ^01:30
*** salv-orlando has quit IRC01:30
*** nnex has joined #openstack-cinder01:30
*** chlong has joined #openstack-cinder01:31
*** EinstCrazy has joined #openstack-cinder01:34
openstackgerritAlex Meade proposed openstack/cinder-specs: Promote backend after failover (cheesecake)  https://review.openstack.org/31259101:37
openstackgerritCao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot  https://review.openstack.org/29852201:49
*** houming has joined #openstack-cinder01:57
*** salv-orlando has joined #openstack-cinder01:57
*** gcb has quit IRC01:59
*** savihou has quit IRC02:00
*** savihou has joined #openstack-cinder02:01
openstackgerritPeter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton  https://review.openstack.org/31242702:01
*** ducttape_ has joined #openstack-cinder02:03
*** EinstCra_ has joined #openstack-cinder02:06
*** yangxi has quit IRC02:06
*** yangxi has joined #openstack-cinder02:09
*** EinstCrazy has quit IRC02:09
*** salv-orlando has quit IRC02:09
*** bardia has quit IRC02:10
*** gcb has joined #openstack-cinder02:11
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Continue to deal with other port groups  https://review.openstack.org/31292802:11
*** karthikp has joined #openstack-cinder02:12
*** ducttape_ has quit IRC02:14
*** apoorvad has quit IRC02:16
*** gouthamr_ has quit IRC02:20
*** nnex has quit IRC02:23
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found  https://review.openstack.org/29617602:27
*** raunak has joined #openstack-cinder02:28
openstackgerritWilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers  https://review.openstack.org/28302902:28
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Mask chap password in log  https://review.openstack.org/27159502:31
*** sasukeh has joined #openstack-cinder02:33
*** yangxi has quit IRC02:34
*** mriedem has quit IRC02:36
*** raunak_ has joined #openstack-cinder02:38
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Add replication v2.1 test cases  https://review.openstack.org/31320902:38
*** raunak has quit IRC02:39
*** raunak_ is now known as raunak02:39
*** iceyao has joined #openstack-cinder02:40
*** iceyao_ has quit IRC02:44
*** daneyon has joined #openstack-cinder02:44
*** daneyon has quit IRC02:44
*** daneyon has joined #openstack-cinder02:45
*** ducttape_ has joined #openstack-cinder02:47
openstackgerritzhangguoqing proposed openstack/cinder: Violin: resize volume if cloned lun is larger  https://review.openstack.org/29509402:48
*** daneyon has quit IRC02:51
*** yangxi has joined #openstack-cinder02:51
*** Julien-zte has joined #openstack-cinder03:00
*** sasukeh has quit IRC03:00
*** sasukeh has joined #openstack-cinder03:02
*** raunak has quit IRC03:03
*** raunak has joined #openstack-cinder03:06
openstackgerritWilson Liu proposed openstack/cinder-specs: Migrate volume between backends in an async way  https://review.openstack.org/31285303:14
*** akshai_ has quit IRC03:15
*** itlinux has joined #openstack-cinder03:18
*** sasukeh has quit IRC03:19
*** markvoelker has quit IRC03:20
*** salv-orlando has joined #openstack-cinder03:21
*** nnex has joined #openstack-cinder03:22
*** yangyapeng has joined #openstack-cinder03:28
*** salv-orlando has quit IRC03:30
*** bardia has joined #openstack-cinder03:32
*** julim has quit IRC03:33
*** amit213 has quit IRC03:33
*** julim has joined #openstack-cinder03:33
*** amit213 has joined #openstack-cinder03:33
*** bardia has quit IRC03:38
*** ducttape_ has quit IRC03:40
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Apply storage policy at vmdk level  https://review.openstack.org/31321303:43
*** salv-orlando has joined #openstack-cinder03:44
*** yangxi has quit IRC03:45
*** salv-orlando has quit IRC03:46
*** laughterwym has quit IRC03:50
*** yangxi has joined #openstack-cinder03:50
*** buhman has joined #openstack-cinder03:50
*** laughterwym has joined #openstack-cinder03:50
buhmanhttps://github.com/openstack/python-cinderclient/blob/master/cinderclient/client.py#L63803:51
buhmanwouldn't it make more sense to do extension discovery via pkg_resources, and some openstack.cinderclient.extensions entrypoint?03:51
*** yangxi has quit IRC03:54
*** esker has quit IRC03:55
*** karthikp has quit IRC03:59
*** links has joined #openstack-cinder04:02
openstackgerritCao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot  https://review.openstack.org/29852204:08
*** Julien-zte has quit IRC04:16
*** markvoelker has joined #openstack-cinder04:21
*** dave-mcc_ has quit IRC04:26
*** markvoelker has quit IRC04:26
*** salv-orlando has joined #openstack-cinder04:28
*** salv-orlando has quit IRC04:32
*** ducttape_ has joined #openstack-cinder04:41
*** yangxi has joined #openstack-cinder04:43
*** daneyon_ has joined #openstack-cinder04:46
*** ducttape_ has quit IRC04:46
*** lennyb__ has joined #openstack-cinder04:49
*** lennyb__ has quit IRC04:50
*** salv-orlando has joined #openstack-cinder05:01
*** salv-orlando has quit IRC05:04
*** yangxi has quit IRC05:07
*** Poornima has joined #openstack-cinder05:07
*** yangxi has joined #openstack-cinder05:09
*** laughter_ has joined #openstack-cinder05:11
*** raunak has quit IRC05:13
*** laughterwym has quit IRC05:14
*** salv-orlando has joined #openstack-cinder05:17
*** markvoelker has joined #openstack-cinder05:22
*** salv-orlando has quit IRC05:24
*** markvoelker has quit IRC05:26
*** jseiler_ has joined #openstack-cinder05:30
*** jseiler__ has quit IRC05:34
*** salv-orlando has joined #openstack-cinder05:35
*** salv-orlando has quit IRC05:39
*** vmtrooper has quit IRC05:42
*** laughter_ has quit IRC05:44
*** laughterwym has joined #openstack-cinder05:44
openstackgerritPranali Deore proposed openstack/cinder: Add logging when filtering returns nothing  https://review.openstack.org/30628505:47
openstackgerritLisaLi proposed openstack/os-brick: Copy encryptors from Nova to os-brick  https://review.openstack.org/24737206:01
*** rdx_matt has joined #openstack-cinder06:06
*** gcb has quit IRC06:13
*** jkraj has joined #openstack-cinder06:17
*** vmtrooper has joined #openstack-cinder06:17
openstackgerritNitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst  https://review.openstack.org/31323706:19
*** zul has quit IRC06:21
*** yangxi has quit IRC06:22
*** markvoelker has joined #openstack-cinder06:23
*** yangxi has joined #openstack-cinder06:25
*** gcb has joined #openstack-cinder06:26
openstackgerritNitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst  https://review.openstack.org/31323706:27
*** markvoelker has quit IRC06:27
openstackgerritCao ShuFeng proposed openstack/cinder: remove extra quota commit of manage snapshot  https://review.openstack.org/29852206:31
*** ircuser-1 has joined #openstack-cinder06:34
*** haplo37 has quit IRC06:38
*** e0ne has joined #openstack-cinder06:39
*** ducttape_ has joined #openstack-cinder06:42
openstackgerritCao ShuFeng proposed openstack/cinder: enhanche error message [1/5]  https://review.openstack.org/29630406:43
openstackgerritCao ShuFeng proposed openstack/cinder: Allow API user to update multiattach attr of volume  https://review.openstack.org/30967406:45
*** salv-orlando has joined #openstack-cinder06:46
*** ducttape_ has quit IRC06:48
*** kfox1111 has quit IRC06:53
*** e0ne has quit IRC06:53
*** salv-orlando has quit IRC06:57
*** e0ne has joined #openstack-cinder06:59
*** e0ne has quit IRC07:00
*** permalac has joined #openstack-cinder07:06
*** itlinux has quit IRC07:14
*** e0ne has joined #openstack-cinder07:16
*** gcb has quit IRC07:21
openstackgerritPeter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton  https://review.openstack.org/31242707:23
*** markvoelker has joined #openstack-cinder07:23
*** salv-orlando has joined #openstack-cinder07:26
*** markvoelker has quit IRC07:28
*** markus_z has joined #openstack-cinder07:29
*** gcb has joined #openstack-cinder07:33
*** lpetrut has joined #openstack-cinder07:36
openstackgerritsurya n proposed openstack/cinder: Minor typo fixes.  https://review.openstack.org/31325907:37
*** openstackstatus has quit IRC07:38
*** openstackstatus has joined #openstack-cinder07:39
*** ChanServ sets mode: +v openstackstatus07:39
*** salv-orlando has quit IRC07:40
*** lpetrut has quit IRC07:43
*** rdx_matt has quit IRC07:43
*** gcb has quit IRC07:43
*** rdx_matt has joined #openstack-cinder07:43
*** lpetrut has joined #openstack-cinder07:45
*** rdx_matt_ has joined #openstack-cinder07:49
*** rdx_matt has quit IRC07:53
*** chlong has quit IRC07:54
*** gcb has joined #openstack-cinder07:55
*** ildikov_ has joined #openstack-cinder07:56
*** ildikov has quit IRC07:58
*** ildikov_ is now known as ildikov07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:02
*** gcb has quit IRC08:03
*** mkrcmari__ has joined #openstack-cinder08:05
*** e0ne has quit IRC08:07
*** mvk_ has quit IRC08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** gcb has joined #openstack-cinder08:16
*** laughterwym has quit IRC08:17
*** yuriy_n17 has quit IRC08:19
*** salv-orlando has joined #openstack-cinder08:19
*** laughterwym has joined #openstack-cinder08:19
*** yuriy_n17 has joined #openstack-cinder08:19
*** e0ne has joined #openstack-cinder08:20
*** salv-orlando has quit IRC08:24
*** gcb has quit IRC08:25
*** lpetrut has quit IRC08:26
*** laughterwym has quit IRC08:29
*** laughterwym has joined #openstack-cinder08:30
*** jistr has joined #openstack-cinder08:32
openstackgerritYuriy Nesenenko proposed openstack/cinder: Handle SSL termination proxies for version list  https://review.openstack.org/30515208:33
*** laughterwym has quit IRC08:34
*** rdx_matt_ has quit IRC08:35
*** rdx_matt has joined #openstack-cinder08:35
*** gcb has joined #openstack-cinder08:39
*** rdx_matt has quit IRC08:40
*** e0ne has quit IRC08:40
openstackgerritYuriy Nesenenko proposed openstack/cinder: Image virtual size doesn't fit to volume size  https://review.openstack.org/31115208:42
*** e0ne has joined #openstack-cinder08:42
*** mkrcmari__ has quit IRC08:43
*** ducttape_ has joined #openstack-cinder08:44
*** ducttape_ has quit IRC08:49
*** jordanP has joined #openstack-cinder08:55
*** sage has quit IRC08:56
*** tesseract has joined #openstack-cinder08:56
*** gcb has quit IRC08:56
*** tesseract is now known as Guest4293008:56
*** sage has joined #openstack-cinder08:57
*** EinstCra_ has quit IRC08:59
*** EinstCrazy has joined #openstack-cinder09:00
*** lpetrut has joined #openstack-cinder09:14
*** vmtrooper has quit IRC09:15
openstackgerritYuriy Nesenenko proposed openstack/cinder: Replace explicit changing of config parameters  https://review.openstack.org/30224909:25
*** mkrcmari__ has joined #openstack-cinder09:28
*** laughterwym has joined #openstack-cinder09:32
*** laughterwym has quit IRC09:37
*** salv-orlando has joined #openstack-cinder09:41
*** lpetrut has quit IRC09:42
openstackgerritGorka Eguileor proposed openstack/cinder: Move OVO methods from CinderObject to Persistent  https://review.openstack.org/31293009:47
openstackgerritGorka Eguileor proposed openstack/cinder: Remove remotable from OVOs  https://review.openstack.org/31292909:47
openstackgerritGorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs  https://review.openstack.org/30707509:47
openstackgerritGorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts  https://review.openstack.org/30707409:47
*** houming has quit IRC09:48
*** salv-orlando has quit IRC09:49
yuriy_n17geguileo: Hi!09:51
geguileoyuriy_n17: Hi09:51
yuriy_n17geguileo: I'd like to discuss on https://review.openstack.org/#/c/302249 if you have free time now.09:52
geguileoyuriy_n17: Sure09:52
yuriy_n17geguileo: See if we use override_config method for overwriting config options there we'll get race conditions.09:53
yuriy_n17geguileo: Now I specially send the patch with that method to show it.09:54
*** yangxi has quit IRC09:54
geguileoyuriy_n17: Ok, I believe we shouldn't have races09:55
*** yangxi has joined #openstack-cinder09:55
yuriy_n17geguileo: Unfortunately, we have09:55
geguileoyuriy_n17: So I'd like to look at the errors to see if we have an underlying problem there09:56
yuriy_n17geguileo: Sure. https://review.openstack.org/#/c/30224909:56
*** jkraj has quit IRC09:57
geguileoyuriy_n17: I am downloading it to test locally, because the gate it going to take a while09:57
geguileoyuriy_n17: And with my memory I'll forget later to look at it  :-(09:57
openstackgerritMerged openstack/cinder: Add devref for conditional updates  https://review.openstack.org/31304009:57
yuriy_n17geguileo: Ok09:58
yuriy_n17geguileo: You can look at http://logs.openstack.org/49/302249/5/check/gate-cinder-python27-db/f6b882d/console.html09:59
geguileoyuriy_n17: Is that from this run?10:01
yuriy_n17geguileo: Yes10:01
ekarlsoasdasd10:03
*** ekarlso has quit IRC10:08
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376610:10
*** haplo37 has joined #openstack-cinder10:16
*** ankit_ag has joined #openstack-cinder10:17
*** yangxi has quit IRC10:19
*** dims has quit IRC10:20
*** chuck__ has joined #openstack-cinder10:25
*** markvoelker has joined #openstack-cinder10:26
*** sdague has joined #openstack-cinder10:28
*** markvoelker has quit IRC10:30
openstackgerritRyan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton  https://review.openstack.org/31242710:31
*** yangyape_ has joined #openstack-cinder10:33
*** alaski is now known as lascii10:36
*** yangyapeng has quit IRC10:36
*** yangyape_ has quit IRC10:37
geguileoyuriy_n17: ping10:42
yuriy_n17geguileo: yes10:42
geguileoyuriy_n17: Ok, the flags method works fine10:43
geguileoyuriy_n17: The problem is that other tests are setting the CONF manually10:43
geguileoyuriy_n17: Instead of using flags10:43
geguileoyuriy_n17: So even if you change that test to do it right, the others are doing it wrong and break your test10:43
yuriy_n17geguileo: You mean in that suitcase or others?10:44
geguileoyuriy_n17: Others10:44
geguileoyuriy_n17: This is one example https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_hitachi_hnas_backend.py#L36210:45
*** ducttape_ has joined #openstack-cinder10:45
geguileoyuriy_n17: So the interactions of that test doing it wrong (and others like that) is what's making the tests you've changed fail10:45
yuriy_n17geguileo: Ok, I see, thanks10:46
geguileoyuriy_n17: As I suspected we have a bigger problem, we have to fix it everywhere  :-(10:46
yuriy_n17geguileo: Well-well...10:46
*** ducttape_ has quit IRC10:49
*** EinstCrazy has quit IRC10:50
*** iceyao has quit IRC10:51
*** lpetrut has joined #openstack-cinder11:01
*** dims has joined #openstack-cinder11:02
*** chuck__ has quit IRC11:06
openstackgerritGorka Eguileor proposed openstack/cinder: Prevent Multitable conditional updates  https://review.openstack.org/31065611:16
openstackgerritYuriy Nesenenko proposed openstack/cinder: Replace manually changing CONF options  https://review.openstack.org/30224911:20
*** gouthamr has joined #openstack-cinder11:21
*** laughterwym has joined #openstack-cinder11:22
*** haplo37 has quit IRC11:25
*** haplo37 has joined #openstack-cinder11:26
*** markvoelker has joined #openstack-cinder11:27
*** gouthamr_ has joined #openstack-cinder11:27
*** chuck__ has joined #openstack-cinder11:28
*** gouthamr has quit IRC11:31
*** markvoelker has quit IRC11:31
*** laughterwym has quit IRC11:36
*** laughterwym has joined #openstack-cinder11:37
*** laughterwym has quit IRC11:41
*** yangyapeng has joined #openstack-cinder11:49
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Use to_utf8() instead of safe_encode() in convert_str()  https://review.openstack.org/31352111:52
*** laughterwym has joined #openstack-cinder11:53
*** salv-orlando has joined #openstack-cinder11:58
*** Lee1092 has quit IRC12:12
*** markvoelker has joined #openstack-cinder12:14
*** maestro has joined #openstack-cinder12:16
*** ekarlso has joined #openstack-cinder12:19
openstackgerritNitin Madhok proposed openstack/cinder-specs: Making doc fixes in backup-snapshots.rst  https://review.openstack.org/31323712:25
*** iceyao has joined #openstack-cinder12:25
*** salv-orlando has quit IRC12:28
*** xyang1 has joined #openstack-cinder12:38
*** iceyao has quit IRC12:38
*** savihou has quit IRC12:39
*** takedakn has joined #openstack-cinder12:40
*** ankit_ag has quit IRC12:41
openstackgerritPranali Deore proposed openstack/cinder: Add logging when filtering returns nothing  https://review.openstack.org/30628512:44
*** iceyao has joined #openstack-cinder12:44
*** ducttape_ has joined #openstack-cinder12:46
*** tpsilva has joined #openstack-cinder12:47
*** ducttape_ has quit IRC12:51
*** iceyao has quit IRC12:51
*** alejandrito has joined #openstack-cinder12:52
*** timcl has joined #openstack-cinder12:53
*** akshai has joined #openstack-cinder12:53
*** rlrossit has joined #openstack-cinder12:54
scottdaameade: Did my comments in https://review.openstack.org/#/c/299052/ make sense, about not supporting the deprecated sort_key and sort_dir?12:56
ameadescottda: yeah, i was being lazy and just leaving in that stuff for future proofing12:56
ameadescottda: once sheel is done with pagination on the server side we can add in the proper client stuff12:57
sheelscottda: I have one query here12:57
scottdaOK, cool. It looks like the server side pagination is fine for the new "sort key:dir" style.12:57
sheelscottda:  we are still using sort_dir and sort_key for volume...12:58
sheelscottda:  is it ok to update that as well?12:58
sheelfor cinderclient12:58
*** iceyao has joined #openstack-cinder12:58
*** sdake has joined #openstack-cinder13:00
scottdasheel: You can see by tracing through the server code that sort_dir and sort_key are used internally, but they are set in the helper function when you use "sort key:dir"....looking for the code...13:00
sheelscottda: yep, its ok in serverside13:01
sheelscottda: my query was for client side13:01
sheelscottda: we are using sort_key and sort_dir in volume get13:01
scottdasheel: The client should only support "sort key:dir" and should not support "--sort_key <key> --sort_dir <direction>" since that is deprecated.13:01
*** EinstCrazy has joined #openstack-cinder13:02
sheelscottda:  yep...13:02
scottdasheel: Yes, it is still in volume get, but it is deprecated in kilo and I assume it will be removed at some point. There is no need to support for the new messages CLI if it's already deprecated.13:02
sheelI'll see for client for volume13:02
sheelscottda:  yes, sounds good..13:03
*** haplo37 has quit IRC13:03
sheelscottda: thanks ..13:03
*** iceyao has quit IRC13:03
scottdaThanks to you and ameade for working on this, sheel13:04
ameade+113:04
sheelscottda:  :)13:04
jordanPhi guys. Could someone have a look at https://review.openstack.org/#/c/310204/ (Assisted snapshot:  use Keystone Session with Nova Client). It's needed to make Cinder compatible with Keystone v3. Thanks !13:05
*** nnex has quit IRC13:11
*** nnex has joined #openstack-cinder13:11
*** savihou has joined #openstack-cinder13:15
scottdadulek: I have a simple question for you: What to do about extensions?13:15
scottdadulek: Just kidding about being simple....13:16
*** nnex has quit IRC13:16
*** mtanino has joined #openstack-cinder13:17
*** iceyao has joined #openstack-cinder13:18
*** mriedem has joined #openstack-cinder13:20
*** iceyao has quit IRC13:24
*** porrua has joined #openstack-cinder13:27
*** mtanino has quit IRC13:27
*** iceyao has joined #openstack-cinder13:28
*** salv-orlando has joined #openstack-cinder13:28
*** jungleboyj has joined #openstack-cinder13:29
*** salv-orlando has quit IRC13:31
*** akerr has joined #openstack-cinder13:31
*** jwcroppe has joined #openstack-cinder13:32
*** cknight has joined #openstack-cinder13:33
openstackgerritSheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete  https://review.openstack.org/31355013:34
*** BigWillie has joined #openstack-cinder13:34
*** gouthamr_ is now known as gouthamr13:34
jgriffithsmcginnis: have you by chance noticed what this:  https://review.openstack.org/#/c/300189/ does to the logs?13:35
*** Poornima has quit IRC13:35
jgriffithhmm... actually, may be something else13:36
*** iceyao has quit IRC13:36
smcginnisjgriffith: Where are the logs?13:36
*** iceyao has joined #openstack-cinder13:36
smcginnisOh, mixed up.13:36
smcginnisjgriffith: What are you seeing?13:36
jgriffithgrab a recent tempest run, look in c-vol and search on ERROR13:36
jgriffithooodles and oodles of these:  http://paste.openstack.org/show/496331/13:37
smcginnisjgriffith: Not seeing anything on the tempest run of that patch.13:37
* smcginnis looking at paste13:37
jgriffithyeah, I don't think it's that patch that did it13:37
jgriffithmay not even be in Cinder honestly13:38
jgriffithI'll look at it when I get in office.  Something in my CI blew up last night and it looks like brick isn't working13:38
smcginnisjgriffith: That looks like the warning that and all the other UUID patches were trying to fix.13:38
jgriffithnoticed when searching log files for "Error" all of those warnings13:38
geguileojgriffith: Those warnings come from a change in oslo versioned object13:38
jgriffithgeguileo: yeah, just saw that13:38
geguileojgriffith: There's been a work in progress by eharney and tbarron to remove them13:39
jgriffithglad we have that in place :)  Even more glad that eharney and tbarron are fixing it13:39
geguileojgriffith: They've removed a lot of them13:39
smcginnisgeguileo: I believe those are all merged now.13:39
jgriffithmissed some :)13:39
smcginnisSo this invalid UUID must be sneaking in somewhere else.13:39
geguileosmcginnis: Yes, they merged, I just don't know if that removed all warnings13:39
smcginnisgeguileo: Apparently not. :)13:39
geguileoI encountered one more yesterday in one of my patches13:40
geguileoAnd I fixed it in that patch13:40
geguileoBut I don't remember on which one of my patches...13:40
smcginnisjgriffith: What's your CI failure. Maybe we can share the misery. Mines still not 100%.13:40
smcginnisgeguileo: Awesome.13:40
geguileoGive me a sec13:40
smcginnisGuess it might take a little while to shake it all out.13:40
jgriffithsmcginnis: 2016-05-06 12:37:27.734 30336 ERROR oslo_messaging.rpc.dispatcher FailedISCSITargetPortalLogin: Could not login to any iSCSI portal.13:41
openstackgerritSean McGinnis proposed openstack/cinder: Change provider_id to StringField  https://review.openstack.org/30950813:41
jgriffithbrick connector.py get_potential_volume_paths is broken maybe?13:42
jgriffithfull logs here:  http://54.164.167.86/solidfire-ci-logs/refs-changes-38-147738-35/logs/c-vol.log.txt13:42
jgriffithStarted last night13:42
smcginnisjgriffith: Not an issue on your array or network issue?13:42
jgriffithhush13:43
*** jwcroppe_ has joined #openstack-cinder13:43
smcginnis:)13:43
*** sdague has quit IRC13:43
smcginnisMaybe new NetApp network security rules got put in place. No iSCSI traffic allowed in the lab. :P13:44
jgriffithLOL... sounds about right13:44
ameadelol13:46
geguileojgriffith: smcginnis Found it:  https://review.openstack.org/#/c/259429/23/cinder/tests/unit/api/fakes.py13:46
ameade>.<13:46
geguileoThat's the one that was giving me trouble with the UUIDs13:46
*** jwcroppe has quit IRC13:46
smcginnisameade: I only joke because it's not too far from the truth here. :)13:46
smcginnisgeguileo: Nice!13:47
*** iceyao has quit IRC13:47
jgriffithgeguileo: very cool13:47
ameadesmcginnis: I'm not saying it isnt a fair statement lmao13:47
smcginnis;)13:47
geguileoIt's not merged yet because it's in one of the API races patches13:47
*** ducttape_ has joined #openstack-cinder13:47
jgriffithgeguileo: here's a +A for your troubles13:47
geguileoXD13:47
smcginnisBeat me to it.13:48
geguileoIt's the 3rd patch in the series, so it won't merge yet  :-(13:48
smcginnisReally with gerrit would make it easy to see dependency chains.13:48
jgriffithalthough my vote has nothing to do with the message (not sure that will fix it anyway)13:48
jgriffithgeguileo: I'm starting to feel bad watching you have to rebase every day13:48
geguileorofl13:49
geguileoI'm mostly afraid that I'll introduce a regression in one of the rebases  :-(13:49
*** crose has joined #openstack-cinder13:52
*** chris_morrell has quit IRC13:52
*** jwcroppe has joined #openstack-cinder13:54
*** edmondsw has joined #openstack-cinder13:55
*** jgregor has joined #openstack-cinder13:56
*** dustins has joined #openstack-cinder13:56
*** dansmith is now known as snotdan13:56
*** chuck__ is now known as zul13:57
*** akshai has quit IRC13:57
*** jwcroppe_ has quit IRC13:57
*** iceyao has joined #openstack-cinder13:58
*** jkraj has joined #openstack-cinder13:58
*** snotdan is now known as superdan13:58
*** akshai has joined #openstack-cinder13:59
*** sdake_ has joined #openstack-cinder14:00
*** ametts has joined #openstack-cinder14:00
*** martyturner has joined #openstack-cinder14:00
*** jdillaman has quit IRC14:00
*** lprice has quit IRC14:01
jgriffithgeguileo: oh geez, don't even think it!14:01
geguileoI'll try  :-)14:01
jgriffithgeguileo: there are down sides to splitting work up  in to dep patches14:01
*** baumann has joined #openstack-cinder14:02
geguileojgriffith: Yeah, but in general it's usually better, because reviews can be done faster and more in depth14:02
geguileoOr at least that's whay I think14:02
*** yangyapeng has quit IRC14:02
*** maestro has quit IRC14:03
jgriffithI agree with you14:03
jgriffithfor me it's more so I can kinda keep in my head what I'm working on14:03
*** sdake has quit IRC14:04
*** alejandrito has quit IRC14:05
*** catintheroof has joined #openstack-cinder14:05
*** ducttape_ has quit IRC14:08
*** mtanino has joined #openstack-cinder14:09
*** jkraj has quit IRC14:11
*** iceyao has quit IRC14:13
-openstackstatus- NOTICE: Zuul is currently recovering from a large number of changes, it will take a few hours until your job is processed. Please have patience and enjoy a great weekend!14:15
*** catintheroof has quit IRC14:15
guitarzansmcginnis: geguileo jgriffith we're going to get rid of requiring UUIDs in most of those fields right?14:15
geguileoguitarzan: No14:15
guitarzangeguileo: why?14:15
*** salv-orlando has joined #openstack-cinder14:15
geguileoguitarzan: What we are doing is making them have real UUIDs14:15
guitarzanthey can't be forced to uuids14:15
guitarzanexisting installs have data that aren't UUIDS14:16
geguileoguitarzan: As the id pattern?14:16
smcginnisguitarzan: For which fields?14:16
guitarzanwe're validating data that isn't actually of that type14:16
*** jwcroppe has quit IRC14:16
guitarzansmcginnis: project_id is the one that I know of for sure14:16
geguileoguitarzan: I think there was a discussion about it somewhere14:16
smcginnisguitarzan: There's my patch for provider_id: https://review.openstack.org/#/c/309508/14:16
smcginnisI haven't seen anything yet for project_id.14:17
geguileoguitarzan: And if I remember correctly it was decided that we would use UUIDs14:17
guitarzangeguileo: you're just saying that people can't upgrade to N14:17
*** links has quit IRC14:17
jgriffithguitarzan: that is a bit of a problem14:17
smcginnisgeguileo: Right, I vaguely remember that discussion and I thought it was decided project_id really should be UUID.14:17
smcginnisBut no idea why now.14:17
*** jwcroppe has joined #openstack-cinder14:17
guitarzansmcginnis: yes, but existing projects in our db AREN'T uuids14:17
jgriffithguitarzan: hey EVERYONE change your project ID etc now!14:17
smcginnisIf there are deployments not using UUIDs, then we probaby should switch that to StringField as well.14:18
guitarzansmcginnis: yeah, that's all I'm saying14:18
smcginnisI don't see any strong reason why that would need to be handled as a UUID.14:18
geguileoguitarzan: It looks like we are changing the project ID type, but other things like volume IDs will still be checked for valid UUIDs14:18
*** raunak has joined #openstack-cinder14:18
geguileos/project/provider14:19
geguileoguitarzan: So your project's ID is not a UUID?14:19
guitarzangeguileo: historically, project ids were not uuids14:19
guitarzanthey could be anything14:20
*** iceyao has joined #openstack-cinder14:20
jgriffithguitarzan: you seem to be under the impression that any of us care to consider or learn from history :)14:20
jgriffithguitarzan: that's not entirely accurate :)14:20
guitarzanjgriffith: true, I assume they had to be strings :)14:21
guitarzanprobably even ascii strings14:21
jgriffithguitarzan: indeed!14:21
geguileoguitarzan: So you have project_ids that are not UUIDs, right?14:21
guitarzangeguileo: yes14:21
geguileoguitarzan: Then we need to change versioned objects14:22
geguileoguitarzan: If there's not a bug for that yet you can create one and I'll fix it14:22
guitarzanyes, I'm looking up nova's object field definitions14:22
geguileoguitarzan: Yes, they have them as strings in the OVOs14:23
geguileoguitarzan: So we need to fix them14:23
guitarzangeguileo: I'll see if I can somehow run our data against master and see if there are other fields with invalid assumptions14:24
geguileoguitarzan: That would be great!!14:25
*** gouthamr has quit IRC14:25
geguileoguitarzan: Please let me know the results14:25
*** jungleboyj has quit IRC14:25
*** ducttape_ has joined #openstack-cinder14:25
*** r-daneel has joined #openstack-cinder14:26
*** iceyao has quit IRC14:27
*** karthikp has joined #openstack-cinder14:27
*** catintheroof has joined #openstack-cinder14:27
openstackgerritGorka Eguileor proposed openstack/cinder: Fix uncaught NotFound exceptions  https://review.openstack.org/31357114:28
*** takedakn has quit IRC14:29
*** ducttape_ has quit IRC14:29
*** lprice has joined #openstack-cinder14:29
*** jwcroppe has quit IRC14:30
*** EinstCrazy has quit IRC14:30
*** dave-mccowan has joined #openstack-cinder14:35
tbarroni would think that what user_id field should be parallel to project_id14:36
tbarronseems to me there should be a conversation with keystone folks about these two fields14:36
smcginnisguitarzan: Is that the case for you? ^^14:36
geguileotbarron: Agreed14:37
geguileotbarron: I'm just writing a patch to fix them14:37
*** iceyao has joined #openstack-cinder14:37
guitarzansmcginnis: I'm not sure about user_id14:37
geguileotbarron: And I've detected a problem in one of our ORM models  :''-(14:37
*** jwcroppe has joined #openstack-cinder14:37
smcginnisgeguileo: Users? I already started project_id too. :)14:37
geguileoguitarzan: According to the ORM model yes14:37
smcginnisJust running unit tests.14:37
geguileosmcginnis: I just finished running them  ;-P14:37
smcginnisgeguileo: OK, you win. :)14:38
geguileoJeje, thanks :-)14:38
tbarronbetter you guys than me, I'm not touching uuids :)14:38
smcginnistbarron: Hah!14:38
guitarzansmcginnis: yeah, we have uuid and non uuid user_ids14:38
smcginnisguitarzan: OK, thanks.14:38
smcginnisgeguileo: Want to take care of both?14:39
geguileosmcginnis: Ok14:39
geguileosmcginnis: I was going to change both in the same patch14:39
geguileosmcginnis: I've detected that we have a problem with the latest Message ORM model14:39
geguileosmcginnis: Because it uses 36 as the size of the project_id14:40
*** raunak has quit IRC14:40
smcginnisgeguileo: Too small?14:41
*** xinli has joined #openstack-cinder14:41
geguileosmcginnis: Well, it's inconsistent with the 255 characters defined everywhere else14:41
geguileosmcginnis: 36 is when we are only using UUID414:41
smcginnisFine if it's just a UUID, but yeah, if we're allowing any string for project_id it should probably be large enough to accomodate whatever's needed.14:42
geguileoAnd be consistent with the other models14:43
smcginnisYeah, consistency is good.14:43
*** e0ne has quit IRC14:43
*** e0ne has joined #openstack-cinder14:44
-openstackstatus- NOTICE: Zuul has been restarted. As a results, we only preserved patches in the gate queue. Be sure to recheck your patches in gerrit if needed.14:45
*** itlinux has joined #openstack-cinder14:46
*** Lee1092 has joined #openstack-cinder14:47
openstackgerritGorka Eguileor proposed openstack/cinder: Change OVOs project_id and user_id to StringField  https://review.openstack.org/31358414:47
*** takedakn has joined #openstack-cinder14:47
*** jwcroppe_ has joined #openstack-cinder14:48
*** itlinux has quit IRC14:51
*** jwcroppe has quit IRC14:52
*** yangxi has joined #openstack-cinder14:54
*** chris_morrell has joined #openstack-cinder14:56
*** yangxi has quit IRC14:57
*** eharney has joined #openstack-cinder14:57
*** sdake_ has quit IRC14:57
*** akapil has joined #openstack-cinder14:58
*** nnex has joined #openstack-cinder14:58
*** ducttape_ has joined #openstack-cinder14:59
*** haplo37 has joined #openstack-cinder14:59
*** lcurtis has joined #openstack-cinder15:00
*** akshai has quit IRC15:01
*** smcginnis_ has joined #openstack-cinder15:02
*** nnex has quit IRC15:03
*** Guest42930 has quit IRC15:05
*** david-lyle has joined #openstack-cinder15:06
*** ociuhandu has joined #openstack-cinder15:07
*** nnex has joined #openstack-cinder15:07
*** arch-nemesis has joined #openstack-cinder15:07
akerrdon't suppose anyone has seen some of the scenario tests fail because Nova couldn't find a valid host with the following error: Require both a host and instance NUMA topology to fit instance on host?15:10
*** jwcroppe has joined #openstack-cinder15:12
*** sdake has joined #openstack-cinder15:16
*** jwcroppe_ has quit IRC15:16
*** jwcroppe has quit IRC15:17
*** iceyao has quit IRC15:19
*** laughterwym has quit IRC15:21
*** laughterwym has joined #openstack-cinder15:22
openstackgerritXinli Guan proposed openstack/cinder: Make IBM drivers return snapshot object for cg ops  https://review.openstack.org/26828315:22
*** itlinux has joined #openstack-cinder15:22
*** martyturner has quit IRC15:23
*** iceyao has joined #openstack-cinder15:24
*** jdillaman has joined #openstack-cinder15:24
*** akshai has joined #openstack-cinder15:25
*** laughterwym has quit IRC15:26
*** chris_morrell has quit IRC15:27
*** itzdilip has joined #openstack-cinder15:27
*** iceyao has quit IRC15:30
*** iceyao_ has joined #openstack-cinder15:30
*** jordanP has quit IRC15:31
*** sdake has quit IRC15:32
openstackgerritxing-yang proposed openstack/cinder-specs: Generic Volume Group  https://review.openstack.org/30389315:33
*** sasukeh has joined #openstack-cinder15:34
openstackgerritMerged openstack/cinder-specs: Making doc fixes in backup-snapshots.rst  https://review.openstack.org/31323715:34
*** martyturner has joined #openstack-cinder15:35
*** catintheroof has quit IRC15:36
*** markus_z has quit IRC15:38
*** sasukeh has quit IRC15:38
*** nnex has quit IRC15:42
*** hemnafk is now known as hemna15:43
*** leeantho has joined #openstack-cinder15:43
*** salv-orlando has quit IRC15:43
*** leeantho_ has joined #openstack-cinder15:44
openstackgerritEric Harney proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718615:46
*** salv-orlando has joined #openstack-cinder15:47
*** leeantho has quit IRC15:47
*** dave-mccowan has quit IRC15:50
*** nnex has joined #openstack-cinder15:52
*** raunak has joined #openstack-cinder15:53
*** enriquetaso has joined #openstack-cinder15:54
*** akapil has quit IRC15:55
hemnaanyone know who the Smaug guy was in our Cinder meetings in Austin ?15:57
smcginnisSaggi15:57
hemnahttps://github.com/openstack/smaug/graphs/contributors15:58
hemnaonly 6 contributors so far15:58
smcginnisI think there are a lot of folks involved, but only a few code committers so far.15:58
*** nnex has quit IRC15:59
*** iceyao_ has quit IRC16:00
xyang1hemna: Saggi Mizrahi was at the cinder meeting16:02
*** lprice has quit IRC16:02
hemnaok thanks16:03
xyang1smcginnis: oh, did not see you already replied:)16:03
smcginnisThis might be interesting if you didn't see it: https://www.youtube.com/watch?v=_tVYuW_YMB816:03
smcginnisxyang1: ;)16:03
*** sasukeh has joined #openstack-cinder16:05
openstackgerritSheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete  https://review.openstack.org/31355016:08
*** johnthetubaguy has quit IRC16:08
*** johnthetubaguy has joined #openstack-cinder16:08
*** apoorvad has joined #openstack-cinder16:14
*** sdake has joined #openstack-cinder16:15
*** yangyapeng has joined #openstack-cinder16:17
*** yangyapeng has quit IRC16:18
*** crose has quit IRC16:20
*** frontrunner has joined #openstack-cinder16:21
*** e0ne has quit IRC16:23
*** bluex has quit IRC16:25
*** sasukeh has quit IRC16:25
*** salv-orlando has quit IRC16:25
*** cebruns has quit IRC16:26
*** catintheroof has joined #openstack-cinder16:27
*** karthikp has quit IRC16:27
*** karthikp has joined #openstack-cinder16:27
*** cebruns has joined #openstack-cinder16:27
*** mpjetta has quit IRC16:28
*** anish has quit IRC16:28
*** takedakn has quit IRC16:28
*** mtreinish has quit IRC16:28
*** anish has joined #openstack-cinder16:29
*** jidar has quit IRC16:29
*** baumann has left #openstack-cinder16:30
*** bluex has joined #openstack-cinder16:32
*** SergeyLukjanov has quit IRC16:32
*** mpjetta has joined #openstack-cinder16:32
*** yuriy_n17 has quit IRC16:33
*** SergeyLukjanov has joined #openstack-cinder16:33
*** bardia has joined #openstack-cinder16:33
*** mtreinish has joined #openstack-cinder16:35
*** jidar has joined #openstack-cinder16:39
*** yangyapeng has joined #openstack-cinder16:42
*** sdake_ has joined #openstack-cinder16:42
openstackgerritSheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete  https://review.openstack.org/31355016:45
*** lprice has joined #openstack-cinder16:45
*** sdake has quit IRC16:45
*** yangyapeng has quit IRC16:47
*** takedakn has joined #openstack-cinder16:47
*** sgotliv has joined #openstack-cinder16:49
*** garthb has joined #openstack-cinder16:50
*** jgregor has quit IRC16:58
openstackgerritxing-yang proposed openstack/cinder-specs: Replication with More Granularity (Tiramisu)  https://review.openstack.org/22972217:01
*** jwcroppe has joined #openstack-cinder17:05
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Add ServiceTopic & ServiceDisabledReason Enum  https://review.openstack.org/31317917:05
*** jseiler__ has joined #openstack-cinder17:06
*** jistr has quit IRC17:07
*** jwcroppe_ has joined #openstack-cinder17:07
*** jseiler_ has quit IRC17:09
guitarzanhmm, so one problem with that versioned objects fields warning is it doesn't know the most important piece of information... which field it's complaining about17:09
*** jwcroppe has quit IRC17:10
*** akapil has joined #openstack-cinder17:12
smcginnisguitarzan: Hah!17:12
smcginnisguitarzan: Guess that would make it too easy. ;)17:12
jgriffithguitarzan: and thus my frustration with the message this morning :(17:13
guitarzanyep17:13
guitarzanalso, the rpc version pinning seems a little odd17:14
jgriffithguitarzan: I'm also unclear on why internal issues like that are something the operator should ever care about or see17:14
jgriffithguitarzan: in other words, that's a dev problem I think17:14
guitarzanit doesn't seem to auto-downgrade the rpc version, even though it knows it needs to17:14
guitarzan"RPCVersionCapError: Requested message version, 2.0 is incompatible.  It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.8."17:15
guitarzandulek: should I expect that to work yet?17:15
guitarzanjgriffith: it is, but the devs are the ones breaking it :D17:15
guitarzansomebody has to be able to find the bugs17:16
guitarzanand you won't find that with devstack17:16
jgriffithguitarzan: good point17:16
jgriffithguitarzan: sounds like you need Jay to write a new hacking check :)17:16
guitarzan"code changed, failed hacking check'17:17
*** dave-mccowan has joined #openstack-cinder17:17
*** uberjay_ has quit IRC17:18
*** haplo37 has quit IRC17:18
*** uberjay has joined #openstack-cinder17:20
guitarzangeguileo: from a simple "volume get" from old production data, project_id and user_id seem to be the only complaints17:20
jgriffithguitarzan: seems doable17:20
jgriffith:)17:20
tbarronguitarzan: good17:21
*** ducttape_ has quit IRC17:21
*** jwcroppe has joined #openstack-cinder17:21
guitarzantbarron: yep, it's not too bad17:21
*** jwcroppe_ has quit IRC17:25
*** uberjay has quit IRC17:27
*** lprice1 has joined #openstack-cinder17:31
*** cFouts is now known as nug17:31
*** nug is now known as Guest4765217:32
*** lprice has quit IRC17:33
*** jwcroppe has quit IRC17:33
*** sasukeh has joined #openstack-cinder17:34
*** ducttape_ has joined #openstack-cinder17:37
*** plastikman has left #openstack-cinder17:37
*** rlrossit has quit IRC17:37
*** baumann has joined #openstack-cinder17:39
openstackgerritSheel Rana proposed openstack/cinder: WADL to RST migration  https://review.openstack.org/31211317:40
*** sasukeh has quit IRC17:40
*** jwcroppe has joined #openstack-cinder17:41
*** Guest47652 is now known as cFouts17:41
*** lpetrut has quit IRC17:43
openstackgerritMerged openstack/cinder: Use utils.convert_str to convert HTTP header values  https://review.openstack.org/30446217:48
*** uberjay has joined #openstack-cinder17:50
*** prometheanfire has joined #openstack-cinder17:51
*** jwcroppe_ has joined #openstack-cinder17:53
*** merooney has joined #openstack-cinder17:55
*** jwcroppe has quit IRC17:56
*** sdake_ is now known as sdake17:57
*** jgregor has joined #openstack-cinder18:03
*** jwcroppe has joined #openstack-cinder18:03
*** jwcroppe_ has quit IRC18:07
openstackgerritKaitlin Farr proposed openstack/cinder: Replace key manager with Castellan  https://review.openstack.org/28049218:07
openstackgerritKaitlin Farr proposed openstack/cinder: Specify key algorithm and size for create_key  https://review.openstack.org/25150318:07
*** gouthamr has joined #openstack-cinder18:08
*** suyash has joined #openstack-cinder18:08
ameadehttps://review.openstack.org/#/c/299052/ "merge me ^.^"18:09
*** sgotliv has quit IRC18:10
*** nnex has joined #openstack-cinder18:10
*** nnex has quit IRC18:11
ameadehttps://review.openstack.org/#/c/300639/ "me too >.>"18:11
*** gouthamr_ has joined #openstack-cinder18:13
*** mriedem1 has joined #openstack-cinder18:15
*** mriedem has quit IRC18:17
*** gouthamr has quit IRC18:17
*** mriedem1 is now known as mriedem18:20
*** rlrossit has joined #openstack-cinder18:23
*** xinli has quit IRC18:24
*** BigWillie has quit IRC18:24
*** salv-orlando has joined #openstack-cinder18:24
*** gouthamr_ is now known as gouthamr18:26
*** BigWillie has joined #openstack-cinder18:26
*** akapil has quit IRC18:30
*** bwallis has joined #openstack-cinder18:30
gouthamrhi, would like some core reviewer attention on a patch: https://review.openstack.org/#/c/246990/ - it's been there too long and would like to get it merged if possible.18:30
*** sdake_ has joined #openstack-cinder18:31
hemnagouthamr, I don't see any chap settings in the cinder.conf18:31
hemnagouthamr, http://goo.gl/jUXaOF18:31
geguileoguitarzan: Thanks!18:32
geguileoguitarzan: The patch is already under review: https://review.openstack.org/#/c/313584/18:33
*** sdake has quit IRC18:33
guitarzangeguileo: ya, it took me a bit to get set up with prod data18:34
guitarzangeguileo: taking a look now18:34
gouthamrhemna: we're not testing chap on the CI jobs.. we had internal tests for it. i'll find out from akerr if we can do something about that18:35
geguileoguitarzan: I imagined it would take you some time18:35
geguileoguitarzan: Thank you for the effort18:35
guitarzangeguileo: now I just need to ask dulek about the rpc upgrade stuff :)18:35
geguileoguitarzan: What RPC upgrade stuff?18:35
geguileoguitarzan: It will not be a problem18:35
smcginnisI think he took an extended stay in Austin. Probably on his way back now.18:35
*** sasukeh has joined #openstack-cinder18:36
geguileoguitarzan: Because serialization of the UUID is transformed to a string18:36
geguileoguitarzan: And the string is serialized as a string as well18:36
guitarzangeguileo: ya, it's not related to this uuid stuff18:36
geguileoguitarzan: Ok, maybe I can help you18:36
openstackgerritSheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete  https://review.openstack.org/31355018:36
guitarzanmy upgrade from L->master didn't quite work right18:36
guitarzanthe c-vols wouldn't start due to the rpc versioning18:36
geguileoguitarzan: They wouldn't start? r:-??18:36
patrickeasthemna: gouthamr: fwiw we don't make other drivers test both chap enabled and disabled18:37
guitarzanI thought they'd downgrade to L versioning18:37
guitarzanL rpc version18:37
geguileoguitarzan: Or they would start with the wrong RPC version?18:37
guitarzangeguileo: they fail to talk to the scheduler18:37
lprice1DuncanT: Would you mind taking another look at https://review.openstack.org/#/c/246990/ when you get a chance?18:37
geguileoguitarzan: Any exception?18:37
guitarzangeguileo: Requested message version, 2.0 is incompatible.  It needs to be equal in major version and less than or equal in minor version as the specified version cap 1.8.18:37
*** jwcroppe_ has joined #openstack-cinder18:38
*** jwcroppe has quit IRC18:38
geguileoguitarzan: Ok, that's because they are out of sync  :-(18:38
guitarzancan you explain that to me?18:38
guitarzanmaster is trying to speak 2.018:38
guitarzanL is 1.8 no?18:38
geguileoguitarzan: I don't know from the top of my head18:39
*** dramakri has joined #openstack-cinder18:39
guitarzanI think it means the scheduler is speaking 1.818:39
guitarzanmaybe?18:39
guitarzanbut the volume service won't or can't downgrade?18:39
*** e0ne has joined #openstack-cinder18:39
guitarzaneither way, it's not a show stopper for testing this uuid stuff18:39
*** savihou has quit IRC18:40
geguileoguitarzan: How did you do the upgrades?18:40
*** sasukeh has quit IRC18:41
*** catintheroof has quit IRC18:41
guitarzanI just upgraded the source tree, blew away the venv and reinstalled18:41
guitarzancinder-manage db sync18:41
guitarzanand go18:41
geguileoOnly 1 c-sch and 1 c-vol?18:42
guitarzanyes, although the db has tons of service entries from prod18:42
akerrpatrickeast: hemna: gouthamr: yea, its not possible to run and report on all possible permutations of our driver's options.  If we turn chap on then we're not testing when chap is off, etc18:42
guitarzanso it wanting the old rpc version (1.8) doesn't surprise me18:42
*** Lee1092 has quit IRC18:42
guitarzanthat was to be expected I think18:42
geguileoguitarzan: And other entries are alive (as in not deleted)?18:43
guitarzangeguileo: so I have a bunch of "down" c-vols18:43
guitarzanyes, lots18:43
patrickeastakerr: yea i ran into the same thing on my ci, for all patches we just picked the most common configuration and do the others internally18:43
geguileoguitarzan: Ok, then the problem is that you are making it pin to an incompatible version18:43
*** ociuhandu has quit IRC18:43
geguileoguitarzan: It detects it as a rolling upgrade18:43
geguileoguitarzan: From L to N18:43
geguileoguitarzan: Which doesn't work18:43
geguileoguitarzan: Rolling upgrades are from N-1 to N only18:44
guitarzangeguileo: ah, that's interesting18:44
geguileoguitarzan: http://docs.openstack.org/releasenotes/cinder/mitaka.html#known-issues18:44
guitarzangeguileo: next time I'll try stable/liberty -> stable/mitaka just for fun18:44
geguileoguitarzan: dulek included an explanation in Mitaka Release Notes Known issues18:45
guitarzanI'm not sure I think that "Known Issue" really explains the problem18:45
guitarzanthe problem is the n-1 to n18:46
guitarzanassuming you're right, which you most likely are :)18:46
geguileoguitarzan: Well, n-1 to n plus the fact that the service list is not "up to date"18:46
*** mkrcmari__ has quit IRC18:46
openstackgerritSheel Rana proposed openstack/python-cinderclient: Support for cinder backup force delete  https://review.openstack.org/31355018:47
geguileoguitarzan: If you try to do L to M, you may have some issues (need to restart the services again) until this patch merges: https://review.openstack.org/#/c/307105/18:47
*** nnex has joined #openstack-cinder18:47
geguileoguitarzan: If you find any other problems with rolling upgrades and dulek is not around you can also ask me, I've fought it a little bit as well ;-)18:48
guitarzangeguileo: great, will do, thanks18:49
geguileonp18:50
akerrameade: (and anyone else who wants to review user messages tests) https://review.openstack.org/#/c/30063918:51
guitarzangeguileo: no surprse, the uuid warnings went away :)18:51
geguileoXD18:51
geguileoThat's good :-)18:51
jgriffithanybody tried testing with chap lately?  Seems like /etc/iscsi/nodes is being written incorrectly???18:52
jgriffithweird18:52
patrickeastjgriffith: doesn't the gate do iscsi chap w/ lvm?18:53
patrickeastjgriffith: or chap isn't enabled by default?18:53
jgriffithpatrickeast: yeah, and that works which is strange... but I'm wondering if chap is broken18:54
jgriffithpatrickeast: it's my backend only that's seeing a problem it seems18:54
patrickeastmm i tested out my changes to the initiator_data stuff yesterday and it seemed to be ok18:54
jgriffithpatrickeast: I looked at the nodes file and the username and secret entries are garbage18:54
*** bwallis has quit IRC18:54
patrickeastjgriffith: oh weird18:54
jgriffithyeah... really strange18:54
patrickeastjgriffith: got logs of the iscsiadm login commands?18:54
jgriffithpatrickeast: they all just time out18:55
jgriffithpatrickeast: because the credentials are wrong18:55
patrickeastjgriffith: oh, thats not good18:55
jgriffithpatrickeast: but yeah... my 17 failures since last night in sfci are all out there18:55
jgriffithmaybe I'll check a different cluster out and try it18:55
jgriffithreally weird18:55
*** bwallis has joined #openstack-cinder18:56
openstackgerritEvgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs  https://review.openstack.org/26154918:56
openstackgerritEvgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support  https://review.openstack.org/26224618:56
openstackgerritEvgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots of in-use volumes  https://review.openstack.org/27646518:56
openstackgerritEvgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py  https://review.openstack.org/26155818:56
patrickeast jgriffith: the sf ci passed on this one at ~1:30 AM https://review.openstack.org/#/c/313200/18:57
jgriffithpatrickeast: yeah, that's the last pass I believe18:57
jgriffithpatrickeast: oh... hmmm18:57
jgriffithpatrickeast: http://54.164.167.86/solidfire-ci-logs/refs-changes-27-312427-5/logs/c-vol.log.txt18:58
jgriffithI'll have to trace back and see exactly when this started, I did apparanty fill the max number of volumes for my little virt cluster last night18:58
mtaninosheel: hi,18:59
jgriffithbut I'm trying to run this manually right now and am seeing the problem18:59
sheelmtanino: hi18:59
*** jwcroppe has joined #openstack-cinder18:59
mtaninosheel: nit question. Can't we run force-delete when the volume is avairable?18:59
patrickeastjgriffith: hmm well you can see it doing the chap username/password setting commands like usual in that log19:00
mtaninosheel: your comment mentioned above19:00
jgriffithpatrickeast: yeah, it does them, but they're wrong :)19:00
jgriffithgo figure19:00
patrickeasthaha19:00
sheelmtanino: sorry i could not understand your question19:00
*** lprice1 has quit IRC19:01
sheelmtanino: ah ok, i got it19:01
mtaninosheel: line 1466 https://review.openstack.org/#/c/313550/5/cinderclient/v3/shell.py19:01
mtaninosheel: allows deleting ....19:01
sheelmtanino:  actually force delete is specific to states other than available and error19:01
sheelmtanino: this can be run for sure in available state as well19:02
sheelmtanino:  but for Available and error state we can simple run backup delete, then we dont need force delete19:02
*** jwcroppe_ has quit IRC19:02
mtaninosheel: ah, ok. I understand the intention.19:02
sheelmtanino:  cool.. happy to answer...19:03
mtaninosheel: just curious.19:03
sheelmtanino:  no problem, always welcome19:03
*** lprice has joined #openstack-cinder19:03
sheelthanks for review19:03
*** rlrossit has quit IRC19:06
*** asselin has joined #openstack-cinder19:07
mtaninosheel: oh, you edit code on the browser directly instead of using git commit...19:07
mtaninogit review19:07
sheelmtanino:  yep... i was playing with gerrit..19:07
sheelUI19:07
mtaninoseems difficult...19:08
sheel:)19:08
*** dave-mccowan has quit IRC19:09
*** xinli has joined #openstack-cinder19:09
*** e0ne has quit IRC19:11
*** ociuhandu has joined #openstack-cinder19:12
*** nnex has quit IRC19:12
openstackgerritAlex Meade proposed openstack/cinder: Add prefix to user message event ids  https://review.openstack.org/31314019:24
*** jwcroppe has quit IRC19:24
*** martyturner has quit IRC19:24
*** sdake has joined #openstack-cinder19:25
*** dave-mccowan has joined #openstack-cinder19:25
*** sdake has quit IRC19:25
*** rcernin has joined #openstack-cinder19:26
*** ducttape_ has quit IRC19:26
*** sdake_ has quit IRC19:26
*** raunak has quit IRC19:27
*** jwcroppe has joined #openstack-cinder19:28
*** mkrcmari__ has joined #openstack-cinder19:31
*** yangyapeng has joined #openstack-cinder19:31
*** bardia has quit IRC19:31
*** bardia has joined #openstack-cinder19:32
*** yangyapeng has quit IRC19:35
*** sasukeh has joined #openstack-cinder19:36
*** salv-orlando has quit IRC19:39
*** timcl has quit IRC19:40
*** apoorvad has quit IRC19:40
*** sasukeh has quit IRC19:41
*** salv-orlando has joined #openstack-cinder19:42
*** akerr has quit IRC19:43
sheelameade:  hi19:43
sheelameade: I think we need not of upgrade scripts here19:43
sheelameade:  in this patch ^^19:44
*** ducttape_ has joined #openstack-cinder19:45
sheelameade:  as no cinder release is done and impact of name of eventIDs is very less which will later be deleted once expiry time is reached... we need not to add upgrade scripts :)19:45
*** bwallis has quit IRC19:45
*** rlrossit has joined #openstack-cinder19:48
*** xinli has quit IRC19:51
openstackgerritPatrick East proposed openstack/cinder: Specify a user-agent in Pure volume drivers  https://review.openstack.org/31367919:56
*** sage has quit IRC19:58
*** xinli has joined #openstack-cinder19:59
*** sage has joined #openstack-cinder20:00
openstackgerritTom Barron proposed openstack/cinder: Remove deprecated cinder RequestBodySizeLimiter  https://review.openstack.org/31368220:01
*** jgregor has quit IRC20:01
*** dramakrishna has joined #openstack-cinder20:06
*** lprice1 has joined #openstack-cinder20:07
*** mvk_ has joined #openstack-cinder20:08
*** lprice has quit IRC20:09
*** mkrcmari__ has quit IRC20:12
*** salv-orlando has quit IRC20:13
*** apoorvad has joined #openstack-cinder20:13
openstackgerritMerged openstack/cinder-specs: Add Stochastic Weighing Scheduler  https://review.openstack.org/30740120:15
scottdaHow do we fix this:20:15
scottdahttps://www.irccloud.com/pastebin/BaGVcFct/20:16
scottdaCollecting cryptography>=1.1 (from paramiko>=1.16.0->tempest-lib>=0.14.0->-r /home/jenkins/workspace/gate-python-cinderclient-pypy/test-requirements.txt (line 16))20:16
scottdaRuntimeError: cryptography 1.0 is not compatible with PyPy < 2.6. Please upgrade PyPy to use this library.20:16
smcginnis_scottda, eewww20:17
sheelyes I saw this in non voting tests today...20:17
sheelthough this is not blocker but still good to fix20:17
eharneyscottda: we still have a pypy job?20:17
*** dramakri has quit IRC20:18
*** dramakrishna is now known as dramakri20:18
scottdaeharney: Yes, but the job is gate-python-cinderclient-pypy, so maybe we shouldn't have it...20:18
scottdaI reckon the job is to verify packages are in pypy?20:18
smcginnis_I think client is the only think that potentially needs to be compatible with pypy.20:18
smcginnis_pypy, not pypi.20:18
openstackgerritMerged openstack/cinder-specs: Implement the oslo_db enginefacade proposal  https://review.openstack.org/27060120:19
smcginnis_Interpreter.20:19
scottdaI'm not sure, but hemna told me it's pronounced Pee-pee20:19
smcginnis_:D20:19
eharneyIMO if we want to test w/ pypy, we should bump up to a more current version20:20
scottdaWho controls the version? us or infra?20:20
eharneypresumably project-config20:20
smcginnis_It should be _at least_ 2.6 or greater I would think.20:22
eharneywell keep in mind that those aren't related to python versions20:22
smcginnis_True20:23
smcginnis_Trusty ships with 2.2.1. :/20:23
smcginnis_Not sure about RH versions. Nothing on their web site I can see.20:24
eharneyi don't think we use it for anything openstack-related20:24
smcginnis_No, but we have broader compatibility requirements for the client.20:25
eharneyright20:25
smcginnis_But if a critical library doesn't work with the older versions, then I think we either need to raise the minimum...20:25
smcginnis_Or remove support for it.20:26
*** ducttape_ has quit IRC20:26
eharneyi think "critical" is questionable here since it came from paramiko/tempest-lib20:26
eharneythe client doesn't actually _use_ it, right?20:26
*** prometheanfire has left #openstack-cinder20:27
*** garthb_ has joined #openstack-cinder20:27
eharneyjust tempest.lib does20:27
smcginnis_Not sure. Seems like something like cryptography would be necessary internally to paramiko.20:28
eharneyright, but paramiko is not necessary for cinderclient20:28
smcginnis_Oh, I see what you're saying. I guess not.20:29
*** garthb has quit IRC20:29
*** e0ne has joined #openstack-cinder20:29
eharneystill a problem, but a tempest/CI problem is different from a client problem20:29
smcginnis_Nope, not just tests: https://github.com/openstack/cinder/blob/9567844f0e180a7278767fdcd4378b19913933be/cinder/ssh_utils.py#L10320:30
eharneyyeah, we have drivers using paramiko20:30
*** BigWillie has quit IRC20:30
eharneybut if we aren't supporting pypy there then there isn't much to worry about20:30
smcginnis_Derp, just realized I was looking in Cinder and not python-cinderclient. :)20:31
smcginnis_Guess it's getting too late on a Friday for me.20:31
*** dramakri has left #openstack-cinder20:31
scottdaSo, if we're not supporting pypy, should be remove this job?20:31
scottdas/be/we20:32
eharneywell, i think we are (supposedly) supporting it for the client20:32
smcginnis_I think we do need to support pypy. But not sure as far as restricting version.20:32
*** openstackgerrit has quit IRC20:32
*** openstackgerrit has joined #openstack-cinder20:33
eharneyscottda: where is that job?20:33
scottdaI'm still looking for where...20:34
*** dramakri has joined #openstack-cinder20:34
scottdaI don't see it in project-config...20:34
*** dramakri has left #openstack-cinder20:34
eharneyscottda: it's in there as pypy-jobs20:35
smcginnis_https://github.com/openstack-infra/project-config/blob/c4eb24d7a1b6cec506315f90e8bc9c76e370bea6/zuul/layout.yaml#L44120:35
smcginnis_I assume it will just use whatever default for the given platform it runs on.20:35
eharneyand it seems to just install pypy from trusty, which is 2.2.120:36
smcginnis_So if it's trusty thats 2.220:36
eharneyscottda: i meant where did you see this fail?20:36
scottdaeharney: https://review.openstack.org/#/c/313550/20:36
scottdahttp://logs.openstack.org/50/313550/5/check/gate-python-cinderclient-pypy/cf9bbc5/console.html#_2016-05-06_18_58_24_49420:37
*** sasukeh has joined #openstack-cinder20:37
eharneythat check has been in python-cryptography since release 1.120:38
*** e0ne has quit IRC20:39
eharneylast year20:39
sheel:( .. seems no one considered being non voting jobs20:39
*** e0ne has joined #openstack-cinder20:40
*** e0ne has quit IRC20:40
eharneysheel: that doesn't mean it was failing20:40
sheeleharney: right20:40
*** ducttape_ has joined #openstack-cinder20:40
eharneyjust saying that there may not be much to do other than figure out how to get a newer pypy20:40
smcginnis_Paramiko just updated to use the new version 9 days ago.20:40
smcginnis_https://github.com/paramiko/paramiko/commit/1e7849ba910365e6008fd276b271e90a718fd61820:40
eharneyahh20:41
*** dramakri has joined #openstack-cinder20:41
*** dramakri has left #openstack-cinder20:41
eharneywell, nobody's going to want to version cap paramiko for long since it's pretty security-sensitive...20:42
*** sasukeh has quit IRC20:42
*** dramakri has joined #openstack-cinder20:42
scottdaQuite a conundrum ...20:42
scottdaI guess I'll start by logging a bug. I need to be somewhere in 15 minutes, so I'm about done here...20:43
eharneyscottda: yeah, definitely do that20:43
smcginnis_I've asked the question over in infra.20:43
smcginnis_Probably not a big deal for now, but good to get a bug filed for it.20:43
smcginnis_Presumably manually installing a newer pypy would be fine.20:43
smcginnis_But we don't have any test coverage to verify it works in the mean time.20:44
eharneycan we ditch trusty and move to xenial already?20:44
smcginnis_hehe, might be a little premature.20:44
eharneyyeaaaah, i know20:44
smcginnis_I've heard they got jobs testing building the image.20:44
smcginnis_I manually did a tempest run and everything passed for me on it.20:44
smcginnis_But the right answer is to move it all to RHEL, right? ;P20:45
eharneywell duh20:45
smcginnis_:)20:45
eharneybut seriously, does anyone use pypy?20:45
*** cknight has quit IRC20:45
smcginnis_I used it when I had to do some stuff on Windows a long time ago. Does that count? :)20:45
eharneyonly if you ran openstack on it :)20:46
smcginnis_Nope!20:47
*** porrua has quit IRC20:47
smcginnis_Good news, according to infra we will soon be running Newton jobs on 16.04. That may solve the issue. At least for our gate jobs.20:47
scottdahttps://bugs.launchpad.net/python-cinderclient/+bug/157922820:48
openstackLaunchpad bug 1579228 in python-cinderclient "gate-python-cinderclient-pypy fails due to paramiko requirement" [Undecided,New]20:48
smcginnis_scottda, +1, thanks!20:48
scottdanp20:48
*** dramakri has left #openstack-cinder20:51
*** dramakri has joined #openstack-cinder20:53
*** raunak has joined #openstack-cinder20:54
*** jay-mehta has quit IRC20:55
jgriffitheharney: :)20:55
jgriffitheharney: I've been using xenial for a week now without any real issues20:56
jgriffithso on another note20:57
jgriffithas I continue to be horrified with our attach/detach workflow :(20:57
hemnawell, on the upside, you are horrified with the rest of us :)20:58
jgriffithWhat do you guys think about just ditching all this nonsense, have a "volume_attach" and "volume_detach" API call and let it just do *everything* and skip this back and forth?  I think hemna may have mentioned something like that at one point20:58
jgriffithhemna: LOL... misery loves company right? :)20:59
hemnahehe, yes it does20:59
jgriffithhemna: it's funny.. I keep looking at git logs asking "how did we get here"20:59
smcginnis_That should theoretically take care of some races and make things simpler if we could.20:59
hemnahehe20:59
hemnaouch20:59
jgriffithhemna: amazing how all these little changes/hacks add up over a few years, and there's a TON of contributions20:59
hemnalaughing through the tears20:59
smcginnis_The simplest we can make the interaction, the better off we'll all be.21:00
smcginnis_But no simpler.21:00
jgriffithsmcginnis_: I can certainly do it, just don't know if it will fly21:00
smcginnis_If it can get rid of code, I think we can convince folks.21:00
hemnaso, yah I think that's what I had hoped at one point, but man it's a big change21:00
smcginnis_Microversions solves everything!!21:00
smcginnis_:)21:00
jgriffithsmcginnis_: to start we could just make attach suck in the initialize_connection and attach methods.  Make them internal21:01
smcginnis_+121:01
scottda++ I've been for that all along.21:01
jgriffithwell, I have my patch almost ready (still need to try and run unit tests which scares me)21:01
hemnapush it up to gerrit....let's all watch21:02
jgriffithit seems to work.. but honestly with all the bull shit that's been added around metadata, optional args etc I'm still not over joyed21:02
hemna:)21:02
smcginnis_Just don't run them with pypy.21:02
jgriffithlol21:02
jgriffithit's ok, I'm on xenial remember :)21:02
smcginnis_Oh yeah. Then carry on.21:02
smcginnis_:)21:02
smcginnis_I've had X on my laptop since release day. I like!21:02
jgriffithsmcginnis_: yeah, I've been pretty happy thus far21:03
jgriffithno surprises/problems21:03
jgriffithseems fairly speedy, can't complain21:03
SwansonHow long has 16.04 lts been out?21:04
smcginnis_My machine actually seems to run better. Trackpad support is far better, which was my biggest complaint before.21:04
smcginnis_Swanson, 2 weeks?21:04
SwansonCool.  Not too late to the party.21:05
*** jungleboyj has joined #openstack-cinder21:07
*** raunak has quit IRC21:09
*** raunak has joined #openstack-cinder21:09
*** dramakri has left #openstack-cinder21:10
*** rlrossit has quit IRC21:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376621:10
openstackgerritPatrick East proposed openstack/cinder: Specify a user-agent in Pure volume drivers  https://review.openstack.org/31367921:11
*** dramakri1 has joined #openstack-cinder21:13
*** ducttape_ has quit IRC21:14
*** dramakri1 has quit IRC21:14
patrickeasthas anyone pinged the openvstorage ci people?21:15
patrickeastit has been borked for a while now21:16
*** xyang1 has quit IRC21:16
*** dramakri has joined #openstack-cinder21:20
smcginnis_I have not.21:20
*** dustins has quit IRC21:21
smcginnis_I think DuncanT said he was going to start reaching out to some of the failing ones.21:21
*** dramakri has left #openstack-cinder21:21
*** baumann has left #openstack-cinder21:23
*** eharney has quit IRC21:24
*** ametts has quit IRC21:24
openstackgerritTom Barron proposed openstack/cinder: Remove deprecated cinder RequestBodySizeLimiter  https://review.openstack.org/31368221:25
jgriffithhey!  That's not bad... 29 failures21:28
jgriffiththat's actually pretty good21:28
jgriffithand those appear to all be because of a missing key in their fakes21:28
jgriffithhuh.. who'd a thunk21:28
*** vilobhmm11 has joined #openstack-cinder21:29
*** vilobhmm11 has quit IRC21:30
*** vilobhmm11 has joined #openstack-cinder21:30
*** lprice1 has quit IRC21:31
*** vilobhmm11 has quit IRC21:33
*** merooney has quit IRC21:33
*** sasukeh has joined #openstack-cinder21:38
*** cknight has joined #openstack-cinder21:43
*** jwcroppe has quit IRC21:43
*** mriedem has quit IRC21:44
*** dramakri has joined #openstack-cinder21:44
*** sasukeh has quit IRC21:46
*** bhbhavsa_ has quit IRC21:46
mtaninojgriffith: hi! Do you want Cheesecake fix? https://review.openstack.org/#/c/313124/221:50
mtaninowaiting someone eat it :)21:50
*** salv-orlando has joined #openstack-cinder21:51
*** ozialien10 has joined #openstack-cinder21:51
*** xinli has quit IRC21:52
jgriffithmtanino: nothing better on a Friday than a good dessert :)21:52
mtaninojgriffith: +1!21:54
openstackgerritMerged openstack/cinder: Use to_utf8() instead of safe_encode() in convert_str()  https://review.openstack.org/31352121:59
mtaninojgriffith: Thank you for eating quickly :)22:01
jgriffithmtanino: haha :)22:01
jgriffithmtanino: thanks as always for helping out!22:01
mtaninojgriffith: sure :)22:02
patrickeastjgriffith: mtanino: maybe dumb question, but adding the policy checks would mean thats potentially a behavior change right? Does that need a micro-version?22:11
patrickeastscottda: ^22:12
mtaninopatrickeast: jgriffith scottda I'd like to know about it too.22:12
jgriffithpatrickeast: oh god no, please22:13
patrickeasthaha22:13
mtaninopatrickeast: In my feeling, it doesn't change behavior of API itself, so I think we don't need microversion.22:13
jgriffithIt's not really a behavior change IMO22:13
patrickeastgood enough for me22:13
jgriffithit's a config option/change22:13
jgriffithchange in the default config22:14
mtaninoAlso cloud admin can change that config too22:14
patrickeastyea I tend to think its fine22:14
patrickeastfigured i'd check22:14
jgriffithpatrickeast: I owe you a beer :)22:14
mtanino!!22:14
openstackmtanino: Error: "!" is not a valid command.22:14
mtaninosweet.22:14
jgriffithlol22:14
jgriffithmtanino: found a new easter egg!22:14
jgriffith!!22:14
openstackjgriffith: Error: "!" is not a valid command.22:14
jgriffith??22:14
jgriffithAhh... I see what the bot did there22:15
jgriffith!eat cheescake22:15
openstackjgriffith: Error: "eat" is not a valid command.22:15
mtaninowow..22:16
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/31371622:16
patrickeasthaha22:16
patrickeast!help22:16
openstackpatrickeast: (help [<plugin>] [<command>]) -- This command gives a useful description of what <command> does. <plugin> is only necessary if the command is in more than one plugin.22:16
mtaninohelp is valid command!22:16
mtaninohelp me22:16
hemna!poop22:16
openstackhemna: Error: "poop" is not a valid command.22:16
hemnadamn22:17
patrickeastlol22:17
hemnayes, I'm 6 years old.22:17
jgriffithlol22:18
*** vilobhmm11 has joined #openstack-cinder22:19
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/31374722:21
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/31375222:22
thingeeglad I'm not the only one22:22
*** jungleboyj has quit IRC22:30
*** david-lyle has quit IRC22:35
*** david-lyle has joined #openstack-cinder22:37
*** sdake has joined #openstack-cinder22:45
*** takedakn1 has joined #openstack-cinder22:46
*** markvoelker has quit IRC22:47
*** sdake_ has joined #openstack-cinder22:48
*** takedakn has quit IRC22:48
*** takedakn1 is now known as takedakn22:48
*** markvoelker has joined #openstack-cinder22:48
*** lcurtis has quit IRC22:50
*** sdake has quit IRC22:50
*** adrianofr has quit IRC22:52
*** r-daneel has quit IRC22:52
*** sdake has joined #openstack-cinder22:55
*** sdake_ has quit IRC22:56
*** tpsilva has quit IRC22:57
*** karthikp has quit IRC23:01
*** martyturner has joined #openstack-cinder23:03
*** sdake has quit IRC23:05
*** yangyapeng has joined #openstack-cinder23:07
openstackgerritMerged openstack/cinder: Add Cheesecake APIs to policy.json file  https://review.openstack.org/31312423:11
*** yangyapeng has quit IRC23:12
*** martyturner has quit IRC23:21
*** sheel has quit IRC23:25
*** rodrigods has joined #openstack-cinder23:29
rodrigodshi... quick question about keystone v3 support23:29
rodrigodsanyone remembers which cycle it happened?23:29
*** sdake has joined #openstack-cinder23:34
*** edmondsw has quit IRC23:34
*** akshai has quit IRC23:35
*** sdake_ has joined #openstack-cinder23:37
*** leeantho_ has quit IRC23:38
*** sdake has quit IRC23:38
*** garthb_ has quit IRC23:38
*** arch-nemesis has quit IRC23:42
*** sasukeh has joined #openstack-cinder23:42
*** sasukeh has quit IRC23:47
*** bardia has quit IRC23:50
*** merooney has joined #openstack-cinder23:51
*** salv-orlando has quit IRC23:56
*** akshai has joined #openstack-cinder23:57
*** akshai_ has joined #openstack-cinder23:58

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