Monday, 2017-02-13

*** ducttape_ has joined #openstack-cinder00:02
*** lucas_ has quit IRC00:05
*** ducttape_ has quit IRC00:09
*** yangyape_ has quit IRC00:09
*** Jack_Iv has joined #openstack-cinder00:11
*** Julien-zte has joined #openstack-cinder00:13
*** Jack_Iv has quit IRC00:15
*** jamielennox is now known as jamielennox|away00:24
*** tommylikehu has quit IRC00:25
*** tommylikehu has joined #openstack-cinder00:26
*** ducttape_ has joined #openstack-cinder00:26
*** stakeda has joined #openstack-cinder00:32
*** catintheroof has joined #openstack-cinder00:36
*** catintheroof has quit IRC00:37
*** wanghao has joined #openstack-cinder00:37
*** lucas_ has joined #openstack-cinder00:38
*** ksumit has quit IRC00:48
*** jamielennox|away is now known as jamielennox00:50
*** ducttape_ has quit IRC01:07
*** ducttape_ has joined #openstack-cinder01:09
*** Jack_Iv has joined #openstack-cinder01:12
*** yangyapeng has joined #openstack-cinder01:14
*** Jack_Iv has quit IRC01:16
*** sileht has quit IRC01:19
*** sileht has joined #openstack-cinder01:22
*** salv-orlando has joined #openstack-cinder01:23
*** knomura has joined #openstack-cinder01:24
*** ducttape_ has quit IRC01:27
*** salv-orlando has quit IRC01:27
*** knomura has quit IRC01:28
*** crushil has joined #openstack-cinder01:30
*** ducttape_ has joined #openstack-cinder01:39
*** lixiaoy1 has joined #openstack-cinder01:54
tommylikehuhey anyone would spend sometime on this simple documenation patch?  The server's code patch has been merged two months ago.  https://review.openstack.org/#/c/395464/01:56
*** jdurgin1 has quit IRC02:05
openstackgerritwangxiyuan proposed openstack/cinder master: Remove "list_volume" from filter  https://review.openstack.org/43280902:06
*** Jack_Iv has joined #openstack-cinder02:13
*** Jack_Iv has quit IRC02:17
*** edtubill has joined #openstack-cinder02:25
*** knomura has joined #openstack-cinder02:25
*** wanghao has quit IRC02:27
*** wanghao has joined #openstack-cinder02:28
*** knomura has quit IRC02:30
*** ducttape_ has quit IRC02:38
*** crushil has quit IRC02:55
*** gouthamr has quit IRC03:03
*** wanghao_ has joined #openstack-cinder03:07
*** The_Ball has quit IRC03:08
*** wanghao has quit IRC03:10
*** diablo_rojo has joined #openstack-cinder03:11
*** ducttape_ has joined #openstack-cinder03:18
*** salv-orlando has joined #openstack-cinder03:24
*** salv-orlando has quit IRC03:28
*** wanghao_ has quit IRC03:30
*** crushil has joined #openstack-cinder03:31
openstackgerritTommyLike proposed openstack/python-cinderclient master: Add filter argument to API Get-Pools  https://review.openstack.org/38398903:34
*** wanghao has joined #openstack-cinder03:37
openstackgerritZhao Liqiang proposed openstack/cinder master: RBD: get replication flags from 'diff'.  https://review.openstack.org/42167803:38
*** ducttape_ has quit IRC03:43
*** alonma has joined #openstack-cinder03:43
*** crushil has quit IRC03:43
*** sudipto has joined #openstack-cinder03:47
*** sudipto_ has joined #openstack-cinder03:47
*** alonma has quit IRC03:47
*** imacdonn_ has quit IRC03:53
*** imacdonn_ has joined #openstack-cinder03:54
*** dfisher has quit IRC03:55
*** dfisher has joined #openstack-cinder03:55
*** ducttape_ has joined #openstack-cinder04:07
*** Dinesh_Bhor has joined #openstack-cinder04:08
*** Julien-zte has quit IRC04:09
*** Julien-z_ has joined #openstack-cinder04:10
*** mdnadeem has joined #openstack-cinder04:10
*** tommylikehu1 has joined #openstack-cinder04:10
*** psachin has joined #openstack-cinder04:10
*** tommylikehu has quit IRC04:14
*** tommylikehu1 is now known as tommylikehu04:14
*** diablo_rojo has quit IRC04:14
*** Julien-z_ has quit IRC04:15
*** Julien-zte has joined #openstack-cinder04:15
*** lucas_ has quit IRC04:20
*** knomura has joined #openstack-cinder04:26
*** sudipto has quit IRC04:28
*** sudipto_ has quit IRC04:28
*** knomura has quit IRC04:31
*** asselin_ has joined #openstack-cinder04:32
*** openstackgerrit has quit IRC04:32
*** asselin has quit IRC04:34
*** hoonetorg has quit IRC04:34
*** ducttape_ has quit IRC04:41
*** faizy has joined #openstack-cinder04:46
*** hoonetorg has joined #openstack-cinder04:48
*** lucas_ has joined #openstack-cinder04:51
*** sudipto has joined #openstack-cinder04:54
*** sudipto_ has joined #openstack-cinder04:54
*** salv-orlando has joined #openstack-cinder04:54
*** lixiaoy1 has quit IRC04:55
*** salv-orlando has quit IRC04:59
*** adisky_ has joined #openstack-cinder05:00
*** lucas_ has quit IRC05:01
*** stakeda has quit IRC05:03
*** lixiaoy1 has joined #openstack-cinder05:04
*** pdeore has joined #openstack-cinder05:11
*** crushil has joined #openstack-cinder05:12
*** mtreinish has quit IRC05:20
*** knomura has joined #openstack-cinder05:28
*** mtreinish has joined #openstack-cinder05:31
*** alonmarx has joined #openstack-cinder05:31
*** knomura has quit IRC05:32
*** ducttape_ has joined #openstack-cinder05:41
*** ducttape_ has quit IRC05:46
*** moiz has joined #openstack-cinder05:47
*** alonma has joined #openstack-cinder05:50
*** Jack_Iv has joined #openstack-cinder05:50
*** asselin has joined #openstack-cinder05:53
*** alonma has quit IRC05:54
*** crushil has quit IRC05:55
*** asselin_ has quit IRC05:55
*** salv-orlando has joined #openstack-cinder05:55
*** bkopilov_ has quit IRC05:58
*** bkopilov_ has joined #openstack-cinder05:59
*** salv-orlando has quit IRC06:00
*** gcb has joined #openstack-cinder06:00
*** bkopilov_ has quit IRC06:04
*** bkopilov_ has joined #openstack-cinder06:04
*** pgadiya has joined #openstack-cinder06:07
*** openstackgerrit has joined #openstack-cinder06:08
openstackgerritLisaLi proposed openstack/cinder master: Use ovo instead of dict  https://review.openstack.org/40678006:08
*** lucas_ has joined #openstack-cinder06:12
openstackgerritLisaLi proposed openstack/cinder master: Use ovo instead of dict  https://review.openstack.org/40678006:16
*** edtubill has quit IRC06:18
*** sandanar has joined #openstack-cinder06:18
*** lpetrut has joined #openstack-cinder06:21
*** knomura has joined #openstack-cinder06:23
sandanarhttps://review.openstack.org/#/c/428661/ this review request is waiting for one more +2 .. Can someone review this? thanks06:28
*** faizy has quit IRC06:31
*** knomura_ has joined #openstack-cinder06:31
*** faizy has joined #openstack-cinder06:31
*** psachin has quit IRC06:33
*** knomura has quit IRC06:33
*** raunak has joined #openstack-cinder06:35
*** faizy has quit IRC06:35
*** knomura_ has quit IRC06:36
*** faizy has joined #openstack-cinder06:36
*** knomura has joined #openstack-cinder06:36
*** faizy has quit IRC06:37
*** knomura has quit IRC06:37
*** knomura has joined #openstack-cinder06:38
*** faizy has joined #openstack-cinder06:38
*** faizy has quit IRC06:41
*** faizy has joined #openstack-cinder06:41
*** knomura has quit IRC06:42
*** salv-orlando has joined #openstack-cinder06:43
*** lucas_ has quit IRC06:46
*** psachin has joined #openstack-cinder06:50
*** psachin has quit IRC06:54
*** psachin has joined #openstack-cinder06:54
*** salv-orl_ has joined #openstack-cinder07:00
*** Jack_Iv has quit IRC07:00
*** Jack_I has joined #openstack-cinder07:02
*** salv-orlando has quit IRC07:03
*** alonma has joined #openstack-cinder07:04
*** pkshiras__ has joined #openstack-cinder07:08
*** pkshiras__ has quit IRC07:08
*** pkshiras__ has joined #openstack-cinder07:08
*** pkshiras__ has quit IRC07:08
*** alonma has quit IRC07:09
*** pkshiras has joined #openstack-cinder07:09
*** pkshiras has quit IRC07:09
*** pkshiras has joined #openstack-cinder07:10
*** pkshiras has quit IRC07:10
*** Poornima_K has joined #openstack-cinder07:10
openstackgerritMerged openstack/cinder master: VNX: Handle error during volume deletion  https://review.openstack.org/43067507:12
*** nkrinner_afk is now known as nkrinner07:12
*** rcernin has joined #openstack-cinder07:15
*** lpetrut has quit IRC07:17
*** huanxuan has joined #openstack-cinder07:19
*** lkuchlan has joined #openstack-cinder07:25
*** Cibo_ has joined #openstack-cinder07:28
*** tesseract has joined #openstack-cinder07:28
*** tesseract is now known as tesseract-RH07:29
*** ircuser-1 has joined #openstack-cinder07:30
*** newmember has quit IRC07:30
*** newmember has joined #openstack-cinder07:31
*** Poornima_K has quit IRC07:37
*** ducttape_ has joined #openstack-cinder07:43
*** arne_r has joined #openstack-cinder07:44
*** ducttape_ has quit IRC07:47
*** pcaruana has joined #openstack-cinder07:49
*** gcb has quit IRC07:57
openstackgerritSilvan Kaiser proposed openstack/cinder master: Removes getfattr from Quobyte Cinder driver  https://review.openstack.org/42783307:58
openstackgerritZhao Liqiang proposed openstack/os-brick master: RBD: Can not delete temporary configuration file.  https://review.openstack.org/43290508:15
*** faizy has quit IRC08:19
*** faizy has joined #openstack-cinder08:20
*** raunak has quit IRC08:31
*** lpetrut has joined #openstack-cinder08:35
*** Jack_Iv has joined #openstack-cinder08:36
*** Poornima_K has joined #openstack-cinder08:44
*** Poornima_K has quit IRC08:48
*** lixiaoy1 has quit IRC08:53
*** Poornima_K has joined #openstack-cinder08:53
*** lpetrut has quit IRC08:56
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
openstackgerritTommyLike proposed openstack/cinder master: [4/4]Reset generic volume group and group snapshot statuses  https://review.openstack.org/39546409:00
*** faizy has quit IRC09:01
*** faizy has joined #openstack-cinder09:02
*** alonmarx has quit IRC09:08
*** faizy_ has joined #openstack-cinder09:11
*** faizy has quit IRC09:15
*** faizy_ has quit IRC09:24
openstackgerritTommyLike proposed openstack/cinder master: [4/4]Reset generic volume group and group snapshot statuses  https://review.openstack.org/39546409:30
*** wanghao has quit IRC09:35
*** Poornima_K has quit IRC09:36
*** alonma has joined #openstack-cinder09:38
*** Jack_Iv has quit IRC09:40
*** faizy has joined #openstack-cinder09:42
*** ducttape_ has joined #openstack-cinder09:44
openstackgerritVipin Balachandran proposed openstack/cinder master: Remove backup-restore methods in driver code  https://review.openstack.org/43294809:44
*** Julien-zte has quit IRC09:45
*** faizy has quit IRC09:47
*** jamielennox is now known as jamielennox|away09:48
*** ducttape_ has quit IRC09:48
*** e0ne has joined #openstack-cinder09:52
*** e0ne has quit IRC09:54
*** The_Ball has joined #openstack-cinder09:57
*** lpetrut has joined #openstack-cinder09:57
*** e0ne has joined #openstack-cinder09:58
*** archies has joined #openstack-cinder09:58
openstackgerritTommyLike proposed openstack/cinder-specs master: Cinder volume revert to snapshot  https://review.openstack.org/31654009:59
*** e0ne has quit IRC10:02
*** e0ne has joined #openstack-cinder10:07
*** e0ne has quit IRC10:09
*** e0ne has joined #openstack-cinder10:13
*** e0ne_ has joined #openstack-cinder10:15
*** e0ne has quit IRC10:15
*** Poornima_K has joined #openstack-cinder10:16
*** dims_ has quit IRC10:16
*** e0ne_ has quit IRC10:21
*** yangyapeng has quit IRC10:22
*** dims has joined #openstack-cinder10:23
*** jamielennox|away is now known as jamielennox10:24
*** salv-orl_ has quit IRC10:24
*** yumiriam has joined #openstack-cinder10:25
*** salv-orlando has joined #openstack-cinder10:25
*** e0ne has joined #openstack-cinder10:25
*** e0ne has quit IRC10:27
*** e0ne has joined #openstack-cinder10:32
*** archies has quit IRC10:33
*** pdeore has quit IRC10:35
*** archies has joined #openstack-cinder10:37
*** mvk has quit IRC10:40
*** e0ne has quit IRC10:41
archies@smcginnis, cinder team, we are trying to implement the cinder tempest cases for automating few sanity scenarios. I never worked on it. Would appreciate any help on getting started with tempest10:43
*** e0ne has joined #openstack-cinder10:45
*** mdovgal has joined #openstack-cinder10:49
*** e0ne has quit IRC10:51
*** Jack_Iv has joined #openstack-cinder10:58
*** dalgaaf has quit IRC10:58
*** sudipto has quit IRC11:01
*** sudipto_ has quit IRC11:01
*** dalgaaf has joined #openstack-cinder11:01
*** pdeore has joined #openstack-cinder11:02
openstackgerritTommyLike proposed openstack/cinder-specs master: Cinder volume revert to snapshot  https://review.openstack.org/31654011:05
*** imi72 has joined #openstack-cinder11:07
*** Poornima_K has quit IRC11:07
*** psachin has quit IRC11:08
*** mvk has joined #openstack-cinder11:09
tommylikehuhey erlon, DuncanT, eharney I uploaded a new version of spec: revert-to-snapshot which focus on your main concerns there, hope you have sometime on that before PTG, thanks:  https://review.openstack.org/#/c/316540/11:13
*** huanxuan has quit IRC11:17
*** Jack_Iv has quit IRC11:17
*** ians has joined #openstack-cinder11:18
*** openstackgerrit has quit IRC11:18
*** ians has quit IRC11:18
*** sdague has joined #openstack-cinder11:19
*** alonma has quit IRC11:21
*** alonma has joined #openstack-cinder11:22
*** Poornima_K has joined #openstack-cinder11:23
*** Julien-zte has joined #openstack-cinder11:25
*** psachin has joined #openstack-cinder11:25
*** alonma has quit IRC11:26
*** alonma has joined #openstack-cinder11:30
*** pgadiya has quit IRC11:31
*** nicolasbock has joined #openstack-cinder11:31
*** Poornima_K has quit IRC11:34
*** e0ne has joined #openstack-cinder11:34
*** openstackgerrit has joined #openstack-cinder11:38
openstackgerritTommyLike proposed openstack/cinder master: [4/4]Reset generic volume group and group snapshot statuses  https://review.openstack.org/39546411:38
*** sudipto has joined #openstack-cinder11:38
*** sudipto_ has joined #openstack-cinder11:38
*** psachin has quit IRC11:39
*** imi72 has quit IRC11:42
*** ducttape_ has joined #openstack-cinder11:44
*** e0ne has quit IRC11:45
*** e0ne has joined #openstack-cinder11:47
*** e0ne has quit IRC11:48
*** e0ne has joined #openstack-cinder11:48
*** e0ne has quit IRC11:48
*** e0ne has joined #openstack-cinder11:49
*** ducttape_ has quit IRC11:49
*** e0ne has quit IRC11:49
*** e0ne has joined #openstack-cinder11:50
*** e0ne has quit IRC11:50
*** e0ne has joined #openstack-cinder11:50
*** e0ne has quit IRC11:51
*** e0ne has joined #openstack-cinder11:51
*** imacdonn_ has quit IRC11:51
*** e0ne has quit IRC11:52
*** porrua has joined #openstack-cinder11:55
*** yangyapeng has joined #openstack-cinder12:00
*** dave-mccowan has joined #openstack-cinder12:04
*** ebalduf has quit IRC12:08
*** psachin has joined #openstack-cinder12:12
*** ebalduf has joined #openstack-cinder12:16
*** alonma has quit IRC12:17
*** alonma has joined #openstack-cinder12:18
*** isaacb has joined #openstack-cinder12:20
*** isaacb_ has joined #openstack-cinder12:20
*** isaacb_ has left #openstack-cinder12:21
*** alonma has quit IRC12:22
*** archies has quit IRC12:23
*** dims has quit IRC12:23
*** dave-mcc_ has joined #openstack-cinder12:28
*** dave-mccowan has quit IRC12:30
*** ChatSharp has joined #openstack-cinder12:36
*** ChatSharp has left #openstack-cinder12:39
*** catintheroof has joined #openstack-cinder12:41
*** ducttape_ has joined #openstack-cinder12:45
*** gcb has joined #openstack-cinder12:50
*** pdeore has quit IRC12:53
*** gouthamr has joined #openstack-cinder12:58
*** EZuck has joined #openstack-cinder13:04
EZuckHi. I have an cinder driver and I would like to apply a bug fix to it.13:04
EZuckI have opened a bug but how can I apply the bug fix to that bug ?13:05
smcginnisEZuck: When you submit the patch, in the commit message, include the line "Closes-bug: #[bugnumber]"13:08
smcginnisEZuck: There's a little more detail here: http://docs.openstack.org/infra/manual/developers.html13:08
EZuckThanks!13:12
smcginnisEZuck: No problem. Just let us know if you have any questions submitting the patch.13:12
*** ociuhandu has quit IRC13:13
*** Poornima_K has joined #openstack-cinder13:14
*** dims has joined #openstack-cinder13:20
*** ducttape_ has quit IRC13:20
*** eharney has quit IRC13:27
*** alonma has joined #openstack-cinder13:28
*** salv-orlando has quit IRC13:29
*** mikal_ has joined #openstack-cinder13:30
*** laurent\_ has joined #openstack-cinder13:30
*** winston-1 has joined #openstack-cinder13:31
*** guitarza1 has joined #openstack-cinder13:31
*** guitarzan has quit IRC13:32
*** winston-d has quit IRC13:32
*** mikal has quit IRC13:32
*** dolphm has quit IRC13:32
*** laurent\ has quit IRC13:32
*** fnordahl has quit IRC13:32
*** dolphm_ has joined #openstack-cinder13:32
*** fnordahl has joined #openstack-cinder13:32
*** dolphm_ is now known as dolphm13:33
openstackgerritMichael Dovgal proposed openstack/cinder master: Fix size update error for snapshot-manage operation  https://review.openstack.org/42616213:34
openstackgerritElad Zucker proposed openstack/cinder master: fix a bug in the xtremio driver which cause a create snapshot from generic group with volumes to fail. Closes-bug :#1664209  https://review.openstack.org/43305913:36
smcginnisEZuck: Can you reformat that commit message to follow these guidelines? https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure13:38
smcginnisEZuck: Just do a "git commit --amend" and edit the commit message.13:38
*** dustins has joined #openstack-cinder13:43
*** dims_ has joined #openstack-cinder13:43
*** dims has quit IRC13:45
*** kwathore has joined #openstack-cinder13:46
EZuckYes.. no problem.13:46
EZucksmcginnis : are you reffering to the number of character per line issue ?13:48
EZucksmcginnis : is there something else ?13:48
*** salv-orlando has joined #openstack-cinder13:49
*** mtanino has joined #openstack-cinder13:49
smcginnisEZuck: Yes, and the format. So the first line should be a summary, ideally within 50 characters. Then a blank line, followed by a more detailed description of the change.13:49
smcginnisEZuck: Then usually another blank line and the "Closed-bug" and commit ID lines.13:50
EZuckgot it.13:50
EZuckthanks13:50
*** ociuhandu has joined #openstack-cinder13:51
openstackgerritElad Zucker proposed openstack/cinder master: create snapshot with generic group fail with XtremIO Driver.  https://review.openstack.org/43305913:51
kwathoreHello folks, Please review  https://review.openstack.org/#/c/409124/13:52
*** eharney has joined #openstack-cinder13:59
*** eharney has quit IRC14:03
*** alonma has quit IRC14:03
*** dave-mcc_ has quit IRC14:03
*** mvk has quit IRC14:03
*** alonma has joined #openstack-cinder14:04
openstackgerritShlomi Avihou proposed openstack/cinder master: Zadara driver - move from httplib to requests  https://review.openstack.org/43306914:04
*** crushil has joined #openstack-cinder14:05
*** dave-mcc_ has joined #openstack-cinder14:07
*** ducttape_ has joined #openstack-cinder14:07
*** ducttape_ has quit IRC14:07
*** ducttape_ has joined #openstack-cinder14:07
*** alonma has quit IRC14:08
*** alonma has joined #openstack-cinder14:09
*** jwcroppe has quit IRC14:13
*** ducttape_ has quit IRC14:14
*** jperry has joined #openstack-cinder14:14
*** mdnadeem has quit IRC14:14
*** eharney has joined #openstack-cinder14:16
*** mvk has joined #openstack-cinder14:18
*** cdelatte has joined #openstack-cinder14:21
*** cknight has joined #openstack-cinder14:23
openstackgerritAshish Billore proposed openstack/cinder master: Cinder tox updates for Python3  https://review.openstack.org/43308114:24
*** HOS has joined #openstack-cinder14:24
*** mriedem has joined #openstack-cinder14:27
*** jwcroppe has joined #openstack-cinder14:27
*** rhefner has joined #openstack-cinder14:28
*** jonesn has joined #openstack-cinder14:32
*** jamielennox is now known as jamielennox|away14:39
*** ducttape_ has joined #openstack-cinder14:44
*** Jack_V has joined #openstack-cinder14:44
*** Jack_I has quit IRC14:45
*** shaner has quit IRC14:46
*** shaner has joined #openstack-cinder14:47
*** pcaruana has quit IRC14:50
*** xyang has joined #openstack-cinder14:52
*** pcaruana has joined #openstack-cinder14:55
*** Poornima_K has quit IRC14:57
*** dave-mcc_ has quit IRC14:57
*** jose-phillips has joined #openstack-cinder15:00
openstackgerritGorka Eguileor proposed openstack/os-brick master: Fix multipath flush when using friendly names  https://review.openstack.org/43310215:03
openstackgerritGorka Eguileor proposed openstack/os-brick master: Retry multipath flush when map is in use  https://review.openstack.org/43310315:03
openstackgerritGorka Eguileor proposed openstack/os-brick master: Fix iSCSI multipath rescan  https://review.openstack.org/43310415:03
smcginnisgeguileo: Only three patches? I expected more from you. :P15:06
*** arne_r has quit IRC15:06
geguileosmcginnis: Yeah, they should be used together wit another open-iscsi patch  ;-P15:06
smcginnis:)15:06
geguileosmcginnis: And that was a little more troublesome: https://github.com/Akrog/open-iscsi/commit/56d7e5704871899ee7d9993341c9923835dd0a3a15:07
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/43311015:10
*** salv-orlando has quit IRC15:11
*** dave-mccowan has joined #openstack-cinder15:11
*** salv-orlando has joined #openstack-cinder15:11
hemnamornin15:12
jungleboyjhemna, Morning.15:12
*** jose-phillips has quit IRC15:16
*** jose-phi_ has joined #openstack-cinder15:17
*** lucas_ has joined #openstack-cinder15:18
*** tlian has joined #openstack-cinder15:18
*** _ducttape_ has joined #openstack-cinder15:21
*** ducttape_ has quit IRC15:25
*** dave-mccowan has quit IRC15:29
*** Jack_Iv has joined #openstack-cinder15:31
*** dave-mccowan has joined #openstack-cinder15:33
*** lucas_ is now known as lucasxu15:34
*** rajinir_ has joined #openstack-cinder15:34
*** mriedem1 has joined #openstack-cinder15:35
prometheanfireany other +2 for https://review.openstack.org/432794 ?15:36
prometheanfire:D15:36
*** sandanar has quit IRC15:37
*** mriedem has quit IRC15:38
*** dfisher has left #openstack-cinder15:42
*** lucasxu has quit IRC15:42
*** lucas_ has joined #openstack-cinder15:43
*** lucas_ has quit IRC15:44
*** psachin has quit IRC15:44
*** lucasxu has joined #openstack-cinder15:44
*** mriedem1 is now known as mriedem15:48
*** lucasxu has quit IRC15:48
-openstackstatus- NOTICE: We are currently investigating an issue with our AFS mirrors which is causing some projects jobs to fail. We are working to correct the issue.15:49
*** lucasxu has joined #openstack-cinder15:49
*** mvk has quit IRC15:53
*** salv-orlando has quit IRC15:54
openstackgerritEric Harney proposed openstack/cinder master: Tests: Remove mocks of nonexistent methods (NetApp)  https://review.openstack.org/43314915:56
*** jose-phi_ has quit IRC15:56
prometheanfireeharney: thanks15:57
*** _ducttape_ has quit IRC15:58
*** ducttape_ has joined #openstack-cinder15:59
*** Julien-z_ has joined #openstack-cinder16:11
*** Julien-zte has quit IRC16:12
*** Jack_Iv has quit IRC16:13
*** ociuhandu has quit IRC16:14
*** alonma has quit IRC16:18
*** adrianofr has joined #openstack-cinder16:18
*** alonma has joined #openstack-cinder16:18
*** rcernin has quit IRC16:19
*** dims_ has quit IRC16:19
smcginnisxyang: Hey, do we need to get those group/cggroup patches backported to stable/ocata?16:20
*** pcaruana has quit IRC16:20
*** alonma has quit IRC16:23
*** dims has joined #openstack-cinder16:24
SwansonGot one trying to sneak into ocata. https://review.openstack.org/#/c/432425/16:26
*** diablo_rojo has joined #openstack-cinder16:26
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX Driver - disable initiator check by default  https://review.openstack.org/43316616:30
*** pcaruana has joined #openstack-cinder16:34
*** e0ne has joined #openstack-cinder16:35
*** _ducttape_ has joined #openstack-cinder16:37
xyang@smcginnis: hi, yes, I was going to submit a patch on that16:39
*** ducttape_ has quit IRC16:40
*** ociuhandu has joined #openstack-cinder16:41
*** crose has joined #openstack-cinder16:42
*** lkuchlan has quit IRC16:42
*** diablo_rojo_phon has joined #openstack-cinder16:42
*** isaacb has quit IRC16:42
*** pjm6 has quit IRC16:44
smcginnisxyang: OK, great.16:44
*** dims has quit IRC16:51
*** e0ne has quit IRC16:54
xyang@smcginnis: https://review.openstack.org/#/c/433185/16:54
*** dims has joined #openstack-cinder16:55
*** tesseract-RH has quit IRC16:55
*** _ducttape_ has quit IRC16:58
*** pcaruana has quit IRC16:58
smcginnisxyang: Thank you16:59
*** pcaruana has joined #openstack-cinder16:59
jungleboyjxyang, Thanks.  +1 from me.16:59
*** mriedem has quit IRC17:00
*** mriedem has joined #openstack-cinder17:01
Swansonjungleboyj, thanks!17:02
*** xyang has quit IRC17:02
jungleboyjSwanson, You are welcome.  Pushing your luck though.  ;-)17:02
*** _ducttape_ has joined #openstack-cinder17:03
Swansonjungleboyj, The eternal struggle. Making the test cases big enough but not so big it scares off reviewers. (They're skitterish.)17:03
*** dims_ has joined #openstack-cinder17:06
*** dims has quit IRC17:07
jungleboyjSwanson, Yep, we are skitterish.  Like cats batting at something.  If it moves we jump.17:11
jungleboyjsmcginnis, There?17:11
*** raunak has joined #openstack-cinder17:14
*** xinli has joined #openstack-cinder17:17
*** nkrinner is now known as nkrinner_afk17:18
-openstackstatus- NOTICE: AFS replication issue has been addressed. Mirrors are currently re-syncing and coming back online.17:18
*** alonmarx has joined #openstack-cinder17:20
*** jperry has quit IRC17:23
*** jperry has joined #openstack-cinder17:24
*** crose has quit IRC17:25
*** Jack_Iv has joined #openstack-cinder17:31
*** mvk has joined #openstack-cinder17:38
*** Apoorva has joined #openstack-cinder17:39
*** pcaruana has quit IRC17:44
*** kmartin has quit IRC17:46
*** lucasxu has quit IRC17:47
*** strigazi is now known as strigazi_AFK17:54
*** salv-orlando has joined #openstack-cinder17:55
*** david-lyle has joined #openstack-cinder17:58
*** ksumit has joined #openstack-cinder18:03
*** lpetrut has quit IRC18:05
smcginnisjungleboyj: Here now18:07
jungleboyjHowdy stranger.18:07
jungleboyjJust as I was about to get up.  ;-)18:08
*** david-lyle_ has joined #openstack-cinder18:08
smcginnisTake a break, I'll be here. :)18:08
jungleboyjsmcginnis, Nah, all good.  So, I brought up the Clustered MySQL DB in the olso meeting this week.18:08
*** david-lyle has quit IRC18:09
jungleboyjAt first they thought the only thing was the change to add the option for setting the db type.18:10
jungleboyjThough someone shared a ML thread between the guy proposing it and Doug.  Looks like there is a lot of discussion happening there.18:10
smcginnisjungleboyj: So still some details to work out before we should do anything in Cinder?18:10
jungleboyjYeah, I added links to the one review as well as the ML thread to the Cinder review.18:12
*** david-lyle_ is now known as david-lyle18:12
jungleboyjBased on what I could tell in the ML thread they were trying to head in the direction that would create the list impact on the consuming projects, but I think it is going to take some time to come to a solution.18:12
*** xyang has joined #openstack-cinder18:14
smcginnisjungleboyj: OK, good to know. My initial thought was there should be very minimum impact to Cinder if the oslo libraries are abstracting things right.18:15
smcginnisjungleboyj: Thanks for the update Cap'n.18:15
jungleboyjYes, I think that is the direction they are trying to move to.  Take a look at the ML thread I put in the Cinder review and see what you think.18:16
jungleboyjsmcginnis, Welcome.  Long story short, I think we just want to keep an eye on that.18:16
jungleboyjsmcginnis, I will maybe bring it up a few times in the oslo meeting and see what happens.18:16
jungleboyjsmcginnis, How hard do we want to push that or do we just want to kind of let it play out?18:17
smcginnisjungleboyj: Cool, just let us know if there's any progress there that the Cinder team should know about.18:17
smcginnisjungleboyj: I think they'll likely come up with the best solution based on the constraints they know about. I think we should be good.18:18
smcginnisjungleboyj: But if you see it taking a left turn and think it would help by having more of us chime in, just let me know.18:18
*** lucasxu has joined #openstack-cinder18:19
jungleboyjOk, sounds like a plan.  I will see if that is on the agenda for the PTG at all as well.18:19
jungleboyjsmcginnis, Doesn't look like it.  We will see what happens.18:21
*** dhinesh has joined #openstack-cinder18:21
*** newmember has quit IRC18:25
*** newmember has joined #openstack-cinder18:26
*** gouthamr has quit IRC18:26
*** gouthamr has joined #openstack-cinder18:27
*** e0ne has joined #openstack-cinder18:30
*** gatuus has joined #openstack-cinder18:31
*** xyang has quit IRC18:35
*** EZuck has quit IRC18:35
alonmarxhttps://review.openstack.org/#/c/432622/ is a bug fix backport for Ocata. Can someone review it? thanks18:35
*** sudipto_ has quit IRC18:35
*** sudipto has quit IRC18:35
openstackgerritHuang Zhiteng proposed openstack/cinder master: SolidFire: Handle connect_volume() failure when create vol from image  https://review.openstack.org/43251418:36
jungleboyjalonmarx, Hey buddy.  Looks good.  smcginnis Can you give it some love?18:38
smcginnisjungleboyj, alonmarx: Sure, taking a look now.18:39
alonmarxjungleboyj: thanks. love and care :)18:39
jungleboyjalonmarx, Any time.18:39
alonmarxsmcginnis: thanks!18:40
smcginnisalonmarx: No problem.18:40
*** e0ne has quit IRC18:44
*** dustins has quit IRC18:46
*** yumiriam has quit IRC18:46
*** yumiriam has joined #openstack-cinder18:48
*** yumiriam has quit IRC18:51
*** lpetrut has joined #openstack-cinder18:53
*** Jack_Iv has quit IRC18:55
*** Jack_Iv has joined #openstack-cinder18:56
*** yumiriam has joined #openstack-cinder18:57
*** Jack_Iv has quit IRC19:00
*** salv-orl_ has joined #openstack-cinder19:00
*** yumiriam has quit IRC19:00
*** salv-orlando has quit IRC19:03
*** yumiriam has joined #openstack-cinder19:04
*** crushil has quit IRC19:05
*** yumiriam has quit IRC19:05
*** Jack_V has quit IRC19:07
*** chris_morrell has joined #openstack-cinder19:08
*** newmember has quit IRC19:08
* jgriffith hmm... multipath is broke, I'll do a glog and grep "multipath"....19:13
*** ociuhandu has quit IRC19:13
* jgriffith holy crap... ummm how many "fix multipath" commits do we have... LOTS19:13
jgriffithand it's still broken :(19:13
patrickeastits always broken19:16
patrickeast:(19:16
jgriffithpatrickeast haha, seems to be19:16
patrickeastusually the fixes break it in another new way (or old one) for some other vendor19:16
jgriffithyeah19:16
patrickeastthen maybe a new version of multipath-tools becomes the default on some OS, and we start all over again19:16
patrickeastor the default multipath.conf updates19:17
jgriffithand now we have my FAVORITE problem of version issues with libs being updated on various cinder releases :(19:17
* patrickeast crys a little inside19:17
* jgriffith crys a LOT inside19:17
patrickeasthemna: hey quick question re: https://review.openstack.org/#/c/432467/2/monitoring/lastcomment-scoreboard/requirements.txt19:17
patrickeasthemna: it doesn't need requests anymore?19:17
jgriffithand thanks to our love of libs it's also an interesting excercise in futility to try and deploy devstack stable/mitaka any more19:18
patrickeasthaha yea19:18
jgriffithlibs are pure friking EVIL!19:18
hemnaoh crap19:18
hemnadidn't realize that there was a requirements.txt there already19:18
hemnaI'll fix it19:18
jgriffithdeath by bloat and deps19:18
* hemna is lazy19:19
patrickeastjgriffith: yea... it gets pretty bad19:19
hemnamulti path is borked?19:19
patrickeastjgriffith: i got pinged to do something on juno, and i'm now sitting here wondering how the hell i'm going to test it19:19
jgriffithpatrickeast yeah.. have fun with that!19:19
jgriffithI ended up deploying RDO an ddebugging off of that19:19
patrickeastyea, thats what i was thinking19:20
jgriffithand that wasn't exactly "easy" either19:20
patrickeastRDO or even just do it by hand19:20
*** shaner has quit IRC19:20
jgriffithhemna well, multi-path breaks anybody that doesn't do multipath (again)19:20
*** shaner has joined #openstack-cinder19:20
*** salv-orl_ has quit IRC19:20
hemnaoops19:20
hemnawhat's going on19:21
jgriffithwe used to have a try/except around the setup stuff to support multi-backend, but it seems it keeps being forgotten in the shuffle19:21
*** salv-orlando has joined #openstack-cinder19:21
jgriffithcustomer upgrades openstack, all their shit breaks... the usual19:21
patrickeastpff, just always multipath19:21
hemnaoh 'fun'19:21
patrickeastits the only safe way19:21
hemnacustomers.  sheesh19:21
patrickeastlol19:21
jgriffithpatrickeast +219:21
*** dustins has joined #openstack-cinder19:21
jgriffithhemna I honestly truly believe that's in fact that attitude that many OS developers subscribe to19:22
jgriffithI'm not talking about this particular item, or even this project19:22
jgriffithjust in general19:22
hemnayah, not many actually deal w/ customers19:22
jgriffithit's turned in to a neat science experiment or academic research activity for a bunch of developers19:22
jgriffithand has become less and less valuable to anybody in the *real* world19:23
jgriffithF#$%#  stack fails again19:23
jgriffithI just keep adding more and more to libs-from git :)19:24
jgriffithsome day maybe I'll get there, but probably not19:24
jgriffithwell, if I ever get this nonsense to deploy I'll post a patch, but chances are.. well; ya know19:25
jgriffith1'st things 1'st... time for Enchiladas and Beer19:25
*** xyang has joined #openstack-cinder19:25
smcginnisjgriffith: Did you see Gorka's latest set of patches? I haven't looked at them yet.19:26
jgriffithhemna you're by the America river not the Feather right?19:26
smcginnisjgriffith: But wondering if that's addressing something you're seeing.19:26
jgriffithsmcginnis oh.. yeah, more patches that likely won't backport or apply :)19:27
smcginnis;)19:27
jgriffithsmcginnis in all seriousness, "no" I haven't, but I'll take a look; good chance he's already fixed the problem, but the trick will be how to get it to a customer19:27
smcginnisI believe he had to patch open-iscsi itself, so that ought to be fun to add to the mix too.19:28
jgriffithoh dear, ok... I've now lost interest19:28
*** crushil has joined #openstack-cinder19:28
smcginnisjgriffith: Well, if it helps: https://review.openstack.org/#/q/status:open+project:openstack/os-brick+branch:master+topic:bug/166403219:28
jgriffithhmm... interesting, but looks like the problem still remains.19:29
patrickeasti'm lookin that those now19:29
patrickeastsmcginnis: any link for the open-iscsi one?19:29
jgriffithparticularly the fact that in a multi-backend env, you can have some devices that use multi-path and some that don't19:30
smcginnispatrickeast: Let me check my history.19:30
jgriffithbut we set the conf setting and pretend that "everybody will just work"19:30
jgriffith:(19:30
patrickeastjgriffith: not even just multi-backend, but in a single multipath backend where you only get a single path up (for whatever reason) will cause problems19:30
jgriffithpatrickeast yeah, it's sort of an all or nothing thing at this point19:30
jgriffithkinda dumb IMHO19:31
jgriffithbut what do I know19:31
jgriffithOh crap!!!19:32
smcginnispatrickeast, jgriffith: https://github.com/Akrog/open-iscsi/commit/56d7e5704871899ee7d9993341c9923835dd0a3a19:32
patrickeastsmcginnis: sweet, thanks!19:34
*** crose has joined #openstack-cinder19:39
openstackgerritJay Bryant proposed openstack/cinder master: Fix Pep8 Failures for Python3  https://review.openstack.org/42987619:43
hemnajgriffith, yah I'm near the American river.  I grew up in Yuba City on the Feather.19:44
hemnaI have lots of friends that got evacuated last night.  almost 200k people got moved19:44
jungleboyjhemna, What for?  Flooding?19:45
hemnaheh19:45
hemnajungleboyj, the Oroville aux. spillway was about to fail19:45
hemnareleasing 30' of water19:45
jungleboyjYikes!19:46
hemnabut it looks like they were able to release enough out of the primary spillway and lowered the water level.19:46
jungleboyjThat is good.  Scary stuff!19:46
*** xinli has quit IRC19:47
hemnayah it was19:47
*** gk-1wm-su has joined #openstack-cinder19:50
*** ociuhandu has joined #openstack-cinder19:51
*** gk-1wm-su has quit IRC19:54
*** kfarr has joined #openstack-cinder19:54
*** ociuhandu has quit IRC19:56
*** Jack_Iv has joined #openstack-cinder19:56
*** salv-orlando has quit IRC19:57
smcginnishemna: Does that mean no more California drought?19:57
hemnanot according to the Governor, we are still in one.19:57
hemna*smh*19:58
*** Jack_Iv has quit IRC20:01
*** crushil has quit IRC20:02
*** Jack_Iv has joined #openstack-cinder20:02
*** xyang has quit IRC20:03
*** xyang has joined #openstack-cinder20:03
*** crushil has joined #openstack-cinder20:05
*** crose has quit IRC20:06
dhineshhey guys..this is the first time i am trying to push changes to cinder, do i need blueprint on Launchpad?20:07
dhineshor am i allowed to create a branch and push the changes to it?20:07
*** Jack_Iv has quit IRC20:07
smcginnisdhinesh: Hi! We don't do pull requests, so you don't create your own branch and put up a PR for OpenStack.20:09
smcginnisdhinesh: This explains a little more how it works: http://docs.openstack.org/infra/manual/developers.html20:10
smcginnisdhinesh: But basically, everything goes through a gerrit review process, then gets merged into master.20:10
smcginnisdhinesh: As far as whether you need a blueprint, that depends what you are trying to do.20:11
smcginnisdhinesh: This link has a little more on that: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-new-feature20:11
*** ociuhandu has joined #openstack-cinder20:12
dhineshi am trying to add a new cinder driver for hedvig along with the unit test for it20:12
smcginnisdhinesh: Great. For new drivers just file a blueprint.20:13
smcginnisdhinesh: This one will be more interesting then: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver20:13
*** jungleboyj has quit IRC20:13
smcginnisdhinesh: I need to update the deadline for Pike in there, but I've been waiting on the official Pike schedule to be finalized.20:14
smcginnisdhinesh: But basically, it will need everything in place by the second milestone.20:14
dhineshsmcginnis: ok i will take a look at the links. Sure thanks for keeping me updated20:15
smcginnisdhinesh: No problem. If anything from there needs clarification, just let us know.20:16
*** Jack_Iv has joined #openstack-cinder20:16
*** Jack_V has joined #openstack-cinder20:18
*** jungleboyj has joined #openstack-cinder20:19
*** alonma has joined #openstack-cinder20:20
*** Jack_Iv has quit IRC20:21
*** ociuhandu has quit IRC20:21
*** Jack_Iv has joined #openstack-cinder20:21
smcginnishemna: Think I've got things to publish CI info every 4 hours here: http://cinderstats-dellstorage.rhcloud.com/20:23
smcginnishemna: http://cinderstats-dellstorage.rhcloud.com/cireport.txt20:24
*** gouthamr has quit IRC20:24
*** Jack_Iv has quit IRC20:25
*** Jack_Iv has joined #openstack-cinder20:25
*** gouthamr has joined #openstack-cinder20:25
*** alonma has quit IRC20:26
*** kong has joined #openstack-cinder20:27
hemnanice :)20:27
*** Jack_V has quit IRC20:28
openstackgerritJose Porrua proposed openstack/cinder master: NetApp: Track SVM and Cluster scoped credentials  https://review.openstack.org/43325120:30
jungleboyjsmcginnis, Nice!20:31
*** ociuhandu has joined #openstack-cinder20:32
*** stevemar has joined #openstack-cinder20:34
*** xinli has joined #openstack-cinder20:34
*** ociuhandu has quit IRC20:36
*** crushil has quit IRC20:44
*** crushil has joined #openstack-cinder20:45
*** cknight has quit IRC20:46
*** kfarr has quit IRC20:46
*** cknight has joined #openstack-cinder20:49
jgriffithpatrickeast FYI, this appears to solve most of the problems I had:  https://review.openstack.org/#/c/325088/220:51
jgriffithafter applying that and manipulating some of the libs manually it looks like I might finally deploy Mitaka20:51
patrickeastoo nice20:52
*** mikal_ is now known as mikal20:54
*** porrua has quit IRC20:57
*** gatuus has quit IRC20:59
*** jamielennox|away is now known as jamielennox21:02
*** _ducttape_ has quit IRC21:05
openstackgerritMerged openstack/cinder master: Fix duplicate lvs2 entry in rootwrap volume filter  https://review.openstack.org/43279421:10
karthikp_scottda: Hi21:10
*** lpetrut has quit IRC21:11
*** xyang has quit IRC21:14
*** xyang has joined #openstack-cinder21:14
*** adisky_ has quit IRC21:19
*** crushil has quit IRC21:19
*** Jack_Iv has quit IRC21:22
prometheanfiresmcginnis: https://review.openstack.org/#/c/433277/ probably should get in for the ocata release21:22
*** akapil has joined #openstack-cinder21:30
*** ksumit has quit IRC21:32
*** dave-mccowan has quit IRC21:36
*** xyang has quit IRC21:37
*** xyang has joined #openstack-cinder21:38
sdagueso, in trying to clean up some of the service catalog extras not really used, I tripped across https://github.com/openstack/cinder/blob/9a879f293647098766e249f53d33707bc1d811e7/cinder/compute/nova.py#L39-L4121:40
sdaguehow disruptive is it going to be to default that to publicURL?21:40
sdagueoh, sorry, bad link21:40
smcginnisprometheanfire: Thanks!21:40
sdaguehttps://github.com/openstack/cinder/blob/cfc617b0cea99ed6994f08e5337fd5b65ea9fd1c/cinder/compute/nova.py#L39-L4121:41
smcginnissdague: nova_catalog_admin_info should come from publicURL and not adminURL?21:43
prometheanfiresmcginnis: yarp, was restarting cinder and it failed, figured out why :D21:43
prometheanfireit's just about time to start packaging ocata21:43
smcginnisprometheanfire: I blame jgriffith. :)21:43
sdaguesmcginnis: there is no different in the features in the endpoitns21:43
prometheanfiresmcginnis: :D21:43
smcginnissdague: Conceptually though?21:43
sdaguesmcginnis: sorry, there is no difference in the features provided by endpoints21:43
sdaguesmcginnis: there is no difference21:43
smcginnissdague: Are we getting rid of adminUrls?21:43
sdaguesmcginnis: they aren't a thing, really, it mostly propogated because of keystone actually having a difference21:44
sdaguebut even that isn't true on v321:44
*** chris_morrell has quit IRC21:44
smcginnissdague: Ah, OK. Shouldn't be an issue to change that, as far as I know.21:44
*** salv-orlando has joined #openstack-cinder21:46
openstackgerritSean Dague proposed openstack/cinder master: Change nova_catalog_admin_info to default to publicURL  https://review.openstack.org/43328921:47
sdagueok, well, proposed, we can discuss in the patch21:47
smcginnissdague: Thanks21:47
*** lucasxu has quit IRC21:58
*** gouthamr has quit IRC21:59
*** gatuus has joined #openstack-cinder21:59
openstackgerritMichael Dovgal proposed openstack/cinder master: Fix size update error for snapshot-manage operation  https://review.openstack.org/42616222:02
*** jonesn has quit IRC22:12
*** eharney has quit IRC22:15
*** alonmarx has quit IRC22:16
*** catintheroof has quit IRC22:20
*** rajinir_ has quit IRC22:20
*** alonma has joined #openstack-cinder22:23
*** akapil has quit IRC22:26
*** alonma has quit IRC22:28
*** dustins has quit IRC22:32
*** lucasxu has joined #openstack-cinder22:35
*** gouthamr has joined #openstack-cinder22:37
*** tommylikehu_ has joined #openstack-cinder22:39
*** ducttape_ has joined #openstack-cinder22:39
*** dhellmann has left #openstack-cinder22:43
*** salv-orlando has quit IRC22:49
*** jperry has quit IRC22:55
jgriffithOMG multipath code in brick :(22:56
jgriffithwonder if there's any way to detect a device doesn't want/do multipath and cut some of this short22:57
jgriffithor if I should try and just fake it in place22:57
jgriffiththis is nutty code though22:57
jgriffithalthough it looks like folks have fixed most things up again in current master22:57
jgriffithMitaka is hosed22:58
patrickeastwhen you say doesn't want to do multipath, you mean like at attach time?22:58
jgriffithpatrickeast yeah, so my driver for example doesn't do multipath22:58
jgriffithbut i'm on a system with other backends that do, and the nova config is sort of broken22:58
jgriffithwell... it's just the wrong way to do it22:58
patrickeastseems like it should be part of the initialize_connection dict22:59
jgriffithpatrickeast that's what I'm wondering22:59
patrickeastlike a "use iscsi AND multipath"22:59
patrickeastinstead of just "iscsi"22:59
jgriffithexactly22:59
jgriffithtrying to understand why it's in nova at all?22:59
patrickeastthe config for it?22:59
jgriffithpatrickeast yes22:59
patrickeastiirc its just to pass into os-brick XD22:59
jgriffithpatrickeast why I oughta....23:00
jgriffith:)23:00
jgriffithbut again, why not put it in the init-con and do it that way?  Seems like the obvious answer, but I don't know much about it so assuming that's no good for some reason23:00
patrickeastimo we can streamline the attach if we have hints like that for behavior23:01
jgriffithpatrickeast I agree/like your opinion :)23:01
jgriffithalthough that ship has already sailed in many cases23:01
*** dhellmann has joined #openstack-cinder23:01
patrickeastif i had to guess, its because either A) thats just the way its always been, B) some backend can't do it... so here we are, C) we tried to make it "easy" and just auto-detect23:01
jgriffiththe good thing is that so far it does appear that just pulling down latest master will just "work" with mitaka23:01
patrickeastand by auto-detect i mean look for a config option and force feed mpio23:01
jgriffithpatrickeast or D). all of the above23:01
jgriffith:)23:01
*** mriedem is now known as mriedem_afk23:02
*** lucasxu has quit IRC23:02
*** lucasxu has joined #openstack-cinder23:02
jgriffithpatrickeast you by chance have a past of a connection_info with multipath handy?23:03
patrickeastjgriffith: yea one sec23:03
jgriffithpatrickeast wonder if I just "lie" and say I'm doing multipath but it's just a single multipath :)23:03
jgriffithoh...  mpi23:03
patrickeastjgriffith: you totally can23:03
*** dhellmann has left #openstack-cinder23:04
jgriffithpatrickeast I think that's the best way around this.... and yes, youmentioned it earlier :)23:04
*** gk--1wm- has joined #openstack-cinder23:04
jgriffithpatrickeast at least that way I don't have to continue fixing it every time the last stable finally hits customers after 6 months23:04
patrickeastjgriffith: yep, just every time multipath-tools packaging updates :D23:06
jgriffith:)23:06
*** gk--1wm- has left #openstack-cinder23:07
jgriffithstoopid storage!  We should all just be 12-factor apps and ephemeral only23:07
jgriffithstateless world FTW23:07
jgriffithwait... hmmm23:07
jgriffith:)23:07
patrickeastjust store it all in s3...23:07
patrickeastwait23:07
patrickeastno23:07
jgriffithpatrickeast LOL23:07
jgriffithYeah, that works great!  Just ask people that actually try and do it!23:07
patrickeastlol23:08
jgriffithsaid noone23:08
jgriffiththat ever tried to do it23:08
*** tommylikehu_ has quit IRC23:10
patrickeastjgriffith: wait, did you want the initiator connection info? or the driver connection info?23:10
jgriffithpatrickeast I guess I don't know :)23:12
jgriffithpatrickeast whatever makes the magic multipath crap in brick work :)23:12
jgriffithI guess I could/should just implement the dang thing23:12
jgriffithalthough it sounds like it's still troublesome23:13
patrickeastjgriffith: so you just need to have the initiator with that multipath flag in it like http://paste.openstack.org/show/598718/23:13
patrickeastbut like that same thing is what os-brick deals with23:13
jgriffithpatrickeast oh, hmm23:13
patrickeasteven though thats the call to cinder23:14
patrickeastjust that "multipath": true bit23:14
jgriffithpatrickeast yeah, got ya23:14
jgriffithbut trying to understand why the calls fail without it.  I mean, going back to the "lie" idea23:14
patrickeastthe attach? does the machine have multipath running? iirc there was/is a check to make sure its *really* got multipathing23:15
patrickeastif the config option is turned on23:15
patrickeastor does it fail to find the path later on?23:15
*** jamielennox is now known as jamielennox|away23:15
dhineshsmcginnis: i created a blueprint https://blueprints.launchpad.net/cinder/+spec/hedvig-cinder23:16
dhineshhow does it get assigned?23:16
patrickeastjgriffith: does it make it past creating the connector object?23:17
jgriffithpatrickeast hey.. wait, this is really stupid!23:17
patrickeasthah23:18
jgriffithpatrickeast so it pukes in get_potential_volume_paths because the multipath discovery section fails23:18
jgriffithpatrickeast BUT... Why would we use the nova.conf flag when the connection_properties would include the multipath flag as per the device?23:18
patrickeastjgriffith: oh, the connection properties builds up from the result of getting the connector, which reads the config option to decide if it should be multipath or not23:19
*** jamielennox|away is now known as jamielennox23:20
jgriffithpatrickeast right, but couldn't we just use the info from initialize_connection to do that instead of the config flag?23:20
jgriffithpatrickeast I guess the problem with that is that the compute node might not be configured for multipath23:20
patrickeastjgriffith: yea, but we don't pass anything back that says one way or the other right now23:20
patrickeastjgriffith: yep23:20
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX Driver - disable initiator check by default  https://review.openstack.org/43316623:21
jgriffithpatrickeast so I can fix this permantly though, add a field in the intiialize_connection return that says "don't f'ing use this"23:21
jgriffithand then that could override the conf flag in nova23:21
jgriffithso we solve the multi-backend config problem23:21
patrickeastyep23:21
jgriffiththis isn't even a "multi-backend on a single cinder-volume node" problem23:22
jgriffiththis is just a multi cinder-volume services problem23:22
patrickeastjgriffith: the only thing that could be a problem is if the backend doesn't know if that host should use multipathing or not, so the "yes" value is kinda vague, but for "no" is can be pretty definitive23:22
jgriffithpatrickeast yeah, I suppose it's not really kosher, just for folks like me that don't support it23:22
jgriffithpatrickeast so then I come back around to "should support it" :(23:23
*** akapil has joined #openstack-cinder23:23
jgriffithpatrickeast I think I'll just quit and go dig a hole somehwere23:23
jgriffithIt's apparent I'm not cut out for this OpenStack thing23:24
*** jwcroppe has quit IRC23:26
*** imacdonn has joined #openstack-cinder23:26
*** jwcroppe has joined #openstack-cinder23:27
*** lucasxu has quit IRC23:28
*** akapil has quit IRC23:28
*** diablo_rojo_phon has quit IRC23:30
*** xinli has quit IRC23:30
*** jwcroppe has quit IRC23:31
*** xyang has quit IRC23:31
*** cknight has quit IRC23:39
*** Julien-z_ has quit IRC23:53
*** catintheroof has joined #openstack-cinder23:55
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: implements: blueprint hedvig-cinder  https://review.openstack.org/43334123:55

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