)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"4f337799a9bdcf25cfce7c84b107ace5dafe7b4c","unresolved":true,"context_lines":[{"line_number":12,"context_line":"2 Source ports tapped into 1 remote IP."},{"line_number":13,"context_line":"1 Source port tapped into 2 remote IPs."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This code has been written by AI."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Icf7f590c965e19f49521b8ca7a7fbedce18e3c85"},{"line_number":18,"context_line":"Signed-off-by: Bharath M V \u003cbmv@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"f0a43345_03da4cee","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":33},"updated":"2026-05-26 15:16:04.000000000","message":"Assisted-By: .... or\nGenerated-By: ....","commit_id":"bcd15a93cfc48fa7ed85c5032d492b40c301324a"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"c191fd7c5aee47eca3cfa598d2f21792c79de4bb","unresolved":false,"context_lines":[{"line_number":12,"context_line":"2 Source ports tapped into 1 remote IP."},{"line_number":13,"context_line":"1 Source port tapped into 2 remote IPs."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This code has been written by AI."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Icf7f590c965e19f49521b8ca7a7fbedce18e3c85"},{"line_number":18,"context_line":"Signed-off-by: Bharath M V \u003cbmv@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"05911995_1b3b97be","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":33},"in_reply_to":"f0a43345_03da4cee","updated":"2026-05-27 06:48:54.000000000","message":"Added","commit_id":"bcd15a93cfc48fa7ed85c5032d492b40c301324a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"10120d1977730aa7671298e5f48a9a6d039d347f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a26046a5_a7b74e49","updated":"2026-06-02 08:07:02.000000000","message":"I think in `neutron-tempest-plugin` it is common to skip according to regex in devstack jobs or according to required extensions (if any name distinguishes ovn feature from ovs one).\nbut in `whitebox-neutron-tempest-plugin` we usually make a check method and automate it to spare CI maintenance (since got all nodes ssh access established).","commit_id":"12b7f72b4b172b2d599ed5f23f2630aa27039e2a"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"75301a284e0bf1ab07603939fcd17d521e479e9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4042d069_d3a81d14","updated":"2026-06-02 07:55:40.000000000","message":"all the devstack jobs with `-ovn` pass, while ones without fail, likely something in tests isn\u0027t supported for ovs jobs.\nif so then skip can be added in test methods, class initial resource making or checks methods of tempest, or using regex in devstack jobs definition (like prefix test method with `ovn_` to have a single regex used, etc).\ndevstack jobs are under zuul.d in repo iirc.","commit_id":"12b7f72b4b172b2d599ed5f23f2630aa27039e2a"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"ad4606029acf2002b6d6b229f85e9710e7ea4cde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"6a1b1298_d2edd3e9","updated":"2026-05-28 09:42:37.000000000","message":"recheck","commit_id":"12b7f72b4b172b2d599ed5f23f2630aa27039e2a"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"053e65ca3b9390f54b450d78477bd24476c31008","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3e367935_2672c5b2","updated":"2026-06-03 09:09:35.000000000","message":"Both tests are skipped in the OVN CI: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_877/openstack/877b142e332448aa98cd49845356c5aa/testr_results.html","commit_id":"341d2a9311d02cdc66a188382c923710e80c2f1d"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"eee1d6e061997fa4b67157cb6df26c3fa9d8fa28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"a01c639f_db334cf8","updated":"2026-06-22 09:32:13.000000000","message":"lgtm, tests pass in CI devstack ovn and skipped for non ovn jobs.","commit_id":"3c2dfef1e8367db7a6c973b277c8690b14d43939"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"fc19bc4e902b4df4640dd0e85e4c4ddb58a279f5","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"cd941210_5a0cce04","updated":"2026-06-22 09:40:06.000000000","message":"nit: I would consider as a future change, if advanced vms can be built once for all test class and reused for per test method in class, since building advanved vms is very time consuming.","commit_id":"3c2dfef1e8367db7a6c973b277c8690b14d43939"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"69fdfe2eed1ac779f3127420722aebeab7baba8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"53cf5dbb_3a05f64b","in_reply_to":"24afb42a_ebfa31ce","updated":"2026-06-22 13:26:50.000000000","message":"I second this for future improvements. These test functions are very long and majority of what they do is `setup` actual testing is just few lines.\nall the setup and cleanup should be moved to fixtures.","commit_id":"3c2dfef1e8367db7a6c973b277c8690b14d43939"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"a0c94f4abdb0c598168698a9216a876d084e96ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"24afb42a_ebfa31ce","in_reply_to":"cd941210_5a0cce04","updated":"2026-06-22 09:43:11.000000000","message":"True. We can consider such approach as a future change.","commit_id":"3c2dfef1e8367db7a6c973b277c8690b14d43939"}],"neutron_tempest_plugin/config.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"de0aa8fb7cf96922fe71c04dca849c50d6d4a9ee","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    cfg.StrOpt(\u0027driver\u0027,"},{"line_number":207,"context_line":"               default\u003dNone,"},{"line_number":208,"context_line":"               choices\u003d[\u0027openvswitch\u0027, \u0027ovn\u0027],"},{"line_number":209,"context_line":"               help\u003d\u0027To differntiate OVS and OVN based TaaS\u0027),"},{"line_number":210,"context_line":"]"},{"line_number":211,"context_line":"taas_group \u003d cfg.OptGroup(name\u003d\u0027taas\u0027,"},{"line_number":212,"context_line":"                          title\u003d\u0027TaaS Tempest Options\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"5212d7ad_f022d429","line":209,"range":{"start_line":209,"start_character":24,"end_line":209,"end_character":36},"updated":"2026-06-16 13:52:50.000000000","message":"nit: \"differentiate\"","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"c2a917b66b8cddf50da76c775d685b6229646a21","unresolved":false,"context_lines":[{"line_number":206,"context_line":"    cfg.StrOpt(\u0027driver\u0027,"},{"line_number":207,"context_line":"               default\u003dNone,"},{"line_number":208,"context_line":"               choices\u003d[\u0027openvswitch\u0027, \u0027ovn\u0027],"},{"line_number":209,"context_line":"               help\u003d\u0027To differntiate OVS and OVN based TaaS\u0027),"},{"line_number":210,"context_line":"]"},{"line_number":211,"context_line":"taas_group \u003d cfg.OptGroup(name\u003d\u0027taas\u0027,"},{"line_number":212,"context_line":"                          title\u003d\u0027TaaS Tempest Options\u0027)"}],"source_content_type":"text/x-python","patch_set":14,"id":"46100005_85093dc3","line":209,"range":{"start_line":209,"start_character":24,"end_line":209,"end_character":36},"in_reply_to":"5212d7ad_f022d429","updated":"2026-06-22 09:37:21.000000000","message":"this approach was abandoned.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0ba11f4408f5c2b0b07bc55d63c3906f667ba2b1","unresolved":true,"context_lines":[{"line_number":203,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":204,"context_line":"               help\u003d\u0027Comma separated list of VLANs to be mirrored \u0027"},{"line_number":205,"context_line":"                    \u0027for a Tap-Flow.\u0027),"},{"line_number":206,"context_line":"    cfg.StrOpt(\u0027driver\u0027,"},{"line_number":207,"context_line":"               default\u003dNone,"},{"line_number":208,"context_line":"               choices\u003d[\u0027openvswitch\u0027, \u0027ovn\u0027],"},{"line_number":209,"context_line":"               help\u003d\u0027To differntiate OVS and OVN based TaaS\u0027),"},{"line_number":210,"context_line":"]"},{"line_number":211,"context_line":"taas_group \u003d cfg.OptGroup(name\u003d\u0027taas\u0027,"},{"line_number":212,"context_line":"                          title\u003d\u0027TaaS Tempest Options\u0027)"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"280fa0b2_2974322e","line":210,"range":{"start_line":206,"start_character":0,"end_line":210,"end_character":0},"updated":"2026-06-17 08:45:25.000000000","message":"Any configuration addition requires maintenance in CI into the future, so if we can avoid it, IMO it is better.\n\nIn this case also NTP has this property already in use to check if CI job is `ovn` or not [1], for example also in scenario code here [2].\n\nWhat about using this instead? sparing the CI configuration and future maintenance.\n\n[1] https://opendev.org/openstack/neutron-tempest-plugin/src/commit/f80cd304cf/neutron_tempest_plugin/api/base.py#L75\n[2] https://opendev.org/openstack/neutron-tempest-plugin/src/commit/f80cd304cf/neutron_tempest_plugin/scenario/base.py#L240","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"b6c171caedfbe37854abd356c3011343c396181d","unresolved":true,"context_lines":[{"line_number":203,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":204,"context_line":"               help\u003d\u0027Comma separated list of VLANs to be mirrored \u0027"},{"line_number":205,"context_line":"                    \u0027for a Tap-Flow.\u0027),"},{"line_number":206,"context_line":"    cfg.StrOpt(\u0027driver\u0027,"},{"line_number":207,"context_line":"               default\u003dNone,"},{"line_number":208,"context_line":"               choices\u003d[\u0027openvswitch\u0027, \u0027ovn\u0027],"},{"line_number":209,"context_line":"               help\u003d\u0027To differntiate OVS and OVN based TaaS\u0027),"},{"line_number":210,"context_line":"]"},{"line_number":211,"context_line":"taas_group \u003d cfg.OptGroup(name\u003d\u0027taas\u0027,"},{"line_number":212,"context_line":"                          title\u003d\u0027TaaS Tempest Options\u0027)"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"8dfa261f_1c004a1f","line":210,"range":{"start_line":206,"start_character":0,"end_line":210,"end_character":0},"in_reply_to":"280fa0b2_2974322e","updated":"2026-06-17 08:57:18.000000000","message":"This is a better example of this property used, more suitable for current need:\nhttps://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/scenario/test_mtu.py#L164","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"c2a917b66b8cddf50da76c775d685b6229646a21","unresolved":false,"context_lines":[{"line_number":203,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":204,"context_line":"               help\u003d\u0027Comma separated list of VLANs to be mirrored \u0027"},{"line_number":205,"context_line":"                    \u0027for a Tap-Flow.\u0027),"},{"line_number":206,"context_line":"    cfg.StrOpt(\u0027driver\u0027,"},{"line_number":207,"context_line":"               default\u003dNone,"},{"line_number":208,"context_line":"               choices\u003d[\u0027openvswitch\u0027, \u0027ovn\u0027],"},{"line_number":209,"context_line":"               help\u003d\u0027To differntiate OVS and OVN based TaaS\u0027),"},{"line_number":210,"context_line":"]"},{"line_number":211,"context_line":"taas_group \u003d cfg.OptGroup(name\u003d\u0027taas\u0027,"},{"line_number":212,"context_line":"                          title\u003d\u0027TaaS Tempest Options\u0027)"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"73fa877d_8aa53ddf","line":210,"range":{"start_line":206,"start_character":0,"end_line":210,"end_character":0},"in_reply_to":"8dfa261f_1c004a1f","updated":"2026-06-22 09:37:21.000000000","message":"done.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"}],"neutron_tempest_plugin/tap_as_a_service/scenario/test_tap_mirror.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a6a66ce4cab4e9f171036d48bf0f8a38de0d86ae","unresolved":true,"context_lines":[{"line_number":312,"context_line":"        # Check monitor1"},{"line_number":313,"context_line":"        output1 \u003d monitor1_client.exec_command(\"cat %s\" % log_location)"},{"line_number":314,"context_line":"        self.assertLess(0, len(output1))"},{"line_number":315,"context_line":"        self.assertIn(\u0027key\u003d0x65\u0027, output1)  # IN:101"},{"line_number":316,"context_line":"        self.assertIn(\u0027key\u003d0x66\u0027, output1)  # OUT:102"},{"line_number":317,"context_line":"        if \u0027BOTH\u0027 in directions1:"},{"line_number":318,"context_line":"            self.assertIn(\u0027key\u003d0x69\u0027, output1)  # BOTH:105"},{"line_number":319,"context_line":"        # Check monitor2"}],"source_content_type":"text/x-python","patch_set":8,"id":"b42de46e_4d007964","line":316,"range":{"start_line":315,"start_character":0,"end_line":316,"end_character":53},"updated":"2026-05-26 15:19:13.000000000","message":"There is a method _check_icmp_mirror_direction for this check, perhaps you can try that one. \nStrange that with OVN the tests are passing but not with OVS","commit_id":"bcd15a93cfc48fa7ed85c5032d492b40c301324a"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"c2a917b66b8cddf50da76c775d685b6229646a21","unresolved":false,"context_lines":[{"line_number":312,"context_line":"        # Check monitor1"},{"line_number":313,"context_line":"        output1 \u003d monitor1_client.exec_command(\"cat %s\" % log_location)"},{"line_number":314,"context_line":"        self.assertLess(0, len(output1))"},{"line_number":315,"context_line":"        self.assertIn(\u0027key\u003d0x65\u0027, output1)  # IN:101"},{"line_number":316,"context_line":"        self.assertIn(\u0027key\u003d0x66\u0027, output1)  # OUT:102"},{"line_number":317,"context_line":"        if \u0027BOTH\u0027 in directions1:"},{"line_number":318,"context_line":"            self.assertIn(\u0027key\u003d0x69\u0027, output1)  # BOTH:105"},{"line_number":319,"context_line":"        # Check monitor2"}],"source_content_type":"text/x-python","patch_set":8,"id":"1e5774fa_3c4aff4a","line":316,"range":{"start_line":315,"start_character":0,"end_line":316,"end_character":53},"in_reply_to":"b42de46e_4d007964","updated":"2026-06-22 09:37:21.000000000","message":"Acknowledged","commit_id":"bcd15a93cfc48fa7ed85c5032d492b40c301324a"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"de0aa8fb7cf96922fe71c04dca849c50d6d4a9ee","unresolved":true,"context_lines":[{"line_number":260,"context_line":"            security_group\u003dself.secgroup[\u0027name\u0027],"},{"line_number":261,"context_line":"            port_security_enabled\u003dFalse,"},{"line_number":262,"context_line":"        )"},{"line_number":263,"context_line":"        user \u003d CONF.neutron_plugin_options.advanced_image_ssh_user"},{"line_number":264,"context_line":"        monitor1_client \u003d remote_client.RemoteClient("},{"line_number":265,"context_line":"            vm_mon1_fip[\u0027floating_ip_address\u0027], user,"},{"line_number":266,"context_line":"            pkey\u003dself.keypair[\u0027private_key\u0027],"}],"source_content_type":"text/x-python","patch_set":14,"id":"927b1789_4eb2114e","line":263,"updated":"2026-06-16 13:52:50.000000000","message":"it seems that you need advanced image to run this test, shouldn\u0027t you then skip test if advanced image is not configured?","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"cc307ab6454fd30100f648f3f30fcbe010acadf4","unresolved":true,"context_lines":[{"line_number":260,"context_line":"            security_group\u003dself.secgroup[\u0027name\u0027],"},{"line_number":261,"context_line":"            port_security_enabled\u003dFalse,"},{"line_number":262,"context_line":"        )"},{"line_number":263,"context_line":"        user \u003d CONF.neutron_plugin_options.advanced_image_ssh_user"},{"line_number":264,"context_line":"        monitor1_client \u003d remote_client.RemoteClient("},{"line_number":265,"context_line":"            vm_mon1_fip[\u0027floating_ip_address\u0027], user,"},{"line_number":266,"context_line":"            pkey\u003dself.keypair[\u0027private_key\u0027],"}],"source_content_type":"text/x-python","patch_set":14,"id":"4a498629_b7c3afbe","line":263,"in_reply_to":"12aa9306_1bc51e9e","updated":"2026-06-22 09:21:07.000000000","message":"ok, I don\u0027t see any easy way to get it done with cirros.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"7c5bb8f08ec545db22720b135c1c1cf2467c1402","unresolved":true,"context_lines":[{"line_number":260,"context_line":"            security_group\u003dself.secgroup[\u0027name\u0027],"},{"line_number":261,"context_line":"            port_security_enabled\u003dFalse,"},{"line_number":262,"context_line":"        )"},{"line_number":263,"context_line":"        user \u003d CONF.neutron_plugin_options.advanced_image_ssh_user"},{"line_number":264,"context_line":"        monitor1_client \u003d remote_client.RemoteClient("},{"line_number":265,"context_line":"            vm_mon1_fip[\u0027floating_ip_address\u0027], user,"},{"line_number":266,"context_line":"            pkey\u003dself.keypair[\u0027private_key\u0027],"}],"source_content_type":"text/x-python","patch_set":14,"id":"12aa9306_1bc51e9e","line":263,"in_reply_to":"2a632715_5cfb83b6","updated":"2026-06-17 13:46:19.000000000","message":"We need to use tcpdump to capture GRE packets, hence we are using advanced image.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"afa38e25fe968edbc46cbee62a423bbdd3fe60fc","unresolved":false,"context_lines":[{"line_number":260,"context_line":"            security_group\u003dself.secgroup[\u0027name\u0027],"},{"line_number":261,"context_line":"            port_security_enabled\u003dFalse,"},{"line_number":262,"context_line":"        )"},{"line_number":263,"context_line":"        user \u003d CONF.neutron_plugin_options.advanced_image_ssh_user"},{"line_number":264,"context_line":"        monitor1_client \u003d remote_client.RemoteClient("},{"line_number":265,"context_line":"            vm_mon1_fip[\u0027floating_ip_address\u0027], user,"},{"line_number":266,"context_line":"            pkey\u003dself.keypair[\u0027private_key\u0027],"}],"source_content_type":"text/x-python","patch_set":14,"id":"ddc65550_0a13c3eb","line":263,"in_reply_to":"4a498629_b7c3afbe","updated":"2026-06-22 09:35:38.000000000","message":"done.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0ba11f4408f5c2b0b07bc55d63c3906f667ba2b1","unresolved":true,"context_lines":[{"line_number":260,"context_line":"            security_group\u003dself.secgroup[\u0027name\u0027],"},{"line_number":261,"context_line":"            port_security_enabled\u003dFalse,"},{"line_number":262,"context_line":"        )"},{"line_number":263,"context_line":"        user \u003d CONF.neutron_plugin_options.advanced_image_ssh_user"},{"line_number":264,"context_line":"        monitor1_client \u003d remote_client.RemoteClient("},{"line_number":265,"context_line":"            vm_mon1_fip[\u0027floating_ip_address\u0027], user,"},{"line_number":266,"context_line":"            pkey\u003dself.keypair[\u0027private_key\u0027],"}],"source_content_type":"text/x-python","patch_set":14,"id":"2a632715_5cfb83b6","line":263,"in_reply_to":"927b1789_4eb2114e","updated":"2026-06-17 08:45:25.000000000","message":"If I may add a question, do these tests need advanced image? why?","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"0ba11f4408f5c2b0b07bc55d63c3906f667ba2b1","unresolved":true,"context_lines":[{"line_number":347,"context_line":"            self._check_icmp_mirror_direction(output2_lines, vm0_ip, vm1_ip,"},{"line_number":348,"context_line":"                                              \"BOTH\", \"key\u003d0x6a\")"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    @testtools.skipUnless("},{"line_number":351,"context_line":"        CONF.taas.driver \u003d\u003d \"ovn\","},{"line_number":352,"context_line":"        \"Does not support OVS, Only supports OVN\")"},{"line_number":353,"context_line":"    @decorators.idempotent_id(\u0027640e8e23-00f4-457c-8e91-04bb011fa94c\u0027)"},{"line_number":354,"context_line":"    def test_two_source_one_dest_remote_ip_mirror(self):"},{"line_number":355,"context_line":"        \"\"\"Test traffic from 2 src VMs mirrored to 1 destination VM remote IP"}],"source_content_type":"text/x-python","patch_set":14,"id":"b09860d9_4e0c66a2","line":352,"range":{"start_line":350,"start_character":0,"end_line":352,"end_character":50},"updated":"2026-06-17 08:45:25.000000000","message":"this can be dropped if using ovn property instead, and also changes under `zuul.d/`.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":35308,"name":"Bharath M V","email":"bmv@redhat.com","username":"bmv"},"change_message_id":"afa38e25fe968edbc46cbee62a423bbdd3fe60fc","unresolved":false,"context_lines":[{"line_number":347,"context_line":"            self._check_icmp_mirror_direction(output2_lines, vm0_ip, vm1_ip,"},{"line_number":348,"context_line":"                                              \"BOTH\", \"key\u003d0x6a\")"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    @testtools.skipUnless("},{"line_number":351,"context_line":"        CONF.taas.driver \u003d\u003d \"ovn\","},{"line_number":352,"context_line":"        \"Does not support OVS, Only supports OVN\")"},{"line_number":353,"context_line":"    @decorators.idempotent_id(\u0027640e8e23-00f4-457c-8e91-04bb011fa94c\u0027)"},{"line_number":354,"context_line":"    def test_two_source_one_dest_remote_ip_mirror(self):"},{"line_number":355,"context_line":"        \"\"\"Test traffic from 2 src VMs mirrored to 1 destination VM remote IP"}],"source_content_type":"text/x-python","patch_set":14,"id":"c9407a05_2788fe8e","line":352,"range":{"start_line":350,"start_character":0,"end_line":352,"end_character":50},"in_reply_to":"ad0e7f7c_43f0700c","updated":"2026-06-22 09:35:38.000000000","message":"The patch for the proposed config was abandoned. \nhttps://review.opendev.org/c/openstack/neutron-tempest-plugin/+/993591\n\nThe VM migration patch will also be modified to remove this non-existing config.","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"39e60e13d8a097e7dfe752c5b256ad45aa37b76e","unresolved":true,"context_lines":[{"line_number":347,"context_line":"            self._check_icmp_mirror_direction(output2_lines, vm0_ip, vm1_ip,"},{"line_number":348,"context_line":"                                              \"BOTH\", \"key\u003d0x6a\")"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    @testtools.skipUnless("},{"line_number":351,"context_line":"        CONF.taas.driver \u003d\u003d \"ovn\","},{"line_number":352,"context_line":"        \"Does not support OVS, Only supports OVN\")"},{"line_number":353,"context_line":"    @decorators.idempotent_id(\u0027640e8e23-00f4-457c-8e91-04bb011fa94c\u0027)"},{"line_number":354,"context_line":"    def test_two_source_one_dest_remote_ip_mirror(self):"},{"line_number":355,"context_line":"        \"\"\"Test traffic from 2 src VMs mirrored to 1 destination VM remote IP"}],"source_content_type":"text/x-python","patch_set":14,"id":"d26a4d7e_a5482b45","line":352,"range":{"start_line":350,"start_character":0,"end_line":352,"end_character":50},"in_reply_to":"b09860d9_4e0c66a2","updated":"2026-06-22 09:08:02.000000000","message":"so do we even need that config option still? If not, it should be probably removed in follow up patch, I\u0027m just asking here :)","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":33341,"name":"Maor Blaustein","email":"mblue@redhat.com","username":"blue"},"change_message_id":"cc307ab6454fd30100f648f3f30fcbe010acadf4","unresolved":true,"context_lines":[{"line_number":347,"context_line":"            self._check_icmp_mirror_direction(output2_lines, vm0_ip, vm1_ip,"},{"line_number":348,"context_line":"                                              \"BOTH\", \"key\u003d0x6a\")"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    @testtools.skipUnless("},{"line_number":351,"context_line":"        CONF.taas.driver \u003d\u003d \"ovn\","},{"line_number":352,"context_line":"        \"Does not support OVS, Only supports OVN\")"},{"line_number":353,"context_line":"    @decorators.idempotent_id(\u0027640e8e23-00f4-457c-8e91-04bb011fa94c\u0027)"},{"line_number":354,"context_line":"    def test_two_source_one_dest_remote_ip_mirror(self):"},{"line_number":355,"context_line":"        \"\"\"Test traffic from 2 src VMs mirrored to 1 destination VM remote IP"}],"source_content_type":"text/x-python","patch_set":14,"id":"ad0e7f7c_43f0700c","line":352,"range":{"start_line":350,"start_character":0,"end_line":352,"end_character":50},"in_reply_to":"d26a4d7e_a5482b45","updated":"2026-06-22 09:21:07.000000000","message":"I think new oslo configuration was abandoned here [1], but still seeing similar occurences for non existing `taas.driver` in patch [2].\n\n@bmv@redhat.com should [2] be edited additionally?\n\n[1]\n993591: [TaaS] Add a driver option to taas test config. | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/993591\n\n[2]\n991062: [Tap-as-a-Service] Adding VM migration scenario test. | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/991062","commit_id":"b13867bcffc240d270df668ba2eb0343c52845d4"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"f4b6e63fe45aee23698cd1d6444a98959c40c0af","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import testtools"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from tempest.common import utils"},{"line_number":16,"context_line":"from tempest import config"}],"source_content_type":"text/x-python","patch_set":16,"id":"16aab1a1_545a3b92","line":13,"updated":"2026-06-17 11:21:28.000000000","message":"pep8: F401 \u0027testtools\u0027 imported but unused","commit_id":"01d3f314a4ad32cf6577238b1f70d36e0d5b394f"}]}