4699a8c73d
The snapshot_support extra spec has always meant two things: a driver can take snapshots and create shares from snapshots. As we add alternate snapshot semantics, it is likely that some drivers will want to support snapshots and some of the new semantics while being unable to create new shares from snapshots. This work adds a new extra spec, create_share_from_snapshot_support, that removes the overloading on snapshot_support. It also makes the existing snapshot_support extra spec optional, allowing admins to create types without setting snapshot_support; shares created with such types will not support snapshots. APIImpact DocImpact Co-Authored-By: Goutham Pacha Ravi <gouthamr@netapp.com> Implements: blueprint add-create-share-from-snapshot-extra-spec Change-Id: Ib0ad5fbfdf6297665c208149b08c8d21b3c232be
92 lines
3.9 KiB
Python
92 lines
3.9 KiB
Python
# Copyright 2014 Mirantis Inc.
|
|
# All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
from tempest import config
|
|
from tempest.lib.common.utils import data_utils
|
|
from testtools import testcase as tc
|
|
|
|
from manila_tempest_tests.tests.api import base
|
|
|
|
|
|
CONF = config.CONF
|
|
|
|
|
|
class ShareMultiBackendTest(base.BaseSharesAdminTest):
|
|
|
|
@classmethod
|
|
def resource_setup(cls):
|
|
super(ShareMultiBackendTest, cls).resource_setup()
|
|
if not CONF.share.multi_backend:
|
|
raise cls.skipException("Manila multi-backend tests are disabled.")
|
|
elif len(CONF.share.backend_names) < 2:
|
|
raise cls.skipException("For running multi-backend tests required"
|
|
" two names in config. Skipping.")
|
|
elif any(not name for name in CONF.share.backend_names):
|
|
raise cls.skipException("Share backend names can not be empty. "
|
|
"Skipping.")
|
|
cls.sts = []
|
|
cls.shares = []
|
|
share_data_list = []
|
|
|
|
# Create share types
|
|
for i in [0, 1]:
|
|
st_name = data_utils.rand_name("share-type-%s" % str(i))
|
|
extra_specs = {
|
|
"share_backend_name": CONF.share.backend_names[i],
|
|
}
|
|
st = cls.create_share_type(
|
|
name=st_name,
|
|
extra_specs=cls.add_extra_specs_to_dict(extra_specs))
|
|
cls.sts.append(st["share_type"])
|
|
st_id = st["share_type"]["id"]
|
|
share_data_list.append({"kwargs": {"share_type_id": st_id}})
|
|
|
|
# Create shares using precreated share types
|
|
cls.shares = cls.create_shares(share_data_list)
|
|
|
|
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
|
|
def test_share_backend_name_reporting(self):
|
|
# Share's 'host' should be like "hostname@backend_name"
|
|
for share in self.shares:
|
|
get = self.shares_client.get_share(share['id'])
|
|
self.assertTrue(len(get["host"].split("@")) == 2)
|
|
|
|
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
|
|
def test_share_share_type(self):
|
|
# Share type should be the same as provided with share creation
|
|
for i in [0, 1]:
|
|
get = self.shares_v2_client.get_share(self.shares[i]['id'],
|
|
version="2.5")
|
|
self.assertEqual(self.sts[i]["name"], get["share_type"])
|
|
|
|
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
|
|
def test_share_share_type_v_2_6(self):
|
|
# Share type should be the same as provided with share creation
|
|
for i in [0, 1]:
|
|
get = self.shares_v2_client.get_share(self.shares[i]['id'],
|
|
version="2.6")
|
|
self.assertEqual(self.sts[i]["id"], get["share_type"])
|
|
self.assertEqual(self.sts[i]["name"], get["share_type_name"])
|
|
|
|
@tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)
|
|
def test_share_backend_name_distinction(self):
|
|
# Different share backends should have different host records
|
|
if CONF.share.backend_names[0] == CONF.share.backend_names[1]:
|
|
raise self.skipException("Share backends "
|
|
"configured with same name. Skipping.")
|
|
get1 = self.shares_client.get_share(self.shares[0]['id'])
|
|
get2 = self.shares_client.get_share(self.shares[1]['id'])
|
|
self.assertNotEqual(get1["host"], get2["host"])
|