Monday, 2017-09-25

*** yangyape_ has quit IRC00:08
*** yangyapeng has joined #openstack-cinder00:09
*** wanghao has joined #openstack-cinder00:12
*** wanghao_ has joined #openstack-cinder00:13
*** yangyapeng has quit IRC00:13
*** wanghao has quit IRC00:16
*** catintheroof has joined #openstack-cinder00:19
*** Shunli has joined #openstack-cinder00:33
*** Apoorva has joined #openstack-cinder00:41
*** markvoelker has joined #openstack-cinder00:42
*** zhurong has joined #openstack-cinder01:00
*** edmondsw has joined #openstack-cinder01:06
openstackgerritJeremy Zhang proposed openstack/cinder master: Add backups v3 views  https://review.openstack.org/50293901:07
*** edmondsw has quit IRC01:10
*** yangyapeng has joined #openstack-cinder01:12
*** markvoelker has quit IRC01:15
*** Apoorva has quit IRC01:15
*** yangyapeng has quit IRC01:16
openstackgerritTommyLike proposed openstack/cinder master: Support count info in List&Detail APIs  https://review.openstack.org/50510101:24
*** catintheroof has quit IRC01:37
*** catintheroof has joined #openstack-cinder01:38
*** sapd has joined #openstack-cinder01:49
*** lhx_ has joined #openstack-cinder01:55
*** catintheroof has quit IRC01:57
*** zhaochao has joined #openstack-cinder02:10
*** yangyapeng has joined #openstack-cinder02:22
*** yangyapeng has quit IRC02:25
*** yangyapeng has joined #openstack-cinder02:25
*** itlinux has joined #openstack-cinder02:37
*** david-lyle has joined #openstack-cinder02:38
openstackgerritVivek Soni proposed openstack/cinder master: 3PAR: Cinder volume revert to snapshot support  https://review.openstack.org/50607202:52
openstackgerritTommyLike proposed openstack/cinder master: [Policy-in-Code] Add support for attachment resource  https://review.openstack.org/50697602:53
openstackgerritTommyLike proposed openstack/cinder master: Support count info in List&Detail APIs  https://review.openstack.org/50510102:54
*** edmondsw has joined #openstack-cinder02:54
openstackgerritTommyLike proposed openstack/cinder master: [Policy-in-Code] Add support for attachment resource  https://review.openstack.org/50697602:55
*** yangyapeng has quit IRC02:55
*** yangyapeng has joined #openstack-cinder02:56
*** edmondsw has quit IRC02:58
*** savvas_ has joined #openstack-cinder03:02
*** savvas has quit IRC03:04
*** zhurong has quit IRC03:08
jamielennoxhey, i'm just doing some policy work and cinder is using is_admin_project and so is being my test bed - unfortunately i don't know enough about cinder03:10
jamielennoxcan someone tell me - does a group get owned by a project or is that kind of a global thing03:10
*** zhurong has joined #openstack-cinder03:11
*** markvoelker has joined #openstack-cinder03:12
*** itlinux has quit IRC03:30
*** wanghao has joined #openstack-cinder03:34
*** Shunli has quit IRC03:35
*** wanghao_ has quit IRC03:37
*** gkadam has joined #openstack-cinder03:38
*** gkadam has quit IRC03:39
*** gkadam has joined #openstack-cinder03:39
*** markvoelker has quit IRC03:47
openstackgerritTommyLike proposed openstack/cinder master: Support count info in List&Detail APIs  https://review.openstack.org/50510103:57
*** chhavi has joined #openstack-cinder04:05
*** lkuchlan has joined #openstack-cinder04:13
*** Dinesh_Bhor has joined #openstack-cinder04:14
*** crushil has joined #openstack-cinder04:14
*** abhitechie has joined #openstack-cinder04:19
*** links has joined #openstack-cinder04:25
*** psachin has joined #openstack-cinder04:31
*** markstur_ has joined #openstack-cinder04:31
*** stakeda has joined #openstack-cinder04:33
*** crushil has quit IRC04:42
*** markvoelker has joined #openstack-cinder04:44
*** ircuser-1 has joined #openstack-cinder05:08
*** markvoelker has quit IRC05:17
*** zhurong has quit IRC05:21
*** ChubYann has quit IRC05:30
*** Shunli has joined #openstack-cinder05:33
*** lkuchlan has quit IRC05:43
*** markstur_ has quit IRC05:45
*** zsli_ has joined #openstack-cinder05:49
*** Shunli has quit IRC05:52
*** wanghao has quit IRC05:52
*** wanghao has joined #openstack-cinder05:55
*** markstur has joined #openstack-cinder06:07
*** wanghao has quit IRC06:08
*** wanghao_ has joined #openstack-cinder06:08
*** zsli_ has quit IRC06:09
*** markstur has quit IRC06:12
*** markvoelker has joined #openstack-cinder06:14
*** zhurong has joined #openstack-cinder06:15
*** kwathore has quit IRC06:24
*** kwathore has joined #openstack-cinder06:25
*** vivsoni_ has quit IRC06:25
*** vivsoni has joined #openstack-cinder06:27
openstackgerrityatin proposed openstack/cinder master: Remove deprecated keymgr code  https://review.openstack.org/46825206:29
*** josecastroleon has joined #openstack-cinder06:33
*** rcernin has joined #openstack-cinder06:41
*** markvoelker has quit IRC06:48
*** lkuchlan has joined #openstack-cinder06:57
*** pcaruana has joined #openstack-cinder07:06
*** zhurong has quit IRC07:14
*** tesseract has joined #openstack-cinder07:23
*** alexchadin has joined #openstack-cinder07:24
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource  https://review.openstack.org/50697607:27
*** chhavi has quit IRC07:28
*** zhurong has joined #openstack-cinder07:32
*** pgadiya has joined #openstack-cinder07:36
*** chenxu1990 has joined #openstack-cinder07:39
*** alexchadin has quit IRC07:39
*** alexchadin has joined #openstack-cinder07:40
*** markvoelker has joined #openstack-cinder07:45
openstackgerritHanxi Liu proposed openstack/cinder master: RBD: Create incremental backup for volume in-use failed  https://review.openstack.org/43218407:53
openstackgerritBoris Pavlovic proposed openstack/cinder master: Switch Rally Task To fromat V2  https://review.openstack.org/50701007:53
chenxu1990hi guys,I test the function of creating 、attatching、detaching volumes ,but when I ran tests concurrently ,detaching will be failed of tired waiting. will attaching volume block detaching on the same server?07:55
*** lhx_ has quit IRC07:55
*** gkadam is now known as gkadam-afk07:55
*** lhx_ has joined #openstack-cinder07:56
*** pgadiya has quit IRC08:04
*** Shunli has joined #openstack-cinder08:06
*** markvoelker has quit IRC08:18
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for backup resource  https://review.openstack.org/50701508:18
*** chenxu1990 has quit IRC08:24
*** lhx_ has quit IRC08:25
*** e0ne has joined #openstack-cinder08:25
*** chenxu1990 has joined #openstack-cinder08:26
*** lhx_ has joined #openstack-cinder08:26
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource  https://review.openstack.org/50697608:27
*** mikal_ has joined #openstack-cinder08:30
*** zhurong has quit IRC08:32
*** mikal has quit IRC08:32
openstackgerritShan Guo proposed openstack/cinder master: Remove SCREEN_LOGDIR from devstack install setting  https://review.openstack.org/50701708:34
*** gkadam-afk is now known as gkadam08:41
*** mvk has joined #openstack-cinder08:44
*** alexchadin has quit IRC08:46
*** alexchadin has joined #openstack-cinder08:47
*** alexchadin has quit IRC08:47
*** finucannot is now known as stephenfin08:47
*** alexchadin has joined #openstack-cinder08:48
*** alexchadin has quit IRC08:48
*** alexchadin has joined #openstack-cinder08:49
*** wanghao_ has quit IRC08:50
kwathore@Team: Requesting upo to review below patch:08:50
kwathorehttps://review.openstack.org/#/c/506102/08:50
*** Shunli has quit IRC08:51
kwathoreyou*08:51
*** zhurong has joined #openstack-cinder08:51
*** alexchadin has quit IRC08:52
*** wanghao has joined #openstack-cinder08:54
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for backup resource  https://review.openstack.org/50701508:55
*** kwathore has quit IRC08:55
*** ghgjgjgjgj has joined #openstack-cinder08:55
*** armax has joined #openstack-cinder08:56
*** chenxu1990 has quit IRC08:59
*** markstur has joined #openstack-cinder09:02
*** markstur has quit IRC09:07
*** lkuchlan has quit IRC09:09
*** markvoelker has joined #openstack-cinder09:15
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource  https://review.openstack.org/50697609:18
*** armax has quit IRC09:19
*** lpetrut has joined #openstack-cinder09:20
*** mvk has quit IRC09:28
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for backup resource  https://review.openstack.org/50701509:28
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for attachment resource  https://review.openstack.org/50697609:31
*** armax has joined #openstack-cinder09:32
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-cinder09:34
*** wanghao has quit IRC09:34
*** wanghao has joined #openstack-cinder09:35
*** wanghao has quit IRC09:35
*** wanghao_ has joined #openstack-cinder09:35
*** wanghao_ has quit IRC09:36
*** wanghao has joined #openstack-cinder09:36
openstackgerritSilvan Kaiser proposed openstack/cinder master: Optimizes volume creation in the Quobyte Driver  https://review.openstack.org/50078209:36
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-cinder09:37
*** dalgaaf has quit IRC09:38
*** markstur has joined #openstack-cinder09:39
*** dalgaaf has joined #openstack-cinder09:40
*** armax has quit IRC09:43
*** ghgjgjgjgj has quit IRC09:43
*** chhavi has joined #openstack-cinder09:43
*** markstur has quit IRC09:44
*** belmoreira has joined #openstack-cinder09:47
*** markvoelker has quit IRC09:48
*** mvk has joined #openstack-cinder09:57
*** kwathore has joined #openstack-cinder10:00
*** nicolasbock has joined #openstack-cinder10:02
openstackgerritSilvan Kaiser proposed openstack/cinder master: Adds a Cache for Volumes Created from Snapshots with Quobyte  https://review.openstack.org/50297410:04
*** belmoreira has quit IRC10:06
*** nicolasbock has quit IRC10:12
*** stakeda has quit IRC10:17
*** armax has joined #openstack-cinder10:19
openstackgerritSilvan Kaiser proposed openstack/cinder master: [WIP] Adds Overlay Volumes Created from Snapshots to Quobyte  https://review.openstack.org/50705010:20
*** zhurong has quit IRC10:21
*** nicolasbock has joined #openstack-cinder10:24
*** abhitechie has quit IRC10:24
*** abhitechie has joined #openstack-cinder10:24
*** zengyingzhe_ has quit IRC10:24
*** zhurong has joined #openstack-cinder10:25
*** zengyingzhe_ has joined #openstack-cinder10:25
*** lkuchlan has joined #openstack-cinder10:26
*** abhitechie has quit IRC10:30
*** abhitechie has joined #openstack-cinder10:30
*** gcb has joined #openstack-cinder10:33
*** alexchadin has joined #openstack-cinder10:41
*** markvoelker has joined #openstack-cinder10:45
*** gcb has quit IRC10:53
*** dalgaaf has quit IRC10:54
*** dalgaaf has joined #openstack-cinder10:57
*** armax has quit IRC10:57
*** AlexeyAbashkin has joined #openstack-cinder10:58
*** sdague has joined #openstack-cinder10:59
*** AlexeyAbashkin has quit IRC11:00
*** AlexeyAbashkin has joined #openstack-cinder11:02
*** kwathore has quit IRC11:02
*** armax has joined #openstack-cinder11:02
*** zhaochao has quit IRC11:02
*** armax has quit IRC11:03
*** armax has joined #openstack-cinder11:03
*** armax has quit IRC11:04
*** bhagyashri_s has joined #openstack-cinder11:04
*** ps_jadhav has joined #openstack-cinder11:04
*** yangyapeng has quit IRC11:05
*** bhagyashris has quit IRC11:07
*** pooja_jadhav has quit IRC11:07
*** zhaochao has joined #openstack-cinder11:13
*** markstur has joined #openstack-cinder11:15
*** markvoelker has quit IRC11:18
*** markstur has quit IRC11:20
*** lhx_ has quit IRC11:30
*** lhx_ has joined #openstack-cinder11:30
*** abhitechie has quit IRC11:30
*** abhitechie has joined #openstack-cinder11:31
*** belmoreira has joined #openstack-cinder11:36
*** abhitechie has quit IRC11:41
*** ps_jadhav is now known as pooja_jadhav11:45
*** markstur has joined #openstack-cinder11:51
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Remove workload for next gen arrays  https://review.openstack.org/50707011:52
*** markstur has quit IRC11:56
*** lkuchlan has quit IRC12:00
*** zhaochao has quit IRC12:02
*** zhaochao has joined #openstack-cinder12:02
*** alexchadin has quit IRC12:07
*** alexchadin has joined #openstack-cinder12:07
*** lhx__ has joined #openstack-cinder12:07
*** lhx_ has quit IRC12:09
*** edmondsw has joined #openstack-cinder12:09
*** jmlowe has quit IRC12:10
*** MVenesio has joined #openstack-cinder12:11
*** bhagyashri_s is now known as bhagyashris12:11
kaisersHi! Is there a known general Jenkins issue currently? I've rebased two changes that were testing ok and now Jenkins fails on two different tests on these changes (http://logs.openstack.org/50/507050/1/check/gate-cinder-python35/abc534f/console.html and http://logs.openstack.org/82/500782/8/check/gate-tempest-dsvm-py35-ubuntu-xenial/644207f/console.html). Note the fails are not related to the changes.12:12
*** abhitechie has joined #openstack-cinder12:13
*** markvoelker has joined #openstack-cinder12:16
*** yangyapeng has joined #openstack-cinder12:20
openstackgerritSilvan Kaiser proposed openstack/cinder master: [WIP] Adds Overlay Volumes Created from Snapshots to Quobyte  https://review.openstack.org/50705012:20
*** MasterofJOKers_ has joined #openstack-cinder12:20
*** MasterofJOKers has quit IRC12:22
*** MasterofJOKers_ is now known as MasterofJOKers12:22
*** alexchadin has quit IRC12:22
*** markvoelker has quit IRC12:23
*** markvoelker has joined #openstack-cinder12:24
*** belmoreira has quit IRC12:24
*** lkuchlan has joined #openstack-cinder12:24
*** abhitechie has quit IRC12:27
*** belmoreira has joined #openstack-cinder12:28
*** abishop_ has joined #openstack-cinder12:29
*** gkadam has quit IRC12:36
*** erlon has joined #openstack-cinder12:38
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Remove workload for next gen arrays  https://review.openstack.org/50707012:49
*** zhurong has quit IRC12:50
*** mchlumsky has joined #openstack-cinder12:51
*** belmoreira has quit IRC12:51
*** belmoreira has joined #openstack-cinder12:53
*** lhx__ has quit IRC12:55
*** lhx_ has joined #openstack-cinder12:55
*** lhx_ has quit IRC12:56
*** zhaochao has quit IRC12:58
*** mriedem has joined #openstack-cinder13:00
*** zhaochao has joined #openstack-cinder13:00
*** liverpooler has joined #openstack-cinder13:02
*** jistr is now known as jistr|mtg13:03
*** mchlumsky has quit IRC13:07
*** jmlowe has joined #openstack-cinder13:07
*** markstur has joined #openstack-cinder13:09
*** mchlumsky has joined #openstack-cinder13:10
*** gcb has joined #openstack-cinder13:10
*** markstur has quit IRC13:14
*** dustins has joined #openstack-cinder13:16
*** gnufied has joined #openstack-cinder13:17
*** links has quit IRC13:23
*** josecastroleon has quit IRC13:25
*** gouthamr has joined #openstack-cinder13:27
*** josecastroleon has joined #openstack-cinder13:27
*** lbragstad has joined #openstack-cinder13:28
*** gouthamr has quit IRC13:29
*** jistr|mtg is now known as jistr13:29
*** gouthamr has joined #openstack-cinder13:30
smcginniskaisers: Looks like maybe a library update breaking things.13:31
kaiserssmcginnis: yeah, just saw that one of the rechecks passed about 20mins ago...13:34
kaisersMaybe it's already over13:34
smcginniskaisers: It appears to be intermittent. I saw it at least one other place before.13:35
*** belmoreira has quit IRC13:37
*** dave-mccowan has joined #openstack-cinder13:41
*** markstur has joined #openstack-cinder13:45
*** chhavi has quit IRC13:46
*** tpsilva has joined #openstack-cinder13:46
*** dpawlik has quit IRC13:49
*** markstur has quit IRC13:50
*** danpawlik has joined #openstack-cinder13:50
*** bswartz has quit IRC13:50
*** amoralej has joined #openstack-cinder13:53
amoralejgeguileo, could you take a look into https://review.openstack.org/#/c/468252 when you have a chance?13:54
amoralejit's fixing a problem we are hitting when deploying master with tripleo13:54
amoralejin fact, with puppet-cinder13:55
amoralejnot only with tripleo13:55
geguileoamoralej: I'll add it to my list, right now I'm swamped :-(13:56
amoralejok, thanks13:56
openstackgerritEric Young proposed openstack/cinder master: ScaleIO Driver - adding cache and refactoring tests.  https://review.openstack.org/50247313:57
openstackgerritcaoyuan proposed openstack/cinder master: cleanup test-requirements  https://review.openstack.org/50711113:57
*** chlong has joined #openstack-cinder13:57
openstackgerritcaoyuan proposed openstack/python-cinderclient master: cleanup test-requirements  https://review.openstack.org/50711213:57
*** kwathore has joined #openstack-cinder14:00
*** belmoreira has joined #openstack-cinder14:00
*** KeithMnemonic has joined #openstack-cinder14:00
*** MasterofJOKers has quit IRC14:01
*** superdan is now known as dansmith14:02
*** MasterofJOKers has joined #openstack-cinder14:03
tommylikehucould anyone take a look at the basic default policy in code patch? This version is much smaller and easier to review :) https://review.openstack.org/#/c/506976/14:04
openstackgerritGyorgy Szombathelyi proposed openstack/cinder master: Remove deprecated nova config options  https://review.openstack.org/50359614:08
*** armax has joined #openstack-cinder14:18
*** josecastroleon has quit IRC14:19
*** abhitechie has joined #openstack-cinder14:20
*** crushil has joined #openstack-cinder14:21
*** abhitechie has quit IRC14:28
*** yongwc has joined #openstack-cinder14:34
*** belmoreira has quit IRC14:38
yongwchi team, about the commit https://review.openstack.org/#/c/502455/, which is related to the inspur storage driver, is there any other rules I need to follow for commits like this?14:39
*** belmoreira has joined #openstack-cinder14:40
smcginnisyongwc: Something is up with your CI. It's showing up in the comment (where it shouldn't) and it isn't showing up in the CI results (where it should).14:42
smcginnisyongwc: Links in the comments are not right. They point to an internal IP address.14:43
*** takedakn has joined #openstack-cinder14:46
*** armax has quit IRC14:47
*** markstur has joined #openstack-cinder14:54
yongwcsmcginnis: then I just need do something like a recheck and change the IP address to public will let the review goes on, right? our CI is not stable, one reason is the limit of our knowledge, others is about the network, clone so many repo is a challenge, I am shamed for that.14:54
*** belmoreira has quit IRC14:54
openstackgerritChris Yang proposed openstack/cinder master: Add support for enhanced features to the QNAP Cinder driver  https://review.openstack.org/47018814:55
*** gcb has quit IRC14:55
smcginnisyongwc: Looks like it's getting there. Just takes some work to get things running.14:57
*** felipemonteiro has joined #openstack-cinder14:58
*** dustins has quit IRC15:01
*** felipemonteiro_ has joined #openstack-cinder15:02
yongwcsmcginnis: thanks, I will make it better.15:03
*** lkuchlan has quit IRC15:04
*** felipemonteiro has quit IRC15:05
openstackgerritMerged openstack/cinder master: Remove SCREEN_LOGDIR from devstack install setting  https://review.openstack.org/50701715:06
*** shaner has quit IRC15:07
*** shaner has joined #openstack-cinder15:07
*** wangyong has joined #openstack-cinder15:08
SwansonWhat is still open for bug fixes? Pike?15:09
*** links has joined #openstack-cinder15:09
*** yongwc has quit IRC15:10
*** crose has joined #openstack-cinder15:11
smcginnisSwanson: Yep. Ocata for security and critical bug fixes.15:14
*** andreykurilin has joined #openstack-cinder15:14
andreykurilinhi folks! Can someone look at trace from our gates - http://logs.openstack.org/18/505918/3/check/gate-rally-dsvm-rally-cinder/160bba7/logs/screen-c-vol.txt.gz#_Sep_25_12_33_28_80806015:15
andreykurilin?15:15
andreykurilinit looks strange15:15
andreykurilin@e0ne ^15:16
*** links has quit IRC15:17
*** Julien-z_ has joined #openstack-cinder15:17
*** Julien-z_ has quit IRC15:17
jgriffithandreykurilin https://review.openstack.org/#/c/505601/ may have something to do with that15:19
e0neandreykurilin: we've got filed bug for it for a while but I don't know how to reproduce it to find a fix15:19
jgriffithe0ne oh, you already have a bug for activation fail on a copy snapshot-->to-volume?15:20
e0neandreykurilin, jgriffith: IMO, it's this bug https://bugs.launchpad.net/cinder/+bug/164211115:20
openstackLaunchpad bug 1642111 in Cinder "create lvm volume from snapshot fails with "device-mapper: reload ioctl on (252:4) failed: Invalid argument"" [Critical,Confirmed]15:20
andreykuriline0ne: it happens sometimes in our gates... can we add some monkey-patching there for more debug-messages ? so next time when it happens, you will have more details15:20
e0nejgriffith: no, I'm talking about another one15:20
jgriffithe0ne right... but that *is* the same bug :)15:21
e0nesorry, guys, I have to disconnect now. bb in ~4 hours15:21
e0nejgriffith: :)15:21
jgriffithe0ne what I mean is the fix I referenced is for that bug... so it doesn't appear to *fix* it :)15:21
*** crose has quit IRC15:22
andreykuriline0ne, jgriffith: the currect error in gates had happend while snapshot->volume15:22
*** e0ne has quit IRC15:23
*** crose has joined #openstack-cinder15:24
andreykurilinjgriffith: today it had happened at least 3 times15:24
jgriffithandreykurilin right... and I'm not sure the activation in that patch which is *after* the lv call to copy the snap is correct.  I raised a question on that15:24
jgriffithandreykurilin and that patch merged to master on the 23'rd so I'm wondering if it may be resulting in this issue being more prevelant15:25
jgriffithsince it was a quiet week-end and all :)15:25
andreykurilinif this issue happens more often, it is a good thing. since it can simplify debugging :)15:26
smcginnisIt's also more prevalent since we just switched back to thin LVM with devstack.15:27
smcginnisNeed to fix the issue, not just hide it by doing thick all the time.15:27
jgriffithsmcginnis +115:27
*** links has joined #openstack-cinder15:30
*** shaner has quit IRC15:30
andreykurilinIf you need any help, feel free to ping me. I'll try to do my best15:30
*** shaner has joined #openstack-cinder15:30
andreykurilinFunny fact: your rally-job is broken now due to a small typo in rally, but the fix fails to be merged due to bug in your code:)15:31
*** takedakn has quit IRC15:32
*** pcaruana has quit IRC15:33
*** pcaruana has joined #openstack-cinder15:34
*** ociuhandu has joined #openstack-cinder15:35
*** jmlowe has quit IRC15:37
*** ntpttr_laptop has joined #openstack-cinder15:39
*** ociuhandu has quit IRC15:40
*** ntpttr_laptop has quit IRC15:41
*** ntpttr_laptop has joined #openstack-cinder15:41
*** ntpttr_laptop has quit IRC15:41
*** jmlowe has joined #openstack-cinder15:44
*** crose has quit IRC15:44
*** wangyong has quit IRC15:47
*** crose has joined #openstack-cinder15:47
*** rcernin has quit IRC15:48
*** ociuhandu has joined #openstack-cinder15:58
*** sbezverk has quit IRC16:00
*** crose has quit IRC16:01
*** bhagyashris has quit IRC16:03
*** pooja_jadhav has quit IRC16:03
*** psachin has quit IRC16:03
*** Dinesh_Bhor has quit IRC16:07
*** Apoorva has joined #openstack-cinder16:09
*** Apoorva has quit IRC16:09
*** Apoorva has joined #openstack-cinder16:10
*** links has quit IRC16:10
*** itlinux has joined #openstack-cinder16:12
*** pcaruana has quit IRC16:15
*** itlinux has quit IRC16:15
*** lpetrut has quit IRC16:16
ApoorvaTintri CI started failing around Sept 19, with the following error related to ImageCopyFailure http://paste.openstack.org/show/621856/, does anyone know about the breaking change or how to fix it?16:16
openstackgerritMerged openstack/cinder-specs master: Update url of the cinder-specs's document  https://review.openstack.org/50109816:17
Apoorvasmcginnis ^^16:17
ApoorvaI also saw error in initialize_connection http://paste.openstack.org/show/621857/, tintri.py uses initialize_connection() from nfs.py16:20
*** ociuhandu has quit IRC16:20
smcginnisApoorva: Both of those look like they are secondary errors. There should be something before that indicating what led to those.16:22
Apoorvasmcginnis ok thanks for the pointer. let me check16:22
*** gkadam has joined #openstack-cinder16:24
openstackgerritEvgeny Antyshev proposed openstack/cinder master: Add ploop to parallels naming conversion  https://review.openstack.org/50719316:25
smcginnissmh16:25
Apoorvasmcginnis maybe this > http://paste.openstack.org/show/621858/?16:27
*** itlinux has joined #openstack-cinder16:27
smcginnisApoorva: I think before then. Looks like it failed to mount it at some point. Then that paste shows it trying to run qemu-img against where it should be mounted.16:28
smcginnisApoorva: Either that our the mount point is OK but it can't access the file for some reason.16:28
*** chhavi has joined #openstack-cinder16:29
*** raunak has joined #openstack-cinder16:39
*** chhavi has quit IRC16:42
*** gkadam has quit IRC16:42
*** chhavi has joined #openstack-cinder16:43
*** chhavi has quit IRC16:43
*** mvk has quit IRC16:49
*** raunak has quit IRC16:50
*** crushil has quit IRC16:55
*** lpetrut has joined #openstack-cinder17:00
*** crushil has joined #openstack-cinder17:04
*** sbezverk has joined #openstack-cinder17:09
*** masber has quit IRC17:10
*** chlong has quit IRC17:10
*** harlowja has joined #openstack-cinder17:20
*** mvk has joined #openstack-cinder17:27
*** chlong has joined #openstack-cinder17:29
*** raunak has joined #openstack-cinder17:29
*** dustins has joined #openstack-cinder17:32
*** crushil_ has joined #openstack-cinder17:34
*** itlinux has quit IRC17:48
*** amoralej is now known as amoralej|off17:50
*** tesseract has quit IRC17:53
*** dhellmann has joined #openstack-cinder17:54
dhellmannhey, folks, I need some help debugging a cinder failure in an oslo.policy patch to see if the failure is related to the patch itself or if there is something else going on17:54
dhellmannthe patch is https://review.openstack.org/#/c/498467/17:54
dhellmannthe failures in http://logs.openstack.org/67/498467/8/check/gate-tempest-dsvm-neutron-src-oslo.policy-ubuntu-xenial/ce6fb77/console.html indicate that "Policy doesn't allow volume:reserve_volume to be performed"17:55
*** felipemonteiro_ has quit IRC17:58
*** itlinux has joined #openstack-cinder17:59
*** raunak_ has joined #openstack-cinder18:01
*** raunak has quit IRC18:04
*** raunak_ is now known as raunak18:04
*** rcernin has joined #openstack-cinder18:11
*** lpetrut has quit IRC18:11
*** crushil_ has quit IRC18:14
smcginnisdhellmann: Hmm, we don't have that in our policy.json18:15
*** jmlowe has quit IRC18:15
dhellmannyeah, I see messages about that throughout the logs a lot18:16
dhellmannI suspect there's a subtle behavior change in the policy library change, but I'm not sure18:16
dhellmannI proposed a small change to its lookup logic to see if I can learn anything from that18:16
smcginnisOK... I don't know enough about oslo.policy either to understand that change and why it would affect reserve.18:17
dhellmannyeah, I don't fully understand either side of the change :-)18:18
smcginnis:)18:18
smcginnisdhellmann: My hunch is behavior changed, where before absense was the same as "target": "" for "always", where now it appears absense of the target may be interpreted as a rule of "!" to disallow.18:29
dhellmannsmcginnis : I think that points to his part of the change: https://review.openstack.org/#/c/498467/9/oslo_policy/policy.py18:29
dhellmanns/his/this/18:31
*** amoralej|off has quit IRC18:32
smcginnisdhellmann: Yeah, I felt a little suspicious of that. Mostly because I'm not sure what it's doing. :)18:32
dhellmannthe point of the patch to the library is to try to add an optional argument with the name of the rule to the check classes18:33
dhellmannbut we can't just add the argument, since we have to find all of the callers and have them test whether or not to pass the new value18:33
smcginnisSo we could potentially have multiple calls that all map to a single policy?18:33
dhellmannso we have the wrapper function _check() that does that, and we're replacing all of the places where we just invoked a rule directly using its __call__ method and replacing that with the wrapper18:34
smcginnisAren't there some issues there with the _check call? Looks like some rules would call _check, which would call the rule, which calls _check, etc.18:37
*** amoralej has joined #openstack-cinder18:38
*** ChubYann has joined #openstack-cinder18:39
dhellmannsome rules have nested rules, yes18:43
dhellmannI think line 732-740 are handling that18:44
smcginnisdhellmann: But something like a NotCheck will get evaluated to True as isinstnace(BaseCheck). So that will call _check, which will call NotCheck._call_, which will call _check,,,,18:46
smcginnisI may just be missing something here, but that doesn't look right.18:47
dhellmannthe argument to the second _check() call is the rule inside the NotCheck, it's not the NotCheck instance again.18:47
dhellmannit might be a NotCheck(TrueCheck()) for example18:47
dhellmannso you get _check() called with NotCheck() and that then calls _check() with TrueCheck()18:48
openstackgerritDavid Paterson proposed openstack/cinder master: Volume backup not supported  https://review.openstack.org/50723518:48
dhellmannsmcginnis : and in all cases current_rule is either a string or it's None18:49
dhellmannit's never a BaseCheck object itself18:49
smcginnisdhellmann: OK, I see how that works now.18:49
smcginnisSo now my only gripe is it's calling an _ method. :)18:50
dhellmannI'm going to let these tests run, and then if they fail I'm going to collapse the else part of that exception handler back into the try part18:50
dhellmannthat's a private function defined in the same library being used by another part of the library. That's allowed by Oslo rules.18:50
dhellmann_check() is private to the library, not the module18:50
smcginnisOK, just seems like it should be _check.check() and not _check._check() but that's pretty irrelevant at thsi point. :)18:51
*** itlinux has quit IRC18:51
dhellmannoh, I guess since the module itself is private that's probably true18:51
dhellmannwell, if I spin a new version I'll change that18:51
dhellmannalthough we did at one point discuss making the contents of the _checks module public18:52
dhellmannand I wouldn't want to do that with this function yet18:52
smcginnisYeah, doesn't really matter.18:52
*** wanghao_ has joined #openstack-cinder18:54
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/50320618:54
*** yangyape_ has joined #openstack-cinder18:54
*** yangyapeng has quit IRC18:55
*** wanghao has quit IRC18:55
*** dave-mccowan has quit IRC18:56
*** chlong has quit IRC19:06
*** felipemonteiro has joined #openstack-cinder19:08
*** jose-phillips has quit IRC19:08
*** lpetrut has joined #openstack-cinder19:09
*** e0ne has joined #openstack-cinder19:12
*** jose-phillips has joined #openstack-cinder19:13
*** lpetrut has quit IRC19:14
*** lifeless has quit IRC19:14
*** lifeless has joined #openstack-cinder19:15
*** jmlowe has joined #openstack-cinder19:19
*** e0ne has quit IRC19:22
*** dave-mccowan has joined #openstack-cinder19:24
*** chlong has joined #openstack-cinder19:31
*** dave-mccowan has quit IRC19:32
*** dave-mccowan has joined #openstack-cinder19:33
*** itlinux has joined #openstack-cinder19:35
*** felipemonteiro_ has joined #openstack-cinder19:37
*** felipemonteiro has quit IRC19:41
*** jmlowe has quit IRC19:42
*** dustins has quit IRC19:45
*** sapd_ has joined #openstack-cinder19:45
*** sapd has quit IRC19:49
*** jmlowe has joined #openstack-cinder19:50
*** e0ne has joined #openstack-cinder19:55
openstackgerritJohn Griffith proposed openstack/cinder master: Check for outstanding attachments during reserve  https://review.openstack.org/50446719:57
openstackgerritBoris Pavlovic proposed openstack/cinder master: Switch Rally Task To fromat V2  https://review.openstack.org/50701019:57
openstackgerritTom Swanson proposed openstack/cinder master: Dell EMC SC: API 4.0 incompatibilities  https://review.openstack.org/50726020:01
*** liverpooler has quit IRC20:04
*** chlong has quit IRC20:05
*** rcernin has quit IRC20:11
*** amoralej has quit IRC20:15
*** amoralej has joined #openstack-cinder20:18
*** e0ne has quit IRC20:18
*** KeithMnemonic has quit IRC20:19
*** KeithMnemonic has joined #openstack-cinder20:19
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Cleanup test-requirements  https://review.openstack.org/50621220:26
*** nicolasbock has quit IRC20:35
openstackgerritMerged openstack/cinder master: HPE 3PAR: fix delete operation of replicated volume  https://review.openstack.org/50610220:35
*** felipemonteiro_ has quit IRC20:43
*** felipemonteiro_ has joined #openstack-cinder20:43
openstackgerritRaunak Kumar proposed openstack/cinder master: nimble: fix for parent volume delete  https://review.openstack.org/50642120:45
*** jmlowe has quit IRC20:47
*** ganso has joined #openstack-cinder20:47
*** dustins has joined #openstack-cinder20:47
*** zengyingzhe_ has quit IRC20:48
*** chenying has joined #openstack-cinder20:48
*** juzuluag_ has joined #openstack-cinder20:48
*** zengyingzhe_ has joined #openstack-cinder20:49
dhellmannsmcginnis : the latest version of https://review.openstack.org/#/c/498467 is passing for reasons I don't fully understand. I can live with that myster for tonight, though.20:49
dhellmann*mystery20:49
openstackgerritSean McGinnis proposed openstack/cinder master: Add index for reservations on (deleted, uuid)  https://review.openstack.org/27504720:49
-openstackstatus- NOTICE: The infra team is bringing Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.20:49
smcginnisdhellmann: OK, thanks for circling back on that. I'll watch for wonkiness.20:50
openstackgerritBoris Pavlovic proposed openstack/cinder master: Switch Rally Task To format V2  https://review.openstack.org/50701020:50
dhellmannsmcginnis : oh, I'm sure it has to do with the patch to oslo.policy. I just don't understand why an "in" check failed where the current implementation worked.20:51
*** chenying_ has quit IRC20:51
*** itlinux has quit IRC20:56
*** mchlumsky has quit IRC21:03
*** gouthamr has quit IRC21:04
*** edmondsw has quit IRC21:07
*** edmondsw has joined #openstack-cinder21:10
*** edmondsw has quit IRC21:14
*** chlong has joined #openstack-cinder21:20
*** juzuluag_ has quit IRC21:22
*** jmlowe has joined #openstack-cinder21:25
*** chlong has quit IRC21:27
*** chlong has joined #openstack-cinder21:28
*** MVenesio has quit IRC21:37
*** MVenesio has joined #openstack-cinder21:37
*** MVenesio has quit IRC21:41
*** dave-mccowan has quit IRC21:43
*** jose-phillips has quit IRC21:46
*** jose-phillips has joined #openstack-cinder21:47
*** jose-phillips has quit IRC21:48
*** dustins has quit IRC21:49
*** armax has joined #openstack-cinder21:49
openstackgerritSean McGinnis proposed openstack/cinder master: Add indexes to SQLAlchemy models  https://review.openstack.org/50728821:50
*** liverpooler has joined #openstack-cinder21:55
*** crushil has quit IRC21:57
*** gouthamr has joined #openstack-cinder22:11
*** amoralej has quit IRC22:18
*** amoralej has joined #openstack-cinder22:23
*** edmondsw has joined #openstack-cinder22:42
openstackgerritSean McGinnis proposed openstack/cinder master: Compact Newton database migrations  https://review.openstack.org/50729822:42
*** dave-mccowan has joined #openstack-cinder22:44
openstackgerritRaunak Kumar proposed openstack/cinder master: nimble: fix for parent volume delete  https://review.openstack.org/50642122:44
Apoorvasmcginnis I ran the tests again, following is what I could see in the logs before other errors http://paste.openstack.org/show/621886/22:46
*** edmondsw has quit IRC22:47
*** armax has quit IRC22:47
openstackgerritJay Bryant proposed openstack/cinder master: Add opt db session arg to quota_allocated_get_all_by_project  https://review.openstack.org/45201222:49
*** crushil has joined #openstack-cinder22:58
*** dave-mcc_ has joined #openstack-cinder23:05
*** dave-mccowan has quit IRC23:07
*** ddixit has joined #openstack-cinder23:10
*** felipemonteiro_ has quit IRC23:12
openstackgerritMerged openstack/cinder master: [DOC] Remove duplicate build of module index  https://review.openstack.org/50320823:20
*** itlinux has joined #openstack-cinder23:36
*** sdague has quit IRC23:39
openstackgerritTommyLike proposed openstack/cinder-specs master: Support create volume from backup  https://review.openstack.org/50600423:43
*** masber has joined #openstack-cinder23:48
*** stakeda has joined #openstack-cinder23:55

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