)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"4f9695fcbf4ff5f09241601d1bd07df23cf998e8","unresolved":false,"context_lines":[{"line_number":10,"context_line":"lbaas-agent is passing in. This way we can figure out what to prevent."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I1a0f809140127ccb9596d83c97add03a853991a3"},{"line_number":13,"context_line":"Closes-bug: #1295491"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAWH%2F%2B938%3D","line":13,"updated":"2014-03-21 10:22:34.000000000","message":"Seems this doesn\u0027t close the bug. Related-Bug?","commit_id":"a2d21f01ead7e2cc7331b5948a5dbd54d938dbf3"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"f66dbe24492f79bf035707fbf42403d6d9db707d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"lbaas-agent is passing in. This way we can figure out what to prevent."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I1a0f809140127ccb9596d83c97add03a853991a3"},{"line_number":13,"context_line":"Closes-bug: #1295491"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAWH%2F%2B7Hg%3D","line":13,"in_reply_to":"AAAAWH%2F%2B938%3D","updated":"2014-03-21 16:50:05.000000000","message":"The bug i opened is just for the missing log status. I probably should have opened a bug for the exception instead.","commit_id":"a2d21f01ead7e2cc7331b5948a5dbd54d938dbf3"}],"neutron/services/loadbalancer/drivers/common/agent_driver_base.py":[{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"037fe83fcea85dc9b3f879b90c8ea18588ae74e6","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":99,"context_line":"                raise n_exc.Invalid(_(\u0027Pool status active requried got: %s\u0027),"},{"line_number":100,"context_line":"                                    pool.status)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"            retval \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2B%2FnU%3D","line":99,"updated":"2014-03-21 04:29:48.000000000","message":"required*","commit_id":"be6ebf1ed15feeabd83bc8c21d453851e41e2d37"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"dbf47b0142c323a12c73e55b1ffcc9c7e8d2e975","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":99,"context_line":"                raise n_exc.Invalid(_(\u0027Pool status active requried got: %s\u0027),"},{"line_number":100,"context_line":"                                    pool.status)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"            retval \u003d {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2B%2Fl0%3D","line":99,"in_reply_to":"AAAAWH%2F%2B%2FnU%3D","updated":"2014-03-21 04:34:34.000000000","message":"whoops.","commit_id":"be6ebf1ed15feeabd83bc8c21d453851e41e2d37"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"a3cd46145234a1a8e3a716074f2eae3770a6f696","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":99,"context_line":"                raise n_exc.Invalid(_(\u0027Pool status active required got: %s\u0027),"},{"line_number":100,"context_line":"                                    pool.status)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"            retval \u003d {}"},{"line_number":103,"context_line":"            retval[\u0027pool\u0027] \u003d self.plugin._make_pool_dict(pool)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2B%2FZg%3D","line":100,"updated":"2014-03-21 05:39:58.000000000","message":"Whoops, one more that I see jenkins is going to fail. Invalid doesn\u0027t take another option in the constructor. \n\nShould be:\n\nn_exc.Invalid(_(\u0027Pool status active required got: %s\u0027) %\n                    pool.status)","commit_id":"2ddde9935ffa44c790f31910b5fc7fb47c75774c"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"4f9695fcbf4ff5f09241601d1bd07df23cf998e8","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":99,"context_line":"                raise n_exc.Invalid(_(\u0027Pool status active required got: %s\u0027) %"},{"line_number":100,"context_line":"                                    pool.status)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"            retval \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2B94w%3D","line":99,"updated":"2014-03-21 10:22:34.000000000","message":"Is it worth adding pool_id to the message as well?","commit_id":"a2d21f01ead7e2cc7331b5948a5dbd54d938dbf3"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"f66dbe24492f79bf035707fbf42403d6d9db707d","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":99,"context_line":"                raise n_exc.Invalid(_(\u0027Pool status active required got: %s\u0027) %"},{"line_number":100,"context_line":"                                    pool.status)"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"            retval \u003d {}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2B7HY%3D","line":99,"in_reply_to":"AAAAWH%2F%2B94w%3D","updated":"2014-03-21 16:50:05.000000000","message":"Done","commit_id":"a2d21f01ead7e2cc7331b5948a5dbd54d938dbf3"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"16061f8e68e7b1be66109f3f38f1ff9483b711e0","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":98,"context_line":"                raise n_exc.Invalid(_(\u0027Pool_id %(pool_id)s status active \u0027"},{"line_number":99,"context_line":"                                      \u0027required got: %(status)s\u0027) %"},{"line_number":100,"context_line":"                                    {\u0027pool_id\u0027: pool_id,"},{"line_number":101,"context_line":"                                     \u0027status\u0027: pool.status})"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F%2B5yA%3D","line":99,"updated":"2014-03-21 20:57:05.000000000","message":"The message looks a bit ambiguous to me. It is better we have more formal sentence in the message, e.g., Pool_id %(pool_id)s status \"ACTIVE\" is expected but the status is %(status)s.","commit_id":"c0bcaef5f2cad48352c79f793883bb472097a15a"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"4c59745bbee9be5e7ab86987f8c20c3cdeb57109","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            pool \u003d qry.one()"},{"line_number":97,"context_line":"            if pool.status !\u003d constants.ACTIVE:"},{"line_number":98,"context_line":"                raise n_exc.Invalid(_(\u0027Pool_id %(pool_id)s status active \u0027"},{"line_number":99,"context_line":"                                      \u0027required got: %(status)s\u0027) %"},{"line_number":100,"context_line":"                                    {\u0027pool_id\u0027: pool_id,"},{"line_number":101,"context_line":"                                     \u0027status\u0027: pool.status})"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F%2B5co%3D","line":99,"in_reply_to":"AAAAWH%2F%2B5yA%3D","updated":"2014-03-21 21:19:57.000000000","message":"Done","commit_id":"c0bcaef5f2cad48352c79f793883bb472097a15a"}]}