)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"141a44defc467a387f2e5c932bf803b35e4a5f58","unresolved":false,"context_lines":[{"line_number":11,"context_line":"   InvalidSharedStorage and InvalidLocalStorage"},{"line_number":12,"context_line":"in _migrate_live so that the \u0027reason\u0027 is passed up"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"fixes-bug:1250523"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I3d7e0626223f2606c5e7205e5022e26585506cfb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2FMGo%3D","line":14,"updated":"2013-11-13 06:42:46.000000000","message":"I think it should be \"Closes-Bug:\" instead\n\nSee https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"1dbf66c1763e7e6287953b17ccb420149dee31ed"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7012d44ee6704e1a1850aed4fe48ffecd472b644","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ryan Moore \u003cryan.moore@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-21 16:09:41 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"handle migration errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"handle errors:"},{"line_number":10,"context_line":"   MigrationPreCheckError, NoValidHost,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F%2BiZ4%3D","line":7,"updated":"2013-11-24 21:59:14.000000000","message":"nit: I kept overlooking this review in my email simply because I couldn\u0027t remember what this patch was, i.e. I kept thinking it was related to database migration changes, which are hard to review and controversial right now.  Once I realized what this was, it\u0027s a no-brainer to score it again. However, that raises the point that the commit message title could use some work here.  I\u0027m not sure what you could say in less than 80 characters though...maybe something like \"Handle more errors in _migrate_live admin API\"?  Just a suggestion, you might get more reviews this way.","commit_id":"7c0cfd9c9974d029da9c389efa79f7105eb6c0a6"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"6a9ff34cf2259dc86f9ff4a3aa6125ec5015465e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ryan Moore \u003cryan.moore@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-11-21 16:09:41 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"handle migration errors"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"handle errors:"},{"line_number":10,"context_line":"   MigrationPreCheckError, NoValidHost,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F%2BiVU%3D","line":7,"in_reply_to":"AAAATn%2F%2BiZ4%3D","updated":"2013-11-24 22:58:15.000000000","message":"Heh, yea db migrations scare a lot of reviewers :-) \n\nGood advice for future changesets.\n\nAlso probably a good idea to be more specific in the commit message - eg mention that it applies to the V2 API etc (the V3 is fine). For this changeset I don\u0027t see it as a blocker to get it merged though, just helps the reviewers.","commit_id":"7c0cfd9c9974d029da9c389efa79f7105eb6c0a6"}]}