[ovs-git] Open vSwitch: ovs-ofctl: Avoid printing false differences on "ovs-ofctl diff-flows". (branch-1.8)
dev at openvswitch.org
dev at openvswitch.org
Wed Jul 25 20:11:41 UTC 2012
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".
The branch, branch-1.8 has been updated
via e23285a6725a21183e31b9a49ee19c0747d1f0fa (commit)
from 64452d5c7bfb01f2eb65ac22e394c4cc9ea1c53d (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit e23285a6725a21183e31b9a49ee19c0747d1f0fa
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e23285a6725a21183e31b9a49ee19c0747d1f0fa
Author: Ben Pfaff <blp at nicira.com>
ovs-ofctl: Avoid printing false differences on "ovs-ofctl diff-flows".
It is possible for "struct ofpact"s to differ bytewise even if they are
equivalent when converted to another representation, such as OpenFlow 1.0
action format or a string representation. This can cause "ovs-ofctl
diff-flows" to print surprising false "differences", e.g. as in the bug
report:
- actions=resubmit(,1)
+ actions=resubmit(,1)
This commit fixes the problem by comparing not just the ofpacts but also
the string representation and printing a difference only if both differ.
Bug #8899.
Reported-by: Luca Giraudo <lgiraudo at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
-----------------------------------------------------------------------
Summary of changes:
tests/ovs-ofctl.at | 25 +++++++++++++++++++++
utilities/ovs-ofctl.c | 56 +++++++++++++++++++++++++++++-------------------
2 files changed, 59 insertions(+), 22 deletions(-)
hooks/post-receive
--
Open vSwitch
More information about the git
mailing list