diff options
Diffstat (limited to 'roles')
85 files changed, 200 insertions, 200 deletions
| diff --git a/roles/cockpit/tasks/main.yml b/roles/cockpit/tasks/main.yml index 066ee3f3b..34754502a 100644 --- a/roles/cockpit/tasks/main.yml +++ b/roles/cockpit/tasks/main.yml @@ -1,7 +1,6 @@  ---  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Install cockpit-ws    package: name={{ item }} state=present diff --git a/roles/contiv/tasks/main.yml b/roles/contiv/tasks/main.yml index 40a0f9e61..cb9196a71 100644 --- a/roles/contiv/tasks/main.yml +++ b/roles/contiv/tasks/main.yml @@ -5,10 +5,10 @@      recurse: yes      state: directory -- include: download_bins.yml +- include_tasks: download_bins.yml -- include: netmaster.yml +- include_tasks: netmaster.yml    when: contiv_role == "netmaster" -- include: netplugin.yml +- include_tasks: netplugin.yml    when: contiv_role == "netplugin" diff --git a/roles/contiv/tasks/netmaster.yml b/roles/contiv/tasks/netmaster.yml index cc52d3a43..6f15af8c2 100644 --- a/roles/contiv/tasks/netmaster.yml +++ b/roles/contiv/tasks/netmaster.yml @@ -1,8 +1,8 @@  --- -- include: netmaster_firewalld.yml +- include_tasks: netmaster_firewalld.yml    when: has_firewalld -- include: netmaster_iptables.yml +- include_tasks: netmaster_iptables.yml    when: not has_firewalld and has_iptables  - name: Netmaster | Check is /etc/hosts file exists @@ -70,8 +70,8 @@      state: started    register: netmaster_started -- include: aci.yml +- include_tasks: aci.yml    when: contiv_fabric_mode == "aci" -- include: default_network.yml +- include_tasks: default_network.yml    when: contiv_default_network == true diff --git a/roles/contiv/tasks/netplugin.yml b/roles/contiv/tasks/netplugin.yml index e861a2591..0b2f91bab 100644 --- a/roles/contiv/tasks/netplugin.yml +++ b/roles/contiv/tasks/netplugin.yml @@ -1,8 +1,8 @@  --- -- include: netplugin_firewalld.yml +- include_tasks: netplugin_firewalld.yml    when: has_firewalld -- include: netplugin_iptables.yml +- include_tasks: netplugin_iptables.yml    when: has_iptables  - name: Netplugin | Ensure localhost entry correct in /etc/hosts @@ -19,7 +19,7 @@      line: '::1 '      state: absent -- include: ovs.yml +- include_tasks: ovs.yml    when: netplugin_driver == "ovs"  - name: Netplugin | Create Netplugin bin symlink diff --git a/roles/contiv/tasks/ovs.yml b/roles/contiv/tasks/ovs.yml index 0c1b994c7..5c92e90e9 100644 --- a/roles/contiv/tasks/ovs.yml +++ b/roles/contiv/tasks/ovs.yml @@ -1,5 +1,5 @@  --- -- include: packageManagerInstall.yml +- include_tasks: packageManagerInstall.yml    when: source_type == "packageManager"    tags:      - binary-update diff --git a/roles/contiv/tasks/packageManagerInstall.yml b/roles/contiv/tasks/packageManagerInstall.yml index e0d48e643..d5726476c 100644 --- a/roles/contiv/tasks/packageManagerInstall.yml +++ b/roles/contiv/tasks/packageManagerInstall.yml @@ -3,7 +3,7 @@    set_fact:      did_install: false -- include: pkgMgrInstallers/centos-install.yml +- include_tasks: pkgMgrInstallers/centos-install.yml    when: (ansible_os_family == "RedHat") and          not is_atomic diff --git a/roles/contiv_facts/tasks/main.yml b/roles/contiv_facts/tasks/main.yml index 7a4972fca..3267a4ab0 100644 --- a/roles/contiv_facts/tasks/main.yml +++ b/roles/contiv_facts/tasks/main.yml @@ -81,8 +81,8 @@      has_iptables: false  # collect information about what packages are installed -- include: rpm.yml +- include_tasks: rpm.yml    when: has_rpm -- include: fedora-install.yml +- include_tasks: fedora-install.yml    when: not is_atomic and ansible_distribution == "Fedora" diff --git a/roles/etcd/tasks/backup.archive.yml b/roles/etcd/tasks/backup.archive.yml index 6daa6dc51..a29a90ea3 100644 --- a/roles/etcd/tasks/backup.archive.yml +++ b/roles/etcd/tasks/backup.archive.yml @@ -1,3 +1,3 @@  --- -- include: backup/vars.yml -- include: backup/archive.yml +- include_tasks: backup/vars.yml +- include_tasks: backup/archive.yml diff --git a/roles/etcd/tasks/backup.copy.yml b/roles/etcd/tasks/backup.copy.yml index cc540cbca..6e8502e3b 100644 --- a/roles/etcd/tasks/backup.copy.yml +++ b/roles/etcd/tasks/backup.copy.yml @@ -1,3 +1,3 @@  --- -- include: backup/vars.yml -- include: backup/copy.yml +- include_tasks: backup/vars.yml +- include_tasks: backup/copy.yml diff --git a/roles/etcd/tasks/backup.fetch.yml b/roles/etcd/tasks/backup.fetch.yml index 26ec15043..d33878804 100644 --- a/roles/etcd/tasks/backup.fetch.yml +++ b/roles/etcd/tasks/backup.fetch.yml @@ -1,3 +1,3 @@  --- -- include: backup/vars.yml -- include: backup/fetch.yml +- include_tasks: backup/vars.yml +- include_tasks: backup/fetch.yml diff --git a/roles/etcd/tasks/backup.force_new_cluster.yml b/roles/etcd/tasks/backup.force_new_cluster.yml index d2e866416..7dd0899ee 100644 --- a/roles/etcd/tasks/backup.force_new_cluster.yml +++ b/roles/etcd/tasks/backup.force_new_cluster.yml @@ -1,5 +1,5 @@  --- -- include: backup/vars.yml +- include_tasks: backup/vars.yml  - name: Move content of etcd backup under the etcd data directory    command: > @@ -9,4 +9,4 @@    command: >      chown -R etcd:etcd "{{ etcd_data_dir }}" -- include: auxiliary/force_new_cluster.yml +- include_tasks: auxiliary/force_new_cluster.yml diff --git a/roles/etcd/tasks/backup.unarchive.yml b/roles/etcd/tasks/backup.unarchive.yml index 77a637360..f92e87c3d 100644 --- a/roles/etcd/tasks/backup.unarchive.yml +++ b/roles/etcd/tasks/backup.unarchive.yml @@ -1,3 +1,3 @@  --- -- include: backup/vars.yml -- include: backup/unarchive.yml +- include_tasks: backup/vars.yml +- include_tasks: backup/unarchive.yml diff --git a/roles/etcd/tasks/backup.yml b/roles/etcd/tasks/backup.yml index c0538e596..60bb82100 100644 --- a/roles/etcd/tasks/backup.yml +++ b/roles/etcd/tasks/backup.yml @@ -1,2 +1,2 @@  --- -- include: backup/backup.yml +- include_tasks: backup/backup.yml diff --git a/roles/etcd/tasks/backup/backup.yml b/roles/etcd/tasks/backup/backup.yml index ca0d29155..afb84eb58 100644 --- a/roles/etcd/tasks/backup/backup.yml +++ b/roles/etcd/tasks/backup/backup.yml @@ -1,5 +1,5 @@  --- -- include: vars.yml +- include_tasks: vars.yml  # TODO: replace shell module with command and update later checks  - name: Check available disk space for etcd backup diff --git a/roles/etcd/tasks/backup_ca_certificates.yml b/roles/etcd/tasks/backup_ca_certificates.yml index a41b032f3..c87359900 100644 --- a/roles/etcd/tasks/backup_ca_certificates.yml +++ b/roles/etcd/tasks/backup_ca_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/backup_ca_certificates.yml +- include_tasks: certificates/backup_ca_certificates.yml diff --git a/roles/etcd/tasks/backup_generated_certificates.yml b/roles/etcd/tasks/backup_generated_certificates.yml index 8cf2a10cc..fa73ea590 100644 --- a/roles/etcd/tasks/backup_generated_certificates.yml +++ b/roles/etcd/tasks/backup_generated_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/backup_generated_certificates.yml +- include_tasks: certificates/backup_generated_certificates.yml diff --git a/roles/etcd/tasks/backup_master_etcd_certificates.yml b/roles/etcd/tasks/backup_master_etcd_certificates.yml index 129e1831c..5526825fa 100644 --- a/roles/etcd/tasks/backup_master_etcd_certificates.yml +++ b/roles/etcd/tasks/backup_master_etcd_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/backup_master_etcd_certificates.yml +- include_tasks: certificates/backup_master_etcd_certificates.yml diff --git a/roles/etcd/tasks/backup_server_certificates.yml b/roles/etcd/tasks/backup_server_certificates.yml index 267ffeb4d..5f3052be1 100644 --- a/roles/etcd/tasks/backup_server_certificates.yml +++ b/roles/etcd/tasks/backup_server_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/backup_server_certificates.yml +- include_tasks: certificates/backup_server_certificates.yml diff --git a/roles/etcd/tasks/ca.yml b/roles/etcd/tasks/ca.yml index cca1e9ad7..dd4b59e24 100644 --- a/roles/etcd/tasks/ca.yml +++ b/roles/etcd/tasks/ca.yml @@ -1,2 +1,2 @@  --- -- include: certificates/deploy_ca.yml +- include_tasks: certificates/deploy_ca.yml diff --git a/roles/etcd/tasks/check_cluster_health.yml b/roles/etcd/tasks/check_cluster_health.yml index 75c110972..3410528eb 100644 --- a/roles/etcd/tasks/check_cluster_health.yml +++ b/roles/etcd/tasks/check_cluster_health.yml @@ -1,2 +1,2 @@  --- -- include: migration/check_cluster_health.yml +- include_tasks: migration/check_cluster_health.yml diff --git a/roles/etcd/tasks/clean_data.yml b/roles/etcd/tasks/clean_data.yml index d131ffd21..12538c2d0 100644 --- a/roles/etcd/tasks/clean_data.yml +++ b/roles/etcd/tasks/clean_data.yml @@ -1,2 +1,2 @@  --- -- include: auxiliary/clean_data.yml +- include_tasks: auxiliary/clean_data.yml diff --git a/roles/etcd/tasks/client_certificates.yml b/roles/etcd/tasks/client_certificates.yml index 2f4108a0d..f3201816d 100644 --- a/roles/etcd/tasks/client_certificates.yml +++ b/roles/etcd/tasks/client_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/fetch_client_certificates_from_ca.yml +- include_tasks: certificates/fetch_client_certificates_from_ca.yml diff --git a/roles/etcd/tasks/disable_etcd.yml b/roles/etcd/tasks/disable_etcd.yml index 9202e6e48..55fb7f6ea 100644 --- a/roles/etcd/tasks/disable_etcd.yml +++ b/roles/etcd/tasks/disable_etcd.yml @@ -1,2 +1,2 @@  --- -- include: auxiliary/disable_etcd.yml +- include_tasks: auxiliary/disable_etcd.yml diff --git a/roles/etcd/tasks/distribute_ca b/roles/etcd/tasks/distribute_ca deleted file mode 100644 index 040c5f7af..000000000 --- a/roles/etcd/tasks/distribute_ca +++ /dev/null @@ -1,2 +0,0 @@ ---- -- include: certificates/distribute_ca.yml diff --git a/roles/etcd/tasks/distribute_ca.yml b/roles/etcd/tasks/distribute_ca.yml new file mode 100644 index 000000000..7d2607844 --- /dev/null +++ b/roles/etcd/tasks/distribute_ca.yml @@ -0,0 +1,2 @@ +--- +- include_tasks: certificates/distribute_ca.yml diff --git a/roles/etcd/tasks/drop_etcdctl.yml b/roles/etcd/tasks/drop_etcdctl.yml index 4c1f609f7..3258ab1a8 100644 --- a/roles/etcd/tasks/drop_etcdctl.yml +++ b/roles/etcd/tasks/drop_etcdctl.yml @@ -1,2 +1,2 @@  --- -- include: auxiliary/drop_etcdctl.yml +- include_tasks: auxiliary/drop_etcdctl.yml diff --git a/roles/etcd/tasks/fetch_backup.yml b/roles/etcd/tasks/fetch_backup.yml index 513eed17a..a28db3d66 100644 --- a/roles/etcd/tasks/fetch_backup.yml +++ b/roles/etcd/tasks/fetch_backup.yml @@ -1,8 +1,8 @@  --- -- include: backup/vars.yml +- include_tasks: backup/vars.yml -- include: backup/archive.yml +- include_tasks: backup/archive.yml -- include: backup/sync_backup.yml +- include_tasks: backup/sync_backup.yml -- include: backup/ +- include_tasks: backup/ diff --git a/roles/etcd/tasks/main.yml b/roles/etcd/tasks/main.yml index 78ec2cedb..5ee9335f5 100644 --- a/roles/etcd/tasks/main.yml +++ b/roles/etcd/tasks/main.yml @@ -7,14 +7,13 @@      etcd_ip: "{{ etcd_ip }}"  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Install etcd    package: name=etcd{{ '-' + etcd_version if etcd_version is defined else '' }} state=present    when: not etcd_is_containerized | bool -- include: drop_etcdctl.yml +- include_tasks: drop_etcdctl.yml    when:    - openshift_etcd_etcdctl_profile | default(true) | bool @@ -101,7 +100,7 @@      when: not l_is_etcd_system_container | bool    - name: Install Etcd system container -    include: system_container.yml +    include_tasks: system_container.yml      when: l_is_etcd_system_container | bool    when: etcd_is_containerized | bool diff --git a/roles/etcd/tasks/migrate.add_ttls.yml b/roles/etcd/tasks/migrate.add_ttls.yml index bc27e4ea1..1dd3c9269 100644 --- a/roles/etcd/tasks/migrate.add_ttls.yml +++ b/roles/etcd/tasks/migrate.add_ttls.yml @@ -1,2 +1,2 @@  --- -- include: migration/add_ttls.yml +- include_tasks: migration/add_ttls.yml diff --git a/roles/etcd/tasks/migrate.configure_master.yml b/roles/etcd/tasks/migrate.configure_master.yml index 3ada6e362..5be9cebd7 100644 --- a/roles/etcd/tasks/migrate.configure_master.yml +++ b/roles/etcd/tasks/migrate.configure_master.yml @@ -1,2 +1,2 @@  --- -- include: migration/configure_master.yml +- include_tasks: migration/configure_master.yml diff --git a/roles/etcd/tasks/migrate.pre_check.yml b/roles/etcd/tasks/migrate.pre_check.yml index 124d21561..4cb67d322 100644 --- a/roles/etcd/tasks/migrate.pre_check.yml +++ b/roles/etcd/tasks/migrate.pre_check.yml @@ -1,2 +1,2 @@  --- -- include: migration/check.yml +- include_tasks: migration/check.yml diff --git a/roles/etcd/tasks/migrate.yml b/roles/etcd/tasks/migrate.yml index 5d5385873..1a75f63f1 100644 --- a/roles/etcd/tasks/migrate.yml +++ b/roles/etcd/tasks/migrate.yml @@ -1,2 +1,2 @@  --- -- include: migration/migrate.yml +- include_tasks: migration/migrate.yml diff --git a/roles/etcd/tasks/migration/check.yml b/roles/etcd/tasks/migration/check.yml index 5c45e5ae1..8ef81da28 100644 --- a/roles/etcd/tasks/migration/check.yml +++ b/roles/etcd/tasks/migration/check.yml @@ -1,7 +1,7 @@  ---  # Check the cluster is healthy -- include: check_cluster_health.yml +- include_tasks: check_cluster_health.yml  # Check if there is at least one v2 snapshot  - name: Check if there is at least one v2 snapshot @@ -39,7 +39,7 @@  # - with_items not supported over block  # Check the cluster status for the first time -- include: check_cluster_status.yml +- include_tasks: check_cluster_status.yml  # Check the cluster status for the second time  - block: @@ -50,7 +50,7 @@        seconds: 5      when: not l_etcd_cluster_status_ok | bool -  - include: check_cluster_status.yml +  - include_tasks: check_cluster_status.yml      when: not l_etcd_cluster_status_ok | bool @@ -63,5 +63,5 @@        seconds: 5      when: not l_etcd_cluster_status_ok | bool -  - include: check_cluster_status.yml +  - include_tasks: check_cluster_status.yml      when: not l_etcd_cluster_status_ok | bool diff --git a/roles/etcd/tasks/remove_ca_certificates.yml b/roles/etcd/tasks/remove_ca_certificates.yml index 36df1a1cc..c1ea4e6c9 100644 --- a/roles/etcd/tasks/remove_ca_certificates.yml +++ b/roles/etcd/tasks/remove_ca_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/remove_ca_certificates.yml +- include_tasks: certificates/remove_ca_certificates.yml diff --git a/roles/etcd/tasks/remove_generated_certificates.yml b/roles/etcd/tasks/remove_generated_certificates.yml index b10a4b32d..8cdeea187 100644 --- a/roles/etcd/tasks/remove_generated_certificates.yml +++ b/roles/etcd/tasks/remove_generated_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/remove_generated_certificates.yml +- include_tasks: certificates/remove_generated_certificates.yml diff --git a/roles/etcd/tasks/retrieve_ca_certificates.yml b/roles/etcd/tasks/retrieve_ca_certificates.yml index bd6c4ec85..2184e669c 100644 --- a/roles/etcd/tasks/retrieve_ca_certificates.yml +++ b/roles/etcd/tasks/retrieve_ca_certificates.yml @@ -1,2 +1,2 @@  --- -- include: certificates/retrieve_ca_certificates.yml +- include_tasks: certificates/retrieve_ca_certificates.yml diff --git a/roles/etcd/tasks/server_certificates.yml b/roles/etcd/tasks/server_certificates.yml index ae26079f9..75c35d59e 100644 --- a/roles/etcd/tasks/server_certificates.yml +++ b/roles/etcd/tasks/server_certificates.yml @@ -1,6 +1,6 @@  --- -- include: ca.yml +- include_tasks: ca.yml    when:    - etcd_ca_setup | default(True) | bool -- include: certificates/fetch_server_certificates_from_ca.yml +- include_tasks: certificates/fetch_server_certificates_from_ca.yml diff --git a/roles/etcd/tasks/upgrade_image.yml b/roles/etcd/tasks/upgrade_image.yml index 9e69027eb..35385cb9a 100644 --- a/roles/etcd/tasks/upgrade_image.yml +++ b/roles/etcd/tasks/upgrade_image.yml @@ -1,2 +1,2 @@  --- -- include: upgrade/upgrade_image.yml +- include_tasks: upgrade/upgrade_image.yml diff --git a/roles/etcd/tasks/upgrade_rpm.yml b/roles/etcd/tasks/upgrade_rpm.yml index 29603d2b6..fbd3cd919 100644 --- a/roles/etcd/tasks/upgrade_rpm.yml +++ b/roles/etcd/tasks/upgrade_rpm.yml @@ -1,2 +1,2 @@  --- -- include: upgrade/upgrade_rpm.yml +- include_tasks: upgrade/upgrade_rpm.yml diff --git a/roles/installer_checkpoint/README.md b/roles/installer_checkpoint/README.md index 6426cd545..68c0357b6 100644 --- a/roles/installer_checkpoint/README.md +++ b/roles/installer_checkpoint/README.md @@ -160,7 +160,7 @@ Health Check               : Complete (0:01:10)  etcd Install               : Complete (0:02:58)  Master Install             : Complete (0:09:20)  Master Additional Install  : In Progress (0:20:04) -    This phase can be restarted by running: playbooks/byo/openshift-master/additional_config.yml +    This phase can be restarted by running: playbooks/openshift-master/additional_config.yml  ```  [set_stats]: http://docs.ansible.com/ansible/latest/set_stats_module.html diff --git a/roles/installer_checkpoint/callback_plugins/installer_checkpoint.py b/roles/installer_checkpoint/callback_plugins/installer_checkpoint.py index 205719215..57444a2a5 100644 --- a/roles/installer_checkpoint/callback_plugins/installer_checkpoint.py +++ b/roles/installer_checkpoint/callback_plugins/installer_checkpoint.py @@ -62,11 +62,11 @@ class CallbackModule(CallbackBase):              },              'installer_phase_master': {                  'title': 'Master Install', -                'playbook': 'playbooks/byo/openshift-master/config.yml' +                'playbook': 'playbooks/openshift-master/config.yml'              },              'installer_phase_master_additional': {                  'title': 'Master Additional Install', -                'playbook': 'playbooks/byo/openshift-master/additional_config.yml' +                'playbook': 'playbooks/openshift-master/additional_config.yml'              },              'installer_phase_node': {                  'title': 'Node Install', @@ -78,7 +78,7 @@ class CallbackModule(CallbackBase):              },              'installer_phase_hosted': {                  'title': 'Hosted Install', -                'playbook': 'playbooks/byo/openshift-cluster/openshift-hosted.yml' +                'playbook': 'playbooks/openshift-hosted/config.yml'              },              'installer_phase_metrics': {                  'title': 'Metrics Install', @@ -90,15 +90,15 @@ class CallbackModule(CallbackBase):              },              'installer_phase_prometheus': {                  'title': 'Prometheus Install', -                'playbook': 'playbooks/byo/openshift-cluster/openshift-prometheus.yml' +                'playbook': 'playbooks/openshift-prometheus/config.yml'              },              'installer_phase_servicecatalog': {                  'title': 'Service Catalog Install', -                'playbook': 'playbooks/byo/openshift-cluster/service-catalog.yml' +                'playbook': 'playbooks/openshift-service-catalog/config.yml'              },              'installer_phase_management': {                  'title': 'Management Install', -                'playbook': 'playbooks/byo/openshift-management/config.yml' +                'playbook': 'playbooks/openshift-management/config.yml'              },          } diff --git a/roles/kuryr/tasks/master.yaml b/roles/kuryr/tasks/master.yaml index 55ab16f74..1cc6d2375 100644 --- a/roles/kuryr/tasks/master.yaml +++ b/roles/kuryr/tasks/master.yaml @@ -1,6 +1,6 @@  --- -- name: Perform OpenShit ServiceAccount config -  include: serviceaccount.yaml +- name: Perform OpenShift ServiceAccount config +  include_tasks: serviceaccount.yaml  - name: Create kuryr manifests tempdir    command: mktemp -d diff --git a/roles/nuage_master/tasks/main.yaml b/roles/nuage_master/tasks/main.yaml index f3c487132..c264427de 100644 --- a/roles/nuage_master/tasks/main.yaml +++ b/roles/nuage_master/tasks/main.yaml @@ -1,7 +1,6 @@  ---  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Set the Nuage certificate directory fact for Atomic hosts    set_fact: @@ -62,7 +61,7 @@    become: yes    file: path={{ nuage_mon_rest_server_logdir }} state=directory -- include: serviceaccount.yml +- include_tasks: serviceaccount.yml  - name: Download the certs and keys    become: yes @@ -82,7 +81,7 @@      - nuage.key      - nuage.kubeconfig -- include: certificates.yml +- include_tasks: certificates.yml  - name: Install Nuage VSD user certificate    become: yes diff --git a/roles/nuage_node/tasks/main.yaml b/roles/nuage_node/tasks/main.yaml index 9db9dbb6a..c6b7a9b10 100644 --- a/roles/nuage_node/tasks/main.yaml +++ b/roles/nuage_node/tasks/main.yaml @@ -31,7 +31,7 @@      - nuage.key      - nuage.kubeconfig -- include: certificates.yml +- include_tasks: certificates.yml  - name: Add additional Docker mounts for Nuage for atomic hosts    become: yes @@ -44,8 +44,7 @@      - restart node    ignore_errors: true -- include: iptables.yml +- include_tasks: iptables.yml  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml diff --git a/roles/openshift_cli/defaults/main.yml b/roles/openshift_cli/defaults/main.yml index ed97d539c..82da0639e 100644 --- a/roles/openshift_cli/defaults/main.yml +++ b/roles/openshift_cli/defaults/main.yml @@ -1 +1,6 @@  --- +system_images_registry_dict: +  openshift-enterprise: "registry.access.redhat.com" +  origin: "docker.io" + +system_images_registry: "{{ system_images_registry_dict[openshift_deployment_type | default('origin')] }}" diff --git a/roles/openshift_cli/tasks/main.yml b/roles/openshift_cli/tasks/main.yml index 14d8a3325..06dc5d14b 100644 --- a/roles/openshift_cli/tasks/main.yml +++ b/roles/openshift_cli/tasks/main.yml @@ -28,13 +28,13 @@  - block:    - name: Pull CLI Image      command: > -      atomic pull --storage ostree {{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.common.cli_image }}:{{ openshift_image_tag }} +      atomic pull --storage ostree {{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.common.cli_image }}:{{ openshift_image_tag }}      register: pull_result      changed_when: "'Pulling layer' in pull_result.stdout"    - name: Copy client binaries/symlinks out of CLI image for use on the host      openshift_container_binary_sync: -      image: "{{ '' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.common.cli_image }}" +      image: "{{ '' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.common.cli_image }}"        tag: "{{ openshift_image_tag }}"        backend: "atomic"    when: diff --git a/roles/openshift_cloud_provider/tasks/main.yml b/roles/openshift_cloud_provider/tasks/main.yml index ab3055c8b..dff492a69 100644 --- a/roles/openshift_cloud_provider/tasks/main.yml +++ b/roles/openshift_cloud_provider/tasks/main.yml @@ -11,11 +11,11 @@      state: directory    when: has_cloudprovider | bool -- include: openstack.yml +- include_tasks: openstack.yml    when: cloudprovider_is_openstack | bool -- include: aws.yml +- include_tasks: aws.yml    when: cloudprovider_is_aws | bool -- include: gce.yml +- include_tasks: gce.yml    when: cloudprovider_is_gce | bool diff --git a/roles/openshift_excluder/tasks/disable.yml b/roles/openshift_excluder/tasks/disable.yml index 5add25b45..21801b994 100644 --- a/roles/openshift_excluder/tasks/disable.yml +++ b/roles/openshift_excluder/tasks/disable.yml @@ -2,11 +2,11 @@  - when: r_openshift_excluder_verify_upgrade    block:    - name: Include verify_upgrade.yml when upgrading -    include: verify_upgrade.yml +    include_tasks: verify_upgrade.yml  # unexclude the current openshift/origin-excluder if it is installed so it can be updated  - name: Disable excluders before the upgrade to remove older excluding expressions -  include: unexclude.yml +  include_tasks: unexclude.yml    vars:      # before the docker excluder can be updated, it needs to be disabled      # to remove older excluded packages that are no longer excluded @@ -15,12 +15,12 @@  # Install any excluder that is enabled  - name: Include install.yml -  include: install.yml +  include_tasks: install.yml  # And finally adjust an excluder in order to update host components correctly. First  # exclude then unexclude  - name: Include exclude.yml -  include: exclude.yml +  include_tasks: exclude.yml    vars:      # Enable the docker excluder only if it is overridden      # BZ #1430612: docker excluders should be enabled even during installation and upgrade @@ -30,7 +30,7 @@  # All excluders that are to be disabled are disabled  - name: Include unexclude.yml -  include: unexclude.yml +  include_tasks: unexclude.yml    vars:      # If the docker override  is not set, default to the generic behaviour      # BZ #1430612: docker excluders should be enabled even during installation and upgrade diff --git a/roles/openshift_excluder/tasks/enable.yml b/roles/openshift_excluder/tasks/enable.yml index fce44cfb5..7c3742a06 100644 --- a/roles/openshift_excluder/tasks/enable.yml +++ b/roles/openshift_excluder/tasks/enable.yml @@ -1,6 +1,6 @@  ---  - name: Install excluders -  include: install.yml +  include_tasks: install.yml  - name: Enable excluders -  include: exclude.yml +  include_tasks: exclude.yml diff --git a/roles/openshift_excluder/tasks/main.yml b/roles/openshift_excluder/tasks/main.yml index db20b4012..93d6ef149 100644 --- a/roles/openshift_excluder/tasks/main.yml +++ b/roles/openshift_excluder/tasks/main.yml @@ -32,7 +32,7 @@      - r_openshift_excluder_upgrade_target is not defined    - name: Include main action task file -    include: "{{ r_openshift_excluder_action }}.yml" +    include_tasks: "{{ r_openshift_excluder_action }}.yml"    when:    - not ostree_booted.stat.exists | bool diff --git a/roles/openshift_excluder/tasks/verify_upgrade.yml b/roles/openshift_excluder/tasks/verify_upgrade.yml index 42026664a..b55a9af23 100644 --- a/roles/openshift_excluder/tasks/verify_upgrade.yml +++ b/roles/openshift_excluder/tasks/verify_upgrade.yml @@ -1,12 +1,12 @@  ---  - name: Verify Docker Excluder version -  include: verify_excluder.yml +  include_tasks: verify_excluder.yml    vars:      excluder: "{{ r_openshift_excluder_service_type }}-docker-excluder"    when: r_openshift_excluder_enable_docker_excluder | bool  - name: Verify OpenShift Excluder version -  include: verify_excluder.yml +  include_tasks: verify_excluder.yml    vars:      excluder: "{{ r_openshift_excluder_service_type }}-excluder"    when: r_openshift_excluder_enable_openshift_excluder | bool diff --git a/roles/openshift_hosted/tasks/storage/glusterfs.yml b/roles/openshift_hosted/tasks/storage/glusterfs.yml index c2954fde1..9b998142a 100644 --- a/roles/openshift_hosted/tasks/storage/glusterfs.yml +++ b/roles/openshift_hosted/tasks/storage/glusterfs.yml @@ -79,14 +79,7 @@        - REGISTRY_STORAGE_MAINTENANCE_READONLY_ENABLED: 'true'    when: openshift.hosted.registry.storage.glusterfs.swap -- name: Unmount registry volume +- name: Unmount registry volume and clean up mount point/fstab    mount: -    state: unmounted -    name: "{{ mktemp.stdout }}" - -- name: Delete temp mount directory -  file: -    dest: "{{ mktemp.stdout }}"      state: absent -  changed_when: False -  check_mode: no +    name: "{{ mktemp.stdout }}" diff --git a/roles/openshift_loadbalancer/tasks/main.yml b/roles/openshift_loadbalancer/tasks/main.yml index 69b061fc5..c87a327a4 100644 --- a/roles/openshift_loadbalancer/tasks/main.yml +++ b/roles/openshift_loadbalancer/tasks/main.yml @@ -1,7 +1,6 @@  ---  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Install haproxy    package: name=haproxy state=present diff --git a/roles/openshift_master/defaults/main.yml b/roles/openshift_master/defaults/main.yml index 3fb94fff8..99bc12514 100644 --- a/roles/openshift_master/defaults/main.yml +++ b/roles/openshift_master/defaults/main.yml @@ -7,6 +7,12 @@ openshift_master_debug_level: "{{ debug_level | default(2) }}"  r_openshift_master_firewall_enabled: "{{ os_firewall_enabled | default(True) }}"  r_openshift_master_use_firewalld: "{{ os_firewall_use_firewalld | default(False) }}" +system_images_registry_dict: +  openshift-enterprise: "registry.access.redhat.com" +  origin: "docker.io" + +system_images_registry: "{{ system_images_registry_dict[openshift_deployment_type | default('origin')] }}" +  openshift_node_ips: []  r_openshift_master_clean_install: false  r_openshift_master_etcd3_storage: false diff --git a/roles/openshift_master/tasks/main.yml b/roles/openshift_master/tasks/main.yml index b6d3539b1..e2f92d597 100644 --- a/roles/openshift_master/tasks/main.yml +++ b/roles/openshift_master/tasks/main.yml @@ -31,8 +31,7 @@    - openshift.common.is_containerized | bool  - name: Open up firewall ports -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Install Master package    package: @@ -172,13 +171,13 @@        no_proxy_etcd_host_ips: "{{ openshift_no_proxy_etcd_host_ips }}"  - name: Update journald config -  include: journald.yml +  include_tasks: journald.yml  - name: Install the systemd units -  include: systemd_units.yml +  include_tasks: systemd_units.yml  - name: Install Master system container -  include: system_container.yml +  include_tasks: system_container.yml    when:    - openshift.common.is_containerized | bool    - openshift.common.is_master_system_container | bool @@ -212,10 +211,10 @@    - restart master api    - restart master controllers -- include: bootstrap_settings.yml +- include_tasks: bootstrap_settings.yml    when: openshift_master_bootstrap_enabled | default(False) -- include: set_loopback_context.yml +- include_tasks: set_loopback_context.yml  - name: Start and enable master api on first master    systemd: @@ -273,7 +272,7 @@  # A separate wait is required here for native HA since notifies will  # be resolved after all tasks in the role. -- include: check_master_api_is_ready.yml +- include_tasks: check_master_api_is_ready.yml    when:    - openshift.master.cluster_method == 'native'    - master_api_service_status_changed | bool @@ -323,5 +322,5 @@    - l_install_result | changed  - name: node bootstrap settings -  include: bootstrap.yml +  include_tasks: bootstrap.yml    when: openshift_master_bootstrap_enabled | default(False) diff --git a/roles/openshift_master/tasks/system_container.yml b/roles/openshift_master/tasks/system_container.yml index 843352532..23386f11b 100644 --- a/roles/openshift_master/tasks/system_container.yml +++ b/roles/openshift_master/tasks/system_container.yml @@ -6,7 +6,7 @@  - name: Pre-pull master system container image    command: > -    atomic pull --storage=ostree {{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }} +    atomic pull --storage=ostree {{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }}    register: l_pull_result    changed_when: "'Pulling layer' in l_pull_result.stdout" @@ -18,7 +18,7 @@  - name: Install or Update HA api master system container    oc_atomic_container:      name: "{{ openshift.common.service_type }}-master-api" -    image: "{{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }}" +    image: "{{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }}"      state: latest      values:      - COMMAND=api @@ -26,7 +26,7 @@  - name: Install or Update HA controller master system container    oc_atomic_container:      name: "{{ openshift.common.service_type }}-master-controllers" -    image: "{{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }}" +    image: "{{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.master.master_system_image }}:{{ openshift_image_tag }}"      state: latest      values:      - COMMAND=controllers diff --git a/roles/openshift_master/tasks/systemd_units.yml b/roles/openshift_master/tasks/systemd_units.yml index b0fa72f19..582185198 100644 --- a/roles/openshift_master/tasks/systemd_units.yml +++ b/roles/openshift_master/tasks/systemd_units.yml @@ -9,7 +9,7 @@    when:    - openshift.common.is_containerized | bool -- include: registry_auth.yml +- include_tasks: registry_auth.yml  - name: Disable the legacy master service if it exists    systemd: diff --git a/roles/openshift_master/tasks/upgrade.yml b/roles/openshift_master/tasks/upgrade.yml index 92371921d..f84cf2f6e 100644 --- a/roles/openshift_master/tasks/upgrade.yml +++ b/roles/openshift_master/tasks/upgrade.yml @@ -1,16 +1,16 @@  --- -- include: upgrade/rpm_upgrade.yml +- include_tasks: upgrade/rpm_upgrade.yml    when: not openshift.common.is_containerized | bool -- include: upgrade/upgrade_scheduler.yml +- include_tasks: upgrade/upgrade_scheduler.yml  # master_config_hook is passed in from upgrade play. -- include: "upgrade/{{ master_config_hook }}" +- include_tasks: "upgrade/{{ master_config_hook }}"    when: master_config_hook is defined -- include: journald.yml +- include_tasks: journald.yml -- include: systemd_units.yml +- include_tasks: systemd_units.yml  - name: Check for ca-bundle.crt    stat: diff --git a/roles/openshift_master_cluster/tasks/main.yml b/roles/openshift_master_cluster/tasks/main.yml index 40705d357..41bfc72cb 100644 --- a/roles/openshift_master_cluster/tasks/main.yml +++ b/roles/openshift_master_cluster/tasks/main.yml @@ -10,5 +10,5 @@    failed_when: false    when: openshift.master.cluster_method == "pacemaker" -- include: configure.yml +- include_tasks: configure.yml    when: "pcs_status | failed and 'Error: cluster is not currently running on this node' in pcs_status.stderr" diff --git a/roles/openshift_metrics/tasks/generate_certificates.yaml b/roles/openshift_metrics/tasks/generate_certificates.yaml index 3dc15d58b..bb842d710 100644 --- a/roles/openshift_metrics/tasks/generate_certificates.yaml +++ b/roles/openshift_metrics/tasks/generate_certificates.yaml @@ -8,4 +8,4 @@      --serial='{{ mktemp.stdout }}/ca.serial.txt'      --name="metrics-signer@{{lookup('pipe','date +%s')}}" -- include: generate_hawkular_certificates.yaml +- include_tasks: generate_hawkular_certificates.yaml diff --git a/roles/openshift_metrics/tasks/generate_hawkular_certificates.yaml b/roles/openshift_metrics/tasks/generate_hawkular_certificates.yaml index 31129a6ac..0fd19c9f8 100644 --- a/roles/openshift_metrics/tasks/generate_hawkular_certificates.yaml +++ b/roles/openshift_metrics/tasks/generate_hawkular_certificates.yaml @@ -1,13 +1,13 @@  ---  - name: generate hawkular-metrics certificates -  include: setup_certificate.yaml +  include_tasks: setup_certificate.yaml    vars:      component: hawkular-metrics      hostnames: "hawkular-metrics,hawkular-metrics.{{ openshift_metrics_project }}.svc.cluster.local,{{ openshift_metrics_hawkular_hostname }}"    changed_when: no  - name: generate hawkular-cassandra certificates -  include: setup_certificate.yaml +  include_tasks: setup_certificate.yaml    vars:      component: hawkular-cassandra      hostnames: hawkular-cassandra diff --git a/roles/openshift_metrics/tasks/install_heapster.yaml b/roles/openshift_metrics/tasks/install_heapster.yaml index 0eb852d91..a33b28ba7 100644 --- a/roles/openshift_metrics/tasks/install_heapster.yaml +++ b/roles/openshift_metrics/tasks/install_heapster.yaml @@ -66,4 +66,4 @@          namespace: "{{ openshift_metrics_project }}"    changed_when: no -- include: generate_heapster_secrets.yaml +- include_tasks: generate_heapster_secrets.yaml diff --git a/roles/openshift_metrics/tasks/install_metrics.yaml b/roles/openshift_metrics/tasks/install_metrics.yaml index fdf4ae57f..49d1d8cf1 100644 --- a/roles/openshift_metrics/tasks/install_metrics.yaml +++ b/roles/openshift_metrics/tasks/install_metrics.yaml @@ -1,8 +1,8 @@  --- -- include: pre_install.yaml +- include_tasks: pre_install.yaml  - name: Install Metrics -  include: "{{ role_path }}/tasks/install_{{ include_file }}.yaml" +  include_tasks: "install_{{ include_file }}.yaml"    with_items:      - support      - heapster @@ -13,11 +13,11 @@    when: not openshift_metrics_heapster_standalone | bool  - name: Install Heapster Standalone -  include: install_heapster.yaml +  include_tasks: install_heapster.yaml    when: openshift_metrics_heapster_standalone | bool  - name: Install Hawkular OpenShift Agent (HOSA) -  include: install_hosa.yaml +  include_tasks: install_hosa.yaml    when: openshift_metrics_install_hawkular_agent | default(false) | bool  - find: @@ -34,7 +34,7 @@    changed_when: no  - name: Create objects -  include: oc_apply.yaml +  include_tasks: oc_apply.yaml    vars:      kubeconfig: "{{ mktemp.stdout }}/admin.kubeconfig"      namespace: "{{ openshift_metrics_project }}" @@ -58,7 +58,7 @@    changed_when: no  - name: Create Hawkular Agent objects -  include: oc_apply.yaml +  include_tasks: oc_apply.yaml    vars:      kubeconfig: "{{ mktemp.stdout }}/admin.kubeconfig"      namespace: "{{ openshift_metrics_hawkular_agent_namespace }}" @@ -67,7 +67,7 @@    with_items: "{{ hawkular_agent_object_defs.results }}"    when: openshift_metrics_install_hawkular_agent | bool -- include: update_master_config.yaml +- include_tasks: update_master_config.yaml  - command: >      {{openshift.common.client_binary}} @@ -80,11 +80,11 @@    changed_when: no  - name: Scaling down cluster to recognize changes -  include: stop_metrics.yaml +  include_tasks: stop_metrics.yaml    when: existing_metrics_rc.stdout_lines | length > 0  - name: Scaling up cluster -  include: start_metrics.yaml +  include_tasks: start_metrics.yaml    tags: openshift_metrics_start_cluster    when:      - openshift_metrics_start_cluster | default(true) | bool diff --git a/roles/openshift_metrics/tasks/install_support.yaml b/roles/openshift_metrics/tasks/install_support.yaml index 584e3be05..c3727d530 100644 --- a/roles/openshift_metrics/tasks/install_support.yaml +++ b/roles/openshift_metrics/tasks/install_support.yaml @@ -19,7 +19,7 @@  - fail: msg="'keytool' is unavailable. Please install java-1.8.0-openjdk-headless on the control node"    when: keytool_check.rc  == 1 -- include: generate_certificates.yaml -- include: generate_serviceaccounts.yaml -- include: generate_services.yaml -- include: generate_rolebindings.yaml +- include_tasks: generate_certificates.yaml +- include_tasks: generate_serviceaccounts.yaml +- include_tasks: generate_services.yaml +- include_tasks: generate_rolebindings.yaml diff --git a/roles/openshift_metrics/tasks/main.yaml b/roles/openshift_metrics/tasks/main.yaml index 10509fc1e..9dfe360bb 100644 --- a/roles/openshift_metrics/tasks/main.yaml +++ b/roles/openshift_metrics/tasks/main.yaml @@ -43,15 +43,15 @@    check_mode: no    tags: metrics_init -- include: install_metrics.yaml +- include_tasks: install_metrics.yaml    when:      - openshift_metrics_install_metrics | bool -- include: uninstall_metrics.yaml +- include_tasks: uninstall_metrics.yaml    when:      - not openshift_metrics_install_metrics | bool -- include: uninstall_hosa.yaml +- include_tasks: uninstall_hosa.yaml    when: not openshift_metrics_install_hawkular_agent | bool  - name: Delete temp directory diff --git a/roles/openshift_metrics/tasks/uninstall_metrics.yaml b/roles/openshift_metrics/tasks/uninstall_metrics.yaml index 403b1252c..1265c7bfd 100644 --- a/roles/openshift_metrics/tasks/uninstall_metrics.yaml +++ b/roles/openshift_metrics/tasks/uninstall_metrics.yaml @@ -1,6 +1,6 @@  ---  - name: stop metrics -  include: stop_metrics.yaml +  include_tasks: stop_metrics.yaml  - name: remove metrics components    command: > diff --git a/roles/openshift_node/defaults/main.yml b/roles/openshift_node/defaults/main.yml index 816338fa1..d249278dc 100644 --- a/roles/openshift_node/defaults/main.yml +++ b/roles/openshift_node/defaults/main.yml @@ -9,6 +9,12 @@ l_is_node_system_container: "{{ (openshift_use_node_system_container | default(o  openshift_deployment_type: "{{ openshift_deployment_type | default('origin') }}"  openshift_service_type: "{{ 'origin' if openshift_deployment_type == 'origin' else 'atomic-openshift' }}" +system_images_registry_dict: +  openshift-enterprise: "registry.access.redhat.com" +  origin: "docker.io" + +system_images_registry: "{{ system_images_registry_dict[openshift_deployment_type | default('origin')] }}" +  openshift_image_tag: ''  default_r_openshift_node_image_prep_packages: diff --git a/roles/openshift_node/tasks/bootstrap.yml b/roles/openshift_node/tasks/bootstrap.yml index b8be50f6c..ac43ef039 100644 --- a/roles/openshift_node/tasks/bootstrap.yml +++ b/roles/openshift_node/tasks/bootstrap.yml @@ -32,8 +32,7 @@      regexp: "^CONFIG_FILE=.*"  - name: include aws sysconfig credentials -  include: aws.yml -  static: yes +  import_tasks: aws.yml    when: not (openshift_node_use_instance_profiles | default(False))  #- name: update the ExecStart to have bootstrap diff --git a/roles/openshift_node/tasks/config.yml b/roles/openshift_node/tasks/config.yml index 2fea33454..741a2234f 100644 --- a/roles/openshift_node/tasks/config.yml +++ b/roles/openshift_node/tasks/config.yml @@ -1,6 +1,6 @@  ---  - name: Install the systemd units -  include: systemd_units.yml +  include_tasks: systemd_units.yml  - name: Start and enable openvswitch service    systemd: @@ -47,8 +47,7 @@      - restart node  - name: include aws provider credentials -  include: aws.yml -  static: yes +  import_tasks: aws.yml    when: not (openshift_node_use_instance_profiles | default(False))  # Necessary because when you're on a node that's also a master the master will be diff --git a/roles/openshift_node/tasks/main.yml b/roles/openshift_node/tasks/main.yml index eae9ca7bc..8e9d1d1b5 100644 --- a/roles/openshift_node/tasks/main.yml +++ b/roles/openshift_node/tasks/main.yml @@ -7,8 +7,7 @@      - not openshift_use_crio | default(false)  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  #### Disable SWAP #####  # https://docs.openshift.com/container-platform/3.4/admin_guide/overcommit.html#disabling-swap-memory @@ -41,7 +40,7 @@  #### End Disable Swap Block ####  - name: include node installer -  include: install.yml +  include_tasks: install.yml  - name: Restart cri-o    systemd: @@ -66,34 +65,34 @@      sysctl_file: "/etc/sysctl.d/99-openshift.conf"      reload: yes -- include: registry_auth.yml +- include_tasks: registry_auth.yml  - name: include standard node config -  include: config.yml +  include_tasks: config.yml  #### Storage class plugins here ####  - name: NFS storage plugin configuration -  include: storage_plugins/nfs.yml +  include_tasks: storage_plugins/nfs.yml    tags:      - nfs  - name: GlusterFS storage plugin configuration -  include: storage_plugins/glusterfs.yml +  include_tasks: storage_plugins/glusterfs.yml    when: "'glusterfs' in openshift.node.storage_plugin_deps"  - name: Ceph storage plugin configuration -  include: storage_plugins/ceph.yml +  include_tasks: storage_plugins/ceph.yml    when: "'ceph' in openshift.node.storage_plugin_deps"  - name: iSCSI storage plugin configuration -  include: storage_plugins/iscsi.yml +  include_tasks: storage_plugins/iscsi.yml    when: "'iscsi' in openshift.node.storage_plugin_deps"  ##### END Storage ##### -- include: config/workaround-bz1331590-ovs-oom-fix.yml +- include_tasks: config/workaround-bz1331590-ovs-oom-fix.yml    when: openshift_node_use_openshift_sdn | default(true) | bool  - name: include bootstrap node config -  include: bootstrap.yml +  include_tasks: bootstrap.yml    when: openshift_node_bootstrap diff --git a/roles/openshift_node/tasks/node_system_container.yml b/roles/openshift_node/tasks/node_system_container.yml index 164a79b39..73dc9e130 100644 --- a/roles/openshift_node/tasks/node_system_container.yml +++ b/roles/openshift_node/tasks/node_system_container.yml @@ -6,14 +6,14 @@  - name: Pre-pull node system container image    command: > -    atomic pull --storage=ostree {{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.node.node_system_image }}:{{ openshift_image_tag }} +    atomic pull --storage=ostree {{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.node.node_system_image }}:{{ openshift_image_tag }}    register: pull_result    changed_when: "'Pulling layer' in pull_result.stdout"  - name: Install or Update node system container    oc_atomic_container:      name: "{{ openshift.common.service_type }}-node" -    image: "{{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.node.node_system_image }}:{{ openshift_image_tag }}" +    image: "{{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.node.node_system_image }}:{{ openshift_image_tag }}"      values:      - "DNS_DOMAIN={{ openshift.common.dns_domain }}"      - "DOCKER_SERVICE={{ openshift.docker.service_name }}.service" diff --git a/roles/openshift_node/tasks/openvswitch_system_container.yml b/roles/openshift_node/tasks/openvswitch_system_container.yml index 0f73ce454..8c3548475 100644 --- a/roles/openshift_node/tasks/openvswitch_system_container.yml +++ b/roles/openshift_node/tasks/openvswitch_system_container.yml @@ -17,14 +17,14 @@  - name: Pre-pull OpenVSwitch system container image    command: > -    atomic pull --storage=ostree {{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.node.ovs_system_image }}:{{ openshift_image_tag }} +    atomic pull --storage=ostree {{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.node.ovs_system_image }}:{{ openshift_image_tag }}    register: pull_result    changed_when: "'Pulling layer' in pull_result.stdout"  - name: Install or Update OpenVSwitch system container    oc_atomic_container:      name: openvswitch -    image: "{{ 'docker:' if openshift.common.system_images_registry == 'docker' else openshift.common.system_images_registry + '/' }}{{ openshift.node.ovs_system_image }}:{{ openshift_image_tag }}" +    image: "{{ 'docker:' if system_images_registry == 'docker' else system_images_registry + '/' }}{{ openshift.node.ovs_system_image }}:{{ openshift_image_tag }}"      state: latest      values:        - "DOCKER_SERVICE={{ l_service_name }}" diff --git a/roles/openshift_node/tasks/systemd_units.yml b/roles/openshift_node/tasks/systemd_units.yml index 9b4c24dfe..6dd4838bd 100644 --- a/roles/openshift_node/tasks/systemd_units.yml +++ b/roles/openshift_node/tasks/systemd_units.yml @@ -11,18 +11,18 @@  - when: openshift.common.is_containerized | bool    block:    - name: include node deps docker service file -    include: config/install-node-deps-docker-service-file.yml +    include_tasks: config/install-node-deps-docker-service-file.yml    - name: include ovs service environment file -    include: config/install-ovs-service-env-file.yml +    include_tasks: config/install-ovs-service-env-file.yml    - name: Install Node system container -    include: node_system_container.yml +    include_tasks: node_system_container.yml      when:      - l_is_node_system_container | bool    - name: Install OpenvSwitch system containers -    include: openvswitch_system_container.yml +    include_tasks: openvswitch_system_container.yml      when:      - openshift_node_use_openshift_sdn | bool      - openshift.common.is_openvswitch_system_container | bool @@ -34,11 +34,11 @@      register: pull_result      changed_when: "'Downloaded newer image' in pull_result.stdout" -  - include: config/install-ovs-docker-service-file.yml +  - include_tasks: config/install-ovs-docker-service-file.yml    when:    - openshift.common.is_containerized | bool    - openshift_node_use_openshift_sdn | bool    - not openshift.common.is_openvswitch_system_container | bool -- include: config/configure-node-settings.yml -- include: config/configure-proxy-settings.yml +- include_tasks: config/configure-node-settings.yml +- include_tasks: config/configure-proxy-settings.yml diff --git a/roles/openshift_node/tasks/upgrade.yml b/roles/openshift_node/tasks/upgrade.yml index 2bca1e974..fb21b39a1 100644 --- a/roles/openshift_node/tasks/upgrade.yml +++ b/roles/openshift_node/tasks/upgrade.yml @@ -10,7 +10,7 @@  # tasks file for openshift_node_upgrade -- include: registry_auth.yml +- include_tasks: registry_auth.yml  - name: Stop node and openvswitch services    service: @@ -48,7 +48,7 @@    - openshift.common.is_containerized | bool    - openshift_use_openshift_sdn | bool -- include: docker/upgrade.yml +- include_tasks: docker/upgrade.yml    vars:      # We will restart Docker ourselves after everything is ready:      skip_docker_restart: True @@ -56,10 +56,10 @@    - l_docker_upgrade is defined    - l_docker_upgrade | bool -- include: "{{ node_config_hook }}" +- include_tasks: "{{ node_config_hook }}"    when: node_config_hook is defined -- include: upgrade/rpm_upgrade.yml +- include_tasks: upgrade/rpm_upgrade.yml    vars:      component: "node"      openshift_version: "{{ openshift_pkg_version | default('') }}" @@ -70,7 +70,7 @@      path: "/etc/systemd/system/docker.service.d/docker-sdn-ovs.conf"      state: absent -- include: upgrade/containerized_node_upgrade.yml +- include_tasks: upgrade/containerized_node_upgrade.yml    when: openshift.common.is_containerized | bool  - name: Ensure containerized services stopped before Docker restart @@ -165,7 +165,7 @@      value: "/etc/origin/node/resolv.conf"  # Restart all services -- include: upgrade/restart.yml +- include_tasks: upgrade/restart.yml  - name: Wait for node to be ready    oc_obj: diff --git a/roles/openshift_node/tasks/upgrade/containerized_node_upgrade.yml b/roles/openshift_node/tasks/upgrade/containerized_node_upgrade.yml index 96b94d8b6..245de60a7 100644 --- a/roles/openshift_node/tasks/upgrade/containerized_node_upgrade.yml +++ b/roles/openshift_node/tasks/upgrade/containerized_node_upgrade.yml @@ -6,7 +6,7 @@      skip_node_svc_handlers: True  - name: Update systemd units -  include: ../systemd_units.yml +  include_tasks: ../systemd_units.yml  # This is a no-op because of skip_node_svc_handlers, but lets us trigger it before end of  # play when the node has already been marked schedulable again. (this would look strange diff --git a/roles/openshift_node_dnsmasq/tasks/main.yml b/roles/openshift_node_dnsmasq/tasks/main.yml index 9bbaafc29..a33b78780 100644 --- a/roles/openshift_node_dnsmasq/tasks/main.yml +++ b/roles/openshift_node_dnsmasq/tasks/main.yml @@ -59,9 +59,9 @@      state: started  # Dynamic NetworkManager based dispatcher -- include: ./network-manager.yml +- include_tasks: ./network-manager.yml    when: network_manager_active | bool  # Relies on ansible in order to configure static config -- include: ./no-network-manager.yml +- include_tasks: ./no-network-manager.yml    when: not network_manager_active | bool diff --git a/roles/openshift_node_dnsmasq/tasks/no-network-manager.yml b/roles/openshift_node_dnsmasq/tasks/no-network-manager.yml index 8a7da66c2..dede2fb8f 100644 --- a/roles/openshift_node_dnsmasq/tasks/no-network-manager.yml +++ b/roles/openshift_node_dnsmasq/tasks/no-network-manager.yml @@ -8,4 +8,4 @@      state: present    notify: restart NetworkManager -- include: ./network-manager.yml +- include_tasks: network-manager.yml diff --git a/roles/openshift_node_group/tasks/main.yml b/roles/openshift_node_group/tasks/main.yml index c7c15683d..43ecf1b8b 100644 --- a/roles/openshift_node_group/tasks/main.yml +++ b/roles/openshift_node_group/tasks/main.yml @@ -1,6 +1,6 @@  ---  - name: Build node config maps -  include: create_config.yml +  include_tasks: create_config.yml    vars:      openshift_node_group_name: "{{ node_group.name }}"      openshift_node_group_edits: "{{ node_group.edits | default([]) }}" diff --git a/roles/openshift_prometheus/tasks/main.yaml b/roles/openshift_prometheus/tasks/main.yaml index a62babaa7..38798e1f5 100644 --- a/roles/openshift_prometheus/tasks/main.yaml +++ b/roles/openshift_prometheus/tasks/main.yaml @@ -20,10 +20,10 @@      mode: 0755    changed_when: False -- include: install_prometheus.yaml +- include_tasks: install_prometheus.yaml    when: openshift_prometheus_state == 'present' -- include: uninstall_prometheus.yaml +- include_tasks: uninstall_prometheus.yaml    when: openshift_prometheus_state == 'absent'  - name: Delete temp directory diff --git a/roles/openshift_provisioners/tasks/install_provisioners.yaml b/roles/openshift_provisioners/tasks/install_provisioners.yaml index 324fdcc82..2d1217c74 100644 --- a/roles/openshift_provisioners/tasks/install_provisioners.yaml +++ b/roles/openshift_provisioners/tasks/install_provisioners.yaml @@ -16,10 +16,10 @@    when: (openshift_provisioners_efs | bool) and openshift_provisioners_efs_aws_secret_access_key is not defined  - name: Install support -  include: install_support.yaml +  include_tasks: install_support.yaml  - name: Install EFS -  include: install_efs.yaml +  include_tasks: install_efs.yaml    when: openshift_provisioners_efs | bool  - find: paths={{ mktemp.stdout }}/templates patterns=*.yaml @@ -32,7 +32,7 @@    changed_when: no  - name: Create objects -  include: oc_apply.yaml +  include_tasks: oc_apply.yaml    vars:      - kubeconfig: "{{ mktemp.stdout }}/admin.kubeconfig"      - namespace: "{{ openshift_provisioners_project }}" @@ -51,5 +51,5 @@    when: ansible_check_mode  - name: Scaling up cluster -  include: start_cluster.yaml +  include_tasks: start_cluster.yaml    when: start_cluster | default(true) | bool diff --git a/roles/openshift_provisioners/tasks/install_support.yaml b/roles/openshift_provisioners/tasks/install_support.yaml index d6db81ab9..93c4c394d 100644 --- a/roles/openshift_provisioners/tasks/install_support.yaml +++ b/roles/openshift_provisioners/tasks/install_support.yaml @@ -10,8 +10,8 @@    changed_when: False    check_mode: no -- include: generate_secrets.yaml +- include_tasks: generate_secrets.yaml -- include: generate_clusterrolebindings.yaml +- include_tasks: generate_clusterrolebindings.yaml -- include: generate_serviceaccounts.yaml +- include_tasks: generate_serviceaccounts.yaml diff --git a/roles/openshift_provisioners/tasks/main.yaml b/roles/openshift_provisioners/tasks/main.yaml index a50c78c97..4ba26b2b8 100644 --- a/roles/openshift_provisioners/tasks/main.yaml +++ b/roles/openshift_provisioners/tasks/main.yaml @@ -12,10 +12,10 @@    check_mode: no    tags: provisioners_init -- include: "{{ role_path }}/tasks/install_provisioners.yaml" +- include_tasks: install_provisioners.yaml    when: openshift_provisioners_install_provisioners | default(false) | bool -- include: "{{ role_path }}/tasks/uninstall_provisioners.yaml" +- include_tasks: uninstall_provisioners.yaml    when: not openshift_provisioners_install_provisioners | default(false) | bool  - name: Delete temp directory diff --git a/roles/openshift_provisioners/tasks/uninstall_provisioners.yaml b/roles/openshift_provisioners/tasks/uninstall_provisioners.yaml index 0be4bc7d2..602dee773 100644 --- a/roles/openshift_provisioners/tasks/uninstall_provisioners.yaml +++ b/roles/openshift_provisioners/tasks/uninstall_provisioners.yaml @@ -1,6 +1,6 @@  ---  - name: stop provisioners -  include: stop_cluster.yaml +  include_tasks: stop_cluster.yaml  # delete the deployment objects that we had created  - name: delete provisioner api objects diff --git a/roles/openshift_storage_nfs/tasks/main.yml b/roles/openshift_storage_nfs/tasks/main.yml index c4e023c1e..24264fa43 100644 --- a/roles/openshift_storage_nfs/tasks/main.yml +++ b/roles/openshift_storage_nfs/tasks/main.yml @@ -1,7 +1,6 @@  ---  - name: setup firewall -  include: firewall.yml -  static: yes +  import_tasks: firewall.yml  - name: Install nfs-utils    package: name=nfs-utils state=present diff --git a/roles/os_firewall/tasks/main.yml b/roles/os_firewall/tasks/main.yml index c477d386c..99084cd3f 100644 --- a/roles/os_firewall/tasks/main.yml +++ b/roles/os_firewall/tasks/main.yml @@ -8,12 +8,12 @@    set_fact:      r_os_firewall_is_atomic: "{{ r_os_firewall_ostree_booted.stat.exists }}" -- include: firewalld.yml +- include_tasks: firewalld.yml    when:    - os_firewall_enabled | bool    - os_firewall_use_firewalld | bool -- include: iptables.yml +- include_tasks: iptables.yml    when:    - os_firewall_enabled | bool    - not os_firewall_use_firewalld | bool | 
