Merge "Fix DriverFilter/GoodnessWeigher string evaluations"

manila/scheduler/evaluator/evaluator.py

@@ -56,9 +56,9 @@ class EvalConstant(object):
         except ValueError:
             try:
                 result = float(result)
-            except ValueError as e:
+            except ValueError:
-                msg = _("ValueError: %s") % e
+                if isinstance(result, str):
-                raise exception.EvaluatorParseException(reason=msg)
+                    result = result.replace('"', '').replace('\'', '')
 
         return result
 
@@ -233,6 +233,7 @@ def _def_parser():
     Combine = pyparsing.Combine
     Forward = pyparsing.Forward
     nums = pyparsing.nums
+    quoted_string = pyparsing.quotedString
     oneOf = pyparsing.oneOf
     opAssoc = pyparsing.opAssoc
     infixNotation = pyparsing.infixNotation
@@ -244,7 +245,7 @@ def _def_parser():
     number = real | integer
     expr = Forward()
     fn = Word(alphas + '_' + '.')
-    operand = number | variable | fn
+    operand = number | variable | fn | quoted_string
 
     signop = oneOf('+ -')
     addop = oneOf('+ -')

manila/tests/scheduler/evaluator/test_evaluator.py

@@ -128,11 +128,12 @@ class EvaluatorTestCase(test.TestCase):
     def test_nonnumber_comparison(self):
         nonnumber = {'test': 'foo'}
         request = {'test': 'bar'}
-        self.assertRaises(
+        self.assertTrue(
-            exception.EvaluatorParseException,
+            evaluator.evaluate("nonnumber.test != request.test",
-            evaluator.evaluate,
+                               nonnumber=nonnumber, request=request))
-            "nonnumber.test != request.test",
+        self.assertFalse(
-            nonnumber=nonnumber, request=request)
+            evaluator.evaluate("nonnumber.test == request.test",
+                               nonnumber=nonnumber, request=request))
 
     def test_div_zero(self):
         self.assertRaises(exception.EvaluatorParseException,

releasenotes/notes/bug-1975715-fix-driverfilter-string-evaluations-3886a68d4d7fa3a1.yaml

@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Goodness_function expects integer or float else raise parseException. This
+    causes example such as "(share.share_proto == 'CIFS') ? 100 : 50" to fail
+    during evaluation. Fix it by adding support of string evalution.