Friday, 2017-01-13

*** chlong has quit IRC00:01
*** gouthamr has quit IRC00:02
*** yumiriam has left #openstack-cinder00:05
*** rajinir has quit IRC00:06
*** pjm6 has quit IRC00:08
openstackgerritMerged openstack/cinder: Support new osprofiler API  https://review.openstack.org/31567600:09
*** lixiaoy1 has joined #openstack-cinder00:10
openstackgerritMerged openstack/cinder: Python3 common patterns  https://review.openstack.org/41335900:11
*** sdague has quit IRC00:12
*** cknight has quit IRC00:12
*** yangyape_ has quit IRC00:14
openstackgerritVivek Agrawal proposed openstack/python-cinderclient: Metadata based snapshop filtering  https://review.openstack.org/41753700:18
*** gouthamr has joined #openstack-cinder00:25
*** viks has joined #openstack-cinder00:27
viks:)00:27
*** jose-phillips has quit IRC00:40
*** knomura has joined #openstack-cinder00:42
*** knomura has quit IRC00:46
*** gouthamr has quit IRC00:47
*** mtani____ has quit IRC00:47
*** stvnoyes has quit IRC00:48
*** stvnoyes has joined #openstack-cinder00:49
*** wanghao has joined #openstack-cinder00:55
*** viks has quit IRC01:14
*** yangyapeng has joined #openstack-cinder01:16
*** wanghao has quit IRC01:19
*** salv-orlando has joined #openstack-cinder01:19
*** wanghao has joined #openstack-cinder01:19
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] ignore me, kicking off a rally/py35 build  https://review.openstack.org/41864301:22
*** salv-orlando has quit IRC01:24
*** Julien-zte has quit IRC01:30
*** diablo_rojo_phon has quit IRC01:30
*** Julien-zte has joined #openstack-cinder01:31
*** kaisers2 has joined #openstack-cinder01:36
*** kaisers3 has joined #openstack-cinder01:36
*** kaisers1 has quit IRC01:37
*** kaisers has quit IRC01:38
*** knomura has joined #openstack-cinder01:43
*** knomura has quit IRC01:48
openstackgerritxing-yang proposed openstack/cinder: Add driver interface for groups  https://review.openstack.org/41971001:51
openstackgerritxing-yang proposed openstack/cinder: Add driver interface for groups  https://review.openstack.org/41971001:56
*** markvoelker has quit IRC02:09
*** kaisers has joined #openstack-cinder02:17
*** dave-mccowan has joined #openstack-cinder02:17
*** ducttape_ has joined #openstack-cinder02:19
*** kaisers_ has quit IRC02:20
*** lixiaoy1 has quit IRC02:27
*** raunak has quit IRC02:28
*** gouthamr has joined #openstack-cinder02:33
openstackgerritMerged openstack/cinder: VMware: Set backend UUID to volume UUID  https://review.openstack.org/40911002:39
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] ignore me, kicking off a rally/py35 build  https://review.openstack.org/41864302:41
*** gouthamr has quit IRC02:43
*** gouthamr has joined #openstack-cinder02:45
openstackgerritJeremy Liu proposed openstack/python-cinderclient: Enable coverage report in console output  https://review.openstack.org/41972502:47
*** mriedem has joined #openstack-cinder02:49
*** breitz has quit IRC02:52
*** ducttape_ has quit IRC02:59
*** porrua has joined #openstack-cinder02:59
*** ducttape_ has joined #openstack-cinder03:00
*** yangyapeng has quit IRC03:01
*** ducttape_ has quit IRC03:04
openstackgerritMerged openstack/cinder: Add CG capability to generic groups in VNX driver  https://review.openstack.org/41392703:04
*** porrua has quit IRC03:04
*** x00350071_ has joined #openstack-cinder03:15
openstackgerritMerged openstack/cinder: RBD: Remove volume_tmp_dir option  https://review.openstack.org/40881803:17
*** edisonxiang has quit IRC03:18
*** ducttape_ has joined #openstack-cinder03:18
*** salv-orlando has joined #openstack-cinder03:21
*** salv-orlando has quit IRC03:25
*** gcb has quit IRC03:26
*** raunak has joined #openstack-cinder03:27
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] ignore me, kicking off a rally/py35 build  https://review.openstack.org/41864303:35
*** ducttape_ has quit IRC03:42
*** ducttape_ has joined #openstack-cinder03:42
*** knomura has joined #openstack-cinder03:44
*** ducttape_ has quit IRC03:44
*** ducttape_ has joined #openstack-cinder03:44
*** wanghao_ has joined #openstack-cinder03:48
*** knomura has quit IRC03:48
*** mriedem has quit IRC03:49
*** wanghao has quit IRC03:50
*** links has joined #openstack-cinder03:54
*** jungleboyj has joined #openstack-cinder03:54
*** yangyapeng has joined #openstack-cinder03:55
*** raunak has quit IRC03:55
*** raunak has joined #openstack-cinder03:56
*** mdnadeem has joined #openstack-cinder03:56
*** yangyapeng has quit IRC03:59
*** gouthamr has quit IRC04:04
jgriffithhaha!  You can't remove a driver without a proper deprecation period, but hey; you can drop an entire project a couple weeks before the release.  That makes sense04:04
*** diablo_rojo has joined #openstack-cinder04:05
*** ducttape_ has quit IRC04:08
*** ducttape_ has joined #openstack-cinder04:09
*** ducttape_ has quit IRC04:11
*** ducttape_ has joined #openstack-cinder04:12
*** pdeore has joined #openstack-cinder04:12
*** psachin has joined #openstack-cinder04:14
*** nicolasbock has quit IRC04:16
*** ducttape_ has quit IRC04:18
*** liverpooler has quit IRC04:21
*** pbandark has joined #openstack-cinder04:24
*** Julien-zte has quit IRC04:25
*** Zer0Byte has joined #openstack-cinder04:25
*** Zer0Byte has quit IRC04:26
*** Zer0Byte has joined #openstack-cinder04:26
*** breitz has joined #openstack-cinder04:28
*** pdeore has quit IRC04:32
*** Zer0Byte has quit IRC04:33
*** wanghao_ has quit IRC04:37
*** wanghao has joined #openstack-cinder04:37
*** Lee1092 has joined #openstack-cinder04:37
*** wanghao_ has joined #openstack-cinder04:41
*** wanghao has quit IRC04:43
*** wanghao_ has quit IRC04:45
*** wanghao has joined #openstack-cinder04:45
*** porrua has joined #openstack-cinder04:48
*** pdeore has joined #openstack-cinder04:50
*** porrua has quit IRC04:52
*** markvoelker has joined #openstack-cinder04:52
*** kaisers has quit IRC04:54
openstackgerritMerged openstack/cinder: Fixes hpelefthandclient AttributeError  https://review.openstack.org/39389404:58
*** Cibo_ has joined #openstack-cinder05:01
*** wanghao_ has joined #openstack-cinder05:05
*** alonmarx has joined #openstack-cinder05:05
*** wanghao has quit IRC05:08
*** tlian has quit IRC05:10
*** sudipto has joined #openstack-cinder05:18
*** sudipto_ has joined #openstack-cinder05:18
*** markvoelker_ has joined #openstack-cinder05:22
*** markvoelker has quit IRC05:26
*** dave-mccowan has quit IRC05:27
*** sticker has quit IRC05:31
*** adisky_ has joined #openstack-cinder05:38
*** knomura has joined #openstack-cinder05:45
*** Jack_I has joined #openstack-cinder05:48
*** Jack_Iv has joined #openstack-cinder05:48
*** knomura has quit IRC05:50
*** kaisers has joined #openstack-cinder05:55
*** lpetrut has joined #openstack-cinder05:59
*** salv-orlando has joined #openstack-cinder06:07
*** alonmarx has quit IRC06:08
openstackgerritchenyingnan proposed openstack/cinder: Replace assertTrue with assertIs.  https://review.openstack.org/41208506:09
openstackgerritchenyingnan proposed openstack/cinder: Replace assertTrue with assertEqual.  https://review.openstack.org/41208506:11
*** Poornima_K has joined #openstack-cinder06:13
*** raunak has quit IRC06:13
*** Jack_Iv has quit IRC06:14
*** swamireddy has quit IRC06:15
*** Jack_Iv has joined #openstack-cinder06:17
*** Jack_Iv has quit IRC06:17
*** Julien-zte has joined #openstack-cinder06:19
*** jwcroppe has quit IRC06:23
*** oomichi has quit IRC06:25
*** oomichi has joined #openstack-cinder06:26
*** oomichi has quit IRC06:30
*** oomichi has joined #openstack-cinder06:31
*** porrua has joined #openstack-cinder06:36
*** oomichi has quit IRC06:38
*** diablo_rojo has quit IRC06:40
*** kaisers has quit IRC06:40
*** oomichi has joined #openstack-cinder06:40
*** porrua has quit IRC06:41
*** mjura has joined #openstack-cinder06:42
*** salv-orlando has quit IRC06:43
*** knomura has joined #openstack-cinder06:47
*** lpetrut has quit IRC06:50
*** knomura has quit IRC06:51
*** oomichi has quit IRC06:55
*** oomichi has joined #openstack-cinder06:57
*** pgadiya has joined #openstack-cinder07:01
*** pdeore has quit IRC07:06
*** pdeore has joined #openstack-cinder07:06
*** diablo_rojo has joined #openstack-cinder07:06
*** tesseract has joined #openstack-cinder07:11
*** nkrinner_afk is now known as nkrinner07:15
*** arne_r has joined #openstack-cinder07:16
*** pdeore has quit IRC07:18
*** pdeore has joined #openstack-cinder07:18
*** lkuchlan_wfh has joined #openstack-cinder07:19
*** chenying has quit IRC07:19
*** chenying has joined #openstack-cinder07:20
*** salv-orlando has joined #openstack-cinder07:27
*** wxy has quit IRC07:28
*** sweston has quit IRC07:28
*** fmccrthy has quit IRC07:28
*** Lee1092 has quit IRC07:29
*** adisky_ has quit IRC07:29
*** supamatt has quit IRC07:29
*** wxy has joined #openstack-cinder07:30
*** dhellmann has quit IRC07:30
*** mwhahaha has quit IRC07:30
*** mwhahaha has joined #openstack-cinder07:31
*** sweston has joined #openstack-cinder07:31
*** fmccrthy has joined #openstack-cinder07:34
*** dhellmann has joined #openstack-cinder07:35
*** supamatt has joined #openstack-cinder07:36
*** adisky_ has joined #openstack-cinder07:36
*** kaisers has joined #openstack-cinder07:36
*** ChubYann has quit IRC07:39
*** AlexeyAbashkin has joined #openstack-cinder07:39
*** lkuchlan_wfh has quit IRC07:45
*** fmccrthy has quit IRC07:57
*** fmccrthy has joined #openstack-cinder07:57
*** nherciu has joined #openstack-cinder07:58
*** Cibo_ has quit IRC08:03
*** diablo_rojo has quit IRC08:07
*** wanghao_ has quit IRC08:20
*** wanghao has joined #openstack-cinder08:20
*** x00350071_ is now known as edisonxiang08:23
*** s-shiono has joined #openstack-cinder08:28
*** kaisers has quit IRC08:41
*** salv-orlando has quit IRC08:43
*** salv-orlando has joined #openstack-cinder08:44
*** knomura has joined #openstack-cinder08:48
*** alonmarx has joined #openstack-cinder08:51
*** knomura has quit IRC08:52
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** pgadiya has quit IRC09:01
*** wanghao_ has joined #openstack-cinder09:02
openstackgerritKazumasa Nomura proposed openstack/cinder: Specify the ports for compute nodes and copy functions in VSP drivers  https://review.openstack.org/41407609:02
*** wanghao has quit IRC09:05
*** ociuhandu has joined #openstack-cinder09:05
*** pgadiya has joined #openstack-cinder09:05
*** porrua has joined #openstack-cinder09:18
*** porrua has quit IRC09:22
*** mvk has quit IRC09:25
*** dansmith has quit IRC09:27
*** dansmith has joined #openstack-cinder09:27
*** dansmith is now known as Guest4575909:27
*** Lee1092 has joined #openstack-cinder09:29
*** Julien-zte has quit IRC09:34
*** s-shiono has quit IRC09:34
*** knomura has joined #openstack-cinder09:49
*** jordanP has joined #openstack-cinder09:50
*** e0ne has joined #openstack-cinder09:50
*** gaurangt has joined #openstack-cinder09:51
*** pdeore has quit IRC09:53
*** knomura has quit IRC09:54
*** mvk has joined #openstack-cinder09:56
*** sheel has joined #openstack-cinder09:56
*** wanghao_ has quit IRC10:01
*** pdeore has joined #openstack-cinder10:02
*** lpetrut has joined #openstack-cinder10:08
*** lkuchlan_wfh has joined #openstack-cinder10:11
*** e0ne has quit IRC10:12
*** JoseMello has joined #openstack-cinder10:18
*** openstackgerrit has quit IRC10:18
*** e0ne has joined #openstack-cinder10:21
*** salv-orl_ has joined #openstack-cinder10:23
*** salv-orlando has quit IRC10:25
*** openstackgerrit has joined #openstack-cinder10:27
openstackgerritMichal Dulko proposed openstack/cinder: Add prefix to user message event ids  https://review.openstack.org/31314010:27
*** kaisers has joined #openstack-cinder10:36
*** lkuchlan_wfh has quit IRC10:43
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] ignore me, kicking off a rally/py35 build  https://review.openstack.org/41864310:44
openstackgerritXuepeng Ji proposed openstack/python-brick-cinderclient-ext: Removes unnecessary utf-8 encoding  https://review.openstack.org/41988210:51
openstackgerritMerged openstack/python-brick-cinderclient-ext: Retrieve storage_protocol from storage backend capabilities  https://review.openstack.org/40935810:52
*** pjm6 has joined #openstack-cinder10:52
*** tommylikehu_ has joined #openstack-cinder10:58
*** yumiriam has joined #openstack-cinder11:01
*** tommylikehu_ has quit IRC11:03
*** sudipto_ has quit IRC11:05
*** sudipto has quit IRC11:05
*** porrua has joined #openstack-cinder11:06
*** porrua has quit IRC11:11
openstackgerritKazumasa Nomura proposed openstack/cinder: Specify the ports for compute nodes and copy functions in VSP drivers  https://review.openstack.org/41407611:12
*** Poornima_K has quit IRC11:24
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Change snapshot names  https://review.openstack.org/41454411:29
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [py35] image file should be open in binary mode  https://review.openstack.org/41989111:33
*** lseki has joined #openstack-cinder11:34
dimse0ne : jgriffith : easy one to get cinder+rally+py35 working ^^ please take a look11:35
e0nedims: great, thanks a lot!11:35
dimse0ne : here's the rally results http://logs.openstack.org/43/418643/5/check/gate-rally-dsvm-py35-cinder-nv/0a95698/rally-plot/results.html.gz11:36
openstackgerritMerged openstack/os-brick: Updated from global requirements  https://review.openstack.org/41886311:39
*** nicolasbock has joined #openstack-cinder11:40
*** kaisers has quit IRC11:43
*** openstackgerrit has quit IRC11:47
*** openstackgerrit has joined #openstack-cinder11:50
openstackgerritPeter Wang proposed openstack/cinder: VNX: Add async migration support  https://review.openstack.org/40527911:50
*** knomura has joined #openstack-cinder11:50
*** sdague has joined #openstack-cinder11:50
*** pdeore has quit IRC11:54
*** knomura has quit IRC11:55
*** salv-orl_ has quit IRC11:55
*** salv-orlando has joined #openstack-cinder11:59
*** adisky_ has quit IRC12:09
*** tommylikehu_ has joined #openstack-cinder12:21
*** timcl has joined #openstack-cinder12:22
*** thiagolib has joined #openstack-cinder12:23
*** catintheroof has joined #openstack-cinder12:30
*** catintheroof has quit IRC12:31
*** catintheroof has joined #openstack-cinder12:32
*** kaisers has joined #openstack-cinder12:39
*** gaurangt has left #openstack-cinder12:43
*** Lee1092 has quit IRC12:46
*** knomura has joined #openstack-cinder12:52
*** knomura has quit IRC12:56
*** liverpooler has joined #openstack-cinder13:00
*** psachin has quit IRC13:02
*** liverpooler has quit IRC13:03
*** liverpooler has joined #openstack-cinder13:04
*** Jack_Iv has joined #openstack-cinder13:16
*** tommylikehu_ has quit IRC13:17
*** tommylikehu_ has joined #openstack-cinder13:17
*** sudipto_ has joined #openstack-cinder13:19
*** sudipto has joined #openstack-cinder13:19
*** kaisers has quit IRC13:27
*** Dinesh_Bhor has quit IRC13:29
*** alonmarx has quit IRC13:31
*** ducttape_ has joined #openstack-cinder13:31
*** mdnadeem has quit IRC13:36
*** pgadiya has quit IRC13:36
*** AlexeyAbashkin has quit IRC13:37
*** AlexeyAbashkin has joined #openstack-cinder13:40
*** timcl1 has joined #openstack-cinder13:42
*** timcl has quit IRC13:45
*** ducttape_ has quit IRC13:55
*** pbandark has quit IRC13:58
openstackgerritGorka Eguileor proposed openstack/cinder: Make Image Volume Cache cluster aware  https://review.openstack.org/38618614:04
openstackgerritGorka Eguileor proposed openstack/cinder: Add replication info to generate_driver_list  https://review.openstack.org/40385414:04
openstackgerritGorka Eguileor proposed openstack/cinder: Prevent claiming and updating races on worker  https://review.openstack.org/41253414:04
openstackgerritGorka Eguileor proposed openstack/cinder: Add active/active info to generate_driver_list  https://review.openstack.org/40418014:04
openstackgerritGorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay  https://review.openstack.org/35306914:04
openstackgerritGorka Eguileor proposed openstack/cinder: Fix replication freeze mechanism  https://review.openstack.org/40292314:04
openstackgerritGorka Eguileor proposed openstack/cinder: Move service and cluster creation in test to utils  https://review.openstack.org/40292214:04
openstackgerritGorka Eguileor proposed openstack/cinder: Allow triggering cleanup from API  https://review.openstack.org/36301014:04
openstackgerritGorka Eguileor proposed openstack/cinder: Make Replication support Active-Active  https://review.openstack.org/38183514:04
openstackgerritGorka Eguileor proposed openstack/cinder: Make notify_service_capabilities cluster aware  https://review.openstack.org/41320014:04
*** xyang1 has joined #openstack-cinder14:06
duleke0ne: geguileo just fixed my remarks on patch you've gave +2 in the morning. I guess we can get it in now.14:07
duleke0ne: That's the one: https://review.openstack.org/36301014:07
duleke0ne: Just a simple fix in there between #41 and #42. :)14:08
*** Jack_Iv has quit IRC14:12
*** jungleboyj has quit IRC14:15
*** gouthamr has joined #openstack-cinder14:16
*** arne_r has quit IRC14:16
*** alonmarx has joined #openstack-cinder14:22
*** kaisers has joined #openstack-cinder14:22
*** e0ne has quit IRC14:23
*** e0ne has joined #openstack-cinder14:25
*** chenying_ has joined #openstack-cinder14:30
*** chenying has quit IRC14:30
*** dustins has joined #openstack-cinder14:31
*** porrua has joined #openstack-cinder14:32
*** tommylikehu has quit IRC14:33
*** tommylikehu has joined #openstack-cinder14:34
*** crose has joined #openstack-cinder14:37
*** salv-orlando has quit IRC14:38
*** tlian has joined #openstack-cinder14:39
e0nedulek, geguileo: I'll give my +2 once CI passed14:43
e0nedims: rally failed for https://review.openstack.org/#/c/419891/1 again:(14:43
*** chlong has joined #openstack-cinder14:45
*** eharney has quit IRC14:46
*** markvoelker has joined #openstack-cinder14:51
*** markvoelker_ has quit IRC14:51
*** liverpooler has quit IRC14:51
*** knomura has joined #openstack-cinder14:53
*** mjura has quit IRC14:54
*** cknight has joined #openstack-cinder14:57
*** sheel has quit IRC14:57
*** knomura has quit IRC14:57
*** mtanino has joined #openstack-cinder14:59
*** dave-mccowan has joined #openstack-cinder15:00
*** ducttape_ has joined #openstack-cinder15:04
*** liverpooler has joined #openstack-cinder15:05
*** eharney has joined #openstack-cinder15:08
*** mriedem has joined #openstack-cinder15:08
*** ducttape_ has quit IRC15:09
*** liverpooler has quit IRC15:11
*** liverpooler has joined #openstack-cinder15:11
*** tommylikehu_ has quit IRC15:17
*** jwcroppe has joined #openstack-cinder15:17
*** xyang1 has quit IRC15:19
*** markvoelker has quit IRC15:28
*** ebalduf has quit IRC15:30
*** ebalduf has joined #openstack-cinder15:31
*** ducttape_ has joined #openstack-cinder15:33
*** nkrinner is now known as nkrinner_afk15:37
dimse0ne : it needs this nova review (https://review.openstack.org/#/c/419476/) don't want to do a depends on since cinder one can merge independently15:39
*** kaisers has quit IRC15:39
dimse0ne : https://review.openstack.org/#/c/418643/ is the real experiment where you can see cinder passes now. i extracted just the bits that needed to be fixed into https://review.openstack.org/#/c/41989115:40
*** diablo_rojo_phon has joined #openstack-cinder15:40
scottdaildikov: ping15:41
ildikovscottda: hi15:41
dimsjgriffith : this one please :) https://review.openstack.org/#/c/41989115:41
scottdaI put some comments in https://bugs.launchpad.net/python-cinderclient/+bug/165610515:41
openstackLaunchpad bug 1656105 in python-cinderclient "api_versions.update_headers doesn't set the microversion header if version is 3.0" [Undecided,Confirmed] - Assigned to Scott DAngelo (scott-dangelo)15:41
e0nedims: got it, thanks!15:41
scottdaI don't know if there is anything to fix here.15:41
scottdaildikov: I'm not thinking this is a bug.15:42
*** dolphm has joined #openstack-cinder15:42
ildikovscottda: to be honest I'm not really an expert on how the clients are working15:42
e0nescottda: I agree. we don't have 3.0 microversion15:42
scottdaildikov: OK, I can help you with your testing. But there's no need for cinderclient to send a version header for 3.0.15:43
scottdae0ne: Thanks. I think I'm going to close that bug. No sense in changing the logic, as far as I'm concerned.15:43
openstackgerritMichael Dovgal proposed openstack/cinder: Remove deprecated RequestBodySizeLimiter class  https://review.openstack.org/41589315:45
ildikovscottda: what I don't fully understand at the moment is that how to get to the latest API microversion in Nova15:46
ildikovscottda: this code snippet is supposed to get the version: version = cinder_client.get_volume_api_from_url(url)15:47
scottdaildikov: No, that is legacy code, and used to get either "1" or "2"15:47
scottdaI *think* it would still work to get "3".15:47
ildikovscottda: and the URL comes from the session or the endpoint_template15:48
scottdaildikov: I had a POC patch that did this: https://review.openstack.org/#/c/385682/3/nova/volume/cinder.py15:48
ildikovyeah, it gets 3 as I set the catalog info and also the endpoint_template now as something is broken with the catalog_info part15:48
ildikovbut I need 3.2415:48
ildikovhow to get it there without hardcoding?15:48
ildikovwho negotiates that?15:48
ildikovI mean if the client puts there 3 I'm not really helped out with that15:49
scottdaildikov: ^^^ L#98-10115:49
scottdaIt may need fixing, but that's the general idea15:49
*** Jack_Iv has joined #openstack-cinder15:50
scottdaildikov: See requirement for cinderclient patch in commit message15:50
ildikovscottda: will the client read that CINDER_API_MICROVERSION up?15:51
*** Jack_Iv has quit IRC15:51
*** mvk has quit IRC15:51
*** jungleboyj has joined #openstack-cinder15:52
ildikovscottda: isn't the min_version in the cinderclient patch is what Matt is missing in that bug report?15:52
ildikovscottda: I will re-read the whole stuff if it was too dumb question :)15:53
*** markvoelker has joined #openstack-cinder15:54
*** markvoelker_ has joined #openstack-cinder15:55
*** markvoelker has quit IRC15:55
ildikovscottda: I really just would like to understand how this whole thing is supposed to work, I don't mean to criticize anything or make your life harder, I just got lost in this whole microversioning when it comes to getting two services working together :/15:56
scottdaIf you ran this against a server from master, the MAX_API_VERSION is 3.1615:56
scottdaSo, the minor version would be '16'15:56
scottdaHere's the flow:15:56
scottdanova instantiates a cinderclient to do an attach.15:57
scottdanova cinder.py calls cinderclient, which calls api_version.get_highest_version()15:57
scottdathis pings the server at endpoint: http://<url>:8776/15:58
scottdaThat returns version info15:58
scottdaThat is parsed, and the current version is obtained15:58
scottdaFor example, this would be '3.16'15:58
scottdaSo, nova would make note of this, and know that the cinder server is at that version15:58
scottdaOnce jgriffith 's patches are merged, Nova would be able to determine, from the version, if the server had the new cinder APIs15:59
scottdaand Nova could either "do it the old way" o r "do it the new way" based on version.15:59
scottdaI often feel it might be easier to translate the term "microversion" into "version"16:00
scottdaThis is all to version the cinder API, and use that info to determine what the API supports..16:00
scottdaAnd allow clients to request the cinder API at a certain version.16:01
ildikovone question, in the above flow, where does that happen that the OpenStack-API-Version gets into the header with the right microversion, which in my case would be 3.24?16:01
ildikovdoes that happen when get_highest_version() is called?16:02
ildikovI understand the rest, Nova tries to call the latest attach calls already as it's hardcoded16:02
ildikovbut currently the Header is not updated in the request, so I get back a resource not found error16:03
scottdaildikov: is this only for 3.0?16:04
scottdaor for 3.18 (or whatever is in the new API patches)?16:04
ildikovhow do you mean 3.0?16:04
ildikovthe endpoint is v316:04
ildikovI don't know where 3.0 comes from16:05
ildikovor 3.x to be exact16:05
scottdaYes, but if  you are using the old way, it won't work:16:05
scottdahttps://review.openstack.org/#/c/385682/3/nova/volume/cinder.py16:05
scottdaooops16:05
* jgriffith is late to the party and frantically back-scrolling....16:05
scottdaversion = cinder_client.get_volume_api_from_url(url)16:06
scottdaThat ^^^^ won't work16:06
ildikovyeah, that was my question earlier :)16:06
*** Jack_Iv has joined #openstack-cinder16:07
scottdaLet me play with that POC patch of mine, I think I see one problem...16:07
*** Jack_Iv has quit IRC16:07
ildikovjgriffith: I think you just got a review request, the long text is my education about microversioning request headers and client and Nova and whatever... :)16:07
jgriffithlol16:07
scottdaAfter we get the microversion, that is when we need to instantiate a client at that version.16:07
jgriffithscottda but it doesn't work that way16:07
ildikovscottda: yeah, that's what I didn't understand as it gets initiated with 316:08
jgriffithscottda I mean I guess we could make it work that way, but you can't get the version with the client without instantiating a client :)16:08
jgriffithscottda and the nova side doesn't work quite as you describe16:08
ildikovscottda: so if the client just thinks the minor version is 0 in that case then it's not good16:08
jgriffithscottda it instantiates a client on init, but that probably doesn't matter16:08
scottdajgriffith: Right. I think I can add a cinderclient.utils function to get the version without a client16:08
scottdajust using a request16:09
ildikovscottda: it should set the latest microversion I guess if nothing else is specified16:09
*** raunak has joined #openstack-cinder16:09
jgriffithscottda so in other words as is right now without extra patches to how ti all works it's not really possible to make this work without hard coding is what you're saying?16:09
scottdaildikov: no, that's not good. Too easy to upgrade the client, and have a higher version thatn the server.16:09
jgriffithildikov yeah, I think that's what the intent of this was16:09
jgriffithscottda ?16:09
ildikovscottda: I don't quite get that, why would it?16:10
jgriffithscottda I think she's correct based on sdague 's docs and other things I've read16:10
scottdajgriffith: Right. That POC will set the version, but the client is already instantiated at theat time.16:10
jgriffithscottda client negotiates with the server the highest supported version (or lcd among multiples) and uses it16:10
jgriffithscottda I feel like I'm in a loop here :)16:10
*** Jack_Iv has joined #openstack-cinder16:10
ildikovscottda: yep, as jgriffith says and I think that's in the bug report too16:10
ildikovscottda: I mean mriedem's bug report from yesterday16:11
scottdaThat is separate from the bug report16:11
scottdaWhat's missing is either:16:11
scottdanegotiate highest supported version16:11
scottdaor get version before init'ing client, and then use that to init a client at that version.16:11
jgriffithscottda well, you can't get that without a client is kind of the point here16:12
ildikovmy vote is on the first option if that matters16:12
scottdajgriffith: not at the moment, but you can hit the endpoint to get version info without AUTH16:12
ildikovthe second sounds like hard coding something somewhere to me16:12
jgriffithscottda I mean you can reinit it or whatever I guess but you keep saying "get the version before you init the client" you can't do that :)16:12
scottdaSo, I should be able to write a utils function that just does that, without a client16:12
scottdaAnd Horizon would like this anyway16:13
jgriffithscottda seems reasonable, but discovery using keystone in the past hasn't exactly gone "well"16:13
scottdaYou don't need a client to get version info...16:13
scottdajust 'curl' to the endpoint16:13
jgriffithscottda WTF? Is that a joke?16:13
*** e0ne has quit IRC16:13
scottdaI'm talking manually test with curl16:13
jgriffithscottda surely you jest?16:13
scottdaThe code would use requests16:14
scottdaLet me play around with it, and I'll show you16:14
jgriffithok, I"m going back to work on some sane code bases for a while :)16:14
jgriffithscottda I'm sure you'll figure out a good way to fix it up.  But if you need/want some help let me know16:14
scottdajgriffith: Will do.16:15
ildikovscottda: jgriffith: I let you figure this out, I hardcoded the version now in Nova for testing, so it's all good16:15
*** cdelatte has joined #openstack-cinder16:15
jgriffithildikov s/jgriffith// on that :)16:16
jgriffithunless someobdy asks for help I'm leaving it to scottda16:16
jgriffithmy opinions on this don't mesh well anyway :)16:16
ildikovjgriffith: I meant that you offered your help, sorry for confusion :)16:16
jgriffithhehe16:16
* jgriffith is just trying to be more like modern day politicians :)16:17
ildikovI have comments for you in the new API patches and I surely wouldn't want to distract you from that with the client stuff :)16:17
ildikovLOL, that cannot hurt :)16:17
*** mdovgal has quit IRC16:19
hemnacan't you make the get version info a static method on the client?16:22
*** raunak has quit IRC16:24
jgriffithno16:29
jgriffithoops... wrong window16:29
jgriffithsorry16:29
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Missing version history  https://review.openstack.org/41968616:34
*** kaisers has joined #openstack-cinder16:35
*** pcaruana has joined #openstack-cinder16:36
*** chlong has quit IRC16:38
*** chlong has joined #openstack-cinder16:43
*** lamt has joined #openstack-cinder16:43
scottdahemna: Yes16:46
scottdahemna: That's the plan.16:46
hemnaah ok :)16:47
scottdajust having weird results from urlparse ATM...16:47
*** jose-phillips has joined #openstack-cinder16:47
*** chlong has quit IRC16:48
*** AlexeyAbashkin has quit IRC16:49
*** Apoorva has joined #openstack-cinder16:50
*** liverpooler is now known as yrabl|lunch16:51
*** lamt has quit IRC16:52
*** Jack_Iv has quit IRC16:53
*** knomura has joined #openstack-cinder16:54
*** knomura has quit IRC16:58
*** chlong has joined #openstack-cinder16:59
*** jordanP has quit IRC17:01
*** sudipto has quit IRC17:01
*** sudipto_ has quit IRC17:01
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - QoS key fix  https://review.openstack.org/42007217:02
*** ociuhandu has quit IRC17:07
*** chlong has quit IRC17:09
*** raunak has joined #openstack-cinder17:11
*** alonmarx has quit IRC17:14
*** alonmarx has joined #openstack-cinder17:17
*** lamt has joined #openstack-cinder17:19
dimsjgriffith : around? ping about https://review.openstack.org/#/c/419891/ :)17:19
jgriffithdims ;)17:19
*** mriedem1 has joined #openstack-cinder17:19
jgriffithLGTM17:20
dimsthanks a ton!17:20
*** mriedem has quit IRC17:20
openstackgerritPavel Gluschak proposed openstack/cinder: vzstorage: fixed snapshot deletion in error state  https://review.openstack.org/41246517:20
dimsnow i have to go chase the Nova folks17:20
jgriffithdims NP, thanks for fixing17:20
*** swamireddy has joined #openstack-cinder17:25
*** nherciu_ has joined #openstack-cinder17:28
*** eharney has quit IRC17:29
*** nherciu has quit IRC17:29
*** kaisers has quit IRC17:30
*** nherciu__ has joined #openstack-cinder17:31
*** lpetrut has quit IRC17:33
*** nherciu_ has quit IRC17:34
SwansonPure CI broken?17:37
*** mvk has joined #openstack-cinder17:38
patrickeastYea, been struggling with some hardware issues the last few weeks17:41
*** _ducttape_ has joined #openstack-cinder17:41
openstackgerritAleksey Ruban proposed openstack/cinder: NexentaStor5: Added extend method to NFS driver  https://review.openstack.org/41799517:43
*** ducttape_ has quit IRC17:44
*** xinli has joined #openstack-cinder17:44
*** salv-orlando has joined #openstack-cinder17:45
*** eharney has joined #openstack-cinder17:51
openstackgerritMerged openstack/cinder: Improve DB method naming consistency  https://review.openstack.org/32144017:53
*** knomura has joined #openstack-cinder17:54
Swansonjgriffith, Whatever happened to using '\' to continue lines? Did we all agree that was peachy or is it still -1 bait?17:56
*** jwcroppe has quit IRC17:56
jgriffithSwanson I don't know that people agreed, I prefer the \ in most cases, and in some it's required, but I don't think there was a real answer there17:57
*** ChubYann has joined #openstack-cinder17:58
*** ociuhandu has joined #openstack-cinder17:58
Swansonjgriffith, alright. Try it and see who complains, I guess.17:59
*** jwcroppe has joined #openstack-cinder17:59
*** Guest45759 is now known as dansmith18:04
*** dustins has quit IRC18:04
hemnaSwanson, I've -1'd it.   But that's because I've been conditioned to do so.18:06
*** dustins has joined #openstack-cinder18:09
*** dustins has quit IRC18:09
*** dustins has joined #openstack-cinder18:10
scottdaI think Sean had a ML post about it...last week or week before..18:10
Swansonhemna, You're a stooge of the Parenthesis industry.18:11
hemnacompletely18:11
*** links has quit IRC18:12
openstackgerritScott DAngelo proposed openstack/python-cinderclient: WIP static method to get_server_version  https://review.openstack.org/42011918:16
*** dustins has quit IRC18:21
*** dustins has joined #openstack-cinder18:21
*** knomura has quit IRC18:22
*** _ducttape_ has quit IRC18:25
*** pcaruana has quit IRC18:26
*** ducttape_ has joined #openstack-cinder18:26
*** kaisers has joined #openstack-cinder18:27
*** Jack_V has joined #openstack-cinder18:28
*** dustins has quit IRC18:29
*** dustins has joined #openstack-cinder18:29
*** Jack_I has quit IRC18:32
*** mriedem1 is now known as mriedem18:33
openstackgerritMerged openstack/cinder: Unreserve volume: fix log message for completion  https://review.openstack.org/38126218:37
*** Jack_V has quit IRC18:37
*** pcaruana has joined #openstack-cinder18:39
*** Jack_I has joined #openstack-cinder18:40
*** Jack_Iv has joined #openstack-cinder18:40
*** lpetrut has joined #openstack-cinder18:43
*** Jack_Iv has quit IRC18:45
*** alonmarx has quit IRC18:46
openstackgerritMerged openstack/cinder: [py35] image file should be open in binary mode  https://review.openstack.org/41989118:47
*** jwcroppe has quit IRC18:48
*** Jack_I has quit IRC18:52
*** alonmarx has joined #openstack-cinder18:52
*** jwcroppe has joined #openstack-cinder18:53
*** tesseract has quit IRC18:53
openstackgerritAlon Marx proposed openstack/cinder: Open the source code of ibm_storage driver  https://review.openstack.org/40495818:55
*** cknight has quit IRC18:57
*** openstack has joined #openstack-cinder19:01
*** kaisers has quit IRC19:01
*** lpetrut has quit IRC19:01
*** ccesario has quit IRC19:02
*** pcaruana has quit IRC19:02
*** gouthamr has joined #openstack-cinder19:03
*** lpetrut has joined #openstack-cinder19:09
*** jamielennox is now known as jamielennox|away19:12
*** chlong has joined #openstack-cinder19:14
*** markvoelker_ has quit IRC19:19
*** markvoelker has joined #openstack-cinder19:20
*** markvoelker has quit IRC19:25
*** yumiriam has quit IRC19:27
patrickeast:*( why did we default that manage snapshot test to be enabled in devstack for any backend? https://github.com/openstack-dev/devstack/commit/afac732d5d9b03da7abbd61f814c2f0d1a9039da19:40
*** dustins has quit IRC19:46
SwansonThe one that only works with LVM and Ceph?19:46
*** Guest87783 is now known as med_19:47
*** med_ has joined #openstack-cinder19:47
patrickeastonly lvm :(19:47
*** med_ is now known as medberry19:47
patrickeastmaybe ceph too, but if it does its just luck that they were implemented similarly19:48
SwansonI thought there was 20 minutes on this in the weekly meeting. Didn't someone put a patch in to remove this and admonish the people who added it without consultation of cinder folk?19:49
patrickeastthere was a patch to remove it19:50
patrickeastbut for whatever reason they didn't merge it, they just put it behind a flag19:50
patrickeastand then defaulted the flag to enabled..19:50
* patrickeast is putting together a devstack patch to fix it...19:50
patrickeastguess i can't complain too much since i've been MIA quite a bit lately19:51
*** cknight has quit IRC19:51
SwansonNice. The process works.19:51
*** gouthamr has quit IRC19:51
*** cknight has joined #openstack-cinder19:54
*** gouthamr has joined #openstack-cinder19:55
*** kaisers has joined #openstack-cinder19:57
*** openstackstatus has quit IRC19:57
*** openstack has joined #openstack-cinder20:00
*** nherciu__ has quit IRC20:01
*** psachin has joined #openstack-cinder20:10
*** rajinir has joined #openstack-cinder20:10
*** eharney has quit IRC20:11
*** alonmarx has quit IRC20:13
hemnaugh20:15
hemnathe ML thread on the PTG seems to be going south20:15
patrickeastwhich one?20:16
jungleboyjhemna: Which one?20:16
* patrickeast loves some ML drama20:16
hemnathe latest20:16
hemnatitled PTG? / Was ...20:16
hemnaman what a mess20:16
patrickeastohh yea20:16
jungleboyjOy ... I had the same thought though.  I guess we will see where this goes.20:18
jungleboyjYeah, I have done all that I can really do.  Now it is the work of physically getting out.20:21
jungleboyjOops, wrong Window.  :-)20:21
diablo_rojo_phonLol20:22
*** eharney has joined #openstack-cinder20:26
*** openstackstatus has quit IRC20:27
*** openstack has joined #openstack-cinder20:30
jungleboyjSwanson: Yes, less interesting than mine.20:30
*** salv-orlando has quit IRC20:32
*** knomura has joined #openstack-cinder20:34
hemnajungleboyj, could have been worse....20:34
jungleboyjhemna: Indeed.  :-)  He He.20:35
*** porrua has quit IRC20:36
*** knomura has quit IRC20:39
patrickeastok... devstack change for that manage_snapshot thing https://review.openstack.org/#/c/42016920:39
patrickeastwould be good for some other cinder folks to look at it and make sure i'm not crazy20:40
*** cdelatte has quit IRC20:43
*** timcl1 has quit IRC20:44
*** edisonxiang has quit IRC20:45
*** edisonxiang has joined #openstack-cinder20:46
*** crose has quit IRC20:50
*** Jack_I has joined #openstack-cinder20:55
*** bardia has joined #openstack-cinder20:59
*** dustins has joined #openstack-cinder21:01
*** xinli has quit IRC21:02
*** Jack_I has quit IRC21:03
*** lpetrut has quit IRC21:05
*** thiagolib has quit IRC21:08
*** kaisers has quit IRC21:10
*** rha has quit IRC21:14
*** Apoorva has joined #openstack-cinder21:16
*** jungleboyj has quit IRC21:25
*** JoseMello has quit IRC21:25
*** xyang1 has joined #openstack-cinder21:28
*** alonmarx has joined #openstack-cinder21:30
*** gouthamr has quit IRC21:31
*** diablo_rojo has joined #openstack-cinder21:32
*** adrianofr has quit IRC21:49
*** viks has joined #openstack-cinder21:49
*** salv-orlando has joined #openstack-cinder21:55
*** ebalduf has quit IRC21:58
*** diablo_rojo has quit IRC21:58
*** jamielennox|away is now known as jamielennox21:59
*** ebalduf has joined #openstack-cinder22:00
*** eharney has quit IRC22:06
*** kaisers has joined #openstack-cinder22:06
*** jdurgin1 has joined #openstack-cinder22:07
viksHi all, one question...can we filter volumes based on metadata which has non ascii characters in value22:08
viksI tried the same on devstack and got error on command line22:08
viksam I missing something here22:09
*** lseki has quit IRC22:10
viksmy setup was older version of OS...22:14
vikson newer version...I do the following22:20
vikscinder create --name фф 122:21
viksthe volume gets created22:21
SwansonAnyone? https://review.openstack.org/39797022:21
*** xyang1 has quit IRC22:21
SwansonShort one... https://review.openstack.org/41797222:22
SwansonOne that is only comments and the driver version ... https://review.openstack.org/41968622:22
viksbut when I do 'cinder list --name фф' I am getting error22:22
viksAm I missing somethig very basic?22:22
*** salv-orl_ has joined #openstack-cinder22:22
SwansonI do not know.22:23
hemnaSwanson, CI ?22:24
hemnanot seeing a CI report on that22:25
hemnamaybe I'm blind22:25
*** salv-orlando has quit IRC22:25
*** baling-wire is now known as david-lyle22:26
*** ducttape_ has quit IRC22:27
*** jungleboyj has joined #openstack-cinder22:34
*** catintheroof has quit IRC22:38
*** alonmarx has quit IRC22:49
*** dustins has quit IRC22:56
*** ducttape_ has joined #openstack-cinder22:59
viks:(23:04
openstackgerritRaunak Kumar proposed openstack/cinder: Fix for live migration for Nimble Storage  https://review.openstack.org/42021423:05
*** tommylikehu_ has joined #openstack-cinder23:06
*** tommylikehu_ has quit IRC23:07
ApoorvaHello. While running CI tests I see following error http://paste.openstack.org/show/594905/ in devstack installation. Should n-net be removed from ENABLED_SERVICES?23:07
*** mriedem has quit IRC23:09
Swansonhemna, that's a good question. Probably hasn't gotten to it yet. Ask again on Monday I will.23:09
*** tommylikehu_ has joined #openstack-cinder23:10
*** kaisers has quit IRC23:14
*** tommylikehu_ has quit IRC23:15
*** yrabl|lunch has quit IRC23:15
*** salv-orl_ has quit IRC23:26
*** jungleboyj has quit IRC23:26
Swansonhemna, this one https://review.openstack.org/#/c/417972/ Has Dell CI approval.23:29
Swanson397970 is still waiting...23:30
karthikp_scottda: Hi23:30
*** knomura has joined #openstack-cinder23:34
*** rajinir has quit IRC23:36
*** cknight has quit IRC23:38
*** e0ne has joined #openstack-cinder23:38
*** chlong has quit IRC23:38
*** knomura has quit IRC23:39
*** tommylikehu has quit IRC23:46
*** viks has quit IRC23:46
*** tommylikehu has joined #openstack-cinder23:46
*** jwcroppe has quit IRC23:47
*** mriedem has joined #openstack-cinder23:52
patrickeastApoorva: yes, you should use neutron networking now23:53
*** gouthamr has joined #openstack-cinder23:54
*** Jack_Iv has joined #openstack-cinder23:55
Apoorvapatrickeast: I thought it would just work by removing n-net from ENABLED_SERVICES but it fails for me. http://openstack-ci.tintri.com/tintri/refs-changes-93-415893-2/testr_results.html23:55
*** harlowja has quit IRC23:56
Apoorvapatrickeast: Looks like neutron is not enabled. Now I am trying to enable it explicitly23:56
patrickeastApoorva: its switched over by default i think, but you can also see what devstack gate does if you specify to turn it on https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate-wrap.sh#L20523:57
patrickeastApoorva: so if you run with a minimal config and look at what the enabled services list looks like it should give some clues23:58
jgriffithpatrickeast Apoorva https://goo.gl/ZSBGrv23:58
jgriffithoh.. hehe23:58
jgriffithpatrickeast got it already23:58
jgriffithnm23:58
jgriffithoh.. that flag no longer does anything I don't think23:59
*** Jack_Iv has quit IRC23:59
jgriffithso devstack master no longer does n-net IIRC23:59
jgriffiththat minimal config I posted should "just" work23:59
patrickeastyea for just devstack you want to just let it use defaults23:59
jgriffithemphasis on *should* :)23:59

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