From b6626452577ba10896a1734dfabeefa6ac88ccd3 Mon Sep 17 00:00:00 2001 From: Al Bailey Date: Wed, 4 Jul 2018 08:01:06 -0500 Subject: [PATCH] Fix sysinv pep8 tox failure Issues: - F401: 'REGION_ONE_NAME' imported but never used - E302: Expected 2 blank lines, found 1 - F841: Local variable 'kspass' is assigned to but never used. Fixes: - Removed the unused import. - Added a blank line. - Removed unused local variable 'kspass'. Story: 2002888 Task: 23097 Change-Id: Ib2fa3b51e7570e3255edbf426b789cbeb59900ea Signed-off-by: Mathieu Robinson --- sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py | 2 +- sysinv/sysinv/sysinv/sysinv/puppet/smapi.py | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py b/sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py index 2ca68fa57a..2863ee6308 100755 --- a/sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py +++ b/sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py @@ -8,7 +8,6 @@ import pecan import socket from rest_api import get_token from rest_api import rest_api_request -from sysinv.common.constants import REGION_ONE_NAME from sysinv.openstack.common import log LOG = log.getLogger(__name__) @@ -23,6 +22,7 @@ def _get_token(): system = pecan.request.dbapi.isystem_get_one() return get_token(system.region_name) + def swact_pre_check(hostname, timeout): """ Sends a Swact Pre-Check command to SM. diff --git a/sysinv/sysinv/sysinv/sysinv/puppet/smapi.py b/sysinv/sysinv/sysinv/sysinv/puppet/smapi.py index 2d86853572..b30f0ba9aa 100644 --- a/sysinv/sysinv/sysinv/sysinv/puppet/smapi.py +++ b/sysinv/sysinv/sysinv/sysinv/puppet/smapi.py @@ -32,7 +32,6 @@ class SmPuppet(openstack.OpenstackBasePuppet): def get_system_config(self): ksuser = self._get_service_user_name(self.SERVICE_NAME) - kspass = self._get_service_password(self.SERVICE_NAME) config = { 'smapi::keystone::authtoken::username': ksuser,