From dad2c05b7ebf2463f31d8d541be53f1bb5987f95 Mon Sep 17 00:00:00 2001
From: Lars Beckers <lars.beckers@rwth-aachen.de>
Date: Wed, 11 Sep 2019 16:34:20 +0200
Subject: [PATCH] php-fpm: name task; fix yamllint issues; fix shell module
 usage

---
 php-fpm/tasks/main.yml | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/php-fpm/tasks/main.yml b/php-fpm/tasks/main.yml
index b0353e8..ab2c8d3 100644
--- a/php-fpm/tasks/main.yml
+++ b/php-fpm/tasks/main.yml
@@ -1,6 +1,7 @@
 ---
 
-- include_vars: "{{ debian_version }}.yml"
+- name: include debian version specific vars
+  include_vars: "{{ debian_version }}.yml"
 
 - name: ensure php-fpm is installed
   apt:
@@ -46,7 +47,13 @@
     - restart php-fpm
 
 - name: get remote active php pools
-  shell: "systemctl list-units --state=loaded | grep uwsgi@ | sed -E 's/.*uwsgi@(.*)\\.service.*/\\1/'"
+  # yamllint disable rule:line-length
+  shell: |
+    set -o pipefail
+    systemctl list-units --state=loaded | grep uwsgi@ | sed -E 's/.*uwsgi@(.*)\\.service.*/\\1/'
+  args:
+    executable: /bin/bash
+  # yamllint enable rule:line-length
   changed_when: false
   register: running_pools
 
@@ -55,6 +62,7 @@
     name: "php-fpm@{{ item }}"
     enabled: false
     state: stopped
+  # yamllint disable-line rule:line-length
   with_items: "{{ running_pools.stdout_lines|difference(fpm_pools|map(attribute=name))|list }}"
 
 - meta: flush_handlers
-- 
GitLab