Completely removed support for Debian:buster #1220

jmjaffe37 · 2025-07-23T23:29:46Z

This distro is over a year EOL and its default package servers went offline recently. This causes builds and PRs to fail since checkJdkDebian needs to run apt-get update on each docker-image that it tests on.

This PR implements option 1 described in this issue: #1218

github-actions

A block has been put on this Pull Request as this repository is temporarily under a code freeze due to an ongoing release cycle.

If this pull request needs to be merged during the release cycle then please comment /merge and a PMC member will be able to remove the block.

If the code freeze is over you can remove this block by commenting /thaw.

jmjaffe37 · 2025-07-23T23:54:13Z

@karianna I believe this is the PR that you want to merge to get "both" effects

steelhead31

Looks good.

sxa

Bear in mind that buster can still be used by any users who wish to (I wonder how many raspberry pis are out there still running a buster-based raspbian!). The repositories for buster have not been removed, but they are now in an archive repository. Adjusting sources.list accordingly means that buster is still usable. For that reason, I'd be somewhat -1 on removing it at this time, but we should perhaps stop it for new java releases.

This is similar to how we have not actively removed support for CentOS/RHEL versions that are out of support, which also require a similar fix to use an vault repository e.g. https://github.com/adoptium/infrastructure/blob/4f112e49583d5ef77be943ed72e5677671afe2ee/ansible/docker/Dockerfile.CentOS7#L8

While I appreciate that anyone on such a distribution may not be expecting security updates, I don't think suddenly removing that ability for our repository without warning is necessarily the best course of action here, and as a minimum it should at least be a PMC agenda item before removing it. Removing it now would (unless I'm mistaken) presumably mean that we would still make old Temurin versions available buster users. IMHO it would also be good if we could get the stats on how many users are hitting the repositories with the buster endpoints before making this decision.

For reference the following command is what's needed to set the repositories correctly to the archive ones - sed -i which is used for the CentOS vault ones should work too. I believe this would have been option 3 in the original issue.

perl -p -i -e 's/deb.debian.org/archive.debian.org/g' /etc/apt/sources.list

sxa · 2025-07-25T09:44:43Z

@steelhead31 Thinking about it I don't think we've got download stats for this repo have we, so we can't necessarily tell how many are accessing the buster URL?

sxa · 2025-07-25T09:49:58Z

(Also noting that there is a valid counter-argument to my earlier comment the other way, which is that if they have to move to the archive repository for the main packages, they should perhaps also adjust the adoptium repo to do s/buster/bookworm or similar, which would also still work and I think may be more consistent with what we do for Ubuntu STS releases)

jmjaffe37 · 2025-07-25T17:41:34Z

@sxa, I see your points. If we are currently doing this for the CentOS image(s), then we could do something similar here. That said, I think this is a good option for now for 2 reasons:

If a user follows all the steps in the README.md at this time, they will be unable to create deb packages without experiencing an error due to a failed test. This will mess-up any automated pipelines that run this (I needed to implement this fix in the Microsoft mirror to get our automation to build these properly).
I agree with your phrasing of the "valid counter-argument"

karianna · 2025-07-27T23:44:51Z

Added PMC Agenda to this. I think it's worthy of a debate there on our "Do we support EOL platforms, and if so, under what circumstances" :-)

sxa

I'm going to put a block on this for now as we were unable to have the discussion at this week's PMC call so it will be deferred until next week and I want to avoid this being merged in the meantime :-)

jmjaffe37 · 2025-08-04T17:48:33Z

@sxa, @karianna, are there any updates on this change? As a reminder, pipelines that build deb files are currently broken without this change since checkJdkDebian will always error until a change is made

karianna · 2025-08-04T18:30:10Z

PMC meeting this week to resolve hopefully

Block was due to a desire for a PMC discussion.

sxa · 2025-08-13T13:55:09Z

@sxa, @karianna, are there any updates on this change? As a reminder, pipelines that build deb files are currently broken without this change since checkJdkDebian will always error until a change is made

I thought we'd updated the distribution which was used for building them independently of this, and this was purely to determine which distributions we would publish to in the apt repository going forward. Are you saying something additional is broken?

steelhead31 · 2025-08-13T14:00:54Z

linux/Jenkinsfile

@@ -532,7 +532,6 @@ def uploadDebArtifacts(String buildArch, String Version) {
            "trixie",   // Debian/13
            "bookworm", // Debian/12
            "bullseye", // Debian/11
-            "buster",   // Debian/10


I think this should be left in the jenkinsfile, as its just used to publish packages in the artifactory repo.

linux_new/Jenkinsfile

.github/workflows/cacert-publish.yml

karianna · 2025-08-14T08:51:40Z

@jmjaffe37 some review comments for you.

…ehavior

sxa

Idendation issue, but I'll defer to @steelhead31 for the fnal review when he's back next week.

sxa · 2025-08-15T08:36:58Z

docs/Guide_To_The_Linux_Installers.md

@@ -507,6 +508,7 @@ def deb_versions = [
 		"trixie",   // Debian/13
 		"bookworm", // Debian/12
 		"bullseye", // Debian/11
+    "buster",   // Debian/10


The indendation here looks inconsistent with the other lines

@sxa, I noticed this in the GH UI as well, but it is consistent in my branch when I look at it locally through VS Code so I am unsure what it happening here

docs/Guide_To_The_Linux_Installers.md

steelhead31

Indentation as above is inconsistent, but otherwise looks good. Approving

Co-authored-by: Scott Fryer <60462088+steelhead31@users.noreply.github.com>

jmjaffe37 · 2025-08-19T16:19:30Z

@karianna, @steelhead31, @sxa, @tellison, all requested changes are in. Please reapprove when possible and we can merge this fix into main :)

sxa · 2025-08-19T16:31:59Z

/thaw

Pull Request unblocked - code freeze is over.

github-actions bot previously requested changes Jul 23, 2025

View reviewed changes

jmjaffe37 mentioned this pull request Jul 23, 2025

Testing on Debian:Buster (past EOL) is causing jdk:debian:checkJdkDebian to fail. This blocks all future Debian builds and PRs #1218

Open

karianna approved these changes Jul 23, 2025

View reviewed changes

steelhead31 approved these changes Jul 25, 2025

View reviewed changes

tellison approved these changes Jul 25, 2025

View reviewed changes

sxa reviewed Jul 25, 2025

View reviewed changes

karianna added the PMC-agenda label Jul 27, 2025

sxa previously requested changes Jul 30, 2025

View reviewed changes

jmjaffe37 force-pushed the jmj/remove_deb_buster_support branch from 2efc153 to 35e0917 Compare August 4, 2025 17:44

steelhead31 requested changes Aug 13, 2025

View reviewed changes

.github/workflows/cacert-publish.yml Outdated Show resolved Hide resolved

jmjaffe37 added 2 commits August 14, 2025 09:29

Removed debian_buster from checkJdkDebian tests to restore expected b…

5264b9c

…ehavior

Removed all remaining references do debian buster

f9354a9

jmjaffe37 force-pushed the jmj/remove_deb_buster_support branch from 35e0917 to f9354a9 Compare August 14, 2025 16:29

Responded to PR feedback

0113fdf

karianna requested review from steelhead31 and sxa August 14, 2025 23:39

sxa reviewed Aug 15, 2025

View reviewed changes

steelhead31 reviewed Aug 19, 2025

View reviewed changes

docs/Guide_To_The_Linux_Installers.md Outdated Show resolved Hide resolved

steelhead31 approved these changes Aug 19, 2025

View reviewed changes

jmjaffe37 and others added 2 commits August 19, 2025 09:11

Committed gh suggestion

bf26063

Co-authored-by: Scott Fryer <60462088+steelhead31@users.noreply.github.com>

fixed formatting

62415da

jmjaffe37 requested review from sxa, karianna, steelhead31 and tellison August 19, 2025 16:17

steelhead31 merged commit 42a88f9 into adoptium:master Aug 20, 2025
67 checks passed

tellison removed the PMC-agenda label Aug 20, 2025

jmjaffe37 deleted the jmj/remove_deb_buster_support branch August 20, 2025 17:16

Uh oh!

Completely removed support for Debian:buster #1220

Completely removed support for Debian:buster #1220

Uh oh!

Conversation

jmjaffe37 commented Jul 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

jmjaffe37 commented Jul 23, 2025

Uh oh!

steelhead31 left a comment

Choose a reason for hiding this comment

Uh oh!

sxa left a comment

Choose a reason for hiding this comment

Uh oh!

sxa commented Jul 25, 2025

Uh oh!

sxa commented Jul 25, 2025

Uh oh!

jmjaffe37 commented Jul 25, 2025

Uh oh!

karianna commented Jul 27, 2025

Uh oh!

sxa left a comment

Choose a reason for hiding this comment

Uh oh!

jmjaffe37 commented Aug 4, 2025

Uh oh!

karianna commented Aug 4, 2025

Uh oh!

sxa commented Aug 13, 2025

Uh oh!

steelhead31 Aug 13, 2025

Choose a reason for hiding this comment

Uh oh!

jmjaffe37 Aug 14, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

karianna commented Aug 14, 2025

Uh oh!

sxa left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sxa Aug 15, 2025

Choose a reason for hiding this comment

Uh oh!

jmjaffe37 Aug 19, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

steelhead31 left a comment

Choose a reason for hiding this comment

Uh oh!

jmjaffe37 commented Aug 19, 2025

Uh oh!

sxa commented Aug 19, 2025

Uh oh!

Uh oh!

Uh oh!

jmjaffe37 commented Jul 23, 2025 •

edited

Loading

sxa left a comment •

edited

Loading