On 03/25/2012 01:42 AM, Martin Kletzander wrote:
Function usage() was called in the code but was missing.
Parameter 'flags' was defined as optional but it was still required in
the code, so I fixed the handling of it.
---
repos/domain/start.py | 15 +++++++++++----
1 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/repos/domain/start.py b/repos/domain/start.py
index cd028bf..38bad7f 100644
--- a/repos/domain/start.py
+++ b/repos/domain/start.py
@@ -1,12 +1,13 @@
#!/usr/bin/env python
"""for testing the start function of domain
mandatory arguments: guestname
+ optional arguments: flags
"""
__author__ = "Osier Yang<jy...@redhat.com>"
__date__ = "Tue Oct 27, 2009"
__version__ = "0.1.0"
-__credits__ = "Copyright (C) 2009 Red Hat, Inc."
+__credits__ = "Copyright (C) 2009, 2012 Red Hat, Inc."
__all__ = ['start', 'check_params', 'parse_opts',
'usage', 'version', 'append_path']
@@ -34,6 +35,11 @@ from exception import LibvirtAPI
NONE = 0
START_PAUSED = 1
+def usage():
+ print '''usage: mandatory arguments: guestname
+ optional arguments: flags
+ '''
+
def return_close(conn, logger, ret):
conn.close()
logger.info("closed hypervisor connection")
@@ -48,7 +54,7 @@ def check_params(params):
logger = params['logger']
- keys = ['guestname', 'flags', 'logger']
+ keys = ['guestname', 'logger']
for key in keys:
if key not in params:
logger.error("key '%s' is required" % key)
@@ -66,7 +72,7 @@ def start(params):
logger -- an object of utils/Python/log.py
mandatory arguments : guestname -- same as the domain name
- optional arguments : flags -- domain create
flags<none|start_paused|noping>
+ optional arguments : flags -- domain create
flags<none|start_paused|noping>
Return 0 on SUCCESS or 1 on FAILURE
"""
@@ -75,7 +81,7 @@ def start(params):
check_params(params)
domname = params['guestname']
logger = params['logger']
- flags = params['flags']
+ flags = params.get('flags', [])
the flags should be string object.
flags = params.get('flags', "") instead.
ACK and pushed with this fixed.
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
