Skip to content

ci: pin GitHub Actions runners to ubuntu-24.04 - #831

Open
redhat-chai-bot wants to merge 1 commit into
flightctl:mainfrom
redhat-chai-bot:pin-ubuntu-24.04
Open

redhat-chai-bot wants to merge 1 commit into
flightctl:mainfrom
redhat-chai-bot:pin-ubuntu-24.04

Conversation

@redhat-chai-bot

@redhat-chai-bot redhat-chai-bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

GitHub will switch the ubuntu-latest runner label to Ubuntu 26.04 on October 19, 2026. Workflows using ubuntu-latest may break when toolchains, system packages, and default versions change on the new image.

This PR pins all runs-on: ubuntu-latest references in GitHub Actions workflows to ubuntu-24.04 to prevent unexpected breakage from the runner image change.

Changes

All runs-on: ubuntu-latest entries in .github/workflows/ have been replaced with runs-on: ubuntu-24.04. No other workflow logic or configuration has been changed. Workflows that already specified ubuntu-24.04 are untouched.


AI-generated. Review for accuracy.

@kkyrazis requested from Slack

Summary

This change updates CI configuration in .github/workflows/ci.yaml, .github/workflows/push-to-main.yaml, and .github/workflows/release.yaml. It replaces ubuntu-latest with ubuntu-24.04 for all nine listed jobs. Other workflow logic is unchanged.

  • Affected areas: .github/workflows/ only. No changes affect libs/ui-components/, libs/types/, libs/i18n/, libs/cypress/, apps/standalone/, apps/ocp-plugin/, proxy/, or packaging/.
  • Impact: The change pins the runner image for CI jobs, including container-build and integration-test jobs. It does not change container-build or E2E test logic.
  • Cross-cutting implications: No shared UI component or platform-specific app code changed. The effect is limited to the GitHub Actions runner environment.

Risk classification

risk:ask applies. The labeling instructions classify semantic CI changes as risk:ask, and changing runs-on is explicitly listed as semantic. risk:ship does not apply because it permits only comment and formatting changes to workflows. risk:show does not apply because the semantic CI change matches an explicit risk:ask trigger.

Ubuntu 26.04 becomes the default runner on Oct 19, 2026.
Pin to ubuntu-24.04 to prevent unexpected breakage from
the runner image change.
@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: flightctl/flightctl-ui/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 222a0ed3-48bc-453f-8803-53352d6bb333

📥 Commits

Reviewing files that changed from the base of the PR and between 7b4eada and 3582c54.


📒 Files selected for processing (3)
  • .github/workflows/ci.yaml
  • .github/workflows/push-to-main.yaml
  • .github/workflows/release.yaml

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 9 remain after this review.



Walkthrough

The CI, push-to-main, and release workflows now run their affected jobs on ubuntu-24.04 instead of ubuntu-latest. Other workflow configuration remains unchanged.

Changes

Runner pinning

Layer / File(s) Summary
Pin workflow job runners
.github/workflows/ci.yaml, .github/workflows/push-to-main.yaml, .github/workflows/release.yaml
The affected CI, main-branch, and release jobs now use ubuntu-24.04. Their other workflow configuration is unchanged.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested labels: risk:ask, ci

Suggested reviewers: celdrake

Merge Risk: ⚪ Minimal · up to 3582c

The workflows are pinned to Ubuntu 24.04 to avoid changes from the moving ubuntu-latest label. No concrete workflow failure is indicated, so no merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: pinning GitHub Actions runners to ubuntu-24.04.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
No-Hardcoded-Secrets Passed PASS. The pull request changes only runs-on values in three workflow files, replacing ubuntu-latest with ubuntu-24.04. The added lines contain no API keys, tokens, passwords, private keys, crede…
No-Weak-Crypto Passed PASS: The pull request only changes GitHub Actions runner labels from ubuntu-latest to ubuntu-24.04 in three workflow files. It introduces no MD5, SHA1, DES, RC4, 3DES, Blowfish, ECB mode, custom …
No-Injection-Vectors Passed PASS: The pull request changes only runs-on labels from ubuntu-latest to ubuntu-24.04 in three workflow files. It does not add or modify eval, exec, dangerouslySetInnerHTML, os.system, `…
Container-Privileges Passed PASS. The pull request changes only GitHub Actions runner labels in three workflow files. The diff introduces no container or Kubernetes manifest and no privileged, hostPID, hostNetwork, `hostIP…
No-Sensitive-Data-In-Logs Passed PASS: The pull request only changes GitHub Actions runner labels from ubuntu-latest to ubuntu-24.04 in three workflow files. It adds no logging, secrets, credentials, or data-handling commands. Ex…
Resource-Leaks Passed The pull request changes only three GitHub Actions workflow files. The diff replaces ubuntu-latest with ubuntu-24.04 and contains no changes to Go files under proxy/. Therefore, it introduces no…
Unchecked-Errors Passed PASS. The pull request changes only three GitHub Actions workflow files. It does not change any Go file under proxy/, so it introduces no unchecked or silently swallowed Go error return covered by t…
Ai-Attribution Passed The PR description includes an AI-generated notice, so the AI-use condition applies. The single PR commit contains no Co-Authored-By trailer and no AI attribution trailer. The explicit failure conditi…
Generated-Files-Not-Hand-Edited Passed The pull request changes only three GitHub Actions workflow files. The reviewed diff contains no changes under the specified generated OpenAPI model paths or libs/i18n/locales/en/translation.json. T…
I18n-Compliance Passed The pull request changes only three GitHub Actions workflow files. The reviewed diff contains no .tsx files and introduces no user-facing strings or t() calls. The i18n-compliance conditions are t…


✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR



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

@coderabbitai coderabbitai Bot added ci risk:ask Ask: medium+ risk — human review required labels Oct 9, 2026
@kkyrazis kkyrazis self-assigned this Oct 9, 2026
@kkyrazis
kkyrazis requested a review from celdrake October 9, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci risk:ask Ask: medium+ risk — human review required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants