nlcache: remove special SIGINT handling

capturing sigint was causing ifquery to hang indefinitely on ctrl-c

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

nlcache: merge existing MTU into new netlink object (vlan/vxlan) before caching

This patch fixes the gap in the vlan and vxlan code

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

ifupdownmain: skipping admin down on deleted vlans

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

ifupdownmain: down ops on vrf-slave should set the slave admin down

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: address: process hwaddress before processing ip addresses

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

lib: nlcache: merge existing MTU into new netlink object before caching

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: mstpctl: Merge if statement with the enclosing one

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

sonarlink: remove unused variable

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: set protodown off on bond slave before bond is deleted

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: addons: bond: Merge if statements with the enclosing ones

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: ignore exception raised during bond-slave speed check

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: make sure bond speed and slaves (swps) speed are matching

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: fix iface.py: Import only needed names or import the module and then use its members.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: allow bond creation without slaves

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: addressvirtual: print exception message in exception handler

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: addressvirtual: Merge if statement with the enclosing one

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: addressvirtual: Rename method "addr_metric_support"

Rename method "addr_metric_support" to prevent any misunderstanding/clash
with field "ADDR_METRIC_SUPPORT" defined on line 80

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

sonarlink: use opposite operator != instead of 'not'

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: addons: address: Merge if statement with the enclosing one

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

add support for systemd logging

New CLI option: --systemd
will enabled journalctl logging when ifupdown2 is run in a systemd context

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

SONAR: Specify an exception class to catch or reraise the exception

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

Merge pull request #256 from tomvil/feat/ethtool_set_ring_max_value

addons: Allow to set `max` value in `--set-ring` option

Merge pull request #258 from aderumier/vlanprotocol-query-check

vlan: query_check: check vlan-protocol for not dotted interface

vlan: query_check: check vlan-protocol for not dotted interface

/etc/network/interfaces
-----------------------
auto eth0.100
iface eth0.100
        vlan-protocol 802.1ad

current
-------
ifquery -a -c

auto eth0.100
iface eth0.100                                                      [pass]

new
---
ifquery -a -c

auto eth0.100
iface eth0.100                                                      [pass]
        vlan-protocol 802.1ad                                       [pass]

addons: Allow to set `max` value in `--set-ring` option

This makes some automation cases much easier

```
auto eth0
iface eth0 inet dhcp
  ring-rx max
  ring-tx max

Ring parameters for eth0:
Pre-set maximums:
RX: 4096
RX Mini: n/a
RX Jumbo: n/a
TX: 4096
Current hardware settings:
RX: 4096
RX Mini: n/a
RX Jumbo: n/a
TX: 4096
```

Signed-off-by: tomvil <tomas.vilemaitis@gmail.com>

Merge pull request #255 from tomvil/master

addons: Add `--set-ring` option to ethtool

remove extra line from ethtool.py

addons: Add `--set-ring` option to ethtool

example:
```
auto eth0
iface eth0 inet dhcp
  ring-tx 1024
  ring-rx 1024
```

Signed-off-by: Tomas Vilemaitis

scheduler: env variable not properly set for user commands

Fixes #218

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

Merge pull request #246 from aderumier/ovsfakebridge2

addons: openvswitch : add support for fakebridge.

addons: openvswitch : add support for fakebridge.

Fake bridge are are ovsbridge child of a parent bridge with a specific vlan tag.

```
auto br-parent
iface br-parent inet manual
        ovs_type OVSBridge

auto br-fakev10
iface br-fakev10 inet manual
        ovs_type OVSBridge
        ovs_bridge br-parent
        ovs_options br-parent 10

```

Merge pull request #242 from talber-nvidia/fix-empty-output-from-subprocess

Fix the return value of utils._execute_subprocess

Fix the return value of utils._execute_subprocess

If the output of the subprocess is empty, then an empty <bytes> object
is returned (b''), and not decoded into an empty <string> object.
This may result in errors such as:
    info[109734]: executing /bin/ip -6 addr show eth0
    networking[109734]: error: eth0: cannot use a string pattern on a
        bytes-like object
    error[109734]: eth0: cannot use a string pattern on a bytes-like object

Merge pull request #239 from sohorx/fix/distutils_dependency

distutils: fix dependency and lib replacement

Merge branch 'master' into fix/distutils_dependency

Merge pull request #238 from sohorx/fix/gvgen_dependency

gvgen: fix module dependency

distutils: fix dependency and lib replacement

As of now, the distutils is deprecated with removal planned for
python 3.12.

The recommended way is to use setuptools which is an enhanced
alternative to distutils maintained by the python packaging authority.

This commit replace with setuptools the distutil import inserted in the
8126ef0c00eb8e2faae2cf1ca9fbaf25c10582cd commit.

This commit also update the debian control runtime dependency.

distutils: https://docs.python.org/3/library/distutils.html
setuptools: https://setuptools.pypa.io/en/latest/

gvgen: fix module dependency

gvgen module require the six module from python3-six package.

This new dependency is only necessary for python2/python3 cross
compatibility on dict.iteritems() (py2) vs dict.items() (py3).

ifupdown2 does not support python2 so we could replace every iteritems
occurrences to it's python3 format but this means the gvgen module will
diff from upstream.

The simplest way is to make ifupdown depends on python3-six.

Merge pull request #237 from svenauhagen/bugfix/bondarpcheck

addons: bond: fix ifquery for IFLA_BOND_ARP_IP_TARGET

sonar: lib: nlcache: catch exception only once

Catch this exception only once;
it is already handled by a previous except clause

Ticket: #3035926

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

update contact info and copyright dates

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

graph: fix GvGen import

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

graph: import gvgen class into ifupdown2 package

python2's ifupdown2 used python-gvgen to output the dependency graph in dot
format. There's not python3 version of that package available in debian.
Resulting in: error: main exception: name 'GvGen' is not defined

The package is in fact a single class that is already compatible with python3
According to the license (MIT) we can pull in the code in our own tree.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: tunnel: support non-disruptive config change

Current design destroys existing tunnel when a config
change is detected. This behaviour causes traffic loss.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: skip bridge mac check and force mac-set on bridge creation

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: fix bridge/vxlan learning sync mechanism

Old code was a bit messy and all over the place. This
resulted in a loophole breaking sync between bridge-learning
and vxlan-learning. This patch simplifies the existing code
and fixes the bug.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: fix sync between bridge-learning and vxlan-learning

if not specified by the user vxlan brport learning is controlled by the
bridge_vxlan_port_learning policy (on by default).

4.4 introduced vxlan-learning off via policy. The syncing code between
bridge-learning and vxlan-learning was incomplete and was written at the
time when we didn't have a default vxlan-learning policy. This patch fixes
the sync-ing gap and makes sure vxlan-learning is sync'd with bridge-learning
which wasn't always the case before.

for every vxlan brport BRPORT_LEARNING is turned on

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: set IFLA_MASTER when updating bond settings

On bond creation and update, ifupdown2 directly caches the netlink
object sent to the kernel. If the bond already exists it overrides
the existing cached object. If the existing bond was enslaved to a
bridge, some bridge vlan data would get purged because the new
netlink object didn't have the IFLA_MASTER attribute (thus tricking
the cache into thinking that the bond got unslaved from the bridge).

Here is a snippet of the bond013 /en/i config in the first topology:

    auto bond013
    iface bond013
        bond-slaves swp1s3
        es-sys-mac 44:38:39:FF:00:02
        bridge-vids 101-144
        bond-lacp-bypass-allow yes
        mstpctl-portadminedge yes
        mstpctl-bpduguard yes

And here in the second topology:

    auto bond013
    iface bond013
          bond-slaves swp1s3
          es-sys-mac 44:38:39:FF:00:01
          bridge-vids 201-210
          bond-lacp-bypass-allow yes
          mstpctl-portadminedge yes
          mstpctl-bpduguard yes

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: set protodown off on unslaved evpn-mh bond slave

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

bond: support for protodown reason bit settings for clag/frr

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: fix ifquery for IFLA_BOND_ARP_IP_TARGET

adding correct support for multiple ip in bond-arp-ip-target

Signed-off-by: Sven Auhagen <sven.auhagen@voleatech.de>

log_error/log_warn: fix stack and format traceback

since we added the debug_handler we are not setting any level
on the root logger, the level is set for each individual handler

log_error and log_warning were using traceback.print_stack and print_exc
which and also only checked the rooter logger's level via getEffectiveLevel

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: vxlan: remote-ip: purge entry removed by user

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: vxlan: vxlan-vni: support vni change on existing config

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: vxlan: add support for l3vxi via vxlan-vni

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

sonar: addons: addressvirtual: Replace this formatting argument with a tuple

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: Fix vrrp addrgenmode

Prior commit (from me) that changed ipv6_addrgenmode for VRRP macvlan
interfaces from RANDOM --> NONE incorrectly disabled this for vrrp6.
The IPv6 link-local address is used as the SIP for vrrp6 hellos, so we
need to make sure that we are only setting addrgenmode to NONE for vrrp4.

Signed-off-by: Trey Aspelund <taspelund@nvidia.com>
Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: set addrgenmode to none for vrrp macvlan

A VRRP macvlan interface should only have addresses matching the
VIP defined by the user.  This change ensures we set ipv6 addrgenmode
to none, that way there won't be any unexpected (from user standpoint)
ipv6 link-local addresses to cause confusion or erroneous Type-2 EVPN
routes (in the case of advertise-default-gw with VRRP instead of VRR)
that advertise IP addresses that don't represent a VIP.

Signed-off-by: Trey Aspelund <taspelund@nvidia.com>
Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: address: ifquery-running: add hwaddress support

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: address: policy: l3_intf_arp_accept accept arp_accept value

the policy l3_intf_arp_accept now accepts boolean value True(1)/False(0) as well
as normal arp_accept values

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

sbin: start-networking: remove dhclient lease files on shutdown

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

log: use default chmod for os.mkdir

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

update ifupdown2 debug logging to /var/log/ifupdown2

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

log: add extra try except when removing log dirs and fix eni.d cp

The extra try/except are necessary just in case something goes wrong
we still want to go through the entire list of extra log dir present
on the system.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

ifupdown2.conf: add persistent debug logging feature

enable persistent ifupdown2 debug logs
ifupdown2 will keep debug logs in /etc/network/ifupdown2/logs
by default the last 42 configurations logs will be kept.
 yes     - (default) enable persistent logging (42 configs)
 no      - disable persistent logging
 [2-9]+  - specify how many configuration logs should be stored
enable_persistent_debug_logging=yes

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

sonar: addons: vrf: debug log formatting issue

Sonar error:
"Replace this formatting argument with a tuple."

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: vrf: unable to create vrf-max-count (missing +1)

A missing "+1" in a loop condition prevents ifupdown2 from creating
N vrfs up to vrf-max-count. We are stuck at vrf-max-count - 1

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: vlan: validate vlan interface name

we have an automation test that adds a vrf named "VRF.TEST":

$ ifquery -a
auto VRF.TEST
iface VRF.TEST
    vrf-table auto

$ ifup -a
error: could not determine vlanid
warning: error while writing to file /sys/class/net/VRF.TEST/mtu: [Errno 22] Invalid argument
$

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: mstpctl: forcevers: add valid vals: stp/mstp

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: vni's portmcrouter shouldn't reset to default if policy is set

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: prevent multiple vxlans to be added to the same VLAN

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: l2 vni evpn: don't warn for static vxlans

Due to missing checks, ifupdown2 may display the following warning for
static vxlan configs.

"warning: possible mis-configuration detected: l2-vni configured
with bridge-learning ON while EVPN is also configured - these two
parameters conflict with each other."

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: syntax-check: user defined vni per svi limit

error: vx-1002: misconfiguration detected: maximum vni allowed per bridge (bridge) svi (1000) is limited to 1 (policy: 'bridge_vni_per_svi_limit')

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: bridge-portmcrouter: reset to default 1 (automatic) if config is removed

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bridge: bridge-hashmax: upper limit of validrange to 65536

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: ifquery-check: es-sys-mac value is not part of netlink dump for non-root run

For some reason es-sys-mac (IFLA_BOND_AD_ACTOR_SYSTEM) is not part
of the netlink dump if requested by non-root user. This commit adds
a log info and will ignore es-sys-mac in that case.

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

bond: ifquery-check: translate user config es-sys-mac before comparing to netlink cache

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: address: clear stale svi fdb entries when svi mac is changed

Three issues:
- when `hwaddress` is removed from an SVI (on top of a bridge), the svi
mac is not resetted back to the bridge mac: fixed
- when `hwaddress` is used on an svi, stale fdb enties were added with the
bridge mac: fixed
- on ifdown we were leaving a leftover fdb entry on the deleted svi: fixed

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: bond: fix ifquery for IFLA_BOND_ARP_IP_TARGET

dropping support for multiple ip in bond-arp-ip-target

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

Merge pull request #138 from svenauhagen/feature/bondarpcheck

This patch adds the ability to use an ARP IP for the bond check.

addons: bridge: allow vlan sub interface in vlan-aware bridge (fixes #92)

New bridge policy:
allow-vlan-sub-interface-in-vlan-aware-bridge (default True)

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Julien Fortin <jfortin@nvidia.com>

addons: address: log error but continue when adding addresses (fixes #130)

Signed-off-by: bauen1 <j2468h@gmail.com>
Signed-off-by: Julien Fortin <jfortin@nvidia.com>

Merge pull request #234 from sohorx/fix/dhcp_client_wait

dhcp: fix dhclient client killed on no wait

addons: dhcp: move policy log info to debug

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

dhcp: fix dhclient client killed on no wait

When a policy dhcp-wait is false the dhcp client is run directly in
background.

But, in most cases, the client won't have the time to find an address
before a check occur.

This is making the client being killed nearly every time.

The obvious solution here is to not check ips differences when no-wait
is asked by the policy.

This patch adds the ability to use an ARP IP for the bond check.

It is only valid for balance-rr and balance-xor.

One Example is:

auto bond0
iface bond0 inet
        bond-slaves ens21 ens22
        bond-mode balance-rr
        bond-arp-interval 100
        bond-arp-ip-target 8.8.8.8
        address 10.10.10.1/24

Signed-off-by: Sven Auhagen <sven.auhagen@voleatech.de>

Merge pull request #190 from tjjh89017/fix_systemd_start_order

networking.service: fix start networking.service before network is marked online

addons: address: dad-interval: adding log.info before sleep (#232)

Signed-off-by: Julien Fortin <jfortin@nvidia.com>

Merge pull request #232 from sohorx/dad_handling_per_itf

Dad handling per itf

address: check interface exists before settling dad

addons: address: update modinfo for dad attributes

Help infos came from the ifupdown (original) manual.

addons: address: up: wait on dad when needed

addons: address: add a settle dad method

_settle_dad will allow ifupdown2 to wait on ipv6 dad tentative.

addons: address: disable dad on no attempts

addons: address: add dad handling module global

Upon ipv6_dad_handling_enabled module global parameter set to true, the
address addon will handle ipv6 dad tentatives by either
* set nodad when adding an ipv6 address
* waiting for dad to finish or warn of a timeout

This handling should be quick close to the ifupdown process.

nlcache: add nodad parameter to addr_add

nlpacket: change IFA_FLAGS to a FourByte attribute

AttributeGeneric cannot be used unless having it's length defined.

This change allow address flags such as optimistic, nodad or others
to be set with the add_attribute packet method.

Merge pull request #231 from CumulusNetworks/revert-230-dad_handling

Revert "Dad handling"

Revert "Dad handling"

Merge pull request #230 from sohorx/dad_handling

addons: address: add support for DAD