)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"dee61b4279bd6e7c20e46261ce29c4895da21a78","unresolved":false,"context_lines":[{"line_number":4002,"context_line":"            return self.driver.get_diagnostics(instance)"},{"line_number":4003,"context_line":"        else:"},{"line_number":4004,"context_line":"            raise exception.InstanceInvalidState("},{"line_number":4005,"context_line":"                attr\u003d\u0027power_state\u0027,"},{"line_number":4006,"context_line":"                instance_uuid\u003dinstance.uuid,"},{"line_number":4007,"context_line":"                state\u003dinstance.power_state,"},{"line_number":4008,"context_line":"                method\u003d\u0027get_diagnostics\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7f43656c","side":"PARENT","line":4005,"updated":"2016-07-07 09:40:47.000000000","message":"Nit: this matches the API I guess, but I could go either way.","commit_id":"84b0c2c6bb1d372b33cc61d4c0b4f7614c59e840"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"dee61b4279bd6e7c20e46261ce29c4895da21a78","unresolved":false,"context_lines":[{"line_number":4023,"context_line":"                attr\u003d\u0027power state\u0027,"},{"line_number":4024,"context_line":"                instance_uuid\u003dinstance.uuid,"},{"line_number":4025,"context_line":"                state\u003dpower_state.STATE_MAP[instance.power_state],"},{"line_number":4026,"context_line":"                method\u003d\u0027get_diagnostics\u0027)"},{"line_number":4027,"context_line":""},{"line_number":4028,"context_line":"    @wrap_exception()"},{"line_number":4029,"context_line":"    @reverts_task_state"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_bf7aedba","line":4026,"range":{"start_line":4026,"start_character":24,"end_line":4026,"end_character":39},"updated":"2016-07-07 09:40:47.000000000","message":"Nit: I guess this should get get_instance_diagnostics as well, but anyways.","commit_id":"4e750dfcc6013e980b3d3698115c6b6b8ad3c96b"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"dd3a40cf65619737a48cfda502f38c6f27d68638","unresolved":false,"context_lines":[{"line_number":4023,"context_line":"                attr\u003d\u0027power state\u0027,"},{"line_number":4024,"context_line":"                instance_uuid\u003dinstance.uuid,"},{"line_number":4025,"context_line":"                state\u003dpower_state.STATE_MAP[instance.power_state],"},{"line_number":4026,"context_line":"                method\u003d\u0027get_diagnostics\u0027)"},{"line_number":4027,"context_line":""},{"line_number":4028,"context_line":"    @wrap_exception()"},{"line_number":4029,"context_line":"    @reverts_task_state"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_aec920ae","line":4026,"range":{"start_line":4026,"start_character":24,"end_line":4026,"end_character":39},"in_reply_to":"1aa78d24_bf7aedba","updated":"2016-07-07 09:59:12.000000000","message":"yeap, I think we may need to replace one of them (get_instance_diagnostics and get_diagnostics)","commit_id":"4e750dfcc6013e980b3d3698115c6b6b8ad3c96b"}]}