[ovs-dev] [rspan 2/2] Document and warn that mirroring to a VLAN is incompatible with SLB bonding.
Ethan Jackson
ethan at nicira.com
Sat Aug 6 01:22:07 UTC 2011
This seems fine, I would go slightly further though.
> + if (s->balance == BM_SLB && port->bridge->cfg->n_flood_vlans) {
> + VLOG_WARN("port %s: SLB bonds are incompatible with flood_vlans, "
> + "please use another bond type or disable flood_vlans",
> + port->name);
> + }
I would change this warning to an error, and actually fail to create
the port in this case. I'm afraid people are going to ignore the
warning in the log. If you disagree, go ahead and merge.
Ethan
More information about the dev
mailing list