diff --git a/.zuul.yaml b/.zuul.yaml index 870e0aa97..300863c15 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -2,7 +2,7 @@ - project: templates: - docs-on-readthedocs - - openstack-python37-jobs-nonvoting + - openstack-python37-jobs - openstack-python36-jobs vars: rtd_webhook_id: '51626' diff --git a/browbeat/perfkit.py b/browbeat/perfkit.py index fae375545..ce95bc42f 100644 --- a/browbeat/perfkit.py +++ b/browbeat/perfkit.py @@ -19,13 +19,13 @@ import shutil import subprocess import time -import elastic -import grafana -from path import get_overcloudrc -from path import get_workload_venv -from path import results_path -import tools -import workloadbase +import browbeat.tools +from browbeat import elastic +from browbeat import grafana +from browbeat import workloadbase +from browbeat.path import get_overcloudrc +from browbeat.path import get_workload_venv +from browbeat.path import results_path class PerfKit(workloadbase.WorkloadBase): @@ -34,7 +34,7 @@ class PerfKit(workloadbase.WorkloadBase): self.overcloudrc = get_overcloudrc() self.config = config self.result_dir_ts = result_dir_ts - self.tools = tools.Tools(self.config) + self.tools = browbeat.tools.Tools(self.config) self.grafana = grafana.Grafana(self.config) self.elastic = elastic.Elastic(self.config, self.__class__.__name__.lower()) diff --git a/browbeat/rally.py b/browbeat/rally.py index a944817a0..8f9eb17eb 100644 --- a/browbeat/rally.py +++ b/browbeat/rally.py @@ -20,13 +20,12 @@ import re import shutil import time -import elastic -import grafana -from path import get_workload_venv -from path import results_path -import workloadbase -import tools - +import browbeat.tools +from browbeat import elastic +from browbeat import grafana +from browbeat import workloadbase +from browbeat.path import get_workload_venv +from browbeat.path import results_path class Rally(workloadbase.WorkloadBase): @@ -34,7 +33,7 @@ class Rally(workloadbase.WorkloadBase): self.logger = logging.getLogger('browbeat.rally') self.config = config self.result_dir_ts = result_dir_ts - self.tools = tools.Tools(self.config) + self.tools = browbeat.tools.Tools(self.config) self.grafana = grafana.Grafana(self.config) self.elastic = elastic.Elastic(self.config, self.__class__.__name__.lower()) diff --git a/browbeat/shaker.py b/browbeat/shaker.py index d87043dfa..1d4c2bdc6 100644 --- a/browbeat/shaker.py +++ b/browbeat/shaker.py @@ -19,13 +19,13 @@ import time import uuid import yaml -import elastic -import grafana -from path import get_overcloudrc -from path import get_workload_venv -from path import results_path -import workloadbase -import tools +import browbeat.tools +from browbeat import elastic +from browbeat import grafana +from browbeat import workloadbase +from browbeat.path import get_overcloudrc +from browbeat.path import get_workload_venv +from browbeat.path import results_path class Shaker(workloadbase.WorkloadBase): @@ -35,7 +35,7 @@ class Shaker(workloadbase.WorkloadBase): self.overcloudrc = get_overcloudrc() self.config = config self.result_dir_ts = result_dir_ts - self.tools = tools.Tools(self.config) + self.tools = browbeat.tools.Tools(self.config) self.grafana = grafana.Grafana(self.config) self.elastic = elastic.Elastic(self.config, self.__class__.__name__.lower()) diff --git a/browbeat/tools.py b/browbeat/tools.py index 911969efa..ca77eccbc 100644 --- a/browbeat/tools.py +++ b/browbeat/tools.py @@ -16,10 +16,10 @@ import os import re import subprocess -import perfkit -import rally -import shaker -import yoda +from browbeat import perfkit +from browbeat import rally +from browbeat import shaker +from browbeat import yoda class Tools(object): @@ -51,7 +51,7 @@ class Tools(object): stderr=subprocess.PIPE) stdout, stderr = process.communicate() output_dict = {} - output_dict['stdout'] = stdout.strip() + output_dict['stdout'] = stdout.strip().decode() output_dict['stderr'] = stderr.strip() output_dict['rc'] = process.returncode if process.returncode > 0: diff --git a/browbeat/yoda.py b/browbeat/yoda.py index e8b011306..c756c1dbd 100644 --- a/browbeat/yoda.py +++ b/browbeat/yoda.py @@ -28,11 +28,11 @@ except ImportError: ostag = None import requests -import elastic -import grafana -from path import results_path -import workloadbase -import tools +import browbeat.tools +from browbeat import elastic +from browbeat import grafana +from browbeat import workloadbase +from browbeat.path import results_path class Yoda(workloadbase.WorkloadBase): @@ -41,7 +41,7 @@ class Yoda(workloadbase.WorkloadBase): self.logger = logging.getLogger('browbeat.yoda') self.config = config self.result_dir_ts = result_dir_ts - self.tools = tools.Tools(self.config) + self.tools = browbeat.tools.Tools(self.config) self.grafana = grafana.Grafana(self.config) self.elastic = elastic.Elastic(self.config, self.__class__.__name__.lower()) self.error_count = 0 diff --git a/tox.ini b/tox.ini index b1a426eae..c653b1c63 100644 --- a/tox.ini +++ b/tox.ini @@ -1,6 +1,6 @@ [tox] minversion = 2.0 -envlist = py27,py35,py36,linters,dist,insights +envlist = py27,py35,py36,py37,linters,dist,insights skipsdist = True [testenv] @@ -62,6 +62,10 @@ commands = pytest {posargs} basepython = python3.6 commands = pytest {posargs} +[testenv:py37] +basepython = python3.7 +commands = pytest {posargs} + [testenv:cover] commands = python setup.py test --coverage --testr-args={posargs}