summaryrefslogtreecommitdiffstats
path: root/tests/functional/common/provisioning/test_storage_class_cases.py
diff options
context:
space:
mode:
authorValerii Ponomarov <vponomar@redhat.com>2018-11-21 22:25:57 +0530
committerValerii Ponomarov <vponomar@redhat.com>2018-11-27 17:56:20 +0530
commitee94c5545589e7c2553d7aead517c2ba587aaa3b (patch)
tree7887bd34b84cc59090c123972453ef4f0761e2a8 /tests/functional/common/provisioning/test_storage_class_cases.py
parentfb0a773bcdb357331e8c3e5f2c09e89aaeeb86fd (diff)
Cleanup and improve configuration
Our config file has dead options as well as unintuitive option names. So, do following improvements: - Delete unused config options from config file example. - Delete setting of dead config options to the base class. - Rename 'storage_class1' and 'storage_class2' config option groups to the 'file_storage_class' and 'block_storage_class' respectively to make it more understandable. - Remove 'secrets' config option group as redundant. Secret names get calculated from custom prefix and randomly generated string. Namespace is used the same as for storage classes. And its 'type' was always storage class's provisioner value. Change-Id: Ifa6d0ab430e8d69591f49bc7bf4c258151662550
Diffstat (limited to 'tests/functional/common/provisioning/test_storage_class_cases.py')
-rw-r--r--tests/functional/common/provisioning/test_storage_class_cases.py24
1 files changed, 16 insertions, 8 deletions
diff --git a/tests/functional/common/provisioning/test_storage_class_cases.py b/tests/functional/common/provisioning/test_storage_class_cases.py
index 7e318eb0..52ac761a 100644
--- a/tests/functional/common/provisioning/test_storage_class_cases.py
+++ b/tests/functional/common/provisioning/test_storage_class_cases.py
@@ -34,12 +34,16 @@ class TestStorageClassCases(cns_baseclass.CnsBaseClass):
parameter (dict): dictionary with storage class parameters
"""
if vol_type == "glusterfile":
- sc = self.cns_storage_class['storage_class1']
- secret = self.cns_secret['secret1']
+ sc = self.cns_storage_class.get(
+ 'storage_class1',
+ self.cns_storage_class.get('file_storage_class'))
+
# Create secret file for usage in storage class
self.secret_name = oc_create_secret(
- self.ocp_master_node[0], namespace=secret['namespace'],
- data_key=self.heketi_cli_key, secret_type=secret['type'])
+ self.ocp_master_node[0],
+ namespace=sc.get('secretnamespace', 'default'),
+ data_key=self.heketi_cli_key,
+ secret_type=sc.get('provisioner', 'kubernetes.io/glusterfs'))
self.addCleanup(
oc_delete, self.ocp_master_node[0], 'secret', self.secret_name)
sc_parameter = {
@@ -48,12 +52,16 @@ class TestStorageClassCases(cns_baseclass.CnsBaseClass):
"volumetype": "replicate:3"
}
elif vol_type == "glusterblock":
- sc = self.cns_storage_class['storage_class2']
- secret = self.cns_secret['secret2']
+ sc = self.cns_storage_class.get(
+ 'storage_class2',
+ self.cns_storage_class.get('block_storage_class'))
+
# Create secret file for usage in storage class
self.secret_name = oc_create_secret(
- self.ocp_master_node[0], namespace=secret['namespace'],
- data_key=self.heketi_cli_key, secret_type=secret['type'])
+ self.ocp_master_node[0],
+ namespace=sc.get('restsecretnamespace', 'default'),
+ data_key=self.heketi_cli_key,
+ secret_type=sc.get('provisioner', 'gluster.org/glusterblock'))
self.addCleanup(
oc_delete, self.ocp_master_node[0], 'secret', self.secret_name)
sc_parameter = {