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

Fix translation of label for system:addons service #4306

Merged
merged 1 commit into from
Sep 8, 2024

Conversation

lolodomo
Copy link
Contributor

The i18n file containing this label is in org.openhab.core bundle.

Fix openhab/openhab-webui#1669

Signed-off-by: Laurent Garnier [email protected]

@lolodomo lolodomo requested a review from a team as a code owner July 10, 2024 22:02
The i18n file containing this label is in org.openhab.core bundle.

Fix openhab/openhab-webui#1669

Signed-off-by: Laurent Garnier <[email protected]>
@lolodomo
Copy link
Contributor Author

It looks like all new PRs build is failing in integration tests:

[INFO] --- bnd-testing:7.0.0:testing (default) @ org.openhab.core.tests ---
[INFO] Matching glob *
[INFO] Testing /var/jenkins_home/workspace/PR-openHAB-Core/itests/org.openhab.core.tests/itest.bndrun:
[INFO] Verification of feature openhab.tp-commons-net/4.3.0.SNAPSHOT succeeded
[INFO] Verification of feature openhab.tp-gson/4.3.0.SNAPSHOT succeeded
[INFO] Verification of feature openhab-core-base/4.3.0.SNAPSHOT succeeded
[INFO] Verification of feature openhab.tp-hivemqclient/4.3.0.SNAPSHOT succeeded
     [java] Error: Could not find or load main class 'self';
     [java] Caused by: java.lang.ClassNotFoundException: 'self';
[INFO] 1 Error(s)
[ERROR] Error   : Exit code remote process 1: /var/jenkins_home/tools/hudson.model.JDK/OpenJDK_17/bin/java -Xms2G -Xmx4G -XX:+UseG1GC -XX:+ExplicitGCInvokesConcurrent -XX:+ParallelRefProcEnabled -XX:+UseStringDeduplication -Dhudson.model.DirectoryBrowserSupport.CSP="default-src 'self'; img-src 'self' data: *; style-src 'self' 'unsafe-inline'; child-src 'self'; frame-src 'self'; script-src 'self' 'unsafe-inline' 'unsafe-eval'; frame-ancestors 'self'" -Djavamelody.application-name=jenkins -cp /var/jenkins_home/workspace/PR-openHAB-Core/itests/org.openhab.core.tests/target/test/tmp/testing/itest/cnf/cache/7.0.0/bnd-cache/biz.aQute.launcher/biz.aQute.launcher.pre.jar -Dlauncher.properties=/var/jenkins_home/workspace/PR-openHAB-Core/itests/org.openhab.core.tests/target/test/tmp/testing/itest/generated/launch420501016096256862.properties -Djdk.util.zip.disableZip64ExtraFieldValidation=true -Dorg.osgi.service.http.port=46159 --add-opens=java.base/java.io=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.security=ALL-UNNAMED --add-opens=java.base/java.text=ALL-UNNAMED --add-opens=java.base/java.time=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.desktop/java.awt.font=ALL-UNNAMED --add-opens=java.naming/javax.naming.spi=ALL-UNNAMED --add-opens=java.rmi/sun.rmi.transport.tcp=ALL-UNNAMED -ea -Dlauncher.runpath=/var/jenkins_home/workspace/PR-openHAB-Core/itests/org.openhab.core.tests/target/test/tmp/testing/itest/cnf/cache/7.0.0/org.openhab.core.bom.runtime-index/org.eclipse.osgi-3.18.0.v20220516-2155.jar,/var/jenkins_home/workspace/PR-openHAB-Core/itests/org.openhab.core.tests/target/test/tmp/testing/itest/cnf/cache/7.0.0/bnd-cache/biz.aQute.launcher/biz.aQute.launcher-7.0.0.jar aQute.launcher.pre.EmbeddedLauncher

@wborn : any idea if something remains to be done after the switch from 4.2 to 4.3 ?

@lolodomo
Copy link
Contributor Author

lolodomo commented Jul 11, 2024

Locally on my Windows PC, the integration tests compile properly. So no idea what's going on.

@wborn wborn added rebuild Triggers the Jenkins PR build and removed rebuild Triggers the Jenkins PR build labels Jul 12, 2024
@wborn
Copy link
Member

wborn commented Jul 12, 2024

[java] Error: Could not find or load main class 'self';
[java] Caused by: java.lang.ClassNotFoundException: 'self';

This was caused by a Java option only used on Jenkins. I disabled it for now so it doesn't cause build issues, but the side effect is that CSS is not applied to the static analysis results when viewed in Jenkins. The parameter didn't propagate into the PR build environment when it was still build on an agent instead of the built-in node.

@J-N-K
Copy link
Member

J-N-K commented Aug 4, 2024

Why are the i18n properties not in the bundle that provides the service?

@lolodomo
Copy link
Contributor Author

lolodomo commented Aug 4, 2024

Copy link
Member

@holgerfriedrich holgerfriedrich left a comment

Choose a reason for hiding this comment

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

LGTM

@holgerfriedrich holgerfriedrich merged commit 3e912ec into openhab:main Sep 8, 2024
5 checks passed
@holgerfriedrich holgerfriedrich added this to the 4.3 milestone Sep 8, 2024
@holgerfriedrich holgerfriedrich added the bug An unexpected problem or unintended behavior of the Core label Sep 8, 2024
@lolodomo lolodomo deleted the system_addons_label branch September 8, 2024 07:42
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 the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Add-on Management" not translated
4 participants