)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"64088fd8f912545b265868301d9fbe54654a1e7c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"functional tests:"},{"line_number":12,"context_line":"    $ tox -e integrated"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It also removes logging from integrated tess."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Partial Implements: blueprint cinder-integrated-tests"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a5de9d1_cdae0498","line":14,"updated":"2016-01-28 19:29:05.000000000","message":"s/tess/tests","commit_id":"bd676b259d2e57a3f4c828d4a7e083282622df25"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"fc991c8b8f6456ac941543f081402d0ebe9fdecf","unresolved":false,"context_lines":[{"line_number":11,"context_line":"functional tests:"},{"line_number":12,"context_line":"    $ tox -e functional"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It also removes logging from functional tess."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Related Implements: blueprint cinder-integrated-tests"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a5de9d1_a31626a5","line":14,"updated":"2016-01-29 04:16:44.000000000","message":"s/tess/tests","commit_id":"fb28d3ad18f4a8208c6f78d6d2bdef021839d436"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0f9b5b1452c90753e045ddece182b74e04e7ef89","unresolved":false,"context_lines":[{"line_number":11,"context_line":"functional tests:"},{"line_number":12,"context_line":"    $ tox -e functional"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It also removes logging from functional tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Related Implements: blueprint cinder-integrated-tests"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a5de9d1_6c80a88b","line":14,"updated":"2016-02-01 21:18:25.000000000","message":"Wouldn\u0027t we want logging for functional tests, since some of them are going to be quite complex and involve calls out to Keystone, the DB, AMQP, etc.?","commit_id":"5701e3a40d4e828b7e2758e3515c21b8233ff72a"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d7eb756cc9dda7c0ff83524ac538825ea2824d92","unresolved":false,"context_lines":[{"line_number":11,"context_line":"functional tests:"},{"line_number":12,"context_line":"    $ tox -e functional"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"It also removes logging from functional tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Related Implements: blueprint cinder-integrated-tests"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5a5ae5dd_db9c4011","line":14,"in_reply_to":"7a5de9d1_6c80a88b","updated":"2016-02-05 15:01:05.000000000","message":"IMO, tests must not introduce logging. They must describes self. All logging must be placed inside project code, not tests","commit_id":"5701e3a40d4e828b7e2758e3515c21b8233ff72a"}]}