Thursday, 2018-05-17

*** LutzB has quit IRC00:05
*** felipemonteiro has joined #openstack-cinder00:05
*** LutzB has joined #openstack-cinder00:09
*** felipemonteiro has quit IRC00:14
*** mriedem_afk has quit IRC00:15
*** liuzz has quit IRC00:17
*** liuzz has joined #openstack-cinder00:19
*** r-daneel has quit IRC00:29
*** wanghao has joined #openstack-cinder00:31
*** david-lyle has quit IRC00:34
*** wanghao_ has joined #openstack-cinder00:35
*** wanghao has quit IRC00:35
*** dpawlik has joined #openstack-cinder00:35
*** Dinesh_Bhor has joined #openstack-cinder00:39
*** dpawlik has quit IRC00:40
openstackgerritMerged openstack/cinder stable/ocata: Dell EMC SC: API 4.0 incompatibilities  https://review.openstack.org/56062100:49
*** wanghao has joined #openstack-cinder00:53
*** dhinesh has quit IRC00:57
*** wanghao_ has quit IRC00:57
*** kaisers_ has joined #openstack-cinder01:03
*** jiaopengju has joined #openstack-cinder01:04
*** kaisers has quit IRC01:07
*** lixiaoy1 has joined #openstack-cinder01:10
*** zhaochao has joined #openstack-cinder01:10
*** Nel1x has joined #openstack-cinder01:19
*** liverpooler has joined #openstack-cinder01:19
*** edmondsw has quit IRC01:19
*** jiaopengju has quit IRC01:29
*** jiaopeng_ has joined #openstack-cinder01:31
*** hemna_ has quit IRC01:32
*** markvoelker_ has quit IRC01:34
*** markvoelker has joined #openstack-cinder01:38
*** namnh has joined #openstack-cinder01:40
*** jiaopengju has joined #openstack-cinder01:48
*** jiaopeng_ has quit IRC01:51
openstackgerritTommyLike proposed openstack/cinder master: Support availability-zone type  https://review.openstack.org/55224301:59
*** itlinux has joined #openstack-cinder02:02
*** dalgaaf has quit IRC02:04
*** dalgaaf has joined #openstack-cinder02:04
*** Dinesh_Bhor has quit IRC02:09
*** LutzB has quit IRC02:10
*** LutzB has joined #openstack-cinder02:10
*** Dinesh_Bhor has joined #openstack-cinder02:11
*** moshele has quit IRC02:24
*** itlinux has quit IRC02:30
*** moshele has joined #openstack-cinder02:32
*** moshele has quit IRC02:37
*** itlinux has joined #openstack-cinder02:38
*** lbragstad has quit IRC02:39
*** s-shiono has joined #openstack-cinder02:40
*** dhinesh has joined #openstack-cinder02:41
*** Nel1x has quit IRC02:45
*** dhinesh has quit IRC02:45
*** itlinux has quit IRC02:46
*** liverpooler has quit IRC02:46
*** dhinesh has joined #openstack-cinder02:47
*** masber has quit IRC02:49
*** dave-mcc_ has quit IRC02:52
*** masber has joined #openstack-cinder02:52
*** bkopilov has quit IRC02:57
*** psachin has joined #openstack-cinder02:59
*** edmondsw has joined #openstack-cinder02:59
neha_alhatjungleboyj: Hi03:03
openstackgerritMerged openstack/cinder master: [Api-ref] fix incorrect host value in worker cleanup api  https://review.openstack.org/56820103:04
*** edmondsw has quit IRC03:04
*** itlinux has joined #openstack-cinder03:08
openstackgerritzhuangzejian proposed openstack/cinder master: Add doc about backend_default config  https://review.openstack.org/56872503:09
*** markstur has joined #openstack-cinder03:10
*** bkopilov has joined #openstack-cinder03:10
*** markstur has quit IRC03:10
*** neha_alhat has quit IRC03:10
*** markstur has joined #openstack-cinder03:11
*** eharney has quit IRC03:14
*** KeithMnemonic has joined #openstack-cinder03:15
openstackgerritTommyLike proposed openstack/cinder master: Support image signature verification  https://review.openstack.org/55416603:16
*** dhinesh has quit IRC03:21
*** nicolasbock has quit IRC03:23
*** bkopilov has quit IRC03:29
*** itlinux has quit IRC03:34
openstackgerritzhuangzejian proposed openstack/cinder master: Replace os.makedirs to avoid process race  https://review.openstack.org/56900103:35
*** KeithMnemonic has quit IRC03:41
*** masuberu has joined #openstack-cinder03:56
*** gyankum has joined #openstack-cinder03:59
*** masber has quit IRC04:00
*** masber has joined #openstack-cinder04:02
*** masuberu has quit IRC04:05
*** links has joined #openstack-cinder04:24
*** edmondsw has joined #openstack-cinder04:48
*** edmondsw has quit IRC04:53
pooja_jadhavtommylikehu, smcginnis, jungleboyj: Please have a look on https://review.openstack.org/#/c/566530/04:56
*** Dinesh__Bhor has joined #openstack-cinder04:59
*** Dinesh_Bhor has quit IRC04:59
openstackgerritMerged openstack/cinder stable/ocata: Dell EMC SC: Find volume folder API call fails on root folders  https://review.openstack.org/56062205:01
*** bkopilov has joined #openstack-cinder05:13
*** chhagarw has joined #openstack-cinder05:24
chhagarwhttps://bugs.launchpad.net/cinder/+bug/177173705:45
openstackLaunchpad bug 1771737 in Cinder "cinder backup create fails withs Out of memory error while performing 1TB backup" [Undecided,New]05:45
*** liuzz has quit IRC05:47
*** liuzz has joined #openstack-cinder05:48
*** markstur has quit IRC05:51
*** vivsoni has joined #openstack-cinder05:51
*** lkuchlan has joined #openstack-cinder05:57
*** LutzB has quit IRC06:03
tommylikehupooja_jadhav:  sure:)06:11
*** armaan has joined #openstack-cinder06:12
*** liuzz_ has joined #openstack-cinder06:15
*** liuzz has quit IRC06:16
*** Dinesh__Bhor has quit IRC06:17
*** Dinesh__Bhor has joined #openstack-cinder06:17
*** liuzz_ has quit IRC06:20
*** liuzz has joined #openstack-cinder06:21
*** liuzz has quit IRC06:23
*** liuzz has joined #openstack-cinder06:25
openstackgerritzhuangzejian proposed openstack/cinder master: Replace os.makedirs to avoid process race  https://review.openstack.org/56900106:28
*** liuzz_ has joined #openstack-cinder06:32
*** liuzz has quit IRC06:33
*** hamdyk has joined #openstack-cinder06:46
openstackgerritliuyamin proposed openstack/python-cinderclient master: Add the parameter service-id for service cleanup command  https://review.openstack.org/56903606:52
*** lvdombrkr has joined #openstack-cinder06:57
*** tesseract has joined #openstack-cinder07:01
*** Dinesh__Bhor has quit IRC07:02
pooja_jadhavtommylikehu: Thanks for review :)07:18
*** dpawlik has joined #openstack-cinder07:24
*** alexchadin has joined #openstack-cinder07:44
*** AlexeyAbashkin has joined #openstack-cinder07:47
*** lvdombrkr89 has joined #openstack-cinder07:58
*** lvdombrkr has quit IRC08:00
*** rcernin has quit IRC08:12
*** alexchad_ has joined #openstack-cinder08:16
*** s-shiono has quit IRC08:16
*** alexchadin has quit IRC08:18
*** edmondsw has joined #openstack-cinder08:24
*** edmondsw has quit IRC08:29
openstackgerritzhuangzejian proposed openstack/cinder master: Replace os.makedirs to avoid process race  https://review.openstack.org/56900108:30
*** links has quit IRC08:32
*** gyankum has quit IRC08:33
*** gkadam has joined #openstack-cinder08:35
*** e0ne has joined #openstack-cinder08:39
*** alexchadin has joined #openstack-cinder08:42
*** alexchad_ has quit IRC08:44
*** links has joined #openstack-cinder08:49
*** lixiaoy1 has quit IRC08:54
*** ys_ has joined #openstack-cinder08:54
*** ys_ has quit IRC08:55
*** gyankum has joined #openstack-cinder09:04
*** alexchad_ has joined #openstack-cinder09:08
*** gyankum has quit IRC09:08
*** alexchadin has quit IRC09:10
openstackgerritAkhil Jayakumar proposed openstack/cinder master: [DNM] Cisco-CI-Experimental-Branch: cinder-znm-dev  https://review.openstack.org/56906409:12
*** LutzB has joined #openstack-cinder09:13
*** alexchadin has joined #openstack-cinder09:19
*** alexcha__ has joined #openstack-cinder09:20
*** dims has quit IRC09:20
*** storshoo has quit IRC09:21
*** t_storshoo has quit IRC09:21
*** a_storshoo has quit IRC09:21
*** alexchad_ has quit IRC09:22
*** alexchadin has quit IRC09:24
*** t_storshoo has joined #openstack-cinder09:25
*** a_storshoo has joined #openstack-cinder09:25
*** storshoo has joined #openstack-cinder09:29
*** neha_alhat has joined #openstack-cinder09:33
openstackgerritAkhil Jayakumar proposed openstack/cinder master: [DNM] Cisco-CI-Experimental-Branch: test_zm  https://review.openstack.org/56907409:37
*** dalgaaf has quit IRC09:45
*** gyankum has joined #openstack-cinder09:46
*** moshele has joined #openstack-cinder09:50
*** gyankum has quit IRC09:50
*** dalgaaf has joined #openstack-cinder09:51
*** AlexeyAbashkin has quit IRC09:53
*** moshele has quit IRC09:54
*** gyankum has joined #openstack-cinder09:56
*** namnh has quit IRC10:00
*** dalgaaf has quit IRC10:06
*** dims has joined #openstack-cinder10:07
*** dalgaaf has joined #openstack-cinder10:11
*** edmondsw has joined #openstack-cinder10:13
*** LutzB has quit IRC10:13
*** LutzB has joined #openstack-cinder10:13
*** edmondsw has quit IRC10:17
*** dims has quit IRC10:17
*** nicolasbock has joined #openstack-cinder10:30
*** wanghao_ has joined #openstack-cinder10:35
*** wanghao has quit IRC10:37
*** zhaochao has quit IRC10:41
*** hoonetorg has quit IRC10:43
*** zhaochao has joined #openstack-cinder10:47
*** armaan has quit IRC10:48
*** armaan has joined #openstack-cinder10:49
*** edmondsw has joined #openstack-cinder10:52
*** bkopilov has quit IRC10:54
*** AlexeyAbashkin has joined #openstack-cinder10:54
*** hoonetorg has joined #openstack-cinder11:00
*** dave-mccowan has joined #openstack-cinder11:09
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Add VMAX specific metadata to volume  https://review.openstack.org/55744811:14
*** gyan_ has joined #openstack-cinder11:14
*** gyankum has quit IRC11:16
*** links has quit IRC11:16
*** alexcha__ has quit IRC11:17
*** alexchadin has joined #openstack-cinder11:17
*** alexchadin has quit IRC11:18
*** alexchadin has joined #openstack-cinder11:18
*** alexchadin has quit IRC11:18
*** alexchadin has joined #openstack-cinder11:19
*** alexchadin has quit IRC11:19
*** alexchadin has joined #openstack-cinder11:20
*** luizbag has joined #openstack-cinder11:20
*** alexchadin has quit IRC11:20
*** armaan has quit IRC11:23
*** armaan has joined #openstack-cinder11:23
*** psachin has quit IRC11:28
*** links has joined #openstack-cinder11:29
*** lbragstad has joined #openstack-cinder11:33
*** dalgaaf has quit IRC11:34
*** abishop has joined #openstack-cinder11:36
*** psachin has joined #openstack-cinder11:37
*** moshele has joined #openstack-cinder11:40
*** stakeda has quit IRC11:42
*** bhagyashri_s has joined #openstack-cinder11:42
*** bhagyashri_s has quit IRC11:43
*** bhagyashri_s has joined #openstack-cinder11:44
*** bhagyashri_s has quit IRC11:45
*** bhagyashris has quit IRC11:46
*** bhagyashri_s has joined #openstack-cinder11:46
*** pooja-jadhav has joined #openstack-cinder11:48
*** pooja_jadhav has quit IRC11:50
*** vivsoni has quit IRC11:52
*** neha_alhat has quit IRC11:53
*** vivsoni has joined #openstack-cinder11:53
*** mriedem has joined #openstack-cinder11:57
*** liuzz_ has quit IRC11:57
*** lkuchlan has quit IRC11:58
*** alexchadin has joined #openstack-cinder11:58
*** hamdyk has quit IRC12:00
*** pooja_jadhav has joined #openstack-cinder12:02
*** bhagyashris_ has joined #openstack-cinder12:03
*** alexchadin has quit IRC12:04
*** pooja-jadhav has quit IRC12:06
*** bhagyashri_s has quit IRC12:06
openstackgerritTommyLike proposed openstack/cinder master: Support image signature verification  https://review.openstack.org/55416612:08
*** liverpooler has joined #openstack-cinder12:08
openstackgerritTommyLike proposed openstack/cinder master: Support availability-zone type  https://review.openstack.org/55224312:09
*** moshele has quit IRC12:10
*** eharney has joined #openstack-cinder12:12
*** bhagyashri_s has joined #openstack-cinder12:15
*** abhishekk has joined #openstack-cinder12:16
*** bhagyashris_ has quit IRC12:18
*** Shilpa has quit IRC12:18
*** alexchadin has joined #openstack-cinder12:19
*** dalgaaf has joined #openstack-cinder12:21
*** lkuchlan has joined #openstack-cinder12:21
*** belmoreira has joined #openstack-cinder12:24
*** moshele has joined #openstack-cinder12:24
*** bhagyashris has joined #openstack-cinder12:25
*** bkopilov has joined #openstack-cinder12:26
chhagarwgeguileo: Would like to discuss on the bug https://bugs.launchpad.net/cinder/+bug/177173712:27
openstackLaunchpad bug 1771737 in Cinder "cinder backup create fails with cannot allocate memory error while performing 1TB backup" [Undecided,Incomplete]12:27
*** bhagyashri_s has quit IRC12:28
geguileochhagarw: I see that eharney asked for some additional info12:28
chhagarwoh let me check12:29
chhagarwgeguileo: this was working before when tpool Proxy was introduced12:32
*** pchavva has joined #openstack-cinder12:33
*** sapd_ has quit IRC12:34
*** sapd_ has joined #openstack-cinder12:34
*** sapd_ has quit IRC12:35
chhagarwI added the memory usage details for the process12:35
*** sapd_ has joined #openstack-cinder12:35
*** sapd_ has quit IRC12:36
chhagarweharney, geguileo: I updated the memory info, it shows the cinder-backup consuming 47% of the memory12:36
*** sapd_ has joined #openstack-cinder12:36
chhagarwThis issue is seen only when 1TB export is done, things works fine till 100GB12:36
eharneyi see it's backing up to swift, what driver is being used for the volumes?12:37
geguileochhagarw: do you have concurrent backups?12:37
chhagarwno its a single 1TB backup12:38
geguileochhagarw: what's the config value of backup_swift_object_size?12:38
chhagarwvolumes use Storwize V700012:39
chhagarw "backup_swift_block_size"            | "32768"12:39
chhagarw "backup_swift_object_size"           | "52428800"12:40
*** armaan has quit IRC12:40
*** chhavi__ has joined #openstack-cinder12:40
*** armaan has joined #openstack-cinder12:41
*** bhagyashri_s has joined #openstack-cinder12:41
*** pooja_jadhav has quit IRC12:41
*** alexchadin has quit IRC12:42
*** bhagyashris has quit IRC12:42
*** pooja_jadhav has joined #openstack-cinder12:42
*** moshele has quit IRC12:43
*** dims has joined #openstack-cinder12:43
geguileochhagarw: it's weird, I have never seen the service do that... r:-??12:44
*** chhagarw has quit IRC12:44
chhavi__I am suspecting with 1TB if its running out of the tpool size, if the swift is responding slow12:46
chhavi__or tpool process is consuming too much memory thats it cannot spawn any more process for os utils to run like chown ?12:46
chhavi__this issue is not specific to swift, have seen the similar issue on POSIX 1TB backup as well12:47
chhavi__It used to work when eventlet was used12:48
*** moshele has joined #openstack-cinder12:48
*** bhagyashri_s has quit IRC12:48
*** dustins has joined #openstack-cinder12:48
*** pooja-jadhav has joined #openstack-cinder12:49
*** bhagyashri_s has joined #openstack-cinder12:49
*** wanghao_ has quit IRC12:52
geguileochhavi__: have you tried changing the tpool size using Cinder's parameter?12:52
*** pooja_jadhav has quit IRC12:52
*** wanghao has joined #openstack-cinder12:52
geguileochhavi__: Did you see in the logs progress on backing up chunks?12:52
*** belmoreira has quit IRC12:52
*** wanghao has quit IRC12:52
geguileochhavi__: or does it get stuck at some point?12:53
*** bhagyashris_ has joined #openstack-cinder12:53
*** wanghao has joined #openstack-cinder12:53
*** wanghao has quit IRC12:53
*** wanghao has joined #openstack-cinder12:54
*** wanghao has quit IRC12:54
*** wanghao has joined #openstack-cinder12:54
*** gyan_ has quit IRC12:54
*** wanghao has quit IRC12:55
*** wanghao has joined #openstack-cinder12:55
*** wanghao has quit IRC12:55
*** bhagyashri_s has quit IRC12:55
*** wanghao has joined #openstack-cinder12:56
*** jmlowe has quit IRC12:56
*** wanghao has quit IRC12:56
*** lseki has joined #openstack-cinder12:56
chhavi__I did not tried changing the tpool size, it did continued till 7hrs when we started hitting this issue.12:57
*** wanghao has joined #openstack-cinder12:58
*** lkuchlan has quit IRC12:59
geguileochhavi__: did the memory increase bit by bit?13:00
chhavi__don't have that details yet,13:00
*** lkuchlan has joined #openstack-cinder13:01
*** moshele has quit IRC13:04
*** links has quit IRC13:09
*** jiaopengju has quit IRC13:09
*** alexchadin has joined #openstack-cinder13:09
*** armaan has quit IRC13:15
*** armaan has joined #openstack-cinder13:15
*** lkuchlan has quit IRC13:17
*** lkuchlan has joined #openstack-cinder13:17
*** mchlumsky has joined #openstack-cinder13:17
*** erlon has quit IRC13:20
*** LutzB has quit IRC13:24
openstackgerritEric Harney proposed openstack/cinder master: Bump hacking to 1.1.0  https://review.openstack.org/56910813:29
*** belmoreira has joined #openstack-cinder13:31
*** pchavva has quit IRC13:32
*** erlon has joined #openstack-cinder13:33
*** abishop_ has joined #openstack-cinder13:36
*** abishop has quit IRC13:37
*** jmlowe has joined #openstack-cinder13:38
*** felipemonteiro has joined #openstack-cinder13:39
*** sdake_ is now known as sdake13:39
*** throwsb1 has joined #openstack-cinder13:41
*** crose has joined #openstack-cinder13:45
*** felipemonteiro has quit IRC13:46
*** ganso has joined #openstack-cinder13:47
*** zhaochao has quit IRC13:51
*** armaan has quit IRC13:56
*** armaan has joined #openstack-cinder13:57
*** jiaopengju has joined #openstack-cinder13:59
eric-youngI'm updating the documentation for a driver and am seeing errors with 'tox -e docs' in completely unrelated code. Is that still the way to generate/validate the docs?13:59
*** ianychoi_ has quit IRC13:59
*** belmorei_ has joined #openstack-cinder14:00
*** armaan has quit IRC14:00
*** armaan has joined #openstack-cinder14:00
*** abhishekk_ has joined #openstack-cinder14:01
*** belmoreira has quit IRC14:01
smcginniseric-young: We have an issue right now where out locally run docs job treats warnings as errors, but the gate does not.14:02
smcginniseric-young: So this has resulted in problem docs being merged due to folks not running local tests.14:02
smcginniseric-young: I would say for now, if you know it's not you, just push it up for review. It will more than likely pass zuul.14:02
smcginnisIn the mean time, I'm working on getting an enforcing job merged that we can use to prevent this from happening.14:03
eric-youngsmcginnis, thanks. That is what I am seeing. Any way to run local tests and generate the docs so I can be sure I changed the doc correctly?14:03
smcginniseric-young: The couple times this has hit me, I actually just fixed the other docs first so I could test my own.14:04
*** abhishekk has quit IRC14:04
smcginnishttps://review.openstack.org/#/c/564232/ in case anyone is interested.14:04
eric-youngsmcginnis, I knew you would say that :)14:04
smcginnis;)14:04
*** abhishekk_ is now known as abhishekk14:04
*** zengyingzhe has joined #openstack-cinder14:10
*** armaan has quit IRC14:15
*** armaan has joined #openstack-cinder14:15
*** johanssone has quit IRC14:16
*** ianychoi has joined #openstack-cinder14:18
*** hemna_ has joined #openstack-cinder14:24
*** moshele has joined #openstack-cinder14:27
*** KeithMnemonic has joined #openstack-cinder14:28
tommylikehueharney, I addressed your comments, could you take a look again :)https://review.openstack.org/#/c/554166/14:34
*** markstur has joined #openstack-cinder14:36
*** zengyingzhe has quit IRC14:44
*** abishop_ has quit IRC14:46
*** asbishop has joined #openstack-cinder14:47
*** abhishekk has quit IRC14:50
*** dklyle has joined #openstack-cinder14:54
*** jiaopengju has quit IRC14:59
*** jiaopengju has joined #openstack-cinder15:00
*** lvdombrkr89 has quit IRC15:09
*** eharney has quit IRC15:10
*** AlexeyAbashkin has quit IRC15:12
openstackgerritEric Harney proposed openstack/cinder master: Disable test_volume_backup_restore for lio-barbican job  https://review.openstack.org/56914315:14
*** eharney has joined #openstack-cinder15:15
*** AlexeyAbashkin has joined #openstack-cinder15:15
eharneyhttps://review.openstack.org/#/c/569143/ should fix lio-barbican failures for now15:15
eharneythis job still has issues getting Cinder/Swift configured correctly, and a new tempest test was added that is failing currently15:16
*** pchavva has joined #openstack-cinder15:17
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver and docs - change san_rest_port to san_api_port  https://review.openstack.org/56092815:19
*** KeithMnemonic has quit IRC15:22
*** cz2 has quit IRC15:24
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Incorrect service level reporting  https://review.openstack.org/56517915:29
*** jiaopengju has quit IRC15:30
openstackgerritEric Harney proposed openstack/cinder master: Bump hacking to 1.1.0  https://review.openstack.org/56910815:33
*** tpsilva has joined #openstack-cinder15:35
jungleboyjsmcginnis: https://review.openstack.org/56915215:35
jungleboyjos-brick 2.5.0 release.15:35
smcginnisjungleboyj: OK, will watch for zuul to come back on that.15:36
jungleboyjsmcginnis:   Thanks.15:37
eharneyneat, the primary new feature is a driver we don't test :)15:37
smcginnisJoy15:38
jungleboyj:-(15:38
*** eharney has quit IRC15:41
*** lvdombrkr has joined #openstack-cinder15:43
*** dklyle has quit IRC15:43
hemna_lol15:46
*** dklyle has joined #openstack-cinder15:48
openstackgerritEric Young proposed openstack/cinder master: ScaleIO: Rebranding of product to VxFlex OS  https://review.openstack.org/56915515:49
*** alexchadin has quit IRC15:50
*** armaan has quit IRC15:50
*** e0ne has quit IRC15:51
*** armaan has joined #openstack-cinder15:51
*** cz2 has joined #openstack-cinder15:52
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Deleting snapshot that is source of multiple volumes fails  https://review.openstack.org/56524115:55
SwansonI can't lie, I think "the primary new feature is a driver we don't test :)" would make a great openstack slogan.15:58
jungleboyj*SMH*15:59
*** LutzB has joined #openstack-cinder16:00
*** felipemonteiro has joined #openstack-cinder16:02
*** neha_alhat has joined #openstack-cinder16:07
*** felipemonteiro has quit IRC16:12
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - remove deprecated XML option  https://review.openstack.org/56551816:18
*** moshele has quit IRC16:20
neha_alhatjungleboyj: Hi16:26
jungleboyjHi.16:26
*** neha_alhat has quit IRC16:31
*** lkuchlan has quit IRC16:33
* jungleboyj sighs16:34
*** neha_alhat has joined #openstack-cinder16:36
neha_alhatjungleboyj: Monty Taylor has added split-logger functionality[1] in keystoneauth. Currently I am working on enabling this functionality in cinder.[1]: https://review.openstack.org/#/c/505764/16:36
jungleboyjneha_alhat:  Ok.16:36
Swansonjungleboyj, That could be your slogan for summit t-shirts! Think about it!16:37
*** tesseract has quit IRC16:38
neha_alhatjungleboyj: For this I am registering 'split_loggers' conf option in keystoneauth like this:  http://paste.openstack.org/show/721071/16:39
jungleboyjOk.16:40
neha_alhatjungleboyj: After registering this conf option in keystoneauth, need to set its value like split_loggers=True/False in cinder.conf under [nova] section if we are enabling this functionality for cinder to novaclient interaction.16:41
*** dklyle has quit IRC16:42
openstackgerritMerged openstack/cinder stable/queens: Huawei driver supports create_group_from_src  https://review.openstack.org/56819416:44
neha_alhatjungleboyj: At the time of cinder to novaclient interaction this split_loggers conf option value will get loaded from [nova] section from cinder.conf directly because of load_session_from_conf_options16:45
neha_alhatjungleboyj: https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L11216:46
neha_alhatjungleboyj: Using this approach I am able to enable this split-logger functionality for cinder-novaclient interaction.16:48
jungleboyjOk.16:48
neha_alhatjungleboyj: Same approach I am using for cinder-glanceclinet interaction16:50
neha_alhatjungleboyj: But for this(glancelient) session is loaded using 'load_from_options' method from keystoneauth like  https://github.com/openstack/cinder/blob/master/cinder/image/glance.py#L10416:52
jungleboyjOk.16:53
neha_alhatjungleboyj: I need to register session conf options under [glance] if I use 'load_session_from_conf_options' instead of 'load_from_options'. You can refer what changes I need to make in glanceclient: http://paste.openstack.org/show/721095/16:55
jungleboyjneha_alhat:  Ok ...16:56
neha_alhatjungleboyj: Impact of this approch is  need to register the session conf options from keystoneauth under [glance] section which earlier was under [default] section.16:58
neha_alhatjungleboyj: so need your opinion regarding this16:58
jungleboyjneha_alhat:  Hmmm.  Ok.17:00
neha_alhatjungleboyj: Pros of using this approach is: As we are setting conf option in keystoneauth it will directly load value of ‘split_loggers’ from conf file. So no need to pass explicitly ‘split_loggers’ value at the time of loading keystoneauth session for each client intearction17:01
*** belmorei_ has quit IRC17:02
jungleboyjneha_alhat:  Little confused.  So, the keystonauth options you are registering only impacting glance?17:03
jungleboyjAs in they were in the default section but should have been in the glance section?17:03
*** AlexeyAbashkin has quit IRC17:04
*** neha_alhat has quit IRC17:05
*** neha_alhat has joined #openstack-cinder17:06
neha_alhatjungleboyj: For cinder-keystoneclient interaction also need to make same changes, it will also impact on keystone also17:06
jungleboyjneha_alhat:  So, it seems to me that Glance should function the same way as novaclient.17:08
neha_alhatjungleboyj: so what approach I am following is it ok?17:10
jungleboyjThough moving keystoneauth seems a little suspicious.  Seems like Glance should be fixed.17:10
jungleboyjWhat is the option that would move from 'default' to 'glance' section?17:11
neha_alhatjungleboyj: keystoneauth conf options: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/session.py#L16117:14
jungleboyjneha_alhat:  Ok, that is what I thought you were saying.17:15
jungleboyjneha_alhat:  So those would all now be under the [glance] section of the file?17:15
neha_alhatjungleboyj: yes17:16
jungleboyjneha_alhat:  Yeah, that isn't ok.  Because those aren't specific to glance ... right.17:16
jungleboyjThey were in the default section because it impacts interaction with keystone, novaclient, etc.17:16
neha_alhatjungleboj: like done in novaclient. https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L5517:18
*** chhavi__ has quit IRC17:18
jungleboyjneha_alhat:  Yes, where it uses them from keystoneauth1 instead of making another copy of the options.17:20
jungleboyjneha_alhat:  I think glance needs to be fixed to use the options in the same way.17:21
*** dalgaaf has quit IRC17:22
*** lvdombrkr has quit IRC17:23
neha_alhatjungleboyj: ok, and what about cinder-keystoneclient intearction17:23
jungleboyjWhat about that?17:24
neha_alhatsame is case like glance17:24
jungleboyjYes, I think it should act the same way.17:24
jungleboyjSo, since keystoneauth is a library utilized by multiple parts of the code it should be configured in default.17:25
jungleboyjThe split_logs functionality is on a per-client basis and then should be controlled in the config file section for that client.17:26
neha_alhatjungleboyj: so you mean to say the value should be under default section in conf file?17:28
jungleboyjneha_alhat:  No, that is where the keystoneauth options should be ...17:29
*** smcginnis_ has joined #openstack-cinder17:29
jungleboyjthe split_loggers=True/False  will impact each client independently.  Correct?17:30
neha_alhatyes17:30
jungleboyjSo, I can split loggers for Glance but not for Nova ?17:30
neha_alhatjungleboyj: yes17:30
*** imacdonn has quit IRC17:31
*** imacdonn has joined #openstack-cinder17:31
jungleboyjOk.  so, then that should go under the [glance] and [nova] sections.  Should be the same for all client interactions.17:31
neha_alhatjungleboyj: Yes17:33
*** neha_alhat_ has joined #openstack-cinder17:34
jungleboyjneha_alhat:  So, I think that is your answer.  If that won't work for glance and nova.keystoneclient that needs to be fixed.  The functionality should be consistent.17:34
*** dalgaaf has joined #openstack-cinder17:34
*** psachin has quit IRC17:37
neha_alhatjungleboyj: Yes, need to fix for glanceclient nad keystoneclient. Able to add this functionality for novaclinet. Also I have checked for nova to its internal clients interaction, it is alos working fine.17:37
jungleboyjOk.  So does that answer your question neha_alhat17:38
neha_alhatjungleboyj: Yes, Thanks for your inputs17:39
jungleboyjneha_alhat:  Yep, thanks for explaining it.17:39
neha_alhatjungleboyj: Have a good day..!!17:40
jungleboyjneha_alhat:  Thanks.  You too!17:45
*** tpsilva has quit IRC17:45
*** dhinesh has joined #openstack-cinder17:45
*** dalgaaf has quit IRC17:48
*** moshele has joined #openstack-cinder17:53
*** felipemonteiro has joined #openstack-cinder17:54
*** links has joined #openstack-cinder17:59
*** dalgaaf has joined #openstack-cinder18:00
*** neha_alhat has quit IRC18:07
smcginnis_Now that the ocata sync patches are out of the way, it sure would be swell if these got through too: https://review.openstack.org/#/q/topic:config-table+status:open18:18
smcginnis_Actually looking at the nicks in this channel, I wonder how many forgotten IRC bouncers are running out there. :)18:20
*** smcginnis_ has quit IRC18:23
*** eharney has joined #openstack-cinder18:27
jungleboyjsmcginnis:  Poor sad bouncers.18:34
*** links has quit IRC18:35
jungleboyjsmcginnis:  Would it be naughty to just merge those?18:37
smcginnis[m]I think that's fine for doc changes.18:41
smcginnis[m]I've actually been meaning to propose some review guidelines about that.18:41
jungleboyjsmcginnis[m]:  Ok.  Will do.18:41
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation  https://review.openstack.org/43334118:45
eharneyhttps://review.openstack.org/#/c/567890/ could use a look18:45
*** diablo_rojo has joined #openstack-cinder18:46
*** markstur has quit IRC18:49
*** dklyle has joined #openstack-cinder18:56
*** dalgaaf has quit IRC18:58
*** dalgaaf has joined #openstack-cinder18:58
*** eharney has quit IRC19:00
*** crose has quit IRC19:02
*** felipemonteiro__ has joined #openstack-cinder19:04
*** felipemonteiro has quit IRC19:08
*** moshele has quit IRC19:19
*** moshele has joined #openstack-cinder19:21
*** eharney has joined #openstack-cinder19:25
*** jmlowe has quit IRC19:27
*** dustins has quit IRC19:34
*** felipemonteiro__ has quit IRC19:39
*** felipemonteiro has joined #openstack-cinder19:39
*** crose has joined #openstack-cinder19:47
*** moshele has quit IRC19:50
*** jmlowe has joined #openstack-cinder19:57
openstackgerritMerged openstack/cinder master: Disable test_volume_backup_restore for lio-barbican job  https://review.openstack.org/56914320:07
*** gkadam has quit IRC20:08
*** jmlowe has quit IRC20:22
*** markstur has joined #openstack-cinder20:25
*** moshele has joined #openstack-cinder20:33
openstackgerritMerged openstack/cinder master: Make config-table display list defaults in list format  https://review.openstack.org/56578820:37
*** dalgaaf has quit IRC20:41
*** pchavva has quit IRC20:43
*** dalgaaf has joined #openstack-cinder20:46
*** luizbag has quit IRC20:46
openstackgerritLucio Seki proposed openstack/cinder stable/pike: NetApp ONTAP: Set new sub-lun clone limit for ONTAP driver  https://review.openstack.org/56925520:46
*** throwsb1 has quit IRC20:53
*** crose has quit IRC21:07
*** dustins has joined #openstack-cinder21:11
openstackgerritMerged openstack/cinder master: Remove Tintri static config option table  https://review.openstack.org/56571221:21
openstackgerritMerged openstack/cinder master: Remove zonemanager static config option tables  https://review.openstack.org/56599021:21
openstackgerritMerged openstack/cinder master: Remove Synology static config option table  https://review.openstack.org/56571121:21
openstackgerritMerged openstack/cinder master: Remove SolidFire static config option table  https://review.openstack.org/56571021:21
openstackgerritMerged openstack/cinder master: Remove ZFSSA static config option tables  https://review.openstack.org/56570821:21
openstackgerritMerged openstack/cinder master: Disallow multiattach for encrypted volumes  https://review.openstack.org/56789021:23
smcginnisYay, down to one page.21:24
*** felipemonteiro has quit IRC21:28
*** felipemonteiro has joined #openstack-cinder21:28
*** felipemonteiro has quit IRC21:28
*** felipemonteiro has joined #openstack-cinder21:28
*** asbishop has quit IRC21:29
*** hemna_ has quit IRC21:34
openstackgerritEric Harney proposed openstack/cinder stable/queens: Disallow multiattach for encrypted volumes  https://review.openstack.org/56926921:35
openstackgerritLucio Seki proposed openstack/cinder stable/queens: NetApp ONTAP: Set new sub-lun clone limit for ONTAP driver  https://review.openstack.org/56927021:39
*** KeithMnemonic has joined #openstack-cinder21:39
*** edmondsw has quit IRC21:46
*** eharney has quit IRC21:57
openstackgerritMerged openstack/cinder master: Fix mocks for ZFS Storage Appliance unit tests  https://review.openstack.org/53733222:01
*** lseki has quit IRC22:04
*** rcernin has joined #openstack-cinder22:04
*** hemna_ has joined #openstack-cinder22:05
*** dustins has quit IRC22:06
*** felipemonteiro has quit IRC22:20
*** hemna_ has quit IRC22:24
*** hemna_ has joined #openstack-cinder22:24
*** ganso has quit IRC22:29
*** mriedem is now known as mriedem_afk22:36
*** threestrands has joined #openstack-cinder22:37
*** mriedem_afk has quit IRC22:38
*** cloudnull has quit IRC22:48
*** cloudnull has joined #openstack-cinder22:49
*** moshele has quit IRC22:55
*** moshele has joined #openstack-cinder22:59
*** edmondsw has joined #openstack-cinder23:03
*** edmondsw has quit IRC23:08
*** moshele has quit IRC23:23
*** edmondsw has joined #openstack-cinder23:46

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