Oh, OK, looks good, applied to next. On Thu, Jun 14, 2012 at 4:01 AM, Miroslav Rezanina <[email protected]> wrote: > Hi, > I miss this patch when push handling Fedora 17 boot options (repo and ks). > > With Xen PV guest we use autotest http server as repository for installation > packages because xen (RHEL5) does not support cdrom emulation. This change > has to be properly reflected in 'repo' boot option. We have to use > http address instead of cdrom path. > > Signed-off-by: Miroslav RezaOnina <[email protected]> > --- > client/virt/tests/unattended_install.py | 14 +++++++++++--- > 1 files changed, 11 insertions(+), 3 deletions(-) > > diff --git a/client/virt/tests/unattended_install.py > b/client/virt/tests/unattended_install.py > index b20a9dc..8ba6543 100644 > --- a/client/virt/tests/unattended_install.py > +++ b/client/virt/tests/unattended_install.py > @@ -718,9 +718,17 @@ class UnattendedInstallConfig(object): > # install cdrom in /dev/sr1. When we get ks via http, > # we need to change repo configuration to /dev/sr0 > if 'repo=cdrom' in kernel_params: > - kernel_params = re.sub('repo\=cdrom[\:\w\d\/]*', > - 'repo=cdrom:/dev/sr0', > - kernel_params) > + if ((self.vm.driver_type == 'xen') and > + (self.params.get('hvm_or_pv') == 'pv')): > + kernel_params = re.sub('repo\=[\:\w\d\/]*', > + 'repo=http://%s:%s' % > + (self.url_auto_content_ip, > + > self.url_auto_content_port), > + kernel_params) > + else: > + kernel_params = re.sub('repo\=cdrom[\:\w\d\/]*', > + 'repo=cdrom:/dev/sr0', > + kernel_params) > > self.params['kernel_params'] = kernel_params > elif self.params.get('unattended_delivery_method') == 'cdrom': > -- > Miroslav Rezanina > _______________________________________________ > Autotest mailing list > [email protected] > http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
-- Lucas _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
