Skip to content

Commit a7483f7

Browse files
resolving pep8 issues
Signed-off-by: Shiva Shankar Vaddepally <[email protected]>
1 parent 15c0ac9 commit a7483f7

File tree

6 files changed

+10
-7
lines changed

6 files changed

+10
-7
lines changed

plugins/module_utils/common.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
from .logger import log
2222
from .nitro_resource_map import NITRO_RESOURCE_MAP
2323

24+
2425
@trace
2526
def get_netscaler_version(client):
2627
is_exist, response = get_resource(client, "nsversion")
@@ -114,7 +115,7 @@ def get_resource(client, resource_name, resource_id=None, resource_module_params
114115
# for zero bindings and some resources, the response_body will be {'errorcode': 0, 'message': 'Done', 'severity': 'NONE'}
115116
if resource_name in NESTED_POST_DATA_RESOURCES:
116117
if "routerDynamicRouting" in response_body:
117-
return_response = response_body["routerDynamicRouting"][NESTED_POST_DATA_RESOURCES_ALIAS[resource_name]]
118+
return_response = response_body["routerDynamicRouting"][NESTED_POST_DATA_RESOURCES_ALIAS[resource_name]]
118119
else:
119120
return False, []
120121
elif resource_name not in response_body:
@@ -125,8 +126,8 @@ def get_resource(client, resource_name, resource_id=None, resource_module_params
125126
]
126127

127128
return False, []
128-
129-
# `update-only` resources return a dict instead of a list.
129+
130+
# `update-only` resources return a dict instead of a list.
130131
if resource_name not in NESTED_POST_DATA_RESOURCES:
131132
return_response = response_body[resource_name]
132133
# FIXME: NITRO-BUG: for some resources like `policypatset_pattern_binding`, NITRO returns keys with uppercase. eg: `String` for `string`.
@@ -206,6 +207,7 @@ def is_resource_exists(client, resource_name, resource_module_params):
206207
)
207208
return is_exists
208209

210+
209211
@trace
210212
def _check_create_resource_params(resource_name, resource_module_params, action=None):
211213
post_data = {}

plugins/module_utils/module_executor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -389,6 +389,7 @@ def is_attribute_equal(
389389
)
390390
# By default, compare as string values
391391
return str(existing_attribute_value) == str(module_params_attribute_value)
392+
392393
@trace
393394
def is_resource_identical_nested(self):
394395
diff_list = []
@@ -427,7 +428,6 @@ def get_nested_value(data, path):
427428
self.module_result.pop("diff_list", None)
428429
return True
429430

430-
431431
@trace
432432
def is_resource_identical(self):
433433
"""

plugins/modules/bgprouter.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,7 @@
138138
type: str
139139
description:
140140
- Source of routing updates.
141+
extends_documentation_fragment: netscaler.adc.netscaler_adc
141142
"""
142143
EXAMPLES = r"""
143144
---

tests/sanity/ignore-2.15.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -948,4 +948,4 @@ plugins/modules/sslechconfig.py validate-modules:missing-gplv3-license # We use
948948
plugins/modules/sslhpkekey.py validate-modules:missing-gplv3-license # We use MIT license
949949
plugins/modules/sslprofile_sslechconfig_binding.py validate-modules:missing-gplv3-license # We use MIT license
950950
plugins/modules/authenticationprotecteduseraction.py validate-modules:missing-gplv3-license # We use MIT license
951-
plugins/modules/bgpRouter.py validate-modules:missing-gplv3-license # We use MIT license
951+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license

tests/sanity/ignore-2.16.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -948,4 +948,4 @@ plugins/modules/sslechconfig.py validate-modules:missing-gplv3-license # We use
948948
plugins/modules/sslhpkekey.py validate-modules:missing-gplv3-license # We use MIT license
949949
plugins/modules/sslprofile_sslechconfig_binding.py validate-modules:missing-gplv3-license # We use MIT license
950950
plugins/modules/authenticationprotecteduseraction.py validate-modules:missing-gplv3-license # We use MIT license
951-
plugins/modules/bgpRouter.py validate-modules:missing-gplv3-license # We use MIT license
951+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license

tests/sanity/ignore-2.17.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -948,4 +948,4 @@ plugins/modules/sslechconfig.py validate-modules:missing-gplv3-license # We use
948948
plugins/modules/sslhpkekey.py validate-modules:missing-gplv3-license # We use MIT license
949949
plugins/modules/sslprofile_sslechconfig_binding.py validate-modules:missing-gplv3-license # We use MIT license
950950
plugins/modules/authenticationprotecteduseraction.py validate-modules:missing-gplv3-license # We use MIT license
951-
plugins/modules/bgpRouter.py validate-modules:missing-gplv3-license # We use MIT license
951+
plugins/modules/bgprouter.py validate-modules:missing-gplv3-license # We use MIT license

0 commit comments

Comments
 (0)