4906: Integrations module should use named modules

mconley

There are numerous instances of files under reviewboard/reviewboard/integrations/ using this pattern:

import logging

# ...
logging.warning("Some warning message")
logging.error("Some error message")
logging.exception("Handled some exception")
# etc

etc. While this is useful, these logging messages can be made more useful if they are "named", meaning that the logging mechanism automatically includes information about the module where the logging message came from which can be handy when diagnosing problems from logs.

Named logging looks like this:

import logging

logger = logging.getLogger(__name__)
# ^-- make sure to do this at the top of the script execution scope so that `logger` is available throughout the script.

#...

logger.warning("This is a warning to a named logger")
logger.error("This is a error to a named logger")
logger.exception("This is a exception to a named logger")
# etc

The goal here is to replace the non-named usages to use named logging. The log messages themselves shouldn't change, and there's no need to add new log messages - just convert the ones that exist in this module directory to named logging.

mconley
#1 mconley
  • +EasyFix
#2 amohapatra

I can take care of this.

jordanvandenbruel
#3 jordanvandenbruel

I can look into this

Chaoyu
#4 Chaoyu

I can take one instead.

david
#5 david

Fixed for release-5.0.x (0efa050). This will ship in 5.0.

  • -New
    +Fixed