)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"567133d11ad036bf40d429e040b9edc159e718a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae724524_e73764ca","updated":"2023-12-14 06:14:07.000000000","message":"I was hoping this would go a different direction:\n\nhttps://review.opendev.org/c/openstack/swift/+/903636","commit_id":"1c5433967c73fed5524c1db071e18310dc2b9dc0"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"454d2f386259dc45c8bd46c0c3918c008984b603","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3c7f0887_f69f1a5f","updated":"2023-12-14 05:33:00.000000000","message":"I\u0027m almost positive the \"fix\" here is to remove the helper for setting up that weird ?format\u003draw manifest and move it into just the one class - then we can remove the slo-part num tests that tried to use it like it made any sense.","commit_id":"1c5433967c73fed5524c1db071e18310dc2b9dc0"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"5b7b33b1783bd35eb31e09d7be0cc31e43053635","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"428012ea_26cc9378","updated":"2023-12-14 10:04:27.000000000","message":"Let\u0027s go in the other direction 😊","commit_id":"1c5433967c73fed5524c1db071e18310dc2b9dc0"}],"test/unit/common/middleware/test_slo.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"ef2ed9b0bf8a279792ac8040faebff0aa597e050","unresolved":true,"context_lines":[{"line_number":2188,"context_line":"        _raw_ranges_manifest \u003d ["},{"line_number":2189,"context_line":"            {\u0027name\u0027: \u0027/gettest/b_10\u0027, \u0027hash\u0027: md5hex(\u0027b\u0027 * 10), \u0027bytes\u0027: \u002710\u0027,"},{"line_number":2190,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2191,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u00271-3\u0027},"},{"line_number":2192,"context_line":"            {\u0027name\u0027: \u0027/gettest/c_15\u0027, \u0027hash\u0027: md5hex(\u0027c\u0027 * 15), \u0027bytes\u0027: \u002715\u0027,"},{"line_number":2193,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2194,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u002710-15\u0027},"}],"source_content_type":"text/x-python","patch_set":2,"id":"215a2835_e40538bd","line":2191,"updated":"2023-12-12 20:15:17.000000000","message":"I\u0027m concerned that the original author may have intended these ranges to be invalid w.r.t. the segment size??? to verify they aren\u0027t checked during a manifest get???","commit_id":"1c5433967c73fed5524c1db071e18310dc2b9dc0"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"454d2f386259dc45c8bd46c0c3918c008984b603","unresolved":true,"context_lines":[{"line_number":2188,"context_line":"        _raw_ranges_manifest \u003d ["},{"line_number":2189,"context_line":"            {\u0027name\u0027: \u0027/gettest/b_10\u0027, \u0027hash\u0027: md5hex(\u0027b\u0027 * 10), \u0027bytes\u0027: \u002710\u0027,"},{"line_number":2190,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2191,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u00271-3\u0027},"},{"line_number":2192,"context_line":"            {\u0027name\u0027: \u0027/gettest/c_15\u0027, \u0027hash\u0027: md5hex(\u0027c\u0027 * 15), \u0027bytes\u0027: \u002715\u0027,"},{"line_number":2193,"context_line":"             \u0027last_modified\u0027: \u00271970-01-01T00:00:00.000000\u0027,"},{"line_number":2194,"context_line":"             \u0027content_type\u0027: \u0027text/plain\u0027, \u0027range\u0027: \u002710-15\u0027},"}],"source_content_type":"text/x-python","patch_set":2,"id":"244f458b_7454134e","line":2191,"in_reply_to":"215a2835_e40538bd","updated":"2023-12-14 05:33:00.000000000","message":"possibly, certainly in the ?format\u003draw TestCase there\u0027s no attempts to fetch/validate the segments - the use of the alphabet object names might have been an unfortunate coupling.  I don\u0027t think this strange manifests should be a common helper.","commit_id":"1c5433967c73fed5524c1db071e18310dc2b9dc0"}]}