[ovs-dev] [PATCH] tests: Ignore more error messages for hidden rules test.

[Ben Pfaff]

Thanks, applied to master and branch-2.4.

On Tue, Sep 29, 2015 at 02:06:37PM -0700, Justin Pettit wrote:
> Acked-by: Justin Pettit <jpettit at nicira.com>
> 
> --Justin
> 
> 
> > On Sep 29, 2015, at 10:19 AM, Ben Pfaff <blp at nicira.com> wrote:
> > 
> > This test intentionally configures an unreachable controller.  It ignored
> > some error messages in the log, specifically
> >    br0: cannot find route for controller (240.0.0.1): ...
> > but a bug report says that other forms of messages can also appear, e.g.
> >    br0<->tcp:240.0.0.1:6653: connection dropped (No route to host)
> > This commit therefore expands the logged error messages that will be
> > ignored to any message that includes the IP address 240.0.0.1.
> > 
> > Reported-by: "Kurek, Tytus" <Tytus.Kurek at pega.com>
> > Reported-at: http://openvswitch.org/pipermail/discuss/2015-September/018910.html
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > AUTHORS          | 1 +
> > tests/ofproto.at | 2 +-
> > 2 files changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/AUTHORS b/AUTHORS
> > index 3f01a0f..cba0535 100644
> > --- a/AUTHORS
> > +++ b/AUTHORS
> > @@ -371,6 +371,7 @@ Teemu Koponen           koponen at nicira.com
> > Thomas Morin            thomas.morin at orange.com
> > Timothy Chen            tchen at nicira.com
> > Torbjorn Tornkvist      kruskakli at gmail.com
> > +Tytus Kurek             Tytus.Kurek at pega.com
> > Valentin Bud            valentin at hackaserver.com
> > Vasiliy Tolstov         v.tolstov at selfip.ru
> > Vasu Dasari             vdasari at gmail.com
> > diff --git a/tests/ofproto.at b/tests/ofproto.at
> > index 1ad80d3..c012a34 100644
> > --- a/tests/ofproto.at
> > +++ b/tests/ofproto.at
> > @@ -1411,7 +1411,7 @@ ditto() {
> > }
> > (head_table classifier; ditto 1 253) > expout
> > AT_CHECK([ovs-ofctl dump-tables br0], [0], [expout])
> > -OVS_VSWITCHD_STOP(["/cannot find route for controller/d"])
> > +OVS_VSWITCHD_STOP(["/240\.0\.0\.1/d"])
> > AT_CLEANUP
> > 
> > AT_SETUP([ofproto - flow table configuration (OpenFlow 1.2)])
> > -- 
> > 2.1.3
> > 
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
>