Thursday, 2017-02-02

*** erlon has quit IRC00:00
*** xyang has quit IRC00:01
openstackgerritMerged openstack/cinder: ZTE: Added CI_WIKI_NAME  https://review.openstack.org/42782300:06
*** xyang has joined #openstack-cinder00:06
*** mriedem has quit IRC00:06
*** xyang has quit IRC00:09
*** xyang1 has quit IRC00:12
*** esker has quit IRC00:19
*** sdague has quit IRC00:21
*** ducttape_ has quit IRC00:39
*** ducttape_ has joined #openstack-cinder00:39
*** esker has joined #openstack-cinder00:41
*** ducttape_ has quit IRC00:41
*** _ducttape_ has joined #openstack-cinder00:41
*** ksumit has quit IRC00:41
*** _ducttape_ has quit IRC00:42
*** ducttape_ has joined #openstack-cinder00:42
*** knomura has joined #openstack-cinder00:51
*** knomura has quit IRC00:55
*** openstackgerrit has quit IRC01:02
*** esker has quit IRC01:03
*** dave-mccowan has quit IRC01:05
*** rushigerrard has joined #openstack-cinder01:10
*** raunak has quit IRC01:13
*** raunak has joined #openstack-cinder01:15
*** raunak has quit IRC01:15
*** rushigerrard has quit IRC01:22
*** alonma has joined #openstack-cinder01:31
*** mriedem has joined #openstack-cinder01:32
*** tlian has quit IRC01:32
*** alonma has quit IRC01:36
*** alonma has joined #openstack-cinder01:38
*** tlian has joined #openstack-cinder01:41
*** Apoorva has quit IRC01:41
*** alonma has quit IRC01:42
*** jose-phillips has quit IRC01:44
*** windmoon has joined #openstack-cinder01:50
*** mtanino has quit IRC01:53
*** windmoon has quit IRC01:57
*** Apoorva has joined #openstack-cinder02:02
*** jdillaman has quit IRC02:04
*** Apoorva has quit IRC02:07
*** zhurong has joined #openstack-cinder02:08
*** jdillaman has joined #openstack-cinder02:17
*** cdelatte has quit IRC02:19
*** dhinesh has quit IRC02:20
*** zhurong has quit IRC02:20
*** openstackgerrit has joined #openstack-cinder02:36
openstackgerritjiamin proposed openstack/cinder master: support bulk create for CG and VG  https://review.openstack.org/42494502:36
*** raunak has joined #openstack-cinder02:38
*** diablo_rojo_phon has quit IRC02:40
*** raunak has quit IRC02:40
*** raunak has joined #openstack-cinder02:42
*** kaisers_ has joined #openstack-cinder02:53
*** kaisers has quit IRC02:56
*** raunak has joined #openstack-cinder03:01
*** gmann has joined #openstack-cinder03:01
*** bkopilov has quit IRC03:02
*** tlian has quit IRC03:06
*** tlian has joined #openstack-cinder03:18
*** mtanino has joined #openstack-cinder03:20
*** gouthamr has quit IRC03:24
*** mtanino has quit IRC03:24
*** gaurangt has joined #openstack-cinder03:36
*** alonma has joined #openstack-cinder03:45
*** alonma has quit IRC03:49
*** tlian has quit IRC03:52
*** nicolasbock has quit IRC04:01
*** raunak has quit IRC04:01
*** gaurangt has left #openstack-cinder04:09
*** gaurangt has joined #openstack-cinder04:15
*** gaurangt has left #openstack-cinder04:16
*** gaurangt has joined #openstack-cinder04:17
*** psachin has joined #openstack-cinder04:18
*** bkopilov has joined #openstack-cinder04:19
*** test123 has joined #openstack-cinder04:19
*** akapil has joined #openstack-cinder04:21
*** test123 has quit IRC04:24
*** akapil has quit IRC04:25
*** Lee1092 has joined #openstack-cinder04:29
*** mdnadeem has joined #openstack-cinder04:29
*** ducttape_ has quit IRC04:36
*** pdeore has joined #openstack-cinder04:47
*** adisky_ has joined #openstack-cinder04:50
*** archies has joined #openstack-cinder04:55
*** knomura has joined #openstack-cinder04:57
*** pbandark has joined #openstack-cinder05:01
*** knomura has quit IRC05:02
*** bkopilov_ has joined #openstack-cinder05:25
*** ducttape_ has joined #openstack-cinder05:37
*** ducttape_ has quit IRC05:42
*** Jack_Iv has joined #openstack-cinder05:49
*** Jack_Iv has quit IRC05:50
*** Jack_Iv has joined #openstack-cinder05:53
*** alonmarx has joined #openstack-cinder06:01
*** mdnadeem has quit IRC06:06
*** Jack_Iv has quit IRC06:08
*** kaisers_ has quit IRC06:09
*** chenying_ has joined #openstack-cinder06:09
*** markvoelker has joined #openstack-cinder06:10
*** zhonghua2 has joined #openstack-cinder06:10
*** pkoniszewski has quit IRC06:11
*** mdnadeem has joined #openstack-cinder06:12
*** Jack_Iv has joined #openstack-cinder06:12
*** zhonghua has quit IRC06:12
*** markvoelker_ has quit IRC06:12
*** mnaser has quit IRC06:12
*** chenying has quit IRC06:13
*** rhefner has quit IRC06:13
*** pkoniszewski has joined #openstack-cinder06:15
*** lpetrut has joined #openstack-cinder06:18
*** jdurgin1 has quit IRC06:19
*** mnaser has joined #openstack-cinder06:19
*** Jack_Iv has quit IRC06:28
*** Jack_Iv has joined #openstack-cinder06:28
*** pgadiya has joined #openstack-cinder06:30
*** kaisers has joined #openstack-cinder06:30
*** diablo_rojo has joined #openstack-cinder06:31
*** mjura has joined #openstack-cinder06:31
*** Jack_Iv has quit IRC06:33
*** sudipto_ has joined #openstack-cinder06:38
*** sudipto has joined #openstack-cinder06:38
*** psachin has quit IRC06:42
*** kwathore has quit IRC06:49
*** Julien-zte has joined #openstack-cinder06:52
*** kwathore has joined #openstack-cinder06:55
*** raunak has joined #openstack-cinder06:56
*** pdeore has quit IRC06:57
*** psachin has joined #openstack-cinder06:58
*** nikeshm has joined #openstack-cinder06:59
*** tommylikehu_ has joined #openstack-cinder07:04
*** pdeore has joined #openstack-cinder07:04
*** rcernin has joined #openstack-cinder07:06
*** lpetrut has quit IRC07:07
*** ducttape_ has joined #openstack-cinder07:08
*** diablo_rojo has quit IRC07:09
*** alonma has joined #openstack-cinder07:11
*** ducttape_ has quit IRC07:13
*** zhurong has joined #openstack-cinder07:13
*** alonma has quit IRC07:16
*** tesseract has joined #openstack-cinder07:16
*** kaisers has quit IRC07:16
*** kaisers has joined #openstack-cinder07:17
*** ChubYann has quit IRC07:17
*** lkuchlan_ has joined #openstack-cinder07:24
*** e0ne has joined #openstack-cinder07:28
*** lpetrut has joined #openstack-cinder07:29
*** Apoorva has joined #openstack-cinder07:29
*** psachin has quit IRC07:30
*** Jack_Iv has joined #openstack-cinder07:30
*** arne_r has joined #openstack-cinder07:30
*** e0ne has quit IRC07:33
*** Jack_Iv has quit IRC07:34
*** Apoorva has quit IRC07:36
*** e0ne has joined #openstack-cinder07:38
*** stakeda has joined #openstack-cinder07:38
*** nikeshm has quit IRC07:39
*** pcaruana has joined #openstack-cinder07:39
*** raunak has quit IRC07:40
*** psachin has joined #openstack-cinder07:42
*** e0ne has quit IRC07:42
*** pcaruana has quit IRC07:42
*** pcaruana has joined #openstack-cinder07:43
*** nkrinner_afk is now known as nkrinner07:45
*** e0ne has joined #openstack-cinder07:46
*** alonma has joined #openstack-cinder07:47
*** alonma has quit IRC07:51
*** e0ne has quit IRC07:53
*** e0ne has joined #openstack-cinder07:57
*** e0ne has quit IRC08:01
*** alonma has joined #openstack-cinder08:02
*** e0ne has joined #openstack-cinder08:06
*** lpetrut has quit IRC08:07
*** lkuchlan_ has quit IRC08:08
*** lkuchlan has joined #openstack-cinder08:08
openstackgerritKazumasa Nomura proposed openstack/cinder master: Restrain the unmanage snapshot for the VSP drivers  https://review.openstack.org/42109508:09
*** e0ne has quit IRC08:11
*** e0ne has joined #openstack-cinder08:16
*** lpetrut has joined #openstack-cinder08:18
*** alonmarx has quit IRC08:20
*** e0ne has quit IRC08:21
*** shausy has joined #openstack-cinder08:22
*** e0ne has joined #openstack-cinder08:26
*** zhurong has quit IRC08:28
*** rcernin has quit IRC08:28
*** e0ne has quit IRC08:34
*** ducttape_ has joined #openstack-cinder08:38
*** e0ne has joined #openstack-cinder08:38
*** tommylikehu_ has quit IRC08:40
*** akapil has joined #openstack-cinder08:41
openstackgerritSilvan Kaiser proposed openstack/cinder master: Removes getfattr from Quobyte Cinder driver  https://review.openstack.org/42783308:41
*** ducttape_ has quit IRC08:43
*** akapil has quit IRC08:44
*** e0ne has quit IRC08:50
*** akapil has joined #openstack-cinder08:51
openstackgerritMarc Koderer proposed openstack/cinder master: Add check if copyoffload tool is available  https://review.openstack.org/42761308:52
*** rcernin has joined #openstack-cinder08:52
openstackgerritMarc Koderer proposed openstack/cinder master: Add check if copyoffload tool is available  https://review.openstack.org/42761308:53
*** Jack_Iv has joined #openstack-cinder08:54
*** e0ne has joined #openstack-cinder08:54
*** lpetrut has quit IRC08:57
*** cyberik has joined #openstack-cinder08:58
*** pbandark has quit IRC08:58
*** cyberik has quit IRC08:59
*** zzzeek has quit IRC09:00
*** lpetrut has joined #openstack-cinder09:01
*** zzzeek has joined #openstack-cinder09:01
*** BobBall has joined #openstack-cinder09:02
*** pbandark has joined #openstack-cinder09:02
*** e0ne has quit IRC09:02
BobBallHowdy - Looks from https://bugs.launchpad.net/cinder/+bug/1661144 that https://review.openstack.org/#/c/396269/ has broken support with Ubuntu 14.04.  Was that intentional?  Do we care?09:04
openstackLaunchpad bug 1661144 in Cinder "Failed to initialize driver due to unrecognized option '--ignoreskippedcluster" [Undecided,New]09:04
*** Jack_Iv has quit IRC09:04
*** shubham0d has joined #openstack-cinder09:11
*** shubham0d has quit IRC09:12
*** s-shiono_ has quit IRC09:18
*** lpetrut has quit IRC09:19
*** alonma has quit IRC09:21
*** alonma has joined #openstack-cinder09:22
*** jordanP has joined #openstack-cinder09:22
Dinesh_BhorBobBall: I am also getting this error after pulling new code on ubuntu 14.0409:23
*** alonma has quit IRC09:26
*** lkuchlan has quit IRC09:27
*** alonma has joined #openstack-cinder09:30
*** lkuchlan has joined #openstack-cinder09:41
*** stakeda has quit IRC09:41
*** yumiriam has joined #openstack-cinder09:48
*** lkuchlan has quit IRC09:50
*** akapil has quit IRC09:56
*** lkuchlan has joined #openstack-cinder10:02
*** akapil has joined #openstack-cinder10:04
*** ducttape_ has joined #openstack-cinder10:09
*** alyson_ has joined #openstack-cinder10:10
*** ducttape_ has quit IRC10:14
*** nikeshm has joined #openstack-cinder10:22
*** dalgaaf has quit IRC10:43
openstackgerritxianming.mao proposed openstack/cinder-specs master: Add support for volume online-migrate when backend has RBD  https://review.openstack.org/42453810:43
*** erlon has joined #openstack-cinder10:45
*** dalgaaf has joined #openstack-cinder10:49
*** alonma has quit IRC10:49
*** alonma has joined #openstack-cinder10:50
*** alonma has quit IRC10:53
*** alonma has joined #openstack-cinder10:53
*** lpetrut has joined #openstack-cinder11:01
*** pbandark has quit IRC11:03
*** shaner has quit IRC11:03
*** shaner has joined #openstack-cinder11:04
openstackgerritAlon Marx proposed openstack/cinder master: xiv delete cg fails if removed on backend  https://review.openstack.org/42808611:05
*** aarefiev_afk is now known as aarefiev11:08
*** ducttape_ has joined #openstack-cinder11:10
*** pbandark has joined #openstack-cinder11:12
*** ducttape_ has quit IRC11:14
*** shubham0d has joined #openstack-cinder11:15
*** ociuhandu has quit IRC11:17
*** kwathore has quit IRC11:18
*** nicolasbock has joined #openstack-cinder11:18
*** sudipto_ has quit IRC11:19
*** sudipto has quit IRC11:19
shubham0dI have recently installed devstack, now if I am changing code on /opt/stack/cinder then did I need to run stack.sh again or there any other way to save those changes.11:19
*** akapil has quit IRC11:25
*** Jack_Iv has joined #openstack-cinder11:30
*** ducttape_ has joined #openstack-cinder11:30
openstackgerritMichal Dulko proposed openstack/cinder master: Make get_capabilities look for clusters  https://review.openstack.org/42772411:39
*** shubham0d has quit IRC11:40
openstackgerritKazumasa Nomura proposed openstack/cinder master: Restrain the unmanage snapshot for the VSP drivers  https://review.openstack.org/42109511:42
*** psachin has quit IRC11:43
*** akapil has joined #openstack-cinder11:45
*** catintheroof has joined #openstack-cinder11:47
*** nikeshm has quit IRC11:48
*** akapil has quit IRC11:50
*** psachin has joined #openstack-cinder11:55
dulekgeguileo: ping - yesterday's discussion about get-pools tempest tests.11:57
*** ducttape_ has quit IRC11:57
*** kwathore has joined #openstack-cinder11:57
dulekgeguileo: I don't think adding a parameter to show volume API help us - we're constrained by API microversions now.11:57
*** bkopilov_ has quit IRC11:57
dulekgeguileo: And the problem is that this get-pools tests in Tempest are run against V2 API, which doesn't have a notion of clusters.11:58
*** bkopilov has quit IRC11:58
*** archies has quit IRC11:59
*** ducttape_ has joined #openstack-cinder12:00
*** e0ne has joined #openstack-cinder12:00
*** openstackgerrit has quit IRC12:03
*** akapil has joined #openstack-cinder12:06
*** openstackgerrit has joined #openstack-cinder12:08
openstackgerritVipin Balachandran proposed openstack/cinder master: VMware: Set vSphere opID prefix  https://review.openstack.org/42810712:08
*** sdague has joined #openstack-cinder12:09
*** dave-mccowan has joined #openstack-cinder12:15
openstackgerritMerged openstack/cinder master: Fix notification short-circuit  https://review.openstack.org/42764712:16
*** pgadiya has quit IRC12:17
*** ociuhandu has joined #openstack-cinder12:21
*** ducttape_ has quit IRC12:22
*** alonma has quit IRC12:24
*** alonma has joined #openstack-cinder12:25
*** pgadiya has joined #openstack-cinder12:29
*** alonma has quit IRC12:30
*** alonma has joined #openstack-cinder12:31
*** alonma has quit IRC12:35
*** ducttape_ has joined #openstack-cinder12:42
*** Lee1092 has quit IRC12:46
adisky_hii12:47
adisky_cinder team12:47
geguileodulek: But I believe that if you enable that new cluster extension that would be out of the scope of the microversions and it would be sent to v2 as well12:47
BobBallSo for some reason it looks like the lvm version check added in https://review.openstack.org/#/c/396269 is failing; the ignoreskippedcluster is being added to all calls even though my LVM version is 2.02.9812:48
geguileoAnybody knows what's the deal with the gate-tempest-dsvm-full-ceph-plugin-src-os-brick-ubuntu-xenial gate?12:48
adisky_can anybody tell me how can we abort ongoing volume migration??12:49
dulekgeguileo: Hm, most probably yes. I think scottda can help us to asses if that's appropriate. Anyway I'll post tempest fix in a moment - maybe it will be a decent workaround.12:49
geguileodulek: cool12:50
geguileodulek: While you do useful stuff I'm stuck with a patch changing a debug level message that doesn't want to merge...  :-(12:50
*** alonma has joined #openstack-cinder12:51
dulekgeguileo: Hey, I'm sure 10+ patches in A/A chain were worse!12:53
geguileodulek: lol   true that!12:53
*** ducttape_ has quit IRC12:53
*** alonma has quit IRC12:55
*** mvk has quit IRC12:55
*** lseki has joined #openstack-cinder12:59
*** gcb has joined #openstack-cinder13:00
*** alonma has joined #openstack-cinder13:01
*** ducttape_ has joined #openstack-cinder13:02
dulekgeguileo: https://review.openstack.org/42812613:02
geguileodulek: thanks13:03
*** gouthamr has joined #openstack-cinder13:11
openstackgerritTzur Eliyahu proposed openstack/cinder master: Xiv create vol from replicated source fails  https://review.openstack.org/42812813:11
*** pgadiya has quit IRC13:16
openstackgerritMerged openstack/cinder master: Remove redundant log during initialize connection  https://review.openstack.org/42726513:17
*** dalgaaf has quit IRC13:20
*** dalgaaf has joined #openstack-cinder13:27
*** eharney has quit IRC13:29
*** mvk has joined #openstack-cinder13:29
openstackgerritTzur Eliyahu proposed openstack/cinder master: Xiv create vol from replicated source fails  https://review.openstack.org/42812813:33
*** Jack_Iv has quit IRC13:36
*** shausy has quit IRC13:36
*** knomura has joined #openstack-cinder13:41
*** windmoon has joined #openstack-cinder13:41
*** akapil has quit IRC13:45
*** Jack_Iv has joined #openstack-cinder13:46
*** windmoon has quit IRC13:46
kwathorePlease review cherry pick:13:47
kwathorehttps://review.openstack.org/#/c/424455/13:47
Dinesh_Bhorgcb: ping13:47
gcbDinesh_Bhor: pong13:48
*** zhurong has joined #openstack-cinder13:49
*** alonma has quit IRC13:50
Dinesh_Bhorgcb: about this: https://review.openstack.org/#/c/413446/1 I know you must be busy in some other priority tasks but have you figured out why the warning is still coming?13:50
*** alonma has joined #openstack-cinder13:50
*** Jack_Iv has quit IRC13:51
*** pbandark has quit IRC13:52
gcbDinesh_Bhor: yeah, just in vacation for Chinese New Year, I'm not sure it's a bug for debtcollector  https://github.com/openstack/debtcollector/blob/master/debtcollector/updating.py#L5513:52
gcbDinesh_Bhor: will back to office tomorrow, then dig more13:53
Dinesh_Bhorgcb: ok, thanks13:54
gcbDinesh_Bhor: np :-)13:54
*** alonma has quit IRC13:55
*** psachin has quit IRC13:55
*** crushil has quit IRC13:56
*** alonma has joined #openstack-cinder13:56
*** ducttape_ has quit IRC13:57
*** Jack_Iv has joined #openstack-cinder13:57
*** alonma has quit IRC13:57
*** alonma has joined #openstack-cinder13:58
openstackgerritAlon Marx proposed openstack/cinder master: xiv delete cg fails if removed on backend  https://review.openstack.org/42808613:59
*** eharney has joined #openstack-cinder14:02
*** Jack_Iv has quit IRC14:04
*** gcb has quit IRC14:05
*** zhurong has quit IRC14:06
*** eharney has quit IRC14:07
*** akapil has joined #openstack-cinder14:08
*** dustins has joined #openstack-cinder14:09
Dinesh_Bhorgeguileo: ping, about your comment: https://review.openstack.org/#/c/427560/1/cinder/api/contrib/resource_common_manage.py14:09
geguileoDinesh_Bhor: pong14:10
geguileoaaaaaaaaand I'm gone for lunch14:11
Dinesh_Bhorgeguileo: At this moment I don't know which are the other exceptions will receive in RemoteError so all those should be raised that's why I have put else there14:11
Dinesh_Bhorgeguileo: ok then, will talk later14:12
openstackgerritSean McGinnis proposed openstack/cinder master: Clean up release notes for Ocata  https://review.openstack.org/42789614:16
*** ducttape_ has joined #openstack-cinder14:16
*** zhurong has joined #openstack-cinder14:17
*** dustins has quit IRC14:17
*** dustins has joined #openstack-cinder14:17
*** crushil has joined #openstack-cinder14:19
*** eharney has joined #openstack-cinder14:19
*** bkopilov_ has joined #openstack-cinder14:19
*** bkopilov has joined #openstack-cinder14:19
dulekgeguileo: I've just got clear Tempest run with A/A and all the fixes applied. :)14:21
*** shubham0d has joined #openstack-cinder14:22
*** alonma has quit IRC14:23
*** alonma has joined #openstack-cinder14:23
*** psachin has joined #openstack-cinder14:26
*** jperry has joined #openstack-cinder14:27
smcginnisdulek: +114:27
smcginnisdulek: That almost warrants a #success tag. :)14:27
*** ducttape_ has quit IRC14:27
*** alonma has quit IRC14:28
*** xyang has joined #openstack-cinder14:28
*** porrua has joined #openstack-cinder14:28
*** Jack_Iv has joined #openstack-cinder14:30
*** alonma has joined #openstack-cinder14:30
*** liverpooler has joined #openstack-cinder14:30
*** Jack_Iv has quit IRC14:30
*** liverpooler has quit IRC14:30
*** liverpooler has joined #openstack-cinder14:30
*** sdague has quit IRC14:34
*** alonma has quit IRC14:34
*** sandanar has joined #openstack-cinder14:35
*** psachin has quit IRC14:37
*** Jack_Iv has joined #openstack-cinder14:39
*** zzzeek has quit IRC14:40
*** zzzeek has joined #openstack-cinder14:40
*** mriedem has quit IRC14:41
*** Jack_Iv has quit IRC14:42
*** shubham0d has quit IRC14:42
openstackgerritMerged openstack/cinder master: Add command information to cinder-manage.rst  https://review.openstack.org/35716214:43
*** crushil has quit IRC14:43
*** dustins has quit IRC14:43
*** crushil has joined #openstack-cinder14:44
*** dustins has joined #openstack-cinder14:44
geguileodulek: Awesome!!14:44
geguileodulek: Now we just need to get them merged14:44
geguileoDinesh_Bhor: Yes, I know, but the raise does raise the RemoteException exception, and not the actual exception, what I was suggesting was to transform the remote exception to the actual exception and raise it14:46
*** alonma has joined #openstack-cinder14:46
openstackgerritBob Ball proposed openstack/cinder master: Check for support on self, not for the method.  https://review.openstack.org/42818014:47
openstackgerritBob Ball proposed openstack/cinder master: Check for support on self, not for the method.  https://review.openstack.org/42818014:48
*** pdeore has quit IRC14:49
BobBallsmcginnis: I can't find your username on Gerrit, but could https://review.openstack.org/#/c/428180/ be reviewed?  It's a bug fix for a recent cinder commit which broke the XenServer CI (running on Nova/Neutron but not Cinder)14:50
*** alonma has quit IRC14:50
openstackgerritMerged openstack/python-cinderclient master: Bump MAX_VERSION to 3.27  https://review.openstack.org/42727014:51
BobBallsmcginnis: OK - now I found you on gerrit - not sure why it didn't complete before...14:52
*** Jack_Iv has joined #openstack-cinder14:53
BobBallAlso ignore me... This is going to be a bit trickier... :)14:53
*** xyang has quit IRC14:54
* jungleboyj should abandon this review?14:54
Dinesh_Bhorgeguileo: ok, got it. I will addressed that comment now accordingly. Thanks for your time.14:54
openstackgerritRohan Arora proposed openstack/cinder master: Nosec audit url open issue from Bandit  https://review.openstack.org/42435614:54
Dinesh_Bhorgeguileo: s/addressed/address14:55
BobBallsmcginnis: In the short term, can we revert https://review.openstack.org/#/c/396269/ unless you have an easy way of calculating the property from a static method?14:55
*** Jack_Iv has quit IRC14:55
*** xyang has joined #openstack-cinder14:56
openstackgerritBob Ball proposed openstack/cinder master: Revert "Cluster volume group fix in lvm"  https://review.openstack.org/42818914:56
geguileoDinesh_Bhor: Although I'm thinking about it...14:56
geguileoDinesh_Bhor: Don't do it, it will be a nightmare14:57
geguileoDinesh_Bhor: Because you'll only be able to do it easily for cinder.exception cases, all other will be a nightmare14:57
Dinesh_Bhorgeguileo: yah, thats what i was thinking now14:58
geguileoDinesh_Bhor: If you just remove the else I'll give it a +214:58
Dinesh_Bhorgeguileo: ok, thanks i will do that14:59
*** Yogi1 has joined #openstack-cinder15:00
*** alonma has joined #openstack-cinder15:02
*** sudipto_ has joined #openstack-cinder15:03
*** sudipto has joined #openstack-cinder15:03
*** alonma has quit IRC15:06
*** mriedem has joined #openstack-cinder15:07
*** zhurong has quit IRC15:09
*** mtanino has joined #openstack-cinder15:13
*** ducttape_ has joined #openstack-cinder15:14
*** Julien-zte has quit IRC15:14
*** Julien-zte has joined #openstack-cinder15:15
*** Julien-zte has quit IRC15:15
*** xyang has quit IRC15:16
openstackgerritTzur Eliyahu proposed openstack/cinder master: Xiv create vol from replicated source fails  https://review.openstack.org/42812815:16
*** xyang has joined #openstack-cinder15:17
*** dave-mccowan has quit IRC15:18
*** alonma has joined #openstack-cinder15:18
*** sdague has joined #openstack-cinder15:21
*** Jack_Iv has joined #openstack-cinder15:21
*** kfarr has joined #openstack-cinder15:22
*** alonma has quit IRC15:22
*** alonma has joined #openstack-cinder15:29
*** sdague has quit IRC15:30
*** rajinir has joined #openstack-cinder15:30
*** gabor_antal_ has joined #openstack-cinder15:32
*** gabor_antal has quit IRC15:32
*** knomura_ has joined #openstack-cinder15:32
*** sdague has joined #openstack-cinder15:33
*** dave-mccowan has joined #openstack-cinder15:33
*** breitz has quit IRC15:33
*** alonma has quit IRC15:34
*** breitz has joined #openstack-cinder15:34
*** knomura has quit IRC15:35
rajinirhttps://www.irccloud.com/pastebin/TGTlllfA/qos%20tempest15:35
*** sandanar has quit IRC15:36
*** thiagolib has quit IRC15:38
*** mdnadeem has quit IRC15:40
*** alonma has joined #openstack-cinder15:45
*** Cibo_ has joined #openstack-cinder15:45
*** arne_r has quit IRC15:46
*** alonma has quit IRC15:49
rajinirTurns out it was not related to CI.15:49
rajinirIgnore15:49
smcginnisBobBall: Oh, the issue is it's just not being called. We should just fix that.15:50
smcginnisBobBall: Oh wait, that's a property, not a static method.15:51
BobBallsmcginnis: Indeed - but the LVM version check takes a root helper which is provided to the instance instantiation15:51
smcginnisBobBall: So this one looked right: https://review.openstack.org/#/c/428180/15:52
*** alonma has joined #openstack-cinder15:52
smcginnisBobBall: That didn't work?15:52
BobBallIt's a static method so can't use self15:53
*** e0ne has quit IRC15:53
BobBallI thought it was the obvious fix so just sent it to gerrit before I actually thought about it :)15:53
smcginnisBobBall: No, it's a @Property.15:53
BobBallget_all_physical_volumes is static15:53
BobBallso can't use self.<property>15:53
* smcginnis must be looking at different source...15:54
*** lkuchlan has quit IRC15:54
smcginnisBobBall: No, it's a property on the instance: https://github.com/openstack/cinder/blob/bc9704f0622a95a003cfa2226d5ee0bd4aedb6c6/cinder/brick/local_dev/lvm.py#L26115:54
BobBallOh - but it passes the roothelper there - so yes, we can make an easy fix...15:54
BobBallsmcginnis: Look at the function name I said :)15:54
BobBallsmcginnis: The function which calls the property is a static function https://github.com/openstack/cinder/blob/bc9704f0622a95a003cfa2226d5ee0bd4aedb6c6/cinder/brick/local_dev/lvm.py#L34015:55
smcginnisBobBall: Ah! Gotcha. The calling method is trying to get an instance property without an instance.15:55
BobBallbut it receives the root helper so actually it can pass that to a newly static https://github.com/openstack/cinder/blob/bc9704f0622a95a003cfa2226d5ee0bd4aedb6c6/cinder/brick/local_dev/lvm.py#L26215:55
smcginnisBobBall: Yep, looks like that should work.15:56
dulekgeguileo, smcginnis: Do we want these A/A fixes before RC-1? It's https://review.openstack.org/#/c/427356/ and https://review.openstack.org/#/c/427724/15:56
geguileodulek: YES!!15:57
BobBallI'll hack something up.15:57
geguileodulek: And this one as well https://review.openstack.org/#/c/425768/115:57
smcginnisBobBall: Awesome, thanks!15:57
dulekgeguileo, smcginnis: No driver will officially support A/A in Ocata, but they'll definitely help with testing as they're enabling Tempest with clustered env.15:58
BobBallNo worries.  I need to get this fixed to get my CI back online :(15:58
geguileodulek: that was my thought as well15:58
smcginnis+115:58
geguileodulek: We want vendors to be able to test their drivers with stable/ocata15:58
*** _ducttape_ has joined #openstack-cinder16:00
*** _ducttape_ has quit IRC16:01
karthikp_Hi, qucik quesiton.. Is "tox -e all-plugin volume" the msot preferred way to run Tempest tests?16:02
karthikp_because some of my tests keep failing this way!16:03
*** ducttape_ has quit IRC16:03
smcginniskarthikp_: That looks correct. I believe that is what my CI runs, with some additional regex to skip certain tests.16:04
*** stvnoyes has quit IRC16:06
*** ducttape_ has joined #openstack-cinder16:06
karthikp_smcginnis: I see...it is runnijg the tests in serial...i thought concurrent was the default16:06
*** stvnoyes has joined #openstack-cinder16:06
smcginniskarthikp_: I thought it defaulted to the number of cores.16:17
duleksmcginnis: Correct, it is.16:18
dulekSo I'm running mine through: "tempest run --regex tempest.api.volume"16:18
smcginniskarthikp_: You can override that with "tox -e all-plugin -- volume --concurrency=n" I think.16:19
smcginnisdulek: OK, good. That would be odd if that changed.16:19
*** dulek has left #openstack-cinder16:19
*** dulek has joined #openstack-cinder16:19
dulekIt is using the venv created by DevStack.16:19
*** xinli has joined #openstack-cinder16:20
*** kberger has joined #openstack-cinder16:25
*** alonma has quit IRC16:25
karthikp_smcginnis: dulek : Thanks!16:26
*** alonma has joined #openstack-cinder16:26
*** tlian has joined #openstack-cinder16:28
*** rcernin has quit IRC16:29
*** alonma has quit IRC16:31
*** x00350071_ has joined #openstack-cinder16:32
*** alonma has joined #openstack-cinder16:32
*** xiangxinyong has quit IRC16:35
*** alonma has quit IRC16:36
*** ducttape_ has quit IRC16:38
*** ducttape_ has joined #openstack-cinder16:39
*** alonma has joined #openstack-cinder16:40
openstackgerritJustin A Wilson proposed openstack/cinder master: Convert dict to OVO in backup export_record rpcapi  https://review.openstack.org/42539216:40
*** tesseract has quit IRC16:41
openstackgerritBob Ball proposed openstack/cinder master: Check for support on self, not for the method.  https://review.openstack.org/42818016:42
scottdabswartz: and others, you can sign up for interest in PTG discussion of Consistent Versioned Endpoints and the Service Catalog here: https://etherpad.openstack.org/p/ptg-architecture-workgroup16:43
scottdaAlso a talk for API Versioning16:43
scottdaand FYI, master etherpad for PTG is: https://wiki.openstack.org/wiki/PTG/Pike/Etherpads16:43
*** alonma has quit IRC16:44
*** knomura_ has quit IRC16:46
*** knomura has joined #openstack-cinder16:54
*** xyang has quit IRC16:56
*** knomura has quit IRC16:57
*** knomura has joined #openstack-cinder16:57
*** tlian has quit IRC16:58
*** alonmarx has joined #openstack-cinder17:00
openstackgerritBob Ball proposed openstack/cinder master: Convert ignoreskippedcluster check to be static  https://review.openstack.org/42818017:00
BobBallsmcginnis: I believe that the above is now working.  Thanks for your suggestions.17:01
smcginnisBobBall: Great, thanks!17:01
*** liverpooler has quit IRC17:02
*** liverpooler has joined #openstack-cinder17:03
*** hoonetorg has quit IRC17:03
*** Apoorva has joined #openstack-cinder17:03
*** Apoorva has quit IRC17:03
*** hoonetorg has joined #openstack-cinder17:04
*** xyang has joined #openstack-cinder17:06
*** xyang has quit IRC17:07
*** knomura has quit IRC17:08
*** raunak has joined #openstack-cinder17:08
*** kfarr has quit IRC17:10
*** akapil_ has joined #openstack-cinder17:15
*** xyang has joined #openstack-cinder17:15
*** xyang has quit IRC17:16
*** mriedem1 has joined #openstack-cinder17:17
*** mriedem has quit IRC17:17
*** mtanino_ has joined #openstack-cinder17:18
*** kfarr has joined #openstack-cinder17:19
*** mtanino__ has joined #openstack-cinder17:19
*** akapil has quit IRC17:19
*** mtanino has quit IRC17:19
*** timcl has quit IRC17:19
*** akapil_ has quit IRC17:20
*** mtanino_ has quit IRC17:22
*** timcl has joined #openstack-cinder17:24
*** raunak has quit IRC17:27
*** raunak has joined #openstack-cinder17:28
smcginnisjbernard: ping17:34
jbernardsmcginnis: heya17:34
smcginnisjbernard: Wondering if this is complete: https://blueprints.launchpad.net/cinder/+spec/rbd-replication17:34
openstackgerritMerged openstack/cinder master: Clean up release notes for Ocata  https://review.openstack.org/42789617:34
smcginnisjbernard: Or it there are any outstanding pieces to that yet.17:34
openstackgerritMerged openstack/cinder master: Make get_capabilities look for clusters  https://review.openstack.org/42772417:35
jbernardscottda: yes17:35
jbernardsmcginnis: yes17:35
jbernardscottda: sorry :)17:35
openstackgerritMerged openstack/cinder master: Fix host assignment when clustered  https://review.openstack.org/42735617:35
*** xyang has joined #openstack-cinder17:35
smcginnisjbernard: Yes, it's complete? Or yes, there's more to it? :)17:35
openstackgerritMerged openstack/cinder master: Add lock acquire/release debug messages  https://review.openstack.org/42576817:35
jbernardsmcginnis: complete ;)17:36
smcginnisjbernard: Cool, thanks! :)17:36
*** xyang has quit IRC17:37
smcginnisjgriffith: Anything done with this? https://blueprints.launchpad.net/cinder/+spec/make-lio-default-iscsi-helper17:37
*** hoonetorg has quit IRC17:38
*** jordanP has quit IRC17:38
*** chlong has joined #openstack-cinder17:39
*** ducttape_ has quit IRC17:39
*** hoonetorg has joined #openstack-cinder17:41
*** nkrinner is now known as nkrinner_afk17:41
*** mriedem1 is now known as mriedem17:43
*** aarefiev is now known as aarefiev_afk17:44
*** hoonetorg has quit IRC17:47
adrianofrjungleboyj: Hey Jay. We have a happy CI now :)   https://review.openstack.org/#/c/426878/17:47
jgriffithsmcginnis I'm using it as my default now :)17:52
jgriffithbut I don't think that's what you're looking for17:52
jgriffithsmcginnis short answer "no"17:52
jgriffithsmcginnis I haven't come up with a way to migrate17:52
smcginnisjgriffith: OK, that's what I thought but wanted to make sure.17:52
*** sudipto_ has quit IRC17:53
*** sudipto has quit IRC17:53
jgriffiththe IET TGT version was easy it turns out because they can co-exist17:53
jgriffithLIO and TGT not so much17:53
jgriffithat least I haven't had luck trying it17:53
smcginnisjgriffith: Oh well. At least we have that out there, so maybe some day a solution will present itself.17:54
*** Apoorva has joined #openstack-cinder17:54
jgriffithI'm going to harass eharney in ATL to see if he has ideas17:54
*** mjura has quit IRC17:54
jgriffithsince it's been default in RHEL for a while it's kinda stupid for us to be stuck17:55
jgriffithor we could make it default, except if targets exist (ie it's an upgrade)17:55
*** jose-phillips has joined #openstack-cinder17:55
smcginnisjgriffith: That seems reasonable. New deployments get it, existing deployments aren't affected.17:56
*** lpetrut has quit IRC17:56
jgriffith@smcginnis yeah, that's the best I've come up with so far17:56
*** xyang has joined #openstack-cinder17:57
*** hoonetorg has joined #openstack-cinder17:59
jungleboyjadrianofr, That is good news.17:59
*** xyang has quit IRC18:00
*** raunak has quit IRC18:01
SwansonSo if policy for consistency groups is nobody (default) should I actually be able to create one?18:02
*** kfarr has quit IRC18:02
*** raunak has joined #openstack-cinder18:03
smcginnisSwanson: policy.conf?18:04
Swansonya18:05
Swansonsmcginnis, Seems like it should have failed.18:05
*** xyang has joined #openstack-cinder18:06
adrianofrjungleboyj:  ;)18:08
*** knomura has joined #openstack-cinder18:09
smcginnisSwanson: Paste of your policy.conf?18:09
Swansonsmcginnis, I've changed it since.18:11
SwansonIt was all like this ...18:11
Swanson    "consistencygroup:create" : "group:nobody",18:11
Swanson    "consistencygroup:delete": "group:nobody",18:11
Swanson    "consistencygroup:update": "group:nobody",18:11
Swanson    "consistencygroup:get": "group:nobody",18:11
Swanson    "consistencygroup:get_all": "group:nobody",18:11
*** dhinesh has joined #openstack-cinder18:11
Swansongroup:create was set to "", tho.18:12
openstackgerritLee Yarwood proposed openstack/cinder master: WIP - volume: Recreate the original volume attachments during retype  https://review.openstack.org/42833118:12
smcginnisSwanson: Yeah, I thought that meant it would be blocked. Hmm.18:12
Swansonsmcginnis, I told you xyang broke groups.18:13
*** knomura has quit IRC18:13
smcginnis:)18:14
*** akapil has joined #openstack-cinder18:14
*** e0ne has joined #openstack-cinder18:19
xyangSwanson: what did I do?:)18:22
*** ericyoung has joined #openstack-cinder18:23
smcginnisxyang: Sounds like maybe we have a missing policy check on the CG create call?18:25
*** akapil has quit IRC18:26
xyang@smcginnis: I joined late so I didn't see what happened18:26
xyang@smcginnis: were you able to create groups with group:nobody?18:26
smcginnisSwanson: group:nobody or blank?18:26
xyangare  you talking about settings for "consistencygroup" or "group"?18:27
xyangwhich command exactly were you using?  CG or groups command?18:28
smcginnisOh sure, now he's hiding on us.18:28
xyang:)18:28
xyangthe bot went offline:)18:28
smcginnishaha18:28
patrickeastfwiw blank policy == ok to do stuff18:29
patrickeastgroup:nobody and it wont work18:30
smcginnispatrickeast: Ah, thanks. I thought blank meant no.18:31
xyangpatrickeast: you are right.  I was wondering which command he was using18:31
smcginnisHaven't had need to play around with policies much.18:31
jungleboyjsmcginnis, I had that confusion the other day too.  Glad it wasn't just me.18:31
smcginnisjungleboyj: ;)18:31
jungleboyjsmcginnis, great minds.  ;-)18:34
*** mvk has quit IRC18:35
xyang@smcginnis: is there a fix for that supports_pvs_ignoreskippedcluster issue?18:36
smcginnisxyang: Yeah, one sec, I'll find it.18:37
xyangok18:37
smcginnishttps://review.openstack.org/#/c/428180/18:37
xyang@smcginnis: great, thanks!18:38
smcginnisnp18:38
*** ducttape_ has joined #openstack-cinder18:40
*** Jack_Iv has quit IRC18:40
*** salv-orlando has joined #openstack-cinder18:41
*** alonma has joined #openstack-cinder18:41
*** Jack_Iv has joined #openstack-cinder18:41
*** ducttape_ has quit IRC18:44
*** alonma has quit IRC18:45
*** alonma has joined #openstack-cinder18:47
*** xyang has quit IRC18:48
*** xyang has joined #openstack-cinder18:50
*** alonma has quit IRC18:51
*** shubham0d has joined #openstack-cinder18:59
Swansonxyang, consisgroup19:00
xyangSwanson: does it work now?19:00
*** eharney has quit IRC19:00
xyangSwanson: or you were saying the policy check was missing?19:00
*** yumiriam has quit IRC19:00
SwansonIt let me create it with group:nobody set for all consisgroup stuff except the group:create19:01
*** ociuhandu has quit IRC19:01
xyangSwanson: "group:create" is policy for groups API though19:01
xyangSwanson: I thought you were using CG API?19:01
Swansonxyang, I am. Just noting it. I didn't change my policy settings from devstack default. Created the group and then remembered I was supposed to change the policy.19:02
Swansonxyang, I don't think the cg group should have been created.19:02
xyangSwanson: let me test it.19:03
*** raunak has quit IRC19:06
smcginnisSwanson: I think we figured out - if the policy was blank (not *:nobody) then that means it is allowed.19:06
Swansonsmcginnis, it wasn't blank.19:07
xyangSwanson, @smcginnis: looks like a bug19:07
xyangSwanson, @smcginnis: I can create it with "nobody" too.  but I can't list it19:07
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Disable multiattach for all drivers  https://review.openstack.org/42836519:08
*** lpetrut has joined #openstack-cinder19:08
*** porrua has quit IRC19:08
hemnaildikov, scottda jgriffith smcginnis ^^19:08
SwansonYeah, I couldn't list it either. Once I changed the policy it was there.19:08
xyangSwanson, @smcginnis: It was introduced by the migration CG changes19:08
*** knomura has joined #openstack-cinder19:10
scottdahemna: We probably don't want to merge that in Ocata....might want to WIP just to be sure19:10
*** raunak has joined #openstack-cinder19:11
hemnaare you sure?19:11
hemnathe flag really doesn't do much today other than allow you to create a volume with multi attach capability19:12
hemnabut nova doesn't even allow it19:12
*** cknight has joined #openstack-cinder19:12
hemnare: it doesn't work19:12
scottdaSome people have hacks that use multiattach today19:12
jungleboyjscottda, True, PowerVC is doing that.19:12
jungleboyjDoesn't mean that we shouldn't explicitly disable it and then maybe make people aware, if they have a hack, they are going to want to re-enable it?19:13
scottdamaybe, but that's why I'd wait until Pike19:13
hemnait doesn't really work now19:13
smcginnisYeah, I see questions every once and awhile from people actually using it. Probably safer to wait until Pike.19:14
jungleboyj*shrug*  Don't have a strong feeling.19:14
hemnaok marked it as tip19:14
*** knomura has quit IRC19:14
hemnawip19:14
hemnadamn xchat autocorrect19:14
jungleboyjJust the TIP?19:17
smcginnisdulek: Don't suppose you're still around. Wondering your thoughts on my comment here: https://review.openstack.org/#/c/354683/19:17
hemnajust19:17
*** ksumit has joined #openstack-cinder19:19
*** shubham0d has quit IRC19:21
*** Jack_Iv has quit IRC19:24
*** Jack_Iv has joined #openstack-cinder19:28
*** ociuhandu has joined #openstack-cinder19:28
*** jungleboyj has quit IRC19:30
scottdaAlright, something is definately wrong with my Tempest runs....19:31
scottdaI get 21 failures with just vanilla devstack and zero changes19:31
hemnaoops19:31
*** ducttape_ has joined #openstack-cinder19:32
scottdaSo, I'm not sure the nova-using-cinder-v3 failures mean anything, since I'm getting the same with cinder v219:32
*** chris_morrell has joined #openstack-cinder19:33
smcginnisscottda: LVM backend?19:33
scottdayes19:34
scottdajust untouched devstack19:34
scottdaand "tempest run --regex tempest.api.volume"19:34
scottdaso, I must be doing something wrong in configuring tempest...19:35
*** xyang has quit IRC19:37
*** eharney has joined #openstack-cinder19:37
*** xyang has joined #openstack-cinder19:39
*** mtanino__ has quit IRC19:39
openstackgerritxing-yang proposed openstack/cinder master: Add back policy check for CG  https://review.openstack.org/42839619:41
xyangSwanson: hi, see if this one fixed the problem for you: https://review.openstack.org/#/c/428396/19:42
scottdaCrap, never moved over tempest.conf to /etc/tempest19:45
Swansonxyang, taking a look19:49
xyangok19:49
Swansonthanks!19:49
mtreinishscottda: do you have a link?19:50
scottdamtreinish: IT's just on my local VM19:50
mtreinishah, ok19:50
scottdabut thanks19:50
scottdamtreinish: You are an expert. Do you use 'tempest init' or just copy over tempest.conf or something else?19:51
*** kfarr has joined #openstack-cinder19:52
mtreinishscottda: well having written the tempest init workflow I should encourage you to use that. But personally I just run a test runner from the tempest repo (because old habits die hard)19:52
mtreinishI don't think we ever landed the patch in devstack that populates /etc/tempest/tempest.conf which would mean you'd have to do all those steps manually19:53
scottdaok, I'll try to start using your tempest init. My old habit will die hard as well...19:54
mtreinishgetting devstack and devstack-gate using tempest init and tempest run was my on my todo list before I had my employment hickups and took a couple months off19:54
scottdathanks19:54
mtreinishscottda: sure, np19:54
mtreinishscottda: if you need help with any of that feel free to ask :)19:54
scottdamtreinish: Will do, thanks19:55
*** raunak has quit IRC20:00
*** ChubYann has joined #openstack-cinder20:00
*** eric_young has joined #openstack-cinder20:05
*** eric_young has quit IRC20:05
*** ericyoung has quit IRC20:05
*** ericyoung has joined #openstack-cinder20:06
*** ericyoung has quit IRC20:07
*** ericyoung has joined #openstack-cinder20:07
*** xyang has quit IRC20:08
*** xyang has joined #openstack-cinder20:09
Swansonxyang, bug squashed20:09
*** salv-orlando has quit IRC20:09
xyangSwanson: cool:)20:10
*** knomura has joined #openstack-cinder20:11
*** Jack_Iv has quit IRC20:12
*** lpetrut has quit IRC20:15
*** knomura has quit IRC20:15
*** kfarr has quit IRC20:16
*** Jack_Iv has joined #openstack-cinder20:18
*** kfarr has joined #openstack-cinder20:18
*** Jack_Iv has quit IRC20:23
openstackgerritMerged openstack/cinder master: Convert ignoreskippedcluster check to be static  https://review.openstack.org/42818020:23
openstackgerritMerged openstack/cinder master: Add SUPPORTED flag to Lenovo iSCSI driver  https://review.openstack.org/42788620:24
*** Jack_Iv has joined #openstack-cinder20:25
*** Jack_Iv has quit IRC20:26
*** Jack_Iv has joined #openstack-cinder20:26
*** Jack_Iv has quit IRC20:26
*** crushil has quit IRC20:26
*** crushil has joined #openstack-cinder20:26
*** Jack_Iv has joined #openstack-cinder20:27
*** Jack_Iv has quit IRC20:28
SwansonWhen adding a volume to a consistency group on create_volume are we still getting consistencygroup_id sent down?20:28
xyangyes20:28
SwansonReally?20:28
*** Apoorva has quit IRC20:28
xyangyou still need to provide the id20:28
xyangthe CLI didn't change20:29
SwansonCause I think this volume.get('consistencygroup_id') is returning Zip.20:30
xyangdo you get group_id?20:31
SwansonA moment...20:31
*** kaisers_ has joined #openstack-cinder20:32
*** salv-orlando has joined #openstack-cinder20:35
*** kaisers has quit IRC20:35
SwansonI just dumped __dict__ and am seeing _obj_consistencygroup_id is None and _obj_group_id is my consisgroup_id sent in.20:35
Swansonxyang, ^^20:35
xyangthat's right because it is a group object being created20:36
xyangso you need to handle both consistencygroup_id and group_id in create_volume20:36
xyangif group_id is not there, check consistencygroup_id20:36
xyangCLI will show consistencygroup_id, but internally it is group_id20:37
*** mvk has joined #openstack-cinder20:38
smcginnisxyang: I wonder if we should duplicate that value into consistencygroup_id so it's backwards compatible for drivers that have not updated to the group construct yet.20:38
SwansonAh. So this was required in Ocata and not Pike?20:38
xyang@smcginnis: I can look into that.  for drivers that have not added new logic yet, it should still work20:38
smcginnisxyang: Well, not if we're hiding the ID somewhere they aren't looking.20:39
Swansonxyang, "it should still work" as a goal or it "it should still work" as in it should be working for me now?20:39
* smcginnis is starting to think he's going to be up late again waiting to cut the RC...20:40
xyangSwanson: I mean you should still be able to create CG, unless you are saying it does not work20:40
smcginnis:)20:40
Swansonxyang, it does not work20:40
xyangSwanson: in that case, we need to add that20:40
xyang@smcginnis: what is the deadline for cutting RC?20:41
Swansonxyang, smcginnis 10 minutes after the fix for this merges?20:41
*** chris_morrell has quit IRC20:42
smcginnisI had hoped to do that this morning, but since we've had so many good bug fixes in flight today I haven't even prep'd it yet.20:42
*** Jack_Iv has joined #openstack-cinder20:47
jgriffith@smcginnis RC2 ?20:49
jgriffith:)20:49
smcginnisjgriffith: Yep. I think it's always inevitable.20:49
xyang@smcginnis: this one is a little tricky.  I need to spend some time on it.  group_id is a key to a group and consistencygroup_id is a key to a consistencygroup.  so I need to be sure not to cause errors by having both20:50
*** zhonghua2 has quit IRC20:50
*** Jack_Iv has quit IRC20:51
*** zhonghua has joined #openstack-cinder20:52
*** alonma has joined #openstack-cinder20:55
*** alonma has quit IRC20:59
*** chris_morrell has joined #openstack-cinder20:59
*** akapil has joined #openstack-cinder21:00
*** harlowja has quit IRC21:00
smcginnisxyang: OK. We can backport it once we know it's good. No sense rushing it.21:02
xyang@smcginnis: ok, that's better.21:03
Swansonxyang, You want a launchpad bug for this?21:03
xyangSwanson: sure, thanks21:03
Swansonxyang, I'll have a short bug up in a moment.21:03
xyangok21:04
*** salv-orlando has quit IRC21:05
*** liverpooler has quit IRC21:06
*** _ducttape_ has joined #openstack-cinder21:06
*** alyson_ has quit IRC21:07
Swansonhttps://bugs.launchpad.net/cinder/+bug/166139321:07
openstackLaunchpad bug 1661393 in Cinder "During a volume create the consisgroup_id not set on the volume. Only the group_id is set." [Undecided,New]21:07
SwansonShort21:07
xyangSwanson: got it, thanks21:09
*** raunak has joined #openstack-cinder21:09
*** ducttape_ has quit IRC21:10
*** raunak has quit IRC21:10
*** dave-mccowan has quit IRC21:10
*** raunak has joined #openstack-cinder21:12
*** catintheroof has quit IRC21:14
*** xyang has quit IRC21:15
*** catintheroof has joined #openstack-cinder21:15
smcginnisscottda, hemna: Mind taking a look. Swanson validated it fixes the problem.21:15
scottdalooking..21:15
scottdawait, no patch up yet21:15
scottda:)21:15
smcginnisscottda: Oh, oops: https://review.openstack.org/#/c/428396/21:17
smcginnis:)21:17
*** xyang has joined #openstack-cinder21:17
scottdaWell, I couldn't find it when I clicked the bug, because it's not linked to the bug...21:17
scottdaso there's that.21:17
smcginnisHmm, I don't think we got a bug filed for that one.21:18
smcginnisProbably should have one.21:18
*** xyang has quit IRC21:18
smcginnisSwanson, xyang: Can we get a bug linked to the missing policy patch?21:18
scottdaahhhg...I accidently +A21:19
*** catintheroof has quit IRC21:19
*** Apoorva has joined #openstack-cinder21:19
scottdacan I undo that?21:19
smcginnisThat's fine. Given the timing, I'd like to have it going through. Maybe we can retractively get a bug filed and just comment with the patch that fixed it.21:20
scottdaok.21:20
SwansonJust flaunting the rules more and more.21:21
*** raunak has quit IRC21:23
*** _ducttape_ has quit IRC21:23
*** ducttape_ has joined #openstack-cinder21:24
*** ksumit has quit IRC21:24
*** ksumit has joined #openstack-cinder21:25
*** raunak has joined #openstack-cinder21:26
*** mriedem has quit IRC21:29
*** alonmarx has quit IRC21:33
*** Yogi1 has quit IRC21:43
*** alonmarx has joined #openstack-cinder21:45
*** crose has quit IRC21:45
*** dave-mccowan has joined #openstack-cinder21:47
bswartzscottda: to un-workflow a change, you have to push another patchset to it -- a dummy patchset is fine21:47
scottdabswartz: Thanks for the tip21:48
bswartzI usually just append "REMOVE FROM GATE" to the end of the commit message, and then let the author fix it before it gets workflowed correctly21:49
smcginnisYou can just -A as well. If it was a Workflow-1 it will not merge it.21:50
bswartzsmcginnis: unless they've improved things since I last tried that's not true21:50
smcginnisbswartz: I've done. I don't remember when that was though.21:51
*** cknight has quit IRC21:52
*** porrua has joined #openstack-cinder21:55
scottdaDo tempest tests for the check queue use any concurrency?21:56
scottdaI keep getting failures on pristine, unchanged devstack.21:57
scottdaBut they don't fail when I run the tests singly.21:57
*** salv-orlando has joined #openstack-cinder21:57
smcginnisMy CI is mostly stable, so devstack and tempest _should_ be OK.21:58
scottdaYeah, I'm sure it's me. Just cannot find root cause21:59
SwansonSo after tonite master is pike?22:01
*** dhinesh has quit IRC22:01
*** xyang has joined #openstack-cinder22:02
smcginnisSwanson: I'll qualify that with something more specific: Once the stable/ocata branch is created, master is pike.22:02
SwansonThat is created around the RC1 label, right?22:03
*** xyang has quit IRC22:03
smcginnisYeah, when I do the request I can request the branch is created, though it doesn't necessarily have to be done together.22:03
openstackgerritSilvan Kaiser proposed openstack/cinder master: Removes getfattr from Quobyte Cinder driver  https://review.openstack.org/42783322:04
smcginnisZuul claims the last Cinder patch in the gate queue should merge in 35 minutes. So in a couple hours when that actually does merge I will probably request it.22:04
*** akapil has quit IRC22:07
SwansonI see what you did there.22:08
*** chris_morrell has quit IRC22:10
openstackgerritNicholas Jones proposed openstack/cinder master: Fix Block Storage API reference nesting  https://review.openstack.org/42847022:11
*** knomura has joined #openstack-cinder22:12
*** breitz has quit IRC22:14
*** salv-orl_ has joined #openstack-cinder22:15
*** jungleboyj has joined #openstack-cinder22:16
*** knomura has quit IRC22:17
*** xyang has joined #openstack-cinder22:18
*** salv-orlando has quit IRC22:18
*** crushil has quit IRC22:19
*** ksumit has quit IRC22:21
jungleboyjsmcginnis, Anything you need more eyes on before cutting RC?22:22
smcginnisjungleboyj: Nah, I think we're probably good. I'm sure we'll have a few backports that will require an RC-2. But nothing close enough to try to rush in today.22:22
SwansonRC-2 is, what? 12 hours after RC-1?22:23
jungleboyjSwanson, You have been on this project long enough to know ...22:24
smcginnisFinal RCs need to be done by the week of the 13th to be actually released the week of the 20.22:24
SwansonRelease 45 minutes after that. Short cycles suck.22:24
*** mriedem has joined #openstack-cinder22:24
xyang@smcginnis, scotta: https://bugs.launchpad.net/cinder/+bug/166141022:25
openstackLaunchpad bug 1661410 in Cinder "CG policy check is missing" [High,Fix committed] - Assigned to Xing Yang (xing-yang)22:25
smcginnisxyang: Thanks for getting the filed. That will help if anyone else comes looking for it.22:25
*** xinli has quit IRC22:25
xyangsure22:25
*** harlowja has joined #openstack-cinder22:25
*** breitz has joined #openstack-cinder22:26
*** ducttape_ has quit IRC22:26
*** tlian has joined #openstack-cinder22:30
*** ducttape_ has joined #openstack-cinder22:31
*** harlowja has quit IRC22:32
*** harlowja has joined #openstack-cinder22:33
*** dustins has quit IRC22:39
*** erlon has quit IRC22:40
*** ducttape_ has quit IRC22:43
*** kaisers_ has quit IRC22:47
-openstackstatus- NOTICE: Restarting gerrit due to performance problems22:50
*** mtanino has joined #openstack-cinder22:51
scottdaThere goes smcginnis plans to sleep before midnight22:51
SwansonHis 35 minutes to completion mark 45 minutes ago is back up to an hour.22:52
*** salv-orl_ has quit IRC22:53
*** alonmarx has quit IRC22:53
*** lseki has quit IRC23:00
*** alonma has joined #openstack-cinder23:02
smcginnisNever fails...23:02
SwansonThat patch removing the emc folder should get through the system sometime after 10pm23:05
*** alonma has quit IRC23:06
jungleboyjYay Gerrit.  Crumbling under the pressure.23:16
*** xyang has quit IRC23:26
*** dave-mccowan has quit IRC23:28
*** ntpttr has quit IRC23:31
*** ntpttr has joined #openstack-cinder23:31
openstackgerritMerged openstack/cinder master: Add SUPPORTED flag to HP MSA driver  https://review.openstack.org/42788123:31
*** gouthamr has quit IRC23:33
*** tlian has quit IRC23:35
*** jungleboyj has quit IRC23:37
*** ksumit has joined #openstack-cinder23:45
*** salv-orlando has joined #openstack-cinder23:48
*** salv-orlando has quit IRC23:52
*** ksumit has quit IRC23:56

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