From 8e786de613c67a963bcb7c7641fd599191f4d9df Mon Sep 17 00:00:00 2001 From: Amarjeet LNU Date: Tue, 25 Aug 2026 16:15:08 -0700 Subject: [PATCH] docs(train): drop --dist loadfile from the shallow suite's README The flag was removed from createCIShallowIntegBuildSpec in SageMakerMLFPySDKInfraCDK: it pins one file's tests to one xdist worker, and each test here holds a concurrency slot until its training job reaches a terminal state (~75s), so the 17-test RLVR file alone took 19m45s against the project's 30-minute timeout. Job names are unique per invocation rather than per test function (see unique_name in harness.py), so tests are free to spread across workers, which is what the wall-clock estimate in harness.py assumes. Documentation only -- the invocation itself lives in the CDK package. --- X-AI-Prompt: The fast-integ-tests CodeBuild job is still failing on PRs now that the CDK project is deployed -- diagnose and fix it X-AI-Tool: claude-code --- sagemaker-train/tests/integ/train/shallow/README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sagemaker-train/tests/integ/train/shallow/README.md b/sagemaker-train/tests/integ/train/shallow/README.md index 07c8912185..a2687b0165 100644 --- a/sagemaker-train/tests/integ/train/shallow/README.md +++ b/sagemaker-train/tests/integ/train/shallow/README.md @@ -19,7 +19,7 @@ submit a job and wait for it, and this suite covers those code paths instead. > — which is nearly all of them — without exposing those credentials to PR code. > > **Consequence for editing this suite:** the marker selection above (`-n 8`, -> `--dist loadfile`, `-m "not gpu_intensive and not us_east_1"`) lives in +> `-m "not gpu_intensive and not us_east_1"`) lives in > `createCIShallowIntegBuildSpec` in the `SageMakerMLFPySDKInfraCDK` package, not in > this repo. Adding a file under `shallow/` is picked up automatically, but changing > *how* the suite is invoked means a change there, which deploys through a pipeline