openstack/os-vif
Code Issues Proposed changes

Merge "trivial: Remove some rules from flake8 ignore list"

Browse Source
This commit is contained in:
Zuul 2020-05-01 12:29:27 +00:00 committed by Gerrit Code Review
commit 222e286661
3 changed files with 8 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
tox.ini
View File

@ -61,30 +61,14 @@ commands =
# E123, E125 skipped as they are invalid PEP-8. # E123, E125 skipped as they are invalid PEP-8.
# Following checks are ignored on purpose. # Following checks are ignored on purpose.
# #
# E251 unexpected spaces around keyword / parameter equals # H404, H405 skipped on purpose per jay pipes discussion.
# reason: no improvement in readability
#
# E265 block comment should start with '# '
# reason: no improvement in readability
#
# H904 wrap long lines in parentheses instead of a backslash
# reason: removed in hacking (https://review.opendev.org/#/c/101701/)
#
# H404 skipped on purpose per jay pipes discussion.
#
# W504 line break after binary operator # W504 line break after binary operator
#
# Due to the upgrade to hacking 0.9.2 the following checking are
# ignored on purpose for the moment and should be re-enabled.
# H106: Don’t put vim configuration in source files
# H203: Use assertIs(Not)None to check for None
show-source = True show-source = True
ignore = E123,E125,E126,E127,E128,E251,E265,H302,H405,H904,H404,W504 ignore = E123,E125,E126,E127,E128,H404,H405,W504
enable-extensions=H106,H203 enable-extensions = H106,H203
builtins = _ builtins = _
exclude=.venv,.git,.tox,dist,*lib/python*,*egg,build exclude = .venv,.git,.tox,dist,*lib/python*,*egg,build
max-complexity=30 max-complexity = 30
[hacking] [hacking]
import_exceptions = os_vif.i18n import_exceptions = os_vif.i18n

2
vif_plug_linux_bridge/tests/unit/test_linux_net.py
View File

@ -130,7 +130,7 @@ class LinuxNetTest(testtools.TestCase):
calls = [mock.call('fake-interface', 1500), calls = [mock.call('fake-interface', 1500),
mock.call('fake-bridge', 1500)] mock.call('fake-bridge', 1500)]
mock_set_mtu.assert_has_calls(calls) mock_set_mtu.assert_has_calls(calls)
calls = [mock.call('fake-bridge', state = 'up'), calls = [mock.call('fake-bridge', state='up'),
mock.call('fake-interface', master='fake-bridge', state='up', mock.call('fake-interface', master='fake-bridge', state='up',
check_exit_code=[0, 2, 254])] check_exit_code=[0, 2, 254])]
mock_ip_set.assert_has_calls(calls) mock_ip_set.assert_has_calls(calls)

6
vif_plug_ovs/ovs.py
View File

@ -100,14 +100,12 @@ class OvsPlugin(plugin.PluginBase):
def describe(self): def describe(self):
pp_ovs = objects.host_info.HostPortProfileInfo( pp_ovs = objects.host_info.HostPortProfileInfo(
profile_object_name= profile_object_name=objects.vif.VIFPortProfileOpenVSwitch.__name__, # noqa
objects.vif.VIFPortProfileOpenVSwitch.__name__,
min_version="1.0", min_version="1.0",
max_version="1.0", max_version="1.0",
) )
pp_ovs_representor = objects.host_info.HostPortProfileInfo( pp_ovs_representor = objects.host_info.HostPortProfileInfo(
profile_object_name= profile_object_name=objects.vif.VIFPortProfileOVSRepresentor.__name__, # noqa
objects.vif.VIFPortProfileOVSRepresentor.__name__,
min_version="1.0", min_version="1.0",
max_version="1.0", max_version="1.0",
) )