)]}'
{"playbooks/roles/mirror-update/tasks/rsync.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"50944034d5d0518dcc723dc7da75573c0f38a001","unresolved":true,"context_lines":[{"line_number":17,"context_line":"      - centos-stream"},{"line_number":18,"context_line":"      - epel"},{"line_number":19,"context_line":"      - fedora"},{"line_number":20,"context_line":"      - openeuler"},{"line_number":21,"context_line":"      - yum-puppetlabs"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Cleanup old unused rsync mirroring scripts"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8d2d97f9_dea92383","side":"PARENT","line":20,"updated":"2025-09-09 19:56:52.000000000","message":"Do you want to leave this script in place but update it to delete the content before we remove the script in a followup change? We\u0027ve done that with other cleanups in the past as it avoids an infra root needing to do things manually.","commit_id":"9a8f2e5f7f0de2fc9b253bded3ed535b15b414a3"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"c45116fdb0035d67754ee869c2e4ea54eed1d52e","unresolved":true,"context_lines":[{"line_number":17,"context_line":"      - centos-stream"},{"line_number":18,"context_line":"      - epel"},{"line_number":19,"context_line":"      - fedora"},{"line_number":20,"context_line":"      - openeuler"},{"line_number":21,"context_line":"      - yum-puppetlabs"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Cleanup old unused rsync mirroring scripts"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b1af56a3_111291e0","side":"PARENT","line":20,"in_reply_to":"8d2d97f9_dea92383","updated":"2025-09-17 17:57:40.000000000","message":"Thanks, I missed that was a feature of the script.","commit_id":"9a8f2e5f7f0de2fc9b253bded3ed535b15b414a3"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"80d79d8d8b960503ae310c446e5a511064e41fcf","unresolved":false,"context_lines":[{"line_number":17,"context_line":"      - centos-stream"},{"line_number":18,"context_line":"      - epel"},{"line_number":19,"context_line":"      - fedora"},{"line_number":20,"context_line":"      - openeuler"},{"line_number":21,"context_line":"      - yum-puppetlabs"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Cleanup old unused rsync mirroring scripts"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4e794440_ddb447fd","side":"PARENT","line":20,"in_reply_to":"b1af56a3_111291e0","updated":"2025-09-17 18:24:36.000000000","message":"Done","commit_id":"9a8f2e5f7f0de2fc9b253bded3ed535b15b414a3"}],"testinfra/test_mirror-update.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"80d79d8d8b960503ae310c446e5a511064e41fcf","unresolved":true,"context_lines":[{"line_number":35,"context_line":"        assert f.exists"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"def test_rsync_scripts_removed(host):"},{"line_number":38,"context_line":"    for script in [\u0027opensuse\u0027, \u0027centos\u0027]:"},{"line_number":39,"context_line":"        f \u003d host.file(\u0027/usr/local/bin/%s-mirror-update\u0027 % script)"},{"line_number":40,"context_line":"        assert not f.exists"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"0fc1f111_5dc5bf90","line":38,"updated":"2025-09-17 18:24:36.000000000","message":"Nit you can add the removed script to this list.","commit_id":"91c6af0c3b27d6c7d554cb93024fbceeb948ec6a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a6f6aa21f08db7ab37a286a94cabd6b76ff1813f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        assert f.exists"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"def test_rsync_scripts_removed(host):"},{"line_number":38,"context_line":"    for script in [\u0027opensuse\u0027, \u0027centos\u0027]:"},{"line_number":39,"context_line":"        f \u003d host.file(\u0027/usr/local/bin/%s-mirror-update\u0027 % script)"},{"line_number":40,"context_line":"        assert not f.exists"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dfbcde1f_26a426a0","line":38,"in_reply_to":"0fc1f111_5dc5bf90","updated":"2025-09-23 20:02:27.000000000","message":"Done","commit_id":"91c6af0c3b27d6c7d554cb93024fbceeb948ec6a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"80d79d8d8b960503ae310c446e5a511064e41fcf","unresolved":true,"context_lines":[{"line_number":60,"context_line":"def test_keytabs_removed(host):"},{"line_number":61,"context_line":"    for keytab in [\u0027/etc/opensuse.keytab\u0027, \u0027/etc/centos.keytab\u0027]:"},{"line_number":62,"context_line":"        f \u003d host.file(keytab)"},{"line_number":63,"context_line":"        assert not f.exists"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"def test_afs_release_script(host):"},{"line_number":66,"context_line":"    f \u003d host.file(\u0027/opt/afs-release/release-volumes.py\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"27b52537_faa80012","line":63,"updated":"2025-09-17 18:24:36.000000000","message":"Nit you could add the removed keytab to this test.","commit_id":"91c6af0c3b27d6c7d554cb93024fbceeb948ec6a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a6f6aa21f08db7ab37a286a94cabd6b76ff1813f","unresolved":false,"context_lines":[{"line_number":60,"context_line":"def test_keytabs_removed(host):"},{"line_number":61,"context_line":"    for keytab in [\u0027/etc/opensuse.keytab\u0027, \u0027/etc/centos.keytab\u0027]:"},{"line_number":62,"context_line":"        f \u003d host.file(keytab)"},{"line_number":63,"context_line":"        assert not f.exists"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"def test_afs_release_script(host):"},{"line_number":66,"context_line":"    f \u003d host.file(\u0027/opt/afs-release/release-volumes.py\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"b7b9e219_23559d8a","line":63,"in_reply_to":"27b52537_faa80012","updated":"2025-09-23 20:02:27.000000000","message":"Done","commit_id":"91c6af0c3b27d6c7d554cb93024fbceeb948ec6a"}]}