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

Backport of CI: Dynamic runs-on values for oss/ent into release/1.3.x #17817

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #17775 to be assessed for backporting due to the inclusion of the label backport/1.3.x.

The below text is copied from the body of the original PR.


In order to fetch secrets from Vault in our Enterprise repo, we need to use self-hosted runners, but here in this public repo, we may only use public runners.

To reduce merge conflicts between OSS and ENT (and backports..), this PR adds conditions that will result in public runners here in OSS, but self-hosted runners in ENT. I have done a lot of trial-and-error to end up with what I hope is a nice middle-ground between complexity, explicitness, and non-mergeflictery.

A subsequent PR will add the steps to (again, conditionally) fetch secrets from Vault.

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/dynamic-runs-on/slightly-new-newt branch from 1df887f to f49991a Compare July 6, 2023 17:41
@hc-github-team-nomad-core hc-github-team-nomad-core merged commit 1d528b1 into release/1.3.x Jul 6, 2023
@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/dynamic-runs-on/slightly-new-newt branch from 99e8935 to 87995d2 Compare July 6, 2023 17:41
@hc-github-team-nomad-core hc-github-team-nomad-core deleted the backport/dynamic-runs-on/slightly-new-newt branch July 6, 2023 17:41
@vercel vercel bot temporarily deployed to Preview – nomad July 6, 2023 17:45 Inactive
@vercel vercel bot temporarily deployed to Preview – nomad-storybook-and-ui July 6, 2023 17:50 Inactive
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