Skip to content

Conversation

@SurbhiJainUSC
Copy link
Collaborator

@SurbhiJainUSC SurbhiJainUSC commented Dec 12, 2025

Description

  • Refactor docker_build_dependency_image.sh and delete unnecessary docker build modes
  • Deleted unused Dockerfiles: maxtext_jax_ai_image.Dockerfile, jetstream_pathways.Dockerfile, maxengine_server.Dockerfile, maxtext_db_dependencies, maxtext_custom_wheels.Dockerfile
  • Updated relevant documentation & Github tests
  • Fixes: b/446945869

Tests

https://paste.googleplex.com/4931634182881280

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 refactor_docker branch 19 times, most recently from 35e49d0 to 823aa33 Compare December 17, 2025 23:24
@SurbhiJainUSC SurbhiJainUSC changed the title Refactor dockerfiles and deprecate stable_stack, jax_ai_image, custom wheels mode Refactor dockerfiles and deprecate stable_stack and jax_ai_image mode Dec 17, 2025
@SurbhiJainUSC SurbhiJainUSC force-pushed the refactor_docker branch 3 times, most recently from 90c5dff to 7986057 Compare December 18, 2025 17:59
@SurbhiJainUSC SurbhiJainUSC force-pushed the refactor_docker branch 6 times, most recently from 2f2243a to 5726f1d Compare December 18, 2025 19:07
@codecov
Copy link

codecov bot commented Dec 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants