From 68c355d346203d2c32bc65f9f99d5bf7728f105a Mon Sep 17 00:00:00 2001
From: Monty Taylor <mordred@inaugust.com>
Date: Fri, 27 Jul 2018 11:46:39 -0400
Subject: [PATCH] Add linters check to make sure site-variables is yaml

A YAML syntax error in site-variables was merged recently. Errors in
that file break basically everything, so add a check that the file at
least parses.

Change-Id: I9b2c47a0461275e1d53aabe387f0309719d46b77
---
 tox.ini | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tox.ini b/tox.ini
index 83b5cec47a..430e318b10 100644
--- a/tox.ini
+++ b/tox.ini
@@ -41,6 +41,8 @@ commands =
   bash -c "find playbooks -type f -regex '.*.ya?ml' -exec \
     ansible-playbook --syntax-check -i {toxinidir}/tests/inventory \
     -e @tests/vars.yaml \{\} + > /dev/null"
+  # Make sure site-variables at least parses
+  python -c 'import yaml ; yaml.load(open("zuul/site-variables.yaml", "r"))'
 
 [testenv:gerrit]
 deps = PyYAML