Change Autotest status parser from fail-hard to fail-safe.

tko/parsers/version_0.py

 import re, os
 
 from autotest_lib.client.common_lib import utils as common_utils
 from autotest_lib.tko import utils as tko_utils, models, status_lib
 from autotest_lib.tko.parsers import base
 
 
 class NoHostnameError(Exception):
     pass
 
@@ skipping 235 matching lines @@
     def is_status_line(line):
         return re.search(r"^\t*(\S[^\t]*\t){3}", line) is not None
 
 
     @classmethod
     def parse_line(cls, line):
         if not status_line.is_status_line(line):
             return None
         indent, line = re.search(r"^(\t*)(.*)$", line, flags=re.DOTALL).groups()
         indent = len(indent)
-        line = line.strip()
 
         # split the line into the fixed and optional fields
-        parts = line.split("\t")
-        status, subdir, testname = parts[0:3]
-        reason = parts[-1]
-        optional_parts = parts[3:-1]
+        parts = line.rstrip("\n").split("\t")
 
-        # all the optional parts should be of the form "key=value"
-        assert sum('=' not in part for part in optional_parts) == 0
-        optional_fields = dict(part.split("=", 1)
-                               for part in optional_parts)
+        part_index = 3
+        status, subdir, testname = parts[0:part_index]
+
+        # all optional parts should be of the form "key=value". once we've found
+        # a non-matching part, treat it and the rest of the parts as the reason.
+        optional_fields = {}
+        while part_index < len(parts):
+          kv = re.search(r"^(\w+)=(.+)", parts[part_index])
+          if not kv:
+            break
+
+          optional_fields[kv.group(1)] = kv.group(2)
+          part_index += 1
+
+        reason = "\t".join(parts[part_index:])
 
         # build up a new status_line and return it
         return cls(indent, status, subdir, testname, reason,
                    optional_fields)
 
 
 class parser(base.parser):
     @staticmethod
     def make_job(dir):
         return job(dir)
@@ skipping 160 matching lines @@
             testname = "boot.%d" % boot_count
             reason = "machine did not return from reboot"
             tko_utils.dprint(("Adding: ABORT\nSubdir:----\n"
                               "Testname:%s\n%s")
                              % (testname, reason))
             new_test = test.parse_test(self.job, None, testname,
                                        "ABORT", reason,
                                        current_kernel, None, None)
             new_tests.append(new_test)
         yield new_tests