)]}'
{".zuul.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1da40bef06f0e94ac0eb7a754b7752c10005d074","unresolved":false,"context_lines":[{"line_number":1445,"context_line":"    semaphore: infra-prod-playbook"},{"line_number":1446,"context_line":"    run: playbooks/zuul/run-production-playbook.yaml"},{"line_number":1447,"context_line":"    required-projects:"},{"line_number":1448,"context_line":"      - opendev/system-config"},{"line_number":1449,"context_line":"    vars:"},{"line_number":1450,"context_line":"      infra_prod_ansible_forks: 5"},{"line_number":1451,"context_line":"      infra_prod_playbook_collect_log: false"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_281ef9b9","line":1448,"updated":"2020-04-13 18:16:48.000000000","message":"Should we go ahead and add this to all the jobs since we are setting the var to point at it below on line 1452?","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"2d25331fd5e1ed34f53ff12940c0a87406145e2c","unresolved":false,"context_lines":[{"line_number":1445,"context_line":"    semaphore: infra-prod-playbook"},{"line_number":1446,"context_line":"    run: playbooks/zuul/run-production-playbook.yaml"},{"line_number":1447,"context_line":"    required-projects:"},{"line_number":1448,"context_line":"      - opendev/system-config"},{"line_number":1449,"context_line":"    vars:"},{"line_number":1450,"context_line":"      infra_prod_ansible_forks: 5"},{"line_number":1451,"context_line":"      infra_prod_playbook_collect_log: false"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_a8184981","line":1448,"in_reply_to":"3f4c43b2_281ef9b9","updated":"2020-04-13 18:19:07.000000000","message":"It is - this is the base job.\n\nOh - project-config, no it\u0027s only used in 3 of them. We could set the var on those - but this seemed more expedient.","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"55905c468606138a89c058e7930293af15b82cfe","unresolved":false,"context_lines":[{"line_number":1445,"context_line":"    semaphore: infra-prod-playbook"},{"line_number":1446,"context_line":"    run: playbooks/zuul/run-production-playbook.yaml"},{"line_number":1447,"context_line":"    required-projects:"},{"line_number":1448,"context_line":"      - opendev/system-config"},{"line_number":1449,"context_line":"    vars:"},{"line_number":1450,"context_line":"      infra_prod_ansible_forks: 5"},{"line_number":1451,"context_line":"      infra_prod_playbook_collect_log: false"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_7548d5cb","line":1448,"in_reply_to":"3f4c43b2_a8184981","updated":"2020-04-13 20:13:14.000000000","message":"Ya sorry I meant openstack/project-config. It just seems odd to me that we set thie var on line 1452 below, then use that var in every run production playbook run, but only ensure the repo is synced when we directly consume it.\n\nSlightly concerned that the next thing that needs project-config will completely miss that.","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"0fab5839b96fc44752be159d0bfc1a3166a268ee","unresolved":false,"context_lines":[{"line_number":1449,"context_line":"    vars:"},{"line_number":1450,"context_line":"      infra_prod_ansible_forks: 5"},{"line_number":1451,"context_line":"      infra_prod_playbook_collect_log: false"},{"line_number":1452,"context_line":"      project_config_src: /home/zuulcd/src/opendev.org/openstack/project-config"},{"line_number":1453,"context_line":"    nodeset:"},{"line_number":1454,"context_line":"      nodes: []"},{"line_number":1455,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_4391a699","line":1452,"updated":"2020-04-13 18:25:23.000000000","message":"This is where we set it to not-/opt/project-config for real runs. But good catch - that\u0027s not going to get passed through to the inner invocation ... fixing.","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"}],"playbooks/group_vars/puppet.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3157e8a1f115b639bacb5e505876947949551443","unresolved":false,"context_lines":[{"line_number":3,"context_line":"copy_puppet: true"},{"line_number":4,"context_line":"puppet_reports: none"},{"line_number":5,"context_line":"manage_config: true"},{"line_number":6,"context_line":"manifest: /opt/system-config/production/manifests/site.pp"},{"line_number":7,"context_line":"manifest_base: /opt/system-config/production"},{"line_number":8,"context_line":"mgmt_manifestpath: /home/zuul/src/opendev.org/opendev/system-config/"},{"line_number":9,"context_line":"puppet_logdest: syslog"},{"line_number":10,"context_line":"mgmt_hieradata: /etc/ansible/hosts"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"3f4c43b2_8769da26","line":7,"range":{"start_line":6,"start_character":0,"end_line":7,"end_character":44},"updated":"2020-04-15 00:09:22.000000000","message":"Does this need updating too?","commit_id":"4da20a8bb76f09be3dbe4068da8349a7986a96d4"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"8c80f829f4c9c32379b896f24fef9e7b5c334ac4","unresolved":false,"context_lines":[{"line_number":3,"context_line":"copy_puppet: true"},{"line_number":4,"context_line":"puppet_reports: none"},{"line_number":5,"context_line":"manage_config: true"},{"line_number":6,"context_line":"manifest: /opt/system-config/production/manifests/site.pp"},{"line_number":7,"context_line":"manifest_base: /opt/system-config/production"},{"line_number":8,"context_line":"mgmt_manifestpath: /home/zuul/src/opendev.org/opendev/system-config/"},{"line_number":9,"context_line":"puppet_logdest: syslog"},{"line_number":10,"context_line":"mgmt_hieradata: /etc/ansible/hosts"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"3f4c43b2_c7e1229d","line":7,"range":{"start_line":6,"start_character":0,"end_line":7,"end_character":44},"in_reply_to":"3f4c43b2_8769da26","updated":"2020-04-15 00:11:39.000000000","message":"No - those are remote paths on the server where puppet apply runs.","commit_id":"4da20a8bb76f09be3dbe4068da8349a7986a96d4"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3157e8a1f115b639bacb5e505876947949551443","unresolved":false,"context_lines":[{"line_number":9,"context_line":"puppet_logdest: syslog"},{"line_number":10,"context_line":"mgmt_hieradata: /etc/ansible/hosts"},{"line_number":11,"context_line":"mgmt_puppet_module_dir: /etc/puppet/modules"},{"line_number":12,"context_line":"puppet_hieradata_link_dest: /opt/system-config/hieradata"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"3f4c43b2_e776de06","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":56},"updated":"2020-04-15 00:09:22.000000000","message":"Ditto","commit_id":"4da20a8bb76f09be3dbe4068da8349a7986a96d4"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"8c80f829f4c9c32379b896f24fef9e7b5c334ac4","unresolved":false,"context_lines":[{"line_number":9,"context_line":"puppet_logdest: syslog"},{"line_number":10,"context_line":"mgmt_hieradata: /etc/ansible/hosts"},{"line_number":11,"context_line":"mgmt_puppet_module_dir: /etc/puppet/modules"},{"line_number":12,"context_line":"puppet_hieradata_link_dest: /opt/system-config/hieradata"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"3f4c43b2_a7033642","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":56},"in_reply_to":"3f4c43b2_e776de06","updated":"2020-04-15 00:11:39.000000000","message":"Same - that\u0027s remote hieradata location","commit_id":"4da20a8bb76f09be3dbe4068da8349a7986a96d4"}],"playbooks/roles/sync-project-config/defaults/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1da40bef06f0e94ac0eb7a754b7752c10005d074","unresolved":false,"context_lines":[{"line_number":1,"context_line":"project_config_dest: /opt/project-config"},{"line_number":2,"context_line":"project_config_subdir: \"\""},{"line_number":3,"context_line":"# When running manage-projects locally, use /opt/project-config instead of zuul checkout"},{"line_number":4,"context_line":"project_config_src: /opt/project-config"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_08769dd9","line":4,"updated":"2020-04-13 18:16:48.000000000","message":"We don\u0027t seem to ever set this to /home/zuul/src/opendev.org/openstack/project-config outside of testing?\n\nAlso we may need something to update /opt/project-config in the manual trigger case? That way it doesn\u0027t stay extra stale outside of the zuul triggered cases.","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"0fab5839b96fc44752be159d0bfc1a3166a268ee","unresolved":false,"context_lines":[{"line_number":1,"context_line":"project_config_dest: /opt/project-config"},{"line_number":2,"context_line":"project_config_subdir: \"\""},{"line_number":3,"context_line":"# When running manage-projects locally, use /opt/project-config instead of zuul checkout"},{"line_number":4,"context_line":"project_config_src: /opt/project-config"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3f4c43b2_a35b4a2c","line":4,"in_reply_to":"3f4c43b2_08769dd9","updated":"2020-04-13 18:25:23.000000000","message":"For the manual triggered case, both /opt/system-config and /opt/project-config will need to be updated by hand - because they are only ever used for by-hand runs, so there is no \"natural\" state for them.","commit_id":"f7f6ffe217cb9fe82ce997aad1b1b6c8a4caf099"}],"tools/apply-test.sh":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c719d1a66d9a1e29e83f626547f411188a3688a7","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    mkdir applytest"},{"line_number":21,"context_line":"fi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"if [[ -d /home/zuul -a ! -d /home/zuulcd ]] ; then"},{"line_number":24,"context_line":"    ln -s /home/zuul /home/zuulcd"},{"line_number":25,"context_line":"fi"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-sh","patch_set":16,"id":"3f4c43b2_b8cfdb7d","line":23,"updated":"2020-04-14 19:13:54.000000000","message":"linters: E040 Syntax error: syntax error in conditional expression","commit_id":"64f7d80b9650eba3de3f3953710f53b0933ec664"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c719d1a66d9a1e29e83f626547f411188a3688a7","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    mkdir applytest"},{"line_number":21,"context_line":"fi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"if [[ -d /home/zuul -a ! -d /home/zuulcd ]] ; then"},{"line_number":24,"context_line":"    ln -s /home/zuul /home/zuulcd"},{"line_number":25,"context_line":"fi"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-sh","patch_set":16,"id":"3f4c43b2_98cc9f8a","line":23,"updated":"2020-04-14 19:13:54.000000000","message":"linters: E040 Syntax error: syntax error near `-a\u0027","commit_id":"64f7d80b9650eba3de3f3953710f53b0933ec664"}]}