Monday, 2016-08-22

*** rkrum has joined #openstack-cinder00:00
*** yangyapeng has quit IRC00:06
openstackgerritzhangyanxian proposed openstack/cinder: Fix typo in the huawei_driver.py  https://review.openstack.org/34027600:21
*** ducttape_ has quit IRC00:25
*** ducttape_ has joined #openstack-cinder00:31
*** ducttape_ has quit IRC00:31
*** ducttape_ has joined #openstack-cinder00:32
*** ducttape_ has quit IRC00:40
*** ducttape_ has joined #openstack-cinder00:41
*** ducttape_ has quit IRC00:45
*** wanghao_ has joined #openstack-cinder00:49
*** laughterwym has joined #openstack-cinder00:50
*** laughterwym has quit IRC00:51
*** laughterwym has joined #openstack-cinder00:51
*** laughter_ has joined #openstack-cinder00:51
*** ebalduf has quit IRC00:55
*** laughterwym has quit IRC00:55
*** markvoelker has joined #openstack-cinder00:59
*** cknight1 has joined #openstack-cinder01:03
*** cknight has quit IRC01:04
*** markvoelker has quit IRC01:04
*** salv-orlando has joined #openstack-cinder01:06
*** cknight1 has quit IRC01:06
*** cknight has joined #openstack-cinder01:06
*** salv-orlando has quit IRC01:08
*** jplanow has quit IRC01:11
*** cknight has quit IRC01:17
*** cknight has joined #openstack-cinder01:17
*** yangyapeng has joined #openstack-cinder01:17
*** cknight1 has joined #openstack-cinder01:19
*** cknight has quit IRC01:19
openstackgerritwanghao proposed openstack/cinder-specs: Report backend state in service list  https://review.openstack.org/25292101:20
*** lixiaoy1 has joined #openstack-cinder01:21
*** lixiaoy1 has quit IRC01:22
*** lixiaoy1 has joined #openstack-cinder01:22
*** cknight1 has quit IRC01:24
*** cknight has joined #openstack-cinder01:24
*** cknight1 has joined #openstack-cinder01:26
*** cknight has quit IRC01:26
*** cknight has joined #openstack-cinder01:30
*** cknight1 has quit IRC01:30
*** EinstCrazy has joined #openstack-cinder01:34
*** cknight has quit IRC01:34
*** cknight has joined #openstack-cinder01:34
*** EinstCrazy has quit IRC01:34
*** EinstCrazy has joined #openstack-cinder01:34
*** gouthamr has quit IRC01:35
*** ducttape_ has joined #openstack-cinder01:35
*** cknight1 has joined #openstack-cinder01:39
*** cknight has quit IRC01:39
*** cknight has joined #openstack-cinder01:45
*** cknight1 has quit IRC01:45
*** cknight1 has joined #openstack-cinder01:49
*** cknight has quit IRC01:49
*** cknight1 has quit IRC01:53
*** cknight has joined #openstack-cinder01:53
*** cknight has quit IRC01:57
*** cknight has joined #openstack-cinder01:57
*** cknight1 has joined #openstack-cinder01:59
*** cknight has quit IRC01:59
*** cknight has joined #openstack-cinder02:01
*** cknight1 has quit IRC02:01
*** alonma has joined #openstack-cinder02:01
*** alonma has quit IRC02:05
*** EinstCra_ has joined #openstack-cinder02:06
*** cknight1 has joined #openstack-cinder02:08
*** cknight has quit IRC02:08
*** EinstCrazy has quit IRC02:09
openstackgerritxianming.mao proposed openstack/cinder: Drop unused logging import  https://review.openstack.org/35836602:09
*** cknight has joined #openstack-cinder02:12
*** cknight1 has quit IRC02:12
*** yangyapeng has quit IRC02:13
*** EinstCrazy has joined #openstack-cinder02:13
*** yangyapeng has joined #openstack-cinder02:14
*** cknight1 has joined #openstack-cinder02:15
*** yangyape_ has joined #openstack-cinder02:15
*** yangyapeng has quit IRC02:15
*** cknight has quit IRC02:15
*** salv-orlando has joined #openstack-cinder02:16
*** EinstCra_ has quit IRC02:17
*** ebalduf has joined #openstack-cinder02:19
*** salv-orlando has quit IRC02:19
*** yangyape_ has quit IRC02:20
*** cknight1 has quit IRC02:27
openstackgerritSoffie Huang proposed openstack/cinder: FalconStor: Restore option "san_secondary_ip" and drop the "copy_image_to_volume" feature  https://review.openstack.org/35583602:29
openstackgerritLisaLi proposed openstack/cinder: Attachment lost after migrating in-use volume  https://review.openstack.org/35699202:29
*** ducttape_ has quit IRC02:32
*** yangyapeng has joined #openstack-cinder02:39
*** GB21 has joined #openstack-cinder02:39
*** bardia has joined #openstack-cinder02:41
*** ducttape_ has joined #openstack-cinder02:45
*** GB21 has quit IRC02:47
*** Lee1092 has joined #openstack-cinder02:52
*** laughter_ has quit IRC02:53
*** laughterwym has joined #openstack-cinder02:54
*** gouthamr has joined #openstack-cinder03:07
*** krtaylor has quit IRC03:10
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox from image/test_glance.py  https://review.openstack.org/35572103:22
*** chrichip has quit IRC03:23
*** chrichip has joined #openstack-cinder03:23
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_qos_specs.py  https://review.openstack.org/35576603:24
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_quota.py  https://review.openstack.org/35617403:26
*** salv-orlando has joined #openstack-cinder03:26
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v1/ tests  https://review.openstack.org/35618503:27
*** julim has quit IRC03:27
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v2/ tests  https://review.openstack.org/35634703:28
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v3/ tests  https://review.openstack.org/35684103:28
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/contrib/ tests (1)  https://review.openstack.org/35685603:29
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/contrib/ tests (end)  https://review.openstack.org/35685903:30
*** ducttape_ has quit IRC03:30
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/*.py tests  https://review.openstack.org/35764803:30
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/volume/drivers/*.py  https://review.openstack.org/35765203:31
*** sdague has quit IRC03:32
*** salv-orlando has quit IRC03:33
*** yangyapeng has quit IRC03:36
openstackgerritDao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_volume.py  https://review.openstack.org/35581503:44
*** yangyapeng has joined #openstack-cinder03:53
*** aagrawal has joined #openstack-cinder04:01
*** laughterwym has quit IRC04:06
*** laughterwym has joined #openstack-cinder04:07
*** laughterwym has quit IRC04:08
*** laughterwym has joined #openstack-cinder04:08
*** alonma has joined #openstack-cinder04:09
*** gouthamr has quit IRC04:11
*** alonma has quit IRC04:14
*** laughterwym has quit IRC04:16
*** laughterwym has joined #openstack-cinder04:18
*** bardia has quit IRC04:20
*** lixiaoy1 has quit IRC04:22
*** bardia has joined #openstack-cinder04:23
*** laughterwym has quit IRC04:23
*** alonma has joined #openstack-cinder04:23
*** laughterwym has joined #openstack-cinder04:23
*** laughterwym has quit IRC04:25
*** alonma has quit IRC04:28
*** alonma has joined #openstack-cinder04:32
*** bardia has quit IRC04:32
*** laughterwym has joined #openstack-cinder04:32
*** laughterwym has quit IRC04:32
*** laughterwym has joined #openstack-cinder04:33
*** bardia has joined #openstack-cinder04:33
*** alonma has quit IRC04:36
*** salv-orlando has joined #openstack-cinder04:36
*** david-lyle has quit IRC04:37
*** ManishD has joined #openstack-cinder04:38
ManishDHI, I see Xen CI failure for various patches on review board with the following error: tempest.exceptions.TimeoutException: Request timed out04:39
*** takedakn has joined #openstack-cinder04:39
ManishDAppears issue with Xen infra, can someone help or take care of it04:39
ManishDIts afecting many patches04:39
*** takedakn has quit IRC04:43
*** salv-orlando has quit IRC04:43
*** krtaylor has joined #openstack-cinder04:50
*** Poornima has joined #openstack-cinder04:51
*** pbandark has joined #openstack-cinder04:53
*** yangyape_ has joined #openstack-cinder04:54
*** yangyapeng has quit IRC04:54
*** pcaruana has quit IRC04:55
*** laughterwym has quit IRC04:57
*** laughterwym has joined #openstack-cinder04:57
*** zhangguoqing has joined #openstack-cinder04:59
*** pgadiya has joined #openstack-cinder04:59
*** shausy has joined #openstack-cinder05:01
*** ducttape_ has joined #openstack-cinder05:01
*** ducttape_ has quit IRC05:06
*** pdeore has joined #openstack-cinder05:07
*** bardia has quit IRC05:09
*** lkuchlan has joined #openstack-cinder05:14
*** jamielennox is now known as jamielennox|away05:18
*** lixiaoy1 has joined #openstack-cinder05:20
*** aswadr_ has joined #openstack-cinder05:22
*** bardia has joined #openstack-cinder05:27
*** bardia has quit IRC05:32
*** links has joined #openstack-cinder05:38
*** links has quit IRC05:38
*** alonma has joined #openstack-cinder05:40
*** alonma has quit IRC05:44
*** salv-orlando has joined #openstack-cinder05:46
*** bardia has joined #openstack-cinder05:46
*** alonma has joined #openstack-cinder05:47
*** aagrawal has quit IRC05:50
*** alonma has quit IRC05:52
*** salv-orlando has quit IRC05:54
*** alonma has joined #openstack-cinder05:57
*** Wenjun has quit IRC05:57
*** vladani_ has joined #openstack-cinder06:01
*** alonma has quit IRC06:01
*** ducttape_ has joined #openstack-cinder06:02
*** Dinesh_Bhor has joined #openstack-cinder06:03
*** vladani has quit IRC06:05
*** yangyape_ has quit IRC06:05
*** yangyapeng has joined #openstack-cinder06:06
*** yangyapeng has quit IRC06:06
*** alonma has joined #openstack-cinder06:06
*** yangyapeng has joined #openstack-cinder06:06
*** yangyapeng has quit IRC06:06
*** yangyapeng has joined #openstack-cinder06:07
*** salv-orlando has joined #openstack-cinder06:07
*** yangyapeng has quit IRC06:07
*** yangyape_ has joined #openstack-cinder06:07
*** ducttape_ has quit IRC06:08
*** yangyapeng has joined #openstack-cinder06:08
*** alonma has quit IRC06:11
*** yangyape_ has quit IRC06:12
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/35762306:13
*** ebalduf has quit IRC06:15
*** _coolsvap_ has joined #openstack-cinder06:15
*** _coolsvap_ is now known as coolsvap06:15
*** coolsvap has quit IRC06:16
*** coolsvap has joined #openstack-cinder06:16
*** alonma has joined #openstack-cinder06:17
*** mjura has joined #openstack-cinder06:17
*** mpjetta has quit IRC06:18
*** alonma has quit IRC06:19
*** nkrinner_afk is now known as nkrinner06:19
*** mpjetta has joined #openstack-cinder06:20
*** mjura has quit IRC06:22
*** mjura has joined #openstack-cinder06:23
*** EinstCrazy has quit IRC06:23
*** yangyapeng has quit IRC06:23
*** alonma has joined #openstack-cinder06:23
*** lkuchlan has quit IRC06:23
*** yangyapeng has joined #openstack-cinder06:24
*** EinstCrazy has joined #openstack-cinder06:24
openstackgerritavnish proposed openstack/cinder: Use upper constraints for all jobs in tox.ini  https://review.openstack.org/35841806:26
*** alonma has quit IRC06:28
openstackgerritvenkata krishna proposed openstack/cinder: Add replication failback in Kaminario K2 drivers  https://review.openstack.org/35647506:30
*** wxy__ has quit IRC06:30
*** alonma has joined #openstack-cinder06:33
*** aagrawal has joined #openstack-cinder06:37
*** smile-luobin has joined #openstack-cinder06:37
*** alonma has quit IRC06:37
*** alonma has joined #openstack-cinder06:40
*** pbandark has quit IRC06:41
*** Apoorva has joined #openstack-cinder06:41
*** pbandark has joined #openstack-cinder06:43
*** bardia has quit IRC06:43
*** aagrawal has quit IRC06:43
*** aagrawal has joined #openstack-cinder06:45
*** alonma has quit IRC06:45
*** yangyape_ has joined #openstack-cinder06:48
*** yangyapeng has quit IRC06:48
*** abhinavtechie has joined #openstack-cinder06:48
*** aagrawal has quit IRC06:49
*** yangyapeng has joined #openstack-cinder06:49
*** alonma has joined #openstack-cinder06:51
*** rcernin has joined #openstack-cinder06:51
*** pcaruana has joined #openstack-cinder06:52
*** Apoorva has quit IRC06:52
*** yangyape_ has quit IRC06:53
*** lkuchlan has joined #openstack-cinder06:54
*** alonma has quit IRC06:55
*** wxy__ has joined #openstack-cinder06:58
*** abhinavtechie has quit IRC06:58
*** alonma has joined #openstack-cinder06:58
*** alonma has quit IRC07:00
*** alonma has joined #openstack-cinder07:00
*** abhinavtechie has joined #openstack-cinder07:01
openstackgerritMerged openstack/os-brick: Fixes with customized ceph cluster name  https://review.openstack.org/35100507:02
*** ducttape_ has joined #openstack-cinder07:04
*** ManishD has quit IRC07:05
*** arecknag has joined #openstack-cinder07:05
*** ducttape_ has quit IRC07:08
*** tesseract- has joined #openstack-cinder07:09
*** salv-orl_ has joined #openstack-cinder07:15
*** salv-orlando has quit IRC07:18
*** mkoderer__ has quit IRC07:19
*** alonma has quit IRC07:23
*** rcernin has quit IRC07:25
flip214currently pylint gives me big amounts of "Too many positional arguments for function call" ... for things like "self.function(first_any_only_arg)" etc.07:28
*** alonma has joined #openstack-cinder07:30
*** alonma has quit IRC07:31
*** alonma has joined #openstack-cinder07:32
*** salv-orl_ has quit IRC07:32
*** rkrum has quit IRC07:34
*** laughterwym has quit IRC07:58
*** laughterwym has joined #openstack-cinder07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:01
*** ducttape_ has joined #openstack-cinder08:05
*** markvoelker has joined #openstack-cinder08:05
*** laughterwym has quit IRC08:06
openstackgerritGuy Kim proposed openstack/cinder: ITRI DISCO cinder volume driver  https://review.openstack.org/34710008:06
*** laughterwym has joined #openstack-cinder08:06
*** laughterwym has quit IRC08:07
*** laughterwym has joined #openstack-cinder08:07
*** xiexs has quit IRC08:08
*** ducttape_ has quit IRC08:10
*** markvoelker has quit IRC08:10
*** salv-orlando has joined #openstack-cinder08:14
*** laughterwym has quit IRC08:16
*** laughterwym has joined #openstack-cinder08:17
*** laughterwym has quit IRC08:18
*** laughter_ has joined #openstack-cinder08:18
*** alonma has quit IRC08:18
*** coolsvap is now known as _coolsvap_08:18
*** alonma has joined #openstack-cinder08:18
openstackgerritGuy Kim proposed openstack/cinder: ITRI DISCO cinder backup driver  https://review.openstack.org/34931808:19
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: NexentaStor4 NFS driver cache capacity and use sessions  https://review.openstack.org/35614208:24
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Added uses of sessions for NexentaEdge drivers  https://review.openstack.org/35847408:35
openstackgerritEvgeny Antyshev proposed openstack/cinder: Unify ploop format names in Openstack projects  https://review.openstack.org/35767908:37
openstackgerritEvgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots  https://review.openstack.org/27646508:37
*** laughter_ has quit IRC08:44
*** laughterwym has joined #openstack-cinder08:45
*** laughter_ has joined #openstack-cinder08:46
openstackgerritNam Nguyen Hoai proposed openstack/cinder: Clean import in code  https://review.openstack.org/35848008:48
*** laughterwym has quit IRC08:49
*** laughter_ has quit IRC08:51
*** lixiaoy1 has quit IRC08:52
*** pgadiya has quit IRC08:52
openstackgerritTuan proposed openstack/cinder: Remove self.__dict__ for formatting strings  https://review.openstack.org/35849109:05
*** aarefiev_ is now known as aarefiev09:06
*** pgadiya has joined #openstack-cinder09:06
*** ducttape_ has joined #openstack-cinder09:06
*** markvoelker has joined #openstack-cinder09:06
openstackgerritMikhail Feoktistov proposed openstack/cinder: Initialize LVM volumes via ploop  https://review.openstack.org/35798909:06
openstackgerritMikhail Feoktistov proposed openstack/cinder: Initialize LVM volumes via ploop  https://review.openstack.org/35798909:08
geguileoDuncanT: ping - https://review.openstack.org/#/c/351005/709:10
*** markvoelker has quit IRC09:11
*** ducttape_ has quit IRC09:11
openstackgerritSzymon Borkowski proposed openstack/cinder: Convert backup_device to OVO  https://review.openstack.org/32933209:18
*** pgadiya has quit IRC09:18
winston-dgeguileo: not sure DuncanT is around right now, but I am. :)09:18
*** pgadiya has joined #openstack-cinder09:18
DuncanTHi, sorry, I'm on a call right now, I should be clear in 5 minutes09:18
geguileowinston-d: Hi, he told me to ping him  O:-)09:19
geguileoDuncanT: Ok, no hurry09:19
DuncanTThanks09:19
*** sheel has joined #openstack-cinder09:22
sheeldulek: hi09:27
duleksheel: Hello, what's up?09:28
sheeldulek: all good.. thnx09:28
* dulek just returned from vacations by the way.09:28
sheeldulek: you would be looking at one of your patch "https://review.openstack.org/#/c/330391/".. I rebased one other patch but this patch is getting rebased automatically09:29
sheeldulek: during git review below message is coming:09:29
sheelThe outstanding commits are:09:29
sheel6b19c4f (HEAD, message1) Merge commit 'refs/changes/40/313140/14' of git://git.openstack.org/openstack/cinder into message109:29
sheel41a901d Add prefix to user message event ids09:29
sheelded43ab Add generic code for online data migrations09:29
sheeldulek: should i go ahead, or something to look for09:29
sheel?09:30
sheeldulek: please take your time, you can reply whenever feel comfortable... take your time09:30
DuncanTgeguileo: Sorry, I'm here now09:30
geguileoDuncanT: np09:30
DuncanTgeguileo: You've objections to the approach?09:30
geguileoDuncanT: We are neither fixing it as we should (we all agree this is not possible right now)09:30
duleksheel: The merge on the top seems a little strange, but go on, I'll fix the patch if it will break.09:31
geguileoDuncanT: Nor are we following Ceph's standard behavior09:31
sheeldulek: thank you09:31
duleksheel: I need to rebase and take care of my patches anyway. :)09:31
DuncanTgeguileo: What do you consider ceph's standard behaviour?09:31
sheeldulek: :)09:31
geguileoDuncanT: Which is having the config in /etc/ceph/cluster_name.conf09:31
winston-dgeguileo: by 'ceph's standard behavior' you are refering to?09:31
geguileoConfig and keys are by default stored in /etc/ceph09:32
geguileoAnd they use cluster_name.conf for the config09:32
DuncanTgeguileo: But that means having to copy the config files about... which several people objected to as tighter coupling09:32
geguileoAnd cluster_name.user_name.keyring for the key09:32
geguileoDuncanT: But we already have tight coupling if we have to copy the keys09:32
DuncanTgeguileo: Yes, and we're planning on fixing that too, but there's no need to do it all in one patch09:33
*** alonma has quit IRC09:33
DuncanTgeguileo: As mentioned in the commit message, and the associated bug09:33
geguileoWhat I mean is that if we are going to be coupled, at least do it in a consistent way with Ceph09:33
winston-dto be fair, it's up to deployer to place the configure file and key, the only default is /etc/ceph/ceph.conf09:34
DuncanTgeguileo: We're partially decoupled right now. One more patch and we'll be decoupled09:34
DuncanTgeguileo: If we added the keyring passing round to this patch, would you be happier?09:34
geguileoDuncanT: You mean passing the key as well?09:35
DuncanTgeguileo: Yes09:35
geguileoDuncanT: I probably would have been happier   XD09:35
geguileoDuncanT: But no biggie, it'll come  :-)09:35
winston-dwould it worry you if the keyring is being passed in clear text?09:35
DuncanTgeguileo: So that is coming in a follow-on patch, and I fully support one-thing-per-patch for readability and testability09:36
geguileowinston-d: It would worry me, yes09:36
DuncanTwinston-d: It's only clear text if you've foolishly not configured rabbit fo rTLS09:36
geguileoDuncanT: Yes, that is true09:36
DuncanTwinston-d: and using barbican is only better against passive sniffing of the undercloud network - with any active attach it is worse, since all you need to get the key material out of barbican is a user token09:37
winston-drTLS is not a must in openstack deployment doc and I know lots of ppl deploy openstack without it09:37
DuncanTwinston-d: We should make passing the keys via rabbit optional I think, but there's no magic bullet to making them unreadable09:38
DuncanTwinston-d: If they're in a local file then they are vunerable to one exploit, if they're in the RPC another, and barbican a third09:38
DuncanTwinston-d: Security is hard, and only as good as the weakest link09:39
*** alonma has joined #openstack-cinder09:39
geguileoDuncanT: I think making it optional would be a sensible option09:40
*** lpetrut has joined #openstack-cinder09:41
DuncanTI'm actually wondering if there's anything you might sensibly want to put in /etc/ceph/cluster_name.conf other than the things we're passing over the wire currently?09:41
*** Thelo_ has joined #openstack-cinder09:42
*** Thelo_ has left #openstack-cinder09:43
*** alonma has quit IRC09:44
winston-dDuncanT: security is hard, yes.  making passing keyring optional is the right thing to do.09:44
DuncanTwinston-d: +109:44
*** sborkows has joined #openstack-cinder09:45
winston-dlooks like we are agree what next patch should be. productive conversation. :)09:47
*** sandanar has joined #openstack-cinder09:47
openstackgerritSheel Rana proposed openstack/cinder: Add prefix to user message event ids  https://review.openstack.org/31314009:52
*** e0ne has joined #openstack-cinder09:54
*** laughterwym has joined #openstack-cinder09:57
*** sborkows has quit IRC09:57
*** sborkows has joined #openstack-cinder09:58
*** rkrum has joined #openstack-cinder09:58
*** laughterwym has quit IRC10:01
*** rkrum has quit IRC10:02
*** ducttape_ has joined #openstack-cinder10:07
*** markvoelker has joined #openstack-cinder10:07
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376610:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376610:10
*** alonma has joined #openstack-cinder10:10
*** ducttape_ has quit IRC10:11
*** ManishD has joined #openstack-cinder10:11
*** markvoelker has quit IRC10:12
*** yangyapeng has quit IRC10:12
*** Poornima has quit IRC10:17
*** shausy has quit IRC10:19
*** shausy has joined #openstack-cinder10:21
*** laughterwym has joined #openstack-cinder10:32
swamireddyDuncanT: Hi - Can  you please review https://review.openstack.org/#/c/335067/ (Updated the patch as per the review comments - i.e extracting the volume summary from dB instread of python calls)...10:36
DuncanTswamireddy: Reviewed. Only one issue I can see10:44
*** laughterwym has quit IRC10:45
DuncanTswamireddy: I do wonder if this API is going to back us into a corner when it comes to people wanting a slightly richer summary though - e.g. the info you provide but by volume type10:45
*** laughterwym has joined #openstack-cinder10:45
swamireddyDuncanT: Thanks...will remove  the unused param...10:46
DuncanTswamireddy: Looks like filters= permeates up the call stack10:46
DuncanTswamireddy: Also, what should the admin view of this be? Being able to get the summary for any tenant?10:47
swamireddyDuncanT:  If required, we can add a few more details to the summary...but I prefer summary should be the summary (not much info)...10:47
swamireddyDuncanT: Its same as other APIs...for ex: with --all-tenants - can be all tenants details..10:48
DuncanTswamireddy: Have you tested with admin?10:48
swamireddyyes10:48
DuncanTswamireddy: I think you start needing to implement filters to get that working10:48
swamireddyDuncanT: yes...API call tested with and without --all-tenants10:49
DuncanTswamireddy: Ok, thanks10:49
*** laughterwym has quit IRC10:49
DuncanTswamireddy: I'll have a think about what level of detail we ought to have - total and per volume type seems the most sensible granularity. We can always add the per-type in a later micro-version (and patch), but it would be nice if the json returned by your version was structured in such a way you didn't need to re-write the parser completely when using the10:51
DuncanTlater microversion10:51
*** EinstCrazy has quit IRC10:53
swamireddyDuncanT: yes, I will improve the same in the next version...as we do all in one time...may be confusion...so I prefer to add the current support and next version, surely update with json response,etc with other inputs. Thank you10:53
DuncanTswamireddy: Well, this version returns some json. Can that json be extended to include the per-type summary and still make sense? Or will you end up having to restructure it, not just extend it?10:54
*** sv7577 has joined #openstack-cinder11:00
*** GB21 has joined #openstack-cinder11:02
swamireddyDuncanT: Can you please clarify - is per-type ¨volume_type¨?11:02
DuncanTswamireddy: Sure... on a system with different classes of service for volumes these are represented by types - e.g. gold, silver and bronze11:02
swamireddyDuncanT: by default - volume type is ¨None¨11:02
DuncanTswamireddy: You can set the default type in your cinder.conf11:03
*** openstackgerrit has quit IRC11:03
nikeshmhi how can we see automatic generated bug by DocImpact11:04
DuncanTswamireddy: You only generally have more than one type when you've more than one backend, in my experience, though some arrays can do classes of service on one array via types11:04
swamireddyDuncanT: That can be doable....I can update the sumary with per types.... Please let me know if change reqruied now or can we do it next version?11:04
*** openstackgerrit has joined #openstack-cinder11:04
DuncanTswamireddy: As I said, I just want to check if the json your current version returns can be extended to cover the second case11:05
swamireddyDuncanT: yes.,..agreed...11:05
DuncanTswamireddy: Can you paste up a response to your current API?11:05
DuncanTswamireddy: I've not got a handy devstack to try it on11:06
swamireddyDuncanT: sure...11:06
swamireddyDuncanT: nop...will share the json response...give 1 min...11:06
*** sdague has joined #openstack-cinder11:06
*** ducttape_ has joined #openstack-cinder11:08
*** markvoelker has joined #openstack-cinder11:08
nikeshmhi how can we see automatic generated bug for openstack manual by DocImpact for our cinder patch11:10
*** ducttape_ has quit IRC11:12
*** markvoelker has quit IRC11:12
swamireddyDuncanT: Here is the API call - curl -i http://10.0.2.15:8776/v3/c42144f34d934c1398bf424bab61baeb/volumes/summary -X GET -H "Accept: application/json" -H "Content-Type: application/json" -H "X-Auth-Project-Id: demo" -H "X-Auth-Token:91a188a0f6dd4fd7be205e43f0bfd159"  -H "OpenStack-API-Version: volume 3.12"11:15
swamireddyDuncanT: here is the json response:  {"volume-summary": {"volume_size": 4, "volume_count": 2}}11:16
*** wN has quit IRC11:18
DuncanTso I guess we can (in future) make that {"volume-summary": {"volume_size": "volume_size": 4, "volume_count": 2}, "types":[{"type": "gold", "volume_size": 1, "volume_count": 1}, {"type": "silver", "volume_size": 3, "volume_count": 1}]}11:18
*** wN has joined #openstack-cinder11:18
*** wN has quit IRC11:18
*** wN has joined #openstack-cinder11:18
DuncanTSo you won't need to change parser11:18
DuncanTGreat11:18
*** salv-orlando has quit IRC11:19
DuncanTMy only nit is I wonder if "volume_size" wouldn't be more clear as "total_size" ?11:19
swamireddyDuncanT: yes....no changes for parser side...11:19
DuncanTswamireddy: Thanks for walking through it with me. I'll leave that nit up to you / otehr reviewers.11:20
swamireddyDuncanT: We can extend the summary based on the suers feedback in the next version...so for now, (as a first version) will go with size and count only...Is this fine with you?11:21
*** salv-orlando has joined #openstack-cinder11:21
DuncanTYes, that's fine with me11:21
*** salv-orl_ has joined #openstack-cinder11:22
swamireddyDuncanT: Thank you...will address your comment on unused param and submit the next patch...11:22
DuncanTswamireddy: Great, I'll keep an eye out for it11:22
*** sham has joined #openstack-cinder11:25
shamHI11:25
*** salv-orlando has quit IRC11:25
shamI have an infra of openstack gizzly11:26
shamGot following error on the cinder-api log11:26
shamhttp://paste.openstack.org/show/562186/11:28
*** maestropandy has joined #openstack-cinder11:28
shamAnyone please have a look on those errors11:28
*** sv7577 has quit IRC11:29
*** laughterwym has joined #openstack-cinder11:31
openstackgerritVinita Deshpande proposed openstack/cinder: Bugfix for CoprHD ScaleIO volume driver  https://review.openstack.org/35857011:33
*** pdeore has quit IRC11:34
*** openstackstatus has quit IRC11:36
*** openstackstatus has joined #openstack-cinder11:37
*** ChanServ sets mode: +v openstackstatus11:37
*** ManishD has quit IRC11:38
*** laughterwym has quit IRC11:42
*** alyson_ has joined #openstack-cinder11:45
*** sham has left #openstack-cinder11:51
*** maestropandy has left #openstack-cinder11:53
*** salv-orl_ has quit IRC11:53
*** asselin has joined #openstack-cinder11:53
*** salv-orlando has joined #openstack-cinder11:54
scottdageguileo: I filed a bug for the issue I was having testing HA friday. It had nothing to do with your patches and reproduces on master. https://bugs.launchpad.net/cinder/+bug/161513411:54
openstackLaunchpad bug 1615134 in Cinder "volume create fails in /usr/sbin/thin_check" [Undecided,New]11:54
geguileoscottda: I saw it this morning, and was relieved to see it had nothing to do with my work  :-)11:55
*** asselin_ has quit IRC11:56
openstackgerritVinita Deshpande proposed openstack/cinder: Bugfix for CoprHD Volume driver  https://review.openstack.org/35858111:57
*** e0ne has quit IRC11:59
*** e0ne has joined #openstack-cinder12:00
*** maestropandy1 has joined #openstack-cinder12:03
*** maestropandy1 has left #openstack-cinder12:04
*** laughterwym has joined #openstack-cinder12:05
*** takedakn has joined #openstack-cinder12:06
scottdageguileo: yes, you are " off the hook". You had said you cannot repro on RedHat os? Could you put that in the bug report?12:06
*** ducttape_ has joined #openstack-cinder12:08
*** markvoelker has joined #openstack-cinder12:09
*** wanghao_ has quit IRC12:09
geguileoscottda: Done12:09
*** ducttape_ has quit IRC12:12
*** ducttape_ has joined #openstack-cinder12:13
*** markvoelker has quit IRC12:13
nikeshmsham:  hi12:18
*** laughterwym has quit IRC12:19
nikeshmsham: rpc_response_timeout=some_value in [DEFAULT] section of cinder.conf on cinder-api node12:19
*** markvoelker has joined #openstack-cinder12:20
*** laughterwym has joined #openstack-cinder12:20
nikeshmsham: and restart cinder-api service, try out this12:20
*** e0ne has quit IRC12:21
*** e0ne has joined #openstack-cinder12:21
*** laughterwym has quit IRC12:25
*** zhangguoqing has quit IRC12:25
*** xyang1 has joined #openstack-cinder12:26
*** tpsilva has joined #openstack-cinder12:26
*** edmondsw has joined #openstack-cinder12:30
*** e0ne has quit IRC12:31
*** e0ne has joined #openstack-cinder12:32
*** xyang has joined #openstack-cinder12:35
*** pgadiya has quit IRC12:36
*** gouthamr has joined #openstack-cinder12:37
*** yangyapeng has joined #openstack-cinder12:38
*** sandanar has quit IRC12:42
*** GB21 has quit IRC12:43
*** laughterwym has joined #openstack-cinder12:43
*** xyang has quit IRC12:44
*** xyang has joined #openstack-cinder12:46
*** ducttape_ has quit IRC12:46
nikeshmxyang: hi12:47
nikeshmaddress yur reviews on https://review.openstack.org/#/c/35647512:47
nikeshmCI and jenkins are passed12:47
nikeshmif you get time please have a look12:48
*** salv-orlando has quit IRC12:50
*** abhinavtechie has quit IRC12:51
*** e0ne has quit IRC12:54
*** e0ne has joined #openstack-cinder12:55
dulekkarthikp: Thanks for making multinode grenade job happen!12:55
dulekscottda: What's happening next with multinode gate testing?12:57
dulekscottda: Multinode grenade job needs to wait for https://review.openstack.org/#/c/355678 and then we can try to move it to check queue as non-voting job.12:58
*** clenimar has joined #openstack-cinder12:58
*** mgagne_ has quit IRC13:00
*** mgagne_ has joined #openstack-cinder13:00
scottdadulek: I don't think we've any patches up yet for running an experimental multi-node job. I think that will be next, but we'll also need some tests that use multi-node13:01
*** enriquetaso has joined #openstack-cinder13:03
*** gouthamr is now known as gouthamr|afk13:04
dulekscottda: Like migration and retype with on-demand migration?13:05
xyangnikeshm: ok13:06
scottdadulek: Yes, that's a good candidate. I'm still trying to get the single-node version merged13:06
*** rlrossit has joined #openstack-cinder13:06
*** edmondsw has quit IRC13:09
*** ducttape_ has joined #openstack-cinder13:15
openstackgerritMichal Dulko proposed openstack/cinder: Pin RPC server's serializer to min obj version  https://review.openstack.org/35053413:17
openstackgerritMichal Dulko proposed openstack/cinder: Cleanup upgrade-related FIXME's  https://review.openstack.org/35053313:17
*** julim has joined #openstack-cinder13:18
*** superdan is now known as dansmith13:21
*** porrua has joined #openstack-cinder13:21
*** ducttape_ has quit IRC13:26
*** sdague has quit IRC13:32
*** yuriy_n17__ has joined #openstack-cinder13:33
*** e0ne_ has joined #openstack-cinder13:34
*** aarefiev_ has joined #openstack-cinder13:34
*** melwitt_ is now known as melwitt13:34
*** akerr has joined #openstack-cinder13:35
*** aarefiev has quit IRC13:36
*** e0ne has quit IRC13:36
*** yuriy_n17_ has quit IRC13:37
*** ducttape_ has joined #openstack-cinder13:38
*** jungleboyj has joined #openstack-cinder13:41
*** ducttape_ has quit IRC13:42
*** dmsimard has joined #openstack-cinder13:46
*** lixiaoy1 has joined #openstack-cinder13:46
*** sdake has joined #openstack-cinder13:46
dmsimardHello #openstack-cinder o/ RDO CI picked up failures after a recent merge https://review.openstack.org/#/c/251503/ .. it looks like encrypted volumes are now broken when used with barbican. We reported the issue in https://bugs.launchpad.net/cinder/+bug/1615059 .13:47
openstackLaunchpad bug 1615059 in Cinder "Cinder fails to create a crypted volume when barbican is used as key manager" [Undecided,New]13:47
dmsimardIs it possible to have some eyes on the problem ? It effectively breaks trunk testing for upstream puppet-openstack reviews.13:48
*** bigdogstl has joined #openstack-cinder13:48
dmsimardhttp://logs.openstack.org/45/357645/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3273ca7/logs/cinder/cinder-api.txt.gz#_2016-08-22_08_01_11_49213:49
*** avishay has joined #openstack-cinder13:49
*** sdake_ has joined #openstack-cinder13:50
*** briancurtin has quit IRC13:51
openstackgerritJon Bernard proposed openstack/cinder: RBD: Implement v2.1 replication  https://review.openstack.org/33356513:51
*** briancurtin has joined #openstack-cinder13:52
*** bigdogstl has quit IRC13:53
*** sdake has quit IRC13:53
lixiaoy1dmsimard: I am going to look at the issue13:54
dmsimardlixiaoy1: thank you, please let me know if you have any questions13:55
*** Julien-zte has joined #openstack-cinder13:55
*** pschaef has joined #openstack-cinder13:55
*** pschaef has quit IRC13:56
*** shausy has quit IRC13:57
*** rlrossit_ has joined #openstack-cinder13:58
lixiaoy1dmsimard: the parameter is wrong, it should be length=length14:01
*** rlrossit has quit IRC14:02
dmsimardI guess CI didn't pick it up because it's not a tested scenario or something like that ?14:02
lixiaoy1dmsimard: I think CI doesn't test barbican14:03
dmsimardlixiaoy1: are you taking care of submitting the fix, then ?14:04
lixiaoy1dmsimard: do you have Barbican environment? I can submit a fix, could you kindly have a test?14:04
dmsimardlixiaoy1: we picked up the issue through puppet-openstack-integration: https://github.com/openstack/puppet-openstack-integration#description14:05
*** leeantho has joined #openstack-cinder14:05
dmsimardlixiaoy1: since puppet doesn't install from source, I can test with a package built with your patch14:05
*** Julien-zte has quit IRC14:06
*** JoseMello has joined #openstack-cinder14:06
lixiaoy1dmsimard: thank you. I will submit a patch soon14:07
*** edmondsw has joined #openstack-cinder14:07
*** lprice1 has joined #openstack-cinder14:10
*** ducttape_ has joined #openstack-cinder14:11
*** haplo37__ has joined #openstack-cinder14:11
*** lprice has quit IRC14:11
*** ducttape_ has quit IRC14:11
*** ducttape_ has joined #openstack-cinder14:12
*** takedakn has quit IRC14:14
openstackgerritLisaLi proposed openstack/cinder: Fix incorrect parameter in create_key  https://review.openstack.org/35867014:15
openstackgerritSatish Venkatasubramanian proposed openstack/cinder: Add volumes/summary API support  https://review.openstack.org/33506714:16
*** sdake_ is now known as sdake14:18
lixiaoy1dmsimard: the patch is here: https://review.openstack.org/#/c/358670/114:18
lixiaoy1jgriffith: ping14:18
*** mtanino has joined #openstack-cinder14:19
dmsimardlixiaoy1: ok, will try it, thanks.14:19
*** lprice1 has quit IRC14:19
*** pbandark has quit IRC14:21
*** xinli has joined #openstack-cinder14:25
*** sheel has quit IRC14:26
*** ducttape_ has quit IRC14:26
*** avishay has quit IRC14:27
*** ducttape_ has joined #openstack-cinder14:28
*** bigdogstl has joined #openstack-cinder14:33
*** lprice has joined #openstack-cinder14:36
*** cdelatte has joined #openstack-cinder14:36
*** alonma has quit IRC14:43
*** bigdogstl has quit IRC14:43
*** eharney has joined #openstack-cinder14:49
*** eharney has quit IRC14:49
*** alonma has joined #openstack-cinder14:50
*** cknight has joined #openstack-cinder14:51
*** lkuchlan has quit IRC14:51
*** nkrinner is now known as nkrinner_afk14:52
*** salv-orlando has joined #openstack-cinder14:54
openstackgerritEvgeny Antyshev proposed openstack/cinder: Unify ploop format names in Openstack projects  https://review.openstack.org/35767914:54
openstackgerritEvgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots  https://review.openstack.org/27646514:54
openstackgerritMerged openstack/cinder: Reduce the runtime of drivers/dell/test_dellsc  https://review.openstack.org/35774814:55
*** xinli has quit IRC14:55
*** alonma has quit IRC14:55
*** xinli has joined #openstack-cinder14:56
*** salv-orlando has quit IRC14:57
*** _ducttape_ has joined #openstack-cinder14:58
*** sborkows has quit IRC14:59
*** ducttape_ has quit IRC15:02
*** eharney has joined #openstack-cinder15:03
openstackgerritLisaLi proposed openstack/cinder: Fix incorrect parameter in create_key  https://review.openstack.org/35867015:03
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077815:06
dmsimardlixiaoy1: doing some pre-requisite work to test your patch and then I'll test it15:08
*** jdurgin1 has joined #openstack-cinder15:08
lixiaoy1dmsimard: ok thanks15:08
*** mjura has quit IRC15:09
*** akerr is now known as akerr_away15:10
lixiaoy1Hi Anyone helps to review this patch https://review.openstack.org/#/c/319078/ ? This is related to retype volumes with different encryptions15:11
*** akerr_away is now known as akerr15:11
*** akerr is now known as akerr_away15:11
*** akerr_away is now known as akerr15:11
*** akerr is now known as akerr_away15:12
*** abhinavtechie has joined #openstack-cinder15:12
*** akerr_away is now known as akerr15:13
*** akerr is now known as akerr_away15:13
*** akerr_away is now known as akerr15:13
*** bigdogstl has joined #openstack-cinder15:14
*** erhudy has joined #openstack-cinder15:19
*** rlrossit has joined #openstack-cinder15:20
*** rlrossit_ has quit IRC15:23
*** _coolsvap_ is now known as coolsvap15:24
openstackgerritJustin A Wilson proposed openstack/cinder: Added config option to enable SSL  https://review.openstack.org/35295815:25
*** lixiaoy1 has quit IRC15:27
*** cdelatte has quit IRC15:30
*** bigdogstl has quit IRC15:32
*** alonma has joined #openstack-cinder15:32
*** bigdogstl has joined #openstack-cinder15:33
openstackgerritMike Rooney proposed openstack/cinder: NetApp: Report multiattach as enabled  https://review.openstack.org/35663515:34
*** Administrator__ has quit IRC15:36
*** Administrator__ has joined #openstack-cinder15:37
*** alonma has quit IRC15:37
openstackgerritCedric Zhuang proposed openstack/cinder: Performance issue for VNX attache LUN  https://review.openstack.org/35872015:38
*** laughterwym has quit IRC15:39
*** harlowja_at_home has joined #openstack-cinder15:40
openstackgerritxing-yang proposed openstack/python-cinderclient: Add generic volume groups  https://review.openstack.org/32262715:41
openstackgerritJay Bryant proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718615:43
*** laughterwym has joined #openstack-cinder15:44
*** thiagolib has joined #openstack-cinder15:45
*** bwallis has quit IRC15:45
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Plugin - Target WWN change for redundancy  https://review.openstack.org/34230015:45
*** _ducttape_ has quit IRC15:48
*** bigdogstl has quit IRC15:48
*** laughterwym has quit IRC15:49
*** jgregor has joined #openstack-cinder15:50
*** laughterwym has joined #openstack-cinder15:50
*** xyang has quit IRC15:51
*** sdake has quit IRC15:54
*** laughterwym has quit IRC15:55
*** sdake has joined #openstack-cinder15:55
*** bigdogstl has joined #openstack-cinder15:56
*** dgonzalez has quit IRC16:03
*** bradjones has quit IRC16:03
openstackgerritxing-yang proposed openstack/python-cinderclient: Add generic volume groups  https://review.openstack.org/32262716:04
*** dgonzalez has joined #openstack-cinder16:05
*** eharney_ has joined #openstack-cinder16:05
*** aorourke has joined #openstack-cinder16:05
*** bradjones has joined #openstack-cinder16:06
*** bradjones has quit IRC16:06
*** bradjones has joined #openstack-cinder16:06
*** e0ne_ has quit IRC16:07
*** ManishD has joined #openstack-cinder16:08
*** pkoniszewski has quit IRC16:08
*** eharney has quit IRC16:08
*** eharney_ is now known as eharney16:09
*** pkoniszewski has joined #openstack-cinder16:10
*** salv-orlando has joined #openstack-cinder16:11
*** bigdogstl has quit IRC16:12
*** alonma has joined #openstack-cinder16:14
*** bigdogstl has joined #openstack-cinder16:14
*** bwallis has joined #openstack-cinder16:16
*** alonma has quit IRC16:18
*** arecknag has quit IRC16:21
*** Thelo_ has joined #openstack-cinder16:24
*** tesseract- has quit IRC16:28
*** xyang has joined #openstack-cinder16:29
*** pcaruana has quit IRC16:29
*** porrua has quit IRC16:33
*** sandanar has joined #openstack-cinder16:33
*** yuriy_n17__ has quit IRC16:34
*** eharney_ has joined #openstack-cinder16:35
*** bigdogstl has quit IRC16:36
*** harlowja_at_home has quit IRC16:37
*** jmccrory_away is now known as jmccrory16:37
*** eharney has quit IRC16:37
*** erlon has joined #openstack-cinder16:41
*** eharney_ is now known as eharney16:44
*** abhinavtechie has quit IRC16:48
*** dims has quit IRC16:54
openstackgerritJay Bryant proposed openstack/cinder: Change NFS driver to not throw exception without nfs shares file  https://review.openstack.org/34128216:55
*** kfarr has joined #openstack-cinder16:55
*** alonma has joined #openstack-cinder16:55
*** dims has joined #openstack-cinder16:56
ManishDHI, I see Xen CI failure for various patches on review board with the following error: tempest.exceptions.TimeoutException: Request timed out16:56
ManishDAppears issue with Xen infra, can someone help or take care of it16:56
ManishDIts afecting many patches16:56
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Failback replications limited to 5 at a time  https://review.openstack.org/35877216:57
Swansonsmcginnis, ^^^ ManishD comment...16:59
*** alonma has quit IRC17:00
*** Thelo_ has quit IRC17:04
*** porrua has joined #openstack-cinder17:07
*** bigdogstl has joined #openstack-cinder17:10
*** bigdogstl has quit IRC17:16
*** jdurgin1 has quit IRC17:17
hemnaManishD, url ?17:20
*** Suyi has joined #openstack-cinder17:20
ManishDhttps://review.openstack.org/#/c/345191/17:21
*** xyang has quit IRC17:25
*** e0ne has joined #openstack-cinder17:26
openstackgerritRaunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage  https://review.openstack.org/34908617:27
*** lpetrut has quit IRC17:28
*** rajinir has joined #openstack-cinder17:28
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Added use of sessions for NexentaEdge drivers  https://review.openstack.org/35847417:29
openstackgerritMerged openstack/cinder: Use constraints for all tox environments  https://review.openstack.org/35564617:30
hemnaManishD, that's a nova patch first off.17:31
hemnaManishD, so any questions to be asked in the nova channel, not cinder17:31
openstackgerritChuck Fouts proposed openstack/cinder: NetApp: Fixes issue with busy snapshot deletion.  https://review.openstack.org/35878917:31
hemnaand ManishD also make sure the issue you are seeing isn't in the list of known rechecks here: http://status.openstack.org/elastic-recheck/17:32
*** ManishD has quit IRC17:33
*** alonma has joined #openstack-cinder17:37
*** Thelo_ has joined #openstack-cinder17:39
*** zul has quit IRC17:40
*** alonma has quit IRC17:41
*** raunak has joined #openstack-cinder17:41
*** sandanar has quit IRC17:42
*** e0ne has quit IRC17:43
*** Apoorva has joined #openstack-cinder17:44
*** e0ne has joined #openstack-cinder17:46
*** zul has joined #openstack-cinder17:48
*** haplo37__ has quit IRC17:49
*** bardia has joined #openstack-cinder17:55
*** e0ne has quit IRC17:57
*** e0ne has joined #openstack-cinder18:14
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add new supported driver checks  https://review.openstack.org/35560818:14
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Add new supported driver checks  https://review.openstack.org/35560818:14
*** alonma has joined #openstack-cinder18:18
*** lcurtis has joined #openstack-cinder18:19
*** alonma has quit IRC18:24
*** e0ne has quit IRC18:25
openstackgerritJon Bernard proposed openstack/os-brick: RBD: Fix typo in rados timeout assignment  https://review.openstack.org/35880618:26
openstackgerritEric Harney proposed openstack/cinder: Tests: Fix racy volume unit test  https://review.openstack.org/35880718:28
*** sdake has quit IRC18:29
*** ducttape_ has joined #openstack-cinder18:29
*** e0ne has joined #openstack-cinder18:30
*** markvoelker has quit IRC18:33
scottdahemna Have you seen anything in os_brick timeout during Nova live migration with encrypted volumes, during lun removal?18:35
scottdahttps://www.irccloud.com/pastebin/QqQmsMSd/18:35
hemnaI haven't really used encrypted vols18:36
scottdahemna: OK, thanks.18:36
scottdahemna: There are obvious problems with this patch, but do you think it's feasible to have a configurable retry on wait_for_volume_removal? https://review.openstack.org/#/c/326294/18:39
scottdaI might have the required bug, and I might need this fix18:40
hemnaah that one18:40
hemnaso18:40
hemnaI had plans on reworking the retry mechanisms in os-brick18:40
hemnato make them more configurable18:40
hemnasince they are currently decorators on methods18:40
*** lpetrut has joined #openstack-cinder18:40
*** e0ne has quit IRC18:40
hemnawhich doesn't really leave room for per request configuration18:40
hemnaI just haven't tackled it yet18:41
scottdahmmmm...Well, I've the above failure in Nova live migration....might be a driver bug, not sure yet. But if it needs more time, we'd want to fix in the retry, I think.18:42
*** rlrossit_ has joined #openstack-cinder18:43
hemnanot really sure why it didn't get removed though18:45
scottdahemna: Yeah, just gathering data at this point. And trying to repro on devstack.18:46
*** rlrossit has quit IRC18:46
*** xyang has joined #openstack-cinder18:48
*** lpetrut has quit IRC18:52
*** ducttap__ has joined #openstack-cinder18:52
*** lcurtis has quit IRC18:52
*** ducttape_ has quit IRC18:53
*** e0ne has joined #openstack-cinder18:56
*** haplo37__ has joined #openstack-cinder18:57
*** alonma has joined #openstack-cinder19:00
*** alonma has quit IRC19:05
*** lkuchlan has joined #openstack-cinder19:07
openstackgerritEric Harney proposed openstack/cinder: Nexenta: Raise NotImplementedError instead of NotImplemented  https://review.openstack.org/35882119:08
*** Thelo__ has joined #openstack-cinder19:08
*** xyang has quit IRC19:09
*** Thelo_ has quit IRC19:10
*** Thelo__ is now known as Thelo_19:10
*** xyang has joined #openstack-cinder19:12
*** salv-orl_ has joined #openstack-cinder19:16
*** raunak has quit IRC19:16
*** e0ne has quit IRC19:18
*** e0ne_ has joined #openstack-cinder19:18
*** salv-orlando has quit IRC19:19
*** haplo37__ has quit IRC19:26
*** salv-orl_ has quit IRC19:29
openstackgerritAlex Meade proposed openstack/python-cinderclient: Add v3 user messages with pagination  https://review.openstack.org/29905219:29
*** david-lyle has joined #openstack-cinder19:30
*** markvoelker has joined #openstack-cinder19:32
*** xyang has quit IRC19:33
*** markvoelker_ has joined #openstack-cinder19:34
*** xyang has joined #openstack-cinder19:35
*** e0ne_ has quit IRC19:35
*** markvoelker has quit IRC19:37
*** rcernin has joined #openstack-cinder19:41
*** alonma has joined #openstack-cinder19:42
*** david-lyle has quit IRC19:42
*** sdake has joined #openstack-cinder19:42
*** lkuchlan has quit IRC19:44
openstackgerritEric Harney proposed openstack/cinder: Group specs: Fix exception name  https://review.openstack.org/35882919:45
*** david-lyle has joined #openstack-cinder19:46
*** alonma has quit IRC19:46
*** markvoelker has joined #openstack-cinder19:48
*** ducttap__ has quit IRC19:49
*** markvoelker_ has quit IRC19:51
*** lpetrut has joined #openstack-cinder19:52
*** ducttape_ has joined #openstack-cinder19:53
*** raunak has joined #openstack-cinder19:59
openstackgerritEric Harney proposed openstack/cinder: Pylint: Print line information  https://review.openstack.org/35883220:01
jungleboyjeharney: I am seeing one of my NFS patches failing because it is using sudo commands.  Any idea what that is?20:04
jungleboyjhttp://logs.openstack.org/82/341282/3/check/gate-cinder-python34-db/a697691/console.html20:04
eharneyjungleboyj: looks like a test isn't mocking out something correctly, so it ends up running "sudo cinder-rootwrap /etc/cinder/rootwrap.conf mount.nfs" for real while running tests20:05
eharneyjungleboyj: this means an execute mock is missing somewhere20:05
*** porrua has quit IRC20:06
jungleboyjeharney: Weird, ok.  I will look closer.20:06
eharneyjungleboyj: it happens on my machine w/ your patchset too -- these things are easy to spot if you have sudo set to require a password :)20:07
jungleboyjeharney: Weird, I didn't see that when running on my system.20:08
eharneyjungleboyj: it probably runs "sudo ... mount.nfs" and succeeds rather than prompting for a password20:08
jungleboyjeharney: Right.  How did you run that check on your system?20:10
jungleboyjeharney: I have an idea what is causing it.20:10
eharneyjungleboyj: well, my fedora machine by default wants a password for sudo, so it just hangs with "enter password" when you run the unit tests20:11
jungleboyjOh.  Ok.20:11
jungleboyjeharney: Yeah, I think I know what it is.20:12
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: NexentaStor4 NFS driver cache capacity and use sessions  https://review.openstack.org/35614220:14
*** markvoelker has quit IRC20:14
eharneyjungleboyj: it's the mount.nfs check in do_setup20:15
jungleboyjeharney: Yep.  That was what I found.  :-)20:15
eharneyjungleboyj: if you cherry-pick in  https://review.openstack.org/#/c/163972/ it will tell you  (sadly i never got that to work well enough to land it)20:16
jungleboyjeharney: Cool.  Thanks.20:16
*** gouthamr|afk is now known as gouthamr20:16
*** sdake has quit IRC20:21
*** alonma has joined #openstack-cinder20:24
*** alonma has quit IRC20:28
*** salv-orlando has joined #openstack-cinder20:29
*** salv-orlando has quit IRC20:34
*** sdake has joined #openstack-cinder20:35
openstackgerritJay Bryant proposed openstack/cinder: Change NFS driver to not throw exception without nfs shares file  https://review.openstack.org/34128220:38
*** david-lyle has quit IRC20:38
jungleboyjeharney: Thanks for the help.  Should be fixed now.  :-)20:38
eharneyjungleboyj: great20:39
jungleboyjeharney: I will learn this unit test thing eventually.20:39
*** david-lyle has joined #openstack-cinder20:39
eharneyjungleboyj: i'll let you know if i ever figure them out20:39
jungleboyj:-)20:40
jungleboyjeharney: Likewise.20:40
*** rlrossit_ has quit IRC20:43
*** david-lyle has quit IRC20:44
*** david-lyle_ has joined #openstack-cinder20:44
*** david-lyle_ has quit IRC20:44
*** rlrossit has joined #openstack-cinder20:46
*** akerr has quit IRC20:47
*** yohoffman has joined #openstack-cinder20:54
*** cknight has quit IRC20:55
yohoffmanHey there, it seems like using the cinder python client to update the volume, snapshot, and gigabyte quota doesn't do anything if set to -1 or 0, but says that it was successful. It fails if it's set to anything less than -1 and says that the minimum is -1. Is this expected behavior?20:56
yohoffman*python cinder client20:57
*** kfarr has quit IRC20:59
*** dgonzalez has quit IRC21:01
*** dgonzalez has joined #openstack-cinder21:03
*** salv-orlando has joined #openstack-cinder21:04
openstackgerritMerged openstack/cinder: VMware: Fix retype when storage policy is enabled  https://review.openstack.org/33767221:05
*** alonma has joined #openstack-cinder21:05
*** julim has quit IRC21:05
yohoffmanShould a quota of -1 or 0 be allowed for gigabytes, snapshots, or volumes?21:08
openstackgerritMerged openstack/cinder: Validate name in qos-spec  https://review.openstack.org/34875621:08
*** alonma has quit IRC21:10
openstackgerritJustin A Wilson proposed openstack/cinder: Added config option to enable SSL  https://review.openstack.org/35295821:10
*** Thelo_ has quit IRC21:11
*** salv-orlando has quit IRC21:19
*** salv-orlando has joined #openstack-cinder21:19
*** david-lyle_ has joined #openstack-cinder21:19
*** salv-orlando has quit IRC21:20
*** salv-orlando has joined #openstack-cinder21:20
*** JoseMello has quit IRC21:20
*** cknight has joined #openstack-cinder21:28
*** rlrossit has quit IRC21:30
*** jgregor has quit IRC21:32
*** david-lyle_ has quit IRC21:34
*** bardia has quit IRC21:37
*** eharney has quit IRC21:37
*** bardia has joined #openstack-cinder21:37
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo  https://review.openstack.org/31904021:39
xyangeharney: ping21:42
*** cknight has quit IRC21:43
*** alonma has joined #openstack-cinder21:47
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo  https://review.openstack.org/31904021:47
*** cknight has joined #openstack-cinder21:48
*** pauloewerton has quit IRC21:49
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo  https://review.openstack.org/31904021:50
*** alonma has quit IRC21:52
*** markvoelker has joined #openstack-cinder21:52
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Add dell_server_os configuration option  https://review.openstack.org/35887822:02
*** lixiaoy1 has joined #openstack-cinder22:02
*** alyson_ has quit IRC22:03
openstackgerritAlexey Khodos proposed openstack/cinder: NexentaStor5: add extend method to NFS driver  https://review.openstack.org/29788022:07
*** alkhodos- has joined #openstack-cinder22:07
*** sdague has joined #openstack-cinder22:08
lixiaoy1jgriffith: here?22:09
*** edmondsw has quit IRC22:10
*** bigdogst_ has joined #openstack-cinder22:10
*** xinli has quit IRC22:10
*** cknight has quit IRC22:11
*** bigdogst_ has quit IRC22:14
*** lpetrut has quit IRC22:14
*** jamielennox|away is now known as jamielennox22:15
lixiaoy1jgriffith: do you still have any concerns about this patch https://review.openstack.org/#/c/358670/ ? how about the new attach flow you work?22:18
*** tpsilva has quit IRC22:18
*** rajinir has quit IRC22:25
*** cknight has joined #openstack-cinder22:27
*** cknight has quit IRC22:28
*** cknight has joined #openstack-cinder22:32
*** bwallis has quit IRC22:33
*** cknight has quit IRC22:35
*** jungleboyj has quit IRC22:35
*** david-lyle_ has joined #openstack-cinder22:37
*** Julien-zte has joined #openstack-cinder22:38
*** markvoelker has quit IRC22:39
*** xyang has quit IRC22:39
*** ducttape_ has quit IRC22:41
*** david-lyle has joined #openstack-cinder22:49
*** xyang has joined #openstack-cinder22:52
*** david-lyle_ has quit IRC22:52
*** dkehn_ has quit IRC22:55
*** xyang has quit IRC22:56
*** bwallis has joined #openstack-cinder22:56
karthikp_dulek: :) .. it was all you... happy to help out23:01
*** tbarron is now known as tbarron|gone23:02
*** david-lyle has quit IRC23:02
*** leeantho has quit IRC23:04
*** gouthamr has quit IRC23:05
*** dkehn_ has joined #openstack-cinder23:08
*** lpetrut has joined #openstack-cinder23:13
*** lixiaoy1 has quit IRC23:14
*** sdake has quit IRC23:16
*** lpetrut has quit IRC23:18
*** sdague has quit IRC23:18
*** xyang1 has quit IRC23:25
*** dkehn_ has quit IRC23:25
*** ducttape_ has joined #openstack-cinder23:26
*** sdake has joined #openstack-cinder23:33
*** Julien-zte has quit IRC23:33
*** dkehn_ has joined #openstack-cinder23:38
*** Lee1092 has quit IRC23:43
*** xyang has joined #openstack-cinder23:45
*** ducttape_ has quit IRC23:50
*** alonma has joined #openstack-cinder23:52
*** sdake has quit IRC23:55
*** alonma has quit IRC23:57
*** mtanin___ has joined #openstack-cinder23:58
*** mtanino has quit IRC23:59

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