Fix bugs in ~6 files.

Issues:
Fixes #<issueid>

Problem:

Analysis:

Tests:

Author: zancas

f5/bigip/tm/cm/device_group.py

@@ -37,7 +37,7 @@ def __init__(self, cm):
         super(Device_Groups, self).__init__(cm)
         self._meta_data['allowed_lazy_attributes'] = [Device_Group]
         self._meta_data['attribute_registry'] =\
-            {'tm:cm:device:device-groupstate': Device_Group}
+            {'tm:cm:device-group:device-groupstate': Device_Group}
 
 
 class Device_Group(Resource):

f5/bigip/tm/ltm/persistence.py

@@ -29,10 +29,11 @@
 
 
 from f5.bigip.resource import Collection
+from f5.bigip.resource import OrganizingCollection
 from f5.bigip.resource import Resource
 
 
-class Persistences(Collection):
+class Persistences(OrganizingCollection):
     '''A Collection concrete subclass docstring.'''
     def __init__(self, ltm):
         '''Auto generated constructor.'''

f5/bigip/tm/net/fdb.py

@@ -29,10 +29,11 @@
 
 
 from f5.bigip.resource import Collection
+from f5.bigip.resource import OrganizingCollection
 from f5.bigip.resource import Resource
 
 
-class Fdbs(Collection):
+class Fdbs(OrganizingCollection):
     '''A Collection concrete subclass docstring.'''
     def __init__(self, net):
         '''Auto generated constructor.'''

f5/bigip/tm/net/tunnels.py

@@ -28,10 +28,11 @@
 """
 
 from f5.bigip.resource import Collection
+from f5.bigip.resource import OrganizingCollection
 from f5.bigip.resource import Resource
 
 
-class Tunnels_s(Collection):
+class Tunnels_s(OrganizingCollection):
     """BIG-IP® network tunnels collection"""
     def __init__(self, net):
         super(Tunnels_s, self).__init__(net)

f5/bigip/tm/sys/application.py

@@ -28,13 +28,14 @@
 """
 
 from f5.bigip.resource import Collection
+from f5.bigip.resource import OrganizingCollection
 from f5.bigip.resource import KindTypeMismatch
 from f5.bigip.resource import Resource
 
 from requests import HTTPError
 
 
-class Applications(Collection):
+class Applications(OrganizingCollection):
     """BIG-IP® iApp collection."""
     def __init__(self, sys):
         super(Applications, self).__init__(sys)

f5/utils/testutils/registrytools.py

@@ -13,6 +13,12 @@
 # limitations under the License.
 #
 
+import logging
+from pprint import pprint as pp
+
+from f5.bigip.mixins import UnsupportedTmosVersion
+from f5.bigip.resource import OrganizingCollection
+from f5.bigip.resource import Collection
 
 def register_collection_atoms(collection):
     '''Given a collection return a registry of all of its atoms (elements).
@@ -21,8 +27,17 @@ def register_collection_atoms(collection):
     A registry provides a snapshot of all resources of a type on the device.
     '''
     resource_registry = {}
-    for resource in collection.get_collection():
-            resource_registry[resource.selfLink] = resource
+    try:
+        resources = collection.get_collection()
+    except Exception as ex:
+        logging.debug(ex)
+        return resource_registry
+    for resource in resources:
+            try:
+                resource_registry[resource.selfLink] = resource
+            except KeyError as ex:
+                pp(resource.raw)
+                raise ex
     return resource_registry
 
 
@@ -35,6 +50,38 @@ def register_OC_atoms(organizing_collection):
     OC_atoms_registry = {}
     OC_types = organizing_collection._meta_data['allowed_lazy_attributes']
     for OC_type in OC_types:
-        collection = getattr(organizing_collection, OC_type.__name__.lower())
-        OC_atoms_registry.update(register_collection_atoms(collection))
+        try:
+            lazy_instance =\
+                getattr(organizing_collection, OC_type.__name__.lower())
+        except UnsupportedTmosVersion as ex:
+            logging.debug(ex)
+            continue
+        if isinstance(lazy_instance, Collection):
+            OC_atoms_registry.update(register_collection_atoms(lazy_instance))
+        elif isinstance(lazy_instance, OrganizingCollection):
+            OC_atoms_registry.update(register_OC_atoms(lazy_instance))
+        #else:
+        #    OC_atoms_registry.update({lazy_instance.selfLink: lazy_instance})
     return OC_atoms_registry
+
+
+def register_loadbalancer_elements(mgmt_rt):
+    monitor_registry = register_OC_atoms(mgmt_rt.tm.ltm.monitor)
+    pool_registry = register_collection_atoms(mgmt_rt.tm.ltm.pools)
+    snat_registry = register_collection_atoms(mgmt_rt.tm.ltm.snats)
+    virtual_registry = register_collection_atoms(mgmt_rt.tm.ltm.virtuals)
+    virtual_address_s_registry =\
+        register_collection_atoms(mgmt_rt.tm.ltm.virtual_address_s)
+    member_registry = {}
+    for pool in pool_registry.values():
+        mc = pool.members_s
+        member_registry.update(reqister_collection_atoms(mc))        
+    folder_registry = register_collection_atoms(mgmt_rt.tm.sys.folders)
+    registries = {'monitor_registry': monitor_registry,
+                  'pool_registry': pool_registry,
+                  'snat_registry': snat_registry,
+                  'virtual_registry': virtual_registry,
+                  'virtual_address_s_registry': virtual_address_s_registry,
+                  'member_registry': member_registry,
+                  'folder_registry': folder_registry}
+    return registries