)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"fa760dc7279d94df3c0992a6daef312d8903295c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bdfa48bb_87796142","updated":"2022-03-18 12:38:45.000000000","message":"recheck","commit_id":"2c50451b90ce17f2c1d1a7126050adfb44cf9e88"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"7a95d30500287b1eeb1be126c7fb275c1c05ce2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0a7d143d_d4a86594","updated":"2022-03-21 10:23:55.000000000","message":"recheck","commit_id":"46a6f6a35304ad40e58b70cce067a7e554c11dd4"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"d6edc3bd4cbc69f66d8ef3d492259fd4f013055a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"89fa0fca_3ff0d99b","updated":"2022-03-24 13:52:46.000000000","message":"recheck","commit_id":"269fdc718bb92759e87ed58eb5e22514388187b4"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"efd3b41c6960d127979063f8c0e3aa710c0fb88c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"dfcc68e4_e64fb71c","updated":"2022-03-24 13:48:11.000000000","message":"recheck","commit_id":"269fdc718bb92759e87ed58eb5e22514388187b4"}],"logscraper/logscraper.py":[{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"4f7d74faed96329e2c3489d7445c5fba7419fbed","unresolved":true,"context_lines":[{"line_number":181,"context_line":"        config_files \u003d load_config(self.config)"},{"line_number":182,"context_line":"        if not config_files:"},{"line_number":183,"context_line":"            logging.critical(\"Can not get information from config file\")"},{"line_number":184,"context_line":"            return"},{"line_number":185,"context_line":"        for f in config_files[\"files\"]:"},{"line_number":186,"context_line":"            if file_object in f[\"name\"] or \\"},{"line_number":187,"context_line":"                    file_object.replace(\".gz\", \"\") in f[\"name\"]:"}],"source_content_type":"text/x-python","patch_set":8,"id":"337fb1cb_368a8ba3","line":184,"updated":"2022-03-21 12:18:46.000000000","message":"Perhaps it would be better to manage the config in the Config object? That way the yaml file is loaded only once, and if it is missing config_files then the logscraper should not start.","commit_id":"46a6f6a35304ad40e58b70cce067a7e554c11dd4"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"699f0aae8b1dfbc82368b2b19d1eba23e0fd9e8f","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        config_files \u003d load_config(self.config)"},{"line_number":182,"context_line":"        if not config_files:"},{"line_number":183,"context_line":"            logging.critical(\"Can not get information from config file\")"},{"line_number":184,"context_line":"            return"},{"line_number":185,"context_line":"        for f in config_files[\"files\"]:"},{"line_number":186,"context_line":"            if file_object in f[\"name\"] or \\"},{"line_number":187,"context_line":"                    file_object.replace(\".gz\", \"\") in f[\"name\"]:"}],"source_content_type":"text/x-python","patch_set":8,"id":"e7ebc36c_197a91a0","line":184,"in_reply_to":"337fb1cb_368a8ba3","updated":"2022-03-23 10:55:45.000000000","message":"Done","commit_id":"46a6f6a35304ad40e58b70cce067a7e554c11dd4"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"d6edc3bd4cbc69f66d8ef3d492259fd4f013055a","unresolved":true,"context_lines":[{"line_number":601,"context_line":"        # add missing informations"},{"line_number":602,"context_line":"        build[\"tenant\"] \u003d config.tenant"},{"line_number":603,"context_line":"        build[\"build_args\"] \u003d args"},{"line_number":604,"context_line":"        build[\"config_file\"] \u003d config.config_file"},{"line_number":605,"context_line":"        builds.append(build)"},{"line_number":606,"context_line":""},{"line_number":607,"context_line":"    logging.info(\"Processing %d builds\", len(builds))"}],"source_content_type":"text/x-python","patch_set":12,"id":"1f0a282c_ade05f4e","line":604,"updated":"2022-03-24 13:52:46.000000000","message":"nit: if we need more, then it might be good to just attach the whole config to the build, e.g. `build[\"config\"] \u003d config`.","commit_id":"269fdc718bb92759e87ed58eb5e22514388187b4"},{"author":{"_account_id":20676,"name":"daniel.pawlik","display_name":"Daniel Pawlik","email":"dpawlik@redhat.com","username":"daniel.pawlik"},"change_message_id":"cd79896a114aa630efcf459eb19f2b44c58eb66a","unresolved":false,"context_lines":[{"line_number":601,"context_line":"        # add missing informations"},{"line_number":602,"context_line":"        build[\"tenant\"] \u003d config.tenant"},{"line_number":603,"context_line":"        build[\"build_args\"] \u003d args"},{"line_number":604,"context_line":"        build[\"config_file\"] \u003d config.config_file"},{"line_number":605,"context_line":"        builds.append(build)"},{"line_number":606,"context_line":""},{"line_number":607,"context_line":"    logging.info(\"Processing %d builds\", len(builds))"}],"source_content_type":"text/x-python","patch_set":12,"id":"8714caf4_4844b13c","line":604,"in_reply_to":"1f0a282c_ade05f4e","updated":"2022-03-24 14:05:23.000000000","message":"ack","commit_id":"269fdc718bb92759e87ed58eb5e22514388187b4"}]}