Fix pylint errors about refstack

JIRA: FUNCTEST-907

Change-Id: Ib08a88523c700d35894b9b89b7585425a02065c3
Signed-off-by: Linda Wang <wangwulin@huawei.com>

1 files changed, 20 insertions, 15 deletions

diff --git a/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py b/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py
index db745227..5764f366 100644
--- a/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py
+++ b/functest/opnfv_tests/openstack/refstack_client/tempest_conf.py
@@ -5,6 +5,9 @@
 # are made available under the terms of the Apache License, Version 2.0
 # which accompanies this distribution, and is available at
 # http://www.apache.org/licenses/LICENSE-2.0
+
+""" Used to generate tempest.conf """
+
 import logging
 import pkg_resources
 
@@ -14,24 +17,23 @@ from functest.utils.constants import CONST
 from functest.opnfv_tests.openstack.tempest.tempest \
     import TempestResourcesManager
 
-""" logging configuration """
-logger = logging.getLogger(__name__)
+LOGGER = logging.getLogger(__name__)
 
 
 class TempestConf(object):
+    """ TempestConf class"""
     def __init__(self, **kwargs):
-        self.VERIFIER_ID = conf_utils.get_verifier_id()
-        self.VERIFIER_REPO_DIR = conf_utils.get_verifier_repo_dir(
-            self.VERIFIER_ID)
-        self.DEPLOYMENT_ID = conf_utils.get_verifier_deployment_id()
-        self.DEPLOYMENT_DIR = conf_utils.get_verifier_deployment_dir(
-            self.VERIFIER_ID, self.DEPLOYMENT_ID)
+        self.verifier_id = conf_utils.get_verifier_id()
+        self.deployment_id = conf_utils.get_verifier_deployment_id()
+        self.deployment_dir = conf_utils.get_verifier_deployment_dir(
+            self.verifier_id, self.deployment_id)
         self.confpath = pkg_resources.resource_filename(
             'functest',
             'opnfv_tests/openstack/refstack_client/refstack_tempest.conf')
         self.resources = TempestResourcesManager(**kwargs)
 
     def generate_tempestconf(self):
+        """ Generate tempest.conf file"""
         try:
             openstack_utils.source_credentials(
                 CONST.__getattribute__('openstack_creds'))
@@ -39,29 +41,32 @@ class TempestConf(object):
                                               use_custom_images=True,
                                               use_custom_flavors=True)
             conf_utils.configure_tempest_defcore(
-                self.DEPLOYMENT_DIR,
+                self.deployment_dir,
                 image_id=resources.get("image_id"),
                 flavor_id=resources.get("flavor_id"),
                 image_id_alt=resources.get("image_id_alt"),
                 flavor_id_alt=resources.get("flavor_id_alt"),
                 tenant_id=resources.get("project_id"))
-        except Exception as e:
-            logger.error("error with generating refstack client "
-                         "reference tempest conf file: %s", e)
+        except Exception as err:  # pylint: disable=broad-except
+            LOGGER.error("error with generating refstack client "
+                         "reference tempest conf file: %s", err)
 
     def main(self):
+        """ The main function called by entry point"""
         try:
             self.generate_tempestconf()
-            logger.info("a reference tempest conf file generated "
+            LOGGER.info("a reference tempest conf file generated "
                         "at %s", self.confpath)
-        except Exception as e:
-            logger.error('Error with run: %s', e)
+        except Exception as err:  # pylint: disable=broad-except
+            LOGGER.error('Error with run: %s', err)
 
     def clean(self):
+        """Clean up the resources"""
         self.resources.cleanup()
 
 
 def main():
+    """Entry point"""
     logging.basicConfig()
     tempestconf = TempestConf()
     tempestconf.main()