Adds security log profile api (#1427)

Issues:
Fixes #1426

Problem:
The API did not exist

Analysis:
This adds it

Tests:
* functional
* unit

Author: caphrim007

f5-sdk-dist/scripts/build_exceptions.py

@@ -108,7 +108,7 @@ def __init__(self, *args, **kargs):
         super(RedhatError, self).__init__(*args, **kargs)
 
 
-class TestError(BuildError):
+class ErrorInTest(BuildError):
     """TestError
 
 An Error occurred during testing...
@@ -117,7 +117,7 @@ class TestError(BuildError):
 
     def __init__(self, *args, **kargs):
         # exception-specific logic here...
-        super(TestError, self).__init__(*args, **kargs)
+        super(ErrorInTest, self).__init__(*args, **kargs)
 
 
 # vim: set fileencoding=utf-8

f5-sdk-dist/scripts/install_test.py

@@ -44,7 +44,7 @@
 from . import build_expectations
 
 from . terminal import terminal
-from . build_exceptions import TestError
+from . build_exceptions import ErrorInTest
 
 # Globals:
 builds = build_expectations.Builds()
@@ -128,7 +128,7 @@ def _set_expected_opts(self, opts):
         if opts:
             if not isinstance(opts, tuple) or \
                     not isinstance(opts[0][0], OperationSettings):
-                raise TestError(msg=err_msg, frame=gfi(cf()),
+                raise ErrorInTest(msg=err_msg, frame=gfi(cf()),
                                 errnum=errno.ESPIPE)
             self.__expected_opts = opts
 
@@ -180,7 +180,7 @@ def collect_pkgs(self):
                 pkg_search = dist + "/rpms/build/*.rpm"
                 pkg_re = re.compile('el(\d+)\.noarch')
             else:
-                raise TestError(opt.os_type, frame=gfi(cf()),
+                raise ErrorInTest(opt.os_type, frame=gfi(cf()),
                                 errnum=errno.ESPIPE,
                                 msg='opt.os_type(%s) is not recognized!')
             entropy = glob.glob(pkg_search)
@@ -208,7 +208,7 @@ def collect_pkgs(self):
                         break
                     if not found:
                         self._set_failure_reason = \
-                            TestError(opt.os_type, opt.version,
+                            ErrorInTest(opt.os_type, opt.version,
                                       frame=gfi(cf()), errno=errno.ESPIPE,
                                       msg='No pkg found built for')
                         print(str(self.failure_reason))
@@ -266,7 +266,7 @@ def test(self, pkg):
             msg = "Failed to build docker container to test with"
         if frame:
             self._set_failure_reason = \
-                TestError(pkg.pkg, msg=msg, frame=frame, errnum=errno.ESPIPE)
+                ErrorInTest(pkg.pkg, msg=msg, frame=frame, errnum=errno.ESPIPE)
             raise self.failure_reason
 
 
@@ -301,13 +301,13 @@ def main():
     try:
         tests = InstallTest()
         tests.execute_tests()
-    except TestError as Error:
+    except ErrorInTest as Error:
         print(str(Error))
         exit(1)
         traceback.print_exc()
     except Exception as Error:
         print(str(Error))
-        TestError(Error, frame=gfi(cf()), errno=-1)
+        ErrorInTest(Error, frame=gfi(cf()), errno=-1)
         print(str(Error))
         traceback.print_exc()
 

f5/bigip/cm/system.py

@@ -47,9 +47,9 @@ def __init__(self, authn):
 
 
 class Tmos_s(Collection):
-    def __init__(self, providers):
-        super(Tmos_s, self).__init__(providers)
-        if (self._meta_data['bigip']._meta_data['tmos_version'] < '13.1.0'):
+    def __init__(self, container):
+        super(Tmos_s, self).__init__(container)
+        if container._meta_data['bigip']._meta_data['tmos_version'] < '13.1.0':
             # Starting from bigip v13.1.0 tmos resources 'kind' was changed
             # from 'mcpremoteproviderstate' to 'authproviderstate'
             # and tmos collection 'kind'
@@ -67,9 +67,9 @@ def __init__(self, providers):
 
 
 class Tmos(Resource):
-    def __init__(self, tokens):
-        super(Tmos, self).__init__(tokens)
-        if (self._meta_data['bigip']._meta_data['tmos_version'] < '13.1.0'):
+    def __init__(self, container):
+        super(Tmos, self).__init__(container)
+        if container._meta_data['bigip']._meta_data['tmos_version'] < '13.1.0':
             # Starting from bigip v13.1.0 tmos resource 'kind' was changed
             # from 'mcpremoteproviderstate' to 'authproviderstate'
             self._meta_data['required_json_kind'] = 'cm:system:authn:providers:tmos:mcpremoteproviderstate'

f5/bigip/cm/test/unit/test_system.py

@@ -23,7 +23,13 @@
 @pytest.fixture
 def FakeTmos():
     mo = mock.MagicMock()
+    r = {'tmos_version': '11.6.0'}
+    m = mock.MagicMock()
+    m.__getitem__.side_effect = r.__getitem__
+    m.__iter__.side_effect = r.__iter__
+    mo._meta_data['bigip']._meta_data = m
     resource = Tmos(mo)
+
     return resource
 
 

f5/bigip/tm/security/__init__.py

@@ -31,6 +31,7 @@
 from f5.bigip.tm.security.analytics import Analytics
 from f5.bigip.tm.security.dos import Dos
 from f5.bigip.tm.security.firewall import Firewall
+from f5.bigip.tm.security.log import Log
 from f5.bigip.tm.security.protocol_inspection import Protocol_Inspection
 
 
@@ -39,4 +40,10 @@ class Security(OrganizingCollection):
 
     def __init__(self, tm):
         super(Security, self).__init__(tm)
-        self._meta_data['allowed_lazy_attributes'] = [Dos, Firewall, Analytics, Protocol_Inspection]
+        self._meta_data['allowed_lazy_attributes'] = [
+            Analytics,
+            Dos,
+            Firewall,
+            Log,
+            Protocol_Inspection,
+        ]