Fix pep8 tox job
pep8 was configured incorrectly in tox.ini and wasn't catching violations. This commit fixes the job and the issues that got in while the job was silently ignoring them. Change-Id: Id4d1e3192c558edf3c4babae54c87c1ba10d617f
This commit is contained in:
parent
b9cabcdbc8
commit
ffbd59d792
|
@ -31,6 +31,6 @@ class BaseAdminKuryrTest(base.BaseAdminNetworkTest):
|
||||||
@classmethod
|
@classmethod
|
||||||
def resource_setup(cls):
|
def resource_setup(cls):
|
||||||
super(BaseAdminKuryrTest, cls).resource_setup()
|
super(BaseAdminKuryrTest, cls).resource_setup()
|
||||||
# TODO (dmellado): Config k8s client in a cleaner way
|
# TODO(dmellado): Config k8s client in a cleaner way
|
||||||
kubernetes.config.load_kube_config()
|
kubernetes.config.load_kube_config()
|
||||||
cls.k8s_client = kubernetes.client.CoreV1Api()
|
cls.k8s_client = kubernetes.client.CoreV1Api()
|
||||||
|
|
|
@ -17,9 +17,9 @@ import time
|
||||||
from kubernetes import client as k8s_client
|
from kubernetes import client as k8s_client
|
||||||
from kubernetes import config as k8s_config
|
from kubernetes import config as k8s_config
|
||||||
|
|
||||||
from tempest.scenario import manager
|
|
||||||
from tempest.lib.common.utils import data_utils
|
|
||||||
from tempest import config
|
from tempest import config
|
||||||
|
from tempest.lib.common.utils import data_utils
|
||||||
|
from tempest.scenario import manager
|
||||||
|
|
||||||
CONF = config.CONF
|
CONF = config.CONF
|
||||||
|
|
||||||
|
@ -41,7 +41,7 @@ class BaseKuryrScenarioTest(manager.NetworkScenarioTest):
|
||||||
def resource_setup(cls):
|
def resource_setup(cls):
|
||||||
super(BaseKuryrScenarioTest, cls).resource_setup()
|
super(BaseKuryrScenarioTest, cls).resource_setup()
|
||||||
cls.pod_fips = []
|
cls.pod_fips = []
|
||||||
# TODO (dmellado): Config k8s client in a cleaner way
|
# TODO(dmellado): Config k8s client in a cleaner way
|
||||||
k8s_config.load_kube_config()
|
k8s_config.load_kube_config()
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
|
@ -68,7 +68,7 @@ class BaseKuryrScenarioTest(manager.NetworkScenarioTest):
|
||||||
body=pod)
|
body=pod)
|
||||||
status = ""
|
status = ""
|
||||||
while status != "Running":
|
while status != "Running":
|
||||||
# TODO (dmellado) add timeout config to tempest plugin
|
# TODO(dmellado) add timeout config to tempest plugin
|
||||||
time.sleep(1)
|
time.sleep(1)
|
||||||
status = self.get_pod_status(name, namespace)
|
status = self.get_pod_status(name, namespace)
|
||||||
|
|
||||||
|
|
|
@ -12,13 +12,11 @@
|
||||||
# See the License for the specific language governing permissions and
|
# See the License for the specific language governing permissions and
|
||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
import testtools
|
|
||||||
|
|
||||||
from oslo_log import log as logging
|
from oslo_log import log as logging
|
||||||
from tempest import config
|
from tempest import config
|
||||||
|
from tempest.lib.common.utils import data_utils
|
||||||
from tempest.lib import decorators
|
from tempest.lib import decorators
|
||||||
from tempest.lib import exceptions
|
from tempest.lib import exceptions
|
||||||
from tempest.lib.common.utils import data_utils
|
|
||||||
|
|
||||||
from kuryr_tempest_plugin.tests.scenario import base
|
from kuryr_tempest_plugin.tests.scenario import base
|
||||||
|
|
||||||
|
@ -61,6 +59,7 @@ class TestCrossPingScenario(base.BaseKuryrScenarioTest):
|
||||||
'from host "%s" to "%s".' % (
|
'from host "%s" to "%s".' % (
|
||||||
fip['floating_ip_address'],
|
fip['floating_ip_address'],
|
||||||
pod_fip['floatingip']['floating_ip_address']))
|
pod_fip['floatingip']['floating_ip_address']))
|
||||||
|
LOG.error(msg)
|
||||||
self.assertEqual('0', result.rstrip('\n'))
|
self.assertEqual('0', result.rstrip('\n'))
|
||||||
except exceptions.SSHExecCommandFailed:
|
except exceptions.SSHExecCommandFailed:
|
||||||
LOG.error("Couldn't ping server")
|
LOG.error("Couldn't ping server")
|
||||||
|
|
|
@ -14,8 +14,8 @@
|
||||||
|
|
||||||
|
|
||||||
from oslo_log import log as logging
|
from oslo_log import log as logging
|
||||||
from tempest.lib import decorators
|
|
||||||
from tempest.lib.common.utils import data_utils
|
from tempest.lib.common.utils import data_utils
|
||||||
|
from tempest.lib import decorators
|
||||||
|
|
||||||
from kuryr_tempest_plugin.tests import base
|
from kuryr_tempest_plugin.tests import base
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue