Comment 11 for bug 2036952

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to octavia (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/octavia/+/897777
Committed: https://opendev.org/openstack/octavia/commit/57833dbdad964f8a7861a0ab1d2847159f483577
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 57833dbdad964f8a7861a0ab1d2847159f483577
Author: Gregory Thiemonge <email address hidden>
Date: Mon Sep 25 07:48:01 2023 -0400

    Retry to set loadbalancer prov status on failures

    In case of DB outages when a flow is running, an exception is caught and
    the flow is reverted. In most of the flows, the revert function of the
    first task's (the last to be reverted) unlocks the load balancer by
    setting its provisioning status (to ERROR or ACTIVE, depending on the
    flow), but it fails if the DB is not reachable, leaving the LB in
    a PENDING_* state.
    This commit adds tenacity.retry to those functions, Octavia retries to
    set the status during ~2h45 (2000 attempts, 1 sec initial delay, 5 sec
    max delay).

    Note: stable/2023.1 and older, the patch also includes modifications for
    v1/tasks/lifecycle_tasks.py

    Conflicts:
            octavia/tests/unit/controller/worker/test_task_utils.py
            octavia/controller/worker/v1/tasks/lifecycle_tasks.py

    Closes-Bug: #2036952
    Change-Id: I458dd6d6f5383edc24116ea0fa27e3a593044146
    (cherry picked from commit be91493332786365b8e997fcf88779a12d1ae130)
    (cherry picked from commit 96782e2c543bfb488c7629e9ba2cf009b2a6b033)