ci: move static-validation jobs off the trusted evalops-internal lane#822
Open
haasonsaas wants to merge 1 commit into
Open
ci: move static-validation jobs off the trusted evalops-internal lane#822haasonsaas wants to merge 1 commit into
haasonsaas wants to merge 1 commit into
Conversation
tag-current-version and version-bump only tag/commit/open PRs via git and the GitHub API; they have no need for private GKE or GCP identity, so the maestro-internal branch of these workflows belongs on the Hetzner-backed evalops-private-ci lane per the dual-cloud runner lane policy. Ref: evalops/deploy#7260
Contributor
Author
|
Chief-of-staff correction: do not merge this in its current form. Routing static or metadata-only work from evalops-internal into evalops-private-ci moves load into the pool that is saturated today. Revise each job by resource need: fork-safe static validation goes GitHub-hosted; code-free metadata or reducer work goes evalops-runner-guard; private CI is allowed only with a named private network, secret, or identity dependency. Add a placement test and before/after queue evidence, then request readmission. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tag-release.ymlandversion-bump.ymlare shared verbatim betweenevalops/maestroandevalops/maestro-internal, and route to the trustedevalops-internallane whenevergithub.repository == 'evalops/maestro-internal'.This PR updates that branch to
evalops-private-ci(companion PR open onmaestro-internal for the same files).
policy/dual-cloud-runner-lanes.yamlin evalops/deploy forbids thestatic_validationandaudit_metadata_onlyworkflow families on theevalops-internallane, anddocs/operations/internal-runner-capacity.mdstates the rule this PR enforces:
tag-current-versionandversion-bumptag/commit/push a release branchand open a PR via git and the GitHub API — no private GKE, GCP Secret
Manager, or Terraform access — so the internal-repo branch belongs on the
Hetzner-backed
evalops-private-cilane.Found by the org-wide runner audit: evalops/deploy#7260
Jobs moved
tag-release.yml :: tag-current-version(maestro-internal branch only)version-bump.yml :: version-bump(maestro-internal branch only)Diff scope
runs-on:lines only, preserving the existing conditional expressionshape.