)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"977895c72599de9d4d99748b57605d7caf29e6c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d1aaec00_ab43f158","updated":"2023-03-07 08:31:31.000000000","message":"Lets check how it will work in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/876692","commit_id":"5465cdde1b1652f66bf319c0f826a40fa5c28af8"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"053da2f5d89b2764d3bef24c3fa8875e65b34596","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c9539fd8_5903d691","updated":"2023-03-08 21:08:33.000000000","message":"Looks like it\u0027s still failing in the stateless tempest test: https://zuul.opendev.org/t/openstack/build/18320a74fae44a92b9defe5a7485e5af/console\n\nI\u0027ll have to investigate before we land this patch.","commit_id":"5465cdde1b1652f66bf319c0f826a40fa5c28af8"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"004f230f9eb2e9e13ec0d78b7aceb872e7f5b59f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"044f89ff_caac7ac8","updated":"2023-03-09 00:44:47.000000000","message":"The problem with the neutron tempest plugin failures was because of the last minute edition to the patch where I added tcp_src \u003d\u003d 80 to the match string to narrow it down somewhat. I\u0027ve added it after I validated that the patch works; then I only worked on migrations / inconsistencies, but I haven\u0027t re-validated that metadata is still working.\n\nThe next revision should pass tempest tests.","commit_id":"5465cdde1b1652f66bf319c0f826a40fa5c28af8"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"87950fd6831dc2d96452c5e2ef22c67c8423c8c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"daeee891_a3598144","updated":"2023-03-07 08:21:36.000000000","message":"recheck - unrelated rally job timeout","commit_id":"5465cdde1b1652f66bf319c0f826a40fa5c28af8"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ab019e009de212917e55fd5c5d690748c51eb5f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"74e7655b_ba4a0706","updated":"2023-03-09 10:05:08.000000000","message":"Same as in the other patch, a FT checking \"configure_service_traffic_acls\" could improve the code coverage.","commit_id":"d95efeefa6d7c119d719e1ea79fdb472894829d1"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1a5f331a45085065bafffd28ef90d2463afcc3e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"35729343_cdd8c6a6","updated":"2023-03-09 13:47:55.000000000","message":"The tempest run passed stateless tests without metadata workaround here: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_acd/876692/2/check/neutron-tempest-plugin-ovn/acd3733/testr_results.html","commit_id":"d95efeefa6d7c119d719e1ea79fdb472894829d1"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"4b75f4300fc8dc1c69f3bbc5bbc0286a48799c88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cada7f89_380a2048","in_reply_to":"74e7655b_ba4a0706","updated":"2023-03-09 18:32:23.000000000","message":"What makes sense is extending the existing test cases that already update stateful field from TestSecurityGroup(base.TestOVNFunctionalBase) to also check the new metadata ACL.","commit_id":"d95efeefa6d7c119d719e1ea79fdb472894829d1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"03139a0b2ae201af299b2420dab4be8c0a0e8ffd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4156025f_3f8a993f","updated":"2023-03-21 16:06:33.000000000","message":"As commented via mail, the metadata service should be explicitly allowed by the SG rules. These rules will depend on the SG type (egress for stateful, or egress+ingress for stateless).","commit_id":"4c80527d8292141a2b4b8f28f270fad07d487f62"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"9447cef85cfaba07736e02e49eff78555e7a40e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"155cb410_386a9881","updated":"2023-03-14 15:33:20.000000000","message":"recheck bug 2011600\n\nfunctional test failure in test_ovs_lib, timout failure","commit_id":"4c80527d8292141a2b4b8f28f270fad07d487f62"}],"neutron/common/ovn/acl.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d7258169c8a2ec6f67688afae5140f4fc3ab4cf7","unresolved":true,"context_lines":[{"line_number":276,"context_line":"        txn.add(ovn.pg_acl_add(**acl, may_exist\u003dTrue))"},{"line_number":277,"context_line":""},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"def get_stateless_acls(security_group_id):"},{"line_number":280,"context_line":"    metadata_rule \u003d {"},{"line_number":281,"context_line":"        \u0027direction\u0027: const.INGRESS_DIRECTION,"},{"line_number":282,"context_line":"        \u0027ethertype\u0027: const.IPv4,"}],"source_content_type":"text/x-python","patch_set":3,"id":"6c7f7df6_e71ad607","line":279,"updated":"2023-03-14 16:52:43.000000000","message":"add comment about potential future adjustment for when ipv6 metadata is supported by the backend (we would have to add a separate ACL for metadata v6 ip / port.)","commit_id":"4c80527d8292141a2b4b8f28f270fad07d487f62"}]}