)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00fcba428557719ffde9ff95af545ce1d1b029a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7865fa1d_a72786ba","updated":"2023-05-16 15:10:59.000000000","message":"-1 just for visibility\n\nSome comments related to Miguel\u0027s bug description.\n1) That was tested (by me) that this issue was affecting ML2/OVS and not ML2/OVN (probably ML2/LB too, but never tested). We should find an approach only for the affected backends.\n\n2) https://review.opendev.org/c/openstack/neutron/+/876716/1/neutron/api/rpc/handlers/securitygroups_rpc.py is only fixing the \"SecurityGroupServerAPIShim\" class, that is used only by ML2/OVS, not ML2/LB.\n\n3) As commented by Miguel, we are breaking the semantics: we should not send a SG_rule event in a SG action. In any case, we can consider dealing with this in the SG delete event. The problem is that we are not currently subscribed to these events in, for example, the ML2/OVS agent.","commit_id":"80c2875dac5819d3ce786558d8c18e78d4a8ad60"}],"neutron/db/securitygroups_db.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"25e0dbda4d445539411de8d4fef3f4b6744ec177","unresolved":true,"context_lines":[{"line_number":286,"context_line":"                             metadata\u003d{\u0027security_group_rule_ids\u0027: sgr_ids,"},{"line_number":287,"context_line":"                                       \u0027name\u0027: sg[\u0027name\u0027]}))"},{"line_number":288,"context_line":"        for rule in rules_ids_as_remote:"},{"line_number":289,"context_line":"            # Some security groups, like the default created for projects,"},{"line_number":290,"context_line":"            # have rules that point to it as remote group. No need to publish"},{"line_number":291,"context_line":"            # event for those rules, since they will be deleted with the"},{"line_number":292,"context_line":"            # security group"},{"line_number":293,"context_line":"            if rule[\u0027security_group_id\u0027] !\u003d rule[\u0027remote_group_id\u0027]:"},{"line_number":294,"context_line":"                registry.publish("}],"source_content_type":"text/x-python","patch_set":2,"id":"21a912a8_27516d5c","line":291,"range":{"start_line":289,"start_character":12,"end_line":291,"end_character":72},"updated":"2023-05-16 07:52:24.000000000","message":"true, I missed this, default groups are not my favourite....","commit_id":"80c2875dac5819d3ce786558d8c18e78d4a8ad60"}]}