Merge pull request #51 from AdriMarteau/fix/airflow_extra

fix warnings with lists install and jinja

CHANGELOG.md

@@ -3,6 +3,8 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/) and [Keep a changelog](https://github.com/olivierlacan/keep-a-changelog).
 
 ## [Unreleased](https://github.com/idealista/airflow-role/tree/develop)
+### Fixed
+- *[#47](https://github.com/idealista/airflow-role/issues/50) Fix deprecation warning from jinja templates @adrimarteau
 
 ## [1.7.3](https://github.com/idealista/airflow-role/tree/1.7.3)
 [Full Changelog](https://github.com/idealista/airflow-role/compare/1.7.2...1.7.3)

tasks/install.yml

@@ -26,10 +26,9 @@
 
 - name: Airflow | Installing dependencies
   apt:
-    pkg: "{{ item }}"
+    pkg: "{{ airflow_required_libs }}"
     state: present
     update_cache: yes
-  with_items: "{{ airflow_required_libs }}"
 
 - name: Airflow | Installing Python pip dependencies
   pip:
@@ -71,9 +70,8 @@
 - name: Airflow | Installing Airflow Extra Packages
   pip:
     executable: "{{ airflow_pip_executable }}"
-    name: apache-airflow[{{ item }}]
+    name: apache-airflow[{{ airflow_extra_packages }}]
     version: "{{ airflow_version }}"
-  with_items: "{{ airflow_extra_packages }}"
   when: airflow_extra_packages
 
 - name: Airflow | Installing DAGs dependencies
@@ -104,4 +102,4 @@
     mode: 0644
   notify: restart {{ item.key }}
   with_dict: "{{ airflow_services }}"
-  when: "{{ item.value.enabled }}"
+  when: item.value.enabled

tasks/service.yml

@@ -7,5 +7,5 @@
     enabled: "{{ item.value.enabled }}"
     daemon_reload: yes
   with_dict: "{{ airflow_services }}"
-  when: "{{ item.value.enabled }}"
+  when: item.value.enabled
   changed_when: false