Add Free Disk Space step to E2E workflows#4759
Conversation
Co-authored-by: hzxuzhonghu <13374016+hzxuzhonghu@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR addresses E2E test failures caused by disk space exhaustion on GitHub Actions runners by adding the jlumbroso/free-disk-space action to all E2E workflows that were previously missing it.
- Adds Free Disk Space cleanup step to 9 E2E workflow files
- Uses configuration matching the existing
e2e_spark.yamlimplementation - Places the cleanup step as the first step in each workflow to free disk space before other operations
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/e2e_vcctl.yaml | Adds disk space cleanup step before Volcano CLI E2E tests |
| .github/workflows/e2e_sequence.yaml | Adds disk space cleanup step before TensorFlow/Ray sequence E2E tests |
| .github/workflows/e2e_scheduling_basic.yaml | Adds disk space cleanup step before basic scheduling E2E tests |
| .github/workflows/e2e_scheduling_actions.yaml | Adds disk space cleanup step before scheduling actions E2E tests |
| .github/workflows/e2e_parallel_jobs.yaml | Adds disk space cleanup step before parallel jobs E2E tests |
| .github/workflows/e2e_hypernode.yaml | Adds disk space cleanup step before hypernode E2E tests |
| .github/workflows/e2e_dra.yml | Adds disk space cleanup step before DRA E2E tests |
| .github/workflows/e2e_cronjob.yaml | Adds disk space cleanup step before cronjob E2E tests |
| .github/workflows/e2e_admission.yaml | Adds disk space cleanup step to both admission policy and webhook E2E test jobs |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/lgtm |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JesseStutler The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/cherry-pick release-1.13 |
|
@JesseStutler: #4759 failed to apply on top of branch "release-1.13": DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@JesseStutler: new issue created for failed cherrypick: #4761 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Co-authored-by: JesseStutler <38534065+JesseStutler@users.noreply.github.com>
|
/cherry-pick release-1.12 |
|
@JesseStutler: #4759 failed to apply on top of branch "release-1.12": DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@JesseStutler: new issue created for failed cherrypick: #4769 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/cherrypick release-1.12 |
|
@JesseStutler: #4759 failed to apply on top of branch "release-1.12": DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@JesseStutler: new issue created for failed cherrypick: #4771 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
What type of PR is this?
CI/Build fix
What this PR does / why we need it:
E2E tests (TensorFlow, Ray plugin, etc.) are frequently failing due to disk space exhaustion on GitHub Actions runners. This adds the
jlumbroso/free-disk-spaceaction to all E2E workflows that were missing it.Workflows updated:
e2e_sequence.yaml(TensorFlow/Ray tests)e2e_parallel_jobs.yamle2e_admission.yaml(both jobs)e2e_cronjob.yamle2e_dra.ymle2e_hypernode.yamle2e_scheduling_actions.yamle2e_scheduling_basic.yamle2e_vcctl.yamle2e_spark.yamlalready had this step and was used as reference.Which issue(s) this PR fixes:
Fixes #4760
Special notes for your reviewer:
Configuration matches existing
e2e_spark.yamlimplementation.Does this PR introduce a user-facing change?
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.