fix: reduce idle runner polling by default

Raise the default idle fetch backoff ceiling to reduce steady-state FetchTask traffic from idle runner fleets while keeping the value configurable for installations that need faster idle polling.

Fixes: https://gitea.com/gitea/runner/issues/762

Co-Authored-By: GPT-5 Codex <codex@openai.com>

internal/app/run/runner.go

@@ -366,7 +366,6 @@ func (r *Runner) run(ctx context.Context, task *runnerv1.Task, reporter *report.
 	} else if t := task.Secrets["GITHUB_TOKEN"]; t != "" {
 		preset.Token = t
 	}
-	applyPullRequestTargetCheckoutContext(preset)
 
 	if actionsIDTokenRequestURL := taskContext["actions_id_token_request_url"].GetStringValue(); actionsIDTokenRequestURL != "" {
 		envs["ACTIONS_ID_TOKEN_REQUEST_URL"] = actionsIDTokenRequestURL
@@ -575,41 +574,6 @@ func postInternalCache(url, secret string, body map[string]string) error {
 	return nil
 }
 
-func applyPullRequestTargetCheckoutContext(preset *model.GithubContext) {
-	if preset == nil || preset.EventName != "pull_request_target" {
-		return
-	}
-	headSHA := nestedString(preset.Event, "pull_request", "head", "sha")
-	if headSHA == "" {
-		return
-	}
-	preset.Sha = headSHA
-	preset.Ref = headSHA
-	if headRef := nestedString(preset.Event, "pull_request", "head", "ref"); headRef != "" {
-		preset.HeadRef = headRef
-		preset.RefName = headRef
-	}
-}
-
-func nestedString(m map[string]any, keys ...string) string {
-	var current any = m
-	for _, key := range keys {
-		next, ok := current.(map[string]any)
-		if !ok {
-			return ""
-		}
-		current, ok = next[key]
-		if !ok {
-			return ""
-		}
-	}
-	value := current.(string)
-	if value == "" {
-		return ""
-	}
-	return value
-}
-
 func (r *Runner) RunningCount() int64 {
 	return r.runningCount.Load()
 }

internal/app/run/runner_test.go

@@ -7,7 +7,6 @@ import (
 	"context"
 	"testing"
 
-	"gitea.com/gitea/runner/act/model"
 	clientmocks "gitea.com/gitea/runner/internal/pkg/client/mocks"
 	"gitea.com/gitea/runner/internal/pkg/config"
 	"gitea.com/gitea/runner/internal/pkg/ver"
@@ -93,67 +92,6 @@ func TestNewRunnerInitializesLabelsAndEnvironment(t *testing.T) {
 	require.Nil(t, r.cacheHandler)
 }
 
-func TestApplyPullRequestTargetCheckoutContextUsesHeadSHA(t *testing.T) {
-	preset := &model.GithubContext{
-		EventName: "pull_request_target",
-		Sha:       "base-sha",
-		Ref:       "refs/heads/main",
-		RefName:   "main",
-		HeadRef:   "feature",
-		Event: map[string]any{
-			"pull_request": map[string]any{
-				"head": map[string]any{
-					"sha": "head-sha",
-					"ref": "contributor-branch",
-				},
-			},
-		},
-	}
-
-	applyPullRequestTargetCheckoutContext(preset)
-
-	require.Equal(t, "head-sha", preset.Sha)
-	require.Equal(t, "head-sha", preset.Ref)
-	require.Equal(t, "contributor-branch", preset.RefName)
-	require.Equal(t, "contributor-branch", preset.HeadRef)
-}
-
-func TestApplyPullRequestTargetCheckoutContextNoOpsWithoutHeadSHA(t *testing.T) {
-	preset := &model.GithubContext{
-		EventName: "pull_request_target",
-		Sha:       "base-sha",
-		Ref:       "refs/heads/main",
-		RefName:   "main",
-		Event:     map[string]any{},
-	}
-
-	applyPullRequestTargetCheckoutContext(preset)
-
-	require.Equal(t, "base-sha", preset.Sha)
-	require.Equal(t, "refs/heads/main", preset.Ref)
-	require.Equal(t, "main", preset.RefName)
-}
-
-func TestApplyPullRequestTargetCheckoutContextNoOpsForOtherEvents(t *testing.T) {
-	preset := &model.GithubContext{
-		EventName: "pull_request",
-		Sha:       "merge-sha",
-		Ref:       "refs/pull/1/merge",
-		Event: map[string]any{
-			"pull_request": map[string]any{
-				"head": map[string]any{
-					"sha": "head-sha",
-				},
-			},
-		},
-	}
-
-	applyPullRequestTargetCheckoutContext(preset)
-
-	require.Equal(t, "merge-sha", preset.Sha)
-	require.Equal(t, "refs/pull/1/merge", preset.Ref)
-}
-
 func taskWithDefaultActionsURL(url string) *runnerv1.Task {
 	return &runnerv1.Task{
 		Context: &structpb.Struct{

internal/pkg/config/config.example.yaml

@@ -39,7 +39,7 @@ runner:
   # The maximum interval for fetching the job from the Gitea instance.
   # The runner uses exponential backoff when idle, increasing the interval up to this maximum.
   # Set to 0 or same as fetch_interval to disable backoff.
-  fetch_interval_max: 5s
+  fetch_interval_max: 1m
   # While idle, remove stale bind-workdir task directories and orphaned host-mode
   # scratch directories (left behind when a host cleanup delete stalls) older than
   # this duration. Setting either workdir_cleanup_age or idle_cleanup_interval to 0

internal/pkg/config/config.go

@@ -183,7 +183,7 @@ func LoadDefault(file string) (*Config, error) {
 		cfg.Runner.FetchInterval = 2 * time.Second
 	}
 	if cfg.Runner.FetchIntervalMax <= 0 {
-		cfg.Runner.FetchIntervalMax = 5 * time.Second
+		cfg.Runner.FetchIntervalMax = time.Minute
 	}
 	if cfg.Runner.WorkdirCleanupAge == 0 && !definedRunnerKeys["workdir_cleanup_age"] {
 		cfg.Runner.WorkdirCleanupAge = 24 * time.Hour

internal/pkg/config/config_test.go

@@ -48,6 +48,12 @@ func TestLoadDefault_DefaultsWorkdirCleanupAge(t *testing.T) {
 	assert.Equal(t, 10*time.Minute, cfg.Runner.IdleCleanupInterval)
 }
 
+func TestLoadDefault_DefaultsIdleFetchBackoffMax(t *testing.T) {
+	cfg, err := LoadDefault("")
+	require.NoError(t, err)
+	assert.Equal(t, time.Minute, cfg.Runner.FetchIntervalMax)
+}
+
 func TestLoadDefault_UsesConfiguredWorkdirCleanupAge(t *testing.T) {
 	dir := t.TempDir()
 	path := filepath.Join(dir, "config.yaml")