Add packaged Java tests to nightly tests - #23732
Conversation
* Move the java-build artifact download into ci/test_packaged_java.sh so PR and nightly runs call the same script. * Download cudf_java_$(arch)_cu$(cuda_major) via rapids-download-from-github. * Replace the inline pr.yaml java-tests job with shared-workflows/custom-job.yaml. * Add java-build-matrix and java-tests to test.yaml.
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
/ok to test fa976dd |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesJava package testing
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This PR aligns packaged Java testing across pull-request and nightly workflows; no actionable merge-blocking risk remains, so it is merge-ready after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
bdice
left a comment
There was a problem hiding this comment.
I discussed this with @paul-aiyedun over a call and I think this makes sense.
|
/merge |
* Move the `java-build` artifact download into `ci/test_packaged_java.sh` so PR and nightly runs call the same script. * Download `cudf_java_$(arch)_cu$(cuda_major)` via `rapids-download-from-github`. * Replace the inline `pr.yaml` `java-tests` job with `shared-workflows/custom-job.yaml`. * Add` java-build-matrix` and `java-tests` to `test.yaml`. Authors: - https://github.com/paul-aiyedun Approvers: - Bradley Dice (https://github.com/bdice) URL: NVIDIA#23732 (cherry picked from commit 30fa357)
## Description * Rebuild classifier JARs on RAPIDS `ci-wheel` (Rocky 8) so the bundled JNI library links against a libstdc++ ABI that Ubuntu 22.04 and Rocky Linux 8 can satisfy. * Skip optional `nvcomp` in `NativeDepsLoader` when its native resource is absent. * Run the existing Java suite against the packaged classifier JAR in PR and nightly CI so tests cover the published artifact. * Add a tag-gated `java-publish` job that uploads the gathered Maven repo to Maven Central. ## Checklist - [x] I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/cudf/blob/HEAD/CONTRIBUTING.md). - [x] New or existing tests cover these changes. - [x] The documentation is up to date with these changes.
Description
Move the
java-buildartifact download intoci/test_packaged_java.shso PR and nightly runs call the same script.Download
cudf_java_$(arch)_cu$(cuda_major)viarapids-download-from-github.Replace the inline
pr.yamljava-testsjob withshared-workflows/custom-job.yaml.Add
java-build-matrixandjava-teststotest.yaml.Checklist