)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"626c9eba1eea828db39a3bd0fece210315dc4376","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b9a16f77_0aa97550","updated":"2022-06-24 15:32:32.000000000","message":"Wouldn\u0027t be better for you to test for the zone[\u0027ttl\u0027] is actually zero, verses relying on the underlying create call code to throw an error? This relies on the create call.","commit_id":"b8ec728a439af61a8a5a246641edadac2ffba7ec"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"020fbb9cff990c32d58d846f6a4454a5e7dfd98b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ae6551dc_49281f79","updated":"2022-06-28 21:44:14.000000000","message":"LGTM","commit_id":"4054a229f1cdb247b18f55a2a7a9f71ff920adee"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"de9cba60a7d3ceb7e1ececb6ddbf1651fa0961da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d3fb0008_abc899ee","updated":"2022-06-29 14:13:22.000000000","message":"LGTM","commit_id":"4054a229f1cdb247b18f55a2a7a9f71ff920adee"}],"designate_tempest_plugin/tests/api/v2/test_zones.py":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"626c9eba1eea828db39a3bd0fece210315dc4376","unresolved":true,"context_lines":[{"line_number":413,"context_line":"        LOG.info(\u0027Create a PRIMARY zone\u0027)"},{"line_number":414,"context_line":"        zone_name \u003d dns_data_utils.rand_zone_name("},{"line_number":415,"context_line":"            name\u003d\"test_create_zone_ttl_zero\", suffix\u003dself.tld_name)"},{"line_number":416,"context_line":"        zone \u003d self.client.create_zone(name\u003dzone_name, ttl\u003d0)[1]"},{"line_number":417,"context_line":"        self.addCleanup(self.wait_zone_delete, self.client, zone[\u0027id\u0027])"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"07f27069_3c1bbea6","line":416,"updated":"2022-06-24 15:32:32.000000000","message":"Wouldn\u0027t be better for you to test for the zone[\u0027ttl\u0027] is actually zero, verses relying on the underlying create call code to throw an error? This relies on the create call.","commit_id":"b8ec728a439af61a8a5a246641edadac2ffba7ec"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"67d7a2115c039098724d331984fe6988c92ad153","unresolved":true,"context_lines":[{"line_number":413,"context_line":"        LOG.info(\u0027Create a PRIMARY zone\u0027)"},{"line_number":414,"context_line":"        zone_name \u003d dns_data_utils.rand_zone_name("},{"line_number":415,"context_line":"            name\u003d\"test_create_zone_ttl_zero\", suffix\u003dself.tld_name)"},{"line_number":416,"context_line":"        zone \u003d self.client.create_zone(name\u003dzone_name, ttl\u003d0)[1]"},{"line_number":417,"context_line":"        self.addCleanup(self.wait_zone_delete, self.client, zone[\u0027id\u0027])"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d19fee44_b18f6671","line":416,"in_reply_to":"07f27069_3c1bbea6","updated":"2022-06-24 21:11:15.000000000","message":"+1 I agree there should at least be an assert here.","commit_id":"b8ec728a439af61a8a5a246641edadac2ffba7ec"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"0d9ed23741b750ac06348ccb26b91059782d92cf","unresolved":false,"context_lines":[{"line_number":413,"context_line":"        LOG.info(\u0027Create a PRIMARY zone\u0027)"},{"line_number":414,"context_line":"        zone_name \u003d dns_data_utils.rand_zone_name("},{"line_number":415,"context_line":"            name\u003d\"test_create_zone_ttl_zero\", suffix\u003dself.tld_name)"},{"line_number":416,"context_line":"        zone \u003d self.client.create_zone(name\u003dzone_name, ttl\u003d0)[1]"},{"line_number":417,"context_line":"        self.addCleanup(self.wait_zone_delete, self.client, zone[\u0027id\u0027])"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"be6d5736_c18f1d30","line":416,"in_reply_to":"c3706129_4053b21b","updated":"2022-06-26 06:58:12.000000000","message":"Done","commit_id":"b8ec728a439af61a8a5a246641edadac2ffba7ec"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"61e239d6e71a9067cd674cef170c2e77b2b9d991","unresolved":true,"context_lines":[{"line_number":413,"context_line":"        LOG.info(\u0027Create a PRIMARY zone\u0027)"},{"line_number":414,"context_line":"        zone_name \u003d dns_data_utils.rand_zone_name("},{"line_number":415,"context_line":"            name\u003d\"test_create_zone_ttl_zero\", suffix\u003dself.tld_name)"},{"line_number":416,"context_line":"        zone \u003d self.client.create_zone(name\u003dzone_name, ttl\u003d0)[1]"},{"line_number":417,"context_line":"        self.addCleanup(self.wait_zone_delete, self.client, zone[\u0027id\u0027])"},{"line_number":418,"context_line":""},{"line_number":419,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"c3706129_4053b21b","line":416,"in_reply_to":"d19fee44_b18f6671","updated":"2022-06-26 06:55:42.000000000","message":"Yep, that\u0027s correct and will be fixed.","commit_id":"b8ec728a439af61a8a5a246641edadac2ffba7ec"}]}