[PATCH] Enhancements to the koan installation in the virt module

[Matthew Hicks <mhicks@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>]

- Adding support for virt-name and virt-path for the install command
- Removing --virt-graphics from the koan options as it's no longer supported
-- Koan enables graphical installations by default and the option isn't
  supported in the current release
---
 func/minion/modules/virt.py |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/func/minion/modules/virt.py b/func/minion/modules/virt.py
index 0828484..92107fe 100644
--- a/func/minion/modules/virt.py
+++ b/func/minion/modules/virt.py
@@ -184,7 +184,7 @@ class Virt(func_module.FuncModule):
                 pass
         return results
 
-    def install(self, server_name, target_name, system=False):
+    def install(self, server_name, target_name, system=False, virt_name=None, virt_path=None):
 
         """
         Install a new virt system by way of a named cobbler profile.
@@ -192,6 +192,7 @@ class Virt(func_module.FuncModule):
 
         # Example:
         # install("bootserver.example.org", "fc7webserver", True)
+        # install("bootserver.example.org", "client.example.org", True, "client-disk0", "HostVolGroup00")
 
         conn = self.__get_conn()
 
@@ -204,15 +205,19 @@ class Virt(func_module.FuncModule):
         if system:
             target = "system"
 
-        # TODO: FUTURE: set --virt-path in cobbler or here
         koan_args = [
             "/usr/bin/koan",
             "--virt",
-            "--virt-graphics",  # enable VNC
             "--%s=%s" % (target, target_name),
             "--server=%s" % server_name
         ]
 
+        if virt_name:
+            koan_args.append("--virt-name=%s" % virt_name)
+
+        if virt_path:
+            koan_args.append("--virt-path=%s" % virt_path)
+
         rc = sub_process.call(koan_args,shell=False)
         if rc == 0:
             return 0
-- 
1.5.3.6


--------------050105050603070203070104
Content-Type: text/plain;
 name="0002-Adding-freemem-functionality.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="0002-Adding-freemem-functionality.patch"

_______________________________________________
Func-list mailing list
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list