)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"3e49455d7f67aa949e888c6eb4fcc2e559db55b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44185b94_ff27f0e7","updated":"2022-07-07 01:02:34.000000000","message":"recheck\n\nthis actually appears to be a mirror error getting snmp packages","commit_id":"045e8597ea4b213f1e414b3d8cbc255e58fc1253"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e9958dcaf132d44ea4d7a23f69fb4b8bd48621b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fbf31d34_47c9bbfe","updated":"2022-08-04 06:23:34.000000000","message":"approving this based on prior review; only changed function name to _get_run_cmd","commit_id":"0836432d272a2df9263f57833064034340ca897c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"2e5b8c4883cb83dfefcc6015d37be5f01b110a6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"43ba8c62_fb9f3dfe","updated":"2022-08-04 06:25:55.000000000","message":"recheck","commit_id":"0836432d272a2df9263f57833064034340ca897c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"90e435333eedbd1560a00b86cc688a2f92115df9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"20653256_455e7641","updated":"2022-08-05 05:12:16.000000000","message":"I will fix up the name in a follow-on to avoid having to rebase everything ontop of this","commit_id":"ed485c1bbf12c98e10a34dea89e2c0a52efd6723"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"ae0ce09f6e995afa5bc152149aca4b908b69fc33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"016ccfb3_4cc01f4e","updated":"2022-08-08 02:16:07.000000000","message":"recheck","commit_id":"ed485c1bbf12c98e10a34dea89e2c0a52efd6723"}],"testinfra/test_mirror.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"34d5ef109770acd2237c004049de20e65c73107b","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    apache \u003d host.service(\u0027apache2\u0027)"},{"line_number":22,"context_line":"    assert apache.is_running"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"def _run_cmd(host, port, scheme\u003d\u0027https\u0027, url\u003d\u0027\u0027):"},{"line_number":25,"context_line":"    hostname \u003d host.backend.get_hostname()"},{"line_number":26,"context_line":"    return f\u0027curl --resolve {hostname}:127.0.0.1 {scheme}://{hostname}:{port}{url}\u0027"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"03d61845_079c9cbb","line":24,"updated":"2022-07-19 22:09:52.000000000","message":"Nit this function name implies that this will run a command but all it does is return the command to run. Might be more appropriately called `_build_cmd_str` or similar.","commit_id":"a2c8410bc58ee31131fbb4141bf0c4e407cb1c4e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8df21fc3560fa1d83c77264e14e21b1f2a8cc11b","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    apache \u003d host.service(\u0027apache2\u0027)"},{"line_number":22,"context_line":"    assert apache.is_running"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"def _run_cmd(host, port, scheme\u003d\u0027https\u0027, url\u003d\u0027\u0027):"},{"line_number":25,"context_line":"    hostname \u003d host.backend.get_hostname()"},{"line_number":26,"context_line":"    return f\u0027curl --resolve {hostname}:127.0.0.1 {scheme}://{hostname}:{port}{url}\u0027"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"8dc7ed99_fc767063","line":24,"in_reply_to":"03d61845_079c9cbb","updated":"2022-08-04 06:22:50.000000000","message":"renamed _get_run_cmd()","commit_id":"a2c8410bc58ee31131fbb4141bf0c4e407cb1c4e"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"34d5ef109770acd2237c004049de20e65c73107b","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"def _run_cmd(host, port, scheme\u003d\u0027https\u0027, url\u003d\u0027\u0027):"},{"line_number":25,"context_line":"    hostname \u003d host.backend.get_hostname()"},{"line_number":26,"context_line":"    return f\u0027curl --resolve {hostname}:127.0.0.1 {scheme}://{hostname}:{port}{url}\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"def test_base_mirror(host):"},{"line_number":29,"context_line":"    # base mirror"}],"source_content_type":"text/x-python","patch_set":4,"id":"db05d4f2_2e5bc25e","line":26,"updated":"2022-07-19 22:09:52.000000000","message":"I\u0027m not sure why the old code felt it needed to talk to apache through all the IP addresses. Talking to localhost alone is probably fine. Maybe we wanted to ensure it was listening on the external IPs?","commit_id":"a2c8410bc58ee31131fbb4141bf0c4e407cb1c4e"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8df21fc3560fa1d83c77264e14e21b1f2a8cc11b","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"def _run_cmd(host, port, scheme\u003d\u0027https\u0027, url\u003d\u0027\u0027):"},{"line_number":25,"context_line":"    hostname \u003d host.backend.get_hostname()"},{"line_number":26,"context_line":"    return f\u0027curl --resolve {hostname}:127.0.0.1 {scheme}://{hostname}:{port}{url}\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"def test_base_mirror(host):"},{"line_number":29,"context_line":"    # base mirror"}],"source_content_type":"text/x-python","patch_set":4,"id":"e9382efd_ba2b6234","line":26,"in_reply_to":"db05d4f2_2e5bc25e","updated":"2022-08-04 06:22:50.000000000","message":"Maybe ipv6; but I find testing of that is a bit non-deterministic due to cloud differences.  I feel like this is enough coverage; I think the main gist of this is making sure apache is configured properly.","commit_id":"a2c8410bc58ee31131fbb4141bf0c4e407cb1c4e"}]}