From ecd9d1543fce0a7fff46ed55600ac66fbc23671c Mon Sep 17 00:00:00 2001
From: Jeremy Lecour <jlecour@evolix.fr>
Date: Mon, 21 Nov 2022 15:46:46 +0100
Subject: [PATCH] varnish: better package facts usage with check mode and tags

---
 CHANGELOG.md           |  1 +
 varnish/tasks/main.yml | 16 +++++++++++++++-
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 81a9ea7b..dac275af 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -31,6 +31,7 @@ The **patch** part changes is incremented if multiple releases happen the same m
 * redis: some values should be quoted
 * redis: variable to disable transparent hugepage (default: do nothing)
 * squid: whitelist deb.freexian.com
+* varnish: better package facts usage with check mode and tags
 * varnish: systemd override depends on Varnish version instead of Debian version
 
 ### Fixed
diff --git a/varnish/tasks/main.yml b/varnish/tasks/main.yml
index 67fe2d6e..f2545264 100644
--- a/varnish/tasks/main.yml
+++ b/varnish/tasks/main.yml
@@ -9,13 +9,27 @@
 - name: Fetch packages
   package_facts:
     manager: auto
+  check_mode: no
   tags:
     - varnish
+    - config
+    - update-config
 
 - set_fact:
-    varnish_package_facts: ansible_facts.packages['varnish'] | first
+    varnish_package_facts: "{{ ansible_facts.packages['varnish'] | first }}"
+  check_mode: no
   tags:
     - varnish
+    - config
+    - update-config
+
+# - debug:
+#     var: varnish_package_facts
+#   check_mode: no
+#   tags:
+#     - varnish
+#     - config
+#     - update-config
 
 - name: Remove default varnish configuration files
   file: