Tuesday, 2016-08-23

*** ducttape_ has joined #openstack-cinder00:00
*** david-lyle has joined #openstack-cinder00:00
*** lixiaoy1 has joined #openstack-cinder00:03
*** david-lyle_ has joined #openstack-cinder00:03
*** bardia_ has joined #openstack-cinder00:04
*** bardia has quit IRC00:04
*** david-lyle has quit IRC00:05
*** yangyapeng has quit IRC00:05
*** david-lyle_ has quit IRC00:09
*** wilson1 has joined #openstack-cinder00:11
*** wilson_liu has quit IRC00:15
*** bigdogstl has joined #openstack-cinder00:15
*** lixiaoy1 has quit IRC00:16
*** sdake has joined #openstack-cinder00:16
*** ducttape_ has quit IRC00:18
*** bigdogstl has quit IRC00:21
*** bigdogstl has joined #openstack-cinder00:21
*** Suyi has quit IRC00:21
openstackgerritRodrigo Freire proposed openstack/os-brick: Multipath: fix race when detaching devices  https://review.openstack.org/33137500:26
*** ducttape_ has joined #openstack-cinder00:29
*** alonma has joined #openstack-cinder00:34
*** Julien-zte has joined #openstack-cinder00:34
*** bigdogstl has quit IRC00:37
*** alonma has quit IRC00:38
*** bigdogstl has joined #openstack-cinder00:38
*** laughterwym has joined #openstack-cinder00:40
*** laughterwym has joined #openstack-cinder00:40
*** bigdogstl has quit IRC00:41
*** Suyi has joined #openstack-cinder00:42
*** bigdogstl has joined #openstack-cinder00:43
*** bigdogst_ has joined #openstack-cinder00:47
*** bigdogstl has quit IRC00:48
*** gouthamr has joined #openstack-cinder00:52
*** bigdogst_ has quit IRC00:53
*** cheneydc has joined #openstack-cinder00:56
*** cheneydc has quit IRC00:57
*** Apoorva has quit IRC00:59
*** bigdogstl has joined #openstack-cinder01:01
*** bigdogstl has quit IRC01:06
*** sdake_ has joined #openstack-cinder01:06
*** sdake has quit IRC01:10
*** jamielennox is now known as jamielennox|away01:11
*** alonma has joined #openstack-cinder01:15
*** salv-orl_ has joined #openstack-cinder01:15
*** jamielennox|away is now known as jamielennox01:16
*** salv-orlando has quit IRC01:18
*** sdake_ has quit IRC01:19
*** alonma has quit IRC01:20
*** yangyapeng has joined #openstack-cinder01:21
*** sdake has joined #openstack-cinder01:22
*** salv-orl_ has quit IRC01:26
*** ducttape_ has quit IRC01:28
*** ducttape_ has joined #openstack-cinder01:28
*** Lee1092 has joined #openstack-cinder01:29
*** yangyape_ has joined #openstack-cinder01:32
*** yangyapeng has quit IRC01:35
*** alonma has joined #openstack-cinder01:36
*** raunak has quit IRC01:37
*** coolsvap has quit IRC01:39
*** alonma has quit IRC01:40
*** andymaier has joined #openstack-cinder01:41
*** markvoelker has joined #openstack-cinder01:44
*** markvoelker_ has joined #openstack-cinder01:46
openstackgerritCao ShuFeng proposed openstack/cinder: Add filters support to get_pools  https://review.openstack.org/32149201:47
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output  https://review.openstack.org/34780401:48
*** yangyape_ has quit IRC01:49
*** lixiaoy1 has joined #openstack-cinder01:50
*** lixiaoy1 has quit IRC01:50
*** lixiaoy1 has joined #openstack-cinder01:50
*** markvoelker has quit IRC01:50
*** EinstCra_ has joined #openstack-cinder02:01
*** mtanin___ has quit IRC02:07
*** bardia_ has quit IRC02:07
*** zhangguoqing has joined #openstack-cinder02:15
*** EinstCra_ has quit IRC02:17
*** alonma has joined #openstack-cinder02:18
openstackgerritMerged openstack/os-brick: RBD: Fix typo in rados timeout assignment  https://review.openstack.org/35880602:20
*** tommylikehu has joined #openstack-cinder02:21
*** alonma has quit IRC02:22
*** EinstCra_ has joined #openstack-cinder02:22
*** raunak has joined #openstack-cinder02:28
*** yangyapeng has joined #openstack-cinder02:36
*** yangyape_ has joined #openstack-cinder02:37
*** EinstCra_ has quit IRC02:39
*** yangyapeng has quit IRC02:40
*** yangyape_ has quit IRC02:47
*** jamielennox is now known as jamielennox|away02:49
*** yangyape_ has joined #openstack-cinder02:50
*** yangyap__ has joined #openstack-cinder02:56
*** gouthamr_ has joined #openstack-cinder02:57
*** gouthamr has quit IRC02:58
*** yangyape_ has quit IRC02:58
*** alonma has joined #openstack-cinder02:59
*** yangyapeng has joined #openstack-cinder03:01
*** gouthamr_ is now known as gouthamr03:01
*** andymaier has quit IRC03:03
*** alonma has quit IRC03:04
*** yangyap__ has quit IRC03:05
*** jamielennox|away is now known as jamielennox03:06
*** lixiaoy1 has quit IRC03:06
*** lixiaoy1 has joined #openstack-cinder03:06
*** yangyapeng has quit IRC03:17
*** ebalduf has joined #openstack-cinder03:20
*** yangyapeng has joined #openstack-cinder03:22
*** EinstCrazy has joined #openstack-cinder03:24
*** Suyi has quit IRC03:25
*** salv-orlando has joined #openstack-cinder03:30
*** bigdogstl has joined #openstack-cinder03:31
*** yangyape_ has joined #openstack-cinder03:34
*** yangyapeng has quit IRC03:36
*** EinstCrazy has quit IRC03:38
*** salv-orlando has quit IRC03:38
*** asselin_ has joined #openstack-cinder03:38
*** ducttape_ has quit IRC03:39
*** bigdogstl has quit IRC03:41
*** zul has quit IRC03:41
*** asselin has quit IRC03:42
*** gouthamr has quit IRC03:44
*** bigdogstl has joined #openstack-cinder03:44
*** zul has joined #openstack-cinder03:46
*** abhitechie has joined #openstack-cinder03:47
*** yangyape_ has quit IRC03:50
openstackgerritxing-yang proposed openstack/cinder: Add group snapshots  https://review.openstack.org/32805204:01
*** Jaison has joined #openstack-cinder04:15
*** markvoelker has joined #openstack-cinder04:21
*** markvoelker_ has quit IRC04:22
*** Suyi has joined #openstack-cinder04:23
*** Jaison has quit IRC04:24
*** jraju has joined #openstack-cinder04:26
openstackgerritxing-yang proposed openstack/cinder: Add group snapshots  https://review.openstack.org/32805204:28
*** markvoelker has quit IRC04:28
*** EinstCrazy has joined #openstack-cinder04:30
*** laughter_ has joined #openstack-cinder04:36
*** lixiaoy1 has quit IRC04:36
*** yangyape_ has joined #openstack-cinder04:37
*** salv-orlando has joined #openstack-cinder04:37
openstackgerritKarthik Prabhu Vinod proposed openstack/python-cinderclient: Showing the metadata readonly value as a separate field  https://review.openstack.org/33248004:38
*** laughterwym has quit IRC04:39
*** ducttape_ has joined #openstack-cinder04:40
*** ducttape_ has quit IRC04:44
*** yangyape_ has quit IRC04:45
*** EinstCrazy has quit IRC04:47
*** lixiaoy1 has joined #openstack-cinder04:48
*** salv-orlando has quit IRC04:49
*** jamielennox is now known as jamielennox|away04:49
openstackgerritvenkata krishna proposed openstack/cinder: Add replication failback in Kaminario K2 drivers  https://review.openstack.org/35647504:50
*** yangyape_ has joined #openstack-cinder04:50
*** enriquetaso has quit IRC04:51
*** bigdogstl has quit IRC04:53
*** bigdogstl has joined #openstack-cinder04:53
*** yangyape_ has quit IRC04:55
*** bigdogstl has quit IRC04:56
*** bigdogstl has joined #openstack-cinder04:56
*** salv-orlando has joined #openstack-cinder04:58
*** bigdogstl has quit IRC05:01
*** yangyape_ has joined #openstack-cinder05:04
*** lkuchlan has joined #openstack-cinder05:05
*** laughter_ has quit IRC05:06
*** laughterwym has joined #openstack-cinder05:07
*** yangyape_ has quit IRC05:09
*** ducttape_ has joined #openstack-cinder05:10
*** abhinavtechie has joined #openstack-cinder05:12
*** Suyi has quit IRC05:12
*** bigdogstl has joined #openstack-cinder05:13
*** abhitechie has quit IRC05:14
*** aagrawal has joined #openstack-cinder05:14
*** ducttape_ has quit IRC05:15
*** abhitechie has joined #openstack-cinder05:16
*** bigdogstl has quit IRC05:17
*** abhinavtechie has quit IRC05:18
*** lkuchlan has quit IRC05:19
*** aagrawal has quit IRC05:20
*** yangyape_ has joined #openstack-cinder05:20
*** lkuchlan has joined #openstack-cinder05:21
*** sdake_ has joined #openstack-cinder05:21
*** jamielennox|away is now known as jamielennox05:23
*** sdake has quit IRC05:24
*** alonma has joined #openstack-cinder05:25
*** yangyape_ has quit IRC05:28
*** bkopilov has joined #openstack-cinder05:29
*** markvoelker has joined #openstack-cinder05:29
*** laughterwym has quit IRC05:29
*** alonma has quit IRC05:30
*** laughterwym has joined #openstack-cinder05:30
*** laughter_ has joined #openstack-cinder05:31
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class  https://review.openstack.org/30928305:32
*** laughter_ has quit IRC05:32
openstackgerritgengchc2 proposed openstack/cinder: Using oslo_config's min attribute  https://review.openstack.org/35896005:32
*** yangyape_ has joined #openstack-cinder05:33
*** laughterwym has quit IRC05:34
*** raunak has quit IRC05:35
*** raunak has joined #openstack-cinder05:36
*** sandanar has joined #openstack-cinder05:37
*** markvoelker has quit IRC05:39
*** sdake_ has quit IRC05:41
*** raunak has quit IRC05:42
*** nkrinner_afk is now known as nkrinner05:48
*** pbandark has joined #openstack-cinder05:49
*** yangyape_ has quit IRC05:50
*** laughterwym has joined #openstack-cinder05:52
*** laughterwym has quit IRC05:52
*** laughterwym has joined #openstack-cinder05:53
*** mdenny has quit IRC05:55
*** mdenny has joined #openstack-cinder05:55
*** laughterwym has quit IRC05:57
*** yangyape_ has joined #openstack-cinder05:59
*** yangyap__ has joined #openstack-cinder06:00
*** yangyap__ has quit IRC06:01
*** alonma has joined #openstack-cinder06:01
*** yangyap__ has joined #openstack-cinder06:02
*** iceyao has joined #openstack-cinder06:02
*** iceyao has quit IRC06:02
*** arecknag has joined #openstack-cinder06:03
*** yangyapeng has joined #openstack-cinder06:03
*** asselin__ has joined #openstack-cinder06:04
*** yangyape_ has quit IRC06:05
*** gcb_ has joined #openstack-cinder06:05
*** alonma has quit IRC06:06
*** yangyapeng has quit IRC06:07
*** yangyap__ has quit IRC06:07
*** asselin_ has quit IRC06:07
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/35762306:08
*** jamielennox is now known as jamielennox|away06:11
*** abhitechie has quit IRC06:11
*** abhinavtechie has joined #openstack-cinder06:11
*** ducttape_ has joined #openstack-cinder06:11
*** pcaruana has joined #openstack-cinder06:14
*** ducttape_ has quit IRC06:17
*** Poornima_K has joined #openstack-cinder06:20
openstackgerritCao ShuFeng proposed openstack/cinder: Add filters support to get_pools  https://review.openstack.org/32149206:22
*** EinstCrazy has joined #openstack-cinder06:32
*** laughterwym has joined #openstack-cinder06:34
*** lpetrut has joined #openstack-cinder06:35
*** laughterwym has quit IRC06:36
*** markvoelker has joined #openstack-cinder06:36
*** yangyape_ has joined #openstack-cinder06:38
*** EinstCrazy has quit IRC06:38
*** laughterwym has joined #openstack-cinder06:41
*** laughterwym has joined #openstack-cinder06:42
*** yangyapeng has joined #openstack-cinder06:42
*** yangyape_ has quit IRC06:42
*** markvoelker has quit IRC06:42
*** Poornima_K has quit IRC06:43
*** EinstCra_ has joined #openstack-cinder06:43
*** yangyape_ has joined #openstack-cinder06:44
*** GB21 has joined #openstack-cinder06:47
*** Poornima_K has joined #openstack-cinder06:48
*** yangyapeng has quit IRC06:48
*** lixiaoy1 has quit IRC06:55
*** lixiaoy1 has joined #openstack-cinder06:57
*** mjura has joined #openstack-cinder06:59
*** EinstCra_ has quit IRC07:01
*** yangyape_ has quit IRC07:01
*** abhitechie has joined #openstack-cinder07:02
*** abhinavtechie has quit IRC07:05
*** yangyapeng has joined #openstack-cinder07:05
*** EinstCrazy has joined #openstack-cinder07:05
*** fmccrthy has quit IRC07:06
*** fmccrthy has joined #openstack-cinder07:08
*** ducttape_ has joined #openstack-cinder07:12
*** salv-orl_ has joined #openstack-cinder07:16
*** ducttape_ has quit IRC07:16
*** tesseract- has joined #openstack-cinder07:18
*** andymaier has joined #openstack-cinder07:18
*** salv-orlando has quit IRC07:19
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Add volume name in vCenter to conn info  https://review.openstack.org/34406907:20
*** Poornima_K has quit IRC07:21
*** salv-orl_ has quit IRC07:21
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Remove extra 'u' in "cinder show"  https://review.openstack.org/35899507:21
*** qeelee has joined #openstack-cinder07:21
*** EinstCrazy has quit IRC07:24
*** EinstCrazy has joined #openstack-cinder07:25
*** salv-orlando has joined #openstack-cinder07:25
*** lpetrut has quit IRC07:26
*** wxy__ has quit IRC07:28
*** wxy__ has joined #openstack-cinder07:29
*** Poornima_K has joined #openstack-cinder07:29
*** EinstCra_ has joined #openstack-cinder07:36
*** markvoelker has joined #openstack-cinder07:38
*** EinstCrazy has quit IRC07:39
*** lpetrut has joined #openstack-cinder07:40
*** markvoelker has quit IRC07:43
*** e0ne has joined #openstack-cinder07:45
*** GB21 has quit IRC07:45
*** zouyapeng has joined #openstack-cinder07:48
*** abhitechie has quit IRC07:53
*** abhitechie has joined #openstack-cinder07:53
*** ebalduf has quit IRC07:54
*** rcernin has quit IRC07:56
*** zzzeek has quit IRC08:00
openstackgerritMichal Dulko proposed openstack/cinder: Pin RPC server's serializer to min obj version  https://review.openstack.org/35053408:00
openstackgerritMichal Dulko proposed openstack/cinder: Cleanup upgrade-related FIXME's  https://review.openstack.org/35053308:00
*** zzzeek has joined #openstack-cinder08:01
*** lpetrut has quit IRC08:01
*** openstackgerrit has quit IRC08:03
*** openstackgerrit has joined #openstack-cinder08:05
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** ducttape_ has joined #openstack-cinder08:13
openstackgerritMatan Sabag proposed openstack/cinder: Sending ScaleIO volume id in attach and detach volume  https://review.openstack.org/35523108:16
*** ducttape_ has quit IRC08:17
*** pgadiya has joined #openstack-cinder08:22
*** coolsvap has joined #openstack-cinder08:25
*** alonma has joined #openstack-cinder08:26
*** qeelee has quit IRC08:30
*** alonma has quit IRC08:31
openstackgerritMatan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver  https://review.openstack.org/35612008:36
*** markvoelker has joined #openstack-cinder08:39
*** lpetrut has joined #openstack-cinder08:41
*** laughterwym has quit IRC08:43
*** salv-orl_ has joined #openstack-cinder08:44
*** markvoelker has quit IRC08:44
*** salv-orlando has quit IRC08:45
*** laughterwym has joined #openstack-cinder08:45
openstackgerritMatan Sabag proposed openstack/cinder: ScaleIO over subscription support  https://review.openstack.org/35654708:47
*** salv-orl_ has quit IRC08:49
*** salv-orlando has joined #openstack-cinder08:50
*** lpetrut has quit IRC08:50
*** laughterwym has quit IRC08:51
openstackgerritEvgeny Antyshev proposed openstack/cinder: Unify ploop format names in Openstack projects  https://review.openstack.org/35767908:54
openstackgerritEvgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots  https://review.openstack.org/27646508:54
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Merge two fake drivers into the one FakeLoggingVolumeDriver  https://review.openstack.org/34842808:57
openstackgerritxianming.mao proposed openstack/cinder: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/35905308:59
lixiaoy1dulek: may I have a question about your patch https://review.openstack.org/#/c/350534/ ?09:08
duleklixiaoy1: Sure.09:08
lixiaoy1dulek: if a old object is returned, but in new services, what happens if new attribute is used?09:09
*** kaisers_ has joined #openstack-cinder09:10
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077809:10
duleklixiaoy1: Newer services should be able to handle missing data.09:10
duleklixiaoy1: Because they can always get older version of an object from older services.09:10
lixiaoy1dulek: yes.09:11
duleklixiaoy1: Similar thing happened when we were adding cascade to volume delete RPC calls - when running in compatibility mode we've just raised an exception on API level.09:11
duleklixiaoy1: To make sure a missing parameter that was accepted won't be ignored further when calls will get backported.09:12
lixiaoy1dulek: but seems the problem can't be prohibited effectively. Dev may add a field, and doesn't update rpcapi interface. But as we transfer such object in rpcapi, this may lead problems09:14
duleklixiaoy1: Worst thing that can happen is that some kind of user's request will get ignored. Consider that:09:15
lixiaoy1dulek: would like to confirm: If I have an object with version 1.0, and then add a field in version 1.1. When I got its instance wiht version 1.0, I can't access the new field, yes?09:15
duleklixiaoy1: It's not enteirely true.09:16
duleklixiaoy1: We have a new service trying to send 1.1 object to other service (it don't know which service version will get it).09:17
duleklixiaoy1: So the object gets backported to 1.0 and a field is removed.09:17
duleklixiaoy1: Now it happens that object is received by newer service that can understand 1.1.09:17
duleklixiaoy1: Serializer will actually unserialize it to 1.1 with that new field unset.09:18
duleklixiaoy1: That's it.09:18
duleklixiaoy1: https://github.com/openstack/cinder/blob/625cab15b05339fbdb1d71250ba62ab30c1294b5/cinder/objects/base.py#L476-L48109:19
*** aarefiev_ is now known as aarefiev09:19
duleklixiaoy1: Good thing is that we should be able to find any problems with multinode grenade job. It's in experimental queue now and is testing older c-vol with newer c-api and c-sch.09:20
lixiaoy1dulek: but here self.version_cap is min obj version09:21
*** salv-orlando has quit IRC09:21
*** salv-orlando has joined #openstack-cinder09:21
*** lpetrut has joined #openstack-cinder09:21
duleklixiaoy1: Ah, you're right - this is serialization, deserialization is just left as o.vo base version.09:22
lixiaoy1dulek: oh, this is just for serialize_entity. we should have a deserialization function to change 1.0 to 1.1?09:24
duleklixiaoy1: We were considering that at the mid cycle in Raleigh - to for example refetch the object from the DB.09:24
duleklixiaoy1: We've decided to leave that to programmer to be handled explicitly.09:25
lixiaoy1lixiaoy1: got it. I remember this point.09:27
lixiaoy1dulek: with this patch, I think problems will be exploded if there are.09:28
*** nkrinner has quit IRC09:28
duleklixiaoy1: Hm… Before this patch we've had backporting of parameters sent to RPC calls/casts.09:29
duleklixiaoy1: With it we're adding backporting to objects returned from RPC calls/casts.09:29
lixiaoy1dulek: yes09:29
duleklixiaoy1: I'm not sure why this is fundamentally different. Can you elaborate?09:30
lixiaoy1dulek:  not much. think again, this patch just backport the objects returned, and seems that we have no such case from mitaka09:32
*** pgadiya_ has joined #openstack-cinder09:33
lixiaoy1dulek: thank you for helping me clear09:33
duleklixiaoy1: Actually we've had such cases.09:33
lixiaoy1dulek: which case?09:33
duleklixiaoy1: We're returning volume or snapshot object in backup_device dictionary.09:33
*** pgadiya has quit IRC09:34
duleklixiaoy1: That's why I'm introducing the patch - I've tested that it may happen that older c-bak will receive a Volume object from newer c-vol. If that happens - it will not understand the object.09:34
lixiaoy1dulek: yes09:34
openstackgerritYuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc  https://review.openstack.org/35907009:35
lixiaoy1dulek: thank you. I am going to leave office. Have a nice day09:35
duleklixiaoy1: Bye, have a nice evening! :)09:36
*** EinstCra_ is now known as EinstCrazy09:36
*** lixiaoy1 has quit IRC09:36
*** lixiaoy1 has joined #openstack-cinder09:36
openstackgerritNguyen Hung Phuong proposed openstack/cinder: Update code to use Newton, Mitaka as the code name  https://review.openstack.org/35907109:37
*** wxy__ has quit IRC09:38
*** gcb_ has quit IRC09:38
*** wxy__ has joined #openstack-cinder09:39
*** gcb_ has joined #openstack-cinder09:39
*** markvoelker has joined #openstack-cinder09:40
*** nkrinner has joined #openstack-cinder09:42
*** markvoelker has quit IRC09:44
*** laughterwym has joined #openstack-cinder09:57
*** EinstCrazy has quit IRC09:57
*** EinstCrazy has joined #openstack-cinder09:57
*** EinstCra_ has joined #openstack-cinder09:58
*** EinstCra_ is now known as EinstCrazy_09:59
*** EinstCrazy has quit IRC10:02
*** qeelee has joined #openstack-cinder10:05
*** lpetrut1 has joined #openstack-cinder10:10
*** ducttape_ has joined #openstack-cinder10:10
*** lpetrut has quit IRC10:11
*** lpetrut1 is now known as lpetrut10:11
*** alonma has joined #openstack-cinder10:14
*** ducttape_ has quit IRC10:16
*** Julien-zte has quit IRC10:18
*** alonma has quit IRC10:19
openstackgerritCedric Zhuang proposed openstack/cinder: Performance issue for VNX attache LUN  https://review.openstack.org/35872010:20
*** lprice has quit IRC10:22
*** lprice has joined #openstack-cinder10:24
*** yangyapeng has quit IRC10:28
*** EinstCrazy_ has quit IRC10:28
*** ntpttr has quit IRC10:30
openstackgerritSzymon Borkowski proposed openstack/cinder: Convert backup_device to OVO  https://review.openstack.org/32933210:31
*** ntpttr has joined #openstack-cinder10:35
*** qeelee has quit IRC10:43
*** coolsvap is now known as coolsvap_10:53
*** alonma has joined #openstack-cinder10:56
*** alonma has quit IRC11:00
*** sheel has joined #openstack-cinder11:02
*** alyson_ has joined #openstack-cinder11:06
*** markvoelker has joined #openstack-cinder11:09
*** ducttape_ has joined #openstack-cinder11:13
*** mjura_ has joined #openstack-cinder11:14
*** ducttape_ has quit IRC11:17
*** bkopilov has quit IRC11:19
*** markvoelker has quit IRC11:20
*** JoseMello has joined #openstack-cinder11:21
*** thiagolib has quit IRC11:23
*** thiagolib has joined #openstack-cinder11:24
openstackgerritMatan Sabag proposed openstack/cinder: ScaleIO over subscription support  https://review.openstack.org/35654711:30
swamireddyDuncanT: Hi - Can  you please review https://review.openstack.org/#/c/335067/  ( Updated the patch as per your review comments)..11:31
*** yuriy_n17 has joined #openstack-cinder11:33
openstackgerritxiexs proposed openstack/cinder: Convert InvalidVolumeMetadataSize to webob.exc.*  https://review.openstack.org/35621311:34
*** xiexs has joined #openstack-cinder11:37
openstackgerritJiWei proposed openstack/cinder: Implement nfs manage/unmanage of a volume  https://review.openstack.org/34914611:38
*** netapp has quit IRC11:41
*** rcernin has joined #openstack-cinder11:43
yuriy_n17geguileo: Hi. Very sorry for disturbing. Please review https://review.openstack.org/#/c/356624/ whenever you have free time.11:44
*** netapp has joined #openstack-cinder11:48
*** abhinavtechie has joined #openstack-cinder11:50
geguileoyuriy_n17: Will do11:50
*** abhitechie has quit IRC11:51
openstackgerritGorka Eguileor proposed openstack/os-brick: Fix cmd execution stderr, stdout unicode errors  https://review.openstack.org/34108511:52
*** abhinavtechie has quit IRC11:52
yuriy_n17geguileo: Thanks.11:53
*** abhinavtechie has joined #openstack-cinder11:53
*** lkuchlan has quit IRC11:53
openstackgerritxiexs proposed openstack/cinder: [Trivial] Refactor the case test_storwize_vdisk_copy_ops  https://review.openstack.org/35619412:00
*** yangyapeng has joined #openstack-cinder12:01
openstackgerritxiexs proposed openstack/cinder: [Trivial] Refactor the using of dict.get() in the test assertion  https://review.openstack.org/35532312:03
*** pgadiya_ is now known as pgadiya12:03
*** abhinavtechie has quit IRC12:04
*** Poornima_K has quit IRC12:04
*** xyang1 has joined #openstack-cinder12:05
openstackgerritxiexs proposed openstack/cinder: Convert InvalidVolumeMetadataSize to webob.exc.*  https://review.openstack.org/35621312:08
*** tpsilva has joined #openstack-cinder12:08
openstackgerritMatan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver  https://review.openstack.org/35612012:09
*** ducttape_ has joined #openstack-cinder12:10
*** kaisers_ has quit IRC12:11
*** andymaier has quit IRC12:13
*** Lee1092 has quit IRC12:14
dmsimardlixiaoy1: I added a comment in your review https://review.openstack.org/#/c/358670/12:16
dmsimardlixiaoy1: it looks like barbican encrypted volumes are really broken now :(12:16
*** Poornima_K has joined #openstack-cinder12:18
*** pauloewerton has joined #openstack-cinder12:18
openstackgerritTuan proposed openstack/cinder: Remove self.__dict__ for formatting strings  https://review.openstack.org/35849112:21
*** ducttape_ has quit IRC12:21
*** wxy__ has quit IRC12:22
*** wxy__ has joined #openstack-cinder12:23
*** rcernin has quit IRC12:23
*** mjura_ has quit IRC12:23
*** andymaier has joined #openstack-cinder12:25
*** gouthamr has joined #openstack-cinder12:26
*** rcernin has joined #openstack-cinder12:28
*** Poornima_K has quit IRC12:34
*** coolsvap_ is now known as coolsvap12:36
*** edmondsw has joined #openstack-cinder12:40
e0neDuncanT: hi. are you around?12:41
e0neDuncanT: I tried to implement noauth support for cinderclient12:42
*** takedakn has joined #openstack-cinder12:42
e0neDuncanT: it's very depended on keystone12:43
e0neDuncanT: according to help, it can work w/o keystone by provider cinder url and other options, but at the first glance, this feature is broken12:43
e0neDuncanT: I'll try to take a look on it once again later this week12:44
*** Poornima_K has joined #openstack-cinder12:48
DuncanTe0ne: It was very broken last I looked for sure12:50
*** lkuchlan has joined #openstack-cinder12:51
DuncanTe0ne: If I deleted enough code then I could make it work, but it was a bit here, a bit there, so nothing easy to clean up12:51
*** lkuchlan has quit IRC12:52
DuncanTe0ne: It wouldn't surprise me if it has gotten worse since then. Many things are now written to assume keystone12:52
*** lkuchlan has joined #openstack-cinder12:52
*** lkuchlan is now known as lkuchlan_wfh12:52
dulekAnyone knows where we're keeping v3 api-ref?12:54
dulekI can only see v1 and v2 folders in cinder/api-ref…12:55
*** chlong has quit IRC12:56
*** xyang has joined #openstack-cinder12:57
*** dustins has joined #openstack-cinder13:00
*** coolsvap is now known as coolsvap_13:00
*** rlrossit has joined #openstack-cinder13:00
scottdadulek: There was a patch up to add v3 api-ref...let me look...13:02
*** rcernin has quit IRC13:03
swamireddyDuncanT: Hi - Can  you please review https://review.openstack.org/#/c/335067/  ( Updated the patch as per your review comments)..13:03
DuncanTswamireddy: Probably tomorrow before I get time, sorry13:04
swamireddyDuncanT: Thank you.. NOP.13:06
swamireddydulek: Thanks for the review (https://review.openstack.org/#/c/335067/ ) with +2.13:07
dulekscottda: Oh, cool. No need to block 335067 because of that however.13:07
scottdadulek: Maybe I'm confusing patches, i'm not finding it. We might not have it yet, but I thought there was a review out there...13:07
dulekscottda: Thanks anyway! :)13:07
scottdadulek: Filed a bug to track the need for v3 api-ref: https://bugs.launchpad.net/cinder/+bug/161607213:11
openstackLaunchpad bug 1616072 in Cinder "Cinder lacks v3 api-ref" [Undecided,New]13:11
*** porrua has joined #openstack-cinder13:11
*** andymaier has quit IRC13:12
*** chlong has joined #openstack-cinder13:13
*** qeelee has joined #openstack-cinder13:13
*** qeelee has quit IRC13:15
*** raunak has joined #openstack-cinder13:18
*** akerr has joined #openstack-cinder13:21
*** julim has joined #openstack-cinder13:21
*** ducttape_ has joined #openstack-cinder13:21
*** andymaier has joined #openstack-cinder13:22
*** arecknag has quit IRC13:22
nikeshmhi if any one gets time, please give one more +2 on https://review.openstack.org/#/c/356475, its already have one +2, thanks13:22
*** pgadiya has quit IRC13:23
*** andymaier has quit IRC13:23
*** bigdogstl has joined #openstack-cinder13:24
*** andymaier_ has joined #openstack-cinder13:24
*** david-lyle has joined #openstack-cinder13:24
*** timcl has joined #openstack-cinder13:24
*** takedakn has quit IRC13:24
*** takedakn has joined #openstack-cinder13:25
patrickeasthemna: so finally got some good test runs, looks like https://review.openstack.org/#/c/231135/ is indeed needed to make FC moar happy13:26
patrickeasthemna: i still had some intermittent mpath devices left behind : / but I'm suspicious of my multipath.conf being wrong13:27
*** takedakn has quit IRC13:30
smcginnispatrickeast: Oh good, I was going to ask if you ever figured that out.13:30
*** Julien-zte has joined #openstack-cinder13:30
smcginnispatrickeast: Didn't think you would be online so early. ;)13:30
patrickeastsmcginnis: haha yea, in NY this week13:32
*** bkopilov has joined #openstack-cinder13:32
patrickeastUsually wouldn't be awake, or at least working yet13:32
*** jungleboyj has joined #openstack-cinder13:33
smcginnispatrickeast: Oh, that's right.13:33
smcginnispatrickeast: Are you attending the ops midcycle too? Or heading back after East is over?13:33
patrickeastsmcginnis: heading back Thursday morning13:35
*** nkrinner is now known as nkrinner_afk13:35
smcginnispatrickeast: Ah, too bad. I get in late Wednesday. Couldn't do OpenStack East but wanted to go to the Ops event.13:36
*** sdake has joined #openstack-cinder13:36
*** sheel has quit IRC13:36
patrickeastsmcginnis: ah gotcha, how late Wednesday? :D13:37
*** sdake_ has joined #openstack-cinder13:37
smcginnispatrickeast: 11:20 PM (what was I thinking!)13:37
*** ducttape_ has quit IRC13:38
dulekpatrickeast: OpenStack East seems like an event with name crafted for you. :)13:38
patrickeastsmcginnis: ah perfect, can just head right over to whatever bar we're at13:39
patrickeastdulek: haha exactly!13:39
*** raunak has quit IRC13:39
erlonsmcginnis: Hi Sean, can you give a look on this 2 patches when you have a chance? It's being around for a while with no core reviews:13:40
erlonsmcginnis: https://review.openstack.org/#/c/333992/13:40
erlonsmcginnis: https://review.openstack.org/#/c/310785/13:40
*** jraju has quit IRC13:40
*** raunak has joined #openstack-cinder13:40
*** sdake has quit IRC13:41
*** raunak has quit IRC13:42
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: NexentaStor4 NFS driver cache capacity and use sessions  https://review.openstack.org/35614213:43
*** bigdogstl has quit IRC13:43
smcginnispatrickeast: Hah, maybe. ;)13:43
smcginniserlon: Got it in my queue now.13:43
*** sdague has joined #openstack-cinder13:43
erlonsmcginnis: thanks!13:44
*** ducttape_ has joined #openstack-cinder13:45
*** sandanar has quit IRC13:45
*** diablo_rojo_ has joined #openstack-cinder13:48
*** Poornima_K has quit IRC13:48
*** rlrossit_ has joined #openstack-cinder13:51
*** eharney has joined #openstack-cinder13:53
*** rlrossit has quit IRC13:55
*** jgregor has joined #openstack-cinder13:55
*** bigdogstl has joined #openstack-cinder13:57
*** kaisers_ has joined #openstack-cinder13:59
*** bigdogstl has quit IRC14:02
*** pcaruana has quit IRC14:02
*** david-lyle has quit IRC14:04
*** kaisers_ has quit IRC14:04
*** tbarron|gone is now known as tbarron14:05
*** coolsvap_ is now known as coolsvap14:05
openstackgerritEric Harney proposed openstack/cinder: Tests: Fix racy volume unit test  https://review.openstack.org/35880714:09
*** ebalduf has joined #openstack-cinder14:12
*** takedakn has joined #openstack-cinder14:12
*** alyson_ has quit IRC14:13
*** alonma has joined #openstack-cinder14:14
*** cknight has joined #openstack-cinder14:15
*** pcaruana has joined #openstack-cinder14:17
erlonjungleboyj: ping14:18
jungleboyjerlon: Pong14:19
*** alonma has quit IRC14:19
erlonjungleboyj: Hey Jay14:19
erlonjungleboyj: I see you have updated the NFS patch14:19
*** rajinir has joined #openstack-cinder14:19
jungleboyjerlon: Yes.14:19
erlonjungleboyj: have you found a way to attach a disk with snapshots?14:20
jungleboyjI haven't gotten that far yet.  Just got it rebased and fixed failing test cases.14:21
jungleboyjGoing to start on the other issues next week hopefully.14:21
erlonjungleboyj: for what I have found, the problem (at least the bigger part of it) is that nova is not considering the snapshot files in the attachment14:21
erlonjungleboyj: so, let me try to update you on what I have discovered so far14:22
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups  https://review.openstack.org/32245914:23
erlonjungleboyj: there are several permission problems, when libvirt creates snapshot  from a file owned by ubuntu:ubuntu, it  gives some files to qemu-kvm:kvm, while others goes to root14:24
jungleboyjerlon: Interesting.14:24
erlonjungleboyj: those problems does not happens if you use nfs_secure = defaut14:24
erlonjungleboyj: so, Im using this to continue investigating the other problem14:25
jungleboyjInteresting.  I have seen other issues around using the default option for security.  I need to investigate those.  Wonder if we have something more systemic.14:26
*** pbandark has quit IRC14:27
erlonjungleboyj: the other problem is that, once you create a snapshot chain for a volume, Nova only considers the base file, so, for example, you attach a file A, create online snapshots A.S1, A.S2, A.S3. Great, this works, Nova writes now to A.S3. But, if you detach the volume, and re-attach it, Nova will attach only A, disconsidering A.S1, A.S2 amd A.S314:28
*** sdague has quit IRC14:28
*** david-lyle has joined #openstack-cinder14:28
erlonjungleboyj: what do you mean by systemic?14:28
erlonjungleboyj: in this last problem, it also happens if you create the snapshot offline and tries to attach it.14:29
erlonjungleboyj: so, I started to try to attach a volume with a backing chain in libvirt, disconsidering Nova14:30
*** sage___ has quit IRC14:30
eharneysomething is going very wrong if Nova is not loading the correct file...14:31
jungleboyjerlon: I am wondering if there are other problems coming from how security/permissions are being handled.14:31
erlonjungleboyj: annnd, I'm still here. It can be a libvirt or a QEMU problem, which is what I'm at the moment14:31
eharneybecause Cinder should be telling it to load the file at the tip of the chain14:31
*** timcl has quit IRC14:31
*** sage has joined #openstack-cinder14:32
erloneharney: yes, Cinder should send the backing chain to Nova, and nova should attach the top file of the chain14:32
eharneyCinder doesn't send the whole chain, libvirt inspects that when it attaches the files14:32
erloneharney: exactly, but this information is not even being sent to Nova14:32
eharneyerlon: that information doesn't need to be sent to Nova, only the last filename does14:32
erloneharney: it doesn't, should it?14:32
eharneyerlon: why should it?14:33
erloneharney: hmm, yeah, that should be enought, but as I didnt know how libvirt does the attach I tried to send all chain14:33
eharneyerlon: libvirt will follow the qcow2 backing file pointers in the files, and you can see the result in libvirt dumpxml on the domain after it's attached14:34
erloneharney: I mean libvirt is not inspecting the file chain14:34
erloneharney: hmmm14:34
eharneyerlon: this should all mostly work in some form regardless, but what version of libvirt?14:35
erloneharney: well, I tryed to do that with libvirt but couldn't14:35
*** sdake_ has quit IRC14:35
erloneharney: jungleboyj: https://www.redhat.com/archives/libvirt-users/2016-August/msg00048.html14:35
erloneharney: 1.3.114:36
*** Lee1092 has joined #openstack-cinder14:36
eharneyerlon: does qemu-img info show the backing file pointer?14:36
*** vern has quit IRC14:36
erloneharney: yes, have doubled checked that14:36
erloneharney: all with the correct permissions14:36
eharneyerlon: i haven't tried doing this w/ virsh attach-device14:36
erloneharney: the guys that is replying the thread said he coud attach but, didn't give enough details and I couldn't reproduce14:38
eharneyerlon: but the <disk> block in your dumpxml and <backingStore> look correct14:38
*** takedakn has quit IRC14:38
eharneyerlon: certainly possible that it's permissions issues somewhere14:39
erloneharney: mhm, I just got the <disk> snip with the snapshot and tried to attach14:39
erloneharney: mhm, yep, libvirt might be setting permissions and then asking qemu to read the files14:39
erloneharney: because looking at libvirt log, I can see the error returned from qemu14:40
*** sdake has joined #openstack-cinder14:41
erloneharney: I'll try to add the disk direct to qemu, to bypass libvirt14:41
openstackgerritMichal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls  https://review.openstack.org/25245814:46
openstackgerritMichal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO  https://review.openstack.org/25303314:46
openstackgerritMichal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype)  https://review.openstack.org/25730514:46
openstackgerritxing-yang proposed openstack/cinder: Add group snapshots  https://review.openstack.org/32805214:47
*** lprice1 has joined #openstack-cinder14:48
*** lprice has quit IRC14:49
*** bigdogstl has joined #openstack-cinder14:52
*** Yogi1 has joined #openstack-cinder14:53
*** xinli has joined #openstack-cinder14:54
*** david-lyle has quit IRC14:54
*** timcl has joined #openstack-cinder14:55
*** bwallis has quit IRC14:55
*** raunak has joined #openstack-cinder14:55
*** alonma has joined #openstack-cinder14:56
jungleboyjerlon: Thank you for working on that.  Keep me updated on your progress.  If you are working through that I can work on the fact that cloning is currently listed as supported but isn't implemented.14:57
*** bigdogstl has quit IRC14:57
*** enriquetaso has joined #openstack-cinder14:58
erlonjungleboyj: ok I will, hmmm that would be good thanks15:00
*** alonma has quit IRC15:00
jungleboyjerlon: Yeah, that one kind-of caught us by surprise when we found it.15:00
scottdageguileo: I think that snapshot-create is now cluster aware with your latest HA patches. Is that correct?15:02
geguileoscottda: Yes15:03
geguileoscottda: All operations should be cluster aware now15:04
*** vern has joined #openstack-cinder15:04
scottdageguileo: Cool. That is what I'm seeing. Your blog on manual testing still talks about snapshot-create Not being cluster aware, but I'd remembered you'd changed that. Testing confirms that it works.15:04
geguileoscottda: I believe I updated the post...15:05
openstackgerritYuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc  https://review.openstack.org/35907015:05
scottdageguileo: I don't see an update in http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/ after a browser refresh . Section 2.215:06
geguileoscottda: Oh, I must have forgotten to change it there, let me see15:06
geguileoscottda: I updated it in "Current cluster aware operations"15:07
openstackgerritYuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc  https://review.openstack.org/35907015:07
geguileoscottda: Oooops, yes, I should update that one as well  XD15:07
scottdageguileo: NP. It's great stuff. And all working as expected.15:07
geguileoscottda: Great to hear it, I'm working on the API stuff now15:08
*** salv-orlando has quit IRC15:08
*** salv-orlando has joined #openstack-cinder15:08
openstackgerritYuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc  https://review.openstack.org/35907015:10
*** leeantho has joined #openstack-cinder15:11
scottdageguileo: Is it correct that after a c-vol has been disabled, that the heartbeat is still updated?15:12
geguileoscottda: I believe that is true for master as well15:12
geguileoscottda: They are different things15:12
scottdageguileo: OK, just checking. I need to look at that code....15:13
karthikp_eharney: Hi. I wanted to check with you about https://review.openstack.org/#/c/322977/.... Since there was other changes like volumeAttach status ..i thought it was good to have this15:13
*** mjura has quit IRC15:15
*** Julien-zte has quit IRC15:15
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Add dell_server_os configuration option  https://review.openstack.org/35887815:16
*** alonma has joined #openstack-cinder15:16
geguileoscottda: Disabled services will still receive requests to attach/detach, delete, etc.  So it's important that they appear as up  ;-)15:17
openstackgerritNam Nguyen Hoai proposed openstack/cinder: Clean import in code  https://review.openstack.org/35848015:17
scottdageguileo: Hmmm...Well, this is after I've killed c-vol. It shows as "down" in service-list.15:19
geguileoscottda: If you kill c-vol it should appear as down15:19
geguileoscottda: If you only have 1 node15:19
*** andymaier_ has quit IRC15:19
geguileoscottda: Is it reporting as down when you only stop 1 of the 2 nodes?15:19
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add new supported driver checks  https://review.openstack.org/35560815:20
*** bwallis has joined #openstack-cinder15:21
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Mask out passwords when tracing  https://review.openstack.org/35437015:22
*** david-lyle has joined #openstack-cinder15:23
*** xinli has quit IRC15:23
scottdageguileo: cinder-volume for 1st c-vol is 'down' in 'cinder service-list' after I kill it. And it is 'up' for c-vol2. As expected. It is the 'last heartbeat' field from 'cluster-list' that confused me. But that is the heartbeat for the cluster, not the hosts, so it is correct.15:23
*** xinli has joined #openstack-cinder15:23
geguileoscottda: Oh, ok, yes, that's correct15:24
* geguileo I should have mentioned that in the post15:25
*** cdelatte has joined #openstack-cinder15:27
*** david-lyle has quit IRC15:33
*** alonma has quit IRC15:36
*** bwallis has quit IRC15:37
*** sdague has joined #openstack-cinder15:38
*** tesseract- has quit IRC15:46
*** bigdogstl has joined #openstack-cinder15:46
*** alyson_ has joined #openstack-cinder15:47
stevemarcan anyone make heads or tails of this? https://bugs.launchpad.net/python-cinderclient/+bug/1616070 ?15:48
openstackLaunchpad bug 1616070 in python-openstackclient "all volume v1 commands broken" [Undecided,New] - Assigned to Rui Chen (kiwik-chenrui)15:48
*** cdelatte has quit IRC15:48
*** kaisers_ has joined #openstack-cinder15:48
*** mmedvede has quit IRC15:53
*** kaisers_ has quit IRC15:53
*** bigdogstl has quit IRC15:53
*** sdague has quit IRC15:54
smcginnisstevemar: Hmm, logger is not getting initialized somehow.15:55
*** pcaruana has quit IRC15:56
openstackgerritShay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions  https://review.openstack.org/34683115:57
smcginnisstevemar: Passing in self.logger instead of self._logger: http://git.openstack.org/cgit/openstack/python-cinderclient/tree/cinderclient/client.py#n12015:57
smcginnisWow, since April.16:00
smcginnisThat's a good sign that not many people are still using v1. :)16:00
scottdaI don't see the issue in cinderclient.16:00
smcginnisscottda: Passing in v1?16:01
smcginnisI was able to repro (and verify fix) here.16:01
scottdasmcginnis: Oh! I had '--debug' and it worked.16:01
scottdasmcginnis: But without '--debug' it doesn't16:01
smcginnisscottda: Yeah, works fine with debug.16:01
*** david-lyle has joined #openstack-cinder16:01
scottdagot it. Sorry16:01
stevemarscottda: ah16:01
smcginnisscottda: Which is actually weird. How is that set with debug but not otherwise?16:01
stevemarsounds like cinderclient has a legit bug :O16:02
smcginnisstevemar: Yeah, looks legit.16:02
scottdasmcginnis: Do you have time to put up a patch, or do you want me to?16:02
*** lkuchlan_wfh has quit IRC16:02
smcginnisscottda: I added that to the bug report. He has it assigned to himself, so I was going to see if they would take care of it.16:03
scottdasmcginnis: Forget it, I see it is assigned16:03
smcginnisscottda: ;)16:03
smcginnisscottda: If no activity soon we can just grab it I suppose.16:03
smcginnisBut if it's been there since april maybe not a huge rush.16:03
scottdaYeah, like you guys said, the most interesting thing is that noone's found it sooner, meaning not much use of /v116:04
smcginnisMust be something in keystoneauth1 that sets logger in Adapter on debug but not otherwise. But should still be self._logger.16:04
scottdaDuncanT, aspiers, geguileo, scottda, dulek, ddeja, patrickeast Want to talk about HA?16:04
geguileoscottda: Sure, though I don't have news16:05
aspiersI have a question about active/passive cinder-volume, but not sure if it's in scope for this meeting16:05
scottda#startmeeting cinder_aaha16:05
openstackMeeting started Tue Aug 23 16:05:24 2016 UTC and is due to finish in 60 minutes.  The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot.16:05
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:05
openstackThe meeting name has been set to 'cinder_aaha'16:05
geguileoscottda: I always welcome the opportunity to beg for reviews  ;-)16:05
scottdageguileo: Yeah, that's a good use of this meeting. Everyone Review!!! and Test !!!16:05
scottdaAnyone interested in HA/AA should run through geguileo 's manual tests: http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/16:06
scottdaThey make some of the inner workings very clear.16:07
geguileoThere are 2 patches that can be merged in this cycle: https://review.openstack.org/#/c/344224/  https://review.openstack.org/#/c/35306816:07
*** bwallis has joined #openstack-cinder16:07
scottdaThere may not be much else to talk about...so maybe aspiers , you should ask about active/passive?16:07
aspiersok :)16:07
aspiersso an alternative approach to a/p has been proposed16:07
aspiersI guess using a virtual hostname is not unusual16:08
aspiersbut this conflicts with cinder-scheduler16:08
aspiersso the suggestion is a kind of hack:16:08
aspiersif a machine needs to run both cinder-scheduler and cinder-volume, use separate cinder.conf files for each16:08
aspierswhich only differ with the hostname setting16:09
aspiersdoes that make sense?16:09
geguileoaspiers: You don't need different cinder.conf files16:09
*** pcaruana has joined #openstack-cinder16:09
geguileoaspiers: You just need the cinder-volume to add an additional config file16:09
geguileoaspiers: That just changes the host value16:09
aspiersoh ok, even cleaner16:09
aspiersbut the same principle16:09
geguileoaspiers: That's how I do it in my HA A/A tests16:10
aspiersI didn't know it was possible to overlay config files16:10
geguileoaspiers: Yes, as many as you want16:10
aspierscool, thanks! is that documented somewhere?16:10
geguileoaspiers: They overwrite the previous values16:10
aspiersI could work with beekhof to get it into the ha-guide16:10
aspiersso you specify -c multiple times? or reference the underlying cinder.conf from within the one on top?16:11
scottdaaspiers: Just like a normal config file, and just add to the startup line:16:14
scottdauser@localhost:$ usr/bin/cinder-volume --config-file /etc/cinder/cinder.conf --config-file /etc/cinder/host1.conf & echo $! >/opt/stack/status/stack/c-vol.pid; fg || echo "c-vol failed to start" | tee "/opt/stack/status/stack/c-vol.failure"16:14
aspiersscottda: perfect, thanks a lot!16:15
scottdaaspiers: Just that "cluster = mycluster" will be overriden (or added if it didn't exist)16:15
aspiersyup, makes sense16:16
scottdaok, keep talking amongst yourselfs. But meeting seems over...16:16
openstackMeeting ended Tue Aug 23 16:16:39 2016 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:16
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.html16:16
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.txt16:16
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.log.html16:16
*** rlrossit_ has quit IRC16:21
*** bigdogstl has joined #openstack-cinder16:23
*** rlrossit has joined #openstack-cinder16:25
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Add exclude_domain_ip option  https://review.openstack.org/35932816:27
*** Yogi1 has quit IRC16:27
*** bigdogstl has quit IRC16:27
*** zhangguoqing has quit IRC16:29
*** eharney has quit IRC16:35
*** rmcall has joined #openstack-cinder16:35
*** zul has quit IRC16:36
*** Suyi has joined #openstack-cinder16:36
*** jraju has joined #openstack-cinder16:38
*** fernnest has joined #openstack-cinder16:38
*** jraju has quit IRC16:39
*** laughterwym has quit IRC16:43
nikeshmsmcginnis:  hi16:47
*** lkuchlan_wfh has joined #openstack-cinder16:47
*** asselin has joined #openstack-cinder16:47
nikeshmhemna: hi16:48
*** zul has joined #openstack-cinder16:48
nikeshmhi gentle reminder, if u get time, please review https://review.openstack.org/#/c/35647516:51
*** e0ne has quit IRC16:52
nikeshmalready have +2 from xyang, it is replication failback support for kaminario drivers16:52
openstackgerritRaunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage  https://review.openstack.org/34908616:55
*** mtanino has joined #openstack-cinder16:57
karthikp_smcginnis: Hi17:01
karthikp_smcginnis: I wanted to know what you feel about this change: https://review.openstack.org/#/c/322977/17:03
*** Apoorva has joined #openstack-cinder17:04
smcginniskarthikp_: You should probably talk to eharney about it.17:04
*** Apoorva has quit IRC17:04
smcginniskarthikp_: But I believe we've switched some other stuff over to enums, so not sure.17:05
*** cdelatte has joined #openstack-cinder17:05
*** Apoorva has joined #openstack-cinder17:05
smcginniskarthikp_: What we have does work, but enums help make things clear and limited to correct, expected values. So not sure. Guess I don't have as strong of an opinion as Eric does.17:05
*** Guest25180 is now known as med_17:06
*** med_ has joined #openstack-cinder17:06
*** med_ is now known as medberry17:06
*** medberry is now known as med_17:06
karthikp_smcginnis: Sure. Yes we have changed many things to enums! So I am really not sure why not this. I will check with him17:06
openstackgerritRaunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage  https://review.openstack.org/34908617:07
smcginniskarthikp_: Cool. I think it would be good to discuss it with him. Looks fine to me.17:07
*** asselin has quit IRC17:09
*** lkuchlan_wfh has quit IRC17:09
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Live Volume Autofailover  https://review.openstack.org/35078217:10
*** mtanin___ has joined #openstack-cinder17:10
*** mtanino has quit IRC17:12
*** asselin has joined #openstack-cinder17:15
*** bigdogstl has joined #openstack-cinder17:15
karthikp_dulek: geguileo: Can I work on the review to create a mapping of release to object & rpc version pins?? mentioned by Gorka in https://review.openstack.org/#/c/350533/1/cinder/objects/service.py@17017:16
geguileokarthikp_: I have no objections and I'm actually glad if you do  :-)17:17
*** rmk has quit IRC17:18
*** cdelatte has quit IRC17:19
*** bigdogstl has quit IRC17:20
*** alonma has joined #openstack-cinder17:21
dmsimardlixiaoy1: hi, are you around ?17:24
dmsimardlixiaoy1: do you think you could address the comment in https://review.openstack.org/#/c/358670/ ? Other than that I think it looks okay.17:24
*** alonma has quit IRC17:26
*** zul has quit IRC17:26
*** xyang has quit IRC17:27
*** xyang has joined #openstack-cinder17:28
*** cdelatte has joined #openstack-cinder17:30
*** cdelatte has quit IRC17:31
*** lpetrut has quit IRC17:33
openstackgerritTom Barron proposed openstack/cinder: Use constraints for api-ref environment  https://review.openstack.org/35936217:34
*** gcb_ has quit IRC17:36
*** bwallis has quit IRC17:36
*** gcb_ has joined #openstack-cinder17:36
*** kaisers_ has joined #openstack-cinder17:37
*** e0ne has joined #openstack-cinder17:39
*** cdelatte has joined #openstack-cinder17:40
*** eharney has joined #openstack-cinder17:40
*** kaisers_ has quit IRC17:42
*** cdelatte has quit IRC17:42
*** rmcall has quit IRC17:44
*** lkuchlan_wfh has joined #openstack-cinder17:45
*** rmcall has joined #openstack-cinder17:49
*** bigdogstl has joined #openstack-cinder17:51
*** zhugaoxiao has joined #openstack-cinder17:53
*** e0ne has quit IRC17:55
*** zul has joined #openstack-cinder17:55
*** Administrator__ has quit IRC17:55
*** eharney has quit IRC17:56
*** lkuchlan_wfh has quit IRC17:56
*** bigdogstl has quit IRC17:58
*** e0ne has joined #openstack-cinder18:00
*** jamielennox|away is now known as jamielennox18:01
*** cdelatte has joined #openstack-cinder18:07
*** eharney has joined #openstack-cinder18:08
*** bardia has joined #openstack-cinder18:10
openstackgerritAnthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers  https://review.openstack.org/34806018:10
*** alonma has joined #openstack-cinder18:23
*** raunak has quit IRC18:23
*** alonma has quit IRC18:30
*** lpetrut has joined #openstack-cinder18:33
*** cdelatte has quit IRC18:37
*** xyang has quit IRC18:38
*** zul has quit IRC18:46
openstackgerritSofia Enriquez proposed openstack/cinder: RBD: Error when manage an already-managed volume  https://review.openstack.org/35428918:50
*** xyang has joined #openstack-cinder18:51
*** timcl has quit IRC18:53
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: LeftHand add entry point tracing  https://review.openstack.org/33912818:53
*** raunak has joined #openstack-cinder18:55
*** raunak has quit IRC18:56
*** markvoelker has joined #openstack-cinder18:58
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Fixes the length of resource field in database for quota tables  https://review.openstack.org/35521418:58
*** markvoelker has quit IRC19:01
*** zul has joined #openstack-cinder19:01
*** xyang has quit IRC19:02
*** xyang has joined #openstack-cinder19:03
dmsimardAnyone familiar with encrypted volumes ? Seeing issues after recent merges.19:06
dmsimardParticularly curious around the docs (and the tempest test) claiming to use aes-512 while that doesn't seem to really exist19:07
*** alee has joined #openstack-cinder19:08
dmsimardFWIW it's mentioned in the review that introduced the issue https://review.openstack.org/#/c/25150319:10
dmsimard"Another fix will be needed to address the use case of aes-xts with a key size of 512 -- key managers may not be able to create 512 bit AES keys."19:11
*** timcl has joined #openstack-cinder19:11
*** cdelatte has joined #openstack-cinder19:13
eharneydmsimard: what platform is the test running on?19:20
dmsimardeharney: CentOS + RDO trunk installed/configured through puppet-openstack-integration19:20
*** pcaruana has quit IRC19:20
*** andymaier_ has joined #openstack-cinder19:21
dmsimardI just added a comment in https://bugs.launchpad.net/cinder/+bug/151454619:21
openstackLaunchpad bug 1514546 in Cinder "Cinder volume encryption uses default parameters for keys" [Undecided,Fix released] - Assigned to Lisa Li (lisali)19:21
*** cdelatte has quit IRC19:22
*** kaisers_ has joined #openstack-cinder19:26
* eharney waiting for gerrit to work19:26
*** salv-orlando has quit IRC19:28
*** salv-orlando has joined #openstack-cinder19:28
eharneydmsimard: kind of a mess there, but i wonder if we can work around it for now by setting key size to 256 in tempest19:29
dmsimardeharney: I was thinking about that as well, at least temporarily19:29
eharneydmsimard: it looks like 512 "should" work, but there are a few issues in the way19:30
dmsimardeharney: even so, the key length issue is actually hidden behind https://review.openstack.org/#/c/358670/19:30
*** kfarr has joined #openstack-cinder19:30
eharneydmsimard: right19:30
kfarrdmsimard ping19:30
dmsimardkfarr: ohai o/19:30
dmsimardkfarr: can we chat cinder+barbican? :D19:30
*** kaisers_ has quit IRC19:30
eharneyjust the person we need :)19:30
kfarrdmsimard, yes!19:30
kfarrsorry for all the trouble D:19:31
dmsimardkfarr: it's okay, I think it highlights a gap in CI19:31
aleekfarr, we've been missing you!19:31
dmsimardRDO tends to catch a lot of those gaps :P19:31
kfarrdmsimard, so it looks like lisali has the fix up for the incorrect parameter, (which I saw you said you integrated already)19:32
kfarrdmsimard though to address the issue about aes 512...19:32
aleekfarr, so whats up with cinder asking for aes 512?19:32
*** xinli has quit IRC19:32
kfarryou're correct that aes 512 keys don't really exist19:32
dmsimardkfarr: yes, I think lisa will need to fix those tests first -- I bumped into the key length issue when manually packaging lisa's fix19:32
kfarrthough like you pointed out, the xts changes things19:32
*** salv-orlando has quit IRC19:32
aleekfarr, in this case, barbican simple_crypto plugin rejects the request19:34
kfarrif we could change the tests to use xts-256, that would be great19:34
aleekfarr, because oit fails the supports()) method19:34
dmsimardkfarr: yeah I think changing the key length in the tempest test is the /easy/ fix19:34
kfarralee, hmm I thought simple_crypto wasn't picky about bit lengths19:34
aleekfarr, you mean aes-256?19:34
aleekfarr, up to 256 :)19:35
dmsimardkfarr: but that fixes just tempest, the docs might warrant a note about this19:35
kfarrdmsimard, true19:35
*** rlrossit_ has joined #openstack-cinder19:35
kfarrI can attempt to explain the xts 512 bits vs 256 bits, though reaperhulk does a much better job19:37
smcginnishemna: Can you take a look at this if/when you have a chance? https://review.openstack.org/#/c/231135/19:37
kfarralee, to support aes-xts-512, we'd essentially need to create two 256 bit keys and concatenate them19:38
*** rlrossit has quit IRC19:38
dmsimardkfarr, alee: https://review.openstack.org/35939119:38
dmsimardeharney: ^19:38
eharneydmsimard: great19:39
kfarrthanks dmsimard19:39
dmsimardkfarr: now we need those two reviews merged and we're in business :)19:40
aleekfarr, the simple plugin checks to see if we have an algorithm in the supported list and key size in that list too19:41
aleekfarr, see https://github.com/openstack/barbican/blob/master/barbican/plugin/crypto/crypto.py#L4719:41
aleekfarr, the checking isn't sophisticated -- we dont match algorithm to key length19:42
aleekfarr, but it is the simple plugin19:43
aleekfarr, if we need to support 512 bit keys for some reason , then we should update barbican simple key plugin too19:43
kfarralee, most key managers probably won't support 512 bit keys though, KMIP doesn't19:43
aleekfarr, right - I was curious about the original choice of size in the docs/test ..19:44
*** raunak has joined #openstack-cinder19:45
kfarrthose docs were written when we were still using ConfKeyManager, and it didn't really care about key size19:45
*** andymaier_ has quit IRC19:46
aleekfarr, nice round number :)19:47
aleekfarr, I think we need a bug to change the docs ..19:47
kfarralee, yes, that would probably be good19:47
*** Apoorva has quit IRC19:48
*** andymaier_ has joined #openstack-cinder19:49
kfarralee, brb19:50
*** andymaier_ has quit IRC19:50
*** andymaier_ has joined #openstack-cinder19:51
*** Apoorva has joined #openstack-cinder19:52
*** kfarr has quit IRC19:55
*** markvoelker has joined #openstack-cinder19:57
*** kfarr has joined #openstack-cinder19:57
dmsimardkfarr, alee: thanks for your help19:58
dmsimardI wish we had the capacity (both human and hardware) to do third party CI on everything, we'd catch this sort of thing earlier in the process :)19:59
aleedmsimard, fun fun :)19:59
*** lpetrut has quit IRC20:03
*** porrua has quit IRC20:04
*** dmsimard has left #openstack-cinder20:09
*** akerr has quit IRC20:09
*** e0ne has quit IRC20:12
openstackgerritxing-yang proposed openstack/cinder: Fix policy check for group types  https://review.openstack.org/35940320:13
*** edmondsw has quit IRC20:13
*** haplo37__ has joined #openstack-cinder20:14
*** alkhodos- has quit IRC20:14
*** xinli has joined #openstack-cinder20:18
*** e0ne has joined #openstack-cinder20:25
*** rhefner has quit IRC20:31
*** aswadr_ has quit IRC20:31
*** coolsvap has quit IRC20:31
*** rhefner has joined #openstack-cinder20:32
*** rhefner has quit IRC20:32
*** rhefner has joined #openstack-cinder20:33
*** e0ne has quit IRC20:37
*** lprice1 has quit IRC20:39
xyanghemna: ping20:42
*** rhefner has quit IRC20:43
scottdaHere's a spec update with a couple of +2's...if anyone else want to have a look...20:46
*** rhefner has joined #openstack-cinder20:48
smcginnisscottda: Wish it was easy to diff between the two files in gerrit.20:49
smcginnisscottda: What's the main difference?20:49
scottdaI agree...maybe you can diff em in the repo...20:49
smcginnisscottda: Yeah, I can pull it down.20:49
smcginnisscottda: Or just review the new file and make sure it makes sense. ;)20:50
scottdasmcginnis: I reviewed it a while back, but I cannot recall the issues. I just was going through the H/A bp and trying to update status.20:50
*** cknight has quit IRC20:50
*** alonma has joined #openstack-cinder20:51
*** markvoelker has quit IRC20:52
*** alonma has quit IRC20:56
*** timcl has quit IRC20:56
openstackgerritMerged openstack/cinder-specs: Update Job Distribution for A/A Specs  https://review.openstack.org/32728320:57
*** gouthamr has quit IRC20:58
*** chris_morrell has quit IRC21:09
*** jgregor has quit IRC21:11
xyangsmcginnis: ScaleIO CI is broken now without this fix: https://review.openstack.org/#/c/355231/21:12
*** julim has quit IRC21:12
xyangsmcginnis: henna found out this issue21:13
xyangsorry, hemna21:14
xyanghemna: it is not me.  it was auto corrected:)21:14
*** kaisers_ has joined #openstack-cinder21:15
*** david-lyle has quit IRC21:15
smcginnisxyang: Done21:16
*** david-lyle has joined #openstack-cinder21:18
*** enriquetaso has quit IRC21:19
*** kaisers_ has quit IRC21:20
*** pauloewerton has quit IRC21:22
*** ducttape_ has quit IRC21:23
*** bardia_ has joined #openstack-cinder21:24
*** bardia has quit IRC21:24
*** eharney has quit IRC21:27
openstackgerritMerged openstack/python-cinderclient: Add group types and group specs  https://review.openstack.org/32015721:30
*** rlrossit_ has quit IRC21:30
*** salv-orlando has joined #openstack-cinder21:32
*** gouthamr has joined #openstack-cinder21:34
*** Lee1092 has quit IRC21:44
*** sdake has quit IRC21:45
*** sdake has joined #openstack-cinder21:45
*** sdake has quit IRC21:45
*** sdake has joined #openstack-cinder21:46
*** ducttape_ has joined #openstack-cinder21:46
openstackgerritRaunak Kumar proposed openstack/cinder: Add Fibre Channel support for Nimble Storage  https://review.openstack.org/35944821:56
*** andymaier_ has quit IRC22:15
*** rmcall has quit IRC22:15
*** alee has quit IRC22:16
openstackgerritKaitlin Farr proposed openstack/cinder: Fix incorrect parameter in create_key  https://review.openstack.org/35867022:18
*** rmk has joined #openstack-cinder22:21
*** lpetrut has joined #openstack-cinder22:24
*** sdague has joined #openstack-cinder22:26
*** lpetrut has quit IRC22:28
ntpttr___hey everyone, I'm trying to back up to swift using devstack and running into this error, anyone know what's going wrong? http://paste.openstack.org/show/562764/22:29
*** sdague has quit IRC22:31
DuncanTntpttr___: Is it a clean devstack install or particularly modified? Devstack usually gets rootwrap filters right- it would show up on the gate pretty quick if it didn't I'd have expected22:32
ntpttr___DuncanT: it's customized in that it's multinode - I have swift running on the controller and c-bak on another node22:33
*** xyang1 has quit IRC22:36
*** xinli has quit IRC22:36
ntpttr___here are my local.confs, it's possible something is wrong in there http://paste.openstack.org/show/562765/ http://paste.openstack.org/show/562766/22:36
*** sdake has quit IRC22:44
*** sdake has joined #openstack-cinder22:44
*** jungleboyj has quit IRC22:53
*** alonma has joined #openstack-cinder22:56
karthikp_ntpttr___: did you try harcoding the service_host in local.conf22:57
karthikp_ntpttr___: maybe it cant get the IP address there22:57
*** leeantho has quit IRC22:58
ntpttr___karthikp_: the service_host is getting populated correctly23:00
ntpttr___those are just ansible variables to get it dynamically, I made sure they were getting the right ones23:00
*** alonma has quit IRC23:01
*** alee has joined #openstack-cinder23:02
*** chlong has quit IRC23:03
*** alyson_ has quit IRC23:03
*** xyang has quit IRC23:14
*** kfarr has quit IRC23:19
*** erhudy has quit IRC23:22
*** Julien-zte has joined #openstack-cinder23:22
*** bigdogstl has joined #openstack-cinder23:25
*** bigdogstl has quit IRC23:29
*** salv-orlando has quit IRC23:36
*** bigdogstl has joined #openstack-cinder23:45
*** Julien-zte has quit IRC23:47
*** bigdogstl has quit IRC23:50
*** ducttape_ has quit IRC23:57
*** ducttape_ has joined #openstack-cinder23:58

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