)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"612bf2e3213176ca4fabcafc91e57f8b0f6d3320","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add host show api"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add host show api."},{"line_number":10,"context_line":"I cannot bump the api version to 1.5 as the unit test fail. seem"},{"line_number":11,"context_line":"pecan check the request version to all controller function which"},{"line_number":12,"context_line":"lead to unit test fail."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff346bd7_35bde801","line":10,"range":{"start_line":10,"start_character":33,"end_line":10,"end_character":36},"updated":"2017-07-27 16:30:57.000000000","message":"I think it is totally fine to consolidate all host API changes into version 1.4","commit_id":"1304e0aec5bc9e3e36e123dbdc7a5259e263da94"}],"zun/api/controllers/v1/hosts.py":[{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"128040906f78e9eca9c41a93aadd1d919db821e0","unresolved":false,"context_lines":[{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"def _get_host(host_id):"},{"line_number":29,"context_line":"    host \u003d None"},{"line_number":30,"context_line":"    if uuidutils.is_uuid_like(host_id):"},{"line_number":31,"context_line":"        host \u003d objects.ComputeNode.get_by_uuid(pecan.request.context, host_id)"},{"line_number":32,"context_line":"    else:"},{"line_number":33,"context_line":"        host \u003d objects.ComputeNode.get_by_hostname(pecan.request.context,"},{"line_number":34,"context_line":"                                                   host_id)"},{"line_number":35,"context_line":"    if not host:"},{"line_number":36,"context_line":"        pecan.abort(404, (\u0027Not found; the host you requested \u0027"},{"line_number":37,"context_line":"                          \u0027does not exist.\u0027))"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    return host"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def check_policy_on_host(host, action):"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff346bd7_5ce4453c","line":39,"range":{"start_line":28,"start_character":0,"end_line":39,"end_character":15},"updated":"2017-07-26 15:55:18.000000000","message":"can we use this, https://github.com/openstack/zun/blob/master/zun/api/controllers/v1/containers.py#L41\nMore cleaner. Thoughts?","commit_id":"1304e0aec5bc9e3e36e123dbdc7a5259e263da94"},{"author":{"_account_id":8264,"name":"Shunli Zhou","username":"ShunliZhou"},"change_message_id":"85e35535d7f53f7e8b39075ed56ea80ed9a12332","unresolved":false,"context_lines":[{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"def _get_host(host_id):"},{"line_number":29,"context_line":"    host \u003d None"},{"line_number":30,"context_line":"    if uuidutils.is_uuid_like(host_id):"},{"line_number":31,"context_line":"        host \u003d objects.ComputeNode.get_by_uuid(pecan.request.context, host_id)"},{"line_number":32,"context_line":"    else:"},{"line_number":33,"context_line":"        host \u003d objects.ComputeNode.get_by_hostname(pecan.request.context,"},{"line_number":34,"context_line":"                                                   host_id)"},{"line_number":35,"context_line":"    if not host:"},{"line_number":36,"context_line":"        pecan.abort(404, (\u0027Not found; the host you requested \u0027"},{"line_number":37,"context_line":"                          \u0027does not exist.\u0027))"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    return host"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"def check_policy_on_host(host, action):"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff346bd7_25da724b","line":39,"range":{"start_line":28,"start_character":0,"end_line":39,"end_character":15},"in_reply_to":"ff346bd7_5ce4453c","updated":"2017-07-27 01:00:23.000000000","message":"I tried this method, but the ComputeNode has no method get_by_name, so we can not reuse it.","commit_id":"1304e0aec5bc9e3e36e123dbdc7a5259e263da94"}]}