Skip to content
This repository has been archived by the owner on Jul 7, 2020. It is now read-only.

Ansible/deprecation warnings #556

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Jan 15, 2019

While running tests, several deprecation warnings are reported. With this PR the warnings get reduced. PR #554 addresses the remaining ones.


This change is Reviewable

Ansible complains about the following:

  [WARNING]: Found both group and host with same name: master

As the hostname is "master", rename the group to "masters" (with "s",
even though there is only one).

Signed-off-by: Niels de Vos <ndevos@redhat.com>
Ansible complains that 'include' is deprecated:

    [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.

Instead of 'include', use 'include_tasks'.

Signed-off-by: Niels de Vos <ndevos@redhat.com>
@nixpanic nixpanic requested a review from jarrpa January 15, 2019 12:08
Ansible complains that checking the return value of a filter action is
deprecated. It seems required to use "is" instead of the pipe ("|").

    [DEPRECATION WARNING]: Using tests as filters is deprecated. Instead of using
    `result|version_compare` use `result is version_compare`. This feature will be
    removed in version 2.9. Deprecation warnings can be disabled by setting
    deprecation_warnings=False in ansible.cfg.

Signed-off-by: Niels de Vos <ndevos@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant