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