Skip to content

Update MaxText package tests to run pathways tests & remove redundant CI tests - #2835

Merged
copybara-service[bot] merged 1 commit into
mainfrom
ci_tests
Dec 16, 2025
Merged

Update MaxText package tests to run pathways tests & remove redundant CI tests#2835
copybara-service[bot] merged 1 commit into
mainfrom
ci_tests

Conversation

@SurbhiJainUSC

@SurbhiJainUSC SurbhiJainUSC commented Dec 16, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

  • Remove redundant Tests Github Action
  • Add pathways tests to MaxText Package Tests Github Action

Tests

CI testing

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@SurbhiJainUSC
SurbhiJainUSC force-pushed the ci_tests branch 2 times, most recently from dad3dfe to b57be72 Compare December 16, 2025 01:06
@SurbhiJainUSC SurbhiJainUSC changed the title Update MaxText package tests to run with GPU wheel Update MaxText package tests to run pathways tests & remove redundant CI tests Dec 16, 2025
@SurbhiJainUSC
SurbhiJainUSC force-pushed the ci_tests branch 7 times, most recently from 3418787 to 84a84e7 Compare December 16, 2025 18:34
@SurbhiJainUSC
SurbhiJainUSC marked this pull request as ready for review December 16, 2025 18:59
shralex
shralex reviewed Dec 16, 2025
View reviewed changes
Comment thread .github/workflows/build_and_test_maxtext.yml Resolved
@copybara-service
copybara-service Bot merged commit 2e2776a into main Dec 16, 2025
22 of 23 checks passed
@copybara-service
copybara-service Bot deleted the ci_tests branch December 16, 2025 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shralex shralex shralex approved these changes
@xuefgu xuefgu xuefgu approved these changes
@gobbleturk gobbleturk Awaiting requested review from gobbleturk gobbleturk is a code owner
@khatwanimohit khatwanimohit Awaiting requested review from khatwanimohit khatwanimohit is a code owner
@parambole parambole Awaiting requested review from parambole parambole is a code owner
@bvandermoon bvandermoon Awaiting requested review from bvandermoon bvandermoon is a code owner
@richjames0 richjames0 Awaiting requested review from richjames0
@jacoguzo jacoguzo Awaiting requested review from jacoguzo
@RissyRan RissyRan Awaiting requested review from RissyRan RissyRan is a code owner
@gagika gagika Awaiting requested review from gagika gagika is a code owner
@A9isha A9isha Awaiting requested review from A9isha A9isha is a code owner
@jiangjy1982 jiangjy1982 Awaiting requested review from jiangjy1982 jiangjy1982 is a code owner
@vipannalla vipannalla Awaiting requested review from vipannalla vipannalla is a code owner

Assignees

No one assigned

Labels

pull ready

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SurbhiJainUSC @shralex @xuefgu