Fix check mode (#38)

Author: alessfg

CHANGELOG.md

@@ -17,6 +17,7 @@ BUG FIXES:
 *   Prevent TravisCI from trying to build (and failing) NGINX Plus images on external PRs.
 *   Fix naming for SELinux Ansible facts.
 *   Correctly import `app_protect` global directives in template.
+*   Role now runs correctly when using Ansible's check mode.
 
 ## 0.2.0 (September 24, 2020)
 

handlers/main.yml

@@ -1,16 +1,19 @@
 ---
 - name: (Handler - NGINX Config) Check NGINX
   command: nginx -t
-  register: config
+  register: config_check
   ignore_errors: yes
+  check_mode: no
   changed_when: false
   listen: (Handler - NGINX Config) Run NGINX
 
 - name: (Handler - NGINX Config) Print NGINX error if syntax check fails
   debug:
-    var: config.stderr_lines
-  failed_when: config.rc != 0
-  when: config.rc != 0
+    var: config_check.stderr_lines
+  failed_when: config_check.rc != 0
+  when:
+    - config_check.stderr_lines is defined
+    - config_check.rc != 0
   listen: (Handler - NGINX Config) Run NGINX
 
 - name: (Handler - NGINX Config) Start/reload NGINX

tasks/config/debug-output.yml

@@ -1,9 +1,12 @@
 ---
 - name: Register NGINX config
   command: nginx -T
+  ignore_errors: "{{ ansible_check_mode }}"
+  check_mode: no
   changed_when: false
-  register: config
+  register: config_full
 
 - name: Print NGINX config
   debug:
-    var: config.stdout_lines
+    var: config_full.stdout_lines
+  when: config_full.stdout_lines is defined