)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c5b61ead4a622fc66359250d6d111b6b5db7595","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Make use of new Console and ConsolePool objects"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change replace direct db calls in nova/console code with calls"},{"line_number":10,"context_line":"that use the newly introduced Console and ConsolePools objects."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related to blueprint mitaka-objects"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a5de9d1_71a8b88d","line":10,"updated":"2016-01-31 10:25:02.000000000","message":"You actually do a few changes here, including removing unused DB methods, I think?","commit_id":"6db278f8f76275f1cc9cf15ffbcb820580c5b0d0"}],"nova/api/openstack/compute/legacy_v2/consoles.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c5b61ead4a622fc66359250d6d111b6b5db7595","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"def _translate_keys(cons):"},{"line_number":25,"context_line":"    \"\"\"Coerces a console instance into proper dictionary format.\"\"\""},{"line_number":26,"context_line":"    pool \u003d cons.pool"},{"line_number":27,"context_line":"    info \u003d {\u0027id\u0027: cons.id,"},{"line_number":28,"context_line":"            \u0027console_type\u0027: pool.console_type}"},{"line_number":29,"context_line":"    return dict(console\u003dinfo)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_b1ecf0f3","line":26,"updated":"2016-01-31 10:25:02.000000000","message":"Nit: I quite like this bit of the conversion being a separate patch.","commit_id":"6db278f8f76275f1cc9cf15ffbcb820580c5b0d0"}],"nova/db/api.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"0c5b61ead4a622fc66359250d6d111b6b5db7595","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"    return IMPL.console_delete(context, console_id)"},{"line_number":1437,"context_line":""},{"line_number":1438,"context_line":""},{"line_number":1439,"context_line":"def console_get_by_pool_instance(context, pool_id, instance_uuid):"},{"line_number":1440,"context_line":"    \"\"\"Get console entry for a given instance and pool.\"\"\""},{"line_number":1441,"context_line":"    return IMPL.console_get_by_pool_instance(context, pool_id, instance_uuid)"},{"line_number":1442,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_b1c39056","side":"PARENT","line":1439,"updated":"2016-01-31 10:25:02.000000000","message":"Nit: its nice to have these removals in a separate patch.","commit_id":"c13977018659018b20867416f65a874f0d60f87e"}]}