Skip to content

ci: Mac CI with self-hosted runner for machine tests#4

Open
timcoding1988 wants to merge 3 commits into
mainfrom
ci/mac-dedicated
Open

ci: Mac CI with self-hosted runner for machine tests#4
timcoding1988 wants to merge 3 commits into
mainfrom
ci/mac-dedicated

Conversation

@timcoding1988
Copy link
Copy Markdown

@timcoding1988 timcoding1988 commented May 19, 2026

using self host mac for the mac tests

@timcoding1988 timcoding1988 force-pushed the ci/mac-dedicated branch 6 times, most recently from 0437fa6 to d3856a0 Compare May 21, 2026 12:33
@timcoding1988 timcoding1988 requested a review from ashley-cui May 26, 2026 16:30
Comment thread .github/workflows/mac.yml Outdated
Comment on lines +93 to +97
- name: Build podman + ginkgo + mac-helper
run: |
make .install.ginkgo
make podman-remote
make podman-mac-helper
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

would it be better to install the binaries built from the build task? this seems like duplicate work

@@ -0,0 +1,15 @@
#!/usr/bin/env bash
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We can leave this for now, but I think once we set up the github runner, there are pre-and post run hooks we can specify to clean the vm after runs.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

correct. think we can remove this

Comment thread .github/workflows/mac.yml Outdated
Comment on lines +7 to +14
- '.github/workflows/mac.yml'
- 'cmd/**'
- 'pkg/**'
- 'libpod/**'
- 'contrib/pkginstaller/**'
- 'Makefile'
- 'go.mod'
- 'go.sum'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the original conditional in cirrus is
PR title has [CI:ALL]
Or one of these files is changed:

  • .cirrus.yml (probably should be updated to /.github/workflows/mac.yml)
  • Makefile
  • contrib/cirrus/** (Update to contrib/gha/mac/*)
  • vendor/**
  • test/tools/**
  • test/registries*.conf
  • hack/**
  • version/rawversion/*
  • cmd/podman/machine/**
  • pkg/machine/**
  • **/machine.go

@timcoding1988 timcoding1988 force-pushed the ci/mac-dedicated branch 5 times, most recently from ec5c99d to 0134d0f Compare May 27, 2026 02:24
Signed-off-by: Tim Zhou <tizhou@redhat.com>
Pins runs-on to the MacM1-4 label so both matrix arms execute on a
single known-good host. max-parallel: 1 since both providers now share
one runner.

Signed-off-by: Tim Zhou <tizhou@redhat.com>
Signed-off-by: Tim Zhou <tizhou@redhat.com>
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