)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"44c7168582317b1f665494e312b8c81cafc5384d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"08e0d325_1ce31bba","updated":"2022-08-16 07:38:03.000000000","message":"recheck","commit_id":"2d33cb3d0245a21ac3f95326baec118e9ce1ec01"}],"logscraper/logscraper.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"562fc6f37b8df66598f729d091d8ab1b23a46144","unresolved":true,"context_lines":[{"line_number":108,"context_line":"def get_arguments():"},{"line_number":109,"context_line":"    parser \u003d argparse.ArgumentParser(description\u003d\"Fetch and push last Zuul \""},{"line_number":110,"context_line":"                                     \"CI job logs into gearman.\")"},{"line_number":111,"context_line":"    parser.add_argument(\"--config\", help\u003d\"Logscraper config file\")"},{"line_number":112,"context_line":"    parser.add_argument(\"--file-list\", help\u003d\"File list to download\")"},{"line_number":113,"context_line":"    parser.add_argument(\"--zuul-api-url\", help\u003d\"URL(s) for Zuul API. Parameter\""},{"line_number":114,"context_line":"                        \" can be set multiple times.\", action\u003d\u0027append\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"67ed10b4_0c6ff30b","line":111,"updated":"2022-08-22 17:51:18.000000000","message":"Shouldn\u0027t we set this argument `required` attribute to true?","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"301ecbbc554cd300b6a5454899ed6ac181155863","unresolved":false,"context_lines":[{"line_number":108,"context_line":"def get_arguments():"},{"line_number":109,"context_line":"    parser \u003d argparse.ArgumentParser(description\u003d\"Fetch and push last Zuul \""},{"line_number":110,"context_line":"                                     \"CI job logs into gearman.\")"},{"line_number":111,"context_line":"    parser.add_argument(\"--config\", help\u003d\"Logscraper config file\")"},{"line_number":112,"context_line":"    parser.add_argument(\"--file-list\", help\u003d\"File list to download\")"},{"line_number":113,"context_line":"    parser.add_argument(\"--zuul-api-url\", help\u003d\"URL(s) for Zuul API. Parameter\""},{"line_number":114,"context_line":"                        \" can be set multiple times.\", action\u003d\u0027append\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"84d36481_634e4d81","line":111,"in_reply_to":"67ed10b4_0c6ff30b","updated":"2022-08-25 12:17:48.000000000","message":"Done","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"}],"logscraper/logsender.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"562fc6f37b8df66598f729d091d8ab1b23a46144","unresolved":true,"context_lines":[{"line_number":207,"context_line":"        key_name \u003d \"service_%s_memorycurrent\" % service.get(\u0027service\u0027)"},{"line_number":208,"context_line":"        current_mem \u003d service.get(\u0027MemoryCurrent\u0027, 0)"},{"line_number":209,"context_line":"        if not isinstance(current_mem, int):"},{"line_number":210,"context_line":"            logging.debug(\"Incorrect service %s memory consumption %s.\""},{"line_number":211,"context_line":"                          \"Setting value to 0\" % (service, current_mem))"},{"line_number":212,"context_line":"            fields[key_name] \u003d 0"},{"line_number":213,"context_line":"            continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"d05459e5_ce0bb14e","line":210,"updated":"2022-08-22 17:51:18.000000000","message":"Shouldn\u0027t this be a warning?","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"301ecbbc554cd300b6a5454899ed6ac181155863","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        key_name \u003d \"service_%s_memorycurrent\" % service.get(\u0027service\u0027)"},{"line_number":208,"context_line":"        current_mem \u003d service.get(\u0027MemoryCurrent\u0027, 0)"},{"line_number":209,"context_line":"        if not isinstance(current_mem, int):"},{"line_number":210,"context_line":"            logging.debug(\"Incorrect service %s memory consumption %s.\""},{"line_number":211,"context_line":"                          \"Setting value to 0\" % (service, current_mem))"},{"line_number":212,"context_line":"            fields[key_name] \u003d 0"},{"line_number":213,"context_line":"            continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"ae315ddc_bebad611","line":210,"in_reply_to":"d05459e5_ce0bb14e","updated":"2022-08-25 12:17:48.000000000","message":"As I see now in the logs, it might be a lot of the warning content.\nIf we just skip that value, debug is enough","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"562fc6f37b8df66598f729d091d8ab1b23a46144","unresolved":true,"context_lines":[{"line_number":209,"context_line":"        if not isinstance(current_mem, int):"},{"line_number":210,"context_line":"            logging.debug(\"Incorrect service %s memory consumption %s.\""},{"line_number":211,"context_line":"                          \"Setting value to 0\" % (service, current_mem))"},{"line_number":212,"context_line":"            fields[key_name] \u003d 0"},{"line_number":213,"context_line":"            continue"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        fields[key_name] \u003d current_mem"}],"source_content_type":"text/x-python","patch_set":3,"id":"91fbd691_148b9b0b","line":212,"updated":"2022-08-22 17:51:18.000000000","message":"why not skipping the value if it is empty?","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"301ecbbc554cd300b6a5454899ed6ac181155863","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        if not isinstance(current_mem, int):"},{"line_number":210,"context_line":"            logging.debug(\"Incorrect service %s memory consumption %s.\""},{"line_number":211,"context_line":"                          \"Setting value to 0\" % (service, current_mem))"},{"line_number":212,"context_line":"            fields[key_name] \u003d 0"},{"line_number":213,"context_line":"            continue"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"        fields[key_name] \u003d current_mem"}],"source_content_type":"text/x-python","patch_set":3,"id":"796ec09f_77c3fe1e","line":212,"in_reply_to":"91fbd691_148b9b0b","updated":"2022-08-25 12:17:48.000000000","message":"Done","commit_id":"d9dd554d511736500f9193bd01d3f6eafd8e40c3"}]}