)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"9595d64a343f499d74419f45c59e1448b7d6f77f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"364a6849_f8e1b69f","updated":"2023-12-14 10:01:17.000000000","message":"These manifests caused me some confusion when we tried to use them in new test cases. I think it is prudent to bury them in their intended context.","commit_id":"1c31973d33223978b5ae096eeb46f809465cea5e"}],"test/unit/common/middleware/test_slo.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"9595d64a343f499d74419f45c59e1448b7d6f77f","unresolved":true,"context_lines":[{"line_number":2157,"context_line":"        _raw_ranges_manifest \u003d ["},{"line_number":2158,"context_line":"            {\u0027name\u0027: \u0027/gettest/b_10\u0027, \u0027hash\u0027: md5hex(\u0027b\u0027 * 10), \u0027bytes\u0027: \u002710\u0027,"},{"line_number":2159,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2160,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u00271-99\u0027},"},{"line_number":2161,"context_line":"            {\u0027name\u0027: \u0027/gettest/c_15\u0027, \u0027hash\u0027: md5hex(\u0027c\u0027 * 15), \u0027bytes\u0027: \u002715\u0027,"},{"line_number":2162,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2163,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u0027100-200\u0027},"}],"source_content_type":"text/x-python","patch_set":1,"id":"8068d03f_92a56459","side":"PARENT","line":2160,"range":{"start_line":2160,"start_character":43,"end_line":2160,"end_character":58},"updated":"2023-12-14 10:01:17.000000000","message":"This manifest was quite confusing to me because the ranges are actually invalid for the given byte size, so it\u0027s not useful for tests that actually try to GET the SLO / have SLO parse the manifest.","commit_id":"7a3124d82d2d18dbdf3251265784a87f62abcb51"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"9595d64a343f499d74419f45c59e1448b7d6f77f","unresolved":true,"context_lines":[{"line_number":2398,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u00271-99\u0027},"},{"line_number":2399,"context_line":"            {\u0027name\u0027: \u0027/gettest/not_checked\u0027, \u0027hash\u0027: md5hex(\u0027bar\u0027),"},{"line_number":2400,"context_line":"             \u0027bytes\u0027: \u0027303\u0027, \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2401,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u0027100-200\u0027},"},{"line_number":2402,"context_line":"        ]"},{"line_number":2403,"context_line":"        self._setup_manifest("},{"line_number":2404,"context_line":"            \u0027raw-ranges\u0027, _raw_ranges_manifest, extra_headers\u003d{"}],"source_content_type":"text/x-python","patch_set":1,"id":"c0b71473_8552f8a5","line":2401,"range":{"start_line":2401,"start_character":43,"end_line":2401,"end_character":61},"updated":"2023-12-14 10:01:17.000000000","message":"ok, that makes more sense","commit_id":"1c31973d33223978b5ae096eeb46f809465cea5e"}]}