Open Bug 1669816 Opened 5 years ago Updated 1 year ago

fix logging and remove caplogpp pytest fixture (socorro)

Categories

(Socorro :: General, task, P3)

Product:
Socorro
Component:
General
Type:
task

Tracking

(Not tracked)

People

(Reporter: willkg, Unassigned)

Details

A while back, I reworked logging and in order to get tests to make assertions on things that were logged, I wrote a caplogpp test fixture which tweaks the "propagate" value on loggers.

Turns out that's all set up wrong and we shouldn't need to have a caplogpp at all.

This bug covers reworking the logging configuration:

  1. only the root logger specifies a handler
  2. the root logger sets the level to WARNING
  3. non-root loggers set propagate=True
  4. remove caplogpp (pretty sure we can remove it at this point)
Summary: fix logging and remove caplogpp pytest fixture → fix logging and remove caplogpp pytest fixture (socorro)

This would have been nice to do, but it's not needed for the GCP migration. Removing it from the GCP migration.

No longer blocks: 1687802
You need to log in before you can comment on or make changes to this bug.