removed el_patching_check_mode variable to follow standard playbook check mode

Author: voidquark

README.md

@@ -24,7 +24,6 @@ Only dnf must be available on the target machine.
 | `el_patching_auto_reboot` | `false` | By default do not reboot the target host. Only verify if a reboot is required.
 | `el_patching_reboot_timeout` | `600` | By default auto reboot is disabled but the default timeout value is set to 5 minutes. Value is in `seconds`.
 | `el_patching_method` | `"security"` | By default apply only `security` patches on the target host. Possible values `"security"/"bugfix"/"all"`
-| `el_patching_check_mode` | `false` | By default task is not executed in check mode. You can enable check mode to simulate patching. Keep in mind that check mode can't predict if a reboot is required. It only shows you how this role is executed on multiple/single hosts.
 
 - **group_vars** or **host_vars** variables.
 
@@ -47,6 +46,19 @@ Create the following playbook.
     - el_patching
 ```
 
+## Example execution
+
+- Normal Execution 
+```shell
+ansible-playbook -i inventory/hosts playbook.yml
+```
+
+- If you want to run playbook in check mode
+```shell
+ansible-playbook -i inventory/hosts playbook.yml --check
+```
+
+
 ## License
 
 MIT

defaults/main.yml

@@ -4,7 +4,6 @@ el_patching_required_packages: "yum-utils"
 el_patching_reboot_timeout: 600
 el_patching_method: "security"
 el_patching_auto_reboot: false
-el_patching_check_mode: false
 
 # Optional exclude variable for group_vars or host_vars
 # el_patching_exclude_packages:

tasks/main.yml

@@ -11,7 +11,6 @@
     name: "*"
     state: latest
     exclude: "{{ el_patching_exclude_packages | default(omit) }}"
-  check_mode: "{{ el_patching_check_mode }}"
   when: el_patching_method == "all"
 
 - name: Apply security patches only
@@ -20,7 +19,6 @@
     security: true
     state: latest
     exclude: "{{ el_patching_exclude_packages | default(omit) }}"
-  check_mode: "{{ el_patching_check_mode }}"
   when: el_patching_method == "security"
 
 - name: Apply bugfix patches only
@@ -29,7 +27,6 @@
     bugfix: true
     state: latest
     exclude: "{{ el_patching_exclude_packages | default(omit) }}"
-  check_mode: "{{ el_patching_check_mode }}"
   when: el_patching_method == "bugfix"
 
 - name: Verify if restart is required
@@ -46,5 +43,4 @@
 - name: Reboot host
   ansible.builtin.reboot:
     reboot_timeout: "{{ el_patching_reboot_timeout }}"
-  check_mode: "{{ el_patching_check_mode }}"
   when: el_patching_auto_reboot | bool and __el_patching_need_restart.rc == 1