Gitweb: http://git.fedorahosted.org/git/?p=fence-agents.git;a=commitdiff;h=ae6ad01ad... Commit: ae6ad01adfba06b7d573e674ba71363fac48abe2 Parent: 96f9239fd502d6146e89997b720952f489dbc147 Author: Marek 'marx' Grac mgrac@redhat.com AuthorDate: Sun Nov 25 14:18:06 2012 +0100 Committer: Marek 'marx' Grac mgrac@redhat.com CommitterDate: Mon Feb 4 16:05:54 2013 +0100
COMPATIBILITY BREAK: fence_lpar replace --n / --partition / partition with standard -n / --plug / port
--- fence/agents/lib/fencing.py.py | 3 +-- fence/agents/lpar/fence_lpar.py | 26 ++++++++------------------ 2 files changed, 9 insertions(+), 20 deletions(-)
diff --git a/fence/agents/lib/fencing.py.py b/fence/agents/lib/fencing.py.py index 920fe50..e639931 100644 --- a/fence/agents/lib/fencing.py.py +++ b/fence/agents/lib/fencing.py.py @@ -769,8 +769,7 @@ def fence_action(tn, options, set_power_fn, get_power_fn, get_outlet_list = None try: ## Process options that manipulate fencing device ##### - if (options["--action"] == "list") and \ - 0 == options["device_opt"].count("port") and 0 == options["device_opt"].count("partition"): + if (options["--action"] == "list") and 0 == options["device_opt"].count("port"): print "N/A" return elif (options["--action"] == "list" and get_outlet_list == None): diff --git a/fence/agents/lpar/fence_lpar.py b/fence/agents/lpar/fence_lpar.py index e8e50c6..55b8294 100644 --- a/fence/agents/lpar/fence_lpar.py +++ b/fence/agents/lpar/fence_lpar.py @@ -22,15 +22,15 @@ BUILD_DATE=""
def get_power_status(conn, options): if options["--hmc-version"] == "3": - conn.send("lssyscfg -r lpar -m " + options["--managed"] + " -n " + options["--partition"] + " -F name,state\n") + conn.send("lssyscfg -r lpar -m " + options["--managed"] + " -n " + options["--plug"] + " -F name,state\n") conn.log_expect(options, options["--command-prompt"], int(options["--power-timeout"]))
try: - status = re.compile("^" + options["--partition"] + ",(.*?),.*$", re.IGNORECASE | re.MULTILINE).search(conn.before).group(1) + status = re.compile("^" + options["--plug"] + ",(.*?),.*$", re.IGNORECASE | re.MULTILINE).search(conn.before).group(1) except AttributeError: fail(EC_STATUS_HMC) elif options["--hmc-version"] == "4": - conn.send("lssyscfg -r lpar -m "+ options["--managed"] +" --filter 'lpar_names=" + options["--partition"] + "'\n") + conn.send("lssyscfg -r lpar -m "+ options["--managed"] +" --filter 'lpar_names=" + options["--plug"] + "'\n") conn.log_expect(options, options["--command-prompt"], int(options["--power-timeout"]))
try: @@ -50,18 +50,18 @@ def get_power_status(conn, options): def set_power_status(conn, options): if options["--hmc-version"] == "3": conn.send("chsysstate -o " + options["--action"] + " -r lpar -m " + options["--managed"] - + " -n " + options["--partition"] + "\n") + + " -n " + options["--plug"] + "\n") conn.log_expect(options, options["--command-prompt"], int(options["--power-timeout"])) elif options["--hmc-version"] == "4": if options["--action"] == "on": conn.send("chsysstate -o on -r lpar -m " + options["--managed"] + - " -n " + options["--partition"] + + " -n " + options["--plug"] + " -f `lssyscfg -r lpar -F curr_profile " + " -m " + options["--managed"] + - " --filter "lpar_names="+ options["--partition"] +""`\n" ) + " --filter "lpar_names="+ options["--plug"] +""`\n" ) else: conn.send("chsysstate -o shutdown -r lpar --immed" + - " -m " + options["--managed"] + " -n " + options["--partition"] + "\n") + " -m " + options["--managed"] + " -n " + options["--plug"] + "\n") conn.log_expect(options, options["--command-prompt"], int(options["--power-timeout"]))
def get_lpar_list(conn, options): @@ -115,17 +115,10 @@ def define_new_opts(): "shortdesc" : "Force HMC version to use (3 or 4)", "default" : "4", "order" : 1 } - all_opt["partition"] = { - "getopt" : "n:", - "longopt" : "partition", - "help" : "-n <id> Name of the partition", - "required" : "0", - "shortdesc" : "Partition name", - "order" : 1 }
def main(): device_opt = [ "ipaddr", "ipport", "login", "passwd", "secure", "cmd_prompt", \ - "partition", "managed", "hmc_version" ] + "port", "managed", "hmc_version" ]
atexit.register(atexit_handler)
@@ -145,9 +138,6 @@ def main(): if 0 == options.has_key("--managed"): fail_usage("Failed: You have to enter name of managed system")
- if (0 == ["list", "monitor"].count(options["--action"].lower())) and (0 == options.has_key("--partition")): - fail_usage("Failed: You have to enter name of the partition") - if 1 == options.has_key("--hmc-version") and (options["--hmc-version"] != "3" and options["--hmc-version"] != "4"): fail_usage("Failed: You have to enter valid version number: 3 or 4")
cluster-commits@lists.fedorahosted.org