From bb2aee1e58c1cd30087b935798060e6bf3c698c8 Mon Sep 17 00:00:00 2001 From: Viacheslav Hletenko Date: Fri, 20 Mar 2026 18:41:24 +0200 Subject: T8410: Fix typos and mistakes for operational and configuration commands Fix typos and mistakes in the commands and comments No functional changes --- data/templates/firewall/sysctl-firewall.conf.j2 | 2 +- data/templates/igmp-proxy/igmpproxy.conf.j2 | 2 +- data/templates/login/tacplus_servers.j2 | 2 +- data/templates/ndppd/ndppd.conf.j2 | 2 +- data/templates/sflow/hsflowd.conf.j2 | 2 +- debian/changelog | 2 +- interface-definitions/container.xml.in | 2 +- .../accel-ppp/max-concurrent-sessions.xml.i | 2 +- .../include/bgp/neighbor-local-role.xml.i | 2 +- .../include/firewall/global-options.xml.i | 4 ++-- .../include/firewall/match-ether-type.xml.i | 2 +- .../include/interface/arp-cache-timeout.xml.i | 2 +- .../interface/ipv6-dup-addr-detect-transmits.xml.i | 2 +- interface-definitions/include/nat64/protocol.xml.i | 2 +- .../include/ospf/default-information.xml.i | 2 +- .../include/stunnel/protocol-value-cifs.xml.i | 2 +- interface-definitions/include/vrrp/garp.xml.i | 4 ++-- interface-definitions/interfaces_openvpn.xml.in | 6 ++--- interface-definitions/interfaces_tunnel.xml.in | 2 +- .../interfaces_virtual-ethernet.xml.in | 2 +- .../load-balancing_haproxy.xml.in | 2 +- interface-definitions/policy.xml.in | 2 +- interface-definitions/protocols_igmp-proxy.xml.in | 2 +- interface-definitions/service_dns_dynamic.xml.in | 2 +- .../service_monitoring_telegraf.xml.in | 2 +- interface-definitions/service_router-advert.xml.in | 4 ++-- interface-definitions/system_option.xml.in | 2 +- .../system_static-host-mapping.xml.in | 2 +- interface-definitions/vpn_ipsec.xml.in | 6 ++--- interface-definitions/vpn_openconnect.xml.in | 4 ++-- .../generate-openvpn-config-client.xml.in | 2 +- op-mode-definitions/monitor-protocol.xml.in | 4 ++-- op-mode-definitions/show-hardware.xml.in | 2 +- python/vyos/accel_ppp_util.py | 4 ++-- python/vyos/config.py | 6 ++--- python/vyos/config_mgmt.py | 2 +- python/vyos/configdep.py | 2 +- python/vyos/configdict.py | 8 +++---- python/vyos/configdiff.py | 4 ++-- python/vyos/configsession.py | 2 +- python/vyos/configsource.py | 8 +++---- python/vyos/configtree.py | 4 ++-- python/vyos/configverify.py | 4 ++-- python/vyos/debug.py | 2 +- python/vyos/defaults.py | 2 +- python/vyos/ethtool.py | 8 +++---- python/vyos/frrender.py | 12 +++++----- python/vyos/ifconfig/bond.py | 14 ++++++------ python/vyos/ifconfig/bridge.py | 4 ++-- python/vyos/ifconfig/control.py | 6 ++--- python/vyos/ifconfig/ethernet.py | 8 +++---- python/vyos/ifconfig/interface.py | 18 +++++++-------- python/vyos/ifconfig/loopback.py | 4 ++-- python/vyos/ifconfig/tunnel.py | 2 +- python/vyos/ifconfig/vxlan.py | 4 ++-- python/vyos/ifconfig/wireguard.py | 4 ++-- python/vyos/ifconfig/wireless.py | 4 ++-- python/vyos/ipsec.py | 2 +- python/vyos/kea.py | 4 ++-- python/vyos/system/compat.py | 2 +- python/vyos/system/disk.py | 4 ++-- python/vyos/utils/assertion.py | 2 +- python/vyos/utils/auth.py | 2 +- python/vyos/utils/dict.py | 4 ++-- python/vyos/utils/network.py | 2 +- python/vyos/utils/process.py | 4 ++-- python/vyos/utils/system.py | 2 +- python/vyos/utils/vti_updown_db.py | 2 +- python/vyos/vpp/control_host.py | 2 +- python/vyos/vyconf_session.py | 2 +- python/vyos/xml_ref/definition.py | 2 +- python/vyos/xml_ref/generate_cache.py | 2 +- python/vyos/xml_ref/generate_op_cache.py | 4 ++-- schema/interface_definition.rnc | 2 +- schema/interface_definition.rng | 2 +- src/conf_mode/container.py | 4 ++-- src/conf_mode/interfaces_bonding.py | 4 ++-- src/conf_mode/interfaces_bridge.py | 2 +- src/conf_mode/interfaces_dummy.py | 2 +- src/conf_mode/interfaces_ethernet.py | 10 ++++----- src/conf_mode/interfaces_geneve.py | 2 +- src/conf_mode/interfaces_input.py | 2 +- src/conf_mode/interfaces_l2tpv3.py | 2 +- src/conf_mode/interfaces_loopback.py | 2 +- src/conf_mode/interfaces_macsec.py | 6 ++--- src/conf_mode/interfaces_openvpn.py | 4 ++-- src/conf_mode/interfaces_pppoe.py | 2 +- src/conf_mode/interfaces_pseudo-ethernet.py | 2 +- src/conf_mode/interfaces_sstpc.py | 2 +- src/conf_mode/interfaces_tunnel.py | 2 +- src/conf_mode/interfaces_virtual-ethernet.py | 4 ++-- src/conf_mode/interfaces_vti.py | 2 +- src/conf_mode/interfaces_vxlan.py | 8 +++---- src/conf_mode/interfaces_wireguard.py | 2 +- src/conf_mode/interfaces_wireless.py | 4 ++-- src/conf_mode/interfaces_wwan.py | 2 +- src/conf_mode/pki.py | 12 +++++----- src/conf_mode/policy.py | 2 +- src/conf_mode/policy_local-route.py | 4 ++-- src/conf_mode/protocols_bgp.py | 18 +++++++-------- src/conf_mode/protocols_eigrp.py | 2 +- src/conf_mode/protocols_isis.py | 2 +- src/conf_mode/protocols_nhrp.py | 2 +- src/conf_mode/protocols_ospf.py | 2 +- src/conf_mode/protocols_ospfv3.py | 2 +- src/conf_mode/protocols_rpki.py | 4 ++-- src/conf_mode/protocols_static.py | 4 ++-- src/conf_mode/qos.py | 2 +- src/conf_mode/service_console-server.py | 2 +- src/conf_mode/service_dhcp-server.py | 4 ++-- src/conf_mode/service_dhcpv6-server.py | 4 ++-- src/conf_mode/service_https.py | 6 ++--- src/conf_mode/service_mdns_repeater.py | 2 +- src/conf_mode/service_monitoring_network_event.py | 2 +- src/conf_mode/service_monitoring_telegraf.py | 4 ++-- src/conf_mode/service_ntp.py | 2 +- src/conf_mode/service_salt-minion.py | 2 +- src/conf_mode/service_snmp.py | 2 +- src/conf_mode/service_ssh.py | 4 ++-- src/conf_mode/service_webproxy.py | 2 +- src/conf_mode/system_conntrack.py | 2 +- src/conf_mode/system_console.py | 6 ++--- src/conf_mode/system_host-name.py | 2 +- src/conf_mode/system_login.py | 2 +- src/conf_mode/system_option.py | 2 +- src/conf_mode/vpn_ipsec.py | 2 +- src/conf_mode/vpp.py | 6 ++--- src/conf_mode/vrf.py | 6 ++--- src/etc/bash_completion.d/vyatta-op | 6 ++--- .../custom_scripts/vyos_services_input_filter.py | 2 +- src/etc/udev/rules.d/90-vyos-serial.rules | 2 +- src/helpers/config_dependency.py | 4 ++-- src/helpers/geoip-update.py | 4 ++-- src/helpers/vyos-failover.py | 2 +- src/init/vyos-router | 2 +- src/migration-scripts/bgp/5-to-6 | 2 +- src/migration-scripts/container/0-to-1 | 4 ++-- src/migration-scripts/dns-dynamic/2-to-3 | 2 +- src/migration-scripts/firewall/11-to-12 | 2 +- src/migration-scripts/firewall/14-to-15 | 2 +- src/migration-scripts/interfaces/1-to-2 | 4 ++-- src/migration-scripts/interfaces/5-to-6 | 2 +- src/migration-scripts/interfaces/7-to-8 | 2 +- src/migration-scripts/interfaces/8-to-9 | 2 +- src/migration-scripts/interfaces/9-to-10 | 4 ++-- src/migration-scripts/ipoe-server/1-to-2 | 2 +- src/migration-scripts/l2tp/5-to-6 | 2 +- src/migration-scripts/nat/6-to-7 | 2 +- src/migration-scripts/nat66/1-to-2 | 2 +- src/migration-scripts/nhrp/0-to-1 | 4 ++-- src/migration-scripts/openconnect/1-to-2 | 2 +- src/migration-scripts/policy/3-to-4 | 2 +- src/migration-scripts/quagga/11-to-12 | 2 +- src/migration-scripts/snmp/1-to-2 | 2 +- src/migration-scripts/snmp/2-to-3 | 2 +- src/migration-scripts/sstp/0-to-1 | 2 +- src/migration-scripts/system/22-to-23 | 2 +- src/migration-scripts/system/23-to-24 | 2 +- src/op_mode/bridge.py | 2 +- src/op_mode/container.py | 2 +- src/op_mode/firewall.py | 2 +- src/op_mode/flow_accounting_op.py | 2 +- src/op_mode/format_disk.py | 4 ++-- src/op_mode/generate_ovpn_client_file.py | 6 ++--- src/op_mode/generate_tech-support_archive.py | 8 +++---- src/op_mode/image_installer.py | 10 ++++----- src/op_mode/interfaces_wireless.py | 2 +- src/op_mode/ipoe-control.py | 2 +- src/op_mode/ipsec.py | 8 +++---- src/op_mode/maya_date.py | 4 ++-- src/op_mode/nat.py | 2 +- src/op_mode/ping.py | 2 +- src/op_mode/show_acceleration.py | 4 ++-- src/op_mode/show_openconnect_otp.py | 2 +- src/op_mode/snmp.py | 2 +- src/op_mode/snmp_ifmib.py | 2 +- src/op_mode/stp.py | 2 +- src/op_mode/traceroute.py | 2 +- src/op_mode/vpp.py | 2 +- src/op_mode/vrrp.py | 4 ++-- src/op_mode/wireguard_client.py | 2 +- .../vyatta-op/functions/interpreter/vyatta-op-run | 2 +- .../vyatta-op/functions/interpreter/vyatta-unpriv | 2 +- src/services/api/graphql/README.graphql | 2 +- .../api/graphql/generate/schema_from_composite.py | 2 +- .../graphql/generate/schema_from_config_session.py | 2 +- .../api/graphql/generate/schema_from_op_mode.py | 2 +- src/services/api/graphql/routers.py | 2 +- src/services/api/rest/models.py | 2 +- src/services/api/rest/routers.py | 26 +++++++++++----------- src/services/vyos-http-api-server | 2 +- 191 files changed, 327 insertions(+), 327 deletions(-) diff --git a/data/templates/firewall/sysctl-firewall.conf.j2 b/data/templates/firewall/sysctl-firewall.conf.j2 index 6c33ffdc8..52806d066 100644 --- a/data/templates/firewall/sysctl-firewall.conf.j2 +++ b/data/templates/firewall/sysctl-firewall.conf.j2 @@ -1,6 +1,6 @@ # Autogenerated by firewall.py -# gloabl options +# global options net.ipv4.icmp_echo_ignore_all = {{ 0 if global_options.all_ping == 'enable' else 1 }} net.ipv4.icmp_echo_ignore_broadcasts = {{ 0 if global_options.broadcast_ping == 'enable' else 1 }} net.ipv4.conf.all.bc_forwarding = {{ 1 if global_options.directed_broadcast == 'enable' else 0 }} diff --git a/data/templates/igmp-proxy/igmpproxy.conf.j2 b/data/templates/igmp-proxy/igmpproxy.conf.j2 index 85a04de7d..6e0d691ea 100644 --- a/data/templates/igmp-proxy/igmpproxy.conf.j2 +++ b/data/templates/igmp-proxy/igmpproxy.conf.j2 @@ -9,7 +9,7 @@ # option can be used in order to define legal multicast sources. # # The "quickleave" should be used to avoid saturation of the upstream link. The -# option should only be used if it's absolutely nessecary to accurately imitate +# option should only be used if it's absolutely necessary to accurately imitate # just one Client. # ######################################################## diff --git a/data/templates/login/tacplus_servers.j2 b/data/templates/login/tacplus_servers.j2 index 23e8e495e..c38ddc593 100644 --- a/data/templates/login/tacplus_servers.j2 +++ b/data/templates/login/tacplus_servers.j2 @@ -41,7 +41,7 @@ vrf={{ tacacs.vrf }} {% if tacacs.source_address is vyos_defined %} # Sets the IPv4 address used as the source IP address when communicating with # the TACACS+ server. IPv6 addresses are not supported, nor are hostnames. -# The address must work when passsed to the bind() system call, that is, it must +# The address must work when passed to the bind() system call, that is, it must # be valid for the interface being used. source_ip={{ tacacs.source_address }} {% endif %} diff --git a/data/templates/ndppd/ndppd.conf.j2 b/data/templates/ndppd/ndppd.conf.j2 index 6369dbdeb..fabe7feb8 100644 --- a/data/templates/ndppd/ndppd.conf.j2 +++ b/data/templates/ndppd/ndppd.conf.j2 @@ -10,7 +10,7 @@ route-ttl {{ route_refresh }} proxy {{ iface }} { # Turn on or off the router flag for Neighbor Advertisements router {{ 'yes' if iface_config.enable_router_bit is vyos_defined else 'no' }} - # Control how long to wait for a Neighbor Advertisment message before invalidating the entry (milliseconds) + # Control how long to wait for a Neighbor Advertisement message before invalidating the entry (milliseconds) timeout {{ iface_config.timeout }} # Control how long a valid or invalid entry remains in the cache (milliseconds) ttl {{ iface_config.ttl }} diff --git a/data/templates/sflow/hsflowd.conf.j2 b/data/templates/sflow/hsflowd.conf.j2 index fc251e92b..2bc249176 100644 --- a/data/templates/sflow/hsflowd.conf.j2 +++ b/data/templates/sflow/hsflowd.conf.j2 @@ -1,4 +1,4 @@ -# Genereated by /usr/libexec/vyos/conf_mode/system_sflow.py +# Generated by /usr/libexec/vyos/conf_mode/system_sflow.py # Parameters http://sflow.net/host-sflow-linux-config.php sflow { diff --git a/debian/changelog b/debian/changelog index 68aeecac3..c9fed1f82 100644 --- a/debian/changelog +++ b/debian/changelog @@ -4,7 +4,7 @@ vyos-1x (999.0) unstable; urgency=medium This is a internal VyOS package and the VyOS package process does not use the debian package changelog for its changes, please refer to the GitHub commitlog and the vyos release-notes for more details. - The correct verion number of this package is auto-generated by GIT + The correct version number of this package is auto-generated by GIT on build-time -- VyOS maintainers and contributors Sun, 10 Sep 2023 15:42:53 +0200 diff --git a/interface-definitions/container.xml.in b/interface-definitions/container.xml.in index f19beedb9..8a0da46a4 100644 --- a/interface-definitions/container.xml.in +++ b/interface-definitions/container.xml.in @@ -55,7 +55,7 @@ sys-admin - Administation operations (quotactl, mount, sethostname, setdomainame) + Administration operations (quotactl, mount, sethostname, setdomainname) sys-module diff --git a/interface-definitions/include/accel-ppp/max-concurrent-sessions.xml.i b/interface-definitions/include/accel-ppp/max-concurrent-sessions.xml.i index f6ef41019..20b40f60e 100644 --- a/interface-definitions/include/accel-ppp/max-concurrent-sessions.xml.i +++ b/interface-definitions/include/accel-ppp/max-concurrent-sessions.xml.i @@ -9,7 +9,7 @@ - Maximum concurent sessions must be in range 0-65535 + Maximum concurrent sessions must be in range 0-65535 diff --git a/interface-definitions/include/bgp/neighbor-local-role.xml.i b/interface-definitions/include/bgp/neighbor-local-role.xml.i index 6ddb4908f..472e5ab8b 100644 --- a/interface-definitions/include/bgp/neighbor-local-role.xml.i +++ b/interface-definitions/include/bgp/neighbor-local-role.xml.i @@ -33,7 +33,7 @@ - Neighbor must send this exact capability, otherwise a role missmatch notification will be sent + Neighbor must send this exact capability, otherwise a role mismatch notification will be sent diff --git a/interface-definitions/include/firewall/global-options.xml.i b/interface-definitions/include/firewall/global-options.xml.i index 7ec07100d..5ff5acfd8 100644 --- a/interface-definitions/include/firewall/global-options.xml.i +++ b/interface-definitions/include/firewall/global-options.xml.i @@ -62,7 +62,7 @@ arp - Adress Resolution Protocol (ARP) + Address Resolution Protocol (ARP) dhcp @@ -371,7 +371,7 @@ - RFC1337 TCP TIME-WAIT assasination hazards protection + RFC1337 TCP TIME-WAIT assassination hazards protection enable disable diff --git a/interface-definitions/include/firewall/match-ether-type.xml.i b/interface-definitions/include/firewall/match-ether-type.xml.i index abfa9034d..536e59bb3 100644 --- a/interface-definitions/include/firewall/match-ether-type.xml.i +++ b/interface-definitions/include/firewall/match-ether-type.xml.i @@ -15,7 +15,7 @@ arp - Adress Resolution Protocol + Address Resolution Protocol _ipv4 diff --git a/interface-definitions/include/interface/arp-cache-timeout.xml.i b/interface-definitions/include/interface/arp-cache-timeout.xml.i index 06d7ffe96..f3cbac24d 100644 --- a/interface-definitions/include/interface/arp-cache-timeout.xml.i +++ b/interface-definitions/include/interface/arp-cache-timeout.xml.i @@ -4,7 +4,7 @@ ARP cache entry timeout in seconds u32:1-86400 - ARP cache entry timout in seconds + ARP cache entry timeout in seconds diff --git a/interface-definitions/include/interface/ipv6-dup-addr-detect-transmits.xml.i b/interface-definitions/include/interface/ipv6-dup-addr-detect-transmits.xml.i index 3b9294dd0..82bf52876 100644 --- a/interface-definitions/include/interface/ipv6-dup-addr-detect-transmits.xml.i +++ b/interface-definitions/include/interface/ipv6-dup-addr-detect-transmits.xml.i @@ -4,7 +4,7 @@ Number of NS messages to send while performing DAD u32:0 - Disable Duplicate Address Dectection (DAD) + Disable Duplicate Address Detection (DAD) u32:1-n diff --git a/interface-definitions/include/nat64/protocol.xml.i b/interface-definitions/include/nat64/protocol.xml.i index a640873b5..3905f8190 100644 --- a/interface-definitions/include/nat64/protocol.xml.i +++ b/interface-definitions/include/nat64/protocol.xml.i @@ -1,7 +1,7 @@ - Apply translation address to a specfic protocol + Apply translation address to a specific protocol diff --git a/interface-definitions/include/ospf/default-information.xml.i b/interface-definitions/include/ospf/default-information.xml.i index 50cda54a4..dc44617f7 100644 --- a/interface-definitions/include/ospf/default-information.xml.i +++ b/interface-definitions/include/ospf/default-information.xml.i @@ -1,7 +1,7 @@ - Default route advertisment settings + Default route advertisement settings diff --git a/interface-definitions/include/stunnel/protocol-value-cifs.xml.i b/interface-definitions/include/stunnel/protocol-value-cifs.xml.i index 5b9484750..12eae6584 100644 --- a/interface-definitions/include/stunnel/protocol-value-cifs.xml.i +++ b/interface-definitions/include/stunnel/protocol-value-cifs.xml.i @@ -1,6 +1,6 @@ cifs - Proprietary (undocummented) extension of CIFS protocol + Proprietary (undocumented) extension of CIFS protocol diff --git a/interface-definitions/include/vrrp/garp.xml.i b/interface-definitions/include/vrrp/garp.xml.i index b56b490df..54bb34572 100644 --- a/interface-definitions/include/vrrp/garp.xml.i +++ b/interface-definitions/include/vrrp/garp.xml.i @@ -32,7 +32,7 @@ - Minimum time interval for refreshing gratuitous ARPs while beeing master + Minimum time interval for refreshing gratuitous ARPs while being master u32:0 No refresh @@ -49,7 +49,7 @@ - Number of gratuitous ARP messages to send at a time while beeing master + Number of gratuitous ARP messages to send at a time while being master u32:1-255 Number of gratuitous ARP messages diff --git a/interface-definitions/interfaces_openvpn.xml.in b/interface-definitions/interfaces_openvpn.xml.in index 9ce05db9c..005d7c015 100644 --- a/interface-definitions/interfaces_openvpn.xml.in +++ b/interface-definitions/interfaces_openvpn.xml.in @@ -374,11 +374,11 @@ _ipv4 - Accept connections on or initate connections to IPv4 addresses only + Accept connections on or initiate connections to IPv4 addresses only _ipv6 - Accept connections on or initate connections to IPv6 addresses only + Accept connections on or initiate connections to IPv6 addresses only dual-stack @@ -811,7 +811,7 @@ enable - Enable chalenge-response + Enable challenge-response (disable|enable) diff --git a/interface-definitions/interfaces_tunnel.xml.in b/interface-definitions/interfaces_tunnel.xml.in index fe1dad371..74849be5d 100644 --- a/interface-definitions/interfaces_tunnel.xml.in +++ b/interface-definitions/interfaces_tunnel.xml.in @@ -158,7 +158,7 @@ ERSPAN version 1 index field u32:0-63 - Platform-depedent field for specifying port number and direction + Platform-dependent field for specifying port number and direction diff --git a/interface-definitions/interfaces_virtual-ethernet.xml.in b/interface-definitions/interfaces_virtual-ethernet.xml.in index 2dfbd50b8..1559a6038 100644 --- a/interface-definitions/interfaces_virtual-ethernet.xml.in +++ b/interface-definitions/interfaces_virtual-ethernet.xml.in @@ -42,7 +42,7 @@ veth[0-9]+ - Virutal Ethernet interface must be named vethN + Virtual Ethernet interface must be named vethN diff --git a/interface-definitions/load-balancing_haproxy.xml.in b/interface-definitions/load-balancing_haproxy.xml.in index 163ffa869..f3baf86ca 100644 --- a/interface-definitions/load-balancing_haproxy.xml.in +++ b/interface-definitions/load-balancing_haproxy.xml.in @@ -295,7 +295,7 @@ - Global perfomance parameters and limits + Global performance parameters and limits #include diff --git a/interface-definitions/policy.xml.in b/interface-definitions/policy.xml.in index d2082fd7a..c93c55c03 100644 --- a/interface-definitions/policy.xml.in +++ b/interface-definitions/policy.xml.in @@ -1567,7 +1567,7 @@ - Border Gateway Protocl (BGP) origin code + Border Gateway Protocol (BGP) origin code igp egp incomplete diff --git a/interface-definitions/protocols_igmp-proxy.xml.in b/interface-definitions/protocols_igmp-proxy.xml.in index 5cde484f5..a1deeb3fc 100644 --- a/interface-definitions/protocols_igmp-proxy.xml.in +++ b/interface-definitions/protocols_igmp-proxy.xml.in @@ -26,7 +26,7 @@ - Unicast source networks allowed for multicast traffic to be proxyed + Unicast source networks allowed for multicast traffic to be proxied ipv4net IPv4 network diff --git a/interface-definitions/service_dns_dynamic.xml.in b/interface-definitions/service_dns_dynamic.xml.in index 75e5520b7..9f8f1a1b5 100644 --- a/interface-definitions/service_dns_dynamic.xml.in +++ b/interface-definitions/service_dns_dynamic.xml.in @@ -55,7 +55,7 @@ Pattern to skip from the HTTP(S) respose txt - Pattern to skip from the HTTP(S) respose to extract the external IP address + Pattern to skip from the HTTP(S) response to extract the external IP address diff --git a/interface-definitions/service_monitoring_telegraf.xml.in b/interface-definitions/service_monitoring_telegraf.xml.in index 2ac0d940e..1ea2a23dd 100644 --- a/interface-definitions/service_monitoring_telegraf.xml.in +++ b/interface-definitions/service_monitoring_telegraf.xml.in @@ -124,7 +124,7 @@ table-per-metric - One table per gorups of metric by the metric name + One table per groups of metric by the metric name (single-table|table-per-metric) diff --git a/interface-definitions/service_router-advert.xml.in b/interface-definitions/service_router-advert.xml.in index 878b53e78..31ed503d5 100644 --- a/interface-definitions/service_router-advert.xml.in +++ b/interface-definitions/service_router-advert.xml.in @@ -49,7 +49,7 @@ - Default router livetime bust be 0 or between 4 and 9000 + Default router lifetime must be 0 or between 4 and 9000 @@ -85,7 +85,7 @@ - Link MTU value placed in RAs, exluded in RAs if unset + Link MTU value placed in RAs, excluded in RAs if unset u32:1280-9000 Link MTU value in RAs diff --git a/interface-definitions/system_option.xml.in b/interface-definitions/system_option.xml.in index e6a2b0b65..4f9103e56 100644 --- a/interface-definitions/system_option.xml.in +++ b/interface-definitions/system_option.xml.in @@ -415,7 +415,7 @@ - Enable root partition auto-extention on system boot + Enable root partition auto-extension on system boot diff --git a/interface-definitions/system_static-host-mapping.xml.in b/interface-definitions/system_static-host-mapping.xml.in index 492741f11..2e72b97ec 100644 --- a/interface-definitions/system_static-host-mapping.xml.in +++ b/interface-definitions/system_static-host-mapping.xml.in @@ -23,7 +23,7 @@ .{1,63} - invalid alias hostname, needs to be between 1 and 63 charactes + invalid alias hostname, needs to be between 1 and 63 characters diff --git a/interface-definitions/vpn_ipsec.xml.in b/interface-definitions/vpn_ipsec.xml.in index ffdfeaedf..3e3bece61 100644 --- a/interface-definitions/vpn_ipsec.xml.in +++ b/interface-definitions/vpn_ipsec.xml.in @@ -1223,7 +1223,7 @@ - Defult ESP group name + Default ESP group name vpn ipsec esp-group @@ -1246,11 +1246,11 @@ yes - Enable remote host re-autentication during an IKE re-key. Currently broken due to a strong swan bug + Enable remote host re-authentication during an IKE re-key. Currently broken due to a strong swan bug no - Disable remote host re-authenticaton during an IKE re-key. + Disable remote host re-authentication during an IKE re-key. inherit diff --git a/interface-definitions/vpn_openconnect.xml.in b/interface-definitions/vpn_openconnect.xml.in index 13e5142fd..fc90e5865 100644 --- a/interface-definitions/vpn_openconnect.xml.in +++ b/interface-definitions/vpn_openconnect.xml.in @@ -65,7 +65,7 @@ - Use RADIUS server for user autentication + Use RADIUS server for user authentication @@ -247,7 +247,7 @@ #include - If the groupconfig option is set, then config-per-user will be overriden, and all configuration will be read from RADIUS. + If the groupconfig option is set, then config-per-user will be overridden, and all configuration will be read from RADIUS. diff --git a/op-mode-definitions/generate-openvpn-config-client.xml.in b/op-mode-definitions/generate-openvpn-config-client.xml.in index 351742f1e..e91b67933 100644 --- a/op-mode-definitions/generate-openvpn-config-client.xml.in +++ b/op-mode-definitions/generate-openvpn-config-client.xml.in @@ -30,7 +30,7 @@ - Cerificate used by client + Certificate used by client pki certificate diff --git a/op-mode-definitions/monitor-protocol.xml.in b/op-mode-definitions/monitor-protocol.xml.in index fec30c8a5..7c32ca335 100644 --- a/op-mode-definitions/monitor-protocol.xml.in +++ b/op-mode-definitions/monitor-protocol.xml.in @@ -701,7 +701,7 @@ - Disable OSPFv3 Link State Advertisments debugging + Disable OSPFv3 Link State Advertisements debugging vtysh -c "no debug ospf6 ${@:5}" @@ -956,7 +956,7 @@ - Enable OSPFv3 Link State Advertisments debugging + Enable OSPFv3 Link State Advertisements debugging vtysh -c "debug ospf6 ${@:5}" diff --git a/op-mode-definitions/show-hardware.xml.in b/op-mode-definitions/show-hardware.xml.in index f7803df9a..84ecd8559 100644 --- a/op-mode-definitions/show-hardware.xml.in +++ b/op-mode-definitions/show-hardware.xml.in @@ -9,7 +9,7 @@ - Show CPU informaion + Show CPU information ${vyos_op_scripts_dir}/cpu.py show diff --git a/python/vyos/accel_ppp_util.py b/python/vyos/accel_ppp_util.py index 85e8a964c..844bc56e5 100644 --- a/python/vyos/accel_ppp_util.py +++ b/python/vyos/accel_ppp_util.py @@ -130,7 +130,7 @@ def verify_accel_ppp_authentication(config, local_users=True): Common helper function which must be used by all Accel-PPP services based on get_config_dict() """ - # vertify auth settings + # verify auth settings if local_users and dict_search("authentication.mode", config) == "local": if ( dict_search("authentication.local_users", config) is None @@ -173,7 +173,7 @@ def verify_accel_ppp_authentication(config, local_users=True): user_config = config["authentication"]["interface"][interface] if "mac" not in user_config: raise ConfigError( - f'Users MAC addreses are not configured for interface "{interface}"') + f'Users MAC addresses are not configured for interface "{interface}"') if dict_search('authentication.radius.dynamic_author.server', config): if not dict_search('authentication.radius.dynamic_author.key', config): diff --git a/python/vyos/config.py b/python/vyos/config.py index 0476d7945..827246d05 100644 --- a/python/vyos/config.py +++ b/python/vyos/config.py @@ -53,7 +53,7 @@ VyOS has two distinct modes: operational mode and configuration mode. When a use the CLI is in the operational mode. In this mode, only the running (effective) config is accessible for reading. When a user enters the "configure" command, a configuration session is setup. Every config session -has its *proposed* (or *session*) config built on top of the current running config. When changes are commited, if commit succeeds, +has its *proposed* (or *session*) config built on top of the current running config. When changes are committed, if commit succeeds, the proposed config is merged into the running config. In configuration mode, "base" functions like `exists`, `return_value` return values from the session config, @@ -259,7 +259,7 @@ class Config(object): def session_changed(self): """ Returns: - True if the config session has uncommited changes, False otherwise. + True if the config session has uncommitted changes, False otherwise. """ return self._config_source.session_changed() @@ -544,7 +544,7 @@ class Config(object): Note: This function is safe to use in operational mode. In configuration mode, - it ignores uncommited changes. + it ignores uncommitted changes. """ if self._running_config is None: return False diff --git a/python/vyos/config_mgmt.py b/python/vyos/config_mgmt.py index ec29f82b7..e1e38b26f 100644 --- a/python/vyos/config_mgmt.py +++ b/python/vyos/config_mgmt.py @@ -841,7 +841,7 @@ def run(): rollback = subparsers.add_parser('rollback', help='Rollback to earlier config') rollback.add_argument('--rev', type=int, help='Revision number for rollback') rollback.add_argument( - '-y', dest='no_prompt', action='store_true', help='Excute without prompt' + '-y', dest='no_prompt', action='store_true', help='Execute without prompt' ) rollback_soft = subparsers.add_parser( diff --git a/python/vyos/configdep.py b/python/vyos/configdep.py index 04de66493..f1dd7895a 100644 --- a/python/vyos/configdep.py +++ b/python/vyos/configdep.py @@ -186,7 +186,7 @@ def graph_from_dependency_dict(d: dict) -> dict: for k in list(d): g[k] = set() # add the dependencies for every sub-case; should there be cases - # that are mutally exclusive in the future, the graphs will be + # that are mutually exclusive in the future, the graphs will be # distinguished for el in list(d[k]): g[k] |= set(d[k][el]) diff --git a/python/vyos/configdict.py b/python/vyos/configdict.py index b16c3cf1d..0603b42b8 100644 --- a/python/vyos/configdict.py +++ b/python/vyos/configdict.py @@ -430,7 +430,7 @@ def get_interface_dict(config, base, ifname='', recursive_defaults=True, with_pk """ Common utility function to retrieve and mangle the interfaces configuration from the CLI input nodes. All interfaces have a common base where value - retrival is identical. This function must be used whenever possible when + retrieval is identical. This function must be used whenever possible when working on the interfaces node! Return a dictionary with the necessary interface config keys. @@ -489,14 +489,14 @@ def get_interface_dict(config, base, ifname='', recursive_defaults=True, with_pk bond = is_member(config, ifname, 'bonding') if bond: dict.update({'is_bond_member' : bond}) - # Check if any DHCP options changed which require a client restat + # Check if any DHCP options changed which require a client restart dhcp = is_node_changed(config, base + [ifname, 'dhcp-options']) if dhcp: dict.update({'dhcp_options_changed' : {}}) dhcpv6 = is_node_changed(config, base + [ifname, 'dhcpv6-options']) if dhcpv6: dict.update({'dhcpv6_options_changed' : {}}) # Some interfaces come with a source_interface which must also not be part - # of any other bond or bridge interface as it is exclusivly assigned as the + # of any other bond or bridge interface as it is exclusively assigned as the # Kernels "lower" interface to this new "virtual/upper" interface. if 'source_interface' in dict: # Check if source interface is member of another bridge @@ -666,7 +666,7 @@ def get_accel_dict(config, base, chap_secrets, with_pki=False): """ Common utility function to retrieve and mangle the Accel-PPP configuration from different CLI input nodes. All Accel-PPP services have a common base - where value retrival is identical. This function must be used whenever + where value retrieval is identical. This function must be used whenever possible when working with Accel-PPP services! Return a dictionary with the necessary interface config keys. diff --git a/python/vyos/configdiff.py b/python/vyos/configdiff.py index 5e21a16e5..b7c3ef7c6 100644 --- a/python/vyos/configdiff.py +++ b/python/vyos/configdiff.py @@ -263,7 +263,7 @@ class ConfigDiff(object): to provide full dict; for example, Diff.MERGE will expand dict['merge'] into dict under value - no_detaults=False: if expand_nodes & Diff.MERGE, do not merge default + no_defaults=False: if expand_nodes & Diff.MERGE, do not merge default values to ret['merge'] recursive: if true, use config_tree diff algorithm provided by diff_tree class @@ -343,7 +343,7 @@ class ConfigDiff(object): to provide full dict; for example, Diff.MERGE will expand dict['merge'] into dict under value - no_detaults=False: if expand_nodes & Diff.MERGE, do not merge default + no_defaults=False: if expand_nodes & Diff.MERGE, do not merge default values to ret['merge'] recursive: if true, use config_tree diff algorithm provided by diff_tree class diff --git a/python/vyos/configsession.py b/python/vyos/configsession.py index 62ce34b22..8fee8eca1 100644 --- a/python/vyos/configsession.py +++ b/python/vyos/configsession.py @@ -219,7 +219,7 @@ class ConfigSession(object): ) if output: print( - 'cli-shell-api teardownSession output for sesion {0}: {1}'.format( + 'cli-shell-api teardownSession output for session {0}: {1}'.format( self.__session_id, output ), file=sys.stderr, diff --git a/python/vyos/configsource.py b/python/vyos/configsource.py index 33852a182..b54f6283a 100644 --- a/python/vyos/configsource.py +++ b/python/vyos/configsource.py @@ -51,7 +51,7 @@ class ConfigSource: def session_changed(self): """ Returns: - True if the config session has uncommited changes, False otherwise. + True if the config session has uncommitted changes, False otherwise. """ raise NotImplementedError(f"function not available for {type(self)}") @@ -198,7 +198,7 @@ class ConfigSourceSession(ConfigSource): def session_changed(self): """ Returns: - True if the config session has uncommited changes, False otherwise. + True if the config session has uncommitted changes, False otherwise. """ try: self._run(self._make_command('sessionChanged', '')) @@ -245,7 +245,7 @@ class ConfigSourceSession(ConfigSource): # FIXUP: by default, showConfig will give you a diff # if there are uncommitted changes. # The config parser obviously cannot work with diffs, - # so we need to supress diff production using appropriate + # so we need to suppress diff production using appropriate # options for getting either running (active) # or proposed (working) config. if effective: @@ -364,7 +364,7 @@ class ConfigSourceVyconfSession(ConfigSource): def session_changed(self): """ Returns: - True if the config session has uncommited changes, False otherwise. + True if the config session has uncommitted changes, False otherwise. """ try: return self._vyconf_session.session_changed() diff --git a/python/vyos/configtree.py b/python/vyos/configtree.py index afe1d7fe4..61ef6428c 100644 --- a/python/vyos/configtree.py +++ b/python/vyos/configtree.py @@ -282,8 +282,8 @@ class ConfigTree(object): """Set new entry in VyOS configuration. path: configuration path e.g. 'system dns forwarding listen-address' value: value to be added to node, e.g. '172.18.254.201' - replace: True: current occurance will be replaced - False: new value will be appended to current occurances - use + replace: True: current occurrence will be replaced + False: new value will be appended to current occurrences - use this for adding values to a multi node """ diff --git a/python/vyos/configverify.py b/python/vyos/configverify.py index 6e2fd1618..00cde7cdf 100644 --- a/python/vyos/configverify.py +++ b/python/vyos/configverify.py @@ -33,7 +33,7 @@ dynamic_interface_pattern = r'(ppp|pppoe|sstpc|l2tp|ipoe)[0-9]+' def verify_mtu(config): """ Common helper function used by interface implementations to perform - recurring validation if the specified MTU can be used by the underlaying + recurring validation if the specified MTU can be used by the underlying hardware. """ from vyos.ifconfig import Interface @@ -456,7 +456,7 @@ def verify_common_route_maps(config): # XXX: This function is called in combination with a previous call to: # tmp = conf.get_config_dict(['policy']) - see protocols_ospf.py as example. # We should NOT call this with the key_mangling option as this would rename - # route-map hypens '-' to underscores '_' and one could no longer distinguish + # route-map hyphens '-' to underscores '_' and one could no longer distinguish # what should have been the "proper" route-map name, as foo-bar and foo_bar # are two entire different route-map instances! for route_map in ['route-map', 'route_map']: diff --git a/python/vyos/debug.py b/python/vyos/debug.py index 1a6924ef5..3b71693ab 100644 --- a/python/vyos/debug.py +++ b/python/vyos/debug.py @@ -64,7 +64,7 @@ def enabled(flag): - command: print command run with result Having the flag setup on the filesystem is required to have - debuging at boot time, however, setting the flag via environment + debugging at boot time, however, setting the flag via environment does not require a seek to the filesystem and is more efficient it can be done on the shell on via .bashrc for the user diff --git a/python/vyos/defaults.py b/python/vyos/defaults.py index c584ddd77..6aa93ebed 100644 --- a/python/vyos/defaults.py +++ b/python/vyos/defaults.py @@ -52,7 +52,7 @@ systemd_services = { } internal_ports = { - 'certbot_haproxy' : 65080, # Certbot running behing haproxy + 'certbot_haproxy' : 65080, # Certbot running behind haproxy } config_files = { diff --git a/python/vyos/ethtool.py b/python/vyos/ethtool.py index ea3ed0b7c..79db25324 100644 --- a/python/vyos/ethtool.py +++ b/python/vyos/ethtool.py @@ -34,9 +34,9 @@ _drivers_without_bonding_support = _drivers_without_mac_change + [] class Ethtool: """ - Class is used to retrive and cache information about an ethernet adapter + Class is used to retrieve and cache information about an ethernet adapter """ - # dictionary containing driver featurs, it will be populated on demand and + # dictionary containing driver features, it will be populated on demand and # the content will look like: # [{'esp-hw-offload': {'active': False, 'fixed': True, 'requested': False}, # 'esp-tx-csum-hw-offload': {'active': False, @@ -79,7 +79,7 @@ class Ethtool: if driver: self._driver_name = driver.group(1) - # Build a dictinary of supported link-speed and dupley settings. + # Build a dictionary of supported link-speed and dupley settings. # [ { # "ifname": "eth0", # "supported-ports": [ "TP" ], @@ -187,7 +187,7 @@ class Ethtool: def check_speed_duplex(self, speed, duplex): """ Check if the passed speed and duplex combination is supported by - the underlaying network adapter. """ + the underlying network adapter. """ if isinstance(speed, int): speed = str(speed) if speed != 'auto' and not speed.isdigit(): diff --git a/python/vyos/frrender.py b/python/vyos/frrender.py index 1aed31f96..ff69f8266 100644 --- a/python/vyos/frrender.py +++ b/python/vyos/frrender.py @@ -14,7 +14,7 @@ # along with this library. If not, see . """ -Helper class attached to vyos-configd to interfact between our CLI configuration +Helper class attached to vyos-configd to interact between our CLI configuration and FRR. Class will render one full FRR configuration and apply this via frr-reload.py, if the configuration has no errors. @@ -102,7 +102,7 @@ def get_frrender_dict(conf: Config, argv=None) -> dict: def dict_helper_ospf_defaults(ospf, path): # We have gathered the dict representation of the CLI, but there are default - # options which we need to update into the dictionary retrived. + # options which we need to update into the dictionary retrieved. default_values = conf.get_config_defaults(path, key_mangling=('-', '_'), get_first_key=True, recursive=True) @@ -140,7 +140,7 @@ def get_frrender_dict(conf: Config, argv=None) -> dict: def dict_helper_ospfv3_defaults(ospfv3, path): # We have gathered the dict representation of the CLI, but there are default - # options which we need to update into the dictionary retrived. + # options which we need to update into the dictionary retrieved. default_values = conf.get_config_defaults(path, key_mangling=('-', '_'), get_first_key=True, recursive=True) @@ -168,7 +168,7 @@ def get_frrender_dict(conf: Config, argv=None) -> dict: def dict_helper_pim_defaults(pim, path): # We have gathered the dict representation of the CLI, but there are default - # options which we need to update into the dictionary retrived. + # options which we need to update into the dictionary retrieved. default_values = conf.get_config_defaults(path, key_mangling=('-', '_'), get_first_key=True, recursive=True) @@ -510,7 +510,7 @@ def get_frrender_dict(conf: Config, argv=None) -> dict: bgp_vrf_path = ['vrf', 'name', vrf_name, 'protocols', 'bgp'] if 'bgp' in vrf_config.get('protocols', []): # We have gathered the dict representation of the CLI, but there are default - # options which we need to update into the dictionary retrived. + # options which we need to update into the dictionary retrieved. default_values = conf.get_config_defaults(bgp_vrf_path, key_mangling=('-', '_'), get_first_key=True, recursive=True) @@ -818,7 +818,7 @@ class FRRender: output += inline_helper(vrf_config['protocols']) - # remove any accidently added empty newline to not confuse FRR + # remove any accidentally added empty newline to not confuse FRR output = os.linesep.join([s for s in output.splitlines() if s]) if '!!' in output: diff --git a/python/vyos/ifconfig/bond.py b/python/vyos/ifconfig/bond.py index ea38b9c42..ace84c6f2 100644 --- a/python/vyos/ifconfig/bond.py +++ b/python/vyos/ifconfig/bond.py @@ -405,9 +405,9 @@ class BondIf(Interface): return self.set_interface('bond_system_mac', mac) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ # Use ref-counting function to place an interface into admin down state. @@ -435,7 +435,7 @@ class BondIf(Interface): # administratively down # # We can not move the upper "shutdown_required" code path here - as this - # would break initial bond creation and inital mode assignment during + # would break initial bond creation and initial mode assignment during # interface creation! if self.get_admin_state() == 'down': # Bonding policy/mode - default value, always present @@ -460,10 +460,10 @@ class BondIf(Interface): # result in the following exception: OSError: [Errno 22] Invalid argument. # # We remove ALL addresses prior to adding new ones, this will remove - # addresses manually added by the user too - but as we are limited to 16 adresses + # addresses manually added by the user too - but as we are limited to 16 addresses # from the kernel side this looks valid to me. We won't run into an error - # when a user added manual adresses which would result in having more - # then 16 adresses in total. + # when a user added manual addresses which would result in having more + # then 16 addresses in total. arp_tgt_addr = list(map(str, self.get_arp_ip_target().split())) for addr in arp_tgt_addr: self.set_arp_ip_target('-' + addr) @@ -503,7 +503,7 @@ class BondIf(Interface): self.set_mac(tmp_if.get_mac()) is_first = False - # Assign underlaying interface to logical bond + # Assign underlying interface to logical bond self.add_port(interface) bond_members.append(interface) diff --git a/python/vyos/ifconfig/bridge.py b/python/vyos/ifconfig/bridge.py index ba06e3757..6b5f49778 100644 --- a/python/vyos/ifconfig/bridge.py +++ b/python/vyos/ifconfig/bridge.py @@ -273,9 +273,9 @@ class BridgeIf(Interface): return self.set_interface('vlan_protocol', map[protocol]) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ # Set ageing time diff --git a/python/vyos/ifconfig/control.py b/python/vyos/ifconfig/control.py index 3cf3297b4..1ef42f981 100644 --- a/python/vyos/ifconfig/control.py +++ b/python/vyos/ifconfig/control.py @@ -33,13 +33,13 @@ class Control(Section): ifname = None def __init__(self, **kargs): - # some commands (such as operation comands - show interfaces, etc.) + # some commands (such as operation commands - show interfaces, etc.) # need to query the interface statistics. If the interface # code is used and the debugging is enabled, the screen output # will include both the command but also the debugging for that command - # to prevent this, debugging can be explicitely disabled + # to prevent this, debugging can be explicitly disabled - # if debug is not explicitely disabled the the config, enable it + # if debug is not explicitly disabled the the config, enable it self.debug = '' if kargs.get('debug', True) and debug.enabled('ifconfig'): self.debug = 'ifconfig' diff --git a/python/vyos/ifconfig/ethernet.py b/python/vyos/ifconfig/ethernet.py index fb29c018c..c70190e38 100644 --- a/python/vyos/ifconfig/ethernet.py +++ b/python/vyos/ifconfig/ethernet.py @@ -244,7 +244,7 @@ class EthernetIf(Interface): cmd += f' speed {speed} duplex {duplex} autoneg off' return self._cmd(cmd) except PermissionError: - # Some NICs do not tell that they don't suppport settings speed/duplex, + # Some NICs do not tell that they don't support settings speed/duplex, # but they do not actually support it either. # In that case it's probably better to ignore the error # than end up with a broken config. @@ -450,7 +450,7 @@ class EthernetIf(Interface): cmd = f'ethtool --set-ring {ifname} {rx_tx} {size}' output, code = self._popen(cmd) # ethtool error codes: - # 80 - value already setted + # 80 - value already set # 81 - does not possible to set value if code and code != 80: print(f'could not set "{rx_tx}" ring-buffer for {ifname}') @@ -550,9 +550,9 @@ class EthernetIf(Interface): self._cmd(f'/sbin/devlink dev eswitch set pci/{addr} mode legacy') def update(self, config): - """General helper function which works on a dictionary retrived by + """General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface.""" # disable ethernet flow control (pause frames) diff --git a/python/vyos/ifconfig/interface.py b/python/vyos/ifconfig/interface.py index c49f497fc..febd68f44 100644 --- a/python/vyos/ifconfig/interface.py +++ b/python/vyos/ifconfig/interface.py @@ -498,7 +498,7 @@ class Interface(Control): def get_mac(self): """ - Get current interface MAC (Media Access Contrl) address used. + Get current interface MAC (Media Access Control) address used. Example: >>> from vyos.ifconfig import Interface @@ -554,7 +554,7 @@ class Interface(Control): def set_mac(self, mac): """ - Set interface MAC (Media Access Contrl) address to given value. + Set interface MAC (Media Access Control) address to given value. Example: >>> from vyos.ifconfig import Interface @@ -572,7 +572,7 @@ class Interface(Control): self.set_interface('mac', mac) - # Turn an interface to the 'up' state if it was changed to 'down' by this fucntion + # Turn an interface to the 'up' state if it was changed to 'down' by this function if prev_state == 'up': self.set_admin_state('up') @@ -1528,7 +1528,7 @@ class Interface(Control): render(systemd_override_file, 'dhcp-client/override.conf.j2', self.config) render(dhclient_config_file, 'dhcp-client/ipv4.j2', self.config) - # Reload systemd unit definitons as some options are dynamically generated + # Reload systemd unit definitions as some options are dynamically generated self._cmd('systemctl daemon-reload') # When the DHCP client is restarted a brief outage will occur, as @@ -1543,7 +1543,7 @@ class Interface(Control): if is_systemd_service_active(systemd_service): self._cmd(f'systemctl stop {systemd_service}') - # Smoketests occationally fail if the lease is not removed from the Kernel fast enough: + # Smoketests occasionally fail if the lease is not removed from the Kernel fast enough: # AssertionError: 2 unexpectedly found in {17: [{'addr': '52:54:00:00:00:00', # 'broadcast': 'ff:ff:ff:ff:ff:ff'}], 2: [{'addr': '192.0.2.103', 'netmask': '255.255.255.0', # @@ -1587,7 +1587,7 @@ class Interface(Control): render(config_file, 'dhcp-client/ipv6.j2', config) render(script_file, 'dhcp-client/dhcp6c-script.j2', config, permission=0o755) - # Reload systemd unit definitons as some options are dynamically generated + # Reload systemd unit definitions as some options are dynamically generated self._cmd('systemctl daemon-reload') # We must ignore any return codes. This is required to enable @@ -1610,7 +1610,7 @@ class Interface(Control): # Please refer to the document for details # - https://man7.org/linux/man-pages/man8/tc.8.html # - https://man7.org/linux/man-pages/man8/tc-mirred.8.html - # Depening if we are the source or the target interface of the port + # Depending if we are the source or the target interface of the port # mirror we need to setup some variables. # Don't allow for netns yet @@ -1744,9 +1744,9 @@ class Interface(Control): os.unlink(wpa_supplicant_conf) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ if self.debug: diff --git a/python/vyos/ifconfig/loopback.py b/python/vyos/ifconfig/loopback.py index f4fc2c906..e2cf69a15 100644 --- a/python/vyos/ifconfig/loopback.py +++ b/python/vyos/ifconfig/loopback.py @@ -56,9 +56,9 @@ class LoopbackIf(Interface): self.del_addr(addr) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ address = config.get('address', []) diff --git a/python/vyos/ifconfig/tunnel.py b/python/vyos/ifconfig/tunnel.py index f96364161..eb8fcdd1f 100644 --- a/python/vyos/ifconfig/tunnel.py +++ b/python/vyos/ifconfig/tunnel.py @@ -161,7 +161,7 @@ class TunnelIf(Interface): return self.set_interface('multicast', enable) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered interface setup code and provide a single point of entry when workin on any interface. """ diff --git a/python/vyos/ifconfig/vxlan.py b/python/vyos/ifconfig/vxlan.py index 0f55acf10..974a53566 100644 --- a/python/vyos/ifconfig/vxlan.py +++ b/python/vyos/ifconfig/vxlan.py @@ -190,9 +190,9 @@ class VXLANIf(Interface): self._cmd(f'bridge vni add dev {self.ifname} vni {vni}') def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ # call base class last diff --git a/python/vyos/ifconfig/wireguard.py b/python/vyos/ifconfig/wireguard.py index c4e70056c..fb576bac0 100644 --- a/python/vyos/ifconfig/wireguard.py +++ b/python/vyos/ifconfig/wireguard.py @@ -266,9 +266,9 @@ class WireGuardIf(Interface): return public_keys def update(self, config): - """General helper function which works on a dictionary retrived by + """General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface.""" # Wireguard base command is identical for every peer diff --git a/python/vyos/ifconfig/wireless.py b/python/vyos/ifconfig/wireless.py index 69fd87347..1026fcd57 100644 --- a/python/vyos/ifconfig/wireless.py +++ b/python/vyos/ifconfig/wireless.py @@ -42,9 +42,9 @@ class WiFiIf(Interface): self._cmd(cmd) def update(self, config): - """ General helper function which works on a dictionary retrived by + """ General helper function which works on a dictionary retrieved by get_config_dict(). It's main intention is to consolidate the scattered - interface setup code and provide a single point of entry when workin + interface setup code and provide a single point of entry when working on any interface. """ # We can not call add_to_bridge() until wpa_supplicant is running, thus diff --git a/python/vyos/ipsec.py b/python/vyos/ipsec.py index 81f3d0812..0d03f8cf9 100644 --- a/python/vyos/ipsec.py +++ b/python/vyos/ipsec.py @@ -65,7 +65,7 @@ def get_vici_sas_by_name(ike_name: str, tunnel: str) -> list: Find installed SAs by IKE_SA name and/or CHILD_SA name and return list with SASs info. If tunnel is not None return a list contained only - CHILD_SAs wich names equal tunnel value. + CHILD_SAs which names equal tunnel value. :param ike_name: IKE SA name :type ike_name: str :param tunnel: CHILD SA name diff --git a/python/vyos/kea.py b/python/vyos/kea.py index a8b38d999..c950a0d67 100644 --- a/python/vyos/kea.py +++ b/python/vyos/kea.py @@ -79,7 +79,7 @@ def _format_hex_string(in_str): def _find_list_of_dict_index(lst, key='ip', value=''): """ Find the index entry of list of dict matching the dict value - Exampe: + Example: % lst = [{'ip': '192.0.2.1'}, {'ip': '192.0.2.2'}] % _find_list_of_dict_index(lst, key='ip', value='192.0.2.2') % 1 @@ -658,7 +658,7 @@ def kea_get_server_leases(config, inet, vrf_name, pools=[], state=[], origin=Non now = datetime.now(timezone.utc) if lease['valid-lft'] > 0 and lease['expire_time'] > now: - # substraction gives us a timedelta object which can't be formatted + # subtraction gives us a timedelta object which can't be formatted # with strftime so we use str(), split gets rid of the microseconds data_lease['remaining'] = str(lease['expire_time'] - now).split('.')[0] diff --git a/python/vyos/system/compat.py b/python/vyos/system/compat.py index 23a34d38a..f4a50b237 100644 --- a/python/vyos/system/compat.py +++ b/python/vyos/system/compat.py @@ -179,7 +179,7 @@ def prune_vyos_versions(root_dir: str = '') -> None: def update_cfg_ver(root_dir:str = '') -> int: - """Get minumum version of image-tools across all installed images + """Get minimum version of image-tools across all installed images Args: root_dir (str): an optional path to the root directory diff --git a/python/vyos/system/disk.py b/python/vyos/system/disk.py index 268a3b195..7ae9a15bb 100644 --- a/python/vyos/system/disk.py +++ b/python/vyos/system/disk.py @@ -50,7 +50,7 @@ def find_persistence() -> str: """Find a mountpoint for persistence storage Returns: - str: Path where 'persistance' pertition is mounted, Empty if not found + str: Path where 'persistence' pertition is mounted, Empty if not found """ mounted_partitions = disk_partitions() for partition in mounted_partitions: @@ -77,7 +77,7 @@ def parttable_create(drive_path: str, root_size: int) -> None: -n3:0:+{root_size_text}K -t3:8300 {drive_path}' run(command) - # update partitons in kernel + # update partitions in kernel sync() run(f'partx -u {drive_path}') diff --git a/python/vyos/utils/assertion.py b/python/vyos/utils/assertion.py index aa0614743..35baa556b 100644 --- a/python/vyos/utils/assertion.py +++ b/python/vyos/utils/assertion.py @@ -57,7 +57,7 @@ def assert_mac(m, test_all_zero=True): split = m.split(':') size = len(split) - # a mac address consits out of 6 octets + # a mac address consists out of 6 octets if size != 6: raise ValueError(f'wrong number of MAC octets ({size}): {m}') diff --git a/python/vyos/utils/auth.py b/python/vyos/utils/auth.py index f9551419f..7123bd0a5 100644 --- a/python/vyos/utils/auth.py +++ b/python/vyos/utils/auth.py @@ -28,7 +28,7 @@ from vyos.utils.process import cmd # Minimum UID used when adding system users MIN_USER_UID: int = 1000 -# Maximim UID used when adding system users +# Maximum UID used when adding system users MAX_USER_UID: int = 59999 # List of local user accounts that must be preserved SYSTEM_USER_SKIP_LIST: frozenset = { diff --git a/python/vyos/utils/dict.py b/python/vyos/utils/dict.py index 5d41900b6..3fcda902f 100644 --- a/python/vyos/utils/dict.py +++ b/python/vyos/utils/dict.py @@ -187,7 +187,7 @@ def dict_search_args(dict_object, *path): return dict_object def dict_search_recursive(dict_object, key, path=[]): - """ Traverse a dictionary recurisvely and return the value of the key + """ Traverse a dictionary recursively and return the value of the key we are looking for. Thankfully copied from https://stackoverflow.com/a/19871956 @@ -380,7 +380,7 @@ def check_mutually_exclusive_options(d, keys, required=False): class FixedDict(dict): """ - FixedDict: A dictionnary not allowing new keys to be created after initialisation. + FixedDict: A dictionary not allowing new keys to be created after initialisation. >>> f = FixedDict(**{'count':1}) >>> f['count'] = 2 diff --git a/python/vyos/utils/network.py b/python/vyos/utils/network.py index de283e546..366433146 100644 --- a/python/vyos/utils/network.py +++ b/python/vyos/utils/network.py @@ -178,7 +178,7 @@ def get_interface_address(interface): def get_interface_namespace(interface: str): """ - Returns wich netns the interface belongs to + Returns which netns the interface belongs to """ # Bail out early if netns does not exist tmp = cmd(f'ip --json netns ls') diff --git a/python/vyos/utils/process.py b/python/vyos/utils/process.py index 74546a172..94d6670dc 100644 --- a/python/vyos/utils/process.py +++ b/python/vyos/utils/process.py @@ -56,7 +56,7 @@ def popen(command, flag='', shell=None, input=None, timeout=None, env=None, - STDOUT, send the data to be merged with stdout - DEVNULL, discard the output decode: specify the expected text encoding (utf-8, ascii, ...) - the default is explicitely utf-8 which is python's own default + the default is explicitly utf-8 which is python's own default vrf: run command in a VRF context netns: run command in the named network namespace buffered: define how process output shall be presented to stdout @@ -69,7 +69,7 @@ def popen(command, flag='', shell=None, input=None, timeout=None, env=None, """ # airbag must be left as an import in the function as otherwise we have a - # a circual import dependency + # a circular import dependency from vyos import debug from vyos import airbag diff --git a/python/vyos/utils/system.py b/python/vyos/utils/system.py index e2197daf2..578537544 100644 --- a/python/vyos/utils/system.py +++ b/python/vyos/utils/system.py @@ -80,7 +80,7 @@ def sysctl_apply(sysctl_dict: dict[str, str], revert: bool = True) -> bool: return True def find_device_file(device): - """ Recurively search /dev for the given device file and return its full path. + """ Recursively search /dev for the given device file and return its full path. If no device file was found 'None' is returned """ from fnmatch import fnmatch diff --git a/python/vyos/utils/vti_updown_db.py b/python/vyos/utils/vti_updown_db.py index 0587b48e8..2931df86c 100644 --- a/python/vyos/utils/vti_updown_db.py +++ b/python/vyos/utils/vti_updown_db.py @@ -111,7 +111,7 @@ class VTIUpDownDB: """ Removes a matching entry from the DB. - If no matching entry can be fonud, the operation returns successfully. + If no matching entry can be found, the operation returns successfully. """ ifspec = f"{interface}:{connection}:{protocol}" if (connection is not None and protocol is not None) else interface if ifspec in self._ifspecs: diff --git a/python/vyos/vpp/control_host.py b/python/vyos/vpp/control_host.py index 9808c9418..707f56740 100644 --- a/python/vyos/vpp/control_host.py +++ b/python/vyos/vpp/control_host.py @@ -420,7 +420,7 @@ def get_eth_mac(iface_name: str) -> str: def xdp_remove(iface_name: str) -> None: - """Remove XDP BPF program from an interfce + """Remove XDP BPF program from an interface Args: iface_name (str): name of an interface diff --git a/python/vyos/vyconf_session.py b/python/vyos/vyconf_session.py index 5748ab2dd..91e51338c 100644 --- a/python/vyos/vyconf_session.py +++ b/python/vyos/vyconf_session.py @@ -111,7 +111,7 @@ class VyconfSession: def exit_config_mode(self): if self.session_changed(): - return 'Uncommited changes', Errnum.UNCOMMITED_CHANGES + return 'Uncommitted changes', Errnum.UNCOMMITED_CHANGES out = vyconf_client.send_request('exit_configuration_mode', token=self.__token) return self.output(out), out.status diff --git a/python/vyos/xml_ref/definition.py b/python/vyos/xml_ref/definition.py index 015e7ee6e..cee762b02 100644 --- a/python/vyos/xml_ref/definition.py +++ b/python/vyos/xml_ref/definition.py @@ -104,7 +104,7 @@ class Xml: if self.exists(path): if self.is_valueless(path) or not self.is_leaf(path): # It's a complete path for a valueless node - # or a path to an empy non-leaf node + # or a path to an empty non-leaf node return (path, None) else: raise ValueError(f'Path "{path}" needs a value or children') diff --git a/python/vyos/xml_ref/generate_cache.py b/python/vyos/xml_ref/generate_cache.py index f0a3ec35b..daea50dc9 100755 --- a/python/vyos/xml_ref/generate_cache.py +++ b/python/vyos/xml_ref/generate_cache.py @@ -52,7 +52,7 @@ def non_trivial(s): return s def main(): - parser = ArgumentParser(description='generate and save dict from xml defintions') + parser = ArgumentParser(description='generate and save dict from xml definitions') parser.add_argument('--xml-dir', type=str, required=True, help='transcluded xml interface-definition directory') parser.add_argument('--internal-cache', type=str, required=True, diff --git a/python/vyos/xml_ref/generate_op_cache.py b/python/vyos/xml_ref/generate_op_cache.py index 1c183fc8c..ca939f550 100755 --- a/python/vyos/xml_ref/generate_op_cache.py +++ b/python/vyos/xml_ref/generate_op_cache.py @@ -89,7 +89,7 @@ def translate_command(s: str, pos: list[str]) -> str: s = translate_exec(s) s = translate_position(s, pos) - # If there are any untranslated occurences of '_place_holder_", + # If there are any untranslated occurrences of "_place_holder_", # it means the command is incorrect, # e.g., it references "$6" when it only has five words. if re.search(r'_place_holder_', s): @@ -317,7 +317,7 @@ def main(): # pylint: disable=global-statement global CHECK_XML_CONSISTENCY - parser = ArgumentParser(description='generate dict from xml defintions') + parser = ArgumentParser(description='generate dict from xml definitions') parser.add_argument( '--xml-dir', type=str, diff --git a/schema/interface_definition.rnc b/schema/interface_definition.rnc index bded5b0a6..0c929ad2c 100644 --- a/schema/interface_definition.rnc +++ b/schema/interface_definition.rnc @@ -187,7 +187,7 @@ valueHelp = element valueHelp # completionHelp tags contain information about allowed values of a node that is used for generating # tab completion in the CLI frontend and drop-down lists in GUI frontends -# It is only meaninful for leaf nodes +# It is only meaningful for leaf nodes # Allowed values can be given as a fixed list of values (e.g. foo bar baz), # as a configuration path (e.g. interfaces ethernet), # or as a path to a script file that generates the list (e.g. diff --git a/schema/interface_definition.rng b/schema/interface_definition.rng index cb9b46ecb..2c46266d3 100644 --- a/schema/interface_definition.rng +++ b/schema/interface_definition.rng @@ -340,7 +340,7 @@