EMC VNX Direct Driver needs to be cleaned up

[Xing Yang]

There are quite a few review comments on the EMC VNX Direct Driver:

https://review.openstack.org/#/c/73672

These issues need to be addressed right after the code is merged.

[John Griffith (john-griffith)]

Umm... this seems really wrong to me. First if the patch wasn't ready it shouldn't have been merged, it's either ready or it's not. Granted if it's nit picky things that don't effect the functionality of the patch it's not so bad to land it and work on it later.

But there's been this weird trend started lately of creating these "generic catch-all bugs", like "fix cert test issues" which is complete which does us no good whatsoever for trackign info and defeats the purpose of bug tracking. This is another example of useless non-informational bugs ("alot of review comments")??

If you need to file a bug, file a bug with real bug information. If you have cleanup work to do in the driver submit a patch with that cleanup work and document what you're doing. We can make an exception for a merge in RC for things like drivers if we need to and if the changes are actually important.

I'm not even remotely concerned about spelling errors or comment forwarding at this point, any of those things should have been fixed in the first place.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/78782
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=48955e56b886c0da51c1555aca62e099761ad99b
Submitter: Jenkins
Branch: master

commit 48955e56b886c0da51c1555aca62e099761ad99b
Author: Xing Yang <email address hidden>
Date: Thu Mar 6 16:50:26 2014 -0500

    Clean Up EMC VNX Direct Driver in Cinder

    This patch cleans up issues discovered during the review of
    EMC VNX Direct Driver.

    https://review.openstack.org/#/c/73672/

    Implements blueprint emc-vnx-direct-driver
    Closes-Bug: #1287944

    Change-Id: I4002ef9ea14e2d843dd8cbccffa025997a54c738