)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"e9e01314f5f4929d216ede7ee8ce03a3fec8d439","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix redis cluster unit test for assertRaisesRegexp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The redis cluster unit test in was updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1491918"},{"line_number":12,"context_line":"Change-Id: I89f0da62df7933786424ce2e96815765a66625aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a2fa921_64390441","line":9,"updated":"2015-10-07 17:41:45.000000000","message":"wrap this ...","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"2a908efb6ed763a940da10f555d926df5a957856","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix redis cluster unit test for assertRaisesRegexp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The redis cluster unit test in was updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1491918"},{"line_number":12,"context_line":"Change-Id: I89f0da62df7933786424ce2e96815765a66625aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a2fa921_c407b81a","line":9,"in_reply_to":"7a2fa921_64390441","updated":"2015-10-07 17:52:21.000000000","message":"Commit message lines should be wrapped to 72 chars\nMore information here https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"97cf87e98202bf01ad0a0bc5c36994c040ebd9ff","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix redis cluster unit test for assertRaisesRegexp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The redis cluster unit test was updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1491918"},{"line_number":12,"context_line":"Change-Id: I89f0da62df7933786424ce2e96815765a66625aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a2fa921_461085a8","line":9,"updated":"2015-10-07 21:43:37.000000000","message":"I think you missed this one, other than that it looks good. I\u0027ll wait for jenkins to finish and then approve this.","commit_id":"179aee5799214b9ec51443b0b76eb93f98581d65"},{"author":{"_account_id":18432,"name":"sandra muñoz vargas","email":"sandrely2611@gmail.com","username":"samv"},"change_message_id":"c6277fb6f8c091156777f675d72ee9ec00039626","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix redis cluster unit test for assertRaisesRegexp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The redis cluster unit test was updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1491918"},{"line_number":12,"context_line":"Change-Id: I89f0da62df7933786424ce2e96815765a66625aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a2fa921_c151dfd9","line":9,"in_reply_to":"7a2fa921_461085a8","updated":"2015-10-07 22:46:28.000000000","message":"what do you mean with \"you missed this one\" ?","commit_id":"179aee5799214b9ec51443b0b76eb93f98581d65"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"6f74ae8ac11e4697e59dd7b20610fdc9ac19fb32","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix redis cluster unit test for assertRaisesRegexp"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The redis cluster unit test was updated to use assertRaisesRegexp rather than assertRaises to ensure that the correct messages are being generated"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #1491918"},{"line_number":12,"context_line":"Change-Id: I89f0da62df7933786424ce2e96815765a66625aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a2fa921_dbe67d14","line":9,"in_reply_to":"7a2fa921_c151dfd9","updated":"2015-10-08 19:32:45.000000000","message":"I think he meant that the line is more than 72 characters long (which is the \u0027standard\u0027 here - the first line has no period at the end and is \u003c\u003d 50, other lines \u003c\u003d 72). Although, the pep8 test (interestingly) does not seem to complain, so one could argue it\u0027s good to go...","commit_id":"179aee5799214b9ec51443b0b76eb93f98581d65"}],"trove/tests/unittests/cluster/test_cluster_redis_controller.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"af10735dc91c3aa468fc4b5ac9b2727c86efa41f","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        mock_get_datastore_version.return_value \u003d (Mock(), datastore_version)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        self.assertRaisesRegexp(exception.TroveError,"},{"line_number":302,"context_line":"\t\t\t\t\"Clusters not supported for\","},{"line_number":303,"context_line":"                                self.controller.create,"},{"line_number":304,"context_line":"                                req,"},{"line_number":305,"context_line":"                                body,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_24f54c82","line":302,"updated":"2015-10-07 17:41:30.000000000","message":"remove the tabs","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"2a908efb6ed763a940da10f555d926df5a957856","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        mock_get_datastore_version.return_value \u003d (Mock(), datastore_version)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        self.assertRaisesRegexp(exception.TroveError,"},{"line_number":302,"context_line":"\t\t\t\t\"Clusters not supported for\","},{"line_number":303,"context_line":"                                self.controller.create,"},{"line_number":304,"context_line":"                                req,"},{"line_number":305,"context_line":"                                body,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_1f373de4","line":302,"in_reply_to":"7a2fa921_24f54c82","updated":"2015-10-07 17:52:21.000000000","message":"spaces should be used instead of tabs\nMore info here https://www.python.org/dev/peps/pep-0008/","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"af10735dc91c3aa468fc4b5ac9b2727c86efa41f","unresolved":false,"context_lines":[{"line_number":349,"context_line":"        cluster.datastore_version.manager \u003d \u0027redis\u0027"},{"line_number":350,"context_line":"        mock_cluster_load.return_value \u003d cluster"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"        self.assertRaisesRegexp(exception.TroveError,\"No action \u0027do_stuff2\u0027 supplied by strategy for manager \u0027redis\u0027\", self.controller.action,                                req, body, tenant_id, id)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    @patch.object(strategy, \u0027load_api_strategy\u0027)"},{"line_number":355,"context_line":"    @patch.object(models.Cluster, \u0027load\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_842c6003","line":352,"updated":"2015-10-07 17:41:30.000000000","message":"line too long","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"2a908efb6ed763a940da10f555d926df5a957856","unresolved":false,"context_lines":[{"line_number":349,"context_line":"        cluster.datastore_version.manager \u003d \u0027redis\u0027"},{"line_number":350,"context_line":"        mock_cluster_load.return_value \u003d cluster"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"        self.assertRaisesRegexp(exception.TroveError,\"No action \u0027do_stuff2\u0027 supplied by strategy for manager \u0027redis\u0027\", self.controller.action,                                req, body, tenant_id, id)"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"    @patch.object(strategy, \u0027load_api_strategy\u0027)"},{"line_number":355,"context_line":"    @patch.object(models.Cluster, \u0027load\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a2fa921_df19c57a","line":352,"in_reply_to":"7a2fa921_842c6003","updated":"2015-10-07 17:52:21.000000000","message":"lines should be wrapped to 79 chars\nMore information here https://www.python.org/dev/peps/pep-0008/","commit_id":"ecee63dcc92815a90e2514afe949dbbe9accb607"}]}