diff options
| author | Ben Pfaff <blp@nicira.com> | 2011-09-15 15:55:45 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2011-09-15 15:55:45 -0700 |
| commit | 9ed3ba29b36ab99b75d6519660f58d6f5f9bd190 (patch) | |
| tree | 0e27e4f2e15d69ce23aa51ffe132b2c44dc2f90e /vswitchd | |
| parent | a193dde952a9ec7bb388e08b86bd2929edd3a353 (diff) | |
| download | openvswitch-9ed3ba29b36ab99b75d6519660f58d6f5f9bd190.tar.gz | |
ovs-brcompatd: Delete ports when netdevs on fake bridges disappear.
Until now, when a network device disappeared, netdev_changed_cb() passed
the name of the bridge that contained the network device to ovs-vsctl as
part of the "del-port" command. However, when the network device was
actually a "fake bridge", it would pass the name of the real bridge, which
ovs-vsctl rejected as wrong (expecting the name of the fake bridge) and
not remove the port.
This fixes the problem by dropping the bridge name, which is simpler than
trying to get the name of the fake bridge in this case.
Reported-by: Tyler Coumbes <coumbes@gmail.com>
Tested-by: Tyler Coumbes <coumbes@gmail.com>
Diffstat (limited to 'vswitchd')
| -rw-r--r-- | vswitchd/ovs-brcompatd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vswitchd/ovs-brcompatd.c b/vswitchd/ovs-brcompatd.c index 41187b171..bfa13ca26 100644 --- a/vswitchd/ovs-brcompatd.c +++ b/vswitchd/ovs-brcompatd.c @@ -790,7 +790,7 @@ netdev_changed_cb(const struct rtnetlink_link_change *change, port_name, br_name); run_vsctl(vsctl_program, VSCTL_OPTIONS, - "--", "--if-exists", "del-port", br_name, port_name, + "--", "--if-exists", "del-port", port_name, "--", "comment", "ovs-brcompatd:", port_name, "disappeared", (char *) NULL); } |