HADOOP-19819. Remove obsolete OS platform Dockerfiles#8257
Open
pan3793 wants to merge 5 commits intoapache:trunkfrom
Open
HADOOP-19819. Remove obsolete OS platform Dockerfiles#8257pan3793 wants to merge 5 commits intoapache:trunkfrom
pan3793 wants to merge 5 commits intoapache:trunkfrom
Conversation
|
💔 -1 overall
This message was automatically generated. |
pan3793
commented
Feb 19, 2026
| # Install packages | ||
| #### | ||
| RUN pkg-resolver/install-maven.sh debian:13 | ||
| RUN pkg-resolver/install-cmake.sh debian:13 |
Member
Author
There was a problem hiding this comment.
Debian 13 APT repo contains CMake 3.31.6, which satisfies the version requirement - 3.19 or newer
chengpan@18fd9dedaa29:~/hadoop$ cmake --version
cmake version 3.31.6
pan3793
commented
Feb 19, 2026
| # Install packages | ||
| ###### | ||
| RUN pkg-resolver/install-maven.sh rockylinux:8 | ||
| RUN pkg-resolver/install-cmake.sh rockylinux:8 |
Member
Author
There was a problem hiding this comment.
RockyLinux 8 YUM repo contains CMake 3.26.5, also satisfies the version requirement - 3.19 or newer
[chengpan@045e7222babb hadoop]$ cmake --version
cmake version 3.26.5
CMake suite maintained and supported by Kitware (kitware.com/cmake).
pan3793
commented
Feb 19, 2026
| @@ -1,55 +0,0 @@ | |||
| #!/usr/bin/env bash | |||
Member
Author
There was a problem hiding this comment.
previously used by CentOS 7, useless now.
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
Contributor
steveloughran
left a comment
steveloughran
left a comment
There was a problem hiding this comment.
not a docker code but cutting old stuff does make sense. I do think I used that ubuntu_20 for the 3.4.3 release so this isn't for backporting
Member
Author
@steveloughran, absolutely, it is for trunk only. I might be more conservative about backporting than you, generally. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
pan3793
force-pushed
the
HADOOP-19819
branch
from
c2c5227 to
8488805
Compare
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
pan3793
force-pushed
the
HADOOP-19819
branch
from
8488805 to
d5d5eb7
Compare
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
pan3793
commented
Feb 22, 2026
| ], | ||
| "debian:13": [ | ||
| "fuse3", | ||
| "libfuse3-dev" |
Member
Author
There was a problem hiding this comment.
it's a bug, fuse and libfuse-dev should be used, otherwise, -Drequire.fuse causes failure.
Contributor
There was a problem hiding this comment.
I’m a bit confused—why do we need to remove Debian 12 when its LTS doesn’t expire until after June 30, 2028?
Contributor
There was a problem hiding this comment.
These two PRs (HADOOP-19659: upgrading the build-environment Dockerfile from Debian 10 to 11; HADOOP-19709: adding Debian 12/13 as build platforms and making JDK 17 the default) were both merged within the last six months.
- HADOOP-19659. Upgrade Debian 10 to 11 in build env Dockerfile (HADOOP-19659. Upgrade Debian 10 to 11 in build env Dockerfile #7898) Contributed by Cheng Pan. - 2025-09-01
- HADOOP-19709. [JDK17] Add debian:12 and debian:13 as a build platform with JDK-17 as default (HADOOP-19709. [JDK17] Add debian:12 and debian:13 as a build platform with JDK-17 as default #8001) Contributed by Vinayakumar B. - 2025-10-22
Member
Author
There was a problem hiding this comment.
@slfan1989, I restored Debian 12, and verified it locally (because it is not covered by Jenkins)
pan3793
force-pushed
the
HADOOP-19819
branch
from
d5d5eb7 to
c9c6e62
Compare
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
(!) A patch to the testing environment has been detected. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of PR
fuseandlibfuse-devshould be used, instead offuse3andlibfuse3-devHow was this patch tested?
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?AI Tooling
No usage.