Paul Rütter created FELIX-6875:
----------------------------------
Summary: Modules not built by CI fail to build or test on JDK 21
and 25
Key: FELIX-6875
URL: https://issues.apache.org/jira/browse/FELIX-6875
Project: Felix
Issue Type: Task
Components: Bundle Repository (OBR), Configuration Admin, Coordinator
Service, Declarative Services (SCR), Features, File Install, Health Checks,
Inventory, Metatype Service, Web Console
Reporter: Paul Rütter
Assignee: Paul Rütter
h2. Problem
CI builds 17 of the repository's 34 top-level modules. The other modules are
never built, so they have drifted: the Dependabot bumps merged into them were
merged without a build.
Building the unbuilt modules that have been maintained recently (or only need a
trivial change) with \{{mvn clean verify}} on JDK 21 and JDK 25 shows that 21
of the 22 build units fail on master. Only \{{main.distribution}} passes. The
two JDKs fail the same way. None of the failures is caused by source code using
a JDK construct that was removed; they come from tooling and test
infrastructure:
||Cause||Modules||
|\{{--release 7}} is no longer accepted (JDK 21), through felix-parent
5/6|metatype, coordinator, healthcheck/api, healthcheck/annotation|
|felix-parent 3 runs a JavaScript Ant task; Nashorn was removed in JDK
15|scr-ext-anno|
|felix-parent 7 uses ianal-maven-plugin 1.0-alpha-1, which cannot open
\{{java.io.File}} on JDK 17+|inventory, features, webconsole-plugins (5),
healthcheck/webconsoleplugin|
|Mockito 1.x defines classes through cglib, which JDK 17+ refuses; Mockito 5.17
and earlier cannot mock on JVM 25|configadmin-plugins/interpolation,
healthcheck/generalchecks, healthcheck/core, healthcheck/webconsoleplugin|
|Integration tests pinned to an old framework release|healthcheck/core|
|\{{bnd-maven-plugin}} 4.1.0 throws a
ConcurrentModificationException|healthcheck/api|
|Depends on a SNAPSHOT of its own module that no longer exists, and compiles to
Java 1.5|bundlerepository.osgi-ct|
|A test that sets \{{java.io.tmpdir}} at runtime, which JDK 12+ reads only once
at startup|fileinstall|
h2. Change
* Move the affected modules from felix-parent 2.1-7 to felix-parent 9, as the
other modules already were.
* Use mockito-core 5.18.0 and rename the \{{org.mockito.Matchers}} imports
(removed in Mockito 4) to \{{ArgumentMatchers}}. Keep a \{{null}} argument
matching with \{{nullable(...)}} in HttpRequestsCheckTest.
* Run healthcheck/core's integration tests on framework 8.0.0.
* Use bnd-maven-plugin 7.1.0 in healthcheck/api.
* Point osgi-ct at the released bundlerepository 2.0.10.
* Make fileinstall's testInvalidTempDir configure the temp directory through
\{{felix.fileinstall.tmpdir}}.
h2. Impact
* *Java level:* bundles that were compiled for Java 1.5 to 7 are now compiled
for Java 8 (felix-parent 9). The next release of each of those bundles will
therefore require Java 8, and its \{{Require-Capability: osgi.ee}} changes
accordingly. Released artifacts are not affected.
* No change to the bundles' code or runtime behaviour. Everything else is build
configuration and tests.
h2. Out of scope
* eventadmin, systemready and rootcause: not released since 2021/2022 and not
maintained, so they are left as they are.
* Adding these modules to the CI workflow, so that they cannot drift again.
h2. Related
* FELIX-6807 (Java 25 LTS support)
* FELIX-6870 (tests run against released bundles instead of the code being
built)
--
This message was sent by Atlassian Jira
(v10.3.26#10030026)
