diff options
| author | Ashish Varma <ashishvarma.ovs@gmail.com> | 2018-08-06 15:53:37 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2018-08-06 16:44:45 -0700 |
| commit | ba35931267b6f900863bc9abd137fdb12cbaae6e (patch) | |
| tree | a9a2875179a645041b1c080907baad7d16af1a28 /tests | |
| parent | ce679280889f0eb4ebc95b62558a20a7a5f7c0fb (diff) | |
| download | openvswitch-ba35931267b6f900863bc9abd137fdb12cbaae6e.tar.gz | |
tests: Test for ovs-ofctl snoop command
Added test for snoop command to check for the initial handshake messages
when a bridge connects to a controller via 'unix' connection method.
Signed-off-by: Ashish Varma <ashishvarma.ovs@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/ovs-ofctl.at | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/tests/ovs-ofctl.at b/tests/ovs-ofctl.at index 06597d7b5..fef6aa407 100644 --- a/tests/ovs-ofctl.at +++ b/tests/ovs-ofctl.at @@ -3184,3 +3184,30 @@ AT_CHECK([grep -q "ct_dpif|DBG|.*ct_flush: zone 123" ovs-vswitchd.log]) OVS_VSWITCHD_STOP AT_CLEANUP + + +AT_SETUP([ovs-ofctl snoop]) +OVS_VSWITCHD_START + +dnl setup controller for br0 before starting the controller +AT_CHECK([ovs-vsctl -vsyslog:off set-controller br0 unix:testcontroller]) + +dnl then start listening on the '.snoop' connection +on_exit 'kill `cat ovs-ofctl-snoop.pid`' +AT_CHECK([ovs-ofctl -vsyslog:off --detach --pidfile=ovs-ofctl-snoop.pid snoop br0 > snoopbr0.txt 2>&1]) + +dnl finally start the controller +on_exit 'kill `cat ovs-testcontroller.pid`' +AT_CHECK([ovs-testcontroller -vsyslog:off --detach --pidfile punix:testcontroller], [0], [ignore]) +OVS_WAIT_UNTIL([test -e testcontroller]) + +dnl check for some of the initial handshake messages +OVS_WAIT_UNTIL([egrep "OFPT_FEATURES_REQUEST" snoopbr0.txt >/dev/null 2>&1]) +OVS_WAIT_UNTIL([egrep "OFPT_FEATURES_REPLY" snoopbr0.txt >/dev/null 2>&1]) +OVS_WAIT_UNTIL([egrep "OFPT_SET_CONFIG" snoopbr0.txt >/dev/null 2>&1]) + +dnl need to suppress the 'connection failed' WARN message in ovs-vswitchd +dnl because we need ovs-vswitchd to have the controller config before starting +dnl the controller to 'snoop' the OpenFlow messages from beginning +OVS_VSWITCHD_STOP(["/connection failed (No such file or directory)/d"]) +AT_CLEANUP |