[ovs-dev] [PATCH 2/4] ovs-brcompatd: Delete Bridge record when deleting a bridge.

Justin Pettit jpettit at nicira.com
Wed Feb 24 22:27:08 PST 2010


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





More information about the dev mailing list