[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openvswitch-dev
Subject:    [ovs-dev] [PATCH 2/4] ovs-brcompatd: Delete Bridge record when deleting a bridge.
From:       jpettit () nicira ! com (Justin Pettit)
Date:       2010-02-25 6:27:08
Message-ID: 81B13A92-AA9E-48E6-A038-6419B70A5A4F () nicira ! com
[Download RAW message or body]

Looks good.

--Justin


On Feb 23, 2010, at 2:35 PM, Ben Pfaff wrote:

> del_bridge() removed the reference to the Bridge record from the
> Open_vSwitch table, but it didn't actually delete the Bridge record itself.
> This fixes the problem.
> 
> Bug #2425.
> ---
> vswitchd/ovs-brcompatd.c |    3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/vswitchd/ovs-brcompatd.c b/vswitchd/ovs-brcompatd.c
> index d0568a4..bf571d7 100644
> --- a/vswitchd/ovs-brcompatd.c
> +++ b/vswitchd/ovs-brcompatd.c
> @@ -509,6 +509,9 @@ del_bridge(const struct ovsrec_open_vswitch *ovs, const char *br_name)
>     ovsrec_open_vswitch_set_bridges(ovs, bridges, n);
>     free(bridges);
> 
> +    /* Delete the bridge itself. */
> +    ovsrec_bridge_delete(br);
> +
>     VLOG_INFO("delbr %s: success", br_name);
> 
>     return 0;
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org




[prev in list] [next in list] [prev in thread] [next in thread]