Skip to content

Start running hip-tests on TheRock - #2706

Closed
cjatin wants to merge 1 commit into
mainfrom
users/jatin/enable_hip_tests
Closed

cjatin wants to merge 1 commit into
mainfrom
users/jatin/enable_hip_tests

Conversation

@cjatin

@cjatin cjatin commented Dec 24, 2025

Copy link
Copy Markdown
Contributor

Co author: @agunashe

Motivation

Enable running hip-tests on TheRock CI.

Technical Details

Add scripts which shard/run the hip-tests in chunks of 4.

Test Plan

Since we are already building hip-tests on the TheRock, this patch should start running the tests.

Test Result

Passed locally, awaiting CI results.

Submission Checklist

@stellaraccident

Copy link
Copy Markdown
Collaborator

Hi @cjatin - I noticed this PR and #2001 both address enabling hip-tests on TheRock. PR #2001 was updated more recently (Jan 7), while this one hasn't had activity since it was opened.

Could you clarify which PR is the active effort? If #2001 is the main one, would you mind converting this PR to draft (or closing it) to reduce confusion? If this PR represents a different/simplified approach, happy to help get reviewers assigned.

Thanks!

@stellaraccident stellaraccident mentioned this pull request Jan 8, 2026
1 task done
@cjatin

cjatin commented Jan 9, 2026

Copy link
Copy Markdown
Contributor Author

Hi @cjatin - I noticed this PR and #2001 both address enabling hip-tests on TheRock. PR #2001 was updated more recently (Jan 7), while this one hasn't had activity since it was opened.

Could you clarify which PR is the active effort? If #2001 is the main one, would you mind converting this PR to draft (or closing it) to reduce confusion? If this PR represents a different/simplified approach, happy to help get reviewers assigned.

Thanks!

Its #2001, closing this.

@cjatin cjatin closed this Jan 9, 2026
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Jan 9, 2026
@cjatin
cjatin deleted the users/jatin/enable_hip_tests branch January 9, 2026 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants