[Xen-devel] [PATCH OSSTEST v2 4/5] cambridge: Do not try to push harness to XenProject instance output

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

By arranging for cr-publish-flight-logs to ignore --push-harness if
either of HarnessPublishGitRepoDir or HarnessPublishGitUserHost are
not specified

Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
v2:
 - Avoid logm which isn't available here, wasn't saying much of use
   anyway.
 - Syntax fix (is not a function, so exit not return)

Perhaps we should have our own tree for such things, but for now just
nobble it.
---
 cr-publish-flight-logs      | 2 ++
 production-config-cambridge | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/cr-publish-flight-logs b/cr-publish-flight-logs
index 1feb52d..2e96f60 100755
--- a/cr-publish-flight-logs
+++ b/cr-publish-flight-logs
@@ -43,6 +43,8 @@ if ($push_harness) {
     my $githost= $c{HarnessPublishGitUserHost};
     my $gitdir= $c{HarnessPublishGitRepoDir};
 
+    exit 0 if (!$githost || !$gitdir);
+
     system_checked("git push $githost:$gitdir HEAD:refs/heads/flight-$flight");
     system_checked("ssh $githost 'cd $gitdir && git update-server-info'");
 }
diff --git a/production-config-cambridge b/production-config-cambridge
index 1404d10..e44f3b0 100644
--- a/production-config-cambridge
+++ b/production-config-cambridge
@@ -50,9 +50,6 @@ GlobalLockDir /export/home/osstest/testing.git
 LogsPublish= "$c{Publish}/logs"
 ResultsPublish= "$c{Publish}/results"
 
-HarnessPublishGitUserHost xen@xxxxxxxxxxxxxxxxxxxxx
-HarnessPublishGitRepoDir git/osstest.git
-
 ReportTrailer= <<END
 Logs, config files, etc. are available at
     $c{ReportHtmlPubBaseUrl}
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel