Skip to content

[None][infra] Add mingyangHao to blossom-ci allowlist#14132

Merged
ZhanruiSunCh merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:user/zhanruis/add_mingyangHao_to_allowlist
May 15, 2026
Merged

[None][infra] Add mingyangHao to blossom-ci allowlist#14132
ZhanruiSunCh merged 1 commit into
NVIDIA:mainfrom
ZhanruiSunCh:user/zhanruis/add_mingyangHao_to_allowlist

Conversation

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator

@ZhanruiSunCh ZhanruiSunCh commented May 14, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow authorization configuration.

Note: This release contains only internal operational updates with no user-visible changes.

Review Change Stack

Description

Test Coverage

PR Checklist

Please review the following before submitting your PR:

  • PR description clearly explains what and why. If using CodeRabbit's summary, please make sure it makes sense.

  • PR Follows TRT-LLM CODING GUIDELINES to the best of your knowledge.

  • Test cases are provided for new code paths (see test instructions)

  • If PR introduces API changes, an appropriate PR label is added - either api-compatible or api-breaking. For api-breaking, include BREAKING in the PR title.

  • Any new dependencies have been scanned for license and vulnerabilities

  • CODEOWNERS updated if ownership changes

  • Documentation updated as needed

  • Update tava architecture diagram if there is a significant design change in PR.

  • The reviewers assigned automatically/manually are appropriate for the PR.

  • Please check this after reviewing the above items as appropriate for this PR.

GitHub Bot Help

To see a list of available CI bot commands, please comment /bot help.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: ZhanruiSunCh <184402041+ZhanruiSunCh@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 14, 2026

📝 Walkthrough

Walkthrough

This PR adds a single GitHub actor (mingyangHao) to the hard-coded authorized actors allowlist in the Blossom-CI workflow's authorization gate, allowing that actor's workflow runs to proceed.

Changes

Blossom-CI Authorization

Layer / File(s) Summary
Authorization actor allowlist
.github/workflows/blossom-ci.yml
The username mingyangHao is inserted into the fromJson([...]) authorized-actors allowlist used by the Authorization job's conditional check.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is empty except for template boilerplate; the actual problem statement, solution, and test coverage sections are completely unfilled. Fill in the Description section explaining why mingyangHao needs to be added to the allowlist and what authorization this grants. Provide test coverage details and complete the PR checklist appropriately.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding mingyangHao to the blossom-ci allowlist, with appropriate [None][infra] tags.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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

@ZhanruiSunCh
Copy link
Copy Markdown
Collaborator Author

/bot skip --comment "Only add allowlist"

@ZhanruiSunCh ZhanruiSunCh enabled auto-merge (squash) May 15, 2026 04:24
@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48514 [ skip ] triggered by Bot. Commit: ad0616b Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48514 [ skip ] completed with state SUCCESS. Commit: ad0616b
Skipping testing for commit ad0616b

Link to invocation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants