From e10f22759a615a3241d73cb8fb2e1cb6cea043a1 Mon Sep 17 00:00:00 2001 From: Benjamin Ludwig Date: Wed, 22 Jan 2025 17:49:45 +0100 Subject: [PATCH] Revert method: Change availability zone back Don't log the current LB's AZ, since that would be one additional DB query, and the new AZ has already been logged by the execute method. --- octavia_f5/api/drivers/f5_driver/tasks/reschedule_tasks.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/octavia_f5/api/drivers/f5_driver/tasks/reschedule_tasks.py b/octavia_f5/api/drivers/f5_driver/tasks/reschedule_tasks.py index 5b25b12..0d7bac4 100644 --- a/octavia_f5/api/drivers/f5_driver/tasks/reschedule_tasks.py +++ b/octavia_f5/api/drivers/f5_driver/tasks/reschedule_tasks.py @@ -133,6 +133,8 @@ def revert(self, result, load_balancer: models.LoadBalancer, candidate: str, rem LOG.error("RewriteLoadBalancerEntry: Unable to update loadbalancer %s", load_balancer.id) return - LOG.warning("RewriteLoadBalancerEntry: Reverting host change of loadbalancer %s from '%s' to '%s'", + + LOG.warning("RewriteLoadBalancerEntry: Reverting host change of loadbalancer %s: Changing host from '%s' to '%s' and availability zone back to '%s'.", load_balancer.id, candidate, removal_host) - self._loadbalancer_repo.update(db_apis.get_session(), load_balancer.id, server_group_id=removal_host) + self._loadbalancer_repo.update(db_apis.get_session(), load_balancer.id, + server_group_id=removal_host, availability_zone=load_balancer.availability_zone)