[ishovkun]

Dev Engineer Review

• .github/workflows/blossom-ci.yml includes skasetty-Siri in the authorized-user allowlist.
• No test results or review findings were provided.

QA Engineer Review

No test changes.

Per-File QA Perspective

• .github/workflows/blossom-ci.yml: The allowlist update permits skasetty-Siri to issue authorized Blossom CI commands. Verify that the intended account can issue those commands and that existing authorization checks remain unchanged.

Description

Add skasetty-Siri to the Blossom CI authorized users so Sirisha Kasetty can issue authorized CI commands, following the existing allowlist process in #20025.

Test Coverage

• pre-commit run --files .github/workflows/blossom-ci.yml (passed, including YAML validation)
• git diff --check (passed)

PR Checklist

• Reviewed the checklist for this allowlist-only change. No API, dependency, ownership, documentation, or architecture changes are involved.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: NVIDIA/TensorRT-LLM
• Branch: main
• File: .coderabbit.yaml
• Commit: 1f9b707
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 2240d216-f6a4-4a87-b5ee-0f39a21bf8cc

Commits

Reviewing files that changed from the base of the PR and between 1f9b707 and 2182d74.

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 Authorization job’s GitHub actor allowlist now includes skasetty-Siri. The existing comment-command and actor checks are unchanged.

Changes

Authorization allowlist

Layer / File(s)	Summary
Add actor to allowlist .github/workflows/blossom-ci.yml	The Authorization job’s allowlist now includes skasetty-Siri. Existing checks are unchanged.

Priority: Low

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

Change: Other

Merge Risk: Minimal · up to 2182d

The change only adds the requested authorized actor and introduces no verified merge-blocking risk.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly identifies the infrastructure change and names the authorized GitHub user being added.
Description check		The description explains the change, its purpose, test coverage, and checklist impact. It is complete for this allowlist-only update.
Docstring Coverage		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		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

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.}