openstack/cinder
Code Issues Proposed changes

Merge "Fix backup NFS share mount with default backup_mount_options"

Browse Source
This commit is contained in:
Jenkins 2016-10-03 20:26:28 +00:00 committed by Gerrit Code Review
commit 3151fdfdd7
2 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/backup/drivers/nfs.py
View File

@ -53,7 +53,7 @@ class NFSBackupDriver(posix.PosixBackupDriver):
self._check_configuration()
self.backup_mount_point_base = CONF.backup_mount_point_base
self.backup_share = CONF.backup_share
self.mount_options = CONF.backup_mount_options or {}
self.mount_options = CONF.backup_mount_options
backup_path = self._init_backup_repo_path()
LOG.debug("Using NFS backup repository: %s", backup_path)
super(NFSBackupDriver, self).__init__(context,

12
cinder/tests/unit/backup/drivers/test_backup_nfs.py
View File

@ -73,7 +73,8 @@ class BackupNFSShareTestCase(test.TestCase):
self.assertRaises(exception.ConfigNotFound,
driver._check_configuration)
def test_init_backup_repo_path(self):
@mock.patch.object(remotefs_brick, 'RemoteFsClient')
def test_init_backup_repo_path(self, mock_remotefs_client_class):
self.override_config('backup_share', FAKE_BACKUP_SHARE)
self.override_config('backup_mount_point_base',
FAKE_BACKUP_MOUNT_POINT_BASE)
@ -81,8 +82,7 @@ class BackupNFSShareTestCase(test.TestCase):
mock_remotefsclient.get_mount_point = mock.Mock(
return_value=FAKE_BACKUP_PATH)
self.mock_object(nfs.NFSBackupDriver, '_check_configuration')
self.mock_object(remotefs_brick, 'RemoteFsClient',
mock.Mock(return_value=mock_remotefsclient))
mock_remotefs_client_class.return_value = mock_remotefsclient
self.mock_object(utils, 'get_root_helper')
with mock.patch.object(nfs.NFSBackupDriver, '_init_backup_repo_path'):
driver = nfs.NFSBackupDriver(self.ctxt)
@ -91,6 +91,12 @@ class BackupNFSShareTestCase(test.TestCase):
self.assertEqual(FAKE_BACKUP_PATH, path)
utils.get_root_helper.called_once()
mock_remotefs_client_class.assert_called_once_with(
'nfs',
utils.get_root_helper(),
nfs_mount_point_base=FAKE_BACKUP_MOUNT_POINT_BASE,
nfs_mount_options=None
)
mock_remotefsclient.mount.assert_called_once_with(FAKE_BACKUP_SHARE)
mock_remotefsclient.get_mount_point.assert_called_once_with(
FAKE_BACKUP_SHARE)