)]}'
{"nova/compute/api.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"065df6042121789e5c3e7e796b88687c88c20cdf","unresolved":false,"context_lines":[{"line_number":4941,"context_line":"            raise exception.ServiceNotFound(service_id\u003dservice_id)"},{"line_number":4942,"context_line":"        service.destroy()"},{"line_number":4943,"context_line":""},{"line_number":4944,"context_line":"    # TODO(mriedem): Nothing outside of tests is using this now so we should"},{"line_number":4945,"context_line":"    # be able to remove it."},{"line_number":4946,"context_line":"    def service_delete(self, context, service_id):"},{"line_number":4947,"context_line":"        \"\"\"Deletes the specified service found via id or uuid.\"\"\""},{"line_number":4948,"context_line":"        self._service_delete(context, service_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_1e354ddb","line":4945,"range":{"start_line":4944,"start_character":0,"end_line":4945,"end_character":27},"updated":"2018-07-18 06:52:51.000000000","message":"Why we can\u0027t remove it now?","commit_id":"fa12fa53971d09e4c058a339862df870ecd14778"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"e0029a5d4a662ea3a0e89d44c20f6b51c3f10381","unresolved":false,"context_lines":[{"line_number":4941,"context_line":"            raise exception.ServiceNotFound(service_id\u003dservice_id)"},{"line_number":4942,"context_line":"        service.destroy()"},{"line_number":4943,"context_line":""},{"line_number":4944,"context_line":"    # TODO(mriedem): Nothing outside of tests is using this now so we should"},{"line_number":4945,"context_line":"    # be able to remove it."},{"line_number":4946,"context_line":"    def service_delete(self, context, service_id):"},{"line_number":4947,"context_line":"        \"\"\"Deletes the specified service found via id or uuid.\"\"\""},{"line_number":4948,"context_line":"        self._service_delete(context, service_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_555c03ef","line":4945,"range":{"start_line":4944,"start_character":0,"end_line":4945,"end_character":27},"in_reply_to":"5f7c97a3_1e354ddb","updated":"2018-07-18 08:24:16.000000000","message":"Searched in the code, seems some tests and cells v1 code are still calling this?","commit_id":"fa12fa53971d09e4c058a339862df870ecd14778"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"525c305396adc4c08d6f199abfdcadd404a8d322","unresolved":false,"context_lines":[{"line_number":4941,"context_line":"            raise exception.ServiceNotFound(service_id\u003dservice_id)"},{"line_number":4942,"context_line":"        service.destroy()"},{"line_number":4943,"context_line":""},{"line_number":4944,"context_line":"    # TODO(mriedem): Nothing outside of tests is using this now so we should"},{"line_number":4945,"context_line":"    # be able to remove it."},{"line_number":4946,"context_line":"    def service_delete(self, context, service_id):"},{"line_number":4947,"context_line":"        \"\"\"Deletes the specified service found via id or uuid.\"\"\""},{"line_number":4948,"context_line":"        self._service_delete(context, service_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_579163f6","line":4945,"range":{"start_line":4944,"start_character":0,"end_line":4945,"end_character":27},"in_reply_to":"5f7c97a3_555c03ef","updated":"2018-07-18 12:55:41.000000000","message":"As Kevin mentioned, I wanted to deal with the cells v1 stuff separately because that\u0027s going to require some brain power since I want to avoid any RPC API changes in that cells v1 path, so I\u0027m not sure how far I can cut that out without making a mess. So I left it for a follow up TODO.","commit_id":"fa12fa53971d09e4c058a339862df870ecd14778"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"e007b010ad5a367b18aee31664e1b59b572d13c2","unresolved":false,"context_lines":[{"line_number":4941,"context_line":"            raise exception.ServiceNotFound(service_id\u003dservice_id)"},{"line_number":4942,"context_line":"        service.destroy()"},{"line_number":4943,"context_line":""},{"line_number":4944,"context_line":"    # TODO(mriedem): Nothing outside of tests is using this now so we should"},{"line_number":4945,"context_line":"    # be able to remove it."},{"line_number":4946,"context_line":"    def service_delete(self, context, service_id):"},{"line_number":4947,"context_line":"        \"\"\"Deletes the specified service found via id or uuid.\"\"\""},{"line_number":4948,"context_line":"        self._service_delete(context, service_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_4024360e","line":4945,"range":{"start_line":4944,"start_character":0,"end_line":4945,"end_character":27},"in_reply_to":"5f7c97a3_579163f6","updated":"2018-07-18 15:00:22.000000000","message":"I got it, I only read the comment said only tests using it. It makes sense separate cellv1 stuff, cellv2 stuff is more complex.","commit_id":"fa12fa53971d09e4c058a339862df870ecd14778"}]}