)]}'
{"ovn_bgp_agent/privileged/vtysh.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"124e7f09c90be9107679ef14308a60a6d589090d","unresolved":true,"context_lines":[{"line_number":24,"context_line":"@ovn_bgp_agent.privileged.vtysh_cmd.entrypoint"},{"line_number":25,"context_line":"def run_vtysh_config(frr_config_file):"},{"line_number":26,"context_line":"    full_args \u003d [\u0027/usr/bin/vtysh\u0027, \u0027--vty_socket\u0027, constants.FRR_SOCKET_PATH,"},{"line_number":27,"context_line":"                 \u0027-f\u0027, frr_config_file]"},{"line_number":28,"context_line":"    try:"},{"line_number":29,"context_line":"        return processutils.execute(*full_args)"},{"line_number":30,"context_line":"    except Exception as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"d9731116_effafd91","line":27,"range":{"start_line":27,"start_character":17,"end_line":27,"end_character":39},"updated":"2024-01-18 10:08:25.000000000","message":"The only information I\u0027ve seen about \"-f\" parameter is to validate the configuration, not to apply it. Can you provide this information?","commit_id":"c9854d3b2aa3e94b395c12d1bbdf7db428a179d3"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"ecc956af206a66ebf7c8f16f9dc778debfa44870","unresolved":false,"context_lines":[{"line_number":24,"context_line":"@ovn_bgp_agent.privileged.vtysh_cmd.entrypoint"},{"line_number":25,"context_line":"def run_vtysh_config(frr_config_file):"},{"line_number":26,"context_line":"    full_args \u003d [\u0027/usr/bin/vtysh\u0027, \u0027--vty_socket\u0027, constants.FRR_SOCKET_PATH,"},{"line_number":27,"context_line":"                 \u0027-f\u0027, frr_config_file]"},{"line_number":28,"context_line":"    try:"},{"line_number":29,"context_line":"        return processutils.execute(*full_args)"},{"line_number":30,"context_line":"    except Exception as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"59c5faf0_bfcd3d56","line":27,"range":{"start_line":27,"start_character":17,"end_line":27,"end_character":39},"in_reply_to":"2894b286_1be3185c","updated":"2024-01-18 10:44:45.000000000","message":"And apparently it has also been included in the earliest github tag i could find.\n\nhttps://github.com/FRRouting/frr/blob/frr-2.0-rc1/vtysh/vtysh_main.c#144","commit_id":"c9854d3b2aa3e94b395c12d1bbdf7db428a179d3"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"b8d0f1547cc3a0a75a4b44580dfdabf5a2fe9105","unresolved":false,"context_lines":[{"line_number":24,"context_line":"@ovn_bgp_agent.privileged.vtysh_cmd.entrypoint"},{"line_number":25,"context_line":"def run_vtysh_config(frr_config_file):"},{"line_number":26,"context_line":"    full_args \u003d [\u0027/usr/bin/vtysh\u0027, \u0027--vty_socket\u0027, constants.FRR_SOCKET_PATH,"},{"line_number":27,"context_line":"                 \u0027-f\u0027, frr_config_file]"},{"line_number":28,"context_line":"    try:"},{"line_number":29,"context_line":"        return processutils.execute(*full_args)"},{"line_number":30,"context_line":"    except Exception as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"d84d3d63_9410d338","line":27,"range":{"start_line":27,"start_character":17,"end_line":27,"end_character":39},"in_reply_to":"59c5faf0_bfcd3d56","updated":"2024-01-18 10:53:49.000000000","message":"When trying to install frr on ubuntu 16.04 with frr-6 (v6.0.3) from their own deb repo (https://deb.frrouting.org/) it also comes with the -f option.\n\n```\nroot@8a33f96fe34c:/# vtysh --help\nUsage : vtysh [OPTION...]\n\nIntegrated shell for FRR.\n\n-b, --boot               Execute boot startup configuration\n-c, --command            Execute argument as command\n-d, --daemon             Connect only to the specified daemon\n-f, --inputfile          Execute commands from specific file and exit\n...\n```","commit_id":"c9854d3b2aa3e94b395c12d1bbdf7db428a179d3"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"ad693ecf194cdac946e442c50c8416eed4e930a7","unresolved":true,"context_lines":[{"line_number":24,"context_line":"@ovn_bgp_agent.privileged.vtysh_cmd.entrypoint"},{"line_number":25,"context_line":"def run_vtysh_config(frr_config_file):"},{"line_number":26,"context_line":"    full_args \u003d [\u0027/usr/bin/vtysh\u0027, \u0027--vty_socket\u0027, constants.FRR_SOCKET_PATH,"},{"line_number":27,"context_line":"                 \u0027-f\u0027, frr_config_file]"},{"line_number":28,"context_line":"    try:"},{"line_number":29,"context_line":"        return processutils.execute(*full_args)"},{"line_number":30,"context_line":"    except Exception as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"2894b286_1be3185c","line":27,"range":{"start_line":27,"start_character":17,"end_line":27,"end_character":39},"in_reply_to":"9391cd4e_f73c92c5","updated":"2024-01-18 10:33:04.000000000","message":"In the oldest docker image i could find (7.3.0) it is already present.\n\nApparently it has been backported into 7.2 https://github.com/FRRouting/frr/commit/0846286b096be2cf437a9b95eb4c484156806df3","commit_id":"c9854d3b2aa3e94b395c12d1bbdf7db428a179d3"},{"author":{"_account_id":25468,"name":"Michel Nederlof","email":"michel@nederlof.info","username":"pellucid"},"change_message_id":"ef94741b1225ec54332b1979d3f5ee6e3f7fdb95","unresolved":true,"context_lines":[{"line_number":24,"context_line":"@ovn_bgp_agent.privileged.vtysh_cmd.entrypoint"},{"line_number":25,"context_line":"def run_vtysh_config(frr_config_file):"},{"line_number":26,"context_line":"    full_args \u003d [\u0027/usr/bin/vtysh\u0027, \u0027--vty_socket\u0027, constants.FRR_SOCKET_PATH,"},{"line_number":27,"context_line":"                 \u0027-f\u0027, frr_config_file]"},{"line_number":28,"context_line":"    try:"},{"line_number":29,"context_line":"        return processutils.execute(*full_args)"},{"line_number":30,"context_line":"    except Exception as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9391cd4e_f73c92c5","line":27,"range":{"start_line":27,"start_character":17,"end_line":27,"end_character":39},"in_reply_to":"d9731116_effafd91","updated":"2024-01-18 10:14:25.000000000","message":"The option you are referring to is the `-C` i guess. But it could be that this is a v8+ feature.\n\nBelow is the `--help` output from the vtysh with v9\n\n```\nroot@my-host:~# docker exec -it ovn-bgp-agent vtysh  --help\nUsage : vtysh [OPTION...]\n\nIntegrated shell for FRR (version 9.0.1_git).\nConfigured with:\n    \u0027--prefix\u003d/usr\u0027 \u0027--sbindir\u003d/usr/lib/frr\u0027 \u0027--sysconfdir\u003d/etc/frr\u0027 \u0027--libdir\u003d/usr/lib\u0027 \u0027--localstatedir\u003d/var/run/frr\u0027 \u0027--enable-rpki\u0027 \u0027--enable-vtysh\u0027 \u0027--enable-multipath\u003d64\u0027 \u0027--enable-vty-group\u003dfrrvty\u0027 \u0027--enable-user\u003dfrr\u0027 \u0027--enable-group\u003dfrr\u0027 \u0027--enable-pcre2posix\u0027 \u0027--enable-scripting\u0027 \u0027CC\u003dgcc\u0027 \u0027CXX\u003dg++\u0027\n\n-b, --boot               Execute boot startup configuration\n-c, --command            Execute argument as command\n-d, --daemon             Connect only to the specified daemon\n-f, --inputfile          Execute commands from specific file and exit\n-E, --echo               Echo prompt and command in -c mode\n-C, --dryrun             Check configuration for validity and exit\n-m, --markfile           Mark input file with context end\n    --vty_socket         Override vty socket path\n    --config_dir         Override config directory path\n-N  --pathspace          Insert prefix into config \u0026 socket paths\n-u  --user               Run as an unprivileged user\n-w, --writeconfig        Write integrated config (frr.conf) and exit\n-H, --histfile           Override history file\n-t, --timestamp          Print a timestamp before going to shell or reading the configuration\n    --no-fork            Don\u0027t fork clients to handle daemons (slower for large configs)\n-h, --help               Display this help and exit\n\nNote that multiple commands may be executed from the command\nline by passing multiple -c args, or by embedding linefeed\ncharacters in one or more of the commands.\n\nReport bugs to https://github.com/frrouting/frr/issues\n```","commit_id":"c9854d3b2aa3e94b395c12d1bbdf7db428a179d3"}]}