On Thu, Apr 06, 2017 at 08:58:02AM -0700, Ben Pfaff wrote:
> On Mon, Mar 27, 2017 at 02:56:12PM -0400, Lance Richardson wrote:
> > Add rbac "roles" and "permissions" tables to ovn southbound
> > database schema.
> >
> > Signed-off-by: Lance Richardson <lrich...@redhat.com>
>
> I see that master has changed a little and will require a respin here,
> but I doubt it's a big deal.
>
> This is where it becomes important to have good documentation for how
> OVSDB RBAC works; I guess that this is not yet available (other than the
> commit message) in the previous patch.
>
> I see that this adds some documentation for the ovn-sb RBAC tables, but
> there's a real need for a description of the overall design and its
> goals, because otherwise the users won't understand what's important
> here. I'd expect that that would go into the ovn-architecture or
> possibly the ovn-northd manpage.
Also, Clang says:
../ovn/northd/ovn-northd.c:5567:10: error: missing field 'auth' initializer
[-Werror,-Wmissing-field-initializers]
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
