[ovs-dev] [PATCH] vtep/ovs-vtep: support multiple ovs-vtep processes

Ben Pfaff blp at ovn.org
Wed May 18 17:22:14 UTC 2016


Thanks nick, Justin, Darrell, I applied this to master.

On Wed, May 18, 2016 at 09:15:27AM -0700, Justin Pettit wrote:
> Thanks for checking:
> 
> Acked-by: Justin Pettit <jpettit at ovn.org>
> 
> --Justin
> 
> 
> > On May 18, 2016, at 8:40 AM, Ben Pfaff <blp at ovn.org> wrote:
> > 
> > The pull request seems OK to me.  Justin, do you have an opinion on this
> > patch?
> > 
> > On Wed, May 18, 2016 at 06:22:23PM +0800, nickcooper-zhangtonghao wrote:
> >> Hi Ben,
> >> 	Thank you for your advice. In general case, we run an ovs-vtep process on the hypervisor. 
> >> In our network virtualization solution, we want to run multiple copies of the vtep simulator - ovs-vtep for the different network, 
> >> such as the data/management/storage network.
> >> 
> >> Start the VTEP emulator:
> >> 	ovs-vtep --detach mgmt-network
> >> 	ovs-vtep --detach stor-network
> >> 	ovs-vtep --detach data-network
> >> 
> >> Bind the logical switch to a port:
> >> 	vtep-ctl bind-ls mgmt-network vnet0 0 ls0
> >> 	vtep-ctl bind-ls stor-network vnet1 0 ls1
> >> 	vtep-ctl bind-ls data-network vnet2 0 ls2
> >> 
> >> 
> >> In fact, the vnet0/vnet1/vnet2 have been connected to the “vtep_ls1” bridge. So, the VMs of the different physical switches have been connected. 
> >> This is arguably an ovs-vtep bug. I fixed this bug by including the physical switch name as part of the logical switch name.  
> >> Darrell Ball did an initial test using the OVN gateway test case and did not see any issues with the proposed diff.
> >> I have submited a pull request. https://github.com/openvswitch/ovs/pull/126 <https://github.com/openvswitch/ovs/pull/126>
> >> 
> >> 
> >> 
> >> Thanks,
> >> nickcooper-zhangtonghao.
> >> 
> >> 
> >>> On May 18, 2016, at 10:26 AM, Ben Pfaff <blp at ovn.org> wrote:
> >>> 
> >>> On Sun, May 01, 2016 at 01:44:26PM +0800, nickcooper-zhangtonghao wrote:
> >>>> signed-off-by: nickcooper-zhangtonghao at opencloud.tech <mailto:nickcooper-zhangtonghao at opencloud.tech>
> >>>> 
> >>>> The computer can run multiple ovs-vtep processes, but ovs-vtep processes share the “vtep_lsX” bridge when binding the physical-switch and logical-switch. The bridge created by logical-switch should be named  according to logical-switch.
> >>> 
> >>> Hi Nick, thanks for submitting a patch.
> >>> 
> >>> You'll need to resubmit it for a number of reasons.
> >>> 
> >>> Your return email address was totally wrong and definitely not usable.
> >>> I think I've corrected it; we'll see if my reply bounces.
> >>> 
> >>> Please use a short (<75 character) title.
> >>> 
> >>> Please limit lines in the commit message to 75 characters or so.
> >>> 
> >>> "Signed-off-by" goes at the end of the message and it should be
> >>> formatted as:
> >>>       Signed-off-by: Your Real Name <email at host>
> >>> not as you did it.
> >>> 
> >>> Thanks,
> >>> 
> >>> Ben.
> >> 
> 



More information about the dev mailing list