Skip to content

[None][infra] Update AutoDeploy CODEOWNERS coverage - #12013

Merged
lucaslie merged 2 commits into
NVIDIA:mainfrom
nv-auto-deploy:ll/codeowners-examples-autodeploy
Mar 9, 2026
Merged

[None][infra] Update AutoDeploy CODEOWNERS coverage#12013
lucaslie merged 2 commits into
NVIDIA:mainfrom
nv-auto-deploy:ll/codeowners-examples-autodeploy

Conversation

@lucaslie

@lucaslie lucaslie commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove @NVIDIA/trt-llm-doc-owners from examples/auto_deploy — AutoDeploy examples should only require AutoDeploy dev review, consistent with other AutoDeploy paths
  • Add docs/source/features/auto_deploy with @NVIDIA/trt-llm-torch-autodeploy-devs + @NVIDIA/trt-llm-doc-owners
  • Add tests/unittest/auto_deploy (new test location, forgot to modify this during the AD test refactor) with @NVIDIA/trt-llm-torch-autodeploy-devs
  • Add tests/integration/defs/accuracy/test_llm_api_autodeploy.py with @NVIDIA/trt-llm-torch-autodeploy-devs + @NVIDIA/trt-llm-qa-function
  • Remove stale tests/unittest/_torch/auto_deploy entry (tests have been moved to tests/unittest/auto_deploy)

Test plan

  • No functional changes; CODEOWNERS file update only

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

No user-facing changes

This release includes internal administrative updates to code ownership and test path configurations. These changes do not impact functionality or user experience.

lucaslie added 2 commits March 8, 2026 15:01
AutoDeploy examples should only require AutoDeploy dev review,
not doc-owners, consistent with other AutoDeploy paths.

Signed-off-by: Lucas Liebenwein <11156568+lucaslie@users.noreply.github.com>
- Add docs/source/features/auto_deploy (autodeploy-devs + doc-owners)
- Add tests/unittest/auto_deploy (new test location)
- Add tests/integration/defs/accuracy/test_llm_api_autodeploy.py
- Remove stale tests/unittest/_torch/auto_deploy (tests moved)

Signed-off-by: Lucas Liebenwein <11156568+lucaslie@users.noreply.github.com>
@lucaslie
lucaslie requested a review from a team March 8, 2026 23:18
@lucaslie
lucaslie requested review from a team as code owners March 8, 2026 23:18
@lucaslie
lucaslie requested review from dpitman-nvda and niukuo March 8, 2026 23:18
@lucaslie

lucaslie commented Mar 8, 2026

Copy link
Copy Markdown
Contributor Author

/bot skip --comment "CODEOWNERS-only change, no functional code modified"

@coderabbitai

coderabbitai Bot commented Mar 8, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Update CODEOWNERS configuration to expand ownership coverage for TensorRT-LLM's AutoDeploy feature by adding documentation and integration test paths, while adjusting unittest directory references.

Changes

Cohort / File(s) Summary
CODEOWNERS Configuration
.github/CODEOWNERS
Rebalanced ownership assignments for AutoDeploy feature across examples, documentation, and test paths. Removed /tests/unittest/_torch/auto_deploy entry and added /docs/source/features/auto_deploy, /tests/unittest/auto_deploy, and /tests/integration/defs/accuracy/test_llm_api_autodeploy.py entries with appropriate team assignments.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately summarizes the main change: updating CODEOWNERS coverage for the AutoDeploy feature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed PR description clearly explains all changes with specific CODEOWNERS modifications, rationale for each change, and explicitly states there are no functional changes.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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

@lucaslie
lucaslie enabled auto-merge (squash) March 8, 2026 23:22
@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #38156 [ skip ] triggered by Bot. Commit: 8c9ad58 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #38156 [ skip ] completed with state SUCCESS. Commit: 8c9ad58
Skipping testing for commit 8c9ad58

Link to invocation

@lucaslie
lucaslie merged commit 6ec0aad into NVIDIA:main Mar 9, 2026
10 checks passed
tianyuz-nv pushed a commit to wanqian-nv/TensorRT-LLM that referenced this pull request Mar 19, 2026
Signed-off-by: Lucas Liebenwein <11156568+lucaslie@users.noreply.github.com>
limin2021 pushed a commit to limin2021/TensorRT-LLM that referenced this pull request Mar 19, 2026
Signed-off-by: Lucas Liebenwein <11156568+lucaslie@users.noreply.github.com>
longcheng-nv pushed a commit to longcheng-nv/TensorRT-LLM that referenced this pull request Mar 31, 2026
Signed-off-by: Lucas Liebenwein <11156568+lucaslie@users.noreply.github.com>
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