Added fix for #491

Author: wojtek0806

f5/bigip/mixins.py

@@ -16,9 +16,10 @@
 #
 # NOTE:  Code taken from Effective Python Item 26
 
-import logging
 
+from distutils.version import LooseVersion
 from f5.sdk_exception import F5SDKError
+import logging
 
 
 class InvalidCommand(F5SDKError):
@@ -117,10 +118,13 @@ def __getattr__(container, name):
 
     def _check_supported_versions(container, attribute):
         tmos_v = container._meta_data['bigip'].tmos_version
-        if tmos_v not in attribute._meta_data['supported_versions']:
+        minimum = attribute._meta_data['minimum_version']
+        if LooseVersion(tmos_v) < LooseVersion(minimum):
             error = "There was an attempt to access API which " \
                     "has not been implemented or supported " \
-                    "in the device's TMOS version: {}".format(tmos_v)
+                    "in the device's TMOS version: {}. "\
+                    "Minimum TMOS version supported is {}".format(
+                        tmos_v, minimum)
             raise UnsupportedTmosVersion(error)
 
 

f5/bigip/resource.py

@@ -179,7 +179,7 @@ def __init__(self, container):
         self._set_meta_data_uri()
         # Supported versions for each class will be defined here.
         # List can be modified downstream in each sub-class
-        self._meta_data['supported_versions'] = set(['11.6.0', '12.0.0'])
+        self._meta_data['minimum_version'] = '11.6.0'
         # Commands you can run on a resource or collection, we define it here
         self._meta_data['allowed_commands'] = []
         # Define required command parameters

f5/bigip/tm/cm/trust.py

@@ -38,7 +38,7 @@ def __init__(self, cm):
         self._meta_data['required_json_kind'] = \
             'tm:cm:add-to-trust:runstate'
         self._meta_data['allowed_commands'].append('run')
-        self._meta_data['supported_versions'].discard('11.6.0')
+        self._meta_data['minimum_version'] = '12.0.0'
 
 
 class Remove_From_Trust(PathElement, UnnamedResourceMixin,
@@ -61,4 +61,4 @@ def __init__(self, cm):
         self._meta_data['required_json_kind'] = \
             'tm:cm:remove-from-trust:runstate'
         self._meta_data['allowed_commands'].append('run')
-        self._meta_data['supported_versions'].discard('11.6.0')
+        self._meta_data['minimum_version'] = '12.0.0'

f5/bigip/tm/ltm/profile.py

@@ -512,7 +512,7 @@ def __init__(self, profile):
         self._meta_data['allowed_lazy_attributes'] = [Iiop]
         self._meta_data['attribute_registry'] = \
             {'tm:ltm:profile:iiop:iiopstate': Iiop}
-        self._meta_data['supported_versions'].discard('11.6.0')
+        self._meta_data['minimum_version'] = '12.0.0'
 
 
 class Iiop(Resource):
@@ -1056,7 +1056,7 @@ def __init__(self, profile):
         self._meta_data['allowed_lazy_attributes'] = [Tftp]
         self._meta_data['attribute_registry'] = \
             {'tm:ltm:profile:tftp:tftpstate': Tftp}
-        self._meta_data['supported_versions'].discard('11.6.0')
+        self._meta_data['minimum_version'] = '12.0.0'
 
 
 class Tftp(Resource):

f5/bigip/tm/shared/bigip_failover_state.py

@@ -48,7 +48,7 @@ def __init__(self, shared):
         super(Bigip_Failover_State, self).__init__(shared)
         self._meta_data['required_load_parameters'] = set()
         self._meta_data['required_json_kind'] = ''
-        self._meta_data['supported_versions'].discard('11.6.0')
+        self._meta_data['minimum_version'] = '12.0.0'
 
     def update(self, **kwargs):
         '''Update is not supported for BIG-IP® failover state.