)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"62e0145dd1f76dec1adcea5d72f6035aaefe5c8e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[OVN] Fix inconsistent IGMP configuration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Prior to this patch the IGMP configuration for ML2/OVN was incosistent"},{"line_number":10,"context_line":"with the configuration option description and also the ML2/OVS driver"},{"line_number":11,"context_line":"because it was flooding traffic to unregistered VMs [0]."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f621f24_3ede69d9","line":9,"range":{"start_line":9,"start_character":59,"end_line":9,"end_character":70},"updated":"2020-11-16 12:00:20.000000000","message":"nit: inconsistent","commit_id":"3a13f289854b06fb2569860e02144f0713882221"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b11758f74a4a8071138bd854cf3966e0b4f7ef68","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[OVN] Fix inconsistent IGMP configuration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Prior to this patch the IGMP configuration for ML2/OVN was incosistent"},{"line_number":10,"context_line":"with the configuration option description and also the ML2/OVS driver"},{"line_number":11,"context_line":"because it was flooding traffic to unregistered VMs [0]."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f621f24_24581c86","line":9,"range":{"start_line":9,"start_character":59,"end_line":9,"end_character":70},"in_reply_to":"1f621f24_3ede69d9","updated":"2020-11-16 14:25:41.000000000","message":"ops","commit_id":"3a13f289854b06fb2569860e02144f0713882221"}],"doc/source/admin/ovn/igmp.rst":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"62e0145dd1f76dec1adcea5d72f6035aaefe5c8e","unresolved":false,"context_lines":[{"line_number":37,"context_line":"The ``igmp_snooping_enable`` configuration from Neutron is translated"},{"line_number":38,"context_line":"into the ``mcast_snoop`` option set in the ``other_config`` column"},{"line_number":39,"context_line":"from the ``Logical_Switch`` table in the OVN Northbound Database"},{"line_number":40,"context_line":"(``mcast_flood_unregistered`` is always \"false\"):"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":".. code-block:: bash"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f621f24_9e97bdec","line":40,"range":{"start_line":40,"start_character":3,"end_line":40,"end_character":27},"updated":"2020-11-16 12:00:20.000000000","message":"Let me ask you: if igmp_snooping_enable\u003dFalse (default), we should allow mcast traffic to all ports, this is the default behavior. Why this parameter is always false?","commit_id":"3a13f289854b06fb2569860e02144f0713882221"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"b11758f74a4a8071138bd854cf3966e0b4f7ef68","unresolved":false,"context_lines":[{"line_number":37,"context_line":"The ``igmp_snooping_enable`` configuration from Neutron is translated"},{"line_number":38,"context_line":"into the ``mcast_snoop`` option set in the ``other_config`` column"},{"line_number":39,"context_line":"from the ``Logical_Switch`` table in the OVN Northbound Database"},{"line_number":40,"context_line":"(``mcast_flood_unregistered`` is always \"false\"):"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":".. code-block:: bash"},{"line_number":43,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1f621f24_24b49c7f","line":40,"range":{"start_line":40,"start_character":3,"end_line":40,"end_character":27},"in_reply_to":"1f621f24_9e97bdec","updated":"2020-11-16 14:25:41.000000000","message":"Yes, this will work (tested and talked with dumitru, the core OVN engineer that implemented the IGMP feature).\n\nBasically if mcast_snoop\u003dfalse (which is set accordingly to igmp_snooping_enable config) the mcast_flood_unregistered configuration will be ignored and the multicast traffic will be flooded.\n\nSo having it set to \"false\" always in this patch does two things:\n\n1) It will be consistent with the current ML2/OVS behavior\n\n2) It will make backport easier\n\nTo expand on 2), I believe we need a new configuration option for this behavior but that would make backports more complex (adding new config). So I decided to fix it like this first and later add a new configuration so people can tweak this behavior for their envs as they like.\n\nWhat you think ?","commit_id":"3a13f289854b06fb2569860e02144f0713882221"}],"releasenotes/notes/ovn-igmp-flood-unregistered-82aeb640f5dded84.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e2d0a7a206d3d37da5ef0aa05ae5f733d099ff0e","unresolved":false,"context_lines":[{"line_number":2,"context_line":"issues:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Even with the \"igmp_snooping_enable\" configuration option stating that"},{"line_number":5,"context_line":"    traffic would not be flooaded to unregistered VMs when this option was"},{"line_number":6,"context_line":"    enabled, the ML2/OVN driver didn\u0027t follow that behavior. This"},{"line_number":7,"context_line":"    has now been fixed and ML2/OVN will no longer flood traffic to"},{"line_number":8,"context_line":"    unregistered VMs when this configuration option is set to True."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fffc6b78_0a47ba1f","line":5,"range":{"start_line":5,"start_character":25,"end_line":5,"end_character":33},"updated":"2020-11-20 09:25:06.000000000","message":"nit: should be \"flooded\" :)","commit_id":"3a13f289854b06fb2569860e02144f0713882221"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"95a22afd3ae10c377650dd4dccdab68fa8f580c6","unresolved":false,"context_lines":[{"line_number":2,"context_line":"issues:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Even with the \"igmp_snooping_enable\" configuration option stating that"},{"line_number":5,"context_line":"    traffic would not be flooaded to unregistered VMs when this option was"},{"line_number":6,"context_line":"    enabled, the ML2/OVN driver didn\u0027t follow that behavior. This"},{"line_number":7,"context_line":"    has now been fixed and ML2/OVN will no longer flood traffic to"},{"line_number":8,"context_line":"    unregistered VMs when this configuration option is set to True."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"160d18b7_d792e9dc","line":5,"range":{"start_line":5,"start_character":25,"end_line":5,"end_character":33},"in_reply_to":"fffc6b78_0a47ba1f","updated":"2020-11-25 16:46:38.000000000","message":"Ops, thanks Slaweq!","commit_id":"3a13f289854b06fb2569860e02144f0713882221"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"609d2fcb2ce05ce46479e2b3c196b1d2b899246b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"issues:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Even with the \"igmp_snooping_enable\" configuration option stating that"},{"line_number":5,"context_line":"    traffic would not be flooded to unregistered VMs when this option was"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"26d3b0dc_5157c9c0","line":2,"updated":"2021-04-03 06:34:56.000000000","message":"The should be \"fixes\", because IIUC it describes a resolved issue, not a known open issue.","commit_id":"9dc8bca7401106313cb227cb85d07eba7d987cb5"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"de0f773388b1c0cada083bf348c2d30a17c1f379","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"issues:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Even with the \"igmp_snooping_enable\" configuration option stating that"},{"line_number":5,"context_line":"    traffic would not be flooded to unregistered VMs when this option was"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"32a3de81_ba1bb2e0","line":2,"in_reply_to":"26d3b0dc_5157c9c0","updated":"2021-04-06 08:24:23.000000000","message":"Ops, true... do you think it worth sending another patch fixing this release note ?","commit_id":"9dc8bca7401106313cb227cb85d07eba7d987cb5"}]}