From: Sean McGinnis <sean_mcginnis@dell.com>
Date: Tue, 18 Aug 2015 13:08:18 +0000 (-0500)
Subject: Dell SC: Fix error causing missed log message
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=21e892207a606314ae762bd37041342e10fa7f30;p=openstack-build%2Fcinder-build.git

Dell SC: Fix error causing missed log message

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
---

diff --git a/cinder/volume/drivers/dell/dell_storagecenter_api.py b/cinder/volume/drivers/dell/dell_storagecenter_api.py
index dab55ec23..7fdf5b841 100644
--- a/cinder/volume/drivers/dell/dell_storagecenter_api.py
+++ b/cinder/volume/drivers/dell/dell_storagecenter_api.py
@@ -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