)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"f195afed4fb398970177b90599d5d62a4a4f6a24","unresolved":false,"context_lines":[{"line_number":12,"context_line":"message."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I116f65ece94f839e63b085480ba4f8974119d2bb"},{"line_number":15,"context_line":"Fixes: bug 1198239"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAP3%2F%2FzD4%3D","line":15,"updated":"2013-07-11 08:54:23.000000000","message":"Maybe it\u0027s just my preference, but I prefer the Change-Id to be the last line. I just looked through the changes, and I think most of the commit messages are organised that way. The Change-Id is more like a metadata, I prefer not to mix it with the data.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages\n\nis not asking for this strictly!","commit_id":"a3a4ea656ab5ec375263dedd1ac90ba76ece203d"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"ce276e88b60ee6465ecf971c43d70914839458be","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the pool).  In this case make sure that we do give a decent error"},{"line_number":12,"context_line":"message."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Fixes: bug 1198239"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I116f65ece94f839e63b085480ba4f8974119d2bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAQn%2F%2Fb60%3D","line":14,"updated":"2013-07-31 08:33:47.000000000","message":"I think we need DocImpact here, sorry.","commit_id":"d59c99543a634ef23013a4c66b7b672699f36904"}],"nova/virt/xenapi/vm_utils.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3edf8b5dda1c028769cdde899fe971bb4a5825af","unresolved":false,"context_lines":[{"line_number":1667,"context_line":"                \"install guest instances on. Please check your \""},{"line_number":1668,"context_line":"                \"configuration (e.g. set a default SR for the pool) \""},{"line_number":1669,"context_line":"                \"and/or configure the flag \u0027sr_matching_filter\u0027.\"))"},{"line_number":1670,"context_line":"    return None"},{"line_number":1671,"context_line":""},{"line_number":1672,"context_line":""},{"line_number":1673,"context_line":"def _safe_find_iso_sr(session):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAP3%2F%2F6KI%3D","line":1670,"updated":"2013-07-09 12:10:53.000000000","message":"If this really is an error, should we not throw an exception here? Not sure what the general rule is about throwing exceptions during the nova-compute startup process though.\n\nIt would stops people not spotting the error at least.","commit_id":"06dc70b1b5f1220396a48f551cdbbf219f17d12d"}]}