From baa7374a075e1a978afe02f278277f8883e58fcb Mon Sep 17 00:00:00 2001 From: Moritz Kiemer Date: Wed, 11 Dec 2024 15:45:57 +0100 Subject: [PATCH] Revert "Reland "Ensure that sites can still communicate after me..."" This reverts commit 99bc03183588aa8564c3d2521c0daa89f18f3122. Change-Id: I75be4965d1256ad17ea05f130f560b6319ba64a9 --- .../message_broker/test_message_broker.py | 21 ------------------- 1 file changed, 21 deletions(-) diff --git a/tests/composition/message_broker/test_message_broker.py b/tests/composition/message_broker/test_message_broker.py index 38d0bc52b36..7cf63d2e693 100644 --- a/tests/composition/message_broker/test_message_broker.py +++ b/tests/composition/message_broker/test_message_broker.py @@ -12,7 +12,6 @@ from tests.testlib.site import Site from tests.composition.message_broker.utils import ( - assert_message_exchange_not_working, assert_message_exchange_working, broker_pong, broker_stopped, @@ -102,23 +101,3 @@ def test_message_broker_remote_remote_p2p( broker_stopped(central_site), ): check_broker_ping(remote_site_2, remote_site.id) - - def test_rabbitmq_port_change(self, central_site: Site, remote_site: Site) -> None: - """Ensure that sites can still communicate after the message broker port is changed""" - site_connection = central_site.openapi.show_site(remote_site.id)["extensions"] - site_connection_port = int( - site_connection["configuration_connection"]["message_broker_port"] - ) - assert site_connection_port == remote_site.message_broker_port - next_port = _next_free_port(remote_site, "RABBITMQ_PORT", str(site_connection_port + 1)) - - remote_site.set_config("RABBITMQ_PORT", str(next_port), with_restart=True) - - site_connection["configuration_connection"]["message_broker_port"] = str(next_port) - central_site.openapi.update_site(remote_site.id, site_connection) - - # ensure changes are not in effect before activated - assert_message_exchange_not_working(central_site, remote_site) - central_site.openapi.activate_changes_and_wait_for_completion() - - assert_message_exchange_working(central_site, remote_site)