)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"af716a7515d5a0faac4339208fd208c9e4446eb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f1b18611_2fdaad5f","updated":"2024-02-21 16:30:22.000000000","message":"Seems reasonable, once we fix pep8.","commit_id":"58aca50bc4bcd27e7184c3924ac10810aebfe9e6"}],"test/unit/proxy/controllers/test_obj.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"af716a7515d5a0faac4339208fd208c9e4446eb7","unresolved":true,"context_lines":[{"line_number":224,"context_line":"        codes \u003d [404] * num_handoff"},{"line_number":225,"context_line":"        with mocked_http_conn(*codes):"},{"line_number":226,"context_line":"            resp \u003d req.get_response(self.app)"},{"line_number":227,"context_line":"        self.assertEqual(resp.status_int, 503)"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"    def test_iter_nodes_local_first_noops_when_no_affinity(self):"},{"line_number":230,"context_line":"        # this test needs a stable node order - most don\u0027t"}],"source_content_type":"text/x-python","patch_set":1,"id":"747205bb_24abbc2a","line":227,"updated":"2024-02-21 16:30:22.000000000","message":"Yup, seems right -- and no config change can make it a 404.\n\n(I had a brief thought about `rebalance_missing_suppression_count`, but that\u0027s more about going the *other* way: treating primaries more like handoffs, rather than considering handoffs as authoritative as primaries.)","commit_id":"58aca50bc4bcd27e7184c3924ac10810aebfe9e6"}]}