)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34460,"name":"Enrique Vallespí Gil","email":"evallesp@redhat.com","username":"evallesp"},"change_message_id":"4965c240f980711e8e8c8457c351e980a8305c5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f0addaf5_5efeed47","updated":"2022-05-31 11:50:43.000000000","message":"LGTM.","commit_id":"5e4c38f8726f0195ee8d3b2c3cbde5541e6e3290"},{"author":{"_account_id":34460,"name":"Enrique Vallespí Gil","email":"evallesp@redhat.com","username":"evallesp"},"change_message_id":"80253b582274803487177725f01be7452159f41d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5fee3ed5_c9565ac7","updated":"2022-06-03 07:56:43.000000000","message":"I like the last patch which avoids preprocessing a DEBUG line. Just one comment noted.","commit_id":"58db989faafbb248aea1c1eb72f28a0dd835d28a"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"4720573be02b63370837aafe7bfafba76a6c76dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ac0f8377_b28153f5","updated":"2022-06-07 06:22:52.000000000","message":"rebase was needed. ","commit_id":"e6491cc3d48e9b7a1763c7d1b807ff08f235dd8e"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"4f2a80b99433570f1ab3e2fb17d4ea6f67c5b638","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a1487026_0aa20736","updated":"2022-06-07 06:41:52.000000000","message":"recheck","commit_id":"e6491cc3d48e9b7a1763c7d1b807ff08f235dd8e"}],"logscraper/logsender.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"5a519b60cbca32821192f466824cfa6b85c793f0","unresolved":true,"context_lines":[{"line_number":299,"context_line":"        fields[\"@timestamp\"] \u003d ts.isoformat()"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        message \u003d get_message(line)"},{"line_number":302,"context_line":"        if not message or (skip_debug and \u0027DEBUG\u0027 in message):"},{"line_number":303,"context_line":"            continue"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"        fields[\"message\"] \u003d message"}],"source_content_type":"text/x-python","patch_set":2,"id":"b89e6342_5e26290a","line":302,"updated":"2022-06-02 11:42:29.000000000","message":"I think it would make more sense to do this test in the logline_iter, so that we don\u0027t pre-process DEBUG line.","commit_id":"5e4c38f8726f0195ee8d3b2c3cbde5541e6e3290"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"d06c0a091cb1b0d43426f0125291e788d59f0f9c","unresolved":false,"context_lines":[{"line_number":299,"context_line":"        fields[\"@timestamp\"] \u003d ts.isoformat()"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        message \u003d get_message(line)"},{"line_number":302,"context_line":"        if not message or (skip_debug and \u0027DEBUG\u0027 in message):"},{"line_number":303,"context_line":"            continue"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"        fields[\"message\"] \u003d message"}],"source_content_type":"text/x-python","patch_set":2,"id":"8da7741f_0a8d6d08","line":302,"in_reply_to":"b89e6342_5e26290a","updated":"2022-06-02 13:28:38.000000000","message":"oh, right","commit_id":"5e4c38f8726f0195ee8d3b2c3cbde5541e6e3290"},{"author":{"_account_id":34460,"name":"Enrique Vallespí Gil","email":"evallesp@redhat.com","username":"evallesp"},"change_message_id":"80253b582274803487177725f01be7452159f41d","unresolved":true,"context_lines":[{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    try:"},{"line_number":318,"context_line":"        if build_file.endswith(\u0027performance.json\u0027):"},{"line_number":319,"context_line":"            docs \u003d doc_iter(json_iter(build_file), index, es_fields, doc_type)"},{"line_number":320,"context_line":"            return helpers.bulk(es_client, docs, chunk_size\u003dchunk_size)"},{"line_number":321,"context_line":""},{"line_number":322,"context_line":"        docs \u003d doc_iter(logline_iter(build_file, skip_debug), index, es_fields,"}],"source_content_type":"text/x-python","patch_set":3,"id":"0797c25c_9c4e0ea1","line":319,"updated":"2022-06-03 07:56:43.000000000","message":"Is it not necessary to be able to skip_debug at json_iter, right? It makes more sense what Tristan said.","commit_id":"58db989faafbb248aea1c1eb72f28a0dd835d28a"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"ed40f419a85859ea1388eb5df3a07f0bd3e548d3","unresolved":false,"context_lines":[{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    try:"},{"line_number":318,"context_line":"        if build_file.endswith(\u0027performance.json\u0027):"},{"line_number":319,"context_line":"            docs \u003d doc_iter(json_iter(build_file), index, es_fields, doc_type)"},{"line_number":320,"context_line":"            return helpers.bulk(es_client, docs, chunk_size\u003dchunk_size)"},{"line_number":321,"context_line":""},{"line_number":322,"context_line":"        docs \u003d doc_iter(logline_iter(build_file, skip_debug), index, es_fields,"}],"source_content_type":"text/x-python","patch_set":3,"id":"134add4d_cac750c3","line":319,"in_reply_to":"0797c25c_9c4e0ea1","updated":"2022-06-03 08:03:14.000000000","message":"exactly. That\u0027s why I move to logline_iter.","commit_id":"58db989faafbb248aea1c1eb72f28a0dd835d28a"}]}