)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"f26f78db52f8f653b86c1c45bad168f6018fce95","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"79868c14_b66862b7","updated":"2021-10-20 09:27:01.000000000","message":"recheck","commit_id":"d6a6de5da84b34646badcef23f2c79bd7c6550f9"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"9717103bc1be17cf42a055893237430a8c9f7afc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d3dbdd48_76bdb765","updated":"2021-10-20 07:02:26.000000000","message":"recheck","commit_id":"d6a6de5da84b34646badcef23f2c79bd7c6550f9"}],"designate_tempest_plugin/tests/api/v2/test_tld.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":40,"context_line":"        super(TldAdminTest, cls).setup_clients()"},{"line_number":41,"context_line":"        cls.admin_client \u003d cls.os_admin.tld_client"},{"line_number":42,"context_line":"        cls.primary_client \u003d cls.os_primary.tld_client"},{"line_number":43,"context_line":"        cls.primary_zone_client \u003d cls.os_primary.zones_client"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @classmethod"},{"line_number":46,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"30fe16fe_bd5f183c","line":43,"updated":"2021-10-05 23:30:31.000000000","message":"This patch will need to be updated for the secure RBAC changes.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        super(TldAdminTest, cls).setup_clients()"},{"line_number":41,"context_line":"        cls.admin_client \u003d cls.os_admin.tld_client"},{"line_number":42,"context_line":"        cls.primary_client \u003d cls.os_primary.tld_client"},{"line_number":43,"context_line":"        cls.primary_zone_client \u003d cls.os_primary.zones_client"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    @classmethod"},{"line_number":46,"context_line":"    def resource_setup(cls):"}],"source_content_type":"text/x-python","patch_set":1,"id":"06e2c42b_a4aa9148","line":43,"in_reply_to":"30fe16fe_bd5f183c","updated":"2021-10-06 13:40:51.000000000","message":"Ack","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    @decorators.idempotent_id(\u0027961bd2e8-d4d0-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":71,"context_line":"    def test_create_duplicated_tlds(self):"},{"line_number":72,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":73,"context_line":"        for attempt in range(2):"},{"line_number":74,"context_line":"            if attempt \u003d\u003d 0:"},{"line_number":75,"context_line":"                LOG.info(\u0027Create a first \"org\" TLD\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"e51bb472_6ebcfb16","line":72,"updated":"2021-10-05 23:30:31.000000000","message":"It would be nice to put the test name in the description field to allow better resource tracking.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    @decorators.idempotent_id(\u0027961bd2e8-d4d0-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":71,"context_line":"    def test_create_duplicated_tlds(self):"},{"line_number":72,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":73,"context_line":"        for attempt in range(2):"},{"line_number":74,"context_line":"            if attempt \u003d\u003d 0:"},{"line_number":75,"context_line":"                LOG.info(\u0027Create a first \"org\" TLD\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"81c9bb0f_c4449fb9","line":72,"in_reply_to":"e51bb472_6ebcfb16","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":70,"context_line":"    @decorators.idempotent_id(\u0027961bd2e8-d4d0-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":71,"context_line":"    def test_create_duplicated_tlds(self):"},{"line_number":72,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":73,"context_line":"        for attempt in range(2):"},{"line_number":74,"context_line":"            if attempt \u003d\u003d 0:"},{"line_number":75,"context_line":"                LOG.info(\u0027Create a first \"org\" TLD\u0027)"},{"line_number":76,"context_line":"                tld \u003d self.admin_client.create_tld("}],"source_content_type":"text/x-python","patch_set":1,"id":"5ce375a5_1705841b","line":73,"updated":"2021-10-05 23:30:31.000000000","message":"There is no shared code here, so the loop should not be needed, it should just be the two blocks sequentially.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    @decorators.idempotent_id(\u0027961bd2e8-d4d0-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":71,"context_line":"    def test_create_duplicated_tlds(self):"},{"line_number":72,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":73,"context_line":"        for attempt in range(2):"},{"line_number":74,"context_line":"            if attempt \u003d\u003d 0:"},{"line_number":75,"context_line":"                LOG.info(\u0027Create a first \"org\" TLD\u0027)"},{"line_number":76,"context_line":"                tld \u003d self.admin_client.create_tld("}],"source_content_type":"text/x-python","patch_set":1,"id":"0b16ed09_652e3863","line":73,"in_reply_to":"5ce375a5_1705841b","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":88,"context_line":"    def test_create_multiply_tlds(self):"},{"line_number":89,"context_line":"        tlds \u003d [\u0027abc\u0027, \u0027def\u0027, \u0027gih\u0027]"},{"line_number":90,"context_line":"        for tld_name in tlds:"},{"line_number":91,"context_line":"            tld_data \u003d {\"name\": tld_name, \"description\": \"sample tld\"}"},{"line_number":92,"context_line":"            LOG.info(\u0027Create a \"{}\" TLD\u0027.format(tld_name))"},{"line_number":93,"context_line":"            tld \u003d self.admin_client.create_tld("},{"line_number":94,"context_line":"                tld_data[\u0027name\u0027], tld_data[\u0027description\u0027])[1]"}],"source_content_type":"text/x-python","patch_set":1,"id":"6a9ac6c0_dfddb9b6","line":91,"updated":"2021-10-05 23:30:31.000000000","message":"Same, would be nice to have the test name in the description field.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    def test_create_multiply_tlds(self):"},{"line_number":89,"context_line":"        tlds \u003d [\u0027abc\u0027, \u0027def\u0027, \u0027gih\u0027]"},{"line_number":90,"context_line":"        for tld_name in tlds:"},{"line_number":91,"context_line":"            tld_data \u003d {\"name\": tld_name, \"description\": \"sample tld\"}"},{"line_number":92,"context_line":"            LOG.info(\u0027Create a \"{}\" TLD\u0027.format(tld_name))"},{"line_number":93,"context_line":"            tld \u003d self.admin_client.create_tld("},{"line_number":94,"context_line":"                tld_data[\u0027name\u0027], tld_data[\u0027description\u0027])[1]"}],"source_content_type":"text/x-python","patch_set":1,"id":"091e64c4_ef3863b4","line":91,"in_reply_to":"6a9ac6c0_dfddb9b6","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":99,"context_line":"    def test_create_invalid_tld(self):"},{"line_number":100,"context_line":"        invalid_tlds \u003d [\u0027   \u0027, \u0027fff.\u0027, \u0027^\u0026$\u0027, \u0027t\u0027 * 1000, \u0027@p%\u0027]"},{"line_number":101,"context_line":"        for tld_name in invalid_tlds:"},{"line_number":102,"context_line":"            tld_data \u003d {\"name\": tld_name, \"description\": \"sample tld\"}"},{"line_number":103,"context_line":"            LOG.info(\u0027Create a \"{}\" TLD\u0027.format(tld_name))"},{"line_number":104,"context_line":"            self.assertRaises("},{"line_number":105,"context_line":"                lib_exc.BadRequest, self.admin_client.create_tld,"}],"source_content_type":"text/x-python","patch_set":1,"id":"6895956d_cc4508c7","line":102,"updated":"2021-10-05 23:30:31.000000000","message":"Same, would be nice to have the test name in the description field.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":99,"context_line":"    def test_create_invalid_tld(self):"},{"line_number":100,"context_line":"        invalid_tlds \u003d [\u0027   \u0027, \u0027fff.\u0027, \u0027^\u0026$\u0027, \u0027t\u0027 * 1000, \u0027@p%\u0027]"},{"line_number":101,"context_line":"        for tld_name in invalid_tlds:"},{"line_number":102,"context_line":"            tld_data \u003d {\"name\": tld_name, \"description\": \"sample tld\"}"},{"line_number":103,"context_line":"            LOG.info(\u0027Create a \"{}\" TLD\u0027.format(tld_name))"},{"line_number":104,"context_line":"            self.assertRaises("},{"line_number":105,"context_line":"                lib_exc.BadRequest, self.admin_client.create_tld,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7477019f_760344be","line":102,"in_reply_to":"6895956d_cc4508c7","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":107,"context_line":"        LOG.info(\u0027Create a \"{}\" TLD with huge size description.\u0027)"},{"line_number":108,"context_line":"        self.assertRaises("},{"line_number":109,"context_line":"            lib_exc.BadRequest, self.admin_client.create_tld,"},{"line_number":110,"context_line":"            tld_name\u003d\u0027org\u0027, description\u003d\u0027example\u0027 * 1000)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    @decorators.idempotent_id(\u002706deced8-d4de-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":113,"context_line":"    def test_create_zone_for_not_existing_tld(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"0110fca2_cadb3623","line":110,"updated":"2021-10-05 23:30:31.000000000","message":"Same, would be nice to have the test name in the description field.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        LOG.info(\u0027Create a \"{}\" TLD with huge size description.\u0027)"},{"line_number":108,"context_line":"        self.assertRaises("},{"line_number":109,"context_line":"            lib_exc.BadRequest, self.admin_client.create_tld,"},{"line_number":110,"context_line":"            tld_name\u003d\u0027org\u0027, description\u003d\u0027example\u0027 * 1000)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    @decorators.idempotent_id(\u002706deced8-d4de-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":113,"context_line":"    def test_create_zone_for_not_existing_tld(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3af60c09_769dd60b","line":110,"in_reply_to":"0110fca2_cadb3623","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":112,"context_line":"    @decorators.idempotent_id(\u002706deced8-d4de-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":113,"context_line":"    def test_create_zone_for_not_existing_tld(self):"},{"line_number":114,"context_line":"        LOG.info(\u0027Create an \"org\" TLD\u0027)"},{"line_number":115,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":116,"context_line":"        tld \u003d self.admin_client.create_tld("},{"line_number":117,"context_line":"            tld_data[\u0027name\u0027], tld_data[\u0027description\u0027])[1]"},{"line_number":118,"context_line":"        self.addCleanup(self.admin_client.delete_tld, tld[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"320cc732_891316c1","line":115,"updated":"2021-10-05 23:30:31.000000000","message":"Same, would be nice to have the test name in the description field.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    @decorators.idempotent_id(\u002706deced8-d4de-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":113,"context_line":"    def test_create_zone_for_not_existing_tld(self):"},{"line_number":114,"context_line":"        LOG.info(\u0027Create an \"org\" TLD\u0027)"},{"line_number":115,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":116,"context_line":"        tld \u003d self.admin_client.create_tld("},{"line_number":117,"context_line":"            tld_data[\u0027name\u0027], tld_data[\u0027description\u0027])[1]"},{"line_number":118,"context_line":"        self.addCleanup(self.admin_client.delete_tld, tld[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"d11ffc89_d6a81125","line":115,"in_reply_to":"320cc732_891316c1","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":119,"context_line":"        self.assertEqual(tld_data[\"name\"], tld[\u0027name\u0027])"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        LOG.info(\u0027Create a Primary zone with \"org\" (existing) TLD.\u0027)"},{"line_number":122,"context_line":"        zone \u003d self.primary_zone_client.create_zone(name\u003d\u0027example.org.\u0027)[1]"},{"line_number":123,"context_line":"        self.addCleanup("},{"line_number":124,"context_line":"            self.wait_zone_delete, self.primary_zone_client, zone[\u0027id\u0027])"},{"line_number":125,"context_line":"        LOG.info(\u0027Ensure we respond with CREATE+PENDING\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2e745d0d_4c0a10d3","line":122,"updated":"2021-10-05 23:30:31.000000000","message":"Why is this needed? Shouldn\u0027t just creating the TLD cause the \"zzz\" zone create to fail?","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        self.assertEqual(tld_data[\"name\"], tld[\u0027name\u0027])"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        LOG.info(\u0027Create a Primary zone with \"org\" (existing) TLD.\u0027)"},{"line_number":122,"context_line":"        zone \u003d self.primary_zone_client.create_zone(name\u003d\u0027example.org.\u0027)[1]"},{"line_number":123,"context_line":"        self.addCleanup("},{"line_number":124,"context_line":"            self.wait_zone_delete, self.primary_zone_client, zone[\u0027id\u0027])"},{"line_number":125,"context_line":"        LOG.info(\u0027Ensure we respond with CREATE+PENDING\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"946df18d_058b1de0","line":122,"in_reply_to":"2e745d0d_4c0a10d3","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"770ff32cc381f9c14548c8696ce5494f07487275","unresolved":true,"context_lines":[{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    @decorators.idempotent_id(\u0027757019c0-d4e2-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":135,"context_line":"    def test_create_tld_as_primary_user(self):"},{"line_number":136,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":137,"context_line":"        LOG.info(\u0027Try to create a TLD as primary user.\u0027)"},{"line_number":138,"context_line":"        self.assertRaises("},{"line_number":139,"context_line":"            lib_exc.Forbidden, self.primary_client.create_tld,"}],"source_content_type":"text/x-python","patch_set":1,"id":"08d55412_85915227","line":136,"updated":"2021-10-05 23:30:31.000000000","message":"Same, would be nice to have the test name in the description field.","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"94f87cde17056e346e94755bea6575c85ccb7e44","unresolved":false,"context_lines":[{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    @decorators.idempotent_id(\u0027757019c0-d4e2-11eb-b8ee-74e5f9e2a801\u0027)"},{"line_number":135,"context_line":"    def test_create_tld_as_primary_user(self):"},{"line_number":136,"context_line":"        tld_data \u003d {\"name\": \"org\", \"description\": \"sample tld\"}"},{"line_number":137,"context_line":"        LOG.info(\u0027Try to create a TLD as primary user.\u0027)"},{"line_number":138,"context_line":"        self.assertRaises("},{"line_number":139,"context_line":"            lib_exc.Forbidden, self.primary_client.create_tld,"}],"source_content_type":"text/x-python","patch_set":1,"id":"69db5ef5_0907758c","line":136,"in_reply_to":"08d55412_85915227","updated":"2021-10-06 13:40:51.000000000","message":"Done","commit_id":"a35d48f04c2bd46e9c5819834286ced9023e967d"}]}