09:00:42 #startmeeting qa 09:00:43 Meeting started Thu Dec 10 09:00:42 2015 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. 09:00:44 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 09:00:46 The meeting name has been set to 'qa' 09:00:57 hi, who's here today? 09:01:01 o/ 09:01:02 o/ 09:01:02 Hi gmann o/ 09:01:08 dmellado: hi 09:01:14 jlanoux: hi 09:01:34 hi gmann 09:01:43 let's for min to join more people 09:01:47 sure 09:02:07 hi 09:02:28 ylobankov: hi 09:02:44 ok let's start 09:02:51 #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Proposed_Agenda_for_December_10th_2015_.280900_UTC.29 09:02:59 && Today meeting agenda 09:03:18 #topic Specs Reviews 09:03:38 we have most of spec under review 09:03:41 #link https://review.openstack.org/#/q/status:open+project:openstack/qa-specs,n,z 09:04:15 andrea updated tempest-resource spec, good to review that 09:04:18 #link https://review.openstack.org/#/c/173334/ 09:04:19 gi 09:04:20 hi 09:04:27 jordanP: hi 09:04:27 morning jordanP ;) 09:04:50 There are other 2 in list of agenda 09:04:53 # https://review.openstack.org/#/c/230183/ 09:05:17 I have not gone through it yet, but its under review 09:05:25 anyone has anything on this to discuss 09:05:58 anyways next one is list plugin one 09:06:04 #link https://review.openstack.org/#/c/247575/ 09:06:13 I haven't gone over there yet, but I'd like to somehow couple the config script inners with the bp-resources 09:06:20 this one was straight forward 09:06:22 I'll update it when I've some time 09:06:29 dmellado: cool, Thanks 09:06:38 plugin list one is merged. 09:06:54 yep 09:07:11 I also found difficulty while debugging ironic plugin which was in progress 09:07:23 jordanP: yea it will be nice addition 09:07:34 anything on spec 09:07:58 or any other spec anyone want to bring up for discussion 09:08:06 I have one - https://review.openstack.org/#/c/255023/ 09:08:27 dwalleck: sure, please go ahead 09:08:54 There's been a lot of discussion about how the remote client is tied to very generic Linux implementation. This spec allows for an extendable implementation 09:09:30 I know there's some work around this in the resources spec, but I don't think it will go far enough to solve the problem. I put this up as another possibility to consider 09:09:54 dwalleck: i see, so it will be having drives for each connectivity protocol ? 09:10:09 dwalleck, I like this spec. The current linux client and implementation is a bit all over the place, a refactor and architecture change would be good 09:10:23 with more consistency 09:10:25 +also adding support for the dreaded windows 09:10:27 xD 09:10:43 jordanP: yea looks like nice one 09:10:45 but, are we *that* interested in testing something else than Linux ? 09:10:49 gmann: Right. A driver per protocol, and the remote client implementation picks the driver it needs 09:10:57 I mean it's gonna be some work, you sure it's worth it ? 09:11:15 dwalleck: 1 question does this one contain how our tests goind to switch between those 09:11:18 The dreaded windows work is done :D If you check one of the bottom links, I link to an external implementation 09:11:28 dwalleck: ok, so remote client will pick up dynamically 09:11:29 dwalleck: jordanP: also, will be the gates running anything else? otherwise this might end up being outdated in case it gets implemented 09:11:50 dmellado: nice point 09:11:57 gmann: My thought was to have it be a property in the resource file for the image 09:12:16 dwalleck: i see. 09:12:21 dwalleck: Thanks 09:12:34 dmellado: We run a lot of Windows gates at Rackspace. But yes, testing of multiple drivers is one of the possible pain points 09:12:54 well at least if it's unit tested, it's a good start 09:13:16 But I'd be more than glad to help with that. I also suggest making the remote client extensions plugins, which would put the maintenance back to the developer of the plugin 09:13:23 +1 from me, if we can couple it with unit tests 09:13:50 and if missing one can be added as we planed for resource things 09:13:59 anyways its good one for review and lets start reviewing it and have comment/suggestion there 09:14:05 dwalleck: Thanks . 09:14:11 let's move on 09:14:15 thanks everyone :-) 09:14:18 dwalleck: I'll take a look later and add comments, thanks ;) 09:14:33 #topic Priority Items 09:14:41 dmellado: Thanks, that will be nice 09:15:06 so we have priority items decided in Summit with their targeted deadline 09:15:14 #link https://etherpad.openstack.org/p/mitaka-qa-priorities 09:15:28 first one is Tempest Microversion support 09:15:34 this missed the M1 :( 09:15:50 but patches are in review and good to merge soon 09:16:05 lets discuss this later 09:16:48 Other service client migration we will talk later in agenda 09:17:04 jordanP: one is your 09:17:10 #link Tempest Lib Migrations 09:17:16 yep 09:17:23 jordanP: do you have anything on this 09:17:53 hummm now that andreaf and jlanoux have completed their work, I think I can start to thing about this migration 09:18:03 *think 09:18:09 jordanP: cool 09:18:22 other than that I don"t know, it shouldn't be too hard 09:18:35 jordanP: It is not completed yet. For sure, one change needs to get in, perhaps another one. Then, I'll take care of compute and remote. 09:18:39 jlanoux: jordanP anything pending for ssh thing, need review etc 09:18:52 * andreaf sneaks in 09:18:54 jlanoux: you have link > 09:19:04 #link https://review.openstack.org/#/c/253444/ 09:19:04 andreaf: hi :) 09:19:18 jlanoux: Thanks 09:19:25 i will have look tomorrow 09:19:39 And I don't know if I'll make the wrapper around compute and remote before the migration of after. 09:19:46 *or 09:20:01 gmann: thanks 09:20:11 jlanoux: its better to make it before migration 09:20:27 jlanoux, count me in to review your work 09:20:30 gmann: ok - I'll do that than 09:20:34 jordanP: thanks 09:20:38 *then 09:20:41 jlanoux: after migration we have to maintain all published interfaces with backward compatibility mode 09:20:48 jordanP: great Thanks 09:20:54 gmann: ok 09:21:00 Next one is Tempest CLI Improvements 09:21:35 #link https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/tempest-cli-improvements,n,z 09:21:41 I'm about 2 hours from done with the "tempest run" spec 09:21:42 There are lot work going on 09:21:54 dwalleck: awesome. 09:22:04 masayukig: you there? 09:22:23 i think not :) 09:22:41 Let's move on 09:22:48 #topic Eslint-config-openstack review approval policy 09:23:03 krotscheck : your turn 09:23:38 not sure about this one, anyone has any idea 09:23:50 not really :\ 09:23:57 ok, let's move 09:24:09 #topic Tempest 09:24:21 first one is Service client from oomichi 09:24:29 he updated the progress there 09:24:41 #link https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/consistent-service-method-names,n,z 09:24:52 review progress and work is going good 09:25:05 but still keep doing more and more reviews on those 09:25:34 I'm trying to reviews as many as possible of those 09:25:55 another thing he mentioned is to update patches link on etherpad to track those correctly 09:25:56 I'd appretiate some reviews, once we deal with what to do with javelin ;) 09:25:57 #link https://etherpad.openstack.org/p/mitaka-tempest-service-clients 09:26:12 andreaf: Thanks, it will make thing fast 09:26:27 dmellado: sure, we will discuss on that next 09:26:51 any query on service client work 09:27:13 one i have about volume client doc string update 09:27:26 as we have base client and v1 and v2 on separate classes/file 09:27:49 I think updating v2 doc path is nice as jordanP done in QOS client 09:28:10 yeah v1 is already deprecated 09:28:15 soon to be removed 09:28:21 because there might be question that those base client are for both 09:28:35 jordanP: yea that is my consideration 09:28:41 ok 09:29:05 next is microversion testing support 09:29:09 #link https://review.openstack.org/#/q/status:open++branch:master+topic:bp/api-microversions-testing-support,n,z 09:29:30 as i mentioned it has missed the M1 :( but i think that will be ok 09:29:46 andreaf: i updated devstack and Config patches as per discussion 09:30:00 andreaf: please review those 09:30:13 gmann: cool thanks, I will do after the meeting 09:30:19 andreaf: Thanks 09:30:41 also there are couple of patches there for schema conversioning etc 09:31:10 and last one is to test Nova 2.2 version testing which can be checked to make sure all framework works fine 09:31:13 #link #link https://review.openstack.org/#/q/status:open++branch:master+topic:bp/api-microversions-testing-support,n,z 09:31:25 reviews from everyone are welcome and helpful 09:32:30 let's move on 09:32:35 next one is Javelin script testing 09:32:53 what is javelin exactly ? :) 09:33:11 jordanP: I didn't know that too 09:33:14 as we observed in keystone client separation work, this script was untested on gate 09:33:19 and was asking sdague and checking 09:33:28 and it seems that it was a script used in the beginning of grenade 09:33:31 to create resources 09:33:41 jordanP: dmellado that was actually for creating/deleting resources which was used for upgrade testing 09:33:47 and it isn't used anymore at all, in sdague's words -in grenade- 09:34:16 dmellado: yes, it was removed from grenade 09:34:24 #link https://review.openstack.org/#/c/186559/ 09:34:26 EmilienM, ping 09:34:32 so my point is: shouldn't we remove it as well from tempest if it's not going to be used? 09:34:34 hum it's night time for him 09:34:37 I think this patch ^^ 09:34:55 dmellado: yea nice point 09:35:18 dmellado: i had same discussion with oomichi and he also feels to remove it 09:35:30 so if there is no more usage of this script then we can remove 09:35:31 +1 from me 09:35:51 otherwise we have to modify its unit tests to cover more at deep level 09:35:51 I'll update the patch removing grenade+tests 09:36:09 if that's ok for everyone 09:36:13 dmellado: you mean javelin script? 09:36:24 don't remove grenade ! :) 09:36:34 jordanP: heh :) 09:36:35 gmann: yep, javelin script + test_javelin 09:36:54 dmellado: awesome, please add mtreinish and sdague as reviewer 09:37:00 if anything we are missing about that 09:37:01 gmann: will do 09:37:02 dmellado, gmann, jordanP: I think we may want to bring this to the ML first? 09:37:05 dmellado: cool 09:37:07 As far as I understood this script was used only for grenade. Now it is not used. So I think it is OK to remove the script. 09:37:15 that is wise indeed 09:37:18 andreaf: will do. 09:37:27 we might have hidden users 09:37:29 #action dmellado to remove Javelin script and get feedback on review 09:37:39 I was speaking yesterday with sdague and he was ok, but I'll send a mail just in case 09:37:48 andreaf: ok that is also nice way before going ahead on gerrit 09:37:54 yes 09:37:55 could you review the current patch? we can always modify it later on 09:38:05 dmellado: thanks 09:38:26 #action dmellado to first send ML about removal of Javelin from tempest 09:38:44 jordanP: yea 09:39:04 jordanP: may be someone using it explicitly 09:39:29 ok next one 09:39:30 Neutron LBAAS Scenario Tests 09:39:36 dmellado: your turn please 09:39:56 Basically I'm not sure about the current scope on this and wanted to have feedback. 09:40:09 IIUC, the scenario tests were *not* going to be pulled from main tempest 09:40:16 dmellado: actually they were removed from tempest 09:40:24 what's the idea for the LBAAS, FWAAS and so on? 09:40:27 #link https://review.openstack.org/#/c/186559/ 09:40:31 what's the status of the plugin interface for neutron? 09:40:43 are they not being covered in tempest jobs now, but in neutron? 09:40:54 dmellado: i think your main concern is to run those along with tempest tests right? 09:41:01 gmann: exactly 09:41:20 dmellado: yea so i will say they should have plugin there as you mentioned 09:41:31 but not sure status of that 09:41:34 gmann: but it's the plugin interface for neutron done yet? 09:41:38 who was tackling that? 09:41:50 dmellado: I do not know exactly 09:42:01 otherwise, it feels strange for me to remove then while the interface's not ready 09:42:10 s/then/them 09:42:45 it was question from yfried also many times 09:43:09 dmellado: actually they removed to keep the ownership for those tests cases 09:43:15 which can always be run from their repo 09:43:47 but yea plugin support is always nice to make them run from single entity (Tempest) 09:43:56 but wouldn't that 'split' tempest ownership? I'm well aware of 'tests to its plrojects' 09:44:12 but always thought that as scenario tests were using multiple ones they'd stay within Tempest 09:44:39 it just feels strange to me, tbh 09:44:54 but I can check with dougw 09:45:28 I've also a question, so from now any new LBAAS test should go there? and how about FWAAS or designate? 09:45:32 dmellado: scenario tests for the core 6 services can be in tempest, as they are good integration tests and they run all the time 09:46:04 andreaf: wouldn't lbaas be a part of neutron? or am I just missing something? 09:46:10 dmellado: but lbaas and fwaas are kind of independent from neutron 09:46:17 dmellado: yea scenario tests can be added in tempest as per Big tent scope 09:47:00 dmellado: andreaf yea, they have their own repo and tests ownership etc 09:47:02 dmellado: it's kind of an extra feature, it's not something the other services would heavily rely on for the basic working 09:47:21 gmann: andreaf thanks 09:47:23 dmellado: i remember removal things was for neutron advance services 09:47:32 only 09:47:35 I assume that'll be the same for every neutron 'advance services' 09:47:40 so I think it makes sense for those to run on fwaas / lbaas / neutron changes only 09:47:45 dmellado: we have all neutron tests 09:48:06 dmellado: exactly for all advance services 09:48:13 I see, 09:48:27 so andreaf how are the fwaas/lbaas triggered then on the neutron changes? 09:48:36 do we have a triggered job for it? 09:48:58 or will they be relying on the so called pluggable interface, as Monasca and so on? 09:49:04 dmellado: at the moment they have use pre/post run hooks to setup things and run those tests 09:49:22 andreaf: ack, thanks for all the clarifications 09:49:41 dmellado: using a plugin would be nicer, I agree, but it's not there yet 09:49:58 dmellado: but its not like they should not write plugin :) 09:50:06 as the plugin is a stable interface to add things into tempest, other things may break over time 09:50:08 andreaf: gmann thanks for the summary! 09:50:14 dmellado: how about writing for them 09:50:26 andreaf: nice point 09:50:42 gmann: is there any patch that I could use as an example? If so I could tackle that 09:51:11 dmellado: yea, i have couple of from Congress(my teammate worked on those) 09:51:25 dmellado: and help on Ironic one which is in progress 09:51:30 then I'll ask you after the meeting ;) 09:51:38 dmellado: i will provide the link on qa channel after meeting 09:51:44 dmellado: sure, Thanks 09:51:46 dmellado, gmann: well the idea of plugins is to scale better so project teams can maintain their tests - so if we start doing it it's kind of the opposite direction :D but sure you can do that talk to the fwaas team 09:51:46 thanks gmann 09:52:13 andreaf: yea i just pointed on dmellado :) 09:52:25 andreaf: I see :D 09:52:33 anyways lets move on 09:53:00 we will skip Devstack and grenade if nothing from anyone 09:53:39 let's move on 09:53:43 #topic OpenStack Health 09:53:50 andreaf: do you have anything on this 09:54:24 gmann: well the forth page is now up and running :) the DB migration from uuid to int keys complete 09:54:58 #link http://status.openstack.org/openstack-health/#/ 09:55:01 andreaf: cool 09:55:26 see for instance http://status.openstack.org/openstack-health/#/test/tempest.api.identity.admin.v3.test_groups.GroupsV3TestJSON.test_group_update_with_few_fields 09:55:37 nova is around 97% pass :) 09:56:07 we had for M1 to deliver group-by and filter-by capabilities, but that's still WIP 09:56:18 andreaf: any failure example, can we see log from there? 09:56:23 group-by is likely to be the next main feature on the dashboard 09:56:45 gmann: no, attachments are not available yet 09:56:46 andreaf: and what all category will be in group-by 09:56:55 andreaf: ok 09:57:44 gmann: run metadata keys, even though not all of them will be shown, it's going to be configurable which to display and which not 09:57:59 gmann: metadata keys are basically ZULL_* parameters passed to jobs 09:58:17 andreaf: ok 09:59:00 andreaf: 1 basic question, for how many days data it load by default 09:59:11 if you scroll to the bottom of http://status.openstack.org/openstack-health/#/job/periodic-tempest-dsvm-full-test-accounts-master you can see for instance that object store tests are flacky with pre-prov creds 10:00:37 on the 3rd page (test details) it should be 1 day (because it's a lot of data), but if you select a periodic job it's more, 15days I think 10:00:47 andreaf: where at bottom? 10:00:50 because periodic tests are less frequent 10:00:56 http://status.openstack.org/openstack-health/#/job/periodic-tempest-dsvm-full-test-accounts-master 10:01:12 andreaf: yea because it took long time to load for me 10:01:30 sorry for late, it's almost end, though. 10:01:31 anyways we will check it 10:01:35 andreaf: Thanks a lot 10:01:41 masayukig: hi :) 10:01:54 time up now 10:01:58 gmann: you can also search the test name by ObjectACL 10:02:03 #topic critical reviews 10:02:09 andreaf: i see 10:02:21 and i will check and ping you if any question 10:02:28 any reviews from anyone? 10:02:45 ok Let's end the meeting 10:02:51 Thanks All 10:02:53 thanks :) 10:02:55 thanks! ;) 10:03:02 #endmeeting