From 1d2964a1d8f5f9a15a88a0b5fdf116439c9c537b Mon Sep 17 00:00:00 2001
From: David Mat <david.mat@archimiddle.com>
Date: Tue, 23 Feb 2016 16:36:01 +0100
Subject: Add correct parsing of ec2_security_groups env variable

---
 .../aws/openshift-cluster/tasks/launch_instances.yml      | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

(limited to 'playbooks/aws')

diff --git a/playbooks/aws/openshift-cluster/tasks/launch_instances.yml b/playbooks/aws/openshift-cluster/tasks/launch_instances.yml
index 6090ed6fe..63be06ecf 100644
--- a/playbooks/aws/openshift-cluster/tasks/launch_instances.yml
+++ b/playbooks/aws/openshift-cluster/tasks/launch_instances.yml
@@ -34,26 +34,22 @@
 
 - set_fact:
     ec2_instance_type: "{{ ec2_master_instance_type | default(lookup('env', 'ec2_master_instance_type') | default(lookup('env', 'ec2_instance_type') | default(deployment_vars[deployment_type].type, true), true), true) }}"
-    ec2_security_groups: "{{ ec2_master_security_groups
-                    | default(deployment_vars[deployment_type].security_groups, true) }}"
+    ec2_security_groups: "{{ ec2_master_security_groups | default(lookup('env', 'ec2_master_security_groups') | default(lookup('env', 'ec2_security_groups') | default(deployment_vars[deployment_type].security_groups, true), true), true) }}"
   when: host_type == "master" and sub_host_type == "default"
 
 - set_fact:
     ec2_instance_type: "{{ ec2_etcd_instance_type | default(lookup('env', 'ec2_etcd_instance_type') | default(lookup('env', 'ec2_instance_type') | default(deployment_vars[deployment_type].type, true), true), true) }}"
-    ec2_security_groups: "{{ ec2_etcd_security_groups
-                    | default(deployment_vars[deployment_type].security_groups, true)}}"
+    ec2_security_groups: "{{ ec2_etcd_security_groups | default(lookup('env', 'ec2_etcd_security_groups') | default(lookup('env', 'ec2_security_groups') | default(deployment_vars[deployment_type].security_groups, true), true), true) }}"
   when: host_type == "etcd" and sub_host_type == "default"
 
 - set_fact:
     ec2_instance_type: "{{ ec2_infra_instance_type | default(lookup('env', 'ec2_infra_instance_type') | default(lookup('env', 'ec2_instance_type') | default(deployment_vars[deployment_type].type, true), true), true) }}"
-    ec2_security_groups: "{{ ec2_infra_security_groups
-                    | default(deployment_vars[deployment_type].security_groups, true) }}"
+    ec2_security_groups: "{{ ec2_infra_security_groups | default(lookup('env', 'ec2_infra_security_groups') | default(lookup('env', 'ec2_security_groups') | default(deployment_vars[deployment_type].security_groups, true), true), true) }}"
   when: host_type == "node" and sub_host_type == "infra"
 
 - set_fact:
     ec2_instance_type: "{{ ec2_node_instance_type | default(lookup('env', 'ec2_node_instance_type') | default(lookup('env', 'ec2_instance_type') | default(deployment_vars[deployment_type].type, true), true), true) }}"
-    ec2_security_groups: "{{ ec2_node_security_groups
-                    | default(deployment_vars[deployment_type].security_groups, true) }}"
+    ec2_security_groups: "{{ ec2_node_security_groups | default(lookup('env', 'ec2_node_security_groups') | default(lookup('env', 'ec2_security_groups') | default(deployment_vars[deployment_type].security_groups, true), true), true) }}"
   when: host_type == "node" and sub_host_type == "compute"
 
 - set_fact:
@@ -61,8 +57,7 @@
                           | default(deployment_vars[deployment_type].type, true) }}"
   when: ec2_instance_type is not defined
 - set_fact:
-    ec2_security_groups: "{{ lookup('env', 'ec2_security_groups')
-                    | default(deployment_vars[deployment_type].security_groups, true) }}"
+    ec2_security_groups: "{{ lookup('env', 'ec2_security_groups') | default(deployment_vars[deployment_type].security_groups, true) }}"
   when: ec2_security_groups is not defined
 
 - name: Find amis for deployment_type
-- 
cgit v1.2.3