remove E112 hacking exemption and fix errors
Change-Id: I6031ad2274c677b071a03068e5fb3c000fec7723 Partial-Bug: 1329017
This commit is contained in:
parent
db3f4b3156
commit
ca4d6b173e
|
@ -191,6 +191,7 @@ class N1kvNeutronPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
|
||||||
# Add profiles in database if new profiles were created in VSM
|
# Add profiles in database if new profiles were created in VSM
|
||||||
for pid in vsm_profiles_set - plugin_profiles_set:
|
for pid in vsm_profiles_set - plugin_profiles_set:
|
||||||
self._add_policy_profile(vsm_profiles[pid], pid)
|
self._add_policy_profile(vsm_profiles[pid], pid)
|
||||||
|
|
||||||
# Delete profiles from database if profiles were deleted in VSM
|
# Delete profiles from database if profiles were deleted in VSM
|
||||||
for pid in plugin_profiles_set - vsm_profiles_set:
|
for pid in plugin_profiles_set - vsm_profiles_set:
|
||||||
self._delete_policy_profile(pid)
|
self._delete_policy_profile(pid)
|
||||||
|
|
|
@ -694,7 +694,7 @@ class MidonetPluginV2(db_base_plugin_v2.NeutronDbPluginV2,
|
||||||
self.client.add_dhcp_route_option(
|
self.client.add_dhcp_route_option(
|
||||||
bridge, cidr, ip, METADATA_DEFAULT_IP)
|
bridge, cidr, ip, METADATA_DEFAULT_IP)
|
||||||
else:
|
else:
|
||||||
# IPs have changed. Re-map the DHCP entries
|
# IPs have changed. Re-map the DHCP entries
|
||||||
for cidr, ip, mac in self._dhcp_mappings(
|
for cidr, ip, mac in self._dhcp_mappings(
|
||||||
context, old_ips, mac):
|
context, old_ips, mac):
|
||||||
self.client.remove_dhcp_host(
|
self.client.remove_dhcp_host(
|
||||||
|
|
3
tox.ini
3
tox.ini
|
@ -40,7 +40,6 @@ commands =
|
||||||
commands = {posargs}
|
commands = {posargs}
|
||||||
|
|
||||||
[flake8]
|
[flake8]
|
||||||
# E112 expected an idented block
|
|
||||||
# E122 continuation line missing indentation or outdented
|
# E122 continuation line missing indentation or outdented
|
||||||
# E125 continuation line does not distinguish itself from next logical line
|
# E125 continuation line does not distinguish itself from next logical line
|
||||||
# E126 continuation line over-indented for hanging indent
|
# E126 continuation line over-indented for hanging indent
|
||||||
|
@ -63,7 +62,7 @@ commands = {posargs}
|
||||||
# H405 multi line docstring summary not separated with an empty line
|
# H405 multi line docstring summary not separated with an empty line
|
||||||
# H904 Wrap long lines in parentheses instead of a backslash
|
# H904 Wrap long lines in parentheses instead of a backslash
|
||||||
# TODO(marun) H404 multi line docstring should start with a summary
|
# TODO(marun) H404 multi line docstring should start with a summary
|
||||||
ignore = E112,E122,E125,E126,E128,E129,E251,E265,E713,F402,F811,F812,H104,H237,H302,H304,H305,H307,H401,H402,H404,H405,H904
|
ignore = E122,E125,E126,E128,E129,E251,E265,E713,F402,F811,F812,H104,H237,H302,H304,H305,H307,H401,H402,H404,H405,H904
|
||||||
show-source = true
|
show-source = true
|
||||||
builtins = _
|
builtins = _
|
||||||
exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject
|
exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject
|
||||||
|
|
Loading…
Reference in New Issue