14:59:47 #startmeeting cinder_testing 14:59:48 Meeting started Wed Aug 17 14:59:47 2016 UTC and is due to finish in 60 minutes. The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:59:49 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:59:51 The meeting name has been set to 'cinder_testing' 15:00:22 hey 15:00:28 #link https://etherpad.openstack.org/p/Cinder-testing 15:00:49 o/ 15:01:20 hi 15:02:00 From the agenda: 15:02:02 devstack-gate cinder multi-node patch has merged: https://review.openstack.org/#/c/327619/ 15:02:22 Is dulek around? 15:02:31 Sean McGinnis proposed openstack/os-brick: Add retries to iSCSI connect_volume https://review.openstack.org/356532 15:03:15 OK 15:03:20 Next on agenda: Add a multibackend list to tempest.conf has merged: https://review.openstack.org/#/c/325895 15:03:37 I'm waiting on re-approval of next patch in the series.... 15:04:27 Main thing is that I've updated the status of the patches under review... 15:04:52 I'll go to the QA meeting as well as infra channel and look for help on the ones that need final +2. 15:04:59 Any help with that would also be good. 15:05:16 and some patches have zero reviews, so please have a look. 15:07:40 scottda: Is there a common topic we can pull up all of them? 15:07:40 smcginnis: nope, not yet anyway 15:07:40 smcginnis: maybe...not sure. they are in at least 2 different repos: devstack-gate and devstack. 15:07:40 scottda: OK, probably easier just pulling up patches where you're the owner then. 15:07:41 I'll try to keep the status up-to-date on the etherpad. That may be the best way to track 'em 15:07:50 scottda: That sounds good. 15:08:16 Has anyone other than me tried geguileo 's manual tests for AA-HA? 15:08:20 #link http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/ 15:08:30 Works great. Ship it. 15:09:07 XD 15:09:34 geguileo: I haven't been able to run through and test, but very nice write up! 15:09:59 I hope it helps understand the flow a little bit better 15:10:16 Although it doesn't replace a good devref for adding clenable resources/states 15:11:03 geguileo: do you want to add that to cinder doc 15:11:17 I'm going to test them once I finish groups-related patches testing 15:11:39 xyang1: Not that one, I'll create a doc with an explanation on how to add a new cleanable resource/state 15:11:51 geguileo: ok 15:11:57 xyang1: I don't think adding that info to the cinder doc is that useful 15:12:13 It was more to help people get started in the testing and understand the pieces at play 15:12:16 A "how to" sounds useful. 15:12:21 :) 15:12:21 I'm going to re-test today with the latest set. And I threatened geguileo I would record it an put the webcast on youtube, as I did with api microversions. Some found that helpful... 15:12:31 Because sometimes it's hard just reading the specs 15:13:22 Anyone have anything test-wise to bring up? 15:13:46 scottda: I finally wrote up test description changes. 15:14:08 And switched around what we were saying for "integration" vs "functional", so interested in what folks think of that. 15:14:11 I'm working on fake drivers refactoring and devstack integration. I hope, all patches will be ready this week. 15:14:18 #link https://review.openstack.org/#/c/356177/ 15:14:29 e0ne: I saw you submitted a patch on the fake driver. so we should use that one for functional tests? 15:14:29 scottda: Beat me to it! 15:14:30 smcginnis: looks -1'd by you? 15:14:40 and yes, fake driver could be use for functional and integration tests 15:14:51 xyang1: I hope so 15:14:55 scottda: Just to make sure everyone has a chance to take a look and raise any objections before someone decides to just push it through. 15:15:13 xyang1: I didn't test it with devstack, so it's still in WIP status 15:15:28 smcginnis: Makes sense. Everyone please review. 15:15:30 e0ne: ok 15:16:16 #link https://review.openstack.org/356525 15:16:17 we need refatcor unit/test_volume.py to get split between unit, funcitonal and integration tests 15:16:23 e0ne: 's fake driver patch ^^ 15:16:31 e0ne: +1 15:16:48 correct #link https://review.openstack.org/348428 15:17:03 oops 15:17:18 scottda: that one is abandoned. I missed correct change-id 15:17:25 k 15:18:23 #link https://review.openstack.org/348449 - patch to devstack 15:18:42 xyang1: I'll add FakeGateDriver support to devstack patch ^^ too 15:18:55 e0ne: thanks:) 15:19:42 e0ne: Would you like to put those links in the etherpad? 15:20:07 scottda: I'll do once I test my patch with devstack 15:20:13 thanks 15:20:29 everybody is welcome to review ot now.. maybe I missed something 15:20:49 TBH, I removed few asserts from test_volume 15:21:38 So, in general, we're getting good reviews from cinder folks on these test patches. I still don't know how to get more QA/infra people involved, but there's been some progress. 15:23:11 ¿Hay mas? Anything else? 15:23:25 i had one thing maybe worth discussion 15:23:32 patrickeast: Bring it 15:23:46 in the in-tree cg tempest tests https://review.openstack.org/#/c/355271/ i added a new config opt group for tempest.conf 15:23:49 Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 15:24:10 instead of re-using/adding to existing groups like "volume" or "volume-features-enabled" (or whatever it is) 15:24:48 wanted to make sure no one was strongly opposed to that... it is kinda weird to have so many sections basically for configuring volume/cinder stuff 15:26:14 patrickeast: Thanks. We'll see what people think in the review... 15:26:28 scottda: haha yea, i take silence as a resounding "ship it" 15:27:21 :) 15:27:45 alrighty then....we can finish unless there are other items? 15:28:13 #endmeeting