)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"de289cad2d57d1101dfb5d34c77b54b3feb6bd04","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Commit d02fb560aa35c4931d40000e8ca4c0e4105299b7 to Neutron disabled \u0027mcast_flood\u0027 on localnet ports, in order to avoid multicast packet duplication. This introduces a regression because outbound multicast traffic from VMs is no longer sent to the provider network unless it is part of a multicast group that OVN has snooped. This makes it impossible for something external to OpenStack to subscribe to a multicast group running on a VM (in OpenStack) unless another VM (in OpenStack) subscribes to it first (because the multicast traffic never leaves the OVS)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Opened issue on ovn: https://github.com/ovn-org/ovn/issues/125"},{"line_number":16,"context_line":"Opened bug to track progress: https://bugs.launchpad.net/neutron/+bug/1967996"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I3390b8c65f4b8572be0ab766024ed4b4ec4aa4ae"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f46e8a10_058e7d0f","line":15,"updated":"2022-04-06 09:29:46.000000000","message":"Hi,\n\nBefore we commit to this revert it would be good to have the fix in core OVN otherwise we will introduce the same duplication problem as before. I see you opened an github issue for this, but I don\u0027t think that OVN developers actually look into GH issues instead they use the ovs-discuss mailing list [0] for bug reports. Do you mind reporting this issue there and see how the community responds to this issue ?\n\n[0] https://docs.ovn.org/en/latest/internals/mailing-lists.html","commit_id":"7ac2a320cf365d18c3324833dda9ee34977bf7af"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"43558d0a88071ad5440323d7683d78d97018f87a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"efce3aef_9f6e3c6a","updated":"2022-08-18 13:13:37.000000000","message":"Hi Diko,\n\nHave you tested this after the newest changes in core OVN ? Dumitru refactored part of core OVN for IGMP (see https://github.com/ovn-org/ovn/commit/6aeeccdf272bc60630581e46aa42d97f4f56d4fa) and when I spoke with him he mentioned that we should set mcast_flood to False. This revert changes that, so I think we need to test the latest OVN changes prior to going ahead with this revert.\n\nFor reference, this is a snippet of when I was testing dumitru\u0027s newest IGMP changes in OVN:\n\nAug 03 09:59:30 \u003clucasagomes\u003e   no worries, just to confirm. We need the mcast_flood set to false on all LSPs right ?                                                                         \nAug 03 09:59:35 \u003clucasagomes\u003e   either false or unset                                                                                                                                         \nAug 03 10:04:07 \u003cdceara\u003e    Right.                                                                                                                                                            \nAug 03 10:07:52 \u003clucasagomes\u003e   ++ thanks                                                                                                                                                     \nAug 03 10:08:00 \u003cdceara\u003e    thank you :)","commit_id":"dc83924d23557a5077139b441c365a538aac0dcc"}]}