-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[digitalstrom] Fix: Remove excessive log statements for unknown Application Groups #11433
[digitalstrom] Fix: Remove excessive log statements for unknown Application Groups #11433
Conversation
…Groups Signed-off-by: Rouven Schürch <r.schuerch@gmx.ch>
I imagine this warning was introduced for a reason. Rather than removing it, wouldn't it make sense to rather reduce its level to DEBUG (or even TRACE) ? Like that, it will not be logged for standard users. |
Actually, the log statement had been introduced by me, under the impression of being a good idea - which it was not. At the time of writing my digitalSTROM installation consisted of a few items, the log statement was negligible, now running a few items more, the log spam is considerable. |
Ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…icationGroups (openhab#11433) Signed-off-by: Rouven Schürch <r.schuerch@gmx.ch> Signed-off-by: Dave J Schoepel <dave@theschoepels.com>
…icationGroups (openhab#11433) Signed-off-by: Rouven Schürch <r.schuerch@gmx.ch> Signed-off-by: Nick Waterton <n.waterton@outlook.com>
…icationGroups (openhab#11433) Signed-off-by: Rouven Schürch <r.schuerch@gmx.ch>
…icationGroups (openhab#11433) Signed-off-by: Rouven Schürch <r.schuerch@gmx.ch>
Fixes #10939, excessive log statements. The occurence of the unknown Application Groups is still unclear and a first try to squelch the logging messages failed.
The WARN log statement has been removed competely as there is no means of correcting the digitalSTROM configuration on the DSS or in the binding.
Signed-off-by: Rouven Schürch r.schuerch@gmx.ch