Thursday, 2017-04-27

*** diablo_rojo has joined #openstack-cinder00:05
*** Julien-zte has joined #openstack-cinder00:17
*** yongwc has joined #openstack-cinder00:28
*** yongwc has quit IRC00:31
*** yongwc has joined #openstack-cinder00:32
*** dikonoor has joined #openstack-cinder00:32
*** Apoorva has quit IRC00:37
*** markstur has quit IRC00:41
*** markstur has joined #openstack-cinder00:42
*** wangyong__ has joined #openstack-cinder00:44
*** markstur has quit IRC00:46
*** yongwc has quit IRC00:46
*** gouthamr has joined #openstack-cinder00:46
*** wanghao has joined #openstack-cinder00:48
*** Shunli has joined #openstack-cinder00:55
*** sdague has quit IRC00:58
*** imacdonn_ has joined #openstack-cinder00:59
*** wanghao_ has joined #openstack-cinder01:02
*** wanghao has quit IRC01:02
*** gman-tx has quit IRC01:04
*** yangyapeng has joined #openstack-cinder01:18
*** zhenguo has joined #openstack-cinder01:19
*** Apoorva has joined #openstack-cinder01:19
*** yangyapeng has quit IRC01:20
*** yangyape_ has joined #openstack-cinder01:20
*** Apoorva has quit IRC01:20
openstackgerritTommyLike proposed openstack/cinder master: Use cg object when invoking update cg method.  https://review.openstack.org/46036501:30
*** diablo_rojo has quit IRC01:30
*** yangyape_ has quit IRC01:30
*** yangyapeng has joined #openstack-cinder01:31
*** yangyapeng has quit IRC01:35
*** kaisers has quit IRC01:37
*** yangyapeng has joined #openstack-cinder01:37
*** gcb has joined #openstack-cinder01:38
*** gouthamr has quit IRC01:42
*** dikonoor has quit IRC01:42
*** wangyong__ is now known as yongwc01:51
*** chenying__ has joined #openstack-cinder01:55
*** chenying_ has quit IRC01:55
*** zhurong has joined #openstack-cinder02:03
*** rkmrHonjo has joined #openstack-cinder02:03
*** huanxuan has joined #openstack-cinder02:07
rkmrHonjohemna: Hi. My patch https://review.openstack.org/#/c/446401/ got -2 by you. But I'd afraid that you don't understand my report. I replied to you on gerrit. I'd appreciate it if you read it and reply for me.02:07
*** zhugaoxiao has quit IRC02:10
*** wxy has joined #openstack-cinder02:14
*** dhinesh has quit IRC02:17
*** zhurong has quit IRC02:18
openstackgerritTommyLike proposed openstack/cinder master: Refactor 'update_group' method  https://review.openstack.org/46037102:18
*** zul has quit IRC02:19
openstackgerritPeter Wang proposed openstack/cinder master: Add VNX replication group support  https://review.openstack.org/45920502:22
*** mriedem has quit IRC02:34
*** zhangguoqing has joined #openstack-cinder02:34
*** dave-mccowan has quit IRC02:36
*** dave-mccowan has joined #openstack-cinder02:37
*** dave-mccowan has quit IRC02:37
*** zhurong has joined #openstack-cinder02:43
*** dave-mcc_ has joined #openstack-cinder02:44
*** zhangguoqing has quit IRC02:45
*** chlong has joined #openstack-cinder02:48
*** liverpooler has quit IRC02:50
*** gouthamr has joined #openstack-cinder02:51
*** dhinesh has joined #openstack-cinder03:11
*** huanxuan_ has joined #openstack-cinder03:20
*** huanxuan has quit IRC03:23
*** dave-mcc_ has quit IRC03:24
*** nicolasbock has quit IRC03:27
*** psachin has joined #openstack-cinder03:32
*** zhurong has quit IRC03:42
*** dhinesh has quit IRC03:47
*** dikonoor has joined #openstack-cinder03:57
*** gouthamr has quit IRC04:01
*** kaisers has joined #openstack-cinder04:01
*** rarora has quit IRC04:06
*** dikonoor has quit IRC04:08
*** abhitechie has joined #openstack-cinder04:08
*** Poornima has joined #openstack-cinder04:10
*** gkadam has joined #openstack-cinder04:12
*** rarora has joined #openstack-cinder04:19
*** alonma has joined #openstack-cinder04:34
*** chhavi has joined #openstack-cinder04:35
*** markstur has joined #openstack-cinder04:35
*** kaisers has quit IRC04:39
*** Poornima has quit IRC04:42
*** stakeda has quit IRC04:46
*** stakeda has joined #openstack-cinder04:47
*** salv-orlando has joined #openstack-cinder04:51
*** adisky_ has joined #openstack-cinder04:56
*** pgadiya has joined #openstack-cinder04:58
*** dikonoor has joined #openstack-cinder05:05
*** kaisers has joined #openstack-cinder05:09
*** kaisers_ has joined #openstack-cinder05:14
*** kaisers has quit IRC05:15
*** isaacb has joined #openstack-cinder05:18
*** dikonoor has quit IRC05:21
*** alonma has quit IRC05:45
*** alonma has joined #openstack-cinder05:46
*** e0ne has joined #openstack-cinder05:49
*** alonma has quit IRC05:50
*** bkopilov has quit IRC05:52
*** bkopilov has joined #openstack-cinder05:52
*** e0ne has quit IRC05:55
*** dikonoor has joined #openstack-cinder06:05
*** Poornima has joined #openstack-cinder06:11
*** zhurong has joined #openstack-cinder06:13
openstackgerritfupingxie proposed openstack/cinder master: Spelling error "paramenter"  https://review.openstack.org/46040806:13
*** rkmrHonjo has quit IRC06:15
*** dikonoo has joined #openstack-cinder06:16
*** dikonoor has quit IRC06:16
*** markstur has quit IRC06:28
*** ChubYann has quit IRC06:30
*** zhangguoqing has joined #openstack-cinder06:42
*** zhangguoqing has quit IRC06:42
*** zhangguoqing has joined #openstack-cinder06:42
*** alonma has joined #openstack-cinder06:43
*** zhangguoqing has quit IRC06:43
*** zhangguoqing has joined #openstack-cinder06:44
*** pcaruana has joined #openstack-cinder06:44
*** Julien-zte has quit IRC06:45
*** Julien-zte has joined #openstack-cinder06:46
*** salv-orlando has quit IRC06:48
*** Julien-zte has quit IRC06:54
*** Julien-zte has joined #openstack-cinder06:55
*** isaacb has quit IRC06:57
*** alonma has quit IRC06:57
*** alonma has joined #openstack-cinder06:58
*** arne_r has joined #openstack-cinder06:59
*** lkuchlan has joined #openstack-cinder07:00
*** alonma_ has joined #openstack-cinder07:02
*** zhurong has quit IRC07:02
*** alonma has quit IRC07:02
*** zhurong has joined #openstack-cinder07:06
*** salv-orlando has joined #openstack-cinder07:07
*** Nakato has quit IRC07:08
*** Nakato has joined #openstack-cinder07:08
*** tesseract has joined #openstack-cinder07:09
*** lkuchlan has quit IRC07:09
*** Julien-zte has quit IRC07:17
*** Julien-zte has joined #openstack-cinder07:18
*** lkuchlan has joined #openstack-cinder07:23
*** Julien-zte has quit IRC07:28
*** Julien-zte has joined #openstack-cinder07:30
*** Nakato has quit IRC07:30
*** Nakato has joined #openstack-cinder07:31
*** dikonoo has quit IRC07:32
*** zhurong has quit IRC07:37
openstackgerritwzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression  https://review.openstack.org/45029107:41
*** mlakat_ has joined #openstack-cinder07:47
*** mlakat has quit IRC07:47
*** mlakat_ is now known as mlakat07:47
*** e0ne has joined #openstack-cinder07:52
*** alonma_ has quit IRC07:54
*** alonma_ has joined #openstack-cinder07:56
openstackgerritMichal Dulko proposed openstack/cinder master: Read XtremIO options from self.configuration  https://review.openstack.org/45361707:57
openstackgerritMichal Dulko proposed openstack/cinder master: Move driver options to "backend" section  https://review.openstack.org/45365407:57
openstackgerritMichal Dulko proposed openstack/cinder master: Read rbd options from self.configuration, not CONF  https://review.openstack.org/45359807:57
openstackgerritMichal Dulko proposed openstack/cinder master: Remove usage of CONF from DISCO driver  https://review.openstack.org/45816707:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** Julien-zte has quit IRC08:00
*** Julien-zte has joined #openstack-cinder08:01
*** MasterofJOKers has quit IRC08:02
*** MasterofJOKers has joined #openstack-cinder08:03
*** zhurong has joined #openstack-cinder08:19
*** yongwc has quit IRC08:20
*** bkopilov has quit IRC08:23
*** pgadiya has quit IRC08:26
*** yongwc has joined #openstack-cinder08:26
*** Julien-zte has quit IRC08:27
*** Julien-zte has joined #openstack-cinder08:29
*** alonma_ has quit IRC08:31
*** Julien-zte has quit IRC08:31
*** Julien-zte has joined #openstack-cinder08:33
*** alonma has joined #openstack-cinder08:37
*** abhitechie has quit IRC08:44
*** Julien-zte has quit IRC08:45
*** Julien-zte has joined #openstack-cinder08:46
*** e0ne has quit IRC08:48
*** e0ne has joined #openstack-cinder08:49
*** lpetrut has joined #openstack-cinder08:50
*** bkopilov has joined #openstack-cinder08:52
openstackgerritTommyLike proposed openstack/cinder master: Add support for generalized filtering on list APIs  https://review.openstack.org/44459808:53
*** yongwc has quit IRC09:00
*** yongwc has joined #openstack-cinder09:02
*** Julien-zte has quit IRC09:03
openstackgerritNidhiMittalHada proposed openstack/cinder master: validate_integer doesn't check non int properly  https://review.openstack.org/45131609:03
*** Julien-zte has joined #openstack-cinder09:05
*** zhangguoqing has quit IRC09:06
*** yongwc has quit IRC09:07
openstackgerritfupingxie proposed openstack/cinder master: Spelling error "paramenter"  https://review.openstack.org/46040809:08
*** Julien-zte has quit IRC09:09
*** Julien-zte has joined #openstack-cinder09:10
*** Julien-zte has quit IRC09:11
*** Julien-zte has joined #openstack-cinder09:12
openstackgerritwzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression  https://review.openstack.org/45029109:13
openstackgerritwzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression  https://review.openstack.org/45029109:14
*** aarefiev_afk is now known as aarefiev09:16
*** rarora has quit IRC09:18
*** Julien-zte has quit IRC09:25
*** Julien-zte has joined #openstack-cinder09:27
*** e0ne has quit IRC09:30
*** yongwc has joined #openstack-cinder09:33
*** dalgaaf has quit IRC09:34
*** dalgaaf has joined #openstack-cinder09:36
*** e0ne has joined #openstack-cinder09:36
*** alonma has quit IRC09:40
*** alonma has joined #openstack-cinder09:43
*** alonma has quit IRC09:44
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix host check in is_backend_frozen  https://review.openstack.org/44814709:45
*** stakeda has quit IRC09:45
*** zhangguoqing has joined #openstack-cinder09:48
openstackgerritwzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression  https://review.openstack.org/45029109:48
*** salv-orl_ has joined #openstack-cinder09:49
*** salv-orlando has quit IRC09:52
tommylikehudoes anyone would like to take a look at these two simple patches? https://review.openstack.org/#/c/460365/, https://review.openstack.org/#/c/460371/09:55
tommylikehuthanks in advance :)09:55
*** lpetrut_ has joined #openstack-cinder09:58
*** Julien-zte has quit IRC09:59
*** ociuhandu has quit IRC09:59
*** lpetrut has quit IRC10:00
openstackgerritwzyuliyang proposed openstack/cinder master: Add pigz support for cinder-backup chunked driver compression  https://review.openstack.org/45029110:02
*** nicolasbock has joined #openstack-cinder10:03
*** alonma has joined #openstack-cinder10:03
*** mvk has quit IRC10:04
*** alonma_ has joined #openstack-cinder10:05
*** Shunli has quit IRC10:05
*** alonma has quit IRC10:07
*** zul has joined #openstack-cinder10:09
*** zhurong has quit IRC10:20
*** mlakat has quit IRC10:26
*** huanxuan_ has quit IRC10:27
*** zhurong has joined #openstack-cinder10:28
*** yangyape_ has joined #openstack-cinder10:29
*** rarora has joined #openstack-cinder10:29
*** psachin has quit IRC10:30
*** psachin has joined #openstack-cinder10:33
*** mvk has joined #openstack-cinder10:34
openstackgerritNidhiMittalHada proposed openstack/cinder master: Issue in logs while creating cinder volume  https://review.openstack.org/46049610:36
*** yangyape_ has quit IRC10:37
*** salv-orl_ has quit IRC10:38
*** The_Ball has quit IRC10:38
*** The_Ball has joined #openstack-cinder10:38
*** yangyapeng has quit IRC10:38
*** yangyapeng has joined #openstack-cinder10:39
*** yangyapeng has quit IRC10:43
openstackgerritTommyLike proposed openstack/cinder master: Add support for generalized filtering on list APIs  https://review.openstack.org/44459810:48
*** chhavi has quit IRC10:48
*** edmondsw has joined #openstack-cinder10:53
*** alonma_ has quit IRC10:59
*** alonma has joined #openstack-cinder11:02
*** sdague has joined #openstack-cinder11:04
*** dave-mccowan has joined #openstack-cinder11:09
*** markvoelker_ has joined #openstack-cinder11:11
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST  https://review.openstack.org/46019111:11
*** markvoelker has quit IRC11:12
*** lkuchlan has quit IRC11:13
*** lkuchlan has joined #openstack-cinder11:19
*** jordanP has quit IRC11:22
*** alonma has quit IRC11:22
*** alonma has joined #openstack-cinder11:22
*** zhonghua has quit IRC11:23
*** zhonghua has joined #openstack-cinder11:23
*** alonma has quit IRC11:27
*** alonma has joined #openstack-cinder11:27
*** kaisers_ has quit IRC11:29
*** kaisers has joined #openstack-cinder11:30
*** kaisers has quit IRC11:34
*** yangyapeng has joined #openstack-cinder11:36
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/46051211:37
*** markvoelker_ has quit IRC11:38
*** Poornima has quit IRC11:41
*** kaisers has joined #openstack-cinder11:41
*** abishop has joined #openstack-cinder11:45
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/46051611:45
*** chhavi has joined #openstack-cinder11:53
*** dgonzalez has quit IRC12:02
*** dgonzalez has joined #openstack-cinder12:07
*** zhonghua has quit IRC12:09
*** The_Ball has quit IRC12:09
*** zhonghua has joined #openstack-cinder12:10
*** takedakn has joined #openstack-cinder12:15
*** zhurong has quit IRC12:25
*** ociuhandu has joined #openstack-cinder12:25
*** gman-tx has joined #openstack-cinder12:27
*** dillaman has quit IRC12:29
*** dustins has joined #openstack-cinder12:31
openstackgerritTommyLike proposed openstack/cinder master: Add support for generalized filtering on list APIs  https://review.openstack.org/44459812:33
*** markvoelker has joined #openstack-cinder12:35
*** zhangguoqing has quit IRC12:36
*** liverpooler has joined #openstack-cinder12:38
*** crushil has joined #openstack-cinder12:38
*** gkadam has quit IRC12:41
*** dillaman has joined #openstack-cinder12:42
*** mlakat has joined #openstack-cinder12:42
*** jonesn has joined #openstack-cinder12:47
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: remove custom Lock object  https://review.openstack.org/44750312:48
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: use tooz builtin heartbeat feature  https://review.openstack.org/44707912:48
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: remove un-used method is_active()  https://review.openstack.org/46053212:48
*** dillaman has quit IRC12:49
*** jdillaman has quit IRC12:50
*** arne_r has quit IRC12:52
*** zhonghua has quit IRC12:53
*** zhonghua has joined #openstack-cinder12:53
*** zhangguoqing has joined #openstack-cinder12:54
*** zhurong has joined #openstack-cinder12:58
*** zhangguoqing has quit IRC13:01
*** jdillaman has joined #openstack-cinder13:02
*** dillaman has joined #openstack-cinder13:02
*** wanghao has joined #openstack-cinder13:04
*** wanghao has quit IRC13:05
*** zhangguoqing has joined #openstack-cinder13:06
*** mriedem has joined #openstack-cinder13:06
*** wanghao_ has quit IRC13:07
*** salv-orlando has joined #openstack-cinder13:12
openstackgerritwangxiyuan proposed openstack/cinder master: Fix keystone auth for novaclient  https://review.openstack.org/46054113:14
*** catintheroof has joined #openstack-cinder13:17
jgriffithtommylikehu ping13:18
tommylikehuHey jgriffith13:19
*** bswartz has joined #openstack-cinder13:19
jgriffithtommylikehu Hi!13:20
jgriffithtommylikehu Hey… I finally reviewed https://review.openstack.org/#/c/444598/1713:20
jgriffithtommylikehu thank you!!!!13:20
jgriffithThat looks great13:20
tommylikehujgriffith: thanks:)13:21
*** Julien-zte has joined #openstack-cinder13:21
tommylikehustill working on that. And any suggestion?13:21
Dinesh_Bhorjgriffith: hi, have time to take a look at it? https://review.openstack.org/#/c/454562/13:22
*** zengyingzhe has quit IRC13:25
*** Julien-zte has quit IRC13:26
*** zengyingzhe has joined #openstack-cinder13:26
*** Julien-zte has joined #openstack-cinder13:28
*** Julien-zte has quit IRC13:28
*** Julien-zte has joined #openstack-cinder13:29
*** Julien-zte has quit IRC13:30
*** Julien-zte has joined #openstack-cinder13:31
*** alonma has quit IRC13:31
*** alonma has joined #openstack-cinder13:31
*** Julien-zte has quit IRC13:37
openstackgerritEric Harney proposed openstack/cinder master: LIO: Fix terminate_connection AttributeError  https://review.openstack.org/45819213:38
*** eharney has joined #openstack-cinder13:38
*** alonma has quit IRC13:38
*** Julien-zte has joined #openstack-cinder13:38
*** alonma has joined #openstack-cinder13:39
*** alonma has quit IRC13:39
*** alonma has joined #openstack-cinder13:39
*** diablo_rojo has joined #openstack-cinder13:46
*** lamt has joined #openstack-cinder13:49
*** dillaman has quit IRC13:49
*** adisky_ has quit IRC13:49
*** takedakn has quit IRC13:51
*** zhaochao has joined #openstack-cinder13:55
*** psachin has quit IRC13:56
*** xyang has joined #openstack-cinder13:58
*** xyang has quit IRC14:01
*** dillaman has joined #openstack-cinder14:02
jgriffithtommylikehu no I think it looks really good, I’d suggest breaking the different objects into separate commits if you’re going to keep going with it14:07
jgriffithtommylikehu having everything in one patch is going to be hard to get reviewed IMO14:07
*** yongwc has quit IRC14:08
tommylikehuthanks jgriffith , I don't like that either.  I added them all because most of the logic are similiar. if you or anyone else insist on this, I will split it into different ones.14:09
jgriffithtommylikehu yeah, I get it.. the logic is identical in all of them14:09
jgriffithtommylikehu my thought was just get the first iteration with Volume approved and hammered out / merged before propagating to all the others.  In particular never know if somebody might want to change something in how you did it (or propose a change, or find an issue)14:10
jgriffithtommylikehu I certainly don’t have a strong enough opinion one way or the other on it, so no worries from me.  Multiple patches are nice from a review perspective14:11
jgriffithNot so nice from a “get it done” perspective14:11
tommylikehujgriffith: if that way will not add more review's burden for you, I will split it :)14:13
jgriffithtommylikehu don’t do it for me :)14:14
jgriffithWait and see what others think14:14
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: remove custom Lock object  https://review.openstack.org/44750314:14
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: use tooz builtin heartbeat feature  https://review.openstack.org/44707914:14
*** dillaman has quit IRC14:14
tommylikehujgriffith: thanks14:15
*** uberjay has quit IRC14:18
*** uberjay has joined #openstack-cinder14:20
*** felipemonteiro__ has joined #openstack-cinder14:21
*** felipemonteiro_ has joined #openstack-cinder14:23
*** salv-orlando has quit IRC14:23
dr_gogeta86xyang1, hi14:23
dr_gogeta86back on track14:24
dr_gogeta86I'm clean up the whole stack right now14:24
dr_gogeta86things to check while offline ?14:24
hemnamep14:26
*** felipemonteiro__ has quit IRC14:27
*** diablo_rojo has quit IRC14:27
*** yangyape_ has joined #openstack-cinder14:27
*** dillaman has joined #openstack-cinder14:30
*** yangyapeng has quit IRC14:30
*** gouthamr has joined #openstack-cinder14:31
*** zhangguoqing has quit IRC14:31
*** xyang has joined #openstack-cinder14:31
*** zhangguoqing has joined #openstack-cinder14:32
*** felipemonteiro has joined #openstack-cinder14:33
*** felipemonteiro_ has quit IRC14:35
*** zhangguoqing has quit IRC14:37
*** salv-orlando has joined #openstack-cinder14:38
*** zhurong has quit IRC14:39
*** lpetrut_ has quit IRC14:40
*** lkuchlan has quit IRC14:42
*** darrenc has quit IRC14:43
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: remove custom Lock object  https://review.openstack.org/44750314:53
openstackgerritJulien Danjou proposed openstack/cinder master: coordination: use tooz builtin heartbeat feature  https://review.openstack.org/44707914:53
tommylikehuhey xyang1 , smcginnis  this patch backports the bug you reviewed several hour ago :)14:53
*** alonma has quit IRC14:56
*** alonma has joined #openstack-cinder14:56
*** cknight has joined #openstack-cinder14:57
*** xyang has quit IRC14:59
*** alonma has quit IRC15:01
*** xyang has joined #openstack-cinder15:01
xyangtommylikehu: no idea which patch you are talking about:)15:02
tommylikehuI have no idea either :) I remembered to pasted it here: https://review.openstack.org/#/c/460618/15:03
xyangdr_gogeta86: hi, I didn't find anything obvious in your conf file.  So nothing specific.  Just make sure your SMI-S provider is working with the backend array.  You are using FC?  make sure zoning is set up15:04
*** pcaruana has quit IRC15:05
*** zhonghua has quit IRC15:07
*** zhonghua has joined #openstack-cinder15:08
*** mlakat has quit IRC15:11
openstackgerritMerged openstack/cinder master: Validate uuid parameters strictly for create volume API  https://review.openstack.org/45456215:11
openstackgerritMerged openstack/cinder master: Volume Type Encryption added to v2 api-ref  https://review.openstack.org/45994015:13
smcginnistommylikehu: How did you backport that patch? It doesn't have the necessary cherry pick information in the commit.15:13
smcginnistommylikehu: Oh, I see others have already commented on there. Sorry. :015:14
tommylikehuI just pressed the cherry pick button on the website :(15:14
tommylikehusmcginnis:  anything wrong?15:14
tommylikehusmcginnis: or because it's original patch has not been merged?15:15
*** zigo has quit IRC15:17
smcginnistommylikehu: Hmm, maybe that doesn't work right through gerrit. Not sure, but it should include the cherry pick info in the commit.15:17
smcginnistommylikehu: I think I've used a workflow similar to this: https://docs.openstack.org/contributor-guide/additional-git-workflow/cherry-pick.html15:18
tommylikehusmcginnis:  will add it15:18
smcginnistommylikehu: Maybe it doesn't add that until it's merged in master. Can't remember now.15:18
tommylikehuadd that manually15:19
dr_gogeta86xyang, it's all working ... but pretty slow15:19
tommylikehuI could15:19
smcginnistommylikehu: The commit id looks right, so maybe adding it manually would be the easiest at this point.15:19
xyangdr_gogeta86: it's slow without the driver involved?15:22
dr_gogeta86no15:23
xyangdr_gogeta86: please open a bug with logs uploaded so we can take a look15:26
cFoutshi, xyang. If the consistency group to volume group migration script doesn't also upgrade existing extra-specs that have "consistencygroup_support" defined won't that lead to a bad user experience?15:26
xyangcFouts: migration script should not change volume type definition15:26
cFoutsxyang: You can't change extra-specs now when a volume type is in being used by volumes15:27
xyangcFouts: you can15:27
cFoutsxyang: not without specifying another configuration option15:27
xyangcFouts: unless some patch was merged that I don't know15:27
xyangcFouts: Also consistency group_support is not a required spec15:28
openstackgerritGyorgy Szombathelyi proposed openstack/cinder master: Modernize the nova client in cinder  https://review.openstack.org/46064015:29
cFoutsxyang: There was a patch submitted by jgriffith. Looking for it.15:29
xyangcFout: you can also keep "consistencygroup_support =True" in capability reporting to be safe15:29
cFoutsxyang: at some point the extra-specs have to be updated15:30
cFoutsxyang: unless I am missing something this seems like a bad customer experience during upgrade15:30
xyangcFouts: I don't think migration script should change that though15:30
*** markvoelker has quit IRC15:30
openstackgerritGyorgy Szombathelyi proposed openstack/cinder master: Modernize the nova client in cinder  https://review.openstack.org/46064015:30
*** markvoelker has joined #openstack-cinder15:31
xyangcFouts: you can keep "consistencygroup_support =True" in your driver15:31
cFoutsxyang: yes but at some point that should be dropped, right?15:32
xyangcFouts: it doesn't have to be dropped if we worry that someone keeps it in extra specs forever15:32
xyangcFouts: because it was never a required spec, I didn't think it is a problem.  However it is definitely possible someone might have it there15:33
cFoutsxyang: That was why I was expecting to just migrate the extra-spec as well. Otherwise we have a legacy capability that we can't deprecate, right?15:33
xyangcFouts: I don't think extra spec for a volume type should be changed by migration script15:34
xyangcFouts: may be it can be an extra step for admin to decide to do15:34
*** aarefiev is now known as aarefiev_afk15:35
cFoutsxyang: yeah, the extra step is what I was trying to avoid. Having multiple steps tends to lead to customer support issues15:35
xyangcFouts: have you written in your driver doc that consistencygroup_support should be in extra_specs?15:35
cFoutsxyang: yes15:35
xyangcFouts: please show me15:36
openstackgerritGyorgy Szombathelyi proposed openstack/cinder master: Modernize the nova client in cinder  https://review.openstack.org/46064015:36
xyangcFouts: as I said, you can keep the capability in your driver that will make it backward compatible15:37
cFoutshttps://netapp.github.io/openstack-deploy-ops-guide/mitaka/content/cinder.cdot.fc.configuration.html15:38
*** alonma has joined #openstack-cinder15:40
*** gman-tx has quit IRC15:41
cFoutsok, thanks, xyang15:42
xyangcFouts: np15:42
cFoutsxyang: here is the patch I mentioned above: https://github.com/openstack/cinder/commit/b245225d5e67120dfe7aee5e941f381846c8942315:45
*** gman-tx has joined #openstack-cinder15:47
xyangcFouts: Now I remembered that patch. we should still give admin an option to change it somehow15:47
smcginnisxyang: Why was the extra spec changed for that? I can't remember.15:47
xyangcFouts: so in your case, keep that capability in your driver to maintain backward compatibility.  In the release notes, still ask user not to use it for new volume types and group types15:48
xyang@smcginnis: for generic volume groups, we use "consistent_group_snapshot_enabled", not "consistencygroup_support" which is tightly related to consistency15:48
smcginnisxyang: In retrospect, seems like we should have just kept the same extra spec key.15:49
xyang@smcginnis: so we could have "replication_group_enabled" and "consistent_replication_group_enabled"15:49
xyang@smcginnis: we can't:)15:49
xyang@smcginnis: if we want to support different kinds of groups15:49
*** salv-orl_ has joined #openstack-cinder15:50
xyang@smcginnis: I'd love to keep consistencygroups without adding generic volume groups if everyone loves "consistency"15:50
*** alonma has quit IRC15:50
xyang@smcginnis: tbarron told me NetApp can't support consistencygroup for replication, so we have to drop "consistency" from the key15:51
*** gman-tx has quit IRC15:52
*** salv-orlando has quit IRC15:52
*** imacdonn_ is now known as imacdonn15:52
*** alonma has joined #openstack-cinder15:52
smcginnisWell, I'm not saying keep consistency groups. I'm just not clear why we can't use the same extra spec that we used for consistency groups for groups that are consistent.15:53
tommylikehuhey xyang , eharney , I added the cherry pick messages manually :) https://review.openstack.org/#/c/460618/15:54
*** Apoorva has joined #openstack-cinder15:54
xyang@smcginnis: because "consistencygroup_support" only means consistent group snapshot in Cinder currently.  We can't say it means consistent group replication too15:55
*** gman-tx has joined #openstack-cinder15:56
*** e0ne has quit IRC15:57
tbarronxyang: tbarron can't remember that stuff any more, better check with someone else15:58
Swansontbarron, If I made an Ice Barron joke would you get it?15:59
xyangtbarron: sorry to reference you:)  It's all in the spec review any way.  we are not changing back15:59
tbarronSwanson: seems like I should remember that and get it but my brain is too mushy15:59
tbarronxyang: :D15:59
*** Julien-zte has quit IRC15:59
*** zhaochao has quit IRC16:00
Swansontbarron, ice bear talks in the third person a lot.16:00
*** Julien-zte has joined #openstack-cinder16:02
*** Julien-zte has quit IRC16:02
*** lamt has quit IRC16:03
*** eharney has quit IRC16:05
*** xyang has quit IRC16:06
*** Julien-zte has joined #openstack-cinder16:07
* tbarron is going to spend the afternoon on You Tube catching up with Yuria and the Bear16:07
*** chlong_ has joined #openstack-cinder16:08
*** pewp has quit IRC16:08
*** pewp has joined #openstack-cinder16:08
*** alonma has quit IRC16:13
*** chlong_ has quit IRC16:14
SwansonWhat is consistent group replication exactly?16:17
*** eharney has joined #openstack-cinder16:18
*** gman-tx has quit IRC16:25
*** gouthamr has quit IRC16:31
*** salv-orl_ has quit IRC16:36
*** gouthamr has joined #openstack-cinder16:40
tommylikehuhey eharney , to be clear , the revert-to-snapshot patches are little outdated. we definitely would not support online volume to be reverted without the effort from nova, I will update the spec and code tomorrow :)16:43
tommylikehuwe will focus on the detached ones16:45
smcginniseharney: The concern with reverting boot volumes is the volume is "attached", so cinder has no way to know it's safe.16:46
smcginniseharney: So kind of like cinder-backup, we require the volume is 'available'. At least as a first step.16:46
eharneysmcginnis: it was referring to "bootable" volumes, not "boot volumes" or "attached" volumes16:47
eharneythe "bootable" flag is not interesting here afaik16:47
smcginniseharney: Oh, right! Wording. Should be boot volumes, not "bootable" volumes.16:47
*** lamt has joined #openstack-cinder16:48
*** ociuhandu has quit IRC16:48
tommylikehueharney: smcginnis , thanks16:49
openstackgerritMerged openstack/cinder master: Use cg object when invoking update cg method.  https://review.openstack.org/46036516:52
*** salv-orlando has joined #openstack-cinder17:01
*** xyang has joined #openstack-cinder17:04
*** yangyape_ has quit IRC17:05
smcginnisildikov: Slow progress. ;)17:06
ildikovsmcginnis: well, Cinder and Nova are both really messy IMHO...17:07
smcginnisildikov: I won't argue that.17:07
*** harlowja has quit IRC17:08
ildikovlike we deal with one small thing and a zillion other just fall on our head every second...17:08
ildikovso who's the migrate_volume_completion expert here? :)17:08
smcginnisildikov: Not even sure on that.17:09
ildikovsmcginnis: then I guess I'll have to make myself one...17:12
*** gman-tx has joined #openstack-cinder17:12
smcginnisildikov: :)17:13
*** gman-tx has quit IRC17:13
*** gman-tx has joined #openstack-cinder17:15
*** ChubYann has joined #openstack-cinder17:27
*** dhinesh has joined #openstack-cinder17:37
*** tongli has joined #openstack-cinder17:41
*** harlowja has joined #openstack-cinder17:51
*** pewp has quit IRC17:51
*** pewp has joined #openstack-cinder17:52
*** mvk has quit IRC17:53
*** cknight has quit IRC17:56
*** erlon has joined #openstack-cinder17:57
*** chhavi has quit IRC17:59
*** cknight has joined #openstack-cinder17:59
*** jdurgin has joined #openstack-cinder18:03
*** yangyapeng has joined #openstack-cinder18:06
*** tongli has quit IRC18:06
*** yangyapeng has quit IRC18:10
*** dustins has quit IRC18:18
*** salv-orlando has quit IRC18:27
*** e0ne has joined #openstack-cinder18:29
*** fnordahl_ has joined #openstack-cinder18:38
*** eglute has quit IRC18:38
*** fnordahl has quit IRC18:38
*** eglute has joined #openstack-cinder18:39
*** openstackgerrit has quit IRC18:48
*** crushil has quit IRC18:52
*** crushil has joined #openstack-cinder18:53
*** gman-tx has quit IRC18:54
*** openstackgerrit has joined #openstack-cinder19:00
openstackgerritMerged openstack/cinder master: Spelling error "paramenter"  https://review.openstack.org/46040819:00
*** gman-tx has joined #openstack-cinder19:02
*** crushil has quit IRC19:03
*** kfarr has joined #openstack-cinder19:03
*** yangyapeng has joined #openstack-cinder19:07
*** mvk has joined #openstack-cinder19:08
*** felipemonteiro has quit IRC19:08
*** yangyapeng has quit IRC19:11
*** kfarr has quit IRC19:25
*** salv-orlando has joined #openstack-cinder19:30
*** xyang has quit IRC19:33
*** xyang has joined #openstack-cinder19:39
*** xyang has quit IRC19:40
*** kaisers has quit IRC19:41
*** kaisers has joined #openstack-cinder19:42
*** dustins has joined #openstack-cinder19:46
*** kaisers has quit IRC19:46
*** crushil has joined #openstack-cinder19:47
*** kaisers has joined #openstack-cinder19:48
*** porrua has quit IRC19:49
*** crushil has quit IRC19:52
*** e0ne has quit IRC20:03
*** felipemonteiro has joined #openstack-cinder20:03
*** yangyapeng has joined #openstack-cinder20:08
*** felipemonteiro_ has joined #openstack-cinder20:08
*** shyama has quit IRC20:08
*** e0ne has joined #openstack-cinder20:10
*** felipemonteiro has quit IRC20:11
*** alonma has joined #openstack-cinder20:12
*** yangyapeng has quit IRC20:12
*** crushil has joined #openstack-cinder20:15
*** eharney has quit IRC20:17
*** Cibo_ has joined #openstack-cinder20:23
openstackgerritNicholas Jones proposed openstack/cinder master: Delete limited_by_marker from api/common.py  https://review.openstack.org/46073620:25
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/46051220:26
*** liverpooler has quit IRC20:26
*** e0ne has quit IRC20:34
*** kaisers has quit IRC20:38
*** xyang has joined #openstack-cinder20:38
*** salv-orlando has quit IRC20:39
*** salv-orlando has joined #openstack-cinder20:39
*** xyang has quit IRC20:48
*** xyang has joined #openstack-cinder20:48
*** alonma has quit IRC20:49
*** cknight has quit IRC20:50
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements  https://review.openstack.org/46051220:54
*** jonesn has quit IRC20:57
*** tongli has joined #openstack-cinder20:59
*** gman-tx has quit IRC20:59
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/45883821:03
*** crushil has quit IRC21:04
*** dustins has quit IRC21:05
*** yangyapeng has joined #openstack-cinder21:08
*** gman-tx has joined #openstack-cinder21:13
*** edmondsw has quit IRC21:14
*** yangyapeng has quit IRC21:15
*** tongli has quit IRC21:22
*** abishop has quit IRC21:26
*** zigo has joined #openstack-cinder21:32
*** xyang has quit IRC21:47
*** salv-orl_ has joined #openstack-cinder21:50
*** Nakato has quit IRC21:52
*** salv-orlando has quit IRC21:52
*** Nakato has joined #openstack-cinder21:53
*** catintheroof has quit IRC22:00
*** jose-phillips has joined #openstack-cinder22:00
*** salv-orl_ has quit IRC22:01
*** Apoorva_ has joined #openstack-cinder22:01
*** Apoorva has quit IRC22:04
*** yangyapeng has joined #openstack-cinder22:12
*** cburgess has quit IRC22:12
*** erlon has quit IRC22:13
*** jungleboyj has quit IRC22:14
*** harlowja has quit IRC22:14
*** rha has quit IRC22:15
*** cburgess has joined #openstack-cinder22:16
*** rushiagr has quit IRC22:17
*** sweston has quit IRC22:17
*** tdurakov has quit IRC22:17
*** yangyapeng has quit IRC22:17
*** theanalyst has quit IRC22:18
*** rha has joined #openstack-cinder22:19
*** rha has quit IRC22:19
*** rha has joined #openstack-cinder22:19
*** sweston has joined #openstack-cinder22:21
*** theanalyst has joined #openstack-cinder22:25
*** tdurakov has joined #openstack-cinder22:26
*** felipemonteiro_ has quit IRC22:28
*** rushiagr has joined #openstack-cinder22:32
*** edmondsw has joined #openstack-cinder22:37
*** salv-orlando has joined #openstack-cinder22:37
*** kaisers has joined #openstack-cinder22:38
*** edmondsw has quit IRC22:42
*** kaisers has quit IRC22:42
*** liverpooler has joined #openstack-cinder22:44
*** gman-tx has quit IRC22:50
*** diablo_rojo has joined #openstack-cinder23:03
*** Kvisle has quit IRC23:03
*** Kvisle has joined #openstack-cinder23:05
*** salv-orlando has quit IRC23:08
*** sdague has quit IRC23:08
*** lamt has quit IRC23:09
*** diablo_rojo has quit IRC23:17
*** tesseract has quit IRC23:20
*** harlowja has joined #openstack-cinder23:23
*** mriedem has quit IRC23:30
*** mriedem has joined #openstack-cinder23:30
*** stakeda has joined #openstack-cinder23:33
*** Julien-zte has quit IRC23:34
*** yangyapeng has joined #openstack-cinder23:37
*** Apoorva_ has quit IRC23:38
*** Apoorva has joined #openstack-cinder23:39
*** markvoelker has quit IRC23:40
*** yangyapeng has quit IRC23:42
*** dhinesh has quit IRC23:53
*** jungleboyj has joined #openstack-cinder23:55

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