From c4b14c0b89b2aba05ad2d0d6f058588a1366bd0d Mon Sep 17 00:00:00 2001 From: Erich Blume Date: Sun, 25 Jan 2026 19:40:33 -0800 Subject: [PATCH] Move DOCKER_HOST to runner config instead of workflow The runner's container.env config sets environment variables in all job containers, so we don't need to specify DOCKER_HOST per-workflow. Co-Authored-By: Claude Opus 4.5 --- .forgejo/workflows/build-container.yaml | 3 --- argocd/manifests/forgejo-runner/configmap.yaml | 3 +++ 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.forgejo/workflows/build-container.yaml b/.forgejo/workflows/build-container.yaml index 3bc1efd..59291f0 100644 --- a/.forgejo/workflows/build-container.yaml +++ b/.forgejo/workflows/build-container.yaml @@ -16,9 +16,6 @@ on: jobs: build: runs-on: k8s - env: - # For k8s runner with DinD sidecar, Docker is available via TCP - DOCKER_HOST: tcp://127.0.0.1:2375 steps: - name: Parse tag id: parse diff --git a/argocd/manifests/forgejo-runner/configmap.yaml b/argocd/manifests/forgejo-runner/configmap.yaml index 860f96b..7d8ddef 100644 --- a/argocd/manifests/forgejo-runner/configmap.yaml +++ b/argocd/manifests/forgejo-runner/configmap.yaml @@ -18,3 +18,6 @@ data: network: "host" # Connect to DinD sidecar via TCP (not socket) docker_host: tcp://127.0.0.1:2375 + # Set DOCKER_HOST in job containers so they can run docker commands + env: + DOCKER_HOST: tcp://127.0.0.1:2375