[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[et-mgmt-tools] [PATCH] virt-manager: Don't use storage pool with qemu:///session



Hi,
qemu:///session can't write to the storage pool (except for root of
course) so defaulting to cwd is probably better. This also makes the
distinction between session and system more obvious. (Debian Bug
#500260)
 -- Guido
# HG changeset patch
# User "Guido Günther <agx sigxcpu org>"
# Date 1223654801 -7200
# Node ID 8fd85d2ed3de8ee449122755e321a985c05e2335
# Parent  7442ab043062ab4270b2de7ec23007866c7d5158
qemu://session can't write to the pool so use current directory

diff -r 7442ab043062 -r 8fd85d2ed3de src/virtManager/config.py
--- a/src/virtManager/config.py	Tue Oct 07 10:09:03 2008 -0400
+++ b/src/virtManager/config.py	Fri Oct 10 18:06:41 2008 +0200
@@ -380,11 +380,10 @@
     def get_default_image_dir(self, connection):
         if connection.get_type() == "Xen":
             return DEFAULT_XEN_IMAGE_DIR
-        #elif os.access(DEFAULT_VIRT_IMAGE_DIR, os.W_OK):
-        #    return DEFAULT_VIRT_IMAGE_DIR
-        #else:
-        #    return os.getcwd()
 
+        if connection.is_qemu_session():
+            return os.getcwd()
+        
         # Just return the default dir since the intention is that it
         # is a managed pool and the user will be able to install to it.
         return DEFAULT_VIRT_IMAGE_DIR

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]