On 15 Mar 2025, at 18:43, Aaron Conole wrote:
> When the 'mirror-to' option was conceived, it was intended to allow users
> the ability to arbitrarily build their own ports to use the mirror port
> rather than always using an algorithmically defined mirror port. However,
> the mirror port code never accommodated a port that the user defined as
> part of an OVS bridge. This could be useful for running against all kinds
> of OVS specific ports.
>
> Adjust the 'mirror-to' option so that when the user specifies a port, the
> utility searches through system interfaces, as well as the OVS DB. This
> means that we need to drop the port_exists() check for the mirror port,
> but that should be okay.
>
> Reported-at:
> https://mail.openvswitch.org/pipermail/ovs-discuss/2025-March/053508.html
> Reported-at: https://issues.redhat.com/browse/FDP-1194
> Reported-by: Jun Wang <[email protected]>
> Acked-by: Mike Pattrick <[email protected]>
> Signed-off-by: Aaron Conole <[email protected]>
Thanks Aaron for all the changes. The patch looks good to me.
Acked-by: Eelco Chaudron <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev