Skip to content

Use Temurin for the Blossom Java setup [skip ci] - #5177

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

Replace the deprecated AdoptOpenJDK distribution with Temurin in the Blossom vulnerability-scan job. Keep actions/setup-java@v5, Java 8, and all other workflow settings unchanged.

Validation

  • YAML parsing and structural comparison passed: only the Java distribution changes.
  • git diff --check passed.
  • The self-hosted Blossom job was not run locally.

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:51
@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 it makes the supported distribution migration without altering the vulnerability-scan flow or Java version.

Summary

The PR updates the Blossom vulnerability-scan workflow to use Eclipse Temurin instead of the deprecated AdoptOpenJDK distribution while retaining actions/setup-java@v5 and Java 8.

  • Preserves the Java version and remaining scan configuration.
  • Updates the copyright notice to the current NVIDIA wording.
  • No actionable correctness, security, or maintainability issues were identified.

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

@pxLi pxLi changed the title Use Temurin for the Blossom Java setup Use Temurin for the Blossom Java setup [skip ci] Sep 22, 2026
@pxLi pxLi self-assigned this Sep 22, 2026
@pxLi pxLi added the build label Sep 22, 2026
@pxLi

pxLi commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

build

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants