xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Ian Jackson <ian.jackson@eu.citrix.com>
To: xen-devel@lists.xenproject.org
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Ian Campbell <ian.campbell@citrix.com>
Subject: [OSSTEST PATCH 10/13] rump kernel tests: Specify kernel via runvar
Date: Tue, 15 Jul 2014 16:22:24 +0100	[thread overview]
Message-ID: <1405437747-12753-11-git-send-email-ian.jackson@eu.citrix.com> (raw)
In-Reply-To: <1405437747-12753-1-git-send-email-ian.jackson@eu.citrix.com>

The runvar <gn>_builtimage is set to <distpart>:<distpath>.
Previously we hardcoded `/usr/local/lib/xen/rump-kernel' in
`rumpuserxen'.

The ffs filesystem images, and the demo config template, still come
from a single particular build, which we find from the
`guests_rumpuserxenbuildjob' runvar.  This runvar is then doing double
duty as the build job for the ffs filesystem images and example
configs, and the `default' build job for guest kernels.

The subpath /usr/local/lib/xen/rump-kernel is duplicated, because it
appears once (in make-flight, for setting a runvar) to find the demo
guest's kernel, and once (hardcoded in ts-rumpuserxen-demo-setup) for
the filesystem images and config file.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 make-flight               |    1 +
 ts-rumpuserxen-demo-setup |   17 +++++++++++++----
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/make-flight b/make-flight
index 8aa5396..cede756 100755
--- a/make-flight
+++ b/make-flight
@@ -123,6 +123,7 @@ do_rumpkernel_tests () {
                       test-rumpuserxen xl \
             $xenarch $dom0arch                                       \
             guests_rumpuserxenbuildjob=build-$rumparch-rumpuserxen   \
+            rump_builtimage=rumpuserxen:/usr/local/lib/xen/rump-kernel \
             all_hostflags=$most_hostflags
 }
 
diff --git a/ts-rumpuserxen-demo-setup b/ts-rumpuserxen-demo-setup
index 68ec7d4..34da7cb 100755
--- a/ts-rumpuserxen-demo-setup
+++ b/ts-rumpuserxen-demo-setup
@@ -35,10 +35,19 @@ our $rkdist;
 sub prep () {
     $gho = prepareguest($ho, $gn, $gn, 4096, undef, 30);
 
+    my $cfg_rkdist = target_extract_jobdistpath_subdir
+	($ho, 'rumpuserxen--base', 'rumpuserxen',
+	 $r{guests_rumpuserxenbuildjob});
+    $cfg_rkdist .= '/usr/local/lib/xen/rump-kernel';
+
     my $buildjob = guest_var($gho, 'rumpuserxenbuildjob', undef);
+    my $builtimage = guest_var($gho, 'builtimage', undef);
+    $builtimage =~ m/\:/ or die "$builtimage ?";
+    my $builtimage_part = $`;
+    my $builtimage_subpath = $'; #';
     $rkdist = target_extract_jobdistpath_subdir
-	($ho, 'rumpuserxen', 'rumpuserxen', $buildjob);
-    $rkdist .= '/usr/local/lib/xen/rump-kernel';
+	($ho, "rumpuserxen-$gn", $builtimage_part, $buildjob);
+    my $imagefile = $rkdist.$builtimage_subpath;
 
     my @images;
 
@@ -46,7 +55,7 @@ sub prep () {
 
     my $cfgfile = "$gn.cfg";
     my $cfgpath = "/etc/xen/$cfgfile";
-    target_editfile_root($ho, "$rkdist/domain_config", $cfgfile, $cfgpath,
+    target_editfile_root($ho, "$cfg_rkdist/domain_config", $cfgfile, $cfgpath,
 			 sub {
 	while (<EI>) {
 	    next if m/^\s*\#/;
@@ -55,7 +64,7 @@ sub prep () {
 	    my $in = $& if m/^\w\S*/;
 
 	    s# = .*              # " = '$gn'"          #xe  if $in eq 'name';
-	    s# =\s* ['"](.*)['"] # " = '$rkdist/$1'"   #xe  if $in eq 'kernel';
+	    s# = .*              # " = '$imagefile'"   #xe  if $in eq 'kernel';
 	    s#\b mac=[0-9a-f:]+  # "mac=$gho->{Ether}" #xe  if $in eq 'vif';
 	    s#\b 4             \b# 3                   #xe  if $in eq 'extra';
 
-- 
1.7.10.4

  parent reply	other threads:[~2014-07-15 15:22 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-15 15:22 [OSSTEST PATCH 00/13] Build and test rump kernel xenstore-ls Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 01/13] selectguest: log a message Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 02/13] BuildSupport: make selectbuildhost modify @ARGV Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 03/13] sg-run-job: Support toggling `adding' by repeating `+' Ian Jackson
2014-07-16  9:17   ` Ian Campbell
2014-07-15 15:22 ` [OSSTEST PATCH 04/13] building: Delete when cloning, not when preparing $builddir Ian Jackson
2014-07-16  9:19   ` Ian Campbell
2014-07-16  9:58     ` Ian Jackson
2014-07-16 10:06       ` Ian Campbell
2014-07-16 10:18         ` Ian Jackson
2014-07-16 10:20           ` Ian Campbell
2014-07-16 10:29             ` Ian Jackson
2014-07-16 11:25               ` Ian Campbell
2014-07-15 15:22 ` [OSSTEST PATCH 05/13] make-flight: rumpuserxen tests: use same buildjob for all guests Ian Jackson
2014-07-16  9:19   ` Ian Campbell
2014-07-16 10:10     ` Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 06/13] ts-xen-build: Allow specification of make targets Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 07/13] ts-xen-build: Honour cmdprefix_configure, cmdprefix_make Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 08/13] rump kernel tests: Do a xen.git tools build Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 09/13] rump kernel tests: Use guest-specific fs images location Ian Jackson
2014-07-15 15:22 ` Ian Jackson [this message]
2014-07-15 15:22 ` [OSSTEST PATCH 11/13] rump kernel tests: Specify cmdline via runvar Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 12/13] ts-guest-destroy-hard: New script Ian Jackson
2014-07-16  9:24   ` Ian Campbell
2014-07-16 10:10     ` Ian Jackson
2014-07-16 10:12       ` Ian Campbell
2014-07-16 10:20         ` Ian Jackson
2014-07-16 10:21           ` Ian Campbell
2014-07-16 10:30             ` Ian Jackson
2014-07-15 15:22 ` [OSSTEST PATCH 13/13] rump kernel tests: Run xenstore-ls demo Ian Jackson
2014-07-16  9:26   ` Ian Campbell
2014-07-16 10:15     ` Ian Jackson
2014-07-16 10:17       ` Ian Campbell
2014-07-16  9:26 ` [OSSTEST PATCH 00/13] Build and test rump kernel xenstore-ls Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1405437747-12753-11-git-send-email-ian.jackson@eu.citrix.com \
    --to=ian.jackson@eu.citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).