mirror of
https://github.com/go-gitea/gitea.git
synced 2026-07-02 10:53:27 +00:00
## Summary This PR adds **scoped workflows** to Gitea Actions. Workflows defined centrally in a "source" repository that automatically run on every repository in scope: an organization's repositories, or (for instance admins) every repository on the instance. Each scoped run executes in the consuming repository's own context (its runners, secrets, and branch) while its content is read from the source repository, so an org or instance can mandate shared CI across many repositories without copying workflow files into each one. An owner or instance admin registers source repositories on a settings page and can mark individual workflows as **required**. A required scoped workflow cannot be opted out by a consuming repository and gates its pull-request merges; an optional one can be disabled per repository. Scoped workflows live under a dedicated `SCOPED_WORKFLOW_DIRS` (default `.gitea/scoped_workflows`), kept separate from regular `WORKFLOW_DIRS`. ## Main changes ### Configuration New `SCOPED_WORKFLOW_DIRS` setting, validated to not overlap with `WORKFLOW_DIRS`. Default: `.gitea/scoped_workflows` ### Data model & migration - New `action_scoped_workflow_source` table mapping a registering owner (`owner_id`, where `0` = instance-level) to a source repository, with a per-workflow `WorkflowConfigs` map. - `ActionRun` gains `WorkflowRepoID` / `WorkflowCommitSHA` (the pinned content source) and an `IsScopedRun` flag. ### Detection & run creation On consumer events, scoped workflows from the effective sources (the owner's own sources plus instance-level ones) are matched and turned into runs that execute in the consumer's context, with content pinned to the source repo's default-branch commit. `on: workflow_run` and `on: schedule` are currently not supported. ### Opt-out A consuming repository can disable an optional scoped workflow (tracked separately from regular `DisabledWorkflows`); required scoped workflows can never be disabled, opted out, or bypassed. ### Commit status A scoped run's status context format is `"<source repo full name>: <workflow display name> / <job> (<event>)"` (for example: `my-org/scoped-workflows: db-tests / test-sqlite (pull_request)`), keeping it distinct from a same-named repo-level workflow and from other sources. ### Required status checks Admins mark workflows required and supply status-check patterns. `EffectiveRequiredContexts` appends those patterns to the branch protection's required contexts and they are matched must-present-and-pass. If the status checks from scoped workflows fail, the PR cannot be merged. NOTE: scoped workflows' required status checks patterns can protect any target branch that has a protection rule, even though the rule's "Status Check" is disabled. A target branch with no protection rule cannot be protected. <details> <summary>Screenshots</summary> <img width="1400" alt="image" src="https://github.com/user-attachments/assets/a5d1db33-15ec-487e-93be-2bc04b4e6643" /> </details> ### Reusable workflows (`uses:`) A scoped workflow's local `uses: ./...` resolves against the source repository. `uses:` directory validation honors the instance-configurable `WORKFLOW_DIRS` and `SCOPED_WORKFLOW_DIRS` (previously hardcoded to `.gitea`/`.github/workflows`). ### Manual dispatch `workflow_dispatch` is supported for scoped workflows (web and API), resolving inputs/content from the source repo. ### Performance A process-local LRU cache keyed by source repo ID for the per-source workflow parse, so instance-level and owner-level sources don't open the source repo and parse workflow files on every event. ### UI Org / user / admin pages to register and remove sources, search repositories, and mark workflows required with their status-check patterns. The repository Actions sidebar groups scoped workflows by source with owner/instance labels and required/disabled badges. <details> <summary>Screenshots</summary> Scoped workflows setting page: <img width="1600" alt="image" src="https://github.com/user-attachments/assets/9d19f667-97a5-4935-92b2-e53f105e3642" /> Consumer repo's Actions runs list: <img width="1600" alt="image" src="https://github.com/user-attachments/assets/a77241f9-0aa9-41aa-ba73-12a9a688cb64" /> - `Owner`: this is a owner-level scoped workflows source repo - `Global`: this is a global scoped workflows source repo - `Required`: this scoped workflow is required, repo admin cannot disable it </details> --- Docs: https://gitea.com/gitea/docs/pulls/447 --------- Co-authored-by: bircni <bircni@icloud.com>
190 lines
5.8 KiB
Go
190 lines
5.8 KiB
Go
// Copyright 2026 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package jobparser
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestParseUses(t *testing.T) {
|
|
t.Run("LocalSameRepo", func(t *testing.T) {
|
|
cases := []struct {
|
|
name string
|
|
in string
|
|
want UsesRef
|
|
}{
|
|
{
|
|
name: "gitea dir, .yml",
|
|
in: "./.gitea/workflows/build.yml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/workflows/build.yml"},
|
|
},
|
|
{
|
|
name: "github dir, .yml",
|
|
in: "./.github/workflows/build.yml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".github/workflows/build.yml"},
|
|
},
|
|
{
|
|
name: "gitea dir, .yaml",
|
|
in: "./.gitea/workflows/build.yaml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/workflows/build.yaml"},
|
|
},
|
|
{
|
|
name: "filename containing dots is allowed",
|
|
in: "./.gitea/workflows/foo..bar.yml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/workflows/foo..bar.yml"},
|
|
},
|
|
{
|
|
name: "nested subdirectory",
|
|
in: "./.gitea/workflows/sub/build.yml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/workflows/sub/build.yml"},
|
|
},
|
|
{
|
|
// ParseUses is dir-agnostic; the allowed directories (WORKFLOW_DIRS / SCOPED_WORKFLOW_DIRS) are enforced by ResolveUses.
|
|
name: "scoped workflows dir parses",
|
|
in: "./.gitea/scoped_workflows/lib.yml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/scoped_workflows/lib.yml"},
|
|
},
|
|
{
|
|
name: "non-default dir parses (allowlist enforced downstream)",
|
|
in: "./.gitea/custom_workflows/x.yaml",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/custom_workflows/x.yaml"},
|
|
},
|
|
{
|
|
name: "leading/trailing whitespace is trimmed",
|
|
in: " ./.gitea/workflows/build.yml ",
|
|
want: UsesRef{Kind: UsesKindLocalSameRepo, Path: ".gitea/workflows/build.yml"},
|
|
},
|
|
}
|
|
for _, c := range cases {
|
|
t.Run(c.name, func(t *testing.T) {
|
|
got, err := ParseUses(c.in)
|
|
require.NoError(t, err)
|
|
assert.Equal(t, c.want, *got)
|
|
})
|
|
}
|
|
})
|
|
|
|
t.Run("LocalCrossRepo", func(t *testing.T) {
|
|
cases := []struct {
|
|
name string
|
|
in string
|
|
want UsesRef
|
|
}{
|
|
{
|
|
name: "gitea dir, simple ref",
|
|
in: "owner/repo/.gitea/workflows/build.yml@v1",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".gitea/workflows/build.yml",
|
|
Ref: "v1",
|
|
},
|
|
},
|
|
{
|
|
name: "github dir, branch ref",
|
|
in: "owner/repo/.github/workflows/build.yml@main",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".github/workflows/build.yml",
|
|
Ref: "main",
|
|
},
|
|
},
|
|
{
|
|
name: ".yaml extension",
|
|
in: "owner/repo/.gitea/workflows/build.yaml@abc123",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".gitea/workflows/build.yaml",
|
|
Ref: "abc123",
|
|
},
|
|
},
|
|
{
|
|
name: "ref with slashes (refs/heads/feature)",
|
|
in: "owner/repo/.gitea/workflows/build.yml@refs/heads/feature",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".gitea/workflows/build.yml",
|
|
Ref: "refs/heads/feature",
|
|
},
|
|
},
|
|
{
|
|
name: "nested subdirectory under workflows",
|
|
in: "owner/repo/.gitea/workflows/sub/build.yml@v1",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".gitea/workflows/sub/build.yml",
|
|
Ref: "v1",
|
|
},
|
|
},
|
|
{
|
|
name: "scoped workflows dir parses (allowlist enforced by ResolveUses)",
|
|
in: "owner/repo/.gitea/scoped_workflows/lib.yml@v1",
|
|
want: UsesRef{
|
|
Kind: UsesKindLocalCrossRepo,
|
|
Owner: "owner",
|
|
Repo: "repo",
|
|
Path: ".gitea/scoped_workflows/lib.yml",
|
|
Ref: "v1",
|
|
},
|
|
},
|
|
}
|
|
for _, c := range cases {
|
|
t.Run(c.name, func(t *testing.T) {
|
|
got, err := ParseUses(c.in)
|
|
require.NoError(t, err)
|
|
assert.Equal(t, c.want, *got)
|
|
})
|
|
}
|
|
})
|
|
|
|
t.Run("Errors", func(t *testing.T) {
|
|
cases := []struct {
|
|
name string
|
|
in string
|
|
}{
|
|
{name: "empty string", in: ""},
|
|
{name: "whitespace only", in: " "},
|
|
|
|
// Same-repo malformed (note: a wrong *directory* parses and should be rejected by the caller)
|
|
{name: "same-repo with @ref", in: "./.gitea/workflows/build.yml@v1"},
|
|
{name: "same-repo wrong extension", in: "./.gitea/workflows/build.txt"},
|
|
{name: "same-repo missing extension", in: "./.gitea/workflows/build"},
|
|
{name: "same-repo absolute path", in: "/.gitea/workflows/build.yml"},
|
|
{name: "same-repo path traversal", in: "./.gitea/workflows/../escape.yml"},
|
|
{name: "same-repo double slash", in: "./.gitea/workflows//build.yml"},
|
|
{name: "same-repo redundant ./", in: "./.gitea/workflows/./build.yml"},
|
|
|
|
// Cross-repo malformed
|
|
{name: "cross-repo missing @ref", in: "owner/repo/.gitea/workflows/build.yml"},
|
|
{name: "cross-repo empty ref", in: "owner/repo/.gitea/workflows/build.yml@"},
|
|
{name: "cross-repo missing owner", in: "/repo/.gitea/workflows/build.yml@v1"},
|
|
{name: "cross-repo missing repo", in: "owner//.gitea/workflows/build.yml@v1"},
|
|
{name: "cross-repo wrong extension", in: "owner/repo/.gitea/workflows/build.txt@v1"},
|
|
{name: "cross-repo path traversal", in: "owner/repo/.gitea/workflows/../escape.yml@v1"},
|
|
{name: "cross-repo double slash in path", in: "owner/repo/.gitea/workflows//build.yml@v1"},
|
|
// owner/repo with chars Gitea's name validators reject
|
|
{name: "cross-repo owner with space", in: "bad owner/repo/.gitea/workflows/build.yml@v1"},
|
|
{name: "cross-repo repo with @", in: "owner/re@po/.gitea/workflows/build.yml@v1"},
|
|
}
|
|
for _, c := range cases {
|
|
t.Run(c.name, func(t *testing.T) {
|
|
_, err := ParseUses(c.in)
|
|
assert.Error(t, err)
|
|
})
|
|
}
|
|
})
|
|
}
|