Skip to content

Use Ubuntu-Slim runner for light operations#348

Merged
5ouma merged 2 commits intomainfrom
ci-actions-ubuntu-slim
Dec 6, 2025
Merged

Use Ubuntu-Slim runner for light operations#348
5ouma merged 2 commits intomainfrom
ci-actions-ubuntu-slim

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Dec 6, 2025

⚠️ Issue

close #


✏️ Description

It's more performant and cheaper.


Copilot AI review requested due to automatic review settings December 6, 2025 03:55
@github-actions github-actions bot added the 🎽 CI Changes to CI configuration files and scripts label Dec 6, 2025
Copy link

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 pull request attempts to migrate GitHub Actions workflows from Ubuntu-Latest to Ubuntu-Slim runners to improve performance and reduce costs for lightweight operations. However, there is a critical issue: Ubuntu-Slim is not a valid GitHub Actions runner type.

Key Changes:

  • Updates runner type in 4 workflow files from Ubuntu-Latest to Ubuntu-Slim
  • Increases timeout for pre-commit workflow from 3 to 5 minutes

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/release.yml Changes runner for both tagpr and release jobs to Ubuntu-Slim
.github/workflows/pre-commit.yml Changes runner to Ubuntu-Slim and increases timeout from 3 to 5 minutes
.github/workflows/merge-gatekeeper.yml Changes runner for merge gatekeeper job to Ubuntu-Slim
.github/workflows/labeler.yml Changes runner for labeler job to Ubuntu-Slim

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

@5ouma 5ouma force-pushed the ci-actions-ubuntu-slim branch from de741d6 to 0c89d05 Compare December 6, 2025 03:59
@coderabbitai
Copy link

coderabbitai bot commented Dec 6, 2025

Warning

Rate limit exceeded

@5ouma has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 29 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 4efad7c and e19ff31.

📒 Files selected for processing (3)
  • .github/workflows/labeler.yml (1 hunks)
  • .github/workflows/pre-commit.yml (1 hunks)
  • .github/workflows/release.yml (2 hunks)

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

The pull request updates GitHub Actions runner images from ubuntu-latest to ubuntu-latest-slim across four workflow files. Additionally, the pre-commit workflow timeout is increased from 3 minutes to 5 minutes.

Changes

Cohort / File(s) Change Summary
GitHub Actions runner image updates
.github/workflows/labeler.yml, .github/workflows/merge-gatekeeper.yml, .github/workflows/release.yml
Runner image changed from ubuntu-latest to ubuntu-latest-slim for specified jobs
GitHub Actions runner image and timeout updates
.github/workflows/pre-commit.yml
Runner image changed from ubuntu-latest to ubuntu-latest-slim; job timeout increased from 3 minutes to 5 minutes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

  • Verify that ubuntu-latest-slim image is compatible with existing workflow steps across all files
  • Confirm the 5-minute timeout in pre-commit.yml is sufficient for job execution

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Use Ubuntu-Slim runner for light operations' accurately summarizes the main change - switching CI runner images from Ubuntu-Latest to Ubuntu-Slim across multiple workflows.
Description check ✅ Passed The description is related to the changeset, explaining the rationale (performance and cost improvements) for switching to Ubuntu-Slim runners, though it lacks specific technical details.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@5ouma 5ouma merged commit 8131d2e into main Dec 6, 2025
6 of 7 checks passed
@5ouma 5ouma deleted the ci-actions-ubuntu-slim branch December 6, 2025 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎽 CI Changes to CI configuration files and scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants