Skip to content

[fix][ci] dont build wheel for cpp tests #5443

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 1 commit into from
Jun 25, 2025
Merged

[fix][ci] dont build wheel for cpp tests #5443

merged 1 commit into from
Jun 25, 2025

Conversation

omera-nv
Copy link
Collaborator

[fix] Skip build wheel for CPP tests

The wheel building portion of build_wheel.py can take several minutes, and is not needed for CPP testing.

@omera-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9725 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9725 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #7160 completed with status: 'FAILURE'

@omera-nv omera-nv force-pushed the fix/skip_build_wheel_in_cpp_tests branch from 020f104 to bb31a9e Compare June 24, 2025 15:30
@omera-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9728 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9728 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #7162 completed with status: 'FAILURE'

@omera-nv omera-nv force-pushed the fix/skip_build_wheel_in_cpp_tests branch from bb31a9e to 88a258b Compare June 25, 2025 02:51
@omera-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9796 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

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

@omera-nv
do not build wheel for cpp tests
7f69ff0 
Signed-off-by: Omer Ullman Argov <[email protected]>
@omera-nv omera-nv force-pushed the fix/skip_build_wheel_in_cpp_tests branch from 88a258b to 7f69ff0 Compare June 25, 2025 06:00
@omera-nv
Copy link
Collaborator Author

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9815 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #9815 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #7242 completed with status: 'SUCCESS'
Pipeline passed with automatic retried tests. Check the rerun report for details.

@omera-nv omera-nv enabled auto-merge (squash) June 25, 2025 20:42
@omera-nv omera-nv requested a review from DomBrown June 25, 2025 20:42
Funatiq
Funatiq reviewed Jun 25, 2025
View reviewed changes
@omera-nv omera-nv merged commit bdc8dfe into NVIDIA:main Jun 25, 2025
3 checks passed
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 9, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
2473add 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
300d431 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
0280894 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
3e6179f 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 10, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
d64d10d 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 11, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
982353f 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 11, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
0b45475 
Signed-off-by: Omer Ullman Argov <[email protected]>
dominicshanshan pushed a commit to dominicshanshan/TensorRT-LLM that referenced this pull request Jul 11, 2025
@omera-nv @dominicshanshan
[fix][ci] dont build wheel for cpp tests (NVIDIA#5443)
cf29467 
Signed-off-by: Omer Ullman Argov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Funatiq Funatiq Funatiq approved these changes

@DomBrown DomBrown Awaiting requested review from DomBrown

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
@omera-nv @tensorrt-cicd @Funatiq