[Kimchi-devel] [PATCH 4/5] [WOK] Make dynamic the state_dir path in Kimchi.
pvital at linux.vnet.ibm.com
pvital at linux.vnet.ibm.com
Tue Sep 29 13:08:16 UTC 2015
From: Paulo Vital <pvital at linux.vnet.ibm.com>
Removed the hard-coded state_dir path (/var/lib/kimchi) and make it dynamic to
get based on PluginPaths() value. By default, this value is set to
/var/lib/wok/plugins/kimchi.
Changed the state_dir path (/var/lib/kimchi) to the new path using Wok
structure (/var/lib/wok/plugins/kimchi) in documentation and tests.
Signed-off-by: Paulo Vital <pvital at linux.vnet.ibm.com>
---
src/wok/plugins/kimchi/docs/README.md | 2 +-
src/wok/plugins/kimchi/model/libvirtstoragepool.py | 4 +++-
src/wok/plugins/kimchi/model/storagepools.py | 2 +-
src/wok/plugins/kimchi/tests/test_mock_storagepool.py | 2 +-
src/wok/plugins/kimchi/tests/test_mock_storagevolume.py | 2 +-
src/wok/plugins/kimchi/tests/test_model.py | 2 +-
src/wok/plugins/kimchi/tests/test_model_storagepool.py | 2 +-
src/wok/plugins/kimchi/tests/test_storagepoolxml.py | 2 +-
src/wok/plugins/kimchi/tests/test_template.py | 2 +-
9 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/src/wok/plugins/kimchi/docs/README.md b/src/wok/plugins/kimchi/docs/README.md
index f400333..b52892a 100644
--- a/src/wok/plugins/kimchi/docs/README.md
+++ b/src/wok/plugins/kimchi/docs/README.md
@@ -223,7 +223,7 @@ new template using the "+" button in the upper right corner.
To create a template, you need an ISO on your host or using remote one.
If you are willing to use your own ISO, please copy it to out of box storage
-pool (default path is: /var/lib/kimchi/isos).
+pool (default path is: /var/lib/wok/plugins/kimchi/isos).
Known Issues
------------
diff --git a/src/wok/plugins/kimchi/model/libvirtstoragepool.py b/src/wok/plugins/kimchi/model/libvirtstoragepool.py
index 5da0b3b..108c381 100644
--- a/src/wok/plugins/kimchi/model/libvirtstoragepool.py
+++ b/src/wok/plugins/kimchi/model/libvirtstoragepool.py
@@ -23,6 +23,7 @@ import os
import tempfile
from lxml.builder import E
+from wok.config import PluginPaths
from wok.exception import InvalidParameter, OperationFailed, TimeoutExpired
from wok.rollbackcontext import RollbackContext
from wok.utils import parse_cmd_output, run_command, wok_log
@@ -77,7 +78,8 @@ class NetfsPoolDef(StoragePoolDef):
def __init__(self, poolArgs):
super(NetfsPoolDef, self).__init__(poolArgs)
- self.path = '/var/lib/kimchi/nfs_mount/' + self.poolArgs['name']
+ self.path = PluginPaths('kimchi').state_dir + '/nfs_mount/'
+ self.path = self.path + self.poolArgs['name']
def prepare(self, conn):
mnt_point = tempfile.mkdtemp(dir='/tmp')
diff --git a/src/wok/plugins/kimchi/model/storagepools.py b/src/wok/plugins/kimchi/model/storagepools.py
index db68252..1764fe9 100644
--- a/src/wok/plugins/kimchi/model/storagepools.py
+++ b/src/wok/plugins/kimchi/model/storagepools.py
@@ -79,7 +79,7 @@ class StoragePoolsModel(object):
pools[default_pool] = {'path': '/var/lib/libvirt/images'}
if config.get("server", "create_iso_pool") == "true":
- pools['ISO'] = {'path': '/var/lib/kimchi/isos'}
+ pools['ISO'] = {'path': PluginPaths('kimchi').state_dir + '/isos'}
error_msg = ("Please, check the configuration in %s/template.conf to "
"ensure it has a valid storage pool." %
diff --git a/src/wok/plugins/kimchi/tests/test_mock_storagepool.py b/src/wok/plugins/kimchi/tests/test_mock_storagepool.py
index 5cf5b3e..07c3e07 100644
--- a/src/wok/plugins/kimchi/tests/test_mock_storagepool.py
+++ b/src/wok/plugins/kimchi/tests/test_mock_storagepool.py
@@ -82,7 +82,7 @@ class MockStoragepoolTests(unittest.TestCase):
'path': '/tmp/kimchi-images'},
{'type': 'netfs', 'name': u'kīмсhīUnitTestNSFPool',
'source': {'host': 'localhost',
- 'path': '/var/lib/kimchi/nfs-pool'}},
+ 'path': '/var/lib/wok/plugins/kimchi/nfs-pool'}},
{'type': 'scsi', 'name': u'kīмсhīUnitTestSCSIFCPool',
'source': {'adapter_name': fc_devs[0]}},
{'type': 'iscsi', 'name': u'kīмсhīUnitTestISCSIPool',
diff --git a/src/wok/plugins/kimchi/tests/test_mock_storagevolume.py b/src/wok/plugins/kimchi/tests/test_mock_storagevolume.py
index 9d0a5ad..643fe1b 100644
--- a/src/wok/plugins/kimchi/tests/test_mock_storagevolume.py
+++ b/src/wok/plugins/kimchi/tests/test_mock_storagevolume.py
@@ -77,7 +77,7 @@ class MockStorageVolumeTests(unittest.TestCase):
'path': '/tmp/kimchi-images'},
{'type': 'netfs', 'name': u'kīмсhīUnitTestNSFPool',
'source': {'host': 'localhost',
- 'path': '/var/lib/kimchi/nfs-pool'}},
+ 'path': '/var/lib/wok/plugins/kimchi/nfs-pool'}},
{'type': 'scsi', 'name': u'kīмсhīUnitTestSCSIFCPool',
'source': {'adapter_name': fc_devs[0]}},
{'type': 'iscsi', 'name': u'kīмсhīUnitTestISCSIPool',
diff --git a/src/wok/plugins/kimchi/tests/test_model.py b/src/wok/plugins/kimchi/tests/test_model.py
index 7c89048..4b8ea69 100644
--- a/src/wok/plugins/kimchi/tests/test_model.py
+++ b/src/wok/plugins/kimchi/tests/test_model.py
@@ -49,7 +49,7 @@ invalid_repository_urls = ['www.fedora.org', # missing protocol
'http://www.fedora', # invalid domain name
'file:///home/foobar'] # invalid path
-TMP_DIR = '/var/lib/kimchi/tests/'
+TMP_DIR = '/var/lib/wok/plugins/kimchi/tests/'
UBUNTU_ISO = TMP_DIR + 'ubuntu14.04.iso'
diff --git a/src/wok/plugins/kimchi/tests/test_model_storagepool.py b/src/wok/plugins/kimchi/tests/test_model_storagepool.py
index 5f9b966..7c5d6eb 100644
--- a/src/wok/plugins/kimchi/tests/test_model_storagepool.py
+++ b/src/wok/plugins/kimchi/tests/test_model_storagepool.py
@@ -107,7 +107,7 @@ class StoragepoolTests(unittest.TestCase):
self.assertEquals(len(storagepools) + 3, len(pools))
# Create a pool with an existing path
- tmp_path = tempfile.mkdtemp(dir='/var/lib/kimchi')
+ tmp_path = tempfile.mkdtemp(dir='/var/lib/wok/plugins/kimchi')
rollback.prependDefer(os.rmdir, tmp_path)
req = json.dumps({'name': 'existing_path', 'type': 'dir',
'path': tmp_path})
diff --git a/src/wok/plugins/kimchi/tests/test_storagepoolxml.py b/src/wok/plugins/kimchi/tests/test_storagepoolxml.py
index 7e45cca..86c3198 100644
--- a/src/wok/plugins/kimchi/tests/test_storagepoolxml.py
+++ b/src/wok/plugins/kimchi/tests/test_storagepoolxml.py
@@ -53,7 +53,7 @@ class StoragepoolXMLTests(unittest.TestCase):
<dir path='/var/export'/>
</source>
<target>
- <path>/var/lib/kimchi/nfs_mount/unitTestNFSPool</path>
+ <path>/var/lib/wok/plugins/kimchi/nfs_mount/unitTestNFSPool</path>
</target>
</pool>
"""},
diff --git a/src/wok/plugins/kimchi/tests/test_template.py b/src/wok/plugins/kimchi/tests/test_template.py
index c7de182..ca95c91 100644
--- a/src/wok/plugins/kimchi/tests/test_template.py
+++ b/src/wok/plugins/kimchi/tests/test_template.py
@@ -311,7 +311,7 @@ class TemplateTests(unittest.TestCase):
'path': '/tmp/kimchi-images'},
{'type': 'netfs', 'name': u'kīмсhīUnitTestNSFPool',
'source': {'host': 'localhost',
- 'path': '/var/lib/kimchi/nfs-pool'}},
+ 'path': '/var/lib/wok/plugins/kimchi/nfs-pool'}},
{'type': 'scsi', 'name': u'kīмсhīUnitTestSCSIFCPool',
'source': {'adapter_name': fc_devs[0]}},
{'type': 'iscsi', 'name': u'kīмсhīUnitTestISCSIPool',
--
2.4.3
More information about the Kimchi-devel
mailing list