Comment 11 for bug 2033294

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to distcloud (master)

Reviewed: https://review.opendev.org/c/starlingx/distcloud/+/906112
Committed: https://opendev.org/starlingx/distcloud/commit/91c9d301d8cc9e16520599dd1f12d1a60b6370bb
Submitter: "Zuul (22348)"
Branch: master

commit 91c9d301d8cc9e16520599dd1f12d1a60b6370bb
Author: rlima <email address hidden>
Date: Fri Jan 19 10:42:23 2024 -0300

    Fix issues introduced by pylint/pep8 for dcmanager

    This commit fixes some regressions caused by the merge with
    master in the original commit.
    1 - Small formatting changes to be more consistent with
        existing code;
    2 - Removal of require_audit_flag that was mistakenly
        reintroduced during a rebase;
    3 - Repositioning of eventlet.monkey_patch() so it
        happens before the other imports;
    4 - Fix the admin IP conditional so it matches previous
        behavior;
    5 - Change the determine_image_fields conditional so it
        matches previous behavior.

    Related commit:
     https://review.opendev.org/c/starlingx/distcloud/+/893000

    Test Plan:
    1. Perform `tox` command
    - Pass in py39, pylint, pep8:

    Closes-bug: 2033294

    Change-Id: Icea2a80e2e4eed79f37bf49878e6fdea0db13a56
    Signed-off-by: rlima <email address hidden>