storpool driver doesn't follow recommended logging format

[John Griffith]

storpool driver is super light on Logging (maybe that's good, maybe that's bad). Seems like there could be a bit more logging added here to help operators out.

The entirety of the logging is shown below, and the formatting of the logging messages is incorrect:

                LOG.error(
                    _LE('Could not remove the temp snapshot {n} for {v}: {e}').
                    format(n=name, v=volname, e=six.text_type(e)))

                LOG.error(
                    _LE('Could not remove the temp snapshot {n} for {v}: {e}').
                    format(n=name, v=volname, e=six.text_type(e)))

                LOG.error(_LE("Could not delete the temp snapshot {n}: {msg}").
                          format(n=snapname, msg=six.text_type(e)))

               LOG.error(_LE("StorPoolDriver API initialization failed: {e}").
                         format(e=e))

Should follow guidelines from here: http://docs.openstack.org/developer/oslo.i18n/guidelines.html

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/143762
Reason: Over a month with no update.

[Hardik Parekh (hardik-parekh047)]

This file is no longer exist so This is not a valid bug.