Skip to content

add a new DAG 'maxtext_multi_tier_sav01_save_local' - #81

Open
ernie-chang wants to merge 1 commit into
masterfrom
sav01-save-local
Open

add a new DAG 'maxtext_multi_tier_sav01_save_local'#81
ernie-chang wants to merge 1 commit into
masterfrom
sav01-save-local

Conversation

@ernie-chang

@ernie-chang ernie-chang commented Jun 26, 2025

Copy link
Copy Markdown

Description

This test verifies the Orbax Multitier Checkpointing local saving function, with phase 2 replicator enabled. It conducts the following tasks:

Run the MaxText training by enabling the checkpointing, until the local checkpoints have been saved.
Clean up the saved local ram checkpoints.
Using logging explorer API to query log entries during the tests, and verify they contain the logs that local checkpoints have been saved.
Note: The local checkpoints validation is through the logging explorer API, instead of going into the pod and checking the saved checkpoints files.

Tests

cluster env:

project: cloud-tpu-multipod-dev
name: b425674043-v5p64
zone: europe-west4-b
highScaleCheckpointingConfig: enabled
GcsFuseCsiDriver: enabled
Workload Identity Federation: enabled
gcs bucket:

name: mtc-automation-bucket
Hierarchical namespace: enabled
airflow/composer env:

project: cloud-ml-auto-solutions
name: erniechang-test
composer version: 2.13.4
airflow version: 2.10.5
test result:
Tests have been conducted in the "cloud-ml-auto-solutions" project with cloud composer.
Test link

Checklist

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

  • I have performed a self-review of my code.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run one-shot tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed.

@ernie-chang
ernie-chang force-pushed the sav01-save-local branch 3 times, most recently from 30513ab to 50c20e1 Compare June 26, 2025 07:46
@camiloCienet
camiloCienet force-pushed the sav01-save-local branch 2 times, most recently from 270038b to ab6d305 Compare August 8, 2025 03:57

@CJ-Tsai-cienet CJ-Tsai-cienet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think we need to update the commit message

Comment thread dags/common/test_owner.py Outdated
ABHINAV_S = "ABHINAV S."
XUEFENG_G = "XUEFENG G."
ERNIE_C = "ERNIE C."
CAMILO= "CAMILO Q."

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I found that Severus has another PR, updating the owner's name to github ID., Maybe we need to update these once his PR is merged.

https://github.com/GoogleCloudPlatform/ml-auto-solutions/pull/801/files#diff-a8acbf5a7a49843d6305e7396705a6c608acd4accab04c2fc2c8906d1b5b5650

@camiloCienet
camiloCienet force-pushed the sav01-save-local branch 4 times, most recently from 4e89d8f to 765ab6a Compare August 8, 2025 09:15
[UPDATE] Update with comment

[NEW] Add orbax utils: apply cpc and delete cpc to the dag

[UPDATE] add delete cpc and apply cpc in orbax

[UPDATE] Update format and coding style

[UPDATE]

[UPDATE] Modify the dag structure and add cpc operation

[UPDATE] Modify the dag structure and add cpc operation

[UPDATE] update for the code comment

[UPDATE] update for the code comment

add a new DAG 'maxtext_multi_tier_sav01_save_local' GoogleCloudPlatform#804

Add comment validate()

Change log explorer text filter string

to be discarded

to be discarded

to be discarded
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