Thursday, 2016-06-16

*** openstack has joined #openstack-cinder05:59
*** bardia has joined #openstack-cinder06:00
*** laughterwym has quit IRC06:01
*** Poornima has joined #openstack-cinder06:06
baaterdiff version of cinder have some influence on sync ?06:07
openstackgerritRavi Shekhar Jethani proposed openstack/cinder: Fix db purge for volume_types FK constraint  https://review.openstack.org/33032806:08
*** TaylorHuang has joined #openstack-cinder06:12
*** lpetrut has quit IRC06:13
*** shausy has quit IRC06:14
*** shausy has joined #openstack-cinder06:15
*** rsjethani has joined #openstack-cinder06:18
rsjethaniHi, I am facing a weird issue with cinder tests06:19
rsjethanirunning tests on a single file or for a module is working06:20
rsjethanibut when i run whole test suit..it fails..even on master06:20
*** laughter_ has quit IRC06:23
*** laughterwym has joined #openstack-cinder06:24
*** lpetrut has joined #openstack-cinder06:24
*** arecknag has joined #openstack-cinder06:24
*** rcernin has joined #openstack-cinder06:25
*** EinstCrazy has quit IRC06:28
lixiaoy1baater: https://github.com/openstack/cinder/blob/master/cinder/service.py#L179 it updates db every report_interval(default 10s)06:28
lixiaoy1baater: https://github.com/openstack/cinder/blob/master/cinder/service.py#L51 do you unset the config paramter?06:29
*** EinstCrazy has joined #openstack-cinder06:29
lixiaoy1baater: from kilo, they did same thing to report service06:29
*** EinstCrazy has quit IRC06:30
*** EinstCrazy has joined #openstack-cinder06:30
*** adisky has joined #openstack-cinder06:33
*** lpetrut has quit IRC06:40
*** EinstCrazy has quit IRC06:41
openstackgerritMerged openstack/cinder: Netmask needs corersion to string  https://review.openstack.org/32957406:41
openstackgerritMerged openstack/cinder: Add test type clarification to devref  https://review.openstack.org/32780606:41
openstackgerritMerged openstack/cinder: Remove test_conf.py  https://review.openstack.org/32014406:41
openstackgerritHiroyuki Eguchi proposed openstack/cinder: Unify the type of value for what are True/False values  https://review.openstack.org/33033906:43
*** EinstCrazy has joined #openstack-cinder06:45
openstackgerritCao ShuFeng proposed openstack/cinder: Fix size exceed of dd action when clearing a volume  https://review.openstack.org/32437606:48
*** salv-orlando has quit IRC06:49
openstackgerritCao ShuFeng proposed openstack/cinder: Fix size exceed of dd action when clearing a volume  https://review.openstack.org/32437606:50
*** salv-orlando has joined #openstack-cinder06:50
*** afazekas is now known as afazekas|dentist06:52
*** yangyape_ has joined #openstack-cinder06:54
*** salv-orl_ has joined #openstack-cinder06:55
openstackgerritMerged openstack/cinder: Give more fine grained access to DriverInitiatorData  https://review.openstack.org/32415506:55
openstackgerritMerged openstack/cinder: Remove locks from Pure volume drivers  https://review.openstack.org/32415606:56
*** yangyapeng has quit IRC06:56
openstackgerritMerged openstack/cinder: Enable str2size() to handle long int for i386  https://review.openstack.org/30132606:56
rsjethaniHi, I am facing a weird issue with cinder tests06:56
rsjethanirunning tests on a single file or for a module is working06:56
rsjethanibut when i run whole test suit..it fails..even on master06:56
*** salv-orlando has quit IRC06:57
openstackgerrithaobing proposed openstack/cinder: Allow setting CG name or description to empty value  https://review.openstack.org/32446506:58
*** bardia has quit IRC06:59
*** markvoelker_ has quit IRC07:01
*** markvoelker has joined #openstack-cinder07:01
lixiaoy1rsjethani: you'd better paste the error message07:01
*** yangyapeng has joined #openstack-cinder07:06
*** yangyapeng has quit IRC07:06
*** yangyapeng has joined #openstack-cinder07:07
*** yangyape_ has quit IRC07:08
*** tesseract has joined #openstack-cinder07:09
*** jamielennox is now known as jamielennox|away07:12
*** bharathk has quit IRC07:16
*** bharathk has joined #openstack-cinder07:17
*** akapil has joined #openstack-cinder07:18
*** permalac has quit IRC07:19
openstackgerritwanghao proposed openstack/python-cinderclient: Fixing parsing problem of cascade in client  https://review.openstack.org/32080907:20
openstackgerritwanghao proposed openstack/python-cinderclient: Fixing parsing problem of cascade in client  https://review.openstack.org/32080907:21
*** locust has quit IRC07:24
*** GB21 has quit IRC07:31
*** akapil has quit IRC07:33
*** akapil has joined #openstack-cinder07:34
*** akapil has quit IRC07:37
*** akapil has joined #openstack-cinder07:38
*** akapil has quit IRC07:43
*** akapil has joined #openstack-cinder07:43
*** bharat_ has joined #openstack-cinder07:44
duleksmcginnis: Too lateā€¦ ;) I'll compare DB schemas generated by both anyway.07:44
*** baater has quit IRC07:47
*** salv-orl_ has quit IRC07:49
*** salv-orlando has joined #openstack-cinder07:49
*** akapil has quit IRC07:51
flip214reviews of https://review.openstack.org/#/c/329905/ would be appreciated... thanks a lot in advance!07:56
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** bharat_ has quit IRC08:00
*** lpetrut has joined #openstack-cinder08:07
*** permalac has joined #openstack-cinder08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** lpetrut1 has joined #openstack-cinder08:12
*** akapil has joined #openstack-cinder08:12
*** lpetrut has quit IRC08:12
*** lpetrut1 is now known as lpetrut08:12
*** e0ne has joined #openstack-cinder08:13
*** akapil has quit IRC08:14
*** Julien-zte has joined #openstack-cinder08:15
*** zhongjun_ has quit IRC08:21
*** akapil has joined #openstack-cinder08:21
*** zhongjun_ has joined #openstack-cinder08:29
*** GB21 has joined #openstack-cinder08:32
*** savihou has joined #openstack-cinder08:34
*** jordanP has joined #openstack-cinder08:34
savihoue0ne: Hi Ivan, around ?08:36
e0nesavihou: hi, yes08:36
savihoue0ne: PM'ed you08:37
*** laughterwym has quit IRC08:47
*** zhonghua-lee has quit IRC08:47
*** laughterwym has joined #openstack-cinder08:47
*** laughter_ has joined #openstack-cinder08:50
*** laughterwym has quit IRC08:52
openstackgerritYuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id  https://review.openstack.org/31567308:52
*** laughter_ has quit IRC08:54
*** salv-orlando has quit IRC08:55
*** salv-orlando has joined #openstack-cinder08:56
*** akapil has quit IRC09:01
openstackgerritMichal Dulko proposed openstack/cinder: Add ConsistencyGroupVolumeTypeMapping table  https://review.openstack.org/24585409:04
openstackgerritMichal Dulko proposed openstack/cinder: Add generic code for online data migrations  https://review.openstack.org/33039109:04
openstackgerritnikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver  https://review.openstack.org/32728509:06
openstackgerritMichal Dulko proposed openstack/cinder: Add ConsistencyGroupVolumeTypeMapping table  https://review.openstack.org/24585409:09
openstackgerritningwei proposed openstack/cinder: Added backend driver for huawei FusionStorage  https://review.openstack.org/32888709:11
*** bhavik has quit IRC09:20
*** lprice has quit IRC09:22
*** bhavik has joined #openstack-cinder09:24
*** Divya has joined #openstack-cinder09:25
openstackgerritSzymon Borkowski proposed openstack/cinder: Convert backup_device to OVO  https://review.openstack.org/32933209:25
*** ihrachys has joined #openstack-cinder09:26
*** rsjethani has quit IRC09:27
*** mkoderer__ has quit IRC09:28
*** bhavik has quit IRC09:28
*** lprice has joined #openstack-cinder09:29
*** akapil has joined #openstack-cinder09:32
*** akapil has quit IRC09:33
*** akapil has joined #openstack-cinder09:34
*** yhayashi has quit IRC09:35
*** nikeshm has joined #openstack-cinder09:38
*** ociuhandu has quit IRC09:39
*** rsjethani has joined #openstack-cinder09:41
openstackgerritHiroyuki Eguchi proposed openstack/cinder: Unify the type of value for what are True/False values  https://review.openstack.org/33033909:43
*** bhavik has joined #openstack-cinder09:46
openstackgerritCao ShuFeng proposed openstack/cinder: Support None value of extra_specs in cinder-scheduler filter  https://review.openstack.org/32859709:58
*** cheneydc has quit IRC10:01
nikeshmDuncanT: hi10:03
*** akapil has quit IRC10:04
nikeshmDuncanT: if you get time please review https://review.openstack.org/#/c/32728510:04
duleksmcginnis: I've commented on https://review.openstack.org/#/c/324897. It's failing in a case where older code wasn't.10:06
duleksmcginnis: Sorry I couldn't review it yesterday, it was 10:30 PM my time.10:07
*** salv-orlando has quit IRC10:09
*** jamielennox|away is now known as jamielennox10:15
*** ociuhandu has joined #openstack-cinder10:25
*** zhongjun_ has quit IRC10:26
*** mvk_ has quit IRC10:26
*** zhongjun_ has joined #openstack-cinder10:26
*** iceyao_ has joined #openstack-cinder10:28
*** iceyao has quit IRC10:32
*** iceyao_ has quit IRC10:33
*** Divya has quit IRC10:36
*** winston-d_ has quit IRC10:37
*** akapil has joined #openstack-cinder10:38
*** akapil has quit IRC10:38
*** sdake has joined #openstack-cinder10:40
*** e0ne has quit IRC10:41
*** e0ne has joined #openstack-cinder10:41
*** sdake_ has joined #openstack-cinder10:42
*** sdake has quit IRC10:45
*** akapil has joined #openstack-cinder10:50
*** akapil has quit IRC10:51
*** sborkows has joined #openstack-cinder10:51
*** mvk_ has joined #openstack-cinder10:53
*** GB21 has quit IRC10:53
*** akapil has joined #openstack-cinder10:55
*** akapil has quit IRC10:56
*** cdelatte has joined #openstack-cinder10:57
*** tyrola has joined #openstack-cinder10:58
huyangkindly review: https://review.openstack.org/#/c/325682/11:01
*** salv-orlando has joined #openstack-cinder11:07
*** tyrola has quit IRC11:09
*** salv-orlando has quit IRC11:10
*** sdake_ has quit IRC11:10
*** salv-orlando has joined #openstack-cinder11:11
*** kfarr has joined #openstack-cinder11:21
*** etraitel has joined #openstack-cinder11:22
etraitelHi, we're playing with cinder create --metadata field, we noticed it doesn't work on RedHat OpenStack 8 (claims to be Liberty, cinder api 7.0.1) but works on devstack (cinder api 8.0.0). Isn't devstack Liberty? In which cinder version was --metadata introduced?11:29
duleketraitel: DevStack without explicit setting installs master.11:30
duleketraitel: You're getting error from c-api or python-cinderclient?11:30
*** kaisers1 has joined #openstack-cinder11:34
*** e0ne has quit IRC11:35
etraitelGot it. We're not getting an error, it just seems to ignore it11:36
duleketraitel: Use --debug on cinder command to see if it's sent over REST.11:38
*** kaisers1 has quit IRC11:44
*** salv-orlando has quit IRC11:45
*** kaisers1 has joined #openstack-cinder11:45
openstackgerritningwei proposed openstack/cinder: Added backend driver for huawei FusionStorage  https://review.openstack.org/32888711:46
*** salv-orlando has joined #openstack-cinder11:46
*** GB21 has joined #openstack-cinder11:48
*** iceyao has joined #openstack-cinder11:49
*** Julien-zte has quit IRC11:49
*** ducttape_ has quit IRC11:54
*** ducttape_ has joined #openstack-cinder11:54
*** laughter_ has joined #openstack-cinder11:58
*** bharathk has quit IRC12:00
*** akapil has joined #openstack-cinder12:01
*** sdake has joined #openstack-cinder12:05
*** akapil has quit IRC12:06
*** akapil has joined #openstack-cinder12:06
*** ducttape_ has quit IRC12:07
openstackgerritxiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils  https://review.openstack.org/33050212:15
openstackgerritxiexs proposed openstack/cinder: Make volume-image-metadata to support None type  https://review.openstack.org/33050312:15
*** GB21 has quit IRC12:15
*** GB21 has joined #openstack-cinder12:17
*** adrianofr has joined #openstack-cinder12:18
*** timcl has joined #openstack-cinder12:22
*** rcernin has quit IRC12:24
*** kfarr has quit IRC12:28
*** ducttape_ has joined #openstack-cinder12:36
*** ducttape_ has quit IRC12:38
*** rcernin has joined #openstack-cinder12:39
*** julim has joined #openstack-cinder12:39
*** e0ne has joined #openstack-cinder12:40
*** zul_ has joined #openstack-cinder12:41
e0negeguileor: hi. are you around?12:42
*** zul_ has quit IRC12:42
*** zul_ has joined #openstack-cinder12:42
*** zul_ has quit IRC12:43
*** zul_ has joined #openstack-cinder12:44
*** eharney has quit IRC12:44
*** edmondsw has joined #openstack-cinder12:45
*** zul_ has quit IRC12:49
*** zul has quit IRC12:49
*** zul has joined #openstack-cinder12:50
*** Julien-zte has joined #openstack-cinder12:52
*** GB21 has quit IRC12:53
*** gcb has quit IRC12:53
*** akapil has quit IRC12:53
*** rcernin has quit IRC12:54
*** akapil has joined #openstack-cinder12:54
*** gouthamr has joined #openstack-cinder12:58
*** pauloewerton has joined #openstack-cinder12:58
*** rlrossit has joined #openstack-cinder13:00
*** gouthamr_ has joined #openstack-cinder13:04
*** timcl1 has joined #openstack-cinder13:05
*** timcl has quit IRC13:06
*** gouthamr has quit IRC13:07
*** rcernin has joined #openstack-cinder13:07
duleke0ne: AFAIK geguileo_phone is on PTO now.13:08
*** e0ne has quit IRC13:09
*** gouthamr has joined #openstack-cinder13:11
*** pgadiya has quit IRC13:11
*** e0ne has joined #openstack-cinder13:12
*** hemanthm_ is now known as hemanthm13:14
*** timcl has joined #openstack-cinder13:14
*** gouthamr_ has quit IRC13:15
*** timcl1 has quit IRC13:15
*** faiz89 has joined #openstack-cinder13:16
*** eharney has joined #openstack-cinder13:17
*** mriedem has joined #openstack-cinder13:19
*** dustins has joined #openstack-cinder13:19
*** EinstCrazy has quit IRC13:20
*** salv-orlando has quit IRC13:21
*** yangyape_ has joined #openstack-cinder13:21
*** salv-orlando has joined #openstack-cinder13:21
*** EinstCrazy has joined #openstack-cinder13:21
*** merooney has joined #openstack-cinder13:22
*** bharath has joined #openstack-cinder13:22
*** pgadiya has joined #openstack-cinder13:23
*** yangyape_ has quit IRC13:23
*** yangyapeng has quit IRC13:24
openstackgerritGyorgy Szombathelyi proposed openstack/cinder: Use keystoneauth1 for creating the keystone session  https://review.openstack.org/31849013:25
*** akshai has joined #openstack-cinder13:25
*** pgadiya has quit IRC13:25
*** EinstCrazy has quit IRC13:26
*** akshai_ has joined #openstack-cinder13:27
*** raildo-a` is now known as raildo13:27
*** diogogmt has quit IRC13:29
*** frontrunner has joined #openstack-cinder13:30
geguileo_phonee0ne I'm on PTO on an island with limited internet connection13:31
*** akshai has quit IRC13:31
e0negeguileo_phone: don't worry. I'll ping you later. have a good vacation!13:31
geguileo_phoneI'll be back to civilization next Monday13:31
*** diogogmt has joined #openstack-cinder13:31
*** smoriya has quit IRC13:32
geguileo_phonee0ne Is it urgent?13:32
*** rlrossit has quit IRC13:32
e0negeguileo_phone: no13:32
geguileo_phoneOK, then I'll ping you as soon as I'm back13:32
*** mtanino has joined #openstack-cinder13:35
*** cknight has joined #openstack-cinder13:37
*** cknight1 has joined #openstack-cinder13:37
*** cknight has quit IRC13:37
*** timcl has quit IRC13:40
*** porrua has joined #openstack-cinder13:41
*** lprice has quit IRC13:42
*** Poornima has quit IRC13:42
*** ducttape_ has joined #openstack-cinder13:43
smcginnisdulek: Thanks, I figured it was a long shot you were still around. Thanks for still looking at that. Reading your comments now.13:44
eharneydulek: smcginnis:  https://bugs.launchpad.net/cinder/+bug/1593175  has me thinking we should just revert the db compaction before it's out there and do it again, what do you think?13:44
openstackLaunchpad bug 1593175 in Cinder "Databases created before and after compacting migrations are different" [Critical,New]13:44
dulekameade: I've seen your comment. Rebase successful? :)13:45
smcginniseharney: I would hope it's just easy enough to fix.13:45
smcginniseharney: Haven't looked close enough, but really should be fairly minor I would hope.13:45
ameadedulek: yessir, thank you, setting up an environment now13:45
eharneysmcginnis: great, if you think so -- i didn't have much of a guess there13:45
smcginniseharney: Will see if I change my mind once I look closer. ;)13:45
eharneysmcginnis: if going that route, we have to do it rather quickly IMO13:46
smcginniseharney: Yes, just got to my desk and saw this. Top priority for me this morning.13:46
duleksmcginnis, eharney: I would expect anyone running trunk deployments would have an old DB migrated instead of creating a new one.13:46
smcginnisdulek: True13:46
duleksmcginnis, eharney: So hopefully changing a migration isn't that bad.13:46
eharneyworks for me13:47
dulekBut of course if it will take time I would be much more comfortable with reverting that for a moment.13:47
*** akerr has joined #openstack-cinder13:48
smcginnisdulek, eharney: OK, if that's what you would prefer I'm fine with that.13:48
duleksmcginnis: Only if it will take a lot of time to fix that. :)13:48
*** dulek has left #openstack-cinder13:49
*** dulek has joined #openstack-cinder13:49
smcginnisdulek: Weird, looking at your diff now. I didn't see any of these diffs when I compared the old to new.13:49
smcginnisThis is comparing the initial schema definition without the post-kilo migrations to the state of kilo?13:49
duleksmcginnis: Some more info on mysql_charset. Thing is that when running migrations one-by-one we were updating mysql_charset after each creation. Now we're doing it at the end and that's the difference.13:50
*** etraitel has quit IRC13:50
duleksmcginnis: No, in both cases it's from 0 to trunk.13:50
duleksmcginnis: But with and without your commit.13:50
smcginnisdulek: But it is set. Is MySQL behaving differently based on when it is set?13:50
smcginnisdulek: From 0 to trunk meaning running all migrations?13:51
duleksmcginnis: At least in my env it's failing on creating a volumes table unless I specify mysql_charset.13:51
duleksmcginnis: Yes, all migrations.13:51
*** ducttape_ has quit IRC13:52
smcginnisdulek: But there were changes after Kilo, so it should be different.13:52
smcginnisdulek: Maybe I'm not following.13:52
openstackgerritSean McGinnis proposed openstack/cinder: Revert "Compact Kilo database migrations"  https://review.openstack.org/33057013:52
duleksmcginnis: My perception is that someone creating a DB with your commit will have a little different DB than someone upgrading it continously.13:52
*** akapil has quit IRC13:53
dulekAw, that sounds confusing. ;)13:53
smcginnisdulek: But if you're comparing to the state at Kilo?13:53
duleksmcginnis: So I've loaded master, dropped the DB, created a new one and run cinder-manage DB sync.13:54
openstackgerritMerged openstack/cinder: VMware: Deprecate vCenter version 5.1  https://review.openstack.org/32818913:54
dulekLast migration - 74.13:54
dulekNow I've got back in git to a moment just before your commit and repeated the procedure.13:54
dulekAlso ended up on 74.13:55
dulekI've ended up with two cases of creating a DB from scratch, right?13:55
smcginnisdulek: OK, good. That's what I was hoping you were comparing to, but it sounded like Kilo, which I wouldn't have thought you would compare to but just making sure.13:55
smcginnisdulek: Yeah, that's a good case. Basically what I thought I had done though, so I'm a little confused at this point.13:55
duleksmcginnis: So you've did similar comparison without experiencing my results.13:56
smcginnisBut not too surprised if I had missed something along the way. Let's get the revert through and I'll sort it out.13:56
smcginnisdulek: Yes13:56
duleksmcginnis: MySQL version? OS version? SQLA versionā€¦?13:56
eharneysmcginnis: can you stick a Related-Bug in the commit message just so we can keep track of this later?13:56
dulekAny of these could have caused it on my env.13:56
*** akapil has joined #openstack-cinder13:56
smcginnisdulek: Not sure now, but I'm certainly open to the possiblity that I screwed it up on my end. ;)13:56
smcginniseharney: Sure, good call.13:57
*** timcl has joined #openstack-cinder13:57
openstackgerritSean McGinnis proposed openstack/cinder: Revert "Compact Kilo database migrations"  https://review.openstack.org/33057013:57
* dulek hopes he isn't making all this fuss for nothing because of a mistake.13:57
*** tpsilva has joined #openstack-cinder13:58
smcginnisdulek: The likelihood it's my mistake and not yours is probably higher. :)13:58
*** nag_ has quit IRC13:58
*** kfarr has joined #openstack-cinder13:58
*** geguileo_mobile has joined #openstack-cinder13:59
*** jgregor has joined #openstack-cinder13:59
dulekI would expect it's 50-50. :P13:59
smcginnisI better grab some coffee before I go much further. :)14:00
*** geguileo_phone has quit IRC14:01
*** diogogmt has quit IRC14:01
*** rlrossit_ has joined #openstack-cinder14:01
*** iceyao has left #openstack-cinder14:03
*** sheel has quit IRC14:05
*** crose has joined #openstack-cinder14:07
*** arch-nemesis has joined #openstack-cinder14:08
*** arch-nemesis has quit IRC14:08
*** arch-nemesis has joined #openstack-cinder14:09
*** itzdilip has joined #openstack-cinder14:11
nikeshmsmcginnis patrickeast eharney: addressed the comments on https://review.openstack.org/#/c/327285/10, if you get time please have a look, CI also reporting14:12
smcginnisnikeshm: Opened a tab, probably won't be able to look until later.14:12
nikeshmare yu in india?14:13
smcginnisnikeshm: No, still in the US. I will be in Bangalore the first week of July.14:14
*** crose has quit IRC14:14
nikeshmcrucial time, since close to N114:15
nikeshmJul 08 - 09, 2016 OpenStack Days India14:15
smcginnisnikeshm: Yep, that's the one.14:16
*** arecknag has quit IRC14:18
*** bhavik has quit IRC14:21
*** kaisers_ has joined #openstack-cinder14:21
*** bharath has quit IRC14:21
nikeshmeharney: hi14:23
*** Lee1092 has quit IRC14:23
nikeshmeharney: do i have to change CONF.kaminario_nodedup_substring to self.configuration.kaminario_nodedup_substring14:24
*** laughter_ has quit IRC14:25
*** laughterwym has joined #openstack-cinder14:25
eharneynikeshm: yes because otherwise when people set it in the driver backend section in cinder.conf it won't behave as expected14:26
*** diablo_rojo has joined #openstack-cinder14:26
jgriffithsmcginnis: hmm... what deltas did you guys find?  I loaded it and did a quick check and it seemed ok14:26
dulekjgriffith: I've listed them here: https://bugs.launchpad.net/cinder/+bug/159317514:27
openstackLaunchpad bug 1593175 in Cinder "Databases created before and after compacting migrations are different" [Critical,New]14:27
smcginnisjgriffith: Me too. I'm spinning up a clean instance to do some more checking to make sure I didn't pollute it somehow.14:27
smcginnisI'm guessing that's probably what happened.14:27
jgriffithhmm14:27
smcginnisHonestly not too surprised if something was missed based on all the odd migrations we had in there.14:27
dulekjgriffith: If you've both compared SQL dumps of before and after and found no differences, then it can be my env's fault.14:28
nikeshmeharney:  i m not using this in code self.configuration.append_config_values(kaminario1_opts), so people can not set it in backend section14:28
*** jistr is now known as jistr|mtg14:28
jgriffithsmcginnis: yeah, it's so long overdue it's sort of a monumental thing14:28
smcginnisjgriffith: Yeah, we probably should have been doing this for a while.14:28
nikeshmeharney: i am only using CONF.register_opts(kaminario1_opts)14:28
jgriffithdulek: nah... good chance I didn't compare the dump on the default settings14:28
smcginnisAt least from this point forward (once I straighten this out) it should be trivial to keep up with.14:29
nikeshmeharney: so user can only set it in [DEFAULT]14:29
*** lprice has joined #openstack-cinder14:29
eharneynikeshm: i don't think that is true14:29
jgriffithdulek: smcginnis I think part of the issue may be the fact that we've changed our semantics for the migrations considerably over the years14:30
jgriffithdulek: smcginnis some of the things we used to do with default null etc have changed14:30
*** sheel has joined #openstack-cinder14:30
nikeshmeharney: it is giving error if i try to put kaminario_nodedup_substring in backend section since i am not using self.configuration.append_config_values(kaminario1_opts)14:31
smcginnisThere was some really odd conditional logic on nulls between sqlite and everything else in there. Not sure I entirely understood it, but thought I got it to at least be consistent.14:31
mriedemsmcginnis: speaking of retype bugs https://bugs.launchpad.net/nova/+bug/159305514:32
openstackLaunchpad bug 1593055 in OpenStack Compute (nova) "Retype an in-use volume failed in mitaka" [Undecided,Incomplete]14:32
nikeshmeharney: self.configuration.kaminario_nodedup_substring  is only working when i use self.configuration.append_config_values(kaminario1_opts)14:32
mriedemand remotefs14:32
smcginnismriedem: Looks like a retype that requires migration for an attached LVM volume.14:33
scottdamriedem: Thanks14:33
scottdamriedem: I'll work on a repro now....14:33
smcginnisscottda: Thanks!14:33
smcginnisIs "python setup.py install" broken?14:33
*** bbhavsar_ has joined #openstack-cinder14:34
*** akapil has quit IRC14:35
smcginnisHmm, "pip install ./" gets the same error for me...14:35
nikeshmeharney: NoSuchOptError: no such option in group kaminario-iscsi-dedup: kaminario_nodedup_substring14:35
eharneyin a meeting right now, bbiab14:36
dulek HexChat: 2.9.6 ** OS: Linux 3.16.0-38-generic x86_64 ** Distro: Debian jessie/sid ** CPU: 4 x Intel(R) Core(TM) i5-4300M CPU @ 2.60GHz (GenuineIntel) @ 3,23GHz ** RAM: Physical: 15,4GB, 8,9% free ** Disk: Total: 725,8GB, 77,7% free ** VGA: Intel Corporation 4th Gen Core Processor Integrated Graphics Controller ** Sound: HDA-Intel - HDA Intel HDMI1: HDA-Intel - HDA Intel PCH2: USB-Audio -14:42
dulek Logitech Wireless Headset29: ThinkPad EC - ThinkPad Console Audio Control ** Ethernet: Intel Corporation Ethernet Connection I217-LM ** Uptime: 5w 2d 11h 52m 7s **14:42
*** itzdilip has quit IRC14:42
*** karthikp_ has joined #openstack-cinder14:42
dulekWhoops, I've clicked something bad in my client. Sorry.14:43
eharneydulek: hey, i've done that trick before too... it's a sneaky button :)14:44
smcginnisdulek: We now know exactly what you're running. :)14:44
openstackgerritMerged openstack/cinder: Add debug level logs on OVO backports  https://review.openstack.org/32114314:44
smcginnisdulek: Good thing you don't have any AMD in there.14:44
eharneylol14:44
openstackgerritMerged openstack/cinder: Implement OVO class init mechanism  https://review.openstack.org/32114414:44
*** kaisers_ has quit IRC14:44
dulekYou even know how much disk I'm using. :D14:45
*** kmartin has joined #openstack-cinder14:45
*** karthikp_ has quit IRC14:47
*** kaisers_ has joined #openstack-cinder14:48
*** jungleboyj has joined #openstack-cinder14:48
*** tyrola has joined #openstack-cinder14:49
*** mriedem1 has joined #openstack-cinder14:50
*** mriedem has quit IRC14:51
*** akapil has joined #openstack-cinder14:52
*** karthikp_ has joined #openstack-cinder14:52
*** sdake has quit IRC14:54
*** akapil has quit IRC14:55
*** diogogmt has joined #openstack-cinder14:55
*** salv-orlando has quit IRC14:56
*** salv-orlando has joined #openstack-cinder14:57
*** mriedem1 is now known as mriedem14:57
*** EinstCrazy has joined #openstack-cinder14:58
*** markstur has joined #openstack-cinder14:58
*** sdake has joined #openstack-cinder14:59
*** salv-orlando has quit IRC15:00
eantyshevsmcginnis: Hello, may I ask for your review on this: https://review.openstack.org/261558 (This is a substantial refactoring initiated in Cinder unit tests, and it is ready for review for some time already)15:00
*** salv-orlando has joined #openstack-cinder15:00
smcginniseantyshev: Got a tab open for it. I probably can't get to it until later today, but I'll try to take a look soon.15:00
*** david-lyle_ is now known as david-lyle15:00
eantyshevsmcginnis: Thank you, this really needs some feedback!15:02
*** markstur has left #openstack-cinder15:02
jgriffithgeguileor: It's disturbing that a change like this ends up being >50% unit test changes  https://review.openstack.org/#/c/286598/2415:03
jgriffithgeguileor: I like the simplification15:03
jgriffithgeguileor: number of places we can benefit from this sort of cleanup15:04
*** cbader_ has quit IRC15:05
*** faiz89 has quit IRC15:07
dulekjgriffith: I have a concern on the previous patch in that chain: https://review.openstack.org/#/c/31554115:08
jgriffithdulek: I've just started looking at that one15:08
jgriffithdulek: suppose I should've started at the beginning :)15:08
dulekjgriffith: It's too magical to me. Like we're trying to squash different code fragments into one solution and we end up with too much special cases.15:09
jgriffithdulek: what's the concern?15:09
*** kaisers_ has quit IRC15:09
*** iceyao has joined #openstack-cinder15:09
jgriffithdulek: ahh... a man after my own heart :)15:09
dulekjgriffith: I wonder if you will share that when reviewing or it is just me.15:09
*** kaisers_ has joined #openstack-cinder15:10
jgriffithdulek: let me have a look15:10
*** mattymo_ is now known as mattymo15:10
jgriffithoh15:10
dulekjgriffith: BTW - you've mentioned https://github.com/openstack/cinder/commit/a8744e26ba665a9103f153bd516dd49d99fd0c2f#diff-1b317284bd7cf1f7415c1be6a919fee5R2964 yesterday.15:10
dulekjgriffith: Is there a problem with that?15:10
*** adisky has quit IRC15:10
jgriffithdulek: nah... I found what I was looking for15:10
dulekjgriffith: Okay, cool. :015:11
jgriffithdulek: the "int_no" was bothering me15:11
dulekSure. Different type of deleted column.15:11
jgriffithdulek: kind of to the point you just made, I'm kind of a simpleton15:11
jgriffithdulek: I like having a standard15:11
*** rsjethani has quit IRC15:11
jgriffithdulek: it took me a while to look at why/where some use True, some use "int_no" etc15:12
jgriffithdulek: my conclusion (perhaps wrong) was that it's more transitionary15:12
dulekjgriffith: It's because VolumeTypeProjects has deleted column as int, and not boolean.15:13
dulekjgriffith: There were at least 2 bugs related to that and we've end up with this.15:14
*** iceyao has quit IRC15:14
*** suyash has joined #openstack-cinder15:15
openstackgerritYuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id  https://review.openstack.org/31567315:16
jgriffithdulek: oh... crap15:16
jgriffithyeah... I see now15:16
jgriffithdulek: so there's another place where we could probably normalize the db again no?15:16
jgriffithdulek: easy enough to write a migration that updates existing ones15:17
dulekjgriffith: Probably, but this one is related to the constraint we have on that table.15:17
jgriffithdulek: also, frankly we haven't released VolumeTypeProjects so we could/should fix that15:17
dulekjgriffith: Released?15:18
e0nesmcginnis: just FYI, I've fixed issue with BDD (patch on review). the second path will be for tempest to make CI green15:18
jgriffithdulek: there's no release version of Cinder running out there that has that model15:18
smcginnise0ne: Yay! :)15:18
jgriffithdulek: it doesn't matter either way... just ignore that comment :)15:18
dulekjgriffith: git blame tells me that it's there since 2014ā€¦ :P15:19
*** jistr|mtg is now known as jistr15:19
e0nesmcginnis: there are  2 different issues: with tempest and cinder code:(15:19
jgriffithdulek: VolumeTypeGroup?15:19
smcginnise0ne: Glad to see some progress in addressing them. Thanks for working on that.15:19
jgriffithOhhh15:19
jgriffithprojects :)15:19
jgriffithderp15:19
jgriffithnever mind15:19
e0nesmcginnis: np15:19
dulek:)15:19
dulekhttps://github.com/openstack/cinder/blame/master/cinder/db/sqlalchemy/models.py#L26515:19
jgriffithdulek: yeah15:20
e0nesmcginnis: I'll try to setup some notifications for that job to fix asap if any failure15:20
smcginnise0ne: That would be good. I know it was failing for some time before I noticed.15:20
e0nesmcginnis: yep:(. it's broken almost 1.5 months:(15:21
*** akapil has joined #openstack-cinder15:21
smcginnis:[15:21
*** akapil has quit IRC15:21
jgriffithdulek: I added a comment to that patch15:22
jgriffithdulek: https://review.openstack.org/#/c/315541/1715:23
jgriffithgeguileor: dulek So I do like consolidation, but I think it can be taken too far15:23
*** akapil has joined #openstack-cinder15:24
*** akapil has quit IRC15:24
*** akapil has joined #openstack-cinder15:24
*** akapil has quit IRC15:24
dulekjgriffith: That was also my feeling. Okay, let's discuss with Gorka once he'll return from vacations. Maybe on the meeting.15:25
*** ihrachys has quit IRC15:25
* dulek got to leave office and start preparing to watch Poland-Germany football (soccer) match today. :)15:25
*** ihrachys_ has joined #openstack-cinder15:25
jgriffithdulek: cool... have a good nigth15:25
jgriffithnight15:26
duleksmcginnis: Will you figure out if I was wrong with my bug report?15:26
duleksmcginnis: On the compacted migrations.15:26
smcginnisdulek: Not yet. Getting weirdness trying to generate the before and after schema.15:26
duleksmcginnis: Oh, maybe I can help with that?15:26
smcginnisdulek: Even when I go back to the commit before, it's starting at migration version 45 now on my test host and I can't figure out why.15:26
duleksmcginnis: You need to reinstall Cinder python package?15:27
smcginnisdulek: I did. Then I completely blew all references to it I could find away. Same thing.15:27
duleksmcginnis: My another problem were .pyc files laying around.15:27
duleksmcginnis: And are you dropping whole Cinder DB?15:28
smcginnisdulek: That's what I thought, but did full "rm -fr /usr/local/lib/python2.7/dist-packages/cinder*" and still there.15:28
smcginnisJust using a local config file and generating a new one in the current directory.15:29
smcginniscinder-manage --config-file dbtest.conf db sync15:29
smcginnisGenerates to ./cinder.sqlite15:29
duleksmcginnis: Hm. I was actually running live MySQL.15:29
smcginnisdulek: Thought I'd start simple with sqlite, but maybe I'll just do that.15:30
smcginnisBut odd.15:30
*** faiz89 has joined #openstack-cinder15:30
smcginnisThink I'll spin up a new fresh VM to try the pre-compaction first.15:30
smcginnisThere are differences between sqlite and mysql, so I should do that too.15:30
dulekOkay. I'll try to take a look on IRC later today.15:32
*** geguileo_mobile has quit IRC15:33
*** openstackgerrit has quit IRC15:34
*** openstackgerrit has joined #openstack-cinder15:35
*** geguileo_phone has joined #openstack-cinder15:37
*** eharney_ has joined #openstack-cinder15:38
*** eharney has quit IRC15:39
nikeshmeharney: i replied in patch , please let me know if i need to change still15:40
nikeshmin the code15:40
nikeshmafter that explanation15:40
*** eharney_ is now known as eharney15:44
smcginnisdulek: Got things working. I am seeing some differences. Will work on figuring out what/why now.15:45
*** EinstCrazy has quit IRC15:53
*** jungleboyj has quit IRC15:53
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953115:55
*** belmoreira has joined #openstack-cinder15:56
*** alyson_ has joined #openstack-cinder15:56
*** e0ne has quit IRC15:56
*** fernnest has joined #openstack-cinder15:57
*** timcl has quit IRC15:57
*** garthb has joined #openstack-cinder15:58
*** akshai has joined #openstack-cinder15:58
*** Apoorva has joined #openstack-cinder15:58
*** tesseract has quit IRC16:00
*** mriedem1 has joined #openstack-cinder16:00
*** kaisers_ has quit IRC16:01
*** akshai_ has quit IRC16:01
*** mriedem has quit IRC16:02
*** shausy has quit IRC16:02
*** mriedem1 is now known as mriedem16:03
*** jgregor has quit IRC16:05
*** jungleboyj has joined #openstack-cinder16:06
*** tyrola has quit IRC16:06
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/32967716:07
*** akshai_ has joined #openstack-cinder16:10
*** akshai has quit IRC16:13
*** permalac has quit IRC16:15
*** sborkows has quit IRC16:18
*** sdake has quit IRC16:20
*** laughterwym has quit IRC16:23
*** enriquetaso has joined #openstack-cinder16:27
*** cbader has joined #openstack-cinder16:28
scottdaeharney: jgriffith Here's a patch to enable cinder multibackend tests: https://review.openstack.org/#/c/330678/16:28
*** sdake has joined #openstack-cinder16:28
scottdaAnd the main reason for that patch ^^^ is for testing cinder migration (retype --on-deman) with a volume attached (BFV): https://review.openstack.org/32668116:32
*** bharath has joined #openstack-cinder16:34
*** laughterwym has joined #openstack-cinder16:35
*** david-lyle has quit IRC16:35
*** faiz89 has quit IRC16:36
*** jordanP has quit IRC16:37
*** kfarr has quit IRC16:38
*** laughterwym has quit IRC16:39
*** Lee1092 has joined #openstack-cinder16:40
*** suyash has quit IRC16:41
*** geguileo_phone has quit IRC16:44
*** erlon has joined #openstack-cinder16:44
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546916:45
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407616:45
*** geguileo_phone has joined #openstack-cinder16:47
*** david-lyle has joined #openstack-cinder16:51
*** belmoreira has quit IRC17:00
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring  https://review.openstack.org/28546917:01
*** suyash has joined #openstack-cinder17:02
*** mdenny has joined #openstack-cinder17:06
*** mvk_ has quit IRC17:06
*** akshai_ has quit IRC17:06
*** salv-orl_ has joined #openstack-cinder17:08
*** timcl has joined #openstack-cinder17:09
*** salv-orlando has quit IRC17:12
*** bharath has quit IRC17:12
*** bharath has joined #openstack-cinder17:13
*** rcernin has quit IRC17:21
*** eharney has quit IRC17:23
*** bauzas has quit IRC17:23
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407617:24
*** bauzas has joined #openstack-cinder17:24
*** ihrachys_ has quit IRC17:24
*** geguileo_phone has quit IRC17:25
*** faiz89 has joined #openstack-cinder17:26
*** lpetrut has quit IRC17:27
*** akshai has joined #openstack-cinder17:28
*** angela-s has joined #openstack-cinder17:28
*** eharney has joined #openstack-cinder17:28
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407617:32
*** xyang1 has joined #openstack-cinder17:33
*** geguileo_phone has joined #openstack-cinder17:34
*** ChubYann has joined #openstack-cinder17:38
*** xinli has joined #openstack-cinder17:38
*** mvk_ has joined #openstack-cinder17:39
*** faiz89 has quit IRC17:42
*** rlrossit_ has quit IRC17:42
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge.  https://review.openstack.org/32006017:43
*** faiz89 has joined #openstack-cinder17:43
*** xinli has quit IRC17:43
*** bbhavsar_ has quit IRC17:44
openstackgerritErlon R. Cruz proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078517:44
*** xinli has joined #openstack-cinder17:46
*** eharney has quit IRC17:46
openstackgerritMerged openstack/cinder: Support None value of extra_specs in cinder-scheduler filter  https://review.openstack.org/32859717:55
openstackgerritMerged openstack/cinder: Start/Stop coordinator with Volume service  https://review.openstack.org/26331317:55
openstackgerritMerged openstack/cinder: Replace locks in volume manager  https://review.openstack.org/18564617:56
xinlidulek: could you please help to review the code for patch 251989: https://review.openstack.org/#/c/251989/17:58
*** mtanino has quit IRC17:59
openstackgerritMerged openstack/cinder: Handle quota exceed exception  https://review.openstack.org/28166618:01
openstackgerritMerged openstack/cinder: Nimble driver should return ceiling of volume size  https://review.openstack.org/32551318:01
*** rlrossit has joined #openstack-cinder18:04
*** kmartin has quit IRC18:04
*** ebalduf has joined #openstack-cinder18:06
*** rcernin has joined #openstack-cinder18:07
*** akshai has quit IRC18:08
*** akshai has joined #openstack-cinder18:11
*** mtanino has joined #openstack-cinder18:11
*** eharney has joined #openstack-cinder18:14
openstackgerritAlyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078518:18
*** bardia has joined #openstack-cinder18:18
*** bharath has quit IRC18:19
openstackgerritMerged openstack/cinder: Provide consistency to "Openstack-Api-Version" header  https://review.openstack.org/30987818:20
*** timcl has quit IRC18:21
*** lpetrut has joined #openstack-cinder18:22
*** mriedem1 has joined #openstack-cinder18:23
*** mkoderer__ has joined #openstack-cinder18:24
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix improperly configured privsep  https://review.openstack.org/32958618:24
smcginnisdulek: At least with sqlite, I've found two differences pre-compact commit and post.18:25
smcginnisdulek: One is a foreign key that I did miss. It was originally in there, but our migrations ended up dropping it. I need to remove that from what we have post-compaction.18:25
*** mriedem has quit IRC18:26
smcginnisThe other, which I can't quite figure out how sqlalchemy decides to set because it's not something explicit, is it adds a check constraint that a boolean field is in (0, 1).18:26
smcginnisThat wasn't there before, but I think it actually should be since it's a bool field.18:26
*** kmartin has joined #openstack-cinder18:27
*** mriedem1 is now known as mriedem18:29
*** faiz89 has quit IRC18:34
*** faiz89 has joined #openstack-cinder18:34
*** timcl has joined #openstack-cinder18:42
*** sandanar has joined #openstack-cinder18:44
*** akshai_ has joined #openstack-cinder18:45
*** akshai has quit IRC18:47
*** akshai_ has quit IRC18:50
*** kmartin has quit IRC18:50
*** sandanar_ has joined #openstack-cinder18:50
*** sandanar has quit IRC18:50
*** amit213 has joined #openstack-cinder18:51
*** sage__ has joined #openstack-cinder18:51
*** rlrossit has quit IRC18:52
*** salv-orl_ has quit IRC18:54
*** salv-orlando has joined #openstack-cinder18:55
*** timcl1 has joined #openstack-cinder18:56
*** e0ne has joined #openstack-cinder18:58
*** timcl has quit IRC18:58
*** sgotliv_ has quit IRC19:02
*** ociuhandu has quit IRC19:02
*** sandanar_ has quit IRC19:06
*** sandanar has joined #openstack-cinder19:07
*** sandanar has quit IRC19:09
*** ebalduf has quit IRC19:09
*** lpetrut has quit IRC19:13
openstackgerritXinli Guan proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject  https://review.openstack.org/25198919:14
*** geguileo_mobile has joined #openstack-cinder19:14
*** zhongjun_ has quit IRC19:14
*** zhongjun_ has joined #openstack-cinder19:15
*** geguileo_phone has quit IRC19:17
*** e0ne has quit IRC19:22
openstackgerritSean McGinnis proposed openstack/cinder: Correct DB modifications from compaction  https://review.openstack.org/33075619:22
smcginnisdulek, eharney: Think I got it. ^^19:22
*** ociuhandu has joined #openstack-cinder19:37
*** asselin_ has joined #openstack-cinder19:38
*** asselin__ has quit IRC19:39
*** lpetrut has joined #openstack-cinder19:46
openstackgerritAlyson proposed openstack/cinder: HNAS: Add support for consistency groups in iSCSI driver  https://review.openstack.org/32704319:48
*** rlrossit has joined #openstack-cinder19:51
gouthamrjgriffith: ping19:53
jgriffithgouthamr: pong19:53
*** faiz89 has quit IRC19:53
gouthamrjgriffith: hi! :) i'm trying to implement cheesecake replication on cDOT... I had a couple of questions, now a good time?19:53
jgriffithsure19:53
gouthamrjgriffith: nice.. okay, noticed that solidfire doesn't do the failover to "default"19:54
gouthamrjgriffith: is there some hidden sauce i'm not reading right, or that kind of failover (failback if you will :)) not supported..19:54
jgriffithgouthamr: so the use case we outlined there is no "failback"19:57
jgriffithbut that's something folks are adding now19:57
gouthamrjgriffith: so is it recommended that we do that?19:57
jgriffithgouthamr: Well the problem is people are going to want the failback19:57
jgriffithgouthamr: I personally would like to see it made part of the existing failover cmd19:57
jgriffithgouthamr: whether that be having it work like a toggle, or taking an additional option19:58
jgriffithgouthamr: I just don't want a "new" api method19:58
jgriffithgouthamr: I think patrickeast and Swanson have implemeneted something here19:58
gouthamrjgriffith: +1, i agree.. we could standardize this way of doing it and update the devref so all vendors can do this the same way19:59
patrickeastoh yea19:59
gouthamrjgriffith: coming from the cinderclient and not seeing the "default" trigger in the API is a bit unsettling..19:59
patrickeasti did the default thingy in the pure drivers19:59
SwansonI did what patrickeast did.19:59
gouthamr:D19:59
*** chris_morrell has joined #openstack-cinder20:00
* patrickeast isn't sure how to feel about having implemented that particular feature20:00
jgriffithpatrickeast: dirty, cheap and hollow inside20:00
gouthamrpatrickeast: did you plan on adding documentation for it?20:00
patrickeastgouthamr: so, its totally optional.. and really like if you look at what I did we basically don't do anything other than say 'OK' and go back to the original state20:00
*** salv-orlando has quit IRC20:00
*** ebalduf has joined #openstack-cinder20:01
*** lprice has quit IRC20:01
patrickeastgouthamr: nope, i'm pushing back on that really hard20:01
*** salv-orlando has joined #openstack-cinder20:01
*** lprice has joined #openstack-cinder20:01
patrickeastbecause i kinda expect it to change or go away20:01
Swansongouthamr, patrickeast now if you look at what the dell driver did I should expect you do either do what patrick did or not implement it at all.20:02
*** Apoorva has quit IRC20:02
gouthamrpatrickeast: ah.. sure.. so, we'd let the admin alter the database or the config files or both and get things back to 'normalcy' until we figure this out and give a common way of "failing back"20:02
smcginnisjgriffith: LOL, just reading scrollback and got to your last comment. :D20:02
patrickeastjgriffith: smcginnis: its pretty accurate :D20:03
*** akerr has quit IRC20:03
*** geguileo_mobile has quit IRC20:03
gouthamralright, sounds like we need to hash this out a little more..20:04
patrickeastjgriffith: its almost as bad as my drivers db calls :P20:04
patrickeastgouthamr: yea, if I were you Id wait20:04
gouthamrthe other question i had was regarding the 'replication_device' option20:04
gouthamri see that the spec and the devref had the key as 'backend_id'20:04
gouthamrand the config opt has 'replication_target_id'20:04
patrickeastmm i think backend_id is the right one20:05
patrickeastwhich means that config opt probably should be updated20:05
*** sheel has quit IRC20:05
gouthamrpatrickeast: yep..20:05
gouthamrthere's already a couple vendor drivers that use replication_target_id20:06
*** cknight1 has quit IRC20:06
smcginnisAdded replication failback to the midcycle topics.20:06
patrickeastgouthamr: well, iirc it doesn't matter too much, as long as its what your driver is expecting20:06
gouthamrsorry..20:06
gouthamr"target_device_id"20:07
patrickeastsmcginnis: good idea20:07
gouthamrsmcginnis: +120:07
*** merooney has quit IRC20:07
gouthamrsmcginnis: i should guess, you added it back :)20:07
*** geguileo_phone has joined #openstack-cinder20:08
gouthamrjgriffith patrickeast: thanks! i might bother you guys again.20:08
patrickeastnp20:08
*** porrua has quit IRC20:08
*** xinli has quit IRC20:10
*** timcl1 has quit IRC20:10
*** faiz89 has joined #openstack-cinder20:11
*** rlrossit has quit IRC20:13
*** akshai has joined #openstack-cinder20:13
*** rlrossit has joined #openstack-cinder20:14
openstackgerritPatrick East proposed openstack/cinder-specs: Shared backend config stanza  https://review.openstack.org/33076720:14
*** openstackstatus has joined #openstack-cinder20:18
*** ChanServ sets mode: +v openstackstatus20:18
*** rlrossit has quit IRC20:18
smcginnispatrickeast: Nice! ^20:19
*** zul has quit IRC20:21
*** zul has joined #openstack-cinder20:22
patrickeastsmcginnis: yea i figure that might help shorten some of the conversations i have explaining why things in DEFAULT don't work :(20:22
*** Apoorva has joined #openstack-cinder20:22
smcginnispatrickeast: I haven't read through, but sounds like basically what we talked about back in Tokyo, right?20:25
patrickeastsmcginnis: yep, basically the same thing20:25
smcginnispatrickeast: Yay! Glad to see movement there.20:25
*** rlrossit has joined #openstack-cinder20:27
*** ihrachys has joined #openstack-cinder20:27
*** jungleboyj has quit IRC20:33
*** lpetrut has quit IRC20:38
*** arch-nemesis has quit IRC20:41
*** daneyon has quit IRC20:42
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077820:45
*** hemanthm is now known as hemanthm|afk20:46
patrickeastjgriffith: hey, saw your comment on the config spec, totally on board for the no-drivers in DEFAULT (/me looks at Fuel...), when you say 'internally force them to behave as if they were in a stanza' what did you have in mind?20:49
*** faiz89 has quit IRC20:50
*** lpetrut has joined #openstack-cinder20:51
*** ihrachys has quit IRC20:52
*** zul has quit IRC20:52
*** ihrachys has joined #openstack-cinder20:53
*** arch-nemesis has joined #openstack-cinder20:56
*** jungleboyj has joined #openstack-cinder20:57
smcginnispatrickeast: That may be what I was talking about too the other day.20:57
smcginnispatrickeast: For single backend you can just put the driver in default.20:57
smcginnispatrickeast: But then if you want to add another backend, ruh roh.20:58
patrickeastsmcginnis: ohh, make it so you can mix them?20:58
smcginnispatrickeast: It would be nice to enforce somehow that all backends need to be defined as we do for multibackend.20:58
smcginnispatrickeast: So make it so you HAVE to put them in their own sections.20:58
patrickeastsmcginnis: yea that would be awesome, but probably needs some kind of deprecation period, eh?20:59
patrickeasthah, or start doing config file versions20:59
smcginnispatrickeast: Yeah, and maybe more than our normal deprecation period. But it would be good to get it in the works then so it eventually goes away.20:59
patrickeastsmcginnis: we could probably make it so we treat [DEFAULT] as a backend in multibackend too but log a big warning21:00
smcginnisI think that's what jgriffith was referring to, but I could be wrong. I'd at least like to see it though.21:00
smcginnispatrickeast: That could work.21:00
patrickeasti'll play around with it and see what works21:00
patrickeastfind some nice middle-ground between actually works without too much hacking and does what we want21:00
smcginnispatrickeast: Awesome.21:01
*** sgotliv_ has joined #openstack-cinder21:02
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge.  https://review.openstack.org/32006021:03
*** merooney has joined #openstack-cinder21:03
*** faiz89 has joined #openstack-cinder21:03
*** arch-nemesis has quit IRC21:03
*** faiz89 has quit IRC21:05
*** sgotliv_ has quit IRC21:07
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo  https://review.openstack.org/31904021:08
*** faiz89 has joined #openstack-cinder21:08
*** diablo_rojo has quit IRC21:11
*** akapil has joined #openstack-cinder21:12
*** cknight has joined #openstack-cinder21:14
*** gouthamr has quit IRC21:15
*** krtaylor has quit IRC21:15
*** pauloewerton has quit IRC21:15
*** dustins has quit IRC21:15
*** arch-nemesis has joined #openstack-cinder21:17
*** eharney has quit IRC21:19
*** ihrachys_ has joined #openstack-cinder21:19
*** bardia has quit IRC21:21
*** ihrachys has quit IRC21:22
*** Lee1092 has quit IRC21:23
*** faiz89 has quit IRC21:24
*** laughterwym has joined #openstack-cinder21:25
*** laughterwym has quit IRC21:29
*** porrua has joined #openstack-cinder21:31
*** rlrossit has quit IRC21:32
*** enriquetaso has quit IRC21:32
*** lprice has quit IRC21:33
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Fix improperly configured privsep  https://review.openstack.org/32958621:33
*** lprice has joined #openstack-cinder21:33
*** diogogmt has quit IRC21:34
*** diogogmt has joined #openstack-cinder21:36
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407621:36
*** rcernin has quit IRC21:36
*** gouthamr has joined #openstack-cinder21:37
*** lpetrut has quit IRC21:39
*** lprice has quit IRC21:50
*** mriedem has quit IRC21:51
*** mriedem has joined #openstack-cinder21:52
*** mriedem has quit IRC21:52
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953121:58
openstackgerritTom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication  https://review.openstack.org/32953121:59
*** gouthamr_ has joined #openstack-cinder22:04
*** xyang1 has quit IRC22:04
*** porrua has quit IRC22:05
*** gouthamr has quit IRC22:07
*** lcurtis has joined #openstack-cinder22:08
*** akapil has quit IRC22:09
*** chris_morrell has quit IRC22:11
*** akshai has quit IRC22:11
hemnajgriffith, hey man.   I want to +A this guy, but SF CI isn't reporting....https://review.openstack.org/#/c/326776/22:16
*** ebalduf has quit IRC22:23
jgriffithhemna: hmm... don't know why that thread keeps dieing22:26
jgriffithrather annoying22:26
hemnawasn't sure what was up, thought you should know22:26
jgriffithhemna: yeah, thanks22:27
jgriffithhemna: between the NTAP Exchange server blocking the email notifications I send and me being lazy :)22:27
jgriffithI don't even notice it not running22:27
hemna:)22:27
*** salv-orlando has quit IRC22:29
*** salv-orlando has joined #openstack-cinder22:30
*** jungleboyj has quit IRC22:30
*** geguileo_phone has quit IRC22:31
*** mriedem has joined #openstack-cinder22:32
*** salv-orlando has quit IRC22:34
*** salv-orlando has joined #openstack-cinder22:35
*** geguileo_phone has joined #openstack-cinder22:36
*** karthikp has joined #openstack-cinder22:36
*** bardia has joined #openstack-cinder22:39
*** asselin__ has joined #openstack-cinder22:45
*** asselin_ has quit IRC22:47
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077822:47
*** karthikp has quit IRC22:49
*** geguileo_phone has quit IRC22:51
*** lcurtis has quit IRC22:54
*** geguileo_phone has joined #openstack-cinder22:54
*** alaski_ is now known as alaski23:02
*** edmondsw has quit IRC23:07
*** salv-orl_ has joined #openstack-cinder23:08
*** diogogmt has quit IRC23:10
*** salv-orlando has quit IRC23:11
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo  https://review.openstack.org/31904023:12
*** akerr has joined #openstack-cinder23:14
*** daneyon has joined #openstack-cinder23:16
*** akerr has quit IRC23:17
*** asselin has quit IRC23:17
*** ihrachys_ has quit IRC23:20
*** garthb has quit IRC23:20
*** ihrachys has joined #openstack-cinder23:21
openstackgerritMerged openstack/cinder: 3PAR: Fix delete volume when online clone  https://review.openstack.org/32784023:28
*** daneyon has quit IRC23:33
*** angela-s has quit IRC23:34
*** karthikp has joined #openstack-cinder23:35
*** crinkle has left #openstack-cinder23:35
*** cknight has quit IRC23:37
*** cknight has joined #openstack-cinder23:37
openstackgerritMerged openstack/cinder: Fix Brcd zone driver initiator zone update  https://review.openstack.org/31706123:37
*** cknight has quit IRC23:39
*** cknight1 has joined #openstack-cinder23:39
*** karthikp has quit IRC23:40
*** chlong has quit IRC23:42
*** cknight1 has quit IRC23:44
*** cknight has joined #openstack-cinder23:44
*** salv-orl_ has quit IRC23:47
*** salv-orlando has joined #openstack-cinder23:48
*** cknight1 has joined #openstack-cinder23:50
*** cknight has quit IRC23:51
*** ihrachys has quit IRC23:51
*** Julien-zte has quit IRC23:52
*** adrianofr has quit IRC23:52
*** sdake has quit IRC23:57

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