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 18/26] ts-debian-hvm-install, etc.: Do not hardcode in-iso path
Date: Fri, 18 Sep 2015 18:50:08 +0100 [thread overview]
Message-ID: <1442598616-2884-5-git-send-email-ian.jackson@eu.citrix.com> (raw)
In-Reply-To: <1442598616-2884-1-git-send-email-ian.jackson@eu.citrix.com>
ts-debian-hvm-install hardcoded `install.amd' as the directory in the
.iso in which to find the kernel and initrd. This is wrong for
architectures other than amd64.
Instead, pass this information in runvars (as is done for the netinst
installs in make-distros-flight), and honour it in
ts-debian-hvm-install.
If the runvars are not set, default to the previous hardcoded values.
(This arranges that clones of old flights still work with new osstest,
eg for bisection.)
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
make-flight | 8 ++++++++
ts-debian-hvm-install | 25 +++++++++++++++++++------
2 files changed, 27 insertions(+), 6 deletions(-)
diff --git a/make-flight b/make-flight
index c1f40b9..accf922 100755
--- a/make-flight
+++ b/make-flight
@@ -264,6 +264,12 @@ do_hvm_debian_test_one () {
local arch=$(branch_debianhvm_arch)
+ case "$arch" in
+ amd64) iso_dir='install.amd' ;;
+ i386) iso_dir='install.386' ;;
+ *) iso_dir="install.$arch" ;;
+ esac
+
stubdom_suffix=""
stubdom_runvar=""
if [ x$stubdom = xtrue ]; then
@@ -276,6 +282,8 @@ do_hvm_debian_test_one () {
enable_xsm=$xsm \
$stubdom_runvar \
debianhvm_image=debian-7.2.0-$arch-CD-1.iso \
+ debianhvm_iso_kernel=/$iso_dir/vmlinuz \
+ debianhvm_iso_ramdisk=/$iso_dir/initrd.gz \
bios=$bios \
all_hostflags=$most_hostflags,hvm
}
diff --git a/ts-debian-hvm-install b/ts-debian-hvm-install
index 4b1eca4..3b93ebd 100755
--- a/ts-debian-hvm-install
+++ b/ts-debian-hvm-install
@@ -48,6 +48,8 @@ our $disk_mb= 10000;
our $guesthost= "$gn.guest.osstest";
our $gho;
+our ($kernel, $ramdisk);
+
our $gsuite= guest_var($gho,'suite',$c{GuestDebianSuite});
sub preseed () {
@@ -130,14 +132,14 @@ set default="0"
set timeout=5
menuentry 'debian guest auto Install' {
- linux /install.amd/vmlinuz $cmdline
- initrd /install.amd/initrd.gz
+ linux $kernel $cmdline
+ initrd $ramdisk
}
END
}
sub isolinux_cfg () {
- my $cmdline = gcmdline("initrd=/install.amd/initrd.gz");
+ my $cmdline = gcmdline("initrd=$ramdisk");
return <<"END";
default autoinstall
@@ -145,7 +147,7 @@ sub isolinux_cfg () {
timeout 0
label autoinstall
- kernel /install.amd/vmlinuz
+ kernel $kernel
append $cmdline
END
}
@@ -156,9 +158,9 @@ sub prepare_initrd ($$$) {
rm -rf $initrddir
mkdir $initrddir
cd $initrddir
- gzip -d < $newiso/install.amd/initrd.gz | cpio --extract --make-directories --no-absolute-filename
+ gzip -d < $newiso$ramdisk | cpio --extract --make-directories --no-absolute-filename
cp $preseed_file_path preseed.cfg
- find . | cpio -H newc --create | gzip -9 > $newiso/install.amd/initrd.gz
+ find . | cpio -H newc --create | gzip -9 > $newiso$ramdisk
cd -
rm -rf $initrddir
cd $newiso
@@ -169,6 +171,13 @@ END
our $emptyiso= "/root/$flight.$job.$gn-empty.iso";
+sub iso_path ($$) {
+ my ($which, $deftail) = @_;
+ my $v = guest_var($gho, "iso_$which", "/install.amd/$deftail");
+ die "$gho->{Guest} $which $v ?" unless $v =~ m#^/#;
+ return $v;
+}
+
sub prep () {
target_install_packages_norec($ho, qw(lvm2 rsync genisoimage));
@@ -177,6 +186,10 @@ sub prep () {
$gho= prepareguest($ho, $gn, $guesthost, 22,
$disk_mb + 1,
200);
+
+ $kernel = iso_path('kernel', 'vmlinuz');
+ $ramdisk = iso_path('ramdisk', 'initrd.gz');
+
my $base = "/root/$flight.$job.$gn-";
my $newiso= $base . "newiso";
my $emptydir= $base . "empty-dir";
--
1.7.10.4
next prev parent reply other threads:[~2015-09-18 17:50 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-18 17:50 [OSSTEST PATCH v3 00/15-26] xen.git#staging smoke tests Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 15/26] cr*: Support xen-unstable-smoke Ian Jackson
2015-09-21 9:17 ` Ian Campbell
2015-09-21 10:21 ` Ian Jackson
2015-09-21 10:32 ` Ian Campbell
2015-09-21 10:53 ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 16/26] ap-*: Be able to fetch and push xen.git#smoke Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 17/26] crontab: Enable xen-unstable-smoke branch every 3h at <1mod3>:51 UTC Ian Jackson
2015-09-18 17:50 ` Ian Jackson [this message]
2015-09-21 9:19 ` [OSSTEST PATCH 18/26] ts-debian-hvm-install, etc.: Do not hardcode in-iso path Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 19/26] ts-debian-hvm-install: Cope with images containing only isolinux Ian Jackson
2015-09-21 9:28 ` Ian Campbell
2015-09-21 10:35 ` Ian Jackson
2015-09-21 14:30 ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 20/26] ts-debian-hvm-install: Set $gsuite after $gho Ian Jackson
2015-09-21 9:29 ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 21/26] ts-debian-hvm-install: Do not create EFI partition if EFI not in use Ian Jackson
2015-09-21 9:32 ` Ian Campbell
2015-09-21 10:36 ` Ian Jackson
2015-09-21 10:41 ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 22/26] ts-debian-hvm-install: Use /dev/sda for i386, not /dev/xvda Ian Jackson
2015-09-21 9:40 ` Ian Campbell
2015-09-21 10:38 ` Ian Jackson
2015-09-18 17:50 ` [OSSTEST PATCH 23/26] Timeouts: Introduce target_adjust_timeout Ian Jackson
2015-09-21 9:41 ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 24/26] Timeouts: Honour guest-related timeout-adjustment runvars Ian Jackson
2015-09-21 9:43 ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 25/26] Debian i386 HVM tests: Increase installation timeout Ian Jackson
2015-09-21 9:43 ` Ian Campbell
2015-09-18 17:50 ` [OSSTEST PATCH 26/26] cri-common: Add a missing semicolon Ian Jackson
2015-09-21 9:44 ` 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=1442598616-2884-5-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).