Skip to content

Commit

Permalink
Fix printf debugging of SelfIPs: Print their IDs
Browse files Browse the repository at this point in the history
  • Loading branch information
BenjaminLudwigSAP committed Jun 21, 2024
1 parent e2c91a0 commit 6b268f7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion octavia_f5/controller/worker/controller_worker.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down
2 changes: 1 addition & 1 deletion octavia_f5/controller/worker/l2_sync_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit 6b268f7

Please sign in to comment.