Monday, 2017-12-11

*** masber has joined #openstack-cinder00:07
*** gouthamr has quit IRC00:09
*** yangyapeng has joined #openstack-cinder00:19
*** yangyapeng has quit IRC00:24
*** wanghao_ has quit IRC00:29
*** wanghao has joined #openstack-cinder00:29
*** sbezverk has quit IRC00:40
*** salv-orlando has joined #openstack-cinder00:41
*** Dinesh_Bhor has joined #openstack-cinder00:46
*** salv-orlando has quit IRC00:46
winston-djgriffith, ildikov: Congrats! 6 months + ~180 iterations. That's awful lot of work and determinations.00:47
*** psachin has joined #openstack-cinder00:50
*** Dinesh_Bhor has quit IRC00:52
*** Dinesh_Bhor has joined #openstack-cinder00:52
*** hoangcx has joined #openstack-cinder00:54
*** Apoorva has quit IRC00:58
*** linkmark has quit IRC00:59
*** salv-orlando has joined #openstack-cinder01:00
ildikovwinston-d: and that's just the top of the iceberg :)01:05
*** salv-orlando has quit IRC01:05
*** yangyapeng has joined #openstack-cinder01:07
*** gouthamr has joined #openstack-cinder01:08
winston-dildikov: https://i.imgflip.com/x5twc.jpg01:11
ildikovwinston-d: :)01:18
ildikovwinston-d: in that sense too, thanks for recognizing it!01:18
*** r-daneel has joined #openstack-cinder01:21
*** zhaochao has joined #openstack-cinder01:42
*** asettle_ is now known as asettle01:45
winston-d:)01:49
*** sapd_ has joined #openstack-cinder01:50
*** sapd__ has quit IRC01:50
*** dalgaaf has quit IRC01:51
*** dalgaaf has joined #openstack-cinder01:52
*** Apoorva has joined #openstack-cinder01:59
*** wanghao_ has joined #openstack-cinder02:05
*** wanghao has quit IRC02:08
*** openstackgerrit has joined #openstack-cinder02:10
openstackgerritNguyen Van Trung proposed openstack/cinder-specs master: Update bindep.txt for doc builds  https://review.openstack.org/52241802:10
*** markvoelker has quit IRC02:13
*** dave-mccowan has joined #openstack-cinder02:13
*** dave-mcc_ has joined #openstack-cinder02:15
*** dave-mccowan has quit IRC02:19
*** felipemonteiro_ has quit IRC02:22
*** dave-mccowan has joined #openstack-cinder02:33
*** dave-mcc_ has quit IRC02:35
*** psachin has quit IRC02:49
vivsoniHi Sean & Team, dirverfixes/ocata branch is broken.02:53
vivsonizuul check, 'openstack-tox-py27' is failing with Error "AttributeError: 'module' object has no attribute 'SSL_ST_INIT'"02:53
vivsoniMay i request you to please check02:54
*** zhurong has joined #openstack-cinder02:54
*** psachin has joined #openstack-cinder03:02
*** namnh has joined #openstack-cinder03:02
*** liverpooler has joined #openstack-cinder03:03
vivsonihttp://logs.openstack.org/19/525719/1/check/openstack-tox-py27/f0c6813/job-output.txt.gz#_2017-12-05_18_35_28_79457403:05
*** bkopilov has quit IRC03:14
*** Apoorva has quit IRC03:18
tommylikehuhey vivsoni  is this failure relates to the cryptography release version?03:29
vivsoniseems to be of cryptography latest version03:31
*** vivsoni has quit IRC03:36
*** david-lyle has joined #openstack-cinder03:38
*** dave-mccowan has quit IRC03:41
*** liverpooler has quit IRC03:41
*** abhishekk has joined #openstack-cinder03:42
tommylikehuvisoni: does it work if we switch cryptography from 2.1.4 to 2.1.3?03:43
*** gouthamr has quit IRC03:45
*** david-lyle has quit IRC03:45
*** vivsoni has joined #openstack-cinder03:52
*** liverpooler has joined #openstack-cinder04:06
*** links has joined #openstack-cinder04:07
*** links has quit IRC04:07
*** markvoelker has joined #openstack-cinder04:13
*** zhurong has quit IRC04:26
openstackgerritMerged openstack/cinder master: Fix create from snapshot and source volume  https://review.openstack.org/52658104:29
*** masuberu has joined #openstack-cinder04:30
*** masber has quit IRC04:33
*** jmlowe has quit IRC04:33
*** masuberu has quit IRC04:36
*** armax has joined #openstack-cinder04:36
*** armax has quit IRC04:41
*** markvoelker has quit IRC04:48
*** bkopilov has joined #openstack-cinder04:58
*** Dinesh_Bhor has quit IRC05:01
*** pgadiya has joined #openstack-cinder05:05
*** pgadiya has quit IRC05:05
*** pgadiya has joined #openstack-cinder05:11
*** pgadiya has quit IRC05:11
*** liverpooler has quit IRC05:24
*** liverpooler has joined #openstack-cinder05:25
*** zhurong has joined #openstack-cinder05:32
*** lkuchlan has joined #openstack-cinder05:42
*** masber has joined #openstack-cinder05:42
*** markvoelker has joined #openstack-cinder05:45
*** Dinesh_Bhor has joined #openstack-cinder05:48
*** lkuchlan has quit IRC06:13
*** threestrands has quit IRC06:17
*** bhagyashri_s is now known as bhagyashris06:17
*** markvoelker has quit IRC06:19
openstackgerritChris Yang proposed openstack/cinder master: QNAP: Add support for QES 2.0.0  https://review.openstack.org/52699406:24
openstackgerritEric Harney proposed openstack/cinder master: Deprecate cinder-manage logs commands  https://review.openstack.org/52692806:35
*** ircuser-1 has joined #openstack-cinder06:42
*** abishop has joined #openstack-cinder06:43
*** pooja_jadhav has joined #openstack-cinder06:43
*** pooja-jadhav has joined #openstack-cinder06:43
*** pooja-jadhav has quit IRC06:43
*** bkopilov has quit IRC06:45
*** zhurong has quit IRC06:50
*** moshele has joined #openstack-cinder06:52
*** bkopilov has joined #openstack-cinder07:04
*** salv-orlando has joined #openstack-cinder07:08
*** imran_ansari has joined #openstack-cinder07:10
*** josecastroleon has joined #openstack-cinder07:10
*** vivsoni has quit IRC07:10
*** kevinz has joined #openstack-cinder07:13
kevinzcinder create --allow-multiattach  --display-name my-volume2 507:13
kevinzHi All07:13
kevinzGood morning Cinder07:14
kevinzI've tried to use cinder multiple attach. When create with command:07:14
kevinzcinder create --allow-multiattach  --display-name my-volume2 507:14
kevinzI find the volume status of the created one is error. Do I miss some essential procedure?07:15
*** markvoelker has joined #openstack-cinder07:16
*** hoonetorg has quit IRC07:18
*** lkuchlan has joined #openstack-cinder07:20
*** adisky_ has joined #openstack-cinder07:20
*** namnh has quit IRC07:29
*** namnh_ has joined #openstack-cinder07:29
*** hoonetorg has joined #openstack-cinder07:31
winston-dkevinz: And if you do 'create create --display-name my-volume3 5', the result would be different?07:40
kevinzwinston-d: Yeah, the volume status is "error"07:40
kevinzwinston-d: Thanks for your help. I ues LVM backend. Does it not support?07:41
winston-dsorry, i'm confused, 'my-volume3' status would also be 'error'?07:41
kevinzwinston-d: Yes. I've checked with c-sch, I find "Failed to run task cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid backend was found. No weighed backends available:"07:42
winston-dSo it has nothing to do with multi-attach then.07:42
winston-dDid you check your c-vol service and see if it is working properly?07:43
*** rcernin has quit IRC07:43
*** salv-orlando has quit IRC07:44
*** salv-orlando has joined #openstack-cinder07:44
kevinzwinston-d: Thanks I've added here http://paste.openstack.org/show/628577/. I don't find any difference in c-vol07:46
*** AlexeyAbashkin has joined #openstack-cinder07:46
*** salv-orlando has quit IRC07:48
*** markvoelker has quit IRC07:49
*** moshele has quit IRC07:50
*** imran_ansari has quit IRC07:52
*** salv-orlando has joined #openstack-cinder07:56
*** arnewiebalck_ has quit IRC08:12
*** vivsoni has joined #openstack-cinder08:12
kevinzHi All, what's the volume drivers that could support multiattach volume?08:13
kevinzYour response will be really appreciated!08:14
kevinzI'm working about cinder multiple attach with Zun(container management service for OpenStack)08:14
*** arnewiebalck has joined #openstack-cinder08:15
*** alexchadin has joined #openstack-cinder08:19
*** gkadam has joined #openstack-cinder08:27
*** salv-orlando has quit IRC08:39
*** hoangcx has quit IRC08:41
*** vivsoni has quit IRC08:43
*** vivsoni has joined #openstack-cinder08:43
*** amoralej|off is now known as amoralej08:45
*** markvoelker has joined #openstack-cinder08:46
*** hoangcx has joined #openstack-cinder08:46
*** rcernin has joined #openstack-cinder08:47
*** vivsoni has quit IRC08:50
*** salv-orlando has joined #openstack-cinder09:01
*** e0ne has joined #openstack-cinder09:02
*** Jack_Iv has joined #openstack-cinder09:07
*** vivsoni has joined #openstack-cinder09:12
*** markvoelker has quit IRC09:20
*** sticker has quit IRC09:25
*** abishop has quit IRC09:31
*** wanghao_ has quit IRC09:32
*** Dinesh_Bhor has quit IRC09:43
*** ludovic_ has joined #openstack-cinder09:45
ludovic_someone is familiar with networking issue like urllib3 connection pool full in nova or cinder ?09:45
pooja_jadhavtommylikehu :Hi09:48
tommylikehupooja_jadhav:  hi09:48
tommylikehuhey kevinz  lvm doesn't support multi attach: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/lvm.py#L27909:48
kevinzhey <tommylikehu>09:49
kevinztommylikehu: Thanks very much! I wonder which driver support multiple attach now ?09:49
ludovic_Maybe someone know why with volume terminate-connection i have a lot of urllib3 connection pool full09:50
tommylikehukevinz:  where are you going to use those volumes if you create one?09:50
ludovic_it comes from cinder client apparently09:50
kevinztommylikehu: I'm developer in Zun project, want to make container support multiple attach volume09:51
kevinztommylikehu: Here is the bp: https://blueprints.launchpad.net/zun/+spec/cinder-volume-multi-attach09:51
tommylikehukevinz:  thanks09:52
*** finucannot is now known as stephenfin09:52
kevinztommylikehu: Just nothing to do with Nova, we want to do the same as Nova does to make container multiple attach volume09:52
tommylikehukevinz:  that's the link for new attach/detach APIs09:53
tommylikehukevinz:  I can't find any driver who advertise its support of multi attach in cinder's repos. maybe you can ask jgriffith or ildikov for help they would be around in 2-3 hours09:57
kevinztommylikehu: Yeah. The info here is just for reference. Actually we are not sure which API is workable to realize the Multiple attach09:57
kevinztommylikehu: OK, thanks a lot. I will ping them after later:-)09:58
pooja_jadhavtommylikehu : for volume snapshot API, in V1 there are only 2 values get accepted while updating snapshot like(display_name and display_description) [1] https://github.com/openstack/cinder/blob/driverfixes/mitaka/cinder/api/v1/snapshots.py#L221-L22409:58
pooja_jadhavbut for v2 and v3 valid_update_keys are: 'name', 'display_name', 'description', 'display_description' [2]09:58
pooja_jadhavhttps://github.com/openstack/cinder/blob/driverfixes/mitaka/cinder/api/v2/snapshots.py#L22009:58
*** gmann is now known as gmann_afk09:58
tommylikehukevinz:  you'd better use the new style one:)09:59
pooja_jadhavtommylikehu : Also in api ref for v3 valid_update_keys are 'name' and 'description' only [1]https://developer.openstack.org/api-ref/block-storage/v3/index.html#volumes-volumes. So my question is their any need of 'display_description' and 'display_name' parameters for v3.09:59
kevinztommylikehu: Yes definitely10:00
*** ganso has joined #openstack-cinder10:02
*** ludovic_ has quit IRC10:07
tommylikehuhey pooja_jadhav  I guess both 'name' and 'display_name' are validated in the volume&snapshot V3 controllers , you should add validation for both of them in your jsonschema patch10:11
tommylikehusame for description10:11
*** kevinz has quit IRC10:17
tommylikehupooja_jadhav: display_* is not described in the api ref, but we still can use it.10:19
pooja_jadhavtommylikehu: ohhhk10:34
pooja_jadhavtommylikehu: thanks for info. :)10:35
*** jmlowe has joined #openstack-cinder10:38
*** abhishekk has quit IRC10:39
*** adisky_ has quit IRC10:40
*** alexchadin has quit IRC10:42
*** armaan has joined #openstack-cinder10:46
*** erlon has joined #openstack-cinder10:48
*** alexchadin has joined #openstack-cinder10:52
*** sdague has joined #openstack-cinder10:53
*** alexchadin has quit IRC10:56
*** alexchadin has joined #openstack-cinder10:57
*** abhishekk has joined #openstack-cinder10:59
*** abhishekk has quit IRC10:59
*** namnh_ has quit IRC11:03
*** moshele has joined #openstack-cinder11:04
*** Jack_Iv has quit IRC11:08
openstackgerritMerged openstack/python-cinderclient master: Remove 'end_version' parameter in backup update  https://review.openstack.org/52443211:08
*** stakeda has quit IRC11:11
*** markvoelker has joined #openstack-cinder11:17
*** Jack_Iv has joined #openstack-cinder11:18
*** Jack_Iv has quit IRC11:23
*** AlexeyAbashkin has quit IRC11:25
*** salv-orlando has quit IRC11:26
*** salv-orlando has joined #openstack-cinder11:27
*** AlexeyAbashkin has joined #openstack-cinder11:45
*** liverpooler has quit IRC11:48
*** linkmark has joined #openstack-cinder11:49
*** markvoelker has quit IRC11:50
*** kwathore has quit IRC11:54
*** kwathore has joined #openstack-cinder11:54
*** lpetrut has joined #openstack-cinder11:57
bhagyashristommylikehu: hi12:03
*** nicolasbock has joined #openstack-cinder12:05
*** dave-mccowan has joined #openstack-cinder12:06
*** Jack_Iv has joined #openstack-cinder12:08
*** dave-mcc_ has joined #openstack-cinder12:13
*** lhx_ has joined #openstack-cinder12:15
*** dave-mccowan has quit IRC12:16
*** lhx_ has quit IRC12:16
*** lhx_ has joined #openstack-cinder12:16
*** Jack_Iv_ has joined #openstack-cinder12:23
*** alexchadin has quit IRC12:27
*** Jack_Iv has quit IRC12:27
*** Roamer` has joined #openstack-cinder12:27
*** alexchadin has joined #openstack-cinder12:27
pooja_jadhavtommylikehu: Hi12:34
pooja_jadhavtommylikehu: I have checked data base table for workers api (Table name: workers) and the developers doc [1] as well but I saw inconsistency in parameters: What I observed is the 'status' and 'race_preventer' columns are there in db but I haven't saw this fileds in the doc [1] and code as well so do you have any idea why those fileds are added. [1]https://developer.openstack.org/api-ref/block-storage/v3/#workers-workers.12:36
*** salv-orlando has quit IRC12:38
*** salv-orlando has joined #openstack-cinder12:40
*** markvoelker has joined #openstack-cinder12:47
*** amoralej is now known as amoralej|lunch12:51
*** psachin has quit IRC12:53
*** lkuchlan has quit IRC12:57
*** lennyb has quit IRC13:00
*** lennyb has joined #openstack-cinder13:02
*** ChanServ has quit IRC13:08
*** ChanServ has joined #openstack-cinder13:18
*** barjavel.freenode.net sets mode: +o ChanServ13:18
*** yangyapeng has quit IRC13:19
*** yangyapeng has joined #openstack-cinder13:20
*** markvoelker has quit IRC13:21
*** lhx_ has quit IRC13:21
*** rcernin has quit IRC13:23
*** yangyapeng has quit IRC13:25
*** moshele has quit IRC13:28
*** moshele has joined #openstack-cinder13:29
*** kevzha01 has joined #openstack-cinder13:33
*** dustins has joined #openstack-cinder13:35
*** markvoelker has joined #openstack-cinder13:38
openstackgerritNiraj Singh proposed openstack/cinder master: Add service_token for cinder-glance interaction  https://review.openstack.org/52661113:41
openstackgerritNiraj Singh proposed openstack/cinder master: Add service_token for cinder-nova interaction  https://review.openstack.org/52449713:48
openstackgerritNiraj Singh proposed openstack/cinder master: Add service_token for cinder-glance interaction  https://review.openstack.org/52661113:48
*** zhonghua has quit IRC13:49
*** zhonghua has joined #openstack-cinder13:50
*** liverpooler has joined #openstack-cinder13:51
*** amoralej|lunch is now known as amoralej13:51
*** lhx_ has joined #openstack-cinder13:52
*** mriedem has joined #openstack-cinder13:55
*** yangyapeng has joined #openstack-cinder13:57
*** sbezverk has joined #openstack-cinder13:59
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix available space checking for image conversion  https://review.openstack.org/52709714:08
*** alexchadin has quit IRC14:16
*** alexchadin has joined #openstack-cinder14:16
*** alexchadin has quit IRC14:17
*** alexchadin has joined #openstack-cinder14:17
*** alexchadin has quit IRC14:17
*** lkuchlan has joined #openstack-cinder14:22
*** yangyapeng has quit IRC14:24
*** gcb has joined #openstack-cinder14:24
*** yangyapeng has joined #openstack-cinder14:25
*** gouthamr has joined #openstack-cinder14:26
*** josecastroleon1 has joined #openstack-cinder14:29
*** josecastroleon has quit IRC14:29
*** erlon has quit IRC14:38
*** kevzha01_ has joined #openstack-cinder14:41
*** superdan is now known as dansmith14:42
*** kevzha01 has quit IRC14:44
*** alexchadin has joined #openstack-cinder14:45
*** alexchadin has quit IRC14:48
*** david-lyle has joined #openstack-cinder14:51
*** felipemonteiro_ has joined #openstack-cinder14:55
*** felipemonteiro__ has joined #openstack-cinder14:56
*** shaner has quit IRC14:59
*** kevzha01_ has quit IRC14:59
*** felipemonteiro_ has quit IRC15:00
*** kevzha01_ has joined #openstack-cinder15:04
*** shaner has joined #openstack-cinder15:05
*** stelucz has quit IRC15:08
openstackgerritEric Harney proposed openstack/cinder master: Deprecate cinder-manage logs commands  https://review.openstack.org/52692815:09
*** armaan has quit IRC15:11
*** armaan has joined #openstack-cinder15:12
*** armaan has quit IRC15:12
*** armaan has joined #openstack-cinder15:13
*** alexchadin has joined #openstack-cinder15:17
*** r-daneel has quit IRC15:18
*** moshele has quit IRC15:21
*** alexchadin has quit IRC15:21
*** vivsoni has quit IRC15:25
*** moshele has joined #openstack-cinder15:28
*** armaan has quit IRC15:28
tommylikehuhey pooja_jadhav , I guess we don't need to consider 'status' and 'race_preventer' as they are for internal use  now :)15:30
*** moshele has quit IRC15:32
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Fix available space checking for image conversion  https://review.openstack.org/52709715:33
tommylikehupooja_jadhav: and geguileo knows this better, if he is available now :)15:35
*** alexchadin has joined #openstack-cinder15:40
*** armaan has joined #openstack-cinder15:44
*** alexchadin has quit IRC15:44
*** gcb has quit IRC15:44
*** abhi89 has joined #openstack-cinder15:45
*** josecastroleon1 has quit IRC15:45
abhi89Hi all.. can some folks please review https://review.openstack.org/#/c/525071/15:45
*** Swimingly has quit IRC15:46
*** mvk has quit IRC15:46
*** vivsoni_ has joined #openstack-cinder15:46
jungleboyjabhi89:  Looking.15:46
*** Swimingly has joined #openstack-cinder15:47
*** Swimingly has joined #openstack-cinder15:47
*** yangyapeng has quit IRC15:49
jungleboyjabhi89:  How can this be right?  You are never decrementing the GB's if they are quotaed :  https://review.openstack.org/#/c/525071/1/cinder/volume/manager.py15:49
*** josecastroleon has joined #openstack-cinder15:51
*** alexchadin has joined #openstack-cinder15:56
*** armax has joined #openstack-cinder15:56
*** r-daneel has joined #openstack-cinder15:56
*** markstur has joined #openstack-cinder15:59
*** alexchadin has quit IRC15:59
*** mchlumsky has quit IRC15:59
*** _alastor_ has joined #openstack-cinder16:00
*** mchlumsky has joined #openstack-cinder16:01
*** gouthamr has quit IRC16:01
*** gnufied has joined #openstack-cinder16:09
*** josecastroleon has quit IRC16:09
*** dave-mcc_ is now known as dave-mccowan16:11
*** Apoorva has joined #openstack-cinder16:12
*** lkuchlan has quit IRC16:15
*** bkopilov has quit IRC16:17
*** lhx__ has joined #openstack-cinder16:18
*** lhx_ has quit IRC16:18
e0nedoes anybody know how to get the list of the slowest tests now?16:20
e0newe lost this information since 'stestr' transition:(16:20
smcginnisI thought we still had that in the output?16:21
smcginnisHuh, I guess not.16:21
e0nesmcginnis: we haven't http://logs.openstack.org/94/526994/1/check/openstack-tox-py27/d433ca5/job-output.txt.gz16:22
smcginnise0ne: Looks like you can manually run "stestr slowest" after the test is run.16:24
e0newe've got 5(five!) tests slower than 10 seconds:(16:24
e0nesmcginnis: thank you16:25
smcginnisWow, that's bad!16:25
e0netop slowest tests on my desktop http://paste.openstack.org/show/628631/16:25
openstackgerritSean McGinnis proposed openstack/cinder master: Add output of slowest tests to UT runs  https://review.openstack.org/52714916:26
smcginnisDatacore must be missing some mocks of sleeps in there for it to be that bad.16:27
e0neyea, it looks like my semi-annual task to speed-up UTs16:28
smcginnise0ne: :)16:29
e0ne:)16:29
abhi89jungleboyj: so initially, when capturing a vm we used to reserve gb quota twice (once when we create the group snapshot & once when we actually create the volumes).. so then i changed the code such that we should not increment/reserve the quota twice.. only reserve when creating the group snapshot16:29
*** hemna_ has joined #openstack-cinder16:31
abhi89jungleboyj: so once the capturing of a vm is done, then we delete the group snapshot that was created.. at this time we don't have to decrement the gb quota..16:31
abhi89jungleboyj: coz actually while creating a capture we have used that much gb..16:31
*** alexchadin has joined #openstack-cinder16:32
abhi89initially we deecremented coz we had incremented twice16:32
abhi89https://review.openstack.org/#/c/517725/ is the previous changeset which should ideally have the change which i am making now in https://review.openstack.org/#/c/525071/16:33
*** crose has joined #openstack-cinder16:33
jungleboyjOk.  Quota code makes my head hurt.16:34
jungleboyjabhi89: If you can update the commit with a little more explanation that would help and then we should be good.16:35
*** alexchadin has quit IRC16:36
abhi89jungleboyj: ok, i will try to make the commit message more clear.. always bad with explaning things in commit msg..16:37
jungleboyjabhi89:  No worries.  Thanks for addressing it.16:38
*** thiagolib has joined #openstack-cinder16:39
e0nesmcginnis: thanks for the patch! I would like to merge it even if CI didn't vote16:42
smcginnise0ne: :)16:42
*** Apoorva has quit IRC16:42
smcginnise0ne: I'm actually working on a path to stestr right now to add it as an option so we don't have to call two separate commands.16:42
smcginnise0ne: That will probably take a little while before we can use that though, so this will at least get us the output for now.16:43
e0nesmcginnis: sounds reasonable to me16:43
*** AlexeyAbashkin has quit IRC16:43
*** vivsoni_ has quit IRC16:44
*** kevzha01_ has quit IRC16:44
*** lpetrut has quit IRC16:51
*** zhonghua has quit IRC16:52
*** zhonghua has joined #openstack-cinder16:52
*** moshele has joined #openstack-cinder16:52
*** Jack_Iv_ has quit IRC16:54
*** Jack_Iv has joined #openstack-cinder16:55
*** vivsoni_ has joined #openstack-cinder16:55
*** yangyapeng has joined #openstack-cinder16:59
*** moshele has quit IRC17:00
*** yangyapeng has quit IRC17:03
*** salv-orlando has quit IRC17:04
*** jmlowe has quit IRC17:05
*** Jack_Iv_ has joined #openstack-cinder17:06
*** alexchadin has joined #openstack-cinder17:07
*** josecastroleon has joined #openstack-cinder17:07
*** jmlowe has joined #openstack-cinder17:08
*** Jack_Iv has quit IRC17:09
*** alexchadin has quit IRC17:09
*** armaan has quit IRC17:17
*** armaan has joined #openstack-cinder17:18
*** alexchadin has joined #openstack-cinder17:23
*** lhx__ has quit IRC17:24
*** alexchadin has quit IRC17:25
*** salv-orlando has joined #openstack-cinder17:27
*** crose has quit IRC17:30
*** gkadam has quit IRC17:30
*** vivsoni_ has quit IRC17:31
*** alexchadin has joined #openstack-cinder17:39
*** alexchadin has quit IRC17:42
*** xyang1 has joined #openstack-cinder17:43
*** Apoorva has joined #openstack-cinder17:45
*** Apoorva has quit IRC17:45
*** Apoorva has joined #openstack-cinder17:46
*** diablo_rojo has joined #openstack-cinder17:47
*** yangyapeng has joined #openstack-cinder17:49
*** yangyapeng has quit IRC17:53
*** e0ne has quit IRC17:56
*** bkopilov has joined #openstack-cinder18:04
*** armaan has quit IRC18:06
*** crose has joined #openstack-cinder18:16
*** crose has quit IRC18:22
*** salv-orlando has quit IRC18:26
*** salv-orlando has joined #openstack-cinder18:27
*** AlexeyAbashkin has joined #openstack-cinder18:28
*** Jack_Iv_ has quit IRC18:28
*** salv-orlando has quit IRC18:31
*** AlexeyAbashkin has quit IRC18:32
*** abhi89 has quit IRC18:35
*** Jack_Iv has joined #openstack-cinder18:38
*** abhi89 has joined #openstack-cinder18:38
*** Jack_Iv has quit IRC18:43
*** yangyapeng has joined #openstack-cinder18:49
*** jmlowe has quit IRC18:51
*** yangyapeng has quit IRC18:54
*** Apoorva has quit IRC18:58
*** jmlowe has joined #openstack-cinder18:58
*** gkadam has joined #openstack-cinder18:59
*** moshele has joined #openstack-cinder19:00
*** e0ne has joined #openstack-cinder19:01
*** gkadam has quit IRC19:02
*** gkadam has joined #openstack-cinder19:02
*** armaan has joined #openstack-cinder19:03
*** AlexeyAbashkin has joined #openstack-cinder19:03
*** Jack_Iv has joined #openstack-cinder19:06
*** AlexeyAbashkin has quit IRC19:07
*** josecastroleon has quit IRC19:08
openstackgerritAbhishek Sharma M proposed openstack/cinder master: Removed gb quota decrement in grp snapshot delete  https://review.openstack.org/52507119:08
felipemonteiro__jungleboyj: do you have a minute or two? i have a question about some cinder APIs that return None if a user isn't authorized but returns a dict if the user is authorized... and whether that sort of thing is intentional design or something that should be fixed19:13
jungleboyjfelipemonteiro__:  Go ahead.  I should pull in smcginnis and tommylikehu  though as they know the API's a little better.19:14
felipemonteiro__the link is this (for example):19:14
felipemonteiro__#link https://github.com/openstack/cinder/blob/0cf910d4345c000e8c306b1cb2b2dd291975cf71/cinder/api/contrib/volume_image_metadata.py#L13119:14
felipemonteiro__The detailed explanation is in the commit message here... but then I questioned whether I should even be fixing this sort of thing Tempest-side:19:15
felipemonteiro__#link https://review.openstack.org/#/c/521935/19:15
*** itlinux has joined #openstack-cinder19:17
jungleboyjfelipemonteiro__:  The default is ADMIN_OR_OWNER .19:22
jungleboyjSo this is only an issue if it is made ADMIN only.19:22
*** itlinux has quit IRC19:22
felipemonteiro__Yes, that's correct.19:23
jungleboyjSo is there a different way it should be handled?19:24
felipemonteiro__Tempest should be able to work with any cloud -- that's their use case -- so not sure if it's a Tempest bug or Cinder one.19:25
felipemonteiro__Uncertain because not sure if an API should return None or empty dict, if it returns a dict normally.19:25
*** amoralej is now known as amoralej|off19:26
*** moshele has quit IRC19:27
*** itlinux has joined #openstack-cinder19:27
*** salv-orlando has joined #openstack-cinder19:27
jungleboyjIt seems to me that you should get a 403, not a 'None'.  RIght?19:28
felipemonteiro__Yes, that would make sense too19:29
felipemonteiro__As *usually* (from what I've seen) fatal=False is reserved for adding an attribute to a response body19:29
*** itlinux has quit IRC19:29
*** salv-orlando has quit IRC19:32
*** ntpttr_laptop has joined #openstack-cinder19:32
jungleboyjI am thinking we have a bug here.19:33
jungleboyjWould you be willing to add this to the agenda for Wednesday's team meeting so we could discuss it as a team there and plan what to do with this.19:33
felipemonteiro__jungleboyj: Thanks for your time :D, will do. I'll also look for this pattern elsewhere using code search.19:35
*** e0ne has quit IRC19:38
*** e0ne has joined #openstack-cinder19:38
*** e0ne has quit IRC19:40
*** Apoorva has joined #openstack-cinder19:41
*** abhishek has joined #openstack-cinder19:42
jungleboyjThat wold be great.  Have an examples from elsewhere to help inform our decision.19:43
*** Apoorva has quit IRC19:45
*** abhi89 has quit IRC19:45
*** Apoorva has joined #openstack-cinder19:54
*** moshele has joined #openstack-cinder19:54
*** e0ne has joined #openstack-cinder19:57
*** Jack_Iv has quit IRC19:59
*** e0ne has quit IRC20:02
*** moshele has quit IRC20:15
*** nsingh has joined #openstack-cinder20:16
*** bhagyashri_s has joined #openstack-cinder20:16
*** pooja-jadhav has joined #openstack-cinder20:16
*** gkadam has quit IRC20:17
*** pooja_jadhav has quit IRC20:18
*** niraj_singh has quit IRC20:18
*** bhagyashris has quit IRC20:18
*** salv-orlando has joined #openstack-cinder20:28
*** e0ne has joined #openstack-cinder20:28
*** salv-orlando has quit IRC20:33
abhishekjungleboyj: I have changed the commit message https://review.openstack.org/#/c/525071/..20:38
*** shaner has quit IRC20:38
jungleboyjThanks.  That makes more sense.20:39
*** shaner has joined #openstack-cinder20:39
*** moshele has joined #openstack-cinder20:41
*** liverpooler has quit IRC20:42
*** ntpttr_laptop has quit IRC20:46
*** nsingh has quit IRC20:54
*** felipemonteiro_ has joined #openstack-cinder20:54
*** moshele has quit IRC20:55
*** nsingh has joined #openstack-cinder20:55
*** gnufied has quit IRC20:56
*** felipemonteiro__ has quit IRC20:57
openstackgerritMerged openstack/cinder master: Fix api-ref for v3 group type specs  https://review.openstack.org/52586821:00
*** Apoorva has quit IRC21:11
openstackgerritMerged openstack/cinder master: Update unity tests to use our test base  https://review.openstack.org/52227121:13
*** david-lyle has quit IRC21:13
openstackgerritMerged openstack/cinder master: DS8K: don't swap role after failover group  https://review.openstack.org/51655121:24
*** jmlowe has quit IRC21:27
*** jmlowe has joined #openstack-cinder21:28
*** salv-orlando has joined #openstack-cinder21:29
*** salv-orlando has quit IRC21:34
*** threestrands has joined #openstack-cinder21:41
*** threestrands has quit IRC21:41
*** threestrands has joined #openstack-cinder21:41
*** r-daneel has quit IRC21:43
*** r-daneel has joined #openstack-cinder21:43
*** pooja_jadhav has joined #openstack-cinder21:46
*** bhagyashris_ has joined #openstack-cinder21:47
*** salv-orlando has joined #openstack-cinder21:47
*** abhishek has quit IRC21:48
*** niraj_singh has joined #openstack-cinder21:49
*** cfriesen__ has joined #openstack-cinder21:50
*** strigazi_ has joined #openstack-cinder21:51
*** tonyb_ has joined #openstack-cinder21:51
*** andreykurilin_ has joined #openstack-cinder21:55
*** EmilienM_ has joined #openstack-cinder21:55
*** rmk` has joined #openstack-cinder21:55
*** stephenfin_ has joined #openstack-cinder21:55
*** lhinds|out has joined #openstack-cinder21:55
*** nsingh has quit IRC21:56
*** pooja-jadhav has quit IRC21:56
*** bhagyashri_s has quit IRC21:56
*** lennyb has quit IRC21:56
*** ircuser-1 has quit IRC21:56
*** cfriesen_ has quit IRC21:56
*** EmilienM has quit IRC21:56
*** jdillaman has quit IRC21:56
*** 17SAA4WSA has quit IRC21:56
*** tonyb has quit IRC21:56
*** strigazi has quit IRC21:56
*** rmk has quit IRC21:56
*** mdbooth has quit IRC21:56
*** lhinds has quit IRC21:56
*** stephenfin has quit IRC21:56
*** freerunner has quit IRC21:56
*** andreykurilin has quit IRC21:56
*** rmk` is now known as rmk21:56
*** rmk has quit IRC21:56
*** rmk has joined #openstack-cinder21:56
*** vryzhenkin has joined #openstack-cinder21:56
*** EmilienM_ is now known as EmilienM21:56
*** vryzhenkin is now known as freerunner21:56
*** EmilienM has quit IRC21:56
*** EmilienM has joined #openstack-cinder21:56
*** Apoorva has joined #openstack-cinder21:57
*** ircuser-1 has joined #openstack-cinder21:58
*** mdbooth has joined #openstack-cinder21:58
*** lennyb has joined #openstack-cinder22:00
*** jdillaman has joined #openstack-cinder22:00
*** gouthamr has joined #openstack-cinder22:00
*** yangyapeng has joined #openstack-cinder22:09
*** mvk has joined #openstack-cinder22:10
*** yangyapeng has quit IRC22:13
*** rcernin has joined #openstack-cinder22:19
*** e0ne has quit IRC22:22
*** xyang1 has quit IRC22:34
*** dave-mccowan has quit IRC22:34
*** gouthamr has quit IRC22:40
*** yangyapeng has joined #openstack-cinder22:58
*** yangyapeng has quit IRC23:03
*** threestrands_ has joined #openstack-cinder23:04
*** threestrands_ has joined #openstack-cinder23:04
*** mchlumsky has quit IRC23:06
*** threestrands has quit IRC23:06
*** dustins has quit IRC23:14
*** felipemonteiro_ has quit IRC23:16
*** jappleii__ has joined #openstack-cinder23:20
*** felipemonteiro_ has joined #openstack-cinder23:20
*** jappleii__ has quit IRC23:21
*** jappleii__ has joined #openstack-cinder23:22
*** gmann_afk is now known as gmann23:22
*** threestrands_ has quit IRC23:22
*** gmann is now known as gmann_afk23:32
*** ntpttr_laptop has joined #openstack-cinder23:33
*** ntpttr_laptop has quit IRC23:38
*** felipemonteiro_ has quit IRC23:44
*** linkmark has quit IRC23:47
*** stakeda has joined #openstack-cinder23:48
*** Apoorva has quit IRC23:48
*** Apoorva has joined #openstack-cinder23:48
*** yangyapeng has joined #openstack-cinder23:52
*** enriquetaso has quit IRC23:55
*** wxy has quit IRC23:55
*** DuncanT has quit IRC23:55
*** salv-orlando has quit IRC23:55
*** enriquetaso has joined #openstack-cinder23:56
*** wxy has joined #openstack-cinder23:56
*** yangyapeng has quit IRC23:56
*** TheJulia has quit IRC23:56
*** melwitt has quit IRC23:57
*** ameade has quit IRC23:57
*** apopovych has quit IRC23:57
*** wiggin15 has quit IRC23:57
*** MeltedLux has quit IRC23:57
*** twouters has quit IRC23:57
*** dirk has quit IRC23:57
*** TheJulia has joined #openstack-cinder23:58
*** AlexeyAbashkin has joined #openstack-cinder23:58
*** dirk has joined #openstack-cinder23:59
openstackgerritMerged openstack/cinder master: Check snapshot flag for test_snapshot_create_volume_description_non_ascii_code  https://review.openstack.org/52650223:59

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