diff options
| author |   Stefan Stanacar <stefanx.stanacar@intel.com> | 2013-09-25 12:30:49 +0000 |
|---|---|---|
| committer |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-09-26 16:37:13 +0100 |
| commit | 6fd19a9df0ad25b2822f12e2c3a97f1b71068d4e (patch) | |
| tree | 85bf7fb81e41d5cfda18143f69e67a5cf5b6ed4b /meta | |
| parent | 829edcb6e59e2e20dd4165b727b685f286f38031 (diff) | |
| download | openembedded-core-contrib-6fd19a9df0ad25b2822f12e2c3a97f1b71068d4e.tar.gz | |
lib/oeqa/runtime: ping: wait for 5 echo replies
Instead of considering that ping test passed after 1 reply,
wait for at least 5 consecutive replies in 60 seconds (which should
be enough time for connman to reconfigure the interface in systemd
images and help with the fake ssh/tests fails.)
Signed-off-by: Stefan Stanacar <stefanx.stanacar@intel.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
| -rw-r--r-- | meta/lib/oeqa/runtime/ping.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/meta/lib/oeqa/runtime/ping.py b/meta/lib/oeqa/runtime/ping.py index e163b969f0..0d028f9b22 100644 --- a/meta/lib/oeqa/runtime/ping.py +++ b/meta/lib/oeqa/runtime/ping.py @@ -8,10 +8,13 @@ class PingTest(oeRuntimeTest): def test_ping(self): output = '' - status = None + count = 0 endtime = time.time() + 60 - while status != 0 and time.time() < endtime: + while count < 5 and time.time() < endtime: proc = subprocess.Popen("ping -c 1 %s" % oeRuntimeTest.tc.qemu.ip, shell=True, stdout=subprocess.PIPE) output += proc.communicate()[0] - status = proc.poll() - self.assertEqual(status, 0, msg = "No echo reply, ping output is:\n%s" % output) + if proc.poll() == 0: + count += 1 + else: + count = 0 + self.assertEqual(count, 5, msg = "Expected 5 consecutive replies, got %d.\nping output is:\n%s" % (count,output)) |