All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: ian.jackson@eu.citrix.com
Cc: wei.liu2@citrix.com, Ian Campbell <ian.campbell@citrix.com>,
	xen-devel@lists.xen.org
Subject: [OSSTEST v6 08/24] ts-debian-hvm-install: Only apply EFI workaround for Wheezy
Date: Wed, 6 May 2015 10:56:37 +0100	[thread overview]
Message-ID: <1430906213-30108-8-git-send-email-ian.campbell@citrix.com> (raw)
In-Reply-To: <1430905605.2660.187.camel@citrix.com>

The previous refactoring of preseed hooks makes this easy to do.

The underlying issue is lack of persistent variable store in our OVMF
setup, which we workaround by placing a copy of grub at the removable
media path. Add a comment saying this since I initially thought this
was just a Wheezy bug.

In Jessie the extra copy of grub can be achieved by preseeding
grub-installer/force-efi-extra-removable (since various real h/w has
similar limitations/bugs) however I haven't tested that so I didn't
add it to the preseed yet, I just mention it in the code comment.

Currently this script hardcodes Wheezy, refactor to use the 'suite'
guest_var (or $c{GuestDebianSuite})

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Cc: wei.liu2@citrix.com
---
v6: New patch
---
 ts-debian-hvm-install | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/ts-debian-hvm-install b/ts-debian-hvm-install
index b37f0f9..f3629e1 100755
--- a/ts-debian-hvm-install
+++ b/ts-debian-hvm-install
@@ -41,9 +41,11 @@ our $disk_mb= 10000;
 our $guesthost= "$gn.guest.osstest";
 our $gho;
 
+our $gsuite= guest_var($gho,'suite',$c{GuestDebianSuite});
+
 sub preseed () {
 
-    my $preseed_file = preseed_base($gho,'wheezy','','',());
+    my $preseed_file = preseed_base($gho,$gsuite,'','',());
 
     $preseed_file .= (<<END);
 d-i netcfg/get_hostname string $gn
@@ -71,7 +73,15 @@ d-i partman-auto/expert_recipe string \\
 d-i apt-setup/cdrom/set-first boolean false
 END
 
-preseed_hook_command($gho, 'late_command', '', <<END);
+
+    # Our OVMF does not have a persistent variable store and therefore
+    # cannot remember the boot entry added by the Debian
+    # installer. Therefore we ensure that a copy of the bootloader is
+    # copied to the removable media path, which is the EFI fallback.
+    #
+    # From Jessie onwards this can be worked around by using the fix to
+    # #767037 i.e. preseeding grub-installer/force-efi-extra-removable.
+preseed_hook_command($gho, 'late_command', '', <<END) if $gsuite =~ /wheezy/;
 #!/bin/sh
 set -ex
 
-- 
2.1.4

  parent reply	other threads:[~2015-05-06  9:56 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-06  9:46 [PATCH OSSTEST v5 00/24] add distro domU testing flight Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 01/24] TestSupport: Add helper to fetch a URL on a host Ian Campbell
2015-05-12 15:08   ` Ian Jackson
2015-05-12 15:17     ` Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 02/24] TestSupport: allow caller of prepareguest_part_xencfg to specify viftype Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 03/24] create_webfile: Support use with guests as well as hosts Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 04/24] Debian: refactor code to add preseed commands to the preseed file Ian Campbell
2015-05-12 15:11   ` Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 05/24] Debian: refactor preseeding of .ssh directories Ian Campbell
2015-05-12 15:13   ` Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 06/24] Debian: Refactor installation of overlays, so it can be used for guests too Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 07/24] Debian: add preseed_create_guest helper Ian Campbell
2015-05-12 15:16   ` Ian Jackson
2015-05-12 15:22     ` Ian Campbell
2015-05-06  9:56 ` Ian Campbell [this message]
2015-05-12 15:17   ` [OSSTEST v6 08/24] ts-debian-hvm-install: Only apply EFI workaround for Wheezy Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 09/24] make-flight: Handle $BUILD_LVEXTEND_MAX in mfi-common:create_build_jobs() Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 10/24] distros: add support for installing Debian PV guests via d-i, flight and jobs Ian Campbell
2015-05-12 15:25   ` Ian Jackson
2015-05-12 15:40     ` Ian Campbell
2015-05-12 16:08       ` Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 11/24] distros: support booting Debian PV (d-i installed) guests with pvgrub Ian Campbell
2015-05-12 15:32   ` Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 12/24] distros: Support pvgrub for Wheezy too Ian Campbell
2015-05-12 15:35   ` Ian Jackson
2015-05-12 15:49     ` Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 13/24] distros: support PV guest install from Debian netinst media Ian Campbell
2015-05-12 15:52   ` Ian Jackson
2015-05-12 16:14     ` Ian Jackson
2015-05-13  9:16       ` Ian Campbell
2015-05-13  9:06     ` Ian Campbell
2015-05-14 18:01       ` Ian Jackson
2015-05-15  8:58         ` Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 14/24] Test pygrub and pvgrub on the regular flights Ian Campbell
2015-05-12 15:54   ` Ian Jackson
2015-05-13  9:11     ` Ian Campbell
2015-05-14 17:58       ` Ian Jackson
2015-05-06  9:56 ` [OSSTEST v6 15/24] distros: add branch infrastructure Ian Campbell
2015-05-12 16:03   ` Ian Jackson
2015-05-13  9:12     ` Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 16/24] distros: Run a flight over the weekend Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 17/24] Debian: Handle lack of bootloader support in d-i on ARM Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 18/24] ts-debian-di-install: Refactor root_disk specification Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 19/24] make-flight: refactor PV debian tests Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 20/24] Add testing of non-LVM/phy disk backends Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 21/24] mfi-common: Allow make-*flight to filter the set of build jobs to include Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 22/24] make-distros-flight: don't bother building for XSM Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 23/24] distros: email only me on play flights Ian Campbell
2015-05-06  9:56 ` [OSSTEST v6 24/24] ts-debian-di-install: Use ftp.debian.org directly Ian Campbell
2015-05-06  9:57 ` [PATCH OSSTEST v5 00/24] add distro domU testing flight Ian Campbell
2015-05-12 16:06 ` Ian Jackson
2015-05-13  9:19   ` Ian Campbell
2015-05-13 16:09     ` Ian Jackson

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=1430906213-30108-8-git-send-email-ian.campbell@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.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 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.