)]}'
{"inventory/service/groups.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a2f2db6ce7ae629fe1e6287129f2cce4fe7c54d7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"plugin: yamlgroup"},{"line_number":2,"context_line":"groups:"},{"line_number":3,"context_line":"  adns: adns*.open*.org"},{"line_number":4,"context_line":"  afs: afs[0-9]*.open*.org"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"ff570b3c_55d574b6","line":1,"updated":"2020-05-28 22:20:22.000000000","message":"Why does this file end up in inventory/service and not inventory/base like the openstack inventory does?","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d4ad2ba6313e34d5a86a1605bf04dbdf03914403","unresolved":false,"context_lines":[{"line_number":1,"context_line":"plugin: yamlgroup"},{"line_number":2,"context_line":"groups:"},{"line_number":3,"context_line":"  adns: adns*.open*.org"},{"line_number":4,"context_line":"  afs: afs[0-9]*.open*.org"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"ff570b3c_0081f84a","line":1,"in_reply_to":"ff570b3c_55d574b6","updated":"2020-05-28 22:53:07.000000000","message":"We need 2 sets of hostvars (We only have 1 in this patch) so we put the groups.yaml into one dir and the hosts.yaml into the other dir - ansible will pull hostvars from adjacency to both of them.\n\nI figured groups are really associted more with the logical services and hosts themselves are more about the underlying nodes, so figured putting base hostvars with hosts and service hostvars with groups made sense.","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"}],"playbooks/zuul/run-base.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a2f2db6ce7ae629fe1e6287129f2cce4fe7c54d7","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      include_role:"},{"line_number":12,"context_line":"        name: write-inventory"},{"line_number":13,"context_line":"      vars:"},{"line_number":14,"context_line":"        write_inventory_dest: /home/zuul/src/opendev.org/opendev/system-config/inventory/base/inventory.yaml"},{"line_number":15,"context_line":"        write_inventory_exclude_hostvars:"},{"line_number":16,"context_line":"          - ansible_user"},{"line_number":17,"context_line":"          - ansible_python_interpreter"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"ff570b3c_75035843","line":14,"updated":"2020-05-28 22:20:22.000000000","message":"We don\u0027t need to write the inventory if we are loading it out of the git dir (its already written).","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"8ba68ecf742ac168d0d1e72972ef14762800d041","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      include_role:"},{"line_number":12,"context_line":"        name: write-inventory"},{"line_number":13,"context_line":"      vars:"},{"line_number":14,"context_line":"        write_inventory_dest: /home/zuul/src/opendev.org/opendev/system-config/inventory/base/gate-hosts.yaml"},{"line_number":15,"context_line":"        write_inventory_exclude_hostvars:"},{"line_number":16,"context_line":"          - ansible_user"},{"line_number":17,"context_line":"          - ansible_python_interpreter"}],"source_content_type":"text/x-yaml","patch_set":24,"id":"ff570b3c_cdfb62bb","line":14,"range":{"start_line":14,"start_character":89,"end_line":14,"end_character":109},"updated":"2020-06-03 21:22:45.000000000","message":"Understanding why this is different than base/hosts.yaml is much easier now. Thanks!","commit_id":"053e51fd3acdb07564a8685aaabf1399f71edb4b"}],"tools/prep-apply.sh":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a2f2db6ce7ae629fe1e6287129f2cce4fe7c54d7","unresolved":false,"context_lines":[{"line_number":43,"context_line":"virtualenv --system-site-packages /tmp/apply-ansible-env"},{"line_number":44,"context_line":"/tmp/apply-ansible-env/bin/pip install ansible"},{"line_number":45,"context_line":"# Install puppet"},{"line_number":46,"context_line":"inv\u003d/home/zuul/src/opendev.org/opendev/system-config/inventory/service/hosts"},{"line_number":47,"context_line":"cat \u003e $inv \u003c\u003cEOF"},{"line_number":48,"context_line":"localhost ansible_connection\u003dlocal"},{"line_number":49,"context_line":"EOF"}],"source_content_type":"text/x-sh","patch_set":12,"id":"ff570b3c_5583b4a9","line":46,"range":{"start_line":46,"start_character":63,"end_line":46,"end_character":76},"updated":"2020-05-28 22:20:22.000000000","message":"We use base/hosts and service/hosts in different places. Should we be consistent?","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d4ad2ba6313e34d5a86a1605bf04dbdf03914403","unresolved":false,"context_lines":[{"line_number":43,"context_line":"virtualenv --system-site-packages /tmp/apply-ansible-env"},{"line_number":44,"context_line":"/tmp/apply-ansible-env/bin/pip install ansible"},{"line_number":45,"context_line":"# Install puppet"},{"line_number":46,"context_line":"inv\u003d/home/zuul/src/opendev.org/opendev/system-config/inventory/service/hosts"},{"line_number":47,"context_line":"cat \u003e $inv \u003c\u003cEOF"},{"line_number":48,"context_line":"localhost ansible_connection\u003dlocal"},{"line_number":49,"context_line":"EOF"}],"source_content_type":"text/x-sh","patch_set":12,"id":"ff570b3c_4092d0a0","line":46,"range":{"start_line":46,"start_character":63,"end_line":46,"end_character":76},"in_reply_to":"ff570b3c_5583b4a9","updated":"2020-05-28 22:53:07.000000000","message":"We need this to go into the service dir so that it picks up the hostvars there. We don\u0027t yet have hostvars in base- and for puppet we\u0027re not going to need them (but we do need service-specific vars for puppet, because install-puppet needs puppet vars)","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"}],"tox.ini":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a2f2db6ce7ae629fe1e6287129f2cce4fe7c54d7","unresolved":false,"context_lines":[{"line_number":33,"context_line":"commands \u003d sphinx-build -W -E -b html doc/source doc/build/html"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[testenv:testinfra]"},{"line_number":36,"context_line":"commands \u003d py.test --junit-xml junit.xml --connection\u003dansible --ansible-inventory\u003d/home/zuul/src/opendev.org/opendev/system-config/inventory/base/inventory.yaml -v testinfra {posargs}"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[flake8]"},{"line_number":39,"context_line":"show-source \u003d True"}],"source_content_type":"text/x-properties","patch_set":12,"id":"ff570b3c_d5c064e3","line":36,"range":{"start_line":36,"start_character":141,"end_line":36,"end_character":160},"updated":"2020-05-28 22:20:22.000000000","message":"We are calling this base/hosts.yaml now right?","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d4ad2ba6313e34d5a86a1605bf04dbdf03914403","unresolved":false,"context_lines":[{"line_number":33,"context_line":"commands \u003d sphinx-build -W -E -b html doc/source doc/build/html"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"[testenv:testinfra]"},{"line_number":36,"context_line":"commands \u003d py.test --junit-xml junit.xml --connection\u003dansible --ansible-inventory\u003d/home/zuul/src/opendev.org/opendev/system-config/inventory/base/inventory.yaml -v testinfra {posargs}"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[flake8]"},{"line_number":39,"context_line":"show-source \u003d True"}],"source_content_type":"text/x-properties","patch_set":12,"id":"ff570b3c_a0b80c2a","line":36,"range":{"start_line":36,"start_character":141,"end_line":36,"end_character":160},"in_reply_to":"ff570b3c_d5c064e3","updated":"2020-05-28 22:53:07.000000000","message":"inventory is the one written out by the test setup. hosts.yaml is the real one.","commit_id":"bf6aeda4d88f1da77177e85e99d27a379d56870c"}]}