)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"97d43ba2e7e8c706dd616f789d3cb77bcf5bda20","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Amrith Kumar \u003camrith@tesora.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-19 15:44:47 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"These messages are not setup for localization"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While reviewing some other code, I found these two"},{"line_number":10,"context_line":"messages that are not setup for localization."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_f78c6037","line":7,"updated":"2014-06-19 19:59:40.000000000","message":"Please re-write commit message title.","commit_id":"9b7821f0317c4196c371a76c9b77b81a32026c76"},{"author":{"_account_id":10295,"name":"mariam john","email":"mariamj@us.ibm.com","username":"mariamj"},"change_message_id":"a076a05d4a424f92c03e930f241be624dfa29394","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Amrith Kumar \u003camrith@tesora.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-19 15:44:47 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"These messages are not setup for localization"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While reviewing some other code, I found these two"},{"line_number":10,"context_line":"messages that are not setup for localization."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_723bfe37","line":7,"in_reply_to":"1ae5cdf2_f78c6037","updated":"2014-06-19 20:35:53.000000000","message":"Agree with Dennis here. It could be something like \"Some messages in common/exceptions.py are not setup for localization\". Thanks Amrith.","commit_id":"9b7821f0317c4196c371a76c9b77b81a32026c76"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"97d43ba2e7e8c706dd616f789d3cb77bcf5bda20","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"These messages are not setup for localization"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While reviewing some other code, I found these two"},{"line_number":10,"context_line":"messages that are not setup for localization."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1332310"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_d7b764d6","line":9,"updated":"2014-06-19 19:59:40.000000000","message":"Please rewrite commit message body. Commit body should be impersonalized.","commit_id":"9b7821f0317c4196c371a76c9b77b81a32026c76"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"97d43ba2e7e8c706dd616f789d3cb77bcf5bda20","unresolved":false,"context_lines":[{"line_number":9,"context_line":"While reviewing some other code, I found these two"},{"line_number":10,"context_line":"messages that are not setup for localization."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1332310"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I43895824d405ceae06ba0bc89164b786f112cd50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ae5cdf2_979cfc6b","line":12,"updated":"2014-06-19 19:59:40.000000000","message":"Closes-Bug: #1332310","commit_id":"9b7821f0317c4196c371a76c9b77b81a32026c76"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"d468a8f92ac0dc2573a88094d1bee1b1d9da3aba","unresolved":false,"context_lines":[{"line_number":9,"context_line":"These two messages in exceptions.py are"},{"line_number":10,"context_line":"not setup for localization."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes bug: #1332310"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I43895824d405ceae06ba0bc89164b786f112cd50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1ae5cdf2_e7f9f6c8","line":12,"updated":"2014-06-20 14:52:08.000000000","message":"This needs the dash, i.e. :    Closes-Bug","commit_id":"979c9d533055ff1c84514b128371187ba47d106c"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"9e3c017d4f02e48fd30fd485efa6c2770d7457d8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Amrith Kumar \u003camrith@tesora.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-20 22:30:00 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Two messages in exceptions.py not setup for localization"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These two messages in exceptions.py are"},{"line_number":10,"context_line":"not setup for localization and so I have fixed them."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ae5cdf2_56b83ae8","line":7,"updated":"2014-06-24 14:42:05.000000000","message":"Sorry to nitpit, but the first line should be limited to 50 characters. https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure\n\nHow about:\n  Added missing localization in exceptions.py","commit_id":"11b9f4c5638a011b607dad48756754bfd3c1699a"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"9e3c017d4f02e48fd30fd485efa6c2770d7457d8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Two messages in exceptions.py not setup for localization"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These two messages in exceptions.py are"},{"line_number":10,"context_line":"not setup for localization and so I have fixed them."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-bug: #1332310"},{"line_number":13,"context_line":"Change-Id: I43895824d405ceae06ba0bc89164b786f112cd50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ae5cdf2_562f5aa7","line":10,"updated":"2014-06-24 14:42:05.000000000","message":"How about:\n  NoServiceEndpoint and EmptyCatalog exceptions had messages\n  that were not set up for localization.  These were fixed.","commit_id":"11b9f4c5638a011b607dad48756754bfd3c1699a"}],"trove/common/exception.py":[{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"d468a8f92ac0dc2573a88094d1bee1b1d9da3aba","unresolved":false,"context_lines":[{"line_number":433,"context_line":"class NoServiceEndpoint(TroveError):"},{"line_number":434,"context_line":"    \"\"\"Could not find requested endpoint in Service Catalog.\"\"\""},{"line_number":435,"context_line":"    message \u003d _(\"Endpoint not found for service_type\u003d%(service_type)s, \""},{"line_number":436,"context_line":"               \"endpoint_type\u003d%(endpoint_type)s, \""},{"line_number":437,"context_line":"               \"endpoint_region\u003d%(endpoint_region)s\")"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_672de645","line":436,"updated":"2014-06-20 14:52:08.000000000","message":"Failing pep8 -- you need to indent these two lines one more space I think.","commit_id":"979c9d533055ff1c84514b128371187ba47d106c"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"f19ff7f02cae29723d416f541486593fa52f9b52","unresolved":false,"context_lines":[{"line_number":433,"context_line":"class NoServiceEndpoint(TroveError):"},{"line_number":434,"context_line":"    \"\"\"Could not find requested endpoint in Service Catalog.\"\"\""},{"line_number":435,"context_line":"    message \u003d _(\"Endpoint not found for service_type\u003d%(service_type)s, \""},{"line_number":436,"context_line":"               \"endpoint_type\u003d%(endpoint_type)s, \""},{"line_number":437,"context_line":"               \"endpoint_region\u003d%(endpoint_region)s\")"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_d520134b","line":436,"in_reply_to":"1ae5cdf2_09aa6c70","updated":"2014-06-21 02:28:41.000000000","message":"greg, nikhil, thanks.\n\nI thought I fixed this before git review. In any event, I fixed it now, and re-ran pep8 locally.\n\namrith@amrith-1404-lts:~/source/trove.1332310/trove/common$ pep8  exception.py \namrith@amrith-1404-lts:~/source/trove.1332310/trove/common$ \n\nSorry guys! a new rev is on its way.","commit_id":"979c9d533055ff1c84514b128371187ba47d106c"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"adef4f4b739c8a069718f745b6c7f2e65ae7da35","unresolved":false,"context_lines":[{"line_number":433,"context_line":"class NoServiceEndpoint(TroveError):"},{"line_number":434,"context_line":"    \"\"\"Could not find requested endpoint in Service Catalog.\"\"\""},{"line_number":435,"context_line":"    message \u003d _(\"Endpoint not found for service_type\u003d%(service_type)s, \""},{"line_number":436,"context_line":"               \"endpoint_type\u003d%(endpoint_type)s, \""},{"line_number":437,"context_line":"               \"endpoint_region\u003d%(endpoint_region)s\")"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1ae5cdf2_09aa6c70","line":436,"in_reply_to":"1ae5cdf2_672de645","updated":"2014-06-20 21:34:17.000000000","message":"Yup, just need to fix pep8; all else looks good.","commit_id":"979c9d533055ff1c84514b128371187ba47d106c"}]}