Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Migrate cross-compile task to github-actions #4966

Closed
wants to merge 1 commit into from

Conversation

cevich
Copy link
Member

@cevich cevich commented Aug 9, 2023

What type of PR is this?

/kind other

What this PR does / why we need it:

As of Sept. 1st 2023 Cirrus-CI will start charging for hosted compute time. At the time of this commit, according to the containers-org billing page, github provides 200 "free" minutes per month of MacOS time. However, the same page shows 0 minutes used across-the-board, so perhaps they're not counting minutes at all.

Note: This new workflow will...

  • Run much slower in GHA (~25-30m) vs Cirrus-CI (~5-7m)
  • Execute for all PRs and merges w/o regard to the branch (Cirrus-CI only acted upon the
    branch copy of .cirrus.yml vs GHA always reads workflow YAML from "main").
  • NOT prevent execution of Cirrus-CI MacOS tasks on release branches. They will need
    to be individually disabled so as to not count against minutes-pool.
  • Use an older MacOS environment, currently latest is "MacOS 12
    Monterey", whereas Cirrus-CI was using "MacOS 13 Ventura".
  • NOT synchronize execution with Cirrus-CI tasks. This is possible to do, but fairly complex.
  • NOT mitigate any unintended consequences due to the final CI-states not being joined in any way.

How to verify it

The new workflow will execute and pass, w/n [CI:DOCS] is present in the title.

Which issue(s) this PR fixes:

https://cirrus-ci.org/blog/2023/07/17/limiting-free-usage-of-cirrus-ci

Special notes for your reviewer:

This is an experimental initial implementation. It will introduce several side-effects that need investigating. Please do not merge yet.

Does this PR introduce a user-facing change?

None

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 9, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cevich

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved label Aug 9, 2023
@cevich cevich force-pushed the move_macos_job branch 6 times, most recently from 94071c1 to 6fed082 Compare August 9, 2023 19:17
@cevich cevich changed the title [WIP] [CI:DOCS] Migrate cross-compile task to github-actions [WIP] Migrate cross-compile task to github-actions Aug 9, 2023
@cevich cevich marked this pull request as draft August 9, 2023 19:23
As of Sept. 1st 2023 Cirrus-CI will start charging for hosted compute
time.  At the time of this commit, according to the containers-org
billing page, github provides 200 "free" minutes per month of MacOS
time.  However, the same page shows 0 minutes used across-the-board, so
perhaps they're not counting minutes at all.

Signed-off-by: Chris Evich <[email protected]>
@cevich
Copy link
Member Author

cevich commented Aug 15, 2023

Closing, it was decided to do more research on other options first.

@cevich cevich closed this Aug 15, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant