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.

Sign up for GitHub

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

Jump to bottom

[metering] NoClassDefFoundError - mandatory transitive Micrometer dependencies missing #2288

Closed
pravussum opened this issue Apr 13, 2021 · 1 comment · Fixed by #2289
Closed

[metering] NoClassDefFoundError - mandatory transitive Micrometer dependencies missing #2288

pravussum opened this issue Apr 13, 2021 · 1 comment · Fixed by #2289
Assignees
@pravussum
Labels
bug An unexpected problem or unintended behavior of the Core

Comments

@pravussum
Copy link
Contributor

This has originally been reported in openhab-addons:
openhab/openhab-addons#10515

The actual issue is located within the core, since the Metrics IO addon makes use of the MeterRegistryProvider provided by the core (org.openhab.core.io.monitor bundle). The Micrometer classes depend on a transitive dependency, which is only present in the Metrics IO addon, but not in the core bundle.

The solution is to include mandatory transitive dependencies of Micrometer in the core bunde.
@pravussum
Copy link
Contributor Author

Can someone assign this to me please?

pravussum pushed a commit to pravussum/openhab-core that referenced this issue Apr 13, 2021
[metering] Fix dependency issue in io.monitor bundle (openhab#2288)
ee9a15c 
Signed-off-by: Robert Bach <[email protected]>
@pravussum pravussum mentioned this issue Apr 13, 2021
Merged
@cweitkamp cweitkamp mentioned this issue Apr 14, 2021
Closed
@cweitkamp cweitkamp added the bug An unexpected problem or unintended behavior of the Core label Apr 14, 2021
pravussum pushed a commit to pravussum/openhab-core that referenced this issue Apr 18, 2021
[metering] incorporate feedback from CR; fix build warnings (openhab#…
246460d 
…2288)

Signed-off-by: Robert Bach <[email protected]>
pravussum pushed a commit to pravussum/openhab-core that referenced this issue Apr 18, 2021
[metering] incorporate feedback from CR; fix build warnings (openhab#…
e1411ff 
…2288)

Signed-off-by: Robert Bach <[email protected]>
kaikreuzer pushed a commit that referenced this issue Apr 28, 2021
@pravussum
[metering] Fix dependency issue in io.monitor bundle (#2288) (#2289)
da05f8e 
Signed-off-by: Robert Bach <[email protected]>
fwolter pushed a commit to fwolter/openhab-core that referenced this issue May 24, 2021
@pravussum @fwolter
[metering] Fix dependency issue in io.monitor bundle (openhab#2288) (o…
9293288 
…penhab#2289)

Signed-off-by: Robert Bach <[email protected]>
splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this issue Jul 11, 2023
@pravussum @splatch
[metering] Fix dependency issue in io.monitor bundle (openhab#2288) (o…
095091e 
…penhab#2289)

Signed-off-by: Robert Bach <[email protected]>
GitOrigin-RevId: da05f8e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pravussum pravussum

Labels
bug An unexpected problem or unintended behavior of the Core
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[metering] Fix dependency issue in io.monitor bundle (#2288) pravussum/openhab-core
3 participants
@kaikreuzer @cweitkamp @pravussum