-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-19540. [JDK17] Add ubuntu:noble as a build platform with JDK-17 as default #7608
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
ayushtkn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error message on aarch64 ain't cheese
ayushsaxena@ayushsaxena hadoop % ./start-build-env.sh ubuntu_24
[+] Building 0.0s (2/2) FINISHED
=> [internal] load .dockerignore 0.0s
=> => transferring context: 2B 0.0s
=> [internal] load build definition from Dockerfile_ubuntu_24_aarch64 0.0s
=> => transferring dockerfile: 2B 0.0s
ERROR: failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount1675973539/Dockerfile_ubuntu_24_aarch64: no such file or directory
ayushsaxena@ayushsaxena hadoop %
Can we update the BUILDING.txt to reflect which ones are supported for aarch64.
Rest if the build comes clean, changes LGTM
|
💔 -1 overall
This message was automatically generated. |
|
All the failures in compilation are due to below. Nothing to do with changes in this PR. |
877f792 to
0db1122
Compare
Ack. Updated the BUILDING.txt and added error message in |
|
💔 -1 overall
This message was automatically generated. |
ayushtkn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Something crazy with the last build as well, see if you can get a clean build, before merging, haven't seen this error before, not sure why is it crying in yarn for protoc
|
💔 -1 overall
This message was automatically generated. |
|
Ran the failed tests locally.. |
0db1122 to
bcd73e6
Compare
|
💔 -1 overall
This message was automatically generated. |
...-project/hadoop-hdfs-native-client/src/main/native/libhdfspp/include/hdfspp/block_location.h
Outdated
Show resolved
Hide resolved
cnauroth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 from me, pending resolution of the last round of feedback from @pan3793 . Thanks @vinayakumarb !
…7 as default 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
bcd73e6 to
1bea33c
Compare
|
💔 -1 overall
This message was automatically generated. |
|
Thanks everyone for reviews. |
@vinayakumarb Thank you for the contribution! |
…7 as default (apache#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
|
@vinayakumarb Should the PR supporting Ubuntu 24 be backported to branch-3.4, even though branch-3.4 currently does not support JDK17? cc: @steveloughran |
…7 as default (apache#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
…rm with JDK-17 as default (apache#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
…7 as default (apache#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
…ch (#8105) * HADOOP-19495. Add JDK 24 to Ubuntu 20.04 docker development images. * HADOOP-19540. [JDK17] Add ubuntu:noble as a build platform with JDK-17 as default (#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24` * HADOOP-19570. Upgrade libxxhash to 0.8.3 in Windows 10 (#7689) * The current version of libxxhash - 0.8.1 isn't available on the msys repo. * This is causing the Hadoop Jenkins CI for Windows to fail while building the docker image for Windows 10. * This PR upgrades libxxhash to 0.8.3 to fix this issue. * HADOOP-19636. [JDK17] Remove CentOS 7 Support and Clean Up Dockerfile. (#7822) * HADOOP-19636. [JDK17] Remove CentOS 7 Support and Clean Up Dockerfile. Co-authored-by: Cheng Pan <chengpan@apache.org> Co-authored-by: Istvan Toth <stoty@apache.org> Reviewed-by: Xiaoqiao He <hexiaoqiao@apache.org> Reviewed-by: Steve Loughran <stevel@apache.org> Reviewed-by: Chris Nauroth <cnauroth@apache.org> Reviewed-by: Cheng Pan <chengpan@apache.org> Reviewed-by: Istvan Toth <stoty@apache.org> Signed-off-by: Shilun Fan <slfan1989@apache.org> * HADOOP-19661. Migrate CentOS 8 to Rocky Linux 8 in build env Dockerfile Closes #7900 Signed-off-by: Shilun Fan <slfan1989@apache.org> Signed-off-by: Chris Nauroth <cnauroth@apache.org> * HADOOP-19661. (Follow-up) Rename Dockerfile_centos_8 to Dockerfile_rockylinux_8 (#7917) HADOOP-19661. (Follow-up) Rename Dockerfile_centos_8 to Dockerfile_rockylinux_8. Signed-off-by: Shilun Fan <slfan1989@apache.org> * HADOOP-19659. Upgrade Debian 10 to 11 in build env Dockerfile (#7898) Contributed by Cheng Pan. * HADOOP-19659. Upgrade Debian 10 to 11 in build env Dockerfile Signed-off-by: Shilun Fan <slfan1989@apache.org> * HADOOP-19661. (Follow-up) Install missing deps for Rocky Linux 8 docker image (#7931) Contributed by Cheng Pan. * HADOOP-19661. (Follow-up) Install missing deps for Rocky Linux 8 docker image Reviewed-by: Shilun Fan <slfan1989@apache.org> Signed-off-by: Shilun Fan <slfan1989@apache.org> * HADOOP-19659. (Follow-up) Install missing procps for Debian 11 docker image (#7932) Contributed by Cheng Pan. * HADOOP-19659. (Follow-up) Install missing procps for Debian 11 docker image. Signed-off-by: Shilun Fan <slfan1989@apache.org> --------- Signed-off-by: Shilun Fan <slfan1989@apache.org> Signed-off-by: Chris Nauroth <cnauroth@apache.org> Co-authored-by: Istvan Toth <stoty@apache.org> Co-authored-by: Vinayakumar B <vinayakumarb@apache.org> Co-authored-by: Gautham B A <gautham.bangalore@gmail.com> Co-authored-by: Cheng Pan <chengpan@apache.org>
…7 as default (#7608) 1. Added "ubuntu:noble" to supported docker platforms. 2. Modified `start-build-env.sh` to use expected Dockerfiles dynamically. ex: `bash start-build-env.sh ubuntu_24` where `ubuntu_24` is the extra suffix in the name of the Dockerfile. 3. Added `jdk17++` profile activated when jdk17 is available. Successfully built the entire Hadoop tar with native support on env created using `bash start-build-env.sh ubuntu_24`
|
no, this shouldn't have been backported to branch-3.R as it is breaking releases there. |
|
update, I'm confused, tried to build on the wrong branch. sorry. I'd actually kicked off a build of trunk in an arm64 ec2 host...if it failed its because the docker image wasn't java17 everywhere... |
Description of PR
start-build-env.shto use expected Dockerfiles dynamically. ex:bash start-build-env.sh ubuntu_24whereubuntu_24is the extra suffix in the name of the Dockerfile.jdk17profile activated when jdk17 is available.#include <stdint.h>to header files which misses it to successfully compile with g++ (13.3) in ubuntu 24.How was this patch tested?
Successfully built the entire Hadoop tar with native support on env created using
bash start-build-env.sh ubuntu_24For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?