openstack/zun
Code Issues Proposed changes

Merge "Add check_multiattach to Local volume driver"

Browse Source
This commit is contained in:
Zuul 2018-08-16 07:51:04 +00:00 committed by Gerrit Code Review
commit 3fcebdabbf
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
zun/container/docker/driver.py
View File

@ -1019,7 +1019,8 @@ class DockerDriver(driver.ContainerDriver):
return volume_driver.get_volume_status(context, volume_mapping) return volume_driver.get_volume_status(context, volume_mapping)
def check_multiattach(self, context, volume_mapping): def check_multiattach(self, context, volume_mapping):
return self.volume_driver.check_multiattach(context, volume_mapping) volume_driver = self._get_volume_driver(volume_mapping)
return volume_driver.check_multiattach(context, volume_mapping)
def _get_or_create_docker_network(self, context, network_api, def _get_or_create_docker_network(self, context, network_api,
neutron_net_id): neutron_net_id):

7
zun/volume/driver.py
View File

@ -82,6 +82,9 @@ class VolumeDriver(object):
def get_volume_status(self, *args, **kwargs): def get_volume_status(self, *args, **kwargs):
raise NotImplementedError() raise NotImplementedError()
def check_multiattach(self, context, volume):
raise NotImplementedError()
class Local(VolumeDriver): class Local(VolumeDriver):
@ -117,6 +120,10 @@ class Local(VolumeDriver):
def get_volume_status(self, context, volume): def get_volume_status(self, context, volume):
return 'available' return 'available'
@validate_volume_provider(supported_providers)
def check_multiattach(self, context, volume):
return False
class Cinder(VolumeDriver): class Cinder(VolumeDriver):