Skip to content

[TRTLLMINF-188][test] Validate merged post-merge approval workflow#16575

Closed
chzblych wants to merge 2 commits into
NVIDIA:mainfrom
chzblych:trtllminf-188-upstream-guard-e2e-20260719
Closed

[TRTLLMINF-188][test] Validate merged post-merge approval workflow#16575
chzblych wants to merge 2 commits into
NVIDIA:mainfrom
chzblych:trtllminf-188-upstream-guard-e2e-20260719

Conversation

@chzblych

Copy link
Copy Markdown
Collaborator

Description

Ephemeral draft PR used only to verify the merged default-branch post-merge
approval-label workflow.

The only change is a non-executable marker file. This PR must not be merged.
No /bot command, Blossom workflow, Jenkins job, or downstream CI workload
will be triggered.

Related implementation:
NVIDIA/TensorRT-LLM#16318.

Test Coverage

  • Apply ci: post-merge approved as an active
    NVIDIA/trt-llm-ci-approvers member.
  • Verify the default-branch Guard succeeds, retains the label, and creates no
    denial comment.
  • Change only Revision: 1 to Revision: 2 in a second signed-off commit.
  • Verify the label remains and no synchronize Guard run is created.
  • Remove the label and close this draft PR without merging.

Negative-actor and cancellation behavior was already validated in an isolated
personal fork and will not be repeated against upstream.

PR Checklist

  • This is an isolated, marker-only workflow test and must not be merged.

Signed-off-by: Yanchao Lu <yanchaol@nvidia.com>
@chzblych chzblych added the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026 — with ChatGPT Codex Connector
Signed-off-by: Yanchao Lu <yanchaol@nvidia.com>
@chzblych chzblych removed the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026
@chzblych chzblych closed this Jul 19, 2026
@trtllm-agent trtllm-agent added the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026
@github-actions github-actions Bot removed the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026
@github-actions

Copy link
Copy Markdown

Removed the "ci: post-merge approved" label because @trtllm-agent could not be verified as an active member of NVIDIA/trt-llm-ci-approvers. Ask a member of that team to apply it.

@chzblych chzblych reopened this Jul 19, 2026
@chzblych

Copy link
Copy Markdown
Collaborator Author

/bot run --post-merge

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60187 [ run ] triggered by Bot. Commit: 8ab2045 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60187 Bot args parsing error: Post-merge CI requested by --post-merge or a broad stage selector ('*' or 'Post-Merge') requires the ci: post-merge approved PR label. Ask a member of NVIDIA/trt-llm-ci-approvers to apply it.

Link to invocation

@chzblych chzblych added the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026
@chzblych

Copy link
Copy Markdown
Collaborator Author

/bot run --post-merge

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60188 [ run ] triggered by Bot. Commit: 8ab2045 Link to invocation

@chzblych

Copy link
Copy Markdown
Collaborator Author

/bot kill

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60189 [ kill ] triggered by Bot. Commit: 8ab2045 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60188 [ run ] completed with state ABORTED. Commit: 8ab2045

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #60189 [ kill ] completed with state SUCCESS. Commit: 8ab2045
Successfully killed previous jobs for commit 8ab2045

Link to invocation

@chzblych chzblych removed the ci: post-merge approved Approved by TRT-LLM CI approvers for broad post-merge CI requests label Jul 19, 2026
@chzblych chzblych closed this Jul 19, 2026
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