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

github-actions: Check CI secret to tag selected OSes to run on self-hosted runners #2546

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Nov 22, 2022

Checks setting one of the following secrets:
SELF_HOSTED_MACOS, SELF_HOSTED_LINUX, SELF_HOSTED_WINDOWS
If the value stored in any of them evaluates to true (based on GitHub actions expression truth values) the jobs for the respective OS will be tagged to use self-hosted runners.

Without the secret, jobs will use GitHub cloud runners.

"runs-on" doesn't allow direct use of environment variables so we need to pass them via an extra job.

Closes: #2526

Signed-off-by: Jan Vesely jan.vesely@rutgers.edu

…ted runners

Allows setting one of the following secrets:
SELF_HOSTED_MACOS, SELF_HOSTED_LINUX, SELF_HOSTED_WINDOWS
If the value stored in any of them evaluates to true (based on github
actions expression truth values) the jobs for the respective OS will
be tagged to only use self-hosted runners.

Without the secret, the jobs will allow use of github's cloud runners.

"runs-on" doesn't allow direct use of environment variables so we need
to pass them via an extra job.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
@jvesely jvesely added the CI Continuous Integration label Nov 22, 2022
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely changed the title github-actions: Add env var to force selected OSes to run on self-hosted runner github-actions: Check CI secret to tag selected OSes to run on self-hosted runners Nov 22, 2022
@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 84.166% when pulling 4ed2e7a on jvesely/macos-self into 50c545b on devel.

@jvesely jvesely merged commit 9b9c8c5 into devel Nov 22, 2022
@jvesely jvesely deleted the jvesely/macos-self branch November 22, 2022 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants