From 6b268f7bda4cd250f4fdaf0d2800c035e36149af Mon Sep 17 00:00:00 2001 From: Benjamin Ludwig Date: Fri, 21 Jun 2024 12:17:44 +0200 Subject: [PATCH] Fix printf debugging of SelfIPs: Print their IDs --- octavia_f5/controller/worker/controller_worker.py | 2 +- octavia_f5/controller/worker/l2_sync_manager.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/octavia_f5/controller/worker/controller_worker.py b/octavia_f5/controller/worker/controller_worker.py index f511cd2..7c03ff8 100644 --- a/octavia_f5/controller/worker/controller_worker.py +++ b/octavia_f5/controller/worker/controller_worker.py @@ -117,7 +117,7 @@ def f5sync(network_id, device, *args): self.queue.qsize(), network_id, [lb.id for lb in loadbalancers]) selfips = list(chain.from_iterable( self.network_driver.ensure_selfips(loadbalancers, CONF.host, cleanup_orphans=True))) - LOG.warning(f"FOOBARBAZ: In controller_worker.as3worker.f5_sync: Got SelfIPs: {selfips}") + LOG.warning(f"FOOBARBAZ: In controller_worker.as3worker.f5_sync: Got SelfIPs: {[sip.id for sip in selfips]}") if all(lb.provisioning_status == lib_consts.PENDING_DELETE for lb in loadbalancers): LOG.warning("FOOBARBAZ: In controller_worker.as3worker.f5_sync: all(lb.provisioning_status == lib_consts.PENDING_DELETE for lb in loadbalancers)") self.sync.tenant_delete(network_id, device).raise_for_status() diff --git a/octavia_f5/controller/worker/l2_sync_manager.py b/octavia_f5/controller/worker/l2_sync_manager.py index 5ffcc08..b1c7616 100644 --- a/octavia_f5/controller/worker/l2_sync_manager.py +++ b/octavia_f5/controller/worker/l2_sync_manager.py @@ -282,7 +282,7 @@ def sync_l2_selfips_and_subnet_routes_flow(self, selfips: [network_models.Port], store = {'bigip': bigip, 'network': network} selfips_for_host = [selfip for selfip in selfips if bigip.hostname in selfip.name] - LOG.warning("In sync_l2_selfips_and_subnet_routes_flow. needed_selfips=selfips_for_host == ", selfips_for_host) + LOG.warning(f"In sync_l2_selfips_and_subnet_routes_flow. needed_selfips=selfips_for_host == {[sip.id for sip in selfips_for_host]}") fs[self.executor.submit(self._do_sync_l2_selfips_and_subnet_routes_flow, needed_selfips=selfips_for_host, store=store)] = bigip