remove the domblkinfo which duped with domain_blkinfo.
Signed-off-by: Wanlong Gao <gaowanlong@xxxxxxxxxxxxxx>
---
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
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list