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.
Reviewed: https:/ /review. opendev. org/674940 /git.openstack. org/cgit/ openstack/ keystone/ commit/ ?id=0526718feed 24fe47c469bba75 42a468c204585c
Committed: https:/
Submitter: Zuul
Branch: master
commit 0526718feed24fe 47c469bba7542a4 68c204585c
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: I98a0c06586b18d bd2f6681a24a5af 1ea2de70951
Partial-bug: #1836568