Comment 1 for bug 2038948

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

Reviewed: https://review.opendev.org/c/starlingx/config/+/897852
Committed: https://opendev.org/starlingx/config/commit/352d1ce0b0acb538d5a715976cbb3c0c46cb8d28
Submitter: "Zuul (22348)"
Branch: master

commit 352d1ce0b0acb538d5a715976cbb3c0c46cb8d28
Author: amantri <email address hidden>
Date: Tue Oct 10 12:02:04 2023 -0400

    Fix misleading info log message in the conductor API

    The following is the misleading log

    sysinv 2023-10-06 21:30:35.237 22002 INFO
    sysinv.conductor.manager [-] admin endpoint config applied
    {'personalities': ['controller'],
    'host_uuids': ['13f3d7e8-6e30-4f6a-9de7-31fb5709bc5c'],
    'classes': ['platform::firewall::runtime'], 'force': False,
    'config_type': 'config_apply_runtime_manifest',
    'created_at': datetime.datetime(2023, 10, 6, 21, 30, 35, 237289)}

    While applying puppet class 'platform::firewall::runtime'
    the openstack module could set the
    constants.ADMIN_ENDPOINT_CONFIG_REQUIRED flag, this fix
    removes this and adds the proper log message.

    Testcases:
    PASS: Check sysinv.log and verify that misleading log is no
          shown

    Closes-bug: 2038948

    Change-Id: Ib475bea9a43bf3ff5b865af85a0b1558438f022b
    Signed-off-by: amantri <email address hidden>