)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"b750b5ea86927b19facd0d3ca3b970a3b065daa9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"01985262_41452baf","updated":"2024-12-04 20:17:33.000000000","message":"Will amend the release notes in - having a technical issue.","commit_id":"c31d131c0bcb8104d875d40851b9a5539b45c725"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"25b1a4ce139e746a81adf44155cbb0bfa12fd359","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2495d34d_18db52ef","updated":"2024-12-11 01:28:51.000000000","message":"recheck (timed out)","commit_id":"229717af772c25960156d6fcf1510bda69efdfcf"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"7e6359464f001b926cf54c5364266439756d4ffd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b0221e32_8379cefa","updated":"2025-01-10 09:02:29.000000000","message":"it would be great to have the pice of documentation with an configuration example","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"770fb0ac9508bfb7c63859c5f62ae04b1b39f46f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"78ec11c3_bd7ff99f","in_reply_to":"006534aa_5750e163","updated":"2025-01-28 11:07:41.000000000","message":"https://review.opendev.org/c/openstack/kolla-ansible/+/940295/1 I added this but it\u0027s in an other Merge requests sorry, kind of noob with git review and gerrit :/","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"e7477fc25a657a4d2a6d085217d108fd5e167578","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"006534aa_5750e163","in_reply_to":"445aa8c2_f7c54e59","updated":"2025-01-28 10:03:17.000000000","message":"Acknowledged","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"7739bf4c63e89e511540ed8450bce05cdff46847","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"445aa8c2_f7c54e59","in_reply_to":"6151c60f_16773a2a","updated":"2025-01-27 20:17:23.000000000","message":"there should be a page somewhere on the https://docs.openstack.org/kolla-ansible/ since there is no any info about BGP in networking. the only \u0027neutron-bgp-dragent\u0027 can be found in the code.","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"fd0f036a6ce9d9596ba6b687a34f57cb8a463449","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b645c354_d17daaec","in_reply_to":"6478c0ef_47c55b33","updated":"2025-01-28 20:22:22.000000000","message":"Both changes were merged. Please abandon the second change and continue with the next patch sets in this review.","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"92ad3f9eaf1da9a3061bc753f38582d58b658bbb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6478c0ef_47c55b33","in_reply_to":"78ec11c3_bd7ff99f","updated":"2025-01-28 20:16:52.000000000","message":"why the separate change?","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"fb07cdec158ca22c8367b852ec05095854316876","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ba970866_b245d624","in_reply_to":"b0221e32_8379cefa","updated":"2025-01-27 12:23:40.000000000","message":"Hi Maksim, I really need this feature, what kind of doc should it be ? do you have any example ?","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"f2f3073704d3b6d34596247b1a7e9dee69ac34a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f63da4cb_16e5f3e2","in_reply_to":"b645c354_d17daaec","updated":"2025-01-28 20:23:26.000000000","message":"thanks!!","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"f94b759b2ff3e84a299086ec1279336e99a5c4ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6151c60f_16773a2a","in_reply_to":"ba970866_b245d624","updated":"2025-01-27 12:27:43.000000000","message":"A doc for how to use it ? Is the globals.yaml not enough ? Maybe more details to the globals.yaml ! Is the doc of ovn-bgp-agent not enough ?","commit_id":"8dd77bc7853026f3376a859ac6fa1b61fc1ed28c"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"e7477fc25a657a4d2a6d085217d108fd5e167578","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8200dd8b_17e02250","updated":"2025-01-28 10:03:17.000000000","message":"I just did the rebase i\u0027ll try to write some doc","commit_id":"184d86252886887dd613320d54c6d03424e9f78e"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"1d86d9301344302493e3f0f11142f488d6ef6502","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"fd99d959_e56f5f94","updated":"2025-01-29 18:20:58.000000000","message":"We are going to need to add a note about FRR. Anytime the container restarts it will cause a service impact.\n\nI would suggest a disclaimer that says that.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":37512,"name":"Ivan Tkachuk","display_name":"Ivan Tkachuk","email":"ivan.tkachuk@fotbo.org","username":"Fstream"},"change_message_id":"cef0848fe5584ea517cd137651267195acfa0743","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"891313a8_01eb478f","updated":"2025-02-23 12:36:51.000000000","message":"frr containers failed to start if daemons file was not provided:\nERROR:__main__:MissingRequiredSource: /var/lib/kolla/config_files/daemons file is not found\nthere should be template for daemons file with default values. Or customize frr.json.j2 for the container so that daemons is added to config_files only when the daemons file is provided or custom daemons option enabled.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":37512,"name":"Ivan Tkachuk","display_name":"Ivan Tkachuk","email":"ivan.tkachuk@fotbo.org","username":"Fstream"},"change_message_id":"14e2feacef53061d1b574c25b358072f76d0cd76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a0f54ec6_587944d8","updated":"2025-02-24 08:31:25.000000000","message":"watch_log_file is deprecated and should be removed from config. \nDeprecated: Option \"watch_log_file\" from group \"DEFAULT\" is deprecated for removal (This function is known to have bene broken for long time, and depends on the unmaintained library).  Its value may be silently ignored in the future.\nTraceback (most recent call last):\n  File \"/var/lib/kolla/venv/bin/ovn-bgp-agent\", line 8, in \u003cmodule\u003e\n    sys.exit(start())\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/ovn_bgp_agent/agent.py\", line 78, in start\n    config.setup_logging()\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/ovn_bgp_agent/config.py\", line 265, in setup_logging\n    logging.setup(CONF, \u0027bgp-agent\u0027)\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/oslo_log/log.py\", line 296, in setup\n    _setup_logging_from_conf(conf, product_name, version)\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/oslo_log/log.py\", line 381, in _setup_logging_from_conf\n    raise RuntimeError(\u0027watch_log_file feature was removed because \u0027\nRuntimeError: watch_log_file feature was removed because it has been broken for multiple releases.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"270a8dfb3ce2f133970f2102762e4ff93a93d73a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"91885940_a1cc8b7f","in_reply_to":"49baf131_b676a76d","updated":"2025-04-29 22:04:14.000000000","message":"If FRR is configured to peer to the physical network, what can happen is that if FRR restarts, those peering sessions reset. The resulting action could be an outage.\n\nAnytime FRR containers are restarted this appears to occur.\n\nThere\u0027s nothing I see that can mitigate this. It may be an idea to consider pulling FRR out of this.\n\n@maksim.malchuk@gmail.com would be interested in feedback here. I think we can keep it in here and disclaim it in docs.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"f2fc63973d9ed51296b242e7fe32dcc2e82723e9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f582b6e9_adbc7b0a","in_reply_to":"891313a8_01eb478f","updated":"2025-04-30 17:22:12.000000000","message":"The default daemons file does not enable BGP. I will add a template in to reflect the conditions.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"27c679b6ff48672f9e6c9cfd6f1272919289fab1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"49baf131_b676a76d","in_reply_to":"b78dc4da_827f8d9b","updated":"2025-02-13 15:45:12.000000000","message":"@jayjahns@gmail.com ?","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"3af27e6bf6f175998df80a1cb8bd9d016b6d4114","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"770c3537_606d622d","in_reply_to":"f582b6e9_adbc7b0a","updated":"2025-04-30 18:02:13.000000000","message":"Done","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":37355,"name":"Pablo Colson","display_name":"Pablo Colson","email":"colson.pablo@gmail.com","username":"pabloclsn"},"change_message_id":"213ea7ed96cec4628326f45d53d30a98d7c187d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b78dc4da_827f8d9b","in_reply_to":"fd99d959_e56f5f94","updated":"2025-01-29 19:11:19.000000000","message":"Even if you have multiple network/compute node ?","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":290,"name":"Antony Messerli","email":"antony@mes.ser.li","username":"antonym"},"change_message_id":"dddcb680fae7406c6f8a44d1b5a0de9dc84021e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"0f52c5dc_0daa1252","updated":"2025-04-19 04:01:50.000000000","message":"recheck","commit_id":"26ebb6afe7c776dd8c17c0b7ddec3df786262334"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"9db5a6d11757bf50d940a54703d73000800339c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"65845793_bdd93d88","updated":"2025-05-07 19:14:14.000000000","message":"recheck (upgrade)","commit_id":"abd47860e9017c34134c0d9dcb0ff649bb2465e9"},{"author":{"_account_id":32343,"name":"mikhail","display_name":"Michael Kraynov","email":"kraynovmv@gmail.com","username":"kraynovmv"},"change_message_id":"210395953e5864e959009733bb3f6d66fff7f1e9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"c5296418_607a661a","updated":"2025-12-03 13:50:21.000000000","message":"@jayjahns@gmail.com Hello, can you please send the link where we can find information about native Neutron + OVN/BGP support?","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9dde29b7b8bda6f7e12dae06ef61f42cbb9e1fbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0d422cf1_4debdf09","updated":"2025-06-07 06:18:17.000000000","message":"I don’t believe we want to pursue that while OVN Native BGP approach is being worked on in Neutron and targeted for this or next cycle.","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":290,"name":"Antony Messerli","email":"antony@mes.ser.li","username":"antonym"},"change_message_id":"79cf75897883a1c9307bdad40d52a9f483ad73a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"57db88a0_4f70f0a9","updated":"2025-05-08 01:11:55.000000000","message":"recheck","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":36670,"name":"Jay Jahns","email":"jayjahns@gmail.com","username":"jayjahns"},"change_message_id":"97f3a83fafcf34cdfd1a3e56e5fda599a3926ec5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"5461f77b_db8c5fb1","updated":"2025-05-12 13:45:37.000000000","message":"recheck (dependent merge appears to be green)","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":36998,"name":"Ilia Baikov","display_name":"Ilia Baikov","email":"ilia.baikov@ib.systems","username":"frct1"},"change_message_id":"69109dbc1b48efe34e322c313d1f2612e2708017","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"714a14fd_aac670c0","in_reply_to":"0d422cf1_4debdf09","updated":"2025-06-07 12:45:48.000000000","message":"Could you please share a bit details on this, may be some links or something? I saw some recent OVN\u0027s patches for BGP but didn\u0027t see anything related in neutron.\nThank you","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":38875,"name":"Jake Groves","display_name":"jakegroves","email":"jake.groves@jagex.com","username":"jakegroves"},"change_message_id":"fa337900ef05e0ab7547dc70553a984d2dbe2166","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"f583b328_5f66fc84","in_reply_to":"c5296418_607a661a","updated":"2026-03-02 16:25:02.000000000","message":"\u003e @jayjahns@gmail.com Hello, can you please send the link where we can find information about native Neutron + OVN/BGP support?\nwe are also looking to use kolla for native Neutron + OVN with bgp agents, but can\u0027t seem to find documentation to do so","commit_id":"9f2fb617a6506632543c01098695a94342f36def"}],"ansible/group_vars/all.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6233a970769c1eaef0d0fb670527cef5614f97cd","unresolved":true,"context_lines":[{"line_number":277,"context_line":"####################"},{"line_number":278,"context_line":"network_interface: \"eth0\""},{"line_number":279,"context_line":"neutron_external_interface: \"eth1\""},{"line_number":280,"context_line":"neutron_no_external_interface: \"{{ enable_ovn_bgp_agent | bool }}\""},{"line_number":281,"context_line":"kolla_external_vip_interface: \"{{ network_interface }}\""},{"line_number":282,"context_line":"api_interface: \"{{ network_interface }}\""},{"line_number":283,"context_line":"swift_storage_interface: \"{{ network_interface }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"6cfc8dea_418a1d50","line":280,"updated":"2025-05-13 10:13:03.000000000","message":"Needs an explanation in commit message","commit_id":"184d86252886887dd613320d54c6d03424e9f78e"}],"ansible/roles/frr/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6233a970769c1eaef0d0fb670527cef5614f97cd","unresolved":true,"context_lines":[{"line_number":40,"context_line":"  - \"{{ node_config_directory }}/frr/:{{ container_config_directory }}:ro\""},{"line_number":41,"context_line":"  - \"/etc/localtime:/etc/localtime:ro\""},{"line_number":42,"context_line":"  - \"{{ \u0027/etc/timezone:/etc/timezone:ro\u0027 if ansible_facts.os_family \u003d\u003d \u0027Debian\u0027 else \u0027\u0027 }}\""},{"line_number":43,"context_line":"  - \"kolla_logs:/var/log/kolla/\""},{"line_number":44,"context_line":"  - \"/var/run/:/var/run/\""},{"line_number":45,"context_line":"  - \"/etc/iproute2/rt_tables:/etc/iproute2/rt_tables\""},{"line_number":46,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"60144237_1c7b475b","line":43,"updated":"2025-05-13 10:13:03.000000000","message":"What\u0027s the reason for whole /var/run? needs to go in commit message","commit_id":"184d86252886887dd613320d54c6d03424e9f78e"}],"ansible/roles/frr/handlers/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6233a970769c1eaef0d0fb670527cef5614f97cd","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    common_options: \"{{ docker_common_options }}\""},{"line_number":10,"context_line":"    name: \"{{ service.container_name }}\""},{"line_number":11,"context_line":"    image: \"{{ service.image }}\""},{"line_number":12,"context_line":"    privileged: \"{{ service.privileged | default(False) }}\""},{"line_number":13,"context_line":"    volumes: \"{{ service.volumes }}\""},{"line_number":14,"context_line":"    dimensions: \"{{ service.dimensions }}\""},{"line_number":15,"context_line":"    healthcheck: \"{{ service.healthcheck }}\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"e4b68bbe_6bad11b8","line":12,"updated":"2025-05-13 10:13:03.000000000","message":"nit: not needed","commit_id":"184d86252886887dd613320d54c6d03424e9f78e"}],"ansible/roles/frr/tasks/config.yml":[{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"98648f4aaad08b92aad73b2891e648d0393d90b0","unresolved":true,"context_lines":[{"line_number":68,"context_line":"    - files:"},{"line_number":69,"context_line":"        - \"{{ node_custom_config }}/frr/{{ inventory_hostname }}/daemons\""},{"line_number":70,"context_line":"        - \"{{ node_custom_config }}/frr/daemons\""},{"line_number":71,"context_line":"        - \"{{ role_path }}/frr/daemons.j2\""},{"line_number":72,"context_line":"      skip: true"},{"line_number":73,"context_line":"  when:"},{"line_number":74,"context_line":"    - inventory_hostname in groups[service.group]"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"b700a6d8_fd938776","line":71,"updated":"2025-06-06 20:24:48.000000000","message":"This is not working for me. Container fails to start with:\n\n```\nERROR MissingRequiredSource: /var/lib/kolla/config_files/daemons file is not found\n```\n\nThis should be \"{{ role_path }}/templates/daemons.j2\"","commit_id":"9f2fb617a6506632543c01098695a94342f36def"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"98648f4aaad08b92aad73b2891e648d0393d90b0","unresolved":true,"context_lines":[{"line_number":87,"context_line":"  with_first_found:"},{"line_number":88,"context_line":"    - files:"},{"line_number":89,"context_line":"        - \"{{ node_custom_config }}/frr/{{ inventory_hostname }}/frr.conf\""},{"line_number":90,"context_line":"        - \"{{ node_custom_config }}/frr/frr.conf\""},{"line_number":91,"context_line":"      skip: true"},{"line_number":92,"context_line":"  when:"},{"line_number":93,"context_line":"    - inventory_hostname in groups[service.group]"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"1b8ef555_ca45ee6a","line":90,"updated":"2025-06-06 20:24:48.000000000","message":"FRR container fails to start if this file is missing.\n\nIf the deployer is expected to provide its own, can an example or reference to documentation be provided?\n\nI would also expect a precheck to be created to avoid frr silently failing after a deployment if the deployer is expected to provide its own.","commit_id":"9f2fb617a6506632543c01098695a94342f36def"}],"ansible/roles/ovn-bgp-agent/templates/bgp-agent.conf.j2":[{"author":{"_account_id":37512,"name":"Ivan Tkachuk","display_name":"Ivan Tkachuk","email":"ivan.tkachuk@fotbo.org","username":"Fstream"},"change_message_id":"14e2feacef53061d1b574c25b358072f76d0cd76","unresolved":true,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug\u003d{{ ovn_bgp_agent_logging_debug }}"},{"line_number":3,"context_line":"log_file\u003d\"/var/log/kolla/openvswitch/ovn-bgp-agent.log\""},{"line_number":4,"context_line":"watch_log_file\u003dTrue"},{"line_number":5,"context_line":"ovsdb_connection\u003d\"unix:/var/run/openvswitch/db.sock\""},{"line_number":6,"context_line":"driver\u003d\"{{ ovn_bgp_agent_driver }}\""},{"line_number":7,"context_line":"exposing_method\u003d\"{{ ovn_bgp_agent_exposing_method }}\""}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"dca07dc4_41803698","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":14},"updated":"2025-02-24 08:31:25.000000000","message":"watch_log_file is deprecated and should be removed from config. \nDeprecated: Option \"watch_log_file\" from group \"DEFAULT\" is deprecated for removal (This function is known to have bene broken for long time, and depends on the unmaintained library).  Its value may be silently ignored in the future.\nTraceback (most recent call last):\n  File \"/var/lib/kolla/venv/bin/ovn-bgp-agent\", line 8, in \u003cmodule\u003e\n    sys.exit(start())\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/ovn_bgp_agent/agent.py\", line 78, in start\n    config.setup_logging()\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/ovn_bgp_agent/config.py\", line 265, in setup_logging\n    logging.setup(CONF, \u0027bgp-agent\u0027)\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/oslo_log/log.py\", line 296, in setup\n    _setup_logging_from_conf(conf, product_name, version)\n  File \"/var/lib/kolla/venv/lib64/python3.9/site-packages/oslo_log/log.py\", line 381, in _setup_logging_from_conf\n    raise RuntimeError(\u0027watch_log_file feature was removed because \u0027\nRuntimeError: watch_log_file feature was removed because it has been broken for multiple releases.","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":290,"name":"Antony Messerli","email":"antony@mes.ser.li","username":"antonym"},"change_message_id":"0818e26525396f1dcd22c9e40f864e79aec90619","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[DEFAULT]"},{"line_number":2,"context_line":"debug\u003d{{ ovn_bgp_agent_logging_debug }}"},{"line_number":3,"context_line":"log_file\u003d\"/var/log/kolla/openvswitch/ovn-bgp-agent.log\""},{"line_number":4,"context_line":"watch_log_file\u003dTrue"},{"line_number":5,"context_line":"ovsdb_connection\u003d\"unix:/var/run/openvswitch/db.sock\""},{"line_number":6,"context_line":"driver\u003d\"{{ ovn_bgp_agent_driver }}\""},{"line_number":7,"context_line":"exposing_method\u003d\"{{ ovn_bgp_agent_exposing_method }}\""}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"78348886_59050a00","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":14},"in_reply_to":"dca07dc4_41803698","updated":"2025-04-19 19:35:20.000000000","message":"Done","commit_id":"d4a9da0f8579f0df8a3772ef298427551570f40d"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"98648f4aaad08b92aad73b2891e648d0393d90b0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"ovsdb_connection\u003d\"unix:/var/run/openvswitch/db.sock\""},{"line_number":5,"context_line":"driver\u003d\"{{ ovn_bgp_agent_driver }}\""},{"line_number":6,"context_line":"exposing_method\u003d\"{{ ovn_bgp_agent_exposing_method }}\""},{"line_number":7,"context_line":"bgp_AS\u003d\"{{ bgp_local_asn }}\""},{"line_number":8,"context_line":"bgp_router_id\u003d\"{{ bgp_loopback_ip }}\""},{"line_number":9,"context_line":"clear_vrf_routes_on_startup\u003dTrue"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":15,"id":"67639f22_8e94ccec","line":7,"updated":"2025-06-06 20:24:48.000000000","message":"Those variables (bgp_local_asn and bgp_loopback_ip) are not defined and do not have default values. Ansible fails on deployment.\n\nIf those variables are deployment specific, I would expect a precheck enforcing their definition when ovn-bgp-agent is enabled. And maybe some kind of documentation in all.yml","commit_id":"9f2fb617a6506632543c01098695a94342f36def"}]}