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