Monday, 2016-04-18

*** sdake has joined #openstack-cinder00:06
*** salv-orlando has quit IRC00:06
*** mylu has quit IRC00:07
*** iceyao has joined #openstack-cinder00:10
*** yhayashi has quit IRC00:10
*** mylu has joined #openstack-cinder00:13
*** iceyao has quit IRC00:17
*** mylu has quit IRC00:19
*** mylu has joined #openstack-cinder00:21
*** yhayashi has joined #openstack-cinder00:22
*** qeelee has joined #openstack-cinder00:27
*** laughterwym has joined #openstack-cinder00:30
*** ekarlso has quit IRC00:32
*** mylu has quit IRC00:44
*** ekarlso has joined #openstack-cinder00:44
*** mylu has joined #openstack-cinder00:47
*** cheneydc has joined #openstack-cinder00:55
*** yangyape_ has joined #openstack-cinder01:06
*** sdake_ has joined #openstack-cinder01:14
*** sdake has quit IRC01:18
*** EinstCrazy has joined #openstack-cinder01:18
*** yangyape_ has quit IRC01:23
*** yangxi has joined #openstack-cinder01:26
*** qeelee has quit IRC01:31
*** rchurch_ has quit IRC01:34
*** rchurch has joined #openstack-cinder01:34
*** iceyao has joined #openstack-cinder01:35
*** iceyao_ has joined #openstack-cinder01:36
*** mriedem has quit IRC01:39
*** iceyao has quit IRC01:39
openstackgerritMerged openstack/cinder: Fix minor typos in the migration devref  https://review.openstack.org/30687201:42
*** qeelee has joined #openstack-cinder01:48
*** EinstCra_ has joined #openstack-cinder01:49
*** EinstCrazy has quit IRC01:52
*** laughterwym has quit IRC01:59
*** rchurch has quit IRC01:59
*** yangxi has quit IRC02:00
*** laughterwym has joined #openstack-cinder02:01
*** gouthamr has quit IRC02:03
*** mylu has quit IRC02:10
*** ebalduf has joined #openstack-cinder02:23
*** ebalduf has quit IRC02:26
*** smoriya_ has joined #openstack-cinder02:27
*** iceyao_ has quit IRC02:28
*** iceyao has joined #openstack-cinder02:28
openstackgerritLisaLi proposed openstack/cinder-specs: Retype encrypted volumes  https://review.openstack.org/24859302:33
openstackgerritLisaLi proposed openstack/cinder-specs: Retype encrypted volumes  https://review.openstack.org/24859302:37
*** ebalduf has joined #openstack-cinder02:41
*** haomaiwang has joined #openstack-cinder02:45
*** zhenguo_ has joined #openstack-cinder02:47
*** EinstCra_ has quit IRC02:54
*** Lee1092 has joined #openstack-cinder02:56
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** EinstCrazy has joined #openstack-cinder03:03
*** laughterwym has quit IRC03:08
*** yangxi has joined #openstack-cinder03:08
*** laughterwym has joined #openstack-cinder03:13
*** laughterwym has quit IRC03:18
*** bardia has joined #openstack-cinder03:19
*** bardia has quit IRC03:22
*** laughterwym has joined #openstack-cinder03:25
*** laughterwym has quit IRC03:29
*** sdake_ has quit IRC03:30
*** laughterwym has joined #openstack-cinder03:39
*** haomaiwang has quit IRC03:41
*** haomaiwa_ has joined #openstack-cinder03:41
*** garthb has joined #openstack-cinder03:42
*** laughterwym has quit IRC03:44
*** laughterwym has joined #openstack-cinder03:44
*** qeelee has quit IRC03:47
*** haomaiwa_ has quit IRC04:01
*** haomaiwa_ has joined #openstack-cinder04:01
*** haomaiwa_ has quit IRC04:01
*** 7JTAAPGO1 has joined #openstack-cinder04:01
*** sgotliv_ has joined #openstack-cinder04:06
*** salv-orlando has joined #openstack-cinder04:09
*** jwcroppe has joined #openstack-cinder04:09
*** garthb has quit IRC04:11
*** garthb has joined #openstack-cinder04:11
*** salv-orlando has quit IRC04:13
*** garthb has quit IRC04:14
*** garthb has joined #openstack-cinder04:14
*** garthb has quit IRC04:21
*** houming has joined #openstack-cinder04:21
*** jwcroppe has quit IRC04:22
*** klkumar has joined #openstack-cinder04:23
*** yangxi has quit IRC04:26
*** ekarlso has quit IRC04:27
*** sdake has joined #openstack-cinder04:28
*** qeelee has joined #openstack-cinder04:36
*** ekarlso has joined #openstack-cinder04:40
*** links has joined #openstack-cinder04:41
*** jwcroppe has joined #openstack-cinder04:45
*** yangxi has joined #openstack-cinder04:46
*** salv-orlando has joined #openstack-cinder04:48
*** jwcroppe has quit IRC04:50
*** Poornima has joined #openstack-cinder04:54
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: Let method CONF.set_override with enfore_type=true by default  https://review.openstack.org/25332805:00
*** smoriya_ has quit IRC05:00
*** 7JTAAPGO1 has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** mylu has joined #openstack-cinder05:03
*** sgotliv_ has quit IRC05:04
*** sivaramakrishna has joined #openstack-cinder05:04
*** rcernin has joined #openstack-cinder05:04
sivaramakrishnaHi cinder folks,05:05
sivaramakrishnaany idea if we have FC driver for HP left hand... i can only see iscsi here05:05
*** shausy has joined #openstack-cinder05:11
*** yangyape_ has joined #openstack-cinder05:11
*** mylu has quit IRC05:11
*** e0ne has joined #openstack-cinder05:15
jgriffithsivaramakrishna: correct, there is no FC LHN driver05:15
*** qeelee has quit IRC05:15
jgriffithsivaramakrishna: I didn't even know there was an FC version of the LeftHand device05:16
*** arch-nemesis has joined #openstack-cinder05:16
*** e0ne has quit IRC05:18
*** iceyao has quit IRC05:19
*** shausy has quit IRC05:23
*** iceyao has joined #openstack-cinder05:23
*** karthikp has joined #openstack-cinder05:24
*** mylu has joined #openstack-cinder05:24
*** shausy has joined #openstack-cinder05:26
*** haomaiwang has quit IRC05:28
*** mylu has quit IRC05:29
*** tpatzig_ has quit IRC05:35
*** mkoderer has quit IRC05:36
*** lpetrut has joined #openstack-cinder05:37
*** karthikp has quit IRC05:38
*** mkoderer has joined #openstack-cinder05:39
sivaramakrishnaah okay.. but we can connect LHN through FC05:39
sivaramakrishnajgriffith:do you know of any blueprints addressing this?05:39
openstackgerritMerged openstack/cinder: Implement Cluster Pairing for SolidFire Driver  https://review.openstack.org/29974605:43
*** jwcroppe has joined #openstack-cinder05:47
*** ChubYann has quit IRC05:50
*** jwcroppe has quit IRC05:53
*** ankit has joined #openstack-cinder05:54
*** laughterwym has quit IRC05:55
*** laughterwym has joined #openstack-cinder05:56
*** gcb has quit IRC05:56
*** laughterwym has quit IRC05:57
*** laughterwym has joined #openstack-cinder05:57
*** tpatzig has joined #openstack-cinder05:59
openstackgerritShlomi Avihou proposed openstack/cinder: Add backend driver for Zadara Storage VPSA  https://review.openstack.org/26302606:03
*** Poornima has quit IRC06:16
*** lixiaoy1 has joined #openstack-cinder06:31
*** eharney has quit IRC06:36
*** Guest23960 has quit IRC06:37
*** lixiaoy1 has quit IRC06:39
*** lixiaoy1 has joined #openstack-cinder06:39
*** wilson has joined #openstack-cinder06:41
*** wilson is now known as Guest825606:42
*** eharney has joined #openstack-cinder06:43
*** haomaiwang has joined #openstack-cinder06:46
openstackgerritDeepti Ramakrishna proposed openstack/cinder-specs: New extension API to expose system capabilities  https://review.openstack.org/30693006:49
*** jwcroppe has joined #openstack-cinder06:50
*** jwcroppe has quit IRC06:55
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
openstackgerritDeepti Ramakrishna proposed openstack/cinder-specs: New extension API to expose system capabilities  https://review.openstack.org/30693007:03
*** liverpooler has joined #openstack-cinder07:03
*** lpetrut has quit IRC07:06
*** haomaiwang has quit IRC07:11
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/30670807:13
*** dalgaaf has joined #openstack-cinder07:13
*** abehl has joined #openstack-cinder07:14
*** laughterwym has quit IRC07:15
*** e0ne has joined #openstack-cinder07:16
*** mjura has joined #openstack-cinder07:17
*** ebalduf has quit IRC07:19
openstackgerritwangxiyuan proposed openstack/cinder-specs: Update quota APIs for ConsistencyGroup  https://review.openstack.org/30694607:23
*** laughterwym has joined #openstack-cinder07:23
*** lpetrut has joined #openstack-cinder07:26
openstackgerritDeepti Ramakrishna proposed openstack/cinder-specs: New extension API to expose system capabilities  https://review.openstack.org/30693007:28
*** arch-nemesis has quit IRC07:31
*** arecknag has joined #openstack-cinder07:37
*** laughterwym has quit IRC07:42
*** laughterwym has joined #openstack-cinder07:42
*** laughterwym has quit IRC07:47
*** sgotliv_ has joined #openstack-cinder07:49
*** lpetrut has quit IRC07:49
*** lpetrut has joined #openstack-cinder07:49
*** jwcroppe has joined #openstack-cinder07:51
*** arecknag has quit IRC07:52
*** arecknag has joined #openstack-cinder07:53
*** geguileo has joined #openstack-cinder07:53
*** laughterwym has joined #openstack-cinder07:53
*** vgridnev has joined #openstack-cinder07:54
*** lpetrut has quit IRC07:56
*** vgridnev has quit IRC07:56
*** jwcroppe has quit IRC07:56
*** bjornar has joined #openstack-cinder08:02
*** ndipanov has joined #openstack-cinder08:03
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
bjornarI really dont understand the structure/parameters of keystone_authtoken, are these really not documented anywhere?08:11
bjornarThinking about auth_uri vs auth_url -- I dont find any reference to auth_url in the documentation, but afaik it is needed08:12
*** jistr has joined #openstack-cinder08:21
*** yuriy_n17 has joined #openstack-cinder08:22
*** Fdaisuke has joined #openstack-cinder08:23
*** lpetrut has joined #openstack-cinder08:31
*** rdx_matt has joined #openstack-cinder08:33
*** rdx_matt_ has quit IRC08:33
*** laughterwym has quit IRC08:40
*** laughterwym has joined #openstack-cinder08:40
*** goretoxo has joined #openstack-cinder08:40
*** laughterwym has quit IRC08:44
*** yangxi has quit IRC08:46
*** yangxi has joined #openstack-cinder08:48
*** yangxi_ has joined #openstack-cinder08:50
*** sgotliv has joined #openstack-cinder08:51
*** al_indigo has joined #openstack-cinder08:52
*** jwcroppe has joined #openstack-cinder08:53
*** yangxi has quit IRC08:53
*** jwcroppe has quit IRC08:58
*** yangxi_ has quit IRC09:01
*** al_indigo has quit IRC09:02
*** tesseract has joined #openstack-cinder09:03
*** tesseract is now known as Guest3039409:03
*** jistr has quit IRC09:05
*** sdake has quit IRC09:07
*** sdake has joined #openstack-cinder09:07
*** lpetrut has quit IRC09:09
openstackgerritXu Ao proposed openstack/cinder: Repalce funcs List.get then del with just List.pop  https://review.openstack.org/30700309:11
*** jistr has joined #openstack-cinder09:12
*** sheel has joined #openstack-cinder09:15
*** laughterwym has joined #openstack-cinder09:24
*** lpetrut has joined #openstack-cinder09:27
*** laughterwym has quit IRC09:28
*** sgotliv has quit IRC09:29
*** yangxi has joined #openstack-cinder09:31
openstackgerritwangxiyuan proposed openstack/cinder-specs: Update quota APIs for ConsistencyGroup  https://review.openstack.org/30694609:37
openstackgerritwangxiyuan proposed openstack/cinder: Add ConsistencyGroup Into Quota Management  https://review.openstack.org/30702409:41
*** gcb has joined #openstack-cinder09:42
openstackgerritwangxiyuan proposed openstack/cinder-specs: Add ConsistencyGroup Into Quota Management  https://review.openstack.org/30694609:52
*** jwcroppe has joined #openstack-cinder09:55
openstackgerritwangxiyuan proposed openstack/cinder-specs: Add ConsistencyGroup Into Quota Management  https://review.openstack.org/30694609:55
*** jwcroppe has quit IRC10:00
*** cheneydc has quit IRC10:00
*** lpetrut has quit IRC10:03
*** sgotliv_ has quit IRC10:03
*** salv-orl_ has joined #openstack-cinder10:04
*** salv-orlando has quit IRC10:07
*** lpetrut has joined #openstack-cinder10:08
*** EinstCrazy has quit IRC10:09
*** yangyap__ has joined #openstack-cinder10:09
*** yangyape_ has quit IRC10:13
*** yangyap__ has quit IRC10:13
*** sdague has joined #openstack-cinder10:15
*** avishay has joined #openstack-cinder10:20
*** al_indigo has joined #openstack-cinder10:20
*** sgotliv_ has joined #openstack-cinder10:22
openstackgerritGorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts  https://review.openstack.org/30707410:26
openstackgerritGorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs  https://review.openstack.org/30707510:26
*** lpetrut has quit IRC10:41
*** sgotliv_ has quit IRC10:43
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image  https://review.openstack.org/21637810:43
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from consistency groups  https://review.openstack.org/25942910:43
openstackgerritGorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype  https://review.openstack.org/22144210:43
openstackgerritGorka Eguileor proposed openstack/cinder: Add ordering possibilities to conditional update  https://review.openstack.org/23193610:43
openstackgerritXu Ao proposed openstack/cinder: Repalce funcs Dict.get then del with just Dict.pop  https://review.openstack.org/30700310:49
*** jwcroppe has joined #openstack-cinder10:57
openstackgerritXu Ao proposed openstack/cinder: Replace funcs Dict.get then del with just Dict.pop  https://review.openstack.org/30700310:58
*** yangxi has quit IRC10:58
*** iceyao has quit IRC11:00
*** jwcroppe has quit IRC11:01
*** pck_ has joined #openstack-cinder11:01
*** avishay has quit IRC11:02
*** aorourke__ has joined #openstack-cinder11:02
*** marcusvrn__ has joined #openstack-cinder11:02
*** zhiyan_ has joined #openstack-cinder11:02
*** Lickitysplitted_ has joined #openstack-cinder11:02
*** cppforlife__ has joined #openstack-cinder11:02
*** serverascode_ has joined #openstack-cinder11:03
*** cebruns_ has joined #openstack-cinder11:03
*** wxy_ has joined #openstack-cinder11:03
*** rhefner_ has joined #openstack-cinder11:03
*** krotscheck_ has joined #openstack-cinder11:04
*** jgriffith_ has joined #openstack-cinder11:04
*** thurloat_ has joined #openstack-cinder11:04
*** tobasco_ has joined #openstack-cinder11:04
*** kfox1111_ has joined #openstack-cinder11:04
*** pocketprotector- has joined #openstack-cinder11:05
*** gnarld_ has joined #openstack-cinder11:05
*** jasondotstar_ has joined #openstack-cinder11:05
*** netapp_ has joined #openstack-cinder11:05
*** rhefner has quit IRC11:05
*** cppforlife_ has quit IRC11:05
*** marcusvrn_ has quit IRC11:05
*** melwitt has quit IRC11:05
*** wxy has quit IRC11:05
*** netapp has quit IRC11:05
*** cFouts has quit IRC11:05
*** jgriffith has quit IRC11:05
*** Lickitysplitted has quit IRC11:05
*** thurloat has quit IRC11:05
*** jasondotstar has quit IRC11:05
*** pck has quit IRC11:05
*** serverascode has quit IRC11:06
*** zhiyan has quit IRC11:06
*** rushiagr has quit IRC11:06
*** harlowja has quit IRC11:06
*** krotscheck has quit IRC11:06
*** pocketprotector has quit IRC11:06
*** cebruns has quit IRC11:06
*** tobasco has quit IRC11:06
*** kfox1111 has quit IRC11:06
*** aorourke_ has quit IRC11:06
*** jgriffith_ is now known as jgriffith11:06
*** thurloat_ is now known as thurloat11:06
*** wxy_ is now known as wxy11:06
*** rhefner_ is now known as rhefner11:06
*** jasondotstar_ is now known as jasondotstar11:06
*** jgriffith is now known as Guest7277211:06
*** netapp_ is now known as netapp11:06
*** krotscheck_ is now known as krotscheck11:06
*** marcusvrn__ is now known as marcusvrn_11:07
*** pocketprotector- has quit IRC11:07
*** pocketprotector has joined #openstack-cinder11:07
*** sivaramakrishna has quit IRC11:09
*** sweston has quit IRC11:09
*** cppforlife__ is now known as cppforlife_11:10
*** melwitt has joined #openstack-cinder11:10
*** zhiyan_ is now known as zhiyan11:10
*** melwitt is now known as Guest6400311:10
*** rcernin has quit IRC11:12
*** serverascode_ is now known as serverascode11:12
*** klkumar has quit IRC11:13
*** sweston has joined #openstack-cinder11:15
*** EinstCrazy has joined #openstack-cinder11:23
*** rcernin has joined #openstack-cinder11:24
*** ByPasS has quit IRC11:27
*** ByPasS has joined #openstack-cinder11:28
*** rushiagr has joined #openstack-cinder11:39
openstackgerritwangxiyuan proposed openstack/cinder: [WIP]Add ConsistencyGroup Into Quota Management  https://review.openstack.org/30702411:40
*** EinstCrazy has quit IRC11:40
openstackgerritYuriy Nesenenko proposed openstack/cinder: Deprecate nas_ip and change this to nas_host  https://review.openstack.org/30220711:41
*** liverpooler has quit IRC11:42
*** sgotliv_ has joined #openstack-cinder11:44
*** dave-mccowan has joined #openstack-cinder11:44
*** tpsilva has joined #openstack-cinder11:50
*** cdelatte has joined #openstack-cinder11:52
*** avishay has joined #openstack-cinder11:52
*** jwcroppe has joined #openstack-cinder11:58
*** yhayashi has quit IRC12:02
*** jwcroppe has quit IRC12:03
*** mtreinish has joined #openstack-cinder12:03
*** EinstCrazy has joined #openstack-cinder12:07
openstackgerritYuriy Nesenenko proposed openstack/cinder: Deprecate nas_ip and change this to nas_host  https://review.openstack.org/30220712:09
*** Poornima has joined #openstack-cinder12:10
*** raildo-afk is now known as raildo12:12
*** mylu has joined #openstack-cinder12:15
*** markvoelker has joined #openstack-cinder12:19
*** christian has quit IRC12:23
*** mylu has quit IRC12:26
*** edmondsw has joined #openstack-cinder12:26
*** Poornima_K has joined #openstack-cinder12:27
*** julim has joined #openstack-cinder12:30
*** Poornima has quit IRC12:30
*** qeelee has joined #openstack-cinder12:33
*** mylu has joined #openstack-cinder12:33
*** mylu has quit IRC12:39
*** baumann has joined #openstack-cinder12:43
*** gcb has quit IRC12:47
*** Poornima_K has quit IRC12:50
*** ankit has quit IRC12:52
*** smoriya_afk has joined #openstack-cinder12:53
*** lpetrut has joined #openstack-cinder12:53
*** houming has quit IRC12:53
sheelDuncanT: hi there12:55
sheelDuncanT: regarding comment on https://review.openstack.org/#/c/278605/35/cinder/volume/api.py12:56
sheelDuncanT:  could I take 2 min to discuss?12:56
*** jseiler_ has quit IRC12:56
*** rlrossit has joined #openstack-cinder12:56
DuncanTsheel: I'm in a meeting for the next 30 minutes12:57
sheelDuncanT: ok, no issus..catch you later12:58
*** qeelee has quit IRC12:59
*** eharney has quit IRC13:01
*** cbader has joined #openstack-cinder13:01
avishaysmcginnis: hey13:07
smcginnisavishay: Hi13:08
*** baumann has quit IRC13:08
avishaysorry about the confusion in the review for list manageable - i was referring to this: http://eavesdrop.openstack.org/irclogs/%23openstack-meeting/%23openstack-meeting.2016-04-13.log.html#t2016-04-13T16:20:4713:09
avishaywhen DuncanT said: "avishay: The move to core should probably be a different patch to your addition"13:09
smcginnisavishay: Ah, that was in reference to getting rid of the API extensions.13:10
smcginnisavishay: But all new API additions should be on v3 and with microversions.13:10
DuncanTEach extension can be moved in indemedently13:10
avishayI'm confused...13:10
*** laughterwym has joined #openstack-cinder13:11
DuncanTavishay: We want to move all the extensions into core13:12
avishayDuncanT: that i understand. but should i add these resources first, or later, or is it part of some effort to move them all?13:12
smcginnisavishay: New API additions will not be API extensions.13:13
smcginnisSo I think the point was, that part of the discussion is a separate effort.13:13
*** akerr has joined #openstack-cinder13:13
*** qeelee has joined #openstack-cinder13:13
DuncanTavishay: I think it is reasonable to ask that you put up the patch to move the extension into core at the same time you add to it - the move isn't massively complex13:15
*** asselin__ has joined #openstack-cinder13:15
*** qeelee has quit IRC13:15
avishayDuncanT: I have no problem doing the patch.  I just couldn't find any example in Cinder of microversions, or anything beyond what is in cinder/api/v3/router.py13:16
DuncanThttps://review.openstack.org/#/c/278605 i sthe best example I know, though it might not be perfect13:17
DuncanTWe haven't merged any new microversions yet13:17
akerrDuncanT: i think there was 1 merged, it conflicted with the user messages13:17
smcginnisYeah, I think one merged. This one still in process: https://review.openstack.org/#/c/298052/13:18
scottdaWe have merged a new microversion...mc_nair 's patch I think....13:18
*** eharney has joined #openstack-cinder13:18
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546913:18
*** qeelee has joined #openstack-cinder13:18
DuncanTAh, looks like I haven't run git fetch for a while :-)13:19
avishaySo wouldn't it make sense to add the resource after, so that it won't bump the version twice?  (once for adding the resources in the extensions, and another for adding the new actions?)13:19
scottdaThis is the 3.1 microversion patch: f8ce884013:21
*** jwcroppe has joined #openstack-cinder13:21
*** cknight has joined #openstack-cinder13:23
avishayscottda: can you help walk me through this?13:23
*** ndipanov has quit IRC13:23
scottdaavishay: Sure. You mean how to add a microversion?13:23
avishayscottda: I assume I need to start by adding a resource in cinder/api/v3/router.py?13:24
scottdaavishay: For starters, here's the devref: http://docs.openstack.org/developer/cinder/devref/api_microversion_dev.html13:24
*** ndipanov has joined #openstack-cinder13:25
scottdaavishay: Let me look at your patch....13:25
*** erhudy has joined #openstack-cinder13:25
*** iceyao has joined #openstack-cinder13:25
*** gouthamr has joined #openstack-cinder13:26
*** xyang1 has joined #openstack-cinder13:26
*** jwcroppe has quit IRC13:27
*** mriedem has joined #openstack-cinder13:28
avishayscottda: the code stays in /api/contrib?  I add an entry to router.py, and add @wsgi.Controller.api_version to all the relevant methods?13:28
scottdaThe change will come in api/contrib/volume_manager13:29
*** jwcroppe has joined #openstack-cinder13:30
scottdaYou can use the api_version_request Decorator or logic in anything that inherits from wsgi.Controller13:30
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546913:30
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Deprecating XML config file  https://review.openstack.org/29407613:30
scottdaavishay: So yes, I think you are correct.13:30
avishayscottda: OK, and should this patch go in before my current one, or after?  (i.e., two version bumps or one?)13:32
avishayscottda: I can put them both up now but want to know which should depend on which13:32
scottdaavishay: You should version bump at the same time. Otherwise, a user will have access to this via the /v2 endpoint without using microversions, and that is not what we want.13:32
scottdaavishay: And you should not have to change router.py13:33
scottdaavishay: Oh, to move the extension to core. I'd have to look at that more closely...13:35
avishayscottda: right now the user can access these resources only via /v2.  so the question is to allow accessing the resources with the existing actions via /v3 and then adding the new actions, or adding the actions to /v2 and then adding all the actions to /v3.13:36
openstackgerritMerged openstack/cinder: Fix service version pinning on start  https://review.openstack.org/30468513:36
*** diablo_rojo has joined #openstack-cinder13:36
*** rchurch has joined #openstack-cinder13:40
*** merooney has joined #openstack-cinder13:40
*** BigWillie has joined #openstack-cinder13:41
*** dustins has joined #openstack-cinder13:42
*** qeelee has quit IRC13:44
scottdaavishay: Yes, that's a good question. I'm not sure what's best offhand, and I'd like to get a discussion going on the best place to make the changes. I'm going to have to come back to this later today, as I've some conflicts the next couple hours.13:46
avishayscottda: OK, let me know how to proceed please13:47
scottdaavishay: Yes, sorry. You're a pioneer here in new microversions for extensions.13:47
openstackgerritMerged openstack/cinder-specs: Support backup import on another Storage database  https://review.openstack.org/24917613:51
avishayscottda: I didn't sign up for that ;P13:53
*** cknight has quit IRC13:53
*** baumann has joined #openstack-cinder13:54
*** eharney has quit IRC13:57
*** links has quit IRC13:57
scottdaavishay: You're telling me. I'm off to get a "Microversion" tattoo...13:57
avishayscottda: that's the best kind - you can change it easily13:58
*** jgregor has joined #openstack-cinder14:00
*** qeelee has joined #openstack-cinder14:01
scottdahaha...I never thought about that!14:02
smcginnisNice! ;)14:03
*** diablo_rojo has quit IRC14:03
*** diablo_rojo has joined #openstack-cinder14:04
*** ametts has joined #openstack-cinder14:04
openstackgerritMerged openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/30670814:06
avishayscottda: ha, got something working.  just added 3 lines to router.py, and the decorator to contrib/volume_manage.py.  when i set the version in the decorator to 3.0 i see the resource, when i set to 3.4 i get 404 error.14:06
*** arch-nemesis has joined #openstack-cinder14:07
avishayscottda: i'll finish the patch, then just let me know about the order14:07
scottdaavishay: Cool. And you can weigh in on your opinion of the order. I won't be making the decision, it'll come from any interested persons...so maybe just put it where you think it belongs and we can have a bit of a debate in the review.14:09
*** nkrinner has quit IRC14:10
scottdaavishay: Here's some (probably outdated) code I had for testing microversions with curl: https://github.com/scottdangelo/TestCinderAPImicroversions14:11
scottdaavishay: You might be able to use that to see if your changes are working..14:11
*** eharney has joined #openstack-cinder14:11
scottdaavishay: And here's in-flight patch(es) for addding support to the client: https://review.openstack.org/#/c/30194114:12
*** cknight has joined #openstack-cinder14:15
*** ebalduf has joined #openstack-cinder14:16
ameadegeguileo: https://review.openstack.org/#/c/306402 responded14:17
ameadeDuncanT: ^14:17
*** jseiler has joined #openstack-cinder14:21
dalgaafAnyone around to review os_brick patch: https://review.openstack.org/#/c/300560/ ?14:24
geguileoameade: Hi14:24
geguileoameade: I read your reply, but I don't understand what you mean about this being an option to avoid having too many threads...14:25
ameadegeguileo: I looked at the original patch trying to figure out why it is even a config option at the reasoning seemed to be to have it false when you have a lot of things deleting14:29
ameadeotherwise i imagine it would just always do them in separate threads14:29
ameadeI dont know if that is a legit problem or not14:30
geguileoameade: It should not be a problem14:30
ameadebut it does seem a bit odd to just spin up X threads14:30
geguileoameade: Why is it odd?14:30
geguileoameade: Imagine this scenario14:30
geguileoameade: Your service is running14:30
geguileoAnd you have 20 ongoing delete operations14:31
ameadecan that happen?14:31
geguileoWhen you start you want them to make sure they are actually deleting in the backend14:31
geguileoWhy couldn't there be 20 delete operations?14:31
geguileoWe don't have any restriction for that, do we? r:-??14:32
DuncanTameade: You can have an arbitrary number of ongoing deletes in theory14:32
ameademm14:32
DuncanTameade: And given how slow and CPU loaded backup service can get, pathalogical situations aren't actually that hard to hit14:33
DuncanTI can't remember why we put the default to False, but I remember I asked for the config option - sadly in the pre-channel-logging days which makes the answer hard to find14:34
smcginnisThat patch just moves the threading up to the whole processing instead of internally to the delete loop. Or did I miss something?14:35
geguileoProbably because existing behavior was not to start receiving operations in the service until everything was cleaned up14:35
DuncanTsheel: ping14:35
sheelDuncanT: hi14:35
geguileosmcginnis: The patch will make deletes serially instead of in parallel14:35
DuncanTsheel: You were asking me about your microverisons patch?14:35
sheelDuncanT: I got reply from scottda ... "3.2" seems confusing14:36
DuncanTgeguileo: It will also stop the startup being blocked by the deletes though14:36
sheelDuncanT: so changing that accordingly...14:36
geguileoDuncanT: But we can do that now just by changing configuration option14:36
DuncanTgeguileo: That might have been why there was a config option though14:37
DuncanTgeguileo: just trying ot think through the tenant visible behaviour difference14:38
ameade"backup service could not be found' is the error you get when all the backup services are 'down'14:38
ameadei think it should be up but not pick up requests until it's done, when this option is false14:39
ameadethe tenant gets that error14:39
DuncanTThat behaviour actually came after this patch I think. Requests used to just sit on the queue14:40
DuncanT(I dislike that behaviour, FWIW, I prefered things to get queued)14:40
geguileoameade: Have you tried with the lastest master?14:40
ameadethat's what queues are for lol14:40
*** crose has joined #openstack-cinder14:41
*** jgregor has quit IRC14:42
ameadegeguileo: yeah pretty sure i did both liberty and master14:42
ameadei can play with things more today, what do we hope to discover?14:42
geguileoameade: Because with this patch that just merged: https://review.openstack.org/#/c/304685/14:42
geguileoameade: Service may report up earlier than before14:43
*** merooney has quit IRC14:46
*** erlon_ has joined #openstack-cinder14:48
*** Guest64003 is now known as melwitt14:51
*** haplo37 has joined #openstack-cinder14:51
*** porrua has joined #openstack-cinder14:51
*** ivase has joined #openstack-cinder14:56
*** ociuhandu has joined #openstack-cinder14:59
*** bjornar has quit IRC14:59
*** merooney has joined #openstack-cinder15:00
*** pots has joined #openstack-cinder15:01
bswartzanyone seen smcginnis?15:05
smcginnisbswartz: I have.15:06
bswartzo_O15:06
smcginnis:P15:06
bswartzwow you're not in the channel list -- at least not in my client15:06
* bswartz slaps Pidgin around a bit with a large trout15:06
smcginnisbswartz: Stealth mode I guess. :) What's up?15:06
bswartzdo you guys do blueprints first, specs second? or specs first, blueprints second?15:07
*** qeelee has quit IRC15:07
avishaybswartz: smcginnis is special - in my client he's on top15:07
smcginnisbswartz: Typically specs first, but I just recently found out that's at least backwards from how Nova does it.15:07
bswartzoh you're right avishay!15:07
* smcginnis feels special :)15:07
bswartzhe has +o so he's not alphabetical15:07
avishayyep, i was looking earlier today :)15:08
*** shausy has quit IRC15:08
smcginnisbswartz: How does manila handle specs/bps?15:08
bswartzsmcginnis: we don't have a specs repo and specs are done in wikis only if needed15:08
bswartzso we always start with a BP and then link the spec if necessary15:09
*** jgregor has joined #openstack-cinder15:09
smcginnisbswartz: Oh nice. That works well for you?15:09
bswartzwell it's process-light, which makes me happy15:09
smcginnisbswartz: A little less of that pesky "process" I know you like so much. :)15:09
bswartzit has the downside of making it slightly harder to provide feedback on specs15:09
smcginnisSo at least for us, BPs approval is basically saying, sure that looks like not a terrible idea.15:10
bswartzI'm asking about cinder because I want to contribute a feature to cinder15:10
smcginnisThen if it needs a spec we debate details there.15:10
bswartzokay15:10
smcginnisNova does spec first I believe, then approves the BP once that's squared away.15:10
smcginnisbswartz: Awesome, anything cool you're looking to add?15:11
*** julim has quit IRC15:11
bswartzyes I want to make the scheduler smarter15:11
smcginnisbswartz: Seems like a good area for potential improvement.15:11
bswartzIMO the scheduler does pathological things in a lot of cases and the admin has to be careful to avoid those scenarios15:12
smcginnisbswartz: Ping me when you file the BP and I can take a look.15:12
bswartzok15:12
smcginnisbswartz: Thanks!15:12
*** garthb has joined #openstack-cinder15:12
bswartzsmcginnis: one more thing -- where is your spec template?15:13
* smcginnis goes looking...15:13
* bswartz derp15:13
bswartzI see it in the repo15:13
smcginnisOK, good. I thought it was there.15:13
*** merooney has quit IRC15:14
*** julim has joined #openstack-cinder15:15
*** karthikp has joined #openstack-cinder15:16
openstackgerritJohn Griffith proposed openstack/cinder: Implement replicate_volume & failover on SolidFire  https://review.openstack.org/29974715:17
openstackgerritJohn Griffith proposed openstack/cinder: Add failed_over detection to SolidFire driver  https://review.openstack.org/29974815:18
openstackgerritJohn Griffith proposed openstack/cinder: Handle TFlow short comings in SF driver  https://review.openstack.org/30007815:18
*** iceyao has quit IRC15:19
*** links has joined #openstack-cinder15:19
*** mjura has quit IRC15:20
*** abehl has quit IRC15:20
openstackgerritAvishay Traeger proposed openstack/cinder: List manageable volumes and snapshots  https://review.openstack.org/28529615:21
openstackgerritAvishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3  https://review.openstack.org/30732415:21
*** jungleboyj has joined #openstack-cinder15:21
avishayscottda, smcginnis ^^15:21
avishayDuncanT ^^15:21
DuncanTavishay: Looking, thanks15:22
avishayDuncanT: thank _you_15:24
*** mtanino has joined #openstack-cinder15:25
*** Guest72772 is now known as jgriffith15:28
jgriffithsdake: Umm... is this right?  You require invites to your IRC channel?15:29
jgriffithsdake: I received a highlight and an error message saying it's an invite only channel?15:30
* jgriffith is confused15:30
avishaycrap, how did you get in? ;)15:31
jgriffithscottda: when you get a chance can you explain what all this means and how it works?  (versions, V3 etc)15:32
jgriffithor maybe avishay since you seem to have figured it out15:32
openstackgerritAvishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3  https://review.openstack.org/30732415:33
avishayjgriffith: what exactly do you want to know?15:33
jgriffithavishay: what the F is the process now :)15:33
jgriffithavishay: when do I need to add that, what do I bump it to when I add it etc etc15:34
jgriffithavishay: how do I make shit work when I run stack.sh15:34
avishayjgriffith: any API change requires bumping the microversion, more details here: http://docs.openstack.org/developer/cinder/devref/api_microversion_dev.html15:34
*** e0ne has quit IRC15:34
avishayjgriffith: cinder client doesn't support v3 yet (scottda has a patch up), but you can use curl15:35
jgriffithavishay: yeah, once a process requires more than 3 pages to describe my ADD kicks in and I never get the gist of it15:35
jgriffithI'll try again15:35
*** Guest15381 is now known as medberry15:35
avishayjgriffith: in curl, add this to the end (put whatever version you want): -H "OpenStack-API-Version: volume 3.2"15:35
jgriffithavishay: sort of log the logging nonsense :)15:35
*** medberry is now known as med_15:35
jgriffithavishay: excellent, thanks!15:35
avishayjgriffith: i figured it out today the hard way, let me know if you get stuck. i'm not expert, but can sometimes fake it.15:36
jgriffithavishay: I'll take somebody that made it work over an "expert" any day :)15:36
avishay:)15:36
*** pots has quit IRC15:36
jgriffithavishay: and careful, now you'll be an "expert" yourself15:37
*** Kimmo_ has quit IRC15:38
*** crose has quit IRC15:38
*** jklare has quit IRC15:39
avishayjgriffith: i will deny everything15:40
scottdajgriffith: To test /v3 endpoint with cinderclient you can pull this patch: https://review.openstack.org/#/c/30002815:40
avishayscottda: BTW, the link you sent to your github had: -H "OpenStack-Volume-microversion: 3.1"15:41
scottdajgriffith: To use microversions in cinderclient cherry-pick this on top: https://review.openstack.org/#/c/30194115:41
avishayscottda: But that didn't work for me, this did:  -H "OpenStack-API-Version: volume 3.1"15:41
jgriffithscottda: got it, avishay mentioned it.. thanks!15:41
jgriffithscottda: oh... wait15:41
scottdaavishay: Yes, sorry, that github code is a little out of date.15:41
*** jklare has joined #openstack-cinder15:41
avishayscottda: no worries, just wanted to make sure i didn't do something wrong :)15:43
jgriffithscottda: hehe... that's what I was missing15:43
jgriffithok, cool... tahnks scottda and avishay15:43
avishayyou're welcome (still not an expert)15:43
jgriffithscottda: are you going to push the fixes geguileo pointed out?15:43
jgriffithI'd like to get that stuff merged asap if we could15:43
jgriffithrather than messing with cherry-picks and blah blah blah15:43
scottdajgriffith: Yes, today I'll update patches.15:43
jgriffithscottda: awesome15:44
scottdajgriffith: Amen to that. Reviews and testing are very welcome.15:44
*** lprice has joined #openstack-cinder15:44
jgriffithscottda: well.. I was going to just +2/A it as it is15:44
jgriffithscottda: but I've been told to stop doing that so I just don't do reviews much any more ;)15:45
*** lprice1 has quit IRC15:48
*** harlowja_at_home has joined #openstack-cinder15:48
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement unmanage volume for RBD driver  https://review.openstack.org/30734215:50
*** arecknag has quit IRC15:54
*** apoorvad has joined #openstack-cinder15:54
*** avishay has quit IRC15:59
*** adrianofr has joined #openstack-cinder16:00
*** links has quit IRC16:01
*** andymaier has joined #openstack-cinder16:04
*** salv-orlando has joined #openstack-cinder16:04
*** Guest30394 has quit IRC16:05
openstackgerritSheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list  https://review.openstack.org/27860516:05
*** andymaier has quit IRC16:06
*** salv-orl_ has quit IRC16:06
*** martyturner has joined #openstack-cinder16:06
*** karthikp has quit IRC16:09
*** karthikp has joined #openstack-cinder16:09
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088516:13
ntpttrjgriffith: in case you still weren't sure, the #kolla channel moved to #openstack-kolla. I think #kolla is invite only but the new one is public16:19
*** goretoxo has quit IRC16:20
*** sgotliv_ has quit IRC16:20
*** enriquetaso has joined #openstack-cinder16:23
*** harlowja_at_home has quit IRC16:26
*** kfox1111_ is now known as kfox111116:32
*** jwcroppe_ has joined #openstack-cinder16:32
*** jwcroppe has quit IRC16:35
SwansonBreaking news. The new company name for the post Dell EMC merger is to be Dell McDellface.  That is all.16:35
sbezverkjgriffith hey, do you have a minute?16:38
*** jistr has quit IRC16:39
*** jwcroppe has joined #openstack-cinder16:40
*** jwcroppe_ has quit IRC16:42
*** itlinux has joined #openstack-cinder16:42
*** rcernin has quit IRC16:43
bswartzSwanson: lol16:44
openstackgerritGorka Eguileor proposed openstack/cinder-specs: Resource cleanup to support HA A/A  https://review.openstack.org/23697716:45
*** EinstCrazy has quit IRC16:49
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Add /v3 endpoint support for cinderclient  https://review.openstack.org/30002816:49
scottdajgriffith: I've addressed geguileo 's comments ^^^ for cinderclient /v3 endpoint support16:50
*** vgridnev_ has joined #openstack-cinder16:52
*** laughterwym has quit IRC16:53
geguileoscottda: Thanks!16:57
mriedemhas there been any effort in cinder to recover/rollback from failed volume migrations during the swap volume portion of the flow in nova? https://bugs.launchpad.net/nova/+bug/155063916:59
openstackLaunchpad bug 1550639 in OpenStack Compute (nova) "After migrate volume being attached instance, the instance cann't run normally" [Undecided,Incomplete] - Assigned to YaoZheng_ZTE (zheng-yao1)16:59
geguileoscottda: Done17:00
mriedemalso, how does the cinder volume migration flow track the status of the swap volume operation in nova to see if it failed and the volume migration should be rolled back?17:00
sheelSwanson: Dell McDellface .. this seems something known.... ;)17:01
*** rdx_matt has quit IRC17:07
*** kfarr has joined #openstack-cinder17:10
*** harlowja has joined #openstack-cinder17:11
*** karthikp has quit IRC17:11
sdakejgriffith that isn't right17:12
sdakejgriffith your tryign to join #openstack-kolla?17:12
openstackgerritMerged openstack/cinder: Coding style issue  https://review.openstack.org/30554717:13
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376617:16
*** bjornar has joined #openstack-cinder17:16
jgriffithsdake: I'm in openstack-kolla, but I received a highlight from #kolla, which apparantly exists and is invite-only?  Anyway... no big deal, just was odd17:18
dimsjgriffith : that's how infra marks them as unusable17:18
jgriffithdims: yeah, I dunno.. just couldn't figure out how I got a highlight17:20
jgriffithoh well17:20
sdake#kolla should forward you to openstack-kolla17:20
jgriffithsdake: oh17:20
jgriffithsdake: ok17:20
jgriffithnow I see what you guys were saying17:20
jgriffithI tihnk17:20
sdakeso basically nowhere in thatcall path for dta is an exceeption raised17:21
sdakeonly none is returned17:21
sdakeso no exception = no retry17:21
sdakei dont know about delete or other calls17:21
sdakethey may raise exceptions in which case my patchset 5 was wrong17:21
sdakei ont know if they used to raise exceptions17:22
sdakeyou seem to imply getattr magically raised a ImageNotFound exceptions17:22
sdakebut i dont see how that is possible with current master17:22
*** leeantho has joined #openstack-cinder17:24
*** pots has joined #openstack-cinder17:24
*** Suyash has joined #openstack-cinder17:26
openstackgerritGorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs  https://review.openstack.org/30707517:28
openstackgerritGorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts  https://review.openstack.org/30707417:28
*** ChubYann has joined #openstack-cinder17:39
*** stevemar has quit IRC17:41
*** lpetrut has quit IRC17:43
*** Kimmo_ has joined #openstack-cinder17:43
*** stevemar has joined #openstack-cinder17:44
*** bardia has joined #openstack-cinder18:00
*** angela-s has joined #openstack-cinder18:06
openstackgerritBen Swartzlander proposed openstack/cinder-specs: Add Stochastic Weighing Scheduler  https://review.openstack.org/30740118:10
*** sgotliv_ has joined #openstack-cinder18:10
bswartzsmcginnis, cknight: ^18:10
Swansonbswartz, I don't get the issue.  The load on the new backends will only be allocating volumes.  The traffic load will still be mostly to the old backends.18:12
Swansonbswartz, plus initial startup...18:12
*** salv-orlando has quit IRC18:13
*** salv-orlando has joined #openstack-cinder18:13
*** pots has quit IRC18:15
bswartzSwanson: that depends entirely on traffic patterns18:17
bswartzin some uses cases volumes are short lived, and in others they're long lived18:17
*** salv-orlando has quit IRC18:18
bswartzeven with long lived volumes the read/write mix over time is highly dependent on the application18:18
openstackgerritKendall Nelson proposed openstack/os-brick: Split out factory method  https://review.openstack.org/30244518:18
Swansonbswartz, I should probably finish reading your spec before commenting.18:19
*** pots has joined #openstack-cinder18:20
*** lpetrut has joined #openstack-cinder18:22
anteayajgriffith: ola18:22
anteayajgriffith: so pure storage ci18:22
*** ntpttr_ has joined #openstack-cinder18:22
* patrickeast debates wether to run away18:23
anteayajgriffith: it is commenting on the sandbox repo, would you mind pointing it to the openstack-dev/ci-sandbox repo please?18:23
patrickeastanteaya: oh, yea i can switch that18:23
anteayapatrickeast: your fine :)18:23
anteayapatrickeast: oh are you pure storage?18:23
anteayasorry I thought that was jgriffith18:23
anteayamy mistake18:23
patrickeastanteaya: yep yep, jgriffith is solidfire ci's owner18:23
patrickeastanteaya: no worries18:23
anteayaoh yeah, silly me18:23
anteayasorry jgriffith18:23
anteayathanks patrickeast18:23
anteayaanyone know emc isilon ci?18:24
smcginnisWe can still bug jgriffith with it. He loves CIs.18:24
patrickeastxyang1: might know ^^18:24
smcginnisxyang1: Run!! ^^18:24
patrickeastlol18:24
anteayasmcginnis: never let me stand in the way of you bugging jgriffith18:24
anteayahttps://review.openstack.org/#/c/290505/218:24
smcginnis:)18:24
anteayawe are trying to clean up the sandbox repo18:24
anteayafor new devs and all18:24
*** salv-orlando has joined #openstack-cinder18:24
patrickeastgotcha18:25
xyang1anteaya: hi, what's up18:25
anteayaxyang1: hi18:25
anteayaxyang1: are you associated with emc isilon ci?18:25
xyang1anteaya: yes18:26
anteayawonderful18:26
anteayaxyang1: mind pointing it at openstack-dev/ci-sandbox please?18:26
xyang1anteaya: what's wrong?18:26
anteayainstead of openstack-dev/sandbox, which is where it is now?18:26
anteayasandbox is for new developers18:26
xyang1anteaya: it is for Manila18:27
anteayapleia2: is having a lunch and learn at summit18:27
anteayaand will be using the sandbox repo for new devs18:27
anteayaxyang1: that is fine, cis should be using the ci-sandbox repo for testing18:27
anteayanot the sandbox repo18:27
patrickeastanteaya: we don't want new developers to see examples of 3rd party ci issues? :D18:27
anteayacomment to your heart's content on ci-sandbox18:27
* patrickeast is mostly joking18:27
anteayapatrickeast: what do you think? :)18:28
pleia2not ones that aren't working and so are still living in the sandbox repo ;)18:28
xyang1anteaya: I see what you are saying.  We will change it18:28
pleia2thank you18:28
anteayaxyang1: thanks18:28
anteayanot saying don't test18:28
xyang1anteaya: pleia2 no problem18:28
anteayasaying test over here please, on ci-sandbox18:28
anteayaxyang1: thank you18:29
anteayalast is lennyb18:29
anteayaI'll talk to him at 0800 tomorrow in the third party meeting18:29
openstackgerritAlex Meade proposed openstack/cinder: User messages API for error cases  https://review.openstack.org/29805218:30
openstackgerritAnthony Lee proposed openstack/python-brick-cinderclient-ext: Query volume paths implementation  https://review.openstack.org/26840518:32
*** jwcroppe has quit IRC18:34
*** jwcroppe_ has joined #openstack-cinder18:36
*** rdx_matt has joined #openstack-cinder18:38
*** lpetrut has quit IRC18:39
*** ntpttr_ has quit IRC18:40
*** jwcroppe has joined #openstack-cinder18:41
*** jwcroppe_ has quit IRC18:44
*** lpetrut has joined #openstack-cinder18:45
*** jwcroppe_ has joined #openstack-cinder18:46
*** jwcroppe has quit IRC18:48
*** jwcroppe has joined #openstack-cinder18:48
*** jwcroppe_ has quit IRC18:51
*** ociuhandu has quit IRC18:52
*** mtecer has joined #openstack-cinder18:54
*** lpetrut has quit IRC18:55
mtecerHi guys, I am having issues with bootable cinder volumes, backing image has hw_disk_bus set to scsi. Instance boots with no issues, however instance details has vda as root device, and this causes issues with attaching more volumes. Any ideas ?18:58
smcginnismtecer: Hmm, might be a good question for Nova. Other than recreate your image, I'm not sure what to suggest.18:59
mtecersmcginnis: Thank you. I will ask in that room as well19:00
*** ametts has quit IRC19:00
jgriffithOh swell.. now I get spam from gliffy19:00
smcginnismtecer: Good luck!19:00
smcginnisjgriffith: Lovely, ain't it.19:00
jgriffithsigh19:01
*** jwcroppe_ has joined #openstack-cinder19:01
*** fernnest_ has joined #openstack-cinder19:02
*** jwcroppe has quit IRC19:03
*** rdx_matt has quit IRC19:03
*** jwcroppe has joined #openstack-cinder19:09
*** jwcroppe_ has quit IRC19:12
*** crose has joined #openstack-cinder19:14
*** sheel has quit IRC19:15
*** ametts has joined #openstack-cinder19:17
*** lpetrut has joined #openstack-cinder19:17
*** lpetrut has quit IRC19:29
*** ociuhandu has joined #openstack-cinder19:32
*** geguileo has quit IRC19:39
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Support api-microversions  https://review.openstack.org/30194119:45
*** salv-orlando has quit IRC19:47
*** enriquetaso has quit IRC19:48
jgriffithanybody willing to put a +2/A back on this last one so I don't have to rebase everything again :) https://review.openstack.org/#/c/299747/19:50
smcginnisjgriffith: Looking19:50
smcginnisjgriffith: Any idea how far behind your CI is running?19:51
jgriffithhmm19:51
jgriffithyeah, lemme just recheck now19:52
smcginnisjgriffith: NM, I see you just fixed some minor (but good!) things since it was approved.19:52
smcginnisGood enough for me.19:52
openstackgerritKendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration  https://review.openstack.org/28623419:52
jgriffithsmcginnis: thanks, should kick off here shortly19:52
*** Lee1092 has quit IRC19:52
*** e0ne has joined #openstack-cinder19:52
jgriffithsmcginnis: and I just added a retry around that apt-get update so that problem should hopefully go away19:52
smcginnisjgriffith: You've been getting apt-get failures?19:53
jgriffithYES19:53
jgriffithintermittently apt-get update keeps failing19:53
jgriffithabout 1 in 1019:53
smcginnisjgriffith: I finally figured out setting up a local apt-mirror for apt and bandersnatch for pip. It's very nice having a local repo to pull from.19:53
jgriffithwas talking to Chris here and he's been seeing the same thing19:53
smcginnisjgriffith: Reduces the network load quite a bit too.19:54
jgriffithsmcginnis: yeah.. I used to have one, but rebuilt things and never put it back19:54
bswartzjgriffith: apt-cacher-ng = win19:54
jgriffithI need to do that again... soooo much faster and more reliable19:54
openstackgerritKendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration  https://review.openstack.org/28623419:54
smcginnisbswartz: Never got it to work right, but I'm sure it was just me.19:54
bswartzour corporate firewall just eats deb packages for lunch19:54
smcginnis:)19:54
jgriffithbswartz: tastie19:54
smcginnisjgriffith: Was that when all your lab equipment was "reappropriated"?19:55
jgriffithha.. nah... this was self inflicted19:55
smcginnis:)19:55
openstackgerritKendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration  https://review.openstack.org/28623419:55
jgriffithI thought I'd be *smart* and upgrade to Liberty and Neutron19:55
jgriffithIt's been less than ideal19:55
jgriffithparticularly it seems to suck in multi-node setups19:55
smcginnisI really need to upgrade as well.19:55
jgriffithsmcginnis: DONT19:56
jgriffithtrust me19:56
jgriffithI spent a week finding various things19:56
jgriffithincluding changes in allocation ratios on Nova19:57
smcginnisHaha, maybe in Ocata I'll pull the trigger. ;)19:57
jgriffithif you do upgrade, let me know and I'll compile a list :)19:57
patrickeasti've had problems with upgrading mine19:57
smcginnispatrickeast: Are you still a mix of nova-network in one and neutron in the other?19:57
patrickeastended up being easier to back up all the vm images and just do a clean install : /19:57
jgriffithpatrickeast: yeah.. I had to explicitly set all of the allocation_ratio vars19:57
*** delattec has joined #openstack-cinder19:57
*** martyturner has quit IRC19:57
patrickeasteverything is neutron, but i had to do all-in-one setups19:57
jgriffiththen I'd have funny things like all of a sudden fixed ip's weren't routable19:58
patrickeasti *think* i can do it without that with the distributed router thingy now19:58
patrickeastbut haven't tried it19:58
jgriffithor even better, secondary compute nodes on neutron would all of a sudden just NOT be reachable19:58
jgriffithpatrickeast: yes, I had to revert back to an all-in-one19:58
jgriffith:(19:58
jgriffithvery very sad19:58
patrickeastat some point i might just say screw it and put iscsi traffic through my neutron controller...19:59
jgriffithand even that all in one, I have no f'ing idea if it's actually right or not19:59
smcginnisOuch19:59
patrickeastand reclaim the like 12G of ram that the all-in-one costs on each compute node19:59
*** crose has quit IRC19:59
*** cdelatte has quit IRC19:59
jgriffithpatrickeast: ha.. I was thinking at some point I may say screw it and write my own networking plugin for Nova :)19:59
patrickeastlol19:59
jgriffithie nova-net19:59
jgriffithwhich we seem to be abandoning and leaving for dead even though it works for 90% of us20:00
patrickeastjgriffith: i looked into the linux bridge neutron setup... in theory it looks very nice20:00
patrickeastjgriffith: does almost the same thing that nova-net did20:00
jgriffithpatrickeast: yeah.. in theory it does :)20:00
jgriffithbeen there... beat my head against the wall20:00
patrickeastouch20:00
jgriffithpatrickeast: I do think it's the answer20:00
jgriffithpatrickeast: just can't get the f'ing thing to be reliable20:01
patrickeastjgriffith: you think thats a design issue or implementation?20:01
openstackgerritMerged openstack/cinder: Cleanup volume_attachment in case of reset-state attach_status  https://review.openstack.org/30355920:01
jgriffithdeploy 3 times, get 3 different results20:01
patrickeasthaha yea, all of my nodes have had special issues20:01
patrickeastand special fixes20:01
jgriffithor my favorite, have things suddenly just "stop routing" with no indication as to why20:01
jgriffithpatrickeast: it's really quite sad and disappointing IMO20:01
*** ntpttr_ has joined #openstack-cinder20:02
jgriffithssh hotbox20:02
smcginnisNetwork compression. ;)20:02
*** delattec has quit IRC20:02
*** sdake has quit IRC20:07
*** ntpttr_ has quit IRC20:07
*** delattec has joined #openstack-cinder20:07
*** e0ne has quit IRC20:08
smcginnisjgriffith: Hah, just saw you ML post on the local.conf files.20:09
akerrjgriffith: we do run apt-cacher and devpi but were still also seeing apt-get failures late last week20:09
Swansonjgriffith, what stopped routing with no indication as to why?20:09
smcginnisSwanson: Multinode neutron setup.20:11
akerrupgraded our cache server and rebooted it a few times and the issue seemed to go away20:11
openstackgerritXinli Guan proposed openstack/cinder: IBM FlashSystem: Cleanup host resource leaking  https://review.openstack.org/30745120:11
akerrthe old turn off and on method of trouble solving20:12
Swansonsmcginnis, Oh, joy. One of the things on my todo list.20:12
*** akerr has quit IRC20:13
jgriffithSwanson: networks20:14
jgriffithSwanson: private net20:14
jgriffithSwanson: random IP's in the range would just not route20:14
jgriffithSwanson: also ran into all sorts of issues when Instances ended up on a secondary/tertiary compute node20:14
openstackgerritKendall Nelson proposed openstack/os-brick: Split out factory method  https://review.openstack.org/30244520:15
jgriffithsmcginnis: :)20:16
jgriffithsmcginnis: funny timing eh :)20:16
*** angela-s has quit IRC20:17
*** lpetrut has joined #openstack-cinder20:17
*** lpetrut has quit IRC20:26
*** salv-orlando has joined #openstack-cinder20:33
*** BigWillie has quit IRC20:34
openstackgerritKendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration  https://review.openstack.org/28623420:37
*** e0ne has joined #openstack-cinder20:38
*** ntpttr_ has joined #openstack-cinder20:45
*** eharney has quit IRC20:48
*** cknight has quit IRC20:50
*** e0ne has quit IRC20:50
openstackgerritKendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum  https://review.openstack.org/29088520:51
jgriffithno more rejoin :(20:54
Swansonthe late and lamented rejoin...20:55
jungleboyjjgriffith: You mean for devstack?20:55
jgriffithjungleboyj: yes20:57
jungleboyjjgriffith: Yeah, that confused the hell out of me when it disappeared.  :-(20:57
jungleboyjIt is missed.20:57
SwansonIt is missed.20:57
* jungleboyj hears taps in the distance.20:58
jgriffithanybody know when/why it was removed?20:58
jgriffithI'm looking through git now20:58
jungleboyjjgriffith: It was removed back in like January I think.20:58
jgriffithjungleboyj: no20:58
jungleboyjjgriffith: Reasoning was that it didn't work reliably.20:59
jgriffithjungleboyj: couldn't have been that long ago, I just built this machine last week20:59
jgriffithjungleboyj: and it's completely reliable when using screen :)20:59
jungleboyjjgriffith: I know. I never had any issues with it.20:59
patrickeastwait what? theres no rejoin?20:59
jgriffithpatrickeast: nope20:59
*** sdake has joined #openstack-cinder21:00
jungleboyjjgriffith: I am sorry, you are right.  It was on Mar 14.21:00
patrickeastjgriffith: weird, so what we just do manual screen commands to get in there now?21:00
SwansonIt has been gone for longer than a week.21:00
jungleboyjFeels like longer ago I hit that issue.21:00
* patrickeast needs to stop taking vacation days21:00
jungleboyjpatrickeast: Yep.21:00
jgriffithhmm... weird then21:00
*** jgregor has quit IRC21:00
*** rdx_matt has joined #openstack-cinder21:01
*** jgregor has joined #openstack-cinder21:01
mc_nairjgriffith: yea, not sure why either. I still just check rejoin-stack out from stable/liberty cause I miss it so.21:04
jungleboyjmc_nair: :-)  Good idea.21:04
*** raildo is now known as raildo-afk21:05
smcginnisYeah, I don't get the "unreliable" argument.21:05
smcginnisIn a few corner cases it might not work right, so no one gets to use it?21:05
mc_nairdon't know about good idea, since I'd guess it was removed for a reason. But it's at least a good way to stop the separation anxiety21:06
smcginnismc_nair: Hah! ;)21:06
*** rdx_matt has quit IRC21:06
jungleboyj:-)21:07
jgriffithsmcginnis: patrickeast jungleboyj http://paste.openstack.org/show/494516/21:07
jgriffithnot exactly last week, but not january either21:07
jungleboyjjgriffith: Yeah.  Sorry, don't know why I was thinking January.  TIme is a confusing thign tright now for me.  :-)21:08
*** kfarr has quit IRC21:08
*** ametts has quit IRC21:08
jgriffithhaha.. I hear that!21:08
*** porrua has quit IRC21:09
*** jgregor has quit IRC21:09
*** gouthamr has quit IRC21:09
jgriffithwell, I now have a hook to add that file each time I clone devstack21:10
jgriffithwe'll see how long before it's rendered non-functional21:10
*** ametts has joined #openstack-cinder21:17
*** julim has quit IRC21:20
*** baumann has left #openstack-cinder21:21
*** mylu has joined #openstack-cinder21:22
*** rlrossit has quit IRC21:22
*** mylu has quit IRC21:24
*** mylu has joined #openstack-cinder21:25
sbezverkjgriffith ping21:29
jgriffithsbezverk: hi21:29
sbezverkjgriffith hi, how is it going?21:29
sbezverkjgriffith any ideas how to make this fix working for us ;-)??21:29
jgriffithsbezverk: well... I was going to use steves line "I'm booked solid for the next 3 weeks"21:30
jgriffithsbezverk: but...21:30
jgriffithI am kind of in the middle of some things21:30
sbezverkjgriffith that is fine, we have a temp solution with Steve's version21:30
jgriffithsbezverk: I'm not sure how that can be used21:31
jgriffithsbezverk: I guess cinder doesn't regularly do things like delete so maybe that's why it's *ok*21:31
jgriffithsbezverk: I'm also not completely clear why that version I pushed fails still21:31
jgriffithI couldn't quite gather the problem from the comments21:32
sbezverkjgriffith got it, I guess we wil have to live with this evil until you guys figure out solid solution, without any solution we lose half of VMs21:32
jgriffithsbezverk: I'll try and look at it again tonight21:32
jgriffithsbezverk: I'm going to need your help though I think to understand which call exactly is failing (which method in the gettattr call that is)21:33
sbezverkjgriffith if you need me to setup debug session, just let me know21:33
patrickeastjgriffith: sbezverk was that the glance thing from last week?21:33
sbezverkpatrickeast yes21:33
sbezverkpatrickeast the solution you suggested is not used in kolla21:33
*** dustins has quit IRC21:33
jgriffithpatrickeast: https://review.openstack.org/#/c/306756/21:33
patrickeast:D which solution? using swift?21:34
sbezverkpatrickeast instead glance have a list of api servers and clients need to check one by one for the image before failing..21:34
sbezverkpatrickeast no using image replicator21:34
patrickeastoh yea21:34
patrickeastthat kind of sucks that it needs to potentially hit all the servers21:35
patrickeastbut i guess if it works, it works21:35
jgriffithpatrickeast: sbezverk I've been mulling over the comment about not actually needing the internal raise there21:35
sbezverkpatrickeast I agree but it is the way it is :-)21:35
*** diablo_rojo has quit IRC21:36
sbezverkjgriffith it is better to check with Steve I did not get his exact implementation..21:36
sbezverkI meant with sdake21:36
jgriffithsbezverk: well, I understood his first impl and it makes sense, until you try and do a "delete" call :(21:37
jgriffithwhich returns None21:37
jgriffithso his code caused that to call to raise an exception21:37
jgriffithI'm unclear on why my check of method being called results in the data get failure21:37
jgriffith*unless* it's the iterator on the download just reaching the end21:37
jgriffithhmmm21:38
sbezverkjgriffith to prove it I remove 'data' from the list and it started working21:38
patrickeastwasn't the problem that we were calling data and it returned None?21:38
jgriffithsbezverk: well, in that case :)21:38
patrickeastso we do expect it to do it sometimes21:38
jgriffithsbezverk: that's a significant help for me21:38
smcginnissbezverk: Is that a valid configuration to have multiple nodes but not sync the images?21:39
jgriffithsmcginnis: we could've had HA without all the patches DOH!!!21:39
jgriffithjust don't sync things :)21:40
*** jungleboyj has quit IRC21:40
sbezverk@smcginnis yep, glance offers this along with the list of api servers21:40
sbezverkwhere you can find the image..21:40
jgriffithI suppose I can remove data from that list...21:41
jgriffithI'm assuming that if the image isn't found we'll never get to the download part21:41
jgriffithunless there's a chance that it queries one server, then tries downloading from another in the middle of the sequence?21:41
sbezverkjgriffith right the final exception should be triggered only when all aip servers returned no image found..21:42
jgriffithI guess that also introduces a narrow race21:42
jgriffithI'm willing to propose it, but I'd like some good reviews and hopefully some people will test it fully21:42
jgriffithit's early enough in the release cycle that I'm somewhat comfortable with that21:42
smcginnisThis seems kind of hackish to do any of this on our end.21:44
sbezverkjgriffith From my side I will beat something out pf it ;-)21:44
smcginnisEither the admin needs to sync their nodes...21:44
smcginnisor Glance should have built in smarts to give us the right image no matter where it is.21:44
jgriffithsmcginnis: I'm curious if Nova runs into this?21:45
jgriffithI don't quite understand how they couldn't21:45
patrickeastanywhere they ask for a glance image they should21:45
smcginnisjgriffith: Good point!21:45
patrickeastunless they already have code like this21:45
jgriffithpatrickeast: nope21:45
jgriffithpatrickeast: unless they're checking it somewhere else, which is possible21:45
*** haplo37 has quit IRC21:46
*** sgotliv_ has quit IRC21:46
jgriffithpatrickeast: but the glance.py file is the same one I "leveraged" years ago21:46
patrickeasthaha21:46
patrickeastjgriffith: gotcha21:46
smcginnisHah, "leveraged".21:46
jgriffithpatrickeast: although there's so much indirection with everyone's cleverness any more who knows21:46
openstackgerritJohn Griffith proposed openstack/cinder: Retry when glance returns None on fetch  https://review.openstack.org/30675621:46
patrickeastjgriffith: true, plus they do a bunch of image caching too if i understand correctly21:46
jgriffithpatrickeast: yeah, I just put that in my comment :)21:48
sbezverkjgriffith the reason glance does it this way is because it is file backend.. I suspect when backend will be swift, it will probably behave much bettwe21:48
jgriffithwondering if the cache avoids the issue21:49
patrickeastjgriffith: it would assuming you get the image on the first try21:49
*** _alastor_ has joined #openstack-cinder21:50
patrickeastjgriffith: same thing would happen with our image-volume cache21:50
jgriffithpatrickeast: yeah21:50
patrickeastjgriffith: or even if you keep trying through the failures until you get one to work21:50
patrickeastjgriffith: then you are in business21:50
jgriffithsbezverk: oh... well that's an interesting point21:50
jgriffithpatrickeast: yeah... maybe21:50
jgriffithpatrickeast: seems to me the whole design is just flawed, but what do I know21:51
*** sdake has quit IRC21:51
jgriffithpatrickeast: I see sbezverk 's point about if/when you're using an object store21:51
jgriffiththat makes sense21:51
patrickeastjgriffith: yep21:51
smcginnisLooks like the nova code has changed a bit: https://github.com/openstack/nova/blob/master/nova/image/glance.py#L19121:51
patrickeastjgriffith: we usually just tell folks to use swift to avoid this21:51
jgriffithI guess I think it's just crap to say "sure.. run as many G-servers as you like"21:51
jgriffithoh... psst.. btw, we're not worrying about silly things like synching them21:52
patrickeastno one really uses the file backend, right? right?!21:52
jgriffithyeah... just like nobody uses LVM21:52
jgriffith:)21:52
patrickeastlol21:52
smcginnisLooks like this was supposed to add a sync_images config option:21:54
smcginnishttps://blueprints.launchpad.net/glance/+spec/glance-sync-image-across-regions21:54
*** IlyaG has joined #openstack-cinder21:54
sbezverkjgriffith please do not joke with things like that, you almost gave me a heart attack !! Nobody uses lvm?!?!21:54
smcginnisBut no patch associated with that altho it says Complete.21:54
*** mylu has quit IRC21:54
sbezverkI spent almost 2 month make it working in docker21:54
jgriffithsbezverk: haah.. sorry, it's an old petpeave of mine21:54
smcginnisOh, Obsolete.21:54
jgriffithsbezverk: people trying to tell me LVM is no good and nobody uses it21:55
*** laughterwym has joined #openstack-cinder21:55
jgriffithsbezverk: it's a completely false statement and show ignorance21:55
sbezverkjgriffith please do not listen to them!21:55
smcginnisSo I still think we are trying to work around a Glance issue, and the right answer is to fix Glance.21:55
jgriffithsbezverk: nope21:55
smcginnisBut us being more robust isn't a bad thing.21:55
jgriffithsmcginnis: perhaps21:55
jgriffithsmcginnis: but I do like the work Kolla is doing, and running LVM/iSCSI in containers is pretty kick ass21:55
jgriffithsmcginnis: if this is the one thing that is screwing it up, and there aren't any real side-effects, I say do it21:56
jgriffithsmcginnis: or worst case... make a config option to ignore it21:56
jgriffithdefault to "NO_IGNORE"21:56
smcginnisjgriffith: I'm certainly behind that!21:57
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Oversubscription support  https://review.openstack.org/30749921:57
smcginnis+1 to kolla for sure.21:57
smcginnisJust I think the issue needs to be pushed (or at least raised) with Glance.21:58
*** laughterwym has quit IRC21:59
jgriffithsmcginnis: non arguments from me either21:59
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - QOS support  https://review.openstack.org/30750222:00
*** rdx_matt has joined #openstack-cinder22:02
*** ociuhandu has quit IRC22:02
*** cknight has joined #openstack-cinder22:02
*** rdx_matt has quit IRC22:06
sbezverkjgriffith if you have any good contact in Glance group, maybe we could have a chat about this situation?22:08
patrickeastprobably track someone down in Austin22:09
patrickeastcould*22:09
sbezverkpatrickeast good idea glance hunting in Austin sounds like a movie title ;-)22:16
patrickeasthaha22:16
*** tpsilva has quit IRC22:17
*** dave-mccowan has quit IRC22:19
*** ivase has quit IRC22:23
*** ntpttr_ has quit IRC22:25
*** markvoelker has quit IRC22:28
*** vgridnev_ has quit IRC22:30
*** sdague has quit IRC22:45
*** cknight has quit IRC22:50
*** neophy has joined #openstack-cinder22:53
*** sdague has joined #openstack-cinder22:58
*** ntpttr_ has joined #openstack-cinder22:58
*** rdx_matt has joined #openstack-cinder23:03
*** rdx_matt has quit IRC23:07
*** xyang1 has quit IRC23:07
*** cknight has joined #openstack-cinder23:08
*** cknight has quit IRC23:12
openstackgerritTin Lam proposed openstack/cinder: Added 'volume:get_transfer' to policy.json file  https://review.openstack.org/30685623:14
*** IlyaG has quit IRC23:17
*** raildo-afk is now known as raildo23:18
*** markvoelker has joined #openstack-cinder23:28
*** bjornar has quit IRC23:32
*** markvoelker has quit IRC23:33
openstackgerritMitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status  https://review.openstack.org/28873223:35
*** salv-orlando has quit IRC23:35
*** yhayashi has joined #openstack-cinder23:41
openstackgerritMitsuhiro Tanino proposed openstack/python-cinderclient: Support name option for volume restore  https://review.openstack.org/27591023:46
*** mriedem is now known as mriedem_away23:51
openstackgerritMerged openstack/cinder: Fix RPC and Object version pinning on API  https://review.openstack.org/30468623:55
*** erlon_ has quit IRC23:55

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