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 Rezanina <[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

Reply via email to