Thursday, 2015-06-04

*** alexpilotti has quit IRC00:03
*** alexpilotti has joined #openstack-cinder00:05
*** alexpilotti has quit IRC00:08
*** hemna is now known as hemnafkj00:09
*** xiaohui has joined #openstack-cinder00:12
*** asselin_ has joined #openstack-cinder00:23
openstackgerritJohn Griffith proposed openstack/cinder: Revert "Remove Brick from cinder codebase"  https://review.openstack.org/18822000:27
*** jaypipes has quit IRC00:27
patrickeastwait, why is brick coming back into cinder?00:30
patrickeastjgriffith: ^00:30
jgriffithpatrickeast: read my revert comment00:30
patrickeastbut it was accepting commints and we are using it right now…. right?00:30
jgriffithpatrickeast: it turns out Walt doesn't allow patches to os-brick yet because it's "not ready" doesn't have a CI00:30
patrickeasthuh00:30
jgriffithpatrickeast: https://review.openstack.org/#/c/187811/00:31
patrickeastyea, i saw the test stuff earlier today00:31
winston-djgriffith: maybe just for today00:31
patrickeastdidnt it already merge00:31
patrickeast?00:31
jgriffithpatrickeast: did which already merge?00:31
patrickeasti thought that was like a small hiccup in the process, not a major problem00:31
*** leeantho has quit IRC00:31
patrickeastthis thing00:32
patrickeasthttps://review.openstack.org/#/c/188156/00:32
patrickeastto enable the tests00:32
asselin_that patch is stuck in gate00:32
winston-djgriffith: walt said they are bringing up some 3rd party ci soon00:32
asselin_infra is looking into it00:32
jgriffithpatrickeast: looks like it00:32
jgriffithsigh00:32
jgriffithSo the revert was more to point out the issue00:32
jgriffithNot thinking anybody would actually +2/A it :)00:33
jgriffithalthough... I guess ya never know00:33
patrickeasthaha gotcha00:33
*** lixiaoy1 has joined #openstack-cinder00:34
jgriffithpatrickeast: and I find it even more annoying that it's not consistent: https://review.openstack.org/#/c/188113/00:34
patrickeasti guess worst case is that the gate comes up and we find out its broken00:37
patrickeastwhich makes sense to block commits until then00:37
jgriffithpatrickeast: yeah00:37
jgriffithpatrickeast: makes sense00:37
patrickeastbut yea, should have been consistent00:37
patrickeasti was under the impression requirement updates cause a lot of problems in gate00:37
jgriffithpatrickeast: indeed they do00:37
jgriffithpatrickeast: We should've had this to begin with00:38
jgriffithbut no big deal00:38
jgriffithstill early enough00:38
jgriffithjust good it's getting taken care of now00:38
patrickeasthaha yea, better now than L-4 or something00:38
jgriffithhehe!00:38
*** vilobhmm1 has quit IRC00:40
*** breitz1 has quit IRC00:45
*** breitz has joined #openstack-cinder00:45
*** heyun has joined #openstack-cinder00:53
*** takedakn has joined #openstack-cinder00:53
*** Apoorva has quit IRC00:54
*** takedakn has quit IRC00:54
*** takedakn has joined #openstack-cinder00:54
*** takedakn has quit IRC01:06
*** takedakn has joined #openstack-cinder01:06
*** bswartz has quit IRC01:07
*** primechuck has joined #openstack-cinder01:08
asselin_nikeshm, congrats!01:11
*** dkalleg has quit IRC01:12
nikeshmasselin_ : thanks for all your support,we really appreciate it01:14
nikeshmasselin_ : thanks01:14
nikeshm:)01:14
*** openstack has joined #openstack-cinder01:23
nikeshmasselin_: i added a variable in /etc/environment fc_pci_device='xxxxxxxxxxx' and was able to access in script,same file when i added one more variable it was not able to get the value01:25
*** bswartz has joined #openstack-cinder01:25
nikeshmi was not able to get its value by script01:26
asselin_hmmm...strange...it should get everything01:26
nikeshmyaa that why i am asking did you try01:26
asselin_you can use the command printenv to see all the env variables defined01:26
*** Lee1092 has joined #openstack-cinder01:27
nikeshmbecause looks like our environment was facing same issue for /etc/profile.d01:27
*** openstack has joined #openstack-cinder01:38
*** _cjones_ has quit IRC01:40
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789201:41
*** primechuck has joined #openstack-cinder01:45
*** boris-42 has quit IRC01:48
*** openstack has quit IRC01:52
*** openstack has joined #openstack-cinder01:54
*** ianbrown has quit IRC01:57
*** ian_brown has joined #openstack-cinder01:57
*** bill_az has quit IRC02:11
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update  https://review.openstack.org/18823602:15
*** patrickeast has quit IRC02:25
*** gcb has quit IRC02:35
*** mriedem_away has quit IRC02:39
openstackgerritwanghao proposed openstack/cinder: Fix weird change of volume status in re-scheduling  https://review.openstack.org/18554502:40
openstackgerritchenzongliang proposed openstack/cinder: refactor Huawei Volume driver  https://review.openstack.org/18824002:44
*** asselin_ has joined #openstack-cinder02:44
*** julim has quit IRC02:47
asselin_anteaya, you around?03:00
*** primechuck has quit IRC03:00
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789203:04
*** akerr has joined #openstack-cinder03:04
*** sgotliv has joined #openstack-cinder03:04
*** akerr_ has joined #openstack-cinder03:06
*** dims_ has quit IRC03:07
*** pegmatite has joined #openstack-cinder03:08
*** akerr has quit IRC03:09
*** akerr_ has quit IRC03:09
*** lixiaoy11 has joined #openstack-cinder03:09
openstackgerritDiem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver  https://review.openstack.org/17831903:10
*** lixiaoy12 has joined #openstack-cinder03:13
*** lixiaoy11 has quit IRC03:15
*** asselin_ has quit IRC03:16
*** markvoelker has quit IRC03:23
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add retry to iSCSI delete  https://review.openstack.org/18781103:25
*** pegmatite has left #openstack-cinder03:25
*** takedakn has quit IRC03:28
*** _cjones_ has joined #openstack-cinder03:28
*** _cjones_ has quit IRC03:29
*** _cjones_ has joined #openstack-cinder03:30
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix disconnecting necessary iSCSI sessions issue  https://review.openstack.org/18815803:37
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add HGST Solutions connector  https://review.openstack.org/18658803:39
*** coolsvap|afk is now known as coolsvap03:39
*** hemna has joined #openstack-cinder03:40
*** dannywilson has quit IRC03:46
openstackgerritTomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue  https://review.openstack.org/18815903:47
openstackgerritTomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue  https://review.openstack.org/18815903:47
openstackgerritTomoki Sekiyama proposed openstack/os-brick: Fix disconnecting necessary iSCSI sessions issue  https://review.openstack.org/18815803:47
openstackgerritTomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue  https://review.openstack.org/18815903:48
openstackgerritchenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support  https://review.openstack.org/18825103:49
*** gcb has joined #openstack-cinder03:49
*** tobe has joined #openstack-cinder03:51
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update  https://review.openstack.org/18823603:52
*** asselin_ has joined #openstack-cinder03:53
*** EmilienM|off has quit IRC03:55
*** EmilienM has joined #openstack-cinder04:00
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789204:00
*** hemna has quit IRC04:02
*** pradipta has joined #openstack-cinder04:04
*** avishay has joined #openstack-cinder04:07
*** dims_ has joined #openstack-cinder04:07
*** dims_ has quit IRC04:13
*** Longgeek has joined #openstack-cinder04:14
*** sc has quit IRC04:20
*** sc has joined #openstack-cinder04:23
*** markvoelker has joined #openstack-cinder04:23
*** lixiaoy12 has quit IRC04:25
*** markvoelker has quit IRC04:28
*** yamada-h has joined #openstack-cinder04:37
*** Longgeek has quit IRC04:39
*** rushiagr_away is now known as rushiagr04:42
*** marcusvrn has joined #openstack-cinder04:43
*** Longgeek has joined #openstack-cinder04:53
*** swamireddy1 has joined #openstack-cinder04:54
openstackgerritchenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support  https://review.openstack.org/18825104:57
*** Longgeek_ has joined #openstack-cinder04:57
*** Longgeek has quit IRC05:00
*** yamada-h has quit IRC05:04
*** yamada-h has joined #openstack-cinder05:04
*** lixiaoy1 has joined #openstack-cinder05:05
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789205:08
*** yamada-h has quit IRC05:09
*** tsekiyama has quit IRC05:10
*** xyang has joined #openstack-cinder05:19
*** tobe has quit IRC05:22
*** deepakcs has joined #openstack-cinder05:22
deepakcsanyone who can help with cinder objects issue I am seeing @ http://lists.openstack.org/pipermail/openstack-dev/2015-June/065686.html ?05:27
*** angela-s has quit IRC05:31
*** BharatK has quit IRC05:39
*** tobe has joined #openstack-cinder05:40
*** agarciam has joined #openstack-cinder05:45
*** marcusvrn has quit IRC05:45
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087305:45
*** alonmarx has joined #openstack-cinder05:47
*** alonmarx_ has quit IRC05:49
*** BharatK has joined #openstack-cinder05:52
*** belmoreira has joined #openstack-cinder05:56
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789205:57
*** _cjones_ has quit IRC05:57
*** _cjones_ has joined #openstack-cinder06:01
*** nlevinki has joined #openstack-cinder06:01
*** swamireddy2 has joined #openstack-cinder06:02
*** lpetrut has joined #openstack-cinder06:03
*** pradipta has quit IRC06:04
*** swamireddy1 has quit IRC06:05
*** yamada-h has joined #openstack-cinder06:11
*** markvoelker has joined #openstack-cinder06:13
spradeepvanyone who can help with cinder volume test failure I am facing @ https://ask.openstack.org/en/question/67974/cinder-tempest-volume-tests-failed/06:15
*** markvoelker has quit IRC06:18
*** tobe has quit IRC06:18
*** alonmarx_ has joined #openstack-cinder06:21
*** alonmarx has quit IRC06:23
*** jdurgin1 has joined #openstack-cinder06:39
*** _cjones_ has quit IRC06:44
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632706:44
*** daneyon_ has quit IRC06:50
*** ndipanov has quit IRC06:53
*** _cjones_ has joined #openstack-cinder06:56
*** tobe has joined #openstack-cinder06:57
*** anshul has joined #openstack-cinder06:58
*** pradipta has joined #openstack-cinder06:59
*** tshefi has joined #openstack-cinder07:01
*** _cjones_ has quit IRC07:01
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632707:05
openstackgerritJessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage  https://review.openstack.org/17711307:06
openstackgerritJessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage  https://review.openstack.org/17711307:10
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632707:11
*** markvoelker has joined #openstack-cinder07:13
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632707:15
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789207:17
*** vincent_hou has joined #openstack-cinder07:17
*** markvoelker has quit IRC07:18
vincent_houavishay: Ping. I updated the spec https://review.openstack.org/#/c/186327/. Welcome your comments07:18
avishayvincent_hou: I replied to patch set 10, welcome your replies :)07:19
*** aarefiev has joined #openstack-cinder07:19
vincent_houThank you. I am changing.07:26
*** ronis has joined #openstack-cinder07:35
openstackgerritwanghao proposed openstack/cinder-specs: Incremental backup improvements for L  https://review.openstack.org/18792607:35
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632707:38
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632707:40
*** lpetrut has quit IRC07:44
*** dims_ has joined #openstack-cinder07:44
*** chlong has quit IRC07:44
*** dims_ has quit IRC07:49
*** vincent_hou_ has joined #openstack-cinder07:49
*** jistr has joined #openstack-cinder07:50
*** vincent_hou has quit IRC07:50
*** vincent_hou_ is now known as vincent_hou07:51
*** jistr is now known as jistr|mt07:54
*** jistr|mt is now known as jistr|mtg07:54
*** _cjones_ has joined #openstack-cinder07:57
*** vincent_hou has quit IRC08:01
*** afazekas has joined #openstack-cinder08:01
*** _cjones_ has quit IRC08:02
*** aix has joined #openstack-cinder08:03
*** jdurgin1 has quit IRC08:05
*** ndipanov has joined #openstack-cinder08:07
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects  https://review.openstack.org/18789208:20
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements  https://review.openstack.org/17280808:27
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/cinder: tests: glusterfs: Remove online snapshot related tests  https://review.openstack.org/18830008:27
*** jordanP has joined #openstack-cinder08:27
*** swamireddy1 has joined #openstack-cinder08:33
*** swamireddy2 has quit IRC08:36
openstackgerritwanghao proposed openstack/cinder-specs: Incremental backup improvements for L  https://review.openstack.org/18792608:37
*** daneyon has joined #openstack-cinder08:38
*** tobe has quit IRC08:39
*** tobe has joined #openstack-cinder08:39
openstackgerritwanghao proposed openstack/cinder-specs: Incremental backup improvements for L  https://review.openstack.org/18792608:40
*** gcb has quit IRC08:41
*** daneyon has quit IRC08:43
*** lpetrut has joined #openstack-cinder08:46
*** avishay_ has joined #openstack-cinder09:01
*** avishay has quit IRC09:01
*** markvoelker has joined #openstack-cinder09:02
*** ociuhandu has quit IRC09:02
*** markvoelker has quit IRC09:07
*** jistr|mtg is now known as jistr09:19
*** ociuhandu has joined #openstack-cinder09:20
*** tobe has quit IRC09:33
*** BharatK has quit IRC09:38
*** e0ne has joined #openstack-cinder09:44
*** afazekas is now known as afazekas_mtg09:44
*** dims_ has joined #openstack-cinder09:45
*** ociuhandu has quit IRC09:46
*** yamada-h has quit IRC09:50
openstackgerritwanghao proposed openstack/cinder: Incremental backup improvements for L  https://review.openstack.org/18832809:51
*** dims_ has quit IRC09:51
*** BharatK has joined #openstack-cinder09:51
*** tobe has joined #openstack-cinder09:53
*** lixiaoy1 has quit IRC09:54
*** _cjones_ has joined #openstack-cinder10:00
*** dims_ has joined #openstack-cinder10:03
*** _cjones_ has quit IRC10:05
*** Zhongjun has joined #openstack-cinder10:05
*** yamada-h has joined #openstack-cinder10:10
*** yamada-h has quit IRC10:10
*** ociuhandu has joined #openstack-cinder10:11
*** markvoelker has joined #openstack-cinder10:18
*** markvoelker has quit IRC10:22
luvhi. https://git.openstack.org/cgit/openstack/cinder/tree/cinder/utils.py#n60 this surely doesn't look correct, does it?10:22
luvShould the function not look for a config file in config directory instead of state_path ??10:23
luvthis breaks deploying cinder in virtualenv for me10:24
luvhttp://pastebin.blesmrt.net/pbgdgkmyg10:25
*** daneyon has joined #openstack-cinder10:27
*** coolsvap is now known as coolsvap|afk10:30
*** daneyon has quit IRC10:32
*** lpetrut1 has joined #openstack-cinder10:33
*** lpetrut has quit IRC10:33
*** boris-42 has joined #openstack-cinder10:33
*** lpetrut1 has left #openstack-cinder10:34
*** lpetrut1 has joined #openstack-cinder10:34
DuncanTluv: What do you set for state_path?10:38
DuncanTI believe the point is so that you can confine all of the cinder stuff to a sub-tree - most people will run with '/' I believe10:38
*** ociuhandu has quit IRC10:39
luvDuncanT: /var/lib/cinder/10:40
luvsame as other openstack service i always set state_path to /var/lib/service .... I think ubuntu does the same by default ... let me check10:41
luvyup, it does.10:42
luvstate_path = /var/lib/cinder in etc/cinder/cinder.conf in cinder-common package10:43
deepakcsDuncanT: Hi, looking for some help on a strange side-effect seen in cinder unit test. Details @ http://lists.openstack.org/pipermail/openstack-dev/2015-June/065801.html10:43
*** ociuhandu has joined #openstack-cinder10:44
luvand that's same for upstream cinder as well ... default state_path is /var/lib/cinder10:45
luvhttps://git.openstack.org/cgit/openstack/cinder/tree/cinder/common/config.py#n4410:46
*** markvoelker has joined #openstack-cinder10:46
*** pradipta has quit IRC10:50
*** geguileor has quit IRC10:52
*** Longgeek_ has quit IRC10:53
*** coolsvap|afk is now known as coolsvap10:54
*** Longgeek has joined #openstack-cinder10:54
*** deepakcs has quit IRC10:56
*** IlyaG has quit IRC10:59
*** _cjones_ has joined #openstack-cinder11:01
*** _cjones_ has quit IRC11:06
*** yamada-h has joined #openstack-cinder11:10
*** yamada-h has quit IRC11:15
*** coolsvap is now known as coolsvap|afk11:16
*** xyang has quit IRC11:19
*** timcl has joined #openstack-cinder11:26
*** aix has quit IRC11:30
*** e0ne is now known as e0ne_11:39
*** heyun has quit IRC11:41
*** e0ne_ has quit IRC11:49
DuncanTdeepakcs: That is odd11:51
*** afazekas_mtg has quit IRC11:53
openstackgerritchenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support  https://review.openstack.org/18825111:53
*** agarciam has quit IRC11:54
*** shyama has joined #openstack-cinder12:01
*** _cjones_ has joined #openstack-cinder12:02
*** chhavi has joined #openstack-cinder12:03
*** avishay_ has quit IRC12:05
*** avishay__ has joined #openstack-cinder12:05
openstackgerritPradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector  https://review.openstack.org/18479912:05
openstackgerritchenzongliang proposed openstack/cinder: Add Chap support for Huawei Storage  https://review.openstack.org/18836012:06
*** tobe has quit IRC12:06
*** _cjones_ has quit IRC12:07
chhaviwinston-d: Want to discuss on https://review.openstack.org/#/c/182980/12:08
*** e0ne has joined #openstack-cinder12:13
*** avishay__ is now known as avishay12:13
openstackgerritchenzongliang proposed openstack/cinder: Huawei Storage iSCSI multipath Support  https://review.openstack.org/18836512:14
*** daneyon has joined #openstack-cinder12:16
*** mtanino has joined #openstack-cinder12:16
*** chlong has joined #openstack-cinder12:20
lpetrut1DuncanT: Hi12:20
lpetrut1DuncanT: Could you please take a look over the 2 following bug fixes? https://review.openstack.org/#/c/174291/6 https://review.openstack.org/#/c/158713/12:21
*** daneyon has quit IRC12:21
lpetrut1DuncanT: those are up for review for some time now, but due to the fact that we had some issues w/ the CI, the tests were failing. Those issues were fixed.12:22
lpetrut1DuncanT: Those 2 bug fixes are really important for us so at the moment we have to cherry-pick them on the CI before testing changes, so I would really like to get them in as soon as possible.12:23
*** akerr has joined #openstack-cinder12:23
openstackgerritAlexander Maretskiy proposed openstack/cinder: Changes in rally-jobs/README.rst  https://review.openstack.org/18837112:24
*** aix has joined #openstack-cinder12:30
*** rushil has joined #openstack-cinder12:31
*** rushil has quit IRC12:32
*** yamada-h has joined #openstack-cinder12:35
*** rushiagr is now known as rushiagr_away12:36
*** rwsu has joined #openstack-cinder12:36
openstackgerritRajesh Tailor proposed openstack/cinder: Removed explicit return from __init__ method  https://review.openstack.org/18837712:38
openstackgerritwanghao proposed openstack/cinder: Incremental backup improvements for L  https://review.openstack.org/18832812:38
*** yamada-h has quit IRC12:40
*** xyang1 has joined #openstack-cinder12:41
*** xyang has joined #openstack-cinder12:41
*** fanyaohong has joined #openstack-cinder12:44
*** swamireddy2 has joined #openstack-cinder12:44
*** afazekas_mtg has joined #openstack-cinder12:45
*** BharatK has quit IRC12:46
*** swamireddy1 has quit IRC12:47
*** xyang has quit IRC12:50
*** IanGovett has joined #openstack-cinder12:53
*** merooney has joined #openstack-cinder12:55
*** smoriya has quit IRC12:56
*** delatte has joined #openstack-cinder12:58
*** zhenguo has quit IRC12:58
*** swamireddy1 has joined #openstack-cinder12:59
*** cdelatte has quit IRC12:59
*** delattec has quit IRC12:59
*** BharatK has joined #openstack-cinder13:00
*** delattec has joined #openstack-cinder13:00
*** IlyaG has joined #openstack-cinder13:01
*** e0ne is now known as e0ne_13:01
*** swamireddy2 has quit IRC13:02
*** delatte has quit IRC13:02
*** afazekas_mtg has quit IRC13:04
*** julim has joined #openstack-cinder13:04
*** e0ne_ is now known as e0ne13:04
*** deepakcs has joined #openstack-cinder13:04
*** IlyaG has quit IRC13:05
*** mtanino has quit IRC13:05
*** swamireddy1 has quit IRC13:07
*** Miouge_ has joined #openstack-cinder13:09
*** akshai has joined #openstack-cinder13:12
*** Miouge has quit IRC13:12
*** Miouge_ is now known as Miouge13:12
*** mriedem has joined #openstack-cinder13:15
*** nlevinki has quit IRC13:17
*** nlevinki has joined #openstack-cinder13:19
*** eharney has quit IRC13:20
*** Yogi1 has joined #openstack-cinder13:20
DuncanTlpetrut1: Given the big stripe of red on the CIs, I'll let the recheck run first. Code looks good though, and the couple of CIs I checked the logs off look like unrelated issues13:23
lpetrut1DuncanT: sure, thanks!13:28
*** rushiagr_away is now known as rushiagr13:28
*** gouthamr has joined #openstack-cinder13:30
*** primechuck has joined #openstack-cinder13:32
*** rushil has joined #openstack-cinder13:33
*** merooney has quit IRC13:34
*** dustins has joined #openstack-cinder13:35
*** thangp has joined #openstack-cinder13:36
*** jungleboyj has quit IRC13:39
*** merooney has joined #openstack-cinder13:41
*** merooney has quit IRC13:44
*** rushil has quit IRC13:45
*** BharatK has quit IRC13:47
*** eharney has joined #openstack-cinder13:47
*** chhavi has quit IRC13:47
deepakcsthangp: i saw your comment for 17280813:49
*** nlevinki has quit IRC13:49
thangpdeepakcs: ok13:49
deepakcsthangp: i can work to make the db calls chnage to objects.. but still that doesn't explain the strange unit test failure, does it ?13:49
deepakcsthangp: did u get a chance to look at the unit test side effect I Cc'ed u on ?13:49
thangpdeepakcs: might be related to the db change13:49
deepakcsthangp: you mean the side effect is bcos of not using objects ?13:50
thangpdeepakcs: the test cases that fail use self._create_snapshot to create the snapshot db entry, which contains a volume_id attr13:51
thangpobjects.Snapshot.get_by_id just reads whatever is in the db13:52
thangpthis is for test_delete_no_dev_fails unit test13:52
deepakcsthangp: ok, so i am still not clear, if the create_snap creates a db entry with volume_id attr, why does we get error "volume_id can't be none" ?13:53
thangpdeepakcs: dont know, did you refresh your virt env?13:54
deepakcsthangp: no, but if u doubt it i can delete .venv and try again ?13:54
thangpdeepakcs: looks like something corrupt in it13:54
deepakcsthangp: also its strange bcos test_delete_no_dev_fails when run all by itself works, only when run along with my remotefs test, it fails and i havent' been able to figure how they overalp for somehting in DB to corrupt13:55
deepakcss/test_delete_no_dev_fails / test_delete_no_dev passes13:55
thangpdeepakcs: what happens when you dont put in your test case?13:55
deepakcsthangp: when i don't put in any testcase as part of my patch.. all works fine13:56
thangpdeepakcs: which test case in test_remotefs fails when you put it in?13:56
deepakcsthangp: for eg: i rename my test_xxx to _test_xxx.. that disables it and it works w/o errors13:56
deepakcsthangp: test_create_snapshot_online_success in test_remotefs.py13:57
deepakcsthangp: wait13:57
deepakcsthangp: when i put a new testcase in test_remotefs.py.it passes, but i see errors from VolumeTestCase test suite13:57
deepakcsthangp: i hope you are clear on the side effect ? I explained it in http://lists.openstack.org/pipermail/openstack-dev/2015-June/065801.html13:58
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/18611213:58
*** merooney has joined #openstack-cinder13:58
*** rushil has joined #openstack-cinder13:58
deepakcsthangp: IOW, the testcase that fail weren't even touched by me!13:59
*** e0ne is now known as e0ne_13:59
deepakcsthangp: and thats why its strange fail :)13:59
thangpdeepakcs: ok...try this14:01
*** nlevinki has joined #openstack-cinder14:01
thangpdeepakcs: put "snapshot_obj = objects.Snapshot.get_by_id(self.context, snapshot_id)" in https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_volume.py#L303314:02
*** BharatK has joined #openstack-cinder14:02
thangpactually put - snapshot_obj = objects.Snapshot.get_by_id(context.get_admin_context(), snapshot_id)14:02
thangpand delete the db_snapshot_get14:03
deepakcsthangp: ok, so basicaly u r saying move all db. calls to objects. calls ?14:03
thangpyup, in that unit test14:03
deepakcsin test_volume.py ?14:03
thangpdeepakcs: no, only that unit test14:03
deepakcsthangp: ok, my env is not up, so can't do right now, will note this down and get back.14:04
thangpdeepakcs: because you said you narrowed it down to it14:04
*** _cjones_ has joined #openstack-cinder14:04
*** e0ne_ has quit IRC14:04
deepakcsthangp: no, i narrowed it down to <remotefs unit test followed by volumetestcase tests> fail14:04
deepakcsthangp: in VolumeTestCase there are 21 testcases failing with the same error :)14:04
deepakcsthangp: let me explain in brief, so u r clear...14:05
deepakcsthangp: when i run VolumeTestCase tests, all by themselves , they all run fine14:05
*** daneyon has joined #openstack-cinder14:05
deepakcsthangp: when i run the entire test suite (./run_tests.sh -N) 21 failures in volumetestcase suite14:05
*** e0ne has joined #openstack-cinder14:06
*** nlevinki has quit IRC14:06
deepakcsthangp: so eventually i figured that my testcase when added in test_remotefs.py is causing those 21 failures in volumetestcase test suite14:06
deepakcsthangp: and i have no idea why my testcase in a diff test suite has anything to do with volumetestcase tests!14:06
deepakcsthangp: in short, there are 21 testcase failures all in volumetestcase test class14:07
deepakcsthangp: http://logs.openstack.org/08/172808/6/check/gate-cinder-python27/daea7cd/console.html (grep for 'Captured traceback')14:08
*** _cjones_ has quit IRC14:08
*** daneyon has quit IRC14:10
*** nlevinki has joined #openstack-cinder14:10
*** rmesta has joined #openstack-cinder14:10
*** rushil has quit IRC14:10
*** rushil has joined #openstack-cinder14:12
deepakcsthangp: there ? :)14:12
*** merooney has quit IRC14:13
thangpdeepakcs: i am pulling down your patch and testing it myself14:13
thangpdeepakcs: i think there was some kind of side effect that your patch made14:13
thangpdeepakcs: because i have not seen this problem with other patches14:14
*** IanGovett has quit IRC14:14
*** mtanino has joined #openstack-cinder14:16
deepakcsthangp: yes, and i hope you shoudln't have issues applying it, since i had rebased it recently14:16
*** bswartz has quit IRC14:16
deepakcsthangp: let me know here or best is to reply on top of the -dev list thread, that ways it could help others too14:16
*** IlyaG has joined #openstack-cinder14:17
deepakcsthangp: i spent close to 2 days to figure what could be the side effect, trying to narrow down testcase , but no success :(((14:17
*** cknight has joined #openstack-cinder14:17
deepakcsthangp: all i can narrow down as of today is that even 1 testcase i add in test_remotefs, it caues 21 failures in test_volume.py14:17
deepakcsthangp: so TIA and appreciate your help in resolving this14:18
*** bswartz has joined #openstack-cinder14:18
deepakcsthangp++ :)14:18
*** merooney has joined #openstack-cinder14:18
*** dulek has joined #openstack-cinder14:19
*** jungleboyj has joined #openstack-cinder14:20
*** mtanino_ has joined #openstack-cinder14:21
*** IlyaG has quit IRC14:22
*** mtanino has quit IRC14:23
lpetrut1DuncanT: The CI was happy about this https://review.openstack.org/#/c/174291/ . Well, the 3rd party CIs failing are unrelated to the change.14:23
*** dustins_ has joined #openstack-cinder14:25
*** eharney_ has joined #openstack-cinder14:25
*** eharney has quit IRC14:26
*** eharney_ is now known as eharney14:26
*** dustins has quit IRC14:28
*** tshefi has quit IRC14:29
*** julim has quit IRC14:29
*** dulek has quit IRC14:32
*** dulek_ has joined #openstack-cinder14:32
*** dulek_ has quit IRC14:32
*** dulek_ has joined #openstack-cinder14:32
*** eharney has quit IRC14:32
*** eharney has joined #openstack-cinder14:34
*** afazekas_mtg has joined #openstack-cinder14:35
*** yamada-h has joined #openstack-cinder14:36
*** mtanino_ has quit IRC14:39
*** yamada-h has quit IRC14:41
*** mtanino has joined #openstack-cinder14:43
*** afazekas_mtg has quit IRC14:43
openstackgerritVincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers  https://review.openstack.org/18087314:43
*** rajinir_r has quit IRC14:45
*** rajinir_r has joined #openstack-cinder14:46
*** gouthamr has quit IRC14:46
*** ganso_ has joined #openstack-cinder14:46
openstackgerritAlexander Maretskiy proposed openstack/cinder: Changes in rally-jobs/README.rst  https://review.openstack.org/18837114:46
*** ronis has quit IRC14:47
*** chlong has quit IRC14:47
*** markstur has quit IRC14:51
*** fanyaohong has quit IRC14:53
*** deepakcs has quit IRC14:53
*** nlevinki has quit IRC14:54
*** markstur has joined #openstack-cinder14:55
openstackgerritrakesh mishra proposed openstack/cinder: set-max-size-limit-for-volume  https://review.openstack.org/18590614:57
*** lpabon has joined #openstack-cinder15:00
*** belmoreira has quit IRC15:05
*** _cjones_ has joined #openstack-cinder15:05
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632715:06
*** hemnafkj is now known as hemna15:06
luvDuncanT: really looks like a bug to many15:08
*** julim has joined #openstack-cinder15:08
luvprobably affecting just *very* few deployments in non standard location15:08
*** dustins_ has quit IRC15:10
*** _cjones_ has quit IRC15:10
openstackgerritVincent Hou proposed openstack/cinder-specs: Volume migration improvement for L  https://review.openstack.org/18632715:11
*** alexpilotti has joined #openstack-cinder15:11
*** merooney has quit IRC15:12
*** annegentle has joined #openstack-cinder15:13
*** eharney has quit IRC15:13
openstackgerritTom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls  https://review.openstack.org/18239615:14
*** daneyon has joined #openstack-cinder15:16
*** BharatK has quit IRC15:19
*** dannywilson has joined #openstack-cinder15:21
*** dustins has joined #openstack-cinder15:21
*** jdurgin1 has joined #openstack-cinder15:23
*** eharney has joined #openstack-cinder15:25
*** dustins has quit IRC15:25
openstackgerritMichal Dulko proposed openstack/cinder: Don't send heartbeats if Manager reports a problem  https://review.openstack.org/18767015:27
*** crose has joined #openstack-cinder15:31
*** dulek_ has quit IRC15:32
*** IlyaG has joined #openstack-cinder15:33
openstackgerritJordan Pittier proposed openstack/cinder: Scality SOFS: enhance how the remoteFS mount is detected  https://review.openstack.org/18844215:35
*** IlyaG has quit IRC15:37
*** _cjones_ has joined #openstack-cinder15:38
*** timcl has quit IRC15:39
*** agarciam has joined #openstack-cinder15:39
*** dims_ has quit IRC15:43
*** jordanP has quit IRC15:45
openstackgerritDanny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver  https://review.openstack.org/18660515:45
*** lpetrut1 has quit IRC15:49
thingeexyang1, e0ne https://etherpad.openstack.org/p/cinder-liberty-drivers15:49
e0nethingee: thanks!15:50
openstackgerritThang Pham proposed openstack/cinder: Cinder part of volume snapshot improvements  https://review.openstack.org/17280815:51
xyang1thingee: Thanks15:51
*** lprice has joined #openstack-cinder15:52
*** lprice has left #openstack-cinder15:53
openstackgerritYuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param  https://review.openstack.org/15774915:55
kvidvansthingee: Do drivers, pulled out in kilo becuase of no CI, need a new blueprint submitted? I ask since I didnt see https://review.openstack.org/#/c/178319 in the list. Thanks!15:56
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update  https://review.openstack.org/18823615:58
dannywilsonthingee: thanks, just fixed the latest issue in Pure FC and updated etherpad15:58
*** jistr has quit IRC16:01
*** agarciam has quit IRC16:02
*** julim has quit IRC16:03
*** julim has joined #openstack-cinder16:05
*** annegentle has quit IRC16:08
*** jaypipes has joined #openstack-cinder16:08
*** rmesta has quit IRC16:13
*** rmesta has joined #openstack-cinder16:13
*** rmesta has quit IRC16:14
*** rmesta has joined #openstack-cinder16:14
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/18611216:17
*** jordanP has joined #openstack-cinder16:18
openstackgerritYuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param  https://review.openstack.org/15774916:18
*** rasoto has joined #openstack-cinder16:19
*** jordanP has quit IRC16:19
*** Apoorva has joined #openstack-cinder16:19
thingeekvidvans: yes. purely for tracking purposes16:21
thingeeno details needed, just driver name and link to the review16:21
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/18849016:24
*** jcru has joined #openstack-cinder16:31
*** ronis has joined #openstack-cinder16:32
*** avishay has quit IRC16:32
*** aix has quit IRC16:33
openstackgerritThang Pham proposed openstack/cinder: Complete switch to snapshot objects  https://review.openstack.org/16391016:35
openstackgerritThang Pham proposed openstack/cinder: Prophetstor driver needs to return snapshot objects  https://review.openstack.org/18549516:35
*** yamada-h has joined #openstack-cinder16:37
*** breitz has quit IRC16:37
*** breitz has joined #openstack-cinder16:38
*** lpetrut has joined #openstack-cinder16:38
*** dims_ has joined #openstack-cinder16:39
*** merooney has joined #openstack-cinder16:42
*** timcl has joined #openstack-cinder16:42
*** yamada-h has quit IRC16:42
*** gouthamr has joined #openstack-cinder16:43
*** gouthamr has left #openstack-cinder16:43
*** gouthamr has joined #openstack-cinder16:44
*** jungleboyj has quit IRC16:46
*** IlyaG has joined #openstack-cinder16:49
*** asselin_ has quit IRC16:49
*** rushil has quit IRC16:51
*** IlyaG has quit IRC16:53
*** swamireddy1 has joined #openstack-cinder16:54
*** rushil has joined #openstack-cinder16:57
*** merooney has quit IRC17:01
*** vilobhmm has joined #openstack-cinder17:03
*** swamireddy1 has quit IRC17:04
*** Yogi11 has joined #openstack-cinder17:06
*** rushil has quit IRC17:06
*** merooney has joined #openstack-cinder17:06
*** Yogi1 has quit IRC17:09
*** rushil has joined #openstack-cinder17:10
openstackgerritJessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage  https://review.openstack.org/17711317:11
kvidvansthingee: will file a blueprint. Thanks17:16
*** pradipta has joined #openstack-cinder17:17
*** haomaiwang has joined #openstack-cinder17:19
*** haomaiw__ has quit IRC17:20
*** e0ne has quit IRC17:22
*** dan has quit IRC17:23
*** Viswanath has joined #openstack-cinder17:25
*** daneyon_ has joined #openstack-cinder17:26
*** shyama has quit IRC17:27
*** rbradbury has joined #openstack-cinder17:28
*** daneyon has quit IRC17:29
*** Viswanath has quit IRC17:30
*** jungleboyj has joined #openstack-cinder17:30
*** ndipanov has quit IRC17:30
*** anshul has quit IRC17:33
*** delattec has quit IRC17:34
*** cdelatte has joined #openstack-cinder17:34
*** haomaiwa_ has joined #openstack-cinder17:39
*** cdelatte has quit IRC17:39
*** dims_ has quit IRC17:40
*** haomaiwang has quit IRC17:41
*** IlyaG has joined #openstack-cinder17:47
*** dontalton has joined #openstack-cinder17:47
*** patrickeast has joined #openstack-cinder17:53
*** thangp has quit IRC17:55
*** thangp has joined #openstack-cinder17:55
*** rushil has quit IRC17:57
*** crose has quit IRC17:57
*** rushiagr is now known as rushiagr_away17:57
*** merooney has quit IRC17:57
*** merooney has joined #openstack-cinder18:00
*** annegentle has joined #openstack-cinder18:01
*** Viswanath has joined #openstack-cinder18:01
*** barra204 has joined #openstack-cinder18:01
anteayaasselin: sorry I wasn't around earlier, do you still wish for IBM GPFS CI to be disabled?18:02
asselinanteaya, already done18:02
anteayaoh good thank you18:02
anteayanice work18:02
patrickeastanteaya: asselin: i sent them an email a couple days ago, it sounds like they were at least looking into the issues18:03
patrickeastjungleboyj: ^18:03
anteayapatrickeast: ah okay well thanks for trying to get a hold of them18:03
anteayaI have concerns if after a few days they still can't figure out how to turn their system off18:04
asselinyea I was talking to them via e-mail with jungleboyj. they powered off the system, but it was still commenting. Disabling is the only option left until they figure it out.18:04
anteayawhich is entirely different than fixing it18:04
patrickeast:( yea its still voting failures http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=&timeframe=2418:04
anteayaasselin: agreed18:04
asselinalso they have 2 accounts, which could be causing the issue.....18:04
*** ganso_ has quit IRC18:04
*** Viswanath has quit IRC18:04
anteayaasselin: well 2 accounts doesn't help18:04
asselinboth were disabled18:04
*** kvidvans has quit IRC18:04
anteayaasselin: thanks18:04
patrickeastouch, yea thats not good if they couldn’t get it disabled18:04
anteayapatrickeast: yes, that is my concern18:05
anteayaasselin patrickeast thank you both for trying to help them and also for following up18:05
asselinyeah...cloudfounders is next18:06
*** tellesnobrega_ has joined #openstack-cinder18:06
anteayago clean them up18:06
anteayaand thank you18:06
*** barra204 has quit IRC18:07
dannywilsonasselin: Hi. I am trying to switch our CI to dib and having a weird problem at build time.  Is your nodepool.yaml.erb in os-ext-testing-data the one you are actually using? I am basing mine off that one.18:07
anteayaand the reason that a system may still comment after we disabled them is that gerrit has to wait for the connection to expire18:07
anteayaunless we restart gerrit18:07
patrickeastasselin: heh yea, should poke them, they haven’t voted correctly in a long time (assuming at some point they *did*)18:07
anteayain our next gerrit upgrade we hope to have a feature, close connection, that will allow us to id an ssh connection by name and close it18:07
anteayabut we don't have that yet18:07
asselinanteaya, ok...fungi flushed the cache...but I guess it's not 100%18:08
jungleboyjanteaya: patrickeast and asselin My deepest apologies.  I am dealing with that.18:08
patrickeastanteaya: that would be good to have18:08
anteayaasselin: no not the same18:08
jungleboyjI have been sending them e-mail since 5/22 about this.18:08
anteayajungleboyj: I feel so sorry for you that you don't have better support18:08
jungleboyjNo excuse.18:08
anteayajungleboyj: I know if it was up to you it would have been off at the first hint of problem18:08
*** akerr has quit IRC18:08
*** dims_ has joined #openstack-cinder18:08
asselindannywilson, it's not excatly the one I use...but quite close18:08
anteayapatrickeast: I'm looking forward to it18:08
jungleboyjWell, the person signed up to work on it changed, the new person wasn't responding and now Sasi is trying to clean up.18:08
asselindannywilson, what issue do you have?18:09
anteayajungleboyj: yup that is usually how a mess like this happens18:09
anteayapatrickeast: where is the code for your dashboard again?18:09
dannywilsonasselin:  timeout: failed to run command ‘/opt/stack/new/devstack-gate/devstack-vm-gate.sh’: No such file or directory18:09
*** haomai___ has joined #openstack-cinder18:09
patrickeastanteaya: now it lives in https://github.com/stackforge/third-party-ci-tools/tree/master/monitoring/scoreboard18:09
asselindannywilson, can you provide more context? Is this when you build an image with dib?18:09
*** haomaiwa_ has quit IRC18:10
anteayapatrickeast: great thank you18:10
dannywilsonasselin: image is already built and uploaded, trying to execute a build from sandbox18:10
asselindannywilson, link to logs?18:11
dannywilsonasselin: http://ec2-54-67-119-204.us-west-1.compute.amazonaws.com/91/188291/1/check/PureISCSIDriver-tempest-dsvm-volume/6858040/18:11
asselindannywilson, hmmm...something doesn't look right here: http://ec2-54-67-119-204.us-west-1.compute.amazonaws.com/91/188291/1/check/PureISCSIDriver-tempest-dsvm-volume/6858040/logs/devstack-gate-setup-workspace-new.txt.gz18:12
dannywilsonasselin: indeed, that is weird18:13
*** merooney has quit IRC18:13
jungleboyjanteaya: asselin I will get it worked out.18:13
asselindannywilson, what os are you building the images on?18:13
jungleboyjThanks for the understanding.18:13
patrickeastasselin: its on our jenkins-master, ubuntu 14.0418:13
*** leeantho has joined #openstack-cinder18:14
*** merooney has joined #openstack-cinder18:14
dannywilsonasselin: our nodepool.yaml.erb file is pasted here http://paste.openstack.org/show/263574/18:15
asselindannywilson, can you launch a nova vm using that image and smoke test it?18:15
*** akshai has quit IRC18:17
dannywilsonasselin: will do18:17
asselindannywilson, line 41 and 46 I don't have...18:17
*** pradipta has quit IRC18:17
asselinline 41 I assume is your custom stuff18:17
asselinand 46 I believe uses that regardless of the setting in nodepool.yaml (depends on nodepool version)18:18
*** timcl1 has joined #openstack-cinder18:18
dannywilsonyes, I made an element, that seems to work correctly but I will check it when I start an instance18:18
dannywilsonshould I take 46 out?18:18
*** timcl has quit IRC18:19
asselindannywilson, not at this point....18:19
asselindannywilson, you can compare to ours: http://15.126.198.151/01/184001/6/check/3par-iscsi-driver-master-client-pip-megatron-dsvm/2a1b74a/logs/devstack-gate-setup-workspace-new.txt.gz18:20
*** merooney has quit IRC18:20
asselinwe don't have -n /dev/vdb  so don't have that issue....18:21
*** annegentle has quit IRC18:21
*** annegentle has joined #openstack-cinder18:22
*** merooney has joined #openstack-cinder18:24
*** tellesnobrega_ has quit IRC18:29
*** tellesnobrega_ has joined #openstack-cinder18:29
*** tellesnobrega_ has quit IRC18:29
*** jcru has quit IRC18:30
*** merooney has quit IRC18:31
*** merooney has joined #openstack-cinder18:31
*** tellesnobrega_ has joined #openstack-cinder18:32
*** tellesnobrega_ has quit IRC18:32
hemnathingee, ping18:38
openstackgerritHuang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name  https://review.openstack.org/18854318:38
*** openstackstatus has joined #openstack-cinder18:39
*** ChanServ sets mode: +v openstackstatus18:39
hemnajungleboyj, smcginnis ping18:39
*** Zhongjun has quit IRC18:39
*** tellesnobrega_ has joined #openstack-cinder18:41
-openstackstatus- NOTICE: Gerrit has been restarted to clear an issue with its event stream. Any change events between 17:25 and 18:38 UTC should be rechecked or have their approvals reapplied to initiate testing.18:41
*** tellesnobrega_ has quit IRC18:43
*** raunak has joined #openstack-cinder18:44
*** tellesnobrega_ has joined #openstack-cinder18:45
*** raunak_ has joined #openstack-cinder18:48
smcginnishemna: pong18:51
*** tsbot_ has quit IRC18:51
*** raunak has left #openstack-cinder18:52
smcginnishemna: Request timed out.  :-P18:54
*** krtaylor has quit IRC18:59
*** dannywilson has quit IRC19:01
*** dannywilson has joined #openstack-cinder19:03
*** patrickeast_ has joined #openstack-cinder19:04
*** Apoorva has quit IRC19:04
*** patrickeast has quit IRC19:06
*** patrickeast_ is now known as patrickeast19:06
*** annegentle has quit IRC19:07
hemnahey19:07
hemnasmcginnis, so https://review.openstack.org/#/c/143205  that landed it looks like in K.19:07
hemnawhich is a way for drivers to put their backend volume id associated with the cinder volume id in the volumes table.19:07
hemnasmcginnis, https://review.openstack.org/#/c/188236/  I have that one in flight that adds our volume name to the provider_id19:08
hemnabut we aren't showing the provider_id from the client19:08
hemnacinder show <id>19:08
hemnaso I started digging into the API and the view builder doesn't have a way to distinguish admin vs non admin19:09
hemnawhen it builds the volume dict to return in the API call19:09
hemnawe are kinda hacking in the volume admin metadata into the metadata19:09
hemnaso that rides for free in the metadata dict in the return19:09
hemnakinda seems like the viewbuilder should have a way to filter out admin context or not and show data based upon context19:12
*** Longgeek has quit IRC19:12
hemnawhich it doesn't19:12
hemnawas hoping thingee could chime in as well19:12
*** merooney has quit IRC19:13
thingeehemna: meetings meetings atm19:13
thingeehemna: https://twitter.com/Thingee/status/60650174938220953619:13
hemnahaha19:18
hemnaawesome19:18
*** Viswanath has joined #openstack-cinder19:18
hemnaplanning to plan!19:18
*** rasoto has quit IRC19:20
asselinhemna, https://review.openstack.org/18855719:20
asselinhemna, still need to test it19:20
hemnanice thanks19:20
hemnado I need to post a revert os-brick patch just to prove a point for this patch as well?19:20
hemnamaybe I should post a revert os-brick patch every day19:21
*** Viswanath has quit IRC19:21
hemnauntil some people are happy19:21
*** devious has joined #openstack-cinder19:21
*** rbradbury has quit IRC19:22
*** raunak_ has left #openstack-cinder19:22
*** krtaylor has joined #openstack-cinder19:22
*** merooney has joined #openstack-cinder19:26
*** salv-orlando has joined #openstack-cinder19:26
*** cknight has quit IRC19:27
*** cknight has joined #openstack-cinder19:28
*** akerr has joined #openstack-cinder19:28
*** Lee1092 has quit IRC19:31
jungleboyjhemna: What's up?19:31
openstackgerritScott DAngelo proposed openstack/python-cinderclient: Nova volume-detach --force requires API  https://review.openstack.org/18730519:33
openstackgerritxing-yang proposed openstack/cinder-specs: Non Disruptive Backup  https://review.openstack.org/18689719:35
*** merooney has quit IRC19:38
*** yamada-h has joined #openstack-cinder19:40
*** merooney has joined #openstack-cinder19:43
*** yamada-h has quit IRC19:44
*** dannywilson has quit IRC19:49
*** dannywilson has joined #openstack-cinder19:50
*** ronis has quit IRC19:52
*** devious has quit IRC19:54
*** e0ne has joined #openstack-cinder19:55
*** Viswanath has joined #openstack-cinder19:55
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/18611219:57
*** kvidvans has joined #openstack-cinder19:58
*** Yogi1 has joined #openstack-cinder19:59
*** Viswanath has quit IRC20:00
*** Yogi11 has quit IRC20:01
*** timcl1 has quit IRC20:01
*** adurbin_ has joined #openstack-cinder20:02
*** dannywilson has quit IRC20:02
*** dannywilson has joined #openstack-cinder20:04
*** Viswanath has joined #openstack-cinder20:07
*** Viswanath has quit IRC20:10
*** Apoorva has joined #openstack-cinder20:10
vilobhmmsmcginnis : ping20:16
*** Viswanath has joined #openstack-cinder20:16
vilobhmmIf you have time…a review on this will be helpful…https://review.openstack.org/#/c/185704/20:17
vilobhmmjungleboyj : ^^20:17
jungleboyjOk, need to drop off for a few, but will look.20:17
*** rushil has joined #openstack-cinder20:18
*** Viswanath has quit IRC20:20
vilobhmmjungleboyj: thanks :)20:21
*** jungleboyj has quit IRC20:22
*** angela-s has joined #openstack-cinder20:24
openstackgerritEarle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector  https://review.openstack.org/18658820:24
openstackgerritTom Barron proposed openstack/cinder: Fix remaining memory issues with nfs backup unit tests  https://review.openstack.org/18857720:31
*** julim has quit IRC20:32
*** ociuhandu has quit IRC20:32
*** Yogi1 has quit IRC20:33
openstackgerritEarle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions  https://review.openstack.org/18658020:33
*** Gomeler has joined #openstack-cinder20:37
*** lpetrut has quit IRC20:41
*** markvoelker has quit IRC20:41
thingeehemna: back20:41
*** merooney has quit IRC20:42
hemnaok so I'm trying to figure out if 1) it makes sense to output the new provider_id in volumes for admins and if so 2) make that happen, but only for admins20:43
*** tsekiyama has joined #openstack-cinder20:43
hemnaas a reference :  https://review.openstack.org/#/c/14320520:44
hemnaand for us : https://review.openstack.org/#/c/188236/20:44
hemnafrom what I can tell from how the API works20:44
*** tsbot_ has joined #openstack-cinder20:44
jgriffithhemna: you could just ask me20:44
*** Viswanath has joined #openstack-cinder20:44
hemnajgriffith, ok I didn't see you on, sorry.20:45
jgriffithhemna: and the answer is absolutely doesn't20:45
hemnaso we don't want to show that provider_id for admins even ?20:45
jgriffithhemna: that's called a broken abstraction, showing internal device info to users20:45
hemnaI was hoping we could20:45
jgriffithhemna: why?20:46
hemnait would give the cloud admins a way to associate the cinder volume to the backend volume in their vendor specific, array console20:46
jgriffithhemna: and you "can" mysql -u root cinder; select provider_info from volumes"20:46
hemnafor those of us that have this issue.20:46
hemnasure20:47
hemnayah that's the fallback20:47
jgriffithhemna: you can't do that already?20:47
hemnaI just was wanting to raise this and see if anyone else thinks it has value for the client is all20:47
jgriffithhemna: you don't have metadata, or a "name" for a volume on 3Par?20:47
*** Viswanath has quit IRC20:47
thingeejgriffith: I think he's thinking of those who don't have mysql access, but have cinder admin api access.20:47
jgriffithhemna: I guess if 3par can't "name" volumes that kinda sucks20:47
*** gouthamr has quit IRC20:47
thingeeregardless, I'm not sure I understand why this needs to be exposed.20:47
jgriffiththingee: no... actually I'm saying they should not need that either20:48
hemna3PAR has volume names, that's our unique id, and the name has character limits which prevents us from just using the cinder UUID directly20:48
thingeejgriffith: no arguments there20:48
jgriffithhemna: that's a bummer20:48
jgriffithhemna: be creative :)20:48
hemnayah it is20:48
thingeejgriffith: just trying to guess where this is coming from20:48
hemnawe have already solved that problem though20:48
jgriffiththingee: Yeah20:48
hemnaI created a way to convert the uuid into something that fits on the 3PAR20:48
thingeeDatera folks asked me about this yesterday actually, I couldn't get a reason why though.20:49
hemnabut admins have no way (without the python code) to associated the cinder UUID to the 3PAR volume name20:49
thingeethey shouldn't need to?20:49
hemnaso outputing the provider_id (which is the 3PAR volume name for us in my patch) would let admins see that.20:49
hemnabut like you said20:49
hemnathey can mysql it20:49
jgriffithhemna: so that's not really kosher either20:50
jgriffithhemna: I wasn't understanding you lack the ability to have a general mapping of volumes on 3par20:50
hemnawhat's not ?20:50
jgriffithhemna: saying "access the db"20:50
*** gouthamr has joined #openstack-cinder20:50
jgriffithhemna: write a 3 par extension... or put the tool in that fancy 3par-client thing you have20:51
hemnasure, I'm not advocating that at all, but it's technically a fall back, since there is no way to see the provider_id via the api20:51
hemnasure, so why did we add provider_id then ?20:52
*** merooney has joined #openstack-cinder20:52
*** IlyaG has quit IRC20:54
openstackgerritTom Barron proposed openstack/cinder: Handle ineffective backup compression  https://review.openstack.org/18859120:54
jgriffithhemna: same reason we have things like block-size20:55
jgriffithhemna: internal systems info20:55
*** merooney has quit IRC20:55
thingeedoing a search on provider_id ... we don't really use it20:55
hemnaok that's fine.   I just thought it was worth a discussion to see if it makes sense to others if exposing that to admins is useful.20:55
hemnathingee, yah almost no one uses it20:56
hemnaquobyte I think only20:56
thingeecloudbyte20:56
thingeeone of the bytes ;)20:56
hemnaah yes...*byte :)20:56
*** cknight has quit IRC20:57
hemnajgriffith, ok thanks for the feedback.   I'll shelve it.20:57
jgriffithhemna: it may come up again20:58
jgriffithhemna: I'm sure we'll all talk about it again20:58
jgriffithhemna: IMO until we expose "all" of the provider info I don't want to do "just this"20:58
openstackgerritThang Pham proposed openstack/cinder: WIP: Conversion to volume object  https://review.openstack.org/17705420:59
jgriffithhemna: anywho, if it's a major PITA for you guys we could surely come up with a solution20:59
jgriffiththat doesn't utilize the db query20:59
*** akerr has quit IRC20:59
hemnait's been a small issue for us, but I've just told folks "tough" for now20:59
hemnaheh21:00
jgriffith:)21:00
jgriffithWell... maybe we should all noodle on it21:00
hemnaand then I saw provider_id last night dinking around and thought I could use it to help out21:00
*** thangp has quit IRC21:00
hemnaand figured it could be admin viewable only21:00
*** Gomeler has quit IRC21:00
hemnaas a side note, we pound in admin metadata into the volume metadata as part of the API output for show21:01
hemnawas wondering why we couldn't add an admin flag to the viewbuilder to know if the request had admin privs, to allow output of admin fields or not.21:02
hemnakinda changes the output of the volume details depending on context21:02
jgriffithhemna: Yeah, kinda makes sense... I thought we already did some of that actually21:03
hemnathe viewbuilder doesn't have a flag for that21:03
hemnait always just outputs the same info no matter what.21:04
*** Gomeler has joined #openstack-cinder21:04
hemnaand our API shoves in admin metadata on top of existing metadata for admin contexts.21:04
hemnaanyway, not a big.21:04
scottdathingee: Do cinderclient BPs get a milestone target like cinder BPs?21:04
*** e0ne has quit IRC21:05
*** Gomeler has quit IRC21:05
thingeenope :(21:05
*** Gomeler has joined #openstack-cinder21:05
thingeeI could target to a new release though21:05
thingeethey're independent though21:06
scottdaOK. I've a BP to expose force_detach for the Nova client and I'm wondering how to keep it moving along. Nova side won't land without it: https://blueprints.launchpad.net/python-cinderclient/+spec/nova-force-detach-needs-cinderclient-api21:06
scottdasorry, should be "for Nova" not Nova client21:06
scottdaAnd I think it would have to be in a release, since Nova would need that version in its requirements21:07
*** rushil has quit IRC21:10
tsekiyamathingee: Could you target a BP for python-cinderclient to enable host-attaching of the volumes? This is a requirement for glance-spec of cinder backend to be approved. https://blueprints.launchpad.net/python-cinderclient/+spec/host-attach21:11
*** rushil has joined #openstack-cinder21:11
*** IlyaG has joined #openstack-cinder21:17
thingeexyang1: hi21:18
hemnathingee, jgriffith FYI.  Just wanted to let you know that asselin and I are working on infra, devstack changes to make sure that patches against os-brick kick off volume tests for cinder21:18
thingeexyang1: I think my question was missed here https://review.openstack.org/#/c/183762/21:18
thingeexyang1: where are the xtreme IO ci reports lately?21:18
hemnathingee, jgriffith https://review.openstack.org/#/c/18855721:18
hemnaworking on that.21:18
thingeeoh nevermind21:18
thingeexyang1: found it21:18
hemnathe os-brick patches are now kicking off lib-forward tests as part of their checks21:18
asselini'm peeling away the onion...might need more patches to other repos21:19
jgriffithhemna: https://bugs.launchpad.net/cinder/+bug/146210621:19
openstackLaunchpad bug 1462106 in Cinder "Unit tests belong in cinder/tests/unit/*" [Undecided,New]21:19
jgriffithsmcginnis: ^^21:19
*** eharney has quit IRC21:19
winston-dthingee: ping21:20
smcginnisjgriffith: Doh!21:20
thingeewinston-d: hi21:21
*** tellesnobrega_ has quit IRC21:21
smcginnishemna: Sorry, was out so just read the scrollback.21:21
hemnacrap ok21:21
hemnajgriffith, thanks for pointing that out21:21
smcginnishemna: Someone at the summit was looking for a way to associate LUNs on the host side with cinder volumes.21:21
smcginnisYour question is kinda sorta partly along the same lines.21:21
winston-dthingee: saw your comment regarding to DriverFilter, and you have concern about the change might impact existing users?21:22
thingeesmcginnis: why?21:22
xyang1thingee: Hi, are you all set?21:22
thingeexyang1: yes thanks21:22
smcginnisIf this keeps coming up we may want to think about how to give a little peek into the cloud for admins.21:22
thingeewinston-d: yes21:22
hemnasmcginnis, admins want to be able to see the volume in the array from.21:22
smcginnisJust for cases where somethings not right and they need to dig in to things.21:22
hemnasmcginnis, +121:22
smcginnisAdmin only of course.21:22
hemnawe have customers that have been asking for that for a while21:22
smcginnishemna: It doesn't seem too unreasonable to me.21:23
thingeesmcginnis: I think that's completely up to the vendor.21:23
*** gouthamr has quit IRC21:23
thingeevendors are already doing this today. pick one that does it for you if that's important.21:23
winston-dthingee: there is one case the fix will cause impact, which is the user didn't use CapabilitiesFilter but DriverFilter21:23
smcginnisIt's great to abstract an dhide everything for day to day operations, but when stuff hits the fan you want to be able to poke around under the hood.21:23
smcginnisthingee: True21:23
hemnasmcginnis, +121:23
xyang1thingee: Sorry that we do have some problems with CI lately, we are working on them21:24
thingeewinston-d: correct21:24
smcginnisNot a need for me (yet) but I could see the case for it and this is the second time something similar has been brought up.21:24
thingeexyang1: communication is good if there is planned down time for firmware upgrades.21:24
thingeexyang1: otherwise I take things too personal. it's just who I am :(21:24
*** gouthamr has joined #openstack-cinder21:24
winston-dthingee: and they replies on DriverFilter to filter backends using volume_backend_name21:25
xyang1thingee: Sure, will keep you posted.  No problem:).21:25
thingeexyang1: thank you!21:25
xyang1thingee: Thanks!21:25
winston-dI can't say such case isn't possible but I'd argue most user won't change default filter settings except they know exactly what they are doing21:26
smcginnisjgriffith: You got the unit test bug? Let me know if you want me to fix it. I didn't catch it so I feel responsible.21:26
thingeewinston-d: that's true21:27
thingeewinston-d: don't we have a commit message impact for this...21:27
thingeewinston-d: to make sure it's on the release notes21:27
* thingee checks21:27
thingeeUpgradeImpact21:28
thingeewinston-d: ^21:28
*** rushil has quit IRC21:28
thingeethis is how I found most of the stuff to add in the release notes for Kilo21:28
winston-dok. how about I add UpgradeImpact and detailed explanation to the commit message for the change?21:29
thingeewinston-d: unless others have concerns, sounds fine to me21:29
winston-dthingee: cool, doing that right now. thx!21:30
*** salv-orlando has quit IRC21:35
*** lpabon has quit IRC21:36
*** raunak has joined #openstack-cinder21:40
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Move the dothill tests into unit test dir  https://review.openstack.org/18860121:40
*** salv-orlando has joined #openstack-cinder21:41
*** markvoelker has joined #openstack-cinder21:41
*** yamada-h has joined #openstack-cinder21:41
openstackgerritJohn Griffith proposed openstack/cinder: Move Dothill tests out of root test directory  https://review.openstack.org/18860321:42
*** markvoelker has quit IRC21:46
*** yamada-h has quit IRC21:46
asselinhemna was in first lol ^^21:47
openstackgerritMitsuhiro Tanino proposed openstack/cinder-specs: Efficient volume copy for volume migration  https://review.openstack.org/18620921:48
smcginnisasselin: Should we see how many we can do for the same thing? :-D21:48
asselinbug's assigned to jgriffith so I guess he gets it21:49
* asselin sends an e-mail to CloudFounders.....sigh....21:49
*** mriedem has quit IRC21:51
*** krtaylor has quit IRC21:56
vilobhmmscmcginnis : thanks for the review "+2" from you looks good :)21:56
openstackgerritHuang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name  https://review.openstack.org/18854321:56
winston-dthingee: https://review.openstack.org/188543 commit message updated21:57
openstackgerritEarle F. Philhower, III proposed openstack/cinder: Fix exception parameter name  https://review.openstack.org/18860621:58
jgriffithwinston-d: ping22:01
*** swamireddy1 has joined #openstack-cinder22:05
openstackgerritDanny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver  https://review.openstack.org/18660522:07
*** flip214 has quit IRC22:09
*** swamireddy1 has quit IRC22:09
*** flip214 has joined #openstack-cinder22:10
*** flip214 has quit IRC22:10
*** flip214 has joined #openstack-cinder22:10
*** dannywil_ has joined #openstack-cinder22:10
openstackgerritEarle F. Philhower, III proposed openstack/os-brick: Trivial exception parameter name fix for Huawei  https://review.openstack.org/18861022:12
*** dannywilson has quit IRC22:14
*** dannywil_ has quit IRC22:15
*** IlyaG has quit IRC22:25
*** krtaylor has joined #openstack-cinder22:26
*** primechuck has quit IRC22:27
*** rwsu has quit IRC22:30
*** winston-d has quit IRC22:31
*** winston-d has joined #openstack-cinder22:31
*** dontalton has quit IRC22:35
*** annegentle has joined #openstack-cinder22:35
openstackgerritSean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects  https://review.openstack.org/18853522:40
*** annegentle has quit IRC22:40
*** annegentle has joined #openstack-cinder22:42
*** markvoelker has joined #openstack-cinder22:42
*** markvoelker has quit IRC22:47
*** timcl has joined #openstack-cinder23:02
*** Apoorva has quit IRC23:03
*** Apoorva has joined #openstack-cinder23:04
*** angela-s has quit IRC23:04
*** timcl has quit IRC23:06
*** annegentle has quit IRC23:08
*** akerr has joined #openstack-cinder23:12
*** ian_brown has quit IRC23:19
*** ian_brown has joined #openstack-cinder23:19
*** adurbin_ has quit IRC23:21
*** chlong has joined #openstack-cinder23:27
openstackgerritHuang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name  https://review.openstack.org/18854323:27
*** zhenguo has joined #openstack-cinder23:30
*** leeantho has quit IRC23:31
*** vilobhmm has quit IRC23:38
*** yamada-h has joined #openstack-cinder23:42
*** hemna is now known as hemnafk23:43
*** yamada-h has quit IRC23:47
*** salv-orlando has quit IRC23:53
*** akerr_ has joined #openstack-cinder23:57

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