Skip to content

TEZ-4709: Fix Protobuf linter (buf) failures by upgrading in build environment - #486

Merged
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4709
Apr 28, 2026
Merged

TEZ-4709: Fix Protobuf linter (buf) failures by upgrading in build environment#486
abstractdog merged 1 commit into
apache:masterfrom
Aggarwal-Raghav:TEZ-4709

Conversation

@Aggarwal-Raghav

@Aggarwal-RaghavAggarwal-Raghav commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

Older version (0.21.0): It was buf check
Screenshot 2026-04-18 at 11 43 17 PM

Newer Version (1.68.2): It's buf lint
Screenshot 2026-04-18 at 11 43 31 PM

@Aggarwal-Raghav
Aggarwal-Raghav marked this pull request as ready for review April 18, 2026 18:24
@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-486/1/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗detsecrets0m 1sdetect-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 6sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 1sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
_ Other Tests _
+1 💚asflicense0m 26sThe patch does not generate ASF License warnings.
1m 27s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/1/artifact/out/Dockerfile
GITHUB PR#486
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs
unameLinux 88ffe4c035d8 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-486/src/.yetus/personality.sh
git revisionmaster / 8f721b7
Max. process+thread count57 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/1/console
versionsgit=2.43.0 maven=3.9.14 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-486/2/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 15sDocker 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.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep0m 32sMaven dependency ordering for branch
-1 ❌mvninstall4m 17s/branch-mvninstall-root.txtroot in master failed.
-1 ❌bufcompat0m 2s/branch-bufcompat-stderr.txtError running buf. Please check buf stderr files.
_ Patch Compile Tests _
+0 🆗mvndep0m 9sMaven dependency ordering for patch
+1 💚mvninstall4m 12sthe patch passed
+1 💚codespell1m 23sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚buflint0m 7sNo new issues.
+1 💚hadolint0m 1sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
_ Other Tests _
+1 💚unit5m 58stez-runtime-library in the patch passed.
+1 💚unit72m 26sroot in the patch passed.
+1 💚asflicense1m 24sThe patch does not generate ASF License warnings.
92m 14s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/2/artifact/out/Dockerfile
GITHUB PR#486
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs cc unit buflint bufcompat
unameLinux 6fa57d7d06c9 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-486/src/.yetus/personality.sh
git revisionmaster / 6ad9eba
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/2/testReport/
Max. process+thread count2124 (vs. ulimit of 5500)
modulesC: tez-runtime-library . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/2/console
versionsgit=2.43.0 maven=3.9.14 hadolint=1.18.0-0-g76eee5c buf=1.68.2 codespell=2.4.1 shellcheck=0.7.1
Powered byApache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

@Aggarwal-Raghav

Aggarwal-Raghav commented Apr 18, 2026

Copy link
Copy Markdown
ContributorAuthor

It is running buf, will revert the trigger commit
Screenshot 2026-04-19 at 1 33 44 AM

@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-486/3/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 16sDocker 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 7sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 1sNo new issues.
+1 💚shellcheck0m 0sNo new issues.
_ Other Tests _
+1 💚asflicense0m 24sThe patch does not generate ASF License warnings.
1m 16s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/3/artifact/out/Dockerfile
GITHUB PR#486
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs
unameLinux 8e6dbdf936ee 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-486/src/.yetus/personality.sh
git revisionmaster / 8f721b7
Max. process+thread count55 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/3/console
versionsgit=2.43.0 maven=3.9.14 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-486/4/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 0sDocker mode activated.
-1 ❌docker8m 39sDocker failed to build run-specific yetus/tez:tp-9001}.
SubsystemReport/Notes
GITHUB PR#486
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/4/console
versionsgit=2.34.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-486/5/console in case of problems.

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 47sDocker 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 4sNo new issues.
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚hadolint0m 2sNo new issues.
+1 💚shellcheck0m 1sNo new issues.
_ Other Tests _
+1 💚asflicense0m 20sThe patch does not generate ASF License warnings.
2m 33s
SubsystemReport/Notes
DockerClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/5/artifact/out/Dockerfile
GITHUB PR#486
Optional Testsdupname asflicense codespell detsecrets hadolint shellcheck shelldocs
unameLinux f791d026ee0f 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personality/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-486/src/.yetus/personality.sh
git revisionmaster / 16bc6b3
Max. process+thread count58 (vs. ulimit of 5500)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-486/5/console
versionsgit=2.43.0 maven=3.9.14 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 April 28, 2026 19:18

@abstractdogabstractdog left a comment

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.

+1
I guess we haven't changed proto files recently, hence we haven't faced this problem

@abstractdog
abstractdog merged commit 57c7491 into apache:masterApr 28, 2026
4 checks passed
@Aggarwal-Raghav
Aggarwal-Raghav deleted the TEZ-4709 branch May 1, 2026 05:52
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