| *** hemna is now known as hemnafk | 00:01 | |
| *** dannywilson has quit IRC | 00:02 | |
| *** shakamunyi has joined #openstack-cinder | 00:13 | |
| *** barra204 has joined #openstack-cinder | 00:13 | |
| *** rbradbury has quit IRC | 00:15 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for DotHill Storage Arrays. https://review.openstack.org/177665 | 00:15 |
|---|---|---|
| *** patrickeast has quit IRC | 00:20 | |
| *** leeantho has quit IRC | 00:20 | |
| *** leeantho has joined #openstack-cinder | 00:20 | |
| *** garthb has quit IRC | 00:23 | |
| *** garthb_ has quit IRC | 00:23 | |
| nikeshm | hemna:added option for http or https | 00:23 |
| *** leeantho has quit IRC | 00:26 | |
| *** zhenguo has joined #openstack-cinder | 00:33 | |
| *** xyang1 has quit IRC | 00:35 | |
| *** Apoorva has quit IRC | 00:36 | |
| *** breitz has quit IRC | 00:45 | |
| *** breitz1 has joined #openstack-cinder | 00:45 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Port ProphetStor driver to use ABCmeta driver model https://review.openstack.org/178992 | 00:49 |
| *** mtanino has quit IRC | 00:56 | |
| *** heyun has joined #openstack-cinder | 00:56 | |
| *** bkopilov has quit IRC | 00:58 | |
| *** tsekiyama has quit IRC | 00:59 | |
| *** cknight has joined #openstack-cinder | 01:02 | |
| *** cknight has quit IRC | 01:02 | |
| *** cknight has joined #openstack-cinder | 01:02 | |
| *** _cjones_ has quit IRC | 01:06 | |
| *** bkopilov has joined #openstack-cinder | 01:08 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:10 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:12 | |
| *** lixiaoy1 has quit IRC | 01:12 | |
| *** superfly_ has joined #openstack-cinder | 01:12 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:14 | |
| *** superfly_ is now known as superflyy | 01:14 | |
| *** lixiaoy11 has quit IRC | 01:16 | |
| *** smoriya has joined #openstack-cinder | 01:18 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:20 | |
| *** vilobhmm1 has joined #openstack-cinder | 01:20 | |
| *** vilobhmm has quit IRC | 01:21 | |
| *** lixiaoy1 has quit IRC | 01:22 | |
| *** barra204 has quit IRC | 01:24 | |
| *** shakamunyi has quit IRC | 01:24 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:24 | |
| *** superflyy has quit IRC | 01:24 | |
| *** Yogi1 has joined #openstack-cinder | 01:25 | |
| *** rmesta has quit IRC | 01:25 | |
| *** lixiaoy11 has quit IRC | 01:26 | |
| *** Apoorva has joined #openstack-cinder | 01:30 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:31 | |
| *** tobe has joined #openstack-cinder | 01:33 | |
| *** lixiaoy1 has quit IRC | 01:34 | |
| *** Lee1092 has joined #openstack-cinder | 01:36 | |
| *** rbradbury has joined #openstack-cinder | 01:39 | |
| *** vilobhmm1 has quit IRC | 01:42 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:42 | |
| *** lixiaoy11 has quit IRC | 01:43 | |
| *** Apoorva has quit IRC | 01:44 | |
| *** cknight has quit IRC | 01:48 | |
| *** Apoorva has joined #openstack-cinder | 01:49 | |
| *** rbradbury has quit IRC | 01:52 | |
| nikeshm | hi | 01:53 |
| nikeshm | my dependency patch is outdated and i want to do some changes in my cinder patch and then want to commit | 01:55 |
| nikeshm | following http://docs.openstack.org/infra/manual/developers.html | 01:55 |
| nikeshm | small doubt | 01:55 |
| nikeshm | i did git review -d $PARENT_CHANGE_NUMBER | 01:56 |
| nikeshm | now shall i checkout to by branch again and then do changes and then git add and git commit and it review -x $CHILD_CHANGE_NUMBER and git review | 01:58 |
| *** vivek_nandavanam has joined #openstack-cinder | 02:06 | |
| *** ianbrown has joined #openstack-cinder | 02:08 | |
| *** rushiagr_away is now known as rushiagr | 02:09 | |
| *** asselin_ has joined #openstack-cinder | 02:15 | |
| *** Apoorva has quit IRC | 02:19 | |
| *** IlyaG has joined #openstack-cinder | 02:20 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for HPMSA Storage Arrays. https://review.openstack.org/187707 | 02:22 |
| *** IanGovett has joined #openstack-cinder | 02:26 | |
| *** haigang has quit IRC | 02:26 | |
| *** haigang has joined #openstack-cinder | 02:27 | |
| *** annegentle has joined #openstack-cinder | 02:28 | |
| *** Zhongjun has joined #openstack-cinder | 02:29 | |
| *** haigang has quit IRC | 02:30 | |
| *** haigang has joined #openstack-cinder | 02:32 | |
| *** haigang has quit IRC | 02:33 | |
| *** haigang has joined #openstack-cinder | 02:34 | |
| *** asselin_ has quit IRC | 02:38 | |
| *** haigang has quit IRC | 02:38 | |
| *** IanGovett has quit IRC | 02:39 | |
| *** rushiagr is now known as rushiagr_away | 02:40 | |
| *** avishay has joined #openstack-cinder | 02:42 | |
| nikeshm | i made it what i ask by this http://paste.openstack.org/show/258155/ | 02:42 |
| *** Yogi1 has quit IRC | 02:43 | |
| *** annegentle has quit IRC | 02:44 | |
| *** avishay_ has joined #openstack-cinder | 02:46 | |
| *** haigang has joined #openstack-cinder | 02:46 | |
| *** IlyaG has quit IRC | 02:47 | |
| *** avishay has quit IRC | 02:47 | |
| *** annegentle has joined #openstack-cinder | 02:51 | |
| *** annegentle has quit IRC | 02:51 | |
| *** dims__ has quit IRC | 02:56 | |
| openstackgerrit | John Griffith proposed openstack/os-brick: Add retry to iSCSI delete https://review.openstack.org/187811 | 03:17 |
| *** avishay_ has quit IRC | 03:19 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Return provider_id in SolidFire model update https://review.openstack.org/187233 | 03:22 |
| *** elonden has joined #openstack-cinder | 03:27 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for Lenovo Storage Arrays. https://review.openstack.org/187853 | 03:38 |
| *** markvoelker has quit IRC | 03:44 | |
| *** lcurtis_ has joined #openstack-cinder | 03:48 | |
| *** Longgeek_ has quit IRC | 03:53 | |
| *** alexpilotti has joined #openstack-cinder | 03:54 | |
| *** Longgeek has joined #openstack-cinder | 03:54 | |
| *** sgotliv has joined #openstack-cinder | 03:55 | |
| *** Longgeek_ has joined #openstack-cinder | 03:57 | |
| *** changbl_ has joined #openstack-cinder | 04:00 | |
| *** Longgeek has quit IRC | 04:00 | |
| *** patrickeast has joined #openstack-cinder | 04:01 | |
| *** Longgeek has joined #openstack-cinder | 04:02 | |
| *** patrickeast has quit IRC | 04:02 | |
| *** patrickeast has joined #openstack-cinder | 04:03 | |
| *** Longgeek_ has quit IRC | 04:04 | |
| *** Longgeek_ has joined #openstack-cinder | 04:08 | |
| *** Longgeek has quit IRC | 04:10 | |
| *** avishay_ has joined #openstack-cinder | 04:10 | |
| *** Longgeek_ has quit IRC | 04:12 | |
| *** avishay_ is now known as avishay | 04:15 | |
| *** Longgeek has joined #openstack-cinder | 04:18 | |
| *** Longgeek has quit IRC | 04:18 | |
| *** Longgeek has joined #openstack-cinder | 04:19 | |
| *** Longgeek has quit IRC | 04:20 | |
| *** Longgeek has joined #openstack-cinder | 04:20 | |
| nikeshm | avishay: git fetch git rebase -i master | 04:21 |
| nikeshm | right | 04:21 |
| avishay | don't need -i | 04:21 |
| avishay | git rebase origin/master | 04:21 |
| *** Longgeek_ has joined #openstack-cinder | 04:23 | |
| *** Longgee__ has joined #openstack-cinder | 04:25 | |
| *** Longgeek has quit IRC | 04:25 | |
| *** Longgeek_ has quit IRC | 04:26 | |
| *** Longgeek has joined #openstack-cinder | 04:26 | |
| *** patrickeast has quit IRC | 04:27 | |
| *** Longgeek_ has joined #openstack-cinder | 04:28 | |
| *** Longge___ has joined #openstack-cinder | 04:29 | |
| *** lixiaoy1 has quit IRC | 04:29 | |
| *** Longgee__ has quit IRC | 04:30 | |
| *** coolsvap|afk is now known as coolsvap | 04:30 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for DotHill Storage Arrays. https://review.openstack.org/177665 | 04:30 |
| *** coolsvap is now known as coolsvap|afk | 04:31 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up unused exceptions https://review.openstack.org/187858 | 04:31 |
| *** Longgeek has quit IRC | 04:32 | |
| *** coolsvap|afk is now known as coolsvap | 04:32 | |
| *** Longgeek_ has quit IRC | 04:32 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 04:34 |
| *** changbl_ has quit IRC | 04:40 | |
| *** krtaylor has quit IRC | 04:40 | |
| *** rbradbury has joined #openstack-cinder | 04:40 | |
| *** rmesta has joined #openstack-cinder | 04:44 | |
| *** markvoelker has joined #openstack-cinder | 04:45 | |
| *** BharatK has joined #openstack-cinder | 04:45 | |
| nikeshm | avishay: thanks | 04:45 |
| openstackgerrit | John Griffith proposed openstack/cinder: Add secondary account capability to SolidFire https://review.openstack.org/187311 | 04:47 |
| openstackgerrit | John Griffith proposed openstack/cinder: Add secondary account capability to SolidFire https://review.openstack.org/187311 | 04:48 |
| *** _cjones_ has joined #openstack-cinder | 04:48 | |
| avishay | jgriffith: go to sleep :) | 04:48 |
| *** anuragpalsule has joined #openstack-cinder | 04:48 | |
| *** anuragpalsule has left #openstack-cinder | 04:49 | |
| *** markvoelker has quit IRC | 04:50 | |
| *** lcurtis_ has quit IRC | 04:52 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add secondary account capability to SolidFire https://review.openstack.org/187311 | 04:53 |
| *** _cjones_ has quit IRC | 04:55 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:56 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC ScaleIO Cinder Driver https://review.openstack.org/183762 | 04:58 |
| *** rbradbury has quit IRC | 05:02 | |
| *** nkrinner has joined #openstack-cinder | 05:03 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for Lenovo Storage Arrays. https://review.openstack.org/187853 | 05:09 |
| *** yamada-h has joined #openstack-cinder | 05:09 | |
| *** sks has joined #openstack-cinder | 05:13 | |
| *** dannywilson has joined #openstack-cinder | 05:13 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Adds FC and ISCSI Cinder drivers for HPMSA Storage Arrays. https://review.openstack.org/187707 | 05:22 |
| *** dannywilson has quit IRC | 05:25 | |
| *** dannywilson has joined #openstack-cinder | 05:25 | |
| *** krtaylor has joined #openstack-cinder | 05:27 | |
| *** asselin_ has joined #openstack-cinder | 05:27 | |
| nikeshm | smcginis: are yu ok with patch 9 | 05:35 |
| *** nlevinki has joined #openstack-cinder | 05:35 | |
| nikeshm | hemna: is it good now :) | 05:35 |
| nikeshm | on last patch CI was successful | 05:36 |
| nikeshm | and new patch there is no code change | 05:36 |
| nikeshm | only rebasing | 05:36 |
| *** nihilifer has joined #openstack-cinder | 05:37 | |
| nikeshm | donot know do again CI shown is needed for merge? | 05:37 |
| nikeshm | any how again triggered CI | 05:39 |
| *** asselin_ has quit IRC | 05:40 | |
| *** vilobhmm has joined #openstack-cinder | 05:42 | |
| *** dulek has joined #openstack-cinder | 05:46 | |
| *** ankit_ag has joined #openstack-cinder | 05:51 | |
| *** Maike has joined #openstack-cinder | 05:52 | |
| *** deepakcs has joined #openstack-cinder | 05:54 | |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Add Pure Storage FibreChannel driver https://review.openstack.org/186605 | 05:55 |
| *** lpetrut has joined #openstack-cinder | 05:58 | |
| *** vilobhmm has quit IRC | 05:58 | |
| openstackgerrit | wanghao proposed openstack/cinder: Fix weird change of volume status in re-scheduling https://review.openstack.org/185545 | 06:13 |
| *** sgotliv has quit IRC | 06:15 | |
| *** shakamunyi has joined #openstack-cinder | 06:16 | |
| *** barra204 has joined #openstack-cinder | 06:17 | |
| *** dannywilson has quit IRC | 06:18 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 06:23 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Service object https://review.openstack.org/160417 | 06:25 |
| *** belmoreira has joined #openstack-cinder | 06:26 | |
| *** agarciam has joined #openstack-cinder | 06:29 | |
| *** markvoelker has joined #openstack-cinder | 06:34 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Refactor scheduler's create_volume flow https://review.openstack.org/186439 | 06:35 |
| *** ianbrown has quit IRC | 06:35 | |
| *** Longgeek has joined #openstack-cinder | 06:35 | |
| *** Longge___ has quit IRC | 06:37 | |
| *** markvoelker has quit IRC | 06:38 | |
| *** ronis has joined #openstack-cinder | 06:49 | |
| *** anshul has joined #openstack-cinder | 06:50 | |
| *** anshul has quit IRC | 06:51 | |
| *** anshul has joined #openstack-cinder | 06:51 | |
| *** nlevinki_ has joined #openstack-cinder | 06:51 | |
| *** nlevinki has quit IRC | 06:54 | |
| *** pradipta has joined #openstack-cinder | 06:55 | |
| *** tshefi has joined #openstack-cinder | 06:59 | |
| openstackgerrit | Abhijeet Malawade proposed openstack/cinder: Validate request body element name consistently https://review.openstack.org/187886 | 07:00 |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/187887 | 07:00 |
| *** avishay_ has joined #openstack-cinder | 07:03 | |
| *** avishay has quit IRC | 07:04 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Fix QoS support in Huawei driver https://review.openstack.org/187889 | 07:05 |
| *** nlevinki_ has quit IRC | 07:12 | |
| *** haigang has quit IRC | 07:19 | |
| *** nlevinki_ has joined #openstack-cinder | 07:25 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder-specs: capacity-headroom https://review.openstack.org/170380 | 07:26 |
| nikeshm | hemna smcginnis: now all reviews are addressed and jenkins and Dothill CI are also good | 07:26 |
| *** nlevinki_ has quit IRC | 07:27 | |
| *** nlevinki has joined #openstack-cinder | 07:27 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: WIP: GlusterFS: Switch to Cinder Objects https://review.openstack.org/187892 | 07:33 |
| *** chlong has quit IRC | 07:37 | |
| *** coolsvap is now known as coolsvap|afk | 07:41 | |
| *** belmoreira has quit IRC | 07:42 | |
| *** markus_z has joined #openstack-cinder | 07:45 | |
| *** belmoreira has joined #openstack-cinder | 07:46 | |
| *** coolsvap|afk is now known as coolsvap | 07:46 | |
| *** sgotliv has joined #openstack-cinder | 07:47 | |
| *** jistr has joined #openstack-cinder | 07:51 | |
| *** tobe has quit IRC | 07:52 | |
| *** elonden has quit IRC | 07:54 | |
| *** dims_ has joined #openstack-cinder | 07:58 | |
| *** ndipanov has joined #openstack-cinder | 08:00 | |
| *** rushiagr_away is now known as rushiagr | 08:02 | |
| *** dims_ has quit IRC | 08:03 | |
| *** aarefiev has quit IRC | 08:06 | |
| *** jordanP has joined #openstack-cinder | 08:12 | |
| *** tobe has joined #openstack-cinder | 08:14 | |
| *** aarefiev has joined #openstack-cinder | 08:17 | |
| *** nlevinki has quit IRC | 08:21 | |
| *** markvoelker has joined #openstack-cinder | 08:22 | |
| *** vivek_nandavanam has quit IRC | 08:23 | |
| *** lpetrut has quit IRC | 08:23 | |
| *** markvoelker has quit IRC | 08:27 | |
| *** nlevinki has joined #openstack-cinder | 08:31 | |
| *** nlevinki has quit IRC | 08:33 | |
| *** yamada-h has quit IRC | 08:35 | |
| *** nlevinki has joined #openstack-cinder | 08:37 | |
| *** BharatK has quit IRC | 08:39 | |
| *** ianbrown has joined #openstack-cinder | 08:41 | |
| *** e0ne has joined #openstack-cinder | 08:46 | |
| *** BharatK has joined #openstack-cinder | 08:52 | |
| *** nkrinner has quit IRC | 08:52 | |
| *** lixiaoy1 has quit IRC | 08:52 | |
| *** ianbrown has quit IRC | 08:56 | |
| *** nikeshm has left #openstack-cinder | 08:57 | |
| *** nkrinner has joined #openstack-cinder | 08:57 | |
| *** e0ne is now known as e0ne_ | 09:01 | |
| *** e0ne_ is now known as e0ne | 09:01 | |
| *** lpetrut has joined #openstack-cinder | 09:05 | |
| *** Zhongjun has quit IRC | 09:09 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Don't send heartbeats if Manager reports a problem https://review.openstack.org/187670 | 09:18 |
| *** afazekas has joined #openstack-cinder | 09:22 | |
| *** lpetrut has quit IRC | 09:22 | |
| *** nikeshm has joined #openstack-cinder | 09:26 | |
| *** gcb has joined #openstack-cinder | 09:27 | |
| nikeshm | DuncanT: going to sleep,just to inform that jenkins and CI passed for the dothill patch | 09:32 |
| *** ianbrown has joined #openstack-cinder | 09:47 | |
| *** pradipta has quit IRC | 09:48 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter snapshots data on the DB side https://review.openstack.org/184267 | 09:55 |
| *** e0ne is now known as e0ne_ | 09:55 | |
| *** ianbrown has quit IRC | 09:56 | |
| *** lpetrut has joined #openstack-cinder | 09:57 | |
| *** e0ne_ is now known as e0ne | 09:57 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter snapshots data on the DB side https://review.openstack.org/184267 | 09:58 |
| *** ronis has quit IRC | 09:58 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs: Incremental backup improvements for L https://review.openstack.org/187926 | 09:59 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Don't send heartbeats if Manager reports a problem https://review.openstack.org/187670 | 10:01 |
| *** dims_ has joined #openstack-cinder | 10:06 | |
| dulek | DuncanT, geguileor: Hi, on the summit we decided to divide outstanding work on c-vol A/A topic during one of the meetings. Do we want to do that today? | 10:08 |
| geguileor | dulek: I think we are doing that in an informal way in the Etherpad: https://etherpad.openstack.org/p/cinder-active-active-vol-service-issues | 10:09 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Port remote_fs driver to use new driver model https://review.openstack.org/160688 | 10:10 |
| dulek | geguileor: Yeah, that haven't changed for at least a week. I wonder about status of each of the parts. | 10:11 |
| *** markvoelker has joined #openstack-cinder | 10:11 | |
| geguileor | dulek: I can tell you about my part... No news, I haven't been able to start | 10:11 |
| geguileor | dulek: I've been fighting a concurrency issue on LIO | 10:12 |
| dulek | geguileor: Okay, good to know that. I'll see if I can find someone working on my team to take a look on reproducing at least the issues that Tooz locks are supposed to fix. | 10:13 |
| geguileor | dulek: But I think it would be a good idea to have periodic updates in the meetings | 10:14 |
| geguileor | dulek: I've solved that LIO problem and it's already merge, so I should have some time now to look into the testing | 10:14 |
| *** markvoelker has quit IRC | 10:15 | |
| dulek | geguileor: Okay, I'll add an item to the agenda. | 10:17 |
| dulek | geguileor: And about tests - let's just try to in sync on the topic. | 10:18 |
| geguileor | dulek: Ok | 10:18 |
| dulek | geguileor: Can I add you also to "Led by:"? I'll be traveling at meeting time and I'm not sure if my mobile connectivity will be reliable. | 10:21 |
| geguileor | dulek: Sure | 10:22 |
| *** merooney has joined #openstack-cinder | 10:24 | |
| *** Gomeler has quit IRC | 10:34 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Do Not Merge: test delete flow https://review.openstack.org/187943 | 10:59 |
| *** ronis has joined #openstack-cinder | 11:00 | |
| *** lcurtis_ has joined #openstack-cinder | 11:03 | |
| *** skraynev_ has joined #openstack-cinder | 11:04 | |
| *** skraynev_ has quit IRC | 11:04 | |
| *** julim has joined #openstack-cinder | 11:06 | |
| *** lcurtis_ has quit IRC | 11:10 | |
| *** merooney has quit IRC | 11:16 | |
| *** zul has quit IRC | 11:22 | |
| *** cdelatte has joined #openstack-cinder | 11:25 | |
| *** aix has quit IRC | 11:25 | |
| *** delattec has joined #openstack-cinder | 11:25 | |
| *** rushiagr is now known as rushiagr_away | 11:25 | |
| *** merooney has joined #openstack-cinder | 11:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 11:28 |
| *** zul has joined #openstack-cinder | 11:28 | |
| *** ociuhandu has quit IRC | 11:30 | |
| *** ronis has quit IRC | 11:31 | |
| *** e0ne is now known as e0ne_ | 11:32 | |
| flip214 | Hi. Doing final tests on the DRBD driver in devstack gives me "TypeError: get_volume_stats() got an unexpected keyword argument 'refresh'". | 11:35 |
| flip214 | google says that that's because of incompatible cinder/cinder-client versions. Is that still true, even with a current devstack? | 11:36 |
| *** markvoelker has joined #openstack-cinder | 11:42 | |
| *** e0ne_ has quit IRC | 11:42 | |
| *** heyun has quit IRC | 11:44 | |
| *** markvoelker has quit IRC | 11:46 | |
| *** merooney has quit IRC | 11:50 | |
| *** tobe has quit IRC | 11:56 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/187967 | 11:57 |
| *** tobe has joined #openstack-cinder | 11:57 | |
| *** avishay_ has quit IRC | 11:57 | |
| *** merooney has joined #openstack-cinder | 11:57 | |
| *** e0ne has joined #openstack-cinder | 11:58 | |
| *** aix has joined #openstack-cinder | 12:00 | |
| *** markvoelker has joined #openstack-cinder | 12:01 | |
| *** lpabon has joined #openstack-cinder | 12:02 | |
| *** tobe has quit IRC | 12:02 | |
| *** deepakcs has quit IRC | 12:03 | |
| *** ronis has joined #openstack-cinder | 12:04 | |
| *** spradeepv has joined #openstack-cinder | 12:06 | |
| *** sgotliv has quit IRC | 12:09 | |
| *** lennyb has joined #openstack-cinder | 12:09 | |
| *** sgotliv has joined #openstack-cinder | 12:12 | |
| *** lennyb_ has quit IRC | 12:14 | |
| *** swamireddy1 has joined #openstack-cinder | 12:16 | |
| *** IanGovett has joined #openstack-cinder | 12:17 | |
| *** eharney has joined #openstack-cinder | 12:18 | |
| *** ganso_ has joined #openstack-cinder | 12:20 | |
| openstackgerrit | Seiji Aguchi proposed openstack/cinder: Add VSP Midrange Family support to Hitachi driver https://review.openstack.org/184999 | 12:25 |
| *** timcl has joined #openstack-cinder | 12:27 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/187983 | 12:32 |
| *** akerr has joined #openstack-cinder | 12:36 | |
| *** merooney has quit IRC | 12:37 | |
| *** Adriano_ has joined #openstack-cinder | 12:40 | |
| *** ociuhandu has joined #openstack-cinder | 12:41 | |
| *** bswartz has joined #openstack-cinder | 12:41 | |
| *** bswartz has quit IRC | 12:41 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Backup object https://review.openstack.org/157085 | 12:43 |
| *** gouthamr has joined #openstack-cinder | 12:45 | |
| *** gouthamr has quit IRC | 12:46 | |
| *** BharatK has quit IRC | 12:46 | |
| *** merooney has joined #openstack-cinder | 12:50 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Don't send heartbeats if Manager reports a problem https://review.openstack.org/187670 | 12:51 |
| *** primechuck has joined #openstack-cinder | 12:52 | |
| *** ankit_ag has quit IRC | 12:53 | |
| *** primechuck has quit IRC | 12:53 | |
| *** primechu_ has joined #openstack-cinder | 12:53 | |
| *** rwsu has joined #openstack-cinder | 12:54 | |
| *** smoriya has quit IRC | 12:57 | |
| *** jistr is now known as jistr|mtg | 12:57 | |
| *** sks has quit IRC | 12:57 | |
| *** gouthamr has joined #openstack-cinder | 12:57 | |
| *** BharatK has joined #openstack-cinder | 12:59 | |
| *** merooney has quit IRC | 12:59 | |
| *** tsekiyama has joined #openstack-cinder | 13:01 | |
| *** tsekiyama has quit IRC | 13:04 | |
| *** sks has joined #openstack-cinder | 13:11 | |
| *** asselin_ has joined #openstack-cinder | 13:11 | |
| *** nkrinner has quit IRC | 13:15 | |
| *** Yogi1 has joined #openstack-cinder | 13:23 | |
| *** dustins has joined #openstack-cinder | 13:23 | |
| *** cknight has joined #openstack-cinder | 13:27 | |
| *** jungleboyj has quit IRC | 13:28 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up unused exceptions https://review.openstack.org/187858 | 13:30 |
| *** rushil has joined #openstack-cinder | 13:31 | |
| *** mriedem_away is now known as mriedem | 13:32 | |
| openstackgerrit | Rick Chen proposed openstack/cinder: Port ProphetStor driver to use ABCMeta driver model https://review.openstack.org/178992 | 13:39 |
| *** e0ne is now known as e0ne_ | 13:41 | |
| *** e0ne_ is now known as e0ne | 13:41 | |
| *** changbl_ has joined #openstack-cinder | 13:44 | |
| *** rmesta has joined #openstack-cinder | 13:45 | |
| *** IanGovett has quit IRC | 13:46 | |
| *** rushil has quit IRC | 13:47 | |
| *** xyang1 has joined #openstack-cinder | 13:52 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Port remote_fs driver to use new driver model https://review.openstack.org/160688 | 13:55 |
| *** jistr|mtg is now known as jistr | 13:59 | |
| *** dulek has quit IRC | 14:00 | |
| *** coolsvap is now known as coolsvap|afk | 14:00 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC ScaleIO Cinder Driver https://review.openstack.org/183762 | 14:00 |
| *** merooney has joined #openstack-cinder | 14:01 | |
| *** sks has quit IRC | 14:03 | |
| *** timcl has quit IRC | 14:05 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC ScaleIO Cinder Driver https://review.openstack.org/183762 | 14:07 |
| *** thangp has joined #openstack-cinder | 14:08 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Service object https://review.openstack.org/160417 | 14:11 |
| *** bkopilov has quit IRC | 14:12 | |
| *** marcusvrn1 has joined #openstack-cinder | 14:12 | |
| *** Gomeler has joined #openstack-cinder | 14:13 | |
| *** marcusvrn has quit IRC | 14:14 | |
| *** swamireddy1 has quit IRC | 14:15 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver https://review.openstack.org/178319 | 14:18 |
| *** mtanino has joined #openstack-cinder | 14:20 | |
| *** mdbooth has quit IRC | 14:21 | |
| openstackgerrit | Ph. Marek proposed openstack/cinder: Re-add DRBD driver. https://review.openstack.org/178573 | 14:24 |
| *** david-lyle has quit IRC | 14:25 | |
| *** tsekiyama has joined #openstack-cinder | 14:27 | |
| ameade | thingee: want me to update this? https://wiki.openstack.org/wiki/CrossProjectLiaisons#API_Working_Group | 14:28 |
| *** agarciam has quit IRC | 14:30 | |
| *** agarciam has joined #openstack-cinder | 14:31 | |
| *** rwsu has quit IRC | 14:31 | |
| *** rushil has joined #openstack-cinder | 14:34 | |
| *** asselin__ has joined #openstack-cinder | 14:36 | |
| *** barra204 has quit IRC | 14:36 | |
| *** shakamunyi has quit IRC | 14:36 | |
| *** asselin_ has quit IRC | 14:37 | |
| *** rhe00_ has joined #openstack-cinder | 14:38 | |
| openstackgerrit | Cindy Pallares proposed openstack/cinder: Fix multi-line docstrings to meet hacking rules https://review.openstack.org/184820 | 14:40 |
| thingee | ameade: yes, I ping you about it yesterday. | 14:41 |
| *** Maike has quit IRC | 14:41 | |
| *** afazekas has quit IRC | 14:43 | |
| flip214 | thingee: just for information, the drbd cinder driver seems to work fine, and the devstack plugin looks like it's working, too. | 14:45 |
| *** rajinir_r has joined #openstack-cinder | 14:46 | |
| flip214 | I'll have to ask what the next steps are to get both into the infra CIs, but I'm optimistic that it's done by June 12th. | 14:46 |
| thingee | flip214: just letting you know right now that's cutting it pretty close. | 14:46 |
| openstackgerrit | xing-yang proposed openstack/cinder: Allow provisioning to reach max oversubscription https://review.openstack.org/188031 | 14:48 |
| flip214 | thingee: yes, I'm well aware. I'd like to have been done 3 years ago, too. | 14:48 |
| thingee | I would've preferred before cinder was created if we want to act like that. | 14:49 |
| *** ronis has quit IRC | 14:49 | |
| *** nihilifer has quit IRC | 14:50 | |
| *** timcl has joined #openstack-cinder | 14:53 | |
| *** daneyon has joined #openstack-cinder | 14:57 | |
| *** Tross has quit IRC | 14:58 | |
| *** Tross has joined #openstack-cinder | 14:58 | |
| *** Tross has quit IRC | 14:58 | |
| *** deepakcs has joined #openstack-cinder | 14:58 | |
| *** Tross has joined #openstack-cinder | 14:58 | |
| *** nlevinki has quit IRC | 15:00 | |
| *** Tross1 has joined #openstack-cinder | 15:00 | |
| deepakcs | hi all, I am getting this error "ValueError: Field `volume_id' cannot be None" for some of the VolumeTestCase tests | 15:00 |
| *** anshul has quit IRC | 15:01 | |
| deepakcs | I don't see doing anything in my patch that could be causing that | 15:01 |
| *** daneyon_ has joined #openstack-cinder | 15:01 | |
| deepakcs | anyone has any ideas ? ^^ | 15:01 |
| *** Tross1 has quit IRC | 15:02 | |
| *** Tross1 has joined #openstack-cinder | 15:03 | |
| *** Tross1 has quit IRC | 15:03 | |
| *** Tross1 has joined #openstack-cinder | 15:03 | |
| *** Tross has quit IRC | 15:03 | |
| *** Tross1 has quit IRC | 15:03 | |
| *** jungleboyj has joined #openstack-cinder | 15:03 | |
| *** Tross1 has joined #openstack-cinder | 15:03 | |
| *** daneyon has quit IRC | 15:04 | |
| *** Tross1 has quit IRC | 15:04 | |
| asselin__ | meeting? | 15:05 |
| *** Gomeler has quit IRC | 15:05 | |
| asselin__ | thingee, meeting? | 15:06 |
| jungleboyj | asselin__: Not for another hour. | 15:06 |
| asselin__ | or is my day/time off | 15:06 |
| asselin__ | ok | 15:06 |
| *** Tross1 has joined #openstack-cinder | 15:07 | |
| jungleboyj | asselin__: Sorry about the GPFS CI. I have been sending them notes for two weeks. Got a couple lines of management involved. It is being worked now. | 15:07 |
| jungleboyj | asselin__: No excuse for that. I apologize. | 15:07 |
| asselin__ | np...guess it takes a bit of external influence :) | 15:07 |
| *** Gomeler has joined #openstack-cinder | 15:07 | |
| jungleboyj | :-) Thanks for the help. | 15:08 |
| *** leeantho has joined #openstack-cinder | 15:09 | |
| *** changbl_ has quit IRC | 15:10 | |
| *** zhipeng has joined #openstack-cinder | 15:10 | |
| *** Zhongjun has joined #openstack-cinder | 15:11 | |
| deepakcs | jungleboyj: I am getting this error "ValueError: Field `volume_id' cannot be None" for some of the VolumeTestCase tests, unable to relate it to my patch in any way, any sugestions ? | 15:11 |
| deepakcs | jungleboyj: Its happening when the tests call objects.Snapshot.get_by_id(self.context, snapshot_id) | 15:11 |
| *** changbl_ has joined #openstack-cinder | 15:11 | |
| deepakcs | eharney: ^^ | 15:12 |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Non Disruptive Backup https://review.openstack.org/186897 | 15:16 |
| *** jordanP has quit IRC | 15:17 | |
| *** rwsu has joined #openstack-cinder | 15:19 | |
| *** mdbooth has joined #openstack-cinder | 15:20 | |
| DuncanT | xyang1: Are you about? | 15:21 |
| xyang1 | DuncanT: Hi | 15:21 |
| DuncanT | Hi | 15:21 |
| DuncanT | Looking at that blueprint I had a thought - is there a significant downside to asking the user to take a snapshot themselves, then allowing them to back that up? It's one less thing to keep track of inside cinder then | 15:22 |
| *** zhipeng has quit IRC | 15:22 | |
| xyang1 | DuncanT: That is an option, but not automatic | 15:23 |
| xyang1 | DuncanT: You mean this way we don't need a temp snapshot? | 15:23 |
| *** merooney has quit IRC | 15:24 | |
| *** alexpilotti has quit IRC | 15:24 | |
| xyang1 | DuncanT: One downside I can see is that the timestamp could be way off | 15:25 |
| xyang1 | DuncanT: So backup timestamp could be very far away from snapshot timestamp | 15:25 |
| xyang1 | DuncanT: This also means a later backup could be from an earlier snapshot | 15:26 |
| xyang1 | DuncanT: Unless if we create timestamp based on snapshot timestamp? Doesn't seem right | 15:28 |
| DuncanT | No, it would mean the tenant would need to make sure they took the snap at the right time and did the backup immediately if they want the timestamps to match | 15:29 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Deprecate the HPLeftHandISCSIDriver CLIQ driver https://review.openstack.org/187799 | 15:29 |
| DuncanT | It does mean that they can turn snaps into backup later if they wanted | 15:29 |
| DuncanT | Given the code to backup a snap is needed either way, might be worth writing and exposing that first, then adding the backup-live-volume-with-temp-snapshot as a second patch? No reason not to allow both that I can see | 15:30 |
| xyang1 | DuncanT: How can we garantee they take snapshot immediately? Seems that user can make mistakes easily? | 15:31 |
| *** jdurgin1 has joined #openstack-cinder | 15:31 | |
| geguileor | I agree with DuncanT, we should allow both | 15:31 |
| xyang1 | DuncanT: I am thinking incremental backup could be affected | 15:31 |
| geguileor | So it makes sense to first do the snapshot backup and then the live-backup | 15:31 |
| xyang1 | DuncanT: We automatically find the latest timestamp | 15:32 |
| DuncanT | Well, the time of the snap is the time they froze the data, when they do the backup doesn't actually matter | 15:32 |
| *** e0ne is now known as e0ne_ | 15:32 | |
| xyang1 | DuncanT: It will screw up incremental backup | 15:32 |
| DuncanT | So we can take the time from the snap to find the best backup to use, that shouldn't be difficult | 15:32 |
| geguileor | xyang1: How would it screw incremental backup? | 15:33 |
| xyang1 | DuncanT: Unless we require user to enter uuid of parent backup again | 15:33 |
| xyang1 | geguileor: We automatically find the latest backup based on timestamp | 15:33 |
| *** rushil has quit IRC | 15:34 | |
| xyang1 | Timestamp will be messed up | 15:34 |
| DuncanT | xyang1: The time the snap was created is right there in the DB, no problem | 15:34 |
| geguileor | But backup timestamp should be snapshot timestamp, right? | 15:34 |
| DuncanT | xyang1: Just use that to pick the best parent | 15:34 |
| geguileor | Oh, sorry, we don't have backup timestamp and data timestamp | 15:34 |
| geguileor | :( | 15:34 |
| geguileor | Could it be interesting to add metadata to the backup specifying source (volume, snapshot) and original timestamp? | 15:35 |
| *** e0ne_ is now known as e0ne | 15:35 | |
| geguileor | And make incremental use those if present? | 15:36 |
| xyang1 | DuncanT: Unless we manually put timestamp of snapshot into backup, we can't reliably find the latest backup timestamp any more | 15:38 |
| geguileor | xyang1: And adding that to the metadata wouldn't work? | 15:38 |
| DuncanT | xyang1: Ah, I see | 15:39 |
| *** david-lyle has joined #openstack-cinder | 15:39 | |
| xyang1 | DuncanT: If we require user to enter a parent id, that may give us flexibility | 15:40 |
| DuncanT | xyang1: but that is a painful UI | 15:40 |
| xyang1 | DuncanT: That is true | 15:40 |
| geguileor | I agree | 15:41 |
| *** hemnafk is now known as hemna | 15:41 | |
| DuncanT | We can add a new field to the backup record to track the 'data' date of the backup... maybe we do the hidden temp snapshot first and add user controlled backup-of-snap later | 15:42 |
| DuncanT | Never mind, thanks for talking it through | 15:42 |
| xyang1 | DuncanT: Sure, thanks | 15:42 |
| *** earlephilhower has quit IRC | 15:43 | |
| *** jdurgin1 has quit IRC | 15:46 | |
| *** david-lyle has quit IRC | 15:47 | |
| openstackgerrit | xing-yang proposed openstack/cinder-specs: Non Disruptive Backup https://review.openstack.org/186897 | 15:48 |
| *** alexpilotti has joined #openstack-cinder | 15:48 | |
| *** patrickeast has joined #openstack-cinder | 15:48 | |
| *** vincent_hou has joined #openstack-cinder | 15:49 | |
| *** mdbooth has quit IRC | 15:49 | |
| *** adurbin__ has joined #openstack-cinder | 15:50 | |
| *** mdbooth has joined #openstack-cinder | 15:50 | |
| *** david-lyle has joined #openstack-cinder | 15:53 | |
| *** lpabon has quit IRC | 15:54 | |
| *** avishay has joined #openstack-cinder | 15:55 | |
| avishay | hello all | 15:56 |
| e0ne | avishay: hi | 15:57 |
| *** dannywilson has joined #openstack-cinder | 15:57 | |
| vincent_hou | avishay: openstack-meeting? | 15:57 |
| xyang1 | avishay: Hi | 15:57 |
| avishay | e0ne: vincent_hou xyang1 hey | 15:57 |
| avishay | vincent_hou: i guess so, soon | 15:57 |
| vincent_hou | hey folks | 15:58 |
| *** dulek has joined #openstack-cinder | 15:59 | |
| *** rmesta has quit IRC | 16:00 | |
| *** tshefi has quit IRC | 16:03 | |
| *** rushiagr_away is now known as rushiagr | 16:03 | |
| *** nestorf has joined #openstack-cinder | 16:04 | |
| *** belmoreira has quit IRC | 16:07 | |
| *** _cjones_ has joined #openstack-cinder | 16:07 | |
| *** swamireddy1 has joined #openstack-cinder | 16:09 | |
| *** jistr has quit IRC | 16:11 | |
| *** afazekas has joined #openstack-cinder | 16:12 | |
| *** dulek has quit IRC | 16:13 | |
| *** dulek has joined #openstack-cinder | 16:14 | |
| *** anshul has joined #openstack-cinder | 16:14 | |
| *** vilobhmm has joined #openstack-cinder | 16:16 | |
| *** dims_ has quit IRC | 16:18 | |
| *** dims_ has joined #openstack-cinder | 16:18 | |
| *** dims__ has joined #openstack-cinder | 16:20 | |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| *** BharatK has quit IRC | 16:21 | |
| *** barra204 has joined #openstack-cinder | 16:21 | |
| *** david-lyle has quit IRC | 16:22 | |
| *** anshul has quit IRC | 16:23 | |
| openstackgerrit | Patrick East proposed openstack/cinder-specs: Generic image volume cache functionality https://review.openstack.org/182520 | 16:23 |
| *** dims_ has quit IRC | 16:24 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Expanded comments and update var names https://review.openstack.org/187729 | 16:24 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls https://review.openstack.org/182396 | 16:24 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for alternate iscsi portals https://review.openstack.org/182760 | 16:24 |
| *** earlephilhower has joined #openstack-cinder | 16:25 | |
| vincent_hou | avishay: Thanks, dude. | 16:27 |
| *** barra204 has quit IRC | 16:28 | |
| *** ronis has joined #openstack-cinder | 16:32 | |
| winston-d | avishay: hi | 16:32 |
| avishay | winston-d: yo what's up | 16:32 |
| avishay | vincent_hou: sure thing | 16:32 |
| *** dulek has quit IRC | 16:33 | |
| *** swamireddy1 has quit IRC | 16:33 | |
| *** dulek has joined #openstack-cinder | 16:36 | |
| *** Apoorva has joined #openstack-cinder | 16:36 | |
| *** rmesta has joined #openstack-cinder | 16:38 | |
| *** avishay has quit IRC | 16:40 | |
| *** agarciam has quit IRC | 16:42 | |
| *** agarciam has joined #openstack-cinder | 16:42 | |
| *** deepakcs has quit IRC | 16:44 | |
| *** akshai has joined #openstack-cinder | 16:47 | |
| *** rushil has joined #openstack-cinder | 16:48 | |
| *** thangp_ has joined #openstack-cinder | 16:49 | |
| *** thangp has quit IRC | 16:49 | |
| *** agarciam has quit IRC | 16:52 | |
| *** afazekas has quit IRC | 16:55 | |
| *** Longgeek has quit IRC | 16:55 | |
| *** vincent_hou has quit IRC | 16:55 | |
| *** juzuluag has joined #openstack-cinder | 16:56 | |
| *** merooney has joined #openstack-cinder | 16:56 | |
| openstackgerrit | John Griffith proposed openstack/os-brick: Add retry to iSCSI delete https://review.openstack.org/187811 | 16:57 |
| *** dulek has quit IRC | 16:58 | |
| *** belmoreira has joined #openstack-cinder | 16:58 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver https://review.openstack.org/178319 | 16:58 |
| *** aarefiev has quit IRC | 16:58 | |
| asselin__ | nikeshm, | 17:00 |
| asselin__ | you're being requested in -meeting | 17:00 |
| asselin__ | but meeting is over...read the minutes | 17:00 |
| *** markus_z has quit IRC | 17:01 | |
| winston-d | hemna, scottda: do we have a doc for cinder-nova interaction? | 17:01 |
| *** dannywilson has quit IRC | 17:01 | |
| *** merooney has quit IRC | 17:03 | |
| *** dannywilson has joined #openstack-cinder | 17:03 | |
| scottda | winston-d: https://etherpad.openstack.org/p/CinderNovaAPI | 17:03 |
| scottda | just rough notes, but it is a place to start | 17:03 |
| *** thangp_ has quit IRC | 17:03 | |
| *** patrickeast has quit IRC | 17:03 | |
| vilobhmm | thingee, DuncanT, winston-d : https://blueprints.launchpad.net/nova/+spec/fix-nova-cinder-interaction | 17:05 |
| *** merooney has joined #openstack-cinder | 17:05 | |
| vilobhmm | filed the blueprint with Nova | 17:05 |
| *** thangp has joined #openstack-cinder | 17:06 | |
| DuncanT | vilobhmm: Awesome | 17:06 |
| *** dannywilson has quit IRC | 17:06 | |
| winston-d | scottda: cool, thx | 17:08 |
| winston-d | vilobhmm: nice | 17:08 |
| winston-d | xyang1: around? | 17:08 |
| xyang1 | winston-d: Hi | 17:09 |
| *** afazekas has joined #openstack-cinder | 17:09 | |
| winston-d | want to check with your change: https://review.openstack.org/#/c/188031/ | 17:09 |
| *** xyang has joined #openstack-cinder | 17:10 | |
| xyang1 | winston-d: Sure | 17:10 |
| winston-d | if provisioned capacity is already at max, do you really want to allow more volumes to be created? | 17:10 |
| nikeshm | asselin : sorry was tight day yesterday and slept late | 17:10 |
| xyang1 | winston-d: This will only reach max after create the new volume | 17:10 |
| asselin__ | nikeshm, suggest you read the last parts of the meeting minutes | 17:11 |
| xyang1 | winston-d: So at the time, it has not reached max yet, after we create the current new volume, it will reach max | 17:11 |
| winston-d | no, in the case when provisioned_ratio equals max_provision_ratio, it's already max | 17:11 |
| xyang1 | winston-d: See line 90-91 | 17:12 |
| xyang1 | winston-d: It will be max after we add volume size | 17:12 |
| *** merooney has quit IRC | 17:12 | |
| winston-d | ah, missed that. | 17:12 |
| winston-d | thx for pointing out. | 17:13 |
| xyang1 | winston-d: Thanks! | 17:13 |
| *** david-lyle has joined #openstack-cinder | 17:13 | |
| *** merooney has joined #openstack-cinder | 17:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 17:13 |
| *** david-lyle has quit IRC | 17:13 | |
| *** rasoto has joined #openstack-cinder | 17:14 | |
| *** david-lyle has joined #openstack-cinder | 17:15 | |
| *** rushiagr is now known as rushiagr_away | 17:15 | |
| *** anshul has joined #openstack-cinder | 17:15 | |
| *** anshul has quit IRC | 17:16 | |
| *** kaisers has quit IRC | 17:16 | |
| *** kaisers1 has quit IRC | 17:17 | |
| *** e0ne has quit IRC | 17:17 | |
| thingee | ameade: just so there are no surprises, I do treat different protocols as a different driver https://blueprints.launchpad.net/cinder/+spec/add-fibre-channel-support-to-netapp-eseries ... I'm assuming this won't be a problem? http://www.openstack.org/blog/2015/05/deadline-for-new-cinder-volume-drivers-in-liberty/ | 17:17 |
| thingee | just noticed this is still marked as not started | 17:18 |
| *** Zhongjun has quit IRC | 17:19 | |
| ameade | thingee: yeah i know, oh it shouldn't be not started | 17:21 |
| thingee | ameade: whew ok | 17:22 |
| ameade | thingee: hehe thanks for reaching out though | 17:22 |
| *** kaisers has joined #openstack-cinder | 17:22 | |
| ameade | thingee: setting up the CI right now actually | 17:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 17:22 |
| *** kaisers1 has joined #openstack-cinder | 17:22 | |
| *** swamireddy1 has joined #openstack-cinder | 17:22 | |
| *** patrickeast has joined #openstack-cinder | 17:24 | |
| anteaya | thingee: you asked my opinion on rebranded drivers, I have no opinion | 17:26 |
| anteaya | thanks for making me aware though | 17:26 |
| anteaya | just anyone who operates a ci needs to follow the infra requirements | 17:26 |
| anteaya | all else on cinder things come from you and the cinder team | 17:26 |
| *** reed_ has joined #openstack-cinder | 17:26 | |
| *** lpetrut has quit IRC | 17:27 | |
| *** david-lyle has quit IRC | 17:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/186112 | 17:31 |
| thingee | anteaya: posted for discussion http://lists.openstack.org/pipermail/openstack-dev/2015-June/065701.html | 17:33 |
| nikeshm | thingee: sorry was tough night yesterday so unable to attend meeting, | 17:34 |
| anteaya | thingee: thanks I'll share my thoughts on the list | 17:37 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/188113 | 17:38 |
| nikeshm | thingee: rebranded drivers are giving logs messages with rebranded drivers names,like in this rebranded driver https://review.openstack.org/#/c/187853/ ,in lenovo_common.py,line no 49,we have self.vendor_name="Lenovo", so in log files,we will get messages with Lenovo | 17:38 |
| nikeshm | same variable in parent class too | 17:39 |
| openstackgerrit | Earle F. Philhower, III proposed openstack/os-brick: Add HGST Solutions connector https://review.openstack.org/186588 | 17:39 |
| nikeshm | and using this variable in log messages | 17:39 |
| nikeshm | like LOG.debug("Disconnected from %s Array.", self.vendor_name) | 17:40 |
| thingee | nikeshm: how about config options that mention the driver being rebranded? | 17:41 |
| thingee | in the name and description | 17:41 |
| thingee | and/or* | 17:41 |
| *** nestorf has quit IRC | 17:42 | |
| xyang | thingee: can you add incremental backup support to the release notes? the doc was finally merged: https://review.openstack.org/#/c/175576/ | 17:46 |
| nikeshm | thingee: like this we have to give for lenovo driver | 17:46 |
| nikeshm | http://paste.openstack.org/show/260039/ | 17:46 |
| nikeshm | in cinder.conf | 17:46 |
| thingee | nikeshm: do those also show up correct with the cinder.conf.sample? http://lists.openstack.org/pipermail/openstack-dev/2015-June/065701.html | 17:47 |
| thingee | correctly* | 17:47 |
| thingee | xyang: sure | 17:47 |
| thingee | xyang: and thank you | 17:47 |
| xyang | thingee: thanks! | 17:47 |
| patrickeast | just curious, why can’t they all just use a ‘RealStor’ (or whatever it is under the hood) driver if they are all the same backend? is the idea that customers really wouldn’t know it wasn’t lenovo or hp or whatever? | 17:49 |
| nikeshm | thingee: we will update the document for each driver in http://docs.openstack.org/kilo/config-reference/content/section_volume-drivers.html | 17:52 |
| *** reed_ has quit IRC | 17:52 | |
| thingee | nikeshm: that's good for the docs, but not for what I'm talking about | 17:52 |
| *** marcusvrn1 has quit IRC | 17:53 | |
| thingee | nikeshm: https://github.com/openstack/cinder/blob/master/tools/config/generate_sample.sh | 17:53 |
| thingee | patrickeast: fyi https://blueprints.launchpad.net/cinder/+spec/nimble-image-cloning-optimization | 17:55 |
| nikeshm | thingee: i will look into that | 17:55 |
| patrickeast | thingee: that looks familiar :D | 17:56 |
| *** marcusvrn has joined #openstack-cinder | 17:56 | |
| nikeshm | thingee: but if we provide CI,then is it fine for rebranded drivers | 17:56 |
| thingee | nikeshm: I'm still trying to understand the gain Cinder gets from rebranded drivers. | 17:57 |
| thingee | there's a ML post to discuss this | 17:57 |
| thingee | nikeshm: http://lists.openstack.org/pipermail/openstack-dev/2015-June/065701.html | 17:57 |
| patrickeast | thingee: oh hey, somewhat related, does your immenent approval of the image cache include its dependant cinder tenant one? https://review.openstack.org/#/c/186232/4 | 17:58 |
| thingee | ah good point | 17:59 |
| patrickeast | thingee: they are linked in gerrit, but i haven’t done a very good job of advertising the other one | 17:59 |
| thingee | patrickeast: so I was confused on the progress with that one. jgriffith I thought suggested this special tenant, then he was against it on IRC...I may have that totally wrong | 18:00 |
| *** juzuluag has quit IRC | 18:00 | |
| patrickeast | haha yea… he suggested it, i ran with the idea and told other people to use it at the summit, and then there was some push back | 18:00 |
| patrickeast | so i dunno | 18:00 |
| patrickeast | no one has -2’ed the spec | 18:01 |
| patrickeast | so thats a good sign | 18:01 |
| jgriffith | thingee: patrickeast no, I didn't say I was against the tenant idea | 18:01 |
| smcginnis | patrickeast: I think it's a good idea. Some good problems having that could solve. | 18:01 |
| jgriffith | thingee: patrickeast what I *did* say was that with the work some others were trying to do with Glance (again) that it might obsolete the whole idea/concept that was being proposed | 18:01 |
| jgriffith | thingee: patrickeast so rather than caching on a device you would just use cinder as the Glance backend | 18:02 |
| thingee | jgriffith: what is the work? | 18:02 |
| thingee | ah | 18:02 |
| *** belmoreira has quit IRC | 18:02 | |
| jgriffith | thingee: patrickeast AND furthermore I didn't say that was the answer either | 18:02 |
| thingee | is that actually being worked on? I thought that was abandoned/broken | 18:02 |
| *** sgotliv has quit IRC | 18:02 | |
| patrickeast | jgriffith: gotcha, that makes sense | 18:02 |
| jgriffith | thingee: patrickeast I simply suggested that it was interesting, that there was overlap and it might be worthwhile to join the two ideas and make sure there isn't a combined solution that should be looked at | 18:02 |
| jgriffith | thingee: it's active again | 18:03 |
| jgriffith | tsekiyama: wasn't that you that was starting work on that again? | 18:03 |
| patrickeast | so, just to make things clear, even with a glance cinder (cinder glance?) backend i do want an image cache, mainly because for backends like mine with a fairly low volume limit it doesn’t make sense to store *all* the images on it, but a smaller cache would help out and make a difference | 18:03 |
| openstackgerrit | Diem Tran proposed openstack/cinder: Re-integrate Oracle iSCSI Cinder driver https://review.openstack.org/178319 | 18:03 |
| patrickeast | imo having both options available is pretty neat | 18:03 |
| thingee | jgriffith: Maybe I'm not fully understanding things (which usually the case these days), but using Cinder as a backend in glance and this seem similar except having this tenant logic in Cinder or glance? | 18:03 |
| jgriffith | patrickeast: and I think that's probably an absolutely GOOD call on your part | 18:04 |
| *** asselin__ has quit IRC | 18:04 | |
| tsekiyama | jgriffith: what work? | 18:04 |
| thingee | tsekiyama: :) | 18:04 |
| patrickeast | lol | 18:04 |
| jgriffith | patrickeast: but what I'm interested in is any possibility of a backend-cache managed by Glance | 18:04 |
| tsekiyama | ah | 18:04 |
| thingee | the magic OpenStack work fairies | 18:04 |
| jgriffith | tsekiyama: Cinder backend for Glance | 18:04 |
| * thingee is feel sassy today and apologizes | 18:05 | |
| thingee | feeling* | 18:05 |
| patrickeast | jgriffith: oh, i see, so the cache lives over in glance and uses the cinder backend to do stuff… yea i can see that being pretty viable too | 18:05 |
| smcginnis | :) | 18:05 |
| jgriffith | patrickeast: it gives us some "nice" things in terms of quotas, permissions etc | 18:05 |
| tsekiyama | jgriffith: yes, I'm proposing glance-spec for it | 18:05 |
| ameade | zhiyan: ^^ | 18:05 |
| jgriffith | patrickeast: yes, that's what I was thinking | 18:05 |
| thingee | well, in that case.... | 18:05 |
| jgriffith | thingee: ^^ tsekiyama and zhiyan are the "OpenStack work fairies" if that's what you want to call it | 18:06 |
| jgriffith | thingee: patrickeast so... just to reiterate. I'm not saying patrickeast shouldn't go forward with what he has | 18:06 |
| thingee | what would be the benefit of having both? | 18:06 |
| jgriffith | thingee: patrickeast I'm saying, it would be wise to consider what tsekiyama and zhiyan are workign on, what their timelines are and what potential there may be for a more elegant combined solution | 18:06 |
| jgriffith | thingee: None | 18:06 |
| jgriffith | thingee: but if it means you use one for Liberty and the other in M that's another story | 18:07 |
| *** asselin_ has joined #openstack-cinder | 18:07 | |
| *** afazekas has quit IRC | 18:07 | |
| jgriffith | thingee: I don't konw that everythign would be in place to even entertain the leveraging of the Glance pieces in L | 18:07 |
| thingee | jgriffith: ok, so you're saying short term, what patrick is suggesting and we know there's a good chance of it happening, as oppose to long term with tsekiyama's improvements on the glance backend, which we have no idea their timeline or coming through with the work? | 18:08 |
| jgriffith | thingee: patrickeast and others all collectively decided to tackle and make it happen | 18:08 |
| jgriffith | thingee: yes, pretty much | 18:08 |
| *** swamireddy1 has quit IRC | 18:08 | |
| thingee | jgriffith: cool | 18:08 |
| jgriffith | thingee: the solution patrickeast has proposed is fairly easy, and should be "easy" to have in Cinder for L1 or L2 IMHO | 18:08 |
| thingee | +1 | 18:08 |
| patrickeast | sounds good to me, and im happy to help out with the glance efforts too | 18:08 |
| jgriffith | thingee: the Glance backend stuff is a bit more complicated/involved | 18:08 |
| patrickeast | but yea, i definitely want this for L, whichever way that ends up being | 18:08 |
| thingee | why not just have patrickeast help on the glance efforts if we know we can count on him? | 18:09 |
| jgriffith | tsekiyama: sorry to pull you in on that and put you on the spot :) | 18:09 |
| tsekiyama | jgriffith: np | 18:09 |
| thingee | ah complicated | 18:09 |
| jgriffith | thingee: I'm not fully convinced on that approach being viable yet | 18:09 |
| jgriffith | thingee: and "time" | 18:09 |
| thingee | gotcha | 18:09 |
| thingee | jgriffith: can we talk about the transition? if and when? | 18:09 |
| jgriffith | thingee: what "transition"; between the two approaches? | 18:10 |
| thingee | yes | 18:10 |
| jgriffith | thingee: I would say "if" and that's a big "if" the cinder-backend-for-glance happens in L, then it would something for M most likely | 18:10 |
| jgriffith | thingee: but I'm not working on these things so I can't really say | 18:11 |
| patrickeast | i dont think it would be bad if they overlapped, they should be able to co-exist | 18:11 |
| jgriffith | thingee: and given neither of them are functional or exist fully right now, I can't really guess what a transition would look like | 18:11 |
| patrickeast | imo get the quick fix in cinder, then whenever the glance stuff finishes we deprecate it | 18:11 |
| thingee | ok... that's my only concern. It could be though that we end up just using patrick's depending on what's discovered with the glance backend approach | 18:11 |
| jgriffith | patrickeast: that's a valid point as well | 18:11 |
| patrickeast | and maybe remove after another release | 18:11 |
| thingee | patrickeast: seems reasonable. just was wondering if the transition is going to be doable, but maybe we don't have that information yet | 18:11 |
| jgriffith | patrickeast: sure, that's what deprecation periods are for | 18:12 |
| jgriffith | patrickeast: although this is simply an implemenation | 18:12 |
| thingee | I guess transition is you switch to the glance backend, and have a bunch of cache misses | 18:12 |
| thingee | no big deal | 18:12 |
| jgriffith | patrickeast: and given that the image-volume and all the caching is "magic" hidden from everybody in the outside world who cares | 18:12 |
| jgriffith | thingee: no | 18:12 |
| thingee | haha | 18:12 |
| jgriffith | thingee: if you migrate, you can certainly migrate the cached images | 18:12 |
| thingee | sure, I'm just saying cache misses are no big deal imo | 18:13 |
| jgriffith | thingee: but we're kinda getting ahead of our skis here I think for now | 18:13 |
| jgriffith | thingee: Oh, yes, I would agree with you on that | 18:13 |
| patrickeast | yea, i think core functionality of image -> volume would never be interupted throughout this process | 18:13 |
| * thingee has been skiing wrong this whole time | 18:13 | |
| patrickeast | just how fast it is | 18:13 |
| jgriffith | patrickeast: right | 18:13 |
| jgriffith | andon that note... I'm out of here once again :( | 18:14 |
| jgriffith | maybe tomorrow will be a normal day again | 18:14 |
| jgriffith | tbarron: BTW, thanks for the good catches on that iscsi review :) | 18:14 |
| tbarron | jgriffith: yw | 18:15 |
| tbarron | jgriffith: I'm learning a lot now that I have some time to do reviews. | 18:15 |
| *** angela-s has quit IRC | 18:15 | |
| thingee | jgriffith: bummer, see ya around | 18:15 |
| *** angela-s has joined #openstack-cinder | 18:16 | |
| nikeshm | thingee: I am sitting with my manager. Reviewed meeting logs related to DH, HP and Lenovo cinder drivers. | 18:17 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder-specs: Efficient volume copy for volume migration https://review.openstack.org/186209 | 18:17 |
| thingee | nikeshm: I'm sitting in a coffee shop, next to a couple of strangers. | 18:18 |
| nikeshm | thingee: we will provide individuals and support the drivers | 18:18 |
| thingee | nikeshm: I would recommend keeping up with the discussion on the list. | 18:19 |
| thingee | nikeshm: I'm on the fence with this, so seeing what others think | 18:19 |
| *** marcusvrn1 has joined #openstack-cinder | 18:22 | |
| nikeshm | thingee: it appears that most are leaning towards - if you bring a driver - you bring a CI. We are good with it. | 18:23 |
| *** dannywilson has joined #openstack-cinder | 18:23 | |
| *** annegentle has joined #openstack-cinder | 18:23 | |
| thingee | cool | 18:23 |
| jungleboyj | :-) | 18:24 |
| *** marcusvrn has quit IRC | 18:24 | |
| *** dannywilson has quit IRC | 18:25 | |
| thingee | I'll bbl | 18:25 |
| *** dannywilson has joined #openstack-cinder | 18:25 | |
| mtanino | thingee: thank you for your approval for our strage driver. | 18:26 |
| *** lpetrut has joined #openstack-cinder | 18:27 | |
| nikeshm | hemna smcginnis: hi | 18:28 |
| *** bswartz has joined #openstack-cinder | 18:28 | |
| *** merooney has quit IRC | 18:32 | |
| *** david-lyle has joined #openstack-cinder | 18:33 | |
| *** aix has quit IRC | 18:39 | |
| hemna | bbiab....lunch | 18:48 |
| *** rmesta has quit IRC | 18:52 | |
| *** cknight has quit IRC | 18:52 | |
| *** pradipta has joined #openstack-cinder | 18:53 | |
| *** merooney has joined #openstack-cinder | 18:54 | |
| *** merooney has quit IRC | 18:56 | |
| *** cpallares has quit IRC | 18:56 | |
| *** barra204 has joined #openstack-cinder | 18:57 | |
| *** asselin_ has quit IRC | 18:59 | |
| *** barra204 has quit IRC | 19:02 | |
| *** sjcherry has joined #openstack-cinder | 19:07 | |
| *** Yogi1 has quit IRC | 19:12 | |
| *** dalgaaf has joined #openstack-cinder | 19:12 | |
| *** e0ne has joined #openstack-cinder | 19:13 | |
| *** rasoto has quit IRC | 19:20 | |
| *** pradipta has quit IRC | 19:20 | |
| *** Yogi1 has joined #openstack-cinder | 19:22 | |
| *** cpallares has joined #openstack-cinder | 19:22 | |
| asselin | jgriffith, hemna thingee: https://review.openstack.org/188156 | 19:22 |
| asselin | os-brick job change ^^ | 19:23 |
| patrickeast | whats lib-forward-testing do? | 19:23 |
| asselin | it sets up devstack with pip install of the proposed library patch change, instead of using it from pypi | 19:24 |
| *** rmesta has joined #openstack-cinder | 19:24 | |
| asselin | patrickeast, http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/devstack-gate.yaml#n127 | 19:24 |
| *** ronis has quit IRC | 19:24 | |
| asselin | more comments documented there ^^ | 19:24 |
| *** jungleboyj has quit IRC | 19:25 | |
| patrickeast | oh nice | 19:25 |
| patrickeast | that sounds like a pretty good thing to test | 19:25 |
| *** kvidvans has joined #openstack-cinder | 19:25 | |
| asselin | yes | 19:25 |
| asselin | hemna and I were talking about it yesterday | 19:26 |
| anteaya | so lib-forward-testing is a zuul template: http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n293 | 19:27 |
| asselin | will add some 3rd party ci for it, but probably no need for all drivers to test....but should get a few | 19:27 |
| anteaya | and uses gate-tempest-dsvm-neutron-src-{name} in check and gate | 19:27 |
| anteaya | just confirming this is as you would like | 19:28 |
| *** merooney has joined #openstack-cinder | 19:28 | |
| patrickeast | i was planning to point the pure storage ci at os-brick later next week | 19:28 |
| patrickeast | anteaya: thanks for the link! | 19:28 |
| anteaya | patrickeast: sure | 19:28 |
| anteaya | patrickeast: while we are chatting, can I add you to https://review.openstack.org/#/admin/groups/440,members | 19:28 |
| anteaya | it allows you to add names to https://review.openstack.org/#/admin/groups/270,members | 19:29 |
| patrickeast | anteaya: sounds good to me | 19:29 |
| anteaya | the purpose of that group is those accounts don't send out gerrit email when they comment | 19:29 |
| anteaya | thanks | 19:29 |
| anteaya | you are patrickeast in gerrit? | 19:29 |
| anteaya | gotcha | 19:30 |
| anteaya | thank you | 19:30 |
| *** Lee1092 has quit IRC | 19:31 | |
| *** dustins_ has joined #openstack-cinder | 19:31 | |
| *** dustins has quit IRC | 19:34 | |
| *** rushil has quit IRC | 19:36 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix disconnecting necessary iSCSI sessions issue https://review.openstack.org/188158 | 19:39 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/os-brick: Fix connecting unnecessary iSCSI sessions issue https://review.openstack.org/188159 | 19:39 |
| *** merooney has quit IRC | 19:47 | |
| *** ociuhandu has quit IRC | 19:48 | |
| *** dustins_ has quit IRC | 19:50 | |
| *** timcl has quit IRC | 19:54 | |
| *** cpallares has quit IRC | 19:54 | |
| *** barra204 has joined #openstack-cinder | 19:58 | |
| *** thangp has quit IRC | 20:00 | |
| *** rmesta has quit IRC | 20:01 | |
| *** xyang1 has quit IRC | 20:01 | |
| *** patrickeast_ has joined #openstack-cinder | 20:01 | |
| *** adurbin_ has quit IRC | 20:02 | |
| *** patrickeast has quit IRC | 20:02 | |
| *** patrickeast_ is now known as patrickeast | 20:02 | |
| *** barra204 has quit IRC | 20:03 | |
| *** rmesta has joined #openstack-cinder | 20:05 | |
| *** merooney has joined #openstack-cinder | 20:05 | |
| *** mtanino has quit IRC | 20:06 | |
| *** daneyon_ has quit IRC | 20:08 | |
| *** daneyon has joined #openstack-cinder | 20:08 | |
| *** rushil has joined #openstack-cinder | 20:09 | |
| *** e0ne is now known as e0ne_ | 20:09 | |
| *** jungleboyj has joined #openstack-cinder | 20:11 | |
| *** daneyon_ has joined #openstack-cinder | 20:11 | |
| *** cpallares has joined #openstack-cinder | 20:13 | |
| *** e0ne_ is now known as e0ne | 20:14 | |
| *** e0ne has quit IRC | 20:14 | |
| *** daneyon has quit IRC | 20:14 | |
| *** alexpilotti has quit IRC | 20:17 | |
| winston-d | asselin: when will 3rd party ci against os-brick be ready? | 20:24 |
| winston-d | hemna: there are a few changes ready to land in os-brick, those 2 from tsekiyama and the iscsi retry from jgriffith, do you want to wait for 3rd party ci or we can just merge them? | 20:26 |
| *** merooney has quit IRC | 20:27 | |
| hemna | I was hoping to see if I can get the 3rd party CI working here | 20:27 |
| winston-d | hemna: like right now? | 20:27 |
| hemna | we tried quickly last night to run it | 20:27 |
| hemna | but it didn't pull that patch and install that version of os-brick prior to running cinder tests. | 20:28 |
| hemna | still working on that part. | 20:28 |
| hemna | let me chat with asselin and see where we are | 20:28 |
| winston-d | ok, i +2 to those, you can +A when you feel comfortable | 20:28 |
| hemna | ok thanks for the help! | 20:29 |
| *** ociuhandu has joined #openstack-cinder | 20:29 | |
| winston-d | np | 20:29 |
| nikeshm | hemna:hi afternoon | 20:32 |
| nikeshm | hemna: is there any concern from community on using urllib2 for http request | 20:33 |
| nikeshm | hemna: saw jaypipes comment on that | 20:34 |
| smcginnis | nikeshm: Just left a comment on your review. | 20:34 |
| nikeshm | smcginnis: thanks | 20:44 |
| *** Yogi1 has quit IRC | 20:47 | |
| *** rmesta has quit IRC | 20:47 | |
| hemna | nikeshm, don't use httplib2 | 20:48 |
| hemna | nikeshm, it has issues with SSL and not being able to set the fallback for TLS | 20:48 |
| hemna | nikeshm, use requests if you can. | 20:49 |
| kmartin_ | angela-s, hi just checking to see if you resolved bug https://bugs.launchpad.net/cinder/+bug/1442606, or do you still need help? | 20:49 |
| openstack | Launchpad bug 1442606 in Cinder "Either, the 3PAR WS is not running or the version of the WS is invalid" [Undecided,New] | 20:49 |
| *** rmesta has joined #openstack-cinder | 20:50 | |
| *** dustins has joined #openstack-cinder | 20:50 | |
| asselin | winston-d, I'm working on adding 1st party ci first...I'll add our 3rd party soon | 20:52 |
| asselin | winston-d https://review.openstack.org/188156 | 20:52 |
| *** rushil has quit IRC | 20:52 | |
| nikeshm | hemna:agreed but wanted to go with urllib2 for now,we will look for requests for future enhancement to the driver. | 20:55 |
| scottda | Quick Poll: Should force_detach be exposed to non-admin user in the case of a bootable volume? | 20:55 |
| hemna | nikeshm, fwiw, we are looking at replacing our usage of urllib2 as well for the L release. | 20:55 |
| hemna | winston-d, so for now, I want to hold of merging any os-brick patches | 20:56 |
| hemna | until asselin can get the gate forward checks in place. | 20:56 |
| winston-d | hemna: i'm fine. | 20:56 |
| hemna | that basically ensures that a release of os-brick with the outstanding patch doesn't break cinder. | 20:57 |
| angela-s | kmartin_: its not my bug. saw it was tagged as devstack bug and changed it to cinder | 20:57 |
| asselin | winston-d: shouldn't take too long. Waiting for jenkins to pass & cinder core +1s. https://review.openstack.org/188156 | 20:57 |
| asselin | infra cores will then merge | 20:57 |
| winston-d | asselin: already spent my 1 cent on that. :) | 20:58 |
| winston-d | scottda: mind remind me what exactly does force detach do in cinder? or review url? | 20:58 |
| nikeshm | hemna: CI schedule is very tight and we are working on CI for hpmsa drivers to meet that,as of now can you give +2,in L-2 we can replace urllib2 | 20:59 |
| kmartin_ | angela-s, just saw that...thanks. | 20:59 |
| nikeshm | for dothill drivers | 20:59 |
| scottda | winston-d: Here's the spec: https://review.openstack.org/#/c/172213/5/specs/liberty/implement-force-detach-for-safe-cleanup.rst | 21:01 |
| scottda | currently, it just calls terminate_connection. | 21:01 |
| hemna | nikeshm, yah I'd be ok with that | 21:02 |
| scottda | But Nova needs it exposed in the client in order to implement 'nova --force volume-detach' | 21:02 |
| hemna | nikeshm, land the drivers, then immediately file a bug to upgrade to requests. | 21:02 |
| hemna | then fix that bug prior to L3 | 21:02 |
| scottda | If it were non-admin, it could be written inside regular detach with a force flag. | 21:02 |
| nikeshm | hemna:sure | 21:03 |
| *** Apoorva has quit IRC | 21:04 | |
| asselin | winston-d, you're core right? | 21:04 |
| *** xyang has quit IRC | 21:04 | |
| *** tsekiyama has quit IRC | 21:05 | |
| *** akerr has quit IRC | 21:05 | |
| *** annegentle has quit IRC | 21:06 | |
| earlephilhower | asselin: Thanks for the openstack-dev-ext repos!! You made it only /almost/ impossible for me to set up a CI system, instead of just impossible. :) | 21:10 |
| asselin | you're welcome...:) | 21:10 |
| winston-d | asselin: yeah, cinder core. | 21:10 |
| asselin | earlephilhower, there's progress in the horizon: http://specs.openstack.org/openstack-infra/infra-specs/specs/openstackci.html | 21:11 |
| *** marcusvrn1 has quit IRC | 21:11 | |
| *** rushil has joined #openstack-cinder | 21:12 | |
| *** rmesta has quit IRC | 21:13 | |
| earlephilhower | Cool. One other thing commercial companies may be interested is a CI-as-a-Service for this kind of stuff. We're good at making storage or networking or mobos, but not necessarially experts at CI systems... | 21:13 |
| *** Gomeler has quit IRC | 21:13 | |
| asselin | earlephilhower, there's a lot of ppl taking about that :) | 21:14 |
| asselin | talking* | 21:14 |
| winston-d | scottda: what's the counterpart for Nova of this spec? | 21:18 |
| scottda | https://review.openstack.org/#/c/84048/ | 21:18 |
| winston-d | scottda: trying to understand what's done at Nova side for 'nova --force volume-detach' | 21:19 |
| winston-d | scottda: thx | 21:19 |
| scottda | That's an old spec that has just been un-abandoned. | 21:19 |
| nikeshm | hemna: need +2 from you and drivers will land :) and we will make sure will add requests before L-3 | 21:19 |
| scottda | It could use some clean-up. But I spoke with Nova at Vancouver summit and they are supportive of this. | 21:19 |
| *** sjcherry has quit IRC | 21:20 | |
| winston-d | scottda: i'm not very clear about what will be done at Nova side when 'nova --force vol-deatch' is issued after reading the nova spec. | 21:27 |
| *** dalgaaf has quit IRC | 21:28 | |
| winston-d | scottda: will that be total different code path compared to 'nova vol-detach' or just be a few enhancement on current one? | 21:28 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Added verify cert option for REST calls https://review.openstack.org/182396 | 21:31 |
| scottda | yes, I'd like to make some things clearer in that spec... | 21:31 |
| scottda | The basic idea is to have a top-to-bottom cleanup for a volume stuck in detaching/attaching. | 21:31 |
| scottda | So (not necessarily in this order): Nova would cleanup BlockDeviceMapping table to remove the entry | 21:32 |
| scottda | Nova would call libvirt.detatch | 21:32 |
| scottda | Nova would call cinder force_detach | 21:32 |
| scottda | Cinder would call driver force_detach. Default implementation would just terminate_connnection and detach() as it does today. | 21:33 |
| scottda | Cinder would set volume state in DB to 'available' | 21:33 |
| scottda | And entire process would carry on regardless of what happened along the way, and be idempotent. | 21:33 |
| *** lpetrut has quit IRC | 21:34 | |
| scottda | Nova implementation would skip state checks for cinder. And have try: blocks so that if libvirt.detach failed, or the BDM entry was missing, it would still call Cinder force_detach | 21:34 |
| winston-d | new cinder api will need to check for instance uuid to make sure it doesn't shoot the wrong guy, if the volume is currently 'in-use'/attached to some other instance. | 21:37 |
| scottda | Well, OK, if it is non-admin. But if it were admin-only maybe we should trust that the admin knows what he/she is doing and allow it no matter what. | 21:38 |
| *** dontalton has joined #openstack-cinder | 21:38 | |
| asselin | winston-d hemna approved...will be merged soon | 21:40 |
| winston-d | scottda: well, admin makes mistakes too, what if he/she accidently pasted the wrong vol uuid from one of his/her tickets. | 21:40 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Added support for alternate iscsi portals https://review.openstack.org/182760 | 21:40 |
| scottda | I agree. It'd been hurting my head to think of all the corner cases, and how to implement. | 21:40 |
| winston-d | scottda: but i think nova api gets that covered, api will get BDM to make sure instance-vol pair exists before it proceeds to compute/api | 21:41 |
| scottda | I guess one question is, how do you clean up these things? and others? | 21:43 |
| scottda | I'd like to replace the use of SQL on a production DB. | 21:43 |
| scottda | and all the clunky steps, some of which aren't known to the operator. | 21:43 |
| winston-d | and we did saw inconsistency between api cell DB and child cell DB, for BDM records, if using nova cell | 21:44 |
| scottda | What if there is no BDM entry, but cinder DB shows 'in-use' and there is still an iscsi connection to the compute host? Should this get cleaned up from Nova, or should the admin need to use cinder force-detach? | 21:44 |
| *** Apoorva has joined #openstack-cinder | 21:45 | |
| winston-d | i think we met this case several times too | 21:45 |
| scottda | The basic issue is that state exists in all these places, and gets out of sync. How to give safe tools to the operator to clean this up? | 21:45 |
| *** alexpilotti has joined #openstack-cinder | 21:45 | |
| *** Apoorva has quit IRC | 21:46 | |
| winston-d | one thing i am not sure is, how would it impact the hypervisor when disconnecting a iSCSI session from target side? | 21:46 |
| winston-d | would it just leave some logs in syslog or cause kernel panic? | 21:47 |
| scottda | I think Nova should call libvirt.detach first, but I am not sure if that is the correct solution. | 21:47 |
| winston-d | scottda: in the case you just mentioned, not BDM entry, should nova still try libvirt.detach? | 21:48 |
| winston-d | we know we should | 21:48 |
| scottda | My thinking is yes, it should try everything and expect that some things are going to fail or already be cleaned up. | 21:48 |
| winston-d | let me check the nova code see if it would puke in such case | 21:48 |
| scottda | ok, I appreciate it. | 21:49 |
| winston-d | well, libvirt.detach_volume will check against instance XML to find the device, if not, it raises DiskNotFound. | 21:51 |
| *** ianbrown has joined #openstack-cinder | 21:52 | |
| scottda | my thinking is... try: libvirt.detach_volume except: #log some stuff but don't raise | 21:53 |
| scottda | if there's no XML, fine, proceed to check BDM table. If nothing there, fine, proceed to call Cinder for cleanup on cinder side (DB and backend export) | 21:54 |
| winston-d | yeah, and call libvirt.driver.disconnect_volume() to remove the iscsi session | 21:54 |
| scottda | yes. But also give Cinder a chance with terminate_connection just in case. Belt and suspenders mentality. | 21:55 |
| winston-d | right | 21:55 |
| *** lpetrut has joined #openstack-cinder | 21:55 | |
| *** lpetrut has quit IRC | 21:56 | |
| *** mriedem has quit IRC | 21:57 | |
| *** adurbin__ has quit IRC | 21:57 | |
| scottda | Do you have any ideas for the cinder force_detach spec? I need to clean some stuff up. You had pointed out that I was mistaken about not being able to attach a volume to >1 instance if reset-state was used to go back to 'available'. | 21:58 |
| scottda | Please let me know if you see anything else that raises a flag. i want to upload another patch, but I hate to lose all those +1's :) | 21:58 |
| *** ianbrown has quit IRC | 22:00 | |
| *** tsekiyama has joined #openstack-cinder | 22:04 | |
| winston-d | scottda: i will review the cinder/nova spec. | 22:04 |
| winston-d | scottda: and you will have my +1s :) | 22:04 |
| *** ianbrown has joined #openstack-cinder | 22:05 | |
| winston-d | we suffer a lot from attach/detach, so i'd love see this sliver bullet being made for cleaning up | 22:05 |
| scottda | Thanks. I really appreciate your insight into this. The force_detach is my highest priority, but then I want to work on the general Nova <-> cinder api, so we'll be talking more about that as well, i am sure. | 22:05 |
| winston-d | sure | 22:06 |
| winston-d | back to boot from volume though, why a non-admin user can't do force detach, if he/she owns the vol and instance? | 22:07 |
| scottda | I don't really have a good reason. I'm just asking the question in case others have reasons. | 22:08 |
| scottda | It is much less clunky to use existing detach with a force flag, which is non-admin. Saves a lot of code duplication. | 22:08 |
| *** tsekiyama has quit IRC | 22:09 | |
| scottda | And you might want to force-detach a boot volume, since the persistent volume is your 'pet' and a new instance could then be spun up easily. | 22:09 |
| scottda | as thingee said, if it is safe, why not let the non-admin user do it? | 22:09 |
| winston-d | maybe it's 'cos 'delete instance' is a better option in such case? it does what 'force detach' does? | 22:11 |
| scottda | Yes. But is that confusing for operators? I think that there are many that don't know as much as Cinder/Nova devs about all the moving parts involved. | 22:14 |
| *** primechu_ has quit IRC | 22:15 | |
| winston-d | unplugging (hotplug) the disk of / partition or c:/ from a running OS isn't fun. so Nova doesn't allow 'detaching' volume if volume is the root disk. | 22:20 |
| *** jamielennox|away is now known as jamielennox | 22:20 | |
| *** primechuck has joined #openstack-cinder | 22:20 | |
| winston-d | doesn't allow force detach of root disk, is at least consistent | 22:20 |
| winston-d | it starts to make sense: if a volume is the root disk, cleaning things up will take 2 steps: 1) delete the instance; 2) force detach. | 22:22 |
| *** bill_az has joined #openstack-cinder | 22:23 | |
| scottda | fair enough. But Nova will call the cinderclient force_detach, so this will be exposed. | 22:24 |
| scottda | Can cinderclient or cinder force_detach verify that a volume is not a root disk? | 22:24 |
| *** Adriano_ has quit IRC | 22:26 | |
| thingee | Drivers for liberty - this is how I see it so far DuncanT jgriffith smcginnis eharney jungleboyj hemna winston-d https://etherpad.openstack.org/p/cinder-liberty-drivers | 22:27 |
| winston-d | scottda: this https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/volumes.py#L412 will prevent nova from taking further actions, i think force detach should follow. | 22:29 |
| *** rushil has quit IRC | 22:29 | |
| *** gouthamr has quit IRC | 22:31 | |
| scottda | yes, from the Nova side for 'nova --force volume-detach' . But in order for Nova to call into cinder force_detach we'll need to expose it using the cinderclient, right? | 22:31 |
| winston-d | scottda: if bfv instance has been deleted, then the volume is no longer root disk, everything can move on | 22:32 |
| winston-d | just like normal non-root volumes | 22:32 |
| scottda | Yes, and I'm not sure I understand correctly, but to have Nova be able to call cinder force_detach, won't this need to be available in the cinderclient? Meaning, could an admin call 'cinder force-detach <volume>' on a bootable volume? | 22:33 |
| *** mriedem has joined #openstack-cinder | 22:34 | |
| scottda | Even if the volume was still attached to an instance as a root boot-from-volume | 22:35 |
| *** mriedem has quit IRC | 22:37 | |
| winston-d | we could allow admin to do that, but it would leave a dead instance for him/her to clean up after this. | 22:37 |
| scottda | OK. I didn't think that there was much we could do about that case. | 22:38 |
| *** mriedem has joined #openstack-cinder | 22:38 | |
| winston-d | yes, so I'd vote for don't allow detach root-disk volume. | 22:38 |
| scottda | I agree, from the Nova side where it can be checked. | 22:38 |
| winston-d | right | 22:39 |
| *** vilobhmm has quit IRC | 22:41 | |
| *** vilobhmm has joined #openstack-cinder | 22:41 | |
| *** vilobhmm1 has joined #openstack-cinder | 22:44 | |
| *** vilobhmm has quit IRC | 22:46 | |
| *** Apoorva has joined #openstack-cinder | 22:47 | |
| *** barra204 has joined #openstack-cinder | 22:48 | |
| *** Apoorva has quit IRC | 22:51 | |
| *** barra204 has quit IRC | 22:53 | |
| nikeshm | hemna: hi | 22:54 |
| *** Apoorva has joined #openstack-cinder | 22:56 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: ConsistencyGroup: Return 400 instead of 500 for invalid body https://review.openstack.org/186669 | 22:58 |
| *** akshai has quit IRC | 23:01 | |
| *** dustins has quit IRC | 23:03 | |
| *** primechuck has quit IRC | 23:05 | |
| nikeshm | thingee: hi looks like hemna is not online,can you give +2 on https://review.openstack.org/#/c/177665/ | 23:14 |
| *** ganso_ has quit IRC | 23:14 | |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Return provider_id in SolidFire model update https://review.openstack.org/187233 | 23:19 |
| *** chlong has joined #openstack-cinder | 23:19 | |
| *** IlyaG has joined #openstack-cinder | 23:21 | |
| *** tsekiyama has joined #openstack-cinder | 23:25 | |
| *** dontalton has quit IRC | 23:37 | |
| *** rwsu has quit IRC | 23:41 | |
| *** dkalleg has joined #openstack-cinder | 23:45 | |
| *** smoriya has joined #openstack-cinder | 23:48 | |
| *** dims_ has joined #openstack-cinder | 23:51 | |
| nikeshm | hemna: we filed the bug | 23:52 |
| nikeshm | https://bugs.launchpad.net/cinder/+bug/1461718 | 23:52 |
| openstack | Launchpad bug 1461718 in Cinder "switch from urllib2 to requests in DotHill Drivers" [Undecided,New] - Assigned to Nikesh (nikesh-mahalaka) | 23:52 |
| *** dims__ has quit IRC | 23:55 | |
| nikeshm | hemna: thanks,whats your view on HPMSA | 23:59 |
| nikeshm | drivers | 23:59 |
| nikeshm | we are working on CI | 23:59 |
| nikeshm | for that | 23:59 |
| nikeshm | https://review.openstack.org/#/c/187707/ | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!