openinfra/python-tempestconf
Code Issues Proposed changes

Fix manila backend_names and enable_protocols

Browse Source 
A backend in manila can have multiple storage pools
and capabilities for each pool are considered to
create configuration for tempest tests. These
capabilities however may overlap with each other,
so we need some de-duplication.

Change-Id: I108615ab197b8959fe944e21ae7c1a36125ca531
This commit is contained in:
Goutham Pacha Ravi 2021-03-19 12:45:48 -07:00
parent 84e406d530
commit d625ec79c2
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
config_tempest/services/share.py
View File

@ -44,12 +44,12 @@ class ShareService(VersionedService):
"pools - storage back-ends can't be discovered.")
return
if pools:
backends = []
enable_protocols = []
backends = set()
enable_protocols = set()
for pool in pools:
backends.append(pool['backend'])
backends.add(pool['backend'])
protocol = pool['capabilities']['storage_protocol'].lower()
enable_protocols.extend(protocol.split('_'))
enable_protocols.update(protocol.split('_'))
conf.set('share', 'backend_names', ','.join(backends))
conf.set('share', 'enable_protocols', ','.join(enable_protocols))