]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Dell SC: Fix error causing missed log message
authorSean McGinnis <sean_mcginnis@dell.com>
Tue, 18 Aug 2015 13:08:18 +0000 (08:08 -0500)
committerSean McGinnis <sean_mcginnis@dell.com>
Tue, 18 Aug 2015 13:08:18 +0000 (08:08 -0500)
Extremely trivial fix. An extra comma at the end of a
logging line actually made it into a statement that
doesn't perform logging. This patch removes the extra
comma so that the message gets logged as expected.

Change-Id: I032d5cdd54e3ed8924115744671d8ef9231e62c0

cinder/volume/drivers/dell/dell_storagecenter_api.py

index dab55ec238676aa9d2e81f226f76f3be194a3442..7fdf5b84177f1bd3ebd9cdfa83905e680c49c69e 100644 (file)
@@ -1951,7 +1951,7 @@ class StorageCenterApi(object):
             if (self._update_volume_profiles(self.find_volume(vol['id']),
                                              addid=None,
                                              removeid=profileid)):
-                LOG.info(_LI('Removed %s from cg.'), vol['id']),
+                LOG.info(_LI('Removed %s from cg.'), vol['id'])
             else:
                 LOG.error(_LE('Failed to remove %s from cg.'), vol['id'])
                 return False