Skip to content

Commit dd18ffe

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Fix Grafana role support for Ansible check mode"
2 parents b575c28 + b59fe6c commit dd18ffe

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

ansible/roles/grafana/tasks/config.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,9 @@
100100
file:
101101
state: absent
102102
path: "{{ node_config_directory }}/grafana/dashboards/"
103-
when: service | service_enabled_and_mapped_to_host
103+
when:
104+
- not ansible_check_mode
105+
- service | service_enabled_and_mapped_to_host
104106

105107
- name: Copying over custom dashboards
106108
vars:

ansible/roles/grafana/tasks/post_config.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
retries: 30
1313
delay: 2
1414
run_once: true
15+
when: not ansible_check_mode
1516

1617
- name: Remove old grafana docker volume
1718
become: true

0 commit comments

Comments
 (0)