Skip to content

test: Unwaive test for nvbug_5150466 #3552

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 2 commits into from
Apr 18, 2025
Merged

test: Unwaive test for nvbug_5150466 #3552

merged 2 commits into from
Apr 18, 2025

Conversation

hchings
Copy link
Collaborator

@hchings hchings commented Apr 15, 2025

Using this MR to verify whether a flaky test is no longer an issue, since I cannot repro the issue locally after 50+ continuous runs and there had been a related fix.

@hchings hchings requested a review from Superjomn April 15, 2025 00:38
@hchings hchings self-assigned this Apr 15, 2025
@hchings
Copy link
Collaborator Author

hchings commented Apr 15, 2025

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2233 [ run ] triggered by Bot

@juney-nvidia juney-nvidia changed the title Unwaive test for nvbug_5150466 test: Unwaive test for nvbug_5150466 Apr 15, 2025
@tensorrt-cicd
Copy link
Collaborator

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

@hchings
Copy link
Collaborator Author

hchings commented Apr 15, 2025

/bot run

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2283 [ run ] triggered by Bot

@Superjomn
Copy link
Collaborator

/bot run --add-multi-gpu-test --disable-fail-fast

Superjomn
Superjomn approved these changes Apr 15, 2025
View reviewed changes
Copy link
Collaborator

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

But let's try to run the CI at least twice to make sure there are no flaky issues.

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2295 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2283 [ run ] completed with state ABORTED

@tensorrt-cicd
Copy link
Collaborator

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

@hchings
Copy link
Collaborator Author

hchings commented Apr 15, 2025

/bot run --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2354 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

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

@hchings
Copy link
Collaborator Author

hchings commented Apr 16, 2025

/bot run --only-multi-gpu-test

@hchings
Copy link
Collaborator Author

hchings commented Apr 16, 2025

/bot kill

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2486 [ kill ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2486 [ kill ] completed with state SUCCESS
Successfully killed previous jobs for commit 8247895

@hchings
Copy link
Collaborator Author

hchings commented Apr 16, 2025

/bot run --add-multi-gpu-test

@hchings
Copy link
Collaborator Author

hchings commented Apr 17, 2025

/bot kill

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2680 [ kill ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2680 [ kill ] completed with state SUCCESS
Successfully killed previous jobs for commit 4560609

@hchings
unwaive
9c691b4 
Signed-off-by: Erin Ho <[email protected]>
@hchings
Copy link
Collaborator Author

hchings commented Apr 17, 2025

/bot run --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2682 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

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

@hchings hchings changed the base branch from main to release/0.19 April 18, 2025 06:44
@hchings hchings requested a review from a team as a code owner April 18, 2025 06:44
@hchings hchings changed the base branch from release/0.19 to main April 18, 2025 06:44
@Superjomn
Copy link
Collaborator

/bot reuse-pipeline

@Superjomn Superjomn enabled auto-merge (squash) April 18, 2025 07:00
@tensorrt-cicd
Copy link
Collaborator

PR_Github #2738 [ reuse-pipeline ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #2738 [ reuse-pipeline ] completed with state SUCCESS
Reusing PR_Github #2682 for commit edaa1d3

@Superjomn Superjomn merged commit 4fedf0b into NVIDIA:main Apr 18, 2025
3 checks passed
@hchings hchings deleted the nvbug_5150466 branch April 18, 2025 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Superjomn Superjomn Superjomn approved these changes

Assignees

@hchings hchings

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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