kvm_utils.py: remote_login(): improve regular expression matching
1. Make the 'login:' regular expression stricter so it doesn't match
'Last login: ...' messages.
2. Make the 'password:' regular expression stricter.
3. Handle 'Connection refused' messages.
Signed-off-by: Michael Goldish <mgoldish@redhat.com>
git-svn-id: http://test.kernel.org/svn/autotest/trunk@3249 592f7852-d20e-0410-864c-8624ca9c26a4
diff --git a/client/tests/kvm/kvm_utils.py b/client/tests/kvm/kvm_utils.py
index 27bc7c1..90de88f 100644
--- a/client/tests/kvm/kvm_utils.py
+++ b/client/tests/kvm/kvm_utils.py
@@ -644,8 +644,8 @@
while True:
(match, text) = sub.read_until_last_line_matches(
- ["[Aa]re you sure", "[Pp]assword:", "[Ll]ogin:",
- "[Cc]onnection.*closed", prompt],
+ [r"[Aa]re you sure", r"[Pp]assword:\s*$", r"^\s*[Ll]ogin:\s*$",
+ r"[Cc]onnection.*closed", r"[Cc]onnection.*refused", prompt],
timeout=timeout, internal_timeout=0.5)
if match == 0: # "Are you sure you want to continue connecting"
logging.debug("Got 'Are you sure...'; sending 'yes'")
@@ -669,11 +669,15 @@
logging.debug("Got 'Connection closed'")
sub.close()
return None
- elif match == 4: # prompt
+ elif match == 4: # "Connection refused"
+ kvm_log.debug("Got 'Connection refused'")
+ sub.close()
+ return None
+ elif match == 5: # prompt
logging.debug("Got shell prompt -- logged in")
return sub
else: # match == None
- logging.debug("Timeout or process terminated")
+ logging.debug("Timeout elapsed or process terminated")
sub.close()
return None
@@ -707,7 +711,7 @@
while True:
(match, text) = sub.read_until_last_line_matches(
- ["[Aa]re you sure", "[Pp]assword:", "lost connection"],
+ [r"[Aa]re you sure", r"[Pp]assword:\s*$", r"lost connection"],
timeout=_timeout, internal_timeout=0.5)
if match == 0: # "Are you sure you want to continue connecting"
logging.debug("Got 'Are you sure...'; sending 'yes'")