windmill/ansible-role-zuul
Code Issues Proposed changes

Define zookeeper hosts for zuul configuration

Browse Source 
Zuul 3.7.0 introduced a breaking configuration change, we now need to
define zookeeper hosts by default.

Disable 306 for ansible-lint until we can decided best fix.

Change-Id: Idd54f1ea75cccbb0dab7978ca4d7778fbc05bf42
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2019-03-22 18:14:19 -04:00
parent 7c804cd9b5
commit a4a10fc062
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
templates/etc/zuul/zuul.conf
View File

@ -5,6 +5,9 @@
port = 4730 port = 4730
server = 127.0.0.1 server = 127.0.0.1
[zookeeper]
hosts = localhost:2181
[gearman_server] [gearman_server]
listen_address = 127.0.0.1 listen_address = 127.0.0.1
log_config = /etc/zuul/gearman-logging.conf log_config = /etc/zuul/gearman-logging.conf

2
tox.ini
View File

@ -31,7 +31,7 @@ commands =
flake8 flake8
# Ansible Lint Check # Ansible Lint Check
bash -c "find . -not -path '*/\.*' -type f -regex '.*.y[a]?ml' -print0 | \ bash -c "find . -not -path '*/\.*' -type f -regex '.*.y[a]?ml' -print0 | \
xargs -t -n1 -0 ansible-lint" xargs -t -n1 -0 ansible-lint -x 306"
# Ansible Syntax Check # Ansible Syntax Check
bash -c "find tests -type f -regex '.*.y[a]?ml' -print | xargs -t -n1 \ bash -c "find tests -type f -regex '.*.y[a]?ml' -print | xargs -t -n1 \
ansible-playbook --syntax-check -i tests/inventory \ ansible-playbook --syntax-check -i tests/inventory \