From 022bf9440f62e028c104066e814b98a3e2ffe7ae Mon Sep 17 00:00:00 2001
From: John <hello@johnthenerd.com>
Date: Thu, 30 Jul 2020 13:32:00 -0230
Subject: [PATCH] turned reboot into a handler

---
 roles/raspbian/tasks/main.yml | 10 ++--------
 site.yml                      |  3 +++
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/roles/raspbian/tasks/main.yml b/roles/raspbian/tasks/main.yml
index 707d37d..04c931a 100644
--- a/roles/raspbian/tasks/main.yml
+++ b/roles/raspbian/tasks/main.yml
@@ -14,7 +14,7 @@
     regexp: '^((?!.*\bcgroup_enable=cpuset cgroup_memory=1 cgroup_enable=memory\b).*)$'
     line: '\1 cgroup_enable=cpuset cgroup_memory=1 cgroup_enable=memory'
     backrefs: true
-  register: cgroup
+  notify: reboot
   when:
     - raspbian is true
 
@@ -22,6 +22,7 @@
   iptables:
     flush: true
   when: raspbian
+  changed_when: false   # iptables flush always returns changed
 
 - name: Changing to iptables-legacy
   alternatives:
@@ -36,10 +37,3 @@
     name: ip6tables
   register: ip6_legacy
   when: raspbian
-
-- name: Rebooting
-  reboot:
-  when:
-    - cgroup.changed
-  tags:
-    - skip_ansible_lint
diff --git a/site.yml b/site.yml
index 8862e96..46a24bd 100644
--- a/site.yml
+++ b/site.yml
@@ -8,6 +8,9 @@
     - role: download
     - role: raspbian
     - role: ubuntu
+  handlers:
+    - name: reboot
+      reboot:
 
 - hosts: master
   become: yes