Skip to content

Commit 553ce81

Browse files
carlfriedrichguenhter
authored andcommitted
fix: Prevent config file dir created with conflicting permissions
The task for creating the system-level configuration directory with permissions 0700 was running unconditionally, even during container-based installations. This caused a file permission conflict with the container mount point setup, which already created this directory with 0755 permission, leading to two recurring changed states on every playbook run. Adding a conditional ensures this task is now skipped when `gitlab_runner_container_install` is true, resolving the idempotency issue.
1 parent 4929c82 commit 553ce81

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

tasks/global-setup.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
state: directory
66
mode: "0700"
77
become: "{{ gitlab_runner_system_mode }}"
8+
when: not gitlab_runner_container_install
89

910
- name: Ensure config.toml exists
1011
ansible.builtin.file:

0 commit comments

Comments
 (0)