Merge "Decouple ceilometerclient without aodh services running" into stable/mitaka
This commit is contained in:
@@ -17,6 +17,7 @@ from keystoneclient.auth.identity import v3 as v3_auth
|
|||||||
from keystoneclient import exceptions as ks_exc
|
from keystoneclient import exceptions as ks_exc
|
||||||
from keystoneclient import session as ks_session
|
from keystoneclient import session as ks_session
|
||||||
import mock
|
import mock
|
||||||
|
import requests
|
||||||
|
|
||||||
from ceilometerclient import client
|
from ceilometerclient import client
|
||||||
from ceilometerclient import exc
|
from ceilometerclient import exc
|
||||||
@@ -218,6 +219,15 @@ class ClientTestWithAodh(ClientTest):
|
|||||||
self.assertEqual('True', (client.alarm_client.http_client.
|
self.assertEqual('True', (client.alarm_client.http_client.
|
||||||
auth_plugin.opts['insecure']))
|
auth_plugin.opts['insecure']))
|
||||||
|
|
||||||
|
def test_ceilometerclient_available_without_aodh_services_running(self):
|
||||||
|
env = FAKE_ENV.copy()
|
||||||
|
env.pop('auth_plugin', None)
|
||||||
|
with mock.patch('ceilometerclient.openstack.common.apiclient.client.'
|
||||||
|
'HTTPClient.client_request') as mocked_request:
|
||||||
|
mocked_request.side_effect = requests.exceptions.ConnectionError
|
||||||
|
ceiloclient = client.get_client(2, **env)
|
||||||
|
self.assertIsInstance(ceiloclient, v2client.Client)
|
||||||
|
|
||||||
|
|
||||||
class ClientAuthTest(utils.BaseTestCase):
|
class ClientAuthTest(utils.BaseTestCase):
|
||||||
|
|
||||||
|
|||||||
@@ -16,6 +16,8 @@
|
|||||||
# under the License.
|
# under the License.
|
||||||
import copy
|
import copy
|
||||||
|
|
||||||
|
import requests
|
||||||
|
|
||||||
from ceilometerclient import client as ceiloclient
|
from ceilometerclient import client as ceiloclient
|
||||||
from ceilometerclient.v2 import alarms
|
from ceilometerclient.v2 import alarms
|
||||||
from ceilometerclient.v2 import capabilities
|
from ceilometerclient.v2 import capabilities
|
||||||
@@ -125,3 +127,5 @@ class Client(object):
|
|||||||
return None
|
return None
|
||||||
except kc_exc.EndpointNotFound:
|
except kc_exc.EndpointNotFound:
|
||||||
return None
|
return None
|
||||||
|
except requests.exceptions.ConnectionError:
|
||||||
|
return None
|
||||||
|
|||||||
Reference in New Issue
Block a user