[ovs-dev] [PATCH 3/3] ovn-controller-vtep.at: Fix intermittent test failure.
rbryant at redhat.com
Sun Aug 16 21:15:40 UTC 2015
On 08/16/2015 12:56 PM, Alex Wang wrote:
> When testing the recreation of 'chassis' table entry by 'ovn-controller-
> vtep'. The removal of 'chassis' table entry by the 'ovn-sbctl' could
> cause 'Broken pipe' warning in ovsdb-server.log. This is due to the
> race between 'ovn-sbctl' exiting and 'ovn-controller-vtep' adding
> the chassis back. So, if the 'ovn-sbctl' exits right when the
> ovsdb-server tries to send update of readd of the deleted 'chassis',
> the sending will fail with 'Broken pipe' error.
> Signed-off-by: Alex Wang <alexw at nicira.com>
> tests/ovn-controller-vtep.at | 4 ++++
> 1 file changed, 4 insertions(+)
> diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
> index 82f6166..12d0134 100644
> --- a/tests/ovn-controller-vtep.at
> +++ b/tests/ovn-controller-vtep.at
> @@ -104,6 +104,10 @@ OVS_WAIT_UNTIL([test -n "`grep WARN ovn-controller-vtep.log`"])
> AT_CHECK([sed -n 's/^.*\(|WARN|.*\)$/\1/p' ovn-controller-vtep.log], , [dnl
> |WARN|Chassis for VTEP physical switch (br-vtep) disappears, maybe deleted by ovn-sbctl, adding it back
> +# this removal of chassis could cause 'Broken pipe' warning in the ovsdb-server.log,
> +# due to the race between 'ovn-sbctl' exiting and 'ovn-controller-vtep' adding
> +# the chassis back. so just removes the 'Broken pipe' warning from ovsdb-server.log.
> +AT_CHECK([sed -i '/Broken pipe/d' ovsdb-server.log])
> # changes the tunnel_ip on physical switch, watches the update of chassis's
> # encap.
Acked-by: Russell Bryant <rbryant at redhat.com>
More information about the dev