Docs migration from XPK to CTK(4/5) - #5187
KevinYenky wants to merge 14 commits into
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Code Review
This pull request updates the documentation for emergency and multi-tier checkpointing to transition from XPK-based instructions to Cluster Toolkit (gcluster) workflows. It includes detailed steps for configuring GKE clusters, setting up Cloud Storage buckets with Hierarchical Namespace (HNS), granting necessary IAM permissions, and submitting workloads. The review comments correctly identify that the documented commands for setting gcluster configuration defaults are incorrect and should be updated from gcluster job config set to gcluster config set to prevent command execution errors.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…ys run name in checkpointing guides TAG=agy CONV=ae9cc2c4-0e92-4deb-90ca-13acf319b40e
…ting.md TAG=agy CONV=ae9cc2c4-0e92-4deb-90ca-13acf319b40e
…ariables in checkpointing guides TAG=agy CONV=0bbf46ed-e401-4e1e-9730-8919db285e37
Description
This PR is the fourth part of decomposing the XPK to Cluster Toolkit document migration. It focuses on updating our documentation regarding Checkpointing Solutions to use Cluster Toolkit (CTK) instead of the deprecated XPK launcher.
Specifically, this PR updates the following guides:
gclusterequivalents, updating environment variables and volume mount instructions as necessary.Why this is being made:
CTK is replacing XPK as the primary submission mechanism for Google Kubernetes Engine (GKE) workloads. Breaking down the massive PR #5175 into smaller, logical chunks makes the migration significantly easier to review and test.
Tests
Tested by scripts that run the
gclustercommands provided in the documents, with results stored in a GCS bucket.Checklist
gemini-reviewlabel.