Function image_snapshot_revert has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def image_snapshot_revert(call=None, kwargs=None):
    '''
    Reverts an image state to a previous snapshot.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if ip_.is_private:
                        private_ips.append(adrs)
                    else:
                        public_ips.append(adrs)

Avoid deeply nested control flow statements.

Open

                    if device.deviceInfo.label in list(devices['disk'].keys()):
                        disk_spec = None
                        if 'size' in devices['disk'][device.deviceInfo.label]:
                            size_gb = float(devices['disk'][device.deviceInfo.label]['size'])
                            size_kb = int(size_gb * 1024.0 * 1024.0)

Avoid deeply nested control flow statements.

Open

                    if master_supports_ipv6 and _valid_ip6(current_ip.ipAddress):
                        log.info(
                            "[ %s ] Successfully retrieved IPv6 information "
                            "in %s seconds", vm_ref.name, time_counter
                        )

Avoid deeply nested control flow statements.

Open

                    if _valid_ip(current_ip.ipAddress) and not ipv4_address:
                        # Delay return in case we have a valid IPv6 available
                        ipv4_address = current_ip
        if ipv4_address:

Avoid deeply nested control flow statements.

Open

                    if spec.device.deviceInfo.label == devices['cd'][cd_drive_label]['controller']:
                        controller_key = spec.device.key
                        ide_controllers[controller_key] = 0
                        break
            else:

Avoid deeply nested control flow statements.

Open

                    if hasattr(device, 'addressType'):
                        device_full_info[device.deviceInfo.label]['addressType'] = device.addressType

Avoid deeply nested control flow statements.

Open

                    if device.deviceInfo.label in list(devices['cd'].keys()):
                        device_type = devices['cd'][device.deviceInfo.label]['device_type'] if 'device_type' in devices['cd'][device.deviceInfo.label] else ''
                        mode = devices['cd'][device.deviceInfo.label]['mode'] if 'mode' in devices['cd'][device.deviceInfo.label] else ''
                        iso_path = devices['cd'][device.deviceInfo.label]['iso_path'] if 'iso_path' in devices['cd'][device.deviceInfo.label] else ''
                        cd_drive_spec = _edit_existing_cd_or_dvd_drive(device, device_type, mode, iso_path)

Avoid deeply nested control flow statements.

Open

                    if num_devices < 2:
                        controller_key = ide_controller_key
                        break

Function show_storage_keys has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def show_storage_keys(kwargs=None, conn=None, call=None):
    '''
    .. versionadded:: 2015.8.0

    Show storage account keys

• Read up
  Read up

Function add_service_certificate has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def add_service_certificate(kwargs=None, conn=None, call=None):
    '''
    .. versionadded:: 2015.8.0

    Add a new service certificate

• Read up
  Read up

Function image_snapshot_flatten has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def image_snapshot_flatten(call=None, kwargs=None):
    '''
    Flattens the snapshot of an image and discards others.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Function secgroup_clone has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def secgroup_clone(call=None, kwargs=None):
    '''
    Clones an existing security group.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Function template_instantiate has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def template_instantiate(call=None, kwargs=None):
    '''
    Instantiates a new virtual machine from a template.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if hasattr(device, 'device'):
                        device_full_info[device.deviceInfo.label]['deviceKeys'] = device.device

Avoid deeply nested control flow statements.

Open

                    if hasattr(device, 'videoRamSizeInKB'):
                        device_full_info[device.deviceInfo.label]['videoRamSizeInKB'] = device.videoRamSizeInKB

Avoid deeply nested control flow statements.

Open

                    if isinstance(device, vim.vm.device.VirtualDisk):
                        device_full_info[device.deviceInfo.label]['capacityInKB'] = device.capacityInKB
                        device_full_info[device.deviceInfo.label]['diskMode'] = device.backing.diskMode
                        device_full_info[device.deviceInfo.label]['fileName'] = device.backing.fileName

Function vn_free_ar has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def vn_free_ar(call=None, kwargs=None):
    '''
    Frees a reserved address range from a virtual network.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Function _get_ips has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def _get_ips(node, addr_type='public'):
    ret = []
    for _, interface in node.addresses.items():
        for addr in interface:
            if addr_type in ('floating', 'fixed') and addr_type == addr.get('OS-EXT-IPS:type'):

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if spec.device.deviceInfo.label == devices['disk'][disk_label]['controller']:
                        disk_spec.device.controllerKey = spec.device.key
                        break