Comment 17 for bug 1836568

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

Reviewed: https://review.opendev.org/674940
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=0526718feed24fe47c469bba7542a468c204585c
Submitter: Zuul
Branch: master

commit 0526718feed24fe47c469bba7542a468c204585c
Author: Colleen Murphy <email address hidden>
Date: Tue Aug 6 14:40:30 2019 -0700

    Make policy deprecation reasons less verbose

    oslo.policy emits a deprecation warning for every single rule that we
    have deprecated at least once when a request comes in, and possibly
    subsequent times if keystone is running in multiple processes, and even
    more after keystone is reloaded. It's not useful to repeat nearly
    identical multi-line messages for each policy every time: logs aren't
    meant to justify our reasoning, they're just meant to be informative.
    The relevant information and rationalization can be found in the release
    notes. Repeating these identical multi-line messages starts to use up
    space quickly and makes the logs unreadable.

    This patch reduces the DEPRECATED_REASON for each policy to one brief
    line, which should help make the logs more readable in general and
    especially when run in a multiprocess environment, reduce the disk
    footprint of the logs, and help with log indexing.

    Change-Id: I98a0c06586b18dbd2f6681a24a5af1ea2de70951
    Partial-bug: #1836568