[libvirt] [libvirt-test-API][PATCH 0/2] Add a new case for storagePoolLookupByVolume API
This API will be tested in dir/logicl/netfs pool type. jiahu (2): Add pool_lookup_by_volume.py to conf Add new case to test storagePoolLookupByVolume cases/storage_dir.conf | 6 +++++ cases/storage_logical.conf | 6 +++++ cases/storage_netfs.conf | 6 +++++ repos/storage/pool_lookup_by_volume.py | 46 ++++++++++++++++++++++++++++++++++ 4 files changed, 64 insertions(+) create mode 100644 repos/storage/pool_lookup_by_volume.py -- 1.8.1.4
--- cases/storage_dir.conf | 6 ++++++ cases/storage_logical.conf | 6 ++++++ cases/storage_netfs.conf | 6 ++++++ 3 files changed, 18 insertions(+) diff --git a/cases/storage_dir.conf b/cases/storage_dir.conf index 38b349d..97e8676 100644 --- a/cases/storage_dir.conf +++ b/cases/storage_dir.conf @@ -20,6 +20,12 @@ storage:create_dir_volume capacity $defaultvolumesize +storage:pool_lookup_by_volume + poolname + $defaultpoolname + volname + $defaultvolumename + storage:vol_clone poolname $defaultpoolname diff --git a/cases/storage_logical.conf b/cases/storage_logical.conf index d374dfa..c4c8c79 100644 --- a/cases/storage_logical.conf +++ b/cases/storage_logical.conf @@ -22,6 +22,12 @@ storage:create_logical_volume capacity $defaultvolumesize +storage:pool_lookup_by_volume + poolname + $defaultpoolname + volname + $defaultvolumename + storage:vol_clone poolname $defaultpoolname diff --git a/cases/storage_netfs.conf b/cases/storage_netfs.conf index f486ff4..4fa9c88 100644 --- a/cases/storage_netfs.conf +++ b/cases/storage_netfs.conf @@ -24,6 +24,12 @@ storage:create_netfs_volume capacity $defaultvolumesize +storage:pool_lookup_by_volume + poolname + $defaultpoolname + volname + $defaultvolumename + storage:vol_clone poolname $defaultpoolname -- 1.8.1.4
Added a new case to validate storagePoolLookupByVolume api --- repos/storage/pool_lookup_by_volume.py | 46 ++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) create mode 100644 repos/storage/pool_lookup_by_volume.py diff --git a/repos/storage/pool_lookup_by_volume.py b/repos/storage/pool_lookup_by_volume.py new file mode 100644 index 0000000..ee5c9c9 --- /dev/null +++ b/repos/storage/pool_lookup_by_volume.py @@ -0,0 +1,46 @@ +#!/usr/bin/env python +#test storagePoolLookupByVolume() API for libvirt + +import os +import libvirt +from libvirt import libvirtError +from src import sharedmod + +required_params = ('poolname','volname',) +optional_params = {} + +def pool_lookup_by_volume(params): + """ + test API for storagePoolLookupByVolume in class virStoragePool + """ + logger = params['logger'] + poolname = params['poolname'] + volname = params['volname'] + logger.info("The given pool name is %s" % (poolname)) + logger.info("The given vol name is %s" % (volname)) + conn = sharedmod.libvirtobj['conn'] + pool = conn.storagePoolLookupByName(poolname) + pre_vol = pool.storageVolLookupByName(volname) + volpath = pre_vol.path() + logger.info("The given volume path is %s" % (volpath)) + temp = volpath.split("/") + temp.pop(0) + temp.pop(-1) + temp1 = "/" + "/".join(temp) + if not os.path.exists(temp1): + logger.warning("volume path file %s is not exist" % temp1) + + try: + vol = conn.storageVolLookupByPath(volpath) + pool_name = vol.storagePoolLookupByVolume().name() + logger.info("The pool name is %s from API" % (pool_name)) + + if not pool_name == poolname: + return 1 + + except libvirtError, e: + logger.error("API error message: %s, error code is %s" \ + % (e.message, e.get_error_code())) + return 1 + + return 0 -- 1.8.1.4
On 07/02/2015 05:08 PM, jiahu wrote:
Added a new case to validate storagePoolLookupByVolume api --- repos/storage/pool_lookup_by_volume.py | 46 ++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) create mode 100644 repos/storage/pool_lookup_by_volume.py
diff --git a/repos/storage/pool_lookup_by_volume.py b/repos/storage/pool_lookup_by_volume.py new file mode 100644 index 0000000..ee5c9c9 --- /dev/null +++ b/repos/storage/pool_lookup_by_volume.py @@ -0,0 +1,46 @@ +#!/usr/bin/env python +#test storagePoolLookupByVolume() API for libvirt + +import os +import libvirt +from libvirt import libvirtError +from src import sharedmod + +required_params = ('poolname','volname',) +optional_params = {} + +def pool_lookup_by_volume(params): + """ + test API for storagePoolLookupByVolume in class virStoragePool + """ + logger = params['logger'] + poolname = params['poolname'] + volname = params['volname'] + logger.info("The given pool name is %s" % (poolname)) + logger.info("The given vol name is %s" % (volname)) + conn = sharedmod.libvirtobj['conn'] + pool = conn.storagePoolLookupByName(poolname) + pre_vol = pool.storageVolLookupByName(volname) + volpath = pre_vol.path() + logger.info("The given volume path is %s" % (volpath)) + temp = volpath.split("/") + temp.pop(0) + temp.pop(-1) + temp1 = "/" + "/".join(temp) + if not os.path.exists(temp1): + logger.warning("volume path file %s is not exist" % temp1) + + try: + vol = conn.storageVolLookupByPath(volpath) + pool_name = vol.storagePoolLookupByVolume().name() + logger.info("The pool name is %s from API" % (pool_name)) + + if not pool_name == poolname: + return 1 + + except libvirtError, e: + logger.error("API error message: %s, error code is %s" \ + % (e.message, e.get_error_code())) + return 1 + + return 0 ACK and Pushed
Thanks Hongming
participants (2)
-
hongming -
jiahu