Skip to content
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

[jdbc] Fix logging not working #15722

Merged
merged 1 commit into from
Oct 8, 2023
Merged

[jdbc] Fix logging not working #15722

merged 1 commit into from
Oct 8, 2023

Conversation

wborn
Copy link
Member

@wborn wborn commented Oct 8, 2023

The dependency change in #15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

The dependency change in openhab#15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

Signed-off-by: Wouter Born <github@maindrain.net>
@wborn wborn added bug An unexpected problem or unintended behavior of an add-on regression Regression that happened during the development of a release. Not shown on final release notes. labels Oct 8, 2023
@wborn wborn requested a review from a team as a code owner October 8, 2023 15:40
@jlaur
Copy link
Contributor

jlaur commented Oct 8, 2023

@wborn - thanks! I should have reacted. 😞 I saw the same on my development system with latest snapshot versions, but thought my system was just getting too messy after the pain I had put it through.

@jlaur jlaur merged commit 756b32d into openhab:main Oct 8, 2023
3 checks passed
@jlaur jlaur added this to the 4.1 milestone Oct 8, 2023
@wborn wborn deleted the jdbc-fix-logging branch October 8, 2023 18:00
@lolodomo
Copy link
Contributor

lolodomo commented Oct 8, 2023

Hey guys, Kai asked to merge nothing until tonight...

@jlaur
Copy link
Contributor

jlaur commented Oct 8, 2023

Hey guys, Kai asked to merge nothing until tonight...

Please see latest update. 😉

@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/openhab-4-1-milestone-discussion/149502/52

pat-git023 pushed a commit to pat-git023/openhab-addons that referenced this pull request Oct 13, 2023
The dependency change in openhab#15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

Signed-off-by: Wouter Born <github@maindrain.net>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 21, 2023
The dependency change in openhab#15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

Signed-off-by: Wouter Born <github@maindrain.net>
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 29, 2023
The dependency change in openhab#15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: querdenker2k <querdenker2k@gmx.de>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
The dependency change in openhab#15659 caused slf4j to be embedded into the bundle.

As a result the bundle no longer uses Pax Logging so it cannot find an SLF4J provider and logs the following when it is installed:

SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.

Signed-off-by: Wouter Born <github@maindrain.net>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on regression Regression that happened during the development of a release. Not shown on final release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants