x/vmware-nsx
Code Issues Proposed changes

Merge "enable F812 check for flake8"

Browse Source
This commit is contained in:
Jenkins 2014-11-18 00:56:20 +00:00 committed by Gerrit Code Review
commit a67d68e862
5 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
neutron/plugins/cisco/db/l3/device_handling_db.py
View File

@ -303,8 +303,8 @@ class DeviceHandlingMixin(object):
agents = [hosting_device.cfg_agent for hosting_device in query agents = [hosting_device.cfg_agent for hosting_device in query
if hosting_device.cfg_agent is not None] if hosting_device.cfg_agent is not None]
if active is not None: if active is not None:
agents = [agent for agent in agents if not agents = [a for a in agents if not
self.is_agent_down(agent['heartbeat_timestamp'])] self.is_agent_down(a['heartbeat_timestamp'])]
return agents return agents
def auto_schedule_hosting_devices(self, context, agent_host): def auto_schedule_hosting_devices(self, context, agent_host):

3
neutron/plugins/sriovnicagent/pci_lib.py
View File

@ -63,8 +63,7 @@ class PciDeviceIPWrapper(ip_lib.IPWrapper):
vf_details = self._parse_vf_link_show(vf_line) vf_details = self._parse_vf_link_show(vf_line)
if vf_details: if vf_details:
vf_details_list.append(vf_details) vf_details_list.append(vf_details)
return [vf_details.get("MAC") for vf_details in return [details.get("MAC") for details in vf_details_list]
vf_details_list]
def get_vf_state(self, vf_index): def get_vf_state(self, vf_index):
"""Get vf state {True/False} """Get vf state {True/False}

2
neutron/plugins/vmware/common/sync.py
View File

@ -330,7 +330,7 @@ class NsxSynchronizer():
for network in networks: for network in networks:
lswitches = neutron_nsx_mappings.get(network['id'], []) lswitches = neutron_nsx_mappings.get(network['id'], [])
lswitches = [lswitch.get('data') for lswitch in lswitches] lswitches = [lsw.get('data') for lsw in lswitches]
self.synchronize_network(ctx, network, lswitches) self.synchronize_network(ctx, network, lswitches)
def synchronize_router(self, context, neutron_router_data, def synchronize_router(self, context, neutron_router_data,

4
neutron/services/vpn/device_drivers/ipsec.py
View File

@ -692,9 +692,7 @@ class IPsecDriver(device_drivers.DeviceDriver):
# Delete any IPSec processes running # Delete any IPSec processes running
# VPN that do not have an associated router. # VPN that do not have an associated router.
process_ids = [process_id process_ids = [pid for pid in self.processes if pid not in router_ids]
for process_id in self.processes
if process_id not in router_ids]
for process_id in process_ids: for process_id in process_ids:
self.destroy_router(process_id) self.destroy_router(process_id)
self.report_status(context) self.report_status(context)

4
tox.ini
View File

@ -65,7 +65,7 @@ commands = python setup.py build_sphinx
# E128 continuation line under-indented for visual indent # E128 continuation line under-indented for visual indent
# E129 visually indented line with same indent as next logical line # E129 visually indented line with same indent as next logical line
# E265 block comment should start with ‘# ‘ # E265 block comment should start with ‘# ‘
# F812 list comprehension redefines name from line # H104 file contains nothing but comments
# H237 module is removed in Python 3 # H237 module is removed in Python 3
# H305 imports not grouped correctly # H305 imports not grouped correctly
# H307 like imports should be grouped together # H307 like imports should be grouped together
@ -74,7 +74,7 @@ commands = python setup.py build_sphinx
# 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 = E125,E126,E128,E129,E265,F812,H237,H305,H307,H401,H402,H404,H405,H904 ignore = E125,E126,E128,E129,E265,H237,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,rally-scenarios exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject,rally-scenarios