14:00:50 <igordcard> #startmeeting network_common_flow_classifier
14:00:51 <openstack> Meeting started Tue Jun 13 14:00:50 2017 UTC and is due to finish in 60 minutes.  The chair is igordcard. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:52 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:54 <openstack> The meeting name has been set to 'network_common_flow_classifier'
14:01:04 <reedip_> o/
14:01:41 <igordcard> hi all
14:01:48 <igordcard> just giving 3 minutes to people
14:01:59 <reedip_> k
14:02:26 <bcafarel> hello
14:02:58 <davidsha> Hey
14:03:23 <igordcard> hi all again
14:03:28 <igordcard> so let's start
14:03:48 <igordcard> agenda: https://wiki.openstack.org/wiki/Neutron/CommonClassificationFramework#Discussion_Topic_13_June_2017
14:03:54 <igordcard> #topic Spec status
14:04:30 <igordcard> #link https://review.openstack.org/#/c/333993
14:05:43 <reedip_> igordcard : going through it
14:06:04 <igordcard> just some background before diving in, I was able to get rights to the neutron-classifier after talking to kevinbenton
14:06:18 <igordcard> that prompted more people to review the spec
14:06:39 <igordcard> so there are a couple of new comments in the spec that I will address in the next patchset
14:07:31 <igordcard> since davidsha is/was working on the PoC, I've added him to neutron-classifier as well
14:07:40 <igordcard> and based on that...
14:07:45 <igordcard> #topic Implementation status
14:08:11 <igordcard> now that we have access to the repo, we can potentially start merging code
14:09:16 <igordcard> I'd just wait until the existing neutron-classifier members are OK with the spec, and then we can start working on the repo
14:10:01 <igordcard> most of them have moved on, but e.g. sean has commented on the spec
14:10:42 <davidsha> kk, just the update on the implementation, Classifications, Classification Groups and list classification types are done. At the moment I'm looking into the unit tests and then I can submit the code for review.
14:11:18 <igordcard> davidsha: cool, is it a different patch from https://review.openstack.org/#/c/445577/ ?
14:12:18 <davidsha> For the classifications we have ipv4, ipv6, tcp, udp and ethernet. No, that is the patch I'll submit it to.
14:12:50 <igordcard> davidsha: alright
14:14:30 <davidsha> Are there any questions about the implementation people would like to ask?
14:15:46 <igordcard> davidsha: we have to bring the repo to a clean state (i.e. wipe the unused code)
14:16:25 <igordcard> davidsha: we can do that after our changes or before... before makes it a bit clear as to when the CCF changes start I'd say
14:16:40 <igordcard> davidsha: but it's fine either way
14:17:43 <davidsha> igordcard: true I can rebase over that repo wipe or we can clean up after the patch as a "retirement" of the old library.
14:18:02 <igordcard> davidsha: yeah
14:18:24 <igordcard> davidsha: I'll submit a quick patch with just the cookiecutter stuff to see what happens in the gate
14:18:30 <igordcard> davidsha: DNM patch
14:18:48 <davidsha> Cool, add me to it!
14:19:29 <igordcard> davidsha: alright!
14:21:22 <igordcard> I don't have anything else.. just going to fix the spec and make that test patch in neutron-classifier
14:21:57 <igordcard> #topic Open discussion
14:22:33 * igordcard looks at bcafarel's coffee
14:23:06 <bcafarel> sorry, the CCF does not allow coffee classification :)
14:24:20 <davidsha> bcafarel: yet*
14:24:21 <igordcard> regarding the neutron-classifier core team.. me and davidsha are there due to the spec/code authorship, but as people engage in the project and make contributions, of course we will follow the good community habits, and anyone can join the team
14:24:40 <reedip_> hahaha
14:24:49 <igordcard> bcafarel: it's all about extensibility
14:25:00 <reedip_> igordcard : thats a good idea :)
14:25:52 <bcafarel> igordcard: don't forget to add that in the next spec update
14:25:55 <reedip_> I have been a bit distracted with other projects , but I will pitch in whenever I have some free time
14:26:09 <igordcard> bcafarel: coffee classification type? OK
14:26:27 <bcafarel> nice
14:26:41 <igordcard> reedip_: alright, let us know if there's any clear blocekr
14:27:42 <igordcard> many people have shown interest in the ccf over time, so it's a bit hard to manually invite everyone to the next patchsets, as such my recommendation is that you enable neutron-classifier email notifications in gerrit
14:27:46 <reedip_> igordcard : sure
14:28:08 <bcafarel> ack, will do
14:28:10 <igordcard> there won't be too many notifications since this is a small project
14:28:27 <reedip_> big things start small :P
14:29:28 <igordcard> reedip_: yes
14:29:48 <igordcard> alright, bye all
14:30:47 <bcafarel> see ya, waiting to get that cookicutter patch mail :)
14:30:57 <davidsha> Same, I'll talk to ye!
14:31:12 <igordcard> #endmeeting