[ovs-dev] [PATCH 5/7] ovn-nb: Add Load-balancer table to schema.

[Zong Kai LI]

Hi, Guru.


> +  <table name="Load_Balancer" title="load balancer">
> +    <p>
> +      Each row represents one load balancer.
> +    </p>
> +
> +    <column name="vips">
> +      <p>
> +        A map of virtual IPv4 addresses (and an optional port number
> separated
> +        by <code>:</code>) associated with this load balancer and
> +        their corresponding endpoint IPv4 addresses (and optional port
> numbers
> +        separated by <code>:</code>) separated by commas.  If
>

I'm not sure whether will this confuse others, but it confused me.
At first, I thougth it would be:
ovn-nbctl create load-balancer vips:'"30.0.0.1:8000,40.0.0.1:5000"'='"
10.0.0.1:80,10.0.0.2:80,10.0.0.3:80"'
it inserts successfully into DB, but fails in ovn-northd for "bad ip port
for load balancer key".
Later, when I read your patch 7/7, I recognize it should be:
ovn-nbctl create load-balancer vips:'"30.0.0.1:8000"'='"10.0.0.1:80,
10.0.0.2:80,10.0.0.3:80"' vips:'"30.0.0.1:5000"'='"10.0.0.1:80,10.0.0.2:80,
10.0.0.3:80"', this time it get correctly parsed in ovn-northd.

Woud you mind to update the description here to notice it's a list of maps,
or/and give some examples?

Thanks.
Zong Kai, LI