13:06:50 #startmeeting sriov 13:06:51 Meeting started Tue Mar 1 13:06:50 2016 UTC and is due to finish in 60 minutes. The chair is ndipanov_. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:06:52 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:06:54 The meeting name has been set to 'sriov' 13:07:02 awesome 13:07:30 ok so I thought we should talk about pending patches 13:07:45 and any bugs that are bad enough we wanna try and get them in for Mitaka 13:07:59 I thought we could use https://etherpad.openstack.org/p/sriov_meeting_agenda 13:08:19 to list the reviews that might be pending 13:08:41 I don't think a lot of bugfixes are going to realistically get merged this week 13:08:43 I have two pending, let me put them on the etherpad 13:08:44 as it's FF week 13:08:49 lbeliveau, please do 13:09:40 the first one is and easy one, you put +2 on it, I guess it just need another core to double check 13:10:28 yeah so let me re-arrange that :) 13:12:27 lbeliveau, ok I remember the first one - I know johnthetubaguy has looked at it before so we'll ping him once the FF craze is over 13:12:47 perfect thanks 13:15:12 anything else on the pending patches front? 13:15:33 I'm good 13:16:03 I have one 13:16:06 https://review.openstack.org/#/c/283198/4 13:16:19 vladikr, yeah I saw it - will try to review that one today as well 13:16:30 great, thanks 13:16:56 so there is also this https://review.openstack.org/#/c/242087/ 13:17:11 I've seen this refactoring before, and jaypipes mentioned it as well 13:17:12 vladikr: have you see my comment about if the fix needs to be applied anywhere else in the compute manager for cleanup ? 13:17:22 but haven't really looked at it 13:17:50 lbeliveau, oh, yes, sorry.. 13:18:04 vladdikr: just a question. no worries :) 13:18:20 lbeliveau, I was trying to look into that, but I don't see any other places where this would apply 13:18:20 anyone else looked at that? 13:18:35 I looked at it 13:19:52 ok lbeliveau I see you commented quite recently 13:20:17 is that one on track for FF? 13:20:45 lbeliveau, do you have any thoughts? 13:20:53 that you would like to share :) 13:22:13 Should ask jay pipes, I think this patch is part of multiple patches for remove pci stats, dunno if it's finished or not 13:22:41 lbeliveau, right, but what is your take on the work? 13:22:50 * ndipanov_ hasn't really looked at it tbh 13:23:48 I think it makes senses just to keep pci devices and get away from pci stats 13:24:17 lbeliveau, but won't that make us query ALL the pci devices ever in the scheduler 13:24:26 which is potentially 10s of thousands of rows? 13:25:27 I guess I should look at the series 13:26:37 it's part of the blueprint for uniform resource provides, haven't follow that closely and wasn't part of the discussions 13:27:59 ok I'll try to wrap my head around it today 13:28:29 I see that there is a bug that it fixes too - will look at that as well 13:29:34 just looking at the list of bugs there seems to be nothing critical other than resize which we know doesn't work 13:29:58 ndipanov: what do you want to do about this one https://review.openstack.org/#/c/242573, if you have more comments I can work on it today 13:30:05 I was going to look at it if I get a chance after I'm done with some live migration stuff I am trying to figure out 13:30:24 great 13:30:41 lbeliveau, yeah I have that on my list - I think I was happy with it mostly but it's a big patch so I didn't vote :( 13:30:55 moshele, https://bugs.launchpad.net/nova/+bug/1400574 <- can we mark that one as incomplete? 13:30:55 Launchpad bug 1400574 in OpenStack Compute (nova) "Create VMs sometimes failure when use mellonax nic as SR-IOV" [Low,Confirmed] - Assigned to KaiLin (linkai3) 13:30:57 I understand, let me know 13:31:21 yes 13:32:18 ndipanov_: he didn't respond to my question 13:32:23 moshele, yeah 13:32:27 so I was looking at this 13:32:29 http://bit.ly/1QjJuVf 13:32:40 this is basically a search for pci in nova bugs 13:32:54 but that might not get all the bugs we care about 13:33:24 for example neutron integration 13:33:43 lbeliveau: in progress 13:34:02 so if there are some bugs that you are aware of that we should bump in priority and try to get fixed in M - that would be awesome 13:34:10 so here are all neutron sr-iov bugs https://bugs.launchpad.net/neutron/+bugs?field.tag=sriov-pci-pt 13:36:30 ndipanov_: we should fix the migration issue so I guess lbeliveau https://review.openstack.org/#/c/242573 is important 13:37:07 right, after it get merged, I proposed myself to look at the resize issue 13:37:08 moshele, yeah I guess so - it's really annoying that it's there 13:38:42 ok well if that's all we have 13:39:12 let's get back to reviewing/coding and hopefully get these in soon 13:39:49 we could add an sriov tag to bugs 13:39:55 to make it easier to track 13:40:05 I might do that and go back and tag useful bugs with it 13:40:44 ndipanov, looks like that series https://review.openstack.org/#/c/242087/ 13:41:00 vladikr, yeah 13:41:03 ndipanov, completely removes all of the vf-pf stuff 13:41:08 :) 13:41:13 lol nice 13:41:24 back to the drawing board 13:41:31 jeez 13:41:40 yeah, going to look at this closely today 13:41:50 vladikr, please do - I will too 13:41:57 maybe I'm missing something 13:42:34 I couldn't follow your work on that (was busy on prodcut stuff), but let me know if I can help 13:43:39 lbeliveau, well the main drawback of that approach is that it's not used in consume_requests which means there will be races 13:43:48 that approach = vladik's series 13:44:00 otherwise it's very contained 13:44:05 and straightforward 13:44:27 lbeliveau, but please if you get a chance take a look at it and even take it for a spin 13:44:33 and report any bugs that you find 13:45:02 vladikr: what is your series in question ? 13:45:12 lbeliveau, it's merged already 13:45:34 lbeliveau, I should have a link somewhere.. sec 13:45:54 but I think gerrit doesn't keep links between patches once they merge 13:46:11 vladikr, but if you have all the links... 13:46:38 ndipanov, yeah, it's messed up 13:46:51 https://review.openstack.org/#/q/topic:pf-neutron-port 13:47:40 thanks, will have a look, will be helpful to get perspective for https://review.openstack.org/#/c/242087/ 13:48:05 lbeliveau, thanks 13:49:44 ok cool - let's give ourselves 10 min back 13:50:20 chat in 2 weeks - I am assuming the topic will be the same - so keep an eye out for bugs :) 13:50:31 bye folks 13:50:34 ndipanov, thanks 13:50:36 bye guys 13:50:42 bye 13:51:00 #endmeeting sriov 13:51:26 err.. 13:52:07 that didn't work 14:01:15 dprince: Error: Can't start another meeting, one is in progress. Use #endmeeting first. 14:01:19 o/ 14:01:26 o/ 14:01:37 o/ 14:01:48 o/ 14:01:50 \o 14:02:07 the last meeting needs to be ended first 14:02:26 #endmeeting 14:02:37 ruh roh 14:02:48 derekh: I think it needs to come from the person who started the meeting 14:03:09 o/ 14:03:16 o/ 14:03:19 ndipanov: can you #endmeeting? 14:03:21 ndipanov_ ^^ 14:03:24 this is the openstack equivalent of turf wars 14:03:38 slagle: I don't think it was him either 14:03:44 * leanderthal starts the west side story soundtrack 14:03:48 ndipanov: I guess it didn't end because you started it as "ndipanov_" 14:03:50 *snap snap* 14:04:26 #endmeeting sriov