Bug 2029916 - Excluding system facts logs as WARN causing log files to increase dramatically
Summary: Excluding system facts logs as WARN causing log files to increase dramatically
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Candlepin
Classification: Community
Component: candlepin
Version: 3.1
Hardware: x86_64
OS: Linux
unspecified
high
Target Milestone: ---
: ---
Assignee: candlepin-bugs
QA Contact:
URL:
Whiteboard:
Depends On: 2029548
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-12-07 14:47 UTC by Rehana
Modified: 2022-01-20 10:16 UTC (History)
5 users (show)

Fixed In Version: candlepin-4.1.9-1
Clone Of: 2029548
Environment:
Last Closed: 2022-01-20 10:16:49 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github candlepin candlepin pull 3184 0 None Merged 2029916: lower logging level for excluded facts 2021-12-15 09:18:32 UTC

Description Rehana 2021-12-07 14:47:07 UTC
+++ This bug was initially created as a clone of Bug #2029548 +++

Description of problem:
When adding the option 'candlepin.consumer.facts.match_regex' /etc/candlepin/candlepin.conf we see all facts that are being excluded logged as WARN. Since this option is not included in the candlepin.conf file by default and it is manually being placed here, I do not believe the exclusions should be or a WARN type. Since this option is also used to exclude systems facts on hosts where the number of these facts are causing long queries or a lot of system resource usage, I believe these should be logged as DEBUG level in the logs to limit the unnecessary logging of INFO+.

Version-Release number of selected component (if applicable):
candlepin-3.1.28-1.el7sat.noarch

How reproducible:
Every time

Steps to Reproduce:
1. Add regex exclusion to the candlepin.conf file for host facts and watch the logs for the matches to appear as WARN level.
2.
3.

Actual results:
Candlepin log files increase in size 3-4x 

Expected results:
This logging should be set on the level of DEBUG as this is an optional parameter to add for systems with a lot of facts.

Additional info:


Note You need to log in before you can comment on or make changes to this bug.