From 91a5b5be2f1be19d22413cc4194474a48b6a2967 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Wed, 16 Oct 2013 09:28:47 -0700 Subject: [PATCH] oftest: Update default controller port number to match recent OFTest. Also, document how to work around it if you use an older OFTest. Reported-by: Simon Horman Reported-by: Lori Jakab Signed-off-by: Ben Pfaff --- README-OFTest | 7 +++++++ tests/run-oftest | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/README-OFTest b/README-OFTest index 1b8516be5..cdca06d37 100644 --- a/README-OFTest +++ b/README-OFTest @@ -50,6 +50,13 @@ default) and enable verbose logging: make check-oftest OFT= OFTFLAGS='--verbose -T basic.Echo' +If you use OFTest that does not include commit 4d1f3eb2c792 (oft: +change default port to 6653), merged into the OFTest repository in +October 2013, then you need to add an option to use the IETF-assigned +controller port: + + make check-oftest OFT= OFTFLAGS='--port=6653' + Interpreting OFTest Results --------------------------- diff --git a/tests/run-oftest b/tests/run-oftest index d12a22fc0..c73c25c90 100755 --- a/tests/run-oftest +++ b/tests/run-oftest @@ -74,7 +74,7 @@ for port in p1 p2 p3 p4; do options:pstream=punix:$OVS_RUNDIR/$port done run ovs-vsctl \ - -- set-controller br0 tcp:127.0.0.1 \ + -- set-controller br0 tcp:127.0.0.1:6653 \ -- set controller br0 connection-mode=out-of-band max-backoff=1000 # Run OFTest. -- 2.47.0