Skip to content

Resolve GitHub Actions deprecation warnings [skip ci] - #16076

Merged
pxLi merged 1 commit into
NVIDIA:mainfrom
pxLi:fix-actions-deprecations
Sep 22, 2026
Merged

pxLi merged 1 commit into
NVIDIA:mainfrom
pxLi:fix-actions-deprecations

Conversation

@pxLi

@pxLi pxLi commented Sep 22, 2026 •

Copy link
Copy Markdown
Member

Description

Resolve GitHub Actions deprecation warnings without changing CI coverage:

  • Upgrade the withResource report upload from actions/upload-artifact@v4 to the Node 24-based v7, preserving the artifact name, path, and failure-handling settings.
  • Replace the six deprecated AdoptOpenJDK distribution settings with Temurin while preserving every Java version and build matrix.

No workflow triggers, audit rules, product code, or shared actions are changed. This is a CI-only dependency update with no change to Spark execution or performance-sensitive code, so runtime performance testing is not required.

Validation:

  • YAML parsing and structural comparison passed; only the intended dependency settings changed.
  • git diff --check passed.
  • Verified the selected action runtime is Node 24. Live Maven and Blossom workflow runs remain to be validated in CI.

Checklists

Documentation

  • Updated for new or modified user-facing features or behaviors
  • No user-facing change

Testing

  • Added or modified tests to cover new code paths
  • Covered by existing tests
    (Existing mvn[compile,RAT,scalastyle,docgen] and Blossom-CI workflows; execution pending in CI.)
  • Not required

Performance

  • Tests ran and results are added in the PR description
  • Issue filed with a link in the PR description
  • Not required

Deprecation References

Signed-off-by: Peixin Li <pxLi@nyu.edu>
@pxLi
pxLi requested a review from a team as a code owner September 22, 2026 03:57
@pxLi pxLi added the build Related to CI / CD or cleanly building label Sep 22, 2026
@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the dependency substitutions preserve the existing CI matrix and run on compatible runners.

Summary

This PR updates CI dependencies without changing workflow coverage or product behavior.

  • Replaces deprecated AdoptOpenJDK distribution selections with Temurin while retaining Java versions and matrices.
  • Upgrades the withResource audit report upload from actions/upload-artifact@v4 to @v7.
  • Preserves workflow triggers, artifact settings, and failure handling.

Reviews (1) · Last reviewed commit: "Resolve GitHub Actions deprecation warni..."

@pxLi pxLi changed the title Resolve GitHub Actions deprecation warnings [reduced-it] Resolve GitHub Actions deprecation warnings [skip ci] Sep 22, 2026
@pxLi pxLi self-assigned this Sep 22, 2026
@pxLi

pxLi commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

build

@pxLi
pxLi merged commit 74e52b7 into NVIDIA:main Sep 22, 2026
63 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Related to CI / CD or cleanly building

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants