)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ccb214f3eb4129065574310e6c5b348bdc465f6f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"virt: Make sure block device info is persisted"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are several methods that we wrap with a @update_db decorator,"},{"line_number":10,"context_line":"which is to say that we want the data that changed on the object to be"},{"line_number":11,"context_line":"persisted once the method has done it\u0027s work."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa041b7_9b55ca51","line":9,"updated":"2015-04-03 17:07:39.000000000","message":"s/a @update_db/an @update_db/","commit_id":"c083743062e2044ff9af0b7d7ccb19a935b448e2"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"af82c21f4da2ade1ebdc08659aff4c0f23c698b7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are several methods that we wrap with a @update_db decorator,"},{"line_number":10,"context_line":"which is to say that we want the data that changed on the object to be"},{"line_number":11,"context_line":"persisted once the method has done it\u0027s work."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"However in case of exceptions, we would do some local cleanup and"},{"line_number":14,"context_line":"propagate the exception to through the stack, which would cause the db"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa041b7_dccede59","line":11,"updated":"2015-04-02 12:41:48.000000000","message":"pedantic nit: -\u003e its work.","commit_id":"c083743062e2044ff9af0b7d7ccb19a935b448e2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ccb214f3eb4129065574310e6c5b348bdc465f6f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"persisted once the method has done it\u0027s work."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"However in case of exceptions, we would do some local cleanup and"},{"line_number":14,"context_line":"propagate the exception to through the stack, which would cause the db"},{"line_number":15,"context_line":"save to not happen (due to the way the \u0027update_db\u0027 decorator was"},{"line_number":16,"context_line":"written)."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa041b7_1b61ba39","line":14,"updated":"2015-04-03 17:07:39.000000000","message":"s/to through/up/","commit_id":"c083743062e2044ff9af0b7d7ccb19a935b448e2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ccb214f3eb4129065574310e6c5b348bdc465f6f","unresolved":false,"context_lines":[{"line_number":16,"context_line":"written)."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This is not ideal since some of those methods such as \u0027attach()\u0027 do"},{"line_number":19,"context_line":"several interaction (with the Cinder API and libvirt for example), and"},{"line_number":20,"context_line":"making sure that results of successfull ones are persisted in case of"},{"line_number":21,"context_line":"a non immediate failure is important for later cleanup, and keeping"},{"line_number":22,"context_line":"the system in a consistent state."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa041b7_fbca9619","line":19,"updated":"2015-04-03 17:07:39.000000000","message":"s/interaction/interactions/","commit_id":"c083743062e2044ff9af0b7d7ccb19a935b448e2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ccb214f3eb4129065574310e6c5b348bdc465f6f","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This is not ideal since some of those methods such as \u0027attach()\u0027 do"},{"line_number":19,"context_line":"several interaction (with the Cinder API and libvirt for example), and"},{"line_number":20,"context_line":"making sure that results of successfull ones are persisted in case of"},{"line_number":21,"context_line":"a non immediate failure is important for later cleanup, and keeping"},{"line_number":22,"context_line":"the system in a consistent state."},{"line_number":23,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa041b7_9bdcaae5","line":20,"updated":"2015-04-03 17:07:39.000000000","message":"s/successfull/successful/","commit_id":"c083743062e2044ff9af0b7d7ccb19a935b448e2"}]}