starlingx
/
config
Code Issues Proposed changes

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 <mathieu.robinson@windriver.com>
This commit is contained in:
Al Bailey 2018-07-04 08:01:06 -05:00 committed by Mathieu Robinson
parent be59494af5
commit b662645257
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sysinv/sysinv/sysinv/sysinv/api/controllers/v1/sm_api.py
View File

@ -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.

1
sysinv/sysinv/sysinv/sysinv/puppet/smapi.py
View File

@ -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,