Skip to content

Commit 0cede11

Browse files
committed
fix flake8 findings
Signed-off-by: hwassman <[email protected]>
1 parent d0b2cb0 commit 0cede11

File tree

3 files changed

+6
-8
lines changed

3 files changed

+6
-8
lines changed

source/confgenerator.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
import yaml
3030
except ImportError as e:
3131
yaml = e
32-
import analytics
3332
from messages import MSG, ERR
3433

3534

source/zimonGrafanaIntf.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -339,16 +339,16 @@ def main(argv):
339339
# register Prometheus config generator endpoints (only if PyYaml available)
340340
try:
341341
conf_generator = PrometheusConfigGenerator(logger,
342-
mdHandler,
343-
args,
344-
ENDPOINTS.get('prometheus',{}))
342+
mdHandler,
343+
args,
344+
ENDPOINTS.get('prometheus', {}))
345345
cherrypy.tree.mount(conf_generator, '/prometheus.yml',
346346
{'/':
347347
{'request.dispatch': cherrypy.dispatch.MethodDispatcher()}
348348
}
349349
)
350350
registered_apps.append("Prometheus Config Generator Api")
351-
except ImportError as e:
351+
except ImportError:
352352
logger.warning("Prometheus Config Generator Api requires python PyYaml packages. Skip registering API.")
353353

354354
# register Profiler reporter endpoint

tests/test_confgenerator.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
import os
33
from unittest import mock
44
from nose2.tools.decorators import with_setup
5-
from source.queryHandler.QueryHandler import ColumnInfo, Key
65
from source.profiler import Profiler
76
from source.metadata import MetadataHandler
87
from source.confgenerator import PrometheusConfigGenerator
@@ -68,7 +67,7 @@ def test_case01():
6867
with mock.patch('source.metadata.MetadataHandler._MetadataHandler__initializeTables') as md_init:
6968
with mock.patch('source.metadata.MetadataHandler._MetadataHandler__getSupportedMetrics') as md_supp:
7069
with mock.patch('source.metadata.MetadataHandler.SensorsConfig', return_value=sensors_conf) as md_sensConf:
71-
with mock.patch('source.confgenerator.PrometheusConfigGenerator.host_ip', return_value='127.0.0.1') as pc_ip:
70+
with mock.patch('source.confgenerator.PrometheusConfigGenerator.host_ip', return_value='127.0.0.1'):
7271
logger = logging.getLogger(__name__)
7372
args['logger'] = logger
7473
md = MetadataHandler(**args)
@@ -86,7 +85,7 @@ def test_case02():
8685
with mock.patch('source.metadata.MetadataHandler._MetadataHandler__initializeTables') as md_init:
8786
with mock.patch('source.metadata.MetadataHandler._MetadataHandler__getSupportedMetrics') as md_supp:
8887
with mock.patch('source.metadata.MetadataHandler.SensorsConfig', return_value=sensors_conf) as md_sensConf:
89-
with mock.patch('source.confgenerator.PrometheusConfigGenerator.host_ip', return_value='127.0.0.1') as pc_ip:
88+
with mock.patch('source.confgenerator.PrometheusConfigGenerator.host_ip', return_value='127.0.0.1'):
9089
logger = logging.getLogger(__name__)
9190
args['logger'] = logger
9291
md = MetadataHandler(**args)

0 commit comments

Comments
 (0)