222 lines
11 KiB
Python
222 lines
11 KiB
Python
# 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.
|
|
|
|
from oslo_utils import uuidutils
|
|
from unittest import mock
|
|
|
|
from neutron.agent.l3 import agent as l3_agent
|
|
from neutron.agent.l3 import l3_agent_extension_api as l3_ext_api
|
|
from neutron.agent.l3 import router_info as l3router
|
|
from neutron.api.rpc.callbacks.consumer import registry
|
|
from neutron.api.rpc.callbacks import events
|
|
from neutron.api.rpc.handlers import resources_rpc
|
|
from neutron.tests.unit.agent.l3 import test_agent
|
|
from neutron_lib import constants
|
|
from neutron_lib import context
|
|
|
|
from neutron_dynamic_routing.objects import bgp_associations as assoc_objects
|
|
from neutron_dynamic_routing.privileged import driver as priv_driver
|
|
from neutron_dynamic_routing.services.bgp.agent.l3 import bgp_extension
|
|
|
|
BGP_ROUTER_ASSOC = assoc_objects.BgpSpeakerRouterAssociation.obj_name()
|
|
BGP_PEER_ASSOC = assoc_objects.BgpSpeakerPeerAssociation.obj_name()
|
|
HOSTNAME = 'testhost'
|
|
FAKE_BGPSPEAKER_UUID = uuidutils.generate_uuid()
|
|
FAKE_ROUTER_ASSOC_UUID = uuidutils.generate_uuid()
|
|
FAKE_PEER_ASSOC_UUID = uuidutils.generate_uuid()
|
|
FAKE_ROUTER_UUID = uuidutils.generate_uuid()
|
|
FAKE_BGPPEER_UUID = uuidutils.generate_uuid()
|
|
FAKE_BGP_SPEAKER = {'id': FAKE_BGPSPEAKER_UUID,
|
|
'ip_version': 4,
|
|
'local_as': 12345,
|
|
'peers': [{'remote_as': '2345',
|
|
'peer_ip': '1.1.1.1',
|
|
'auth_type': 'none',
|
|
'password': ''}],
|
|
'router_associations': [{
|
|
'bgp_speaker_id': FAKE_BGPSPEAKER_UUID,
|
|
'router_id': FAKE_ROUTER_UUID,
|
|
'advertise_extra_routes': True}],
|
|
'peer_associations': [{
|
|
'bgp_speaker_id': FAKE_BGPSPEAKER_UUID,
|
|
'peer_id': FAKE_BGPPEER_UUID}],
|
|
'advertised_routes': []}
|
|
FAKE_BGP_PEER = {'id': FAKE_BGPPEER_UUID,
|
|
'remote_as': '2345',
|
|
'peer_ip': '1.1.1.1',
|
|
'auth_type': 'none',
|
|
'password': ''}
|
|
FAKE_ROUTES = ['192.168.145.0/24', '145.0.0.0/24']
|
|
FAKE_EXTRA_ROUTE = ['145.0.0.0/24']
|
|
|
|
|
|
class BgpL3AgentExtensionBaseTestCase(
|
|
test_agent.BasicRouterOperationsFramework):
|
|
|
|
def setUp(self):
|
|
super(BgpL3AgentExtensionBaseTestCase, self).setUp()
|
|
|
|
self.bgp_ext = bgp_extension.BgpAgentExtension()
|
|
self.agent_api = l3_ext_api.L3AgentExtensionAPI(None, None)
|
|
self.bgp_ext.consume_api(self.agent_api)
|
|
self.connection = mock.Mock()
|
|
self.context = context.get_admin_context()
|
|
self.agent = l3_agent.L3NATAgent(HOSTNAME, self.conf)
|
|
self.router_id = FAKE_ROUTER_UUID
|
|
self.router = {'id': self.router_id,
|
|
'ha': False,
|
|
'distributed': False}
|
|
self.router_info = l3router.RouterInfo(self.agent, self.router_id,
|
|
self.router, **self.ri_kwargs)
|
|
self.agent.router_info[self.router['id']] = self.router_info
|
|
|
|
self.get_router_info = mock.patch(
|
|
'neutron.agent.l3.l3_agent_extension_api.'
|
|
'L3AgentExtensionAPI.get_router_info').start()
|
|
self.get_router_info.return_value = self.router_info
|
|
|
|
|
|
class BgpL3AgentExtensionInitializeTestCase(BgpL3AgentExtensionBaseTestCase):
|
|
@mock.patch.object(registry, 'register')
|
|
@mock.patch.object(resources_rpc, 'ResourcesPushRpcCallback')
|
|
def test_initialize_subscribed_to_rpc(self, rpc_mock, subscribe_mock):
|
|
call_to_patch = 'neutron_lib.rpc.Connection'
|
|
with mock.patch(call_to_patch,
|
|
return_value=self.connection) as create_connection:
|
|
self.bgp_ext.initialize(
|
|
self.connection, constants.L3_AGENT_MODE)
|
|
create_connection.assert_has_calls([mock.call()])
|
|
self.connection.create_consumer.assert_has_calls(
|
|
[mock.call(
|
|
resources_rpc.resource_type_versioned_topic(
|
|
BGP_ROUTER_ASSOC),
|
|
[rpc_mock()],
|
|
fanout=True),
|
|
mock.call(
|
|
resources_rpc.resource_type_versioned_topic(
|
|
BGP_PEER_ASSOC),
|
|
[rpc_mock()],
|
|
fanout=True)]
|
|
)
|
|
subscribe_mock.assert_any_call(
|
|
mock.ANY, BGP_ROUTER_ASSOC)
|
|
subscribe_mock.assert_called_with(
|
|
mock.ANY, BGP_PEER_ASSOC)
|
|
|
|
|
|
class BGPL3AgentExtensionRouterAssocTestCase(BgpL3AgentExtensionBaseTestCase):
|
|
|
|
def setUp(self):
|
|
super(BGPL3AgentExtensionRouterAssocTestCase, self).setUp()
|
|
self.bgp_ext.initialize(
|
|
self.connection, constants.L3_AGENT_MODE)
|
|
router_assoc_info = {'id': FAKE_ROUTER_ASSOC_UUID,
|
|
'router_id': self.router_id,
|
|
'bgp_speaker_id': FAKE_BGPSPEAKER_UUID,
|
|
'advertise_extra_routes': True}
|
|
self.router_assoc = assoc_objects.BgpSpeakerRouterAssociation(
|
|
self.context, **router_assoc_info)
|
|
peer_assoc_info = {'id': FAKE_PEER_ASSOC_UUID,
|
|
'peer_id': FAKE_BGPPEER_UUID,
|
|
'bgp_speaker_id': FAKE_BGPSPEAKER_UUID}
|
|
self.peer_assoc = assoc_objects.BgpSpeakerPeerAssociation(
|
|
self.context, **peer_assoc_info)
|
|
self.bgp_ext.rpc_plugin = mock.Mock()
|
|
self.plugin_rpc = self.bgp_ext.rpc_plugin
|
|
self.plugin_rpc.get_bgp_speaker_info.return_value = FAKE_BGP_SPEAKER
|
|
self.plugin_rpc.get_bgp_peer_info.return_value = FAKE_BGP_PEER
|
|
self.plugin_rpc.get_routes_to_advertise.return_value = FAKE_ROUTES
|
|
self.plugin_rpc.get_routes_to_withdraw.return_value = FAKE_EXTRA_ROUTE
|
|
self.plugin_rpc.get_speaker_associated_to_router.return_value = [
|
|
router_assoc_info]
|
|
|
|
@mock.patch.object(priv_driver, 'add_bgp_speaker')
|
|
def test_handle_create_router_assoc(self, driver_mock):
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[self.router_assoc],
|
|
events.CREATED)
|
|
driver_mock.assert_called_once_with(FAKE_BGPSPEAKER_UUID,
|
|
FAKE_BGP_SPEAKER['local_as'],
|
|
mock.ANY,
|
|
mock.ANY, 4)
|
|
|
|
@mock.patch.object(priv_driver, 'advertise_routes')
|
|
def test_handle_update_router_assoc_with_advertise_extra_routes(
|
|
self, driver_mock):
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[self.router_assoc],
|
|
events.UPDATED)
|
|
driver_mock.assert_called_once_with(tuple(FAKE_ROUTES))
|
|
|
|
@mock.patch.object(priv_driver, 'withdraw_routes')
|
|
def test_handle_update_router_assoc_without_advertise_extra_routes(
|
|
self, driver_mock):
|
|
assoc_info = {'id': FAKE_ROUTER_ASSOC_UUID,
|
|
'router_id': self.router_id,
|
|
'bgp_speaker_id': FAKE_BGPSPEAKER_UUID,
|
|
'advertise_extra_routes': False}
|
|
router_assoc = assoc_objects.BgpSpeakerPeerAssociation(
|
|
self.context, **assoc_info)
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[router_assoc],
|
|
events.UPDATED)
|
|
driver_mock.assert_called_once_with(tuple(FAKE_EXTRA_ROUTE))
|
|
|
|
@mock.patch.object(priv_driver, 'del_bgp_speaker')
|
|
def test_handle_delete_router_assoc(self, driver_mock):
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[self.router_assoc],
|
|
events.DELETED)
|
|
driver_mock.assert_called_once_with(FAKE_BGPSPEAKER_UUID, mock.ANY)
|
|
|
|
@mock.patch.object(priv_driver, 'advertise_routes')
|
|
@mock.patch.object(priv_driver, 'add_bgp_neighbor')
|
|
def test_handle_create_peer_assoc(self, driver_mock, advertise_mock):
|
|
priv_driver.add_bgp_speaker = mock.Mock()
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[self.router_assoc],
|
|
events.CREATED)
|
|
self.bgp_ext._handle_peer_notification(self.context, mock.Mock(),
|
|
[self.peer_assoc],
|
|
events.CREATED)
|
|
driver_mock.assert_called_once_with(FAKE_BGPSPEAKER_UUID,
|
|
FAKE_BGP_PEER['peer_ip'],
|
|
FAKE_BGP_PEER['remote_as'],
|
|
mock.ANY)
|
|
advertise_mock.assert_called_once_with(tuple(FAKE_ROUTES))
|
|
|
|
@mock.patch.object(priv_driver, 'del_bgp_neighbor')
|
|
def test_handle_delete_peer_assoc(self, driver_mock):
|
|
self.bgp_ext._handle_peer_notification(self.context, mock.Mock(),
|
|
[self.peer_assoc],
|
|
events.DELETED)
|
|
driver_mock.assert_called_once_with(FAKE_BGPSPEAKER_UUID,
|
|
FAKE_BGP_PEER['peer_ip'],
|
|
mock.ANY)
|
|
|
|
@mock.patch.object(priv_driver, 'advertise_routes')
|
|
@mock.patch.object(priv_driver, 'withdraw_routes')
|
|
def test_update_router(self, mock_withdraw, mock_advertise):
|
|
priv_driver.add_bgp_speaker = mock.Mock()
|
|
priv_driver.add_bgp_neighbor = mock.Mock()
|
|
self.bgp_ext._handle_router_notification(self.context, mock.Mock(),
|
|
[self.router_assoc],
|
|
events.CREATED)
|
|
self.bgp_ext._handle_peer_notification(self.context, mock.Mock(),
|
|
[self.peer_assoc],
|
|
events.CREATED)
|
|
new_route = ['155.0.0.0/24']
|
|
self.plugin_rpc.get_routes_to_advertise.return_value = new_route
|
|
self.bgp_ext.update_router(self.context, self.router)
|
|
mock_advertise.assert_called_with(tuple(new_route))
|
|
mock_withdraw.assert_called_with(tuple(FAKE_ROUTES))
|