From 44279ce34f08c3c2df242ff399542d20d36950f8 Mon Sep 17 00:00:00 2001
From: Martynas Mickevicius <mmartynas@gmail.com>
Date: Thu, 3 Oct 2013 11:41:34 +0300
Subject: [PATCH] Allow leading ranges in the inventory host entries.

---
 lib/ansible/inventory/expand_hosts.py | 3 +--
 lib/ansible/inventory/ini.py          | 2 +-
 test/TestInventory.py                 | 6 ++++++
 test/inventory/test_leading_range     | 3 +++
 4 files changed, 11 insertions(+), 3 deletions(-)
 create mode 100644 test/inventory/test_leading_range

diff --git a/lib/ansible/inventory/expand_hosts.py b/lib/ansible/inventory/expand_hosts.py
index aab271fe2c..a1db9f1c6a 100644
--- a/lib/ansible/inventory/expand_hosts.py
+++ b/lib/ansible/inventory/expand_hosts.py
@@ -41,8 +41,7 @@ def detect_range(line = None):
 
     Returnes True if the given line contains a pattern, else False.
     '''
-    if (not line.startswith("[") and
-        line.find("[") != -1 and
+    if (line.find("[") != -1 and
         line.find(":") != -1 and
         line.find("]") != -1 and
         line.index("[") < line.index(":") < line.index("]")):
diff --git a/lib/ansible/inventory/ini.py b/lib/ansible/inventory/ini.py
index d983a80b16..eadf5db510 100644
--- a/lib/ansible/inventory/ini.py
+++ b/lib/ansible/inventory/ini.py
@@ -65,7 +65,7 @@ class InventoryParser(object):
         active_group_name = 'ungrouped'
 
         for line in self.lines:
-            if line.startswith("["):
+            if line.startswith("[") and line.strip().endswith("]"):
                 active_group_name = line.split(" #")[0].replace("[","").replace("]","").strip()
                 if line.find(":vars") != -1 or line.find(":children") != -1:
                     active_group_name = active_group_name.rsplit(":", 1)[0]
diff --git a/test/TestInventory.py b/test/TestInventory.py
index 8cc0c67c2a..998a011471 100644
--- a/test/TestInventory.py
+++ b/test/TestInventory.py
@@ -294,6 +294,12 @@ class TestInventory(unittest.TestCase):
         expected_hosts=['host1A','host2A','host1B','host2B']
         assert sorted(hosts) == sorted(expected_hosts)
 
+    def test_leading_range(self):
+        i = Inventory(os.path.join(self.test_dir, 'inventory','test_leading_range'))
+        hosts = i.list_hosts('test')
+        expected_hosts=['1.host','2.host','A.host','B.host']
+        assert sorted(hosts) == sorted(expected_hosts)
+
     ###################################################
     ### Inventory API tests
 
diff --git a/test/inventory/test_leading_range b/test/inventory/test_leading_range
new file mode 100644
index 0000000000..d6e8503319
--- /dev/null
+++ b/test/inventory/test_leading_range
@@ -0,0 +1,3 @@
+[test]
+[1:2].host
+[A:B].host