From: Emma Foley <emma.l.foley@intel.com>
Date: Fri, 7 Dec 2018 15:29:57 +0000 (+0000)
Subject: Fix pylint warnings in fetch_url_and_verify
X-Git-Tag: opnfv-8.0.0~93^2
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F59%2F65659%2F1;p=yardstick.git

Fix pylint warnings in fetch_url_and_verify

Disable pylint checking in ansible module, to avoid
unused-wildcard-import and wrong-import-position
The wildcard import is required for ansile to be able to
use the module.

JIRA: YARDSTICK-1527
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Change-Id: Icc358555f59c8cc088f6f22e4fab8ee2205daff2
---

diff --git a/ansible/library/fetch_url_and_verify.py b/ansible/library/fetch_url_and_verify.py
index 6c5c0a8c2..77ff916cc 100644
--- a/ansible/library/fetch_url_and_verify.py
+++ b/ansible/library/fetch_url_and_verify.py
@@ -13,6 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# pylint: disable-all
+
 DOCUMENTATION = '''
 ---
 module: fetch_url_and_verify
@@ -59,7 +61,7 @@ def main():
         module.exit_json(changed=False, dest=dest, url=url,
                          sha256sum=sha256sum)
 
-    for retry in range(retries):
+    for _ in range(retries):
         curl_rc, stdout, stderr = module.run_command(
             ['curl', '-sS', '-o', dest, url], cwd=image_dir)
         if curl_rc == 0:
@@ -74,7 +76,7 @@ def main():
 
 
 # <<INCLUDE_ANSIBLE_MODULE_COMMON>>
-from ansible.module_utils.basic import *  # noqa
+from ansible.module_utils.basic import *
 
 if __name__ == '__main__':
     main()