[ovs-dev] [PATCH ovn] ovn-controller-vtep: Set chassis_name for newly created Encap.

[Odintsov Vladislav]

Hi,

Thanks for the quick reaction!
I've tested your patch against 20.06.3 and it works well for me.
May I ask you to backport your patch down to supported branches?
 
Tested-by: Vladislav Odintsov <odivlad at gmail.com>

On 20.04.2021, 20:58, "dev on behalf of Mark Michelson" <ovs-dev-bounces at openvswitch.org on behalf of mmichels at redhat.com> wrote:

    Acked-by: Mark Michelson <mmichels at redhat.com>

    On 4/20/21 6:32 AM, Dumitru Ceara wrote:
    > Without this ovn-controller-vtep fails to register new chassis when RBAC
    > is enabled.
    > 
    > Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2021-April/382207.html
    > Reported-by: Odintsov Vladislav <VlOdintsov at croc.ru>
    > Fixes: d06760b64276 ("ovn: Restrict encap modification to its creating chassis")
    > Signed-off-by: Dumitru Ceara <dceara at redhat.com>
    > ---
    >   controller-vtep/gateway.c | 1 +
    >   1 file changed, 1 insertion(+)
    > 
    > diff --git a/controller-vtep/gateway.c b/controller-vtep/gateway.c
    > index 636dab365..e9419138b 100644
    > --- a/controller-vtep/gateway.c
    > +++ b/controller-vtep/gateway.c
    > @@ -59,6 +59,7 @@ create_chassis_rec(struct ovsdb_idl_txn *txn, const char *name,
    >       sbrec_encap_set_ip(encap_rec, encap_ip);
    >       const struct smap options = SMAP_CONST1(&options, "csum", "false");
    >       sbrec_encap_set_options(encap_rec, &options);
    > +    sbrec_encap_set_chassis_name(encap_rec, name);
    >       sbrec_chassis_set_encaps(chassis_rec, &encap_rec, 1);
    >   
    >       return chassis_rec;
    > 

    _______________________________________________
    dev mailing list
    dev at openvswitch.org
    https://mail.openvswitch.org/mailman/listinfo/ovs-dev