[LNT] Fixed incorrect orders behavior in case of miss formatted llvm_project_revision

LNT expected that llvm_project_revision is a version in the format 1.2.3. But it is a string and may contain any symbols. Someone may use hex numbers, dashes, etc. If llvm_project_revision does not contain any digit at all the behavior is undefined and caused inconsistent DB. hash(str) is a workaround to fix this behavior. Note it does not fix the sorting.

Reviewed By: cmatthews

Differential Revision: https://reviews.llvm.org/D109577

Author: Pavel Kosov

lnt/server/ui/util.py

@@ -1,4 +1,5 @@
 import colorsys
+import hashlib
 import math
 import re
 from lnt.server.reporting.analysis import REGRESSED
@@ -262,25 +263,26 @@ def convert_revision(dotted, cache=None):
     that is ordered and sortable.
     "1" -> (1)
     "1.2.3" -> (1,2,3)
+    "1a2,3-45:b6;" -> (1,2,3,45,6)
+    "abc" -> (hash("abc"))
 
     :param dotted: the string revision to convert
     :param cache: a dict to use as a cache or None for no cache.
         because this is called many times, it is a nice performance
         increase to cache these conversions.
     :return: a tuple with the numeric bits of this revision as ints.
-
+        return a hash in case of miss formatted version to avoid wrong equals.
     """
     if cache is not None:
         val = cache.get(dotted)
         if val:
             return val
-        else:
-            dotted_parsed = integral_rex.findall(dotted)
-            val = tuple([int(d) for d in dotted_parsed])
-            cache[dotted] = val
-            return val
+
     dotted_parsed = integral_rex.findall(dotted)
-    val = tuple([int(d) for d in dotted_parsed])
+    val = tuple([int(d) for d in dotted_parsed] or 
+                [int(hashlib.sha1(dotted.encode("utf-8")).hexdigest(), 16)])
+    if cache is not None:
+        cache[dotted] = val
     return val