Browse Source

Merge pull request #5441 from mgugino-upstream-stage/fix-reg-auth

Automatic merge from submit-queue

Fix registry auth task ordering

Currently, registry authentication credentials are not
produced until after docker systemd service files are
created.

This commit ensures the credentials are
created before the systemd service files to ensure
the proper boolean is set to include the read-only
mount of credentials inside containerized nodes and
masters.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1316341
OpenShift Merge Robot 7 years ago
parent
commit
fa9d0ee3e1
2 changed files with 4 additions and 4 deletions
  1. 2 2
      roles/openshift_master/tasks/main.yml
  2. 2 2
      roles/openshift_node/tasks/main.yml

+ 2 - 2
roles/openshift_master/tasks/main.yml

@@ -177,6 +177,8 @@
     local_facts:
       no_proxy_etcd_host_ips: "{{ openshift_no_proxy_etcd_host_ips }}"
 
+- include: registry_auth.yml
+
 - name: Install the systemd units
   include: systemd_units.yml
 
@@ -229,8 +231,6 @@
   - restart master controllers
   when: openshift_master_bootstrap_enabled | default(False)
 
-- include: registry_auth.yml
-
 - include: set_loopback_context.yml
   when:
   - openshift.common.version_gte_3_2_or_1_2

+ 2 - 2
roles/openshift_node/tasks/main.yml

@@ -69,12 +69,12 @@
   include: bootstrap.yml
   when: openshift_node_bootstrap
 
+- include: registry_auth.yml
+
 - name: include standard node config
   include: config.yml
   when: not openshift_node_bootstrap
 
-- include: registry_auth.yml
-
 - name: Configure AWS Cloud Provider Settings
   lineinfile:
     dest: /etc/sysconfig/{{ openshift.common.service_type }}-node