Issue #8499
Updated by bmbouter over 3 years ago
## Problem
Currently warnings are implemented with `warnings.warn` which is not being shown in the logs.
`#python` pointed me to [this interesting section of the docs](https://docs.python.org/3/howto/logging.html#when-to-use-logging) contrasting warning.warn() vs logging.warning() which suggests warning.warn is right, but then in the channel 3+ well-known python folks said that even though Python has DeprecatedWarning we shouldn't use it because in practice no one see's it. In order to see it users would have to run Python -Wall which is not practical.
Ultimately our goal with these warnings was to have both users AND developers see them. To do that, the devs in #python recommended we switch to logging.warning().
## Fix
1. Instead we should convert all usage to `logging.warning`.
2. Update the [plugin writer docs](https://docs.pulpproject.org/pulpcore/plugins/plugin-writer/concepts/index.html#plugin-api-stability-and-deprecation-policy) also.
## Downside
Downstreams will emit warnings and they would have to patch the downstream to remove them. Unfortunately I think that is what we'll have to do.
.