From 7589cc2c370a2724f3d7ca41c96d9dff7ef9b00d Mon Sep 17 00:00:00 2001 From: Martin Kopec Date: Mon, 19 Feb 2024 13:33:58 +0100 Subject: [PATCH] Add pep8 job and fix pep8 issues Adding a pep8 job in check and gate pipeline. This commit also fixes a few pep8 issues. Change-Id: I7688ce8a216eafc4d1b1c2082a7e1aad5b1d15fa --- whitebox_neutron_tempest_plugin/tests/scenario/base.py | 2 +- .../tests/scenario/test_vlan_transparency.py | 9 ++++----- zuul.d/project.yaml | 2 ++ 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/whitebox_neutron_tempest_plugin/tests/scenario/base.py b/whitebox_neutron_tempest_plugin/tests/scenario/base.py index 1da6371..a3b4890 100644 --- a/whitebox_neutron_tempest_plugin/tests/scenario/base.py +++ b/whitebox_neutron_tempest_plugin/tests/scenario/base.py @@ -15,8 +15,8 @@ import re import netaddr -from neutron_tempest_plugin.scenario import base from neutron_tempest_plugin.common import utils as common_utils +from neutron_tempest_plugin.scenario import base from oslo_log import log from tempest import config diff --git a/whitebox_neutron_tempest_plugin/tests/scenario/test_vlan_transparency.py b/whitebox_neutron_tempest_plugin/tests/scenario/test_vlan_transparency.py index 30185bb..7c06bf2 100644 --- a/whitebox_neutron_tempest_plugin/tests/scenario/test_vlan_transparency.py +++ b/whitebox_neutron_tempest_plugin/tests/scenario/test_vlan_transparency.py @@ -20,7 +20,6 @@ from neutron_tempest_plugin.common import ssh from neutron_tempest_plugin import config from neutron_tempest_plugin.scenario import base from oslo_log import log as logging -from tempest.lib.common import fixed_network from tempest.lib.common.utils import data_utils from tempest.lib import decorators from tempest.lib import exceptions @@ -326,8 +325,8 @@ class VlanTransparencyDowntreamBaseTest(base.BaseTempestTestCase): sg_rule = self.create_pingable_secgroup_rule( self.security_group['id'], client=self.client) self.addCleanup( - self.client.delete_security_group_rule, - sg_rule['id']) + self.client.delete_security_group_rule, + sg_rule['id']) class MultiVlanTransparencyTest(VlanTransparencyDowntreamBaseTest): @@ -441,8 +440,8 @@ class MultiPortVlanTransparencyTest(VlanTransparencyDowntreamBaseTest): sg_rule = self.create_pingable_secgroup_rule( self.security_group['id'], client=self.client) self.addCleanup( - self.client.delete_security_group_rule, - sg_rule['id']) + self.client.delete_security_group_rule, + sg_rule['id']) self._check_remote_connectivity_vlan_second_port() diff --git a/zuul.d/project.yaml b/zuul.d/project.yaml index c37d2e7..eb2a795 100644 --- a/zuul.d/project.yaml +++ b/zuul.d/project.yaml @@ -14,9 +14,11 @@ - whitebox-neutron-tempest-plugin-jobs check: jobs: + - openstack-tox-pep8 - whitebox-neutron-tempest-plugin-openvswitch - whitebox-neutron-tempest-plugin-ovn gate: jobs: + - openstack-tox-pep8 - whitebox-neutron-tempest-plugin-openvswitch - whitebox-neutron-tempest-plugin-ovn