Adds unit tests for external DNS integration
Adds unit tests for external DNS integration Change-Id: I91a9f6b496a6a6c544e824508dfb349f68de9dbd
This commit is contained in:
parent
2dc6cb7b8d
commit
9df22afb38
@ -0,0 +1,463 @@
|
|||||||
|
# Copyright (c) 2016 IBM
|
||||||
|
# All Rights Reserved.
|
||||||
|
#
|
||||||
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
||||||
|
# not use this file except in compliance with the License. You may obtain
|
||||||
|
# a copy of the License at
|
||||||
|
#
|
||||||
|
# http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
#
|
||||||
|
# Unless required by applicable law or agreed to in writing, software
|
||||||
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
||||||
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||||
|
# License for the specific language governing permissions and limitations
|
||||||
|
# under the License.
|
||||||
|
|
||||||
|
import mock
|
||||||
|
import netaddr
|
||||||
|
|
||||||
|
from neutron import context
|
||||||
|
from neutron.db import dns_db
|
||||||
|
from neutron.extensions import dns
|
||||||
|
from neutron.extensions import providernet as pnet
|
||||||
|
from neutron.plugins.ml2 import config
|
||||||
|
from neutron.plugins.ml2.extensions import dns_integration
|
||||||
|
from neutron.tests.unit.plugins.ml2 import test_plugin
|
||||||
|
|
||||||
|
|
||||||
|
mock_client = mock.Mock()
|
||||||
|
mock_admin_client = mock.Mock()
|
||||||
|
mock_config = {'return_value': (mock_client, mock_admin_client)}
|
||||||
|
DNSDOMAIN = 'domain.com.'
|
||||||
|
DNSNAME = 'port-dns-name'
|
||||||
|
NEWDNSNAME = 'new-port-dns-name'
|
||||||
|
V4UUID = 'v4_uuid'
|
||||||
|
V6UUID = 'v6_uuid'
|
||||||
|
|
||||||
|
|
||||||
|
@mock.patch(
|
||||||
|
'neutron.services.externaldns.drivers.designate.driver.get_clients',
|
||||||
|
**mock_config)
|
||||||
|
class DNSIntegrationTestCase(test_plugin.Ml2PluginV2TestCase):
|
||||||
|
_extension_drivers = ['dns']
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
config.cfg.CONF.set_override('extension_drivers',
|
||||||
|
self._extension_drivers,
|
||||||
|
group='ml2')
|
||||||
|
config.cfg.CONF.set_override('external_dns_driver', 'designate')
|
||||||
|
mock_client.reset_mock()
|
||||||
|
mock_admin_client.reset_mock()
|
||||||
|
super(DNSIntegrationTestCase, self).setUp()
|
||||||
|
dns_integration.DNS_DRIVER = None
|
||||||
|
dns_integration.subscribe()
|
||||||
|
|
||||||
|
def _create_port_for_test(self, provider_net=True, dns_domain=True,
|
||||||
|
dns_name=True, ipv4=True, ipv6=True):
|
||||||
|
net_kwargs = {}
|
||||||
|
if provider_net:
|
||||||
|
net_kwargs = {
|
||||||
|
'arg_list': (pnet.NETWORK_TYPE, pnet.SEGMENTATION_ID,),
|
||||||
|
pnet.NETWORK_TYPE: 'vxlan',
|
||||||
|
pnet.SEGMENTATION_ID: '2016',
|
||||||
|
}
|
||||||
|
if dns_domain:
|
||||||
|
net_kwargs[dns.DNSDOMAIN] = DNSDOMAIN
|
||||||
|
net_kwargs['arg_list'] = \
|
||||||
|
net_kwargs.get('arg_list', ()) + (dns.DNSDOMAIN,)
|
||||||
|
res = self._create_network(self.fmt, 'test_network', True,
|
||||||
|
**net_kwargs)
|
||||||
|
network = self.deserialize(self.fmt, res)
|
||||||
|
if ipv4:
|
||||||
|
self._create_subnet(self.fmt, network['network']['id'],
|
||||||
|
'10.0.0.0/24', ip_version=4)
|
||||||
|
if ipv6:
|
||||||
|
self._create_subnet(self.fmt, network['network']['id'],
|
||||||
|
'fd3d:bdd4:da60::/64', ip_version=6)
|
||||||
|
port_kwargs = {}
|
||||||
|
if dns_name:
|
||||||
|
port_kwargs = {
|
||||||
|
'arg_list': (dns.DNSNAME,),
|
||||||
|
dns.DNSNAME: DNSNAME
|
||||||
|
}
|
||||||
|
res = self._create_port('json', network['network']['id'],
|
||||||
|
**port_kwargs)
|
||||||
|
self.assertEqual(201, res.status_int)
|
||||||
|
port = self.deserialize(self.fmt, res)['port']
|
||||||
|
ctx = context.get_admin_context()
|
||||||
|
dns_data_db = ctx.session.query(
|
||||||
|
dns_db.PortDNS).filter_by(
|
||||||
|
port_id=port['id']).one_or_none()
|
||||||
|
return network['network'], port, dns_data_db
|
||||||
|
|
||||||
|
def _update_port_for_test(self, port, new_dns_name=NEWDNSNAME,
|
||||||
|
**kwargs):
|
||||||
|
mock_client.reset_mock()
|
||||||
|
records_v4 = [ip['ip_address'] for ip in port['fixed_ips']
|
||||||
|
if netaddr.IPAddress(ip['ip_address']).version == 4]
|
||||||
|
records_v6 = [ip['ip_address'] for ip in port['fixed_ips']
|
||||||
|
if netaddr.IPAddress(ip['ip_address']).version == 6]
|
||||||
|
recordsets = []
|
||||||
|
if records_v4:
|
||||||
|
recordsets.append({'id': V4UUID, 'records': records_v4})
|
||||||
|
if records_v4:
|
||||||
|
recordsets.append({'id': V6UUID, 'records': records_v6})
|
||||||
|
mock_client.recordsets.list.return_value = recordsets
|
||||||
|
mock_admin_client.reset_mock()
|
||||||
|
body = {}
|
||||||
|
if new_dns_name is not None:
|
||||||
|
body['dns_name'] = new_dns_name
|
||||||
|
body.update(kwargs)
|
||||||
|
data = {'port': body}
|
||||||
|
req = self.new_update_request('ports', data, port['id'])
|
||||||
|
res = req.get_response(self.api)
|
||||||
|
self.assertEqual(200, res.status_int)
|
||||||
|
port = self.deserialize(self.fmt, res)['port']
|
||||||
|
ctx = context.get_admin_context()
|
||||||
|
dns_data_db = ctx.session.query(
|
||||||
|
dns_db.PortDNS).filter_by(
|
||||||
|
port_id=port['id']).one_or_none()
|
||||||
|
return port, dns_data_db
|
||||||
|
|
||||||
|
def _verify_port_dns(self, net, port, dns_data_db, dns_name=True,
|
||||||
|
dns_domain=True, ptr_zones=True, delete_records=False,
|
||||||
|
provider_net=True, dns_driver=True, original_ips=None,
|
||||||
|
current_dns_name=DNSNAME, previous_dns_name=''):
|
||||||
|
if dns_name:
|
||||||
|
self.assertEqual(current_dns_name, port[dns.DNSNAME])
|
||||||
|
if dns_name and dns_domain and provider_net and dns_driver:
|
||||||
|
self.assertEqual(current_dns_name, dns_data_db['current_dns_name'])
|
||||||
|
self.assertEqual(previous_dns_name,
|
||||||
|
dns_data_db['previous_dns_name'])
|
||||||
|
if current_dns_name:
|
||||||
|
self.assertEqual(net[dns.DNSDOMAIN],
|
||||||
|
dns_data_db['current_dns_domain'])
|
||||||
|
else:
|
||||||
|
self.assertFalse(dns_data_db['current_dns_domain'])
|
||||||
|
records_v4 = [ip['ip_address'] for ip in port['fixed_ips']
|
||||||
|
if netaddr.IPAddress(ip['ip_address']).version
|
||||||
|
== 4]
|
||||||
|
records_v6 = [ip['ip_address'] for ip in port['fixed_ips']
|
||||||
|
if netaddr.IPAddress(ip['ip_address']).version
|
||||||
|
== 6]
|
||||||
|
expected = []
|
||||||
|
expected_delete = []
|
||||||
|
if records_v4:
|
||||||
|
if current_dns_name:
|
||||||
|
expected.append(
|
||||||
|
mock.call(net[dns.DNSDOMAIN], current_dns_name, 'A',
|
||||||
|
records_v4))
|
||||||
|
if delete_records:
|
||||||
|
expected_delete.append(mock.call(net[dns.DNSDOMAIN],
|
||||||
|
V4UUID))
|
||||||
|
if records_v6:
|
||||||
|
if current_dns_name:
|
||||||
|
expected.append(
|
||||||
|
mock.call(net[dns.DNSDOMAIN], current_dns_name, 'AAAA',
|
||||||
|
records_v6))
|
||||||
|
if delete_records:
|
||||||
|
expected_delete.append(mock.call(net[dns.DNSDOMAIN],
|
||||||
|
V6UUID))
|
||||||
|
mock_client.recordsets.create.assert_has_calls(expected,
|
||||||
|
any_order=True)
|
||||||
|
self.assertTrue(
|
||||||
|
len(mock_client.recordsets.create.call_args_list) ==
|
||||||
|
len(expected))
|
||||||
|
mock_client.recordsets.delete.assert_has_calls(expected_delete,
|
||||||
|
any_order=True)
|
||||||
|
self.assertTrue(
|
||||||
|
len(mock_client.recordsets.delete.call_args_list) ==
|
||||||
|
len(expected_delete))
|
||||||
|
expected = []
|
||||||
|
expected_delete = []
|
||||||
|
if ptr_zones:
|
||||||
|
records = records_v4 + records_v6
|
||||||
|
recordset_name = '%s.%s' % (current_dns_name,
|
||||||
|
net[dns.DNSDOMAIN])
|
||||||
|
for record in records:
|
||||||
|
in_addr_name = netaddr.IPAddress(record).reverse_dns
|
||||||
|
in_addr_zone_name = self._get_in_addr_zone_name(
|
||||||
|
in_addr_name)
|
||||||
|
if current_dns_name:
|
||||||
|
expected.append(mock.call(in_addr_zone_name,
|
||||||
|
in_addr_name, 'PTR',
|
||||||
|
[recordset_name]))
|
||||||
|
if delete_records and not original_ips:
|
||||||
|
expected_delete.append(mock.call(in_addr_zone_name,
|
||||||
|
in_addr_name))
|
||||||
|
if delete_records and original_ips:
|
||||||
|
for record in original_ips:
|
||||||
|
in_addr_name = netaddr.IPAddress(record).reverse_dns
|
||||||
|
in_addr_zone_name = self._get_in_addr_zone_name(
|
||||||
|
in_addr_name)
|
||||||
|
expected_delete.append(mock.call(in_addr_zone_name,
|
||||||
|
in_addr_name))
|
||||||
|
mock_admin_client.recordsets.create.assert_has_calls(
|
||||||
|
expected, any_order=True)
|
||||||
|
self.assertTrue(
|
||||||
|
len(mock_admin_client.recordsets.create.call_args_list) ==
|
||||||
|
len(expected))
|
||||||
|
mock_admin_client.recordsets.delete.assert_has_calls(
|
||||||
|
expected_delete, any_order=True)
|
||||||
|
self.assertTrue(
|
||||||
|
len(mock_admin_client.recordsets.delete.call_args_list) ==
|
||||||
|
len(expected_delete))
|
||||||
|
else:
|
||||||
|
self.assertTrue(dns_data_db is None)
|
||||||
|
self.assertFalse(mock_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(mock_client.recordsets.delete.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.delete.call_args_list)
|
||||||
|
|
||||||
|
def _get_in_addr_zone_name(self, in_addr_name):
|
||||||
|
units = self._get_bytes_or_nybles_to_skip(in_addr_name)
|
||||||
|
return '.'.join(in_addr_name.split('.')[int(units):])
|
||||||
|
|
||||||
|
def _get_bytes_or_nybles_to_skip(self, in_addr_name):
|
||||||
|
if 'in-addr.arpa' in in_addr_name:
|
||||||
|
return ((
|
||||||
|
32 - config.cfg.CONF.designate.ipv4_ptr_zone_prefix_size) / 8)
|
||||||
|
return (128 - config.cfg.CONF.designate.ipv6_ptr_zone_prefix_size) / 4
|
||||||
|
|
||||||
|
def test_create_port(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
self._verify_port_dns(net, port, dns_data_db)
|
||||||
|
|
||||||
|
def test_create_port_tenant_network(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(provider_net=False)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, provider_net=False)
|
||||||
|
|
||||||
|
def test_create_port_no_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(dns_name=False)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, dns_name=False)
|
||||||
|
|
||||||
|
def test_create_port_no_dns_domain(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(dns_domain=False)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, dns_domain=False)
|
||||||
|
|
||||||
|
def test_create_port_no_dns_driver(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('external_dns_driver', '')
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, dns_driver=False)
|
||||||
|
|
||||||
|
def test_create_port_no_ipv6(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(ipv6=False)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db)
|
||||||
|
|
||||||
|
def test_create_port_no_ipv4(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(ipv4=False)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db)
|
||||||
|
|
||||||
|
def test_create_port_no_ptr_zones(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
config.cfg.CONF.set_override('allow_reverse_dns_lookup', False,
|
||||||
|
group='designate')
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, ptr_zones=False)
|
||||||
|
config.cfg.CONF.set_override('allow_reverse_dns_lookup', True,
|
||||||
|
group='designate')
|
||||||
|
|
||||||
|
def test_update_port(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
port, dns_data_db = self._update_port_for_test(port)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=NEWDNSNAME,
|
||||||
|
previous_dns_name=DNSNAME)
|
||||||
|
|
||||||
|
def test_update_port_with_current_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=DNSNAME)
|
||||||
|
self.assertEqual(DNSNAME, dns_data_db['current_dns_name'])
|
||||||
|
self.assertEqual(DNSDOMAIN, dns_data_db['current_dns_domain'])
|
||||||
|
self.assertEqual('', dns_data_db['previous_dns_name'])
|
||||||
|
self.assertEqual('', dns_data_db['previous_dns_domain'])
|
||||||
|
self.assertFalse(mock_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(mock_client.recordsets.delete.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.delete.call_args_list)
|
||||||
|
|
||||||
|
def test_update_port_tenant_network(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(provider_net=False)
|
||||||
|
port, dns_data_db = self._update_port_for_test(port)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=NEWDNSNAME,
|
||||||
|
previous_dns_name=DNSNAME, provider_net=False)
|
||||||
|
|
||||||
|
def test_update_port_no_dns_domain(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(dns_domain=False)
|
||||||
|
port, dns_data_db = self._update_port_for_test(port)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=NEWDNSNAME,
|
||||||
|
previous_dns_name=DNSNAME, dns_domain=False)
|
||||||
|
|
||||||
|
def test_update_port_add_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test(dns_name=False)
|
||||||
|
port, dns_data_db = self._update_port_for_test(port)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=False,
|
||||||
|
current_dns_name=NEWDNSNAME,
|
||||||
|
previous_dns_name='')
|
||||||
|
|
||||||
|
def test_update_port_clear_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
port, dns_data_db = self._update_port_for_test(port, new_dns_name='')
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name='', previous_dns_name=DNSNAME)
|
||||||
|
|
||||||
|
def test_update_port_non_dns_name_attribute(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
port_name = 'port_name'
|
||||||
|
kwargs = {'name': port_name}
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=None,
|
||||||
|
**kwargs)
|
||||||
|
self.assertEqual(DNSNAME, dns_data_db['current_dns_name'])
|
||||||
|
self.assertEqual(DNSDOMAIN, dns_data_db['current_dns_domain'])
|
||||||
|
self.assertEqual('', dns_data_db['previous_dns_name'])
|
||||||
|
self.assertEqual('', dns_data_db['previous_dns_domain'])
|
||||||
|
self.assertFalse(mock_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(mock_client.recordsets.delete.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.delete.call_args_list)
|
||||||
|
self.assertEqual(port_name, port['name'])
|
||||||
|
|
||||||
|
def test_update_port_fixed_ips(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
original_ips = [ip['ip_address'] for ip in port['fixed_ips']]
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id'],
|
||||||
|
'ip_address': str(netaddr.IPAddress(ip['ip_address']) + 1)})
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=None,
|
||||||
|
**kwargs)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=DNSNAME,
|
||||||
|
previous_dns_name=DNSNAME,
|
||||||
|
original_ips=original_ips)
|
||||||
|
|
||||||
|
def test_update_port_fixed_ips_with_subnet_ids(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
original_ips = [ip['ip_address'] for ip in port['fixed_ips']]
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id']})
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=None,
|
||||||
|
**kwargs)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=DNSNAME,
|
||||||
|
previous_dns_name=DNSNAME,
|
||||||
|
original_ips=original_ips)
|
||||||
|
|
||||||
|
def test_update_port_fixed_ips_with_new_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
original_ips = [ip['ip_address'] for ip in port['fixed_ips']]
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id'],
|
||||||
|
'ip_address': str(netaddr.IPAddress(ip['ip_address']) + 1)})
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=NEWDNSNAME,
|
||||||
|
**kwargs)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=NEWDNSNAME,
|
||||||
|
previous_dns_name=DNSNAME,
|
||||||
|
original_ips=original_ips)
|
||||||
|
|
||||||
|
def test_update_port_fixed_ips_with_current_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
original_ips = [ip['ip_address'] for ip in port['fixed_ips']]
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id'],
|
||||||
|
'ip_address': str(netaddr.IPAddress(ip['ip_address']) + 1)})
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name=DNSNAME,
|
||||||
|
**kwargs)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name=DNSNAME,
|
||||||
|
previous_dns_name=DNSNAME,
|
||||||
|
original_ips=original_ips)
|
||||||
|
|
||||||
|
def test_update_port_fixed_ips_clearing_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
original_ips = [ip['ip_address'] for ip in port['fixed_ips']]
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id'],
|
||||||
|
'ip_address': str(netaddr.IPAddress(ip['ip_address']) + 1)})
|
||||||
|
port, dns_data_db = self._update_port_for_test(port,
|
||||||
|
new_dns_name='',
|
||||||
|
**kwargs)
|
||||||
|
self._verify_port_dns(net, port, dns_data_db, delete_records=True,
|
||||||
|
current_dns_name='', previous_dns_name=DNSNAME,
|
||||||
|
original_ips=original_ips)
|
||||||
|
|
||||||
|
def test_update_fixed_ips_no_effect_after_clearing_dns_name(self, *mocks):
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
port, dns_data_db_1 = self._update_port_for_test(port,
|
||||||
|
new_dns_name='')
|
||||||
|
kwargs = {'fixed_ips': []}
|
||||||
|
for ip in port['fixed_ips']:
|
||||||
|
kwargs['fixed_ips'].append(
|
||||||
|
{'subnet_id': ip['subnet_id'],
|
||||||
|
'ip_address': str(netaddr.IPAddress(ip['ip_address']) + 1)})
|
||||||
|
mock_client.reset_mock()
|
||||||
|
mock_admin_client.reset_mock()
|
||||||
|
port, dns_data_db_2 = self._update_port_for_test(port,
|
||||||
|
new_dns_name='',
|
||||||
|
**kwargs)
|
||||||
|
self.assertEqual('', dns_data_db_2['current_dns_name'])
|
||||||
|
self.assertEqual('', dns_data_db_2['current_dns_domain'])
|
||||||
|
self.assertEqual(dns_data_db_1['current_dns_name'],
|
||||||
|
dns_data_db_2['current_dns_name'])
|
||||||
|
self.assertEqual(dns_data_db_1['current_dns_domain'],
|
||||||
|
dns_data_db_2['current_dns_domain'])
|
||||||
|
self.assertEqual(dns_data_db_1['previous_dns_name'],
|
||||||
|
dns_data_db_2['previous_dns_name'])
|
||||||
|
self.assertEqual(dns_data_db_1['previous_dns_domain'],
|
||||||
|
dns_data_db_2['previous_dns_domain'])
|
||||||
|
self.assertFalse(mock_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.create.call_args_list)
|
||||||
|
self.assertFalse(mock_client.recordsets.delete.call_args_list)
|
||||||
|
self.assertFalse(
|
||||||
|
mock_admin_client.recordsets.delete.call_args_list)
|
||||||
|
|
||||||
|
def test_dns_driver_loaded_after_server_restart(self, *mocks):
|
||||||
|
dns_integration.DNS_DRIVER = None
|
||||||
|
config.cfg.CONF.set_override('dns_domain', DNSDOMAIN)
|
||||||
|
net, port, dns_data_db = self._create_port_for_test()
|
||||||
|
self._verify_port_dns(net, port, dns_data_db)
|
Loading…
Reference in New Issue
Block a user