Skip to content

[None][infra] Add blossom-ci authorized users - #21243

Merged
hanjingtian merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:auto/blossom-ci-add-users-20261029T054101
Oct 29, 2026
Merged

hanjingtian merged 1 commit into
NVIDIA:mainfrom
trtllm-agent:auto/blossom-ci-add-users-20261029T054101

Conversation

@trtllm-agent

@trtllm-agent trtllm-agent commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Auto-generated by blossom-ci-onboard-bot from the CI-onboarding sheet.

Adds the following user(s) to the Blossom-CI trigger allowlist:

  • skasetty-Siri
  • yifjiang

Dev Engineer Review

The workflow allowlist adds skasetty-Siri and yifjiang as authorized Blossom-CI users. No other change is shown. Current review findings were not provided.

QA Engineer Review

No test changes.

Per-File QA Perspective

  • .github/workflows/blossom-ci.yml: The allowlist change permits the two added users to trigger the workflow’s authorization path. Verify that each account should have this access.

Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com>
@hanjingtian

Copy link
Copy Markdown
Collaborator

/bot skip --comment "add blossom user"

@github-actions

Copy link
Copy Markdown

Automatically added "ci: full pre-merge approved" because this PR has satisfied the required GitHub review approvals. Unresolved review conversations and other required checks remain independent merge requirements.

@hanjingtian
hanjingtian enabled auto-merge (squash) October 29, 2026 05:47
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 544c45a and 824b4d5.

Files selected for processing: 1
  • .github/workflows/blossom-ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The Blossom-CI workflow adds skasetty-Siri and yifjiang to its authorized-user lists.

Changes

Blossom-CI authorized-user allowlist

Layer / File(s) Summary
Add authorized users
.github/workflows/blossom-ci.yml
The workflow adds skasetty-Siri and yifjiang to authorized-user lists.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: yuanjingx87

Merge Risk: Minimal · up to 824b4

No concrete merge-blocking risk was established for this allowlist-only change.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the infrastructure change and follows the required [None][type] format.
Description check Passed The description clearly states that the onboarding bot added skasetty-Siri and yifjiang to the Blossom-CI trigger allowlist. It does not include test coverage or checklist details, but those secti…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Description check

Explanation

The description clearly states that the onboarding bot added skasetty-Siri and yifjiang to the Blossom-CI trigger allowlist. It does not include test coverage or checklist details, but those sections are non-critical for this allowlist-only change.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #81356 [ skip ] triggered by Bot. Commit: 824b4d5 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #81356 [ skip ] completed with state SUCCESS. Commit: 824b4d5
Skipping testing for commit 824b4d5

Link to invocation

@hanjingtian
hanjingtian merged commit d788792 into NVIDIA:main Oct 29, 2026
19 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.

3 participants