)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"67075f24ffaba657ad744b009f13dd3745fc6339","unresolved":false,"context_lines":[{"line_number":13,"context_line":"* Dumps out stdout/stderr if they are found inline (makes for easier"},{"line_number":14,"context_line":"  debug"},{"line_number":15,"context_line":"* Dumps out pythonlogging on failures if found inline"},{"line_number":16,"context_line":"* Prints skip reasons"},{"line_number":17,"context_line":"* Prints an RST style failure summary at the end"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Based on I1b529546e005f47aba56b451e1c0d8b0da09fca3, but because"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"9ad9bd40_f47ba7f2","line":16,"updated":"2014-05-06 13:45:32.000000000","message":"Don\u0027t we already print these?  Sounds like printing run time per test case is new.","commit_id":"883ad4a9a1cc26cb1a9e3e7e3a288218dd26ec97"}],"tools/subunit-trace.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"5ab05f60b082eb3862abc4073bd82737a0dfa2f6","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    stream.write(\u0027{%s}%s [%s]%s\\n\u0027 % ("},{"line_number":113,"context_line":"        worker, name, test[\u0027status\u0027], duration))"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    if test[\u0027status\u0027] in (\u0027inprogress\u0027, \u0027unknown\u0027, \u0027fail\u0027, \u0027uxsuccess\u0027):"},{"line_number":116,"context_line":"        for name, detail in test[\u0027details\u0027].items():"},{"line_number":117,"context_line":"            if detail.content_type.type \u003d\u003d \u0027test\u0027:"},{"line_number":118,"context_line":"                detail.content_type.type \u003d \u0027text\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"9ad9bd40_f5776939","line":115,"updated":"2014-05-02 21:55:53.000000000","message":"You need to handle all the status codes in this routine - http://bazaar.launchpad.net/~subunit/subunit/trunk/view/head:/README#L268 // https://github.com/testing-cabal/testtools/blob/master/testtools/testresult/real.py#L345 and https://github.com/testing-cabal/testtools/blob/master/testtools/testresult/real.py#L590\n\nSpecifically - you\u0027re missing skip here as one where you probably want to show attachments. (This is why \u0027reason\u0027 isn\u0027t being shown per your IRC question).","commit_id":"168577dbf711f2631a0479458843de1ca2a3e075"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"0618d583e2c15a985006f5c88aac3ded21b39d24","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"def cleanup_test_name(name):"},{"line_number":81,"context_line":"    # strip out tempest at the beginning, it\u0027s redundant"},{"line_number":82,"context_line":"    if name[:8] \u003d\u003d \"tempest\":"},{"line_number":83,"context_line":"        name \u003d name[8:]"},{"line_number":84,"context_line":"    # strip out the tags, we don\u0027t care"},{"line_number":85,"context_line":"    if name.find(\u0027[\u0027) \u003e 0:"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad9bd40_f86bb0ee","line":82,"updated":"2014-05-05 20:27:56.000000000","message":"8 char substring will never equal 7 char \"tempest\".\n\nI don\u0027t feel strongly about it but many times I have copy/pasted a prefix of  this part of the output into my own testr command which would no longer be possible if we strip it.","commit_id":"04cfdcc0f01ebd4759871188d973e50d3cee2bd6"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a3faf74b0b537d18be90ab5784897262aa6f3d04","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"def cleanup_test_name(name):"},{"line_number":81,"context_line":"    # strip out tempest at the beginning, it\u0027s redundant"},{"line_number":82,"context_line":"    if name[:8] \u003d\u003d \"tempest\":"},{"line_number":83,"context_line":"        name \u003d name[8:]"},{"line_number":84,"context_line":"    # strip out the tags, we don\u0027t care"},{"line_number":85,"context_line":"    if name.find(\u0027[\u0027) \u003e 0:"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad9bd40_db9c2612","line":82,"in_reply_to":"9ad9bd40_f86bb0ee","updated":"2014-05-05 20:48:41.000000000","message":"right, good point, I think I\u0027m good on removing this.","commit_id":"04cfdcc0f01ebd4759871188d973e50d3cee2bd6"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"0618d583e2c15a985006f5c88aac3ded21b39d24","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    # strip out tempest at the beginning, it\u0027s redundant"},{"line_number":82,"context_line":"    if name[:8] \u003d\u003d \"tempest\":"},{"line_number":83,"context_line":"        name \u003d name[8:]"},{"line_number":84,"context_line":"    # strip out the tags, we don\u0027t care"},{"line_number":85,"context_line":"    if name.find(\u0027[\u0027) \u003e 0:"},{"line_number":86,"context_line":"        name \u003d name[:name.find(\u0027[\u0027)]"},{"line_number":87,"context_line":"    return name"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad9bd40_787fa0ab","line":84,"updated":"2014-05-05 20:27:56.000000000","message":"Matt asked in the previous version \"why don\u0027t we care?\" Not sure if there was a consensus to remove the tags.","commit_id":"04cfdcc0f01ebd4759871188d973e50d3cee2bd6"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a3faf74b0b537d18be90ab5784897262aa6f3d04","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    # strip out tempest at the beginning, it\u0027s redundant"},{"line_number":82,"context_line":"    if name[:8] \u003d\u003d \"tempest\":"},{"line_number":83,"context_line":"        name \u003d name[8:]"},{"line_number":84,"context_line":"    # strip out the tags, we don\u0027t care"},{"line_number":85,"context_line":"    if name.find(\u0027[\u0027) \u003e 0:"},{"line_number":86,"context_line":"        name \u003d name[:name.find(\u0027[\u0027)]"},{"line_number":87,"context_line":"    return name"}],"source_content_type":"text/x-python","patch_set":7,"id":"9ad9bd40_1b712ee0","line":84,"in_reply_to":"9ad9bd40_787fa0ab","updated":"2014-05-05 20:48:41.000000000","message":"I think this ends up making the lines extremely long. The point of test results is to map tests to results. Tags don\u0027t help with that. Tags only help with test selection. Which is not relevant by the time we get here.","commit_id":"04cfdcc0f01ebd4759871188d973e50d3cee2bd6"}]}