Monday, 2018-05-07

*** jiaohaolin has left #openstack-cinder00:31
*** r-daneel has joined #openstack-cinder00:38
*** r-daneel has quit IRC00:42
*** r-daneel has joined #openstack-cinder00:44
*** dave-mccowan has joined #openstack-cinder00:48
*** dpawlik has joined #openstack-cinder01:03
*** zhaochao has joined #openstack-cinder01:06
*** dpawlik has quit IRC01:07
*** kaisers_ has joined #openstack-cinder01:14
*** liuzz has joined #openstack-cinder01:17
*** kaisers has quit IRC01:18
*** edmondsw has joined #openstack-cinder01:32
*** edmondsw has quit IRC01:36
*** lixiaoy1 has joined #openstack-cinder01:42
*** dalgaaf has quit IRC01:46
*** dalgaaf has joined #openstack-cinder01:49
*** links has joined #openstack-cinder02:17
*** namnh has joined #openstack-cinder02:18
*** ircuser-1 has joined #openstack-cinder02:23
*** psachin has joined #openstack-cinder02:42
*** EvilienM is now known as EmilienM02:45
*** r-daneel has quit IRC02:50
*** dklyle has joined #openstack-cinder02:51
*** Dinesh_Bhor has joined #openstack-cinder02:54
*** dklyle has quit IRC02:57
*** dpawlik has joined #openstack-cinder03:03
*** dpawlik has quit IRC03:11
*** edmondsw has joined #openstack-cinder03:20
*** moshele has quit IRC03:21
*** dave-mccowan has quit IRC03:23
*** edmondsw has quit IRC03:24
*** pooja_jadhav has joined #openstack-cinder03:45
openstackgerritNeha Alhat proposed openstack/cinder master: V3 jsonschema validation: volume_type_encryption
*** threestrands has joined #openstack-cinder04:04
*** namnh has quit IRC04:08
*** namnh has joined #openstack-cinder04:09
*** gyankum has joined #openstack-cinder04:10
*** mmethot has joined #openstack-cinder04:10
*** pgadiya has joined #openstack-cinder04:25
*** pgadiya has quit IRC04:28
*** abhishekk has joined #openstack-cinder04:35
*** moshele has joined #openstack-cinder04:45
openstackgerritShunei Shiono proposed openstack/cinder master: NEC driver: Delete max volumes per pool limit.
*** Dinesh_Bhor has quit IRC04:59
*** dpawlik has joined #openstack-cinder05:02
*** Nel1x has quit IRC05:06
*** dpawlik has quit IRC05:06
*** hoonetorg has quit IRC05:07
*** hoonetorg has joined #openstack-cinder05:20
*** moshele has quit IRC05:21
*** lkuchlan has joined #openstack-cinder05:22
*** evin has quit IRC05:46
*** pcaruana has joined #openstack-cinder06:02
*** allegra__ has quit IRC06:05
*** allegra__ has joined #openstack-cinder06:10
*** evin has joined #openstack-cinder06:13
*** lkuchlan has quit IRC06:20
*** andreas_s has joined #openstack-cinder06:26
*** andreas_s has quit IRC06:26
*** hamdyk has joined #openstack-cinder06:27
*** andreas_s has joined #openstack-cinder06:28
*** armaan has quit IRC06:34
*** armaan has joined #openstack-cinder06:35
*** armaan has quit IRC06:42
*** armaan has joined #openstack-cinder06:43
*** Dinesh_Bhor has joined #openstack-cinder06:49
*** edmondsw has joined #openstack-cinder06:56
*** armaan has quit IRC06:58
*** armaan has joined #openstack-cinder06:58
*** edmondsw has quit IRC07:00
*** dpawlik has joined #openstack-cinder07:03
*** dpawlik has quit IRC07:08
*** alexchadin has joined #openstack-cinder07:12
*** alexchad_ has joined #openstack-cinder07:14
*** alexchadin has quit IRC07:17
*** alexchad_ is now known as alexchadin07:18
openstackgerritfupingxie proposed openstack/cinder master: Remove stale pip-missing-reqs tox test
*** lkuchlan has joined #openstack-cinder07:25
*** threestrands has quit IRC07:39
openstackgerritNeha Alhat proposed openstack/cinder master: V3 jsonschema validation: scheduler_hints
*** moshele has joined #openstack-cinder07:55
*** liuzz_ has joined #openstack-cinder08:25
*** liuzz has quit IRC08:27
*** edmondsw has joined #openstack-cinder08:44
*** alexchadin has quit IRC08:47
*** edmondsw has quit IRC08:49
openstackgerritNeha Alhat proposed openstack/cinder master: V3 jsonschema validation: volume_type_encryption
*** lixiaoy1 has quit IRC08:55
*** dpawlik has joined #openstack-cinder09:04
*** kaisers_ has quit IRC09:06
*** e0ne has joined #openstack-cinder09:07
*** dpawlik has quit IRC09:08
*** dpawlik has joined #openstack-cinder09:11
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: os-services
*** alexchadin has joined #openstack-cinder09:11
*** dpawlik has quit IRC09:16
*** Dinesh_Bhor has quit IRC09:32
*** kaisers has joined #openstack-cinder09:36
*** armaan has quit IRC09:38
*** erlon has quit IRC09:39
*** armaan has joined #openstack-cinder09:40
*** alexchadin has quit IRC09:55
*** afazekas is now known as afazekas|pto09:59
openstackgerritTommyLike proposed openstack/cinder master: Support filter backend based on operation type
*** namnh has quit IRC10:15
*** lkuchlan has quit IRC10:28
*** edmondsw has joined #openstack-cinder10:32
*** moshele has quit IRC10:33
*** abhishekk has quit IRC10:34
*** moshele has joined #openstack-cinder10:34
*** edmondsw has quit IRC10:36
*** armaan has quit IRC10:45
*** armaan has joined #openstack-cinder10:45
*** stakeda has quit IRC10:46
*** luizbag has joined #openstack-cinder10:53
*** s-shiono has quit IRC10:53
*** pgadiya has joined #openstack-cinder11:00
*** pgadiya has quit IRC11:00
*** andreas_s has quit IRC11:11
*** andreas_s has joined #openstack-cinder11:11
*** dpawlik has joined #openstack-cinder11:12
strigazihello, since last week I'm getting the following error when creating a devstack env: error in setup command: Error parsing /opt/stack/cinder/setup.cfg: SyntaxError: '<' operator not allowed in environment markers11:12
*** andreas_s_ has joined #openstack-cinder11:12
strigaziHas anyone seen this?11:13
*** dave-mccowan has joined #openstack-cinder11:14
*** andreas_s_ has quit IRC11:14
*** andreas_s_ has joined #openstack-cinder11:14
*** andreas_s has quit IRC11:16
*** dpawlik has quit IRC11:16
*** andreas_s_ has quit IRC11:18
*** andreas_s has joined #openstack-cinder11:19
*** andreas_s_ has joined #openstack-cinder11:22
*** andreas_s has quit IRC11:23
*** dave-mccowan has quit IRC11:26
*** dave-mccowan has joined #openstack-cinder11:27
*** erlon has joined #openstack-cinder11:27
*** moshele has quit IRC11:38
*** moshele has joined #openstack-cinder11:40
tommylikehuhey geguileo, done! can you review it again?
*** abishop has joined #openstack-cinder11:40
*** r-daneel has joined #openstack-cinder11:53
*** moshele has quit IRC11:54
*** moshele has joined #openstack-cinder11:55
*** alexchadin has joined #openstack-cinder12:09
*** nicolasbock has joined #openstack-cinder12:09
geguileotommylikehu: thanks, you have my +2  :-)12:11
*** markvoelker has joined #openstack-cinder12:17
tommylikehugeguileo:  thanks :)12:25
*** liverpooler has joined #openstack-cinder12:25
*** tpsilva has joined #openstack-cinder12:30
*** lkuchlan has joined #openstack-cinder12:31
openstackgerritTommyLike proposed openstack/cinder master: Add missing image metadata when creating volume from backup
*** edmondsw has joined #openstack-cinder12:32
*** edmondsw has quit IRC12:36
*** edmondsw has joined #openstack-cinder12:38
*** edmondsw has quit IRC12:42
*** edmondsw has joined #openstack-cinder12:44
*** liuzz has joined #openstack-cinder12:45
*** liuzz_ has quit IRC12:45
*** felipemonteiro has joined #openstack-cinder12:48
*** edmondsw has quit IRC12:48
*** edmondsw has joined #openstack-cinder12:49
*** mmedvede has quit IRC12:58
*** lkuchlan has quit IRC12:59
*** bhagyashri_s has joined #openstack-cinder13:01
*** mmethot_ has joined #openstack-cinder13:01
*** jmlowe_ has joined #openstack-cinder13:02
*** vivsoni__ has joined #openstack-cinder13:02
*** psachin` has joined #openstack-cinder13:03
*** links has quit IRC13:05
*** lbragstad has joined #openstack-cinder13:06
*** felipemonteiro has quit IRC13:07
*** zigo_ has joined #openstack-cinder13:07
*** eric-young has joined #openstack-cinder13:09
*** sdake_ has joined #openstack-cinder13:09
*** sdake_ has quit IRC13:09
*** sdake_ has joined #openstack-cinder13:09
*** storshoo___ has joined #openstack-cinder13:09
*** rmk` has joined #openstack-cinder13:09
*** armaan has quit IRC13:09
*** cburgess_ has joined #openstack-cinder13:09
*** johnthetubaguy_ has joined #openstack-cinder13:10
*** Reepicheep_ has joined #openstack-cinder13:10
*** armaan has joined #openstack-cinder13:10
*** mmethot has quit IRC13:11
*** psachin has quit IRC13:11
*** a_storshoo has quit IRC13:11
*** jmlowe has quit IRC13:11
*** vivsoni_ has quit IRC13:11
*** sdake has quit IRC13:11
*** bhagyashris has quit IRC13:11
*** openstackgerrit has quit IRC13:11
*** rmk has quit IRC13:11
*** cburgess has quit IRC13:11
*** Reepicheep has quit IRC13:11
*** zigo has quit IRC13:11
*** johnthetubaguy has quit IRC13:11
*** ericyoung has quit IRC13:11
*** bswartz has quit IRC13:11
*** rmk` is now known as rmk13:11
*** rmk has quit IRC13:11
*** rmk has joined #openstack-cinder13:11
*** dalgaaf has quit IRC13:14
*** bswartz has joined #openstack-cinder13:17
*** dalgaaf has joined #openstack-cinder13:18
*** psachin` has quit IRC13:24
*** mmedvede has joined #openstack-cinder13:26
*** AlexeyAbashkin has joined #openstack-cinder13:28
*** moshele has quit IRC13:30
*** armaan has quit IRC13:34
*** armaan has joined #openstack-cinder13:34
*** pchavva has joined #openstack-cinder13:36
*** superdan is now known as dansmith13:40
*** dustins has joined #openstack-cinder13:42
*** erlon has quit IRC13:44
*** eharney has joined #openstack-cinder13:46
*** gnufied has joined #openstack-cinder13:49
*** lpetrut has joined #openstack-cinder13:51
*** chhagarw has joined #openstack-cinder13:53
*** r-daneel has quit IRC13:56
mnasersmcginnis: whenever you're around.. is rbd multiattach still a thing that might make it to queens?13:57
*** erlon has joined #openstack-cinder13:57
mnasergot my answer13:57
*** chhagarw has quit IRC14:00
*** chhagarw has joined #openstack-cinder14:00
*** gyankum has quit IRC14:05
*** chhagarw has quit IRC14:06
*** evin has quit IRC14:10
smcginnismnaser: Yeah, there were a few random patches trying to enable that, so Eric added that comment.14:10
mnasersmcginnis: gotcha, looks like it'll be a little while before that gets settled down14:14
*** armaan has quit IRC14:16
*** throwsb1 has joined #openstack-cinder14:17
*** armaan has joined #openstack-cinder14:17
*** arnewiebalck_ has joined #openstack-cinder14:17
smcginnismnaser: Yeah, I wouldn't expect it any time soon.14:18
mnasersmcginnis: no worries, our cinder is on queens now so i was hoping maybe we could break it ;)14:19
*** armaan has quit IRC14:20
*** armaan has joined #openstack-cinder14:21
*** ganso has joined #openstack-cinder14:24
tpsilvajungleboyj: ping14:26
*** hemna_ has joined #openstack-cinder14:27
*** r-daneel has joined #openstack-cinder14:28
*** moshele has joined #openstack-cinder14:29
*** felipemonteiro has joined #openstack-cinder14:32
*** hamdyk has quit IRC14:34
*** AlexeyAbashkin has quit IRC14:36
*** felipemonteiro_ has joined #openstack-cinder14:42
*** felipemonteiro has quit IRC14:45
jungleboyjtpsilva:  Pong.14:47
*** arnewiebalck_ has quit IRC14:47
*** lpetrut has quit IRC14:50
*** lkuchlan has joined #openstack-cinder14:51
tpsilvajungleboyj: hello! have a question14:59
tpsilvajungleboyj: after this patch merged:
jungleboyjtpsilva: Looking.15:00
tpsilvawe found that our NFS host does not match that particular regex15:00
tpsilvajungleboyj: oh wait, not this patch, just a sec15:00
tpsilvathis one15:02
tpsilvait's the same topic, anyway15:02
tpsilvait validates the parameters for the API15:02
tpsilvajungleboyj: and now it validates if the hostname matches a regex ( when managing a volume15:03
tpsilvathe problem is, our NFS driver uses the path to the nfs export as its hostname, so it doest not match that regex15:04
tpsilvajungleboyj: my question: would it be ok to change this hostname regex to accept our implementation?15:05
*** dpawlik has joined #openstack-cinder15:05
*** markstur has joined #openstack-cinder15:05
jungleboyjtpsilva: smcginnis and I were just talking and if it worked in the past and now doesn't.15:06
jungleboyjSo, it would seem it is a bug that should be resolved.15:06
tpsilvajungleboyj: a bug in our driver?15:07
jungleboyjNo, a bug introduced by that change.15:07
*** alexchad_ has joined #openstack-cinder15:07
smcginnisA lot of those schema validation changes do kind of need some runtime and checking for odd cases like these.15:07
eharneydid the driver CI fail on that change?15:08
tpsilvayep, and for some weird reason our manage/unmanage tests are disabled on our CI15:08
tpsilvawe are looking into that15:08
tpsilvaeharney: but it started failing on our internal functional tests routine15:08
tpsilvaunfortunately after ir merged15:08
*** pcaruana has quit IRC15:09
tpsilvaok, it is a simple fix and we actually already have a fix ready to go upstream15:09
*** alexchadin has quit IRC15:09
*** dpawlik has quit IRC15:10
*** felipemonteiro_ has quit IRC15:10
*** felipemonteiro_ has joined #openstack-cinder15:10
*** openstackgerrit has joined #openstack-cinder15:10
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix google backup driver
tpsilvabut it might be interesting to get attention from other vendors to check their drivers as well, if they already didn't15:10
*** mriedem has joined #openstack-cinder15:10
smcginnistpsilva: Did third party CI not fail for this?15:11
tpsilvasmcginnis: nope... our ci is not running manage/unmanage tests, which is the feature that it's failing15:11
smcginnisAh, that's too bad.15:12
tpsilvayep, we're looking into why it is disabled15:12
*** rajinir has joined #openstack-cinder15:12
jungleboyjtpsilva: Ok.  Thanks for letting us know.  I think it is ok to get a patch up to resolve the issue for your driver.15:14
tpsilvajungleboyj: awesome, we'll just set things here downstream and we should have a patch up soon15:15
tpsilvathanks everyone!15:15
smcginnisThe manage/unmanage APIs are hard to tempest test since that particular functionality needs somewhat driver-specific input.15:15
eharneydo we not have a tempest test that leverages list-manageable for that?15:15
tpsilvawe test that thoroughly with our functional tests, but it does not run upstream15:16
jungleboyjeharney:  Not sure.  That would be one that would be hard to set up a test for.15:16
eharneyjungleboyj: how so?15:16
*** dpawlik has joined #openstack-cinder15:16
jungleboyjOh, actually I suppose you could unmanage a volume and make sure it then shows up as one that could be managed in.15:17
*** dpawlik has quit IRC15:20
*** felipemonteiro__ has joined #openstack-cinder15:22
*** felipemonteiro_ has quit IRC15:26
*** sapd_ has quit IRC15:32
*** markstur_ has joined #openstack-cinder15:37
*** andreas_s_ has quit IRC15:40
*** andreas_s has joined #openstack-cinder15:40
*** markstur has quit IRC15:41
*** alexchad_ has quit IRC15:45
*** andreas_s has quit IRC15:45
*** lkuchlan has quit IRC15:45
*** kaisers has quit IRC15:46
*** armaan has quit IRC15:49
*** dklyle has joined #openstack-cinder15:52
*** moshele has quit IRC15:54
*** dklyle has quit IRC15:57
*** dklyle has joined #openstack-cinder15:57
*** zhaochao has quit IRC16:13
*** e0ne has quit IRC16:13
*** openstackgerrit has quit IRC16:19
*** alexchadin has joined #openstack-cinder16:20
*** lpetrut has joined #openstack-cinder16:20
*** finucannot is now known as stephenfin16:20
*** diablo_rojo has joined #openstack-cinder16:20
*** kaisers has joined #openstack-cinder16:23
*** alexchadin has quit IRC16:24
*** lkuchlan has joined #openstack-cinder16:28
*** openstackgerrit has joined #openstack-cinder16:31
openstackgerritEric Harney proposed openstack/cinder master: Validate releasenotes with yamllint
openstackgerritMerged openstack/cinder master: Support filter backend based on operation type
smcginniseharney: I like that idea, but I don't think that's going to work -
smcginniseharney: At least without adding a crap-ton pile of ignore-notes to the index pages.16:35
eharneysmcginnis: i was wondering about that... but not having any way to validate these seems like a bad idea too16:37
smcginniseharney: There's a "reno lint" subcommand.16:37
smcginnisBut still would have the issue of needing to update old notes.16:38
smcginniseharney: Doug was saying there are some plans in the works to just make these rst files too.16:38
eharneyi dunno, will think about it some more later... what we have committed so far is quite inconsistent :/16:40
*** eharney has quit IRC16:40
SwansonHey, smcginnis,
smcginnisSure, if your head is clamped into place. Otherwise two spacing is just plain wrong!16:49
Swanson2 spaces is as god intended.16:50
smcginnisIt's the work of the devil!16:51
smcginnisExtra spaces spread things out. So if it's printed, it needs more paper. More paper means cutting down more trees. That leads to global warming and natural disasters. That disrupts established civilations. So double spacing is leading to the end of the world and it's all your fault. :P16:52
SwansonNo one prints anymore. And twitter doubled the tweet size! All the space in the world. We can go to 3 spaces.16:54
SwansonMy typing teacher is rolling over in his grave.16:55
openstackgerritMerged openstack/cinder master: QNAP: driver fails to detach while uploading volume to image
*** jmlowe_ has quit IRC17:10
*** mriedem has quit IRC17:10
erlonsmcginnis, jungleboyj tpsilva: If Im not wrong, the NetApp driver does not runs the upstream tempest tests because we need to pass an specific driver provided handle that only works by default on LVM17:14
*** dpawlik has joined #openstack-cinder17:17
*** alexchadin has joined #openstack-cinder17:18
*** dpawlik has quit IRC17:21
*** e0ne has joined #openstack-cinder17:35
_alastor_Anyone here with experience in the OpenStackSDK?  I've been trying to hail the SDK folks and had limited success17:37
*** moshele has joined #openstack-cinder17:40
_alastor_It's somewhere in the Connection constructor, because volume_api_version is '2' when returned from 'get_one'17:50
*** alexchadin has quit IRC17:52
*** lkuchlan has quit IRC17:52
_alastor_whoops, wrong window17:56
*** eharney has joined #openstack-cinder17:58
*** jmlowe has joined #openstack-cinder18:04
Swansonsmcginnis, Jeff sez you're a two space denier.18:07
Swanson(Wait, that joke has implications. Pretend I didn't type it.)18:12
*** jmlowe has quit IRC18:13
*** jmlowe has joined #openstack-cinder18:15
clarkb_alastor_: the defaults come from openstacksdk/openstack/config/defaults.json and volume api is defaulted to v218:15
*** mvenesio has joined #openstack-cinder18:18
smcginnisSwanson: LOL18:36
*** dave-mccowan has quit IRC18:36
*** dave-mcc_ has joined #openstack-cinder18:36
*** mriedem has joined #openstack-cinder18:39
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation
*** kaisers has quit IRC18:44
*** kaisers has joined #openstack-cinder18:45
e0nedoes anybody know what python2 version do we use at gates?18:47
_alastor_clarkb: I'm adding API v3 support for volume api, but the volume API is defaulting to 3 and ignoring the value in 'volume_api_version' in the config18:47
clarkb_alastor_: defaulting to 2 you mean?18:48
_alastor_clarkb: so the _proxy.Proxy object is always v3 when it should be configurable18:48
_alastor_You can see my changes in this review:
clarkbas is that should continue to default to v2 since defaults.json isn't updated. YOu are saying it always uses v3 instead? If so interesting and probably beyond my understanding of the sdk18:50
_alastor_clarkb: It's defaulting to 3 which is why I'm confused.  I know they treat cinder special in the SDK because of our versioning, but I can't nail down why this is happening18:52
clarkb_alastor_: I do know the order of the list in also has some impact as first one there is used in some cases I think18:53
clarkbServiceFilter._get_valid_version returns first item in that list18:54
_alastor_clarkb: Yeah, it uses whichever is first in the list, but changing identity_api_version gets you the correct Keystone proxy even though it has multiple entries18:55
smcginnise0ne: I think it's 2.7.14. Whatever is the default on Xenial.18:57
smcginnise0ne: Might have to ask in -infra to make sure.18:57
e0nesmcginnis: thanks. let me check on my second env first18:58
eharneye0ne: zuul-info/host-info in the unit test job logs will tell you18:58
e0neI've got strange errors on my macbook since I updated python to 2.7.1518:59
e0neeharney: thank you, Eric!19:00
smcginnisIs it the same as ansible_python?19:00
eharneyi was assuming so19:01
smcginnisYeah, looks reasonable. I'm seeing
smcginnise0ne: Can you actually run tests on Mac? I've never been able to get our tests to work on there.19:02
clarkbansible currently always runs under python2 on the host aiui. But you can run your jobs on any python you get installed19:02
e0nesmcginnis: unit tests worked for me except test_cmd.py19:02
smcginnisclarkb: So py2 jobs are safe to assume running under what's shown for ansible_python_version?19:02
eharneythere's a dpkg-l.txt.gz on tempest jobs that will show the python package as well19:03
clarkbsmcginnis: yes unless you manage to do something weird with bindep19:03
smcginnise0ne: I had to mock out some rtslib things, but then I was running in to some other things.19:03
smcginnisclarkb: Thanks!19:03
e0nesmcginnis: it's pretty sad, but mac doesn't fit good for openstack development :(19:04
smcginnise0ne: Yeah. I at least would like to run the docs jobs.19:04
*** alexchadin has joined #openstack-cinder19:04
smcginnise0ne: I started working through some issues. If I ever get it all working I'll submit it.19:04
smcginnise0ne: Even if we can't do a full tox run, if we can at least get most of it to run on Mac I think it would help.19:05
e0nesmcginnis: +119:05
*** dave-mccowan has joined #openstack-cinder19:07
*** e0ne_ has joined #openstack-cinder19:07
*** dave-mcc_ has quit IRC19:07
clarkb_alastor_: seems like CloudRegion.get_service_type() is probably what needs debugging?19:08
*** eharney has quit IRC19:09
*** e0ne has quit IRC19:11
_alastor_clarkb: I saw that and had a laugh.  It seems like a candidate, but that's not the same format as volume_api_version19:12
*** eharney has joined #openstack-cinder19:13
_alastor_clarkb: I also tried putting volume_api_version in the format of 'volumev2' etc, but it didn't affect anything19:13
clarkb_alastor_: I think its converting volume_api_version to use teh volumev2 and volumev3 types19:13
clarkbvol_ver = self.get_api_version('volume') being the first step in that mapping19:14
_alastor_clarkb: Ah, I see.  I'll push on that a bit19:14
_alastor_clarkb: Nope, something else.  'volumev2' gets returned from that function as expected, but the proxy is still for v319:17
_alastor_clarkb: looks like the constructor argument is v3 already before that function is even called19:21
*** evin has joined #openstack-cinder19:24
clarkb_alastor_: even though self.get_api_version('volume') returns 2?19:24
*** jmlowe has quit IRC19:27
_alastor_clarkb: config.get_api_version('block_store') returns None.19:28
_alastor_So it's defaulting19:28
_alastor_In openstack/
clarkboh so it likely should be block_storage_api_version ?19:30
clarkbI wonder if this is just an internal mixup between using service_type volume and block_store19:30
_alastor_clarkb: I think that's our smoking gun.  Here's my config:
_alastor_It has volume_api_version, but no block_store19:33
clarkbya but it sort of handles both in that get_service_type method19:34
_alastor_clarkb: But it doesn't since get_proxy_class relies on what is returned from the config dict and 'block_storage' isn't populated19:36
_alastor_So instead of constructing 'v2' for the proxy class, it defaults19:36
clarkbya I don't think it is sufficient handling of both19:36
clarkbI bet that is the bug19:37
*** alexchadin has quit IRC19:38
_alastor_clarkb: A simple patch would be to put the special check for volume/block-store into get_api_version() to ensure volume_api_version is used instead of block_store19:39
clarkbor maybe update get_session_endpoint to handle both?19:41
clarkband let the "low level" get_api_version return just what it asked for?19:42
_alastor_clarkb: get_session_endpoint isn't called when instantiating the connection object directly with a config19:51
_alastor_clarkb: At least not the one in cloud_region19:51
_alastor_clarkb: And opening causes my vim session to freak out :P19:52
*** lpetrut has quit IRC19:54
*** moshele has quit IRC20:02
*** alexchadin has joined #openstack-cinder20:03
*** liverpooler has quit IRC20:04
_alastor_clarkb: even doesn't really use get_session_endpoint.  There's a single call to it uses to return a boolean (has_service()), but otherwise it's unused20:06
openstackgerritVipin Balachandran proposed openstack/cinder master: Log backup_service.restore() exception
*** eharney has quit IRC20:26
*** alexchadin has quit IRC20:30
*** pchavva has quit IRC20:33
*** mvenesio has quit IRC20:38
*** mvenesio has joined #openstack-cinder20:39
*** fmccrthy_ has joined #openstack-cinder20:45
*** tpsilva_ has joined #openstack-cinder20:46
*** xdfil_ has joined #openstack-cinder20:46
*** guyr-infinidat_ has joined #openstack-cinder20:46
*** chrisyang_0660_ has joined #openstack-cinder20:46
*** knikolla_ has joined #openstack-cinder20:47
*** simondodsley_ has joined #openstack-cinder20:47
*** mriedem has quit IRC20:48
*** tezar_ has joined #openstack-cinder20:48
*** luizbag has quit IRC20:48
*** mvenesio has quit IRC20:53
*** tpsilva has quit IRC20:53
*** knikolla has quit IRC20:53
*** mvk has quit IRC20:53
*** tezar has quit IRC20:53
*** wN_ has quit IRC20:53
*** fmccrthy has quit IRC20:53
*** simondodsley has quit IRC20:53
*** xdfil has quit IRC20:53
*** guyr-infinidat has quit IRC20:53
*** chrisyang_0660 has quit IRC20:53
*** chrisyang_0660_ is now known as chrisyang_066020:53
*** knikolla_ is now known as knikolla20:53
*** tpsilva_ is now known as tpsilva20:53
*** fmccrthy_ is now known as fmccrthy20:53
*** guyr-infinidat_ is now known as guyr-infinidat20:53
*** simondodsley_ is now known as simondodsley20:53
*** xdfil_ is now known as xdfil20:53
*** mvk has joined #openstack-cinder20:54
*** storshoo___ has quit IRC20:55
*** johnthetubaguy_ has quit IRC20:56
*** wN_ has joined #openstack-cinder20:57
*** johnthetubaguy has joined #openstack-cinder20:58
*** a_storshoo has joined #openstack-cinder20:58
_alastor_clarkb: The only place I can see where we can insert the fix (other than get_api_version) is get_proxy_class in OpenStackServiceDescription21:00
_alastor_clarkb: Because that's the method that needs to return the correct proxy and we can intercept the call to get_api_version based on self.service_type21:01
*** abishop has quit IRC21:10
*** dustins has quit IRC21:13
*** edmondsw has quit IRC21:15
*** edmondsw has joined #openstack-cinder21:15
*** edmondsw has quit IRC21:20
*** dpawlik has joined #openstack-cinder21:21
*** evin has quit IRC21:22
*** dpawlik has quit IRC21:26
_alastor_jungleboyj smcginnis: should we update the openrc in devstack to use APIv3?21:29
_alastor_It's a oneline change from what I can tell21:29
_alastor_jungleboyj smcginnis:
*** moshele has joined #openstack-cinder21:34
clarkb_alastor_: I guess there is already precedent for that in get_service_type21:35
clarkb_alastor_: I would make something that works then mordred and others can weigh in better than I can21:36
jungleboyj_alastor_: it isn't set that way already?21:36
_alastor_jungleboyj: Nope, it defaults to 2 unless you set CINDER_VERSION explicitly21:36
smcginnisI've been trying to get tempest to switch over to v3 for a long time now.21:37
smcginnisWe might need that done first, but I suppose we could switch devstack and see.21:37
smcginnisI would like it all switched over ideally.21:37
clarkb_alastor_: cool21:38
_alastor_jungleboyj: Devstack sets up v3 API cinder, it's just the openrc that sets the version used to 221:39
*** e0ne_ has quit IRC21:50
*** throwsb1 has quit IRC21:51
jungleboyjsmcginnis:  ++ Given that we have deprecated v2 we should get everything moved.21:55
*** dave-mccowan has quit IRC22:01
jungleboyj_alastor_:  Thanks for the patch.22:01
*** smcginnis[m] has joined #openstack-cinder22:03
* hemna_ just discovered the awesomeness that is lnav....22:08
hemna_that is all22:08
*** moshele has quit IRC22:13
*** felipemonteiro__ has quit IRC22:24
*** rcernin has joined #openstack-cinder22:25
*** ganso has quit IRC22:25
*** dave-mccowan has joined #openstack-cinder22:55
*** _alastor_ has quit IRC22:55
-openstackstatus- NOTICE: Any devstack job failure due to rsync errors related to tripleo-incubator can safely be rechecked now22:56
*** _alastor_ has joined #openstack-cinder23:18
*** cloudnull is now known as cloudkiller23:20
*** cloudkiller is now known as cloudnull23:21
*** r-daneel has quit IRC23:28
*** pooja_jadhav has quit IRC23:28
*** pooja_jadhav has joined #openstack-cinder23:28
*** Nel1x has joined #openstack-cinder23:41
*** stakeda has joined #openstack-cinder23:47

Generated by 2.15.3 by Marius Gedminas - find it at!