diff options
| author | Rafael Weingärtner <rafael@apache.org> | 2022-09-07 14:20:50 -0300 |
|---|---|---|
| committer | Edward Hope-Morley <edward.hope-morley@canonical.com> | 2023-03-21 16:22:49 +0000 |
| commit | dcae2040026da432721687a36716b6b9c3e01078 (patch) | |
| tree | 9d2173b53de62a801596ea18b11ddbff8cc56694 | |
| parent | 9f6481163c3d86c6b38578a2fef8c6bdd29ae463 (diff) | |
| download | ceilometer-dcae2040026da432721687a36716b6b9c3e01078.tar.gz | |
Fix barbican credentials retrieval
Change-Id: Iec0d885e1fdd42edb2da37d4a71a80168c690c98
(cherry picked from commit 264229ea44ad4ddcb347bda153d36255e07322ee)
(cherry picked from commit 17648c7195d48c35c97e3015f7f85ccccfc837eb)
| -rw-r--r-- | ceilometer/polling/discovery/non_openstack_credentials_discovery.py | 2 | ||||
| -rw-r--r-- | ceilometer/tests/unit/polling/test_non_openstack_credentials_discovery.py | 10 |
2 files changed, 7 insertions, 5 deletions
diff --git a/ceilometer/polling/discovery/non_openstack_credentials_discovery.py b/ceilometer/polling/discovery/non_openstack_credentials_discovery.py index 61459452..0b3ccec8 100644 --- a/ceilometer/polling/discovery/non_openstack_credentials_discovery.py +++ b/ceilometer/polling/discovery/non_openstack_credentials_discovery.py @@ -38,7 +38,7 @@ class NonOpenStackCredentialsDiscovery(EndpointDiscovery): if not param: return [barbican_secret] barbican_endpoints = super(NonOpenStackCredentialsDiscovery, - self).discover("key-manager") + self).discover(manager, "key-manager") if not barbican_endpoints: LOG.warning("No Barbican endpoints found to execute the" " credentials discovery process to [%s].", diff --git a/ceilometer/tests/unit/polling/test_non_openstack_credentials_discovery.py b/ceilometer/tests/unit/polling/test_non_openstack_credentials_discovery.py index c1fffd87..4e257415 100644 --- a/ceilometer/tests/unit/polling/test_non_openstack_credentials_discovery.py +++ b/ceilometer/tests/unit/polling/test_non_openstack_credentials_discovery.py @@ -95,8 +95,8 @@ class TestNonOpenStackCredentialsDiscovery(base.BaseTestCase): @mock.patch('keystoneclient.v2_0.client.Client') def test_discover_response_ok(self, client_mock): - def discover_mock(self, manager, param=None): - return ["barbican_url"] + discover_mock = mock.MagicMock() + discover_mock.return_value = ["barbican_url"] original_discover_method = EndpointDiscovery.discover EndpointDiscovery.discover = discover_mock @@ -108,9 +108,11 @@ class TestNonOpenStackCredentialsDiscovery(base.BaseTestCase): client_mock.session.get.return_value = return_value - response = self.discovery.discover( - manager=self.FakeManager(client_mock), param="param") + fake_manager = self.FakeManager(client_mock) + response = self.discovery.discover(manager=fake_manager, param="param") self.assertEqual(["content"], response) + discover_mock.assert_has_calls([ + mock.call(fake_manager, "key-manager")]) EndpointDiscovery.discover = original_discover_method |