pylint/pep8: bad-whitespace issue

[Danny Al-Gaaf]

pylint found some pep8 bad-whitespace issues:

************* Module cinder.volume.drivers.netapp.nfs
C:645, 0: No space allowed before bracket
class NetAppDirectNfsDriver (NetAppNFSDriver):
                            ^ (bad-whitespace)
C:723, 0: No space allowed before bracket
class NetAppDirectCmodeNfsDriver (NetAppDirectNfsDriver):
                                 ^ (bad-whitespace)
C:1284, 0: No space allowed before bracket
class NetAppDirect7modeNfsDriver (NetAppDirectNfsDriver):
                                 ^ (bad-whitespace)

[OpenStack Infra (hudson-openstack)]

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

[Duncan Thomas (duncan-thomas)]

The correct fix for this is to fix our PEP8 run in cinder to detect this, just fixing it up by hand does nothing to avoid it catching us out again in future

[Danny Al-Gaaf (danny-al-gaaf)]

It seems that pylint complains but pep8 is fuzzy about this case. From the pep8 code for the E211 code (commit #5c5fde71ace3176d1a9bddaa5f11da4cc577d607):

# Syntax "class A (B):" is allowed, but avoid it

and:

# This is not prohibited by PEP8, but avoid it.
class Foo (Bar, Baz):

If a check is needed it needs to be implemented outside of pep8.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Walter A. Boring IV (hemna) (<email address hidden>) on branch: master
Review: https://review.openstack.org/114042
Reason: abandoned by request of "akerr" in irc.

[Sean McGinnis (sean-mcginnis)]

Automatically unassigning due to inactivity.

[Sean McGinnis (sean-mcginnis)]

I don't believe this is still an issue. Please reopen (or just propose patch to fix) if it is.