On 1 July 2016 at 01:21, Zong Kai LI <zealo...@gmail.com> wrote: > 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? > The patch0 of the series did give a couple of examples. But I see your point. I will add examples in the man page as part of v2.
> > Thanks. > Zong Kai, LI > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
