From d6a12ac03398ecc752139908688b82aa473c29ef Mon Sep 17 00:00:00 2001
From: Michael Gugino <mgugino@redhat.com>
Date: Mon, 20 Nov 2017 18:05:13 -0500
Subject: Remove is_openvswitch_system_container from facts

Simply logic for this variable.
---
 playbooks/init/facts.yml                     | 2 --
 roles/openshift_node/defaults/main.yml       | 1 +
 roles/openshift_node/tasks/systemd_units.yml | 4 ++--
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/playbooks/init/facts.yml b/playbooks/init/facts.yml
index 7439313b7..be3984fae 100644
--- a/playbooks/init/facts.yml
+++ b/playbooks/init/facts.yml
@@ -29,7 +29,6 @@
   - name: initialize_facts set fact for containerized and l_is_*_system_container
     set_fact:
       l_is_containerized: "{{ (l_is_atomic | bool) or (containerized | default(false) | bool) }}"
-      l_is_openvswitch_system_container: "{{ (openshift_use_openvswitch_system_container | default(openshift_use_system_containers | default(false)) | bool) }}"
       l_is_master_system_container: "{{ (openshift_use_master_system_container | default(openshift_use_system_containers | default(false)) | bool) }}"
 
   # TODO: Should this be moved into health checks??
@@ -113,7 +112,6 @@
         hostname: "{{ openshift_hostname | default(None) }}"
         ip: "{{ openshift_ip | default(None) }}"
         is_containerized: "{{ l_is_containerized | default(None) }}"
-        is_openvswitch_system_container: "{{ l_is_openvswitch_system_container | default(false) }}"
         is_master_system_container: "{{ l_is_master_system_container | default(false) }}"
         public_hostname: "{{ openshift_public_hostname | default(None) }}"
         public_ip: "{{ openshift_public_ip | default(None) }}"
diff --git a/roles/openshift_node/defaults/main.yml b/roles/openshift_node/defaults/main.yml
index d249278dc..62208c155 100644
--- a/roles/openshift_node/defaults/main.yml
+++ b/roles/openshift_node/defaults/main.yml
@@ -14,6 +14,7 @@ system_images_registry_dict:
   origin: "docker.io"
 
 system_images_registry: "{{ system_images_registry_dict[openshift_deployment_type | default('origin')] }}"
+l_is_openvswitch_system_container: "{{ (openshift_use_openvswitch_system_container | default(openshift_use_system_containers | default(false)) | bool) }}"
 
 openshift_image_tag: ''
 
diff --git a/roles/openshift_node/tasks/systemd_units.yml b/roles/openshift_node/tasks/systemd_units.yml
index 9b4c24dfe..2344e33ca 100644
--- a/roles/openshift_node/tasks/systemd_units.yml
+++ b/roles/openshift_node/tasks/systemd_units.yml
@@ -25,7 +25,7 @@
     include: openvswitch_system_container.yml
     when:
     - openshift_node_use_openshift_sdn | bool
-    - openshift.common.is_openvswitch_system_container | bool
+    - l_is_openvswitch_system_container | bool
 
 - block:
   - name: Pre-pull openvswitch image
@@ -38,7 +38,7 @@
   when:
   - openshift.common.is_containerized | bool
   - openshift_node_use_openshift_sdn | bool
-  - not openshift.common.is_openvswitch_system_container | bool
+  - not l_is_openvswitch_system_container | bool
 
 - include: config/configure-node-settings.yml
 - include: config/configure-proxy-settings.yml
-- 
cgit v1.2.3