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

Han Zhou zhouhan at gmail.com
Thu Jun 23 06:33:39 UTC 2016


Thanks Babu for taking over this. I'd like to repost my comment here:

On Wed, Jun 22, 2016 at 10:05 PM, <bschanmu at redhat.com> wrote:

> diff --git a/ovn/ovn-sb.ovsschema b/ovn/ovn-sb.ovsschema
> index 06e8a07..22f7ad0 100644
> --- a/ovn/ovn-sb.ovsschema
> +++ b/ovn/ovn-sb.ovsschema
> @@ -1,7 +1,7 @@
>  {
>      "name": "OVN_Southbound",
> -    "version": "1.3.0",
> -    "cksum": "654726257 5528",
> +    "version": "1.4.0",
> +    "cksum": "2272541964 5853",
>      "tables": {
>          "Chassis": {
>              "columns": {
> @@ -28,6 +28,14 @@
>                                       "min": 0,
>                                       "max": "unlimited"}},
>                  "ip": {"type": "string"}}},
> +        "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.

Thanks,
Han



More information about the dev mailing list