openstack/cinder
Code Issues Proposed changes

Consist terminate_connection function signature

Browse Source 
Clean up the "force=False" in function signature for terminate_connection
since it already accept **kwargs.

Thus make all function signature for terminate_connection consist to
    def terminate_connection(self, volume, connector, **kwargs)

Fixes: bug #1190419
Change-Id: I8765cb8f2aa3cfbb49ff57ae034b148aee35a8c4
This commit is contained in:
xiaoxi_chen
2013-07-09 10:18:30 +08:00
parent e958394668
commit f76d1ee4dd
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/volume/driver.py
View File

@@ -159,7 +159,7 @@ class VolumeDriver(object):
"""Allow connection to connector and return connection info."""
raise NotImplementedError()
def terminate_connection(self, volume, connector, force=False, **kwargs):
def terminate_connection(self, volume, connector, **kwargs):
"""Disallow connection from connector"""
raise NotImplementedError()

2
cinder/volume/drivers/scality.py
View File

@@ -206,7 +206,7 @@ class ScalityDriver(driver.VolumeDriver):
}
}
def terminate_connection(self, volume, connector, force=False, **kwargs):
def terminate_connection(self, volume, connector, **kwargs):
"""Disallow connection from connector."""
pass

2
cinder/volume/drivers/xenapi/sm.py
View File

@@ -118,7 +118,7 @@ class XenAPINFSDriver(driver.VolumeDriver):
)
)
def terminate_connection(self, volume, connector, force=False, **kwargs):
def terminate_connection(self, volume, connector, **kwargs):
pass
def check_for_setup_error(self):