)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7cc9e8ff2519d22b02dc884c56b9020c1066b0cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72de9f07_41449364","updated":"2022-05-03 15:01:45.000000000","message":"it\u0027s just tests - i forgot how this worked, it\u0027s a great tool.","commit_id":"0b1cc8b0c4c7480549770596aacb439158ba8060"}],"test/unit/proxy/controllers/test_obj.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"7cc9e8ff2519d22b02dc884c56b9020c1066b0cd","unresolved":true,"context_lines":[{"line_number":1629,"context_line":""},{"line_number":1630,"context_line":"        # overloaded primary after double rebalance"},{"line_number":1631,"context_line":"        # ... opts should increase rebalance_missing_suppression_count"},{"line_number":1632,"context_line":"        policy_opts.rebalance_missing_suppression_count \u003d 2"},{"line_number":1633,"context_line":"        do_test([Timeout(), 404, 404], 503)"},{"line_number":1634,"context_line":""},{"line_number":1635,"context_line":"        # two primaries out, but no rebalance"}],"source_content_type":"text/x-python","patch_set":1,"id":"edf4f85a_9130a95d","line":1632,"updated":"2022-05-03 15:01:45.000000000","message":"if we\u0027d have pushed a config with this when we first got reports of the error we could have closed the entire incident without messing with timeouts and like the normal client backoff retry handle the elevated errors until the rebalance caught up.","commit_id":"0b1cc8b0c4c7480549770596aacb439158ba8060"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"2ca27aaf9717d33c6fb8d1633ff02634fd885b93","unresolved":true,"context_lines":[{"line_number":1629,"context_line":""},{"line_number":1630,"context_line":"        # overloaded primary after double rebalance"},{"line_number":1631,"context_line":"        # ... opts should increase rebalance_missing_suppression_count"},{"line_number":1632,"context_line":"        policy_opts.rebalance_missing_suppression_count \u003d 2"},{"line_number":1633,"context_line":"        do_test([Timeout(), 404, 404], 503)"},{"line_number":1634,"context_line":""},{"line_number":1635,"context_line":"        # two primaries out, but no rebalance"}],"source_content_type":"text/x-python","patch_set":1,"id":"1cc07836_a21c71a6","line":1632,"in_reply_to":"edf4f85a_9130a95d","updated":"2022-05-03 15:23:01.000000000","message":"Remind me, do we have this plumbed through as a per-policy option yet? If not, should we maybe do that sooner rather than later? I bet the EC policy didn\u0027t have the same 404-when-we-should-503 behavior (though it likely *did* see an increase in 503s).","commit_id":"0b1cc8b0c4c7480549770596aacb439158ba8060"}]}