summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatt Clay <matt@mystile.com>2017-01-27 15:45:23 -0800
committerGitHub <noreply@github.com>2017-01-27 15:45:23 -0800
commit95789f3949b01c79c729315ff375d0e759f64970 (patch)
treef1e2ed9fcc6aaa5e1ed56a20dd1e609e55489cb6
parent802fbcadf855db34bfdbcec0c6310c537ff3a5d0 (diff)
downloadansible-95789f3949b01c79c729315ff375d0e759f64970.tar.gz
PEP 8 whitespace cleanup. (#20783)
* PEP 8 E271 whitespace cleanup. * PEP 8 W293 whitespace cleanup. * Fix whitespace issue from recent PR.
-rwxr-xr-xcontrib/inventory/abiquo.py2
-rwxr-xr-xcontrib/inventory/apache-libcloud.py4
-rwxr-xr-xcontrib/inventory/consul_io.py4
-rwxr-xr-xcontrib/inventory/fleet.py2
-rwxr-xr-xcontrib/inventory/freeipa.py6
-rwxr-xr-xcontrib/inventory/spacewalk.py2
-rw-r--r--contrib/inventory/stacki.py2
-rwxr-xr-xhacking/dump_playbook_attributes.py8
-rw-r--r--lib/ansible/executor/stats.py2
-rw-r--r--lib/ansible/galaxy/login.py6
-rw-r--r--lib/ansible/galaxy/role.py2
-rw-r--r--lib/ansible/galaxy/token.py8
-rw-r--r--lib/ansible/module_utils/azure_rm_common.py6
-rw-r--r--lib/ansible/module_utils/facts.py4
-rw-r--r--lib/ansible/module_utils/redhat.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_asg.py6
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_customer_gateway.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_elb_facts.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_tag.py8
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_vol_facts.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_vpc_dhcp_options.py6
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_vpc_net_facts.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/ec2_vpc_vgw_facts.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/iam.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/rds.py6
-rw-r--r--lib/ansible/modules/cloud/amazon/rds_subnet_group.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/route53.py2
-rw-r--r--lib/ansible/modules/cloud/amazon/s3_website.py14
-rw-r--r--lib/ansible/modules/cloud/amazon/sqs_queue.py4
-rw-r--r--lib/ansible/modules/cloud/azure/azure.py4
-rw-r--r--lib/ansible/modules/cloud/azure/azure_rm_storageblob.py2
-rw-r--r--lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py2
-rw-r--r--lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py2
-rw-r--r--lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py2
-rw-r--r--lib/ansible/modules/cloud/docker/docker_image.py2
-rw-r--r--lib/ansible/modules/cloud/docker/docker_service.py4
-rw-r--r--lib/ansible/modules/cloud/google/gc_storage.py18
-rw-r--r--lib/ansible/modules/cloud/google/gcdns_zone.py2
-rw-r--r--lib/ansible/modules/cloud/google/gce_img.py2
-rw-r--r--lib/ansible/modules/cloud/google/gcpubsub.py6
-rw-r--r--lib/ansible/modules/cloud/google/gcpubsub_facts.py4
-rw-r--r--lib/ansible/modules/cloud/linode/linode.py4
-rw-r--r--lib/ansible/modules/cloud/misc/ovirt.py4
-rw-r--r--lib/ansible/modules/cloud/misc/proxmox_kvm.py2
-rw-r--r--lib/ansible/modules/cloud/misc/virt_net.py2
-rw-r--r--lib/ansible/modules/cloud/openstack/_quantum_router_gateway.py2
-rw-r--r--lib/ansible/modules/cloud/openstack/_quantum_router_interface.py4
-rw-r--r--lib/ansible/modules/cloud/openstack/os_zone.py2
-rw-r--r--lib/ansible/modules/cloud/packet/packet_device.py6
-rw-r--r--lib/ansible/modules/cloud/packet/packet_sshkey.py2
-rw-r--r--lib/ansible/modules/cloud/rackspace/rax_facts.py4
-rw-r--r--lib/ansible/modules/cloud/vmware/vmware_vswitch.py14
-rw-r--r--lib/ansible/modules/cloud/vmware/vsphere_guest.py2
-rw-r--r--lib/ansible/modules/cloud/webfaction/webfaction_app.py2
-rw-r--r--lib/ansible/modules/cloud/webfaction/webfaction_db.py10
-rw-r--r--lib/ansible/modules/cloud/webfaction/webfaction_domain.py2
-rw-r--r--lib/ansible/modules/cloud/webfaction/webfaction_mailbox.py2
-rw-r--r--lib/ansible/modules/cloud/webfaction/webfaction_site.py4
-rw-r--r--lib/ansible/modules/clustering/consul.py2
-rw-r--r--lib/ansible/modules/crypto/openssl_privatekey.py2
-rw-r--r--lib/ansible/modules/crypto/openssl_publickey.py2
-rw-r--r--lib/ansible/modules/database/misc/elasticsearch_plugin.py4
-rw-r--r--lib/ansible/modules/database/misc/kibana_plugin.py6
-rw-r--r--lib/ansible/modules/database/misc/redis.py2
-rw-r--r--lib/ansible/modules/database/misc/riak.py2
-rw-r--r--lib/ansible/modules/database/mssql/mssql_db.py2
-rw-r--r--lib/ansible/modules/database/postgresql/postgresql_ext.py2
-rw-r--r--lib/ansible/modules/database/postgresql/postgresql_lang.py8
-rw-r--r--lib/ansible/modules/database/postgresql/postgresql_user.py2
-rw-r--r--lib/ansible/modules/database/vertica/vertica_configuration.py2
-rw-r--r--lib/ansible/modules/database/vertica/vertica_role.py2
-rw-r--r--lib/ansible/modules/files/patch.py2
-rw-r--r--lib/ansible/modules/monitoring/boundary_meter.py6
-rw-r--r--lib/ansible/modules/monitoring/newrelic_deployment.py2
-rw-r--r--lib/ansible/modules/monitoring/pagerduty.py4
-rw-r--r--lib/ansible/modules/monitoring/zabbix_host.py4
-rwxr-xr-xlib/ansible/modules/network/bigswitch/bigmon_chain.py2
-rw-r--r--lib/ansible/modules/network/cloudflare_dns.py2
-rw-r--r--lib/ansible/modules/network/dellos10/dellos10_facts.py10
-rw-r--r--lib/ansible/modules/network/dellos6/dellos6_facts.py2
-rw-r--r--lib/ansible/modules/network/dnsmadeeasy.py20
-rw-r--r--lib/ansible/modules/network/eos/eos_eapi.py4
-rw-r--r--lib/ansible/modules/network/lldp.py6
-rw-r--r--lib/ansible/modules/network/nxos/nxos_aaa_server.py2
-rw-r--r--lib/ansible/modules/network/nxos/nxos_aaa_server_host.py4
-rw-r--r--lib/ansible/modules/network/nxos/nxos_bgp_af.py2
-rw-r--r--lib/ansible/modules/network/nxos/nxos_pim.py2
-rw-r--r--lib/ansible/modules/network/nxos/nxos_pim_interface.py6
-rw-r--r--lib/ansible/modules/network/nxos/nxos_snmp_location.py2
-rw-r--r--lib/ansible/modules/network/nxos/nxos_snmp_traps.py2
-rw-r--r--lib/ansible/modules/network/wakeonlan.py2
-rw-r--r--lib/ansible/modules/notification/flowdock.py2
-rw-r--r--lib/ansible/modules/packaging/os/apt.py2
-rwxr-xr-xlib/ansible/modules/packaging/os/apt_rpm.py8
-rw-r--r--lib/ansible/modules/packaging/os/layman.py4
-rw-r--r--lib/ansible/modules/packaging/os/rhn_channel.py8
-rw-r--r--lib/ansible/modules/packaging/os/urpmi.py10
-rw-r--r--lib/ansible/modules/packaging/os/yum.py12
-rw-r--r--lib/ansible/modules/packaging/os/zypper.py2
-rw-r--r--lib/ansible/modules/packaging/os/zypper_repository.py2
-rw-r--r--lib/ansible/modules/remote_management/foreman/katello.py10
-rw-r--r--lib/ansible/modules/remote_management/stacki/stacki_host.py50
-rw-r--r--lib/ansible/modules/source_control/github_hooks.py4
-rw-r--r--lib/ansible/modules/system/group.py2
-rw-r--r--lib/ansible/modules/system/locale_gen.py8
-rw-r--r--lib/ansible/modules/system/pam_limits.py2
-rw-r--r--lib/ansible/modules/system/sysctl.py2
-rwxr-xr-xlib/ansible/modules/web_infrastructure/jira.py2
-rw-r--r--lib/ansible/modules/web_infrastructure/letsencrypt.py2
-rw-r--r--lib/ansible/modules/windows/win_package.py2
-rw-r--r--lib/ansible/modules/windows/win_stat.py2
-rw-r--r--lib/ansible/playbook/role/metadata.py4
-rw-r--r--lib/ansible/playbook/role_include.py2
-rw-r--r--lib/ansible/plugins/action/package.py2
-rw-r--r--lib/ansible/plugins/action/service.py2
-rw-r--r--lib/ansible/plugins/callback/jabber.py2
-rw-r--r--lib/ansible/plugins/callback/syslog_json.py2
-rw-r--r--lib/ansible/plugins/connection/iocage.py2
-rw-r--r--lib/ansible/plugins/lookup/first_found.py2
-rw-r--r--lib/ansible/plugins/lookup/hashi_vault.py14
-rw-r--r--lib/ansible/plugins/strategy/__init__.py6
-rw-r--r--lib/ansible/vars/__init__.py2
-rw-r--r--test/integration/cleanup_ec2.py4
-rw-r--r--test/sanity/pep8/legacy-files.txt24
-rw-r--r--test/sanity/pep8/legacy-ignore.txt2
-rw-r--r--test/units/cli/test_galaxy.py2
-rw-r--r--test/units/module_utils/gcp/test_auth.py4
-rw-r--r--test/units/module_utils/test_basic.py6
-rw-r--r--test/units/modules/cloud/google/test_gce_tag.py6
-rw-r--r--test/units/plugins/lookup/test_lastpass.py4
-rw-r--r--test/units/plugins/strategy/test_strategy_base.py8
-rw-r--r--test/units/template/test_templar.py2
132 files changed, 287 insertions, 313 deletions
diff --git a/contrib/inventory/abiquo.py b/contrib/inventory/abiquo.py
index 04f7d4e60d..0b5533bb6b 100755
--- a/contrib/inventory/abiquo.py
+++ b/contrib/inventory/abiquo.py
@@ -142,7 +142,7 @@ def generate_inv_from_api(enterprise_entity,config):
break
else:
vm_nic = None
-
+
vm_state = True
# From abiquo.ini: Only adding to inventory VMs deployed
if ((config.getboolean('defaults', 'deployed_only') == True) and (vmcollection['state'] == 'NOT_ALLOCATED')):
diff --git a/contrib/inventory/apache-libcloud.py b/contrib/inventory/apache-libcloud.py
index f7d64c257c..fd020caae8 100755
--- a/contrib/inventory/apache-libcloud.py
+++ b/contrib/inventory/apache-libcloud.py
@@ -145,7 +145,7 @@ class LibcloudInventory(object):
self.cache_path_cache = cache_path + "/ansible-libcloud.cache"
self.cache_path_index = cache_path + "/ansible-libcloud.index"
self.cache_max_age = config.getint('cache', 'cache_max_age')
-
+
def parse_cli_args(self):
'''
@@ -225,7 +225,7 @@ class LibcloudInventory(object):
# Inventory: Group by key pair
if node.extra['key_name']:
self.push(self.inventory, self.to_safe('key_' + node.extra['key_name']), dest)
-
+
# Inventory: Group by security group, quick thing to handle single sg
if node.extra['security_group']:
self.push(self.inventory, self.to_safe('sg_' + node.extra['security_group'][0]), dest)
diff --git a/contrib/inventory/consul_io.py b/contrib/inventory/consul_io.py
index 2f226e7b50..40b3d35eba 100755
--- a/contrib/inventory/consul_io.py
+++ b/contrib/inventory/consul_io.py
@@ -289,7 +289,7 @@ class ConsulInventory(object):
and the node name add each entry in the dictionary to the the node's
metadata '''
node = node_data['Node']
- if self.config.has_config('kv_metadata'):
+ if self.config.has_config('kv_metadata'):
key = "%s/%s/%s" % (self.config.kv_metadata, self.current_dc, node['Node'])
index, metadata = self.consul_api.kv.get(key)
if metadata and metadata['Value']:
@@ -305,7 +305,7 @@ class ConsulInventory(object):
kv_groups config value and the node name add the node address to each
group found '''
node = node_data['Node']
- if self.config.has_config('kv_groups'):
+ if self.config.has_config('kv_groups'):
key = "%s/%s/%s" % (self.config.kv_groups, self.current_dc, node['Node'])
index, groups = self.consul_api.kv.get(key)
if groups and groups['Value']:
diff --git a/contrib/inventory/fleet.py b/contrib/inventory/fleet.py
index 788e1a5f51..4db4ea199c 100755
--- a/contrib/inventory/fleet.py
+++ b/contrib/inventory/fleet.py
@@ -77,7 +77,7 @@ def get_a_ssh_config(box_name):
if options.list:
ssh_config = get_ssh_config()
hosts = { 'coreos': []}
-
+
for data in ssh_config:
hosts['coreos'].append(data['Host'])
diff --git a/contrib/inventory/freeipa.py b/contrib/inventory/freeipa.py
index 377326c710..64ede999a4 100755
--- a/contrib/inventory/freeipa.py
+++ b/contrib/inventory/freeipa.py
@@ -18,7 +18,7 @@ def initialize():
except AttributeError:
#FreeIPA < 4.0 compatibility
api.Backend.xmlclient.connect()
-
+
return api
def list_groups(api):
@@ -39,7 +39,7 @@ def list_groups(api):
if 'member_host' in hostgroup:
members = [host for host in hostgroup['member_host']]
if 'memberindirect_host' in hostgroup:
- members += (host for host in hostgroup['memberindirect_host'])
+ members += (host for host in hostgroup['memberindirect_host'])
inventory[hostgroup['cn'][0]] = {'hosts': [host for host in members]}
for member in members:
@@ -48,7 +48,7 @@ def list_groups(api):
inventory['_meta'] = {'hostvars': hostvars}
inv_string = json.dumps(inventory, indent=1, sort_keys=True)
print(inv_string)
-
+
return None
def parse_args():
diff --git a/contrib/inventory/spacewalk.py b/contrib/inventory/spacewalk.py
index bee92dfb89..3d0005e5bb 100755
--- a/contrib/inventory/spacewalk.py
+++ b/contrib/inventory/spacewalk.py
@@ -220,7 +220,7 @@ elif options.host:
print('Problem executing the command "%s inventory": %s' %
(SW_REPORT, str(e)), file=sys.stderr)
sys.exit(2)
-
+
if options.human:
print('Host: %s' % options.host)
for k, v in iteritems(host_details):
diff --git a/contrib/inventory/stacki.py b/contrib/inventory/stacki.py
index 9dbabf8562..db90df7bef 100644
--- a/contrib/inventory/stacki.py
+++ b/contrib/inventory/stacki.py
@@ -58,7 +58,7 @@ try:
import requests
except:
sys.exit('requests package is required for this inventory script')
-
+
CONFIG_FILES = ['/etc/stacki/stacki.yml', '/etc/ansible/stacki.yml']
diff --git a/hacking/dump_playbook_attributes.py b/hacking/dump_playbook_attributes.py
index 8ccd6f752b..204fe3fd32 100755
--- a/hacking/dump_playbook_attributes.py
+++ b/hacking/dump_playbook_attributes.py
@@ -3,10 +3,10 @@
import optparse
from jinja2 import Environment, FileSystemLoader
-from ansible.playbook import Play
-from ansible.playbook.block import Block
-from ansible.playbook.role import Role
-from ansible.playbook.task import Task
+from ansible.playbook import Play
+from ansible.playbook.block import Block
+from ansible.playbook.role import Role
+from ansible.playbook.task import Task
template_file = 'playbooks_directives.rst.j2'
oblist = {}
diff --git a/lib/ansible/executor/stats.py b/lib/ansible/executor/stats.py
index 989a380be7..f91874d17a 100644
--- a/lib/ansible/executor/stats.py
+++ b/lib/ansible/executor/stats.py
@@ -59,7 +59,7 @@ class AggregateStats:
if host is None:
host = '_run'
- if host not in self.custom:
+ if host not in self.custom:
self.custom[host] = {which: what}
else:
self.custom[host][which] = what
diff --git a/lib/ansible/galaxy/login.py b/lib/ansible/galaxy/login.py
index 3dd90c84b1..29b6c094de 100644
--- a/lib/ansible/galaxy/login.py
+++ b/lib/ansible/galaxy/login.py
@@ -44,12 +44,12 @@ class GalaxyLogin(object):
''' Class to handle authenticating user with Galaxy API prior to performing CUD operations '''
GITHUB_AUTH = 'https://api.github.com/authorizations'
-
+
def __init__(self, galaxy, github_token=None):
self.galaxy = galaxy
self.github_username = None
self.github_password = None
-
+
if github_token == None:
self.get_credentials()
@@ -61,7 +61,7 @@ class GalaxyLogin(object):
display.display("The password will not be displayed." + u'\n\n', screen_only=True)
display.display("Use " + stringc("--github-token",'yellow') +
" if you do not want to enter your password." + u'\n\n', screen_only=True)
-
+
try:
self.github_username = raw_input("Github Username: ")
except:
diff --git a/lib/ansible/galaxy/role.py b/lib/ansible/galaxy/role.py
index da6743bd81..9883abdc8e 100644
--- a/lib/ansible/galaxy/role.py
+++ b/lib/ansible/galaxy/role.py
@@ -208,7 +208,7 @@ class GalaxyRole(object):
# create tar file from scm url
tmp_file = RoleRequirement.scm_archive_role(**self.spec)
elif self.src:
- if os.path.isfile(self.src):
+ if os.path.isfile(self.src):
# installing a local tar.gz
local_file = True
tmp_file = self.src
diff --git a/lib/ansible/galaxy/token.py b/lib/ansible/galaxy/token.py
index 4346dc8760..742c7b39f0 100644
--- a/lib/ansible/galaxy/token.py
+++ b/lib/ansible/galaxy/token.py
@@ -42,7 +42,7 @@ class GalaxyToken(object):
self.config = yaml.safe_load(self.__open_config_for_read())
if not self.config:
self.config = {}
-
+
def __open_config_for_read(self):
if os.path.isfile(self.file):
display.vvv('Opened %s' % self.file)
@@ -57,11 +57,11 @@ class GalaxyToken(object):
def set(self, token):
self.config['token'] = token
self.save()
-
+
def get(self):
return self.config.get('token', None)
-
+
def save(self):
with open(self.file,'w') as f:
yaml.safe_dump(self.config,f,default_flow_style=False)
-
+
diff --git a/lib/ansible/module_utils/azure_rm_common.py b/lib/ansible/module_utils/azure_rm_common.py
index 78ba496d5a..d646da93b0 100644
--- a/lib/ansible/module_utils/azure_rm_common.py
+++ b/lib/ansible/module_utils/azure_rm_common.py
@@ -333,7 +333,7 @@ class AzureRMModuleBase(object):
def _get_credentials(self, params):
# Get authentication credentials.
# Precedence: module parameters-> environment variables-> default profile in ~/.azure/credentials.
-
+
self.log('Getting credentials')
arg_credentials = dict()
@@ -345,11 +345,11 @@ class AzureRMModuleBase(object):
self.log('Retrieving credentials with profile parameter.')
credentials = self._get_profile(arg_credentials['profile'])
return credentials
-
+
if arg_credentials['subscription_id']:
self.log('Received credentials from parameters.')
return arg_credentials
-
+
# try environment
env_credentials = self._get_env_credentials()
if env_credentials:
diff --git a/lib/ansible/module_utils/facts.py b/lib/ansible/module_utils/facts.py
index 8cd4fff954..72c775c111 100644
--- a/lib/ansible/module_utils/facts.py
+++ b/lib/ansible/module_utils/facts.py
@@ -368,7 +368,7 @@ class Facts(object):
self.facts['service_mgr'] = proc_1_map.get(proc_1, proc_1)
# start with the easy ones
- elif self.facts['distribution'] == 'MacOSX':
+ elif self.facts['distribution'] == 'MacOSX':
#FIXME: find way to query executable, version matching is not ideal
if LooseVersion(platform.mac_ver()[0]) >= LooseVersion('10.4'):
self.facts['service_mgr'] = 'launchd'
@@ -2155,7 +2155,7 @@ class AIX(Hardware):
rc, out, err = self.module.run_command(cmd)
if rc == 0 and out:
pp_size = re.search(r'PP SIZE:\s+(\d+\s+\S+)',out).group(1)
- for n in re.finditer(r'(\S+)\s+(\w+)\s+(\d+)\s+(\d+).*',m.group(0)):
+ for n in re.finditer(r'(\S+)\s+(\w+)\s+(\d+)\s+(\d+).*',m.group(0)):
pv_info = { 'pv_name': n.group(1),
'pv_state': n.group(2),
'total_pps': n.group(3),
diff --git a/lib/ansible/module_utils/redhat.py b/lib/ansible/module_utils/redhat.py
index 905990cf43..ed2a92956f 100644
--- a/lib/ansible/module_utils/redhat.py
+++ b/lib/ansible/module_utils/redhat.py
@@ -127,7 +127,7 @@ class Rhsm(RegistrationBase):
for k,v in kwargs.items():
if re.search(r'^(system|rhsm)_', k):
args.append('--%s=%s' % (k.replace('_','.'), v))
-
+
self.module.run_command(args, check_rc=True)
@property
diff --git a/lib/ansible/modules/cloud/amazon/ec2_asg.py b/lib/ansible/modules/cloud/amazon/ec2_asg.py
index d1020c5cfd..888e7183c8 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_asg.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_asg.py
@@ -644,7 +644,7 @@ def replace(connection, module):
instances = props['instances']
if replace_instances:
instances = replace_instances
-
+
#check if min_size/max_size/desired capacity have been specified and if not use ASG values
if min_size is None:
min_size = as_group.min_size
@@ -674,7 +674,7 @@ def replace(connection, module):
if not old_instances:
changed = False
return(changed, props)
-
+
# set temporary settings and wait for them to be reached
# This should get overwritten if the number of instances left is less than the batch size.
@@ -827,7 +827,7 @@ def wait_for_term_inst(connection, module, term_instances):
lifecycle = instance_facts[i]['lifecycle_state']
health = instance_facts[i]['health_status']
log.debug("Instance {0} has state of {1},{2}".format(i,lifecycle,health ))
- if lifecycle == 'Terminating' or health == 'Unhealthy':
+ if lifecycle == 'Terminating' or health == 'Unhealthy':
count += 1
time.sleep(10)
diff --git a/lib/ansible/modules/cloud/amazon/ec2_customer_gateway.py b/lib/ansible/modules/cloud/amazon/ec2_customer_gateway.py
index a8a74926cd..631a07a9a5 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_customer_gateway.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_customer_gateway.py
@@ -148,7 +148,7 @@ class Ec2CustomerGatewayManager:
CustomerGatewayId=gw_id
)
return response
-
+
def ensure_cgw_present(self, bgp_asn, ip_address):
response = self.ec2.create_customer_gateway(
DryRun=False,
diff --git a/lib/ansible/modules/cloud/amazon/ec2_elb_facts.py b/lib/ansible/modules/cloud/amazon/ec2_elb_facts.py
index c4857f6a3c..710331fe80 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_elb_facts.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_elb_facts.py
@@ -215,7 +215,7 @@ class ElbInformation(object):
elb_array.append(existing_lb)
else:
elb_array = all_elbs
-
+
return list(map(self._get_elb_info, elb_array))
def main():
diff --git a/lib/ansible/modules/cloud/amazon/ec2_tag.py b/lib/ansible/modules/cloud/amazon/ec2_tag.py
index 7655dd5c36..13653d3111 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_tag.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_tag.py
@@ -143,14 +143,14 @@ def main():
resource = module.params.get('resource')
tags = module.params.get('tags')
state = module.params.get('state')
-
+
ec2 = ec2_connect(module)
-
+
# We need a comparison here so that we can accurately report back changed status.
# Need to expand the gettags return format and compare with "tags" and then tag or detag as appropriate.
filters = {'resource-id' : resource}
gettags = ec2.get_all_tags(filters=filters)
-
+
dictadd = {}
dictremove = {}
baddict = {}
@@ -170,7 +170,7 @@ def main():
tagger = ec2.create_tags(resource, dictadd)
gettags = ec2.get_all_tags(filters=filters)
module.exit_json(msg="Tags %s created for resource %s." % (dictadd,resource), changed=True)
-
+
if state == 'absent':
if not tags:
module.fail_json(msg="tags argument is required when state is absent")
diff --git a/lib/ansible/modules/cloud/amazon/ec2_vol_facts.py b/lib/ansible/modules/cloud/amazon/ec2_vol_facts.py
index d471fef31c..ba5350789a 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_vol_facts.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_vol_facts.py
@@ -97,7 +97,7 @@ def get_volume_info(volume):
},
'tags': volume.tags
}
-
+
return volume_info
def list_ec2_volumes(connection, module):
diff --git a/lib/ansible/modules/cloud/amazon/ec2_vpc_dhcp_options.py b/lib/ansible/modules/cloud/amazon/ec2_vpc_dhcp_options.py
index 4caee64451..15baf15142 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_vpc_dhcp_options.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_vpc_dhcp_options.py
@@ -289,7 +289,7 @@ def main():
changed = False
new_options = collections.defaultdict(lambda: None)
-
+
region, ec2_url, boto_params = get_aws_connection_info(module)
connection = connect_to_aws(boto.vpc, region, **boto_params)
@@ -378,9 +378,9 @@ def main():
# and remove old ones if that was requested
if params['delete_old'] and existing_options:
remove_dhcp_options_by_id(connection, existing_options.id)
-
+
module.exit_json(changed=changed, new_options=new_options, dhcp_options_id=dhcp_option.id)
-
+
from ansible.module_utils.basic import *
from ansible.module_utils.ec2 import *
diff --git a/lib/ansible/modules/cloud/amazon/ec2_vpc_net_facts.py b/lib/ansible/modules/cloud/amazon/ec2_vpc_net_facts.py
index 14e1c4920f..1915fd813e 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_vpc_net_facts.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_vpc_net_facts.py
@@ -72,7 +72,7 @@ def get_vpc_info(vpc):
classic_link = vpc.classic_link_enabled
except AttributeError:
classic_link = False
-
+
vpc_info = { 'id': vpc.id,
'instance_tenancy': vpc.instance_tenancy,
'classic_link_enabled': classic_link,
diff --git a/lib/ansible/modules/cloud/amazon/ec2_vpc_vgw_facts.py b/lib/ansible/modules/cloud/amazon/ec2_vpc_vgw_facts.py
index 6e378dc271..d6e92e112d 100644
--- a/lib/ansible/modules/cloud/amazon/ec2_vpc_vgw_facts.py
+++ b/lib/ansible/modules/cloud/amazon/ec2_vpc_vgw_facts.py
@@ -133,7 +133,7 @@ def list_virtual_gateways(client, module):
snaked_vgws = [camel_dict_to_snake_dict(get_virtual_gateway_info(vgw))
for vgw in all_virtual_gateways['VpnGateways']]
-
+
module.exit_json(virtual_gateways=snaked_vgws)
diff --git a/lib/ansible/modules/cloud/amazon/iam.py b/lib/ansible/modules/cloud/amazon/iam.py
index 79a3e34622..907c96af79 100644
--- a/lib/ansible/modules/cloud/amazon/iam.py
+++ b/lib/ansible/modules/cloud/amazon/iam.py
@@ -457,7 +457,7 @@ def delete_group(module=None, iam=None, name=None):
iam.delete_group_policy(name, policy)
try:
iam.delete_group(name)
- except boto.exception.BotoServerError as err:
+ except boto.exception.BotoServerError as err:
error_msg = boto_exception(err)
if ('must detach all policies first') in error_msg:
module.fail_json(changed=changed, msg="All inline polices have been removed. Though it appears"
diff --git a/lib/ansible/modules/cloud/amazon/rds.py b/lib/ansible/modules/cloud/amazon/rds.py
index d12593f320..04f6516b82 100644
--- a/lib/ansible/modules/cloud/amazon/rds.py
+++ b/lib/ansible/modules/cloud/amazon/rds.py
@@ -280,7 +280,7 @@ EXAMPLES = '''
command: reboot
instance_name: database
wait: yes
-
+
# Restore a Postgres db instance from a snapshot, wait for it to become available again, and
# then modify it to add your security group. Also, display the new endpoint.
# Note that the "publicly_accessible" option is allowed here just as it is in the AWS CLI
@@ -298,7 +298,7 @@ EXAMPLES = '''
tags:
Name: pg1_test_name_tag
register: rds
-
+
- local_action:
module: rds
command: modify
@@ -844,7 +844,7 @@ def promote_db_instance(module, conn):
valid_vars = ['backup_retention', 'backup_window']
params = validate_parameters(required_vars, valid_vars, module)
instance_name = module.params.get('instance_name')
-
+
result = conn.get_db_instance(instance_name)
if not result:
module.fail_json(msg="DB Instance %s does not exist" % instance_name)
diff --git a/lib/ansible/modules/cloud/amazon/rds_subnet_group.py b/lib/ansible/modules/cloud/amazon/rds_subnet_group.py
index d5ace785b7..cf7f7f8038 100644
--- a/lib/ansible/modules/cloud/amazon/rds_subnet_group.py
+++ b/lib/ansible/modules/cloud/amazon/rds_subnet_group.py
@@ -130,7 +130,7 @@ def main():
except BotoServerError as e:
if e.error_code != 'DBSubnetGroupNotFoundFault':
module.fail_json(msg = e.error_message)
-
+
if state == 'absent':
if exists:
conn.delete_db_subnet_group(group_name)
diff --git a/lib/ansible/modules/cloud/amazon/route53.py b/lib/ansible/modules/cloud/amazon/route53.py
index 6452fefb35..266456e15b 100644
--- a/lib/ansible/modules/cloud/amazon/route53.py
+++ b/lib/ansible/modules/cloud/amazon/route53.py
@@ -470,7 +470,7 @@ def main():
if command_in == 'create':
if ( weight_in!=None or region_in!=None or failover_in!=None ) and identifier_in==None:
module.fail_json(msg= "If you specify failover, region or weight you must also specify identifier")
- elif ( weight_in==None and region_in==None and failover_in==None ) and identifier_in!=None:
+ elif ( weight_in==None and region_in==None and failover_in==None ) and identifier_in!=None:
module.fail_json(msg= "You have specified identifier which makes sense only if you specify one of: weight, region or failover.")
diff --git a/lib/ansible/modules/cloud/amazon/s3_website.py b/lib/ansible/modules/cloud/amazon/s3_website.py
index b8e1503b2d..d81dcdf0a4 100644
--- a/lib/ansible/modules/cloud/amazon/s3_website.py
+++ b/lib/ansible/modules/cloud/amazon/s3_website.py
@@ -57,7 +57,7 @@ options:
- "Suffix that is appended to a request that is for a directory on the website endpoint (e.g. if the suffix is index.html and you make a request to samplebucket/images/ the data that is returned will be for the object with the key name images/index.html). The suffix must not include a slash character."
required: false
default: index.html
-
+
extends_documentation_fragment:
- aws
- ec2
@@ -76,14 +76,14 @@ EXAMPLES = '''
- s3_website:
name: mybucket.com
state: absent
-
+
# Configure an s3 bucket as a website with index and error pages
- s3_website:
name: mybucket.com
suffix: home.htm
error_key: errors/404.htm
state: present
-
+
'''
RETURN = '''
@@ -170,7 +170,7 @@ def _create_website_configuration(suffix, error_key, redirect_all_requests):
def enable_or_update_bucket_as_website(client_connection, resource_connection, module):
-
+
bucket_name = module.params.get("name")
redirect_all_requests = module.params.get("redirect_all_requests")
# If redirect_all_requests is set then don't use the default suffix that has been set
@@ -263,7 +263,7 @@ def main():
redirect_all_requests=dict(type='str', required=False)
)
)
-
+
module = AnsibleModule(
argument_spec=argument_spec,
mutually_exclusive = [
@@ -273,7 +273,7 @@ def main():
if not HAS_BOTO3:
module.fail_json(msg='boto3 required for this module')
-
+
region, ec2_url, aws_connect_params = get_aws_connection_info(module, boto3=True)
if region:
@@ -288,7 +288,7 @@ def main():
enable_or_update_bucket_as_website(client_connection, resource_connection, module)
elif state == 'absent':
disable_bucket_as_website(client_connection, module)
-
+
from ansible.module_utils.basic import *
from ansible.module_utils.ec2 import *
diff --git a/lib/ansible/modules/cloud/amazon/sqs_queue.py b/lib/ansible/modules/cloud/amazon/sqs_queue.py
index bad72f96bb..984a761ced 100644
--- a/lib/ansible/modules/cloud/amazon/sqs_queue.py
+++ b/lib/ansible/modules/cloud/amazon/sqs_queue.py
@@ -187,7 +187,7 @@ def create_or_update_sqs_queue(connection, module):
queue = connection.create_queue(queue_name)
update_sqs_queue(queue, **queue_attributes)
result['changed'] = True
-
+
if not module.check_mode:
result['queue_arn'] = queue.get_attributes('QueueArn')['QueueArn']
result['default_visibility_timeout'] = queue.get_attributes('VisibilityTimeout')['VisibilityTimeout']
@@ -195,7 +195,7 @@ def create_or_update_sqs_queue(connection, module):
result['maximum_message_size'] = queue.get_attributes('MaximumMessageSize')['MaximumMessageSize']
result['delivery_delay'] = queue.get_attributes('DelaySeconds')['DelaySeconds']
result['receive_message_wait_time'] = queue.get_attributes('ReceiveMessageWaitTimeSeconds')['ReceiveMessageWaitTimeSeconds']
-
+
except BotoServerError:
result['msg'] = 'Failed to create/update sqs queue due to error: ' + traceback.format_exc()
module.fail_json(**result)
diff --git a/lib/ansible/modules/cloud/azure/azure.py b/lib/ansible/modules/cloud/azure/azure.py
index aca6e719eb..870533c9ae 100644
--- a/lib/ansible/modules/cloud/azure/azure.py
+++ b/lib/ansible/modules/cloud/azure/azure.py
@@ -365,7 +365,7 @@ def create_virtual_machine(module, azure):
azure.get_role(name, name, name)
except AzureMissingException:
# vm does not exist; create it
-
+
if os_type == 'linux':
# Create linux configuration
disable_ssh_password_authentication = not password
@@ -563,7 +563,7 @@ def main():
cloud_service_raw = None
if module.params.get('state') == 'absent':
(changed, public_dns_name, deployment) = terminate_virtual_machine(module, azure)
-
+
elif module.params.get('state') == 'present':
# Changed is always set to true when provisioning new instances
if not module.params.get('name'):
diff --git a/lib/ansible/modules/cloud/azure/azure_rm_storageblob.py b/lib/ansible/modules/cloud/azure/azure_rm_storageblob.py
index 9ff235fd96..70ae31862d 100644
--- a/lib/ansible/modules/cloud/azure/azure_rm_storageblob.py
+++ b/lib/ansible/modules/cloud/azure/azure_rm_storageblob.py
@@ -372,7 +372,7 @@ class AzureRMStorageBlob(AzureRMModuleBase):
self.log('Create container %s' % self.container)
tags = None
- if not self.blob and self.tags:
+ if not self.blob and self.tags:
# when a blob is present, then tags are assigned at the blob level
tags = self.tags
diff --git a/lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py b/lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py
index 40010df8eb..132df72fd1 100644
--- a/lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py
+++ b/lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py
@@ -918,7 +918,7 @@ class AzureRMVirtualMachine(AzureRMModuleBase):
interface_dict['properties'] = nic_dict['properties']
# Expand public IPs to include config properties
- for interface in result['properties']['networkProfile']['networkInterfaces']:
+ for interface in result['properties']['networkProfile']['networkInterfaces']:
for config in interface['properties']['ipConfigurations']:
if config['properties'].get('publicIPAddress'):
pipid_dict = azure_id_to_dict(config['properties']['publicIPAddress']['id'])
diff --git a/lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py b/lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py
index 97494444ee..c5f52283ec 100644
--- a/lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py
+++ b/lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py
@@ -347,7 +347,7 @@ class AzureRMVirtualNetwork(AzureRMModuleBase):
try:
poller = self.network_client.virtual_networks.create_or_update(self.resource_group, self.name, vnet)
new_vnet = self.get_poller_result(poller)
- except Exception as exc:
+ except Exception as exc:
self.fail("Error creating or updating virtual network {0} - {1}".format(self.name, str(exc)))
return virtual_network_to_dict(new_vnet)
diff --git a/lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py b/lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py
index 2aee631395..868b6e7935 100644
--- a/lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py
+++ b/lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py
@@ -232,7 +232,7 @@ class AnsibleCloudStackInstanceFacts(AnsibleCloudStack):
if not instance:
self.module.fail_json(msg="Instance not found: %s" % self.module.params.get('name'))
self.facts['cloudstack_instance'] = self.get_result(instance)
- return self.facts
+ return self.facts
def get_result(self, instance):
diff --git a/lib/ansible/modules/cloud/docker/docker_image.py b/lib/ansible/modules/cloud/docker/docker_image.py
index 4643c63ed6..b91ee2096a 100644
--- a/lib/ansible/modules/cloud/docker/docker_image.py
+++ b/lib/ansible/modules/cloud/docker/docker_image.py
@@ -293,7 +293,7 @@ class ImageManager(DockerBaseClass):
if repo_tag:
self.name = repo
self.tag = repo_tag
-
+
if self.state in ['present', 'build']:
self.present()
elif self.state == 'absent':
diff --git a/lib/ansible/modules/cloud/docker/docker_service.py b/lib/ansible/modules/cloud/docker/docker_service.py
index ee7c35329f..537d60a348 100644
--- a/lib/ansible/modules/cloud/docker/docker_service.py
+++ b/lib/ansible/modules/cloud/docker/docker_service.py
@@ -639,7 +639,7 @@ class ContainerManager(DockerBaseClass):
return options
def cmd_up(self):
-
+
start_deps = self.dependencies
service_names = self.services
detached = True
@@ -943,7 +943,7 @@ class ContainerManager(DockerBaseClass):
short_id=container.short_id
))
result['actions'].append(service_res)
-
+
if not self.check_mode and result['changed']:
_, fd_name = tempfile.mkstemp(prefix="ansible")
try:
diff --git a/lib/ansible/modules/cloud/google/gc_storage.py b/lib/ansible/modules/cloud/google/gc_storage.py
index 7d78d0c505..98f0d1480d 100644
--- a/lib/ansible/modules/cloud/google/gc_storage.py
+++ b/lib/ansible/modules/cloud/google/gc_storage.py
@@ -165,7 +165,7 @@ def grant_check(module, gs, obj):
module.fail_json(msg= str(e))
return True
-
+
def key_check(module, gs, bucket, obj):
try:
@@ -228,7 +228,7 @@ def delete_key(module, gs, bucket, obj):
module.exit_json(msg="Object deleted from bucket ", changed=True)
except gs.provider.storage_response_error as e:
module.fail_json(msg= str(e))
-
+
def create_dirkey(module, gs, bucket, obj):
try:
bucket = gs.lookup(bucket)
@@ -325,7 +325,7 @@ def handle_put(module, gs, bucket, obj, overwrite, src, expiration):
module.exit_json(msg="WARNING: Checksums do not match. Use overwrite parameter to force upload.", failed=True)
else:
upload_gsfile(module, gs, bucket, obj, src, expiration)
-
+
if not bucket_rc:
create_bucket(module, gs, bucket)
upload_gsfile(module, gs, bucket, obj, src, expiration)
@@ -333,7 +333,7 @@ def handle_put(module, gs, bucket, obj, overwrite, src, expiration):
# If bucket exists but key doesn't, just upload.
if bucket_rc and not key_rc:
upload_gsfile(module, gs, bucket, obj, src, expiration)
-
+
def handle_delete(module, gs, bucket, obj):
if bucket and not obj:
if bucket_check(module, gs, bucket):
@@ -350,7 +350,7 @@ def handle_delete(module, gs, bucket, obj):
module.exit_json(msg="Bucket does not exist.", changed=False)
else:
module.fail_json(msg="Bucket or Bucket & object parameter is required.", failed=True)
-
+
def handle_create(module, gs, bucket, obj):
if bucket and not obj:
if bucket_check(module, gs, bucket):
@@ -417,7 +417,7 @@ def main():
gs = boto.connect_gs(gs_access_key, gs_secret_key)
except boto.exception.NoAuthHandlerFound as e:
module.fail_json(msg = str(e))
-
+
if mode == 'get':
if not bucket_check(module, gs, bucket) or not key_check(module, gs, bucket, obj):
module.fail_json(msg="Target bucket/key cannot be found", failed=True)
@@ -425,7 +425,7 @@ def main():
download_gsfile(module, gs, bucket, obj, dest)
else:
handle_get(module, gs, bucket, obj, overwrite, dest)
-
+
if mode == 'put':
if not path_check(src):
module.fail_json(msg="Local object for PUT does not exist", failed=True)
@@ -434,10 +434,10 @@ def main():
# Support for deleting an object if we have both params.
if mode == 'delete':
handle_delete(module, gs, bucket, obj)
-
+
if mode == 'create':
handle_create(module, gs, bucket, obj)
-
+
if mode == 'get_url':
if bucket and obj:
if bucket_check(module, gs, bucket) and key_check(module, gs, bucket, obj):
diff --git a/lib/ansible/modules/cloud/google/gcdns_zone.py b/lib/ansible/modules/cloud/google/gcdns_zone.py
index 683cb88189..4506bf1bda 100644
--- a/lib/ansible/modules/cloud/google/gcdns_zone.py
+++ b/lib/ansible/modules/cloud/google/gcdns_zone.py
@@ -93,7 +93,7 @@ EXAMPLES = '''
# Basic zone creation example.
- name: Create a basic zone with the minimum number of parameters.
gcdns_zone: zone=example.com
-
+
# Zone removal example.
- name: Remove a zone.
gcdns_zone: zone=example.com state=absent
diff --git a/lib/ansible/modules/cloud/google/gce_img.py b/lib/ansible/modules/cloud/google/gce_img.py
index e340808539..1c14a1421f 100644
--- a/lib/ansible/modules/cloud/google/gce_img.py
+++ b/lib/ansible/modules/cloud/google/gce_img.py
@@ -103,7 +103,7 @@ EXAMPLES = '''
- gce_img:
name: test-image
source: https://storage.googleapis.com/bucket/path/to/image.tgz
-
+
# Alternatively use the gs scheme
- gce_img:
name: test-image
diff --git a/lib/ansible/modules/cloud/google/gcpubsub.py b/lib/ansible/modules/cloud/google/gcpubsub.py
index 027629f740..d0c7b4c39a 100644
--- a/lib/ansible/modules/cloud/google/gcpubsub.py
+++ b/lib/ansible/modules/cloud/google/gcpubsub.py
@@ -194,7 +194,7 @@ def publish_messages(message_list, topic):
attrs = message['attributes']
batch.publish(bytes(msg), **attrs)
return True
-
+
def pull_messages(pull_params, sub):
"""
:rtype: tuple (output, changed)
@@ -203,7 +203,7 @@ def pull_messages(pull_params, sub):
max_messages=pull_params.get('max_messages', None)
message_ack = pull_params.get('message_ack', 'no')
return_immediately = pull_params.get('return_immediately', False)
-
+
output= []
pulled = sub.pull(return_immediately=return_immediately,
max_messages=max_messages)
@@ -237,7 +237,7 @@ def main():
if not HAS_PYTHON26:
module.fail_json(
msg="GCE module requires python's 'ast' module, python v2.6+")
-
+
if not HAS_GOOGLE_CLOUD_PUBSUB:
module.fail_json(msg="Please install google-cloud-pubsub library.")
diff --git a/lib/ansible/modules/cloud/google/gcpubsub_facts.py b/lib/ansible/modules/cloud/google/gcpubsub_facts.py
index 08a60841f3..8484b0b351 100644
--- a/lib/ansible/modules/cloud/google/gcpubsub_facts.py
+++ b/lib/ansible/modules/cloud/google/gcpubsub_facts.py
@@ -98,7 +98,7 @@ except ImportError as e:
def list_func(data, member='name'):
"""Used for state=list."""
return [getattr(x, member) for x in data]
-
+
def main():
module = AnsibleModule(argument_spec=dict(
@@ -112,7 +112,7 @@ def main():
if not HAS_PYTHON26:
module.fail_json(
msg="GCE module requires python's 'ast' module, python v2.6+")
-
+
if not HAS_GOOGLE_CLOUD_PUBSUB:
module.fail_json(msg="Please install google-cloud-pubsub library.")
diff --git a/lib/ansible/modules/cloud/linode/linode.py b/lib/ansible/modules/cloud/linode/linode.py
index 5b45267446..393ba0fe66 100644
--- a/lib/ansible/modules/cloud/linode/linode.py
+++ b/lib/ansible/modules/cloud/linode/linode.py
@@ -247,7 +247,7 @@ def linodeServers(module, api, state, name, plan, distribution, datacenter, lino
# - need linode_id (entity)
# - need disk_id for linode_id - create disk from distrib
# - need config_id for linode_id - create config (need kernel)
-
+
# Any create step triggers a job that need to be waited for.
if not servers:
for arg in (name, plan, distribution, datacenter):
@@ -424,7 +424,7 @@ def linodeServers(module, api, state, name, plan, distribution, datacenter, lino
instance['status'] = 'Restarting'
changed = True
instances.append(instance)
-
+
elif state in ('absent', 'deleted'):
for server in servers:
instance = getInstanceDetails(api, server)
diff --git a/lib/ansible/modules/cloud/misc/ovirt.py b/lib/ansible/modules/cloud/misc/ovirt.py
index af89998258..048bf34bc4 100644
--- a/lib/ansible/modules/cloud/misc/ovirt.py
+++ b/lib/ansible/modules/cloud/misc/ovirt.py
@@ -306,7 +306,7 @@ def create_vm(conn, vmtype, vmname, zone, vmdisk_size, vmcpus, vmnic, vmnetwork,
# define network parameters
network_net = params.Network(name=vmnetwork)
nic_net1 = params.NIC(name=vmnic, network=network_net, interface='virtio')
-
+
try:
conn.vms.add(vmparams)
except:
@@ -502,7 +502,7 @@ def main():
else:
vm_stop(c, vmname)
module.exit_json(changed=True, msg="VM %s is shutting down" % vmname)
-
+
if state == 'restart':
if vm_status(c, vmname) == 'up':
vm_restart(c, vmname)
diff --git a/lib/ansible/modules/cloud/misc/proxmox_kvm.py b/lib/ansible/modules/cloud/misc/proxmox_kvm.py
index 59e778aa33..9397d11ff1 100644
--- a/lib/ansible/modules/cloud/misc/proxmox_kvm.py
+++ b/lib/ansible/modules/cloud/misc/proxmox_kvm.py
@@ -1038,7 +1038,7 @@ def main():
time.sleep(1)
except Exception as e:
module.fail_json(msg="deletion of VM %s failed with exception: %s" % ( vmid, e ))
-
+
elif state == 'current':
status = {}
try:
diff --git a/lib/ansible/modules/cloud/misc/virt_net.py b/lib/ansible/modules/cloud/misc/virt_net.py
index 849e44a304..8e7df65e89 100644
--- a/lib/ansible/modules/cloud/misc/virt_net.py
+++ b/lib/ansible/modules/cloud/misc/virt_net.py
@@ -421,7 +421,7 @@ class VirtNetwork(object):
def create(self, entryid):
return self.conn.create(entryid)
-
+
def modify(self, entryid, xml):
return self.conn.modify(entryid, xml)
diff --git a/lib/ansible/modules/cloud/openstack/_quantum_router_gateway.py b/lib/ansible/modules/cloud/openstack/_quantum_router_gateway.py
index 6d0c715090..7568d297a6 100644
--- a/lib/ansible/modules/cloud/openstack/_quantum_router_gateway.py
+++ b/lib/ansible/modules/cloud/openstack/_quantum_router_gateway.py
@@ -180,7 +180,7 @@ def _add_gateway_router(neutron, module, router_id, network_id):
module.fail_json(msg = "Error in adding gateway to router: %s" % e.message)
return True
-def _remove_gateway_router(neutron, module, router_id):
+def _remove_gateway_router(neutron, module, router_id):
try:
neutron.remove_gateway_router(router_id)
except Exception as e:
diff --git a/lib/ansible/modules/cloud/openstack/_quantum_router_interface.py b/lib/ansible/modules/cloud/openstack/_quantum_router_interface.py
index 09de7dd901..9af05b1b44 100644
--- a/lib/ansible/modules/cloud/openstack/_quantum_router_interface.py
+++ b/lib/ansible/modules/cloud/openstack/_quantum_router_interface.py
@@ -193,7 +193,7 @@ def _get_port_id(neutron, module, router_id, subnet_id):
module.fail_json( msg = "Error in listing ports: %s" % e.message)
if not ports['ports']:
return None
- for port in ports['ports']:
+ for port in ports['ports']:
for subnet in port['fixed_ips']:
if subnet['subnet_id'] == subnet_id:
return port['id']
@@ -209,7 +209,7 @@ def _add_interface_router(neutron, module, router_id, subnet_id):
module.fail_json(msg = "Error in adding interface to router: %s" % e.message)
return True
-def _remove_interface_router(neutron, module, router_id, subnet_id):
+def _remove_interface_router(neutron, module, router_id, subnet_id):
kwargs = {
'subnet_id': subnet_id
}
diff --git a/lib/ansible/modules/cloud/openstack/os_zone.py b/lib/ansible/modules/cloud/openstack/os_zone.py
index a733d80ab2..51d59ec195 100644
--- a/lib/ansible/modules/cloud/openstack/os_zone.py
+++ b/lib/ansible/modules/cloud/openstack/os_zone.py
@@ -205,7 +205,7 @@ def main():
else:
if masters is None:
masters = []
-
+
pre_update_zone = zone
changed = _system_state_change(state, email,
description, ttl,
diff --git a/lib/ansible/modules/cloud/packet/packet_device.py b/lib/ansible/modules/cloud/packet/packet_device.py
index 7a6501013d..e1f34b1ed3 100644
--- a/lib/ansible/modules/cloud/packet/packet_device.py
+++ b/lib/ansible/modules/cloud/packet/packet_device.py
@@ -346,7 +346,7 @@ def get_hostname_list(module):
_msg = ("If you set count>1, you should only specify one hostname "
"with the %d formatter, not a list of hostnames.")
raise Exception(_msg)
-
+
if (len(hostnames) == 1) and (count > 0):
hostname_spec = hostnames[0]
count_range = range(count_offset, count_offset + count)
@@ -382,7 +382,7 @@ def get_device_id_list(module):
raise Exception("You specified too many devices, max is %d" %
MAX_DEVICES)
return device_ids
-
+
def create_single_device(module, packet_conn, hostname):
@@ -430,7 +430,7 @@ def wait_for_ips(module, packet_conn, created_devices):
if all_have_public_ip(refreshed):
return refreshed
time.sleep(5)
-
+
raise Exception("Waiting for IP assignment timed out. Hostnames: %s"
% [d.hostname for d in created_devices])
diff --git a/lib/ansible/modules/cloud/packet/packet_sshkey.py b/lib/ansible/modules/cloud/packet/packet_sshkey.py
index db82264869..6f362e2bd2 100644
--- a/lib/ansible/modules/cloud/packet/packet_sshkey.py
+++ b/lib/ansible/modules/cloud/packet/packet_sshkey.py
@@ -204,7 +204,7 @@ def act_on_sshkeys(target_state, module, packet_conn):
new_key_response = packet_conn.create_ssh_key(
newkey['label'], newkey['key'])
changed = True
-
+
matching_sshkeys.append(new_key_response)
else:
# state is 'absent' => delete mathcing keys
diff --git a/lib/ansible/modules/cloud/rackspace/rax_facts.py b/lib/ansible/modules/cloud/rackspace/rax_facts.py
index 8c49e8df98..b480fb2bcf 100644
--- a/lib/ansible/modules/cloud/rackspace/rax_facts.py
+++ b/lib/ansible/modules/cloud/rackspace/rax_facts.py
@@ -101,9 +101,9 @@ def rax_facts(module, address, name, server_id):
servers.append(cs.servers.get(server_id))
except Exception as e:
pass
-
+
servers[:] = [server for server in servers if server.status != "DELETED"]
-
+
if len(servers) > 1:
module.fail_json(msg='Multiple servers found matching provided '
'search parameters')
diff --git a/lib/ansible/modules/cloud/vmware/vmware_vswitch.py b/lib/ansible/modules/cloud/vmware/vmware_vswitch.py
index ef14f2d6bf..cbe5e37197 100644
--- a/lib/ansible/modules/cloud/vmware/vmware_vswitch.py
+++ b/lib/ansible/modules/cloud/vmware/vmware_vswitch.py
@@ -93,7 +93,7 @@ def find_vswitch_by_name(host, vswitch_name):
class VMwareHostVirtualSwitch(object):
-
+
def __init__(self, module):
self.host_system = None
self.content = None
@@ -132,7 +132,7 @@ class VMwareHostVirtualSwitch(object):
# Source from
# https://github.com/rreubenur/pyvmomi-community-samples/blob/patch-1/samples/create_vswitch.py
-
+
def state_create_vswitch(self):
vss_spec = vim.host.VirtualSwitch.Specification()
vss_spec.numPorts = self.number_of_ports
@@ -146,7 +146,7 @@ class VMwareHostVirtualSwitch(object):
def state_destroy_vswitch(self):
config = vim.host.NetworkConfig()
-
+
for portgroup in self.host_system.configManager.networkSystem.networkInfo.portgroup:
if portgroup.spec.vswitchName == self.vss.name:
portgroup_config = vim.host.PortGroup.Config()
@@ -158,7 +158,7 @@ class VMwareHostVirtualSwitch(object):
portgroup_config.spec.vswitchName = portgroup.spec.vswitchName
portgroup_config.spec.policy = vim.host.NetworkPolicy()
config.portgroup.append(portgroup_config)
-
+
self.host_system.configManager.networkSystem.UpdateNetworkConfig(config, "modify")
self.host_system.configManager.networkSystem.RemoveVirtualSwitch(self.vss.name)
self.module.exit_json(changed=True)
@@ -170,15 +170,15 @@ class VMwareHostVirtualSwitch(object):
host = get_all_objs(self.content, [vim.HostSystem])
if not host:
self.module.fail_json(msg="Unable to find host")
-
+
self.host_system = host.keys()[0]
self.vss = find_vswitch_by_name(self.host_system, self.switch_name)
-
+
if self.vss is None:
return 'absent'
else:
return 'present'
-
+
def main():
argument_spec = vmware_argument_spec()
diff --git a/lib/ansible/modules/cloud/vmware/vsphere_guest.py b/lib/ansible/modules/cloud/vmware/vsphere_guest.py
index e7de7625d4..6bd7405bbe 100644
--- a/lib/ansible/modules/cloud/vmware/vsphere_guest.py
+++ b/lib/ansible/modules/cloud/vmware/vsphere_guest.py
@@ -999,7 +999,7 @@ def reconfigure_vm(vsphere_client, vm, module, esxi, resource_pool, cluster_name
except (KeyError, ValueError):
vsphere_client.disconnect()
module.fail_json(msg="Error in '%s' definition. Size needs to be specified as an integer." % disk)
-
+
# Make sure the new disk size is higher than the current value
dev = dev_list[disk_num]
if disksize < int(dev.capacityInKB):
diff --git a/lib/ansible/modules/cloud/webfaction/webfaction_app.py b/lib/ansible/modules/cloud/webfaction/webfaction_app.py
index 63c00bd177..a967095522 100644
--- a/lib/ansible/modules/cloud/webfaction/webfaction_app.py
+++ b/lib/ansible/modules/cloud/webfaction/webfaction_app.py
@@ -148,7 +148,7 @@ def main():
existing_app = app_map.get(app_name)
result = {}
-
+
# Here's where the real stuff happens
if app_state == 'present':
diff --git a/lib/ansible/modules/cloud/webfaction/webfaction_db.py b/lib/ansible/modules/cloud/webfaction/webfaction_db.py
index 6fe785f76a..545176c4bf 100644
--- a/lib/ansible/modules/cloud/webfaction/webfaction_db.py
+++ b/lib/ansible/modules/cloud/webfaction/webfaction_db.py
@@ -84,7 +84,7 @@ options:
EXAMPLES = '''
# This will also create a default DB user with the same
# name as the database, and the specified password.
-
+
- name: Create a database
webfaction_db:
name: "{{webfaction_user}}_db1"
@@ -145,7 +145,7 @@ def main():
existing_user = user_map.get(db_name)
result = {}
-
+
# Here's where the real stuff happens
if db_state == 'present':
@@ -175,16 +175,16 @@ def main():
# If this isn't a dry run...
if not module.check_mode:
-
+
if not (existing_db or existing_user):
module.exit_json(changed = False,)
-
+
if existing_db:
# Delete the db if it exists
result.update(
webfaction.delete_db(session_id, db_name, db_type)
)
-
+
if existing_user:
# Delete the default db user if it exists
result.update(
diff --git a/lib/ansible/modules/cloud/webfaction/webfaction_domain.py b/lib/ansible/modules/cloud/webfaction/webfaction_domain.py
index 859209c9ce..8c145f18cb 100644
--- a/lib/ansible/modules/cloud/webfaction/webfaction_domain.py
+++ b/lib/ansible/modules/cloud/webfaction/webfaction_domain.py
@@ -121,7 +121,7 @@ def main():
existing_domain = domain_map.get(domain_name)
result = {}
-
+
# Here's where the real stuff happens
if domain_state == 'present':
diff --git a/lib/ansible/modules/cloud/webfaction/webfaction_mailbox.py b/lib/ansible/modules/cloud/webfaction/webfaction_mailbox.py
index 2132eeaffb..84d3763a89 100644
--- a/lib/ansible/modules/cloud/webfaction/webfaction_mailbox.py
+++ b/lib/ansible/modules/cloud/webfaction/webfaction_mailbox.py
@@ -107,7 +107,7 @@ def main():
existing_mailbox = mailbox_name in mailbox_list
result = {}
-
+
# Here's where the real stuff happens
if site_state == 'present':
diff --git a/lib/ansible/modules/cloud/webfaction/webfaction_site.py b/lib/ansible/modules/cloud/webfaction/webfaction_site.py
index 08a9b4d76d..9f415f5f3a 100644
--- a/lib/ansible/modules/cloud/webfaction/webfaction_site.py
+++ b/lib/ansible/modules/cloud/webfaction/webfaction_site.py
@@ -53,7 +53,7 @@ options:
required: false
choices: ['present', 'absent']
default: "present"
-
+
host:
description:
- The webfaction host on which the site should be created.
@@ -141,7 +141,7 @@ def main():
existing_site = site_map.get(site_name)
result = {}
-
+
# Here's where the real stuff happens
if site_state == 'present':
diff --git a/lib/ansible/modules/clustering/consul.py b/lib/ansible/modules/clustering/consul.py
index 203704efb9..8618dbbd4a 100644
--- a/lib/ansible/modules/clustering/consul.py
+++ b/lib/ansible/modules/clustering/consul.py
@@ -395,7 +395,7 @@ def parse_service(module):
module.fail_json( msg="service_name supplied but no service_port, a port is required to configure a service. Did you configure the 'port' argument meaning 'service_port'?")
-class ConsulService():
+class ConsulService():
def __init__(self, service_id=None, name=None, address=None, port=-1,
tags=None, loaded=None):
diff --git a/lib/ansible/modules/crypto/openssl_privatekey.py b/lib/ansible/modules/crypto/openssl_privatekey.py
index d643142c65..8f6ea5c70d 100644
--- a/lib/ansible/modules/crypto/openssl_privatekey.py
+++ b/lib/ansible/modules/crypto/openssl_privatekey.py
@@ -195,7 +195,7 @@ class PrivateKey(object):
}
return result
-
+
def main():
diff --git a/lib/ansible/modules/crypto/openssl_publickey.py b/lib/ansible/modules/crypto/openssl_publickey.py
index 6ac73dc975..e6fe182c24 100644
--- a/lib/ansible/modules/crypto/openssl_publickey.py
+++ b/lib/ansible/modules/crypto/openssl_publickey.py
@@ -163,7 +163,7 @@ class PublicKey(object):
}
return result
-
+
def main():
diff --git a/lib/ansible/modules/database/misc/elasticsearch_plugin.py b/lib/ansible/modules/database/misc/elasticsearch_plugin.py
index 8a16518962..d6ade97f91 100644
--- a/lib/ansible/modules/database/misc/elasticsearch_plugin.py
+++ b/lib/ansible/modules/database/misc/elasticsearch_plugin.py
@@ -161,7 +161,7 @@ def install_plugin(module, plugin_bin, plugin_name, version, url, proxy_host, pr
if rc != 0:
reason = parse_error(out)
module.fail_json(msg=reason)
-
+
return True, cmd, out, err
def remove_plugin(module, plugin_bin, plugin_name):
@@ -177,7 +177,7 @@ def remove_plugin(module, plugin_bin, plugin_name):
if rc != 0:
reason = parse_error(out)
module.fail_json(msg=reason)
-
+
return True, cmd, out, err
def main():
diff --git a/lib/ansible/modules/database/misc/kibana_plugin.py b/lib/ansible/modules/database/misc/kibana_plugin.py
index f3b20386f8..77c87b6c92 100644
--- a/lib/ansible/modules/database/misc/kibana_plugin.py
+++ b/lib/ansible/modules/database/misc/kibana_plugin.py
@@ -181,7 +181,7 @@ def install_plugin(module, plugin_bin, plugin_name, url, timeout):
if rc != 0:
reason = parse_error(out)
module.fail_json(msg=reason)
-
+
return True, cmd, out, err
def remove_plugin(module, plugin_bin, plugin_name):
@@ -191,12 +191,12 @@ def remove_plugin(module, plugin_bin, plugin_name):
if module.check_mode:
return True, cmd, "check mode", ""
-
+
rc, out, err = module.run_command(cmd)
if rc != 0:
reason = parse_error(out)
module.fail_json(msg=reason)
-
+
return True, cmd, out, err
def main():
diff --git a/lib/ansible/modules/database/misc/redis.py b/lib/ansible/modules/database/misc/redis.py
index f99d025742..078a46ea20 100644
--- a/lib/ansible/modules/database/misc/redis.py
+++ b/lib/ansible/modules/database/misc/redis.py
@@ -322,7 +322,7 @@ def main():
e = get_exception()
module.fail_json(msg="unable to connect to database: %s" % e)
-
+
try:
old_value = r.config_get(name)[name]
except Exception:
diff --git a/lib/ansible/modules/database/misc/riak.py b/lib/ansible/modules/database/misc/riak.py
index af4ec9489f..e35e08d925 100644
--- a/lib/ansible/modules/database/misc/riak.py
+++ b/lib/ansible/modules/database/misc/riak.py
@@ -179,7 +179,7 @@ def main():
ring_size = stats['ring_creation_size']
rc, out, err = module.run_command([riak_bin, 'version'] )
version = out.strip()
-
+
result = dict(node_name=node_name,
nodes=nodes,
ring_size=ring_size,
diff --git a/lib/ansible/modules/database/mssql/mssql_db.py b/lib/ansible/modules/database/mssql/mssql_db.py
index 2daf74d011..cbd9bf0378 100644
--- a/lib/ansible/modules/database/mssql/mssql_db.py
+++ b/lib/ansible/modules/database/mssql/mssql_db.py
@@ -181,7 +181,7 @@ def main():
login_password = module.params['login_password']
login_host = module.params['login_host']
login_port = module.params['login_port']
-
+
login_querystring = login_host
if login_port != "1433":
login_querystring = "%s:%s" % (login_host, login_port)
diff --git a/lib/ansible/modules/database/postgresql/postgresql_ext.py b/lib/ansible/modules/database/postgresql/postgresql_ext.py
index 51576eb1dc..f998786823 100644
--- a/lib/ansible/modules/database/postgresql/postgresql_ext.py
+++ b/lib/ansible/modules/database/postgresql/postgresql_ext.py
@@ -178,7 +178,7 @@ def main():
else:
if state == "absent":
changed = ext_delete(cursor, ext)
-
+
elif state == "present":
changed = ext_create(cursor, ext)
except NotSupportedError:
diff --git a/lib/ansible/modules/database/postgresql/postgresql_lang.py b/lib/ansible/modules/database/postgresql/postgresql_lang.py
index 91e5732fb3..650212f1cd 100644
--- a/lib/ansible/modules/database/postgresql/postgresql_lang.py
+++ b/lib/ansible/modules/database/postgresql/postgresql_lang.py
@@ -253,21 +253,21 @@ def main():
if lang_exists(cursor, lang):
lang_trusted = lang_istrusted(cursor, lang)
if (lang_trusted and not trust) or (not lang_trusted and trust):
- if module.check_mode:
+ if module.check_mode:
changed = True
else:
changed = lang_altertrust(cursor, lang, trust)
else:
- if module.check_mode:
+ if module.check_mode:
changed = True
else:
changed = lang_add(cursor, lang, trust)
if force_trust:
changed = lang_altertrust(cursor, lang, trust)
-
+
else:
if lang_exists(cursor, lang):
- if module.check_mode:
+ if module.check_mode:
changed = True
kw['lang_dropped'] = True
else:
diff --git a/lib/ansible/modules/database/postgresql/postgresql_user.py b/lib/ansible/modules/database/postgresql/postgresql_user.py
index 5376499fbe..e2df5a809a 100644
--- a/lib/ansible/modules/database/postgresql/postgresql_user.py
+++ b/lib/ansible/modules/database/postgresql/postgresql_user.py
@@ -688,7 +688,7 @@ def main():
module.fail_json(msg=str(e))
else:
if user_exists(cursor, user):
- if module.check_mode:
+ if module.check_mode:
changed = True
kw['user_removed'] = True
else:
diff --git a/lib/ansible/modules/database/vertica/vertica_configuration.py b/lib/ansible/modules/database/vertica/vertica_configuration.py
index c99627a021..3b018f449a 100644
--- a/lib/ansible/modules/database/vertica/vertica_configuration.py
+++ b/lib/ansible/modules/database/vertica/vertica_configuration.py
@@ -116,7 +116,7 @@ def get_configuration_facts(cursor, parameter_name=''):
'current_value': row.current_value,
'default_value': row.default_value}
return facts
-
+
def check(configuration_facts, parameter_name, current_value):
parameter_key = parameter_name.lower()
if current_value and current_value.lower() != configuration_facts[parameter_key]['current_value'].lower():
diff --git a/lib/ansible/modules/database/vertica/vertica_role.py b/lib/ansible/modules/database/vertica/vertica_role.py
index aff14581a3..f11efcffaa 100644
--- a/lib/ansible/modules/database/vertica/vertica_role.py
+++ b/lib/ansible/modules/database/vertica/vertica_role.py
@@ -135,7 +135,7 @@ def update_roles(role_facts, cursor, role,
cursor.execute("revoke {0} from {1}".format(assigned_role, role))
for assigned_role in set(required) - set(existing):
cursor.execute("grant {0} to {1}".format(assigned_role, role))
-
+
def check(role_facts, role, assigned_roles):
role_key = role.lower()
if role_key not in role_facts:
diff --git a/lib/ansible/modules/files/patch.py b/lib/ansible/modules/files/patch.py
index e8004b8c62..f09b7b625c 100644
--- a/lib/ansible/modules/files/patch.py
+++ b/lib/ansible/modules/files/patch.py
@@ -182,7 +182,7 @@ def main():
# patch need an absolute file name
p.src = os.path.abspath(p.src)
-
+
changed = False
if not is_already_applied(patch_func, p.src, p.basedir, dest_file=p.dest, binary=p.binary, strip=p.strip):
try:
diff --git a/lib/ansible/modules/monitoring/boundary_meter.py b/lib/ansible/modules/monitoring/boundary_meter.py
index 49b3d954bf..7ec661e769 100644
--- a/lib/ansible/modules/monitoring/boundary_meter.py
+++ b/lib/ansible/modules/monitoring/boundary_meter.py
@@ -121,12 +121,12 @@ def build_url(name, apiid, action, meter_id=None, cert_type=None):
elif action == "certificates":
return "https://%s/%s/meters/%s/%s.pem" % (api_host, apiid, meter_id, cert_type)
elif action == "tags":
- return "https://%s/%s/meters/%s/tags" % (api_host, apiid, meter_id)
+ return "https://%s/%s/meters/%s/tags" % (api_host, apiid, meter_id)
elif action == "delete":
- return "https://%s/%s/meters/%s" % (api_host, apiid, meter_id)
+ return "https://%s/%s/meters/%s" % (api_host, apiid, meter_id)
def http_request(module, name, apiid, apikey, action, data=None, meter_id=None, cert_type=None):
-
+
if meter_id is None:
url = build_url(name, apiid, action)
else:
diff --git a/lib/ansible/modules/monitoring/newrelic_deployment.py b/lib/ansible/modules/monitoring/newrelic_deployment.py
index c8f8703230..c671bb0d54 100644
--- a/lib/ansible/modules/monitoring/newrelic_deployment.py
+++ b/lib/ansible/modules/monitoring/newrelic_deployment.py
@@ -123,7 +123,7 @@ def main():
params["application_id"] = module.params["application_id"]
else:
module.fail_json(msg="you must set one of 'app_name' or 'application_id'")
-
+
for item in [ "changelog", "description", "revision", "user", "appname", "environment" ]:
if module.params[item]:
params[item] = module.params[item]
diff --git a/lib/ansible/modules/monitoring/pagerduty.py b/lib/ansible/modules/monitoring/pagerduty.py
index 43d93501c1..0331155a7b 100644
--- a/lib/ansible/modules/monitoring/pagerduty.py
+++ b/lib/ansible/modules/monitoring/pagerduty.py
@@ -209,7 +209,7 @@ def create(module, name, user, passwd, token, requester_id, service, hours, minu
'Content-Type' : 'application/json',
}
request_data = {'maintenance_window': {'start_time': start, 'end_time': end, 'description': desc, 'service_ids': service}}
-
+
if requester_id:
request_data['requester_id'] = requester_id
else:
@@ -235,7 +235,7 @@ def absent(module, name, user, passwd, token, requester_id, service):
'Content-Type' : 'application/json',
}
request_data = {}
-
+
if requester_id:
request_data['requester_id'] = requester_id
else:
diff --git a/lib/ansible/modules/monitoring/zabbix_host.py b/lib/ansible/modules/monitoring/zabbix_host.py
index 08e34c0e6b..3eef1d0bf1 100644
--- a/lib/ansible/modules/monitoring/zabbix_host.py
+++ b/lib/ansible/modules/monitoring/zabbix_host.py
@@ -376,10 +376,10 @@ class Host(object):
if host['proxy_hostid'] != proxy_id:
return True
-
+
if host['name'] != visible_name:
return True
-
+
return False
# link or clear template of the host
diff --git a/lib/ansible/modules/network/bigswitch/bigmon_chain.py b/lib/ansible/modules/network/bigswitch/bigmon_chain.py
index a5ec935301..6004578afa 100755
--- a/lib/ansible/modules/network/bigswitch/bigmon_chain.py
+++ b/lib/ansible/modules/network/bigswitch/bigmon_chain.py
@@ -123,7 +123,7 @@ def chain(module):
if state in ('absent') and not config_present:
module.exit_json(changed=False)
-
+
if state in ('present'):
response = rest.put('chain[name="%s"]' % name, data={'name': name})
if response.status_code == 204:
diff --git a/lib/ansible/modules/network/cloudflare_dns.py b/lib/ansible/modules/network/cloudflare_dns.py
index 621e92ac1f..81dda8bba7 100644
--- a/lib/ansible/modules/network/cloudflare_dns.py
+++ b/lib/ansible/modules/network/cloudflare_dns.py
@@ -391,7 +391,7 @@ class CloudflareAPI(object):
error_msg += "; Failed to parse API response: {0}".format(content)
# received an error status but no data with details on what failed
- if (info['status'] not in [200,304]) and (result is None):
+ if (info['status'] not in [200,304]) and (result is None):
self.module.fail_json(msg=error_msg)
if not result['success']:
diff --git a/lib/ansible/modules/network/dellos10/dellos10_facts.py b/lib/ansible/modules/network/dellos10/dellos10_facts.py
index 679f4e1060..58851d1f00 100644
--- a/lib/ansible/modules/network/dellos10/dellos10_facts.py
+++ b/lib/ansible/modules/network/dellos10/dellos10_facts.py
@@ -183,14 +183,14 @@ class Default(FactsBase):
return sw_name.text
else:
return ""
-
+
def parse_version(self, data):
sw_ver = data.find('./data/system-sw-state/sw-version/sw-version')
if sw_ver is not None:
return sw_ver.text
else:
return ""
-
+
def parse_hostname(self, data):
match = re.search(r'hostname\s+(\S+)', data, re.M)
if match:
@@ -224,7 +224,7 @@ class Hardware(FactsBase):
self.facts['cpu_arch'] = self.parse_cpu_arch(xml_data)
data = self.runner.get_command('show processes memory | grep Total')
-
+
match = self.parse_memory(data)
if match:
self.facts['memtotal_mb'] = int(match[0]) / 1024
@@ -236,7 +236,7 @@ class Hardware(FactsBase):
return cpu_arch.text
else:
return ""
-
+
def parse_memory(self, data):
return re.findall(r'\:\s*(\d+)', data, re.M)
@@ -372,7 +372,7 @@ class Interfaces(FactsBase):
lldp_facts[name].append(fact)
return lldp_facts
-
+
FACT_SUBSETS = dict(
default=Default,
hardware=Hardware,
diff --git a/lib/ansible/modules/network/dellos6/dellos6_facts.py b/lib/ansible/modules/network/dellos6/dellos6_facts.py
index 3ab0b2a992..243e04ad66 100644
--- a/lib/ansible/modules/network/dellos6/dellos6_facts.py
+++ b/lib/ansible/modules/network/dellos6/dellos6_facts.py
@@ -327,7 +327,7 @@ class Interfaces(FactsBase):
return match.group(3)
if flag==1:
return "null"
-
+
def parse_type(self, key, properties):
type_val, type_val_next = properties.split('--------- ------- --------------------- --------------------- --------------')
flag=1
diff --git a/lib/ansible/modules/network/dnsmadeeasy.py b/lib/ansible/modules/network/dnsmadeeasy.py
index 7650960e43..bba41159b9 100644
--- a/lib/ansible/modules/network/dnsmadeeasy.py
+++ b/lib/ansible/modules/network/dnsmadeeasy.py
@@ -31,25 +31,25 @@ options:
- Account API Key.
required: true
default: null
-
+
account_secret:
description:
- Account Secret Key.
required: true
default: null
-
+
domain:
description:
- Domain to work with. Can be the domain name (e.g. "mydomain.com") or the numeric ID of the domain in DNS Made Easy (e.g. "839989") for faster resolution.
required: true
default: null
-
+
record_name:
description:
- Record name to get/create/delete/update. If record_name is not specified; all records for the domain will be returned in "result" regardless of the state argument.
required: false
default: null
-
+
record_type:
description:
- Record type.
@@ -63,20 +63,20 @@ options:
- "If record_value is not specified; no changes will be made and the record will be returned in 'result' (in other words, this module can be used to fetch a record's current id, type, and ttl)"
required: false
default: null
-
+
record_ttl:
description:
- record's "Time to live". Number of seconds the record remains cached in DNS servers.
required: false
default: 1800
-
+
state:
description:
- whether the record should exist or not
required: true
choices: [ 'present', 'absent' ]
default: null
-
+
validate_certs:
description:
- If C(no), SSL certificates will not be validated. This should only be used
@@ -89,7 +89,7 @@ options:
notes:
- The DNS Made Easy service requires that machines interacting with the API have the proper time and timezone set. Be sure you are within a few seconds of actual time by using NTP.
- This module returns record(s) in the "result" element when 'state' is set to 'present'. This value can be be registered and used in your playbooks.
-
+
requirements: [ hashlib, hmac ]
author: "Brice Burgess (@briceburg)"
'''
@@ -102,7 +102,7 @@ EXAMPLES = '''
domain: my.com
state: present
register: response
-
+
# create / ensure the presence of a record
- dnsmadeeasy:
account_key: key
@@ -130,7 +130,7 @@ EXAMPLES = '''
state: present
record_name: test
register: response
-
+
# delete a record / ensure it is absent
- dnsmadeeasy:
account_key: key
diff --git a/lib/ansible/modules/network/eos/eos_eapi.py b/lib/ansible/modules/network/eos/eos_eapi.py
index 9a7a4197ae..d54af2935c 100644
--- a/lib/ansible/modules/network/eos/eos_eapi.py
+++ b/lib/ansible/modules/network/eos/eos_eapi.py
@@ -283,7 +283,7 @@ def map_config_to_obj(module):
'state': parse_state(out)
}
-def map_params_to_obj(module):
+def map_params_to_obj(module):
obj = {
'http': module.params['http'],
'http_port': module.params['http_port'],
@@ -310,7 +310,7 @@ def collect_facts(module, result):
for each in out[0]['urls']:
intf, url = each.split(' : ')
key = str(intf).strip()
- if key not in facts['eos_eapi_urls']:
+ if key not in facts['eos_eapi_urls']:
facts['eos_eapi_urls'][key] = list()
facts['eos_eapi_urls'][key].append(str(url).strip())
result['ansible_facts'] = facts
diff --git a/lib/ansible/modules/network/lldp.py b/lib/ansible/modules/network/lldp.py
index f222d765fe..f10c85acde 100644
--- a/lib/ansible/modules/network/lldp.py
+++ b/lib/ansible/modules/network/lldp.py
@@ -38,7 +38,7 @@ EXAMPLES = '''
# Retrieve switch/port information
- name: Gather information from lldp
lldp:
-
+
- name: Print each switch/port
debug:
msg: "{{ lldp[item]['chassis']['name'] }} / {{ lldp[item]['port']['ifalias'] }}"
@@ -73,7 +73,7 @@ def gather_lldp():
current_dict = current_dict[path_component]
current_dict[final] = value
return output_dict
-
+
def main():
module = AnsibleModule({})
@@ -84,7 +84,7 @@ def main():
module.exit_json(ansible_facts=data)
except TypeError:
module.fail_json(msg="lldpctl command failed. is lldpd running?")
-
+
# import module snippets
from ansible.module_utils.basic import *
diff --git a/lib/ansible/modules/network/nxos/nxos_aaa_server.py b/lib/ansible/modules/network/nxos/nxos_aaa_server.py
index 34f732993e..964854d073 100644
--- a/lib/ansible/modules/network/nxos/nxos_aaa_server.py
+++ b/lib/ansible/modules/network/nxos/nxos_aaa_server.py
@@ -492,7 +492,7 @@ def main():
)
module = get_network_module(argument_spec=argument_spec,
supports_check_mode=True)
-
+
server_type = module.params['server_type']
global_key = module.params['global_key']
encrypt_type = module.params['encrypt_type']
diff --git a/lib/ansible/modules/network/nxos/nxos_aaa_server_host.py b/lib/ansible/modules/network/nxos/nxos_aaa_server_host.py
index fd023adf9c..6efe25f6dd 100644
--- a/lib/ansible/modules/network/nxos/nxos_aaa_server_host.py
+++ b/lib/ansible/modules/network/nxos/nxos_aaa_server_host.py
@@ -428,7 +428,7 @@ def _match_dict(match_list, key_map):
def get_aaa_host_info(module, server_type, address):
aaa_host_info = {}
command = 'show run | inc {0}-server.host.{1}'.format(server_type, address)
-
+
body = execute_show_command(command, module, command_type='cli_show_ascii')
if body:
@@ -574,7 +574,7 @@ def main():
results['updates'] = cmds
results['changed'] = changed
results['end_state'] = end_state
-
+
module.exit_json(**results)
diff --git a/lib/ansible/modules/network/nxos/nxos_bgp_af.py b/lib/ansible/modules/network/nxos/nxos_bgp_af.py
index 11f1a9bfaf..a8ca2cd993 100644
--- a/lib/ansible/modules/network/nxos/nxos_bgp_af.py
+++ b/lib/ansible/modules/network/nxos/nxos_bgp_af.py
@@ -612,7 +612,7 @@ def get_custom_string_value(config, arg, module):
elif arg.startswith('dampening'):
REGEX = re.compile(r'(?:{0}\s)(?P<value>.*)$'.format(
PARAM_TO_COMMAND_KEYMAP[arg]), re.M)
- if arg == 'dampen_igp_metric' or arg == 'dampening_routemap':
+ if arg == 'dampen_igp_metric' or arg == 'dampening_routemap':
value = ''
if PARAM_TO_COMMAND_KEYMAP[arg] in config:
value = REGEX.search(config).group('value')
diff --git a/lib/ansible/modules/network/nxos/nxos_pim.py b/lib/ansible/modules/network/nxos/nxos_pim.py
index 41571522a3..c972705a99 100644
--- a/lib/ansible/modules/network/nxos/nxos_pim.py
+++ b/lib/ansible/modules/network/nxos/nxos_pim.py
@@ -295,7 +295,7 @@ def main():
supports_check_mode=True)
splitted_ssm_range = module.params['ssm_range'].split('.')
- if len(splitted_ssm_range) != 4 and module.params['ssm_range'] != 'none':
+ if len(splitted_ssm_range) != 4 and module.params['ssm_range'] != 'none':
module.fail_json(msg="Valid ssm_range values are multicast addresses "
"or the keyword 'none'.")
diff --git a/lib/ansible/modules/network/nxos/nxos_pim_interface.py b/lib/ansible/modules/network/nxos/nxos_pim_interface.py
index 60cabe5b41..4bf3a58625 100644
--- a/lib/ansible/modules/network/nxos/nxos_pim_interface.py
+++ b/lib/ansible/modules/network/nxos/nxos_pim_interface.py
@@ -495,10 +495,10 @@ def get_interface_mode(interface, intf_type, module):
def get_pim_interface(module, interface):
pim_interface = {}
command = 'show ip pim interface {0}'.format(interface)
-
+
body = execute_show_command(command, module,
command_type='cli_show_ascii', text=True)
-
+
if body:
if 'not running' not in body[0]:
body = execute_show_command(command, module)
@@ -552,7 +552,7 @@ def get_pim_interface(module, interface):
return {}
command = 'show run interface {0}'.format(interface)
-
+
body = execute_show_command(command, module, command_type='cli_show_ascii')
jp_configs = []
diff --git a/lib/ansible/modules/network/nxos/nxos_snmp_location.py b/lib/ansible/modules/network/nxos/nxos_snmp_location.py
index e3b90973e7..193e6afb1f 100644
--- a/lib/ansible/modules/network/nxos/nxos_snmp_location.py
+++ b/lib/ansible/modules/network/nxos/nxos_snmp_location.py
@@ -378,7 +378,7 @@ def main():
)
module = get_network_module(argument_spec=argument_spec,
supports_check_mode=True)
-
+
location = module.params['location']
state = module.params['state']
diff --git a/lib/ansible/modules/network/nxos/nxos_snmp_traps.py b/lib/ansible/modules/network/nxos/nxos_snmp_traps.py
index a3a88191ff..cbc7a501be 100644
--- a/lib/ansible/modules/network/nxos/nxos_snmp_traps.py
+++ b/lib/ansible/modules/network/nxos/nxos_snmp_traps.py
@@ -463,7 +463,7 @@ def main():
)
module = get_network_module(argument_spec=argument_spec,
supports_check_mode=True)
-
+
group = module.params['group'].lower()
state = module.params['state']
diff --git a/lib/ansible/modules/network/wakeonlan.py b/lib/ansible/modules/network/wakeonlan.py
index e5a38c20df..08bd3b5d80 100644
--- a/lib/ansible/modules/network/wakeonlan.py
+++ b/lib/ansible/modules/network/wakeonlan.py
@@ -97,7 +97,7 @@ def wakeonlan(module, mac, broadcast, port):
int(mac, 16)
except ValueError:
module.fail_json(msg="Incorrect MAC address format: %s" % mac_orig)
-
+
# Create payload for magic packet
data = ''
padding = ''.join(['FFFFFFFFFFFF', mac * 20])
diff --git a/lib/ansible/modules/notification/flowdock.py b/lib/ansible/modules/notification/flowdock.py
index e0584295af..2d133f9ded 100644
--- a/lib/ansible/modules/notification/flowdock.py
+++ b/lib/ansible/modules/notification/flowdock.py
@@ -142,7 +142,7 @@ def main():
url = "https://api.flowdock.com/v1/messages/team_inbox/%s" % (token)
else:
url = "https://api.flowdock.com/v1/messages/chat/%s" % (token)
-
+
params = {}
# required params
diff --git a/lib/ansible/modules/packaging/os/apt.py b/lib/ansible/modules/packaging/os/apt.py
index 5637680cd9..796ef979c8 100644
--- a/lib/ansible/modules/packaging/os/apt.py
+++ b/lib/ansible/modules/packaging/os/apt.py
@@ -782,7 +782,7 @@ def get_cache(module):
else:
module.fail_json(msg=str(e))
return cache
-
+
def main():
module = AnsibleModule(
diff --git a/lib/ansible/modules/packaging/os/apt_rpm.py b/lib/ansible/modules/packaging/os/apt_rpm.py
index fe7a78990f..e0d0e2ca50 100755
--- a/lib/ansible/modules/packaging/os/apt_rpm.py
+++ b/lib/ansible/modules/packaging/os/apt_rpm.py
@@ -110,7 +110,7 @@ def update_package_db(module):
module.fail_json(msg="could not update package db: %s" % err)
def remove_packages(module, packages):
-
+
remove_c = 0
# Using a for loop in case of error, we can report the package that failed
for package in packages:
@@ -122,7 +122,7 @@ def remove_packages(module, packages):
if rc != 0:
module.fail_json(msg="failed to remove %s: %s" % (package, err))
-
+
remove_c += 1
if remove_c > 0:
@@ -162,7 +162,7 @@ def main():
state = dict(default='installed', choices=['installed', 'removed', 'absent', 'present']),
update_cache = dict(default=False, aliases=['update-cache'], type='bool'),
package = dict(aliases=['pkg', 'name'], required=True)))
-
+
if not os.path.exists(APT_PATH) or not os.path.exists(RPM_PATH):
module.fail_json(msg="cannot find /usr/bin/apt-get and/or /usr/bin/rpm")
@@ -182,6 +182,6 @@ def main():
# this is magic, see lib/ansible/module_common.py
from ansible.module_utils.basic import *
-
+
if __name__ == '__main__':
main()
diff --git a/lib/ansible/modules/packaging/os/layman.py b/lib/ansible/modules/packaging/os/layman.py
index 440001b48a..f18189f69c 100644
--- a/lib/ansible/modules/packaging/os/layman.py
+++ b/lib/ansible/modules/packaging/os/layman.py
@@ -158,7 +158,7 @@ def install_overlay(module, name, list_url=None):
if layman.is_installed(name):
return False
-
+
if module.check_mode:
mymsg = 'Would add layman repo \'' + name + '\''
module.exit_json(changed=True, msg=mymsg)
@@ -195,7 +195,7 @@ def uninstall_overlay(module, name):
if not layman.is_installed(name):
return False
-
+
if module.check_mode:
mymsg = 'Would remove layman repo \'' + name + '\''
module.exit_json(changed=True, msg=mymsg)
diff --git a/lib/ansible/modules/packaging/os/rhn_channel.py b/lib/ansible/modules/packaging/os/rhn_channel.py
index c2e87f1bd9..5f35a3f78a 100644
--- a/lib/ansible/modules/packaging/os/rhn_channel.py
+++ b/lib/ansible/modules/packaging/os/rhn_channel.py
@@ -144,18 +144,18 @@ def main():
saturl = module.params['url']
user = module.params['user']
password = module.params['password']
-
+
#initialize connection
client = xmlrpclib.Server(saturl, verbose=0)
session = client.auth.login(user, password)
-
+
# get systemid
sys_id = get_systemid(client, session, systname)
# get channels for system
chans = base_channels(client, session, sys_id)
-
-
+
+
if state == 'present':
if channelname in chans:
module.exit_json(changed=False, msg="Channel %s already exists" % channelname)
diff --git a/lib/ansible/modules/packaging/os/urpmi.py b/lib/ansible/modules/packaging/os/urpmi.py
index beae17d13d..597fc2eaa5 100644
--- a/lib/ansible/modules/packaging/os/urpmi.py
+++ b/lib/ansible/modules/packaging/os/urpmi.py
@@ -119,10 +119,10 @@ def update_package_db(module):
rc, stdout, stderr = module.run_command(cmd, check_rc=False)
if rc != 0:
module.fail_json(msg="could not update package db")
-
+
def remove_packages(module, packages):
-
+
remove_c = 0
# Using a for loop in case of error, we can report the package that failed
for package in packages:
@@ -135,7 +135,7 @@ def remove_packages(module, packages):
if rc != 0:
module.fail_json(msg="failed to remove %s" % (package))
-
+
remove_c += 1
if remove_c > 0:
@@ -189,7 +189,7 @@ def main():
force = dict(default=True, type='bool'),
no_recommends = dict(default=True, aliases=['no-recommends'], type='bool'),
package = dict(aliases=['pkg', 'name'], required=True)))
-
+
if not os.path.exists(URPMI_PATH):
module.fail_json(msg="cannot find urpmi, looking for %s" % (URPMI_PATH))
@@ -212,6 +212,6 @@ def main():
# import module snippets
from ansible.module_utils.basic import *
-
+
if __name__ == '__main__':
main()
diff --git a/lib/ansible/modules/packaging/os/yum.py b/lib/ansible/modules/packaging/os/yum.py
index 8a626d7c32..6f42b96394 100644
--- a/lib/ansible/modules/packaging/os/yum.py
+++ b/lib/ansible/modules/packaging/os/yum.py
@@ -390,12 +390,12 @@ def is_available(module, repoq, pkgspec, conf_file, qf=def_qf, en_repos=None, di
except Exception:
e = get_exception()
module.fail_json(msg="Failure talking to yum: %s" % e)
-
+
return [ po_to_nevra(p) for p in pkgs ]
else:
myrepoq = list(repoq)
-
+
r_cmd = ['--disablerepo', ','.join(dis_repos)]
myrepoq.extend(r_cmd)
@@ -442,7 +442,7 @@ def is_update(module, repoq, pkgspec, conf_file, qf=def_qf, en_repos=None, dis_r
for pkg in pkgs:
if pkg in updates:
retpkgs.append(pkg)
-
+
return set([ po_to_nevra(p) for p in retpkgs ])
else:
@@ -455,12 +455,12 @@ def is_update(module, repoq, pkgspec, conf_file, qf=def_qf, en_repos=None, dis_r
cmd = myrepoq + ["--pkgnarrow=updates", "--qf", qf, pkgspec]
rc,out,err = module.run_command(cmd)
-
+
if rc == 0:
return set([ p for p in out.split('\n') if p.strip() ])
else:
module.fail_json(msg='Error from repoquery: %s: %s' % (cmd, err))
-
+
return set()
def what_provides(module, repoq, req_spec, conf_file, qf=def_qf, en_repos=None, dis_repos=None, installroot='/'):
@@ -725,7 +725,7 @@ def install(module, items, repoq, yum_basecmd, conf_file, en_repos, dis_repos, i
if is_installed(module, repoq, spec, conf_file, en_repos=en_repos, dis_repos=dis_repos, installroot=installroot):
found = True
res['results'].append('package providing %s is already installed' % (spec))
-
+
if found:
continue
diff --git a/lib/ansible/modules/packaging/os/zypper.py b/lib/ansible/modules/packaging/os/zypper.py
index 8628d52c27..5b36489d5a 100644
--- a/lib/ansible/modules/packaging/os/zypper.py
+++ b/lib/ansible/modules/packaging/os/zypper.py
@@ -373,7 +373,7 @@ def package_present(m, name, want_latest):
def package_update_all(m):
"run update or patch on all available packages"
-
+
retvals = {'rc': 0, 'stdout': '', 'stderr': ''}
if m.params['type'] == 'patch':
cmdname = 'patch'
diff --git a/lib/ansible/modules/packaging/os/zypper_repository.py b/lib/ansible/modules/packaging/os/zypper_repository.py
index 97ca36100b..adad2bc195 100644
--- a/lib/ansible/modules/packaging/os/zypper_repository.py
+++ b/lib/ansible/modules/packaging/os/zypper_repository.py
@@ -142,7 +142,7 @@ EXAMPLES = '''
- zypper_repository:
repo: 'http://download.opensuse.org/repositories/systemsmanagement/openSUSE_Leap_42.1/'
auto_import_keys: yes
-
+
# Force refresh of a repository
- zypper_repository:
repo: 'http://my_internal_ci_repo/repo
diff --git a/lib/ansible/modules/remote_management/foreman/katello.py b/lib/ansible/modules/remote_management/foreman/katello.py
index 5679ad229d..6d2bf446ae 100644
--- a/lib/ansible/modules/remote_management/foreman/katello.py
+++ b/lib/ansible/modules/remote_management/foreman/katello.py
@@ -243,7 +243,7 @@ class NailGun(object):
return True
except Exception:
e = get_exception()
-
+
if "Import is the same as existing data" in e.message:
return False
else:
@@ -263,7 +263,7 @@ class NailGun(object):
product.create()
return True
-
+
def sync_product(self, params):
org = self.find_organization(params['organization'])
product = self.find_product(params['name'], org.name)
@@ -287,7 +287,7 @@ class NailGun(object):
repository.create()
return True
-
+
def sync_repository(self, params):
org = self.find_organization(params['organization'])
repository = self.find_repository(params['name'], params['product'], org.name)
@@ -357,7 +357,7 @@ class NailGun(object):
for name in products:
product = self.find_product(name, org.name)
ids.append(product.id)
-
+
sync_plan.add_products(data={'product_ids': ids})
return True
@@ -373,7 +373,7 @@ class NailGun(object):
content_view.update()
else:
content_view = content_view.create()
-
+
if params['repositories']:
repos = []
diff --git a/lib/ansible/modules/remote_management/stacki/stacki_host.py b/lib/ansible/modules/remote_management/stacki/stacki_host.py
index e4d95febc1..cf971d37e4 100644
--- a/lib/ansible/modules/remote_management/stacki/stacki_host.py
+++ b/lib/ansible/modules/remote_management/stacki/stacki_host.py
@@ -127,7 +127,7 @@ class StackiHost:
auth_creds = {'USERNAME': module.params['stacki_user'],
'PASSWORD': module.params['stacki_password']}
-
+
# Get Intial CSRF
cred_a = self.do_request(self.module, self.endpoint, method="GET")
cookie_a = cred_a.headers.get('Set-Cookie').split(';')
@@ -144,7 +144,7 @@ class StackiHost:
# Endpoint to get final authentication header
login_endpoint = self.endpoint + "/login"
-
+
# Get Final CSRF and Session ID
login_req = self.do_request(self.module, login_endpoint, headers=header,
payload=urllib.urlencode(auth_creds), method="POST")
@@ -175,41 +175,41 @@ class StackiHost:
def stack_check_host(self):
-
+
res = self.do_request(self.module, self.endpoint, payload=json.dumps({"cmd": "list host"}),
headers=self.header, method="POST")
-
+
if self.hostname in res.read():
return True
else:
return False
-
-
+
+
def stack_sync(self):
-
+
res = self.do_request(self.module, self.endpoint, payload=json.dumps({ "cmd": "sync config"}),
headers=self.header, method="POST")
-
+
res = self.do_request(self.module, self.endpoint, payload=json.dumps({"cmd": "sync host config"}),
headers=self.header, method="POST")
-
+
def stack_force_install(self):
-
+
data = dict()
changed = False
-
+
data['cmd'] = "set host boot {0} action=install" \
.format(self.hostname)
res = self.do_request(self.module, self.endpoint, payload=json.dumps(data),
headers=self.header, method="POST")
changed = True
-
+
self.stack_sync()
-
+
result['changed'] = changed
result['stdout'] = "api call successful".rstrip("\r\n")
-
+
def stack_add_interface(self):
@@ -217,35 +217,35 @@ class StackiHost:
.format(self.hostname, self.prim_intf, self.prim_intf_ip, self.network, self.prim_intf_mac)
res = self.do_request(self.module, self.endpoint, payload=json.dumps(data),
headers=self.header, method="POST")
-
+
def stack_add(self, result):
-
+
data = dict()
changed = False
-
+
data['cmd'] = "add host {0} rack={1} rank={2} appliance={3}"\
.format(self.hostname, self.rack, self.rank, self.appliance)
res = self.do_request(self.module, self.endpoint, payload=json.dumps(data),
headers=self.header, method="POST")
-
+
self.stack_sync()
-
+
result['changed'] = changed
result['stdout'] = "api call successful".rstrip("\r\n")
-
-
+
+
def stack_remove(self, result):
-
+
data = dict()
-
+
data['cmd'] = "remove host {0}"\
.format(self.hostname)
res = self.do_request(self.module, self.endpoint, payload=json.dumps(data),
headers=self.header, method="POST")
-
+
self.stack_sync()
-
+
result['changed'] = True
result['stdout'] = "api call successful".rstrip("\r\n")
diff --git a/lib/ansible/modules/source_control/github_hooks.py b/lib/ansible/modules/source_control/github_hooks.py
index 37ed698261..56bf49fdb6 100644
--- a/lib/ansible/modules/source_control/github_hooks.py
+++ b/lib/ansible/modules/source_control/github_hooks.py
@@ -118,7 +118,7 @@ def _clean504(module, hookurl, oauthkey, repo, user):
# print "Last response was an ERROR for hook:"
# print hook['id']
_delete(module, hookurl, oauthkey, repo, user, hook['id'])
-
+
return 0, current_hooks
def _cleanall(module, hookurl, oauthkey, repo, user):
@@ -130,7 +130,7 @@ def _cleanall(module, hookurl, oauthkey, repo, user):
# print "Last response was an ERROR for hook:"
# print hook['id']
_delete(module, hookurl, oauthkey, repo, user, hook['id'])
-
+
return 0, current_hooks
def _create(module, hookurl, oauthkey, repo, user, content_type):
diff --git a/lib/ansible/modules/system/group.py b/lib/ansible/modules/system/group.py
index 132d71ad2d..4242e6e008 100644
--- a/lib/ansible/modules/system/group.py
+++ b/lib/ansible/modules/system/group.py
@@ -302,7 +302,7 @@ class DarwinGroup(Group):
(rc, out, err) = self.execute_command(cmd)
return (rc, out, err)
return (None, '', '')
-
+
def get_lowest_available_system_gid(self):
# check for lowest available system gid (< 500)
try:
diff --git a/lib/ansible/modules/system/locale_gen.py b/lib/ansible/modules/system/locale_gen.py
index 8e6a1835e2..ad94f8093f 100644
--- a/lib/ansible/modules/system/locale_gen.py
+++ b/lib/ansible/modules/system/locale_gen.py
@@ -154,14 +154,14 @@ def apply_change(targetState, name):
else:
# Delete locale.
set_locale(name, enabled=False)
-
+
localeGenExitValue = call("locale-gen")
if localeGenExitValue!=0:
raise EnvironmentError(localeGenExitValue, "locale.gen failed to execute, it returned "+str(localeGenExitValue))
def apply_change_ubuntu(targetState, name):
"""Create or remove locale.
-
+
Keyword arguments:
targetState -- Desired state, either present or absent.
name -- Name including encoding such as de_CH.UTF-8.
@@ -188,7 +188,7 @@ def apply_change_ubuntu(targetState, name):
# Purge locales and regenerate.
# Please provide a patch if you know how to avoid regenerating the locales to keep!
localeGenExitValue = call(["locale-gen", "--purge"])
-
+
if localeGenExitValue!=0:
raise EnvironmentError(localeGenExitValue, "locale.gen failed to execute, it returned "+str(localeGenExitValue))
@@ -227,7 +227,7 @@ def main():
else:
prev_state = "absent"
changed = (prev_state!=state)
-
+
if module.check_mode:
module.exit_json(changed=changed)
else:
diff --git a/lib/ansible/modules/system/pam_limits.py b/lib/ansible/modules/system/pam_limits.py
index ab508c5727..f730924d9a 100644
--- a/lib/ansible/modules/system/pam_limits.py
+++ b/lib/ansible/modules/system/pam_limits.py
@@ -272,7 +272,7 @@ def main():
nf.close()
except:
pass
-
+
res_args = dict(
changed = changed, msg = message
)
diff --git a/lib/ansible/modules/system/sysctl.py b/lib/ansible/modules/system/sysctl.py
index a4031a6b2a..8a270b4387 100644
--- a/lib/ansible/modules/system/sysctl.py
+++ b/lib/ansible/modules/system/sysctl.py
@@ -279,7 +279,7 @@ class SysctlModule(object):
sysctl_args = [self.sysctl_cmd, '-p', self.sysctl_file]
if self.args['ignoreerrors']:
sysctl_args.insert(1, '-e')
-
+
rc,out,err = self.module.run_command(sysctl_args)
if rc != 0:
diff --git a/lib/ansible/modules/web_infrastructure/jira.py b/lib/ansible/modules/web_infrastructure/jira.py
index 7e58b5cfb1..64877f0f6e 100755
--- a/lib/ansible/modules/web_infrastructure/jira.py
+++ b/lib/ansible/modules/web_infrastructure/jira.py
@@ -417,7 +417,7 @@ def main():
# Dispatch
try:
-
+
# Lookup the corresponding method for this operation. This is
# safe as the AnsibleModule should remove any unknown operations.
thismod = sys.modules[__name__]
diff --git a/lib/ansible/modules/web_infrastructure/letsencrypt.py b/lib/ansible/modules/web_infrastructure/letsencrypt.py
index 7e8f76329d..0027cb6e70 100644
--- a/lib/ansible/modules/web_infrastructure/letsencrypt.py
+++ b/lib/ansible/modules/web_infrastructure/letsencrypt.py
@@ -775,7 +775,7 @@ def main():
),
supports_check_mode = True,
)
-
+
# AnsibleModule() changes the locale, so change it back to C because we rely on time.strptime() when parsing certificate dates.
locale.setlocale(locale.LC_ALL, "C")
diff --git a/lib/ansible/modules/windows/win_package.py b/lib/ansible/modules/windows/win_package.py
index bea20fe675..f66364aeb7 100644
--- a/lib/ansible/modules/windows/win_package.py
+++ b/lib/ansible/modules/windows/win_package.py
@@ -100,7 +100,7 @@ EXAMPLES = r'''
path: https://download.microsoft.com/download/A/F/0/AF0071F3-B198-4A35-AA90-C68D103BDCCF/rdcman.msi
product_id: '{0240359E-6A4C-4884-9E94-B397A02D893C}'
state: absent
-
+
# Specify the expected non-zero return code when successful
# In this case 3010 indicates 'reboot required'
- name: 'Microsoft .NET Framework 4.5.1'
diff --git a/lib/ansible/modules/windows/win_stat.py b/lib/ansible/modules/windows/win_stat.py
index 73c8e43683..5f7140fbc2 100644
--- a/lib/ansible/modules/windows/win_stat.py
+++ b/lib/ansible/modules/windows/win_stat.py
@@ -192,7 +192,7 @@ stat:
type: string
sample: BUILTIN\\Administrators
sharename:
- description: the name of share if folder is shared
+ description: the name of share if folder is shared
returned: success, path exists, file is a directory and isshared == True
type: string
sample: file-share
diff --git a/lib/ansible/playbook/role/metadata.py b/lib/ansible/playbook/role/metadata.py
index dfbe20b632..d62f2526ac 100644
--- a/lib/ansible/playbook/role/metadata.py
+++ b/lib/ansible/playbook/role/metadata.py
@@ -69,7 +69,7 @@ class RoleMetadata(Base):
if ds:
if not isinstance(ds, list):
raise AnsibleParserError("Expected role dependencies to be a list.", obj=self._ds)
-
+
for role_def in ds:
if isinstance(role_def, string_types) or 'role' in role_def or 'name' in role_def:
roles.append(role_def)
@@ -86,7 +86,7 @@ class RoleMetadata(Base):
current_role_path = None
if self._owner:
current_role_path = os.path.dirname(self._owner._role_path)
-
+
try:
return load_list_of_roles(roles, play=self._owner._play, current_role_path=current_role_path, variable_manager=self._variable_manager, loader=self._loader)
except AssertionError:
diff --git a/lib/ansible/playbook/role_include.py b/lib/ansible/playbook/role_include.py
index 0ba26c90e6..4ed9369cba 100644
--- a/lib/ansible/playbook/role_include.py
+++ b/lib/ansible/playbook/role_include.py
@@ -108,7 +108,7 @@ class IncludeRole(Task):
# build options for role includes
for key in ['tasks', 'vars', 'defaults']:
from_key ='%s_from' % key
- if ir.args.get(from_key):
+ if ir.args.get(from_key):
ir._from_files[key] = basename(ir.args.get(from_key))
#FIXME: find a way to make this list come from object ( attributes does not work as per below)
diff --git a/lib/ansible/plugins/action/package.py b/lib/ansible/plugins/action/package.py
index 0b725759a9..45fb85abd2 100644
--- a/lib/ansible/plugins/action/package.py
+++ b/lib/ansible/plugins/action/package.py
@@ -51,7 +51,7 @@ class ActionModule(ActionBase):
if module == 'auto':
facts = self._execute_module(module_name='setup', module_args=dict(filter='ansible_pkg_mgr', gather_subset='!all'), task_vars=task_vars)
display.debug("Facts %s" % facts)
- if 'ansible_facts' in facts and 'ansible_pkg_mgr' in facts['ansible_facts']:
+ if 'ansible_facts' in facts and 'ansible_pkg_mgr' in facts['ansible_facts']:
module = getattr(facts['ansible_facts'], 'ansible_pkg_mgr', 'auto')
if module != 'auto':
diff --git a/lib/ansible/plugins/action/service.py b/lib/ansible/plugins/action/service.py
index b6caf67ac9..33ec6de35f 100644
--- a/lib/ansible/plugins/action/service.py
+++ b/lib/ansible/plugins/action/service.py
@@ -50,7 +50,7 @@ class ActionModule(ActionBase):
if module == 'auto':
facts = self._execute_module(module_name='setup', module_args=dict(gather_subset='!all', filter='ansible_service_mgr'), task_vars=task_vars)
self._display.debug("Facts %s" % facts)
- if 'ansible_facts' in facts and 'ansible_service_mgr' in facts['ansible_facts']:
+ if 'ansible_facts' in facts and 'ansible_service_mgr' in facts['ansible_facts']:
module = facts['ansible_facts']['ansible_service_mgr']
if not module or module == 'auto' or module not in self._shared_loader_obj.module_loader:
diff --git a/lib/ansible/plugins/callback/jabber.py b/lib/ansible/plugins/callback/jabber.py
index d06016d715..ff839716ff 100644
--- a/lib/ansible/plugins/callback/jabber.py
+++ b/lib/ansible/plugins/callback/jabber.py
@@ -66,7 +66,7 @@ class CallbackModule(CallbackBase):
def v2_runner_on_ok(self, result):
self._clean_results(result._result, result._task.action)
self.debug = self._dump_results(result._result)
-
+
def v2_playbook_on_task_start(self, task, is_conditional):
self.task = task
diff --git a/lib/ansible/plugins/callback/syslog_json.py b/lib/ansible/plugins/callback/syslog_json.py
index e93b97b568..85257ca170 100644
--- a/lib/ansible/plugins/callback/syslog_json.py
+++ b/lib/ansible/plugins/callback/syslog_json.py
@@ -18,7 +18,7 @@ class CallbackModule(CallbackBase):
make sure you have in ansible.cfg:
callback_plugins = <path_to_callback_plugins_folder>
and put the plugin in <path_to_callback_plugins_folder>
-
+
This plugin makes use of the following environment variables:
SYSLOG_SERVER (optional): defaults to localhost
SYSLOG_PORT (optional): defaults to 514
diff --git a/lib/ansible/plugins/connection/iocage.py b/lib/ansible/plugins/connection/iocage.py
index d6a960cb5c..89d648b4a7 100644
--- a/lib/ansible/plugins/connection/iocage.py
+++ b/lib/ansible/plugins/connection/iocage.py
@@ -56,7 +56,7 @@ class Connection(Jail):
stdin=subprocess.PIPE,
stdout=subprocess.PIPE,
stderr=subprocess.STDOUT)
-
+
stdout, stderr = p.communicate()
# otherwise p.returncode would not be set
p.wait()
diff --git a/lib/ansible/plugins/lookup/first_found.py b/lib/ansible/plugins/lookup/first_found.py
index 36e86188d1..fc302bf26e 100644
--- a/lib/ansible/plugins/lookup/first_found.py
+++ b/lib/ansible/plugins/lookup/first_found.py
@@ -47,7 +47,7 @@ __metaclass__ = type
# /tmp/staging/foo
# ${inventory_hostname}
# bar
-
+
# - name: copy first existing file found to /some/file
# action: copy src=$item dest=/some/file
# with_first_found:
diff --git a/lib/ansible/plugins/lookup/hashi_vault.py b/lib/ansible/plugins/lookup/hashi_vault.py
index e1953fea18..74719967bc 100644
--- a/lib/ansible/plugins/lookup/hashi_vault.py
+++ b/lib/ansible/plugins/lookup/hashi_vault.py
@@ -48,11 +48,11 @@ class HashiVault:
raise AnsibleError("Please pip install hvac to use this module")
self.url = kwargs.get('url', ANSIBLE_HASHI_VAULT_ADDR)
-
+
self.token = kwargs.get('token')
if self.token==None:
raise AnsibleError("No Vault Token specified")
-
+
# split secret arg, which has format 'secret/hello:value' into secret='secret/hello' and secret_field='value'
s = kwargs.get('secret')
if s==None:
@@ -76,13 +76,13 @@ class HashiVault:
data = self.client.read(self.secret)
if data is None:
raise AnsibleError("The secret %s doesn't seem to exist" % self.secret)
-
+
if self.secret_field=='': # secret was specified with trailing ':'
return data['data']
-
+
if self.secret_field not in data['data']:
raise AnsibleError("The secret %s does not contain the field '%s'. " % (self.secret, self.secret_field))
-
+
return data['data'][self.secret_field]
@@ -105,6 +105,6 @@ class LookupModule(LookupBase):
key = term.split()[0]
value = vault_conn.get()
ret.append(value)
-
+
return ret
-
+
diff --git a/lib/ansible/plugins/strategy/__init__.py b/lib/ansible/plugins/strategy/__init__.py
index 08ae4ca9a7..5451108241 100644
--- a/lib/ansible/plugins/strategy/__init__.py
+++ b/lib/ansible/plugins/strategy/__init__.py
@@ -322,7 +322,7 @@ class StrategyBase:
# get the original host and task. We then assign them to the TaskResult for use in callbacks/etc.
original_host = get_original_host(task_result._host)
original_task = iterator.get_original_task(original_host, task_result._task)
-
+
task_result._host = original_host
task_result._task = original_task
@@ -662,7 +662,7 @@ class StrategyBase:
if group_name not in host.get_groups():
new_group.add_host(real_host)
changed = True
-
+
if changed:
# clear cache of group dict, which is used in magic host variables
self._inventory.clear_group_dict_cache()
@@ -899,7 +899,7 @@ class StrategyBase:
elif meta_action == 'end_play':
if _evaluate_conditional(target_host):
for host in self._inventory.get_hosts(iterator._play.hosts):
- if not host.name in self._tqm._unreachable_hosts:
+ if not host.name in self._tqm._unreachable_hosts:
iterator._host_states[host.name].run_state = iterator.ITERATING_COMPLETE
msg="ending play"
#elif meta_action == 'reset_connection':
diff --git a/lib/ansible/vars/__init__.py b/lib/ansible/vars/__init__.py
index dfa3131c66..70dd6ad0e4 100644
--- a/lib/ansible/vars/__init__.py
+++ b/lib/ansible/vars/__init__.py
@@ -371,7 +371,7 @@ class VariableManager:
# special case for the 'environment' magic variable, as someone
# may have set it as a variable and we don't want to stomp on it
if task:
- if 'environment' not in all_vars:
+ if 'environment' not in all_vars:
all_vars['environment'] = task.environment
else:
display.warning("The variable 'environment' appears to be used already, which is also used internally for environment variables set on the task/block/play. You should use a different variable name to avoid conflicts with this internal variable")
diff --git a/test/integration/cleanup_ec2.py b/test/integration/cleanup_ec2.py
index 559a5c0ae1..64178394a6 100644
--- a/test/integration/cleanup_ec2.py
+++ b/test/integration/cleanup_ec2.py
@@ -75,10 +75,10 @@ def prompt_and_delete(item, prompt, assumeyes):
assumeyes = raw_input(prompt).lower() == 'y'
assert hasattr(item, 'delete') or hasattr(item, 'terminate') , "Class <%s> has no delete or terminate attribute" % item.__class__
if assumeyes:
- if hasattr(item, 'delete'):
+ if hasattr(item, 'delete'):
item.delete()
print ("Deleted %s" % item)
- if hasattr(item, 'terminate'):
+ if hasattr(item, 'terminate'):
item.terminate()
print ("Terminated %s" % item)
diff --git a/test/sanity/pep8/legacy-files.txt b/test/sanity/pep8/legacy-files.txt
index f7170762ee..16013a7429 100644
--- a/test/sanity/pep8/legacy-files.txt
+++ b/test/sanity/pep8/legacy-files.txt
@@ -6,7 +6,6 @@ contrib/inventory/consul_io.py
contrib/inventory/digital_ocean.py
contrib/inventory/docker.py
contrib/inventory/ec2.py
-contrib/inventory/fleet.py
contrib/inventory/freeipa.py
contrib/inventory/gce.py
contrib/inventory/linode.py
@@ -16,7 +15,6 @@ contrib/inventory/rax.py
contrib/inventory/softlayer.py
contrib/inventory/spacewalk.py
contrib/inventory/ssh_config.py
-contrib/inventory/stacki.py
contrib/inventory/vbox.py
contrib/inventory/vmware_inventory.py
contrib/inventory/windows_azure.py
@@ -27,7 +25,6 @@ docs/docsite/conf.py
docs/docsite/rst/conf.py
docs/docsite/rst/dev_guide/conf.py
examples/scripts/uptime.py
-hacking/dump_playbook_attributes.py
hacking/metadata-tool.py
hacking/module_formatter.py
hacking/tests/gen_distribution_version_testcase.py
@@ -142,16 +139,13 @@ lib/ansible/modules/cloud/azure/azure_rm_networkinterface.py
lib/ansible/modules/cloud/azure/azure_rm_networkinterface_facts.py
lib/ansible/modules/cloud/azure/azure_rm_securitygroup.py
lib/ansible/modules/cloud/azure/azure_rm_securitygroup_facts.py
-lib/ansible/modules/cloud/azure/azure_rm_storageblob.py
lib/ansible/modules/cloud/azure/azure_rm_subnet.py
lib/ansible/modules/cloud/azure/azure_rm_virtualmachine.py
-lib/ansible/modules/cloud/azure/azure_rm_virtualnetwork.py
lib/ansible/modules/cloud/centurylink/clc_loadbalancer.py
lib/ansible/modules/cloud/cloudscale/cloudscale_server.py
lib/ansible/modules/cloud/cloudstack/cs_firewall.py
lib/ansible/modules/cloud/cloudstack/cs_host.py
lib/ansible/modules/cloud/cloudstack/cs_instance.py
-lib/ansible/modules/cloud/cloudstack/cs_instance_facts.py
lib/ansible/modules/cloud/cloudstack/cs_iso.py
lib/ansible/modules/cloud/cloudstack/cs_portforward.py
lib/ansible/modules/cloud/cloudstack/cs_securitygroup_rule.py
@@ -163,7 +157,6 @@ lib/ansible/modules/cloud/docker/docker_image.py
lib/ansible/modules/cloud/docker/docker_service.py
lib/ansible/modules/cloud/google/gc_storage.py
lib/ansible/modules/cloud/google/gcdns_record.py
-lib/ansible/modules/cloud/google/gcdns_zone.py
lib/ansible/modules/cloud/google/gce_img.py
lib/ansible/modules/cloud/google/gce_lb.py
lib/ansible/modules/cloud/google/gce_mig.py
@@ -212,7 +205,6 @@ lib/ansible/modules/cloud/profitbricks/profitbricks_volume_attachments.py
lib/ansible/modules/cloud/rackspace/rax_cbs.py
lib/ansible/modules/cloud/rackspace/rax_cbs_attachments.py
lib/ansible/modules/cloud/rackspace/rax_clb.py
-lib/ansible/modules/cloud/rackspace/rax_facts.py
lib/ansible/modules/cloud/rackspace/rax_identity.py
lib/ansible/modules/cloud/rackspace/rax_network.py
lib/ansible/modules/cloud/rackspace/rax_queue.py
@@ -220,7 +212,6 @@ lib/ansible/modules/cloud/softlayer/sl_vm.py
lib/ansible/modules/cloud/vmware/vca_fw.py
lib/ansible/modules/cloud/vmware/vmware_guest.py
lib/ansible/modules/cloud/vmware/vmware_local_user_manager.py
-lib/ansible/modules/cloud/vmware/vmware_vswitch.py
lib/ansible/modules/cloud/vmware/vsphere_copy.py
lib/ansible/modules/cloud/vmware/vsphere_guest.py
lib/ansible/modules/cloud/webfaction/webfaction_app.py
@@ -234,10 +225,8 @@ lib/ansible/modules/clustering/consul_kv.py
lib/ansible/modules/clustering/consul_session.py
lib/ansible/modules/commands/command.py
lib/ansible/modules/commands/script.py
-lib/ansible/modules/crypto/openssl_privatekey.py
lib/ansible/modules/crypto/openssl_publickey.py
lib/ansible/modules/database/influxdb/influxdb_retention_policy.py
-lib/ansible/modules/database/misc/elasticsearch_plugin.py
lib/ansible/modules/database/misc/kibana_plugin.py
lib/ansible/modules/database/misc/mongodb_parameter.py
lib/ansible/modules/database/misc/mongodb_user.py
@@ -254,7 +243,6 @@ lib/ansible/modules/database/postgresql/postgresql_lang.py
lib/ansible/modules/database/postgresql/postgresql_privs.py
lib/ansible/modules/database/postgresql/postgresql_schema.py
lib/ansible/modules/database/postgresql/postgresql_user.py
-lib/ansible/modules/database/vertica/vertica_configuration.py
lib/ansible/modules/database/vertica/vertica_role.py
lib/ansible/modules/database/vertica/vertica_schema.py
lib/ansible/modules/database/vertica/vertica_user.py
@@ -280,7 +268,6 @@ lib/ansible/modules/monitoring/logicmonitor.py
lib/ansible/modules/monitoring/logicmonitor_facts.py
lib/ansible/modules/monitoring/monit.py
lib/ansible/modules/monitoring/nagios.py
-lib/ansible/modules/monitoring/newrelic_deployment.py
lib/ansible/modules/monitoring/pagerduty.py
lib/ansible/modules/monitoring/pagerduty_alert.py
lib/ansible/modules/monitoring/pingdom.py
@@ -297,7 +284,6 @@ lib/ansible/modules/network/a10/a10_virtual_server.py
lib/ansible/modules/network/basics/get_url.py
lib/ansible/modules/network/basics/slurp.py
lib/ansible/modules/network/basics/uri.py
-lib/ansible/modules/network/bigswitch/bigmon_chain.py
lib/ansible/modules/network/cloudengine/ce_command.py
lib/ansible/modules/network/cloudflare_dns.py
lib/ansible/modules/network/dellos10/dellos10_facts.py
@@ -307,7 +293,6 @@ lib/ansible/modules/network/dellos9/dellos9_config.py
lib/ansible/modules/network/dellos9/dellos9_facts.py
lib/ansible/modules/network/dnsimple.py
lib/ansible/modules/network/dnsmadeeasy.py
-lib/ansible/modules/network/eos/eos_eapi.py
lib/ansible/modules/network/exoscale/exo_dns_record.py
lib/ansible/modules/network/f5/bigip_facts.py
lib/ansible/modules/network/f5/bigip_monitor_tcp.py
@@ -382,7 +367,6 @@ lib/ansible/modules/network/nxos/nxos_vxlan_vtep_vni.py
lib/ansible/modules/network/ovs/openvswitch_port.py
lib/ansible/modules/network/panos/panos_nat_policy.py
lib/ansible/modules/network/snmp_facts.py
-lib/ansible/modules/network/wakeonlan.py
lib/ansible/modules/notification/flowdock.py
lib/ansible/modules/notification/hall.py
lib/ansible/modules/notification/irc.py
@@ -507,13 +491,10 @@ lib/ansible/playbook/play_context.py
lib/ansible/playbook/playbook_include.py
lib/ansible/playbook/role/__init__.py
lib/ansible/playbook/role/metadata.py
-lib/ansible/playbook/role_include.py
lib/ansible/playbook/taggable.py
lib/ansible/plugins/action/__init__.py
lib/ansible/plugins/action/add_host.py
lib/ansible/plugins/action/async.py
-lib/ansible/plugins/action/package.py
-lib/ansible/plugins/action/service.py
lib/ansible/plugins/action/set_fact.py
lib/ansible/plugins/action/set_stats.py
lib/ansible/plugins/action/synchronize.py
@@ -522,14 +503,12 @@ lib/ansible/plugins/cache/jsonfile.py
lib/ansible/plugins/callback/default.py
lib/ansible/plugins/callback/dense.py
lib/ansible/plugins/callback/foreman.py
-lib/ansible/plugins/callback/jabber.py
lib/ansible/plugins/callback/logentries.py
lib/ansible/plugins/callback/oneline.py
lib/ansible/plugins/callback/profile_tasks.py
lib/ansible/plugins/callback/selective.py
lib/ansible/plugins/callback/syslog_json.py
lib/ansible/plugins/connection/accelerate.py
-lib/ansible/plugins/connection/iocage.py
lib/ansible/plugins/connection/paramiko_ssh.py
lib/ansible/plugins/connection/ssh.py
lib/ansible/plugins/connection/winrm.py
@@ -581,14 +560,12 @@ test/units/module_utils/basic/test_log.py
test/units/module_utils/basic/test_no_log.py
test/units/module_utils/basic/test_safe_eval.py
test/units/module_utils/basic/test_set_mode_if_different.py
-test/units/module_utils/gcp/test_auth.py
test/units/module_utils/json_utils/test_filter_non_json_lines.py
test/units/module_utils/test_basic.py
test/units/module_utils/test_distribution_version.py
test/units/module_utils/test_facts.py
test/units/module_utils/test_text.py
test/units/modules/cloud/amazon/test_ec2_vpc_nat_gateway.py
-test/units/modules/cloud/google/test_gce_tag.py
test/units/parsing/test_mod_args.py
test/units/parsing/yaml/test_loader.py
test/units/playbook/test_block.py
@@ -599,7 +576,6 @@ test/units/playbook/test_task.py
test/units/plugins/action/test_raw.py
test/units/plugins/action/test_synchronize.py
test/units/plugins/lookup/test_ini.py
-test/units/plugins/lookup/test_lastpass.py
test/units/plugins/lookup/test_password.py
test/units/plugins/strategy/test_strategy_base.py
test/units/template/test_templar.py
diff --git a/test/sanity/pep8/legacy-ignore.txt b/test/sanity/pep8/legacy-ignore.txt
index 3d2daffbb7..ff6aba8046 100644
--- a/test/sanity/pep8/legacy-ignore.txt
+++ b/test/sanity/pep8/legacy-ignore.txt
@@ -8,7 +8,6 @@ E125
E126
E129
E131
-E271
E501
E701
E703
@@ -17,4 +16,3 @@ E712
E721
W191
W291
-W293
diff --git a/test/units/cli/test_galaxy.py b/test/units/cli/test_galaxy.py
index 6506a59850..3bc58c01ab 100644
--- a/test/units/cli/test_galaxy.py
+++ b/test/units/cli/test_galaxy.py
@@ -120,7 +120,7 @@ class TestGalaxy(unittest.TestCase):
with patch.object(ansible.cli.CLI, "execute", return_value=None) as mock_ex:
with patch.object(ansible.cli.CLI, "run", return_value=None) as mock_run:
gc.run()
-
+
# testing
self.assertEqual(mock_run.call_count, 1)
self.assertTrue(isinstance(gc.api, ansible.galaxy.api.GalaxyAPI))
diff --git a/test/units/module_utils/gcp/test_auth.py b/test/units/module_utils/gcp/test_auth.py
index 102a864218..5aa8049be2 100644
--- a/test/units/module_utils/gcp/test_auth.py
+++ b/test/units/module_utils/gcp/test_auth.py
@@ -135,7 +135,7 @@ class GCPAuthTestCase(unittest.TestCase):
require_valid_json=False,
check_libcloud=False)
self.assertTrue(is_valid)
-
+
@mock.patch('ansible.module_utils.gcp._get_gcp_environ_var',
side_effect=fake_get_gcp_environ_var)
def test_get_gcp_environment_credentials(self, mockobj):
@@ -144,7 +144,7 @@ class GCPAuthTestCase(unittest.TestCase):
actual = _get_gcp_environment_credentials(None, None, None)
expected = tuple(['gce-email', None, None])
self.assertEqual(expected, actual)
-
+
fake_env_data = {'GCE_PEM_FILE_PATH': '/path/to/pem.pem'}
expected = tuple([None, '/path/to/pem.pem', None])
actual = _get_gcp_environment_credentials(None, None, None)
diff --git a/test/units/module_utils/test_basic.py b/test/units/module_utils/test_basic.py
index a03e3401f1..b8551cb722 100644
--- a/test/units/module_utils/test_basic.py
+++ b/test/units/module_utils/test_basic.py
@@ -176,10 +176,10 @@ class TestModuleUtilsBasic(ModuleTestCase):
return ("Bar", "2", "Two")
else:
return ("", "", "")
-
+
with patch('platform.linux_distribution', side_effect=_dist):
self.assertEqual(get_distribution(), "Bar")
-
+
with patch('platform.linux_distribution', side_effect=Exception("boo")):
with patch('platform.dist', return_value=("bar", "2", "Two")):
self.assertEqual(get_distribution(), "Bar")
@@ -699,7 +699,7 @@ class TestModuleUtilsBasic(ModuleTestCase):
self.assertRaises(SystemExit, am.set_context_if_different, '/path/to/file', ['foo_u', 'foo_r', 'foo_t', 's0'], True)
am.is_special_selinux_path = MagicMock(return_value=(True, ['sp_u', 'sp_r', 'sp_t', 's0']))
-
+
with patch('selinux.lsetfilecon', return_value=0) as m:
self.assertEqual(am.set_context_if_different('/path/to/file', ['foo_u', 'foo_r', 'foo_t', 's0'], False), True)
m.assert_called_with('/path/to/file', 'sp_u:sp_r:sp_t:s0')
diff --git a/test/units/modules/cloud/google/test_gce_tag.py b/test/units/modules/cloud/google/test_gce_tag.py
index 6a4595e263..81973b71f8 100644
--- a/test/units/modules/cloud/google/test_gce_tag.py
+++ b/test/units/modules/cloud/google/test_gce_tag.py
@@ -52,7 +52,7 @@ class TestGCETag(unittest.TestCase):
want = ['one']
got = _get_changed_items(tags_to_add, existing_tags)
self.assertEqual(want, got)
-
+
# simulate removing tags
# specifying one tag on right that doesn't exist
tags_to_remove = ['one', 'two']
@@ -63,6 +63,6 @@ class TestGCETag(unittest.TestCase):
if __name__ == '__main__':
unittest.main()
-
-
+
+
diff --git a/test/units/plugins/lookup/test_lastpass.py b/test/units/plugins/lookup/test_lastpass.py
index bab974759a..9918d488d7 100644
--- a/test/units/plugins/lookup/test_lastpass.py
+++ b/test/units/plugins/lookup/test_lastpass.py
@@ -114,12 +114,12 @@ class MockLPass(LPass):
class DisconnectedMockLPass(MockLPass):
-
+
_mock_disconnected = True
class LoggedOutMockLPass(MockLPass):
-
+
_mock_logged_out = True
diff --git a/test/units/plugins/strategy/test_strategy_base.py b/test/units/plugins/strategy/test_strategy_base.py
index b5508c42e5..c9c305e0b1 100644
--- a/test/units/plugins/strategy/test_strategy_base.py
+++ b/test/units/plugins/strategy/test_strategy_base.py
@@ -163,7 +163,7 @@ class TestStrategyBase(unittest.TestCase):
mock_tqm._failed_hosts = []
mock_tqm._unreachable_hosts = []
self.assertEqual(strategy_base.get_hosts_remaining(play=mock_play), mock_hosts)
-
+
mock_tqm._failed_hosts = ["host01"]
self.assertEqual(strategy_base.get_hosts_remaining(play=mock_play), mock_hosts[1:])
self.assertEqual(strategy_base.get_failed_hosts(play=mock_play), [mock_hosts[0]])
@@ -186,7 +186,7 @@ class TestStrategyBase(unittest.TestCase):
mock_inventory = MagicMock()
mock_options = MagicMock()
mock_options.module_path = None
-
+
tqm = TaskQueueManager(
inventory=mock_inventory,
variable_manager=mock_var_manager,
@@ -210,7 +210,7 @@ class TestStrategyBase(unittest.TestCase):
self.assertEqual(strategy_base._pending_results, 3)
finally:
tqm.cleanup()
-
+
def test_strategy_base_process_pending_results(self):
mock_tqm = MagicMock()
@@ -240,7 +240,7 @@ class TestStrategyBase(unittest.TestCase):
mock_tqm._stats = MagicMock()
mock_tqm._stats.increment.return_value = None
-
+
mock_play = MagicMock()
mock_host = MagicMock()
diff --git a/test/units/template/test_templar.py b/test/units/template/test_templar.py
index 91f12feded..f2e1be5dfb 100644
--- a/test/units/template/test_templar.py
+++ b/test/units/template/test_templar.py
@@ -103,7 +103,7 @@ class TestTemplar(unittest.TestCase):
def test_template_jinja2_extensions(self):
fake_loader = DictDataLoader({})
templar = Templar(loader=fake_loader)
-
+
old_exts = C.DEFAULT_JINJA2_EXTENSIONS
try:
C.DEFAULT_JINJA2_EXTENSIONS = "foo,bar"