[ovs-dev] [PATCH] OVN: add mac address only support to IPAM/MACAM

[Lorenzo Bianconi]

>
> On Wed, Dec 12, 2018 at 11:53:49AM +0100, Lorenzo Bianconi wrote:
> > Add the capability to assign just L2 address to IPAM/MACAM since
> > in the current implementation either subnet or ipv6_prefix are mandatory
> > to enable IPAM
> >
> > Tested-by: Yossi Segev <ysegev at redhat.com>
> > Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi at redhat.com>
>
> Hi Lorenzo, thank you for the patch.
>
> I have one suggestion for the documentation.  It uses text to say that
> the column is Boolean, but usually we would allow the documentation
> translator to do that by telling it that the value is boolean, with
> type='{"type": "boolean"}' inside the <column> tag:

Hi Ben,

thx for the review. I will fix it in v2.

Regards,
Lorenzo

>
> > +      <column name="other_config" key="mac_only">
> > +        Boolean value used to request to assign L2 address only if neither
> > +        subnet nor ipv6_prefix are specified
> > +      </column>
> >      </group>
>
> Thanks,
>
> Ben.