Skip to content

Upgrade Karaf from 4.4.7 to 4.4.8 - #1779

Merged
wborn merged 3 commits into
openhab:mainfrom
holgerfriedrich:k448
Aug 9, 2025
Merged

Upgrade Karaf from 4.4.7 to 4.4.8#1779
wborn merged 3 commits into
openhab:mainfrom
holgerfriedrich:k448

Conversation

@holgerfriedrich

Copy link
Copy Markdown
Member

@holgerfriedrich holgerfriedrich added the awaiting other PR Depends on another PR label Jul 29, 2025
@holgerfriedrich

Copy link
Copy Markdown
Member Author

@wborn, I could need some help.

Karaf 4.4.8 reached RC1 today, there will be a RC2 tomorrow.
It have patches for all OH repositories in place, the demo app is starting.

Though, there is one error on distro I am struggling with....

[INFO] openHAB Distribution ............................... FAILURE [  8.684 s]
[INFO] openHAB Feature Verification ....................... SUCCESS [01:53 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:29 min (Wall Clock)
[INFO] Finished at: 2025-07-29T23:56:31+02:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.karaf.tooling:karaf-maven-plugin:4.4.8-SNAPSHOT:assembly (default-assembly) on project openhab: Unable to build assembly: Unable to resolve root: missing requirement [root] osgi.identity; osgi.identity=eventadmin; type=karaf.feature; version=4.4.8.SNAPSHOT; filter:="(&(osgi.identity=eventadmin)(type=karaf.feature)(version>=4.4.8.SNAPSHOT))" [caused by: Unable to resolve eventadmin/4.4.8.SNAPSHOT: missing requirement [eventadmin/4.4.8.SNAPSHOT] osgi.identity; osgi.identity=org.apache.karaf.services.eventadmin; type=osgi.bundle; version="[4.4.8.SNAPSHOT,4.4.8.SNAPSHOT]"; resolution:=mandatory [caused by: Unable to resolve org.apache.karaf.services.eventadmin/4.4.8.SNAPSHOT: missing requirement [org.apache.karaf.services.eventadmin/4.4.8.SNAPSHOT] osgi.ee; filter:="(&(osgi.ee=JavaSE)(version=11))"]] -> [Help 1]

I am a bit stuck with that. Any recommendation?
Karaf moved from Java1.8 min to Java 11, could this be related?

@holgerfriedrich

Copy link
Copy Markdown
Member Author

Fixed the issue above by adding <javase>${oh.java.version}</javase>.
Now testing distro with 4.4.8 🥳

@holgerfriedrich

Copy link
Copy Markdown
Member Author

Now testing distro with 4.4.8 🥳

Besides those strange warnings in the log,

23:20:37.461 [WARN ] [nternal.config.MavenConfigurationImpl] - Location C:/Users/hf/.m2/settings.xml is not accessible (incorrect value for option "settings")
23:20:37.461 [WARN ] [nternal.config.MavenConfigurationImpl] - Location null/conf/settings.xml is not accessible (incorrect value for option "settings")
23:20:37.462 [WARN ] [nternal.config.MavenConfigurationImpl] - Location null/conf/settings.xml is not accessible (incorrect value for option "settings")
23:20:37.462 [WARN ] [nternal.config.MavenConfigurationImpl] - Location C:/Users/hf/.m2/settings-security.xml is not accessible (incorrect value for option "security")

everything seems to work fine 🎉

UI is working, plugin installation is fine. Bundle listing looks great!!!

@holgerfriedrich

Copy link
Copy Markdown
Member Author

The log issue will be fixed in RC2 of Karaf 4.4.8.
https://issues.apache.org/jira/browse/KARAF-7995

So it looks like we are ready once Karaf is released! Maybe the modularisation of osgi imports should be discussed. I will add a comment in core repo PR.

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
@holgerfriedrich
holgerfriedrich marked this pull request as ready for review August 7, 2025 18:48
@holgerfriedrich
holgerfriedrich requested a review from a team as a code owner August 7, 2025 18:48

@wborn wborn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job! 👍

@wborn
wborn merged commit 33e60a7 into openhab:main Aug 9, 2025
1 of 2 checks passed
@wborn wborn added enhancement An enhancement or new feature and removed awaiting other PR Depends on another PR labels Aug 9, 2025
@wborn wborn added this to the 5.1 milestone Aug 9, 2025
@holgerfriedrich
holgerfriedrich deleted the k448 branch August 9, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement An enhancement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants