)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"add34ada4dda480c797f47dafa20c89ae9adab6c","unresolved":true,"context_lines":[{"line_number":13,"context_line":"  * bfd-min-tx: 100"},{"line_number":14,"context_line":"  * bfd-mult: 3"},{"line_number":15,"context_line":"* aggressive"},{"line_number":16,"context_line":"  * bfd-min-rx: 1000 * 2"},{"line_number":17,"context_line":"  * bfd-min-tx: 100"},{"line_number":18,"context_line":"  * bfd-mult: 3"},{"line_number":19,"context_line":"* conservative"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bd0ace38_f5852df1","line":16,"range":{"start_line":16,"start_character":16,"end_line":16,"end_character":24},"updated":"2026-02-24 14:37:30.000000000","message":"1000 / 2","commit_id":"844d9de9db8d28ee5e03085d9b800b4899fe7be5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"fe38601e0966861c93f544387a9ce48910d416ad","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  * bfd-min-tx: 100"},{"line_number":14,"context_line":"  * bfd-mult: 3"},{"line_number":15,"context_line":"* aggressive"},{"line_number":16,"context_line":"  * bfd-min-rx: 1000 * 2"},{"line_number":17,"context_line":"  * bfd-min-tx: 100"},{"line_number":18,"context_line":"  * bfd-mult: 3"},{"line_number":19,"context_line":"* conservative"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1c9fdd71_a8aa4034","line":16,"range":{"start_line":16,"start_character":16,"end_line":16,"end_character":24},"in_reply_to":"bd0ace38_f5852df1","updated":"2026-02-27 15:54:26.000000000","message":"Done","commit_id":"844d9de9db8d28ee5e03085d9b800b4899fe7be5"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"d75ab6c0501f3ebe52596bf0fa963dbd53de5297","unresolved":true,"context_lines":[{"line_number":23,"context_line":"* manual: the three parameters values will be read from the"},{"line_number":24,"context_line":"  configuration file."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"The HA failover time can be calculated in the following way:"},{"line_number":27,"context_line":"  failover time \u003d bfd_min_rx * bfd_mult (\u003d\u003d 3 seconds by default)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Closes-Bug: #2142096"},{"line_number":30,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"0076cdb2_2e86c74b","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":65},"updated":"2026-03-04 15:01:05.000000000","message":"IIUC, as per the RFC [0] the detection time is:\n\nmax(bfd-min-tx, bfd-max-tx)*bfd-mult\n\n\"the value of Detect Mult received from the remote\n   system, multiplied by the agreed transmit interval of the remote\n   system (the greater of bfd.RequiredMinRxInterval and the last\n   received Desired Min TX Interval).\"\n\nSince this is seems how OVS implements it [1][2], we can potentially change the ``bfd-min-tx`` to match the ``bfd-max-rx``for a more balanced approach? That way we don\u0027t keep the tx rate at 100ms, we avoid sending unnecessary traffic.\n\nFor example, in the conservative mode, we are sending 10 packets per second but detection happens only every 6 seconds so we\u0027re creating unnecessary network traffic and cpu load (minimal though?).\n\nSo my suggestion would be perhaps matching bfd-min-tx to bfd-min-rx in all cases or at least in the conservative but could also be a follow-up if needed.\n\n[0] https://datatracker.ietf.org/doc/html/rfc5880\n[1] https://github.com/openvswitch/ovs/blob/b9c66cf106f6730e548d04678fdf638172172d09/lib/bfd.c#L1224\n[2] https://github.com/openvswitch/ovs/blob/b9c66cf106f6730e548d04678fdf638172172d09/lib/bfd.c#L999","commit_id":"7d06035adeb3b5fcec149b226e24cf76e6e997ad"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c3067bbb49b788f0ae9a3a5ab523cea65c520bac","unresolved":false,"context_lines":[{"line_number":23,"context_line":"* manual: the three parameters values will be read from the"},{"line_number":24,"context_line":"  configuration file."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"The HA failover time can be calculated in the following way:"},{"line_number":27,"context_line":"  failover time \u003d bfd_min_rx * bfd_mult (\u003d\u003d 3 seconds by default)"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Closes-Bug: #2142096"},{"line_number":30,"context_line":"Signed-off-by: Rodolfo Alonso Hernandez \u003cralonsoh@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"8c894b25_41ac39ee","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":65},"in_reply_to":"0076cdb2_2e86c74b","updated":"2026-03-04 15:16:24.000000000","message":"1) I\u0027ll update the formula. You reference is correct.\n2) The traffic effect for the BFD packets is close to zero. I\u0027ll keep the default values unless we show a practical effect in the dataplane.","commit_id":"7d06035adeb3b5fcec149b226e24cf76e6e997ad"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"16a00c01a7ada24b25532d13e633d03bde0eefef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0e890c42_00664e0a","updated":"2026-02-27 14:14:43.000000000","message":"Add a documentation section","commit_id":"844d9de9db8d28ee5e03085d9b800b4899fe7be5"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c1b63717edf571174ede55dbcd8cd5868fb78646","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"067efe80_fa40d910","updated":"2026-03-03 10:16:30.000000000","message":"recheck neutron-functional","commit_id":"7ff2a9325614648c1dd9595077f67152e7e459f4"},{"author":{"_account_id":23804,"name":"Daniel Alvarez","email":"dalvarez@redhat.com","username":"dalvarez"},"change_message_id":"d75ab6c0501f3ebe52596bf0fa963dbd53de5297","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7f558ba2_a118eb31","updated":"2026-03-04 15:01:05.000000000","message":"LGTM but I wonder if we should change the ``bfd-min-tx`` parameter for every profile. Keeping it at 100ms is effectively fixing the tx freq rate at 100ms regardless of the detection time.\n\nIn the aggresive profile perhaps we could have 500/500/3 which is still 1.5s \n\nAssuming the detection time is:\nmax(bfd-min-tx, bfd-max-tx)*bfd-mult","commit_id":"7d06035adeb3b5fcec149b226e24cf76e6e997ad"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"caebdc80630716d444a65a1b9cb5a2d903b69132","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fbeda65c_c4b533a1","updated":"2026-03-04 14:11:45.000000000","message":"recheck openstack-tox-cover","commit_id":"7d06035adeb3b5fcec149b226e24cf76e6e997ad"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1280c61926aa4a1d7b1fd8df83762304b153c940","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"cc3d63a3_4815d78b","updated":"2026-03-06 08:09:07.000000000","message":"as discussed on the drivers meeting:\nhttps://meetings.opendev.org/meetings/neutron_drivers/2026/neutron_drivers.2026-02-27-14.01.log.html#l-9","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7e04de767635eadea499cdc553bebb6aef36976c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"887f98aa_4a446cbc","updated":"2026-03-09 06:52:14.000000000","message":"recheck cover","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"aa792b418f61ab3e81a8ccd3f47e671ddd7c1fdb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4a9e0395_ad737527","updated":"2026-03-09 10:13:35.000000000","message":"recheck neutron-tempest-plugin-openvswitch","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"35b8eb0a65926094adf5712a569cd42c42a4d3c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f8d42bd9_0b923f42","updated":"2026-03-05 08:30:45.000000000","message":"recheck neutron-tempest-plugin-ovn","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6970934ea362059d26610774d4cdac2eec137a2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"27445ecb_8060d668","updated":"2026-03-06 14:10:31.000000000","message":"recheck tox cap","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1ae725fdf72f894a3421d1b3f3486e374d8430a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fb5b5520_07b4f816","updated":"2026-03-06 10:50:39.000000000","message":"recheck zuul jobs WB merged https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/979118","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"}],"doc/source/admin/ovn/routing.rst":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d7eb19b3881f31ffbcb207cb2f0db81e22c0094c","unresolved":true,"context_lines":[{"line_number":105,"context_line":"The gateway nodes monitor each other in star topology. Compute nodes don\u0027t"},{"line_number":106,"context_line":"monitor each other because that\u0027s not necessary."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Neutron exposes, via configuration file, a set of options to configure the OVS"},{"line_number":109,"context_line":"BFD options. That allows the administrator to configure the HA failover timeout"},{"line_number":110,"context_line":"that is defined as:"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"9f9a9877_c3c8b7e2","line":108,"range":{"start_line":108,"start_character":75,"end_line":108,"end_character":78},"updated":"2026-03-04 16:54:06.000000000","message":"s/OVN ?","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f1b1ec5572f5eba58932c46b9744de0eab00447","unresolved":true,"context_lines":[{"line_number":105,"context_line":"The gateway nodes monitor each other in star topology. Compute nodes don\u0027t"},{"line_number":106,"context_line":"monitor each other because that\u0027s not necessary."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Neutron exposes, via configuration file, a set of options to configure the OVS"},{"line_number":109,"context_line":"BFD options. That allows the administrator to configure the HA failover timeout"},{"line_number":110,"context_line":"that is defined as:"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"516adbbf_54efde87","line":108,"range":{"start_line":108,"start_character":75,"end_line":108,"end_character":78},"in_reply_to":"899701cc_1c17b218","updated":"2026-03-05 15:14:01.000000000","message":"Yes, understood, just all the other comments and config option text says \"OVN BFD\"","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6286c44631d8d06e323e248dcf43d6d827c0b3f9","unresolved":false,"context_lines":[{"line_number":105,"context_line":"The gateway nodes monitor each other in star topology. Compute nodes don\u0027t"},{"line_number":106,"context_line":"monitor each other because that\u0027s not necessary."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Neutron exposes, via configuration file, a set of options to configure the OVS"},{"line_number":109,"context_line":"BFD options. That allows the administrator to configure the HA failover timeout"},{"line_number":110,"context_line":"that is defined as:"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"899701cc_1c17b218","line":108,"range":{"start_line":108,"start_character":75,"end_line":108,"end_character":78},"in_reply_to":"9f9a9877_c3c8b7e2","updated":"2026-03-05 08:30:38.000000000","message":"These are actually Open vSwitch parameters, set by OVN. OVS is in charge of monitoring the interfaces [1]\n\n[1]https://github.com/openvswitch/ovs/blob/b9c66cf106f6730e548d04678fdf638172172d09/lib/bfd.c","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"}],"neutron/conf/plugins/ml2/drivers/ovn/ovn_conf.py":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"aa7e4ab4434308fe491e80295351d5d6798bc231","unresolved":true,"context_lines":[{"line_number":256,"context_line":"                                   \u0027compatibility during the development of \u0027"},{"line_number":257,"context_line":"                                   \u0027the functionality in Nova (LP#2069718\u0027),"},{"line_number":258,"context_line":"                ),"},{"line_number":259,"context_line":"    cfg.StrOpt(\u0027ha_failover_strategy\u0027,"},{"line_number":260,"context_line":"               default\u003dovn_const.OVN_HA_FAILOVER_NORMAL,"},{"line_number":261,"context_line":"               choices\u003dovn_const.OVN_HA_FAILOVER_CHOICES,"},{"line_number":262,"context_line":"               help\u003d_(\u0027HA chassis group failover strategy. Determines how \u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"1927d24c_02c49fa4","line":259,"updated":"2026-03-16 11:15:41.000000000","message":"shouldn\u0027t we keep it unmanaged by neutron by default?\nFor instance suppose these configs were set pre upgrade manually or via some installer, but due to this change it will be overridden bypassing any tuned configuration in the environment.\n\nLong back Jakub also had revert https://review.opendev.org/q/Ia1dc8072ecec1c019dd02039dadd78d544dbd843, this patch i think not purely falls into same category as config only in maintenance task? but we should consider outside of neutron config atalest","commit_id":"78bf1fcc2995629f0a28a19f2b7096efc3059e3f"}]}