Hey Russell,

With your add type/option change and Ben's tunnel-key work in, it is time
to rebase my ovn-controller-vtep series~~

So, now, I think I'd like to adopt your earlier suggestion, which requires
users
to specify this when creating logical port (for vtep switch):

  Logical_Port
    name: <anything>
    type: vtep
    options:
       vtep_physical_switch: <...>
       vtep_logical_switch: <...>
And the vtep_logical_switch column in ovn-sb Chassis table now becomes a
list of vtep logical switch names (the physical switch attached to).

"vtep_logical_switches" : {"type": {"key": "string",
                                                      "min": 0,
                                                      "max": "unlimited"}}},


To make sure, we should also expose similar interface in CMS to allow users
specify the logical port type and options right?

Thanks,
Alex Wang,

On Mon, Jul 27, 2015 at 2:12 PM, Alex Wang <al...@nicira.com> wrote:

> I think I'll hold off this series a bit, and adopt the similar changes as
> you did
> in the "Change strategy for tunnel keys" series.
>
> Also, with Russell's change, the table could be renamed again,
>
> At the same time, I'll help with reviews,
>
> Thanks,
> Alex Wang,
>
> On Mon, Jul 27, 2015 at 1:33 PM, Ben Pfaff <b...@nicira.com> wrote:
>
>> On Thu, Jul 16, 2015 at 12:56:22AM -0700, Alex Wang wrote:
>> > In a gateway like the VTEP L2 gateway, physical vlans belonging to
>> > the same logical network form a "logical switch".  Each logical switch
>> > has a dedicated tunnel key and will keep records of all MACs learned
>> > from the owned vlans.  So user can just send packet to a "logical
>> > switch" and the gateway will figure out the output port and vlan tag
>> > automatically.
>> >
>> > Therefore, it is not really necessary to keep record of the vlan map
>> > for each gateway physical port in the OVN_Southbound database using
>> > "gateway_ports".
>> >
>> > Thusly, this commit removes the "Gateway" table from the OVN_Southbound
>> > database.  In the "Chassis" table, the "gateway_ports" column is
>> replaced
>> > by "vtep_logical_switches" column which maps the vtep logical switch
>> name
>> > in the gateway to a logical port name.
>> >
>> > Signed-off-by: Alex Wang <al...@nicira.com>
>> >
>> > ---
>> > V3->V4:
>> > - change the column name to "vtep_logical_switches".
>> > - adopt Ben's refinement of ovn-sb schema manual.
>> >
>> > ->V3:
>> > - Realize that the Gateway table is not needed.
>>
>> Acked-by: Ben Pfaff <b...@nicira.com>
>>
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to