search

integr8ly / installation

Integreatly Installer
83 stars 91 forks source link

The use of 'include' for tasks has been deprecated #205

Closed camilamacedo86 closed 5 years ago

camilamacedo86 commented 5 years ago

Description

The use of 'include' for tasks has been deprecated

Expected Behavior

Ansible script is not using deprecated features.

Actual Behavior

TASK [Install rhsso] *******************************************************************************************************************************************************************************************
[DEPRECATION WARNING]: The use of 'include' for tasks has been deprecated. Use 'import_tasks' for static inclusions or 'include_tasks' for dynamic inclusions. This feature will be removed in a future 
release. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
[DEPRECATION WARNING]: include is kept for backwards compatibility but usage is discouraged. The module documentation details page may explain more about this rationale.. This feature will be removed in a 
future release. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.

Environment

Server https://master.master-69d5.openshiftworkshop.com:443 openshift v3.10.14 kubernetes v1.10.0+b81c8f8

* Ansible version:
```shell
ansible --version
[WARNING]: log file at /root/ansible.log is not writeable and we cannot create it, aborting

ansible 2.4.6.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/ec2-user/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.5 (default, May 31 2018, 09:41:32) [GCC 4.8.5 20150623 (Red Hat 4.8.5-28)]

Steps to reproduce

  1. Create a new OCP instance via RHDMS. See here
  2. Perform the steps required to set up and run the install playbook.
camilamacedo86 commented 5 years ago

It still faced at 03/12/2018.

camilamacedo86 commented 5 years ago

The PR to fix it is done: https://github.com/integr8ly/installation/pull/235

camilamacedo86 commented 5 years ago

Done.