Skip to content

Change runners temp#18163

Merged
FelixMalfait merged 3 commits intomainfrom
change-runners-temp
Feb 23, 2026
Merged

Change runners temp#18163
FelixMalfait merged 3 commits intomainfrom
change-runners-temp

Conversation

@charlesBochet
Copy link
Copy Markdown
Member

@charlesBochet charlesBochet commented Feb 23, 2026

Temporarily moving all ubuntu-latest 1 core to depot except ci-website

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Migrates all GitHub Actions workflow runners from ubuntu-latest to depot-ubuntu-24.04 across 24 workflow files. This appears to be a temporary infrastructure change to use Depot's Ubuntu 24.04 runners, likely for improved performance or specific environment requirements. The changes include:

  • All workflow jobs now use depot-ubuntu-24.04 as the runner
  • Jobs with service containers (postgres, redis, clickhouse) are included in the migration
  • Minor whitespace cleanup in preview-env-keepalive.yaml (trailing spaces removed)

No functional changes to workflow logic, steps, or configurations.

Confidence Score: 5/5

  • Safe to merge - straightforward infrastructure change with minimal risk
  • This PR makes a simple, consistent change across all workflow files by replacing the runner specification. The change is mechanical, well-understood, and marked as temporary in the commit message. All changes follow the same pattern, and service containers should be fully compatible with the new runner.
  • No files require special attention - all changes are consistent and straightforward

Important Files Changed

Filename Overview
.github/workflows/ci-breaking-changes.yaml Runner change with service containers (postgres, redis, clickhouse) - should be compatible
.github/workflows/ci-front.yaml Runner changes across multiple jobs (merge-reports, front-tasks, e2e-test, status checks)
.github/workflows/preview-env-keepalive.yaml Runner change plus whitespace cleanup (trailing spaces removed)

Last reviewed commit: e500384

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

24 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 24 files

@FelixMalfait FelixMalfait merged commit 129d1ed into main Feb 23, 2026
48 of 56 checks passed
@FelixMalfait FelixMalfait deleted the change-runners-temp branch February 23, 2026 09:53
@twenty-eng-sync
Copy link
Copy Markdown

Hey @charlesBochet! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

charlesBochet added a commit that referenced this pull request Mar 3, 2026
## Summary
- Replaces all `depot-ubuntu-24.04` runners with `ubuntu-latest`
- Replaces all `depot-ubuntu-24.04-8` runners with
`ubuntu-latest-8-cores`
- Updates storybook build cache keys in ci-front.yaml to reflect the
runner name change

Reverts the temporary Depot migration introduced in #18163 / #18179
across all 23 workflow files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants