Monday, 2015-11-09

*** haomaiwang has quit IRC00:01
*** haomaiwa_ has joined #openstack-cinder00:01
*** rady has quit IRC00:06
*** mriedem is now known as mriedem_away00:10
*** zhangjn has joined #openstack-cinder00:11
*** setmason has joined #openstack-cinder00:13
*** zhangjn has quit IRC00:15
openstackgerritMerged openstack/cinder: Port cinder.hacking to Python 3  https://review.openstack.org/24205300:22
*** smoriya has joined #openstack-cinder00:24
*** zhangjn has joined #openstack-cinder00:56
*** zhangjn has quit IRC00:56
*** zhangjn has joined #openstack-cinder00:57
*** haomaiwa_ has quit IRC01:01
*** haomaiwang has joined #openstack-cinder01:01
*** zhangjn_ has joined #openstack-cinder01:02
*** zhangjn has quit IRC01:04
*** EinstCrazy has joined #openstack-cinder01:04
*** setmason has quit IRC01:04
*** EinstCrazy has quit IRC01:04
*** setmason has joined #openstack-cinder01:04
*** EinstCrazy has joined #openstack-cinder01:04
*** setmason has quit IRC01:06
*** setmason has joined #openstack-cinder01:06
*** zhenguo has joined #openstack-cinder01:15
*** yangxi has joined #openstack-cinder01:15
*** haomaiwang has quit IRC01:25
*** Lee1092 has joined #openstack-cinder01:32
*** lixiaoy1 has joined #openstack-cinder01:37
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Change nfs to NFS in the help strings of nfs.py  https://review.openstack.org/20903301:45
*** setmason has quit IRC01:48
*** Yogi1 has joined #openstack-cinder01:55
*** mitz has left #openstack-cinder01:55
*** bswartz has quit IRC01:56
*** yangxi has quit IRC01:58
*** bswartz has joined #openstack-cinder01:59
openstackgerritwanghao proposed openstack/cinder-specs: Get the replicated volume ref for DR  https://review.openstack.org/18215002:01
*** yangxi has joined #openstack-cinder02:04
*** setmason has joined #openstack-cinder02:08
*** liewegas has quit IRC02:11
*** haomaiwa_ has joined #openstack-cinder02:16
*** yangxi has quit IRC02:18
*** haomaiwa_ has quit IRC02:19
*** haomaiwang has joined #openstack-cinder02:19
*** haomaiwang has quit IRC02:19
*** haomaiwang has joined #openstack-cinder02:19
*** bill_az has quit IRC02:20
*** yangxi has joined #openstack-cinder02:21
*** haomaiwang has quit IRC02:21
*** haomaiwang has joined #openstack-cinder02:22
*** Yogi1 has quit IRC02:35
*** sage has joined #openstack-cinder02:38
*** haomaiwang has quit IRC02:40
*** haomaiwang has joined #openstack-cinder02:43
*** yangxi has quit IRC02:49
*** yangxi has joined #openstack-cinder02:52
*** julim_ has quit IRC03:00
*** haomaiwang has quit IRC03:01
*** Yogi1 has joined #openstack-cinder03:01
*** haomaiwang has joined #openstack-cinder03:01
openstackgerritMitsuhiro Tanino proposed openstack/cinder: Change nfs to NFS in the help strings of nfs.py  https://review.openstack.org/20903303:02
*** Lickitysplitted has joined #openstack-cinder03:04
*** Yogi1 has quit IRC03:05
*** houming-wang has joined #openstack-cinder03:08
*** yangxi has quit IRC03:12
*** lei-zh1 has joined #openstack-cinder03:34
*** yangxi has joined #openstack-cinder03:38
*** zhangjn_ has quit IRC03:55
*** zhangjn has joined #openstack-cinder03:58
*** zhangjn has quit IRC03:59
*** zhangjn has joined #openstack-cinder04:00
*** elonden has joined #openstack-cinder04:00
*** haomaiwang has quit IRC04:01
*** haomaiwa_ has joined #openstack-cinder04:01
*** houming-wang has quit IRC04:02
*** houming-wang has joined #openstack-cinder04:04
*** dims has quit IRC04:06
*** zhangjn has quit IRC04:07
*** pradipta has joined #openstack-cinder04:11
*** dave-mccowan has quit IRC04:11
openstackgerritMerged openstack/cinder: Port cinder.utils.monkey_patch() to Python 3  https://review.openstack.org/24214804:19
*** yangxi has quit IRC04:23
*** zhangjn has joined #openstack-cinder04:35
*** chenying_ has quit IRC04:36
*** chenying has joined #openstack-cinder04:37
*** anshul has joined #openstack-cinder04:46
*** yangxi has joined #openstack-cinder04:47
Lickitysplittedanyone have some links to info for iser on openstack kilo04:48
*** jamielennox is now known as jamielennox|away04:57
*** elonden has quit IRC04:59
*** haomaiwa_ has quit IRC05:01
*** haomaiwa_ has joined #openstack-cinder05:01
*** krtaylor has joined #openstack-cinder05:19
*** lei-zh1 has quit IRC05:22
*** jamielennox|away is now known as jamielennox05:24
*** links has joined #openstack-cinder05:29
*** deepakcs has joined #openstack-cinder05:41
*** salv-orlando has joined #openstack-cinder05:47
*** lei-zh has joined #openstack-cinder05:52
*** lpetrut has joined #openstack-cinder05:57
*** haomaiwa_ has quit IRC06:01
*** zhangjn_ has joined #openstack-cinder06:01
*** haomaiwang has joined #openstack-cinder06:01
*** links has quit IRC06:02
openstackgerritMerged openstack/cinder: 3PAR drivers volume size conversion is incorrect  https://review.openstack.org/24255706:03
*** zhangjn has quit IRC06:04
*** lpetrut has quit IRC06:09
*** zhangjn_ has quit IRC06:18
openstackgerritMerged openstack/cinder: Update the devref for volume migration  https://review.openstack.org/21494106:24
*** zhangjn has joined #openstack-cinder06:26
*** lpetrut has joined #openstack-cinder06:28
*** EinstCrazy has quit IRC06:32
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/24280606:32
*** zhangjn has quit IRC06:33
*** zhangjn has joined #openstack-cinder06:35
*** EinstCrazy has joined #openstack-cinder06:36
*** zhangjn has quit IRC06:38
*** salv-orlando has quit IRC06:39
*** links has joined #openstack-cinder06:41
*** zhangjn has joined #openstack-cinder06:44
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (managed)  https://review.openstack.org/23738706:44
*** yangxi has quit IRC06:46
*** yangxi has joined #openstack-cinder06:46
*** EinstCrazy has quit IRC06:46
*** zhangjn has quit IRC06:48
openstackgerritwanghao proposed openstack/cinder: CG creation should be scheduled on backend level  https://review.openstack.org/22440506:55
*** yangxi has quit IRC06:55
*** yangxi has joined #openstack-cinder06:56
*** lpetrut has quit IRC06:57
*** nikeshm has joined #openstack-cinder07:00
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
*** lpetrut has joined #openstack-cinder07:05
*** EinstCrazy has joined #openstack-cinder07:06
openstackgerritwanghao proposed openstack/cinder: Unmanaging vol/snap reduces quota incorrectly  https://review.openstack.org/23243607:17
*** setmason has quit IRC07:18
*** zhangjn has joined #openstack-cinder07:18
*** zhangjn has quit IRC07:24
openstackgerritwanghao proposed openstack/cinder: Retyping volume got error under max vol limit  https://review.openstack.org/23332507:25
*** zhangjn has joined #openstack-cinder07:26
*** EinstCrazy has quit IRC07:27
*** EinstCrazy has joined #openstack-cinder07:27
*** vignesh has joined #openstack-cinder07:35
*** EinstCrazy has quit IRC07:37
*** salv-orlando has joined #openstack-cinder07:39
*** lpetrut has quit IRC07:40
*** salv-orlando has quit IRC07:44
*** zerda has joined #openstack-cinder07:46
*** zhangjn has quit IRC07:47
*** zhangjn has joined #openstack-cinder07:53
*** zhangjn has quit IRC07:54
*** liverpooler has joined #openstack-cinder07:56
openstackgerritwanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata  https://review.openstack.org/14773808:00
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** afazekas is now known as afazekas|sick08:01
*** zhangjn has joined #openstack-cinder08:03
*** zigo has quit IRC08:16
*** zigo has joined #openstack-cinder08:19
*** lpetrut has joined #openstack-cinder08:21
*** nkrinner has joined #openstack-cinder08:25
*** zhangjn has quit IRC08:27
*** kaisers has quit IRC08:31
*** kaisers has joined #openstack-cinder08:32
*** EinstCrazy has joined #openstack-cinder08:33
*** zhangjn has joined #openstack-cinder08:33
*** zhangjn has quit IRC08:36
*** zhangjn has joined #openstack-cinder08:37
*** ZZelle has quit IRC08:38
*** kaisers has quit IRC08:39
*** ZZelle has joined #openstack-cinder08:39
*** kaisers has joined #openstack-cinder08:41
*** jamielennox is now known as jamielennox|away08:42
*** salv-orlando has joined #openstack-cinder08:45
*** e0ne has joined #openstack-cinder08:46
*** jordanP has joined #openstack-cinder08:53
*** salv-orlando has quit IRC08:53
*** lixiaoy1_ has joined #openstack-cinder08:55
lixiaoy1_test my irc08:55
*** e0ne has quit IRC08:55
*** takedakn has joined #openstack-cinder08:56
*** lixiaoy1 has left #openstack-cinder08:57
*** e0ne has joined #openstack-cinder08:57
*** mkoderer has quit IRC08:59
*** haomaiwang has quit IRC09:01
*** mkoderer has joined #openstack-cinder09:01
*** haomaiwang has joined #openstack-cinder09:01
*** takedakn1 has joined #openstack-cinder09:03
*** takedakn has quit IRC09:04
*** takedakn1 is now known as takedakn09:04
*** geguileo_onPTO is now known as geguileo_onLocal09:11
*** geguileo_onLocal is now known as geguileo_Holiday09:11
*** lixiaoy1_ has quit IRC09:12
*** lixiaoy1 has joined #openstack-cinder09:14
*** [1]Thelo has joined #openstack-cinder09:16
*** lixiaoy1 has left #openstack-cinder09:17
openstackgerritwanghao proposed openstack/cinder-specs: Improvement to query consistency group detail  https://review.openstack.org/24299809:17
*** openstack has joined #openstack-cinder09:19
*** takedakn has quit IRC09:19
*** subscope has joined #openstack-cinder09:22
openstackgerritwanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata  https://review.openstack.org/14773809:22
*** lixiaoy1_ is now known as lixiaoy109:27
*** yusuke has quit IRC09:33
*** takedakn has joined #openstack-cinder09:34
openstackgerritzhangsong proposed openstack/cinder: sheepdog:retrive configuration info from specified config group instead of default group  https://review.openstack.org/24300809:35
openstackgerritzhangsong proposed openstack/cinder: sheepdog:retrive configuration info from specified config group instead ofdefault group  https://review.openstack.org/24300809:39
openstackgerritzhangsong proposed openstack/cinder: sheepdog:retrive configuration info from specified config group instead of default group  https://review.openstack.org/24300809:40
*** jistr has joined #openstack-cinder09:41
*** aix has joined #openstack-cinder09:50
*** yuriy_n17 has joined #openstack-cinder09:50
*** takedakn has quit IRC09:57
openstackgerritShay Halsband proposed openstack/cinder: XtremIO fix create CG from src flow  https://review.openstack.org/24124409:58
*** haomaiwang has quit IRC10:01
*** haomaiwang has joined #openstack-cinder10:01
*** zhenguo has quit IRC10:02
openstackgerritPeter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature  https://review.openstack.org/24301710:02
openstackgerritPeter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature  https://review.openstack.org/24301710:05
openstackgerritPeter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature  https://review.openstack.org/24301710:06
*** can8dnSix has joined #openstack-cinder10:07
*** belmoreira has joined #openstack-cinder10:37
*** ndipanov has joined #openstack-cinder10:42
*** ndipanov has quit IRC10:42
*** yangxi has quit IRC10:44
*** deepakcs has quit IRC10:44
*** lei-zh has quit IRC10:45
*** subscope has quit IRC10:51
*** subscope has joined #openstack-cinder10:55
*** lpetrut has quit IRC11:00
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** subscope has quit IRC11:03
openstackgerritVictor Stinner proposed openstack/cinder: Port IBM storewize_svc driver to Python 3  https://review.openstack.org/24287811:10
*** zhangjn has quit IRC11:10
*** EinstCrazy has quit IRC11:10
*** haypo has joined #openstack-cinder11:12
*** dims has joined #openstack-cinder11:15
*** e0ne has quit IRC11:21
*** e0ne has joined #openstack-cinder11:30
*** lpetrut has joined #openstack-cinder11:31
*** zhangjn has joined #openstack-cinder11:36
*** zhangjn has quit IRC11:36
*** zhangjn has joined #openstack-cinder11:36
*** EinstCrazy has joined #openstack-cinder11:37
*** EinstCrazy has quit IRC11:38
*** EinstCrazy has joined #openstack-cinder11:38
openstackgerritVictor Stinner proposed openstack/cinder: Port EMC VNX CLI to Python 3  https://review.openstack.org/23224911:40
*** jaypipes has joined #openstack-cinder11:41
*** weihan has joined #openstack-cinder11:43
*** ndipanov has joined #openstack-cinder11:44
*** liverpooler has quit IRC11:47
*** smatzek has joined #openstack-cinder11:47
*** haomaiwang has quit IRC11:50
*** pradipta has quit IRC11:52
openstackgerritVictor Stinner proposed openstack/cinder: Port EMC Vmax to Python 3  https://review.openstack.org/24305111:56
*** liverpooler has joined #openstack-cinder12:02
*** lpabon has joined #openstack-cinder12:04
*** zerda has quit IRC12:06
*** liverpooler has quit IRC12:11
*** liverpooler has joined #openstack-cinder12:11
*** subscope has joined #openstack-cinder12:14
*** chenying has quit IRC12:14
*** chenying has joined #openstack-cinder12:14
*** ig0r__ has joined #openstack-cinder12:17
*** raildo-afk is now known as raildo12:17
*** can8dnSix has quit IRC12:20
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova (draft)  https://review.openstack.org/22412412:20
*** ociuhandu has joined #openstack-cinder12:22
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova  https://review.openstack.org/22412412:28
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/24006512:28
*** vgridnev has joined #openstack-cinder12:29
*** yangxi has joined #openstack-cinder12:33
openstackgerritIvan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova  https://review.openstack.org/22412412:43
*** lpetrut has quit IRC12:49
*** lpetrut has joined #openstack-cinder12:49
*** liverpoo1er has joined #openstack-cinder12:51
*** salv-orlando has joined #openstack-cinder12:52
*** salv-orlando has quit IRC12:56
*** aix has quit IRC12:56
*** houming-wang has quit IRC12:56
*** weihan has quit IRC12:57
*** subscope has quit IRC12:57
openstackgerritVictor Stinner proposed openstack/cinder: Port EMC Vmax to Python 3  https://review.openstack.org/24305112:57
*** jordanP has quit IRC12:59
*** dims has quit IRC12:59
*** marcusvrn_ has quit IRC13:05
*** jerrygb has quit IRC13:07
*** akerr has joined #openstack-cinder13:08
*** subscope has joined #openstack-cinder13:08
*** haomaiwa_ has joined #openstack-cinder13:13
*** dustins has joined #openstack-cinder13:14
*** aix has joined #openstack-cinder13:14
*** amoturi has joined #openstack-cinder13:18
*** amoturi has left #openstack-cinder13:19
openstackgerritThang Pham proposed openstack/cinder: Update migrate_volume API to use versionedobjects  https://review.openstack.org/22891613:22
*** e0ne has quit IRC13:23
*** xyang1 has joined #openstack-cinder13:25
*** boris-42 has quit IRC13:28
*** edmondsw has joined #openstack-cinder13:30
*** bswartz has quit IRC13:33
nikeshmhi , i will be on vacation this week due to indian festival "deepawali", feel free to email me in case any need on my email "nikeshmahalka@vedams.com"  :)13:33
*** porrua has joined #openstack-cinder13:35
*** diablo_rojo has joined #openstack-cinder13:36
*** e0ne has joined #openstack-cinder13:37
*** cfriesen__ has joined #openstack-cinder13:37
*** jordanP has joined #openstack-cinder13:37
*** timcl has joined #openstack-cinder13:38
*** diogogmt has quit IRC13:40
tbarronnikeshm: have a nice deepawali!13:41
*** diogogmt has joined #openstack-cinder13:42
nikeshmtbarron : thanks13:43
*** setmason has joined #openstack-cinder13:50
*** liverpoo1er has quit IRC13:56
*** salv-orlando has joined #openstack-cinder13:58
*** bswartz has joined #openstack-cinder13:59
*** boris-42 has joined #openstack-cinder13:59
*** haomaiwa_ has quit IRC14:01
*** haomaiwang has joined #openstack-cinder14:01
smcginnisnikeshm: I wish I would have been over there for one of the holidays.14:02
*** yangxi has quit IRC14:07
*** diogogmt has quit IRC14:10
*** jerrygb has joined #openstack-cinder14:13
*** jgregor has joined #openstack-cinder14:16
*** akshai has joined #openstack-cinder14:17
*** julim has joined #openstack-cinder14:17
*** haomaiwang has quit IRC14:19
*** haomaiwa_ has joined #openstack-cinder14:21
*** thangp has joined #openstack-cinder14:22
*** dave-mccowan has joined #openstack-cinder14:23
*** gouthamr has joined #openstack-cinder14:24
*** ndipanov has quit IRC14:24
*** baumann has joined #openstack-cinder14:26
*** superdan is now known as dansmith14:27
*** Yogi1 has joined #openstack-cinder14:28
*** zhangjn has quit IRC14:30
*** mriedem_away is now known as mriedem14:35
*** smatzek has quit IRC14:35
*** dims has joined #openstack-cinder14:36
*** subscope has quit IRC14:38
*** lprice has joined #openstack-cinder14:43
*** lprice1 has quit IRC14:45
*** dustins has quit IRC14:50
*** breitz has joined #openstack-cinder14:51
*** ctina has joined #openstack-cinder14:53
*** ctina_ has joined #openstack-cinder14:54
*** ctina has quit IRC14:54
*** IanGovett has joined #openstack-cinder14:55
*** willsama has joined #openstack-cinder14:56
*** asselin__ has joined #openstack-cinder14:57
*** smatzek has joined #openstack-cinder14:59
openstackgerritzhangsong proposed openstack/cinder: fix the bug of OSError when copy volume to image  https://review.openstack.org/24315215:00
*** baumann has quit IRC15:00
*** haomaiwa_ has quit IRC15:01
*** baumann has joined #openstack-cinder15:01
*** asselin__ has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** salv-orlando has quit IRC15:02
*** asselin__ has joined #openstack-cinder15:03
haypohi. i got a -1 on https://review.openstack.org/#/c/243010/ "tox: add /bin/bash to whitelist_externals" because i didn't open a bug15:04
haypois it mandatory to open a bug for any refactoring/small enhancement change?15:04
smcginnishaypo: Not here. Not sure if horizon has a different policy though.15:05
smcginnishaypo: That does look like a bug fix. I'm just against filing bugs for changes that aren't really "bugs".15:05
haypooops, wrong channel :) i worked on cinder and horizon this morning, i'm confused :)15:06
smcginnishaypo: ;)15:06
hayposmcginnis: this morning a saw a bug report to remove an unused variable :-/ https://bugs.launchpad.net/oslo.db/+bug/1514399 i don't thnik that it's worth to open a bug for that15:07
openstackLaunchpad bug 1514399 in oslo.db "Unused variable 'drivertype' in provision.py" [Undecided,Fix committed] - Assigned to sandeep nandal (nandalemail)15:07
smcginnishaypo: I agree. I'm totally against that.15:07
haypoby the way, we made good progress on py3 port. keep reviewing, we are close! ;)15:07
*** liverpooler has quit IRC15:07
dulekhaypo: Thank you for driving that effort!15:08
haypodulek: i would be faster if you had less drivers! :-D15:08
smcginnishaypo: Yes, thank you! Good to see how that's progressing.15:08
haypoit's amazing how many drivers cinder have15:09
smcginnis:)15:09
haypoi think neutron splitted drivers out of the tree15:09
*** lpetrut has quit IRC15:09
haypoand neutron "core" was already ported to py315:09
*** vgridnev has quit IRC15:11
*** stevemar_ has joined #openstack-cinder15:13
*** ntpttr has quit IRC15:14
*** ntpttr has joined #openstack-cinder15:14
*** eharney has joined #openstack-cinder15:16
*** houming-wang has joined #openstack-cinder15:16
*** openstackgerrit has quit IRC15:17
*** openstackgerrit has joined #openstack-cinder15:17
*** mdenny has joined #openstack-cinder15:19
*** lprice has quit IRC15:20
*** setmason has quit IRC15:20
*** vgridnev has joined #openstack-cinder15:24
*** baumann has quit IRC15:25
*** Yogi1 has quit IRC15:26
*** wlehman__ has joined #openstack-cinder15:26
*** wlehman__ has quit IRC15:26
*** salv-orlando has joined #openstack-cinder15:27
*** mc_nair has joined #openstack-cinder15:27
*** jungleboyj has joined #openstack-cinder15:27
*** aorourke has quit IRC15:28
*** salv-orlando has quit IRC15:28
*** wlehman__ has joined #openstack-cinder15:29
openstackgerritvenkatamahesh proposed openstack/cinder: OpenStack typo  https://review.openstack.org/24316615:29
*** willsama has quit IRC15:29
*** cbader has quit IRC15:29
*** aorourke has joined #openstack-cinder15:30
*** krtaylor has quit IRC15:32
openstackgerritzhangsong proposed openstack/cinder: fix the bug of OSError when copy volume to image  https://review.openstack.org/24315215:32
*** vgridnev has quit IRC15:33
*** links has quit IRC15:34
*** setmason has joined #openstack-cinder15:38
*** wlehman__ has quit IRC15:44
*** mtanino has joined #openstack-cinder15:45
*** krtaylor has joined #openstack-cinder15:45
jungleboyjeharney: I think this was a patch you wanted to weigh in on, it has been out there for a while.15:46
jungleboyjeharney: Mind taking a look?  https://review.openstack.org/#/c/232205/15:46
openstackgerritzhangsong proposed openstack/cinder: sheepdog:retrive configuration info from specified config group  https://review.openstack.org/24300815:46
*** dansmith is now known as thedan15:46
*** thedan is now known as dansmith15:46
*** lpetrut has joined #openstack-cinder15:47
*** cbader has joined #openstack-cinder15:49
eharneyjungleboyj: will take a look today -- i think i like this patch, just trying to make sure i understand it all15:50
*** willsama has joined #openstack-cinder15:53
*** haomaiwang has quit IRC16:01
*** jordanP has quit IRC16:01
*** haomaiwang has joined #openstack-cinder16:01
*** eharney has quit IRC16:02
*** diogogmt has joined #openstack-cinder16:04
*** lprice has joined #openstack-cinder16:06
*** anshul has quit IRC16:06
*** krotscheck has joined #openstack-cinder16:07
*** dustins has joined #openstack-cinder16:08
openstackgerritzhangsong proposed openstack/cinder: Fix the bug of OSError when copy volume to image  https://review.openstack.org/24315216:08
*** salv-orlando has joined #openstack-cinder16:09
*** vignesh has quit IRC16:15
hemnamornin16:15
*** dustins_ has joined #openstack-cinder16:17
*** eharney has joined #openstack-cinder16:17
*** dustins has quit IRC16:19
*** dustins_ is now known as dustins16:19
*** lcurtis has joined #openstack-cinder16:20
e0nehemna: evening:)16:20
dulekhemna: Morning16:23
*** eharney has quit IRC16:24
duleke0ne: Isn't this channel in UGT (http://www.total-knowledge.com/~ilya/mips/ugt.html)? ;)16:24
*** jdurgin1 has joined #openstack-cinder16:24
*** e0ne has quit IRC16:26
hemnadulek, :)16:26
*** mudassirlatif has joined #openstack-cinder16:26
*** lpetrut has quit IRC16:26
*** lpabon has quit IRC16:28
scottdadulek: +116:29
*** salv-orlando has quit IRC16:33
*** nkrinner has quit IRC16:33
*** e0ne has joined #openstack-cinder16:34
duleke0ne: I've got a question on your comment on https://review.openstack.org/#/c/222292/16:34
e0nedulek: :)16:35
duleke0ne: Are we testing this driver in Tempest? It seems very strange for driver change to affect tempest.16:35
e0nedulek: what do you mean "to affect tempest"?16:36
*** eharney has joined #openstack-cinder16:36
e0nedulek: we've got gate-tempest-dsvm-full-bdd-nv job and we (Mirantis) are working to make it green16:36
*** Yogi1 has joined #openstack-cinder16:36
openstackgerritScott DAngelo proposed openstack/cinder-specs: cinder-api-microversions  https://review.openstack.org/22380316:37
openstackgerritVictor Stinner proposed openstack/cinder: Port HP 3PAR driver to Python 3  https://review.openstack.org/24212816:37
openstackgerritVictor Stinner proposed openstack/cinder: Modify test_hp3par to support random hash  https://review.openstack.org/24212716:37
duleke0ne: Aaah, okay. Thanks, it's clear now.16:38
e0nedulek: good. I'll try to post more clear comment next time. thanks!16:39
*** dustins has quit IRC16:39
duleke0ne: So it may be related to this race condition issue Mitsuhiro and I have raised. If you need an idea to solve it - I can help. :)16:41
e0nedulek: TBH, I don't know what race do you mean16:41
e0nedulek: I've found issue with nova volume encryprion and that driver16:42
duleke0ne: Hm… I can try to explain that if you want.16:42
e0nedulek: why not if you have a time?16:42
duleke0ne: So basically the problem is the selection of device where we can put a volume on.16:43
duleke0ne: find_appropriate_size_device method is responsible for this.16:44
e0nedulek: oh... I think, I've got it16:44
*** aix has quit IRC16:45
e0nedulek: interesting issue.. I need to test such case16:45
*** diogogmt has quit IRC16:45
duleke0ne: If you dive into it you'll notice, that it identifies that device is used by DB column.16:45
dulekBut column isn't saved before returning from for example create_volume.16:46
dulekSo here's the race condition - we would need to lock on higher level.16:46
e0neit's definitely an issue what we have to fix16:47
*** lpetrut has joined #openstack-cinder16:47
*** crose has joined #openstack-cinder16:47
duleke0ne: Glad I've helped. :)16:48
e0nedulek: thanks for detailed explanation16:48
*** ntpttr has quit IRC16:48
*** ntpttr has joined #openstack-cinder16:49
*** dustins has joined #openstack-cinder16:49
openstackgerritxing-yang proposed openstack/cinder: Implement update_migrated_volume for the ScaleIO driver  https://review.openstack.org/24093716:50
*** houming-wang has quit IRC16:50
*** wilson-1 has joined #openstack-cinder16:52
mtaninodulek: e0ne, ah, thank you for sharing our concern :)16:53
*** wilson1 has quit IRC16:54
*** daneyon has joined #openstack-cinder16:55
openstackgerritxing-yang proposed openstack/cinder: ScaleIO driver: update_migrated_volume  https://review.openstack.org/24093716:55
*** daneyon_ has joined #openstack-cinder16:56
*** BharatK has joined #openstack-cinder16:58
*** haypo has left #openstack-cinder16:58
*** daneyon has quit IRC16:59
*** haomaiwang has quit IRC17:01
*** aix has joined #openstack-cinder17:01
*** haomaiwang has joined #openstack-cinder17:01
*** leeantho has joined #openstack-cinder17:01
*** p0rtal has joined #openstack-cinder17:03
*** edtubill has joined #openstack-cinder17:06
*** p0rtal has quit IRC17:08
*** edtubill has quit IRC17:08
*** edtubill has joined #openstack-cinder17:08
*** e0ne has quit IRC17:09
*** krotscheck has quit IRC17:12
*** yuriy_n17 has quit IRC17:12
*** lpetrut has quit IRC17:14
*** dustins has quit IRC17:18
*** dustins has joined #openstack-cinder17:18
*** p0rtal has joined #openstack-cinder17:22
*** daneyon has joined #openstack-cinder17:23
*** breitz has quit IRC17:24
*** chris_morrell has joined #openstack-cinder17:26
*** daneyon_ has quit IRC17:26
*** p0rtal has quit IRC17:27
*** krotscheck has joined #openstack-cinder17:29
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Remove the HP CLIQ proxy driver  https://review.openstack.org/24140417:29
*** timcl has quit IRC17:30
*** markstur has quit IRC17:32
*** p0rtal has joined #openstack-cinder17:34
*** edtubill has quit IRC17:38
*** p0rtal has quit IRC17:38
*** daneyon_ has joined #openstack-cinder17:44
*** hodos has joined #openstack-cinder17:46
*** apoorvad has joined #openstack-cinder17:46
*** daneyon has quit IRC17:46
*** edtubill has joined #openstack-cinder17:47
*** stevemar_ has quit IRC17:48
nikeshmsmcginnis: hmm, i am going my hometown to enjoy deepawali with my family, i have a joint family :)17:50
*** markstur has joined #openstack-cinder17:51
nikeshmsmcginnis: my hometown is near to varanasi(http://www.narendramodi.in/varanasi#vhome)  :)17:54
*** IlyaG has joined #openstack-cinder17:58
*** aix has quit IRC18:00
*** haomaiwang has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
*** jistr has quit IRC18:03
*** sghanekar has joined #openstack-cinder18:06
*** angela-s has joined #openstack-cinder18:06
*** belmoreira has quit IRC18:08
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP 3PAR drivers to now be HPE  https://review.openstack.org/24033818:10
*** anshul has joined #openstack-cinder18:12
*** p0rtal has joined #openstack-cinder18:16
*** john5223 is now known as zz_john522318:17
*** timcl has joined #openstack-cinder18:19
*** zz_john5223 is now known as john522318:19
*** anshul has quit IRC18:19
*** p0rtal has quit IRC18:21
*** lpetrut has joined #openstack-cinder18:22
*** timcl1 has joined #openstack-cinder18:22
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed)  https://review.openstack.org/23120118:23
*** timcl has quit IRC18:24
*** asselin__ has quit IRC18:24
*** asselin_ has joined #openstack-cinder18:28
*** breitz has joined #openstack-cinder18:29
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)  https://review.openstack.org/23124518:29
smcginnisnikeshm: Wow, looks amazing.18:34
*** breitz has quit IRC18:34
*** IlyaG has quit IRC18:34
*** crose has quit IRC18:34
*** breitz has joined #openstack-cinder18:35
*** p0rtal has joined #openstack-cinder18:38
*** breitz has quit IRC18:41
*** crose has joined #openstack-cinder18:41
*** breitz has joined #openstack-cinder18:42
*** crose has quit IRC18:47
*** diogogmt has joined #openstack-cinder18:48
*** pv_ has joined #openstack-cinder18:54
*** breitz has quit IRC18:54
pv_i had a quick question on RDO, how do you configure the cinder GIT repo it pulls from? I want to use our driver which we are developing in a private cinder repo we use18:55
pv_is that possible? i know on devstack its possible changing it local.conf18:55
tbarronpv_ someone here may answer but you may want to try #RDO18:59
pv_alright thanks19:00
pv_i19:00
*** e0ne has joined #openstack-cinder19:00
pv_i'll ask there as well*19:00
patrickeastpv_: i'm not 100% sure, but i thought RDO used the rpm's that are packaged and not sources directly from git repos19:00
*** haomaiwang has quit IRC19:01
pv_yeah that's what it looks like19:01
pv_i just wanted to know if its only supposed to be using the packaged drivers, or if we can run development drivers on it as well19:01
patrickeastpv_: you should, in theory, be able to just drop your driver file into the right directory19:01
*** breitz has joined #openstack-cinder19:01
patrickeastpv_: assuming the driver is pretty close version to the RDO release it will probably work19:02
*** haomaiwa_ has joined #openstack-cinder19:02
*** asselin_ has quit IRC19:03
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP LeftHand driver to now be HPE  https://review.openstack.org/24034319:03
pv_alright i'll figure that out19:04
*** stevemar_ has joined #openstack-cinder19:05
*** IlyaG has joined #openstack-cinder19:08
*** IlyaG has quit IRC19:09
*** IlyaG has joined #openstack-cinder19:09
*** e0ne has quit IRC19:10
*** e0ne has joined #openstack-cinder19:10
*** ntpttr1 has joined #openstack-cinder19:14
*** [1]Thelo has joined #openstack-cinder19:16
*** Thelo has quit IRC19:18
*** [1]Thelo is now known as Thelo19:18
*** ociuhandu has quit IRC19:19
*** rady has joined #openstack-cinder19:19
*** garthb has joined #openstack-cinder19:22
*** willsama has quit IRC19:23
ntpttr1I have some questions about https://review.openstack.org/#/c/234839/. jgriffith, you mention that for non-policy holders the entry will be shown as none, which entry do you mean? It looks like the 'get' API call is called no matter who is making the call, so it should have info to fill the dict with. Do you mean a specific entry in the volume type info?19:24
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed)  https://review.openstack.org/23120119:24
*** e0ne has quit IRC19:27
*** Yogi11 has joined #openstack-cinder19:28
*** Yogi11 has quit IRC19:29
*** Yogi1 has quit IRC19:29
mtaninontpttr1: I'm interested in this topic too. I also couldn't catch which info is shown as none when the user is none priority.19:30
*** IlyaG has quit IRC19:30
*** EinstCrazy has quit IRC19:31
*** EinstCrazy has joined #openstack-cinder19:32
*** asselin_ has joined #openstack-cinder19:32
*** asselin__ has joined #openstack-cinder19:32
*** asselin__ has quit IRC19:34
*** willsama has joined #openstack-cinder19:36
*** willsama has quit IRC19:36
*** willsama has joined #openstack-cinder19:37
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)  https://review.openstack.org/23124519:38
openstackgerritDiem Tran proposed openstack/cinder: Fix ZFSSA drivers' local cache bugs  https://review.openstack.org/22955419:42
pv_so no one on rdo responded, can anyone here help?19:46
pv_ I want to use our own driver that doesn't seemed to be packaged with RDO19:47
pv_any ideas how to do that?19:47
pv_i dont know where the rdo files are installed to to just drag my driver in19:47
pv_and run it, the release is close to the rdo release i installed19:47
pv_so it should be compatible, just wanna make sure19:47
eharneypv_: in theory, it should work if you put it in the cinder volume/drivers python directory on the system, just may be a bit messy mixing your code in with packaged code19:49
pv_do you know where that directory is with RDO, with devstack its /opt/stack/cinder for me19:51
pv_?19:51
eharneypv_: /usr/lib/python2.7/site-packages/cinder/ or so19:51
eharneypv_: you can "rpm -qf python-cinder" and see where the other drivers aer19:51
eharneyare*19:52
pv_ahhh no worries thank you so much19:53
pv_this is what i needed!19:53
pv_im not sure why i said no worries lol, but thanky ou19:54
*** garthb_ has joined #openstack-cinder19:58
smcginniseharney: Is RDO still kilo?19:58
*** garthb has quit IRC19:59
eharneysmcginnis: Liberty's there now too19:59
smcginniseharney: OK, cool. Thanks!19:59
*** haomaiwa_ has quit IRC20:01
*** baumann has joined #openstack-cinder20:01
*** haomaiwang has joined #openstack-cinder20:01
*** crose has joined #openstack-cinder20:02
*** terryyao has joined #openstack-cinder20:03
openstackgerritAngela Smith proposed openstack/cinder: Adds friendly zone name support  https://review.openstack.org/18051820:06
*** asselin_ has quit IRC20:07
*** asselin_ has joined #openstack-cinder20:11
*** hodos has quit IRC20:11
*** hodos has joined #openstack-cinder20:12
*** porrua has quit IRC20:18
*** ntpttr1 has left #openstack-cinder20:22
*** Lee1092 has quit IRC20:29
*** martyturner has joined #openstack-cinder20:31
*** crose has quit IRC20:35
openstackgerritMerged openstack/cinder: Change nfs to NFS in the help strings of nfs.py  https://review.openstack.org/20903320:39
openstackgerritHelen Walsh proposed openstack/cinder: fix for randomly selecting a portgroup  https://review.openstack.org/24330420:39
*** vgridnev has joined #openstack-cinder20:44
*** IlyaG has joined #openstack-cinder20:48
*** baumann has quit IRC20:50
openstackgerritMerged openstack/cinder: VMware: Relocate volume only during no disk space  https://review.openstack.org/23519720:51
openstackgerritMerged openstack/cinder: Update cinder-manage man to match current options  https://review.openstack.org/24133720:51
*** akerr has quit IRC20:53
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP 3PAR drivers to now be HPE  https://review.openstack.org/24033820:59
*** haomaiwang has quit IRC21:01
*** haomaiwang has joined #openstack-cinder21:01
*** baumann has joined #openstack-cinder21:03
*** daneyon has joined #openstack-cinder21:05
*** p0rtal has quit IRC21:05
*** crose has joined #openstack-cinder21:05
*** daneyon has quit IRC21:06
*** daneyon has joined #openstack-cinder21:07
*** daneyon_ has quit IRC21:08
*** lpabon has joined #openstack-cinder21:09
*** lpabon has quit IRC21:11
*** terryyao has quit IRC21:12
*** subscope has joined #openstack-cinder21:16
*** daneyon_ has joined #openstack-cinder21:17
*** daneyon has quit IRC21:20
*** smatzek has quit IRC21:22
openstackgerritKendall Nelson proposed openstack/cinder: Removal of deprecated NPIV option in Storwize  https://review.openstack.org/24331821:22
*** IanGovett has quit IRC21:24
*** p0rtal has joined #openstack-cinder21:24
openstackgerritKendall Nelson proposed openstack/cinder: Removal of deprecated NPIV option in Storwize  https://review.openstack.org/24331821:25
jungleboyjeharney: hemna Wasn't there a way to map an old classpath for volume_driver to a new one so that you wouldn't break CI, systems, etc when refactoring your driver?21:27
eharneyjungleboyj: volume/manager.py has a mapping for class name changes21:27
jungleboyjeharney: Thank you!21:28
smcginnishttps://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L11321:28
jungleboyjsmcginnis: Excellent, that was what I was looking for.  Thank you!21:29
jungleboyjjgregor: baumann ^^^21:29
*** diegows has joined #openstack-cinder21:29
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP LeftHand driver to now be HPE  https://review.openstack.org/24034321:31
openstackgerritAnthony Lee proposed openstack/cinder: Refactor HP 3PAR drivers to now be HPE  https://review.openstack.org/24033821:32
*** IlyaG has quit IRC21:32
*** andrewfly has joined #openstack-cinder21:36
*** vgridnev has quit IRC21:38
*** lpetrut has quit IRC21:42
*** sghanekar has quit IRC21:45
*** sghanekar has joined #openstack-cinder21:46
openstackgerritZohar Mamedov proposed openstack/cinder: Nexenta Edge iSCSI backend driver  https://review.openstack.org/15874521:47
*** jaypipes has quit IRC21:49
SwansonCurious this ... extra_spec requirement 'True' does not match 'True'21:51
*** salv-orlando has joined #openstack-cinder21:51
*** jgregor has quit IRC21:52
*** baumann has quit IRC21:53
*** baumann has joined #openstack-cinder21:55
*** bswartz has quit IRC21:56
*** crose has quit IRC21:56
*** gouthamr has quit IRC21:58
*** akerr has joined #openstack-cinder21:59
*** haomaiwang has quit IRC22:01
*** haomaiwang has joined #openstack-cinder22:01
*** thangp has quit IRC22:03
*** julim has quit IRC22:05
*** salv-orlando has quit IRC22:05
*** martyturner has quit IRC22:07
*** vgridnev has joined #openstack-cinder22:12
*** dustins has quit IRC22:16
*** akerr has quit IRC22:17
openstackgerritAlex O'Rourke proposed openstack/cinder: Remove db access from 3PAR and LH cg functions  https://review.openstack.org/23608922:18
openstackgerritAlex O'Rourke proposed openstack/cinder: Remove db access from 3PAR and LH cg functions  https://review.openstack.org/23608922:19
*** timcl1 has quit IRC22:22
aorourkeSwanson, are the types different? Is on string/bytes and the other a bool?22:22
SwansonOne is from extra specs so certainly a string.  The other is bool.  The particular item is replication_enabled.22:23
aorourkeSwanson, ahh, the extra_specs for that is a little strange. replication_enabled needs to be set to "<is> True"22:23
aorourkeSwanson, in the extra_specs22:24
Swansonaorourke, ah!  yes!  Thanks!22:25
aorourkeSwanson, np22:25
*** vgridnev has quit IRC22:28
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix test_connect_volume when skip is bypassed  https://review.openstack.org/24333822:30
*** baumann has quit IRC22:32
*** dims_ has joined #openstack-cinder22:34
*** dims has quit IRC22:35
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix test_connect_volume when skip is bypassed  https://review.openstack.org/24333822:36
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add new Connector APIs for path validation  https://review.openstack.org/19976422:37
*** gouthamr has joined #openstack-cinder22:37
*** lprice has quit IRC22:37
*** jamielennox|away is now known as jamielennox22:38
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder-specs: Brick add new Connector volume APIs  https://review.openstack.org/20418022:38
*** gouthamr_ has joined #openstack-cinder22:38
hemnaI think I need to work on more patches at the same time.22:39
mtaninohemna: May I ask a question? When we add a feature to os-brick, do we need cinder-spec?22:40
hemnaif it's a new API into the lib, then it might need a spec22:40
hemnabut a new capability for an existing connector?  probably not22:41
mtaninohttps://review.openstack.org/#/c/237856/ for this case?22:41
hemnasmcginnis, ?22:41
*** rady has quit IRC22:41
hemnaheh that one22:41
hemnaI couldn't understand the commit message22:41
*** gouthamr has quit IRC22:41
hemnabut from what I can tell, it doesn't look like it needs a spec IMHO22:42
hemnamtanino, https://review.openstack.org/#/c/199764/22:42
hemnathat guy adds a whole new API into each connector object22:42
hemnaso I figured a spec was good22:42
mtaninoI see, so it needs more specific or detail commit message.22:42
*** harlowja_ has quit IRC22:43
mtaninohemna: I understand. Thanks22:43
hemnaI just didn't understand what the patch was trying to accomplish.22:43
hemnaI have a WIP patch I'm working on for adding resize_volume API to brick22:43
hemnawhich will allow live attached volumes to be extended22:43
hemnaI think that probably needs a new spec22:44
mtaninoI understand.22:44
*** IlyaG has joined #openstack-cinder22:44
hemnawill probably also require cinder changes to allow certain volumes to be extended while attached22:44
mtaninoextend attached volume... It's an aggressive.22:44
hemnawhich also will need a call into Nova, which will call the new brick api.22:44
smcginnishemna, mtanino: Just reading scrollback. I agree with hemna's assessment.22:45
mtaninosmcginnis: thanks for the confirmation.22:45
smcginnisBP's wouldn't hurt.22:45
hemnasmcginnis, cool thanks.22:45
smcginnisBut specs we really only need if there are significant changes that could affect consumers.22:45
xyang1hemna: hi, about your comments on this one https://review.openstack.org/#/c/237750/22:45
hemnaI started working on the extend volume stuff on Friday, but got sidetracked today w/ rebasing hell22:45
*** bswartz has joined #openstack-cinder22:45
*** jungleboyj has quit IRC22:45
xyang1hemna: We don't have CI on stable branches22:46
hemnadoh22:46
hemnawe do22:46
mtaninosmcginnis: I understand. thanks.22:46
hemnaI figured that was a given.22:46
*** mudassirlatif has quit IRC22:46
hemnaguess we haven't talked about that at all as a requirement ?22:46
smcginnismtanino: NP, thanks!22:46
xyang1hemna: no22:47
hemnaxyang1, hrmm22:47
smcginnishemna: We only do stable as well.22:47
smcginnisUnfortunately.22:47
hemnaheh, so what are all those 3rd party CI results on that patch then :P22:47
hemnatesting against master?  which is effectively a noop22:47
smcginnishemna: Yeah, probably. I think those just don't filter events to master. Maybe it pull the right patch for Cinder, but I'm guessing master for everything else. Which seems risky.22:48
smcginnishemna: So no surprise the majority of CI results on stable branches are failures. :/22:48
SwansonDear god I created a replicated volume.22:48
hemnayah, that does seem risky22:48
hemnafalse negatives and false positives22:48
xyang1hemna: smcginnis it is good to have, but we are not there yet22:48
xyang1Swanson: Congrats:)22:49
aorourkeSwanson, congrats!!!22:49
hemnaSwanson, w00t!22:49
*** mudassirlatif has joined #openstack-cinder22:49
*** stevemar_ has quit IRC22:49
SwansonI'm afraid to do anything else with this now!22:49
*** delattec has quit IRC22:50
xyang1Swanson: fail it over:)22:50
*** setmason has quit IRC22:51
smcginnisxyang1: Hah, yeah, go for broke.22:51
Swansonxyang1, I'm not ready for that heartbreak, yet22:51
*** setmason has joined #openstack-cinder22:51
hemnanow retype it!22:51
hemna:P22:51
SwansonI'm more thinking of trying resize and delete.  Nothing that is likely to ruin my evening.  :)22:53
*** eharney has quit IRC22:53
xyang1hemna: can you take a look of this again? https://review.openstack.org/#/c/229580/22:54
hemnaxyang1, lol22:54
hemnaI was JUST looking at that22:55
hemnaliterally22:55
xyang1hemna: ok:)22:55
mc_nairhey all - I'm working to cleanup the lock files that Cinder creates once it's done using them (https://bugs.launchpad.net/cinder/+bug/1432387), and was looking for feedback deciding between two approaches.  Initially I wrote a patch that would cleanup lock files after a resource got deleted (https://review.openstack.org/#/c/239678/), but now there's a suggestion to just use a single lock file for Cinder and leverage byte-offsets which honestly seems22:57
openstackLaunchpad bug 1432387 in Cinder "abandoned lock files" [High,In progress] - Assigned to Ryan McNair (rdmcnair)22:57
hemnarm -rf ?22:58
*** timcl has joined #openstack-cinder22:59
*** jerrygb has quit IRC22:59
*** haomaiwang has quit IRC23:01
openstackgerritAngela Smith proposed openstack/cinder: Adds friendly zone name support  https://review.openstack.org/18051823:01
*** haomaiwang has joined #openstack-cinder23:01
*** jerrygb has joined #openstack-cinder23:02
*** krtaylor has quit IRC23:02
mc_nairhemna: haha - I like it.  Will propose taking hammer to machine for next lock cleanup patch.23:02
*** edtubill has quit IRC23:03
mc_nairsmcginnis, jgriffith - ^ not sure if you have any preference on how we cleanup the locks23:03
*** timcl has quit IRC23:03
*** timcl has joined #openstack-cinder23:04
*** setmason has quit IRC23:05
*** setmason has joined #openstack-cinder23:06
hemna:P23:07
*** timcl has quit IRC23:07
smcginnismc_nair: How would you clean up the offsets?23:08
*** IlyaG has quit IRC23:08
smcginnisSorry, gotta run. Maybe online later tonight.23:09
mc_nairsmcginnis: wouldn't actually do any cleanup with the offsets - but it'd just be a single file (maybe 1 MB large) and each lock would come from that single files bucket23:10
smcginnismc_nair: As resource come and go wouldn't the file need to grow if you don't reclaim offsets?23:10
smcginnisI suppose 1MB large would accomodate a lot, but could eventually run out.23:10
mc_nairsmcginnis: we can discuss this more tomorrow - don't let me hold you up23:10
smcginnismc_nair: Thanks. I guess my only concern is if we're just moving to problem (tho to a smaller consumption).23:11
smcginnisBut we can talk later. Thanks!23:12
* smcginnis runs for the door23:12
*** timcl has joined #openstack-cinder23:14
*** krtaylor has joined #openstack-cinder23:14
*** crose has joined #openstack-cinder23:16
jgriffithmc_nair: so this hard codes in a 60 second limit on locks?23:16
jgriffithmc_nair: If that's correct I'd be concerned about that "short" of a duration23:17
jgriffithmc_nair: there are a number of drivers and operations that can take significantly longer than that23:17
mc_nairjgriffith: not quite.  Basically the 60 seconds only comes into play *after* the resource is deleted.  So say that a volume is deleted and some people where waiting on the lock (because the volume_delete locked that) - the 60 seconds just gives some time for things that were waiting on that lock to have the chance to go, notice the volume is deleted and fail in some way or another23:19
*** timcl has quit IRC23:19
jgriffithmc_nair: ahh.. .nice23:19
mc_nairjgriffith: yea - so it won't change anything until the resource is actually deleted, then the timer starts.23:20
jgriffithmc_nair: so in that case, this seems like a descent improvement to me23:20
jgriffithmc_nair: nice idea23:20
*** cdelatte has joined #openstack-cinder23:21
mc_nairjgriffith: yea - it's nice cause it doesn't have much code impact.  But it still does has the flow of having a periodic cleanup.  And the idea of using the byte-offsets is certainly more of a code impact to get to work, but is a cool idea because then we're not creating many many lock files in the first place23:21
jgriffithmc_nair: yeah... the bigger questions I'm honestly no sure about23:22
jgriffithmc_nair: I'm not even fully convinced we're on the right path with any of this lock stuff at all23:22
jgriffithmc_nair: but that's probably not even worth thinking about any more23:23
jgriffithmc_nair: wish we would strive more to be stateless and dump all of this lock stuff :)23:23
mc_nairjgriffith: haha that's what I was getting from your initial bug comment :) it definitely seems like a conversation worth having, but at least for the time being either of these would fix the locks that we're using (as I don't see them going away anytime soon).23:25
jgriffithmc_nair: nah... I don't think there's any point in discussing it further.  I do think what you propose seems like a good incremental improvement23:25
*** willsama has quit IRC23:26
jgriffithmc_nair: at least should address people running out of disk space due to locks :)23:26
*** subscope has quit IRC23:26
mc_naircool - will check with smcginnis about if he's got any opinions about the byte-offset vs periodic cleanup and then move forward with it.23:28
mc_nairhopefully not disk space, but maybe file descriptors23:28
mc_nairjgriffith: thanks for the feedback23:28
jgriffithmc_nair: the byte-offset thing is interesting, but I'm unconvinced long term on it FWIW23:28
jgriffithmc_nair: anywho23:28
openstackgerritZohar Mamedov proposed openstack/cinder: Nexenta Edge iSCSI backend driver  https://review.openstack.org/15874523:30
*** lcurtis has quit IRC23:37
*** diegows has quit IRC23:40
*** sghanekar has quit IRC23:41
*** Yogi1 has joined #openstack-cinder23:47
*** Yogi1 has quit IRC23:51
openstackgerritHelen Walsh proposed openstack/cinder: fix for randomly selecting a portgroup  https://review.openstack.org/24330423:52
*** yusuke has joined #openstack-cinder23:52
*** mriedem is now known as mriedem_away23:54
*** akshai has quit IRC23:57
*** boris-42 has quit IRC23:58
*** sghanekar has joined #openstack-cinder23:59

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