[ovs-dev] [PATCH] ovs-vtep: Support userspace datapaths.

[Daniele Di Proietto]

On 15/06/2015 20:40, "Alex Wang" <alexw at nicira.com> wrote:

>I actually have a similar patch in my ovn for vtep series.  but instead, I
>added
>a new command-line option for that...  But goal is the same, => use
>different
>datapath type (dummy in my case).
>
>So, I'm fine with this change, and will drop mine, looks good,

I had this for a long time on my branches and I thought you might find it
useful.  If you prefer another approach I will drop it.

Thanks Alex and Ben for your input

>
>On Mon, Jun 15, 2015 at 11:36 AM, Ben Pfaff <blp at nicira.com> wrote:
>
>> On Mon, Jun 15, 2015 at 11:28:12AM -0700, Gurucharan Shetty wrote:
>> > On Mon, Jun 15, 2015 at 11:09 AM, Daniele Di Proietto
>> > <diproiettod at vmware.com> wrote:
>> > > With this commit, the VTEP emulator detects the datapath_type of the
>> > > bridge used as a "physical" switch, and creates subsequent bridges
>> > > with the same type.  This allows ovs-vtep to work with the userspace
>> > > datapath.
>> > >
>> > > Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
>>
>> ...
>>
>> > > +        ovs.util.ovs_fatal(0, "couldn't find %s datapath_type" %
>> ps_name, vlog)
>> > Is datapath_type always populated automatically? When I create a
>> > bridge on tip of master, that column does not get populated.
>>
>> Good catch, I think: it's not populated automatically, so treating "" as
>> an error is probably a bad idea.
>> _______________________________________________
>> dev mailing list
>> dev at openvswitch.org
>> 
>>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailm
>>an_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=
>>SmB5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=DRpxd0AAOCFSEkAcAaKydYlirka
>>4I7cNFBpy-xfX59M&s=caNG5Lx2tx3DfOkJNrQBhhvIslCb3qenFwzQDpm8Q-k&e=
>>
>_______________________________________________
>dev mailing list
>dev at openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Sm
>B5nZacmXNq0gKCC1s_Cw5yUNjxgD4v5kJqZ2uWLlE&m=DRpxd0AAOCFSEkAcAaKydYlirka4I7
>cNFBpy-xfX59M&s=caNG5Lx2tx3DfOkJNrQBhhvIslCb3qenFwzQDpm8Q-k&e=