Avoid deeply nested control flow statements.

Open

                    if not enforce and prop in vmconfig['current'][collection]:
                        continue

                    # create set_ dict
                    if 'set_{0}'.format(collection) not in vmconfig['changed']:

Avoid deeply nested control flow statements.

Open

                    if 'set_{0}'.format(collection) not in vmconfig['changed']:
                        vmconfig['changed']['set_{0}'.format(collection)] = {}

                    # add property to changeset
                    vmconfig['changed']['set_{0}'.format(collection)][prop] = vmconfig['state'][collection][prop]

Avoid deeply nested control flow statements.

Open

                    if not isinstance(rret, (bool)) and 'Error' in rret:
                        ret['result'] = False
                        ret['comment'] = 'vm {0} updated, reprovision failed'.format(
                            vmconfig['state']['hostname']
                        )

Avoid deeply nested control flow statements.

Open

                        if key in medias_attr:
                            medias_dict[med][key] = value

Avoid deeply nested control flow statements.

Open

                    if 'error' in updatemed:
                        error.append(updatemed['error'])

Avoid deeply nested control flow statements.

Open

                    if val is not None:
                        vmconfig['state']['internal_metadata'][var] = val

            # process add and update for collection
            if collection in vmconfig['state'] and vmconfig['state'][collection] is not None:

Avoid deeply nested control flow statements.

Open

                        if update_head:
                            ret['changes']['revision'] = {
                                'old': local_rev, 'new': remote_rev
                            }
                            if fast_forward is False:

Avoid deeply nested control flow statements.

Open

                    if instance in vmconfig['state'] and vmconfig['state'][instance] is not None:
                        for state_cfg in vmconfig['state'][instance]:
                            if vmconfig_type['instance'][instance] not in state_cfg:
                                continue

Avoid deeply nested control flow statements.

Open

                    if vmconfig['image_uuid'] not in res:
                        ret['result'] = False
                        ret['comment'] = 'failed to import image {0}'.format(vmconfig['image_uuid'])
            else:

Avoid deeply nested control flow statements.

Open

                    for cdisk in vmconfig['current']['disks']:
                        if cdisk['path'].endswith(disk['path']):
                            path = cdisk['path']
                            break
                if not path:

Avoid deeply nested control flow statements.

Open

                        for tag_name in deleted_tags:
                            try:
                                if not __opts__['test']:
                                    __salt__['git.tag'](
                                        target,

Avoid deeply nested control flow statements.

Open

                    for disk_id in [cache_disk_id] + capacity_disk_ids:
                        __salt__['vsphere.erase_disk_partitions'](
                            disk_id=disk_id, service_instance=si)
                    comments.append('Erased disks of diskgroup #{0}; '

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

Open

def move_config(name, xpath=None, where=None, dst=None, commit=False):
    '''
    Moves a XPATH value to a new location.

    Use the xpath parameter to specify the location of the object to be moved, the where parameter to

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if vmconfig['state']['hostname'] in __salt__['vmadm.list'](order='hostname', search='state=running'):
                        kvm_needs_start = True
                        __salt__['vmadm.stop'](vm=vmconfig['state']['hostname'], key='hostname')

            # do update

Avoid deeply nested control flow statements.

Open

                        if branch not in all_local_branches:
                            actions.append(
                                'New branch \'{0}\' would be checked '
                                'out, with {1} as a starting '
                                'point'.format(branch, remote_loc)

Avoid deeply nested control flow statements.

Open

                        if rev == 'HEAD':
                            checkout_rev = remote_rev
                        else:
                            checkout_rev = desired_upstream \
                                if desired_upstream \

Avoid deeply nested control flow statements.

Open

                    for idx in six_range(0, len(local)):
                        for key in local[idx]:
                            local_val = local[idx][key]
                            remote_val = remote[idx].get(key)
                            if isinstance(local_val, six.string_types):

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

Open

def subvolume_deleted(name, device, commit=False, __dest=None):
    '''
    Makes sure that a btrfs subvolume is removed.

    name

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if configured_list.index(server) != i:
                        ret['changes']['Servers Reordered'].append(server)
                else:

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

Open

def env_present(name,
                value=None,
                user='root'):
    '''
    Verifies that the specified environment variable is present in the crontab

• Read up
  Read up