Skip to content

TEZ-4674: Update Dockerfile dependencies: Protobuf 3.25.5, and stable Python, npm tools - #452

Merged
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4674
Jan 19, 2026
Merged

TEZ-4674: Update Dockerfile dependencies: Protobuf 3.25.5, and stable Python, npm tools#452
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4674

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Jan 18, 2026

Copy link
Copy Markdown
Contributor
  1. pylint is not running because of the missing dependency of imp which has been marked deprecated since python-3.4
  2. protobuf is running with older version 3.21. we need to move to 3.25.5 as same as pom.xml

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/1/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec5m 3sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚codespell0m 5sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 0sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
_ Other Tests _
+1 💚asflicense0m 45sThe patch does not generate ASF License warnings.
6m 42s
SubsystemReport/Notes
DockerClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/1/artifact/out/Dockerfile
GITHUB PR#452
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs
unameLinux 35ef9d125373 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-452/src/.yetus/personality.sh
git revisionmaster / 408f016
Max. process+thread count53 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/1/console
versionsgit=2.43.0 maven=3.8.7 hadolint=1.18.0-0-g76eee5c codespell=2.4.1 shellcheck=0.7.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@tez-yetus

Copy link
Copy Markdown

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/2/console in case of problems.

&& make install
WORKDIR /root
RUN rm -rf /opt/protobuf-src
https://github.com/protocolbuffers/protobuf/releases/download/v25.5/protoc-25.5-linux-x86_64.zip \

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

  1. Protobuf removed Autotools (./configure and make). Wither we use CMake or Bazel to build from source or directly download the protoc which is also available on release page.
  2. URL & Versioning: The version numbering scheme changed. What was previously v3.21.x is now v25.x. The URL pattern and tarball names have changed accordingly.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

change looks good, compiling protobuf from source is an overkill if there is a binary available

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 43sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚codespell0m 5sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 0sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
_ Other Tests _
+1 💚asflicense0m 45sThe patch does not generate ASF License warnings.
3m 19s
SubsystemReport/Notes
DockerClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/2/artifact/out/Dockerfile
GITHUB PR#452
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs
unameLinux 8b83eb15d386 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-452/src/.yetus/personality.sh
git revisionmaster / 2a86d50
Max. process+thread count53 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-452/2/console
versionsgit=2.43.0 maven=3.8.7 hadolint=1.18.0-0-g76eee5c codespell=2.4.1 shellcheck=0.7.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@abstractdog
abstractdog self-requested a review January 19, 2026 08:10
@abstractdog

Copy link
Copy Markdown
Contributor

change looks good to me, good improvement to the Dockerfile, +1
pending github actions

@abstractdog
abstractdog merged commit faa9bb9 into apache:masterJan 19, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4674 branch January 24, 2026 20:26
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Aggarwal-Raghav@tez-yetus@abstractdog