)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"63f89a59a42cd297e997f983962c2dff05e1ef17","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"51affee0_61de71f0","updated":"2023-08-24 16:11:44.000000000","message":"LGTM","commit_id":"54d76920019991854ccf86c213b70298bc02dd8e"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e3e35750276fa99394cef675fe665899a01c9e86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9c80c472_4721c0d2","updated":"2023-08-25 07:01:16.000000000","message":"eh, wrong patch, sorry. testing CI stability, then ;)","commit_id":"54d76920019991854ccf86c213b70298bc02dd8e"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"c3f27b6f7f730a8c7e96b740080c18dc92b5120c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"817de0c4_eb9af171","updated":"2023-08-25 07:00:07.000000000","message":"recheck not sure if this should work yet, let\u0027s see","commit_id":"54d76920019991854ccf86c213b70298bc02dd8e"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"7e2f43c4071170bb98343ec7ed3965c4bd39bf8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1e3b9b20_e41518ae","updated":"2023-08-25 12:34:45.000000000","message":"recheck test_create_and_delete_records_on_existing_zone_12_PTR","commit_id":"54d76920019991854ccf86c213b70298bc02dd8e"}],"designate/producer/service.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f6524956f71334d354d2fe59ddad663957ab4968","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        self._partitioner.watch_partition_change(self._rebalance)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        enabled_tasks \u003d CONF[\u0027service:producer\u0027].enabled_tasks"},{"line_number":89,"context_line":"        extensions \u003d tasks.PeriodicTask.get_extensions(enabled_tasks)"},{"line_number":90,"context_line":"        if not extensions:"},{"line_number":91,"context_line":"            raise exceptions.ConfigurationError("},{"line_number":92,"context_line":"                \u0027No periodic tasks configured to run: %s\u0027 % enabled_tasks"}],"source_content_type":"text/x-python","patch_set":2,"id":"dfedcfc7_a53709c1","line":89,"updated":"2023-08-24 11:09:54.000000000","message":"Not directly related to this patch, but I\u0027m a bit confused by the terminology, so \"task\" and \"extension\" are synonyms in this context? Would it make sense to clean this up to be consistent about the wording?","commit_id":"7b38a5782f6b799230c6dc5e434c4cddafee12bc"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"2eebaa9d416ff6ffa30f80ca33172f952ecfb6f9","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        self._partitioner.watch_partition_change(self._rebalance)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"        enabled_tasks \u003d CONF[\u0027service:producer\u0027].enabled_tasks"},{"line_number":89,"context_line":"        extensions \u003d tasks.PeriodicTask.get_extensions(enabled_tasks)"},{"line_number":90,"context_line":"        if not extensions:"},{"line_number":91,"context_line":"            raise exceptions.ConfigurationError("},{"line_number":92,"context_line":"                \u0027No periodic tasks configured to run: %s\u0027 % enabled_tasks"}],"source_content_type":"text/x-python","patch_set":2,"id":"bc9e2371_ff8c9d10","line":89,"in_reply_to":"dfedcfc7_a53709c1","updated":"2023-08-24 11:43:54.000000000","message":"Done","commit_id":"7b38a5782f6b799230c6dc5e434c4cddafee12bc"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"f6524956f71334d354d2fe59ddad663957ab4968","unresolved":true,"context_lines":[{"line_number":89,"context_line":"        extensions \u003d tasks.PeriodicTask.get_extensions(enabled_tasks)"},{"line_number":90,"context_line":"        if not extensions:"},{"line_number":91,"context_line":"            raise exceptions.ConfigurationError("},{"line_number":92,"context_line":"                \u0027No periodic tasks configured to run: %s\u0027 % enabled_tasks"},{"line_number":93,"context_line":"            )"},{"line_number":94,"context_line":"        for task in extensions:"},{"line_number":95,"context_line":"            LOG.debug(\u0027Registering task %s\u0027, task)"}],"source_content_type":"text/x-python","patch_set":2,"id":"a0bade01_5be6aaef","line":92,"range":{"start_line":92,"start_character":54,"end_line":92,"end_character":56},"updated":"2023-08-24 11:09:54.000000000","message":"This will generate an error that may look like:\n\nNo periodic tasks configured to run: typoed_task\n\nMaybe better make this \u0027...run: enabled_tasks\u003d%s\u0027?\n\nOr \u0027No periodic tasks found matching: %s\u0027?","commit_id":"7b38a5782f6b799230c6dc5e434c4cddafee12bc"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"2eebaa9d416ff6ffa30f80ca33172f952ecfb6f9","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        extensions \u003d tasks.PeriodicTask.get_extensions(enabled_tasks)"},{"line_number":90,"context_line":"        if not extensions:"},{"line_number":91,"context_line":"            raise exceptions.ConfigurationError("},{"line_number":92,"context_line":"                \u0027No periodic tasks configured to run: %s\u0027 % enabled_tasks"},{"line_number":93,"context_line":"            )"},{"line_number":94,"context_line":"        for task in extensions:"},{"line_number":95,"context_line":"            LOG.debug(\u0027Registering task %s\u0027, task)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3bde950b_7c2a440c","line":92,"range":{"start_line":92,"start_character":54,"end_line":92,"end_character":56},"in_reply_to":"a0bade01_5be6aaef","updated":"2023-08-24 11:43:54.000000000","message":"Done","commit_id":"7b38a5782f6b799230c6dc5e434c4cddafee12bc"}]}
