Monday, 2016-11-14

*** ducttape_ has joined #openstack-cinder00:00
*** alonma has joined #openstack-cinder00:03
*** alonma has quit IRC00:07
*** cknight has quit IRC00:09
*** alonma has joined #openstack-cinder00:23
*** stakeda has joined #openstack-cinder00:23
*** alonma has quit IRC00:27
*** lixiaoy1 has joined #openstack-cinder00:32
*** Julien-zte has joined #openstack-cinder00:34
*** alonma has joined #openstack-cinder00:43
*** bardia has joined #openstack-cinder00:46
*** alonma has quit IRC00:47
*** bardia has quit IRC00:49
*** bardia has joined #openstack-cinder00:49
*** diogogmt has quit IRC00:58
*** alonma has joined #openstack-cinder01:03
*** ducttape_ has quit IRC01:04
*** zhangguoqing has joined #openstack-cinder01:07
*** alonma has quit IRC01:08
*** bardia has quit IRC01:11
*** ckonstanski has quit IRC01:11
*** ckonstan` has joined #openstack-cinder01:11
*** markvoelker has joined #openstack-cinder01:19
*** markvoelker has quit IRC01:23
*** alonma has joined #openstack-cinder01:24
*** ckonstan` has quit IRC01:24
*** zengyingzhe has quit IRC01:26
*** xek has quit IRC01:27
*** alonma has quit IRC01:28
*** zengyingzhe has joined #openstack-cinder01:28
*** bardia has joined #openstack-cinder01:30
*** Julien-zte has quit IRC01:34
*** gcb has joined #openstack-cinder01:36
*** alonma has joined #openstack-cinder01:44
*** ducttape_ has joined #openstack-cinder01:45
*** Julien-zte has joined #openstack-cinder01:46
*** ckonstanski has joined #openstack-cinder01:47
*** ckonstanski has quit IRC01:47
*** alonma has quit IRC01:48
*** ckonstanski has joined #openstack-cinder01:48
*** bardia has quit IRC02:03
*** alonma has joined #openstack-cinder02:04
*** namnh has joined #openstack-cinder02:08
*** Dong_Zhi has joined #openstack-cinder02:08
*** alonma has quit IRC02:09
*** liverpooler has quit IRC02:22
*** alonma has joined #openstack-cinder02:24
*** Dong_Zhi has quit IRC02:24
*** Dong_Zhi has joined #openstack-cinder02:28
*** alonma has quit IRC02:29
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987202:30
*** bardia has joined #openstack-cinder02:30
*** Dong_Zhi_ has joined #openstack-cinder02:32
*** Dong_Zhi__ has joined #openstack-cinder02:34
*** Dong_Zhi__ has quit IRC02:35
*** TommyLikeHu has quit IRC02:36
*** Dong_Zhi has quit IRC02:36
*** TommyLikeHu has joined #openstack-cinder02:36
*** bardia has quit IRC02:38
*** Dong_Zhi_ has quit IRC02:38
*** hoonetorg has quit IRC02:40
*** huanxuan has joined #openstack-cinder02:42
*** dims has quit IRC02:43
*** alonma has joined #openstack-cinder02:45
*** alonma has quit IRC02:49
*** ducttape_ has quit IRC02:50
*** ducttape_ has joined #openstack-cinder02:50
*** Dong_Zhi__ has joined #openstack-cinder02:51
*** hoonetorg has joined #openstack-cinder02:52
*** kaisers_ has joined #openstack-cinder02:56
*** Dong_Zhi__ has quit IRC02:57
*** kaisers__ has quit IRC03:00
*** alonma has joined #openstack-cinder03:05
*** ducttape_ has quit IRC03:08
*** ducttape_ has joined #openstack-cinder03:09
*** alonma has quit IRC03:09
*** ducttape_ has quit IRC03:11
*** ducttape_ has joined #openstack-cinder03:11
*** markvoelker has joined #openstack-cinder03:20
*** Lee1092 has joined #openstack-cinder03:24
*** markvoelker has quit IRC03:24
*** alonma has joined #openstack-cinder03:25
*** ducttape_ has quit IRC03:26
*** ducttape_ has joined #openstack-cinder03:26
*** alonma has quit IRC03:30
*** ducttape_ has quit IRC03:30
*** ducttape_ has joined #openstack-cinder03:40
*** alonma has joined #openstack-cinder03:45
*** alonma has quit IRC03:50
*** laughterwym has joined #openstack-cinder03:56
*** laughterwym has quit IRC04:00
*** sudipto has joined #openstack-cinder04:02
*** sudipto_ has joined #openstack-cinder04:02
*** psachin has joined #openstack-cinder04:04
*** psachin has quit IRC04:05
*** alonma has joined #openstack-cinder04:05
*** ducttape_ has quit IRC04:08
*** ducttape_ has joined #openstack-cinder04:08
*** bardia has joined #openstack-cinder04:09
*** alonma has quit IRC04:10
*** psachin has joined #openstack-cinder04:11
*** rhagarty has joined #openstack-cinder04:11
*** ducttape_ has quit IRC04:12
*** psachin has quit IRC04:12
*** ducttape_ has joined #openstack-cinder04:12
*** psachin has joined #openstack-cinder04:13
*** links has joined #openstack-cinder04:15
*** chris_morrell has joined #openstack-cinder04:22
*** sandanar has joined #openstack-cinder04:23
*** alonma has joined #openstack-cinder04:26
*** chris_morrell has quit IRC04:29
*** alonma has quit IRC04:30
*** chris_morrell has joined #openstack-cinder04:33
*** ducttape_ has quit IRC04:36
*** Dong_Zhi__ has joined #openstack-cinder04:41
*** Dong_Zhi__ has quit IRC04:41
*** ducttape_ has joined #openstack-cinder04:45
*** alonma has joined #openstack-cinder04:46
*** alonma has quit IRC04:50
*** sudswas__ has joined #openstack-cinder04:57
*** sudipto_ has quit IRC04:57
*** GB21 has joined #openstack-cinder04:58
*** sudipto has quit IRC04:58
*** sudipto_ has joined #openstack-cinder04:58
*** sudipto_ has quit IRC04:59
*** sudswas__ has quit IRC04:59
*** sandanar has quit IRC05:00
*** pgadiya has joined #openstack-cinder05:04
*** pdeore has joined #openstack-cinder05:05
*** alonma has joined #openstack-cinder05:06
*** ducttape_ has quit IRC05:10
*** alonma has quit IRC05:11
*** ducttape_ has joined #openstack-cinder05:12
*** ducttape_ has quit IRC05:16
*** alonmarx has joined #openstack-cinder05:17
*** markvoelker has joined #openstack-cinder05:21
*** huanxuan has quit IRC05:21
*** markvoelker has quit IRC05:25
*** alonma has joined #openstack-cinder05:26
*** pbandark has joined #openstack-cinder05:31
*** alonma has quit IRC05:31
*** sudipto has joined #openstack-cinder05:38
*** sudipto_ has joined #openstack-cinder05:38
*** bardia has quit IRC05:41
*** bardia has joined #openstack-cinder05:41
*** alonma has joined #openstack-cinder05:47
*** sudipto_ has quit IRC05:50
*** sudipto has quit IRC05:50
*** alonma has quit IRC05:51
*** bkopilov has joined #openstack-cinder05:55
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987205:58
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987206:01
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987206:02
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987206:04
*** bardia has quit IRC06:06
*** alonma has joined #openstack-cinder06:07
openstackgerritxianming.mao proposed openstack/cinder: Use method is_valid_boolstr from oslo_utils  https://review.openstack.org/38572106:10
*** alonma has quit IRC06:11
*** ducttape_ has joined #openstack-cinder06:12
*** ducttape_ has quit IRC06:17
*** bkopilov_ has joined #openstack-cinder06:17
*** yangyapeng has joined #openstack-cinder06:20
*** SinaMaleki has quit IRC06:25
*** alonma has joined #openstack-cinder06:27
*** alonma has quit IRC06:32
*** chenying_ has quit IRC06:33
*** rhagarty has quit IRC06:33
*** rhagarty has joined #openstack-cinder06:40
openstackgerritMerged openstack/cinder: Log message cleanup for volume-usage-audit  https://review.openstack.org/38412406:41
*** belmoreira has joined #openstack-cinder06:45
*** udesale has joined #openstack-cinder06:45
*** Poornima_K has joined #openstack-cinder06:46
*** alonma has joined #openstack-cinder06:47
*** ociuhandu has quit IRC06:49
*** rhagarty has quit IRC06:51
*** AlexeyAbashkin has joined #openstack-cinder06:51
*** ociuhandu has joined #openstack-cinder06:51
*** rhagarty has joined #openstack-cinder06:52
*** alonma has quit IRC06:52
*** chenying has joined #openstack-cinder06:52
*** mjura has joined #openstack-cinder06:54
*** AlexeyAbashkin has quit IRC06:57
*** rhagarty has quit IRC07:07
*** alonma has joined #openstack-cinder07:08
*** bhagyashris has joined #openstack-cinder07:08
*** alonma has quit IRC07:12
openstackgerritPony Chou proposed openstack/cinder: Add QNAP ES Storage Driver  https://review.openstack.org/30987207:23
*** TommyLikeHu has quit IRC07:28
*** alonma has joined #openstack-cinder07:28
*** TommyLikeHu has joined #openstack-cinder07:28
*** e0ne has joined #openstack-cinder07:30
*** alonma has quit IRC07:32
*** rha has quit IRC07:33
*** arecknag has joined #openstack-cinder07:33
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted  https://review.openstack.org/22508107:34
*** nherciu has joined #openstack-cinder07:35
*** pbandark has quit IRC07:42
*** pbandark has joined #openstack-cinder07:43
*** ducttape_ has joined #openstack-cinder07:43
*** ducttape_ has quit IRC07:48
*** alonma has joined #openstack-cinder07:48
*** GB21 has quit IRC07:49
*** alonma has quit IRC07:53
*** AlexeyAbashkin has joined #openstack-cinder07:56
*** pcaruana has joined #openstack-cinder07:57
openstackgerritTina Tang proposed openstack/cinder: Adds Unity Cinder Driver  https://review.openstack.org/39647707:57
*** alonma has joined #openstack-cinder08:08
*** bkopilov has quit IRC08:10
*** GB21 has joined #openstack-cinder08:11
*** alonma has quit IRC08:13
*** e0ne has quit IRC08:15
*** jordanP has joined #openstack-cinder08:20
*** kaisers has quit IRC08:27
*** kaisers has joined #openstack-cinder08:28
*** alonma has joined #openstack-cinder08:29
*** chris_morrell has quit IRC08:32
*** alonma has quit IRC08:33
*** huanxuan has joined #openstack-cinder08:40
*** gcb has quit IRC08:46
*** alonma has joined #openstack-cinder08:49
*** ociuhandu has quit IRC08:51
*** cknight has joined #openstack-cinder08:51
*** sudipto has joined #openstack-cinder08:52
*** cknight has quit IRC08:52
*** alonma has quit IRC08:53
*** lixiaoy1 has quit IRC08:54
*** snarwade has quit IRC08:57
*** gcb has joined #openstack-cinder08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:01
*** e0ne has joined #openstack-cinder09:03
*** AlexeyAbashkin has quit IRC09:04
TommyLikeHuping erlon-airlong09:05
*** ociuhandu has joined #openstack-cinder09:05
*** alonmarx has quit IRC09:05
*** gk__ has joined #openstack-cinder09:07
*** kwathore has joined #openstack-cinder09:09
*** alonma has joined #openstack-cinder09:09
*** snarwade has joined #openstack-cinder09:12
gk__Hi everyone, I am using Ceph as a backend. I get "down" for the state from cinder service-list. I noticed also that the field "updated_at" is not updated for a long time (around three days). In the cinder-volume's logs, ceph is reporting its status periodically. Does anyone knows about how to set the state up back? Thank you.09:13
*** e0ne has quit IRC09:13
*** ducttape_ has joined #openstack-cinder09:13
*** alonma has quit IRC09:13
kwathore@all: I need to configure TLS 1.2 for my cinder backend configuration, how to configure it? so my communcation from devstack to cinder backend through TLS 1.209:14
kwathoremy cinder backed support TLS 1.209:14
kwathorehow to configure in devstack so the communicatation will take place through TLS 1.209:16
*** ducttape_ has quit IRC09:18
*** ChubYann has quit IRC09:19
*** e0ne has joined #openstack-cinder09:21
*** Julien-zte has quit IRC09:28
*** alonma has joined #openstack-cinder09:29
*** alonma has quit IRC09:34
*** alonma has joined #openstack-cinder09:34
*** geguileo_PTO is now known as geguileo09:38
*** e0ne_ has joined #openstack-cinder09:40
*** e0ne has quit IRC09:43
*** openstackgerrit has quit IRC09:47
*** e0ne_ has quit IRC09:47
*** gouthamr has joined #openstack-cinder09:48
*** openstackgerrit has joined #openstack-cinder09:49
*** e0ne has joined #openstack-cinder09:50
*** lpetrut has joined #openstack-cinder09:50
*** gouthamr has quit IRC09:58
*** aarefiev has joined #openstack-cinder10:06
*** udesale has quit IRC10:08
*** e0ne has quit IRC10:11
*** gouthamr has joined #openstack-cinder10:18
openstackgerritAlon Marx proposed openstack/cinder: add fc decorator to ibm_storage driver  https://review.openstack.org/39710010:18
*** aarefiev has quit IRC10:22
*** mvk has quit IRC10:23
*** stakeda has quit IRC10:26
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Allow deletion of snapshot in error state  https://review.openstack.org/39710710:27
*** gouthamr has quit IRC10:32
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Allow deletion of snapshot in error state  https://review.openstack.org/39710710:32
*** sborkows has joined #openstack-cinder10:35
*** nkrinner_afk has quit IRC10:35
*** aarefiev has joined #openstack-cinder10:37
*** Michael-ZTE has joined #openstack-cinder10:37
*** khushbu has joined #openstack-cinder10:38
*** alonma has quit IRC10:39
*** gk__ has quit IRC10:39
*** Michael-ZTE-1 has joined #openstack-cinder10:39
*** amoralej has joined #openstack-cinder10:40
amoralejgeguileo, you may be interested in https://bugs.launchpad.net/cinder/+bug/164131210:41
openstackLaunchpad bug 1641312 in Cinder "CleanableInUse error in a deployment with just one cinder-volume" [Undecided,New]10:41
geguileoamoralej: Qué versión de MySQL estais usando?10:42
amoralejte digo10:42
*** Michael-ZTE has quit IRC10:42
geguileoNo debería ser un problema porque se supone que lo arreglé, pero por si acaso10:43
amoralej10.1.18-310:43
*** Michael-ZTE-1 has quit IRC10:44
*** pdeore has quit IRC10:45
*** nkrinner_afk has joined #openstack-cinder10:47
geguileoamoralej: Oh, MariaDB, no MySQL10:47
geguileoXD10:47
amoralejgeguileo, is the A/A code merged in newton?, it seems we are hitting the issue only in master10:47
amoralejyes geguileo we are using mariadb since long time ago in RDO10:47
geguileoamoralej: Yes, a part of it has merged10:48
*** ociuhandu has quit IRC10:48
geguileoamoralej: We merged it so we could start detecting all these issues10:48
amoraleji just got it again in other job10:48
*** alonma has joined #openstack-cinder10:48
geguileoamoralej: I've just come back from PTO, so I'm a bit swamped10:48
amoralejno problem :)10:49
geguileoamoralej: But I'll have a look right now and see if I can spot the issue10:49
amoralejok, thanks10:49
geguileoamoralej: If I can't I'll have to do a more careful investigation10:49
*** gouthamr has joined #openstack-cinder10:49
*** sborkows has quit IRC10:50
*** pdeore has joined #openstack-cinder10:51
*** pdeore has quit IRC10:51
*** pdeore has joined #openstack-cinder10:52
*** mvk has joined #openstack-cinder10:55
geguileoamoralej: Are c-vol and c-api on different machines/nodes?10:56
amoralejnot10:56
amoralejallinone10:56
geguileothanks10:56
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Attach and detach snapshot  https://review.openstack.org/39384210:58
*** GB21 has quit IRC10:59
openstackgerritAlon Marx proposed openstack/cinder: add fc decorator to ibm_storage driver  https://review.openstack.org/39710011:00
*** namnh has quit IRC11:05
*** e0ne has joined #openstack-cinder11:09
*** Dong_Zhi has joined #openstack-cinder11:11
geguileoamoralej: Could you check the workers table definition on the DB?11:14
amoralejdon't think so, i only have the job logs11:14
amoralejnot sure if i can extract it from there11:15
geguileoamoralej: OK, then I'll need to deploy it here and check it manually :-(11:15
geguileoamoralej: Is there an easy way to deploy RDO with master code?11:18
amoralejyes11:19
*** Poornima_K has quit IRC11:21
*** alonma has quit IRC11:21
*** alonma has joined #openstack-cinder11:22
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/39378911:25
*** pbandark has quit IRC11:25
openstackgerritMichal Dulko proposed openstack/cinder: Rolling upgrade procedure documentation  https://review.openstack.org/39339511:25
*** alonma has quit IRC11:26
*** AlexeyAbashkin has joined #openstack-cinder11:27
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/39534611:33
*** Dong_Zhi has quit IRC11:33
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/39714511:33
*** pbandark has joined #openstack-cinder11:40
openstackgerritxianming.mao proposed openstack/cinder: Drop unused parameter in class EntryCreateTask  https://review.openstack.org/38476611:45
openstackgerritxianming.mao proposed openstack/cinder: cors: update default configuration  https://review.openstack.org/39357711:45
*** pbandark has quit IRC11:45
*** khushbu has quit IRC11:47
openstackgerritMerged openstack/cinder: VMAX driver - Duplicate initiator group name error.  https://review.openstack.org/37943011:48
openstackgerritxianming.mao proposed openstack/cinder: Enable DeprecationWarning in test environments  https://review.openstack.org/38004711:48
openstackgerritxianming.mao proposed openstack/os-brick: Add developer docs url in README.rst(trivial)  https://review.openstack.org/37609111:49
*** nicolasbock has joined #openstack-cinder11:50
*** laughterwym has joined #openstack-cinder11:52
*** alonma has joined #openstack-cinder11:52
*** ociuhandu has joined #openstack-cinder11:56
*** pbandark has joined #openstack-cinder11:57
*** dave-mccowan has joined #openstack-cinder12:01
*** ociuhandu has quit IRC12:01
openstackgerritwanghao proposed openstack/cinder-specs: Report backend state in service list  https://review.openstack.org/25292112:05
*** ociuhandu has joined #openstack-cinder12:12
*** zhangguoqing has quit IRC12:13
openstackgerritKazumasa Nomura proposed openstack/cinder: Deprecate Hitachi Block Storage Driver  https://review.openstack.org/39647412:14
*** ducttape_ has joined #openstack-cinder12:15
*** sudipto has quit IRC12:16
openstackgerritwangxiyuan proposed openstack/cinder: Remove "service" from required_import_options  https://review.openstack.org/39715512:17
openstackgerritwanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted  https://review.openstack.org/22508112:17
*** ducttape_ has quit IRC12:19
*** gouthamr has quit IRC12:19
*** GB21 has joined #openstack-cinder12:19
flip214smcginnis: AFAICT the change https://review.openstack.org/#/c/332943/ causes the DRBD driver CI failure.12:21
flip214"cinder.volume.utils ... Can not find volume ... at notify usage _usage_from_volume /opt/stack/new/cinder/cinder/volume/utils.py:96"12:22
*** gcb has quit IRC12:22
flip214looks like the DB entry isn't saved in the DB, and that change actually creates problems.12:24
*** catintheroof has joined #openstack-cinder12:24
*** pdeore has quit IRC12:29
*** AlexeyAbashkin has quit IRC12:35
openstackgerritwanghao proposed openstack/cinder-specs: Add pike folder in specs  https://review.openstack.org/39717012:35
openstackgerritwanghao proposed openstack/cinder: Support to query volume filter by change since/before  https://review.openstack.org/35268812:41
flip214smcginnis: resp. see here, please:12:42
flip214http://logs.openstack.org/43/332943/8/check/gate-tempest-dsvm-full-drbd-devstack-nv/7dad5e1/logs/screen-c-vol.txt.gz#_2016-11-08_12_20_41_82612:42
*** Poornima has joined #openstack-cinder12:45
*** wanghao_ has joined #openstack-cinder12:48
*** amoralej is now known as amoralej|lunch12:50
*** belmorei_ has joined #openstack-cinder13:00
*** belmoreira has quit IRC13:01
*** sudipto has joined #openstack-cinder13:02
*** sudipto has quit IRC13:03
*** edmondsw has joined #openstack-cinder13:03
*** pgadiya has quit IRC13:04
*** ckonstan` has joined #openstack-cinder13:07
*** ducttape_ has joined #openstack-cinder13:07
*** ckonstan` has quit IRC13:08
*** ckonstanski has quit IRC13:08
*** ckonstan` has joined #openstack-cinder13:09
*** scottda has joined #openstack-cinder13:10
*** gouthamr has joined #openstack-cinder13:11
openstackgerritmichaeldovgal proposed openstack/python-cinderclient: Remove extra 'u' from cli output  https://review.openstack.org/34780413:11
openstackgerritmichaeldovgal proposed openstack/python-cinderclient: Add convertation of query parameters to string  https://review.openstack.org/39568413:11
*** ociuhandu has quit IRC13:14
*** al_indig_ has joined #openstack-cinder13:19
*** ociuhandu has joined #openstack-cinder13:19
*** al_indigo has quit IRC13:19
*** akerr has quit IRC13:21
*** ckonstan` has quit IRC13:23
*** ckonstanski has joined #openstack-cinder13:24
*** mattanReduxio has joined #openstack-cinder13:24
mattanReduxioHey. I'm trying to review a change which was abandoned (https://review.openstack.org/#/c/332013/) . Git review fails. What procedure should I follow to re-review it?13:27
*** amoralej|lunch is now known as amoralej13:28
scottdamattanReduxio: Can you un-abandon it in Gerrit? There should be an option for that when you are logged into the review and choose the "reply..." button.13:28
*** tongli has joined #openstack-cinder13:29
scottdaButton is "restore"13:29
*** gouthamr has quit IRC13:30
scottdamattanReduxio: hmm..looks like you don't have a Gerrit-ID in the review. That might not work then...13:30
scottdamattanReduxio: What does the failure look like when you try "git review"?13:31
*** dims has joined #openstack-cinder13:35
*** dustins has joined #openstack-cinder13:37
*** ducttape_ has quit IRC13:37
mattanReduxiohttps://thepasteb.in/p/Vmh0VJpz2VnI413:38
*** yangyapeng has quit IRC13:38
bhagyashrissmcginnis; Hi, Could please review patch https://review.openstack.org/#/c/372895/ ?13:39
openstackgerritMattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio  https://review.openstack.org/33203213:41
mattanReduxioscottda: Thanks. After restoring it works. I referenced the wrong item. The correct item is: https://review.openstack.org/#/c/332032/13:43
scottdamattanReduxio: Cool.13:44
*** ociuhandu has quit IRC13:45
*** markvoelker has joined #openstack-cinder13:46
*** ociuhandu has joined #openstack-cinder13:46
*** AlexeyAbashkin has joined #openstack-cinder13:49
*** Julien-zte has joined #openstack-cinder13:50
*** ckonstanski has quit IRC13:54
*** ociuhandu has quit IRC13:55
*** ckonstanski has joined #openstack-cinder13:55
*** gouthamr has joined #openstack-cinder13:55
*** psachin has quit IRC13:55
*** gouthamr has quit IRC13:57
*** huanxuan has quit IRC13:57
*** jbernard has quit IRC13:58
*** mriedem has joined #openstack-cinder13:58
*** Poornima has quit IRC13:58
smcginnisflip214: That looks odd. Are you running different versions of the services?14:01
*** gcb has joined #openstack-cinder14:01
flip214no. fresh devstack, all 3 nodes at the "same time" (within 1 hour or so)14:02
smcginnisHmm. geguileo, you happen to see that from earlier?14:02
smcginnisgeguileo: http://logs.openstack.org/43/332943/8/check/gate-tempest-dsvm-full-drbd-devstack-nv/7dad5e1/logs/screen-c-vol.txt.gz#_2016-11-08_12_20_41_82614:02
smcginnisgeguileo: From https://review.openstack.org/#/c/332943/14:02
*** AlexeyAbashkin has quit IRC14:02
* smcginnis runs to grab coffee quick...14:03
geguileosmcginnis: Let me have a look...14:04
*** khamtamtun has joined #openstack-cinder14:04
flip214smcginnis: http://logstash.openstack.org/... take the last 7 or 30 days, and set message "at notify usage _usage_from_volume"14:04
*** clenimar has joined #openstack-cinder14:04
flip214there are LOTS of this message, though I'm not sure whether it's really related or not14:05
flip214looking at the 30 days graph, it all _started_ a few days ago...14:05
smcginnisflip214: Yeah, looks suspiciously around the time that patch merged.14:06
geguileosmcginnis: I see where the problem is14:06
smcginnisflip214: volume.save() wasn't called before.14:06
*** gouthamr has joined #openstack-cinder14:06
*** kaisers_ has quit IRC14:06
smcginnisgeguileo: We missed an arg to that, didn't we?14:06
*** gouthamr has quit IRC14:06
*** xyang1 has joined #openstack-cinder14:06
geguileosmcginnis: No, the problem is that we are passing an ORM instance instead of an OVO instance to the method14:07
smcginnisgeguileo: Ah!14:07
geguileosmcginnis: Do we have a BZ?14:07
*** gouthamr has joined #openstack-cinder14:07
geguileoI mean an LP buz14:07
smcginnisgeguileo: flip214, have you filed a bug?14:07
geguileobug14:07
smcginnisgeguileo: I got you. ;)14:07
flip214no, not yet. I tried to find out whether it's our fault, or not ;)14:08
geguileoflip214: Not your fault14:08
flip214thanks, that's relieving!14:08
geguileoflip214: Fill in a bug and send me the link, I know where the bug is14:08
*** jbernard has joined #openstack-cinder14:08
smcginnisgeguileo, flip214: I'll watch for the patch and review it right away.14:08
*** ociuhandu has joined #openstack-cinder14:09
flip214geguileo: https://bugs.launchpad.net/cinder/+bug/164161114:11
openstackLaunchpad bug 1641611 in Cinder "TypeError: save() takes exactly 2 arguments (1 given)" [Undecided,New]14:11
geguileoflip214: Thanks14:11
*** akerr has joined #openstack-cinder14:12
*** Julien-zte has quit IRC14:12
*** Julien-zte has joined #openstack-cinder14:12
*** Julien-zte has quit IRC14:13
*** Julien-zte has joined #openstack-cinder14:13
*** Julien-zte has quit IRC14:13
*** Julien-zte has joined #openstack-cinder14:14
*** Julien-zte has quit IRC14:14
*** Julien-zte has joined #openstack-cinder14:15
*** Julien-zte has quit IRC14:15
*** sudipto has joined #openstack-cinder14:15
*** Julien-zte has joined #openstack-cinder14:15
*** khamtamtun has quit IRC14:16
flip214geguileo: thanks for taking care of that. I feared that much more work would be needed to find _a_ cause14:16
*** Julien-zte has quit IRC14:16
geguileoflip214: In this case it was easy enough  :-)14:16
*** pcaruana has quit IRC14:16
*** Julien-zte has joined #openstack-cinder14:16
*** Julien-zte has quit IRC14:17
flip214well, you don't know how much time I spent on that -- I'm a few time zones ahead of you...14:17
*** Julien-zte has joined #openstack-cinder14:17
*** Julien-zte has quit IRC14:17
*** Yogi1 has quit IRC14:19
*** akerr has quit IRC14:19
*** dulek_ has joined #openstack-cinder14:19
*** udesale has joined #openstack-cinder14:21
*** alonma has quit IRC14:22
*** alonma has joined #openstack-cinder14:23
geguileoflip214: I'm in UTC+114:24
*** khamtamtun has joined #openstack-cinder14:25
*** ducttape_ has joined #openstack-cinder14:25
*** sudipto has quit IRC14:27
*** alonma has quit IRC14:27
*** mriedem has quit IRC14:31
*** jgregor has joined #openstack-cinder14:32
mattanReduxioDoes someone knows the status of Fujitsu ETERNUS CI? My change has failed, and it has no logs.14:32
*** gcb has quit IRC14:35
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - MVs and SGs not reflecting correct protocol.  https://review.openstack.org/39565714:37
openstackgerritGorka Eguileor proposed openstack/cinder: Pass OVO instance to copy_volume_to_image  https://review.openstack.org/39721914:37
geguileoflip214: smcginnis ^ That should fix it14:39
smcginnisgeguileo: Awesome, I'll take a look.14:39
*** ducttape_ has quit IRC14:40
*** _fortis has joined #openstack-cinder14:41
*** khamtamtun has quit IRC14:41
*** khamtamtun has joined #openstack-cinder14:42
*** khamtamtun has quit IRC14:43
*** alonma has joined #openstack-cinder14:44
*** yangyapeng has joined #openstack-cinder14:44
*** dulek_ has quit IRC14:46
*** yangyapeng has quit IRC14:46
*** yangyapeng has joined #openstack-cinder14:46
*** eharney has joined #openstack-cinder14:47
*** catintheroof has quit IRC14:48
*** catintheroof has joined #openstack-cinder14:51
*** kfarr has joined #openstack-cinder14:51
*** jgregor has quit IRC14:53
*** jgregor has joined #openstack-cinder14:53
flip214geguileo: well, why did I have to find that out, then?? ;)14:57
*** jgregor has quit IRC14:58
*** kwathore_ has joined #openstack-cinder15:01
*** mriedem has joined #openstack-cinder15:01
*** ducttape_ has joined #openstack-cinder15:02
*** nherciu_ has joined #openstack-cinder15:03
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/39378915:03
*** kwathore has quit IRC15:04
*** nherciu has quit IRC15:05
*** ducttape_ has quit IRC15:06
*** rhagarty has joined #openstack-cinder15:07
*** nherciu_ has quit IRC15:09
*** breitz has joined #openstack-cinder15:10
*** GB21 has quit IRC15:13
*** yangyapeng has quit IRC15:13
*** links has quit IRC15:13
*** jgregor has joined #openstack-cinder15:14
*** victorhe has joined #openstack-cinder15:16
*** sudipto has joined #openstack-cinder15:18
*** wanghao_ has quit IRC15:21
*** clenimar has quit IRC15:22
*** rhagarty has quit IRC15:24
xyang1dulek: ping15:31
dulekxyang1: Hi!15:31
xyang1dulek: hi, can you take a look if this https://review.openstack.org/#/c/350350/15:31
*** mattanReduxio has quit IRC15:32
xyang1dulek: addressed your comments.  still WIP, but like to get your feedback15:32
*** laughterwym has quit IRC15:32
dulekxyang1: I will, I'll just finish trying to make ZeroMQ patches passing tests and get to your one. :)15:33
xyang1dulek: sure, thanks!15:33
geguileoflip214: Because I just got back from PTO today and I have a lot of backlog  ;-)15:33
*** pbandark has quit IRC15:34
*** kfarr has quit IRC15:34
*** ducttape_ has joined #openstack-cinder15:35
*** gcb has joined #openstack-cinder15:35
*** liverpooler has joined #openstack-cinder15:35
*** rajinir has joined #openstack-cinder15:36
openstackgerritMerged openstack/cinder: Add multipath enhancement to Storwize iSCSI driver  https://review.openstack.org/30888215:37
*** slayer has joined #openstack-cinder15:40
*** udesale has quit IRC15:40
*** jgregor_ has joined #openstack-cinder15:43
*** victorhe has quit IRC15:46
*** jgregor has quit IRC15:46
*** victorhe has joined #openstack-cinder15:46
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/39378915:48
*** alonma has quit IRC15:48
*** paw has joined #openstack-cinder15:48
*** alonma has joined #openstack-cinder15:48
*** jgregor has joined #openstack-cinder15:50
*** lpetrut has quit IRC15:50
*** alonma has quit IRC15:53
*** arecknag has quit IRC15:53
*** jgregor_ has quit IRC15:54
*** logan- has quit IRC15:54
*** diablo_rojo has joined #openstack-cinder15:55
*** dgonzalez is now known as dgonzalez__15:55
*** dgonzalez__ is now known as dgonzalez15:55
*** jgregor_ has joined #openstack-cinder15:57
*** _ducttape_ has joined #openstack-cinder15:57
*** ducttape_ has quit IRC16:00
*** jgregor has quit IRC16:01
*** logan- has joined #openstack-cinder16:01
*** Swanson has quit IRC16:01
openstackgerritGorka Eguileor proposed openstack/cinder: Make Image Volume Cache cluster aware  https://review.openstack.org/38618616:02
openstackgerritGorka Eguileor proposed openstack/cinder: Add more operations to cluster  https://review.openstack.org/35596816:02
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A on Scheduler operations  https://review.openstack.org/34422616:02
openstackgerritGorka Eguileor proposed openstack/cinder: [PoC][Don't review] Testing delay  https://review.openstack.org/35306916:02
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A in attach/detach operations  https://review.openstack.org/31857316:02
openstackgerritGorka Eguileor proposed openstack/cinder: Allow triggering cleanup from API  https://review.openstack.org/36301016:02
openstackgerritGorka Eguileor proposed openstack/cinder: Support A/A in delete actions and get_capabilities  https://review.openstack.org/34422516:02
openstackgerritGorka Eguileor proposed openstack/cinder: [PoC] Prevent Active-Active on drivers by default  https://review.openstack.org/39346016:02
openstackgerritGorka Eguileor proposed openstack/cinder: Make Replication support Active-Active  https://review.openstack.org/38183516:02
openstackgerritGorka Eguileor proposed openstack/cinder: Cosmetic changes to scheduler  https://review.openstack.org/34604116:02
smcginnisgeguileo: Back at it! ;)16:03
geguileosmcginnis: Yeah, it was about time to get back to it  :-)16:03
*** kaisers_ has joined #openstack-cinder16:04
*** jgregor_ has quit IRC16:04
*** jgregor has joined #openstack-cinder16:05
*** jgregor_ has joined #openstack-cinder16:06
*** jgregor_ has quit IRC16:06
*** jgregor_ has joined #openstack-cinder16:07
*** mjura has quit IRC16:09
*** jgregor has quit IRC16:09
*** belmorei_ has quit IRC16:11
diablo_rojodulek: Thanks for the +2 :) Any idea what other fields that need to get made into enums after VolAttachStatus merges?16:12
dulekdiablo_rojo: Any status/state field I guess.16:13
dulekdiablo_rojo: You probably don't want to touch Volume.status yet. ;)16:13
*** belmoreira has joined #openstack-cinder16:13
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - MVs and SGs not reflecting correct protocol.  https://review.openstack.org/39565716:13
diablo_rojodulek Ha ha yeah...I was saving that for last..if ever..16:14
*** kaisers_ has quit IRC16:14
diablo_rojodulek: I'll look and see what else there is to switch over, but if there are any priorities, let me know.16:14
dulekdiablo_rojo: I don't think there's anything prioritized for now.16:15
diablo_rojogeguileo: I'f you've got a sec in between pushing dozens of patches at a time, wanna give this a look? dulek already +2ed https://review.openstack.org/#/c/290885/16:15
diablo_rojodulek: Okay cool.16:15
dulekdiablo_rojo: By the way - do you have some info on how fast PTG tickets are being reserved? I wonder how much should we push managers, to make sure our team won't register too late.16:16
*** circ-user-Yoajv has joined #openstack-cinder16:16
*** bardia has joined #openstack-cinder16:16
diablo_rojodulek Last I checked about 1/5 of the tickets are gone already, but that was last week so it could be more by now.16:17
dulekdiablo_rojo: Whoa, that's quite fast. Thanks!16:17
diablo_rojodulek: No problem :)16:18
*** alonma has joined #openstack-cinder16:18
*** cknight has joined #openstack-cinder16:18
* dulek thinks that if something should be fixed in world 2.0, then corporate travel policies are a fair candidate.16:19
waj334What happened to DEFAULT_API_VERSION in cinder/api/openstack/api_version_request.py? Does cinder still default to v3.0 when no api version is specified when using the v3 endpoint?16:19
*** bardia has quit IRC16:22
*** alonma has quit IRC16:22
*** cknight1 has joined #openstack-cinder16:23
*** khamtamtun has joined #openstack-cinder16:23
*** pcaruana has joined #openstack-cinder16:24
openstackgerritEric Harney proposed openstack/os-brick: Add developer docs url in README.rst(trivial)  https://review.openstack.org/37609116:25
*** cknight has quit IRC16:25
*** ckonstanski has quit IRC16:28
*** _ducttape_ has quit IRC16:30
*** liverpooler has quit IRC16:30
*** ducttape_ has joined #openstack-cinder16:30
*** jgregor_ has quit IRC16:30
*** ChubYann has joined #openstack-cinder16:31
*** asselin__ has quit IRC16:32
*** asselin has joined #openstack-cinder16:32
*** belmoreira has quit IRC16:33
*** gcb has quit IRC16:34
*** circ-user-Yoajv has quit IRC16:36
*** bardia has joined #openstack-cinder16:36
*** slayer has quit IRC16:41
*** zengyingzhe has quit IRC16:41
*** slayer has joined #openstack-cinder16:42
*** zengyingzhe has joined #openstack-cinder16:42
*** khamtamtun has quit IRC16:43
*** mriedem has quit IRC16:43
*** diablo_rojo has quit IRC16:43
*** lpetrut has joined #openstack-cinder16:43
*** mriedem has joined #openstack-cinder16:46
*** slayer has quit IRC16:46
mriedemildikov: johnthetubaguy: i think i'm going to skip the nova/cinder xp meeting today, we're doing a spec review sprint in nova today and i want to spend my next hour reviewing johnthetubaguy's nova spec for the new cinder APIs16:46
johnthetubaguyhonestly, I have a clash for the first 30 mins, all my comments are in that spec really16:47
*** khamtamtun has joined #openstack-cinder16:49
openstackgerritMerged openstack/cinder: VMAX driver - Useful name convention for CG names  https://review.openstack.org/36566316:51
*** Swanson has joined #openstack-cinder16:51
openstackgerritMerged openstack/cinder: Pass OVO instance to copy_volume_to_image  https://review.openstack.org/39721916:51
ildikovmriedem: johnthetubaguy: thanks for letting me know, we can move some of the discussion to the spec review16:53
ildikovmriedem: johnthetubaguy: also jgriffith started to update the Cinder API patch: https://review.openstack.org/#/c/387712/ you might want to check as well16:54
*** gouthamr has quit IRC16:55
ildikovmriedem: johnthetubaguy: I'll try to get the Cinder folks together maybe still to get closer to an agreement on how the new API should look like on this side16:55
*** diablo_rojo has joined #openstack-cinder16:55
*** gouthamr has joined #openstack-cinder16:56
*** wxy__ has quit IRC17:01
*** wxy__ has joined #openstack-cinder17:02
*** gouthamr has quit IRC17:02
*** gouthamr has joined #openstack-cinder17:05
*** e0ne has quit IRC17:05
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: add fc decorator to ibm_storage driver  https://review.openstack.org/39710017:07
dulekIs anyone able to figure out what's going on here: http://logs.openstack.org/65/396265/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/fb0ee93/logs/17:08
*** chris_morrell has joined #openstack-cinder17:08
dulekGrenade is complaining that cinder_server1 isn't deleted, but n-cpu logs look quite normal.17:08
*** alonma has joined #openstack-cinder17:10
*** wxy__ has quit IRC17:14
*** wxy__ has joined #openstack-cinder17:14
*** alonma has quit IRC17:15
*** slayer has joined #openstack-cinder17:17
*** jgregor has joined #openstack-cinder17:19
*** clsacramento has quit IRC17:21
openstackgerritMerged openstack/os-brick: Mask passwords in utils.trace for func params  https://review.openstack.org/39509917:21
*** lpetrut has quit IRC17:23
*** xek has joined #openstack-cinder17:25
flip214geguileo: no problem, thanks for fixing!17:26
geguileoflip214: Thanks for reporting it  :-)17:26
flip214geguileo: I had to, else our driver would have been kicked out at some time, I guess ;P17:27
geguileoXD17:27
*** khamtamtun has quit IRC17:29
*** khamtamtun has joined #openstack-cinder17:29
*** enriquetaso has joined #openstack-cinder17:30
*** slayer_ has joined #openstack-cinder17:32
*** alonma has joined #openstack-cinder17:32
*** chris_morrell has quit IRC17:35
*** slayer has quit IRC17:36
*** alonma has quit IRC17:37
*** lpetrut has joined #openstack-cinder17:38
*** khamtamtun has quit IRC17:40
*** jordanP has quit IRC17:40
hemnamorning17:42
*** gouthamr has quit IRC17:42
*** khamtamtun has joined #openstack-cinder17:46
*** jordanP has joined #openstack-cinder17:46
*** jgregor has quit IRC17:49
*** jgregor has joined #openstack-cinder17:49
*** cknight1 has quit IRC17:50
*** dustins has quit IRC17:50
*** jordanP has quit IRC17:51
*** jgregor_ has joined #openstack-cinder17:51
*** mvk has quit IRC17:52
*** slayer_ has quit IRC17:52
*** alonma has joined #openstack-cinder17:53
*** jgregor has quit IRC17:54
diablo_rojohemna: afternoon :)17:54
*** slayer has joined #openstack-cinder17:54
*** slayer has quit IRC17:54
hemnadiablo_rojo, something like that17:54
*** rhagarty has joined #openstack-cinder17:56
*** jgregor_ has quit IRC17:56
*** khamtamtun has quit IRC17:57
*** alonma has quit IRC17:57
*** khamtamtun has joined #openstack-cinder17:57
*** jungleboyj has quit IRC17:58
*** slayer has joined #openstack-cinder18:05
*** GB21 has joined #openstack-cinder18:06
*** lpetrut has quit IRC18:06
*** khamtamtun has quit IRC18:07
*** khamtamtun has joined #openstack-cinder18:08
winston-1jgriffith: ping18:12
jgriffithwinston-1: hey18:12
*** alonma has joined #openstack-cinder18:13
*** kaisers_ has joined #openstack-cinder18:14
*** bardia has quit IRC18:14
patrickeastjgriffith: was double booked for the attach api meeting, reading through the log it looks like you were wanting people with shared connections? Pure Arrays do it18:17
jgriffithpatrickeast: oh yeah!!18:17
jgriffithpatrickeast: so you share a single connection to a host for multiple volumes?18:17
*** alonma has quit IRC18:17
patrickeastjgriffith: yea so in general the array maps "initiators" (iscsi or fc) to volumes as a lun #, and its a single connection to that host (in the Nova case, the n-cpu node)18:18
hemnaarg there was a meeting?18:18
*** khamtamtun has quit IRC18:18
hemna:(18:18
*** kaisers_ has quit IRC18:19
patrickeastjgriffith: so for iscsi, as an example, there might be 4 target portals (ip/port/iqn) that are all shared for the initiator, and each volume attachment is an incremented lun #18:19
jgriffithpatrickeast: ok, cool!  So that's actually pretty common; since you have a lun# it can be identified as *unique'ish*18:19
jgriffithpatrickeast: yeah, that's ok I think18:20
*** ociuhandu has quit IRC18:20
patrickeastjgriffith: yep totally, the main issue is that the lun numbers get recycled18:20
jgriffithpatrickeast: so is there anything special you do currently on a disconnect (detach call)18:20
patrickeastjgriffith: so like, if you attach volume A and it gets lun 2, then detach it, and later attach volume B it can get lun 2 as well18:20
patrickeastjgriffith: and volume A on host foo might be lun 2 but on host bar it is lun 8218:20
jgriffithpatrickeast: oh... interesting18:20
patrickeastjgriffith: so the race conditions between attach/detach are particularly dangerous18:21
hemnathe lefthand driver exports the same target portal, iqn and lun# for multiple attachments on the same host18:21
jgriffithpatrickeast: yeah, that makes sense18:21
patrickeastjgriffith: because if the lun comes back, it might not even be the same volume18:21
xyang1patrickeast: that is same as us18:21
hemnaN vms on a host get the same connection_info back from the lefthand driver.  fyi18:21
hemnatruely shared fwiw18:22
jgriffithpatrickeast: hemna so if you have a chance take a look at the patch I put up.  My idea was that when we call terminate connection to a backend it would report if it's sharing that connection on the host18:22
hemnaok sure18:22
patrickeastjgriffith: yea i'll take a look18:22
jgriffithpatrickeast: hemna set that in the response so that nova (and eventually brick) would know that there's something special there18:22
hemnaduring terminate?  not initiatlize_connection ?18:22
jgriffithI don't know how those two platforms deal with it today so18:22
jgriffithhemna: correct, NOT intiialize18:23
patrickeastjgriffith: the one thing that might need some tweaks is the FCZM, it needs to know on terminate connection whether it should actually unzone18:23
jgriffithhemna: there is no initialize18:23
hemnaso quick question18:23
jgriffithpatrickeast: yeah, that's the only case I could think of that was tricky in all of this18:23
jgriffithpatrickeast: but that should be doable (get_attachments_by_host)18:23
hemnais the order of operations still the same between nova and cinder ?   nova calls brick to disconnect, then calls cinder terminate ?18:23
jgriffithhemna: no18:24
hemnaisn't it a bit late to return that shared flag during cinder terminate ?18:24
patrickeastjgriffith: yep, totally18:24
hemnaok, so nova is going to call cinder terminate first18:24
hemnawhich replies with the flag18:24
jgriffithhemna: the proposal has been to call Cinder first, then depending on that response nova terminates in whatever way18:24
*** sudipto has quit IRC18:24
hemnaok18:24
jgriffithhemna: and it's not calling terminate at all18:24
jgriffithnova isn't18:24
jgriffiththere's 3 calls basically:18:24
jgriffith1. attachment_create18:25
jgriffith2. attachment_finalize (maybe change to update or something else)18:25
jgriffith3. attachment_remove18:25
hemnaso if the cinder driver is sharing that volume with other vms on that same host, it will return the shared flag as true, which would tell nova not to call os-brick disconnect_volume ?18:25
patrickeastinteresting... i need to go do some reading18:25
jgriffithhemna: sharing a volume is multi-attach and that's different18:25
hemnaok18:26
jgriffithhemna: as I've tried to describe in response to comments in the review and in the weekly meeting that we've been having on this topic for the last couple months :)18:26
jgriffithhemna: the multi-attach case is actually *easier* to deal with18:26
jgriffithhemna: the shared connection case is IMO really the only one that's kind of tricky, and the only way to solve that that I'm really aware of is to make the driver report that knowledge back up18:27
hemnaI haven't been able to keep track of when the meetings are, they seem to change every week18:27
hemna:(18:27
patrickeastkinda looks like once we have this other stuff in place we just kinda get multi-attach18:27
hemnaanyway18:27
hemnajgriffith, +118:27
jgriffithpatrickeast: that's the plan... exactly18:27
hemnaI think that's the best way really18:27
hemnait's the driver that really should know that information18:28
jgriffithhemna: yeah, that's what i think18:28
jgriffithhemna: FWIW this is what I was talking about last week when we were chatting here.  I don't think I was very clear though in retrospect18:28
*** nherciu has joined #openstack-cinder18:28
*** xyang has joined #openstack-cinder18:28
hemnaok coolio18:28
jgriffithanyway... gotta run for a bit; but I'll be back once I get to the other building18:29
hemnaafter all my onboarding settles down, I'll get back up to speed on this stuff.   it's been chaos for a while18:29
*** mvk has joined #openstack-cinder18:29
ildikovhemna: the only change with the meeting time was daylight saving AFAIK :)18:30
hemnaildikov, ok cool, last time I attended it was a different day18:30
ildikovhemna: hmm,18:31
ildikovhemna: I don't think we changed the day for this one in the last couple of weeks18:32
ildikovpatrickeast: BTW being double booked for this meeting was for today or in general?18:33
jgriffithhemna: onboarding?  Where did you land?18:33
*** alonma has joined #openstack-cinder18:33
hemnajgriffith, I'm with IBM now.18:33
*** rcernin has joined #openstack-cinder18:33
jgriffithhemna: Congrats!18:33
hemnajgriffith, thanks :)18:33
hemnaI'm working on fixing the closed source stuffs.18:34
hemnabut a job and a job working on OpenStack = winning!18:34
xyanghemna: glad you are still working on OpenStack!18:35
hemnaxyang me too!!!18:36
jgriffithhemna: Excellent!!18:36
*** amoralej is now known as amoralej|off18:38
*** alonma has quit IRC18:38
*** khamtamtun has joined #openstack-cinder18:40
*** Dong_Zhi has joined #openstack-cinder18:46
*** xyang has quit IRC18:50
*** khamtamtun has quit IRC18:50
*** GB21 has quit IRC18:52
*** xyang has joined #openstack-cinder18:52
*** lpetrut has joined #openstack-cinder18:53
*** alonma has joined #openstack-cinder18:53
patrickeastildikov: sry, was afk, but it was just today, usually that time slot is open for me18:54
ildikovpatrickeast: ok cool, I was just wondering whether I'll need to find a new meeting time as today was like this for many :)18:55
ildikovpatrickeast: tnx for confirming18:55
patrickeastnp18:55
*** kaisers_ has joined #openstack-cinder18:57
*** bardia has joined #openstack-cinder18:57
*** alonma has quit IRC18:58
*** lpetrut has quit IRC19:00
*** dustins has joined #openstack-cinder19:01
*** khamtamtun has joined #openstack-cinder19:06
*** jgregor has joined #openstack-cinder19:07
*** alonma has joined #openstack-cinder19:13
*** alonma has quit IRC19:17
*** jgregor has quit IRC19:22
*** jgregor has joined #openstack-cinder19:23
*** belmoreira has joined #openstack-cinder19:23
*** rhagarty has quit IRC19:23
*** rhagarty has joined #openstack-cinder19:24
*** khamtamtun has quit IRC19:26
*** paw has quit IRC19:27
*** belmoreira has quit IRC19:27
*** khamtamtun has joined #openstack-cinder19:27
*** jgregor has quit IRC19:28
smcginnisjgriffith: Catching up on the backlog. The Dell SC array sounds like it works the same way as the Pure one.19:28
* smcginnis calls up the lawyers19:28
smcginnisKidding!19:28
patrickeasthaha19:28
patrickeastthey already settled!19:29
smcginnishehe19:29
*** ccesario has quit IRC19:30
openstackgerritMerged openstack/cinder: Updated from global requirements  https://review.openstack.org/39378919:31
*** alonma has joined #openstack-cinder19:34
*** ociuhandu has joined #openstack-cinder19:34
*** alonma has quit IRC19:38
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements  https://review.openstack.org/39534619:40
*** alonmarx has joined #openstack-cinder19:42
*** rhagarty has quit IRC19:47
jgriffithsmcginnis: patrickeast so just to make sure I follow19:48
*** xyang has quit IRC19:48
jgriffithsmcginnis: patrickeast let's say I have volume A and B and attach them to Servers on a host19:48
jgriffithsmcginnis: patrickeast I'll have /dev/disk/by-path/iqn.foo.0 and idqn.foo.119:49
jgriffithsmcginnis: patrickeast is that right?19:49
smcginnisjgriffith: In the simplest scenario, yes.19:49
smcginnisjgriffith: There will be one iSCSI connection with two different LUNs visible.19:49
jgriffithsmcginnis: so it seems to me that Brick is what needs to be smart in it's teardown here19:50
jgriffithsmcginnis: and if it needs a "shared_connections" flag to do that then that's cool19:50
jgriffithsmcginnis: and should work19:50
smcginnisjgriffith: The only reason I say in the simplest scenario is there can be multiple front end ports over multiple controllers, so LUNs can get exposed over different paths. But in that cases it's no longer an issue.19:50
jgriffithsmcginnis: but I don't see in the current code how it deals with this19:50
smcginnisjgriffith: Yeah, I think you're right.19:50
smcginnisjgriffith: Hand wavy magic.19:51
jgriffithsmcginnis: the best kind of magic is "hand wavy" magic19:51
patrickeastjgriffith: there is a check in os-brick now that looks for devices being used by a session19:51
*** slayer has quit IRC19:51
patrickeastjgriffith: and if its the last one for the iscsi session it logs out19:51
jgriffithpatrickeast: Ahhh... BINGO19:51
patrickeastjgriffith: the problem is that its racy and is frequently causing us problems :(19:52
jgriffithpatrickeast: yeah, that totally makes sense19:52
jgriffithpatrickeast: so that in conjunction with a notification from the driver will at least *help*19:52
*** alonma has joined #openstack-cinder19:52
*** enriquetaso has quit IRC19:52
jgriffithit may not eliminate it currently, but it can at least give us the tools and knowledge to improve later; do you think?19:53
patrickeastjgriffith: yea, i think we can avoid a huge amount of the issues if the driver can just pass to nova (or whoever) something like "plz_synchronize_attach_detach"19:53
hemnadid we rename instance to server now ?19:53
jgriffithhemna: yes :)19:53
jgriffithhemna: sorry19:53
hemna*sigh* ok19:53
patrickeastjgriffith: the majority of the issues come about between the os-brick call to detach and calling terminate_connection19:53
hemnapatrickeast, +119:53
patrickeastjgriffith: well, aside from nasty udev async junk and others19:54
SwansonWait, instances are servers now?19:54
jgriffithpatrickeast: so in your opinion does rerversing that order help?19:54
jgriffithSwanson: FTR they've been servers forever19:54
*** slayer_ has joined #openstack-cinder19:54
hemnaso in the case where there are N volumes over the same iSCSI session, brick already handles that19:54
jgriffithSwanson: you can call them whatever you want19:54
patrickeastjgriffith: oh yea, totally, as long as it is safe to rip out the volume from the array side19:54
jgriffithpatrickeast: yeah, so remember the steps would be like:19:54
hemnain disconnect_volume it checks to see if there are other volumes left on the same target portal before calling iscsiadm logout19:54
patrickeastjgriffith: so if you call terminate and then os-brick, you need to be 1000% sure nothing is still writing/reading from the volume19:54
jgriffith1. attachment_details19:54
*** slayer__ has joined #openstack-cinder19:55
jgriffithWhich would include "shared-connections for this attachment"19:55
jgriffithThen Nova/Brick would decide "nuke it" or "just nuke the one thing"19:55
jgriffithor whatever tom foolery magic it's doing :)19:55
jgriffithThen issue the terminate call back to Cinder19:55
jgriffithattachment_remove19:56
hemnahttps://github.com/openstack/os-brick/blob/master/os_brick/initiator/connectors/iscsi.py#L529-L53919:56
jgriffithand again the driver has the opportunity to figure out wha tit shoud/shouldn't do19:56
hemnathat's the code the checks to see if it needs to disconnect the iscsi session19:56
patrickeastjgriffith: that makes sense, i guess i'm not sure how the order is reversed though, that sounds like what we do today?19:56
hemnabased on if there are volumes still using the same target_portal as the volume that was just removed.19:56
*** alonma has quit IRC19:56
*** MVenesio has joined #openstack-cinder19:57
jgriffithpatrickeast: oh... sorry; so I think the proposal from Nova was that they would try the disconnect on their side BEFORE issuing the attachment_remove call out to Cinder19:57
supamattyeah that code is racy19:57
hemnasupamatt, how so?19:57
supamattespecially racy if you have multiple volumes on the same target, and you only want to dismount one volume19:57
*** MVenesio has quit IRC19:58
jgriffithpatrickeast: currently the flow is like "unreserve, terminate-connection and detach"  Nova issues the call to brick to terminate the connection inbetween the terminate and detach calls to cinder IIRC19:58
supamattduring a detach (with multipath), we will see devices show up as faulty when we try to detach them19:58
patrickeastjgriffith: ah yea, ok, that should be OK too but it means we are pulling iscsi exports before flushing/deleting the devices on the intiator, that mostly works as long as we've stopped anyone from using the volume while we pull the export out from underneath it19:58
supamattthen cinder never removes the target session19:58
jgriffithpatrickeast: this would just flip things around a bit and Nova would call brick to remove the connection, then call Cinder19:58
*** slayer_ has quit IRC19:59
patrickeastsupamatt: +1, those errors are a constant regression19:59
patrickeastjgriffith: yea that makes sense19:59
hemnathat shouldn't happen now thought, as we removed the rescans during detach19:59
hemnawe used to have that19:59
jgriffithpatrickeast: right, and in theory we will have removed it from the KVM side of things by then19:59
supamatthemna: it still happens19:59
patrickeastjgriffith: yep, sweet, that should work better then19:59
supamatthemna: I have an env it still happens on19:59
jgriffithpatrickeast: emphasis on *SHOULD* :)19:59
hemnahrmm ok19:59
patrickeastjgriffith: because then we don't have the race of doing an iscsi rescan right after a detach and having old volumes paths come back :D19:59
hemnaI'd like to know more about that20:00
supamatthemna: xtremio array backed, I can give you access20:00
patrickeastjgriffith: its probably going to be great20:00
hemnasupamatt, ok20:00
jgriffithpatrickeast: right, at least that's how all of this works in my head at least ;)20:00
jgriffithpatrickeast: everything will be just fine... we're going to build the best attach flow, it's going to be under-budget, on time and very very goodly20:00
jgriffithpatrickeast: and we're going to make Nova pay for it20:01
smcginnishah20:01
hemnapatrickeast, iscsi rescan causes issues20:03
* ildikov is just relaxing from now on then and watching jgriffith doing some magic and making Nova pay for this stuff :)20:03
patrickeastjgriffith: sounds perfect, i'd bet we can double the eng resources on it and do it in half the time too20:03
*** xyang has joined #openstack-cinder20:04
jgriffithpatrickeast: and hire only people from the RustBelt that used to be steel and auto-workers20:05
jgriffithoh... wait.. no, not that part20:05
patrickeastlol20:05
jgriffithI'm not moving20:05
*** slayer__ has quit IRC20:05
ildikovLOL20:05
*** xyang has quit IRC20:06
*** Lee1092 has quit IRC20:06
*** slayer has joined #openstack-cinder20:07
*** slayer has quit IRC20:07
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - PortGroup rollback error path scenario.  https://review.openstack.org/37951420:09
*** e0ne has joined #openstack-cinder20:09
*** kaisers_ has quit IRC20:11
*** alonma has joined #openstack-cinder20:12
*** kfarr has joined #openstack-cinder20:14
openstackgerritGorka Eguileor proposed openstack/cinder: Add host check on replication actions  https://review.openstack.org/39735720:16
openstackgerritGorka Eguileor proposed openstack/cinder: Fix replication_status on InvalidReplicationTarget  https://review.openstack.org/39735820:16
openstackgerritGorka Eguileor proposed openstack/cinder: Fix status after a VolumeDriverException  https://review.openstack.org/39735920:16
*** xyang has joined #openstack-cinder20:16
*** alonma has quit IRC20:17
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Ensure VMAX volume matches cinder db volume  https://review.openstack.org/37154520:24
*** khamtamtun has quit IRC20:27
*** asselin has quit IRC20:27
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Attach and detach snapshot  https://review.openstack.org/39384220:31
*** khamtamtun has joined #openstack-cinder20:31
*** alonma has joined #openstack-cinder20:32
*** slayer has joined #openstack-cinder20:34
*** rcernin has quit IRC20:35
flip214smcginnis: thanks, looks good so far.20:35
flip214http://graphite.openstack.org/render/?width=600&height=344&_salt=1434709688.361&from=-3days&xFormat=%25b%25d-%25T&title=DRBD%20Cinder%2FDevstack%20stats&colorList=red%2Cgreen%2Cblue&target=stats_counts.zuul.pipeline.check.job.gate-tempest-dsvm-full-drbd-devstack-nv.FAILURE&target=stats_counts.zuul.pipeline.check.job.gate-tempest-dsvm-full-drbd-devstack-nv.SUCCESS&drawNullAsZero=true20:35
flip214has green points again.20:35
flip214geguileo: ^^20:36
smcginnisflip214: Awesome! Glad that got resolved so quickly.20:36
flip214me too!20:36
geguileoflip214: Glad to hear it!!20:37
*** alonma has quit IRC20:37
*** tongli has quit IRC20:40
*** e0ne has quit IRC20:40
*** pcaruana has quit IRC20:45
*** rhagarty has joined #openstack-cinder20:48
*** khamtamtun has quit IRC20:51
*** paw has joined #openstack-cinder20:51
*** rhagarty has quit IRC20:52
*** alonma has joined #openstack-cinder20:53
*** khamtamtun has joined #openstack-cinder20:53
*** xyang has quit IRC20:54
*** jgregor has joined #openstack-cinder20:54
*** khamtamtun has quit IRC20:55
*** alonma has quit IRC20:57
*** rcernin has joined #openstack-cinder21:00
*** kaisers_ has joined #openstack-cinder21:01
*** xyang has joined #openstack-cinder21:04
*** xyang has quit IRC21:07
jgriffithsmcginnis: so what do you think... am I moving fwd with this:  https://review.openstack.org/#/c/387712/21:08
*** dulek_ has joined #openstack-cinder21:08
jgriffithsmcginnis: no pressure :)21:09
smcginnisjgriffith: Yeah, based on the conversation this morning, I think that sounds like our best bet.21:09
smcginnisjgriffith: Everything looks fine to me so far.21:09
jgriffithok, nobody is screaming "no" and hopefully that's not because they just haven't looked at it21:10
smcginnisjgriffith: Yeah, no doubt.21:10
jgriffithI think it's an improvement21:10
jgriffithbut I'm obviously terribly biased21:10
jgriffithit's not exactly what I hoped for, but a descent compromise21:10
smcginnisjgriffith: Biased, or you just really understand the process and have the clearest take on it. ;)21:11
SwansonAnyone want to poke at a simple patch? https://review.openstack.org/#/c/396343/21:11
smcginnisjgriffith: At this point, I doubt we could have gotten anywhere without some sort of compromise.21:11
smcginnisjgriffith: It is an improvement, and I'm hoping it does address some issues we've had.21:11
jgriffithtrue21:11
smcginnisjgriffith: At least the high level API looks good, and once we get there we can start working through any weird corner cases we find.21:12
jgriffithok, I'll start cleaning/finishing it up more and maybe get a basic nova version going.21:12
smcginnisjgriffith: +121:12
*** alonma has joined #openstack-cinder21:13
*** porrua has quit IRC21:17
*** alonma has quit IRC21:17
*** porrua has joined #openstack-cinder21:18
*** asselin has joined #openstack-cinder21:19
*** xyang has joined #openstack-cinder21:21
*** Julien-zte has joined #openstack-cinder21:25
ildikovjgriffith: added my last monologue regarding the name with the wish that that's the biggest thing to sort out on that API by now :)21:25
jgriffithildikov: excellent!21:25
smcginnisIf we're down to just a name, that's pretty darn good. :)21:26
ildikovsmcginnis: +1 :)21:26
ildikovjgriffith: BTW how far this is from what you hoped for?21:27
ildikovjgriffith: just asking for curiosity, no intent to cause any confusions21:27
jgriffithildikov: not far off really21:28
jgriffithildikov: and actually since it's easy enough to create a condensed API call for the attach it's good IMO21:28
*** lpetrut has joined #openstack-cinder21:28
jgriffithildikov: I should say "still what I initially hoped for"21:29
ildikovjgriffith: cool, that sounds good enough :)21:29
jgriffithildikov: I'd like to figure out how to do some fancy backrefs to shared connections in the db; but that's not really necessary/helpful at this stage I don't think21:29
jgriffithjust makes it harder to review21:29
ildikovjgriffith: I had an idea, just wanted to be sure21:30
jgriffithI'd rather iterate on it with succesive merges21:30
jgriffithildikov: yeah, all good21:30
ildikovjgriffith: yeah, I saw concerns about the DB part21:30
jgriffithit's not as simple as "just add a column" by a long shot21:31
ildikovif it can be handled in an iterative way, I think that will be good enough to go21:33
*** alonma has joined #openstack-cinder21:33
*** kaisers_ has quit IRC21:34
ildikovat least for now to have something to start with21:34
ildikovjgriffith: BTW do you plan to update the patch you already have for Nova?21:35
*** rhagarty has joined #openstack-cinder21:35
*** alonma has quit IRC21:37
*** kaisers_ has joined #openstack-cinder21:39
ildikovjgriffith: if that helps I can look into that one, while you sort out the client, DB, etc.21:40
*** victorhe has quit IRC21:41
*** dulek_ has quit IRC21:42
*** victorhe has joined #openstack-cinder21:43
*** xyang has quit IRC21:52
*** alonma has joined #openstack-cinder21:53
*** xyang has joined #openstack-cinder21:54
jgriffithildikov: not sure what I'm doing there, but for sure; once the cinder side is baked help on that would be great21:54
*** slayer has quit IRC21:56
*** alonma has quit IRC21:58
ildikovjgriffith: ok cool, I'll look into that part then21:58
*** xyang has quit IRC22:01
*** edmondsw has quit IRC22:02
*** rhagarty has quit IRC22:04
*** e0ne has joined #openstack-cinder22:04
*** rhagarty has joined #openstack-cinder22:04
*** kaisers has quit IRC22:08
*** dustins has quit IRC22:08
*** dulek_ has joined #openstack-cinder22:09
*** e0ne has quit IRC22:10
*** kaisers has joined #openstack-cinder22:11
*** alonma has joined #openstack-cinder22:14
*** dulek_ has quit IRC22:14
*** victorhe has quit IRC22:15
*** alonma has quit IRC22:18
*** victorhe has joined #openstack-cinder22:20
*** xyang has joined #openstack-cinder22:22
*** nherciu has quit IRC22:25
*** kaisers_ has quit IRC22:27
*** eharney has quit IRC22:30
*** jgregor has quit IRC22:31
*** kfarr has quit IRC22:32
*** jgregor has joined #openstack-cinder22:32
*** alonma has joined #openstack-cinder22:34
*** slayer has joined #openstack-cinder22:35
*** slayer has quit IRC22:35
*** jgregor has quit IRC22:37
*** dave-mccowan has quit IRC22:38
*** alonma has quit IRC22:39
*** kaisers_ has joined #openstack-cinder22:41
*** victorhe has quit IRC22:42
*** kaisers_ has quit IRC22:46
*** lpetrut has quit IRC22:46
*** xyang has quit IRC22:47
*** alonma has joined #openstack-cinder22:54
*** alonma has quit IRC22:58
*** rhagarty_ has joined #openstack-cinder23:02
*** alonmarx has quit IRC23:02
*** rhagarty has quit IRC23:02
*** xyang has joined #openstack-cinder23:02
*** xyang has quit IRC23:04
*** xyang1 has quit IRC23:04
*** alonma has joined #openstack-cinder23:14
*** alonma has quit IRC23:19
*** rhagarty has joined #openstack-cinder23:20
*** alonma has joined #openstack-cinder23:34
*** Julien-zte has quit IRC23:37
*** alonma has quit IRC23:39
*** diablo_rojo has quit IRC23:40
*** ducttape_ has quit IRC23:42
*** rhagarty has quit IRC23:42
*** ducttape_ has joined #openstack-cinder23:43
*** mriedem has quit IRC23:43
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - MVs and SGs not reflecting correct protocol.  https://review.openstack.org/39565723:46
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Attach and detach snapshot  https://review.openstack.org/39384223:46
*** ducttape_ has quit IRC23:47
*** madca0 has quit IRC23:50
*** alonma has joined #openstack-cinder23:55
SwansonGlance image caching on by default now?23:58
*** alonma has quit IRC23:59

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