17:02:35 <LouisF> #startmeeting service_chaining 17:02:36 <openstack> Meeting started Thu Aug 4 17:02:35 2016 UTC and is due to finish in 60 minutes. The chair is LouisF. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:02:37 <pcarver> hi. I'm double booked in a voice meeting. 17:02:38 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 17:02:40 <openstack> The meeting name has been set to 'service_chaining' 17:02:41 <LouisF> hi all 17:02:49 <bcafarel> hi 17:02:50 <LouisF> scsnow: hi 17:03:09 <yamahata> hello 17:03:14 <LouisF> pcarver: ok 17:03:27 <LouisF> yamahata: hi 17:04:11 <LouisF> agrenda for today https://wiki.openstack.org/wiki/Meetings/ServiceFunctionChainingMeeting#Agenda_for_the_Networking-SFC_Meeting_.288.2F4.2F2016.29 17:04:15 <LouisF> fsunaval: hi 17:04:41 <LouisF> bug scrub today 17:04:52 <LouisF> #topic bug scrub 17:05:30 <LouisF> here is the bug list https://bugs.launchpad.net/networking-sfc/+bugs?orderby=importance&start=0 17:05:42 <fsunaval> Hi 17:06:40 <LouisF> starting with high priority bugs 17:06:43 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1557600 17:06:43 <openstack> Launchpad bug 1557600 in networking-sfc "port-chain-delete failing as "SfcDriverError"" [High,In progress] - Assigned to Mohankumar (mohankumar-n) 17:07:20 <LouisF> mohankumar__: assigned to you 17:07:56 <LouisF> this is quite old has this been reproduced in latest build? 17:08:58 <scsnow> I think it should work 17:09:12 <LouisF> i am going to close it 17:09:40 <scsnow> haven't seen such issue with port-chain-delete last months 17:09:55 <LouisF> right - i have not seen it either 17:10:00 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1567644 17:10:00 <openstack> Launchpad bug 1567644 in networking-sfc "Add “Symmetric” parameter in the chain_param of the API " [High,In progress] - Assigned to Pavel Gluschak (scsnow) 17:10:45 <LouisF> mohankumar_: hi we were just discussing Launchpad bug 1557600 in networking-sfc "port-chain-delete failing as "SfcDriverError"" [High,In progress] - Assigned to Mohankumar (mohankumar-n) 17:10:45 <openstack> Launchpad bug 1557600 in networking-sfc "port-chain-delete failing as "SfcDriverError"" [High,In progress] https://launchpad.net/bugs/1557600 - Assigned to Mohankumar (mohankumar-n) 17:11:06 <LouisF> have you seen that bug? 17:11:19 <LouisF> recently? 17:12:01 <mohankumar_> LouisF , nope 17:12:13 <LouisF> mohankumar_: ok i am going to close it 17:12:31 <LouisF> scsnow: back to 1567644 17:12:31 <mohankumar_> LouisF , ok , Please 17:12:56 <LouisF> scsnow: you had a patch for that 17:13:20 <scsnow> yes, there is an ongoing review for that 17:13:30 <scsnow> I'll update patchset tomorrow to fix merge conflict 17:13:40 <LouisF> scsnow: ok thanks 17:14:05 <scsnow> also unit tests should pass as well now as fix with validation is included in neutron-lib 0.3.0 17:14:28 <LouisF> scsnow: great thanks 17:14:34 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1567654 17:14:34 <openstack> Launchpad bug 1567654 in networking-sfc "Add “SFC path-ID” parameter in the chain_param of the API" [High,In progress] - Assigned to cathy Hong Zhang (cathy-h-zhang) 17:15:05 <LouisF> there is a patch for this https://review.openstack.org/#/c/348136 17:15:46 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1587405 17:15:46 <openstack> Launchpad bug 1587405 in networking-sfc "all sfc flows gone after restart ovs agent" [High,In progress] - Assigned to xiaodongwang (xiaodongwang991481) 17:16:14 <LouisF> this also has a patch https://review.openstack.org/#/c/348574 17:16:55 <fsunaval> There are 2 comments mostly talking about the commit message. I will fix them and resend. 17:17:13 <LouisF> fsunaval: thanks 17:17:27 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1588463 17:17:27 <openstack> Launchpad bug 1588463 in networking-sfc "Add port-pair-group parameter" [Medium,In progress] - Assigned to Louis Fourie (lfourie) 17:17:49 <LouisF> this has a patch https://review.openstack.org/#/c/348136 17:18:03 <s3wong> hello 17:18:36 <LouisF> s3wong: hi we are doing a bug scrub 17:18:44 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1588460 17:18:44 <openstack> Launchpad bug 1588460 in networking-sfc "Move path-id generation from ovs driver to plugin" [Medium,In progress] - Assigned to Igor Duarte Cardoso (igordcard) 17:19:24 <LouisF> igordcard: you have a patch https://review.openstack.org/346175 17:20:40 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1586181 17:20:40 <openstack> Launchpad bug 1586181 in networking-sfc "add functional tests coverage" [Medium,In progress] - Assigned to xiaodongwang (xiaodongwang991481) 17:21:05 <LouisF> there is a patch for this also https://bugs.launchpad.net/networking-sfc/+bug/1586181 17:21:46 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1593737 17:21:46 <openstack> Launchpad bug 1593737 in networking-sfc "separate optional and required params in API" [Medium,In progress] - Assigned to Mohankumar (mohankumar-n) 17:22:00 <LouisF> mohankumar_: what is the status on this? 17:22:24 <mohankumar_> LouisF , Will do it as part os osc transaction 17:22:34 <mohankumar_> *of 17:22:51 <LouisF> mohankumar_: ok is there a patch in review? 17:22:54 <scsnow> 1586181: it's tricky to review such huge patches 17:23:12 <mohankumar_> LouisF , not yet 17:24:41 <LouisF> scsnow: what is that patch? 17:25:13 <scsnow> i.e. https://review.openstack.org/#/c/321870/ 17:25:54 <scsnow> I think that commits should be granular to attract more reviewers 17:26:23 <mohankumar_> scsnow , +1 17:26:32 <LouisF> maybe it can be split into two patches - one for functional tests, the other for tempest tests 17:27:09 <LouisF> i will contact xiaodong about that 17:27:46 <scsnow> LouisF, right. Probably first commit should provide just skeleton for functional testing framework 17:28:25 <LouisF> scsnow: ok will look at splitting it up 17:28:49 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1586024 17:28:49 <openstack> Launchpad bug 1586024 in networking-sfc "Switch OVS agent to l2 agent extensions" [Medium,In progress] - Assigned to Bernard Cafarelli (bcafarel) 17:29:17 <LouisF> fsunaval: you were looking at this? 17:29:41 <fsunaval> i already have the diffs.. 17:29:45 <fsunaval> yes. 17:29:57 <LouisF> fsunaval: great 17:30:27 <LouisF> are you ready to post the patch? 17:31:08 <fsunaval> sure. I was hoping we tag the stable/mitaka label before committing though. 17:32:44 <LouisF> fsunaval: ok thanks 17:33:01 <bcafarel> fsunaval: no need I keep on looking at it then? (I was still at the drivers/extensions API look stage) 17:34:02 <LouisF> bcafarel: i can re-assign to farhad? 17:34:09 <fsunaval> if are ready, go ahead. if not, you can reassign to me. 17:34:45 <LouisF> fsunaval: bcafarel can you work together on this? 17:34:58 <fsunaval> Sounds good LouisF 17:35:16 <LouisF> fsunaval: bcafarel thanks 17:35:30 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1599037 17:35:30 <openstack> Launchpad bug 1599037 in networking-sfc "make get_port_mask more efficient by avoiding some unnecessarily mask checking" [Undecided,In progress] - Assigned to ding bo (longddropt) 17:35:52 <LouisF> there is a patch for this https://review.openstack.org/338741 17:36:06 <LouisF> all please review 17:36:47 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1593693 17:36:47 <openstack> Launchpad bug 1593693 in networking-sfc "SFC neutron-openvswitch-agent has same name as vanilla agent" [Undecided,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) 17:37:16 <LouisF> its assigned to ihar - farhad you were looking at this? 17:37:36 <scsnow> LouisF, seems this one is the same as 1586024? 17:37:37 <fsunaval> yes. In review. 17:38:18 <bcafarel> this should be fixed with 1586024 (no more specific agent) 17:38:22 <bcafarel> ? 17:38:29 <LouisF> scsnow: yes it is 17:38:57 <LouisF> i will update the bug info 17:38:58 <scsnow> bcafarel, I believe so. we will separate sfc agent from neutron agent 17:39:44 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1587486 17:39:44 <openstack> Launchpad bug 1587486 in networking-sfc "Support SFC Encapsulation" [Undecided,In progress] - Assigned to Igor Duarte Cardoso (igordcard) 17:39:44 <fsunaval> bcafarel,LouisF: 1593693 will simply change sfc agent name from neutron-openvswitch-agent to neutron-openvswitch-sfc-agent. 17:39:58 <fsunaval> the other one will add support for l2 flow extensions. 17:40:32 <LouisF> fsunaval: ok 17:41:26 <LouisF> igordcard: what is status? 17:41:59 <scsnow> fsunaval, how this will help? in this case neutron-openvswitch-sfc-agent will still provide functionality of vanilla neutron agent. 17:42:47 <scsnow> I think the overall idea is to have separate thin sfc agent 17:42:54 <fsunaval> scsnow: yes, vanilla neutron agent with sfc code will be called neutron-openvswitch-sfc-agent. 17:43:09 <fsunaval> this is what the bug wants for redhat packaging. 17:43:24 <scsnow> fsunaval, ah ok, just to resolve packaging conflict 17:43:52 <fsunaval> scsnow: At this point, it is not worth rewriting the entire sfc agent into a separate thin sfc agent IMHO. 17:44:14 <fsunaval> we will still be a child of neutron-openvswitch-agent but will go with a separate name. 17:45:31 <fsunaval> if the code review for 1577486 goes through, we should be fine. If not, we will have to go through the effort of writing a thin sfc agent. 17:45:50 <bcafarel> If I got it correctly, that's the fix for 1593693, and bug 1586024 is switching all code in sfc agent to a l2 extension (and therefore removing the sfc-agent)? 17:45:50 <openstack> bug 1586024 in networking-sfc "Switch OVS agent to l2 agent extensions" [Medium,In progress] https://launchpad.net/bugs/1586024 - Assigned to Bernard Cafarelli (bcafarel) 17:47:47 <fsunaval> bcafarel: Let us discuss the solution for 1586024 offline. I will send you my diffs. They are less than 20 lines. We do not remove all the code. 17:47:57 <bcafarel> fsunaval: ok :) 17:48:12 <LouisF> fsunaval: bcafarel thanks 17:48:19 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1587486 17:48:19 <openstack> Launchpad bug 1587486 in networking-sfc "Support SFC Encapsulation" [Undecided,In progress] - Assigned to Igor Duarte Cardoso (igordcard) 17:48:40 <LouisF> looks like igor is away 17:48:52 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1582238 17:48:52 <openstack> Launchpad bug 1582238 in networking-sfc "Add "priority” field in flow-classifier" [Undecided,In progress] 17:49:15 <LouisF> mohankumar__: you were doing work on this - what is the status? 17:50:01 <LouisF> sorry mohan left 17:50:05 <scsnow> mohankumar__, did you have a chance to remove FC conflict checks as we discussed earlier? 17:50:41 <scsnow> LouisF, for that change we would need to remove FC conflict checks in API code and add some agents tests 17:51:59 <LouisF> scsnow: correct - need to add to that patch 17:52:25 <LouisF> left me contact mohan on the status 17:52:49 <scsnow> LouisF, thanks. he did API part and I worked on agent changes 17:53:37 <LouisF> scsnow: ok thanks can you update the patch? 17:54:14 <scsnow> LouisF, well, I can update API part as well, but first we should ask Mohan if he doesn't mind 17:54:39 <LouisF> scsnow: can you work with mohan on that? 17:54:43 <scsnow> LouisF, sure 17:54:52 <LouisF> scsnow: thanks 17:55:40 <LouisF> there are a couple of bugs on rpm and debian packaging 17:55:42 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1529766 17:55:43 <openstack> Launchpad bug 1529766 in networking-sfc "Support rpm package changes for SFC" [Wishlist,In progress] 17:55:52 <LouisF> https://bugs.launchpad.net/networking-sfc/+bug/1529767 17:55:53 <openstack> Launchpad bug 1529767 in networking-sfc "Support debain package changes for SFC." [Wishlist,In progress] 17:56:31 <fsunaval> LouisF: Is it similar to renaming the agent name ? 17:56:41 <LouisF> fsunaval: does the rpm packaging relate to what you are doing? 17:56:55 <LouisF> fsunaval: i think so 17:56:58 <scsnow> LouisF, no, I think this is about impelenting spec files to build rpm/deb packages with sfc code 17:57:10 <LouisF> scsnow: ok 17:57:33 <fsunaval> scsnow: ok. 17:57:38 <LouisF> scsnow: can you elaborate? 17:58:02 <scsnow> scsnow, I'm familiar with rpm specs, so I can take over 1529766 17:58:16 <LouisF> scsnow: thanks i will assign to you 17:58:57 <LouisF> thanks all - i think we got through all the bugs that do not have a fix committed 17:59:03 <scsnow> But first I should look how things are done in another projects like nova and neutron 17:59:24 <scsnow> I haven't seen any spec files in nova/neutron repos 17:59:39 <LouisF> scsnow: ok 18:00:01 <scsnow> I suspect, that specs are maintained by RDO people, but I have to check that 18:00:10 <LouisF> scsnow: ok 18:00:25 <LouisF> all for now thanks all - bye 18:00:27 <scsnow> bye 18:00:32 <LouisF> #endmeeting