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

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

 



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@xxxxxxxxxxx>"
# 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
_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux