From 08f61b11e3463530e61ea295ca1c3697ea9e5917 Mon Sep 17 00:00:00 2001 From: Emiliano Vavassori Date: Thu, 21 Jan 2016 21:43:16 +0100 Subject: [PATCH] Using 'is defined' check for addpkg/delpkg in npm checkings. --- roles/networkpackagemanager/tasks/main.yml | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/roles/networkpackagemanager/tasks/main.yml b/roles/networkpackagemanager/tasks/main.yml index 56a7e83..a9b0c49 100644 --- a/roles/networkpackagemanager/tasks/main.yml +++ b/roles/networkpackagemanager/tasks/main.yml @@ -5,18 +5,13 @@ - name: Updating package cache. apt: update_cache=yes cache_valid_time=180 -- name: Defining new variable to manage additional packages - set_fact: - str_addpkg: "{{ addpkg|join(' ')|trim != '' }}" - str_delpkg: "{{ delpkg|join(' ')|trim != '' }}" - - name: Installing packages for specific role from NetworkPackageManager apt: name: '{{ item }}' state: present become: yes with_items: "{{ addpkg }}" - when: str_addpkg + when: addpkg is defined - name: Removing packages for specific role from NetworkPackageManager apt: @@ -24,7 +19,7 @@ state: absent purge: yes with_items: "{{ delpkg }}" - when: str_delpkg + when: delpkg is defined - name: Updating local hash for packages shell: 'rm -f {{ ansible_local.checksum.pkgver_file }} && echo "{{ ansible_local.checksum.pkgver }}" > {{ ansible_local.checksum.pkgver_file }}'