diff options
author | Kaifeng Wang <kaifeng.w@gmail.com> | 2020-05-01 14:10:34 +0800 |
---|---|---|
committer | Kaifeng Wang <kaifeng.w@gmail.com> | 2020-05-01 07:39:21 +0000 |
commit | 3e6dfdb3b9305d62211aa460f045450c26720555 (patch) | |
tree | c1ce34a968d87604ece0830fcb47dbb7f50880e1 /ironic/tests/unit | |
parent | 1bb5e80ae4be54837cdca84f6bfbe180a79f37b2 (diff) | |
download | ironic-3e6dfdb3b9305d62211aa460f045450c26720555.tar.gz |
Remove [conductor]api_url
It was deprecated long before.
Change-Id: I05d8a90dbf6e92ef230b1a9624c6816fc96c6a7f
Diffstat (limited to 'ironic/tests/unit')
-rw-r--r-- | ironic/tests/unit/common/test_pxe_utils.py | 4 | ||||
-rw-r--r-- | ironic/tests/unit/drivers/modules/test_deploy_utils.py | 17 | ||||
-rw-r--r-- | ironic/tests/unit/drivers/modules/test_ipxe.py | 1 | ||||
-rw-r--r-- | ironic/tests/unit/drivers/modules/test_pxe.py | 1 |
4 files changed, 2 insertions, 21 deletions
diff --git a/ironic/tests/unit/common/test_pxe_utils.py b/ironic/tests/unit/common/test_pxe_utils.py index 6950bfe8a..5dd660b22 100644 --- a/ironic/tests/unit/common/test_pxe_utils.py +++ b/ironic/tests/unit/common/test_pxe_utils.py @@ -1155,8 +1155,6 @@ class PXEBuildConfigOptionsTestCase(db_base.DbTestCase): ramdisk_params=None): self.config(debug=debug) self.config(pxe_append_params='test_param', group='pxe') - # NOTE: right '/' should be removed from url string - self.config(api_url='http://192.168.122.184:6385', group='conductor') driver_internal_info = self.node.driver_internal_info driver_internal_info['is_whole_disk_image'] = whle_dsk_img @@ -1415,8 +1413,6 @@ class iPXEBuildConfigOptionsTestCase(db_base.DbTestCase): mode='deploy'): self.config(debug=debug) self.config(pxe_append_params='test_param', group='pxe') - # NOTE: right '/' should be removed from url string - self.config(api_url='http://192.168.122.184:6385', group='conductor') self.config(ipxe_timeout=ipxe_timeout, group='pxe') root_dir = CONF.deploy.http_root diff --git a/ironic/tests/unit/drivers/modules/test_deploy_utils.py b/ironic/tests/unit/drivers/modules/test_deploy_utils.py index a39b33a35..9594b8d87 100644 --- a/ironic/tests/unit/drivers/modules/test_deploy_utils.py +++ b/ironic/tests/unit/drivers/modules/test_deploy_utils.py @@ -773,16 +773,6 @@ class OtherFunctionTestCase(db_base.DbTestCase): @mock.patch('ironic.common.keystone.get_auth') @mock.patch.object(utils, '_get_ironic_session') - def test_get_ironic_api_url_from_config(self, mock_ks, mock_auth): - mock_sess = mock.Mock() - mock_ks.return_value = mock_sess - fake_api_url = 'http://foo/' - self.config(api_url=fake_api_url, group='conductor') - # also checking for stripped trailing slash - self.assertEqual(fake_api_url[:-1], utils.get_ironic_api_url()) - - @mock.patch('ironic.common.keystone.get_auth') - @mock.patch.object(utils, '_get_ironic_session') @mock.patch('ironic.common.keystone.get_adapter') def test_get_ironic_api_url_from_keystone(self, mock_ka, mock_ks, mock_auth): @@ -791,7 +781,6 @@ class OtherFunctionTestCase(db_base.DbTestCase): fake_api_url = 'http://foo/' mock_ka.return_value.get_endpoint.return_value = fake_api_url # NOTE(pas-ha) endpoint_override is None by default - self.config(api_url=None, group='conductor') url = utils.get_ironic_api_url() # also checking for stripped trailing slash self.assertEqual(fake_api_url[:-1], url) @@ -807,7 +796,6 @@ class OtherFunctionTestCase(db_base.DbTestCase): mock_ks.return_value = mock_sess mock_ka.return_value.get_endpoint.side_effect = ( exception.KeystoneFailure()) - self.config(api_url=None, group='conductor') self.assertRaises(exception.InvalidParameterValue, utils.get_ironic_api_url) @@ -818,7 +806,6 @@ class OtherFunctionTestCase(db_base.DbTestCase): mock_sess = mock.Mock() mock_ks.return_value = mock_sess mock_ka.return_value.get_endpoint.return_value = None - self.config(api_url=None, group='conductor') self.assertRaises(exception.InvalidParameterValue, utils.get_ironic_api_url) @@ -1192,13 +1179,13 @@ class AgentMethodsTestCase(db_base.DbTestCase): self._test_tear_down_inband_cleaning(cleaning_error=True) def test_build_agent_options_conf(self): - self.config(api_url='https://api-url', group='conductor') + self.config(endpoint_override='https://api-url', + group='service_catalog') options = utils.build_agent_options(self.node) self.assertEqual('https://api-url', options['ipa-api-url']) @mock.patch.object(utils, '_get_ironic_session') def test_build_agent_options_keystone(self, session_mock): - self.config(api_url=None, group='conductor') sess = mock.Mock() sess.get_endpoint.return_value = 'https://api-url' session_mock.return_value = sess diff --git a/ironic/tests/unit/drivers/modules/test_ipxe.py b/ironic/tests/unit/drivers/modules/test_ipxe.py index cbec1bb0d..2e07c7804 100644 --- a/ironic/tests/unit/drivers/modules/test_ipxe.py +++ b/ironic/tests/unit/drivers/modules/test_ipxe.py @@ -83,7 +83,6 @@ class iPXEBootTestCase(db_base.DbTestCase): driver_internal_info=self.driver_internal_info) self.port = obj_utils.create_test_port(self.context, node_id=self.node.id) - self.config(group='conductor', api_url='http://127.0.0.1:1234/') def test_get_properties(self): expected = pxe_base.COMMON_PROPERTIES diff --git a/ironic/tests/unit/drivers/modules/test_pxe.py b/ironic/tests/unit/drivers/modules/test_pxe.py index ed4fa2b63..5a5542be8 100644 --- a/ironic/tests/unit/drivers/modules/test_pxe.py +++ b/ironic/tests/unit/drivers/modules/test_pxe.py @@ -85,7 +85,6 @@ class PXEBootTestCase(db_base.DbTestCase): driver_internal_info=self.driver_internal_info) self.port = obj_utils.create_test_port(self.context, node_id=self.node.id) - self.config(group='conductor', api_url='http://127.0.0.1:1234/') self.config(my_ipv6='2001:db8::1') def test_get_properties(self): |