)]}'
{"keystone/catalog/controllers.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f6de093494cd58e1f7691cb0db979cd0eef8ce72","unresolved":false,"context_lines":[{"line_number":150,"context_line":"        \"\"\"Specialized route target for PUT /regions/{region_id}.\"\"\""},{"line_number":151,"context_line":"        ref \u003d self._normalize_dict(region)"},{"line_number":152,"context_line":"        ref \u003d self.catalog_api.create_region(region_id, ref)"},{"line_number":153,"context_line":"        return RegionV3.wrap_member(context, ref)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    @controller.protected()"},{"line_number":156,"context_line":"    def create_region(self, context, region):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2BhsY%3D","line":153,"updated":"2014-01-24 18:23:39.000000000","message":"might have to return a different status code, depending on the api spec","commit_id":"e35f62d564cd59154ad966a64aab93a697cfe220"}],"keystone/catalog/core.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d5d08d28a7aee9216994e258259a217f378485c0","unresolved":false,"context_lines":[{"line_number":130,"context_line":"    \"\"\"Interface description for an Catalog driver.\"\"\""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    @abc.abstractmethod"},{"line_number":133,"context_line":"    def create_region(self, region_id, region_ref):"},{"line_number":134,"context_line":"        \"\"\"Creates a new region."},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        :raises: keystone.exception.Conflict"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BhLQ%3D","line":133,"updated":"2014-01-24 20:24:25.000000000","message":"this signature needs to be updated as well","commit_id":"17b5345c611a00bbf881690390b73342d31d5350"}],"keystone/tests/test_v3_catalog.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"f6de093494cd58e1f7691cb0db979cd0eef8ce72","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \"\"\"Call ``PUT /regions/{region_id}``.\"\"\""},{"line_number":30,"context_line":"        ref \u003d self.new_region_ref()"},{"line_number":31,"context_line":"        del ref[\u0027id\u0027]"},{"line_number":32,"context_line":"        region_id \u003d \u0027myregion\u0027"},{"line_number":33,"context_line":"        r \u003d self.put("},{"line_number":34,"context_line":"            \u0027/regions/%s\u0027 % region_id,"},{"line_number":35,"context_line":"            body\u003d{\u0027region\u0027: ref})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F%2Bhsk%3D","line":32,"updated":"2014-01-24 18:23:39.000000000","message":"the ref is never updated with region_id","commit_id":"e35f62d564cd59154ad966a64aab93a697cfe220"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"d5d08d28a7aee9216994e258259a217f378485c0","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        region_id \u003d \u0027myregion\u0027"},{"line_number":29,"context_line":"        r \u003d self.put("},{"line_number":30,"context_line":"            \u0027/regions/myregion\u0027,"},{"line_number":31,"context_line":"            body\u003d{\u0027region\u0027: ref}, expected_status\u003d200)"},{"line_number":32,"context_line":"        self.assertValidRegionResponse(r, ref)"},{"line_number":33,"context_line":"        # Double-check that the region ID was kept as-is and not"},{"line_number":34,"context_line":"        # populated with a UUID, like is the case with POST /regions"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F%2BhL0%3D","line":31,"updated":"2014-01-24 20:24:25.000000000","message":"this should be a 201 created, but i logged a bug and proposed a fix in a dependent patch\n\n  https://bugs.launchpad.net/keystone/+bug/1272501","commit_id":"17b5345c611a00bbf881690390b73342d31d5350"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"94fe4295ba9251e5f55522b75ccb7cdf1e2355c4","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        r \u003d self.post("},{"line_number":82,"context_line":"            \u0027/services\u0027,"},{"line_number":83,"context_line":"            body\u003d{\u0027service\u0027: ref})"},{"line_number":84,"context_line":"        self.assertValidServiceResponse(r, ref)"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    def test_list_services(self):"},{"line_number":87,"context_line":"        \"\"\"Call ``GET /services``.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUX%2F%2FoU0%3D","line":84,"updated":"2014-02-11 21:40:46.000000000","message":"kinda outside the scope of the change, but i\u0027ll let it go","commit_id":"2b0a2013bf5797fff8f61da9c1098c42399877e6"}]}
