From ada550098ac874cdd3ca7c6637421edf6c7f7163 Mon Sep 17 00:00:00 2001
From: Jinwoo 'Joseph' Suh <jsuh@isi.edu>
Date: Mon, 20 Aug 2012 07:43:21 -0400
Subject: Debugged extra_specs_ops.py

The current extra_specs_ops.py under nova/scheduler/filters has a bug:
it crahses if <or> is not followed by a word. For example, if
"<or> gpu <or>" is given in extra_specs, current code tries to pop a
word after the second <or> that causes a crash. This patch lets it
ignore the "<or>" keyword if no word follows the second "<or>" keyword.

Debugged bug 1038979

Change-Id: I80a6eafd7ee57a2f3ccbf2fb869655ab84e92cf2
---
 nova/tests/scheduler/test_host_filters.py | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

(limited to 'nova/tests')

diff --git a/nova/tests/scheduler/test_host_filters.py b/nova/tests/scheduler/test_host_filters.py
index d6f083576..9cda0ccde 100644
--- a/nova/tests/scheduler/test_host_filters.py
+++ b/nova/tests/scheduler/test_host_filters.py
@@ -670,24 +670,48 @@ class HostFiltersTestCase(test.TestCase):
             especs={'opt1': '<in> 12311321'},
             passes=True)
 
+    def test_compute_filter_passes_extra_specs_with_op_in3(self):
+        self._do_test_compute_filter_extra_specs(
+            ecaps={'opt1': '12311321'},
+            especs={'opt1': '<in> 12311321 <in>'},
+            passes=True)
+
     def test_compute_filter_fails_extra_specs_with_op_in(self):
         self._do_test_compute_filter_extra_specs(
             ecaps={'opt1': '12310321'},
             especs={'opt1': '<in> 11'},
             passes=False)
 
+    def test_compute_filter_fails_extra_specs_with_op_in2(self):
+        self._do_test_compute_filter_extra_specs(
+            ecaps={'opt1': '12310321'},
+            especs={'opt1': '<in> 11 <in>'},
+            passes=False)
+
     def test_compute_filter_passes_extra_specs_with_op_or(self):
         self._do_test_compute_filter_extra_specs(
             ecaps={'opt1': '12'},
             especs={'opt1': '<or> 11 <or> 12'},
             passes=True)
 
+    def test_compute_filter_passes_extra_specs_with_op_or2(self):
+        self._do_test_compute_filter_extra_specs(
+            ecaps={'opt1': '12'},
+            especs={'opt1': '<or> 11 <or> 12 <or>'},
+            passes=True)
+
     def test_compute_filter_fails_extra_specs_with_op_or(self):
         self._do_test_compute_filter_extra_specs(
             ecaps={'opt1': '13'},
             especs={'opt1': '<or> 11 <or> 12'},
             passes=False)
 
+    def test_compute_filter_fails_extra_specs_with_op_or2(self):
+        self._do_test_compute_filter_extra_specs(
+            ecaps={'opt1': '13'},
+            especs={'opt1': '<or> 11 <or> 12 <or>'},
+            passes=False)
+
     def test_compute_filter_passes_extra_specs_with_op_le(self):
         self._do_test_compute_filter_extra_specs(
             ecaps={'opt1': 2, 'opt2': 2},
-- 
cgit