Skip to content

Commit 580e918

Browse files
author
Christopher M. Neill
committed
Ansible won't parse a variable not defined even if the directive is to do nothing if not defined. Ok.
Use the "user" module.
1 parent f09419e commit 580e918

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

ansible/redis-slave.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@
99
- { role: container_kill_start,
1010
container_run_args: "redis-server --slaveof {{ redis_host_address }} 6379",
1111
tags: "deploy" }
12+
- { role: datadog }

ansible/roles/datadog/tasks/main.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,10 @@
3232
- name: add docker to dd-agent group
3333
tags: ['datadog']
3434
sudo: yes
35-
when: not no_dd_monitoring
36-
shell: usermod -a -G docker dd-agent
35+
when: no_dd_monitoring != false
36+
user:
37+
name=dd-agent
38+
groups=docker
3739

3840
- name: create dd configdir
3941
tags: ['datadog']
@@ -64,7 +66,7 @@
6466
- name: install docker integration
6567
tags: ['datadog']
6668
sudo: yes
67-
when: not no_dd_monitoring
69+
when: no_dd_monitoring != false
6870
template:
6971
src=docker_daemon.yaml.j2
7072
dest=/etc/dd-agent/conf.d/docker_daemon.yaml

0 commit comments

Comments
 (0)