Details
-
Improvement
-
Status: Closed
-
Normal
-
Resolution: Won't Fix
-
None
-
None
-
None
Description
This log message ...
2015-10-30 11:20:26,424 INFO [clojure-agent-send-off-pool-27] [p.s.j.puppet-environments] Marking all registered environments as expired.
|
... could use some improvement.
First of all, it is logged per-JRuby, and that's confusing - it looks like the cache is being invalidated N times (where N is the number of JRubies); and technically, it is, since the caching is per-JRuby; but if you just see a block of those messages go by, it can be confusing - especially so for an end-user who might not understand the guts of Puppet Server.
So, perhaps it should only be logged once at INFO level for the entire operation, and the per-JRuby messages could be bumped down to DEBUG. It would also be clearer if they mentioned that they were per-JRuby, perhaps by including the JRuby instance ID.