diff mbox

[ovs-dev,v9,01/10] Add useful information to ovn E2E tests

Message ID 1457730385-28923-2-git-send-email-rmoats@us.ibm.com
State Accepted
Headers show

Commit Message

Ryan Moats March 11, 2016, 9:06 p.m. UTC 
From: RYAN D. MOATS <rmoats@us.ibm.com>

Modify E2E test to output the OF flows from all three
hypervisors to help debug when something goes wrong.

Signed-off-by: RYAN D. MOATS <rmoats@us.ibm.com>
---
 tests/ovn.at |   14 ++++++++++++++
 1 files changed, 14 insertions(+), 0 deletions(-)

Comments

Ben Pfaff March 22, 2016, 9:30 p.m. UTC | #1 
On Fri, Mar 11, 2016 at 03:06:16PM -0600, Ryan Moats wrote:
> From: RYAN D. MOATS <rmoats@us.ibm.com>
> 
> Modify E2E test to output the OF flows from all three
> hypervisors to help debug when something goes wrong.
> 
> Signed-off-by: RYAN D. MOATS <rmoats@us.ibm.com>

Applied, thanks.

By the way, none (!) of the patches in this series found their way to my
mailbox, so I've picked them out of patchwork.  That probably breaks the
threading in your MUA (at least because patchwork corrupts the subject
line).  Sorry.
Ryan Moats March 23, 2016, 1:25 p.m. UTC | #2 
Ben Pfaff <blp@ovn.org> wrote on 03/22/2016 04:30:34 PM:

> From: Ben Pfaff <blp@ovn.org>
> To: Ryan Moats/Omaha/IBM@IBMUS
> Cc: dev@openvswitch.org
> Date: 03/22/2016 04:30 PM
> Subject: Re: [ovs-dev,v9,01/10] Add useful information to ovn E2E tests
>
> On Fri, Mar 11, 2016 at 03:06:16PM -0600, Ryan Moats wrote:
> > From: RYAN D. MOATS <rmoats@us.ibm.com>
> >
> > Modify E2E test to output the OF flows from all three
> > hypervisors to help debug when something goes wrong.
> >
> > Signed-off-by: RYAN D. MOATS <rmoats@us.ibm.com>
>
> Applied, thanks.
>
> By the way, none (!) of the patches in this series found their way to my
> mailbox, so I've picked them out of patchwork.  That probably breaks the
> threading in your MUA (at least because patchwork corrupts the subject
> line).  Sorry.
>

Thanks, and that's ok, because I'm in the middle of a major rebase
anyway...

Ryan (regXboi)
diff mbox

Patch

diff --git a/tests/ovn.at b/tests/ovn.at
index 5cb7d8b..6fcec99 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -1022,8 +1022,22 @@  ovn_populate_arp
 # Allow some time for ovn-northd and ovn-controller to catch up.
 # XXX This should be more systematic.
 sleep 1
+echo "------ OVN dump ------"
+ovn-nbctl show
 ovn-sbctl show
 
+echo "------ hv1 dump ------"
+as hv1 ovs-vsctl show
+as hv1 ovs-ofctl -O OpenFlow13 dump-flows br-int
+
+echo "------ hv2 dump ------"
+as hv2 ovs-vsctl show
+as hv2 ovs-ofctl -O OpenFlow13 dump-flows br-int
+
+echo "------ hv3 dump ------"
+as hv3 ovs-vsctl show
+as hv3 ovs-ofctl -O OpenFlow13 dump-flows br-int
+
 # test_packet INPORT DST SRC ETHTYPE OUTPORT...
 #
 # This shell function causes a packet to be received on INPORT.  The packet's