From 6da67c7364c94768cdc713e948ce4322a521ac12 Mon Sep 17 00:00:00 2001
From: Tim Bielawa <tbielawa@redhat.com>
Date: Wed, 4 Jan 2017 10:54:01 -0800
Subject: Add required python-six package to installation

* Moves package list into a defaults/main.yml file
* Closes #3020
---
 roles/openshift_facts/tasks/main.yml | 6 ++----
 roles/openshift_facts/vars/main.yml  | 7 +++++++
 2 files changed, 9 insertions(+), 4 deletions(-)
 create mode 100644 roles/openshift_facts/vars/main.yml

diff --git a/roles/openshift_facts/tasks/main.yml b/roles/openshift_facts/tasks/main.yml
index 70cf49dd4..b7b521f1a 100644
--- a/roles/openshift_facts/tasks/main.yml
+++ b/roles/openshift_facts/tasks/main.yml
@@ -10,11 +10,9 @@
 - set_fact:
     l_is_containerized: "{{ (l_is_atomic | bool) or (containerized | default(false) | bool) }}"
 
-- name: Ensure PyYaml and yum-utils are installed
+- name: Ensure various deps are installed
   package: name={{ item }} state=present
-  with_items:
-    - PyYAML
-    - yum-utils
+  with_items: "{{ required_packages }}"
   when: not l_is_atomic | bool
 
 - name: Gather Cluster facts and set is_containerized if needed
diff --git a/roles/openshift_facts/vars/main.yml b/roles/openshift_facts/vars/main.yml
new file mode 100644
index 000000000..9c3110ff6
--- /dev/null
+++ b/roles/openshift_facts/vars/main.yml
@@ -0,0 +1,7 @@
+---
+required_packages:
+  - iproute
+  - python-dbus
+  - python-six
+  - PyYAML
+  - yum-utils
-- 
cgit v1.2.3