Skip to content

Conversation

@cnauroth
Copy link
Contributor

Description of PR

The full release docs have Python dependencies on pylint and dateutil. These packages are not currently included in the Ubuntu 24 images.

mvn install site site:stage -DskipTests -DskipShade -Pdist,src -Preleasedocs,docs

[INFO] --- exec:1.3.1:exec (releasedocs) @ hadoop-common ---
ERROR:root:This script requires python-dateutil module to be installed. You can install it using:
	 pip install python-dateutil

Older images included these via dev-support/docker/pkg-resolver/install-common-pkgs.sh, which uses pip. This won't work for Ubuntu 24, which enforces that Python packages are managed with apt. We can simply include them in the apt commands.

How was this patch tested?

I successfully built full release docs in both the x86 and ARM images.

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

AI Tooling

If an AI tool was used:

@hadoop-yetus
Copy link

(!) 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/hadoop-multibranch/job/PR-8223/1/console in case of problems.

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 1m 1s 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 🆗 shelldocs 0m 1s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+0 🆗 mvndep 8m 42s Maven dependency ordering for branch
+1 💚 shadedclient 26m 55s branch has no errors when building and testing our client artifacts.
-0 ⚠️ patch 27m 17s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 hadolint 0m 2s No new issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shadedclient 26m 11s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 34s The patch does not generate ASF License warnings.
66m 39s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8223/1/artifact/out/Dockerfile
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname Linux a97d58c26208 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 tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 90150af
Max. process+thread count 611 (vs. ulimit of 5500)
modules C: U:
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8223/1/console
versions git=2.25.1 maven=3.9.11 hadolint=1.11.1-0-g0e692dd shellcheck=0.7.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@cnauroth cnauroth closed this in 0765e27 Feb 2, 2026
@cnauroth
Copy link
Contributor Author

cnauroth commented Feb 2, 2026

@aajisaka and @slfan1989 , thank you for reviewing. I have committed this to trunk.

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.

4 participants