)]}'
{"nodepool/cmd/config_validator.py":[{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"79fe18b72f689fad1e2bad96701aabf946dfa12c","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        try:"},{"line_number":82,"context_line":"            config \u003d yaml.safe_load(open(self.config_file))"},{"line_number":83,"context_line":"        except Exception:"},{"line_number":84,"context_line":"            logging.exception(\u0027YAML parsing failed\u0027)"},{"line_number":85,"context_line":"            return 1"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_16d204d2","line":84,"range":{"start_line":84,"start_character":12,"end_line":84,"end_character":29},"updated":"2019-10-14 14:30:47.000000000","message":"Should be log.exception(), otherwise we get the root logger.","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"007d2536bf181cc9144757cbe7aebcd5203c776e","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        try:"},{"line_number":82,"context_line":"            config \u003d yaml.safe_load(open(self.config_file))"},{"line_number":83,"context_line":"        except Exception:"},{"line_number":84,"context_line":"            logging.exception(\u0027YAML parsing failed\u0027)"},{"line_number":85,"context_line":"            return 1"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_b7ce802a","line":84,"range":{"start_line":84,"start_character":12,"end_line":84,"end_character":29},"in_reply_to":"3fa7e38b_16d204d2","updated":"2019-10-15 04:32:02.000000000","message":"Done","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"79fe18b72f689fad1e2bad96701aabf946dfa12c","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                    get_provider_config(provider_dict).getSchema()"},{"line_number":94,"context_line":"                provider_schema.extend(provider)(provider_dict)"},{"line_number":95,"context_line":"        except Exception:"},{"line_number":96,"context_line":"            logging.exception(\u0027Schema validation failed\u0027)"},{"line_number":97,"context_line":"            return 1"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        errors \u003d False"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_b6dc50e4","line":96,"range":{"start_line":96,"start_character":12,"end_line":96,"end_character":30},"updated":"2019-10-14 14:30:47.000000000","message":"Should be log.exception(), otherwise we get the root logger.","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"007d2536bf181cc9144757cbe7aebcd5203c776e","unresolved":false,"context_lines":[{"line_number":93,"context_line":"                    get_provider_config(provider_dict).getSchema()"},{"line_number":94,"context_line":"                provider_schema.extend(provider)(provider_dict)"},{"line_number":95,"context_line":"        except Exception:"},{"line_number":96,"context_line":"            logging.exception(\u0027Schema validation failed\u0027)"},{"line_number":97,"context_line":"            return 1"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        errors \u003d False"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_d7d1bccd","line":96,"range":{"start_line":96,"start_character":12,"end_line":96,"end_character":30},"in_reply_to":"3fa7e38b_b6dc50e4","updated":"2019-10-15 04:32:02.000000000","message":"Done","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"}],"nodepool/tests/__init__.py":[{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"d1d32515a24014c6b849f9dfe9bb6fab89a336a3","unresolved":false,"context_lines":[{"line_number":368,"context_line":"        validator \u003d ConfigValidator(path)"},{"line_number":369,"context_line":"        ret \u003d validator.validate()"},{"line_number":370,"context_line":"        if ret !\u003d 0:"},{"line_number":371,"context_line":"            raise Exception(\"Config file %s could not be validated\" % path)"},{"line_number":372,"context_line":"        return path"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def replace_config(self, configfile, filename):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_34e0a026","line":371,"updated":"2019-10-09 11:07:56.000000000","message":"nit: a ValueError might be more precise here than a generic exception.","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"007d2536bf181cc9144757cbe7aebcd5203c776e","unresolved":false,"context_lines":[{"line_number":368,"context_line":"        validator \u003d ConfigValidator(path)"},{"line_number":369,"context_line":"        ret \u003d validator.validate()"},{"line_number":370,"context_line":"        if ret !\u003d 0:"},{"line_number":371,"context_line":"            raise Exception(\"Config file %s could not be validated\" % path)"},{"line_number":372,"context_line":"        return path"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"    def replace_config(self, configfile, filename):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_17f51457","line":371,"in_reply_to":"3fa7e38b_34e0a026","updated":"2019-10-15 04:32:02.000000000","message":"Done","commit_id":"5aa50694a6b7794e5dd1bc535818120e9c227dca"}]}