*** alexpilotti has quit IRC | 00:03 | |
*** alexpilotti has joined #openstack-cinder | 00:05 | |
*** alexpilotti has quit IRC | 00:08 | |
*** hemna is now known as hemnafkj | 00:09 | |
*** xiaohui has joined #openstack-cinder | 00:12 | |
*** asselin_ has joined #openstack-cinder | 00:23 | |
openstackgerrit | John Griffith proposed openstack/cinder: Revert "Remove Brick from cinder codebase" https://review.openstack.org/188220 | 00:27 |
---|---|---|
*** jaypipes has quit IRC | 00:27 | |
patrickeast | wait, why is brick coming back into cinder? | 00:30 |
patrickeast | jgriffith: ^ | 00:30 |
jgriffith | patrickeast: read my revert comment | 00:30 |
patrickeast | but it was accepting commints and we are using it right now…. right? | 00:30 |
jgriffith | patrickeast: it turns out Walt doesn't allow patches to os-brick yet because it's "not ready" doesn't have a CI | 00:30 |
patrickeast | huh | 00:30 |
jgriffith | patrickeast: https://review.openstack.org/#/c/187811/ | 00:31 |
patrickeast | yea, i saw the test stuff earlier today | 00:31 |
winston-d | jgriffith: maybe just for today | 00:31 |
patrickeast | didnt it already merge | 00:31 |
patrickeast | ? | 00:31 |
jgriffith | patrickeast: did which already merge? | 00:31 |
patrickeast | i thought that was like a small hiccup in the process, not a major problem | 00:31 |
*** leeantho has quit IRC | 00:31 | |
patrickeast | this thing | 00:32 |
patrickeast | https://review.openstack.org/#/c/188156/ | 00:32 |
patrickeast | to enable the tests | 00:32 |
asselin_ | that patch is stuck in gate | 00:32 |
winston-d | jgriffith: walt said they are bringing up some 3rd party ci soon | 00:32 |
asselin_ | infra is looking into it | 00:32 |
jgriffith | patrickeast: looks like it | 00:32 |
jgriffith | sigh | 00:32 |
jgriffith | So the revert was more to point out the issue | 00:32 |
jgriffith | Not thinking anybody would actually +2/A it :) | 00:33 |
jgriffith | although... I guess ya never know | 00:33 |
patrickeast | haha gotcha | 00:33 |
*** lixiaoy1 has joined #openstack-cinder | 00:34 | |
jgriffith | patrickeast: and I find it even more annoying that it's not consistent: https://review.openstack.org/#/c/188113/ | 00:34 |
patrickeast | i guess worst case is that the gate comes up and we find out its broken | 00:37 |
patrickeast | which makes sense to block commits until then | 00:37 |
jgriffith | patrickeast: yeah | 00:37 |
jgriffith | patrickeast: makes sense | 00:37 |
patrickeast | but yea, should have been consistent | 00:37 |
patrickeast | i was under the impression requirement updates cause a lot of problems in gate | 00:37 |
jgriffith | patrickeast: indeed they do | 00:37 |
jgriffith | patrickeast: We should've had this to begin with | 00:38 |
jgriffith | but no big deal | 00:38 |
jgriffith | still early enough | 00:38 |
jgriffith | just good it's getting taken care of now | 00:38 |
patrickeast | haha yea, better now than L-4 or something | 00:38 |
jgriffith | hehe! | 00:38 |
*** vilobhmm1 has quit IRC | 00:40 | |
*** breitz1 has quit IRC | 00:45 | |
*** breitz has joined #openstack-cinder | 00:45 | |
*** heyun has joined #openstack-cinder | 00:53 | |
*** takedakn has joined #openstack-cinder | 00:53 | |
*** Apoorva has quit IRC | 00:54 | |
*** takedakn has quit IRC | 00:54 | |
*** takedakn has joined #openstack-cinder | 00:54 | |
*** takedakn has quit IRC | 01:06 | |
*** takedakn has joined #openstack-cinder | 01:06 | |
*** bswartz has quit IRC | 01:07 | |
*** primechuck has joined #openstack-cinder | 01:08 | |
asselin_ | nikeshm, congrats! | 01:11 |
*** dkalleg has quit IRC | 01:12 | |
nikeshm | asselin_ : thanks for all your support,we really appreciate it | 01:14 |
nikeshm | asselin_ : thanks | 01:14 |
nikeshm | :) | 01:14 |
*** openstack has joined #openstack-cinder | 01:23 | |
nikeshm | asselin_: 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 value | 01:25 |
*** bswartz has joined #openstack-cinder | 01:25 | |
nikeshm | i was not able to get its value by script | 01:26 |
asselin_ | hmmm...strange...it should get everything | 01:26 |
nikeshm | yaa that why i am asking did you try | 01:26 |
asselin_ | you can use the command printenv to see all the env variables defined | 01:26 |
*** Lee1092 has joined #openstack-cinder | 01:27 | |
nikeshm | because looks like our environment was facing same issue for /etc/profile.d | 01:27 |
*** openstack has joined #openstack-cinder | 01:38 | |
*** _cjones_ has quit IRC | 01:40 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 01:41 |
*** primechuck has joined #openstack-cinder | 01:45 | |
*** boris-42 has quit IRC | 01:48 | |
*** openstack has quit IRC | 01:52 | |
*** openstack has joined #openstack-cinder | 01:54 | |
*** ianbrown has quit IRC | 01:57 | |
*** ian_brown has joined #openstack-cinder | 01:57 | |
*** bill_az has quit IRC | 02:11 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update https://review.openstack.org/188236 | 02:15 |
*** patrickeast has quit IRC | 02:25 | |
*** gcb has quit IRC | 02:35 | |
*** mriedem_away has quit IRC | 02:39 | |
openstackgerrit | wanghao proposed openstack/cinder: Fix weird change of volume status in re-scheduling https://review.openstack.org/185545 | 02:40 |
openstackgerrit | chenzongliang proposed openstack/cinder: refactor Huawei Volume driver https://review.openstack.org/188240 | 02:44 |
*** asselin_ has joined #openstack-cinder | 02:44 | |
*** julim has quit IRC | 02:47 | |
asselin_ | anteaya, you around? | 03:00 |
*** primechuck has quit IRC | 03:00 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 03:04 |
*** akerr has joined #openstack-cinder | 03:04 | |
*** sgotliv has joined #openstack-cinder | 03:04 | |
*** akerr_ has joined #openstack-cinder | 03:06 | |
*** dims_ has quit IRC | 03:07 | |
*** pegmatite has joined #openstack-cinder | 03:08 | |
*** akerr has quit IRC | 03:09 | |
*** akerr_ has quit IRC | 03:09 | |
*** lixiaoy11 has joined #openstack-cinder | 03:09 | |
openstackgerrit | Diem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver https://review.openstack.org/178319 | 03:10 |
*** lixiaoy12 has joined #openstack-cinder | 03:13 | |
*** lixiaoy11 has quit IRC | 03:15 | |
*** asselin_ has quit IRC | 03:16 | |
*** markvoelker has quit IRC | 03:23 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add retry to iSCSI delete https://review.openstack.org/187811 | 03:25 |
*** pegmatite has left #openstack-cinder | 03:25 | |
*** takedakn has quit IRC | 03:28 | |
*** _cjones_ has joined #openstack-cinder | 03:28 | |
*** _cjones_ has quit IRC | 03:29 | |
*** _cjones_ has joined #openstack-cinder | 03:30 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix disconnecting necessary iSCSI sessions issue https://review.openstack.org/188158 | 03:37 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 03:39 |
*** coolsvap|afk is now known as coolsvap | 03:39 | |
*** hemna has joined #openstack-cinder | 03:40 | |
*** dannywilson has quit IRC | 03:46 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue https://review.openstack.org/188159 | 03:47 |
openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue https://review.openstack.org/188159 | 03:47 |
openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix disconnecting necessary iSCSI sessions issue https://review.openstack.org/188158 | 03:47 |
openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue https://review.openstack.org/188159 | 03:48 |
openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support https://review.openstack.org/188251 | 03:49 |
*** gcb has joined #openstack-cinder | 03:49 | |
*** tobe has joined #openstack-cinder | 03:51 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update https://review.openstack.org/188236 | 03:52 |
*** asselin_ has joined #openstack-cinder | 03:53 | |
*** EmilienM|off has quit IRC | 03:55 | |
*** EmilienM has joined #openstack-cinder | 04:00 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 04:00 |
*** hemna has quit IRC | 04:02 | |
*** pradipta has joined #openstack-cinder | 04:04 | |
*** avishay has joined #openstack-cinder | 04:07 | |
*** dims_ has joined #openstack-cinder | 04:07 | |
*** dims_ has quit IRC | 04:13 | |
*** Longgeek has joined #openstack-cinder | 04:14 | |
*** sc has quit IRC | 04:20 | |
*** sc has joined #openstack-cinder | 04:23 | |
*** markvoelker has joined #openstack-cinder | 04:23 | |
*** lixiaoy12 has quit IRC | 04:25 | |
*** markvoelker has quit IRC | 04:28 | |
*** yamada-h has joined #openstack-cinder | 04:37 | |
*** Longgeek has quit IRC | 04:39 | |
*** rushiagr_away is now known as rushiagr | 04:42 | |
*** marcusvrn has joined #openstack-cinder | 04:43 | |
*** Longgeek has joined #openstack-cinder | 04:53 | |
*** swamireddy1 has joined #openstack-cinder | 04:54 | |
openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support https://review.openstack.org/188251 | 04:57 |
*** Longgeek_ has joined #openstack-cinder | 04:57 | |
*** Longgeek has quit IRC | 05:00 | |
*** yamada-h has quit IRC | 05:04 | |
*** yamada-h has joined #openstack-cinder | 05:04 | |
*** lixiaoy1 has joined #openstack-cinder | 05:05 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 05:08 |
*** yamada-h has quit IRC | 05:09 | |
*** tsekiyama has quit IRC | 05:10 | |
*** xyang has joined #openstack-cinder | 05:19 | |
*** tobe has quit IRC | 05:22 | |
*** deepakcs has joined #openstack-cinder | 05:22 | |
deepakcs | anyone 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 IRC | 05:31 | |
*** BharatK has quit IRC | 05:39 | |
*** tobe has joined #openstack-cinder | 05:40 | |
*** agarciam has joined #openstack-cinder | 05:45 | |
*** marcusvrn has quit IRC | 05:45 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 05:45 |
*** alonmarx has joined #openstack-cinder | 05:47 | |
*** alonmarx_ has quit IRC | 05:49 | |
*** BharatK has joined #openstack-cinder | 05:52 | |
*** belmoreira has joined #openstack-cinder | 05:56 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 05:57 |
*** _cjones_ has quit IRC | 05:57 | |
*** _cjones_ has joined #openstack-cinder | 06:01 | |
*** nlevinki has joined #openstack-cinder | 06:01 | |
*** swamireddy2 has joined #openstack-cinder | 06:02 | |
*** lpetrut has joined #openstack-cinder | 06:03 | |
*** pradipta has quit IRC | 06:04 | |
*** swamireddy1 has quit IRC | 06:05 | |
*** yamada-h has joined #openstack-cinder | 06:11 | |
*** markvoelker has joined #openstack-cinder | 06:13 | |
spradeepv | anyone 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 IRC | 06:18 | |
*** tobe has quit IRC | 06:18 | |
*** alonmarx_ has joined #openstack-cinder | 06:21 | |
*** alonmarx has quit IRC | 06:23 | |
*** jdurgin1 has joined #openstack-cinder | 06:39 | |
*** _cjones_ has quit IRC | 06:44 | |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 06:44 |
*** daneyon_ has quit IRC | 06:50 | |
*** ndipanov has quit IRC | 06:53 | |
*** _cjones_ has joined #openstack-cinder | 06:56 | |
*** tobe has joined #openstack-cinder | 06:57 | |
*** anshul has joined #openstack-cinder | 06:58 | |
*** pradipta has joined #openstack-cinder | 06:59 | |
*** tshefi has joined #openstack-cinder | 07:01 | |
*** _cjones_ has quit IRC | 07:01 | |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 07:05 |
openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 07:06 |
openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 07:10 |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 07:11 |
*** markvoelker has joined #openstack-cinder | 07:13 | |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 07:15 |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 07:17 |
*** vincent_hou has joined #openstack-cinder | 07:17 | |
*** markvoelker has quit IRC | 07:18 | |
vincent_hou | avishay: Ping. I updated the spec https://review.openstack.org/#/c/186327/. Welcome your comments | 07:18 |
avishay | vincent_hou: I replied to patch set 10, welcome your replies :) | 07:19 |
*** aarefiev has joined #openstack-cinder | 07:19 | |
vincent_hou | Thank you. I am changing. | 07:26 |
*** ronis has joined #openstack-cinder | 07:35 | |
openstackgerrit | wanghao proposed openstack/cinder-specs: Incremental backup improvements for L https://review.openstack.org/187926 | 07:35 |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 07:38 |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 07:40 |
*** lpetrut has quit IRC | 07:44 | |
*** dims_ has joined #openstack-cinder | 07:44 | |
*** chlong has quit IRC | 07:44 | |
*** dims_ has quit IRC | 07:49 | |
*** vincent_hou_ has joined #openstack-cinder | 07:49 | |
*** jistr has joined #openstack-cinder | 07:50 | |
*** vincent_hou has quit IRC | 07:50 | |
*** vincent_hou_ is now known as vincent_hou | 07:51 | |
*** jistr is now known as jistr|mt | 07:54 | |
*** jistr|mt is now known as jistr|mtg | 07:54 | |
*** _cjones_ has joined #openstack-cinder | 07:57 | |
*** vincent_hou has quit IRC | 08:01 | |
*** afazekas has joined #openstack-cinder | 08:01 | |
*** _cjones_ has quit IRC | 08:02 | |
*** aix has joined #openstack-cinder | 08:03 | |
*** jdurgin1 has quit IRC | 08:05 | |
*** ndipanov has joined #openstack-cinder | 08:07 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 08:20 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 08:27 |
openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: tests: glusterfs: Remove online snapshot related tests https://review.openstack.org/188300 | 08:27 |
*** jordanP has joined #openstack-cinder | 08:27 | |
*** swamireddy1 has joined #openstack-cinder | 08:33 | |
*** swamireddy2 has quit IRC | 08:36 | |
openstackgerrit | wanghao proposed openstack/cinder-specs: Incremental backup improvements for L https://review.openstack.org/187926 | 08:37 |
*** daneyon has joined #openstack-cinder | 08:38 | |
*** tobe has quit IRC | 08:39 | |
*** tobe has joined #openstack-cinder | 08:39 | |
openstackgerrit | wanghao proposed openstack/cinder-specs: Incremental backup improvements for L https://review.openstack.org/187926 | 08:40 |
*** gcb has quit IRC | 08:41 | |
*** daneyon has quit IRC | 08:43 | |
*** lpetrut has joined #openstack-cinder | 08:46 | |
*** avishay_ has joined #openstack-cinder | 09:01 | |
*** avishay has quit IRC | 09:01 | |
*** markvoelker has joined #openstack-cinder | 09:02 | |
*** ociuhandu has quit IRC | 09:02 | |
*** markvoelker has quit IRC | 09:07 | |
*** jistr|mtg is now known as jistr | 09:19 | |
*** ociuhandu has joined #openstack-cinder | 09:20 | |
*** tobe has quit IRC | 09:33 | |
*** BharatK has quit IRC | 09:38 | |
*** e0ne has joined #openstack-cinder | 09:44 | |
*** afazekas is now known as afazekas_mtg | 09:44 | |
*** dims_ has joined #openstack-cinder | 09:45 | |
*** ociuhandu has quit IRC | 09:46 | |
*** yamada-h has quit IRC | 09:50 | |
openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 09:51 |
*** dims_ has quit IRC | 09:51 | |
*** BharatK has joined #openstack-cinder | 09:51 | |
*** tobe has joined #openstack-cinder | 09:53 | |
*** lixiaoy1 has quit IRC | 09:54 | |
*** _cjones_ has joined #openstack-cinder | 10:00 | |
*** dims_ has joined #openstack-cinder | 10:03 | |
*** _cjones_ has quit IRC | 10:05 | |
*** Zhongjun has joined #openstack-cinder | 10:05 | |
*** yamada-h has joined #openstack-cinder | 10:10 | |
*** yamada-h has quit IRC | 10:10 | |
*** ociuhandu has joined #openstack-cinder | 10:11 | |
*** markvoelker has joined #openstack-cinder | 10:18 | |
*** markvoelker has quit IRC | 10:22 | |
luv | hi. https://git.openstack.org/cgit/openstack/cinder/tree/cinder/utils.py#n60 this surely doesn't look correct, does it? | 10:22 |
luv | Should the function not look for a config file in config directory instead of state_path ?? | 10:23 |
luv | this breaks deploying cinder in virtualenv for me | 10:24 |
luv | http://pastebin.blesmrt.net/pbgdgkmyg | 10:25 |
*** daneyon has joined #openstack-cinder | 10:27 | |
*** coolsvap is now known as coolsvap|afk | 10:30 | |
*** daneyon has quit IRC | 10:32 | |
*** lpetrut1 has joined #openstack-cinder | 10:33 | |
*** lpetrut has quit IRC | 10:33 | |
*** boris-42 has joined #openstack-cinder | 10:33 | |
*** lpetrut1 has left #openstack-cinder | 10:34 | |
*** lpetrut1 has joined #openstack-cinder | 10:34 | |
DuncanT | luv: What do you set for state_path? | 10:38 |
DuncanT | I believe the point is so that you can confine all of the cinder stuff to a sub-tree - most people will run with '/' I believe | 10:38 |
*** ociuhandu has quit IRC | 10:39 | |
luv | DuncanT: /var/lib/cinder/ | 10:40 |
luv | same as other openstack service i always set state_path to /var/lib/service .... I think ubuntu does the same by default ... let me check | 10:41 |
luv | yup, it does. | 10:42 |
luv | state_path = /var/lib/cinder in etc/cinder/cinder.conf in cinder-common package | 10:43 |
deepakcs | DuncanT: 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.html | 10:43 |
*** ociuhandu has joined #openstack-cinder | 10:44 | |
luv | and that's same for upstream cinder as well ... default state_path is /var/lib/cinder | 10:45 |
luv | https://git.openstack.org/cgit/openstack/cinder/tree/cinder/common/config.py#n44 | 10:46 |
*** markvoelker has joined #openstack-cinder | 10:46 | |
*** pradipta has quit IRC | 10:50 | |
*** geguileor has quit IRC | 10:52 | |
*** Longgeek_ has quit IRC | 10:53 | |
*** coolsvap|afk is now known as coolsvap | 10:54 | |
*** Longgeek has joined #openstack-cinder | 10:54 | |
*** deepakcs has quit IRC | 10:56 | |
*** IlyaG has quit IRC | 10:59 | |
*** _cjones_ has joined #openstack-cinder | 11:01 | |
*** _cjones_ has quit IRC | 11:06 | |
*** yamada-h has joined #openstack-cinder | 11:10 | |
*** yamada-h has quit IRC | 11:15 | |
*** coolsvap is now known as coolsvap|afk | 11:16 | |
*** xyang has quit IRC | 11:19 | |
*** timcl has joined #openstack-cinder | 11:26 | |
*** aix has quit IRC | 11:30 | |
*** e0ne is now known as e0ne_ | 11:39 | |
*** heyun has quit IRC | 11:41 | |
*** e0ne_ has quit IRC | 11:49 | |
DuncanT | deepakcs: That is odd | 11:51 |
*** afazekas_mtg has quit IRC | 11:53 | |
openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage Multiple Pools Support https://review.openstack.org/188251 | 11:53 |
*** agarciam has quit IRC | 11:54 | |
*** shyama has joined #openstack-cinder | 12:01 | |
*** _cjones_ has joined #openstack-cinder | 12:02 | |
*** chhavi has joined #openstack-cinder | 12:03 | |
*** avishay_ has quit IRC | 12:05 | |
*** avishay__ has joined #openstack-cinder | 12:05 | |
openstackgerrit | Pradeep Sathasivam proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 12:05 |
openstackgerrit | chenzongliang proposed openstack/cinder: Add Chap support for Huawei Storage https://review.openstack.org/188360 | 12:06 |
*** tobe has quit IRC | 12:06 | |
*** _cjones_ has quit IRC | 12:07 | |
chhavi | winston-d: Want to discuss on https://review.openstack.org/#/c/182980/ | 12:08 |
*** e0ne has joined #openstack-cinder | 12:13 | |
*** avishay__ is now known as avishay | 12:13 | |
openstackgerrit | chenzongliang proposed openstack/cinder: Huawei Storage iSCSI multipath Support https://review.openstack.org/188365 | 12:14 |
*** daneyon has joined #openstack-cinder | 12:16 | |
*** mtanino has joined #openstack-cinder | 12:16 | |
*** chlong has joined #openstack-cinder | 12:20 | |
lpetrut1 | DuncanT: Hi | 12:20 |
lpetrut1 | DuncanT: 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 IRC | 12:21 | |
lpetrut1 | DuncanT: 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 |
lpetrut1 | DuncanT: 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-cinder | 12:23 | |
openstackgerrit | Alexander Maretskiy proposed openstack/cinder: Changes in rally-jobs/README.rst https://review.openstack.org/188371 | 12:24 |
*** aix has joined #openstack-cinder | 12:30 | |
*** rushil has joined #openstack-cinder | 12:31 | |
*** rushil has quit IRC | 12:32 | |
*** yamada-h has joined #openstack-cinder | 12:35 | |
*** rushiagr is now known as rushiagr_away | 12:36 | |
*** rwsu has joined #openstack-cinder | 12:36 | |
openstackgerrit | Rajesh Tailor proposed openstack/cinder: Removed explicit return from __init__ method https://review.openstack.org/188377 | 12:38 |
openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 12:38 |
*** yamada-h has quit IRC | 12:40 | |
*** xyang1 has joined #openstack-cinder | 12:41 | |
*** xyang has joined #openstack-cinder | 12:41 | |
*** fanyaohong has joined #openstack-cinder | 12:44 | |
*** swamireddy2 has joined #openstack-cinder | 12:44 | |
*** afazekas_mtg has joined #openstack-cinder | 12:45 | |
*** BharatK has quit IRC | 12:46 | |
*** swamireddy1 has quit IRC | 12:47 | |
*** xyang has quit IRC | 12:50 | |
*** IanGovett has joined #openstack-cinder | 12:53 | |
*** merooney has joined #openstack-cinder | 12:55 | |
*** smoriya has quit IRC | 12:56 | |
*** delatte has joined #openstack-cinder | 12:58 | |
*** zhenguo has quit IRC | 12:58 | |
*** swamireddy1 has joined #openstack-cinder | 12:59 | |
*** cdelatte has quit IRC | 12:59 | |
*** delattec has quit IRC | 12:59 | |
*** BharatK has joined #openstack-cinder | 13:00 | |
*** delattec has joined #openstack-cinder | 13:00 | |
*** IlyaG has joined #openstack-cinder | 13:01 | |
*** e0ne is now known as e0ne_ | 13:01 | |
*** swamireddy2 has quit IRC | 13:02 | |
*** delatte has quit IRC | 13:02 | |
*** afazekas_mtg has quit IRC | 13:04 | |
*** julim has joined #openstack-cinder | 13:04 | |
*** e0ne_ is now known as e0ne | 13:04 | |
*** deepakcs has joined #openstack-cinder | 13:04 | |
*** IlyaG has quit IRC | 13:05 | |
*** mtanino has quit IRC | 13:05 | |
*** swamireddy1 has quit IRC | 13:07 | |
*** Miouge_ has joined #openstack-cinder | 13:09 | |
*** akshai has joined #openstack-cinder | 13:12 | |
*** Miouge has quit IRC | 13:12 | |
*** Miouge_ is now known as Miouge | 13:12 | |
*** mriedem has joined #openstack-cinder | 13:15 | |
*** nlevinki has quit IRC | 13:17 | |
*** nlevinki has joined #openstack-cinder | 13:19 | |
*** eharney has quit IRC | 13:20 | |
*** Yogi1 has joined #openstack-cinder | 13:20 | |
DuncanT | lpetrut1: 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 issues | 13:23 |
lpetrut1 | DuncanT: sure, thanks! | 13:28 |
*** rushiagr_away is now known as rushiagr | 13:28 | |
*** gouthamr has joined #openstack-cinder | 13:30 | |
*** primechuck has joined #openstack-cinder | 13:32 | |
*** rushil has joined #openstack-cinder | 13:33 | |
*** merooney has quit IRC | 13:34 | |
*** dustins has joined #openstack-cinder | 13:35 | |
*** thangp has joined #openstack-cinder | 13:36 | |
*** jungleboyj has quit IRC | 13:39 | |
*** merooney has joined #openstack-cinder | 13:41 | |
*** merooney has quit IRC | 13:44 | |
*** rushil has quit IRC | 13:45 | |
*** BharatK has quit IRC | 13:47 | |
*** eharney has joined #openstack-cinder | 13:47 | |
*** chhavi has quit IRC | 13:47 | |
deepakcs | thangp: i saw your comment for 172808 | 13:49 |
*** nlevinki has quit IRC | 13:49 | |
thangp | deepakcs: ok | 13:49 |
deepakcs | thangp: 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 |
deepakcs | thangp: did u get a chance to look at the unit test side effect I Cc'ed u on ? | 13:49 |
thangp | deepakcs: might be related to the db change | 13:49 |
deepakcs | thangp: you mean the side effect is bcos of not using objects ? | 13:50 |
thangp | deepakcs: the test cases that fail use self._create_snapshot to create the snapshot db entry, which contains a volume_id attr | 13:51 |
thangp | objects.Snapshot.get_by_id just reads whatever is in the db | 13:52 |
thangp | this is for test_delete_no_dev_fails unit test | 13:52 |
deepakcs | thangp: 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 |
thangp | deepakcs: dont know, did you refresh your virt env? | 13:54 |
deepakcs | thangp: no, but if u doubt it i can delete .venv and try again ? | 13:54 |
thangp | deepakcs: looks like something corrupt in it | 13:54 |
deepakcs | thangp: 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 corrupt | 13:55 |
deepakcs | s/test_delete_no_dev_fails / test_delete_no_dev passes | 13:55 |
thangp | deepakcs: what happens when you dont put in your test case? | 13:55 |
deepakcs | thangp: when i don't put in any testcase as part of my patch.. all works fine | 13:56 |
thangp | deepakcs: which test case in test_remotefs fails when you put it in? | 13:56 |
deepakcs | thangp: for eg: i rename my test_xxx to _test_xxx.. that disables it and it works w/o errors | 13:56 |
deepakcs | thangp: test_create_snapshot_online_success in test_remotefs.py | 13:57 |
deepakcs | thangp: wait | 13:57 |
deepakcs | thangp: when i put a new testcase in test_remotefs.py.it passes, but i see errors from VolumeTestCase test suite | 13:57 |
deepakcs | thangp: i hope you are clear on the side effect ? I explained it in http://lists.openstack.org/pipermail/openstack-dev/2015-June/065801.html | 13:58 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 13:58 |
*** merooney has joined #openstack-cinder | 13:58 | |
*** rushil has joined #openstack-cinder | 13:58 | |
deepakcs | thangp: IOW, the testcase that fail weren't even touched by me! | 13:59 |
*** e0ne is now known as e0ne_ | 13:59 | |
deepakcs | thangp: and thats why its strange fail :) | 13:59 |
thangp | deepakcs: ok...try this | 14:01 |
*** nlevinki has joined #openstack-cinder | 14:01 | |
thangp | deepakcs: 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#L3033 | 14:02 |
*** BharatK has joined #openstack-cinder | 14:02 | |
thangp | actually put - snapshot_obj = objects.Snapshot.get_by_id(context.get_admin_context(), snapshot_id) | 14:02 |
thangp | and delete the db_snapshot_get | 14:03 |
deepakcs | thangp: ok, so basicaly u r saying move all db. calls to objects. calls ? | 14:03 |
thangp | yup, in that unit test | 14:03 |
deepakcs | in test_volume.py ? | 14:03 |
thangp | deepakcs: no, only that unit test | 14:03 |
deepakcs | thangp: ok, my env is not up, so can't do right now, will note this down and get back. | 14:04 |
thangp | deepakcs: because you said you narrowed it down to it | 14:04 |
*** _cjones_ has joined #openstack-cinder | 14:04 | |
*** e0ne_ has quit IRC | 14:04 | |
deepakcs | thangp: no, i narrowed it down to <remotefs unit test followed by volumetestcase tests> fail | 14:04 |
deepakcs | thangp: in VolumeTestCase there are 21 testcases failing with the same error :) | 14:04 |
deepakcs | thangp: let me explain in brief, so u r clear... | 14:05 |
deepakcs | thangp: when i run VolumeTestCase tests, all by themselves , they all run fine | 14:05 |
*** daneyon has joined #openstack-cinder | 14:05 | |
deepakcs | thangp: when i run the entire test suite (./run_tests.sh -N) 21 failures in volumetestcase suite | 14:05 |
*** e0ne has joined #openstack-cinder | 14:06 | |
*** nlevinki has quit IRC | 14:06 | |
deepakcs | thangp: so eventually i figured that my testcase when added in test_remotefs.py is causing those 21 failures in volumetestcase test suite | 14:06 |
deepakcs | thangp: and i have no idea why my testcase in a diff test suite has anything to do with volumetestcase tests! | 14:06 |
deepakcs | thangp: in short, there are 21 testcase failures all in volumetestcase test class | 14:07 |
deepakcs | thangp: http://logs.openstack.org/08/172808/6/check/gate-cinder-python27/daea7cd/console.html (grep for 'Captured traceback') | 14:08 |
*** _cjones_ has quit IRC | 14:08 | |
*** daneyon has quit IRC | 14:10 | |
*** nlevinki has joined #openstack-cinder | 14:10 | |
*** rmesta has joined #openstack-cinder | 14:10 | |
*** rushil has quit IRC | 14:10 | |
*** rushil has joined #openstack-cinder | 14:12 | |
deepakcs | thangp: there ? :) | 14:12 |
*** merooney has quit IRC | 14:13 | |
thangp | deepakcs: i am pulling down your patch and testing it myself | 14:13 |
thangp | deepakcs: i think there was some kind of side effect that your patch made | 14:13 |
thangp | deepakcs: because i have not seen this problem with other patches | 14:14 |
*** IanGovett has quit IRC | 14:14 | |
*** mtanino has joined #openstack-cinder | 14:16 | |
deepakcs | thangp: yes, and i hope you shoudln't have issues applying it, since i had rebased it recently | 14:16 |
*** bswartz has quit IRC | 14:16 | |
deepakcs | thangp: let me know here or best is to reply on top of the -dev list thread, that ways it could help others too | 14:16 |
*** IlyaG has joined #openstack-cinder | 14:17 | |
deepakcs | thangp: 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-cinder | 14:17 | |
deepakcs | thangp: 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.py | 14:17 |
deepakcs | thangp: so TIA and appreciate your help in resolving this | 14:18 |
*** bswartz has joined #openstack-cinder | 14:18 | |
deepakcs | thangp++ :) | 14:18 |
*** merooney has joined #openstack-cinder | 14:18 | |
*** dulek has joined #openstack-cinder | 14:19 | |
*** jungleboyj has joined #openstack-cinder | 14:20 | |
*** mtanino_ has joined #openstack-cinder | 14:21 | |
*** IlyaG has quit IRC | 14:22 | |
*** mtanino has quit IRC | 14:23 | |
lpetrut1 | DuncanT: 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-cinder | 14:25 | |
*** eharney_ has joined #openstack-cinder | 14:25 | |
*** eharney has quit IRC | 14:26 | |
*** eharney_ is now known as eharney | 14:26 | |
*** dustins has quit IRC | 14:28 | |
*** tshefi has quit IRC | 14:29 | |
*** julim has quit IRC | 14:29 | |
*** dulek has quit IRC | 14:32 | |
*** dulek_ has joined #openstack-cinder | 14:32 | |
*** dulek_ has quit IRC | 14:32 | |
*** dulek_ has joined #openstack-cinder | 14:32 | |
*** eharney has quit IRC | 14:32 | |
*** eharney has joined #openstack-cinder | 14:34 | |
*** afazekas_mtg has joined #openstack-cinder | 14:35 | |
*** yamada-h has joined #openstack-cinder | 14:36 | |
*** mtanino_ has quit IRC | 14:39 | |
*** yamada-h has quit IRC | 14:41 | |
*** mtanino has joined #openstack-cinder | 14:43 | |
*** afazekas_mtg has quit IRC | 14:43 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 14:43 |
*** rajinir_r has quit IRC | 14:45 | |
*** rajinir_r has joined #openstack-cinder | 14:46 | |
*** gouthamr has quit IRC | 14:46 | |
*** ganso_ has joined #openstack-cinder | 14:46 | |
openstackgerrit | Alexander Maretskiy proposed openstack/cinder: Changes in rally-jobs/README.rst https://review.openstack.org/188371 | 14:46 |
*** ronis has quit IRC | 14:47 | |
*** chlong has quit IRC | 14:47 | |
*** markstur has quit IRC | 14:51 | |
*** fanyaohong has quit IRC | 14:53 | |
*** deepakcs has quit IRC | 14:53 | |
*** nlevinki has quit IRC | 14:54 | |
*** markstur has joined #openstack-cinder | 14:55 | |
openstackgerrit | rakesh mishra proposed openstack/cinder: set-max-size-limit-for-volume https://review.openstack.org/185906 | 14:57 |
*** lpabon has joined #openstack-cinder | 15:00 | |
*** belmoreira has quit IRC | 15:05 | |
*** _cjones_ has joined #openstack-cinder | 15:05 | |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 15:06 |
*** hemnafkj is now known as hemna | 15:06 | |
luv | DuncanT: really looks like a bug to many | 15:08 |
*** julim has joined #openstack-cinder | 15:08 | |
luv | probably affecting just *very* few deployments in non standard location | 15:08 |
*** dustins_ has quit IRC | 15:10 | |
*** _cjones_ has quit IRC | 15:10 | |
openstackgerrit | Vincent Hou proposed openstack/cinder-specs: Volume migration improvement for L https://review.openstack.org/186327 | 15:11 |
*** alexpilotti has joined #openstack-cinder | 15:11 | |
*** merooney has quit IRC | 15:12 | |
*** annegentle has joined #openstack-cinder | 15:13 | |
*** eharney has quit IRC | 15:13 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls https://review.openstack.org/182396 | 15:14 |
*** daneyon has joined #openstack-cinder | 15:16 | |
*** BharatK has quit IRC | 15:19 | |
*** dannywilson has joined #openstack-cinder | 15:21 | |
*** dustins has joined #openstack-cinder | 15:21 | |
*** jdurgin1 has joined #openstack-cinder | 15:23 | |
*** eharney has joined #openstack-cinder | 15:25 | |
*** dustins has quit IRC | 15:25 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Don't send heartbeats if Manager reports a problem https://review.openstack.org/187670 | 15:27 |
*** crose has joined #openstack-cinder | 15:31 | |
*** dulek_ has quit IRC | 15:32 | |
*** IlyaG has joined #openstack-cinder | 15:33 | |
openstackgerrit | Jordan Pittier proposed openstack/cinder: Scality SOFS: enhance how the remoteFS mount is detected https://review.openstack.org/188442 | 15:35 |
*** IlyaG has quit IRC | 15:37 | |
*** _cjones_ has joined #openstack-cinder | 15:38 | |
*** timcl has quit IRC | 15:39 | |
*** agarciam has joined #openstack-cinder | 15:39 | |
*** dims_ has quit IRC | 15:43 | |
*** jordanP has quit IRC | 15:45 | |
openstackgerrit | Danny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver https://review.openstack.org/186605 | 15:45 |
*** lpetrut1 has quit IRC | 15:49 | |
thingee | xyang1, e0ne https://etherpad.openstack.org/p/cinder-liberty-drivers | 15:49 |
e0ne | thingee: thanks! | 15:50 |
openstackgerrit | Thang Pham proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 15:51 |
xyang1 | thingee: Thanks | 15:51 |
*** lprice has joined #openstack-cinder | 15:52 | |
*** lprice has left #openstack-cinder | 15:53 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 15:55 |
kvidvans | thingee: 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 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Return provider_id in 3PAR volume model update https://review.openstack.org/188236 | 15:58 |
dannywilson | thingee: thanks, just fixed the latest issue in Pure FC and updated etherpad | 15:58 |
*** jistr has quit IRC | 16:01 | |
*** agarciam has quit IRC | 16:02 | |
*** julim has quit IRC | 16:03 | |
*** julim has joined #openstack-cinder | 16:05 | |
*** annegentle has quit IRC | 16:08 | |
*** jaypipes has joined #openstack-cinder | 16:08 | |
*** rmesta has quit IRC | 16:13 | |
*** rmesta has joined #openstack-cinder | 16:13 | |
*** rmesta has quit IRC | 16:14 | |
*** rmesta has joined #openstack-cinder | 16:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 16:17 |
*** jordanP has joined #openstack-cinder | 16:18 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Handle incorrect '--config-dir' param https://review.openstack.org/157749 | 16:18 |
*** rasoto has joined #openstack-cinder | 16:19 | |
*** jordanP has quit IRC | 16:19 | |
*** Apoorva has joined #openstack-cinder | 16:19 | |
thingee | kvidvans: yes. purely for tracking purposes | 16:21 |
thingee | no details needed, just driver name and link to the review | 16:21 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/188490 | 16:24 |
*** jcru has joined #openstack-cinder | 16:31 | |
*** ronis has joined #openstack-cinder | 16:32 | |
*** avishay has quit IRC | 16:32 | |
*** aix has quit IRC | 16:33 | |
openstackgerrit | Thang Pham proposed openstack/cinder: Complete switch to snapshot objects https://review.openstack.org/163910 | 16:35 |
openstackgerrit | Thang Pham proposed openstack/cinder: Prophetstor driver needs to return snapshot objects https://review.openstack.org/185495 | 16:35 |
*** yamada-h has joined #openstack-cinder | 16:37 | |
*** breitz has quit IRC | 16:37 | |
*** breitz has joined #openstack-cinder | 16:38 | |
*** lpetrut has joined #openstack-cinder | 16:38 | |
*** dims_ has joined #openstack-cinder | 16:39 | |
*** merooney has joined #openstack-cinder | 16:42 | |
*** timcl has joined #openstack-cinder | 16:42 | |
*** yamada-h has quit IRC | 16:42 | |
*** gouthamr has joined #openstack-cinder | 16:43 | |
*** gouthamr has left #openstack-cinder | 16:43 | |
*** gouthamr has joined #openstack-cinder | 16:44 | |
*** jungleboyj has quit IRC | 16:46 | |
*** IlyaG has joined #openstack-cinder | 16:49 | |
*** asselin_ has quit IRC | 16:49 | |
*** rushil has quit IRC | 16:51 | |
*** IlyaG has quit IRC | 16:53 | |
*** swamireddy1 has joined #openstack-cinder | 16:54 | |
*** rushil has joined #openstack-cinder | 16:57 | |
*** merooney has quit IRC | 17:01 | |
*** vilobhmm has joined #openstack-cinder | 17:03 | |
*** swamireddy1 has quit IRC | 17:04 | |
*** Yogi11 has joined #openstack-cinder | 17:06 | |
*** rushil has quit IRC | 17:06 | |
*** merooney has joined #openstack-cinder | 17:06 | |
*** Yogi1 has quit IRC | 17:09 | |
*** rushil has joined #openstack-cinder | 17:10 | |
openstackgerrit | Jessy Lee proposed openstack/cinder: Add volume drivers for Infortrend Storage https://review.openstack.org/177113 | 17:11 |
kvidvans | thingee: will file a blueprint. Thanks | 17:16 |
*** pradipta has joined #openstack-cinder | 17:17 | |
*** haomaiwang has joined #openstack-cinder | 17:19 | |
*** haomaiw__ has quit IRC | 17:20 | |
*** e0ne has quit IRC | 17:22 | |
*** dan has quit IRC | 17:23 | |
*** Viswanath has joined #openstack-cinder | 17:25 | |
*** daneyon_ has joined #openstack-cinder | 17:26 | |
*** shyama has quit IRC | 17:27 | |
*** rbradbury has joined #openstack-cinder | 17:28 | |
*** daneyon has quit IRC | 17:29 | |
*** Viswanath has quit IRC | 17:30 | |
*** jungleboyj has joined #openstack-cinder | 17:30 | |
*** ndipanov has quit IRC | 17:30 | |
*** anshul has quit IRC | 17:33 | |
*** delattec has quit IRC | 17:34 | |
*** cdelatte has joined #openstack-cinder | 17:34 | |
*** haomaiwa_ has joined #openstack-cinder | 17:39 | |
*** cdelatte has quit IRC | 17:39 | |
*** dims_ has quit IRC | 17:40 | |
*** haomaiwang has quit IRC | 17:41 | |
*** IlyaG has joined #openstack-cinder | 17:47 | |
*** dontalton has joined #openstack-cinder | 17:47 | |
*** patrickeast has joined #openstack-cinder | 17:53 | |
*** thangp has quit IRC | 17:55 | |
*** thangp has joined #openstack-cinder | 17:55 | |
*** rushil has quit IRC | 17:57 | |
*** crose has quit IRC | 17:57 | |
*** rushiagr is now known as rushiagr_away | 17:57 | |
*** merooney has quit IRC | 17:57 | |
*** merooney has joined #openstack-cinder | 18:00 | |
*** annegentle has joined #openstack-cinder | 18:01 | |
*** Viswanath has joined #openstack-cinder | 18:01 | |
*** barra204 has joined #openstack-cinder | 18:01 | |
anteaya | asselin: sorry I wasn't around earlier, do you still wish for IBM GPFS CI to be disabled? | 18:02 |
asselin | anteaya, already done | 18:02 |
anteaya | oh good thank you | 18:02 |
anteaya | nice work | 18:02 |
patrickeast | anteaya: asselin: i sent them an email a couple days ago, it sounds like they were at least looking into the issues | 18:03 |
patrickeast | jungleboyj: ^ | 18:03 |
anteaya | patrickeast: ah okay well thanks for trying to get a hold of them | 18:03 |
anteaya | I have concerns if after a few days they still can't figure out how to turn their system off | 18:04 |
asselin | yea 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 |
anteaya | which is entirely different than fixing it | 18: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=24 | 18:04 |
anteaya | asselin: agreed | 18:04 |
asselin | also they have 2 accounts, which could be causing the issue..... | 18:04 |
*** ganso_ has quit IRC | 18:04 | |
*** Viswanath has quit IRC | 18:04 | |
anteaya | asselin: well 2 accounts doesn't help | 18:04 |
asselin | both were disabled | 18:04 |
*** kvidvans has quit IRC | 18:04 | |
anteaya | asselin: thanks | 18:04 |
patrickeast | ouch, yea thats not good if they couldn’t get it disabled | 18:04 |
anteaya | patrickeast: yes, that is my concern | 18:05 |
anteaya | asselin patrickeast thank you both for trying to help them and also for following up | 18:05 |
asselin | yeah...cloudfounders is next | 18:06 |
*** tellesnobrega_ has joined #openstack-cinder | 18:06 | |
anteaya | go clean them up | 18:06 |
anteaya | and thank you | 18:06 |
*** barra204 has quit IRC | 18:07 | |
dannywilson | asselin: 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 |
anteaya | and the reason that a system may still comment after we disabled them is that gerrit has to wait for the connection to expire | 18:07 |
anteaya | unless we restart gerrit | 18:07 |
patrickeast | asselin: heh yea, should poke them, they haven’t voted correctly in a long time (assuming at some point they *did*) | 18:07 |
anteaya | in 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 it | 18:07 |
anteaya | but we don't have that yet | 18:07 |
asselin | anteaya, ok...fungi flushed the cache...but I guess it's not 100% | 18:08 |
jungleboyj | anteaya: patrickeast and asselin My deepest apologies. I am dealing with that. | 18:08 |
patrickeast | anteaya: that would be good to have | 18:08 |
anteaya | asselin: no not the same | 18:08 |
jungleboyj | I have been sending them e-mail since 5/22 about this. | 18:08 |
anteaya | jungleboyj: I feel so sorry for you that you don't have better support | 18:08 |
jungleboyj | No excuse. | 18:08 |
anteaya | jungleboyj: I know if it was up to you it would have been off at the first hint of problem | 18:08 |
*** akerr has quit IRC | 18:08 | |
*** dims_ has joined #openstack-cinder | 18:08 | |
asselin | dannywilson, it's not excatly the one I use...but quite close | 18:08 |
anteaya | patrickeast: I'm looking forward to it | 18:08 |
jungleboyj | Well, 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 |
asselin | dannywilson, what issue do you have? | 18:09 |
anteaya | jungleboyj: yup that is usually how a mess like this happens | 18:09 |
anteaya | patrickeast: where is the code for your dashboard again? | 18:09 |
dannywilson | asselin: timeout: failed to run command ‘/opt/stack/new/devstack-gate/devstack-vm-gate.sh’: No such file or directory | 18:09 |
*** haomai___ has joined #openstack-cinder | 18:09 | |
patrickeast | anteaya: now it lives in https://github.com/stackforge/third-party-ci-tools/tree/master/monitoring/scoreboard | 18:09 |
asselin | dannywilson, can you provide more context? Is this when you build an image with dib? | 18:09 |
*** haomaiwa_ has quit IRC | 18:10 | |
anteaya | patrickeast: great thank you | 18:10 |
dannywilson | asselin: image is already built and uploaded, trying to execute a build from sandbox | 18:10 |
asselin | dannywilson, link to logs? | 18:11 |
dannywilson | asselin: http://ec2-54-67-119-204.us-west-1.compute.amazonaws.com/91/188291/1/check/PureISCSIDriver-tempest-dsvm-volume/6858040/ | 18:11 |
asselin | dannywilson, 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.gz | 18:12 |
dannywilson | asselin: indeed, that is weird | 18:13 |
*** merooney has quit IRC | 18:13 | |
jungleboyj | anteaya: asselin I will get it worked out. | 18:13 |
asselin | dannywilson, what os are you building the images on? | 18:13 |
jungleboyj | Thanks for the understanding. | 18:13 |
patrickeast | asselin: its on our jenkins-master, ubuntu 14.04 | 18:13 |
*** leeantho has joined #openstack-cinder | 18:14 | |
*** merooney has joined #openstack-cinder | 18:14 | |
dannywilson | asselin: our nodepool.yaml.erb file is pasted here http://paste.openstack.org/show/263574/ | 18:15 |
asselin | dannywilson, can you launch a nova vm using that image and smoke test it? | 18:15 |
*** akshai has quit IRC | 18:17 | |
dannywilson | asselin: will do | 18:17 |
asselin | dannywilson, line 41 and 46 I don't have... | 18:17 |
*** pradipta has quit IRC | 18:17 | |
asselin | line 41 I assume is your custom stuff | 18:17 |
asselin | and 46 I believe uses that regardless of the setting in nodepool.yaml (depends on nodepool version) | 18:18 |
*** timcl1 has joined #openstack-cinder | 18:18 | |
dannywilson | yes, I made an element, that seems to work correctly but I will check it when I start an instance | 18:18 |
dannywilson | should I take 46 out? | 18:18 |
*** timcl has quit IRC | 18:19 | |
asselin | dannywilson, not at this point.... | 18:19 |
asselin | dannywilson, 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.gz | 18:20 |
*** merooney has quit IRC | 18:20 | |
asselin | we don't have -n /dev/vdb so don't have that issue.... | 18:21 |
*** annegentle has quit IRC | 18:21 | |
*** annegentle has joined #openstack-cinder | 18:22 | |
*** merooney has joined #openstack-cinder | 18:24 | |
*** tellesnobrega_ has quit IRC | 18:29 | |
*** tellesnobrega_ has joined #openstack-cinder | 18:29 | |
*** tellesnobrega_ has quit IRC | 18:29 | |
*** jcru has quit IRC | 18:30 | |
*** merooney has quit IRC | 18:31 | |
*** merooney has joined #openstack-cinder | 18:31 | |
*** tellesnobrega_ has joined #openstack-cinder | 18:32 | |
*** tellesnobrega_ has quit IRC | 18:32 | |
hemna | thingee, ping | 18:38 |
openstackgerrit | Huang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name https://review.openstack.org/188543 | 18:38 |
*** openstackstatus has joined #openstack-cinder | 18:39 | |
*** ChanServ sets mode: +v openstackstatus | 18:39 | |
hemna | jungleboyj, smcginnis ping | 18:39 |
*** Zhongjun has quit IRC | 18:39 | |
*** tellesnobrega_ has joined #openstack-cinder | 18: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 IRC | 18:43 | |
*** raunak has joined #openstack-cinder | 18:44 | |
*** tellesnobrega_ has joined #openstack-cinder | 18:45 | |
*** raunak_ has joined #openstack-cinder | 18:48 | |
smcginnis | hemna: pong | 18:51 |
*** tsbot_ has quit IRC | 18:51 | |
*** raunak has left #openstack-cinder | 18:52 | |
smcginnis | hemna: Request timed out. :-P | 18:54 |
*** krtaylor has quit IRC | 18:59 | |
*** dannywilson has quit IRC | 19:01 | |
*** dannywilson has joined #openstack-cinder | 19:03 | |
*** patrickeast_ has joined #openstack-cinder | 19:04 | |
*** Apoorva has quit IRC | 19:04 | |
*** patrickeast has quit IRC | 19:06 | |
*** patrickeast_ is now known as patrickeast | 19:06 | |
*** annegentle has quit IRC | 19:07 | |
hemna | hey | 19:07 |
hemna | smcginnis, so https://review.openstack.org/#/c/143205 that landed it looks like in K. | 19:07 |
hemna | which is a way for drivers to put their backend volume id associated with the cinder volume id in the volumes table. | 19:07 |
hemna | smcginnis, https://review.openstack.org/#/c/188236/ I have that one in flight that adds our volume name to the provider_id | 19:08 |
hemna | but we aren't showing the provider_id from the client | 19:08 |
hemna | cinder show <id> | 19:08 |
hemna | so I started digging into the API and the view builder doesn't have a way to distinguish admin vs non admin | 19:09 |
hemna | when it builds the volume dict to return in the API call | 19:09 |
hemna | we are kinda hacking in the volume admin metadata into the metadata | 19:09 |
hemna | so that rides for free in the metadata dict in the return | 19:09 |
hemna | kinda seems like the viewbuilder should have a way to filter out admin context or not and show data based upon context | 19:12 |
*** Longgeek has quit IRC | 19:12 | |
hemna | which it doesn't | 19:12 |
hemna | was hoping thingee could chime in as well | 19:12 |
*** merooney has quit IRC | 19:13 | |
thingee | hemna: meetings meetings atm | 19:13 |
thingee | hemna: https://twitter.com/Thingee/status/606501749382209536 | 19:13 |
hemna | haha | 19:18 |
hemna | awesome | 19:18 |
*** Viswanath has joined #openstack-cinder | 19:18 | |
hemna | planning to plan! | 19:18 |
*** rasoto has quit IRC | 19:20 | |
asselin | hemna, https://review.openstack.org/188557 | 19:20 |
asselin | hemna, still need to test it | 19:20 |
hemna | nice thanks | 19:20 |
hemna | do I need to post a revert os-brick patch just to prove a point for this patch as well? | 19:20 |
hemna | maybe I should post a revert os-brick patch every day | 19:21 |
*** Viswanath has quit IRC | 19:21 | |
hemna | until some people are happy | 19:21 |
*** devious has joined #openstack-cinder | 19:21 | |
*** rbradbury has quit IRC | 19:22 | |
*** raunak_ has left #openstack-cinder | 19:22 | |
*** krtaylor has joined #openstack-cinder | 19:22 | |
*** merooney has joined #openstack-cinder | 19:26 | |
*** salv-orlando has joined #openstack-cinder | 19:26 | |
*** cknight has quit IRC | 19:27 | |
*** cknight has joined #openstack-cinder | 19:28 | |
*** akerr has joined #openstack-cinder | 19:28 | |
*** Lee1092 has quit IRC | 19:31 | |
jungleboyj | hemna: What's up? | 19:31 |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Nova volume-detach --force requires API https://review.openstack.org/187305 | 19:33 |
openstackgerrit | xing-yang proposed openstack/cinder-specs: Non Disruptive Backup https://review.openstack.org/186897 | 19:35 |
*** merooney has quit IRC | 19:38 | |
*** yamada-h has joined #openstack-cinder | 19:40 | |
*** merooney has joined #openstack-cinder | 19:43 | |
*** yamada-h has quit IRC | 19:44 | |
*** dannywilson has quit IRC | 19:49 | |
*** dannywilson has joined #openstack-cinder | 19:50 | |
*** ronis has quit IRC | 19:52 | |
*** devious has quit IRC | 19:54 | |
*** e0ne has joined #openstack-cinder | 19:55 | |
*** Viswanath has joined #openstack-cinder | 19:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 19:57 |
*** kvidvans has joined #openstack-cinder | 19:58 | |
*** Yogi1 has joined #openstack-cinder | 19:59 | |
*** Viswanath has quit IRC | 20:00 | |
*** Yogi11 has quit IRC | 20:01 | |
*** timcl1 has quit IRC | 20:01 | |
*** adurbin_ has joined #openstack-cinder | 20:02 | |
*** dannywilson has quit IRC | 20:02 | |
*** dannywilson has joined #openstack-cinder | 20:04 | |
*** Viswanath has joined #openstack-cinder | 20:07 | |
*** Viswanath has quit IRC | 20:10 | |
*** Apoorva has joined #openstack-cinder | 20:10 | |
vilobhmm | smcginnis : ping | 20:16 |
*** Viswanath has joined #openstack-cinder | 20:16 | |
vilobhmm | If you have time…a review on this will be helpful…https://review.openstack.org/#/c/185704/ | 20:17 |
vilobhmm | jungleboyj : ^^ | 20:17 |
jungleboyj | Ok, need to drop off for a few, but will look. | 20:17 |
*** rushil has joined #openstack-cinder | 20:18 | |
*** Viswanath has quit IRC | 20:20 | |
vilobhmm | jungleboyj: thanks :) | 20:21 |
*** jungleboyj has quit IRC | 20:22 | |
*** angela-s has joined #openstack-cinder | 20:24 | |
openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 20:24 |
openstackgerrit | Tom Barron proposed openstack/cinder: Fix remaining memory issues with nfs backup unit tests https://review.openstack.org/188577 | 20:31 |
*** julim has quit IRC | 20:32 | |
*** ociuhandu has quit IRC | 20:32 | |
*** Yogi1 has quit IRC | 20:33 | |
openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Implement Cinder Volume driver for HGST Solutions https://review.openstack.org/186580 | 20:33 |
*** Gomeler has joined #openstack-cinder | 20:37 | |
*** lpetrut has quit IRC | 20:41 | |
*** markvoelker has quit IRC | 20:41 | |
thingee | hemna: back | 20:41 |
*** merooney has quit IRC | 20:42 | |
hemna | ok 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 admins | 20:43 |
*** tsekiyama has joined #openstack-cinder | 20:43 | |
hemna | as a reference : https://review.openstack.org/#/c/143205 | 20:44 |
hemna | and for us : https://review.openstack.org/#/c/188236/ | 20:44 |
hemna | from what I can tell from how the API works | 20:44 |
*** tsbot_ has joined #openstack-cinder | 20:44 | |
jgriffith | hemna: you could just ask me | 20:44 |
*** Viswanath has joined #openstack-cinder | 20:44 | |
hemna | jgriffith, ok I didn't see you on, sorry. | 20:45 |
jgriffith | hemna: and the answer is absolutely doesn't | 20:45 |
hemna | so we don't want to show that provider_id for admins even ? | 20:45 |
jgriffith | hemna: that's called a broken abstraction, showing internal device info to users | 20:45 |
hemna | I was hoping we could | 20:45 |
jgriffith | hemna: why? | 20:46 |
hemna | it would give the cloud admins a way to associate the cinder volume to the backend volume in their vendor specific, array console | 20:46 |
jgriffith | hemna: and you "can" mysql -u root cinder; select provider_info from volumes" | 20:46 |
hemna | for those of us that have this issue. | 20:46 |
hemna | sure | 20:47 |
hemna | yah that's the fallback | 20:47 |
jgriffith | hemna: you can't do that already? | 20:47 |
hemna | I just was wanting to raise this and see if anyone else thinks it has value for the client is all | 20:47 |
jgriffith | hemna: you don't have metadata, or a "name" for a volume on 3Par? | 20:47 |
*** Viswanath has quit IRC | 20:47 | |
thingee | jgriffith: I think he's thinking of those who don't have mysql access, but have cinder admin api access. | 20:47 |
jgriffith | hemna: I guess if 3par can't "name" volumes that kinda sucks | 20:47 |
*** gouthamr has quit IRC | 20:47 | |
thingee | regardless, I'm not sure I understand why this needs to be exposed. | 20:47 |
jgriffith | thingee: no... actually I'm saying they should not need that either | 20:48 |
hemna | 3PAR has volume names, that's our unique id, and the name has character limits which prevents us from just using the cinder UUID directly | 20:48 |
thingee | jgriffith: no arguments there | 20:48 |
jgriffith | hemna: that's a bummer | 20:48 |
jgriffith | hemna: be creative :) | 20:48 |
hemna | yah it is | 20:48 |
thingee | jgriffith: just trying to guess where this is coming from | 20:48 |
hemna | we have already solved that problem though | 20:48 |
jgriffith | thingee: Yeah | 20:48 |
hemna | I created a way to convert the uuid into something that fits on the 3PAR | 20:48 |
thingee | Datera folks asked me about this yesterday actually, I couldn't get a reason why though. | 20:49 |
hemna | but admins have no way (without the python code) to associated the cinder UUID to the 3PAR volume name | 20:49 |
thingee | they shouldn't need to? | 20:49 |
hemna | so outputing the provider_id (which is the 3PAR volume name for us in my patch) would let admins see that. | 20:49 |
hemna | but like you said | 20:49 |
hemna | they can mysql it | 20:49 |
jgriffith | hemna: so that's not really kosher either | 20:50 |
jgriffith | hemna: I wasn't understanding you lack the ability to have a general mapping of volumes on 3par | 20:50 |
hemna | what's not ? | 20:50 |
jgriffith | hemna: saying "access the db" | 20:50 |
*** gouthamr has joined #openstack-cinder | 20:50 | |
jgriffith | hemna: write a 3 par extension... or put the tool in that fancy 3par-client thing you have | 20:51 |
hemna | sure, 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 api | 20:51 |
hemna | sure, so why did we add provider_id then ? | 20:52 |
*** merooney has joined #openstack-cinder | 20:52 | |
*** IlyaG has quit IRC | 20:54 | |
openstackgerrit | Tom Barron proposed openstack/cinder: Handle ineffective backup compression https://review.openstack.org/188591 | 20:54 |
jgriffith | hemna: same reason we have things like block-size | 20:55 |
jgriffith | hemna: internal systems info | 20:55 |
*** merooney has quit IRC | 20:55 | |
thingee | doing a search on provider_id ... we don't really use it | 20:55 |
hemna | ok 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 |
hemna | thingee, yah almost no one uses it | 20:56 |
hemna | quobyte I think only | 20:56 |
thingee | cloudbyte | 20:56 |
thingee | one of the bytes ;) | 20:56 |
hemna | ah yes...*byte :) | 20:56 |
*** cknight has quit IRC | 20:57 | |
hemna | jgriffith, ok thanks for the feedback. I'll shelve it. | 20:57 |
jgriffith | hemna: it may come up again | 20:58 |
jgriffith | hemna: I'm sure we'll all talk about it again | 20:58 |
jgriffith | hemna: IMO until we expose "all" of the provider info I don't want to do "just this" | 20:58 |
openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Conversion to volume object https://review.openstack.org/177054 | 20:59 |
jgriffith | hemna: anywho, if it's a major PITA for you guys we could surely come up with a solution | 20:59 |
jgriffith | that doesn't utilize the db query | 20:59 |
*** akerr has quit IRC | 20:59 | |
hemna | it's been a small issue for us, but I've just told folks "tough" for now | 20:59 |
hemna | heh | 21:00 |
jgriffith | :) | 21:00 |
jgriffith | Well... maybe we should all noodle on it | 21:00 |
hemna | and then I saw provider_id last night dinking around and thought I could use it to help out | 21:00 |
*** thangp has quit IRC | 21:00 | |
hemna | and figured it could be admin viewable only | 21:00 |
*** Gomeler has quit IRC | 21:00 | |
hemna | as a side note, we pound in admin metadata into the volume metadata as part of the API output for show | 21:01 |
hemna | was 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 |
hemna | kinda changes the output of the volume details depending on context | 21:02 |
jgriffith | hemna: Yeah, kinda makes sense... I thought we already did some of that actually | 21:03 |
hemna | the viewbuilder doesn't have a flag for that | 21:03 |
hemna | it always just outputs the same info no matter what. | 21:04 |
*** Gomeler has joined #openstack-cinder | 21:04 | |
hemna | and our API shoves in admin metadata on top of existing metadata for admin contexts. | 21:04 |
hemna | anyway, not a big. | 21:04 |
scottda | thingee: Do cinderclient BPs get a milestone target like cinder BPs? | 21:04 |
*** e0ne has quit IRC | 21:05 | |
*** Gomeler has quit IRC | 21:05 | |
thingee | nope :( | 21:05 |
*** Gomeler has joined #openstack-cinder | 21:05 | |
thingee | I could target to a new release though | 21:05 |
thingee | they're independent though | 21:06 |
scottda | OK. 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-api | 21:06 |
scottda | sorry, should be "for Nova" not Nova client | 21:06 |
scottda | And I think it would have to be in a release, since Nova would need that version in its requirements | 21:07 |
*** rushil has quit IRC | 21:10 | |
tsekiyama | thingee: 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-attach | 21:11 |
*** rushil has joined #openstack-cinder | 21:11 | |
*** IlyaG has joined #openstack-cinder | 21:17 | |
thingee | xyang1: hi | 21:18 |
hemna | thingee, 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 cinder | 21:18 |
thingee | xyang1: I think my question was missed here https://review.openstack.org/#/c/183762/ | 21:18 |
thingee | xyang1: where are the xtreme IO ci reports lately? | 21:18 |
hemna | thingee, jgriffith https://review.openstack.org/#/c/188557 | 21:18 |
hemna | working on that. | 21:18 |
thingee | oh nevermind | 21:18 |
thingee | xyang1: found it | 21:18 |
hemna | the os-brick patches are now kicking off lib-forward tests as part of their checks | 21:18 |
asselin | i'm peeling away the onion...might need more patches to other repos | 21:19 |
jgriffith | hemna: https://bugs.launchpad.net/cinder/+bug/1462106 | 21:19 |
openstack | Launchpad bug 1462106 in Cinder "Unit tests belong in cinder/tests/unit/*" [Undecided,New] | 21:19 |
jgriffith | smcginnis: ^^ | 21:19 |
*** eharney has quit IRC | 21:19 | |
winston-d | thingee: ping | 21:20 |
smcginnis | jgriffith: Doh! | 21:20 |
thingee | winston-d: hi | 21:21 |
*** tellesnobrega_ has quit IRC | 21:21 | |
smcginnis | hemna: Sorry, was out so just read the scrollback. | 21:21 |
hemna | crap ok | 21:21 |
hemna | jgriffith, thanks for pointing that out | 21:21 |
smcginnis | hemna: Someone at the summit was looking for a way to associate LUNs on the host side with cinder volumes. | 21:21 |
smcginnis | Your question is kinda sorta partly along the same lines. | 21:21 |
winston-d | thingee: saw your comment regarding to DriverFilter, and you have concern about the change might impact existing users? | 21:22 |
thingee | smcginnis: why? | 21:22 |
xyang1 | thingee: Hi, are you all set? | 21:22 |
thingee | xyang1: yes thanks | 21:22 |
smcginnis | If this keeps coming up we may want to think about how to give a little peek into the cloud for admins. | 21:22 |
thingee | winston-d: yes | 21:22 |
hemna | smcginnis, admins want to be able to see the volume in the array from. | 21:22 |
smcginnis | Just for cases where somethings not right and they need to dig in to things. | 21:22 |
hemna | smcginnis, +1 | 21:22 |
smcginnis | Admin only of course. | 21:22 |
hemna | we have customers that have been asking for that for a while | 21:22 |
smcginnis | hemna: It doesn't seem too unreasonable to me. | 21:23 |
thingee | smcginnis: I think that's completely up to the vendor. | 21:23 |
*** gouthamr has quit IRC | 21:23 | |
thingee | vendors are already doing this today. pick one that does it for you if that's important. | 21:23 |
winston-d | thingee: there is one case the fix will cause impact, which is the user didn't use CapabilitiesFilter but DriverFilter | 21:23 |
smcginnis | It'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 |
smcginnis | thingee: True | 21:23 |
hemna | smcginnis, +1 | 21:23 |
xyang1 | thingee: Sorry that we do have some problems with CI lately, we are working on them | 21:24 |
thingee | winston-d: correct | 21:24 |
smcginnis | Not 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 |
thingee | xyang1: communication is good if there is planned down time for firmware upgrades. | 21:24 |
thingee | xyang1: otherwise I take things too personal. it's just who I am :( | 21:24 |
*** gouthamr has joined #openstack-cinder | 21:24 | |
winston-d | thingee: and they replies on DriverFilter to filter backends using volume_backend_name | 21:25 |
xyang1 | thingee: Sure, will keep you posted. No problem:). | 21:25 |
thingee | xyang1: thank you! | 21:25 |
xyang1 | thingee: Thanks! | 21:25 |
winston-d | I 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 doing | 21:26 |
smcginnis | jgriffith: 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 |
thingee | winston-d: that's true | 21:27 |
thingee | winston-d: don't we have a commit message impact for this... | 21:27 |
thingee | winston-d: to make sure it's on the release notes | 21:27 |
* thingee checks | 21:27 | |
thingee | UpgradeImpact | 21:28 |
thingee | winston-d: ^ | 21:28 |
*** rushil has quit IRC | 21:28 | |
thingee | this is how I found most of the stuff to add in the release notes for Kilo | 21:28 |
winston-d | ok. how about I add UpgradeImpact and detailed explanation to the commit message for the change? | 21:29 |
thingee | winston-d: unless others have concerns, sounds fine to me | 21:29 |
winston-d | thingee: cool, doing that right now. thx! | 21:30 |
*** salv-orlando has quit IRC | 21:35 | |
*** lpabon has quit IRC | 21:36 | |
*** raunak has joined #openstack-cinder | 21:40 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Move the dothill tests into unit test dir https://review.openstack.org/188601 | 21:40 |
*** salv-orlando has joined #openstack-cinder | 21:41 | |
*** markvoelker has joined #openstack-cinder | 21:41 | |
*** yamada-h has joined #openstack-cinder | 21:41 | |
openstackgerrit | John Griffith proposed openstack/cinder: Move Dothill tests out of root test directory https://review.openstack.org/188603 | 21:42 |
*** markvoelker has quit IRC | 21:46 | |
*** yamada-h has quit IRC | 21:46 | |
asselin | hemna was in first lol ^^ | 21:47 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder-specs: Efficient volume copy for volume migration https://review.openstack.org/186209 | 21:48 |
smcginnis | asselin: Should we see how many we can do for the same thing? :-D | 21:48 |
asselin | bug's assigned to jgriffith so I guess he gets it | 21:49 |
* asselin sends an e-mail to CloudFounders.....sigh.... | 21:49 | |
*** mriedem has quit IRC | 21:51 | |
*** krtaylor has quit IRC | 21:56 | |
vilobhmm | scmcginnis : thanks for the review "+2" from you looks good :) | 21:56 |
openstackgerrit | Huang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name https://review.openstack.org/188543 | 21:56 |
winston-d | thingee: https://review.openstack.org/188543 commit message updated | 21:57 |
openstackgerrit | Earle F. Philhower, III proposed openstack/cinder: Fix exception parameter name https://review.openstack.org/188606 | 21:58 |
jgriffith | winston-d: ping | 22:01 |
*** swamireddy1 has joined #openstack-cinder | 22:05 | |
openstackgerrit | Danny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver https://review.openstack.org/186605 | 22:07 |
*** flip214 has quit IRC | 22:09 | |
*** swamireddy1 has quit IRC | 22:09 | |
*** flip214 has joined #openstack-cinder | 22:10 | |
*** flip214 has quit IRC | 22:10 | |
*** flip214 has joined #openstack-cinder | 22:10 | |
*** dannywil_ has joined #openstack-cinder | 22:10 | |
openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Trivial exception parameter name fix for Huawei https://review.openstack.org/188610 | 22:12 |
*** dannywilson has quit IRC | 22:14 | |
*** dannywil_ has quit IRC | 22:15 | |
*** IlyaG has quit IRC | 22:25 | |
*** krtaylor has joined #openstack-cinder | 22:26 | |
*** primechuck has quit IRC | 22:27 | |
*** rwsu has quit IRC | 22:30 | |
*** winston-d has quit IRC | 22:31 | |
*** winston-d has joined #openstack-cinder | 22:31 | |
*** dontalton has quit IRC | 22:35 | |
*** annegentle has joined #openstack-cinder | 22:35 | |
openstackgerrit | Sean Chen proposed openstack/cinder: Tintri driver to manage existing backend storage objects https://review.openstack.org/188535 | 22:40 |
*** annegentle has quit IRC | 22:40 | |
*** annegentle has joined #openstack-cinder | 22:42 | |
*** markvoelker has joined #openstack-cinder | 22:42 | |
*** markvoelker has quit IRC | 22:47 | |
*** timcl has joined #openstack-cinder | 23:02 | |
*** Apoorva has quit IRC | 23:03 | |
*** Apoorva has joined #openstack-cinder | 23:04 | |
*** angela-s has quit IRC | 23:04 | |
*** timcl has quit IRC | 23:06 | |
*** annegentle has quit IRC | 23:08 | |
*** akerr has joined #openstack-cinder | 23:12 | |
*** ian_brown has quit IRC | 23:19 | |
*** ian_brown has joined #openstack-cinder | 23:19 | |
*** adurbin_ has quit IRC | 23:21 | |
*** chlong has joined #openstack-cinder | 23:27 | |
openstackgerrit | Huang Zhiteng proposed openstack/cinder: DriverFilter: don't check volume_backend_name https://review.openstack.org/188543 | 23:27 |
*** zhenguo has joined #openstack-cinder | 23:30 | |
*** leeantho has quit IRC | 23:31 | |
*** vilobhmm has quit IRC | 23:38 | |
*** yamada-h has joined #openstack-cinder | 23:42 | |
*** hemna is now known as hemnafk | 23:43 | |
*** yamada-h has quit IRC | 23:47 | |
*** salv-orlando has quit IRC | 23:53 | |
*** akerr_ has joined #openstack-cinder | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!