Merge pull request #28 from Porkepix/fix-check_mode

Fix check mode
cloudalchemy · Mar 30, 2018 · a4f8c3b · a4f8c3b
2 parents 2340ae0 + 7f5ac16
commit a4f8c3b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 2 deletions.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -45,7 +45,7 @@ We try to provide production ready ansible roles which should be as much zero-co
 
 ### Add tests
 
-We try to have as much tests written in testinfra framework as possible, so when you copy file, some template or starting some server just add couple of lines in [/tests/test)default.py](test_default.py) file. If you want to know how to write tests in testinfra, go to their [docs](http://testinfra.readthedocs.io/en/latest/index.html).
+We try to have as much tests written in testinfra framework as possible, so when you copy file, some template or starting some server just add couple of lines in [/tests/test_default.py](test_default.py) file. If you want to know how to write tests in testinfra, go to their [docs](http://testinfra.readthedocs.io/en/latest/index.html).
 
 ### Follow best practices
 

diff --git a/README.md b/README.md
@@ -15,7 +15,6 @@ Deploy prometheus [node exporter](https://github.com/prometheus/node_exporter) u
 ## Requirements
 
 - Ansible >= 2.3
-- go-lang installed on deployer machine (same one where ansible is installed)
 
 ## Role Variables
 

diff --git a/tasks/main.yml b/tasks/main.yml
@@ -35,6 +35,7 @@
     group: "node-exp"
   notify:
     - restart node exporter
+  when: not ansible_check_mode
 
 - name: Install libcap on Debian systems
   package: