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