Don't fall when just loading barbicanv1 modules
this avoids loading warnings on salt-master that may not have
os-client-config installed, as salt tries to load them irrespective of
__virtual__() result.
Change-Id: I819d46d5dbd0339e7a5e53609d43e34da992db60
Related-Issue: https://mirantis.jira.com/browse/PROD-26920
(cherry picked from commit 7cd57571cd993afea59b5742d5d8cce22612a217)
diff --git a/_modules/barbicanv1/common.py b/_modules/barbicanv1/common.py
index f6a46ff..80d927f 100644
--- a/_modules/barbicanv1/common.py
+++ b/_modules/barbicanv1/common.py
@@ -1,9 +1,16 @@
import functools
import logging
-import os_client_config
import time
from uuid import UUID
+
+try:
+ import os_client_config
+except ImportError:
+ os_client_config = None
+from salt import exceptions
+
+
try:
from urllib.parse import urlsplit
except ImportError:
@@ -50,6 +57,10 @@
def _get_raw_client(cloud_name):
+ if not os_client_config:
+ raise exceptions.SaltInvocationError(
+ "Cannot load os-client-config. Please check your environment "
+ "configuration.")
service_type = 'key-manager'
config = os_client_config.OpenStackConfig()
cloud = config.get_one_cloud(cloud_name)