)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"3ce09129ba46818b3b84b93946f9ff27d137805d","unresolved":true,"context_lines":[{"line_number":12,"context_line":"2) Create src-vm and try to PING dst-vm using its FQDN"},{"line_number":13,"context_line":"   Should PASS if DNS queries are forwarded to DNS resolver as"},{"line_number":14,"context_line":"   expected."},{"line_number":15,"context_line":"Note: this test is covering:"},{"line_number":16,"context_line":"https://docs.openstack.org/neutron/yoga/admin/config-dns-res.html#case"},{"line_number":17,"context_line":"-2b-queries-are-forwarded-to-dns-resolver-s-configured-on-the-host"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ie9f3a9aac500c33c644c6e7c4656cbb0331f3399"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7ff618a6_bc7ee95b","line":17,"range":{"start_line":15,"start_character":0,"end_line":17,"end_character":66},"updated":"2022-09-20 20:08:00.000000000","message":"Note that both case 2a and 2b are invalid for OVN - there is no DHCP agent or dnsmasq.  I think this test is testing something we should not test for ML2/OVN.","commit_id":"2dde7538dc2c8c898e9cb1465c00464a4fa08975"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"32e1742f0a9bba8afec0babd11b59b7876ba538f","unresolved":true,"context_lines":[{"line_number":12,"context_line":"2) Create src-vm and try to PING dst-vm using its FQDN"},{"line_number":13,"context_line":"   Should PASS if DNS queries are forwarded to DNS resolver as"},{"line_number":14,"context_line":"   expected."},{"line_number":15,"context_line":"Note: this test is covering:"},{"line_number":16,"context_line":"https://docs.openstack.org/neutron/yoga/admin/config-dns-res.html#case"},{"line_number":17,"context_line":"-2b-queries-are-forwarded-to-dns-resolver-s-configured-on-the-host"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ie9f3a9aac500c33c644c6e7c4656cbb0331f3399"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"e9b9e596_0fdae26a","line":17,"range":{"start_line":15,"start_character":0,"end_line":17,"end_character":66},"in_reply_to":"7ff618a6_bc7ee95b","updated":"2024-02-21 16:12:33.000000000","message":"Was this concern addressed?","commit_id":"2dde7538dc2c8c898e9cb1465c00464a4fa08975"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"560740e6a5885f1df4f058391dbd14b82fe07d82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"534fdbe0_4e1df060","updated":"2022-10-20 13:01:04.000000000","message":"recheck","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"bfb0f511a1373a157b644f3c6c51a4bd230b9261","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b3caa356_c26cae53","updated":"2024-02-21 21:20:55.000000000","message":"Brian, is it acceptable to merge a test scenario that doesn\u0027t run anywhere in CI here or in designate repo?","commit_id":"ca3e203d7145a1d91fb4e6ff90e62e7168da39b0"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"bdfb25373c9bbf082fcd6ade7600da9203eb3231","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3912d677_3cec2b33","updated":"2024-04-05 13:46:20.000000000","message":"-1 until we can see the test case executed and passing in some job in upstream gates.","commit_id":"34a971f85d2f270cefee8350ee33730a24022f56"}],"neutron_tempest_plugin/scenario/test_dns_integration.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f67d08cb62ae8f5797949f32be6291c941f978b7","unresolved":true,"context_lines":[{"line_number":86,"context_line":"                                      dns_name\u003ddns_name,"},{"line_number":87,"context_line":"                                      dns_domain\u003dself.zone[\u0027name\u0027])"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def _create_server(self, name\u003dNone, setup_sec_group\u003dNone):"},{"line_number":90,"context_line":"        if setup_sec_group:"},{"line_number":91,"context_line":"            port \u003d self.create_port("},{"line_number":92,"context_line":"                self.network, security_groups\u003d[self.secgroup[\u0027id\u0027]])"}],"source_content_type":"text/x-python","patch_set":2,"id":"d404674c_5d2df898","line":89,"updated":"2024-02-16 21:55:44.000000000","message":"since setup_sec_group is a boolean flag, please use a boolean value (False) as its default value in kwarg spec.","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"e5771709e3fee055c03c1e4713a511111680d89f","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                                      dns_name\u003ddns_name,"},{"line_number":87,"context_line":"                                      dns_domain\u003dself.zone[\u0027name\u0027])"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def _create_server(self, name\u003dNone, setup_sec_group\u003dNone):"},{"line_number":90,"context_line":"        if setup_sec_group:"},{"line_number":91,"context_line":"            port \u003d self.create_port("},{"line_number":92,"context_line":"                self.network, security_groups\u003d[self.secgroup[\u0027id\u0027]])"}],"source_content_type":"text/x-python","patch_set":2,"id":"54370269_b12cc970","line":89,"in_reply_to":"d404674c_5d2df898","updated":"2024-02-19 18:31:52.000000000","message":"Done","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f67d08cb62ae8f5797949f32be6291c941f978b7","unresolved":true,"context_lines":[{"line_number":273,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":274,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":275,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"},{"line_number":276,"context_line":"        name \u003d data_utils.rand_name(\u0027dst-vm\u0027)"},{"line_number":277,"context_line":"        dst_vm \u003d self._create_server(name\u003dname, setup_sec_group\u003dTrue)"},{"line_number":278,"context_line":"        dst_vm_ip \u003d dst_vm[\u0027fip\u0027][\u0027floating_ip_address\u0027]"},{"line_number":279,"context_line":"        self._verify_dns_records(dst_vm_ip, name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5c8c3703_57e398fd","line":276,"updated":"2024-02-16 21:55:44.000000000","message":"nit: name -\u003e dst_name (and name-\u003esrc_name in line 283), so that we avoid reusing the same variable for two different things","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"e5771709e3fee055c03c1e4713a511111680d89f","unresolved":false,"context_lines":[{"line_number":273,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":274,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":275,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"},{"line_number":276,"context_line":"        name \u003d data_utils.rand_name(\u0027dst-vm\u0027)"},{"line_number":277,"context_line":"        dst_vm \u003d self._create_server(name\u003dname, setup_sec_group\u003dTrue)"},{"line_number":278,"context_line":"        dst_vm_ip \u003d dst_vm[\u0027fip\u0027][\u0027floating_ip_address\u0027]"},{"line_number":279,"context_line":"        self._verify_dns_records(dst_vm_ip, name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c810ebf9_21c533eb","line":276,"in_reply_to":"5c8c3703_57e398fd","updated":"2024-02-19 18:31:52.000000000","message":"Done","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f67d08cb62ae8f5797949f32be6291c941f978b7","unresolved":true,"context_lines":[{"line_number":277,"context_line":"        dst_vm \u003d self._create_server(name\u003dname, setup_sec_group\u003dTrue)"},{"line_number":278,"context_line":"        dst_vm_ip \u003d dst_vm[\u0027fip\u0027][\u0027floating_ip_address\u0027]"},{"line_number":279,"context_line":"        self._verify_dns_records(dst_vm_ip, name)"},{"line_number":280,"context_line":"        dst_vm_domain_name \u003d name + \u0027.\u0027 + self.zone[\u0027name\u0027].strip(\u0027.\u0027)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        # Create src-vm"},{"line_number":283,"context_line":"        name \u003d data_utils.rand_name(\u0027src-vm\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"036ee156_d101aa61","line":280,"updated":"2024-02-16 21:55:44.000000000","message":"nit: it\u0027s better to keep relevant variables closer to where they are used, in this case, before line 293.","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"e5771709e3fee055c03c1e4713a511111680d89f","unresolved":false,"context_lines":[{"line_number":277,"context_line":"        dst_vm \u003d self._create_server(name\u003dname, setup_sec_group\u003dTrue)"},{"line_number":278,"context_line":"        dst_vm_ip \u003d dst_vm[\u0027fip\u0027][\u0027floating_ip_address\u0027]"},{"line_number":279,"context_line":"        self._verify_dns_records(dst_vm_ip, name)"},{"line_number":280,"context_line":"        dst_vm_domain_name \u003d name + \u0027.\u0027 + self.zone[\u0027name\u0027].strip(\u0027.\u0027)"},{"line_number":281,"context_line":""},{"line_number":282,"context_line":"        # Create src-vm"},{"line_number":283,"context_line":"        name \u003d data_utils.rand_name(\u0027src-vm\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"370f5b43_af6f134d","line":280,"in_reply_to":"036ee156_d101aa61","updated":"2024-02-19 18:31:52.000000000","message":"Done","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f67d08cb62ae8f5797949f32be6291c941f978b7","unresolved":true,"context_lines":[{"line_number":294,"context_line":"        try:"},{"line_number":295,"context_line":"            ssh_client.exec_command(ping_command)"},{"line_number":296,"context_line":"        except lib_exc.SSHExecCommandFailed as ex:"},{"line_number":297,"context_line":"            raise ex"}],"source_content_type":"text/x-python","patch_set":2,"id":"a0a4dec5_e307cd3d","line":297,"updated":"2024-02-16 21:55:44.000000000","message":"I think this try-except-raise wrapper around exec_command is meaningless, because it will do the exact same thing as just calling exec_command. Just leave line 295.","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"e5771709e3fee055c03c1e4713a511111680d89f","unresolved":false,"context_lines":[{"line_number":294,"context_line":"        try:"},{"line_number":295,"context_line":"            ssh_client.exec_command(ping_command)"},{"line_number":296,"context_line":"        except lib_exc.SSHExecCommandFailed as ex:"},{"line_number":297,"context_line":"            raise ex"}],"source_content_type":"text/x-python","patch_set":2,"id":"2057bd93_a4496bca","line":297,"in_reply_to":"a0a4dec5_e307cd3d","updated":"2024-02-19 18:31:52.000000000","message":"Done","commit_id":"a119816d75608f72abb12b20b9761ad7b6063ddd"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"32e1742f0a9bba8afec0babd11b59b7876ba538f","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        super(BaseDNSIntegrationTests, cls).setup_clients()"},{"line_number":55,"context_line":"        cls.zone_client \u003d cls.os_tempest.dns_v2.ZonesClient()"},{"line_number":56,"context_line":"        cls.recordset_client \u003d cls.os_tempest.dns_v2.RecordsetClient()"},{"line_number":57,"context_line":"        cls.dns_client \u003d cls.os_tempest.dns_v2.ZonesClient()"},{"line_number":58,"context_line":"        cls.query_client.build_timeout \u003d 30"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"dcbbea1d_57f9a17c","line":57,"updated":"2024-02-21 16:12:33.000000000","message":"I don\u0027t see where this is used in the patch. Should it be removed?","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"1ebc8d1ab1e15b11dfd3e41606861508c0a2a18a","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        super(BaseDNSIntegrationTests, cls).setup_clients()"},{"line_number":55,"context_line":"        cls.zone_client \u003d cls.os_tempest.dns_v2.ZonesClient()"},{"line_number":56,"context_line":"        cls.recordset_client \u003d cls.os_tempest.dns_v2.RecordsetClient()"},{"line_number":57,"context_line":"        cls.dns_client \u003d cls.os_tempest.dns_v2.ZonesClient()"},{"line_number":58,"context_line":"        cls.query_client.build_timeout \u003d 30"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"0d59290b_f59844a2","line":57,"in_reply_to":"dcbbea1d_57f9a17c","updated":"2024-02-21 18:50:19.000000000","message":"Yep, this is not needed actually.","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"32e1742f0a9bba8afec0babd11b59b7876ba538f","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"f6b20a58_88df4659","line":343,"updated":"2024-02-21 16:12:33.000000000","message":"I just noticed this (and Nate\u0027s comment that this won\u0027t work with ml2/ovn). What\u0027s the strategy to run the test case on environments that support this behavior? (With dhcp agent.)\n\n- Is there a job anywhere that validates the test case actually passes anywhere? (I couldn\u0027t find a job with designate and ml2/ovs enabled.)\n- Should this test case be skipped for ml2/ovn environments only? (Temporarily, until the bug is fixed.)","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"ebc2a1d1e63a1444d2aae848a280672973611b10","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"16368070_45bf00ba","line":343,"in_reply_to":"0b1d59e4_2598d9b0","updated":"2024-03-18 20:08:56.000000000","message":"@Brian, it\u0027s not just about ovn. This test as added is never executed in any job, neutron or designate. (because there is no job that utilizes ml2/ovs and designate.)","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e5e273353f7a0004381935753b26fa306734be22","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"3d746cce_6385528a","line":343,"in_reply_to":"16368070_45bf00ba","updated":"2024-03-28 22:05:22.000000000","message":"Ihar - thanks for the info, I hadn\u0027t noticed the info in the docs.\n\nSo I would agree that we shouldn\u0027t merge any untested tests.\n\nAnd the @testtools directive is wrong anyways - it should be skipIf since we don\u0027t want it run for OVN.","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"0649e4ebf8f6349d54ef44187ccef4e04d9b3e12","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"0b1d59e4_2598d9b0","line":343,"in_reply_to":"37fc51ed_3d21ec0e","updated":"2024-03-03 14:15:51.000000000","message":"OK, I\u0027ll add a skip for OVN in my next patchset, thanks Brian.","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d0eeb171fa394062c9329bc94b6fb5b68b938ea3","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"37fc51ed_3d21ec0e","line":343,"in_reply_to":"7045d41c_109043dc","updated":"2024-03-01 22:16:58.000000000","message":"I would agree that we shouldn\u0027t merge untested code, skipping in an OVN environment for now should be Ok.","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"bfb0f511a1373a157b644f3c6c51a4bd230b9261","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"7045d41c_109043dc","line":343,"in_reply_to":"f101829d_b13bf146","updated":"2024-02-21 21:20:55.000000000","message":"It\u0027s unclear from the state of the bug if anyone is actively working on a fix. (AFAIU the fix would be not trivial and require some way - a custom controller() action? - to forward to hypervisor resolver.) Do you have an idea of the state of the bug fix?\n\nIf the bug fix is not actively worked on, this patch may become dead code for a long time. It\u0027s unclear to me if this is a good idea to merge something that never runs in any of existing CI.\n\n(Sadly, looks like designate CI doesn\u0027t have any ml2/ovs / dhcp agent jobs either for the -scenarios...)","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"1ebc8d1ab1e15b11dfd3e41606861508c0a2a18a","unresolved":true,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"class DNSIntegrationResolversTests(BaseDNSIntegrationTests):"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    @decorators.skip_because(bug\u003d\"1902950\")"},{"line_number":344,"context_line":"    @decorators.idempotent_id(\u002738705b5c-38be-11ed-8091-201e8823901f\u0027)"},{"line_number":345,"context_line":"    def test_vm_uses_dns_resolver(self):"},{"line_number":346,"context_line":"        # Create dst-vm, ensure \"A\" type recordset"}],"source_content_type":"text/x-python","patch_set":4,"id":"f101829d_b13bf146","line":343,"in_reply_to":"f6b20a58_88df4659","updated":"2024-02-21 18:50:19.000000000","message":"The idea is to remove the \"skip\" once OVN is fixed.","commit_id":"47374286a2f137d42d8f7ba123e4f206eb9f702f"}]}