*** openstack has joined #openstack-cinder | 05:59 | |
*** bardia has joined #openstack-cinder | 06:00 | |
*** laughterwym has quit IRC | 06:01 | |
*** Poornima has joined #openstack-cinder | 06:06 | |
baater | diff version of cinder have some influence on sync ? | 06:07 |
---|---|---|
openstackgerrit | Ravi Shekhar Jethani proposed openstack/cinder: Fix db purge for volume_types FK constraint https://review.openstack.org/330328 | 06:08 |
*** TaylorHuang has joined #openstack-cinder | 06:12 | |
*** lpetrut has quit IRC | 06:13 | |
*** shausy has quit IRC | 06:14 | |
*** shausy has joined #openstack-cinder | 06:15 | |
*** rsjethani has joined #openstack-cinder | 06:18 | |
rsjethani | Hi, I am facing a weird issue with cinder tests | 06:19 |
rsjethani | running tests on a single file or for a module is working | 06:20 |
rsjethani | but when i run whole test suit..it fails..even on master | 06:20 |
*** laughter_ has quit IRC | 06:23 | |
*** laughterwym has joined #openstack-cinder | 06:24 | |
*** lpetrut has joined #openstack-cinder | 06:24 | |
*** arecknag has joined #openstack-cinder | 06:24 | |
*** rcernin has joined #openstack-cinder | 06:25 | |
*** EinstCrazy has quit IRC | 06:28 | |
lixiaoy1 | baater: https://github.com/openstack/cinder/blob/master/cinder/service.py#L179 it updates db every report_interval(default 10s) | 06:28 |
lixiaoy1 | baater: https://github.com/openstack/cinder/blob/master/cinder/service.py#L51 do you unset the config paramter? | 06:29 |
*** EinstCrazy has joined #openstack-cinder | 06:29 | |
lixiaoy1 | baater: from kilo, they did same thing to report service | 06:29 |
*** EinstCrazy has quit IRC | 06:30 | |
*** EinstCrazy has joined #openstack-cinder | 06:30 | |
*** adisky has joined #openstack-cinder | 06:33 | |
*** lpetrut has quit IRC | 06:40 | |
*** EinstCrazy has quit IRC | 06:41 | |
openstackgerrit | Merged openstack/cinder: Netmask needs corersion to string https://review.openstack.org/329574 | 06:41 |
openstackgerrit | Merged openstack/cinder: Add test type clarification to devref https://review.openstack.org/327806 | 06:41 |
openstackgerrit | Merged openstack/cinder: Remove test_conf.py https://review.openstack.org/320144 | 06:41 |
openstackgerrit | Hiroyuki Eguchi proposed openstack/cinder: Unify the type of value for what are True/False values https://review.openstack.org/330339 | 06:43 |
*** EinstCrazy has joined #openstack-cinder | 06:45 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 06:48 |
*** salv-orlando has quit IRC | 06:49 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: Fix size exceed of dd action when clearing a volume https://review.openstack.org/324376 | 06:50 |
*** salv-orlando has joined #openstack-cinder | 06:50 | |
*** afazekas is now known as afazekas|dentist | 06:52 | |
*** yangyape_ has joined #openstack-cinder | 06:54 | |
*** salv-orl_ has joined #openstack-cinder | 06:55 | |
openstackgerrit | Merged openstack/cinder: Give more fine grained access to DriverInitiatorData https://review.openstack.org/324155 | 06:55 |
openstackgerrit | Merged openstack/cinder: Remove locks from Pure volume drivers https://review.openstack.org/324156 | 06:56 |
*** yangyapeng has quit IRC | 06:56 | |
openstackgerrit | Merged openstack/cinder: Enable str2size() to handle long int for i386 https://review.openstack.org/301326 | 06:56 |
rsjethani | Hi, I am facing a weird issue with cinder tests | 06:56 |
rsjethani | running tests on a single file or for a module is working | 06:56 |
rsjethani | but when i run whole test suit..it fails..even on master | 06:56 |
*** salv-orlando has quit IRC | 06:57 | |
openstackgerrit | haobing proposed openstack/cinder: Allow setting CG name or description to empty value https://review.openstack.org/324465 | 06:58 |
*** bardia has quit IRC | 06:59 | |
*** markvoelker_ has quit IRC | 07:01 | |
*** markvoelker has joined #openstack-cinder | 07:01 | |
lixiaoy1 | rsjethani: you'd better paste the error message | 07:01 |
*** yangyapeng has joined #openstack-cinder | 07:06 | |
*** yangyapeng has quit IRC | 07:06 | |
*** yangyapeng has joined #openstack-cinder | 07:07 | |
*** yangyape_ has quit IRC | 07:08 | |
*** tesseract has joined #openstack-cinder | 07:09 | |
*** jamielennox is now known as jamielennox|away | 07:12 | |
*** bharathk has quit IRC | 07:16 | |
*** bharathk has joined #openstack-cinder | 07:17 | |
*** akapil has joined #openstack-cinder | 07:18 | |
*** permalac has quit IRC | 07:19 | |
openstackgerrit | wanghao proposed openstack/python-cinderclient: Fixing parsing problem of cascade in client https://review.openstack.org/320809 | 07:20 |
openstackgerrit | wanghao proposed openstack/python-cinderclient: Fixing parsing problem of cascade in client https://review.openstack.org/320809 | 07:21 |
*** locust has quit IRC | 07:24 | |
*** GB21 has quit IRC | 07:31 | |
*** akapil has quit IRC | 07:33 | |
*** akapil has joined #openstack-cinder | 07:34 | |
*** akapil has quit IRC | 07:37 | |
*** akapil has joined #openstack-cinder | 07:38 | |
*** akapil has quit IRC | 07:43 | |
*** akapil has joined #openstack-cinder | 07:43 | |
*** bharat_ has joined #openstack-cinder | 07:44 | |
dulek | smcginnis: Too lateā¦ ;) I'll compare DB schemas generated by both anyway. | 07:44 |
*** baater has quit IRC | 07:47 | |
*** salv-orl_ has quit IRC | 07:49 | |
*** salv-orlando has joined #openstack-cinder | 07:49 | |
*** akapil has quit IRC | 07:51 | |
flip214 | reviews of https://review.openstack.org/#/c/329905/ would be appreciated... thanks a lot in advance! | 07:56 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-cinder | 08:00 | |
*** bharat_ has quit IRC | 08:00 | |
*** lpetrut has joined #openstack-cinder | 08:07 | |
*** permalac has joined #openstack-cinder | 08:09 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
*** lpetrut1 has joined #openstack-cinder | 08:12 | |
*** akapil has joined #openstack-cinder | 08:12 | |
*** lpetrut has quit IRC | 08:12 | |
*** lpetrut1 is now known as lpetrut | 08:12 | |
*** e0ne has joined #openstack-cinder | 08:13 | |
*** akapil has quit IRC | 08:14 | |
*** Julien-zte has joined #openstack-cinder | 08:15 | |
*** zhongjun_ has quit IRC | 08:21 | |
*** akapil has joined #openstack-cinder | 08:21 | |
*** zhongjun_ has joined #openstack-cinder | 08:29 | |
*** GB21 has joined #openstack-cinder | 08:32 | |
*** savihou has joined #openstack-cinder | 08:34 | |
*** jordanP has joined #openstack-cinder | 08:34 | |
savihou | e0ne: Hi Ivan, around ? | 08:36 |
e0ne | savihou: hi, yes | 08:36 |
savihou | e0ne: PM'ed you | 08:37 |
*** laughterwym has quit IRC | 08:47 | |
*** zhonghua-lee has quit IRC | 08:47 | |
*** laughterwym has joined #openstack-cinder | 08:47 | |
*** laughter_ has joined #openstack-cinder | 08:50 | |
*** laughterwym has quit IRC | 08:52 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 08:52 |
*** laughter_ has quit IRC | 08:54 | |
*** salv-orlando has quit IRC | 08:55 | |
*** salv-orlando has joined #openstack-cinder | 08:56 | |
*** akapil has quit IRC | 09:01 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: Add ConsistencyGroupVolumeTypeMapping table https://review.openstack.org/245854 | 09:04 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Add generic code for online data migrations https://review.openstack.org/330391 | 09:04 |
openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 09:06 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Add ConsistencyGroupVolumeTypeMapping table https://review.openstack.org/245854 | 09:09 |
openstackgerrit | ningwei proposed openstack/cinder: Added backend driver for huawei FusionStorage https://review.openstack.org/328887 | 09:11 |
*** bhavik has quit IRC | 09:20 | |
*** lprice has quit IRC | 09:22 | |
*** bhavik has joined #openstack-cinder | 09:24 | |
*** Divya has joined #openstack-cinder | 09:25 | |
openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 09:25 |
*** ihrachys has joined #openstack-cinder | 09:26 | |
*** rsjethani has quit IRC | 09:27 | |
*** mkoderer__ has quit IRC | 09:28 | |
*** bhavik has quit IRC | 09:28 | |
*** lprice has joined #openstack-cinder | 09:29 | |
*** akapil has joined #openstack-cinder | 09:32 | |
*** akapil has quit IRC | 09:33 | |
*** akapil has joined #openstack-cinder | 09:34 | |
*** yhayashi has quit IRC | 09:35 | |
*** nikeshm has joined #openstack-cinder | 09:38 | |
*** ociuhandu has quit IRC | 09:39 | |
*** rsjethani has joined #openstack-cinder | 09:41 | |
openstackgerrit | Hiroyuki Eguchi proposed openstack/cinder: Unify the type of value for what are True/False values https://review.openstack.org/330339 | 09:43 |
*** bhavik has joined #openstack-cinder | 09:46 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: Support None value of extra_specs in cinder-scheduler filter https://review.openstack.org/328597 | 09:58 |
*** cheneydc has quit IRC | 10:01 | |
nikeshm | DuncanT: hi | 10:03 |
*** akapil has quit IRC | 10:04 | |
nikeshm | DuncanT: if you get time please review https://review.openstack.org/#/c/327285 | 10:04 |
dulek | smcginnis: I've commented on https://review.openstack.org/#/c/324897. It's failing in a case where older code wasn't. | 10:06 |
dulek | smcginnis: Sorry I couldn't review it yesterday, it was 10:30 PM my time. | 10:07 |
*** salv-orlando has quit IRC | 10:09 | |
*** jamielennox|away is now known as jamielennox | 10:15 | |
*** ociuhandu has joined #openstack-cinder | 10:25 | |
*** zhongjun_ has quit IRC | 10:26 | |
*** mvk_ has quit IRC | 10:26 | |
*** zhongjun_ has joined #openstack-cinder | 10:26 | |
*** iceyao_ has joined #openstack-cinder | 10:28 | |
*** iceyao has quit IRC | 10:32 | |
*** iceyao_ has quit IRC | 10:33 | |
*** Divya has quit IRC | 10:36 | |
*** winston-d_ has quit IRC | 10:37 | |
*** akapil has joined #openstack-cinder | 10:38 | |
*** akapil has quit IRC | 10:38 | |
*** sdake has joined #openstack-cinder | 10:40 | |
*** e0ne has quit IRC | 10:41 | |
*** e0ne has joined #openstack-cinder | 10:41 | |
*** sdake_ has joined #openstack-cinder | 10:42 | |
*** sdake has quit IRC | 10:45 | |
*** akapil has joined #openstack-cinder | 10:50 | |
*** akapil has quit IRC | 10:51 | |
*** sborkows has joined #openstack-cinder | 10:51 | |
*** mvk_ has joined #openstack-cinder | 10:53 | |
*** GB21 has quit IRC | 10:53 | |
*** akapil has joined #openstack-cinder | 10:55 | |
*** akapil has quit IRC | 10:56 | |
*** cdelatte has joined #openstack-cinder | 10:57 | |
*** tyrola has joined #openstack-cinder | 10:58 | |
huyang | kindly review: https://review.openstack.org/#/c/325682/ | 11:01 |
*** salv-orlando has joined #openstack-cinder | 11:07 | |
*** tyrola has quit IRC | 11:09 | |
*** salv-orlando has quit IRC | 11:10 | |
*** sdake_ has quit IRC | 11:10 | |
*** salv-orlando has joined #openstack-cinder | 11:11 | |
*** kfarr has joined #openstack-cinder | 11:21 | |
*** etraitel has joined #openstack-cinder | 11:22 | |
etraitel | Hi, 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 |
dulek | etraitel: DevStack without explicit setting installs master. | 11:30 |
dulek | etraitel: You're getting error from c-api or python-cinderclient? | 11:30 |
*** kaisers1 has joined #openstack-cinder | 11:34 | |
*** e0ne has quit IRC | 11:35 | |
etraitel | Got it. We're not getting an error, it just seems to ignore it | 11:36 |
dulek | etraitel: Use --debug on cinder command to see if it's sent over REST. | 11:38 |
*** kaisers1 has quit IRC | 11:44 | |
*** salv-orlando has quit IRC | 11:45 | |
*** kaisers1 has joined #openstack-cinder | 11:45 | |
openstackgerrit | ningwei proposed openstack/cinder: Added backend driver for huawei FusionStorage https://review.openstack.org/328887 | 11:46 |
*** salv-orlando has joined #openstack-cinder | 11:46 | |
*** GB21 has joined #openstack-cinder | 11:48 | |
*** iceyao has joined #openstack-cinder | 11:49 | |
*** Julien-zte has quit IRC | 11:49 | |
*** ducttape_ has quit IRC | 11:54 | |
*** ducttape_ has joined #openstack-cinder | 11:54 | |
*** laughter_ has joined #openstack-cinder | 11:58 | |
*** bharathk has quit IRC | 12:00 | |
*** akapil has joined #openstack-cinder | 12:01 | |
*** sdake has joined #openstack-cinder | 12:05 | |
*** akapil has quit IRC | 12:06 | |
*** akapil has joined #openstack-cinder | 12:06 | |
*** ducttape_ has quit IRC | 12:07 | |
openstackgerrit | xiexs proposed openstack/cinder: Move _check_metadata_properties to cinder/utils https://review.openstack.org/330502 | 12:15 |
openstackgerrit | xiexs proposed openstack/cinder: Make volume-image-metadata to support None type https://review.openstack.org/330503 | 12:15 |
*** GB21 has quit IRC | 12:15 | |
*** GB21 has joined #openstack-cinder | 12:17 | |
*** adrianofr has joined #openstack-cinder | 12:18 | |
*** timcl has joined #openstack-cinder | 12:22 | |
*** rcernin has quit IRC | 12:24 | |
*** kfarr has quit IRC | 12:28 | |
*** ducttape_ has joined #openstack-cinder | 12:36 | |
*** ducttape_ has quit IRC | 12:38 | |
*** rcernin has joined #openstack-cinder | 12:39 | |
*** julim has joined #openstack-cinder | 12:39 | |
*** e0ne has joined #openstack-cinder | 12:40 | |
*** zul_ has joined #openstack-cinder | 12:41 | |
e0ne | geguileor: hi. are you around? | 12:42 |
*** zul_ has quit IRC | 12:42 | |
*** zul_ has joined #openstack-cinder | 12:42 | |
*** zul_ has quit IRC | 12:43 | |
*** zul_ has joined #openstack-cinder | 12:44 | |
*** eharney has quit IRC | 12:44 | |
*** edmondsw has joined #openstack-cinder | 12:45 | |
*** zul_ has quit IRC | 12:49 | |
*** zul has quit IRC | 12:49 | |
*** zul has joined #openstack-cinder | 12:50 | |
*** Julien-zte has joined #openstack-cinder | 12:52 | |
*** GB21 has quit IRC | 12:53 | |
*** gcb has quit IRC | 12:53 | |
*** akapil has quit IRC | 12:53 | |
*** rcernin has quit IRC | 12:54 | |
*** akapil has joined #openstack-cinder | 12:54 | |
*** gouthamr has joined #openstack-cinder | 12:58 | |
*** pauloewerton has joined #openstack-cinder | 12:58 | |
*** rlrossit has joined #openstack-cinder | 13:00 | |
*** gouthamr_ has joined #openstack-cinder | 13:04 | |
*** timcl1 has joined #openstack-cinder | 13:05 | |
*** timcl has quit IRC | 13:06 | |
*** gouthamr has quit IRC | 13:07 | |
*** rcernin has joined #openstack-cinder | 13:07 | |
dulek | e0ne: AFAIK geguileo_phone is on PTO now. | 13:08 |
*** e0ne has quit IRC | 13:09 | |
*** gouthamr has joined #openstack-cinder | 13:11 | |
*** pgadiya has quit IRC | 13:11 | |
*** e0ne has joined #openstack-cinder | 13:12 | |
*** hemanthm_ is now known as hemanthm | 13:14 | |
*** timcl has joined #openstack-cinder | 13:14 | |
*** gouthamr_ has quit IRC | 13:15 | |
*** timcl1 has quit IRC | 13:15 | |
*** faiz89 has joined #openstack-cinder | 13:16 | |
*** eharney has joined #openstack-cinder | 13:17 | |
*** mriedem has joined #openstack-cinder | 13:19 | |
*** dustins has joined #openstack-cinder | 13:19 | |
*** EinstCrazy has quit IRC | 13:20 | |
*** salv-orlando has quit IRC | 13:21 | |
*** yangyape_ has joined #openstack-cinder | 13:21 | |
*** salv-orlando has joined #openstack-cinder | 13:21 | |
*** EinstCrazy has joined #openstack-cinder | 13:21 | |
*** merooney has joined #openstack-cinder | 13:22 | |
*** bharath has joined #openstack-cinder | 13:22 | |
*** pgadiya has joined #openstack-cinder | 13:23 | |
*** yangyape_ has quit IRC | 13:23 | |
*** yangyapeng has quit IRC | 13:24 | |
openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder: Use keystoneauth1 for creating the keystone session https://review.openstack.org/318490 | 13:25 |
*** akshai has joined #openstack-cinder | 13:25 | |
*** pgadiya has quit IRC | 13:25 | |
*** EinstCrazy has quit IRC | 13:26 | |
*** akshai_ has joined #openstack-cinder | 13:27 | |
*** raildo-a` is now known as raildo | 13:27 | |
*** diogogmt has quit IRC | 13:29 | |
*** frontrunner has joined #openstack-cinder | 13:30 | |
geguileo_phone | e0ne I'm on PTO on an island with limited internet connection | 13:31 |
*** akshai has quit IRC | 13:31 | |
e0ne | geguileo_phone: don't worry. I'll ping you later. have a good vacation! | 13:31 |
geguileo_phone | I'll be back to civilization next Monday | 13:31 |
*** diogogmt has joined #openstack-cinder | 13:31 | |
*** smoriya has quit IRC | 13:32 | |
geguileo_phone | e0ne Is it urgent? | 13:32 |
*** rlrossit has quit IRC | 13:32 | |
e0ne | geguileo_phone: no | 13:32 |
geguileo_phone | OK, then I'll ping you as soon as I'm back | 13:32 |
*** mtanino has joined #openstack-cinder | 13:35 | |
*** cknight has joined #openstack-cinder | 13:37 | |
*** cknight1 has joined #openstack-cinder | 13:37 | |
*** cknight has quit IRC | 13:37 | |
*** timcl has quit IRC | 13:40 | |
*** porrua has joined #openstack-cinder | 13:41 | |
*** lprice has quit IRC | 13:42 | |
*** Poornima has quit IRC | 13:42 | |
*** ducttape_ has joined #openstack-cinder | 13:43 | |
smcginnis | dulek: Thanks, I figured it was a long shot you were still around. Thanks for still looking at that. Reading your comments now. | 13:44 |
eharney | dulek: 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 |
openstack | Launchpad bug 1593175 in Cinder "Databases created before and after compacting migrations are different" [Critical,New] | 13:44 |
dulek | ameade: I've seen your comment. Rebase successful? :) | 13:45 |
smcginnis | eharney: I would hope it's just easy enough to fix. | 13:45 |
smcginnis | eharney: Haven't looked close enough, but really should be fairly minor I would hope. | 13:45 |
ameade | dulek: yessir, thank you, setting up an environment now | 13:45 |
eharney | smcginnis: great, if you think so -- i didn't have much of a guess there | 13:45 |
smcginnis | eharney: Will see if I change my mind once I look closer. ;) | 13:45 |
eharney | smcginnis: if going that route, we have to do it rather quickly IMO | 13:46 |
smcginnis | eharney: Yes, just got to my desk and saw this. Top priority for me this morning. | 13:46 |
dulek | smcginnis, eharney: I would expect anyone running trunk deployments would have an old DB migrated instead of creating a new one. | 13:46 |
smcginnis | dulek: True | 13:46 |
dulek | smcginnis, eharney: So hopefully changing a migration isn't that bad. | 13:46 |
eharney | works for me | 13:47 |
dulek | But 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-cinder | 13:48 | |
smcginnis | dulek, eharney: OK, if that's what you would prefer I'm fine with that. | 13:48 |
dulek | smcginnis: Only if it will take a lot of time to fix that. :) | 13:48 |
*** dulek has left #openstack-cinder | 13:49 | |
*** dulek has joined #openstack-cinder | 13:49 | |
smcginnis | dulek: Weird, looking at your diff now. I didn't see any of these diffs when I compared the old to new. | 13:49 |
smcginnis | This is comparing the initial schema definition without the post-kilo migrations to the state of kilo? | 13:49 |
dulek | smcginnis: 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 IRC | 13:50 | |
dulek | smcginnis: No, in both cases it's from 0 to trunk. | 13:50 |
dulek | smcginnis: But with and without your commit. | 13:50 |
smcginnis | dulek: But it is set. Is MySQL behaving differently based on when it is set? | 13:50 |
smcginnis | dulek: From 0 to trunk meaning running all migrations? | 13:51 |
dulek | smcginnis: At least in my env it's failing on creating a volumes table unless I specify mysql_charset. | 13:51 |
dulek | smcginnis: Yes, all migrations. | 13:51 |
*** ducttape_ has quit IRC | 13:52 | |
smcginnis | dulek: But there were changes after Kilo, so it should be different. | 13:52 |
smcginnis | dulek: Maybe I'm not following. | 13:52 |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Revert "Compact Kilo database migrations" https://review.openstack.org/330570 | 13:52 |
dulek | smcginnis: 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 IRC | 13:53 | |
dulek | Aw, that sounds confusing. ;) | 13:53 |
smcginnis | dulek: But if you're comparing to the state at Kilo? | 13:53 |
dulek | smcginnis: So I've loaded master, dropped the DB, created a new one and run cinder-manage DB sync. | 13:54 |
openstackgerrit | Merged openstack/cinder: VMware: Deprecate vCenter version 5.1 https://review.openstack.org/328189 | 13:54 |
dulek | Last migration - 74. | 13:54 |
dulek | Now I've got back in git to a moment just before your commit and repeated the procedure. | 13:54 |
dulek | Also ended up on 74. | 13:55 |
dulek | I've ended up with two cases of creating a DB from scratch, right? | 13:55 |
smcginnis | dulek: 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 |
smcginnis | dulek: 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 |
dulek | smcginnis: So you've did similar comparison without experiencing my results. | 13:56 |
smcginnis | But 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 |
smcginnis | dulek: Yes | 13:56 |
dulek | smcginnis: MySQL version? OS version? SQLA versionā¦? | 13:56 |
eharney | smcginnis: can you stick a Related-Bug in the commit message just so we can keep track of this later? | 13:56 |
dulek | Any of these could have caused it on my env. | 13:56 |
*** akapil has joined #openstack-cinder | 13:56 | |
smcginnis | dulek: Not sure now, but I'm certainly open to the possiblity that I screwed it up on my end. ;) | 13:56 |
smcginnis | eharney: Sure, good call. | 13:57 |
*** timcl has joined #openstack-cinder | 13:57 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Revert "Compact Kilo database migrations" https://review.openstack.org/330570 | 13:57 |
* dulek hopes he isn't making all this fuss for nothing because of a mistake. | 13:57 | |
*** tpsilva has joined #openstack-cinder | 13:58 | |
smcginnis | dulek: The likelihood it's my mistake and not yours is probably higher. :) | 13:58 |
*** nag_ has quit IRC | 13:58 | |
*** kfarr has joined #openstack-cinder | 13:58 | |
*** geguileo_mobile has joined #openstack-cinder | 13:59 | |
*** jgregor has joined #openstack-cinder | 13:59 | |
dulek | I would expect it's 50-50. :P | 13:59 |
smcginnis | I better grab some coffee before I go much further. :) | 14:00 |
*** geguileo_phone has quit IRC | 14:01 | |
*** diogogmt has quit IRC | 14:01 | |
*** rlrossit_ has joined #openstack-cinder | 14:01 | |
*** iceyao has left #openstack-cinder | 14:03 | |
*** sheel has quit IRC | 14:05 | |
*** crose has joined #openstack-cinder | 14:07 | |
*** arch-nemesis has joined #openstack-cinder | 14:08 | |
*** arch-nemesis has quit IRC | 14:08 | |
*** arch-nemesis has joined #openstack-cinder | 14:09 | |
*** itzdilip has joined #openstack-cinder | 14:11 | |
nikeshm | smcginnis patrickeast eharney: addressed the comments on https://review.openstack.org/#/c/327285/10, if you get time please have a look, CI also reporting | 14:12 |
smcginnis | nikeshm: Opened a tab, probably won't be able to look until later. | 14:12 |
nikeshm | are yu in india? | 14:13 |
smcginnis | nikeshm: No, still in the US. I will be in Bangalore the first week of July. | 14:14 |
*** crose has quit IRC | 14:14 | |
nikeshm | crucial time, since close to N1 | 14:15 |
nikeshm | Jul 08 - 09, 2016 OpenStack Days India | 14:15 |
smcginnis | nikeshm: Yep, that's the one. | 14:16 |
*** arecknag has quit IRC | 14:18 | |
*** bhavik has quit IRC | 14:21 | |
*** kaisers_ has joined #openstack-cinder | 14:21 | |
*** bharath has quit IRC | 14:21 | |
nikeshm | eharney: hi | 14:23 |
*** Lee1092 has quit IRC | 14:23 | |
nikeshm | eharney: do i have to change CONF.kaminario_nodedup_substring to self.configuration.kaminario_nodedup_substring | 14:24 |
*** laughter_ has quit IRC | 14:25 | |
*** laughterwym has joined #openstack-cinder | 14:25 | |
eharney | nikeshm: yes because otherwise when people set it in the driver backend section in cinder.conf it won't behave as expected | 14:26 |
*** diablo_rojo has joined #openstack-cinder | 14:26 | |
jgriffith | smcginnis: hmm... what deltas did you guys find? I loaded it and did a quick check and it seemed ok | 14:26 |
dulek | jgriffith: I've listed them here: https://bugs.launchpad.net/cinder/+bug/1593175 | 14:27 |
openstack | Launchpad bug 1593175 in Cinder "Databases created before and after compacting migrations are different" [Critical,New] | 14:27 |
smcginnis | jgriffith: 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 |
smcginnis | I'm guessing that's probably what happened. | 14:27 |
jgriffith | hmm | 14:27 |
smcginnis | Honestly not too surprised if something was missed based on all the odd migrations we had in there. | 14:27 |
dulek | jgriffith: 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 |
nikeshm | eharney: i m not using this in code self.configuration.append_config_values(kaminario1_opts), so people can not set it in backend section | 14:28 |
*** jistr is now known as jistr|mtg | 14:28 | |
jgriffith | smcginnis: yeah, it's so long overdue it's sort of a monumental thing | 14:28 |
smcginnis | jgriffith: Yeah, we probably should have been doing this for a while. | 14:28 |
nikeshm | eharney: i am only using CONF.register_opts(kaminario1_opts) | 14:28 |
jgriffith | dulek: nah... good chance I didn't compare the dump on the default settings | 14:28 |
smcginnis | At least from this point forward (once I straighten this out) it should be trivial to keep up with. | 14:29 |
nikeshm | eharney: so user can only set it in [DEFAULT] | 14:29 |
*** lprice has joined #openstack-cinder | 14:29 | |
eharney | nikeshm: i don't think that is true | 14:29 |
jgriffith | dulek: smcginnis I think part of the issue may be the fact that we've changed our semantics for the migrations considerably over the years | 14:30 |
jgriffith | dulek: smcginnis some of the things we used to do with default null etc have changed | 14:30 |
*** sheel has joined #openstack-cinder | 14:30 | |
nikeshm | eharney: 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 |
smcginnis | There 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 |
mriedem | smcginnis: speaking of retype bugs https://bugs.launchpad.net/nova/+bug/1593055 | 14:32 |
openstack | Launchpad bug 1593055 in OpenStack Compute (nova) "Retype an in-use volume failed in mitaka" [Undecided,Incomplete] | 14:32 |
nikeshm | eharney: self.configuration.kaminario_nodedup_substring is only working when i use self.configuration.append_config_values(kaminario1_opts) | 14:32 |
mriedem | and remotefs | 14:32 |
smcginnis | mriedem: Looks like a retype that requires migration for an attached LVM volume. | 14:33 |
scottda | mriedem: Thanks | 14:33 |
scottda | mriedem: I'll work on a repro now.... | 14:33 |
smcginnis | scottda: Thanks! | 14:33 |
smcginnis | Is "python setup.py install" broken? | 14:33 |
*** bbhavsar_ has joined #openstack-cinder | 14:34 | |
*** akapil has quit IRC | 14:35 | |
smcginnis | Hmm, "pip install ./" gets the same error for me... | 14:35 |
nikeshm | eharney: NoSuchOptError: no such option in group kaminario-iscsi-dedup: kaminario_nodedup_substring | 14:35 |
eharney | in a meeting right now, bbiab | 14: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 IRC | 14:42 | |
*** karthikp_ has joined #openstack-cinder | 14:42 | |
dulek | Whoops, I've clicked something bad in my client. Sorry. | 14:43 |
eharney | dulek: hey, i've done that trick before too... it's a sneaky button :) | 14:44 |
smcginnis | dulek: We now know exactly what you're running. :) | 14:44 |
openstackgerrit | Merged openstack/cinder: Add debug level logs on OVO backports https://review.openstack.org/321143 | 14:44 |
smcginnis | dulek: Good thing you don't have any AMD in there. | 14:44 |
eharney | lol | 14:44 |
openstackgerrit | Merged openstack/cinder: Implement OVO class init mechanism https://review.openstack.org/321144 | 14:44 |
*** kaisers_ has quit IRC | 14:44 | |
dulek | You even know how much disk I'm using. :D | 14:45 |
*** kmartin has joined #openstack-cinder | 14:45 | |
*** karthikp_ has quit IRC | 14:47 | |
*** kaisers_ has joined #openstack-cinder | 14:48 | |
*** jungleboyj has joined #openstack-cinder | 14:48 | |
*** tyrola has joined #openstack-cinder | 14:49 | |
*** mriedem1 has joined #openstack-cinder | 14:50 | |
*** mriedem has quit IRC | 14:51 | |
*** akapil has joined #openstack-cinder | 14:52 | |
*** karthikp_ has joined #openstack-cinder | 14:52 | |
*** sdake has quit IRC | 14:54 | |
*** akapil has quit IRC | 14:55 | |
*** diogogmt has joined #openstack-cinder | 14:55 | |
*** salv-orlando has quit IRC | 14:56 | |
*** salv-orlando has joined #openstack-cinder | 14:57 | |
*** mriedem1 is now known as mriedem | 14:57 | |
*** EinstCrazy has joined #openstack-cinder | 14:58 | |
*** markstur has joined #openstack-cinder | 14:58 | |
*** sdake has joined #openstack-cinder | 14:59 | |
*** salv-orlando has quit IRC | 15:00 | |
eantyshev | smcginnis: 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-cinder | 15:00 | |
smcginnis | eantyshev: 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-lyle | 15:00 | |
eantyshev | smcginnis: Thank you, this really needs some feedback! | 15:02 |
*** markstur has left #openstack-cinder | 15:02 | |
jgriffith | geguileor: It's disturbing that a change like this ends up being >50% unit test changes https://review.openstack.org/#/c/286598/24 | 15:03 |
jgriffith | geguileor: I like the simplification | 15:03 |
jgriffith | geguileor: number of places we can benefit from this sort of cleanup | 15:04 |
*** cbader_ has quit IRC | 15:05 | |
*** faiz89 has quit IRC | 15:07 | |
dulek | jgriffith: I have a concern on the previous patch in that chain: https://review.openstack.org/#/c/315541 | 15:08 |
jgriffith | dulek: I've just started looking at that one | 15:08 |
jgriffith | dulek: suppose I should've started at the beginning :) | 15:08 |
dulek | jgriffith: 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 |
jgriffith | dulek: what's the concern? | 15:09 |
*** kaisers_ has quit IRC | 15:09 | |
*** iceyao has joined #openstack-cinder | 15:09 | |
jgriffith | dulek: ahh... a man after my own heart :) | 15:09 |
dulek | jgriffith: I wonder if you will share that when reviewing or it is just me. | 15:09 |
*** kaisers_ has joined #openstack-cinder | 15:10 | |
jgriffith | dulek: let me have a look | 15:10 |
*** mattymo_ is now known as mattymo | 15:10 | |
jgriffith | oh | 15:10 |
dulek | jgriffith: BTW - you've mentioned https://github.com/openstack/cinder/commit/a8744e26ba665a9103f153bd516dd49d99fd0c2f#diff-1b317284bd7cf1f7415c1be6a919fee5R2964 yesterday. | 15:10 |
dulek | jgriffith: Is there a problem with that? | 15:10 |
*** adisky has quit IRC | 15:10 | |
jgriffith | dulek: nah... I found what I was looking for | 15:10 |
dulek | jgriffith: Okay, cool. :0 | 15:11 |
jgriffith | dulek: the "int_no" was bothering me | 15:11 |
dulek | Sure. Different type of deleted column. | 15:11 |
jgriffith | dulek: kind of to the point you just made, I'm kind of a simpleton | 15:11 |
jgriffith | dulek: I like having a standard | 15:11 |
*** rsjethani has quit IRC | 15:11 | |
jgriffith | dulek: it took me a while to look at why/where some use True, some use "int_no" etc | 15:12 |
jgriffith | dulek: my conclusion (perhaps wrong) was that it's more transitionary | 15:12 |
dulek | jgriffith: It's because VolumeTypeProjects has deleted column as int, and not boolean. | 15:13 |
dulek | jgriffith: There were at least 2 bugs related to that and we've end up with this. | 15:14 |
*** iceyao has quit IRC | 15:14 | |
*** suyash has joined #openstack-cinder | 15:15 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 15:16 |
jgriffith | dulek: oh... crap | 15:16 |
jgriffith | yeah... I see now | 15:16 |
jgriffith | dulek: so there's another place where we could probably normalize the db again no? | 15:16 |
jgriffith | dulek: easy enough to write a migration that updates existing ones | 15:17 |
dulek | jgriffith: Probably, but this one is related to the constraint we have on that table. | 15:17 |
jgriffith | dulek: also, frankly we haven't released VolumeTypeProjects so we could/should fix that | 15:17 |
dulek | jgriffith: Released? | 15:18 |
e0ne | smcginnis: just FYI, I've fixed issue with BDD (patch on review). the second path will be for tempest to make CI green | 15:18 |
jgriffith | dulek: there's no release version of Cinder running out there that has that model | 15:18 |
smcginnis | e0ne: Yay! :) | 15:18 |
jgriffith | dulek: it doesn't matter either way... just ignore that comment :) | 15:18 |
dulek | jgriffith: git blame tells me that it's there since 2014ā¦ :P | 15:19 |
*** jistr|mtg is now known as jistr | 15:19 | |
e0ne | smcginnis: there are 2 different issues: with tempest and cinder code:( | 15:19 |
jgriffith | dulek: VolumeTypeGroup? | 15:19 |
smcginnis | e0ne: Glad to see some progress in addressing them. Thanks for working on that. | 15:19 |
jgriffith | Ohhh | 15:19 |
jgriffith | projects :) | 15:19 |
jgriffith | derp | 15:19 |
jgriffith | never mind | 15:19 |
e0ne | smcginnis: np | 15:19 |
dulek | :) | 15:19 |
dulek | https://github.com/openstack/cinder/blame/master/cinder/db/sqlalchemy/models.py#L265 | 15:19 |
jgriffith | dulek: yeah | 15:20 |
e0ne | smcginnis: I'll try to setup some notifications for that job to fix asap if any failure | 15:20 |
smcginnis | e0ne: That would be good. I know it was failing for some time before I noticed. | 15:20 |
e0ne | smcginnis: yep:(. it's broken almost 1.5 months:( | 15:21 |
*** akapil has joined #openstack-cinder | 15:21 | |
smcginnis | :[ | 15:21 |
*** akapil has quit IRC | 15:21 | |
jgriffith | dulek: I added a comment to that patch | 15:22 |
jgriffith | dulek: https://review.openstack.org/#/c/315541/17 | 15:23 |
jgriffith | geguileor: dulek So I do like consolidation, but I think it can be taken too far | 15:23 |
*** akapil has joined #openstack-cinder | 15:24 | |
*** akapil has quit IRC | 15:24 | |
*** akapil has joined #openstack-cinder | 15:24 | |
*** akapil has quit IRC | 15:24 | |
dulek | jgriffith: 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 IRC | 15:25 | |
* dulek got to leave office and start preparing to watch Poland-Germany football (soccer) match today. :) | 15:25 | |
*** ihrachys_ has joined #openstack-cinder | 15:25 | |
jgriffith | dulek: cool... have a good nigth | 15:25 |
jgriffith | night | 15:26 |
dulek | smcginnis: Will you figure out if I was wrong with my bug report? | 15:26 |
dulek | smcginnis: On the compacted migrations. | 15:26 |
smcginnis | dulek: Not yet. Getting weirdness trying to generate the before and after schema. | 15:26 |
dulek | smcginnis: Oh, maybe I can help with that? | 15:26 |
smcginnis | dulek: 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 |
dulek | smcginnis: You need to reinstall Cinder python package? | 15:27 |
smcginnis | dulek: I did. Then I completely blew all references to it I could find away. Same thing. | 15:27 |
dulek | smcginnis: My another problem were .pyc files laying around. | 15:27 |
dulek | smcginnis: And are you dropping whole Cinder DB? | 15:28 |
smcginnis | dulek: That's what I thought, but did full "rm -fr /usr/local/lib/python2.7/dist-packages/cinder*" and still there. | 15:28 |
smcginnis | Just using a local config file and generating a new one in the current directory. | 15:29 |
smcginnis | cinder-manage --config-file dbtest.conf db sync | 15:29 |
smcginnis | Generates to ./cinder.sqlite | 15:29 |
dulek | smcginnis: Hm. I was actually running live MySQL. | 15:29 |
smcginnis | dulek: Thought I'd start simple with sqlite, but maybe I'll just do that. | 15:30 |
smcginnis | But odd. | 15:30 |
*** faiz89 has joined #openstack-cinder | 15:30 | |
smcginnis | Think I'll spin up a new fresh VM to try the pre-compaction first. | 15:30 |
smcginnis | There are differences between sqlite and mysql, so I should do that too. | 15:30 |
dulek | Okay. I'll try to take a look on IRC later today. | 15:32 |
*** geguileo_mobile has quit IRC | 15:33 | |
*** openstackgerrit has quit IRC | 15:34 | |
*** openstackgerrit has joined #openstack-cinder | 15:35 | |
*** geguileo_phone has joined #openstack-cinder | 15:37 | |
*** eharney_ has joined #openstack-cinder | 15:38 | |
*** eharney has quit IRC | 15:39 | |
nikeshm | eharney: i replied in patch , please let me know if i need to change still | 15:40 |
nikeshm | in the code | 15:40 |
nikeshm | after that explanation | 15:40 |
*** eharney_ is now known as eharney | 15:44 | |
smcginnis | dulek: Got things working. I am seeing some differences. Will work on figuring out what/why now. | 15:45 |
*** EinstCrazy has quit IRC | 15:53 | |
*** jungleboyj has quit IRC | 15:53 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 15:55 |
*** belmoreira has joined #openstack-cinder | 15:56 | |
*** alyson_ has joined #openstack-cinder | 15:56 | |
*** e0ne has quit IRC | 15:56 | |
*** fernnest has joined #openstack-cinder | 15:57 | |
*** timcl has quit IRC | 15:57 | |
*** garthb has joined #openstack-cinder | 15:58 | |
*** akshai has joined #openstack-cinder | 15:58 | |
*** Apoorva has joined #openstack-cinder | 15:58 | |
*** tesseract has quit IRC | 16:00 | |
*** mriedem1 has joined #openstack-cinder | 16:00 | |
*** kaisers_ has quit IRC | 16:01 | |
*** akshai_ has quit IRC | 16:01 | |
*** mriedem has quit IRC | 16:02 | |
*** shausy has quit IRC | 16:02 | |
*** mriedem1 is now known as mriedem | 16:03 | |
*** jgregor has quit IRC | 16:05 | |
*** jungleboyj has joined #openstack-cinder | 16:06 | |
*** tyrola has quit IRC | 16:06 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/329677 | 16:07 |
*** akshai_ has joined #openstack-cinder | 16:10 | |
*** akshai has quit IRC | 16:13 | |
*** permalac has quit IRC | 16:15 | |
*** sborkows has quit IRC | 16:18 | |
*** sdake has quit IRC | 16:20 | |
*** laughterwym has quit IRC | 16:23 | |
*** enriquetaso has joined #openstack-cinder | 16:27 | |
*** cbader has joined #openstack-cinder | 16:28 | |
scottda | eharney: jgriffith Here's a patch to enable cinder multibackend tests: https://review.openstack.org/#/c/330678/ | 16:28 |
*** sdake has joined #openstack-cinder | 16:28 | |
scottda | And the main reason for that patch ^^^ is for testing cinder migration (retype --on-deman) with a volume attached (BFV): https://review.openstack.org/326681 | 16:32 |
*** bharath has joined #openstack-cinder | 16:34 | |
*** laughterwym has joined #openstack-cinder | 16:35 | |
*** david-lyle has quit IRC | 16:35 | |
*** faiz89 has quit IRC | 16:36 | |
*** jordanP has quit IRC | 16:37 | |
*** kfarr has quit IRC | 16:38 | |
*** laughterwym has quit IRC | 16:39 | |
*** Lee1092 has joined #openstack-cinder | 16:40 | |
*** suyash has quit IRC | 16:41 | |
*** geguileo_phone has quit IRC | 16:44 | |
*** erlon has joined #openstack-cinder | 16:44 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 16:45 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 16:45 |
*** geguileo_phone has joined #openstack-cinder | 16:47 | |
*** david-lyle has joined #openstack-cinder | 16:51 | |
*** belmoreira has quit IRC | 17:00 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring https://review.openstack.org/285469 | 17:01 |
*** suyash has joined #openstack-cinder | 17:02 | |
*** mdenny has joined #openstack-cinder | 17:06 | |
*** mvk_ has quit IRC | 17:06 | |
*** akshai_ has quit IRC | 17:06 | |
*** salv-orl_ has joined #openstack-cinder | 17:08 | |
*** timcl has joined #openstack-cinder | 17:09 | |
*** salv-orlando has quit IRC | 17:12 | |
*** bharath has quit IRC | 17:12 | |
*** bharath has joined #openstack-cinder | 17:13 | |
*** rcernin has quit IRC | 17:21 | |
*** eharney has quit IRC | 17:23 | |
*** bauzas has quit IRC | 17:23 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 17:24 |
*** bauzas has joined #openstack-cinder | 17:24 | |
*** ihrachys_ has quit IRC | 17:24 | |
*** geguileo_phone has quit IRC | 17:25 | |
*** faiz89 has joined #openstack-cinder | 17:26 | |
*** lpetrut has quit IRC | 17:27 | |
*** akshai has joined #openstack-cinder | 17:28 | |
*** angela-s has joined #openstack-cinder | 17:28 | |
*** eharney has joined #openstack-cinder | 17:28 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 17:32 |
*** xyang1 has joined #openstack-cinder | 17:33 | |
*** geguileo_phone has joined #openstack-cinder | 17:34 | |
*** ChubYann has joined #openstack-cinder | 17:38 | |
*** xinli has joined #openstack-cinder | 17:38 | |
*** mvk_ has joined #openstack-cinder | 17:39 | |
*** faiz89 has quit IRC | 17:42 | |
*** rlrossit_ has quit IRC | 17:42 | |
openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 17:43 |
*** faiz89 has joined #openstack-cinder | 17:43 | |
*** xinli has quit IRC | 17:43 | |
*** bbhavsar_ has quit IRC | 17:44 | |
openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 17:44 |
*** xinli has joined #openstack-cinder | 17:46 | |
*** eharney has quit IRC | 17:46 | |
openstackgerrit | Merged openstack/cinder: Support None value of extra_specs in cinder-scheduler filter https://review.openstack.org/328597 | 17:55 |
openstackgerrit | Merged openstack/cinder: Start/Stop coordinator with Volume service https://review.openstack.org/263313 | 17:55 |
openstackgerrit | Merged openstack/cinder: Replace locks in volume manager https://review.openstack.org/185646 | 17:56 |
xinli | dulek: could you please help to review the code for patch 251989: https://review.openstack.org/#/c/251989/ | 17:58 |
*** mtanino has quit IRC | 17:59 | |
openstackgerrit | Merged openstack/cinder: Handle quota exceed exception https://review.openstack.org/281666 | 18:01 |
openstackgerrit | Merged openstack/cinder: Nimble driver should return ceiling of volume size https://review.openstack.org/325513 | 18:01 |
*** rlrossit has joined #openstack-cinder | 18:04 | |
*** kmartin has quit IRC | 18:04 | |
*** ebalduf has joined #openstack-cinder | 18:06 | |
*** rcernin has joined #openstack-cinder | 18:07 | |
*** akshai has quit IRC | 18:08 | |
*** akshai has joined #openstack-cinder | 18:11 | |
*** mtanino has joined #openstack-cinder | 18:11 | |
*** eharney has joined #openstack-cinder | 18:14 | |
openstackgerrit | Alyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 18:18 |
*** bardia has joined #openstack-cinder | 18:18 | |
*** bharath has quit IRC | 18:19 | |
openstackgerrit | Merged openstack/cinder: Provide consistency to "Openstack-Api-Version" header https://review.openstack.org/309878 | 18:20 |
*** timcl has quit IRC | 18:21 | |
*** lpetrut has joined #openstack-cinder | 18:22 | |
*** mriedem1 has joined #openstack-cinder | 18:23 | |
*** mkoderer__ has joined #openstack-cinder | 18:24 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix improperly configured privsep https://review.openstack.org/329586 | 18:24 |
smcginnis | dulek: At least with sqlite, I've found two differences pre-compact commit and post. | 18:25 |
smcginnis | dulek: 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 IRC | 18:26 | |
smcginnis | The 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 |
smcginnis | That wasn't there before, but I think it actually should be since it's a bool field. | 18:26 |
*** kmartin has joined #openstack-cinder | 18:27 | |
*** mriedem1 is now known as mriedem | 18:29 | |
*** faiz89 has quit IRC | 18:34 | |
*** faiz89 has joined #openstack-cinder | 18:34 | |
*** timcl has joined #openstack-cinder | 18:42 | |
*** sandanar has joined #openstack-cinder | 18:44 | |
*** akshai_ has joined #openstack-cinder | 18:45 | |
*** akshai has quit IRC | 18:47 | |
*** akshai_ has quit IRC | 18:50 | |
*** kmartin has quit IRC | 18:50 | |
*** sandanar_ has joined #openstack-cinder | 18:50 | |
*** sandanar has quit IRC | 18:50 | |
*** amit213 has joined #openstack-cinder | 18:51 | |
*** sage__ has joined #openstack-cinder | 18:51 | |
*** rlrossit has quit IRC | 18:52 | |
*** salv-orl_ has quit IRC | 18:54 | |
*** salv-orlando has joined #openstack-cinder | 18:55 | |
*** timcl1 has joined #openstack-cinder | 18:56 | |
*** e0ne has joined #openstack-cinder | 18:58 | |
*** timcl has quit IRC | 18:58 | |
*** sgotliv_ has quit IRC | 19:02 | |
*** ociuhandu has quit IRC | 19:02 | |
*** sandanar_ has quit IRC | 19:06 | |
*** sandanar has joined #openstack-cinder | 19:07 | |
*** sandanar has quit IRC | 19:09 | |
*** ebalduf has quit IRC | 19:09 | |
*** lpetrut has quit IRC | 19:13 | |
openstackgerrit | Xinli Guan proposed openstack/cinder: Move QualityOfServiceSpecs to use VersionedObject https://review.openstack.org/251989 | 19:14 |
*** geguileo_mobile has joined #openstack-cinder | 19:14 | |
*** zhongjun_ has quit IRC | 19:14 | |
*** zhongjun_ has joined #openstack-cinder | 19:15 | |
*** geguileo_phone has quit IRC | 19:17 | |
*** e0ne has quit IRC | 19:22 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Correct DB modifications from compaction https://review.openstack.org/330756 | 19:22 |
smcginnis | dulek, eharney: Think I got it. ^^ | 19:22 |
*** ociuhandu has joined #openstack-cinder | 19:37 | |
*** asselin_ has joined #openstack-cinder | 19:38 | |
*** asselin__ has quit IRC | 19:39 | |
*** lpetrut has joined #openstack-cinder | 19:46 | |
openstackgerrit | Alyson proposed openstack/cinder: HNAS: Add support for consistency groups in iSCSI driver https://review.openstack.org/327043 | 19:48 |
*** rlrossit has joined #openstack-cinder | 19:51 | |
gouthamr | jgriffith: ping | 19:53 |
jgriffith | gouthamr: pong | 19:53 |
*** faiz89 has quit IRC | 19:53 | |
gouthamr | jgriffith: hi! :) i'm trying to implement cheesecake replication on cDOT... I had a couple of questions, now a good time? | 19:53 |
jgriffith | sure | 19:53 |
gouthamr | jgriffith: nice.. okay, noticed that solidfire doesn't do the failover to "default" | 19:54 |
gouthamr | jgriffith: is there some hidden sauce i'm not reading right, or that kind of failover (failback if you will :)) not supported.. | 19:54 |
jgriffith | gouthamr: so the use case we outlined there is no "failback" | 19:57 |
jgriffith | but that's something folks are adding now | 19:57 |
gouthamr | jgriffith: so is it recommended that we do that? | 19:57 |
jgriffith | gouthamr: Well the problem is people are going to want the failback | 19:57 |
jgriffith | gouthamr: I personally would like to see it made part of the existing failover cmd | 19:57 |
jgriffith | gouthamr: whether that be having it work like a toggle, or taking an additional option | 19:58 |
jgriffith | gouthamr: I just don't want a "new" api method | 19:58 |
jgriffith | gouthamr: I think patrickeast and Swanson have implemeneted something here | 19:58 |
gouthamr | jgriffith: +1, i agree.. we could standardize this way of doing it and update the devref so all vendors can do this the same way | 19:59 |
patrickeast | oh yea | 19:59 |
gouthamr | jgriffith: coming from the cinderclient and not seeing the "default" trigger in the API is a bit unsettling.. | 19:59 |
patrickeast | i did the default thingy in the pure drivers | 19:59 |
Swanson | I did what patrickeast did. | 19:59 |
gouthamr | :D | 19:59 |
*** chris_morrell has joined #openstack-cinder | 20:00 | |
* patrickeast isn't sure how to feel about having implemented that particular feature | 20:00 | |
jgriffith | patrickeast: dirty, cheap and hollow inside | 20:00 |
gouthamr | patrickeast: did you plan on adding documentation for it? | 20:00 |
patrickeast | gouthamr: 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 state | 20:00 |
*** salv-orlando has quit IRC | 20:00 | |
*** ebalduf has joined #openstack-cinder | 20:01 | |
*** lprice has quit IRC | 20:01 | |
patrickeast | gouthamr: nope, i'm pushing back on that really hard | 20:01 |
*** salv-orlando has joined #openstack-cinder | 20:01 | |
*** lprice has joined #openstack-cinder | 20:01 | |
patrickeast | because i kinda expect it to change or go away | 20:01 |
Swanson | gouthamr, 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 IRC | 20:02 | |
gouthamr | patrickeast: 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 |
smcginnis | jgriffith: LOL, just reading scrollback and got to your last comment. :D | 20:02 |
patrickeast | jgriffith: smcginnis: its pretty accurate :D | 20:03 |
*** akerr has quit IRC | 20:03 | |
*** geguileo_mobile has quit IRC | 20:03 | |
gouthamr | alright, sounds like we need to hash this out a little more.. | 20:04 |
patrickeast | jgriffith: its almost as bad as my drivers db calls :P | 20:04 |
patrickeast | gouthamr: yea, if I were you Id wait | 20:04 |
gouthamr | the other question i had was regarding the 'replication_device' option | 20:04 |
gouthamr | i see that the spec and the devref had the key as 'backend_id' | 20:04 |
gouthamr | and the config opt has 'replication_target_id' | 20:04 |
patrickeast | mm i think backend_id is the right one | 20:05 |
patrickeast | which means that config opt probably should be updated | 20:05 |
*** sheel has quit IRC | 20:05 | |
gouthamr | patrickeast: yep.. | 20:05 |
gouthamr | there's already a couple vendor drivers that use replication_target_id | 20:06 |
*** cknight1 has quit IRC | 20:06 | |
smcginnis | Added replication failback to the midcycle topics. | 20:06 |
patrickeast | gouthamr: well, iirc it doesn't matter too much, as long as its what your driver is expecting | 20:06 |
gouthamr | sorry.. | 20:06 |
gouthamr | "target_device_id" | 20:07 |
patrickeast | smcginnis: good idea | 20:07 |
gouthamr | smcginnis: +1 | 20:07 |
*** merooney has quit IRC | 20:07 | |
gouthamr | smcginnis: i should guess, you added it back :) | 20:07 |
*** geguileo_phone has joined #openstack-cinder | 20:08 | |
gouthamr | jgriffith patrickeast: thanks! i might bother you guys again. | 20:08 |
patrickeast | np | 20:08 |
*** porrua has quit IRC | 20:08 | |
*** xinli has quit IRC | 20:10 | |
*** timcl1 has quit IRC | 20:10 | |
*** faiz89 has joined #openstack-cinder | 20:11 | |
*** rlrossit has quit IRC | 20:13 | |
*** akshai has joined #openstack-cinder | 20:13 | |
*** rlrossit has joined #openstack-cinder | 20:14 | |
openstackgerrit | Patrick East proposed openstack/cinder-specs: Shared backend config stanza https://review.openstack.org/330767 | 20:14 |
*** openstackstatus has joined #openstack-cinder | 20:18 | |
*** ChanServ sets mode: +v openstackstatus | 20:18 | |
*** rlrossit has quit IRC | 20:18 | |
smcginnis | patrickeast: Nice! ^ | 20:19 |
*** zul has quit IRC | 20:21 | |
*** zul has joined #openstack-cinder | 20:22 | |
patrickeast | smcginnis: 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-cinder | 20:22 | |
smcginnis | patrickeast: I haven't read through, but sounds like basically what we talked about back in Tokyo, right? | 20:25 |
patrickeast | smcginnis: yep, basically the same thing | 20:25 |
smcginnis | patrickeast: Yay! Glad to see movement there. | 20:25 |
*** rlrossit has joined #openstack-cinder | 20:27 | |
*** ihrachys has joined #openstack-cinder | 20:27 | |
*** jungleboyj has quit IRC | 20:33 | |
*** lpetrut has quit IRC | 20:38 | |
*** arch-nemesis has quit IRC | 20:41 | |
*** daneyon has quit IRC | 20:42 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3 https://review.openstack.org/330778 | 20:45 |
*** hemanthm is now known as hemanthm|afk | 20:46 | |
patrickeast | jgriffith: 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 IRC | 20:50 | |
*** lpetrut has joined #openstack-cinder | 20:51 | |
*** ihrachys has quit IRC | 20:52 | |
*** zul has quit IRC | 20:52 | |
*** ihrachys has joined #openstack-cinder | 20:53 | |
*** arch-nemesis has joined #openstack-cinder | 20:56 | |
*** jungleboyj has joined #openstack-cinder | 20:57 | |
smcginnis | patrickeast: That may be what I was talking about too the other day. | 20:57 |
smcginnis | patrickeast: For single backend you can just put the driver in default. | 20:57 |
smcginnis | patrickeast: But then if you want to add another backend, ruh roh. | 20:58 |
patrickeast | smcginnis: ohh, make it so you can mix them? | 20:58 |
smcginnis | patrickeast: It would be nice to enforce somehow that all backends need to be defined as we do for multibackend. | 20:58 |
smcginnis | patrickeast: So make it so you HAVE to put them in their own sections. | 20:58 |
patrickeast | smcginnis: yea that would be awesome, but probably needs some kind of deprecation period, eh? | 20:59 |
patrickeast | hah, or start doing config file versions | 20:59 |
smcginnis | patrickeast: 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 |
patrickeast | smcginnis: we could probably make it so we treat [DEFAULT] as a backend in multibackend too but log a big warning | 21:00 |
smcginnis | I think that's what jgriffith was referring to, but I could be wrong. I'd at least like to see it though. | 21:00 |
smcginnis | patrickeast: That could work. | 21:00 |
patrickeast | i'll play around with it and see what works | 21:00 |
patrickeast | find some nice middle-ground between actually works without too much hacking and does what we want | 21:00 |
smcginnis | patrickeast: Awesome. | 21:01 |
*** sgotliv_ has joined #openstack-cinder | 21:02 | |
openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 21:03 |
*** merooney has joined #openstack-cinder | 21:03 | |
*** faiz89 has joined #openstack-cinder | 21:03 | |
*** arch-nemesis has quit IRC | 21:03 | |
*** faiz89 has quit IRC | 21:05 | |
*** sgotliv_ has quit IRC | 21:07 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 21:08 |
*** faiz89 has joined #openstack-cinder | 21:08 | |
*** diablo_rojo has quit IRC | 21:11 | |
*** akapil has joined #openstack-cinder | 21:12 | |
*** cknight has joined #openstack-cinder | 21:14 | |
*** gouthamr has quit IRC | 21:15 | |
*** krtaylor has quit IRC | 21:15 | |
*** pauloewerton has quit IRC | 21:15 | |
*** dustins has quit IRC | 21:15 | |
*** arch-nemesis has joined #openstack-cinder | 21:17 | |
*** eharney has quit IRC | 21:19 | |
*** ihrachys_ has joined #openstack-cinder | 21:19 | |
*** bardia has quit IRC | 21:21 | |
*** ihrachys has quit IRC | 21:22 | |
*** Lee1092 has quit IRC | 21:23 | |
*** faiz89 has quit IRC | 21:24 | |
*** laughterwym has joined #openstack-cinder | 21:25 | |
*** laughterwym has quit IRC | 21:29 | |
*** porrua has joined #openstack-cinder | 21:31 | |
*** rlrossit has quit IRC | 21:32 | |
*** enriquetaso has quit IRC | 21:32 | |
*** lprice has quit IRC | 21:33 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix improperly configured privsep https://review.openstack.org/329586 | 21:33 |
*** lprice has joined #openstack-cinder | 21:33 | |
*** diogogmt has quit IRC | 21:34 | |
*** diogogmt has joined #openstack-cinder | 21:36 | |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file https://review.openstack.org/294076 | 21:36 |
*** rcernin has quit IRC | 21:36 | |
*** gouthamr has joined #openstack-cinder | 21:37 | |
*** lpetrut has quit IRC | 21:39 | |
*** lprice has quit IRC | 21:50 | |
*** mriedem has quit IRC | 21:51 | |
*** mriedem has joined #openstack-cinder | 21:52 | |
*** mriedem has quit IRC | 21:52 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 21:58 |
openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Use Live Volume for replication https://review.openstack.org/329531 | 21:59 |
*** gouthamr_ has joined #openstack-cinder | 22:04 | |
*** xyang1 has quit IRC | 22:04 | |
*** porrua has quit IRC | 22:05 | |
*** gouthamr has quit IRC | 22:07 | |
*** lcurtis has joined #openstack-cinder | 22:08 | |
*** akapil has quit IRC | 22:09 | |
*** chris_morrell has quit IRC | 22:11 | |
*** akshai has quit IRC | 22:11 | |
hemna | jgriffith, 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 IRC | 22:23 | |
jgriffith | hemna: hmm... don't know why that thread keeps dieing | 22:26 |
jgriffith | rather annoying | 22:26 |
hemna | wasn't sure what was up, thought you should know | 22:26 |
jgriffith | hemna: yeah, thanks | 22:27 |
jgriffith | hemna: between the NTAP Exchange server blocking the email notifications I send and me being lazy :) | 22:27 |
jgriffith | I don't even notice it not running | 22:27 |
hemna | :) | 22:27 |
*** salv-orlando has quit IRC | 22:29 | |
*** salv-orlando has joined #openstack-cinder | 22:30 | |
*** jungleboyj has quit IRC | 22:30 | |
*** geguileo_phone has quit IRC | 22:31 | |
*** mriedem has joined #openstack-cinder | 22:32 | |
*** salv-orlando has quit IRC | 22:34 | |
*** salv-orlando has joined #openstack-cinder | 22:35 | |
*** geguileo_phone has joined #openstack-cinder | 22:36 | |
*** karthikp has joined #openstack-cinder | 22:36 | |
*** bardia has joined #openstack-cinder | 22:39 | |
*** asselin__ has joined #openstack-cinder | 22:45 | |
*** asselin_ has quit IRC | 22:47 | |
openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3 https://review.openstack.org/330778 | 22:47 |
*** karthikp has quit IRC | 22:49 | |
*** geguileo_phone has quit IRC | 22:51 | |
*** lcurtis has quit IRC | 22:54 | |
*** geguileo_phone has joined #openstack-cinder | 22:54 | |
*** alaski_ is now known as alaski | 23:02 | |
*** edmondsw has quit IRC | 23:07 | |
*** salv-orl_ has joined #openstack-cinder | 23:08 | |
*** diogogmt has quit IRC | 23:10 | |
*** salv-orlando has quit IRC | 23:11 | |
openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 23:12 |
*** akerr has joined #openstack-cinder | 23:14 | |
*** daneyon has joined #openstack-cinder | 23:16 | |
*** akerr has quit IRC | 23:17 | |
*** asselin has quit IRC | 23:17 | |
*** ihrachys_ has quit IRC | 23:20 | |
*** garthb has quit IRC | 23:20 | |
*** ihrachys has joined #openstack-cinder | 23:21 | |
openstackgerrit | Merged openstack/cinder: 3PAR: Fix delete volume when online clone https://review.openstack.org/327840 | 23:28 |
*** daneyon has quit IRC | 23:33 | |
*** angela-s has quit IRC | 23:34 | |
*** karthikp has joined #openstack-cinder | 23:35 | |
*** crinkle has left #openstack-cinder | 23:35 | |
*** cknight has quit IRC | 23:37 | |
*** cknight has joined #openstack-cinder | 23:37 | |
openstackgerrit | Merged openstack/cinder: Fix Brcd zone driver initiator zone update https://review.openstack.org/317061 | 23:37 |
*** cknight has quit IRC | 23:39 | |
*** cknight1 has joined #openstack-cinder | 23:39 | |
*** karthikp has quit IRC | 23:40 | |
*** chlong has quit IRC | 23:42 | |
*** cknight1 has quit IRC | 23:44 | |
*** cknight has joined #openstack-cinder | 23:44 | |
*** salv-orl_ has quit IRC | 23:47 | |
*** salv-orlando has joined #openstack-cinder | 23:48 | |
*** cknight1 has joined #openstack-cinder | 23:50 | |
*** cknight has quit IRC | 23:51 | |
*** ihrachys has quit IRC | 23:51 | |
*** Julien-zte has quit IRC | 23:52 | |
*** adrianofr has quit IRC | 23:52 | |
*** sdake has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!