Fix ansible-lint failure

Update to newest version

Change-Id: I884a12f5c408d238e37a959f56609c823522d40d

.ansible-lint

@@ -1,19 +1,59 @@
 exclude_paths:
     - roles/validate-ui/.travis.yml
+    - roles/virthost-full-cleanup/tasks/main.yml
     # we don't want to see failures from sideloaded repos
     - ../tripleo-upgrade/
+    # These playbooks are from tq repo and checked there
+    - playbooks/baremetal-full-deploy.yml
+    - playbooks/baremetal-prep-virthost.yml
+    - playbooks/deploy-delete.yml
+    - playbooks/multinode-minion.yml
+    - playbooks/multinode-multiple-overcloud.yml
+    - playbooks/multinode.yml
+    - playbooks/quickstart-extras-standalone.yml
+    - playbooks/quickstart-extras.yml
+    - playbooks/scale_nodes.yml
+    # No any idea why those don't pass linter
+    - playbooks/build-images-v2.yml
+    - playbooks/build-test-packages.yml
 parseable: true
 quiet: false
 skip_list:
     # TODO(ssbarnea): Gradually remove these skips ASAP
-    - 204  # Lines should be no longer than 160 chars
+    - syntax-check
-    - 302  # rm used in place of argument state=absent to file module
+    - internal-error
-    - 303  # sed used in place of template, replace or lineinfile module
+    - unnamed-task
-    - 602  # Don't compare to empty string
+    - risky-file-permissions
-    - 504  # Do not use 'local_action', use 'delegate_to: localhost'
+    - ignore-errors
-    - 701  # No 'galaxy_info' found
+    - command-instead-of-module
-    - 208  # [E208] File permissions unset or incorrect
+    - meta-no-info
-    - 106  # [E106] Role name {} does not match ^[a-z][a-z0-9_]+$ pattern
+    - risky-shell-pipe
-    - 207  # [E207] Nested jinja pattern
+    - deprecated-command-syntax
-    - 306  # [E306] Shells that use pipes should set the pipefail option
+    - empty-string-compare
+    - role-name
+    - experimental
+    - no-changed-when
+    - literal-compare
+    - command-instead-of-shell
+    - package-latest
+    - deprecated-local-action
+    - no-jinja-nesting
+
 verbosity: 1
+mock_modules:
+  - authorized_key
+  - virt
+  - ini_file
+  - openvswitch_bridge
+  - os_stack
+  - redhat_subscription
+  - synchronize
+mock_roles:
+  - tripleo-inventory
+  - repo-setup
+  - common
+  - tripleo-upgrade
+  - fetch-images
+  - tripleo
+  - parts/kvm
+  - collect-logs

.pre-commit-config.yaml

@@ -26,7 +26,7 @@ repos:
         types: [file, yaml]
         entry: yamllint --strict -f parsable
   - repo: https://github.com/ansible/ansible-lint
-    rev: v4.3.7
+    rev: v5.3.2
     hooks:
       - id: ansible-lint
         always_run: true