Skip to content

[CI]: GitHub Actions replaces Cirrus-CI#285

Merged
baude merged 1 commit into
mainfrom
ci/gha-migration
Apr 28, 2026
Merged

[CI]: GitHub Actions replaces Cirrus-CI#285
baude merged 1 commit into
mainfrom
ci/gha-migration

Conversation

@timcoding1988
Copy link
Copy Markdown
Collaborator

@timcoding1988 timcoding1988 commented Apr 21, 2026

Disclosure:
I have used Cursor(coding agent) to plan this change, and I have reviewed the change, which aligns with our goal to moving away from cirrus.

goal: is the tests continue to work with github runner instance.
also. this process is not officially supported by Microsoft, we will continuously monitor the stability of CI.

details:
Migrates CI from Cirrus (AWS OIDC + EC2 bare metal) to GitHub Actions.

  • validate (ubuntu-latest): make validate and Windows cross-compile via make build
  • windows-e2e (windows-latest): winmake.ps1 localtest

Removes .cirrus.yml. Updates docs/cirrus-ci-aws-oidc.md with current CI notes and a short historical pointer.

If windows-e2e fails on hosted runners (nested Hyper-V), follow-up can switch that job to a self-hosted Windows runner.

@timcoding1988 timcoding1988 changed the title ci: GitHub Actions replaces Cirrus/AWS for lint and Windows e2e ci: GitHub Actions replaces Cirrus/ Apr 21, 2026
@timcoding1988 timcoding1988 changed the title ci: GitHub Actions replaces Cirrus/ ci: GitHub Actions replaces Cirrus-CI Apr 21, 2026
@timcoding1988 timcoding1988 changed the title ci: GitHub Actions replaces Cirrus-CI [DNM]: GitHub Actions replaces Cirrus-CI Apr 21, 2026
@timcoding1988 timcoding1988 marked this pull request as draft April 21, 2026 13:51
@timcoding1988 timcoding1988 force-pushed the ci/gha-migration branch 2 times, most recently from c2435bf to 10d1a14 Compare April 22, 2026 17:25
@timcoding1988 timcoding1988 changed the title [DNM]: GitHub Actions replaces Cirrus-CI GitHub Actions replaces Cirrus-CI Apr 22, 2026
@timcoding1988 timcoding1988 changed the title GitHub Actions replaces Cirrus-CI [CI GitHub Actions replaces Cirrus-CI Apr 22, 2026
@timcoding1988 timcoding1988 changed the title [CI GitHub Actions replaces Cirrus-CI [CI]: GitHub Actions replaces Cirrus-CI Apr 22, 2026
@timcoding1988 timcoding1988 marked this pull request as ready for review April 22, 2026 18:04
@timcoding1988 timcoding1988 requested review from Luap99 and baude April 22, 2026 18:04
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/test.yml Outdated
@timcoding1988 timcoding1988 force-pushed the ci/gha-migration branch 2 times, most recently from 332f3b3 to 0b29da1 Compare April 23, 2026 19:51
@timcoding1988 timcoding1988 requested a review from kolyshkin April 23, 2026 21:26
@timcoding1988 timcoding1988 force-pushed the ci/gha-migration branch 2 times, most recently from 091a886 to 0b7f1a3 Compare April 27, 2026 14:49
Comment thread .github/workflows/test.yml Outdated
Replace Cirrus EC2 (AWS OIDC, bare metal) with GHA jobs: validate and
cross-build on ubuntu-latest, e2e on windows-latest via winmake.ps1.
Remove .cirrus.yml.

Signed-off-by: Tim Zhou <tizhou@redhat.com>
Copy link
Copy Markdown
Member

@Luap99 Luap99 left a comment

Choose a reason for hiding this comment

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

LGTM, overall. @baude PTAL

There is something to be done about the test log output,https://github.com/containers/libhvee/actions/runs/25051643474/job/73380700310?pr=285

It just spams with progress bars making the actual test output really hard to read in case of failures as you would need to scroll to the right lines first.

@baude baude merged commit e51fb59 into main Apr 28, 2026
3 checks passed
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.

5 participants