openstack/devstack-plugin-nfs
Code Issues Proposed changes

Don't run volume snapshot/clone tests in Tempest

Browse Source 
The Cinder NFS backend doesn't yet support snapshot, clone
or backup operations, so they should be disabled when
running Tempest.

We also have to explicitly disable the image cache in
cinder because that relies on volume cloning which the NFS
volume backend doesn't support.

Depends-On: I5ad578a78d6cfd4186a8230ad472d4a7b92445c9
Change-Id: I8fb421276005f9ba9cbc6c55ba8b6089349fdb69
This commit is contained in:
Matt Riedemann 2016-11-09 20:26:26 -05:00 committed by Diana Clarke
parent ebf915c8bf
commit fc041321ae
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
devstack/override-defaults
View File

@ -3,6 +3,8 @@ ENABLE_NFS_CINDER=$(trueorfalse True ENABLE_NFS_CINDER)
if [[ $ENABLE_NFS_CINDER == "True" ]]; then if [[ $ENABLE_NFS_CINDER == "True" ]]; then
CINDER_DRIVER=nfs CINDER_DRIVER=nfs
CINDER_ENABLED_BACKENDS=nfs:nfs CINDER_ENABLED_BACKENDS=nfs:nfs
# NOTE(mriedem): This is a workaround for Cinder bug 1642394.
CINDER_IMG_CACHE_ENABLED=False
TEMPEST_STORAGE_PROTOCOL=nfs TEMPEST_STORAGE_PROTOCOL=nfs
fi fi

12
devstack/plugin.sh
View File

@ -78,6 +78,14 @@ function configure_cinder_nfs {
} }
# Configures tempest for running Cinder volume API tests with an NFS backend.
function configure_tempest_nfs {
# The Cinder NFS backend doesn't yet support snapshot, backup or clone.
iniset $TEMPEST_CONFIG volume-feature-enabled snapshot False
iniset $TEMPEST_CONFIG volume-feature-enabled backup False
iniset $TEMPEST_CONFIG volume-feature-enabled clone False
}
if [[ "$1" == "stack" && "$2" == "pre-install" ]]; then if [[ "$1" == "stack" && "$2" == "pre-install" ]]; then
echo_summary "Installing NFS" echo_summary "Installing NFS"
@ -91,6 +99,10 @@ elif [[ "$1" == "stack" && "$2" == "post-config" ]]; then
configure_cinder_nfs configure_cinder_nfs
sudo service $NFS_SERVICE restart sudo service $NFS_SERVICE restart
fi fi
elif [[ "$1" == "stack" && "$2" == "test-config" ]]; then
if is_nfs_enabled_for_service cinder; then
configure_tempest_nfs
fi
fi fi
if [[ "$1" == "unstack" ]]; then if [[ "$1" == "unstack" ]]; then