Merge pull request #9 from m4rcu5nl/develop

Bug fixes

README.md

@@ -13,24 +13,24 @@ This roles requires an access token for the Zerotier API. This enables the role
 Role Variables
 --------------
 
-###zerotier_api_url
+### zerotier_api_url
 The url where the Zerotier API lives. Must use https protocol.    
 Default: https://my.zerotier.com
 
-###zerotier_accesstoken
+### zerotier_accesstoken
 The access token needed to authorize with the Zerotier API. You can generate one in your account settings on my.zerotier.com.
 
-###zerotier_network_id (required)
+### zerotier_network_id (required)
 The 16 character network ID of the network the new members should join.
 
 Example Playbook
 ----------------
 
-
+```yaml
     - hosts: servers
       vars:
          zerotier_network_id: 1234567890qwerty
          zerotier_accesstoken: "{{ vault_zerotier_accesstoken }}"
       roles:
          - { role: m4rcu5nl.zerotier }
-
+```

tasks/install.yml

@@ -36,6 +36,7 @@
 
   when:
   - zerotier_repo|succeeded
+  - not ansible_check_mode
   tags:
   - installation
   - packages

tasks/join_network.yml

@@ -29,5 +29,6 @@
 
   when:
   - zerotier_accesstoken is defined
+  - not ansible_check_mode
   tags:
   - configuration

tasks/main.yml

@@ -1,12 +1,13 @@
 ---
 # tasks file for ansible-role-zerotier
-- include: install.yml
+- import_tasks: install.yml
 
 - name: Check for successfully joined networks
   shell: zerotier-cli listnetworks | grep 'OK'| awk '{print $3}'
   register: joinednetworks
+  check_mode: yes
 
-- include: join_network.yml
+- include_tasks: join_network.yml
   when:
   - 'zerotier_network_id not in joinednetworks.stdout'
   - zerotier_accesstoken is defined