Skip to content

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

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>
@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 intended Java distribution migration without changing the Blossom workflow’s execution flow or Java version.

Summary

Updates the Blossom vulnerability-scan workflow to replace the deprecated AdoptOpenJDK distribution identifier with Temurin while retaining actions/setup-java@v5 and Java 8. It also advances the workflow’s NVIDIA copyright year to 2026.

  • The scan and downstream Jenkins-trigger flow remain structurally unchanged.
  • No security issues or repository-rule violations were identified.

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

@pxLi pxLi self-assigned this Sep 22, 2026
@pxLi pxLi added the build label Sep 22, 2026
@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 requested review from YanxuanLiu and yinqingh September 22, 2026 04:29
@pxLi

pxLi commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

build

@pxLi
pxLi merged commit d4258b7 into NVIDIA:main Sep 22, 2026
5 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.

2 participants