Skip to content

CI: free disk in package_c workflow #5081

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
wanghan-iapcm merged 1 commit into from
Dec 4, 2025
Merged

CI: free disk in package_c workflow #5081

wanghan-iapcm merged 1 commit into from
Dec 4, 2025

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Dec 3, 2025 •
edited by coderabbitai bot
Loading

Fix "No space left on device"

Summary by CodeRabbit

  • Chores
    • Enhanced build process optimization for improved workflow performance.

✏️ Tip: You can customize this high-level summary in your review settings.

@njzjz
CI: free disk in package_c workflow
f9303d9 
Fix "No space left on device"
@njzjz njzjz marked this pull request as ready for review December 3, 2025 17:11
Copilot AI review requested due to automatic review settings December 3, 2025 17:11
Copilot AI reviewed Dec 3, 2025
View reviewed changes
Copy link
Contributor

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 a disk space cleanup step to the package_c workflow to resolve "No space left on device" errors encountered during CI builds.

  • Added a disk cleanup step using insightsengineering/disk-space-reclaimer@v1 action before the checkout step
  • Configured comprehensive cleanup options including tools cache, Android SDK, .NET, Haskell, large packages, swap storage, and Docker images

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 3, 2025

📝 Walkthrough

Walkthrough

Added a disk space reclamation step to the GitHub Actions build_c workflow using the insightsengineering/disk-space-reclaimer action configured to clean tools cache, Android, dotnet, haskell, large packages, swap storage, and Docker images.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/package_c.yml		 Added "Free Disk Space (Ubuntu)" step to build_c workflow using insightsengineering/disk-space-reclaimer@v1 with cleanup options for build artifacts and system resources

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify the disk-space-reclaimer action version is stable and appropriate
  • Confirm the selected cleanup options (tools-cache, android, dotnet, haskell, large-packages, swap-storage, docker-images) do not conflict with workflow dependencies

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'CI: free disk in package_c workflow' directly and clearly describes the main change: adding a disk-freeing step to the package_c CI workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d5fa3c and f9303d9.

📒 Files selected for processing (1)
  • .github/workflows/package_c.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
  • GitHub Check: Test Python (4, 3.12)
  • GitHub Check: Test Python (3, 3.9)
  • GitHub Check: Test Python (5, 3.9)
  • GitHub Check: Test Python (3, 3.12)
  • GitHub Check: Test Python (6, 3.12)
  • GitHub Check: Test Python (5, 3.12)
  • GitHub Check: Test Python (6, 3.9)
  • GitHub Check: Test Python (2, 3.12)
  • GitHub Check: Test Python (4, 3.9)
  • GitHub Check: Test Python (1, 3.9)
  • GitHub Check: Test Python (1, 3.12)
  • GitHub Check: Test Python (2, 3.9)
🔇 Additional comments (1)
.github/workflows/package_c.yml (1)

31-40: The disk space reclamation step is correctly placed before the build. The insightsengineering/disk-space-reclaimer@v1 action is a maintained fork listed on the GitHub Marketplace, making it a trusted choice for addressing "No space left on device" errors. The configuration with all cleanup options enabled is reasonable for resource-intensive builds. The version pinning at @v1 follows semantic versioning best practices and prevents breaking changes from v2+.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Dec 3, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.28%. Comparing base (2d5fa3c) to head (f9303d9).
⚠️ Report is 1 commits behind head on devel.

Additional details and impacted files 
@@            Coverage Diff             @@
##            devel    #5081      +/-   ##
==========================================
- Coverage   84.28%   84.28%   -0.01%     
==========================================
  Files         709      709              
  Lines       70561    70562       +1     
  Branches     3618     3618              
==========================================
- Hits        59472    59470       -2     
- Misses       9923     9924       +1     
- Partials     1166     1168       +2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@njzjz njzjz requested a review from wanghan-iapcm December 3, 2025 17:51
@wanghan-iapcm wanghan-iapcm added this pull request to the merge queue Dec 4, 2025
Merged via the queue into deepmodeling:devel with commit 2215a06 Dec 4, 2025
66 checks passed
ChiahsinChu pushed a commit to ChiahsinChu/deepmd-kit that referenced this pull request Dec 17, 2025
@njzjz @ChiahsinChu
CI: free disk in package_c workflow (deepmodeling#5081)
6191b3b 
Fix "No space left on device"

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Enhanced build process optimization for improved workflow performance.

<sub>✏️ Tip: You can customize this high-level summary in your review
settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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

@wanghan-iapcm wanghan-iapcm wanghan-iapcm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@njzjz @wanghan-iapcm