From 4aa181a77cb614dc6a15198d1ce61357ac24c4e5 Mon Sep 17 00:00:00 2001
From: Jeremy Stanley <fungi@yuggoth.org>
Date: Thu, 28 Sep 2017 19:49:56 +0000
Subject: [PATCH] Stop running ansible-lint on playbooks

The following form raises "ANSIBLE0013 Use shell only when shell
functionality is required:

  shell: '! something'

Chances are its heuristic fails to look for use of the negation (!)
shell operator.

Rather than exclude ANSIBLE0013, just stop running it completely for
now since we've had to do the same on openstack-zuul-jobs in the
I14896450b16f8e65128804b44a643da63580812d change.

Change-Id: Ie9da24a4ab20d9dd0bf7ebb2e3d957b9d8563900
Depends-On: I60daa67e7154d2bf621305e0e0aa6f4db49033c1
---
 tox.ini | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/tox.ini b/tox.ini
index e1c23342b..3fb274880 100644
--- a/tox.ini
+++ b/tox.ini
@@ -31,11 +31,6 @@ commands =
   # Ansible Syntax Check
   bash -c "find playbooks -type f -regex '.*.y[a]?ml' -print0 | xargs -n1 -0 \
     ansible-playbook --syntax-check -i tests/inventory 1>/dev/null"
-  # Ansible Lint Check
-  # NOTE(pabelanger): Ignore the following checks:
-  #   ANSIBlE0012: Commands should not change things if nothing needs doing
-  bash -c "find playbooks -type f -regex '.*.y[a]?ml' -print0 | xargs -n1 -0 \
-    ansible-lint -x ANSIBLE0006,ANSIBLE0012 2> /dev/null"
 
 [testenv:venv]
 commands = {posargs}