From: Ian Jackson <ian.jackson@eu.citrix.com>
Date: Fri, 21 Aug 2020 16:40:07 +0000 (+0100)
Subject: shared/reuse: Rely on @ for ts-host-ping-check
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=304c4e315bd45de98698b5d29b7eebebd6c64f63;p=osstest.git

shared/reuse: Rely on @ for ts-host-ping-check

Remove the check for SharedReady.

The existence of this check is perplexing.  It was introduced in
  ts-host-ping-check: Do not run if host is being reused
in 8f1dc3f7c401 (from 2015).

At that time we only share build hosts, and build hosts never ran this
script.  So I don't understand what that was hoping to achieve.  Maybe
it made some difference in a now-lost pre-rebase situation.

Anyway, in our current tree I think we want to rerun the
ts-host-ping-check when we reuse a test host.  My change to add @ to
parts of per-host-prep in sg-run-job deliberately omitted the step
with testid host-ping-check-xen/@.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---

diff --git a/ts-host-ping-check b/ts-host-ping-check
index a670680..512aaec 100755
--- a/ts-host-ping-check
+++ b/ts-host-ping-check
@@ -27,8 +27,6 @@ our ($whhost) = @ARGV;
 $whhost ||= 'host';
 our $ho= selecthost($whhost);
 
-exit 0 if $ho->{SharedReady};
-
 $_ = `ping -D -i 0.2 -c 100 $ho->{Ip} | tee -a /dev/stderr`;
 
 m/\b([0-9.]+)% packet loss\b/ or die "$_ ?";