All of lore.kernel.org
 help / color / mirror / Atom feed
* [OSSTEST PATCH 1/2] ts-host-install: Fix indentation error
@ 2015-05-15 17:14 Ian Jackson
  2015-05-15 17:14 ` [OSSTEST PATCH 2/2] ts-host-install: New --poweron-test-only option Ian Jackson
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Jackson @ 2015-05-15 17:14 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Ian Campbell

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 ts-host-install |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ts-host-install b/ts-host-install
index b13f293..dfee932 100755
--- a/ts-host-install
+++ b/ts-host-install
@@ -34,7 +34,7 @@ while (@ARGV and $ARGV[0] =~ m/^-/) {
     $_= shift @ARGV;
     last if m/^--$/;
     if (m/^--priority(?:=(.*))?$/) {
-    $xopts{DebconfPriority}= defined($1) ? $1 : 'low';
+        $xopts{DebconfPriority}= defined($1) ? $1 : 'low';
     } elsif  (m/^--rescue$/) {
 	$xopts{RescueMode}= 1;
     }
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [OSSTEST PATCH 2/2] ts-host-install: New --poweron-test-only option
  2015-05-15 17:14 [OSSTEST PATCH 1/2] ts-host-install: Fix indentation error Ian Jackson
@ 2015-05-15 17:14 ` Ian Jackson
  2015-05-18  9:06   ` Ian Campbell
  0 siblings, 1 reply; 3+ messages in thread
From: Ian Jackson @ 2015-05-15 17:14 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Ian Campbell

We are having a difficulty with one of the test boxes, which can be
most easily reproduced by running ts-host-install to power cycle the
box and then see if it wakes up enough to fetch a preseed file.

Keep this mode of operation in tree in case it's useful in future.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 ts-host-install |   10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/ts-host-install b/ts-host-install
index dfee932..69de4b6 100755
--- a/ts-host-install
+++ b/ts-host-install
@@ -27,6 +27,7 @@ use Osstest::Logtailer;
 tsreadconfig();
 
 my $debconf_priority;
+my $poweron_test_only;
 
 our %xopts;
 
@@ -35,6 +36,8 @@ while (@ARGV and $ARGV[0] =~ m/^-/) {
     last if m/^--$/;
     if (m/^--priority(?:=(.*))?$/) {
         $xopts{DebconfPriority}= defined($1) ? $1 : 'low';
+    } elsif  (m/^--poweron-test-only$/) {
+	$poweron_test_only= 1;
     } elsif  (m/^--rescue$/) {
 	$xopts{RescueMode}= 1;
     }
@@ -71,6 +74,13 @@ END
     await_webspace_fetch_byleaf(get_timeout($ho,'reboot',$timeout{ReadPreseed})
                                 , 1,
                                 $logtailer, $ho, $ps_url);
+
+    if ($poweron_test_only) {
+	logm("Hooray, power on worked.");
+	power_state($ho, 0);
+	exit 0;
+    }
+
     setup_pxeboot_local($ho);
 
     await_tcp(get_timeout($ho,'reboot',$timeout{Sshd}), 14,$ho);
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [OSSTEST PATCH 2/2] ts-host-install: New --poweron-test-only option
  2015-05-15 17:14 ` [OSSTEST PATCH 2/2] ts-host-install: New --poweron-test-only option Ian Jackson
@ 2015-05-18  9:06   ` Ian Campbell
  0 siblings, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2015-05-18  9:06 UTC (permalink / raw)
  To: Ian Jackson; +Cc: xen-devel

On Fri, 2015-05-15 at 18:14 +0100, Ian Jackson wrote:
> We are having a difficulty with one of the test boxes, which can be
> most easily reproduced by running ts-host-install to power cycle the
> box and then see if it wakes up enough to fetch a preseed file.
> 
> Keep this mode of operation in tree in case it's useful in future.
> 
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

Both patches: Acked-by: Ian Campbell <ian.campbell@citrix.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-05-18 12:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-15 17:14 [OSSTEST PATCH 1/2] ts-host-install: Fix indentation error Ian Jackson
2015-05-15 17:14 ` [OSSTEST PATCH 2/2] ts-host-install: New --poweron-test-only option Ian Jackson
2015-05-18  9:06   ` Ian Campbell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.