On 10/17/2011 05:23 AM, Onkar N Mahajan wrote: > On Thu, 2011-10-13 at 12:39 -0300, Lucas Meneghel Rodrigues wrote: >> On 10/13/2011 01:45 AM, Onkar N Mahajan wrote: >>> From e6346e178166a606078bd760acf154d2f486f7d0 Mon Sep 17 00:00:00 2001 >>> From: Onkar N Mahajan<[email protected]> >>> Date: Thu, 13 Oct 2011 09:58:53 +0530 >>> Subject: [PATCH] Add SLES10 guest support to KVM Autotest Signed-off-by: >>> Onkar N Mahajan<[email protected]> >> >> Hi Omar, thanks for the contribution! I have one doubt and a minor >> comment, and I'd like you to adress them before I get this integrated: >> >>> --- >>> client/tests/kvm/guest-os.cfg.sample | 30 + >>> client/tests/kvm/unattended/SLES-10.xml | 892 >>> +++++++++++++++++++++++++++++++ >>> 2 files changed, 922 insertions(+), 0 deletions(-) >>> create mode 100644 client/tests/kvm/unattended/SLES-10.xml >>> >>> diff --git a/client/tests/kvm/guest-os.cfg.sample >>> b/client/tests/kvm/guest-os.cfg.sample >>> index 17d6114..fc9a504 100644 >>> --- a/client/tests/kvm/guest-os.cfg.sample >>> +++ b/client/tests/kvm/guest-os.cfg.sample >>> @@ -842,11 +842,41 @@ variants: >>> # hold your autoyast file >>> #extra_params += " --append 'autoyast=floppy >>> console=ttyS0,115200 console=tty0'" >>> extra_params += " --append >>> 'autoyast=device://scd0/autoinst.xml console=ttyS0,115200 console=tty0'" >>> + # --- uncomment the below line for other SLES 10 >>> distros only --- >>> + #extra_params += " --append >>> 'autoyast=device://hdb/autoinst.xml console=ttyS0,115200 console=tty0'" >> >> ^ Indentation is off. Also, can you please explain how you get the >> autoinst.xml file on this hdb device? As I don't have SLES10 available >> to test your patch, you need to explain to me how you managed to get the >> autoinst.xml on the hard drive in the first place. Our unattended >> install test does not have yet a facility to create unattended install >> auxiliary hard drives. > > IMHO , these are not auxiliary drives ; While the unattended_install was > being executed I connected to the monitor and accessed the console : > > # nc -U /tmp/mymonitor > QEMU 0.12.1 monitor - type 'help' for more information > (qemu) sendkey ctrl-alt-f2 > sendkey ctrl-alt-f2 > (qemu) info block > info block > ide0-hd0: type=hd removable=0 > file=/tmp/kvm_autotest_root/images/sles10-64.qcow2 ro=0 drv=qcow2 > encrypted=0 > ide1-cd0: type=cdrom removable=1 locked=0 > file=/tmp/kvm_autotest_root/isos/linux/SLES-10-DVD-x86_64-DVD1.iso ro=1 > drv=raw encrypted=0 > ide0-cd1: type=cdrom removable=1 locked=0 > file=/tmp/kvm_autotest_root/images/sles-10-64/autoyast.iso ro=1 drv=raw > encrypted=0 > (qemu) quit > > I noticed that SLES-10-DVD-x86_64-DVD1.iso is detected as /dev/hdc in > guest and mounted as shown in the screenshot-1 ; So I tried to > mount /dev/hdb as iso9660 guessing that this is autoyast.iso image > (please see screenshot-2) ; It is detected as /dev/hdb and there is > indeed autoinst.xml file in here. I tested this a couple of times.
Ah OK, makes sense. So for SLES 10 /dev/scd0 does not appear as the CDROM, does it? if for some reason using extra_params += " --append 'autoyast=device://scd0/autoinst.xml console=ttyS0,115200 console=tty0'" does not work, then I am OK with using extra_params += " --append 'autoyast=device://scd0/autoinst.xml console=ttyS0,115200 console=tty0'" Just the way you did on your original patch. > Please correct me if I am wrong. I just didn't understand the original intent in the first place, thanks for explaining. _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
