Skip to content

Add Free Disk Space step to CI workflows#1537

Merged
kmesh-bot merged 3 commits intomainfrom
copilot/add-disk-free-to-ci-workflows
Nov 28, 2025
Merged

Add Free Disk Space step to CI workflows#1537
kmesh-bot merged 3 commits intomainfrom
copilot/add-disk-free-to-ci-workflows

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 28, 2025 •
edited
Loading

What type of PR is this?

/kind enhancement

What this PR does / why we need it:

Adds jlumbroso/free-disk-space@v1.3.1 as the first step in all CI workflows to prevent disk exhaustion failures.

Workflows updated:

  • e2e-istio-1.24.yml, e2e-istio-1.25.yml, e2e-istio-1.26.yml
  • e2e-ipv6-istio-1.24.yml, e2e-ipv6-istio-1.25.yml, e2e-ipv6-istio-1.26.yml
  • main.yml

Configuration:

- name: Free Disk Space
  uses: jlumbroso/free-disk-space@v1.3.1
  with:
    tool-cache: false
    android: true
    dotnet: true
    haskell: true
    large-packages: true
    docker-images: true
    swap-storage: true

Clears Android SDK, .NET, Haskell, large packages, Docker images, and swap. Tool cache preserved for Go/Python toolchains.

Which issue(s) this PR fixes:
Fixes #815

Special notes for your reviewer:

Configuration matches volcano-sh/volcano#4759. Action pinned to v1.3.1 for reproducibility.

Does this PR introduce a user-facing change?:

NONE

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com//advisories
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>CI Job Disk Free</issue_title>
<issue_description>

What would you like to be added:

We frequently met disk out of use, and this cause ci failure recently.

Please reference
volcano-sh/volcano#4759 to add disk free to every CI workflow</issue_description>

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@hzxuzhonghu
Add Free Disk Space step to CI workflows
e070f25 
Co-authored-by: hzxuzhonghu <13374016+hzxuzhonghu@users.noreply.github.com>
@kmesh-bot kmesh-bot added size/M and removed size/XS labels Nov 28, 2025
@hzxuzhonghu
Pin jlumbroso/free-disk-space action to v1.3.1
617eb99 
Co-authored-by: hzxuzhonghu <13374016+hzxuzhonghu@users.noreply.github.com>
Copilot AI changed the title [WIP] Add disk free check to every CI workflow Add Free Disk Space step to CI workflows Nov 28, 2025
Copilot AI requested a review from hzxuzhonghu November 28, 2025 02:15
@hzxuzhonghu hzxuzhonghu marked this pull request as ready for review November 28, 2025 04:00
Copilot AI review requested due to automatic review settings November 28, 2025 04:00
@hzxuzhonghu
Copy link
Copy Markdown
Member

hzxuzhonghu commented Nov 28, 2025

/lgtm

@kmesh-bot kmesh-bot added the lgtm label Nov 28, 2025
Copilot AI reviewed Nov 28, 2025
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds disk space cleanup to CI workflows to prevent disk exhaustion failures. The jlumbroso/free-disk-space@v1.3.1 action is added as the first step (before checkout) in 7 workflow files that were experiencing disk space issues.

Key Changes:

  • Adds free-disk-space action to all E2E test workflows (6 files: istio 1.24/1.25/1.26 for both IPv4 and IPv6)
  • Adds free-disk-space action to the main CI workflow
  • Preserves tool cache for Go/Python while removing Android SDK, .NET, Haskell, large packages, Docker images, and swap

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/main.yml Adds disk cleanup to main CI build and test workflow
.github/workflows/e2e-istio-1.24.yml Adds disk cleanup to E2E tests with Istio 1.24
.github/workflows/e2e-istio-1.25.yml Adds disk cleanup to E2E tests with Istio 1.25
.github/workflows/e2e-istio-1.26.yml Adds disk cleanup to E2E tests with Istio 1.26
.github/workflows/e2e-ipv6-istio-1.24.yml Adds disk cleanup to E2E IPv6 tests with Istio 1.24
.github/workflows/e2e-ipv6-istio-1.25.yml Adds disk cleanup to E2E IPv6 tests with Istio 1.25
.github/workflows/e2e-ipv6-istio-1.26.yml Adds disk cleanup to E2E IPv6 tests with Istio 1.26

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 28, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.78%. Comparing base (f115d74) to head (617eb99).
⚠️ Report is 12 commits behind head on main.
see 2 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c2b4d1...617eb99. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kmesh-bot
Copy link
Copy Markdown
Collaborator

kmesh-bot commented Nov 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kmesh-bot kmesh-bot merged commit f9c7ddf into main Nov 28, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@hzxuzhonghu hzxuzhonghu Awaiting requested review from hzxuzhonghu

@bitcoffeeiux bitcoffeeiux Awaiting requested review from bitcoffeeiux

@LiZhenCheng9527 LiZhenCheng9527 Awaiting requested review from LiZhenCheng9527

Assignees

@hzxuzhonghu hzxuzhonghu

Copilot code review Copilot

Labels

approved lgtm size/M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CI Job Disk Free Panic with istio 1.24-alpha.96593644e05baa5c1b6aaab3647b2d9335a23942

4 participants

@hzxuzhonghu @kmesh-bot