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