Do not install any potential logical switch "router type"
port arp responders. Logical router port arp responders
should be sufficient in this respect.
It seems a little wierd for a logical switch not proxying
for a remote VIF to be responding to arp requests and we
are not functionally using this capability in ovn.
Signed-off-by: Darrell Ball <dlu...@gmail.com>
---
ovn/northd/ovn-northd.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
index eeeb41d..17ea90f 100644
--- a/ovn/northd/ovn-northd.c
+++ b/ovn/northd/ovn-northd.c
@@ -2699,11 +2699,11 @@ build_lswitch_flows(struct hmap *datapaths, struct hmap
*ports,
}
/*
- * Add ARP/ND reply flows if either the
- * - port is up or
- * - port type is router
+ * Add ARP/ND reply flows if the
+ * - port is up and
+ * - port type is not router
*/
- if (!lsp_is_up(op->nbsp) && strcmp(op->nbsp->type, "router")) {
+ if (!lsp_is_up(op->nbsp) || !strcmp(op->nbsp->type, "router")) {
continue;
}
--
1.9.1
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev