)]}'
{"swift/proxy/controllers/base.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"9d46ef8c076e32418575f1896bb81f23b58bdb8b","unresolved":false,"context_lines":[{"line_number":1671,"context_line":"        path_env[\u0027PATH_INFO\u0027] \u003d \"/v1/%s/%s\" % ("},{"line_number":1672,"context_line":"            str_to_wsgi(account), str_to_wsgi(container))"},{"line_number":1673,"context_line":"        info \u003d get_container_info(path_env, self.app)"},{"line_number":1674,"context_line":"        if not info or not is_success(info.get(\u0027status\u0027)):"},{"line_number":1675,"context_line":"            info \u003d headers_to_container_info({}, 0)"},{"line_number":1676,"context_line":"            info[\u0027partition\u0027] \u003d None"},{"line_number":1677,"context_line":"            info[\u0027nodes\u0027] \u003d None"},{"line_number":1678,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_31a9907b","line":1675,"range":{"start_line":1674,"start_character":8,"end_line":1675,"end_character":51},"updated":"2019-06-24 22:35:21.000000000","message":"Hmmm....\n\nOh, I get it now! Because we make this decision up in the server, before we\u0027ve even created the controller.","commit_id":"82169ead1cc1c60318507fc3f0fbd7deff0f15e9"}],"test/unit/proxy/test_server.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"9d46ef8c076e32418575f1896bb81f23b58bdb8b","unresolved":false,"context_lines":[{"line_number":4679,"context_line":"        req \u003d Request.blank(\u0027/v1/a/c/o\u0027, method\u003d\u0027PUT\u0027, content_length\u003d0)"},{"line_number":4680,"context_line":"        account_status \u003d [200]"},{"line_number":4681,"context_line":"        # no response from primaries but container exists on a handoff!"},{"line_number":4682,"context_line":"        container_status \u003d ([Timeout()] * 3) + [200]"},{"line_number":4683,"context_line":"        object_status \u003d [201, 201, 201]"},{"line_number":4684,"context_line":"        status \u003d account_status + container_status + object_status"},{"line_number":4685,"context_line":"        with mocked_http_conn(*status) as fake_conn:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_915ddc48","line":4682,"updated":"2019-06-24 22:35:21.000000000","message":"Swap the 200 for another 3x 404 and we\u0027ve got a test for https://bugs.launchpad.net/swift/+bug/1833612 :-)","commit_id":"82169ead1cc1c60318507fc3f0fbd7deff0f15e9"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"9d46ef8c076e32418575f1896bb81f23b58bdb8b","unresolved":false,"context_lines":[{"line_number":4712,"context_line":""},{"line_number":4713,"context_line":"        req \u003d Request.blank(\u0027/v1/a/c/o\u0027, method\u003d\u0027PUT\u0027, content_length\u003d0)"},{"line_number":4714,"context_line":"        account_status \u003d [200]"},{"line_number":4715,"context_line":"        container_status \u003d [503] * 5  # 3 replicas + 2 handoffs"},{"line_number":4716,"context_line":"        status \u003d account_status + container_status"},{"line_number":4717,"context_line":"        with mocked_http_conn(*status) as fake_conn:"},{"line_number":4718,"context_line":"            resp \u003d req.get_response(self.app)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_8c106350","line":4715,"updated":"2019-06-24 22:35:21.000000000","message":"This isn\u0027t terribly likely though, yeah? More likely would be\n\n container_status \u003d [503] * 3 + [404] * 2\n\nwhich *also* passes... but we run into trouble when we add another handoff and get responses like\n\n container_status \u003d [503] * 3 + [404] * 3\n\nOh wow... and something like this still gets us a 404??\n\n [Timeout(), Timeout(), 503, 503, 404, 404]","commit_id":"82169ead1cc1c60318507fc3f0fbd7deff0f15e9"}]}
