[libvirt] [PATCH 4/5] domain: remove the dup case

Wanlong Gao gaowanlong at cn.fujitsu.com
Wed Jun 13 06:53:12 UTC 2012


remove the domblkinfo which duped with domain_blkinfo.

Signed-off-by: Wanlong Gao <gaowanlong at cn.fujitsu.com>
---
 repos/domain/domblkinfo.py | 116 ---------------------------------------------
 1 file changed, 116 deletions(-)
 delete mode 100644 repos/domain/domblkinfo.py

diff --git a/repos/domain/domblkinfo.py b/repos/domain/domblkinfo.py
deleted file mode 100644
index 031ce2f..0000000
--- a/repos/domain/domblkinfo.py
+++ /dev/null
@@ -1,116 +0,0 @@
-#!/usr/bin/env python
-# To test "virsh domblkinfo" command
-
-import os
-import sys
-import re
-import commands
-
-import libvirt
-from libvirt import libvirtError
-
-from src import sharedmod
-
-required_params = ('guestname', 'blockdev',)
-optional_params = {}
-
-GET_DOMBLKINFO_MAC = "virsh domblkinfo %s %s | awk '{print $2}'"
-GET_CAPACITY = "du -b %s | awk '{print $1}'"
-GET_PHYSICAL_K = " du -B K %s | awk '{print $1}'"
-VIRSH_DOMBLKINFO = "virsh domblkinfo %s %s"
-
-def get_output(command, logger):
-    """execute shell command
-    """
-    status, ret = commands.getstatusoutput(command)
-    if status:
-        logger.error("executing "+ "\"" +  command  + "\"" + " failed")
-        logger.error(ret)
-    return status, ret
-
-def check_domain_exists(conn, guestname, logger):
-    """ check if the domain exists, may or may not be active """
-    guest_names = []
-    ids = conn.listDomainsID()
-    for id in ids:
-        obj = conn.lookupByID(id)
-        guest_names.append(obj.name())
-
-    guest_names += conn.listDefinedDomains()
-
-    if guestname not in guest_names:
-        logger.error("%s doesn't exist" % guestname)
-        return False
-    else:
-        return True
-
-def check_block_data(blockdev, blkdata, logger):
-    """ check data about capacity,allocation,physical """
-    status, apparent_size = get_output(GET_CAPACITY % blockdev, logger)
-    if not status:
-        if apparent_size == blkdata[0]:
-            logger.info("the capacity of '%s' is %s, checking succeeded" % \
-                        (blockdev, apparent_size))
-        else:
-            logger.error("apparent-size from 'du' is %s, \n\
-                         but from 'domblkinfo' is %s, checking failed" % \
-                        (apparent_size, blkdata[0]))
-            return 1
-    else:
-        return 1
-
-    status, block_size_k = get_output(GET_PHYSICAL_K % blockdev, logger)
-    if not status:
-        block_size_b = int(block_size_k[:-1]) * 1024
-        # Temporarily, we only test the default case, assuming
-        # Allocation value is equal to Physical value
-        if str(block_size_b) == blkdata[1] and str(block_size_b) == blkdata[2]:
-            logger.info("the block size of '%s' is %s, same with \n\
-                        Allocation and Physical value, checking succeeded" % \
-                         (blockdev, block_size_b))
-        else:
-            logger.error("the block size from 'du' is %s, \n\
-                          the Allocation value is %s, Physical value is %s, \n\
-                          checking failed" % (block_size_b, blkdata[1], blkdata[2]))
-            return 1
-
-    return 0
-
-
-def domblkinfo(params):
-    """ using du command to check the data
-        in the output of virsh domblkinfo
-    """
-    logger = params['logger']
-    guestname = params.get('guestname')
-    blockdev = params.get('blockdev')
-
-    logger.info("the name of guest is %s" % guestname)
-    logger.info("the block device is %s" % blockdev)
-
-    conn = sharedmod.libvirtobj['conn']
-
-    if not check_domain_exists(conn, guestname, logger):
-        logger.error("need a defined guest")
-        return 1
-
-    logger.info("the output of virsh domblkinfo is:")
-    status, output = get_output(VIRSH_DOMBLKINFO % (guestname, blockdev), logger)
-    if not status:
-        logger.info("\n" + output)
-    else:
-        return 1
-
-    status, data_str = get_output(GET_DOMBLKINFO_MAC % (guestname, blockdev), logger)
-    if not status:
-        blkdata = data_str.rstrip().split('\n')
-        logger.info("capacity,allocation,physical list: %s" % blkdata)
-    else:
-        return 1
-
-    if check_block_data(blockdev, blkdata, logger):
-        logger.error("checking domblkinfo data FAILED")
-        return 1
-    else:
-        logger.info("checking domblkinfo data SUCCEEDED")
-    return 0
-- 
1.7.11.rc0




More information about the libvir-list mailing list