From 8a40a71e54a99bba1a0f821317fd4d7edf47a3d4 Mon Sep 17 00:00:00 2001
From: Ralph Bean <rbean@redhat.com>
Date: Mon, 17 Nov 2014 14:34:48 +0000
Subject: [PATCH] Try changing our use of changed_when.

---
 roles/collectd/fcomm-queue/tasks/main.yml | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/roles/collectd/fcomm-queue/tasks/main.yml b/roles/collectd/fcomm-queue/tasks/main.yml
index 2110b79793..4dd261e44b 100644
--- a/roles/collectd/fcomm-queue/tasks/main.yml
+++ b/roles/collectd/fcomm-queue/tasks/main.yml
@@ -39,24 +39,25 @@
 - name: check to see if its even installed yet
   shell: semodule -l | grep fi-collectd-fcomm
   register: selinux_grep
+  always_run: True
+  changed_when: selinux_grep.rc == 1
   tags:
   - collectd
   - selinux
-  ignore_errors: True
 
 - debug: var=selinux_grep
   tags:
   - collectd
   - selinux
 
-- debug: msg="{{selinux_grep|failed}}"
+- debug: msg="{{selinux_grep|changed}}"
   tags:
   - collectd
   - selinux
 
 - name: install our fcomm collectd selinux module
   command: semodule -i /usr/share/collectd/fi-collectd-fcomm.pp
-  when: selinux_module|changed or selinux_grep|skipped
+  when: selinux_module|changed or selinux_grep|changed
   tags:
   - collectd
   - selinux