docstrings do not conform to the format in HACKING

[Liusheng]

I have noticed many docstrings in source files do not confirm to the format in HACKING:

http://docs.openstack.org/developer/hacking/#docstrings

For example, the definition of class cinder.api.v2.router.APIRouter:
 class APIRouter(cinder.api.openstack.APIRouter):
    """Routes requests on the API to the appropriate controller and method."""
    ExtensionManager = extensions.ExtensionManager

An extra line should be placed after the docstring.

Not coincidentally, in other projects of openstack, there are many docstrings are lack of standardization.
such as:
def _sanitize_query(query, db_func, on_behalf_of=None):
    '''Check the query to see if:
    1) the request is coming from admin - then allow full visibility
    2) non-admin - make sure that the query includes the requester's
    project.
    '''
    q = copy.copy(query)

in /ceilometer\api\controllers\v2.py:298

I am not sure these should be standardized.

Is this a bug?

[Avishay Traeger (avishay-il)]

Yes, according to HACKING an extra line should be added after the docstring for classes.

[John Griffith (john-griffith)]

I have no problem if someone wants to work on fixing all of these up and turning on additional hacking rules. But as it stands today I wouldn't consider these triaged bugs.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/69867

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/73356