)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"0668d4300909692f2a9d7f6ba71cd03dce4514a7","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"NOTE: the floating IP QoS rules are deleted matching the QoS external_ids"},{"line_number":19,"context_line":"[2], instead of creating the same QoS rule, thus the priority is not"},{"line_number":20,"context_line":"needed for the delation."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[1]https://www.ovn.org/support/dist-docs/ovn-nb.5.html"},{"line_number":23,"context_line":"[2]https://github.com/openstack/neutron/blob/123bd115f3b65ba09560685ad6cf68c6934a6535/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/extensions/qos.py#L451-L456"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b03f7d20_be3df15c","line":20,"range":{"start_line":20,"start_character":15,"end_line":20,"end_character":23},"updated":"2025-05-08 07:10:53.000000000","message":"nitty nit: should be \"deletion\" 😊","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e73fd4edfd849677dcd90414b49d697168b19efb","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"NOTE: the floating IP QoS rules are deleted matching the QoS external_ids"},{"line_number":19,"context_line":"[2], instead of creating the same QoS rule, thus the priority is not"},{"line_number":20,"context_line":"needed for the delation."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[1]https://www.ovn.org/support/dist-docs/ovn-nb.5.html"},{"line_number":23,"context_line":"[2]https://github.com/openstack/neutron/blob/123bd115f3b65ba09560685ad6cf68c6934a6535/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/extensions/qos.py#L451-L456"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1037ffd6_8825ed1f","line":20,"range":{"start_line":20,"start_character":15,"end_line":20,"end_character":23},"in_reply_to":"b03f7d20_be3df15c","updated":"2025-05-08 07:16:55.000000000","message":"Done","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3d26dd9a31aaf7f2f77a062466bbd8e7492cec58","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"62928399_d7546322","updated":"2025-05-06 14:42:39.000000000","message":"WIP: it is needed a maintenance method to update all the existing FIP QoS rules.","commit_id":"3d211dcc91fb255df021e9e2ae083341547d5f49"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2e2fe3493780f4e9bcdf0286b45693941a6b6469","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9f05c98d_baea681e","updated":"2025-05-08 14:50:39.000000000","message":"I just updated the relnote grammar and added the but url","commit_id":"012327bb4dfa8ab1817fb03cac530a8daa109f98"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4b25a689cfcad575ef85b305ad45f5eebe3cb2ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4131138d_a10ced07","updated":"2025-05-09 09:00:50.000000000","message":"recheck neutron-functional","commit_id":"012327bb4dfa8ab1817fb03cac530a8daa109f98"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"5c73c371e9a318bebe730edce4ca5375a519e462","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3d5def81_2d346823","in_reply_to":"9f05c98d_baea681e","updated":"2025-05-08 14:56:17.000000000","message":"Thanks!","commit_id":"012327bb4dfa8ab1817fb03cac530a8daa109f98"}],"releasenotes/notes/ovn-qos-fip-rule-priority-16ad3908790dfa7d.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"0668d4300909692f2a9d7f6ba71cd03dce4514a7","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":"    The OVN QoS floating IP rules have now a higher priority than the OVN QoS"},{"line_number":5,"context_line":"    router rules. In case of having both, the floating IP rule will take"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"0f84939f_7e5479c3","line":2,"updated":"2025-05-08 07:10:53.000000000","message":"The \"issues\" section should contain list of known issues in the release. This is IMO not a known issue but rather fix so if we really need release note for that, I think it should be in the \"fixes\" section, no?","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b10e92572d6e675eb92dba016b7b2520a6947b6b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"issues:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The OVN QoS floating IP rules have now a higher priority than the OVN QoS"},{"line_number":5,"context_line":"    router rules. In case of having both, the floating IP rule will take"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"36ccc384_c80b474c","line":2,"in_reply_to":"0f84939f_7e5479c3","updated":"2025-05-08 07:15:00.000000000","message":"Done","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"0668d4300909692f2a9d7f6ba71cd03dce4514a7","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The OVN QoS floating IP rules have now a higher priority than the OVN QoS"},{"line_number":5,"context_line":"    router rules. In case of having both, the floating IP rule will take"},{"line_number":6,"context_line":"    precedence over the router rule, finishing the selection uncertainty."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"e268d712_89b88896","line":6,"updated":"2025-05-08 07:10:53.000000000","message":"TBH I am not sure if we really need release note for that. This is more about internal implementation in one of the backends, those priorities are not something what is visible for the users and such release note may IMHO be confusing for some of users who will read release notes.","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b10e92572d6e675eb92dba016b7b2520a6947b6b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The OVN QoS floating IP rules have now a higher priority than the OVN QoS"},{"line_number":5,"context_line":"    router rules. In case of having both, the floating IP rule will take"},{"line_number":6,"context_line":"    precedence over the router rule, finishing the selection uncertainty."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"24fc7f0c_088acf84","line":6,"in_reply_to":"e268d712_89b88896","updated":"2025-05-08 07:15:00.000000000","message":"Yes, the priorities are not visible but at affecting the QoS selection. If both rules have the same priority, OVN will chose one randomly. We want FIP rule to have precedence over the router rule. I\u0027ll update this note.","commit_id":"a09414ae8503c649d3739e920e1906bf5ae53169"}]}
