Skip to content

CI: Add Blossom init GHA workflow#1227

Merged
dpressle merged 2 commits intoopenucx:masterfrom
Alexey-Rivkin:blossom_init
Nov 30, 2025
Merged

CI: Add Blossom init GHA workflow#1227
dpressle merged 2 commits intoopenucx:masterfrom
Alexey-Rivkin:blossom_init

Conversation

@Alexey-Rivkin
Copy link
Collaborator

Signed-off-by: Alexey Rivkin <arivkin@nvidia.com>
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Nov 25, 2025

Greptile Overview

Greptile Summary

Added GitHub Actions workflow to integrate Blossom CI pipeline, triggered by /build comments on pull requests. The workflow includes authorization checks, vulnerability scanning, and job triggering on hybrid infrastructure (GitHub + self-hosted runners).

  • Workflow intentionally runs only on /build comments, not on standard PR events
  • Uses internal NVIDIA Blossom templates with required actions/checkout@v2 version
  • Depends on secrets BLOSSOM_KEY and CI_SERVER being configured
  • The Upload-Log job runs independently on workflow_dispatch events for Jenkins log posting

Confidence Score: 4/5

  • Safe to merge with minor caveat about env.args output mechanism
  • The workflow follows NVIDIA's internal Blossom CI integration template and is intentionally designed for comment-triggered builds. Previously raised concerns about trigger conditions and deprecated actions were confirmed as intentional design choices by the development team. The only unresolved concern is whether the blossom-ci script properly sets env.args for job outputs, but this is likely handled by the Blossom tooling.
  • No files require special attention - this is a straightforward CI integration following internal templates

Important Files Changed

File Analysis

Filename Score Overview
.github/workflows/blossom-ci.yml 4/5 Adds Blossom CI workflow for /build comment triggers; previously reported issues acknowledged by developers as intentional design

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Signed-off-by: Alexey Rivkin <arivkin@nvidia.com>
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@dpressle dpressle merged commit 2d81494 into openucx:master Nov 30, 2025
12 checks passed
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.

2 participants