13:03:00 <irenab> #startmeeting pci_passthrough
13:03:00 <sadasu> based on baoli's email he would be ablout 15 min late
13:03:01 <openstack> Meeting started Tue Jul 15 13:03:00 2014 UTC and is due to finish in 60 minutes.  The chair is irenab. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:03:02 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:03:05 <openstack> The meeting name has been set to 'pci_passthrough'
13:03:57 <irenab> I think we can discuss the current issues for integrating patches, till baoli joins
13:05:21 <itzikb> Hi , I got Rob patch.Checked and had small Error
13:05:32 <itzikb> Got a fix from Rob and checking it now
13:05:45 <irenab> sadasu: does it work for you?
13:05:49 <heyongli> hi
13:05:56 <irenab> heyongli: hi
13:06:20 <sadasu> irenab: so far yes
13:06:58 <irenab> rpothier: when do you think the rest of the patches will be out for review?
13:07:14 <irenab> the nova vif_driver part?
13:07:53 <rpothier> irenab: they should be out this week, just working on how to split them up
13:08:36 <irenab> rpothier: thanks a lot
13:08:54 <irenab> I pushed updated MD for review, will upload the agent soon
13:09:12 <irenab> heyongli: did you have a chance to review?
13:09:34 <rpothier> irenab: It will be easier to maintain if we could some of it pushed up
13:09:57 <heyongli> irenab, which patch
13:10:24 <heyongli> do you mean your "part 1" patch?
13:10:39 <irenab> heyongli: yes
13:10:49 <irenab> it allows to run MD without agent
13:10:52 <heyongli> i review, but mostly study process
13:10:59 <irenab> thanks
13:11:10 <heyongli> i try to understand more till have any comments
13:11:29 <irenab> rpothier: regarding patch splitting, I think the patch dependency can help
13:13:28 <baoli> Hi
13:13:45 <irenab> baoli: hi, welcome back
13:13:56 <heyongli> hi baoli
13:15:01 <irenab> baoli: according to email you sent today, you have some issue to discuss
13:15:32 <baoli> I am working on the patch that involves the PCI area.
13:15:52 <baoli> one last fix I have to make is due to the missing devspec module.
13:16:13 <baoli> one test failure, and the way it's written makes the mocking kind of difficult.
13:16:54 <baoli> I'm hoping that I can get it out today.
13:17:29 <heyongli> cool
13:18:03 <baoli> another patch is for the changes made in the libvirt area. So in total we should have four patches to complete the nova change
13:19:10 <irenab> baoli: great! menawhile we got patches from Rob to keep develoment environment working
13:19:18 <baoli> Another issue is that I have a fix that has been there for months. It got approved by reviewers, but just not the core reviewers. Have been trying to reach out to the core reviewers several times.
13:19:37 <baoli> irenab, heard from Rob on that.
13:19:40 <heyongli> i remember there is
13:20:13 <heyongli> i will try to give +1 once again, this might bring patch to top of review list
13:20:28 <irenab> baoli: can you please add the list of all patches for review on team wiki page?
13:20:35 <baoli> all of these: dividing up patches, merging with existing fix, etc, make the whole thing quite chanllenging.
13:20:45 <baoli> irenab, sure will do.
13:21:04 <irenab> I'll add the neuton side as well
13:21:11 <baoli> heyongli, thanks
13:21:43 <irenab> I updated on last meeting, that there is a bug with regards to binding:profile on neutron port-update, will send the fix soon
13:22:02 <baoli> irenab, got that. Thanks for fixing it.
13:22:18 <irenab> baoli: do you use patch dependency?
13:22:39 <irenab> I think to do it with neutron MD..
13:22:57 <baoli> irenab, I haven't
13:23:17 <irenab> It seems to be helpful to have number of patches for same  bp
13:23:31 <irenab> and use patch dependency between them
13:23:53 <baoli> irenab, I think that I may have to do that to control the order of merges.
13:24:15 <baoli> irenab, but would it help in passing unit test?
13:24:33 <irenab> baoli: I think so, will find out soon :-)
13:25:14 <irenab> #link  https://wiki.openstack.org/wiki/Gerrit_Workflow#Add_dependency
13:25:32 <heyongli> it should be dependency for patches in one set, but for unit test might get a little bit annoying
13:26:20 <irenab> heyongli: any advice for workflow?
13:26:44 <heyongli> just follow the lingk
13:26:49 <heyongli> you paste
13:27:14 <irenab> heyongli: ad what are the issue with unit tests?
13:27:32 <heyongli> some unit test need split
13:27:46 <irenab> ok
13:28:06 <irenab> Is there something else to discuss today?
13:28:09 <heyongli> #link http://www.mediawiki.org/wiki/Gerrit/Advanced_usage
13:28:15 <baoli> irenab, I think that it should help with the unit test.
13:28:35 <heyongli> this work flow some time resolve strange things
13:28:44 <baoli> that's all I have from my side.
13:29:10 <heyongli> look forward to see whole patch set -:)
13:29:20 <baoli> yep, it's coming up soon
13:29:39 <irenab> baol: nothing else from my side as well
13:29:41 <heyongli> great thing, lets review
13:29:50 <heyongli> neither me
13:30:22 <irenab> great, so we can work for extra 30 mins :-)
13:30:27 <irenab> thank you all
13:30:32 <baoli> thanks
13:30:33 <heyongli> thanks
13:30:36 <irenab> #endmeeting