]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
enable H401 hacking check
authorMark McClain <mmcclain@yahoo-inc.com>
Wed, 19 Nov 2014 22:30:10 +0000 (17:30 -0500)
committerMark McClain <mmcclain@yahoo-inc.com>
Thu, 20 Nov 2014 15:20:47 +0000 (10:20 -0500)
This change corrects H401 (docstring should not start with a space)
errors and enables the check
Change-Id: I2f798421d5895fcec349c757d27b7f085ee001e7
Partial-Bug: 1329017

neutron/db/l3_dvrscheduler_db.py
neutron/plugins/bigswitch/routerrule_db.py
neutron/plugins/ml2/drivers/arista/arista_l3_driver.py
neutron/plugins/ml2/drivers/freescale/config.py
neutron/plugins/sriovnicagent/sriov_nic_agent.py
neutron/services/l3_router/brocade/l3_router_plugin.py
tox.ini

index d717b0d5d0c38e60e8d07bbdd24b1b99802c5d5e..c2a8aacf346f18b4899585f4424c2d9e234caa64 100644 (file)
@@ -282,7 +282,7 @@ class L3_DVRsch_db_mixin(l3agent_sch_db.L3AgentSchedulerDbMixin):
                   'agent %(id)s', {'router_id': router_id, 'id': agent_id})
 
     def get_snat_bindings(self, context, router_ids):
-        """ Retrieves the dvr snat bindings for a router."""
+        """Retrieves the dvr snat bindings for a router."""
         if not router_ids:
             return []
         query = context.session.query(CentralizedSnatL3AgentBinding)
index d6dcc08c1044408e06b51d1d1941942df897f870..8bdbe7276bc03ab5cad532a60c14e88ac6d90846 100644 (file)
@@ -46,7 +46,7 @@ class NextHop(model_base.BASEV2):
 
 
 class RouterRule_db_mixin(l3_db.L3_NAT_db_mixin):
-    """ Mixin class to support route rule configuration on a router"""
+    """Mixin class to support route rule configuration on a router."""
     def update_router(self, context, id, router):
         r = router['router']
         with context.session.begin(subtransactions=True):
index 0d3190eddcf90dde0c570992f94ed9c65423cb33..36b19be90cba1597f868ef0a49864277c59b3c16 100644 (file)
@@ -408,7 +408,7 @@ class AristaL3Driver(object):
         return socket.inet_ntop(socket.AF_INET6, struct.pack("!QQ", hi, lo))
 
     def _get_router_ip(self, cidr, ip_count, ip_ver):
-        """ For a given IP subnet and IP version type, generate IP for router.
+        """For a given IP subnet and IP version type, generate IP for router.
 
         This method takes the network address (cidr) and selects an
         IP address that should be assigned to virtual router running
index 7739dc1ab9d59bb62b9f7272be51e7062984407c..fe1b3c6fa2dfd38009d82e4670d8ae7ba80eb5e1 100644 (file)
@@ -16,7 +16,7 @@
 from neutronclient.v2_0 import client
 from oslo.config import cfg
 
-""" Freescale CRD Server Configuration used by ML2 Mechanism Driver.
+"""Freescale CRD Server Configuration used by ML2 Mechanism Driver.
 
 The following configuration is used by Freescale Drivers/Plugin
 like, FWaaS Plugin, VPNaaS Plugin etc.. which connect to Cloud Resource
@@ -68,7 +68,7 @@ SERVICE_TYPE = 'crd'
 
 
 def get_crdclient():
-    """ Using the CRD configuration, get and return CRD Client instance."""
+    """Using the CRD configuration, get and return CRD Client instance."""
     crd_client_params = {
         'username': FSLCONF.crd_user_name,
         'tenant_name': FSLCONF.crd_tenant_name,
index dbfc03c73f03f74c3df70238d6017297705e0602..c11bf6db77e10a4c4183666e160957527e954bd2 100644 (file)
@@ -306,7 +306,7 @@ class SriovNicAgentConfigParser(object):
         self._validate()
 
     def _validate(self):
-        """ Validate configuration.
+        """Validate configuration.
 
         Validate that network_device in excluded_device
         exists in device mappings
index 1826b83a6aed7522f7d34fbd2a24612a6941eab1..78c920ea0c40567c8c35e1a9727e026d851f4f4a 100644 (file)
@@ -73,7 +73,7 @@ class BrocadeSVIPlugin(router.L3RouterPlugin):
         LOG.info(_("rbridge id %s"), self._switch['rbridge_id'])
 
     def create_router(self, context, router):
-        """ creates a vrf on NOS device."""
+        """Creates a vrf on NOS device."""
         LOG.debug("BrocadeSVIPlugin.create_router called: ")
         with context.session.begin(subtransactions=True):
             new_router = super(BrocadeSVIPlugin, self).create_router(context,
@@ -98,7 +98,7 @@ class BrocadeSVIPlugin(router.L3RouterPlugin):
         return new_router
 
     def delete_router(self, context, router_id):
-        """ delete a vrf on NOS device."""
+        """Delete a vrf on NOS device."""
         router = super(BrocadeSVIPlugin, self).get_router(context, router_id)
         super(BrocadeSVIPlugin, self).delete_router(context, router_id)
 
diff --git a/tox.ini b/tox.ini
index d38ef0cdf526fb33bdf97940bf39136589072452..ee77332f121c09626439ba80083a9cb68b634357 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -67,12 +67,11 @@ commands = python setup.py build_sphinx
 # E265 block comment should start with ‘# ‘
 # H305 imports not grouped correctly
 # H307 like imports should be grouped together
-# H401 docstring should not start with a space
 # H402 one line docstring needs punctuation
 # H405 multi line docstring summary not separated with an empty line
 # H904 Wrap long lines in parentheses instead of a backslash
 # TODO(marun) H404 multi line docstring should start with a summary
-ignore = E125,E126,E128,E129,E265,H305,H307,H401,H402,H404,H405,H904
+ignore = E125,E126,E128,E129,E265,H305,H307,H402,H404,H405,H904
 show-source = true
 builtins = _
 exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject,rally-scenarios