)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ac890716a7e9ca8e4058d8d69d28f98df897d96d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e1743bde_5f9c2c2c","updated":"2023-05-30 08:38:17.000000000","message":"recheck\nneutron_tempest_plugin.scenario.test_dns_integration failure seems irrelevant","commit_id":"a846113a60d2f2e3b9213e20dafe48a08b495abf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c5904da9eca1f5eafd47b6bf45620ba391c2b641","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4c2b4b15_28b1bdaa","updated":"2023-06-06 12:23:08.000000000","message":"recheck\ngrenade failure:\n\"[ERROR] /opt/stack/new/grenade/inc/upgrade:159 The following services are still running:  nova-api \"","commit_id":"6312b3deac0ba46bf09ed3614e3dd8185397c413"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1a82d1269e3894208fcea43475cfe038d4583c82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e0a7e3eb_3cc79db8","updated":"2023-06-05 08:01:09.000000000","message":"recheck\nneutron-fullstack-with-uwsgi timeout","commit_id":"6312b3deac0ba46bf09ed3614e3dd8185397c413"}],"neutron/api/rpc/handlers/securitygroups_rpc.py":[{"author":{"_account_id":9642,"name":"Guillaume Espanel","email":"guillaume.espanel@gmail.com","username":"quatre"},"change_message_id":"fddf5610972be9900aeac87b7e78c84cfaa3002b","unresolved":true,"context_lines":[{"line_number":308,"context_line":"                                               rule.id)"},{"line_number":309,"context_line":"        # If there\u0027s a rule which remote is the deleted sg, remove that also."},{"line_number":310,"context_line":"        filters \u003d {\u0027remote_group_id\u0027: (existing.id, )}"},{"line_number":311,"context_line":"        for rule in self.rcache.get_resources(\u0027SecurityGroupRule\u0027, filters):"},{"line_number":312,"context_line":"            self.rcache.record_resource_delete(context, \u0027SecurityGroupRule\u0027,"},{"line_number":313,"context_line":"                                               rule.id)"},{"line_number":314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"4dce767b_4d5a46a2","line":311,"updated":"2023-05-26 14:32:37.000000000","message":"I think this also triggers a bulk_pull of the rules from all the agents upon SG deletion. This [1] might be relevant, what do you think?\n\n[1] https://review.opendev.org/c/openstack/neutron/+/883235.","commit_id":"a846113a60d2f2e3b9213e20dafe48a08b495abf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ea59b0f22ea0269075f7d5f6b8c62372c2a740a8","unresolved":true,"context_lines":[{"line_number":308,"context_line":"                                               rule.id)"},{"line_number":309,"context_line":"        # If there\u0027s a rule which remote is the deleted sg, remove that also."},{"line_number":310,"context_line":"        filters \u003d {\u0027remote_group_id\u0027: (existing.id, )}"},{"line_number":311,"context_line":"        for rule in self.rcache.get_resources(\u0027SecurityGroupRule\u0027, filters):"},{"line_number":312,"context_line":"            self.rcache.record_resource_delete(context, \u0027SecurityGroupRule\u0027,"},{"line_number":313,"context_line":"                                               rule.id)"},{"line_number":314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d38b18c4_642228f7","line":311,"in_reply_to":"4dce767b_4d5a46a2","updated":"2023-05-26 15:24:32.000000000","message":"Thanks I will check it next Monday","commit_id":"a846113a60d2f2e3b9213e20dafe48a08b495abf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"604a2b448805e5d0ecac6b793818852196e854e4","unresolved":true,"context_lines":[{"line_number":308,"context_line":"                                               rule.id)"},{"line_number":309,"context_line":"        # If there\u0027s a rule which remote is the deleted sg, remove that also."},{"line_number":310,"context_line":"        filters \u003d {\u0027remote_group_id\u0027: (existing.id, )}"},{"line_number":311,"context_line":"        for rule in self.rcache.get_resources(\u0027SecurityGroupRule\u0027, filters):"},{"line_number":312,"context_line":"            self.rcache.record_resource_delete(context, \u0027SecurityGroupRule\u0027,"},{"line_number":313,"context_line":"                                               rule.id)"},{"line_number":314,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"897f43f3_a12d9847","line":311,"in_reply_to":"d38b18c4_642228f7","updated":"2023-06-02 13:25:08.000000000","message":"finally I got back to this topic :-)\nThanks, I change this call to avoid as much as possible the batch rpc call to the server, thanks for the review/help","commit_id":"a846113a60d2f2e3b9213e20dafe48a08b495abf"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1090c16458314f6d1758ccd3b86a5873792aa7f4","unresolved":true,"context_lines":[{"line_number":307,"context_line":"            self.rcache.record_resource_delete(context, \u0027SecurityGroupRule\u0027,"},{"line_number":308,"context_line":"                                               rule.id)"},{"line_number":309,"context_line":"        # If there\u0027s a rule which remote is the deleted sg, remove that also."},{"line_number":310,"context_line":"        rules \u003d self.rcache.match_resources_with_func("},{"line_number":311,"context_line":"            \u0027SecurityGroupRule\u0027,"},{"line_number":312,"context_line":"            lambda sg_rule: sg_rule.remote_group_id \u003d\u003d existing.id)"},{"line_number":313,"context_line":"        for rule in rules:"}],"source_content_type":"text/x-python","patch_set":3,"id":"0b32831a_ba37bc99","line":310,"updated":"2023-06-30 11:10:46.000000000","message":"This is very similar to [1] implementation.\n\nHowever I this is only for RPC agent mech drivers (that means OVN code is not affected). I need to check if this bug affects OVN.\n\n[1]https://review.opendev.org/c/openstack/neutron/+/883235/3/neutron/api/rpc/handlers/securitygroups_rpc.py#308","commit_id":"67a0b0728788207cee27adb586880fabb8da6f25"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"373fa6ce5a16c3eb96569a508d47a15b0ca19def","unresolved":false,"context_lines":[{"line_number":307,"context_line":"            self.rcache.record_resource_delete(context, \u0027SecurityGroupRule\u0027,"},{"line_number":308,"context_line":"                                               rule.id)"},{"line_number":309,"context_line":"        # If there\u0027s a rule which remote is the deleted sg, remove that also."},{"line_number":310,"context_line":"        rules \u003d self.rcache.match_resources_with_func("},{"line_number":311,"context_line":"            \u0027SecurityGroupRule\u0027,"},{"line_number":312,"context_line":"            lambda sg_rule: sg_rule.remote_group_id \u003d\u003d existing.id)"},{"line_number":313,"context_line":"        for rule in rules:"}],"source_content_type":"text/x-python","patch_set":3,"id":"caf9336a_1df3a692","line":310,"in_reply_to":"0b32831a_ba37bc99","updated":"2023-06-30 13:02:15.000000000","message":"Ok, when the remote SG is deleted, the local SG with the remote_group_id\u003d\u003csg\u003e is deleted from OVN:\n* The local Port_Group is updated.\n* The ACL in the Port_Group is deleted\n* The remote SG Port_Group is deleted.\n\nIn a nutshell, this issue was not affecting ML2/OVN.","commit_id":"67a0b0728788207cee27adb586880fabb8da6f25"}]}