diff --git a/cinder/tests/unit/test_solidfire.py b/cinder/tests/unit/test_solidfire.py
index a3a4b2d7af4..5210e0846d8 100644
--- a/cinder/tests/unit/test_solidfire.py
+++ b/cinder/tests/unit/test_solidfire.py
@@ -1046,6 +1046,6 @@ class SolidFireVolumeTestCase(test.TestCase):
 
         with mock.patch.object(
                 sfv, '_issue_api_request', side_effect=_fake_issue_api_req):
-            updates = sfv._init_volume_mappings(vrefs)
-            self.assertEqual(99, updates[0]['provider_id'])
-            self.assertEqual(1, len(updates))
+            volume_updates, snapshot_updates = sfv.update_provider_info(vrefs)
+            self.assertEqual(99, volume_updates[0]['provider_id'])
+            self.assertEqual(1, len(volume_updates))
diff --git a/cinder/volume/drivers/solidfire.py b/cinder/volume/drivers/solidfire.py
index 48cdfb89d66..f662c382612 100644
--- a/cinder/volume/drivers/solidfire.py
+++ b/cinder/volume/drivers/solidfire.py
@@ -205,7 +205,9 @@ class SolidFireDriver(san.SanISCSIDriver):
         return updates
 
     def update_provider_info(self, vrefs):
-        return self._init_volume_mappings(vrefs)
+        volume_updates = self._init_volume_mappings(vrefs)
+        snapshot_updates = None
+        return (volume_updates, snapshot_updates)
 
     def _create_template_account(self, account_name):
         # We raise an API exception if the account doesn't exist