Browse files

Refactor of linting/config validation

* Use the built-in pykwalify cli validator
* Use set -e on loop inside tox.ini to ensure invalid configs fail CI

Change-Id: I251f7ead8393b97e93de03dc3b6accbdd9670092
  • Loading branch information...
akrzos committed Sep 21, 2017
1 parent 0804dc9 commit 1a65cd1feb291f2cd0d85069bb5e1ed07fb6a984
Showing with 4 additions and 32 deletions.
  1. +0 −27 ci-scripts/linters/
  2. +0 −1 conf/browbeat-workloads.yml
  3. +4 −4 tox.ini

This file was deleted.

Oops, something went wrong.
@@ -3,7 +3,6 @@ browbeat:
results: results/
rerun: 1
cloud_name: openstack
overcloud_credentials: /home/stack/overcloudrc
enabled: false
regather: false
@@ -17,10 +17,10 @@ commands =
bash -c "cd ansible; find . -type f -regex '.*.y[a]?ml' -print0 | xargs -t -n1 -0 \
ansible-lint \
python ci-scripts/linters/ browbeat/validate.yaml browbeat-config.yaml
python ci-scripts/linters/ browbeat/validate.yaml browbeat-complete.yaml
bash -c "for config in $(ls conf/); do \
python ci-scripts/linters/ browbeat/validate.yaml conf/$config; done"
pykwalify -d browbeat-config.yaml -s browbeat/validate.yaml
pykwalify -d browbeat-complete.yaml -s browbeat/validate.yaml
bash -c "set -e; for config in $(ls conf/); do \
pykwalify -d conf/$config -s browbeat/validate.yaml; done"
commands = flake8 {posargs}

0 comments on commit 1a65cd1

Please sign in to comment.