Browse Source

Merge pull request #5740 from mgugino-upstream-stage/ensure_docker_containerized_lb

Automatic merge from submit-queue.

Ensure docker is installed for containerized load balancers

Currently, containerized load balancers may not have docker
configured by openshift-ansible.

This commit ensures that containerized load balancers have
the openshift_docker role applied.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1496756
OpenShift Merge Robot 7 years ago
parent
commit
a36fec164e
1 changed files with 9 additions and 1 deletions
  1. 9 1
      playbooks/common/openshift-loadbalancer/config.yml

+ 9 - 1
playbooks/common/openshift-loadbalancer/config.yml

@@ -9,6 +9,15 @@
         installer_phase_loadbalancer: "In Progress"
       aggregate: false
 
+- name: Configure firewall and docker for load balancers
+  hosts: oo_lb_to_config:!oo_masters_to_config:!oo_nodes_to_config
+  vars:
+    openshift_image_tag: "{{ hostvars[groups.oo_first_master.0].openshift_image_tag }}"
+  roles:
+  - role: os_firewall
+  - role: openshift_docker
+    when: openshift.common.is_containerized | default(False) | bool and not skip_docker_role | default(False) | bool
+
 - name: Configure load balancers
   hosts: oo_lb_to_config
   vars:
@@ -24,7 +33,6 @@
                                           + openshift_loadbalancer_additional_backends | default([]) }}"
     openshift_image_tag: "{{ hostvars[groups.oo_first_master.0].openshift_image_tag }}"
   roles:
-  - role: os_firewall
   - role: openshift_loadbalancer
   - role: tuned