16:08:06 <Sukhdev_> #startmeeting networking_ml2
16:08:07 <openstack> Meeting started Wed Mar 15 16:08:06 2017 UTC and is due to finish in 60 minutes.  The chair is Sukhdev_. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:08:08 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:08:11 <openstack> The meeting name has been set to 'networking_ml2'
16:08:29 <Sukhdev_> #topic: Agenda
16:08:29 <Sukhdev_> #link: https://wiki.openstack.org/wiki/Meetings/ML2#Meeting_March_15.2C_2017
16:08:37 <Sukhdev_> trevormc : yes
16:08:55 <trevormc> ok I will push it up in a moment
16:09:08 <Sukhdev_> #topic: Anouncements
16:09:38 <Sukhdev_> I do not have any specific announcements - I missed neutron core meeting this week
16:09:55 <Sukhdev_> anybody has any announcements for the team?
16:10:19 <Sukhdev_> #topic: Enhanced Agenda Items
16:10:38 <Sukhdev_> We are adding two new items for discussion to the agenda
16:11:00 <Sukhdev_> 1) QinQ type driver
16:11:28 <Sukhdev_> 2) ML2 Driver extension enforcement
16:11:57 <trevormc_> hey sorry my browser froze
16:12:30 <Sukhdev_> rkukura : can you go first? while trevormc_ fixes the browser issue
16:12:56 <rkukura> actually, if I can have another couple minutes, I’ll post my RFE
16:13:05 <trevormc> it may be
16:13:21 <Sukhdev_> rkukura : sounds good - we are here
16:14:00 <Sukhdev_> trevormc_ : let me know when you are ready
16:14:41 <trevormc__> Sukhdev_: ok, browser fixed now.
16:15:13 <Sukhdev_> trevormc__ : cool - so, you go first while rkukura is wrapping up his RFE
16:15:33 <trevormc__> https://review.openstack.org/446047
16:16:36 <trevormc__> I know there are some things wrong with this patch. I'm here to volunteer my time to develop and  debug.
16:17:14 <trevormc__> I'm hoping neutron cores will push me in the right direction so we can land the new network type driver.
16:17:59 <Sukhdev_> trevormc__ : makes sense
16:18:09 <Sukhdev_> #link: https://review.openstack.org/446047
16:18:44 <Sukhdev_> trevormc__ : I will review this patch - one quick observation
16:19:11 <Sukhdev_> I noticed you are adding new DB table. You may want to add the migration scripts as well
16:19:39 <trevormc__> Please do! Or I can learn and do it myself.
16:20:00 <trevormc__> There are a couple alternatives to this approach. I guess I shouldve put it in the commit message
16:21:04 <Sukhdev_> trevormc__ : it is very trivial step that needs to be added - but, we can worry about it later, lets first make sure this driver does the right things
16:21:45 <trevormc__> Instead of adding another table there was talk about modifying the segmentation_id field to hold the values in a dictionary. I think the Trunk team wanted to use this too.
16:22:11 <Sukhdev_> trevormc__ : I like that approach as well -
16:22:33 <trevormc__> I found it difficult to use the SegmentTypeDriver with that approach
16:22:45 <trevormc__> because it relies on the allocated field.
16:22:45 <rkukura> there was a proposal a while back to generalize the “provider” attributes that may be worth digging up, but a dict might suffice
16:22:50 <Sukhdev_> the idea of segmentation_type and segmentation_id is to be able to accomodate various types
16:23:47 <Sukhdev_> rkukura : +1
16:24:50 <Sukhdev_> trevormc__ : you have RFE/Bug filed for this feature, right?
16:25:13 <trevormc__> Sukhdev, no RFE/bug is filed for specifically this.
16:25:29 <trevormc__> I thought I could use https://bugs.launchpad.net/neutron/+bug/1604222
16:25:29 <openstack> Launchpad bug 1604222 in neutron "[RFE] Implement vlan transparent for openvswitch ML2 driver" [Wishlist,Confirmed] - Assigned to Bo Chi (bochi-michael)
16:25:57 <trevormc__> because in the reviews sean commented he prefers a network type driver to solve this problem.
16:26:13 <trevormc__> https://review.openstack.org/#/c/342755/
16:28:19 <Sukhdev_> trevormc : I see -
16:29:07 <Sukhdev_> trevormc : So, if you want to use the same bug for this, then you should refer to it in your patch - so, that people can tie these together and get the history
16:29:26 <trevormc> Sukhdev_: ok I was just thinking about that
16:30:00 <Sukhdev_> trevormc : right - update the commit message to include the appropriate links
16:32:31 <Sukhdev_> this week is very tight for me. I will try to review the patch and post comments either over the weekend or early next week -
16:32:55 <Sukhdev_> I am sure rkukura will have good feedback for you as well
16:33:18 <trevormc> Thank you, I will be working on the unit tests in the meantime.
16:33:21 <rkukura> I’ll try - added myself as a reviewer
16:33:33 <Sukhdev_> anything else on this type driver?
16:33:48 <trevormc> no thats all, thanks.
16:34:34 <Sukhdev_> #topic: Extensions enforcement discussion
16:34:42 <rkukura> Ok
16:34:47 <Sukhdev_> rkukura : are you ready?
16:35:04 <rkukura> At the PTG, I had promised to file an RFE for this
16:35:19 <rkukura> I finally got to it this morning
16:35:36 <Sukhdev_> excellent
16:35:41 <rkukura> #link https://bugs.launchpad.net/neutron/+bug/1673142
16:35:41 <openstack> Launchpad bug 1673142 in neutron "[RFE][ML2] Enforce extension semantics" [Undecided,New]
16:36:03 <rkukura> I tried to just capture what had been discussed at the PTG
16:36:52 <rkukura> I think the only substantive change was realizing that the enforcement needed to be done entirely within the transacation that commits a port binding, otherwise extension values might change before the binding is committed
16:37:42 <rkukura> The RFE is a bit dense, so I don’t think it makes sense to review it in detail here now, but maybe we could discuss it next week
16:38:32 <Sukhdev_> rkukura : right, I tried to scan through it - need time to review
16:39:11 <rkukura> I’ll post a link on the email thread that kevinbenton started on this
16:39:29 <Sukhdev_> rkukura : +1
16:39:59 <rkukura> that’s it for me
16:40:19 <Sukhdev_> trevormc : you may want to review this as well - this may apply to the RFE that you shared with us at PTG
16:40:40 <Sukhdev_> thanks rkukura
16:40:45 <trevormc> Sukhdev, I bookmarked it
16:41:03 <trevormc> will do
16:41:12 <Sukhdev_> Anything else?
16:42:14 <rkukura> not from me
16:42:17 <trevormc> not from me either
16:42:22 <Sukhdev_> since we do not have dasanind here, I am going to skip the port binding stuff - unless anybody has anything to discuss about it
16:42:54 <Sukhdev_> #topic: Open Discussion
16:43:18 <Sukhdev_> The Boston summit is fast approaching -
16:44:09 <Sukhdev_> rkukura : you are going to be our official guide :-):-)
16:44:22 <rkukura> I’ll miss Monday, though
16:44:35 <Sukhdev_> or shall I say I nominate you for the job :-)
16:45:11 <rkukura> snow should be gone by then
16:45:45 <Sukhdev_> we had 78 degrees yesterday - when you were dealing with the snow :-):-)
16:46:15 <rkukura> I was in FL for the last storm, so it was nice to be here this time
16:46:51 <rkukura> of course it was only in the 50s and 60s last week when I was in SJC
16:47:04 <Sukhdev_> as long as you are having fun - that is what matters...
16:47:14 <rkukura> right
16:47:29 <Sukhdev_> I can stand snow for a day or so... not more than that
16:47:30 <rkukura> lets wrap up so I can get some lunch ;)
16:47:34 <Sukhdev_> I am chicken :-):-)
16:47:35 <trevormc> +1
16:47:50 <Sukhdev_> OK -
16:48:03 <Sukhdev_> Thanks for discussion folks
16:48:07 <Sukhdev_> bye
16:48:10 <trevormc> \o
16:48:12 <rkukura> thanks Sukhdev_!
16:48:13 <Sukhdev_> #endmeeting