)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"06dc2c55036ba8b606f6963305d479ac1006a9c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8c591d22_c78e18cf","updated":"2023-11-06 18:47:30.000000000","message":"LGTM","commit_id":"47bb6cf5ef614c731f2b01ee95081436b4d8c49b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"8c76d471a27646874683fc0d214b880e5ee539c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"116457d7_998bd76e","updated":"2023-11-06 19:24:04.000000000","message":"This is a bit bigger than usual unit tests patches, but besides adding test coverage it only touches core areas of the code that was fundamentality broken, e.g. exceptions that were broken due to missing %s operators.","commit_id":"47bb6cf5ef614c731f2b01ee95081436b4d8c49b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"13e7bf354334ede23bd76f9a2117fcf67b4f834a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"db494bef_5fc33287","updated":"2023-11-06 23:13:34.000000000","message":"recheck unrelated failure","commit_id":"47bb6cf5ef614c731f2b01ee95081436b4d8c49b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"7252cb801718601f0a1bc40df04a8742f0147737","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fdb350cf_c76b58ee","updated":"2023-11-06 21:01:57.000000000","message":"recheck zuul fixed","commit_id":"47bb6cf5ef614c731f2b01ee95081436b4d8c49b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"ae301b20ca372d0077dbf3809b2f89171073f5cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"79a187cc_7a689820","updated":"2023-11-08 10:22:17.000000000","message":"recheck record failed","commit_id":"5c88c503490f235397d01a8c4dcac7004c2f0903"}],"designate/objects/fields.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class PolymorphicObject(ovoo_fields.Object):"},{"line_number":35,"context_line":"    def coerce(self, obj, attr, value):"},{"line_number":36,"context_line":"        if hasattr(value, \u0027__bases__\u0027):"},{"line_number":37,"context_line":"            check_value \u003d value.__bases__[0]"},{"line_number":38,"context_line":"            super().coerce(obj, attr, check_value)"},{"line_number":39,"context_line":"        return value"}],"source_content_type":"text/x-python","patch_set":6,"id":"97d667d0_02c8eafe","side":"PARENT","line":36,"updated":"2023-11-05 22:14:10.000000000","message":"This code is unused in all my testing.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":282,"context_line":"    def coerce(self, obj, attr, value):"},{"line_number":283,"context_line":"        value \u003d super().coerce(obj, attr, value)"},{"line_number":284,"context_line":"        if (len(value) \u003e 255):"},{"line_number":285,"context_line":"            raise ValueError(\"NAPTR record flags field cannot be longer than\""},{"line_number":286,"context_line":"                             \" 255 characters\" % value)"},{"line_number":287,"context_line":"        if not constants.RE_NAPTR_FLAGS.match(\"%s\" % value):"},{"line_number":288,"context_line":"            raise ValueError(\"NAPTR record flags can be S, A, U and P\" % value)"}],"source_content_type":"text/x-python","patch_set":6,"id":"d3a43bcf_0b8a1718","side":"PARENT","line":285,"updated":"2023-11-05 22:14:10.000000000","message":"Old code was missing a %s here.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":285,"context_line":"            raise ValueError(\"NAPTR record flags field cannot be longer than\""},{"line_number":286,"context_line":"                             \" 255 characters\" % value)"},{"line_number":287,"context_line":"        if not constants.RE_NAPTR_FLAGS.match(\"%s\" % value):"},{"line_number":288,"context_line":"            raise ValueError(\"NAPTR record flags can be S, A, U and P\" % value)"},{"line_number":289,"context_line":"        return value"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"42f05bef_e464e68b","side":"PARENT","line":288,"updated":"2023-11-05 22:14:10.000000000","message":"Old code was missing a %s here.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":296,"context_line":"    def coerce(self, obj, attr, value):"},{"line_number":297,"context_line":"        value \u003d super().coerce(obj, attr, value)"},{"line_number":298,"context_line":"        if (len(value) \u003e 255):"},{"line_number":299,"context_line":"            raise ValueError(\"NAPTR record service field cannot be longer than\""},{"line_number":300,"context_line":"                             \" 255 characters\" % value)"},{"line_number":301,"context_line":"        if not constants.RE_NAPTR_SERVICE.match(\"%s\" % value):"},{"line_number":302,"context_line":"            raise ValueError(\"%s NAPTR record service is invalid\" % value)"}],"source_content_type":"text/x-python","patch_set":6,"id":"aefeeeea_62038b6f","side":"PARENT","line":299,"updated":"2023-11-05 22:14:10.000000000","message":"Old code was missing a %s here.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":310,"context_line":"    def coerce(self, obj, attr, value):"},{"line_number":311,"context_line":"        value \u003d super().coerce(obj, attr, value)"},{"line_number":312,"context_line":"        if (len(value) \u003e 255):"},{"line_number":313,"context_line":"            raise ValueError(\"NAPTR record regexp field cannot be longer than\""},{"line_number":314,"context_line":"                             \" 255 characters\" % value)"},{"line_number":315,"context_line":"        if value:"},{"line_number":316,"context_line":"            if not constants.RE_NAPTR_REGEXP.match(\"%s\" % value):"}],"source_content_type":"text/x-python","patch_set":6,"id":"c34f181c_e3cd1bb1","side":"PARENT","line":313,"updated":"2023-11-05 22:14:10.000000000","message":"Old code was missing a %s here.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"1710c6d9f61f0342c8dfb02cc14e0ec02490ff5d","unresolved":false,"context_lines":[{"line_number":407,"context_line":"class BaseObject(ovoo_fields.FieldType):"},{"line_number":408,"context_line":"    @staticmethod"},{"line_number":409,"context_line":"    def coerce(obj, attr, value):"},{"line_number":410,"context_line":"        if isinstance(value, object):"},{"line_number":411,"context_line":"            return value"},{"line_number":412,"context_line":"        else:"},{"line_number":413,"context_line":"            raise ValueError(\"BaseObject valid values are not valid\")"}],"source_content_type":"text/x-python","patch_set":6,"id":"02eddeec_1f83e085","side":"PARENT","line":410,"updated":"2023-11-05 22:14:10.000000000","message":"I don\u0027t know how this could ever be false.","commit_id":"2eeba9ea7d526e022a5341acd4a367529b10ca2e"}]}