In 4624c60c18f63e3ac72f25f143189d447aa9c610, I went through and tried to make the logs consistently use `logger` (for no reason other than that's what the majority of the modules were using).  Is there a reason to switch to `LOG`?  Now we have 9 modules using `logger =` and 3 using `LOG =`.  I think we should just pick one and be consistent.