From 3822f6fe9aba4b37f2e862eff31660ffeabc2e0a Mon Sep 17 00:00:00 2001 From: Emil Simeonov Date: Sun, 21 Jun 2026 00:34:07 +0200 Subject: [PATCH] fix: add default(0) to gitea_runner_uid in environment blocks Ansible evaluates environment blocks even when when conditions are false. The deregister scenario sets skip_runner_registration: true but the environment block still references gitea_runner_uid, causing "variable is undefined" errors. Add default(0) filter to prevent this. Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com> --- ansible/roles/gitea-runner/tasks/deregister.yml | 4 ++-- ansible/roles/gitea-runner/tasks/register.yml | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/ansible/roles/gitea-runner/tasks/deregister.yml b/ansible/roles/gitea-runner/tasks/deregister.yml index d846667..51de648 100644 --- a/ansible/roles/gitea-runner/tasks/deregister.yml +++ b/ansible/roles/gitea-runner/tasks/deregister.yml @@ -30,8 +30,8 @@ become: true become_user: "{{ gitea_runner_service_user }}" environment: - XDG_RUNTIME_DIR: "/run/user/{{ gitea_runner_uid }}" - DOCKER_HOST: "unix:///run/user/{{ gitea_runner_uid }}/docker.sock" + XDG_RUNTIME_DIR: "/run/user/{{ gitea_runner_uid | default(0) }}" + DOCKER_HOST: "unix:///run/user/{{ gitea_runner_uid | default(0) }}/docker.sock" when: - runner_file_stat.stat.exists | default(false) | bool - not skip_runner_registration diff --git a/ansible/roles/gitea-runner/tasks/register.yml b/ansible/roles/gitea-runner/tasks/register.yml index 427549e..470e8c4 100644 --- a/ansible/roles/gitea-runner/tasks/register.yml +++ b/ansible/roles/gitea-runner/tasks/register.yml @@ -25,8 +25,8 @@ become: true become_user: "{{ gitea_runner_service_user }}" environment: - XDG_RUNTIME_DIR: "/run/user/{{ gitea_runner_uid }}" - DOCKER_HOST: "unix:///run/user/{{ gitea_runner_uid }}/docker.sock" + XDG_RUNTIME_DIR: "/run/user/{{ gitea_runner_uid | default(0) }}" + DOCKER_HOST: "unix:///run/user/{{ gitea_runner_uid | default(0) }}/docker.sock" when: not runner_registered.stat.exists register: register_output changed_when: "'already exists' not in register_output.stdout | default('')"