Tuesday, 2019-12-17

*** brinzhang has joined #openstack-cinder00:23
*** brinzhang_ has joined #openstack-cinder00:26
*** brinzhang has quit IRC00:29
*** Liang__ has joined #openstack-cinder00:59
*** awalende has joined #openstack-cinder01:19
*** rosmaita has left #openstack-cinder01:20
*** awalende has quit IRC01:24
*** Lucas_Gray has joined #openstack-cinder02:01
*** yikun has joined #openstack-cinder02:16
*** sam_wan has quit IRC02:45
*** whfnst has joined #openstack-cinder03:13
*** psachin has joined #openstack-cinder03:37
*** dave-mccowan has quit IRC03:38
*** awalende has joined #openstack-cinder03:46
*** awalende has quit IRC03:51
*** zainub_wahid has joined #openstack-cinder03:58
*** Lucas_Gray has quit IRC04:17
*** brinzhang has joined #openstack-cinder04:22
*** brinzhang has quit IRC04:23
*** brinzhang has joined #openstack-cinder04:24
*** brinzhang_ has quit IRC04:25
*** Xuchu has joined #openstack-cinder04:34
*** whoami-rajat has joined #openstack-cinder04:47
*** ChipOManiac has joined #openstack-cinder05:00
ChipOManiacHi all. We're currently attempting to use the LINSTOR backend for Cinder in our OpenStack-Ansible setup and are running into a few problems. Has anyone been able to successfully use this backend in their deployments?05:03
ChipOManiacThe problem seems to specifically be with the cinder-volume services appearing offline because the linstor driver cant find the "Linstor" package.05:04
whoami-rajatChipOManiac, Hi, could you share the c-vol logs for the errror traceback?05:05
ChipOManiacSure. Would a pastebin be fine?05:07
ChipOManiacLine 805 is the python exception.05:11
whoami-rajatChipOManiac, i think the error is pretty straight forward05:24
whoami-rajatyou need to define 'enabled_backends=<driver_group_name>' in cinder.conf05:24
whoami-rajatyou would be having a [linstor] (or any name) section in cinder.conf, that would replace the <driver_group_name>05:25
ChipOManiacOh those were there initially. I've already added the enabled_backends.05:30
ChipOManiacThe current problem with the linstor driver is the error at Line 805.05:30
ChipOManiacThe driver complains that it doesn't have the Linstor python package.05:30
ChipOManiacpip doesn't supply one and I tried to do a pip install of the tar.gz file provided on github. But that doesn't seem to work.05:31
whoami-rajatChipOManiac, ok, let me take another look05:32
whoami-rajatChipOManiac, did you try installing 'python-linstor' package?05:36
ChipOManiacNo, I didn't. I'll take a look.05:38
whoami-rajatChipOManiac, and could you tell me which distro and openstack release are you using?05:40
ChipOManiacYeah. I'm using Ubuntu Bionic. Openstack Release Stein.05:41
ChipOManiacNo dice. Says "no matching distribution found".05:49
ChipOManiacFrom pip install.05:50
*** brinzhang has quit IRC05:59
*** brinzhang has joined #openstack-cinder06:00
*** tommylikehu has joined #openstack-cinder06:04
whoami-rajatChipOManiac, so i'm not really sure what is happening, the documentation still says DRBD (replaced by linstor) [1], they've removed the driver requirement for python-linstor[2] so as to support pypi package but it isn't available at pypi also, lastly to check their driver status, their CI is reporting success on every patch but the result page isn't available[3]  [1] https://docs.linbit.com/docs/users-guide-9.0/#p-build-install-configure [2]06:13
whoami-rajathttps://review.opendev.org/#/c/657856/ [3] http://us.linbit.com:8080/CI-LINSTOR/30/313206:13
whoami-rajatChipOManiac, so their driver 'should be' working fine but i'm not seeing it easily, i will wait for woojay to take a look at your issue since he's the driver maintainer.06:14
whoami-rajatChipOManiac, you can also a bug for the same for more visibility here https://bugs.launchpad.net/cinder06:15
ChipOManiacI see.06:16
ChipOManiacFWIW. The python-linstor package is installed, just not via PyPI.06:16
ChipOManiacIf I do an apt search it shows up. But since the cinder-volume is in a venv I guess it doesn't pass through.06:17
ChipOManiacActually scratch that. My linstor controller is the same as the cinder host. I guess that's where the python-linstor apt package comes from.06:18
ChipOManiacDo you think this is more of an OpenStack-Ansible problem? Or is it just that Cinder has overlooked this?06:19
whoami-rajatChipOManiac, from the start it felt like a deployment tool issue to me06:20
whoami-rajatChipOManiac, i was just trying to get a manual workaround for what the deployment tool had missed06:20
whoami-rajatChipOManiac, G2G, will be back in half hour06:21
ChipOManiacTYT. And thanks.06:21
*** bhagyashris has joined #openstack-cinder06:26
*** whoami-rajat has quit IRC06:26
*** lpetrut has joined #openstack-cinder06:48
*** morks has joined #openstack-cinder06:50
*** whoami-rajat__ has joined #openstack-cinder06:51
*** sahid has joined #openstack-cinder06:58
*** kozhukalov has joined #openstack-cinder06:59
*** anastzhyr has joined #openstack-cinder06:59
*** awalende has joined #openstack-cinder07:15
*** awalende has quit IRC07:20
*** bhagyashris has quit IRC07:26
*** bhagyashris has joined #openstack-cinder07:41
*** pcaruana has joined #openstack-cinder07:52
*** lpetrut has quit IRC07:54
*** bhagyashris has quit IRC08:06
*** tkajinam has quit IRC08:18
*** awalende has joined #openstack-cinder08:19
*** brinzhang_ has joined #openstack-cinder08:20
*** tosky has joined #openstack-cinder08:20
*** brinzhang_ has quit IRC08:21
*** brinzhang_ has joined #openstack-cinder08:22
*** brinzhang has quit IRC08:23
*** tesseract has joined #openstack-cinder08:25
*** yikun has quit IRC08:26
*** brinzhang has joined #openstack-cinder08:34
*** brinzhang has quit IRC08:36
*** brinzhang has joined #openstack-cinder08:37
*** brinzhang_ has quit IRC08:37
*** bhagyashris has joined #openstack-cinder09:01
*** jc_ has joined #openstack-cinder09:01
*** dklyle_ has joined #openstack-cinder09:01
*** ildikov_ has joined #openstack-cinder09:03
*** patrickeast_ has joined #openstack-cinder09:03
*** samc-bbc_ has joined #openstack-cinder09:03
*** coreycb_ has joined #openstack-cinder09:03
*** knikolla_ has joined #openstack-cinder09:03
*** adrianc_ has joined #openstack-cinder09:03
*** carloss_ has joined #openstack-cinder09:03
*** tosky_ has joined #openstack-cinder09:04
*** dasp has joined #openstack-cinder09:05
*** benj_- has joined #openstack-cinder09:05
*** Anticime1 has joined #openstack-cinder09:05
*** lpetrut has joined #openstack-cinder09:06
*** cmurphy has joined #openstack-cinder09:07
*** amotoki_ has joined #openstack-cinder09:09
*** anastzhyr has quit IRC09:09
*** tesseract has quit IRC09:10
*** tosky has quit IRC09:10
*** kozhukalov has quit IRC09:10
*** psachin has quit IRC09:10
*** rcernin has quit IRC09:10
*** Dinesh_Bhor has quit IRC09:10
*** tbarron has quit IRC09:10
*** benj_ has quit IRC09:10
*** david-lyle has quit IRC09:10
*** sorrison has quit IRC09:10
*** kambiz has quit IRC09:10
*** adrianc has quit IRC09:10
*** dasp_ has quit IRC09:10
*** jc__ has quit IRC09:10
*** fnordahl has quit IRC09:10
*** jkulik has quit IRC09:10
*** cmorpheus has quit IRC09:10
*** carloss has quit IRC09:10
*** samc-bbc has quit IRC09:10
*** amotoki has quit IRC09:10
*** Anticimex has quit IRC09:10
*** coreycb has quit IRC09:10
*** knikolla has quit IRC09:10
*** ildikov has quit IRC09:10
*** patrickeast has quit IRC09:10
*** patrickeast_ is now known as patrickeast09:12
*** carloss_ is now known as carloss09:12
*** samc-bbc_ is now known as samc-bbc09:12
*** knikolla_ is now known as knikolla09:12
*** coreycb_ is now known as coreycb09:12
*** ildikov_ is now known as ildikov09:12
*** benj_- is now known as benj_09:12
*** Dinesh_Bhor has joined #openstack-cinder09:16
*** tesseract has joined #openstack-cinder09:17
*** rcernin has joined #openstack-cinder09:18
*** tosky_ is now known as tosky09:19
*** Liang__ has quit IRC09:30
*** anastzhyr has joined #openstack-cinder09:42
*** yaawang has quit IRC09:44
*** e0ne has joined #openstack-cinder10:01
*** tommylikehu has quit IRC10:03
*** zainub_wahid has quit IRC10:11
*** ChipOManiac has quit IRC10:27
*** jawad_axd has joined #openstack-cinder10:29
*** ociuhandu has joined #openstack-cinder10:49
*** kozhukalov has joined #openstack-cinder10:56
lennybe0ne, kindly review Python3 trivial Bug fix for NVMe https://review.opendev.org/#/c/698202/11:01
lennybeOne, please ignore ^^11:02
e0nelennyb: it's already approved11:02
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - RDF clean snapvx target fix  https://review.opendev.org/69938911:06
*** bhagyashris has quit IRC11:19
*** bhagyashris has joined #openstack-cinder11:25
*** ociuhandu has quit IRC11:27
*** ociuhandu has joined #openstack-cinder11:28
*** dviroel has joined #openstack-cinder11:31
*** ociuhandu has quit IRC11:32
*** ociuhandu has joined #openstack-cinder11:33
*** ianychoi has quit IRC11:38
*** ociuhandu has quit IRC11:39
*** ianychoi has joined #openstack-cinder11:40
openstackgerritMerged openstack/cinder master: Fixed open tempfile.NamedTemporaryFile as text in Python3  https://review.opendev.org/69820211:47
*** ociuhandu has joined #openstack-cinder11:55
*** brinzhang_ has joined #openstack-cinder11:57
*** brinzhang has quit IRC12:00
*** brinzhang_ has quit IRC12:04
*** Xuchu has quit IRC12:07
*** amotoki_ is now known as amotoki12:12
*** rosmaita has joined #openstack-cinder12:35
*** tesseract has quit IRC12:38
*** tesseract has joined #openstack-cinder12:38
whoami-rajat__smcginnis, Hi, regarding https://review.opendev.org/#/c/698921/ . the patch is to set volume_type = {} so it doesn't pass as None in filter_properties.(this patch isn't modifying resource_type)12:49
smcginniswhoami-rajat__: I see that. Please add a comment on there to note that is the desired side effect.12:50
whoami-rajat__smcginnis, sure will do12:52
openstackgerritRajat Dhasmana proposed openstack/cinder master: Fix: failed to create snapshot with DriverFilter  https://review.opendev.org/69892112:58
whoami-rajat__smcginnis, ^ Done12:58
*** whoami-rajat__ has quit IRC13:09
*** whoami-rajat__ has joined #openstack-cinder13:11
*** udesale has joined #openstack-cinder13:11
*** udesale has quit IRC13:12
*** udesale has joined #openstack-cinder13:13
*** bhagyashris has quit IRC13:28
*** ociuhandu has quit IRC13:28
*** ociuhandu has joined #openstack-cinder13:28
*** ociuhandu has quit IRC13:34
*** udesale has quit IRC13:39
*** dpawlik has joined #openstack-cinder14:00
*** ociuhandu has joined #openstack-cinder14:01
*** tkajinam has joined #openstack-cinder14:04
*** enriquetaso has joined #openstack-cinder14:16
openstackgerritMerged openstack/os-brick master: Remove Python 2.7 support from testing and gates  https://review.opendev.org/69737414:19
*** ociuhandu has quit IRC14:21
*** eharney has joined #openstack-cinder14:42
*** awalende has quit IRC14:42
*** awalende has joined #openstack-cinder14:43
*** awalende_ has joined #openstack-cinder14:46
*** tkajinam has quit IRC14:47
*** awalende has quit IRC14:47
*** jawad_axd has quit IRC14:47
*** jawad_axd has joined #openstack-cinder14:49
openstackgerritLuigi Toscano proposed openstack/os-brick master: Port the os-bricks jobs to Zuul v3  https://review.opendev.org/69738514:49
*** awalende_ has quit IRC14:51
*** jawad_axd has quit IRC14:53
*** jawad_axd has joined #openstack-cinder14:54
*** jawad_axd has quit IRC14:58
*** kozhukalov has quit IRC15:07
*** anastzhyr has quit IRC15:12
*** awalende has joined #openstack-cinder15:14
*** ociuhandu has joined #openstack-cinder15:14
*** ociuhandu has quit IRC15:18
*** awalende has quit IRC15:18
*** ociuhandu has joined #openstack-cinder15:36
*** senrique_ has joined #openstack-cinder15:38
*** enriquetaso has quit IRC15:41
*** eharney has quit IRC15:43
*** ociuhandu has quit IRC15:46
*** senrique__ has joined #openstack-cinder15:58
*** senrique_ has quit IRC16:00
*** senrique_ has joined #openstack-cinder16:01
*** senrique__ has quit IRC16:04
openstackgerritLuigi Toscano proposed openstack/cinder master: WIP Port several legacy tempest jobs to Zuul v3  https://review.opendev.org/67194516:04
*** gkadam has joined #openstack-cinder16:06
*** whoami-rajat__ has quit IRC16:06
*** ociuhandu has joined #openstack-cinder16:07
toskyI wonder if we should restrict some of the regexp for the secondary jobs, like not running api.neutron tests on the multibackend job above ^^16:08
smcginnistosky: Oh, good catch. I definitely agree with that.16:09
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Docs -  corrections and improvements  https://review.opendev.org/68956416:09
*** B_dgfebba has joined #openstack-cinder16:11
*** ociuhandu has quit IRC16:12
*** B_dgfebba has quit IRC16:15
*** gkadam has quit IRC16:15
*** e0ne has quit IRC16:20
*** ociuhandu has joined #openstack-cinder16:27
*** ociuhandu has quit IRC16:32
*** eharney has joined #openstack-cinder16:43
*** lpetrut has quit IRC16:44
*** B_cgcfdhc has joined #openstack-cinder16:51
*** ociuhandu has joined #openstack-cinder16:53
*** efried_pto has quit IRC16:54
*** ociuhandu has quit IRC16:55
*** ociuhandu has joined #openstack-cinder16:55
*** B_cgcfdhc has quit IRC16:55
*** tesseract has quit IRC17:00
geguileosmcginnis: rosmaita Regarding the cinderlib stuff we talked yesterday... I was thinking that maybe we just remove the whole thing and don't support drivers that do weird stuff with the configuration17:49
geguileoinstead of supporting dynamically adding options and reloading configs17:49
geguileowhen we encounter those issues, they have to update their driver17:49
geguileoit will simplify the code in cinderlib17:50
geguileoand prevent us from introducing new bugs around this whole parsing thing17:50
geguileowhat do you guys think?17:51
smcginnisgeguileo: I'm all for simplifying and not doing weird things in the drivers.17:51
geguileoyeah, it made sense doing weird stuff in cinderlib to accomodate drivers when it was my personal project17:52
smcginnisIt would be good to understand why they are doing things like that in the first place.17:52
geguileobut as an OpenStack one, it makes sense that we should "force" drivers to do the right thing instead17:52
smcginnisIt might be a good opportunity to educate them.17:52
geguileosmcginnis: +117:52
geguileoI'll work on a patch to clean it all up17:53
smcginnisAwesome! Thanks geguileo17:53
geguileosmcginnis: thank you for opening my eyes  :-)17:54
*** morks has quit IRC17:55
rosmaitageguileo: sounds great to me18:01
*** sahid has quit IRC18:03
*** ociuhandu has quit IRC18:05
*** jdillaman has joined #openstack-cinder18:05
Roamer`on the topic of fewer drivers doing weird config stuff, does anybody have any idea why Zuul's openstack-tox-lower-constraints doesn't like my trivial config change in https://review.opendev.org/#/c/698934/ - the last failed run is https://zuul.opendev.org/t/openstack/build/3cc4182083b04b04aa47184222bc38d0 and it seems unrelated, it looks a bit like an environment/infra issue; should I recheck18:18
Roamer`again, or should I ask in -infra or something?18:18
toskygeguileo: uhm, I see you added a new cinderlib_pool_name variable in roles/run-cinderlib-tests - but what is the default value?18:23
toskyit's never assigned18:23
eharneyRoamer`: i think that failure is just a flaky cinder unit test, should just recheck it for now18:23
geguileotosky: if it's not assigned it should get None, and then cinderlib will use the first one reported by the driver18:24
geguileotosky: I thought I had added it to the commit message...18:24
toskygeguileo: but then it should get a |default(''), or there is an error when assigning CL_FTEST_POOL_NAME18:24
toskyNone or '' ?18:24
Roamer`eharney, thanks, I'll retry18:26
geguileotosky: oh, you are talking about the ansible role18:26
toskygeguileo: yep, and I wonder how it didn't fail18:27
toskygeguileo: this is what I see in the new os-brick jobs: https://zuul.opendev.org/t/openstack/build/a60e157ef33644c7aa027ae3022f47b1/log/job-output.txt#3603218:27
geguileotosky: I haven't done ansible is a while, but I don't think you need a default18:27
geguileotosky: It will just render empty iirc18:27
tosky(and the updated version of the lio job in cinder-tempest-plugin is going to fail too)18:28
toskygeguileo: if it's not defined, as it is now, you get the error above18:28
toskygeguileo: if you need to distinguish between None and '', the fix is going to be a bit more complicated18:31
rosmaitatosky: thanks for troubleshooting that, i was hoping it was just some random weirdness18:37
toskygeguileo: also: can I add a job which uses the role to cinderlib? So it won't break anymore18:38
*** pcaruana has quit IRC18:38
geguileotosky: did I break the cinder CI/18:40
toskygeguileo: just my yet-uncommitted zuulv3 jobs :)18:41
toskybut if they have been merged already, that would have been a breakage18:41
toskywhich is not good18:41
toskyso let's see: I should convert the existing jobs to use the role, and fix the passed variable18:41
geguileotosky: mmmm, I though there was a job already using it... but I can't find it18:43
geguileotosky: OK, the problem is that I forgot to add the default in defaults/main.yaml18:45
geguileotosky: empty string is fine as the default, by the way18:45
geguileotosky: if you propose the patch to fix this I can review18:46
toskygeguileo: just to be sure, as get('VARIABLE', None) won't return None but '' - if it's fine as value for pool_name?18:53
geguileotosky: https://github.com/openstack/cinderlib/blob/master/cinderlib/objects.py#L30918:55
geguileotosky: https://github.com/openstack/cinderlib/blob/master/cinderlib/objects.py#L34218:55
geguileotosky: it should be fine, I'm doing an or, so it doesn't matter if it's None or ''18:55
geguileoand apparently I forgot to remove the TODO when I added the pool support18:56
*** dave-mccowan has joined #openstack-cinder18:56
openstackgerritLuigi Toscano proposed openstack/cinderlib master: Fix: add a default value for cinderlib_pool_name  https://review.opendev.org/69948118:58
toskygeguileo: ^^18:58
toskylet's fix it for now, I will think about how to refactor the jobs to use run-cinderlib-tests later18:58
geguileotosky: thanks18:59
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Update cinder-tempest-plugin-lvm-lio (revamp)  https://review.opendev.org/67280419:01
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Enable volume_revert tests  https://review.opendev.org/69729119:01
toskylet's see19:01
toskygeguileo: https://review.opendev.org/672804 is the test19:01
toskyand also https://review.opendev.org/69738519:02
geguileotosky: sorry for breaking your patches :-(19:02
*** jawad_axd has joined #openstack-cinder19:05
*** jawad_axd has quit IRC19:10
*** pcaruana has joined #openstack-cinder19:18
*** awalende has joined #openstack-cinder19:20
*** awalende has quit IRC19:25
*** jawad_axd has joined #openstack-cinder19:26
*** e0ne has joined #openstack-cinder19:27
*** jawad_axd has quit IRC19:31
*** eharney has quit IRC19:45
*** ianw is now known as ianw_pto19:50
*** pcaruana has quit IRC19:59
*** jawad_axd has joined #openstack-cinder20:05
*** jawad_axd has quit IRC20:09
*** whfnst has quit IRC20:15
*** Lucas_Gray has joined #openstack-cinder20:25
*** jawad_axd has joined #openstack-cinder20:26
*** jawad_axd has quit IRC20:30
*** senrique_ has quit IRC20:35
*** pcaruana has joined #openstack-cinder20:42
*** eharney has joined #openstack-cinder20:42
*** kozhukalov has joined #openstack-cinder21:02
*** e0ne has quit IRC21:25
*** kozhukalov has quit IRC21:29
*** Lucas_Gray has quit IRC21:37
*** pcaruana has quit IRC21:41
*** rcernin has quit IRC21:46
Roamer`smcginnis, jungleboyj, https://review.opendev.org/#/c/698934/ made it through Zuul... would you have a minute to re-approve it after I actually regenerated cinder/opts.py? Thanks a lot!21:52
*** dpawlik has quit IRC21:53
*** dviroel has quit IRC22:10
*** dave-mccowan has quit IRC22:30
openstackgerritMerged openstack/cinderlib master: Fix: add a default value for cinderlib_pool_name  https://review.opendev.org/69948122:37
*** rcernin has joined #openstack-cinder22:42
*** jawad_axd has joined #openstack-cinder22:51
*** jawad_axd has quit IRC22:55
*** sorrison has joined #openstack-cinder23:10
*** dklyle_ is now known as dklyle23:12
*** tkajinam has joined #openstack-cinder23:13
*** awalende has joined #openstack-cinder23:21
*** awalende has quit IRC23:25
*** ociuhandu has joined #openstack-cinder23:30
*** jawad_axd has joined #openstack-cinder23:32
*** ociuhandu has quit IRC23:35
openstackgerritMerged openstack/cinder master: StorPool: move the config to the shared group.  https://review.opendev.org/69893423:36
*** jawad_axd has quit IRC23:37
*** jawad_axd has joined #openstack-cinder23:53
*** jawad_axd has quit IRC23:57

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