Blame SOURCES/bz1860544-fence_lpar-fix-long-user-host-issue.patch

0abed3
From 3424464d3e447308f171399302cf76eb573a618f Mon Sep 17 00:00:00 2001
0abed3
From: Reid wahl <nrwahl@protonmail.com>
0abed3
Date: Fri, 24 Jul 2020 18:22:24 -0700
0abed3
Subject: [PATCH] fence_lpar: Fix parse error from long command line
0abed3
0abed3
When Pacemaker executes `fence_lpar` and the HMC command line is greater
0abed3
than 80 characters, a parse error causes agent failure. This can happen
0abed3
with a long user name and/or long managed system name. It happens only
0abed3
when Pacemaker spawns the `fence_lpar` process; it does not happen when
0abed3
`fence_lpar` is run from the CLI.
0abed3
0abed3
A long command line gets a carriage return ('\r') added at the 80
0abed3
character mark and wraps back to the beginning of the line with no line
0abed3
feed ('\n'), overwriting the displayed characters. `fence_lpar`'s regex
0abed3
matches handle this fine when it's run from the command line.
0abed3
0abed3
The problem is that when Pacemaker spawns fence_lpar, **for some
0abed3
reason** there are backspace characters in the buffer when we hit the
0abed3
'\r' character. This seems to overwrite some of the `conn.before`
0abed3
string. As a result, the regex doesn't match `conn.before`, and the
0abed3
agent fails.
0abed3
0abed3
This patch works around the `conn.before` weirdness by reading and
0abed3
discarding the first received line **before** any regex processing.
0abed3
0abed3
Resolves: RHBZ#1860544
0abed3
Resolves: RHBZ#1860545
0abed3
0abed3
Signed-off-by: Reid Wahl <nrwahl@protonmail.com>
0abed3
---
0abed3
 agents/lpar/fence_lpar.py | 33 +++++++++++++++++++++++++++------
0abed3
 1 file changed, 27 insertions(+), 6 deletions(-)
0abed3
0abed3
diff --git a/agents/lpar/fence_lpar.py b/agents/lpar/fence_lpar.py
0abed3
index 270bbe3b..9dfabc43 100644
0abed3
--- a/agents/lpar/fence_lpar.py
0abed3
+++ b/agents/lpar/fence_lpar.py
0abed3
@@ -19,6 +19,9 @@
0abed3
 def get_power_status(conn, options):
0abed3
 	if options["--hmc-version"] == "3":
0abed3
 		conn.send("lssyscfg -r lpar -m " + options["--managed"] + " -n " + options["--plug"] + " -F name,state\n")
0abed3
+
0abed3
+		# First line (command) may cause parsing issues if long
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
 		try:
0abed3
@@ -29,6 +32,9 @@ def get_power_status(conn, options):
0abed3
 	elif options["--hmc-version"] in ["4", "IVM"]:
0abed3
 		conn.send("lssyscfg -r lpar -m "+ options["--managed"] +
0abed3
 				" --filter 'lpar_names=" + options["--plug"] + "'\n")
0abed3
+
0abed3
+		# First line (command) may cause parsing issues if long
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
 		try:
0abed3
@@ -49,6 +55,9 @@ def set_power_status(conn, options):
0abed3
 	if options["--hmc-version"] == "3":
0abed3
 		conn.send("chsysstate -o " + options["--action"] + " -r lpar -m " + options["--managed"]
0abed3
 			+ " -n " + options["--plug"] + "\n")
0abed3
+
0abed3
+		# First line (command) may cause parsing issues if long
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 	elif options["--hmc-version"] in ["4", "IVM"]:
0abed3
 		if options["--action"] == "on":
0abed3
@@ -60,17 +69,23 @@ def set_power_status(conn, options):
0abed3
 		else:
0abed3
 			conn.send("chsysstate -o shutdown -r lpar --immed" +
0abed3
 				" -m " + options["--managed"] + " -n " + options["--plug"] + "\n")
0abed3
+
0abed3
+		# First line (command) may cause parsing issues if long
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
 def get_lpar_list(conn, options):
0abed3
 	outlets = {}
0abed3
 	if options["--hmc-version"] == "3":
0abed3
 		conn.send("query_partition_names -m " + options["--managed"] + "\n")
0abed3
+
0abed3
+		## We have to remove first line (command)
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
-		## We have to remove first 3 lines (command + header) and last line (part of new prompt)
0abed3
+		## We have to remove next 2 lines (header) and last line (part of new prompt)
0abed3
 		####
0abed3
-		res = re.search("^.+?\n(.+?\n){2}(.*)\n.*$", conn.before, re.S)
0abed3
+		res = re.search("^(.+?\n){2}(.*)\n.*$", conn.before, re.S)
0abed3
 
0abed3
 		if res == None:
0abed3
 			fail_usage("Unable to parse output of list command")
0abed3
@@ -81,11 +96,14 @@ def get_lpar_list(conn, options):
0abed3
 	elif options["--hmc-version"] == "4":
0abed3
 		conn.send("lssyscfg -r lpar -m " + options["--managed"] +
0abed3
 			" -F name:state\n")
0abed3
+
0abed3
+		## We have to remove first line (command)
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
-		## We have to remove first line (command) and last line (part of new prompt)
0abed3
+		## We have to remove last line (part of new prompt)
0abed3
 		####
0abed3
-		res = re.search("^.+?\n(.*)\n.*$", conn.before, re.S)
0abed3
+		res = re.search("^(.*)\n.*$", conn.before, re.S)
0abed3
 
0abed3
 		if res == None:
0abed3
 			fail_usage("Unable to parse output of list command")
0abed3
@@ -100,11 +118,14 @@ def get_lpar_list(conn, options):
0abed3
 	elif options["--hmc-version"] == "IVM":
0abed3
 		conn.send("lssyscfg -r lpar -m " + options["--managed"] +
0abed3
 			" -F name,state\n")
0abed3
+
0abed3
+		## We have to remove first line (command)
0abed3
+		conn.readline()
0abed3
 		conn.log_expect(options["--command-prompt"], int(options["--power-timeout"]))
0abed3
 
0abed3
-		## We have to remove first line (command) and last line (part of new prompt)
0abed3
+		## We have to remove last line (part of new prompt)
0abed3
 		####
0abed3
-		res = re.search("^.+?\n(.*)\n.*$", conn.before, re.S)
0abed3
+		res = re.search("^(.*)\n.*$", conn.before, re.S)
0abed3
 
0abed3
 		if res == None:
0abed3
 			fail_usage("Unable to parse output of list command")