On Thu, 2015-04-23 at 07:27 +0000, Pang, LongtaoX wrote:
>
>
> > -----Original Message-----
> > From: Ian Campbell [mailto:[email protected]]
> > Sent: Tuesday, April 21, 2015 6:40 PM
> > To: Pang, LongtaoX
> > Cc: [email protected]; [email protected];
> > [email protected]; Hu,
> > Robert
> > Subject: Re: [OSSTEST Nested PATCH v8 5/7] Add new script to customize
> > nested
> > test configuration
> > > Signed-off-by: longtao.pang <[email protected]>
> > > ---
> > > Changes in v8:
> > > 1. Replace '$nested_host' by '$l1->{Guest}'.
> > > ---
> > > ts-nested-setup | 52
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++
> > > 1 file changed, 52 insertions(+)
> > > create mode 100755 ts-nested-setup
> > >
> > > diff --git a/ts-nested-setup b/ts-nested-setup
> > > new file mode 100755
> > > index 0000000..41d5e80
> > > --- /dev/null
> > > +++ b/ts-nested-setup
> > > @@ -0,0 +1,52 @@
> > > +#!/usr/bin/perl -w
> > > +
> > > +use strict qw(vars);
> > > +use DBI;
> > > +use Osstest;
> > > +use Osstest::Debian;
> > > +use Osstest::TestSupport;
> > > +
> > > +tsreadconfig();
> > > +our ($l0,$l1) = ts_get_host_guest(@ARGV);
> > > +
> > > +guest_check_ip($l1);
> > > +target_cmd_root($l1, "mkdir -p /home/osstest/.ssh && cp
> > /root/.ssh/authorized_keys /home/osstest/.ssh/");
> > > +my $url =
> > $c{WebspaceUrl}.$c{WebspaceCommon}.$l0->{Name}."_".'overlay.tar';
> > > +target_cmd_root($l1, <<END);
> > > + wget -O overlay.tar $url
> > > + tar -xf overlay.tar -C /
> > > + rm overlay.tar -f
> > > + update-rc.d osstest-confirm-booted start 99 2 .
> > > +END
> >
> > I cc'd you on some patches which I think should help avoid this
> > duplication.
> >
> For this question, I have merged the v5_patches[04,05,06] which you CC'd to
> me.
> Based on your patches, after finishing installing L1 hvm guest VM with
> 'ts-debian-hvm-install' script, I could ssh into L1 guest as 'osstest' user
> without password, that means I will not need to use below code anymore
> target_cmd_root($l1, "mkdir -p /home/osstest/.ssh && cp
> /root/.ssh/authorized_keys /home/osstest/.ssh/");
Great.
>
> But, inside L1 guest VM, the overly files(osstest-confirm-booted,
> xenbridge, xenlightdaemons ) does not exist at " /etc/init.d"
> directory. Since 'ts-host-reboot' script will use
> 'osstest-confirm-booted' shell script to confirm whether L1 guest boot
> up normally, these overlay files are necessary here.
> If I add below patch based on your patches, and install L1 hvm guest
> VM again, all the overly files exist in "/etc/init.d" directory inside
> L1 guest.
That sounds ok to me, the overlay contains some host level stuff but it
is either stuff which is useful at guest level too
(osstest-configmr-booted) or harmless (most of the rest).
If we think that is a problem then splitting into overlay-host,
overlay-guest and overlay-common would be one way to address that.
Although I would have expect the preseed_hook_overlay to be removed from
elsewhere too?
> diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
> index 6691ff6..4af6957 100644
> --- a/Osstest/Debian.pm
> +++ b/Osstest/Debian.pm
> @@ -624,6 +624,7 @@ sub preseed_base ($$$$;@) {
>
> preseed_ssh($ho, $sfx);
> preseed_hook_overlay($ho, $sfx, $c{OverlayLocal}, 'overlay-local.tar');
> + preseed_hook_overlay($ho, $sfx, 'overlay', 'overlay.tar');
>
> my $preseed = <<"END";
> d-i mirror/suite string $suite
>
> Another question, based on your patches, I find that the below
> commands under ' d-i preseed/late_command string \\' do not work
> anymore in preseed () ' of 'ts-debian-hvm-install' script. For
> example, after finishing installing L1 guest, there is no directory of
> '/boot/efi/EFI/boot' created and 'sources.list' does not be edited by
> sed inside L1 guest. I think you have verified this, maybe something
> wrong of my test environment to cause the question?
I'm not sure. I suspect the issue is that multiple preseed/late_commands
are not supported and the osstest preseed hook commands have used it
already.
IOW the below should probably be switch to use preseed_hook_command($ho,
'late_command'. Do you think this is a bug in my patches or yours?
Smells like mine on first glance.
> d-i preseed/late_command string \\
> in-target mkdir -p /boot/efi/EFI/boot; \\
> in-target cp /boot/efi/EFI/debian/grubx64.efi
> /boot/efi/EFI/boot/bootx64.efi ;\\
> in-target sed -i 's/^deb *cdrom/#&/g' /etc/apt/sources.list;
> END
_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xen.org/xen-devel
