)]}'
{"neutron/db/evpn_db.py":[{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"4fcdc8a4a7bcdb07fc92da3355fef08b946e36d4","unresolved":true,"context_lines":[{"line_number":189,"context_line":"            router_id\u003drouter_id))"},{"line_number":190,"context_line":"        context.session.flush()"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"        context.session.add(evpn_models.EVPNAdvertisedPort("},{"line_number":193,"context_line":"            port_id\u003dport_id,"},{"line_number":194,"context_line":"            network_id\u003dnetwork_id))"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        LOG.debug(\"EVPN advertise port %s on network %s for router %s\","},{"line_number":197,"context_line":"                  port_id, network_id, router_id)"}],"source_content_type":"text/x-python","patch_set":8,"id":"dd66ec8e_1bdab9af","line":194,"range":{"start_line":192,"start_character":0,"end_line":194,"end_character":35},"updated":"2026-08-14 13:58:38.000000000","message":"This was AI change :) . I dont mind this streamline, but I can revert it to the previous, explicit \"create an object\" and then add it to `add`.","commit_id":"420cc6bd657ae9df1ea4c342ec2b996fbe890a13"}],"neutron/services/evpn/exceptions.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"67ae262e7fcb3f69b324c7d5bfb81fefd2319d54","unresolved":true,"context_lines":[{"line_number":48,"context_line":"class EVPNAdvertisedSubnetConflict(exceptions.Conflict):"},{"line_number":49,"context_line":"    message \u003d _(\"Network %(network_id)s already has an advertised subnet \""},{"line_number":50,"context_line":"                \"on EVPN router %(router_id)s. Only one subnet per network \""},{"line_number":51,"context_line":"                \"can be advertised with --advertise-host.\")"}],"source_content_type":"text/x-python","patch_set":6,"id":"b7a5eefb_d21a2e94","line":51,"range":{"start_line":51,"start_character":40,"end_line":51,"end_character":56},"updated":"2026-08-14 13:05:10.000000000","message":"nit: in the exception the -- is perhaps overload as the user can arrive here not just from the CLI but from any API opertion","commit_id":"4fe6e4a4274a9b4610c48b22ae1cfd853fe22f3b"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"72531cbb926435e910a309e6f98f5bd0c5d87063","unresolved":false,"context_lines":[{"line_number":48,"context_line":"class EVPNAdvertisedSubnetConflict(exceptions.Conflict):"},{"line_number":49,"context_line":"    message \u003d _(\"Network %(network_id)s already has an advertised subnet \""},{"line_number":50,"context_line":"                \"on EVPN router %(router_id)s. Only one subnet per network \""},{"line_number":51,"context_line":"                \"can be advertised with --advertise-host.\")"}],"source_content_type":"text/x-python","patch_set":6,"id":"9c60c42a_a4145ff7","line":51,"range":{"start_line":51,"start_character":40,"end_line":51,"end_character":56},"in_reply_to":"b7a5eefb_d21a2e94","updated":"2026-08-14 13:52:13.000000000","message":"Thanks for the review. Good point about the `--`. I\u0027ll remove it. Also, I just pushed new changes based on conversation I had with Jakub offline.","commit_id":"4fe6e4a4274a9b4610c48b22ae1cfd853fe22f3b"}]}
