Thanks Russell

Pls see inline

Darrell


From: Russell Bryant <russ...@ovn.org<mailto:russ...@ovn.org>>
Date: Wednesday, March 2, 2016 at 5:40 AM
To: Darrel Ball <db...@vmware.com<mailto:db...@vmware.com>>
Cc: Mickey Spiegel <emspi...@us.ibm.com<mailto:emspi...@us.ibm.com>>, Darrell 
Lu <dlu...@gmail.com<mailto:dlu...@gmail.com>>, 
"dev@openvswitch.org<mailto:dev@openvswitch.org>" 
<dev@openvswitch.org<mailto:dev@openvswitch.org>>, Han Zhou 
<zhou...@gmail.com<mailto:zhou...@gmail.com>>
Subject: Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint 
separation proposal



On Tue, Mar 1, 2016 at 11:27 PM, Darrell Ball 
<db...@vmware.com<mailto:db...@vmware.com>> wrote:
Hi Mickey

I was going with the assumption that the “localnet” logical port on each HV has 
a unique name linked to HV/logical switch tuple
Localnet configuration uses multiple logical switches to support a single 
localnet.

My reference to base this assumption on was this link 
http://openvswitch.org/pipermail/git/2015-September/007480.html<https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_pipermail_git_2015-2DSeptember_007480.html&d=BQMFaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=dGZmbKhBG9tJHY4odedsGA&m=51R7VJdSgnlV9g1a2Fsx_-eTR9Td3GNirhJsHeMmeys&s=LWvFpYg_BVvmbjxZT1mRBiAAdikjp9e6MeCGmO9TJJ0&e=>

This changed within the last week.  Sorry.  :-)

https://github.com/openvswitch/ovs/commit/6e6c3f9188a19d4e8981eb7813dd87fa54b8e882<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvswitch_ovs_commit_6e6c3f9188a19d4e8981eb7813dd87fa54b8e882&d=BQMFaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=dGZmbKhBG9tJHY4odedsGA&m=51R7VJdSgnlV9g1a2Fsx_-eTR9Td3GNirhJsHeMmeys&s=QsCS1pNeu5Q6wDeqQ6OkW7wpN0WR3zlTYwjK_UX3cfI&e=>

We worked out an acceptable way to model connectivity to a localnet with a 
single logical switch.

>> Ok, that’s fine

Also, if you check the OVN tests for localnet, the configuration uses the same 
approach.

Oops.  This should have been changed when the above patch merged.  Han, is this 
something you could look at doing?

>> that would be good

If this assumption holds, then even for localnet, a logical port is only bound 
to a single physical endpoint (chassis/port/encap)

I think you are suggesting that a single logical port name for a localnet 
network be used across the HVs ?

Then the logical port becomes a single name, such as your example below - 
provnet1-physnet1

I understand the advantage of a using a single logical port name in some cases, 
as it slightly simplifies the configuration at the NB,

although we would still need to configure each HV access point uniquely with 
ovs-vsctl for physical endpoints then.


However, in general, I think there may be some reasons to retain the unique 
logical port names for localnet access points -

to support different addresses and port security per logical access point, per 
the NB schema.


Let me know if you think being able to have a single localnet logical port name 
across

hypervisors is a hard requirement.

It is, based on the current state of the code.  It helped simplify the 
OpenStack Neutron plugin a lot.  It also drastically reduces the number of 
logical flows needed to model many ports connected to a localnet.

>> agreed


I think the explicit bridge-mapping for localnet could be eliminated.

The network_name is associated with a logical port and a

logical port is bound to a physical endpoint chassis, which is the

localnet access bridge….

However, I did not mention it in this patch, because for localnet,

 I just wanted to focus on the encapsulation/tag for now.

Yes, I think moving the bridge mappings info into southbound is a nice 
improvement. We've come across a case where we need to know this information in 
our OpenStack Neutron plugin, so I was hoping that we'd be able to read it from 
the southbound database as a nice side effect of this work.

>> Using the fresh new localnet approach, this will require a physical 
>> endpoints set per logical port, which is fine
>> maybe I’ll roll the bridge-mapping changes in sooner than later

--
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to