Avoid deeply nested control flow statements.

Open

                        if alias not in ret:
                            ret[alias] = {}
                        if driver not in ret[alias]:

Avoid deeply nested control flow statements.

Open

                        if kwargs:
                            ret[alias][driver][vm_name] = self.clouds[fun](
                                vm_name, kwargs, call='action'
                            )
                        else:

Avoid deeply nested control flow statements.

Open

                        if 'id' in vm_details and vm_details['id'] in names:
                            vm_name = vm_details['id']
                        else:
                            log.debug(
                                'vm:%s in provider:%s is not in name '

Avoid deeply nested control flow statements.

Open

                        if name not in ret['create']:
                            # Machine already removed
                            break

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

Open

def _parse_pricing(url, name):
    '''
    Download and parse an individual pricing file from AWS

    .. versionadded:: 2015.8.0

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if not isinstance(vm_details, dict):
                            vm_details = {}
                        if 'id' in vm_details and vm_details['id'] in names:

Avoid deeply nested control flow statements.

Open

                        if driver not in ret[alias]:
                            ret[alias][driver] = {}

                        # Clean kwargs of "__pub_*" data before running the cloud action call.
                        # Prevents calling positional "kwarg" arg before "call" when no kwarg

Avoid deeply nested control flow statements.

Open

                        if overrides is None or isinstance(overrides, bool):
                            # Foo:
                            #   - bar1:
                            #   - bar2:
                            overrides = {}

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

Open

def avail_images(conn=None):
    '''
    Return a dict of all available VM images on the cloud provider with
    relevant data.

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if invalid_functions.get(fun) is None:
                            invalid_functions.update({fun: []})
                        invalid_functions[fun].append(vm_name)

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

Open

def list_nodes(call=None):
    ''' Return a list of the BareMetal servers that are on the provider.
    '''
    if call == 'action':
        raise SaltCloudSystemExit(

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if subnet_id is not None:
                        return subnet_id

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

Open

def image_snapshot_delete(call=None, kwargs=None):
    '''
    Deletes a snapshot from the image.

    .. versionadded:: 2016.3.0

• Read up
  Read up

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

Open

def template_clone(call=None, kwargs=None):
    '''
    Clones an existing virtual machine template.

    .. versionadded:: 2016.3.0

• Read up
  Read up

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

Open

def remove_host(kwargs=None, call=None):
    '''
    Remove the specified host system from this VMware environment

    CLI Example:

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

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

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

Open

def create_snapshot(name, kwargs=None, call=None):
    '''
    Create a snapshot of the specified virtual machine in this VMware
    environment

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if device.unitNumber:
                        device_full_info[device.deviceInfo.label]['unitNumber'] = device.unitNumber

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

Open

def image_persistent(call=None, kwargs=None):
    '''
    Sets the Image as persistent or not persistent.

    .. versionadded:: 2016.3.0

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if device.deviceInfo.label in list(devices['scsi'].keys()):
                        # Modify the existing SCSI controller
                        scsi_controller_properties = devices['scsi'][device.deviceInfo.label]
                        bus_sharing = scsi_controller_properties['bus_sharing'].strip().lower() if 'bus_sharing' in scsi_controller_properties else None
                        if bus_sharing and bus_sharing in ['virtual', 'physical', 'no']: