17:00:45 <krtaylor> #startmeeting ironic_qa
17:00:46 <openstack> Meeting started Wed Jun  1 17:00:45 2016 UTC and is due to finish in 60 minutes.  The chair is krtaylor. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:00:48 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:00:50 <openstack> The meeting name has been set to 'ironic_qa'
17:00:53 <vdrok> o/
17:00:55 <rpioso> \o
17:01:19 <TheJulia> o/
17:01:34 <krtaylor> hi everyone, I'm hosting again this week, jlvillal is out today
17:01:48 <rloo> hi
17:01:52 <krtaylor> The agenda is at: https://wiki.openstack.org/wiki/Meetings/Ironic-QA
17:02:12 <rajinir> o/
17:02:15 <cdearborn> o/
17:02:35 <krtaylor> not much change in the agenda from last week
17:02:39 <krtaylor> #topic Announcements
17:02:53 <krtaylor> so does anyone have any announcements?
17:03:03 <thiagop> I have about OneView CI
17:03:16 <krtaylor> thiagop, the floor is yours
17:03:17 <thiagop> is it here or on the Third Party topic?
17:03:21 <sambetts> o/
17:03:38 <krtaylor> if specific to CI, lets do it there
17:03:49 <thiagop> right :)
17:04:05 * jroll is late but hear
17:04:10 * jroll is here, even
17:04:14 <krtaylor> hehheh
17:04:26 <krtaylor> well if none, lets move on
17:04:28 <krtaylor> #topic Grenade
17:04:43 <krtaylor> #link https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard
17:05:08 <krtaylor> anyone have status? remaining fixes?
17:05:18 <rloo> i added the current status for today.
17:05:25 <rloo> did we drop the ball on reviewing the patches?
17:06:04 <rloo> the 8 patches from last week didn't get merged yet :-(
17:06:06 <krtaylor> yeah, looks like those same 8 are stuck
17:06:13 * jroll promises to review soon
17:06:16 <rloo> does anyone know ?
17:06:19 * jroll feels bad
17:06:28 * rloo feels bad too
17:06:29 <jroll> I haven't actually checked if they're getting reviews
17:06:32 <vdrok> there is one more to ironicclient
17:06:39 <vdrok> will add it to whiteboard
17:06:43 <rloo> vdrok: link? please add to whiteboard. thx.
17:06:44 <thiagop> I'll try to get some on my queue too, although I'm not core
17:07:01 <rloo> cores aren't magical. everyone can review.
17:07:01 <sambetts> myself, and Vasyl have been working to try and fix the source problems of some of the patches we've had to write to make grenade work, e.g. https://review.openstack.org/#/c/323851/
17:07:12 <vdrok> yeah ^^
17:07:40 * krtaylor looks
17:07:48 <rloo> sambetts: so if that works, we don't need some of the other patches? is it clear which patches won't be needed, or whatever?
17:08:01 <rloo> sambetts: I mean, what do you think reviewers should be reviewing?
17:09:01 <krtaylor> can we just abandon unneeded patches (if any) ?
17:09:06 <sambetts> rloo: I need to update the commit message with more info and closes-bug etc, but hopfully with that patch (we're working to prove it works right now) we shouldn't need to do the extra reboots of nova compute during our upgrade process
17:09:26 <thiagop> rloo: are you sure cores aren't magical? :)
17:09:29 <krtaylor> cool
17:09:30 <rloo> sambetts: cool.
17:09:30 <vdrok> but these extra reboots are already merged
17:09:38 <rloo> thiagop: i can only speak for myself :D
17:09:46 <krtaylor> hehheh
17:09:49 <vdrok> so it's not going to reduce the number of patches to review :(
17:10:13 <rloo> vdrok: ok, got it. so if this new patch works, you'd undo the one that does the reboot.
17:10:19 <vdrok> ++
17:10:35 <sambetts> :'(, I'm going to try and look through the remaining ones and attempt to work out if there are more problems we can solve without bandaids
17:10:37 <thiagop> suggestion: keep patches that should be reviewed on top on the whiteboard
17:10:47 <jroll> they are
17:11:27 <krtaylor> ok, so patch review everyone!
17:11:33 <rloo> so there are two sets of patches. the first set to get ironic grenade working.
17:11:43 <rloo> then another set for ironic grenade to work with other projects
17:12:05 <rloo> what's the difference?
17:12:19 <rloo> it actually sez 'to fix ironic grenade to various projects'
17:12:22 <vdrok> rloo, second section can be removed I think, it's history
17:12:25 <jroll> I think they're the same list
17:12:27 <jroll> yeah
17:12:45 <rloo> OH! i'll go through later then and actually LOOK at them and clean it up :)
17:13:46 <krtaylor> anything else for grenade?
17:13:49 <rloo> did we discuss last week, once these patches are in. then what?
17:14:00 <jroll> then we make the jobs non-voting, and shortly after voting
17:14:11 <rloo> jroll: ok
17:14:32 <rloo> and then we add more to the testing? like do we need something for multitenant networking?
17:14:57 <vdrok> as for this nova thing with deletion in ironic virt driver, are we going to do something about it?
17:14:59 <jroll> we need to make sure multitenant networking passes grenade :)
17:15:28 <rloo> jroll: ok, so nothing new to be added, just that the code changes will pass.
17:15:30 <jroll> I'm not sure about testing the switch to multitenant from flat
17:15:32 <jroll> yeah
17:15:41 <jroll> vdrok: the ML thing?
17:15:43 <rloo> jroll: thx
17:15:45 <vdrok> yep
17:15:53 <jroll> I kind of liek sean's suggestion there
17:16:21 <rloo> is this the cleaning/resource avail issue?
17:16:26 <jroll> yes
17:16:30 * jroll still needs to reply there
17:17:49 <krtaylor> anything else for grenade?
17:17:53 <vdrok> seems that's all :)
17:17:58 <jroll> well
17:18:09 <rloo> here's the ML link: http://lists.openstack.org/pipermail/openstack-dev/2016-May/096125.html
17:18:15 <jroll> do other people like the suggestion of a separate cleaning test (probably without nova)?
17:18:31 <rloo> jroll: yes, we should have separate test
17:18:35 <jroll> or should we continue on ML
17:18:37 <jroll> yay
17:18:48 <rloo> well, we should continue on ML.
17:18:59 <jroll> k
17:19:12 <vdrok> jroll, btw, I think vsaienko asked about an experimental job for cleaning and infra said no
17:19:17 <rloo> is that blocking anything vdrok?
17:19:31 <vdrok> nope, not right now
17:19:47 <jroll> vdrok: well, that's a test that uses the normal tests with nova but enables cleaning
17:19:50 <rloo> vdrok: ok, just the nova tempest tests fail when ironic is enabled.
17:20:11 <jroll> I'd like us to separate the jobs for the "nova scenario tests" from the "ironic api tests"
17:20:30 <jroll> have only one or two of the nova tests, and ironic api tests can cover most of the driver differences and such
17:20:32 <vdrok> jroll, ah, gotcha
17:20:50 * jroll will write this up
17:21:51 <krtaylor> ok, anything else for grenade?  :)
17:21:53 <jroll> we can probably move on :)
17:22:06 <rloo> ++
17:22:07 <krtaylor> #topic Functional testing
17:22:17 <krtaylor> not much movement here, anyone?
17:22:30 <jroll> so we haven't really worked on / talked about this for a while, seems people aren't very interested
17:22:34 <jroll> can we just drop it? :)
17:22:43 <rloo> do we want functional tests?
17:22:55 <krtaylor> yes, but agree that it is a lower priority
17:22:58 <thiagop> Maybe we can put it back on the list when this work starts
17:23:01 <krtaylor> ++
17:23:10 <krtaylor> I'll nuke it for now
17:23:11 <rloo> so no work has been done wrt functional tests?
17:23:11 <jroll> if we split out the api test jobs, those become very close to the same thing
17:23:26 <vdrok> we have tests for client which basically trigger our API, maybe they can be considered as functional for ironic too? :)
17:23:41 <krtaylor> jroll, agreed, and a great start
17:24:09 <krtaylor> vdrok, didn't you have a patch proposing that?
17:24:54 <vdrok> krtaylor, not sure what you mean, we have a bunch of them here - https://github.com/openstack/python-ironicclient/tree/master/ironicclient/tests/functional
17:24:55 <thiagop> jroll: food for thought: we want it on tempest api tests or inside ironic (ironic/tests/functional)?
17:25:07 <jroll> thiagop: /shrug?
17:25:10 <jroll> idk
17:25:15 <jroll> I haven't thought much about these
17:25:32 <thiagop> We don't have to decide it now
17:25:43 <vdrok> thiagop, I think the idea is to do it without devstack and all the other things, just ironic rabbit and DB
17:25:57 <krtaylor> right, anyway, we can probably move on
17:26:35 <krtaylor> #topic 3rd party CI
17:26:54 <krtaylor> thiagop, you had a question?
17:27:23 <thiagop> announcement: OneView CI is on a new infra (using puppet-openstackci)
17:27:41 <thiagop> reduced 65% in testing time
17:28:08 <krtaylor> yes!
17:28:11 <jroll> yay
17:28:13 <thiagop> That should help, but we still need to work on the full deploy test
17:28:21 <sambetts> wow?!
17:28:25 <jroll> does it not deploy an instance yet?
17:28:30 <krtaylor> what is your bottleneck?
17:28:34 <rloo> so what is the amount of time now?
17:28:49 <thiagop> The previous infra were deploying, but the new one is on the cloud
17:29:05 <thiagop> we need to find out how to do the pipes to use real blades there
17:29:49 <thiagop> the power and management tests were running in about 2hs, now in 45min
17:30:04 <rloo> thiagop: wow!
17:30:13 <krtaylor> please document what you did to optimize, btw, we also reduced time a bunch by caching
17:30:15 <jroll> nice
17:30:17 <thiagop> (our previus devstack script really sucked...)
17:30:26 <krtaylor> thats similar to our time, great!
17:30:58 <sambetts> \o/ I'd love to hear the optimizations my full tempest run takes 1hr 30
17:31:18 <sambetts> (full tempest being Ironic tempest)
17:31:19 <sambetts> :-P
17:31:23 <thiagop> Next sprint will be interesting
17:31:32 <krtaylor> ?
17:31:48 <thiagop> our spring, I mean
17:31:51 <thiagop> sprint*
17:31:57 <thiagop> (3 weeks)
17:31:58 <krtaylor> ah
17:32:24 <rloo> thiagop: you mean your internal/downstream sprint?
17:33:15 <thiagop> rloo: yes. We are targetting to have pipes + Ironic tempest in the beginning of July
17:33:31 <thiagop> so we can kill the old infra completely
17:33:41 <rloo> thiagop: :)
17:34:13 <thiagop> me -> done; Thanks krtaylor
17:34:27 <krtaylor> thiagop, cool, thanks for the status!
17:35:32 <krtaylor> so, I also wanted to bring up the email thread about a wiki problem found by folks wanting to change the state in the third party driver table, if you have a problem, let me now and I'll change it for you
17:36:04 <thiagop> ack
17:36:10 <krtaylor> it seems it is a problem for member that havn't been around very long
17:36:52 <krtaylor> the wiki registration was shut down to protect from bots, so let me know, I'll be happy to keep it updated
17:37:31 <krtaylor> alright, anything else CI related?
17:37:56 <krtaylor> #topic Open Discussion
17:39:56 <krtaylor> I'll give everyone a couple of minutes, else close meeting at 40 after
17:40:04 <thiagop> Are we moving out of the wiki or just consolidating info for now?
17:40:20 <krtaylor> plan is to move into documentation
17:40:27 <krtaylor> I have started the draft for that
17:40:36 <thiagop> right
17:40:46 <krtaylor> hope to push a patch for docs very soon
17:41:01 <krtaylor> it will start small, but we can go from there
17:42:10 <krtaylor> anything else?
17:42:28 <krtaylor> thanks everyone, good meeting
17:42:53 <jroll> thanks :)
17:43:15 <krtaylor> #endmeeting