Skip to content

[TRTLLM-5633] - Merge current waive list with the TOT waive list #5198

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Jul 30, 2025
Merged

[TRTLLM-5633] - Merge current waive list with the TOT waive list #5198

merged 3 commits into from
Jul 30, 2025

Conversation

yiqingy0
Copy link
Collaborator

@yiqingy0 yiqingy0 commented Jun 13, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features

    • Added automated merging of waive lists during the pipeline process, with artifacts available for download.
    • Waive lists are now updated by downloading and integrating the latest changes if available.

  • Bug Fixes

    • Improved error handling when updating waive lists, ensuring fallback to default if download fails.

  • Chores

    • Enhanced pipeline structure with clearer preparation and waive list merging stages for improved maintainability.

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #8790 [ run ] triggered by Bot

@yiqingy0 yiqingy0 changed the title Waive list Merge current waive list with the ToT waive list Jun 13, 2025
@tensorrt-cicd
Copy link
Collaborator

PR_Github #8790 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6385 (Partly Tested) completed with status: 'FAILURE'

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #8998 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #8998 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6569 (Partly Tested) completed with status: 'FAILURE'

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9012 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9012 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6582 (Partly Tested) completed with status: 'FAILURE'

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9014 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9014 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6584 (Partly Tested) completed with status: 'FAILURE'

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9020 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9020 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6588 (Partly Tested) completed with status: 'FAILURE'

@yiqingy0
Copy link
Collaborator Author

/bot run --skip-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9143 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9143 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #6696 (Partly Tested) completed with status: 'ABORTED'

@yiqingy0 yiqingy0 force-pushed the waive_list branch 6 times, most recently from b302397 to 93d8d97 Compare June 30, 2025 10:22
@coderabbitai coderabbitai bot mentioned this pull request Jul 28, 2025
Closed
@tensorrt-cicd
Copy link
Collaborator

PR_Github #13158 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #9852 completed with status: 'FAILURE'

This was referenced Jul 28, 2025
Merged
Merged
Merged
Merged
Closed
@yiqingy0
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #13305 [ run ] triggered by Bot

This was referenced Jul 29, 2025
Merged
Merged
@tensorrt-cicd
Copy link
Collaborator

PR_Github #13305 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #9940 completed with status: 'FAILURE'

This was referenced Jul 29, 2025
Merged
Open
Closed
Closed
@yiqingy0
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #13509 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #13509 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #10119 completed with status: 'SUCCESS'

@yiqingy0 yiqingy0 merged commit 0cf2f6f into NVIDIA:main Jul 30, 2025
3 checks passed
This was referenced Jul 31, 2025
Draft
Merged
Closed
Merged
Merged
Merged
lancelly pushed a commit to lancelly/TensorRT-LLM that referenced this pull request Aug 6, 2025
@yiqingy0 @lancelly
[TRTLLM-5633] - Merge current waive list with the TOT waive list (NVI…
2592459 
…DIA#5198)

Signed-off-by: Yiqing Yan <[email protected]>
Signed-off-by: Lanyu Liao <[email protected]>
jain-ria pushed a commit to jain-ria/TensorRT-LLM that referenced this pull request Aug 7, 2025
@yiqingy0
[TRTLLM-5633] - Merge current waive list with the TOT waive list (NVI…
b5279ee 
…DIA#5198)

Signed-off-by: Yiqing Yan <[email protected]>
@coderabbitai coderabbitai bot mentioned this pull request Aug 11, 2025
Merged
@coderabbitai coderabbitai bot mentioned this pull request Aug 27, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@chzblych chzblych chzblych approved these changes

@ZhanruiSunCh ZhanruiSunCh Awaiting requested review from ZhanruiSunCh

@niukuo niukuo Awaiting requested review from niukuo

@EmmaQiaoCh EmmaQiaoCh Awaiting requested review from EmmaQiaoCh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yiqingy0 @tensorrt-cicd @chzblych