)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"d6389c1503acbf8eea1b19027f6c01d0873dc514","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"92f1f5c8_490176c0","updated":"2026-11-02 17:03:12.000000000","message":"I like this a lot, just one question inline.","commit_id":"20f4b275911ecc6a72a3141e620aac1612af7f22"}],"test/s3api/__init__.py":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"d6389c1503acbf8eea1b19027f6c01d0873dc514","unresolved":true,"context_lines":[{"line_number":162,"context_line":"    return wrapper"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"skip_if_checksum_persistence_disabled \u003d unittest.skipIf("},{"line_number":166,"context_line":"    not config_true_value(get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027)),"},{"line_number":167,"context_line":"    \u0027s3_checksum_persistence_tests_enabled\u003d%s\u0027"},{"line_number":168,"context_line":"    % get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"f8b187df_1b4e444e","line":165,"updated":"2026-11-02 17:03:12.000000000","message":"I\u0027m wondering if we should use a similar wrapper like 10 lines above?","commit_id":"20f4b275911ecc6a72a3141e620aac1612af7f22"},{"author":{"_account_id":34930,"name":"Jianjian Huo","email":"jhuo@nvidia.com","username":"jhuo"},"change_message_id":"18f424238d11cb5b9b97919f74b5220d73d2a027","unresolved":true,"context_lines":[{"line_number":162,"context_line":"    return wrapper"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"skip_if_checksum_persistence_disabled \u003d unittest.skipIf("},{"line_number":166,"context_line":"    not config_true_value(get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027)),"},{"line_number":167,"context_line":"    \u0027s3_checksum_persistence_tests_enabled\u003d%s\u0027"},{"line_number":168,"context_line":"    % get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"e30cbc3a_c48778ab","line":165,"in_reply_to":"071dedc8_4e5b43af","updated":"2026-11-03 05:53:53.000000000","message":"I pushed up this alternative at https://review.opendev.org/c/openstack/swift/+/1013631, along with other test improvements.","commit_id":"20f4b275911ecc6a72a3141e620aac1612af7f22"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7bc3029d06813ac9e73ec1eea08bba214edfba7e","unresolved":true,"context_lines":[{"line_number":162,"context_line":"    return wrapper"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"skip_if_checksum_persistence_disabled \u003d unittest.skipIf("},{"line_number":166,"context_line":"    not config_true_value(get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027)),"},{"line_number":167,"context_line":"    \u0027s3_checksum_persistence_tests_enabled\u003d%s\u0027"},{"line_number":168,"context_line":"    % get_opt(\u0027s3_checksum_persistence_tests_enabled\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"071dedc8_4e5b43af","line":165,"in_reply_to":"f8b187df_1b4e444e","updated":"2026-11-02 17:37:59.000000000","message":"I don\u0027t mind. It seemed that ``skipIf`` does the same, and I might get dinged for writing it out long-hand ;-). There\u0027s perhaps benefit in having the two decorators look similar.","commit_id":"20f4b275911ecc6a72a3141e620aac1612af7f22"}]}