Merge pull request #2943 from detiber/fixOpenShiftRelease

Properly handle x.y.z formatted versions for openshift_release

roles/openshift_master_facts/lookup_plugins/openshift_master_facts_default_predicates.py

@@ -32,6 +32,7 @@ class LookupModule(LookupBase):
                     short_version = release[1:]
                 else:
                     short_version = release
+                short_version = '.'.join(short_version.split('.')[0:2])
             elif 'openshift_version' in variables:
                 version = variables['openshift_version']
                 short_version = '.'.join(version.split('.')[0:2])
@@ -64,7 +65,6 @@ class LookupModule(LookupBase):
 
         if short_version in ['1.1', '1.2']:
             predicates.append({'name': 'MatchNodeSelector'})
-            predicates.append({'name': 'Hostname'})
 
         if short_version != '1.1':
             predicates.append({'name': 'MaxEBSVolumeCount'})

roles/openshift_master_facts/lookup_plugins/openshift_master_facts_default_priorities.py

@@ -36,6 +36,7 @@ class LookupModule(LookupBase):
                     short_version = release[1:]
                 else:
                     short_version = release
+                short_version = '.'.join(short_version.split('.')[0:2])
             elif 'openshift_version' in variables:
                 version = variables['openshift_version']
                 short_version = '.'.join(version.split('.')[0:2])

roles/openshift_master_facts/test/openshift_master_facts_default_predicates_tests.py

@@ -14,7 +14,6 @@ DEFAULT_PREDICATES_1_1 = [
     {'name': 'PodFitsResources'},
     {'name': 'NoDiskConflict'},
     {'name': 'MatchNodeSelector'},
-    {'name': 'Hostname'}
 ]
 
 DEFAULT_PREDICATES_1_2 = [
@@ -23,7 +22,6 @@ DEFAULT_PREDICATES_1_2 = [
     {'name': 'NoDiskConflict'},
     {'name': 'NoVolumeZoneConflict'},
     {'name': 'MatchNodeSelector'},
-    {'name': 'Hostname'},
     {'name': 'MaxEBSVolumeCount'},
     {'name': 'MaxGCEPDVolumeCount'}
 ]
@@ -211,6 +209,11 @@ class TestOpenShiftMasterFactsDefaultPredicates(object):
             for release, deployment_type, default_predicates in TEST_VARS:
                 yield self.check_defaults_deployment_type_kwarg, release, deployment_type, default_predicates, regions_enabled
 
+    def test_trunc_openshift_release(self):
+        for release, deployment_type, default_predicates in TEST_VARS:
+            release = release + '.1'
+            yield self.check_defaults_release, release, deployment_type, default_predicates, False
+
     @raises(AnsibleError)
     def test_unknown_deployment_types(self):
         facts = copy.deepcopy(self.default_facts)

roles/openshift_master_facts/test/openshift_master_facts_default_priorities_tests.py

@@ -200,6 +200,11 @@ class TestOpenShiftMasterFactsDefaultPredicates(object):
             for short_version, deployment_type, default_priorities in TEST_VARS:
                 yield self.check_defaults_short_version_kwarg, short_version, deployment_type, default_priorities, zones_enabled
 
+    def test_trunc_openshift_release(self):
+        for release, deployment_type, default_priorities in TEST_VARS:
+            release = release + '.1'
+            yield self.check_defaults_release, release, deployment_type, default_priorities, False
+
     @raises(AnsibleError)
     def test_unknown_origin_version(self):
         facts = copy.deepcopy(self.default_facts)