Skip to content

conditionally select GHA runners - #1881

Merged
turkenf merged 1 commit into
crossplane-contrib:mainfrom
jastang:gha-runner-fallback
Dec 4, 2025
Merged

conditionally select GHA runners#1881
turkenf merged 1 commit into
crossplane-contrib:mainfrom
jastang:gha-runner-fallback

Conversation

@jastang

@jastang jastang commented Oct 10, 2025

Copy link
Copy Markdown
Member

Description of your changes

Follow-up to #1838 to conditionally select the runner, since not all repository owners will have access to the CNCF Oracle runners if they fork this repository. Proposed an "equivalent" GitHub-hosted runner by spec for wider coverage on the fallback option.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make generate and committed the results (ideally in a separate commit).
  • Not made any manual changes to generated files, and verified this with make check-diff.

How has this code been tested

ensure the "ternary" expression works on a fork. the trigger from this PR should also show the Oracle runner being selected.

Signed-off-by: Jason Tang <jason@upbound.io>
@turkenf
turkenf force-pushed the gha-runner-fallback branch from f7f8ee6 to 36352e7 Compare December 4, 2025 11:04
@turkenf
turkenf marked this pull request as ready for review December 4, 2025 11:07

@turkenf turkenf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks, @jastang, I'm moving this forward as it has merged into other providers and causes problems in older branches during CVE patches.

@turkenf
turkenf merged commit fc14925 into crossplane-contrib:main Dec 4, 2025
8 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.

2 participants