Workaround ansible-lint installation failure

* Bumps ansible-lint to master vesion (which got fix)
* Resolves bug with vars file placed inside tasks folder which caused
  linter to crash parsing them.
  https://github.com/ansible/ansible-lint/issues/595
* Includes tox.ini fix for HTTP proxies

Partial-Bug: #1848512
Partial-Bug: https://github.com/ansible/ansible-lint/issues/590
Partial-Bug: https://github.com/ansible/ansible-lint/issues/595
Change-Id: Ib505b05ef909d0ded9dd87a67166b8f821d62d14
This commit is contained in:
Sorin Sbarnea 2019-10-21 12:31:54 +01:00
parent f87828ce9c
commit 9afdcca1bc
5 changed files with 11 additions and 3 deletions

View File

@ -1,11 +1,14 @@
---
parseable: true
skip_list:
# One two excuses for adding skips: bugs in linter and recent linter bumping
- ANSIBLE0006
- ANSIBLE0012
- ANSIBLE0016
- '204' # [E204] Lines should be no longer than 120 chars
- '306' # [E306] Shells that use pipes should set the pipefail option
# E602 https://github.com/ansible/ansible-lint/issues/450
- '602' # [E602] Don't compare to empty string
# E405 is too new and we need to enable it in a separate change
- '405' # [E405] Remote package tasks should have a retry
- '701' # [E701] No 'galaxy_info' found

View File

@ -35,8 +35,10 @@ repos:
files: \.(yaml|yml)$
types: [file, yaml]
entry: yamllint --strict -f parsable
- repo: https://github.com/ansible/ansible-lint
rev: v4.0.1
# Revert to upstream branch fix for bug below is released (tagged)
# https://github.com/ansible/ansible-lint/issues/595
- repo: https://github.com/ansible/ansible-lint.git
rev: 0523c63f193d65313dcd743686d7fb0623a72ee8
hooks:
- id: ansible-lint
files: \.(yaml|yml)$

View File

@ -5,7 +5,7 @@
- name: Include vars for libvirt-nodepool
include_vars:
file: "{{ role_path }}/../../setup/overcloud/tasks/vars/libvirt_nodepool_vars.yml"
file: "{{ role_path }}/../../setup/overcloud/vars/libvirt_nodepool_vars.yml"
when: libvirt_nodepool|default(false)
- name: Check if libvirt is available

View File

@ -23,9 +23,12 @@ setenv =
PIP_DISABLE_PIP_VERSION_CHECK=1
passenv =
ANSIBLE_*
CURL_CA_BUNDLE
DOCKER_*
MOLECULE_*
REQUESTS_CA_BUNDLE
SSH_AUTH_SOCK
SSL_CERT_FILE
TERM
deps = -r{toxinidir}/test-requirements.txt
whitelist_externals =