Some missing protected access tagging

cloudbridge/providers/aws/services.py

@@ -179,6 +179,7 @@ class AWSVMFirewallService(BaseVMFirewallService):
               priority=BaseVMFirewallService.STANDARD_EVENT_PRIORITY)
     def create(self, label, network, description=None):
         AWSVMFirewall.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = AWSVMFirewall._generate_name_from_label(label, 'cb-fw')
         network_id = network.id if isinstance(network, Network) else network
         obj = self.svc.create('create_security_group', GroupName=name,

cloudbridge/providers/azure/resources.py

@@ -1182,6 +1182,7 @@ class AzureInstance(BaseInstance):
         """
 
         self.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = self._generate_name_from_label(label, 'cb-img')
 
         if not self._state == 'VM generalized':

cloudbridge/providers/azure/services.py

@@ -112,6 +112,7 @@ class AzureVMFirewallService(BaseVMFirewallService):
               priority=BaseVMFirewallService.STANDARD_EVENT_PRIORITY)
     def create(self, label, network, description=None):
         AzureVMFirewall.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = AzureVMFirewall._generate_name_from_label(label, "cb-fw")
         net = network.id if isinstance(network, Network) else network
         parameters = {"location": self.provider.region_name,
@@ -398,6 +399,7 @@ class AzureVolumeService(BaseVolumeService):
               priority=BaseVolumeService.STANDARD_EVENT_PRIORITY)
     def create(self, label, size, snapshot=None, description=None):
         AzureVolume.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         disk_name = AzureVolume._generate_name_from_label(label, "cb-vol")
         tags = {'Label': label}
 
@@ -490,6 +492,7 @@ class AzureSnapshotService(BaseSnapshotService):
               priority=BaseSnapshotService.STANDARD_EVENT_PRIORITY)
     def create(self, label, volume, description=None):
         AzureSnapshot.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         snapshot_name = AzureSnapshot._generate_name_from_label(label,
                                                                 "cb-snap")
         tags = {'Label': label}
@@ -827,6 +830,7 @@ class AzureInstanceService(BaseInstanceService):
                key_pair=None, vm_firewalls=None, user_data=None,
                launch_config=None, **kwargs):
         AzureInstance.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         instance_name = AzureInstance._generate_name_from_label(label,
                                                                 "cb-ins")
 
@@ -1145,6 +1149,7 @@ class AzureNetworkService(BaseNetworkService):
             'tags': {'Label': label}
         }
 
+        # pylint:disable=protected-access
         network_name = AzureNetwork._generate_name_from_label(label, 'cb-net')
 
         az_network = self.provider.azure_client.create_network(network_name,
@@ -1228,6 +1233,7 @@ class AzureSubnetService(BaseSubnetService):
         AzureSubnet.assert_valid_resource_label(label)
         # Although Subnet doesn't support tags in Azure, we use the parent
         # Network's tags to track its subnets' labels
+        # pylint:disable=protected-access
         subnet_name = AzureSubnet._generate_name_from_label(label, "cb-sn")
 
         network_id = network.id \
@@ -1306,6 +1312,7 @@ class AzureRouterService(BaseRouterService):
     @dispatch(event="provider.networking.routers.create",
               priority=BaseRouterService.STANDARD_EVENT_PRIORITY)
     def create(self, label, network):
+        # pylint:disable=protected-access
         router_name = AzureRouter._generate_name_from_label(label, "cb-router")
 
         parameters = {"location": self.provider.region_name,
@@ -1387,6 +1394,7 @@ class AzureFloatingIPService(BaseFloatingIPService):
             'public_ip_allocation_method': 'Static'
         }
 
+        # pylint:disable=protected-access
         public_ip_name = AzureFloatingIP._generate_name_from_label(
             None, 'cb-fip-')
 

cloudbridge/providers/gcp/services.py

@@ -555,6 +555,7 @@ class GCPInstanceService(BaseInstanceService):
         # The boot disk must be the first disk attached to the instance.
         disks.insert(0, boot_disk)
 
+        # pylint:disable=protected-access
         config = {
             'name': GCPInstance._generate_name_from_label(label, 'cb-inst'),
             'machineType': vm_type.resource_url,
@@ -791,6 +792,7 @@ class GCPNetworkService(BaseNetworkService):
         to add additional subnets later.
         """
         GCPNetwork.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = GCPNetwork._generate_name_from_label(label, 'cbnet')
         body = {'name': name}
         # This results in a custom mode network
@@ -894,6 +896,7 @@ class GCPRouterService(BaseRouterService):
         log.debug("Creating GCP Router Service with params "
                   "[label: %s network: %s]", label, network)
         GCPRouter.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = GCPRouter._generate_name_from_label(label, 'cb-router')
 
         if not isinstance(network, GCPNetwork):
@@ -983,6 +986,7 @@ class GCPSubnetService(BaseSubnetService):
         provider, is used.
         """
         GCPSubnet.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = GCPSubnet._generate_name_from_label(label, 'cbsubnet')
         region_name = self.provider.region_name
 #         for subnet in self.iter(network=network):
@@ -1183,6 +1187,7 @@ class GCPVolumeService(BaseVolumeService):
               priority=BaseVolumeService.STANDARD_EVENT_PRIORITY)
     def create(self, label, size, snapshot=None, description=None):
         GCPVolume.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = GCPVolume._generate_name_from_label(label, 'cb-vol')
         zone_name = self.provider.zone_name
         snapshot_id = snapshot.id if isinstance(
@@ -1286,6 +1291,7 @@ class GCPSnapshotService(BaseSnapshotService):
               priority=BaseSnapshotService.STANDARD_EVENT_PRIORITY)
     def create(self, label, volume, description=None):
         GCPSnapshot.assert_valid_resource_label(label)
+        # pylint:disable=protected-access
         name = GCPSnapshot._generate_name_from_label(label, 'cbsnap')
         volume_name = volume.name if isinstance(volume, GCPVolume) else volume
         labels = {'cblabel': label}

cloudbridge/providers/openstack/services.py

@@ -806,6 +806,7 @@ class OpenStackInstanceService(BaseInstanceService):
             port_def = {
                 "port": {
                     "admin_state_up": True,
+                    # pylint:disable=protected-access
                     "name": OpenStackInstance._generate_name_from_label(
                         label, 'cb-port'),
                     "network_id": net_id,