Repository navigation
release(nzbhydra2): update docker.io/library/eclipse-temurin ( 17 → 25 ) - #277
renovate[bot] wants to merge 1 commit into
Conversation
|
Renovate PR AnalysisUpdate Summary
Release Changes
Breaking ChangesNone affecting our usage — verified empirically, not assumed: Empirical validation (built from this PR's branch and run locally):
Caveats a maintainer should weigh (documented, not observed):
Code Changes RequiredNone. The Dockerfile ENTRYPOINT flags are compatible with JDK 25:
Security ImpactJDK 17 → 25 brings two LTS generations of security fixes (JDK 21 + 24 CPU/PSU cycles and JDK 25 up to 25.0.4), including crypto and TLS hardening. nzbhydra2 is an internet-facing indexer proxy, so staying on a current LTS JRE reduces exposure. No new attack surface introduced by the base image swap itself (same Alpine 3.22 userland, same non-root RecommendationNeeds manual review — the image builds and the app functions correctly on Java 25 (verified by an actual build + run + HTTP/port/process checks from this PR's branch), but this is a major runtime base image bump, which is excluded from auto-merge by repo policy. Additionally, upstream documents Java 17 "not lower, not higher" as the supported runtime and the app runs on an EOL Spring Boot 3.1.x line that predates JDK 25. Functional smoke tests pass; the risk is medium-term (untested-by-upstream runtime, not immediate breakage). A maintainer should decide whether tracking the newest LTS JRE outweighs diverging from upstream's documented runtime. Do not merge under the automated policy. |
3b4db68 to
6e3b3af
Compare
Renovate PR AnalysisUpdate Summary
Release ChangesThis jumps two LTS lines (17 → 21 → 25). Notable JDK changes along the way (Temurin = OpenJDK builds, so the JDK JEP/release notes apply):
Breaking ChangesNone that block our usage, with caveats:
Code Changes RequiredNone required. Optional cleanups a maintainer may consider (not blockers):
Security ImpactMoving 17 → 25 brings the accumulated CPU security fixes of the 21 and 25 LTS lines, which is a net positive for this internet-facing web UI image. The native-access warning above slightly narrows future compatibility, not the current security posture. Verification Performed
RecommendationNeeds manual review — this is a major version bump of a runtime base image (JRE 17 → 25), which changes the container's runtime environment and moves the app outside Spring Boot 3.1's officially supported Java matrix, even though the full test suite passes empirically on Java 25.0.4. Per repo policy this class of update is never auto-merged; a maintainer should decide whether to accept the newer-JVM risk profile (including the tomcat native-access deprecation warning) or hold on 17/21. |
6e3b3af to
accd9f4
Compare
Renovate PR AnalysisUpdate Summary
Release ChangesDocker base image swap; the app layer (nzbhydra2 8.9.0 generic.zip) is untouched. JDK 17→25 deltas relevant at runtime:
Breaking ChangesUnverified-but-material risk; upstream signals conflict with this bump:
Code Changes RequiredNone mandatory for the PR mechanics (single FROM line, Security ImpactJDK 25 LTS receives current security patches; JDK 17 remains in support. No exploitable condition fixed here affects this image today. Note: this FROM line is tag-pinned, not sha256-digest-pinned — pre-existing deviation from repo policy, worth a separate fix. RecommendationNeeds manual review — major runtime base image bump (auto-merge excluded by policy), and upstream documents Java 17 "not higher" with a Spring Boot 3.1.5 stack that does not support JDK 25. Suggested options: (a) close this PR and stay on 17 until upstream moves, (b) retarget to |
accd9f4 to
a204399
Compare
Renovate PR Analysis(Re-analysis: previous comments predate this PR's latest update.) Update Summary
Release ChangesBase image update, not an app release. Verified the target tag exists with linux/amd64 and linux/arm64 manifests ( Breaking ChangesNone observed in CI: PR-branch Code Changes RequiredNone required for build/tests. Considerations for a maintainer:
Security ImpactNeutral-to-positive: newer LTS track for OpenJDK security fixes. No specific CVE motivates this bump (temurin 17 is also actively patched). RecommendationNeeds manual review — major version bump of a runtime base image in apps/ is an explicit never-auto-merge exclusion: it changes the container's runtime environment for every nzbhydra2 user. CI is green, so evidence says it works, but the LTS jump (17 → 25) deserves maintainer sign-off, ideally validated alongside #298's app major. |
a204399 to
6372486
Compare
Renovate PR AnalysisUpdate Summary
Release ChangesJava 17 → 25 spans two LTS cycles (21, 25). Highlights relevant to a Spring Boot JRE deployment:
Breaking Changes
Code Changes RequiredNone mechanically required. Empirical validation performed by building this PR's Dockerfile with app v9.0.5 (post-#303 state):
Security ImpactPositive in principle: a current LTS JRE receives JDK security fixes for longer. No new exposure introduced by the swap itself (same alpine variant, same non-root user, same process model). Note the base is tag-pinned rather than digest-pinned in the Dockerfile — pre-existing condition, unchanged by this PR. RecommendationNeeds manual review — major bump of the runtime base image for an app whose upstream documents "Java 17, not higher" as the supported runtime. My smoke test (fresh install, empty config) passes fully on Temurin 25, but indexer searches, updater behavior, and long-running database handling on Java 25 are untested upstream. Per repo policy, major runtime base bumps are never auto-merged. Suggested maintainer path: either hold at 17 until upstream declares Java 25 support (worth an upstream issue), or accept the empirical evidence here and merge after a soak test with a real |
6372486 to
7ebea6f
Compare
Renovate PR AnalysisUpdate Summary
Release ChangesJump from JDK 17 LTS to JDK 25 LTS (verified running image:
Breaking Changes
Code Changes RequiredNone strictly required to build/run:
Security ImpactJDK 25 LTS is the newest LTS line and will receive the newest JVM/security fixes sooner. Java 17 is still an LTS under support, so there is no urgent unpatched-CVE pressure forcing this bump. Neutral-to-positive for security; the risk here is upstream compatibility, not security. Verification performedBuilt and ran the image locally (Dockerfile with
RecommendationNeeds manual review — this is a major runtime base image bump for an app whose upstream explicitly documents Java 17 "not lower, not higher". Local smoke tests pass (build, startup, HTTP 200 on JDK 25.0.4.1), but upstream does not declare Java 25 support, and the app carries a Java-version guard that has been tightened before. A maintainer should either confirm upstream Java 25 support or defer this PR until the README constraint changes. Not merging per the repo's exclusion rule for major runtime base bumps. |
Renovate PR AnalysisUpdate Summary
Release ChangesJava 17 → 25 spans LTS releases 21 and 25. Notable JDK changes in between: JEP 400 UTF-8 by default (17), virtual threads finalized (21), generational Shenandoah (21), compact object headers and AOT cache (24/25), removal of legacy Solaris-specific code paths, and continuing removal of deprecated security manager APIs (final removal in 25 per JEP 486). For this app the relevant delta is the JRE runtime only — no build toolchain involved (we consume the prebuilt Breaking ChangesNone observed empirically, but upstream support contract is explicitly against it:
Code Changes RequiredNone required to build/run. The Security ImpactPositive in principle: JDK 25 receives active CVE patches (JDK 17 free updates continue only until Oct 2027; 25 until at least 2030). Alpine 3.22 base is unchanged. No new exposure introduced. RecommendationNeeds manual review — do not auto-merge. This is a major runtime-base bump (repo exclusion list) and upstream explicitly documents Java 17 "not higher". Empirical evidence on this PR is good (build + goss green on JDK 25), but future NZBHydra2 releases (VERSION var in |
Renovate PR AnalysisUpdate Summary
Release ChangesJava 17 → 25 spans LTS releases 21 and 25. Notable JDK deltas: JEP 400 UTF-8 default charset (18), finalized virtual threads (21), permanent Security Manager removal (JEP 486, 24), compact object headers + AOT cache improvements (24/25). For this image only the JRE runtime changes — we consume the prebuilt Breaking ChangesNone observed empirically, but upstream's support contract explicitly disagrees:
Code Changes RequiredNone required to build/run. Optional hygiene: Security ImpactPositive: JDK 25 receives active CVE fixes with support to at least Sep 2031 vs Oct 2027 for JDK 17 (Adoptium roadmap). Alpine 3.22 userland unchanged. No new exposure introduced. RecommendationNeeds manual review — do not auto-merge. This is a major version bump of a runtime base image (repo exclusion list), and upstream documents Java 17 "not lower, not higher". Empirical evidence on this PR is green (build + goss pass on JDK 25, run 36775349794), but a maintainer should explicitly accept the risk that future NZBHydra2 releases (VERSION var, currently 9.1.1) may assume JDK 17. Renovate automerge is already disabled for this PR. |
7ebea6f to
72cb086
Compare
This PR contains the following updates:
17-jre-alpine-3.22→25-jre-alpine-3.22Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.