[ovs-dev] [PATCH v2 3/3] ovn: Add address_set() support for ACLs.

Han Zhou zhouhan at gmail.com
Mon May 2 06:34:21 UTC 2016


One more comment, if you will be working on v3.

On Thu, Apr 7, 2016 at 8:46 AM, Russell Bryant <russell at ovn.org> wrote:
> diff --git a/ovn/ovn-nb.ovsschema b/ovn/ovn-nb.ovsschema
> index 40a7a97..c26d8ae 100644
> --- a/ovn/ovn-nb.ovsschema
> +++ b/ovn/ovn-nb.ovsschema
> @@ -1,7 +1,7 @@
>  {
>      "name": "OVN_Northbound",
> -    "version": "2.0.2",
> -    "cksum": "4289495412 4436",
> +    "version": "2.1.0",
> +    "cksum": "2262966655 4761",
>      "tables": {
>          "Logical_Switch": {
>              "columns": {
> @@ -48,6 +48,14 @@
>                               "min": 0, "max": "unlimited"}}},
>              "indexes": [["name"]],
>              "isRoot": false},
> +        "Address_Set": {
> +            "columns": {
> +                "name": {"type": "string"},
> +                "addresses": {"type": {"key": "string",
> +                                       "min": 0,
> +                                       "max": "unlimited"}}},
> +            "indexes": [["name"]],
> +            "isRoot": true},

It may be good to have column "external_ids", so that external names, such
as security-group name in neutron, can be recognized easily.

--
Best regards,
Han



More information about the dev mailing list