Skip to content

Conversation

vinayakumarb
Copy link
Contributor

This commit introduces support for Debian 12 (Bookworm) and Debian 13 (Trixie) as build platforms, following the approach established for Ubuntu 24.

Key changes include:

  • Creation of Dockerfile_debian_12 and Dockerfile_debian_13 based on Dockerfile_ubuntu_24, with appropriate base images and package resolver arguments.
  • Updates to dev-support/docker/pkg-resolver/packages.json to include package definitions for debian:12 and debian:13.
  • Addition of debian:12 and debian:13 to dev-support/docker/pkg-resolver/platforms.json.
  • Modification of BUILDING.txt to list debian_12 and debian_13 as supported OS platforms.

… with JDK-17 as default

This commit introduces support for Debian 12 (Bookworm) and Debian 13 (Trixie)
as build platforms, following the approach established for Ubuntu 24.

Key changes include:
- Creation of `Dockerfile_debian_12` and `Dockerfile_debian_13` based on
  `Dockerfile_ubuntu_24`, with appropriate base images and package resolver
  arguments.
- Updates to `dev-support/docker/pkg-resolver/packages.json` to include
  package definitions for `debian:12` and `debian:13`.
- Addition of `debian:12` and `debian:13` to
  `dev-support/docker/pkg-resolver/platforms.json`.
- Modification of `BUILDING.txt` to list `debian_12` and `debian_13` as
  supported OS platforms.
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 23m 38s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 hadolint 0m 0s hadolint was not available.
+0 🆗 shellcheck 0m 0s Shellcheck was not available.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+0 🆗 jsonlint 0m 0s jsonlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 💚 shadedclient 37m 46s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
-1 ❌ blanks 0m 0s /blanks-eol.txt The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚 shadedclient 34m 39s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 53s The patch does not generate ASF License warnings.
98m 48s
Subsystem Report/Notes
Docker ClientAPI=1.51 ServerAPI=1.51 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8001/1/artifact/out/Dockerfile
GITHUB PR #8001
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs jsonlint
uname Linux 01b5b1df2935 5.15.0-143-generic #153-Ubuntu SMP Fri Jun 13 19:10:45 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 7ccea84
Max. process+thread count 554 (vs. ulimit of 5500)
modules C: . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8001/1/console
versions git=2.43.7 maven=3.9.11
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Copy link
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM

RUN echo 'APT::Install-Suggests "0";' >>  /etc/apt/apt.conf.d/10disableextras

This is there in only in 12, if possible we should sync it with 13 as well, if relevant. Feel free to commit once you do it

@pan3793
Copy link
Member

pan3793 commented Sep 28, 2025

Previously, there were concerns about having many versions of Linux dist Dockerfiles, how about upgrading Debian 10 to 13 directly?

Copy link
Contributor

@cnauroth cnauroth left a comment

Choose a reason for hiding this comment

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

+1. Thanks @vinayakumarb !

@slfan1989
Copy link
Contributor

Previously, there were concerns about having many versions of Linux dist Dockerfiles, how about upgrading Debian 10 to 13 directly?

@vinayakumarb Thank you very much for your contribution. However, I still have some concerns. Expanding support to many operating systems could be a rather heavy undertaking, since it requires us to pay closer attention to their EOL and version lifecycles. I'm wondering if it might be more sustainable to maintain a smaller subset of supported systems instead. If users have other requirements, they could always try customizing the build themselves.

cc: @pan3793 @ayushtkn @cnauroth

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants