)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"e84c399615d1fd5b96573a1387c8e7baf61a42e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"32a5f6d3_b4fbe431","updated":"2021-11-02 16:59:48.000000000","message":"This is testing neutron\u0027s integration with Designate. This test should be under the neutron-tempest-plugin designate scenarios if it is not already covered.\n\nThere are test setup requirements (devstack, etc.) that are required to enable this functionality in neutron that are not enabled in the designate gates (and should not be).","commit_id":"d14a55f6a4e4bda8a06d3c642804094e25ceb3c8"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"9df1d6dcb1868be51bcf04bd9771f5f9da1eae23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"795e96ab_e2e7d244","in_reply_to":"32a5f6d3_b4fbe431","updated":"2022-08-26 11:46:58.000000000","message":"It could be that some of our customers might want to run such a test on their environments and we\u0027ll also be happy to have this test running in D/S CI 😊\n\nI\u0027ve change the functionality and added more skip checks.\nNote: this test is DISABLED by default and will be executed only when enabled in tempest.conf (see help message added for test_create_server_check_recordset_A) + it will always check that Nova and Neutron services are available on OC.","commit_id":"d14a55f6a4e4bda8a06d3c642804094e25ceb3c8"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f5bb1b9f27c106f36b12ef31a9186b08a9941159","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ee2ede52_8710cce4","updated":"2022-08-31 21:29:53.000000000","message":"This is testing neutron functionality and should be moved to the neutron-tempest-plugin.","commit_id":"f7e8d1970504a764124c01398349c461e2650d68"}],"designate_tempest_plugin/tests/scenario/v2/test_server.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"93764659b341b4956d5c2f89362369d96a509599","unresolved":true,"context_lines":[{"line_number":117,"context_line":"        https://docs.openstack.org/neutron/yoga/admin/config-dns-int-ext-serv"},{"line_number":118,"context_line":"        .html#use-case-1-floating-ips-are-published-with-associated-port-dns"},{"line_number":119,"context_line":"        -attribute"},{"line_number":120,"context_line":"        \"\"\""},{"line_number":121,"context_line":"        # Create the VM #"},{"line_number":122,"context_line":"        vm \u003d self.servers_client.create_server("},{"line_number":123,"context_line":"            name\u003ddata_utils.rand_name(\u0027VM1_Server_\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"f3658e99_7a81c162","line":120,"updated":"2022-08-27 16:47:21.000000000","message":"How is this different from https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/scenario/test_dns_integration.py#L117-L123 ? We already run that test in the neutron-tempest-plugin-designate-scenario job.","commit_id":"ebf0e2c33feccbcf9a3f51a627e192ab96c1889c"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"a417cd4719408e750e7efcc80b4025f6b478a753","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        https://docs.openstack.org/neutron/yoga/admin/config-dns-int-ext-serv"},{"line_number":118,"context_line":"        .html#use-case-1-floating-ips-are-published-with-associated-port-dns"},{"line_number":119,"context_line":"        -attribute"},{"line_number":120,"context_line":"        \"\"\""},{"line_number":121,"context_line":"        # Create the VM #"},{"line_number":122,"context_line":"        vm \u003d self.servers_client.create_server("},{"line_number":123,"context_line":"            name\u003ddata_utils.rand_name(\u0027VM1_Server_\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"988e55d8_8e0872f1","line":120,"in_reply_to":"578ddbba_2d104a40","updated":"2022-08-28 09:21:09.000000000","message":"Done","commit_id":"ebf0e2c33feccbcf9a3f51a627e192ab96c1889c"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"3200646c2bfa58ee8d43df27ed6acc5acca43022","unresolved":true,"context_lines":[{"line_number":117,"context_line":"        https://docs.openstack.org/neutron/yoga/admin/config-dns-int-ext-serv"},{"line_number":118,"context_line":"        .html#use-case-1-floating-ips-are-published-with-associated-port-dns"},{"line_number":119,"context_line":"        -attribute"},{"line_number":120,"context_line":"        \"\"\""},{"line_number":121,"context_line":"        # Create the VM #"},{"line_number":122,"context_line":"        vm \u003d self.servers_client.create_server("},{"line_number":123,"context_line":"            name\u003ddata_utils.rand_name(\u0027VM1_Server_\u0027),"}],"source_content_type":"text/x-python","patch_set":4,"id":"578ddbba_2d104a40","line":120,"in_reply_to":"f3658e99_7a81c162","updated":"2022-08-28 09:20:55.000000000","message":"Hi Dr.Jeans!\n\nNice to have you back reviewing Designate test patches 😊\n\nThe difference is that in Designate Tempest Plugin (its Neutron integration part tests) we are concentrating in using Designate APIs (unlike Neutron tests doing E2E using DNS query to the backends which is great) to make sure that the Designate functions as expected.\n\nThis specific patch is just a beginning, we are planning to add more manipulation and negative tests into this test suite, for example:\n1) Updating VM name and making sure that Designate updates the existing record as expected.\n2) Updating VM name with some a string bigger than maximum size set in Designate and making sure that it\u0027s being rejected.\n\nThanks!","commit_id":"ebf0e2c33feccbcf9a3f51a627e192ab96c1889c"}]}