fix: keep pipeline pytest temp files off /tmp#1797
fix: keep pipeline pytest temp files off /tmp#1797nightcityblade wants to merge 1 commit intoNVIDIA-NeMo:mainfrom
Conversation
Signed-off-by: nightcityblade <nightcityblade@gmail.com>
Greptile SummaryThis PR adds a conditional branch in the Confidence Score: 5/5Safe to merge — minimal, targeted CI-only change with no impact on production code or test logic. The change is a single conditional block in the CI workflow, correctly scoped to the No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[cicd-cpu-tests matrix job\nfolder × python-version] --> B{FOLDER == 'pipelines'?}
B -- yes --> C[mkdir -p\n$GITHUB_WORKSPACE/pytest-tmp]
C --> D[coverage run ... pytest\n--basetemp=$GITHUB_WORKSPACE/pytest-tmp]
D --> E[Temp files → workspace\nlarge partition]
B -- no --> F[coverage run ... pytest\ndefault basetemp]
F --> G[Temp files → /tmp\nsmall partition]
Reviews (1): Last reviewed commit: "fix: keep pipeline pytest temp files off..." | Re-trigger Greptile |
Description
For the CPU pipelines test shard, send pytest temporary files to
$GITHUB_WORKSPACE/pytest-tmpinstead of the runner's default/tmp. This keeps the Ray--temp-dir/pytest temp tree off the smaller/tmppartition and matches the workaround described in the issue.Closes #1775.
Usage
Checklist