Skip to content

feat(ci): broaden usage of fast-startup ubuntu-slim runner - #43477

Open
hainenber wants to merge 1 commit into
masterfrom
feat/continued-migration-of-fast-startup-ubuntu-slim-runner
Open

feat(ci): broaden usage of fast-startup ubuntu-slim runner#43477
hainenber wants to merge 1 commit into
masterfrom
feat/continued-migration-of-fast-startup-ubuntu-slim-runner

Conversation

@hainenber

Copy link
Copy Markdown
Contributor

feat(ci): broaden usage of fast-startup ubuntu-slim runner

SUMMARY

As heavy and highly usedubuntu-26.04 might not be readily available in the public pool, we should switch to ubuntu-slim for lightweight CI workloads

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A

TESTING INSTRUCTIONS

Green CI for acceptance threshold

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Signed-off-by: hainenber <dotronghai96@gmail.com>
@dosubot dosubot Bot added the github_actions Pull requests that update GitHub Actions code label Aug 24, 2026
@bito-code-review

bito-code-review Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Bito Automatic Review Skipped - Files Excluded

Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the excluded files settings here, or contact your Bito workspace admin at evan@preset.io.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.90%. Comparing base (1bde62f) to head (7f6db9b).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #43477      +/-   ##
==========================================
- Coverage   78.90%   78.90%   -0.01%     
==========================================
  Files        2878     2878              
  Lines      165031   165031              
  Branches    38135    38135              
==========================================
- Hits       130218   130216       -2     
- Misses      32363    32365       +2     
  Partials     2450     2450              
Flag Coverage Δ
hive 38.05% <ø> (ø)
mysql 57.80% <ø> (ø)
postgres 57.83% <ø> (-0.01%) ⬇️
presto 39.97% <ø> (ø)
python 83.57% <ø> (-0.01%) ⬇️
sqlite 57.53% <ø> (ø)
unit 73.61% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@rusackas

Copy link
Copy Markdown
Member

One thing before I approve, though. validate-all-ghas is actually failing on ubuntu-slim, not flaky: zizmor pulls a Docker image and the runner has no daemon running (failed to connect to the docker API at unix:///var/run/docker.sock). Might be worth keeping that job on a full runner unless ubuntu-slim can run Docker some other way. The rest of the broadened jobs look fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants