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 <jpet...@ovn.org> > > --Justin > > > > On May 18, 2016, at 8:40 AM, Ben Pfaff <b...@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 <b...@ovn.org> wrote: > >>> > >>> On Sun, May 01, 2016 at 01:44:26PM +0800, nickcooper-zhangtonghao wrote: > >>>> signed-off-by: nickcooper-zhangtong...@opencloud.tech > >>>> <mailto:nickcooper-zhangtong...@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@host> > >>> not as you did it. > >>> > >>> Thanks, > >>> > >>> Ben. > >> > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
