On Wed, May 03, 2017 at 02:59:15PM -0700, Andy Zhou wrote:
> On Wed, May 3, 2017 at 8:45 AM, Ben Pfaff <[email protected]> wrote:
> > It's much easier to see what's going on in the southbound database if
> > human-friendly names are available.
> >
> > Really it's too bad that we didn't put the human-friendly name in "name"
> > and the UUID in something like "external_ids:neutron-uuid", but it'll take
> > more coordination to change that at this point and it may not be worth it.
> >
> > Signed-off-by: Ben Pfaff <[email protected]>
> Acked-by: Andy Zhou <[email protected]>
Thanks!
> > @@ -734,6 +739,9 @@ build_datapaths(struct northd_context *ctx, struct hmap
> > *datapaths)
> > if (*name) {
> > smap_add(&ids, "name", name);
> > }
> In the definition of both nbrec_loical_router and
> nbrec_logical_switch, the comment of the 'name' filed
> says it should nonnull. If that is true, then we don't need this check?
Oh, thanks for noticing. I simplified this code.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
