Add nfs as a cinder backup driver option to CinderBackupBackend

Added nfs as an option to where CinderBackupBackend was hardcoded
as either ceph or swift. Also added some parameters for this
driver - CinderBackupNfsShare and CinderBackupNfsMountOptions

Depends-On: Ic0adb294aa2e60243f8adaf167bdd75e42c8e20e
Change-Id: I29a488374726676a28fb82f2f950db891fcf9627
Closes-Bug: #1744174
This commit is contained in:
Marc Methot 2018-01-19 09:14:55 -05:00
parent 4c58be93a7
commit e456e103fb
5 changed files with 22 additions and 4 deletions

View File

@ -15,7 +15,7 @@ parameters:
description: The short name of the Cinder Backup backend to use. description: The short name of the Cinder Backup backend to use.
type: string type: string
constraints: constraints:
- allowed_values: ['swift', 'ceph'] - allowed_values: ['swift', 'ceph', 'nfs']
CinderBackupRbdPoolName: CinderBackupRbdPoolName:
default: backups default: backups
type: string type: string

View File

@ -20,7 +20,7 @@ parameter_defaults:
CinderEnableIscsiBackend: false CinderEnableIscsiBackend: false
## Whether to enable rbd (Ceph) backend for Cinder. ## Whether to enable rbd (Ceph) backend for Cinder.
CinderEnableRbdBackend: true CinderEnableRbdBackend: true
## Cinder Backup backend can be either 'ceph' or 'swift'. ## Cinder Backup backend can be either 'ceph', 'swift' or 'nfs'.
CinderBackupBackend: ceph CinderBackupBackend: ceph
## Whether to enable NFS backend for Cinder. ## Whether to enable NFS backend for Cinder.
# CinderEnableNfsBackend: false # CinderEnableNfsBackend: false

View File

@ -9,7 +9,7 @@ parameters:
description: The short name of the Cinder Backup backend to use. description: The short name of the Cinder Backup backend to use.
type: string type: string
constraints: constraints:
- allowed_values: ['swift', 'ceph'] - allowed_values: ['swift', 'ceph', 'nfs']
CinderBackupRbdPoolName: CinderBackupRbdPoolName:
default: backups default: backups
type: string type: string
@ -25,6 +25,15 @@ parameters:
CephClientUserName: CephClientUserName:
default: openstack default: openstack
type: string type: string
CinderBackupNfsShare:
default: ''
description: NFS share to be mounted
type: string
CinderBackupNfsMountOptions:
default: ''
description: Mount options passed to the NFS client. See NFS man
page for details.
type: string
ServiceData: ServiceData:
default: {} default: {}
description: Dictionary packing service data description: Dictionary packing service data
@ -85,6 +94,8 @@ outputs:
- {get_param: CephClusterName} - {get_param: CephClusterName}
- '.conf' - '.conf'
cinder::backup::swift::backup_swift_container: volumebackups cinder::backup::swift::backup_swift_container: volumebackups
cinder::backup::nfs::backup_share: {get_param: CinderBackupNfsShare}
cinder::backup::nfs::backup_mount_options: {get_param: CinderBackupNfsMountOptions}
step_config: step_config:
str_replace: str_replace:
template: "include ::tripleo::profile::base::cinder::backup::DRIVER" template: "include ::tripleo::profile::base::cinder::backup::DRIVER"

View File

@ -9,7 +9,7 @@ parameters:
description: The short name of the Cinder Backup backend to use. description: The short name of the Cinder Backup backend to use.
type: string type: string
constraints: constraints:
- allowed_values: ['swift', 'ceph'] - allowed_values: ['swift', 'ceph', 'nfs']
CinderBackupRbdPoolName: CinderBackupRbdPoolName:
default: backups default: backups
type: string type: string

View File

@ -0,0 +1,7 @@
---
features:
- |
Adds support for configuring the cinder-backup service with an NFS backend.
fixes:
- |
Fixes `bug 1744174 <https://bugs.launchpad.net/tripleo/+bug/1744174>`__.