)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"23f8fc9f33e267f59d69d5a6f26301ed97770d37","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"96e33193_3de5b401","updated":"2022-10-12 17:47:30.000000000","message":"recheck","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"3ff8d8e7588d6939806e07efa502b3e29f1b8c04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"5b1389d9_8dd22a56","updated":"2022-10-24 03:35:35.000000000","message":"recheck","commit_id":"51611845d457191596b05bedca18d5206b2236aa"}],"playbooks/zuul/run-production-playbook-post.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2bd9c1f532d6bb0e757951f057140bd7940d78e9","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - add-bastion-host"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- hosts: bastion"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - name: Encrypt log"},{"line_number":8,"context_line":"      when: infra_prod_playbook_encrypt_log|default(False)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"3f11b9be_a845462c","line":5,"updated":"2022-10-12 21:51:26.000000000","message":"Should this be bastion[0] ?","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"91b975f17a534957382d7b1c175ca7b08127effa","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - add-bastion-host"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- hosts: bastion"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - name: Encrypt log"},{"line_number":8,"context_line":"      when: infra_prod_playbook_encrypt_log|default(False)"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"935f0edb_6305385d","line":5,"in_reply_to":"3f11b9be_a845462c","updated":"2022-10-14 02:20:55.000000000","message":"The bastion group is only supposed to have one member, but we can do this for clarity","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"}],"playbooks/zuul/run-production-playbook.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2bd9c1f532d6bb0e757951f057140bd7940d78e9","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - add-bastion-host"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- hosts: bastion"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - name: Run the production playbook and capture logs"},{"line_number":8,"context_line":"      block:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"7592856a_c156268a","line":5,"updated":"2022-10-12 21:51:26.000000000","message":"Should this do bastion[0] as well?","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"91b975f17a534957382d7b1c175ca7b08127effa","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - add-bastion-host"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- hosts: bastion"},{"line_number":6,"context_line":"  tasks:"},{"line_number":7,"context_line":"    - name: Run the production playbook and capture logs"},{"line_number":8,"context_line":"      block:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"c966fdee_dbbab771","line":5,"in_reply_to":"7592856a_c156268a","updated":"2022-10-14 02:20:55.000000000","message":"Ack","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"}],"zuul.d/infra-prod.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"2bd9c1f532d6bb0e757951f057140bd7940d78e9","unresolved":true,"context_lines":[{"line_number":58,"context_line":"      - playbooks/roles/install-ansible/"},{"line_number":59,"context_line":"      - playbooks/roles/logrotate/"},{"line_number":60,"context_line":"      - playbooks/roles/root-keys/"},{"line_number":61,"context_line":"      - inventory/service/group_vars/bastion.yaml"},{"line_number":62,"context_line":"      - playbooks/zuul/run-production-playbook.yaml"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"b545211a_51fc6d09","line":61,"updated":"2022-10-12 21:51:26.000000000","message":"This file gets renamed back in https://review.opendev.org/c/opendev/system-config/+/858476. It is probably best to update these matchers there. However since these are only file matchers and the host var file vs group var file will take effect either way I think this is fine here.","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"91b975f17a534957382d7b1c175ca7b08127effa","unresolved":false,"context_lines":[{"line_number":58,"context_line":"      - playbooks/roles/install-ansible/"},{"line_number":59,"context_line":"      - playbooks/roles/logrotate/"},{"line_number":60,"context_line":"      - playbooks/roles/root-keys/"},{"line_number":61,"context_line":"      - inventory/service/group_vars/bastion.yaml"},{"line_number":62,"context_line":"      - playbooks/zuul/run-production-playbook.yaml"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"d915284a_9fc09814","line":61,"in_reply_to":"b545211a_51fc6d09","updated":"2022-10-14 02:20:55.000000000","message":"I will update in that change","commit_id":"2eed5afd3956e3225fa08f5a35581aa699cee9c5"}]}
