Age | Commit message (Collapse) | Author |
|
|
|
(cherry picked from commit 1a814661a0ade01f144398b91dd6998e42018fdd)
|
|
|
|
There is no need to delete the old ingres-qos and egres-qos values as it's not
a multi node and thus the values are simply overwritten. Also address validation
is not required as it's done in a dedicates test.
|
|
|
|
|
|
This commit also extends the smoketest to verify that the exception for this
error is raised.
|
|
|
|
level
Getting rid of "set firewall options" and move it from:
set firewall options interface ethX adjust-mss 1400
set firewall options interface ethX adjust-mss6 1400
to:
set interfaces ethernet ethX ip adjust-mss 1400
set interfaces ethernet ethX ipv6 adjust-mss 1400
In addition add an extra option called clamp-mss-to-pmtu instead of a value.
|
|
interfaces
The MTU size of the source-interface must be greater or equal to the MTU of the
pseudo ethernet interface.
|
|
|
|
|
|
|
|
|
|
This is the smoketest for the fix added in commit cd504035 ("vyos.ifconfig:
T3532: re-create Q-in-Q interface on protocol change").
|
|
For an unknown reason this regularely fails on the QEMU builds, thus the test
for reading back IP addresses is temporary disabled. There is no big deal here,
as this uses the same methods on 802.1q and here it works and is verified.
|
|
|
|
Currently every smoketest does the setup and destruction of the configsession
on its own durin setUp(). This creates a lot of overhead and one configsession
should be re-used during execution of every smoketest script.
In addiion a test that failed will leaf the system in an unconsistent state.
For this reason before the test is executed we will save the running config
to /tmp and the will re-load the config after the test has passed, always
ensuring a clean environment for the next test.
|
|
|
|
When a VIF/VLAN interface is placed in admin down state but the lower
interface, serving the vlan, is moved from admin down -> admin up, all its
vlan interfaces will be placed in admin up state, too.
This is bad as a VLAN interface will become admin up even if its specified as
admin down after a reboot.
To reproduce:
set interfaces ethernet eth1 vif 20 disable
set interfaces ethernet eth1 disable
commit
delete interfaces ethernet eth1 disable
commit
Now check the interface state and it returns UP,LOWER_UP
7: eth1.20@eth1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default qlen 1000
link/ether 00:50:56:b3:09:07 brd ff:ff:ff:ff:ff:ff
inet6 fe80::250:56ff:feb3:907/64 scope link
valid_lft forever preferred_lft forever
|
|
There had been four implementations of "ip -d -j link show interface" scattered
accross the codebase. Those implementations have now been combined into a new
helper:
vyos.util.get_json_iface_options()
|
|
This reverts commit 1992d9aed31b6906806635687c2b2d2d52310990.
After commit b4889039 ("smoketest: refactor setUp() for all interface based
testcases") we can now use dynamic duid tests.
|
|
For a yet undetermined reason incrementing the duid dynamically in the test
script leads to an error - for the time beeing work with a fixed duid.
|
|
UnboundLocalError: local variable 'client_base' referenced before assignment
|
|
|
|
|
|
Commit 35b049aa ("smoketest: interfaces: verify deletion of interfaces")
validated that when the test completed (tearDown()) there must be no
interfaces left registered for the tests. This is invalid for the loopback
interface as it can not be deleted from the system.
|
|
When deleting the test interfaces, ensure after each run that there are no
leftovers from a previous run, indicating that there was a problem while removing
the interfaces.
|
|
Verify interface is in admin down state afterwards.
|
|
|
|
|
|
Previous fix somehow got lost during a rebase :(
|
|
|
|
|
|
Commit a8e4317c ("smoketest: interfaces: test dhcpv6 pd sla-id auto increment")
added a new test, but when executed on multiple interfaces, e.g.:
TEST_ETH="eth1 eth2" /usr/libexec/vyos/tests/smoke/cli/test_interfaces_ethernet.py
A variable was not properly reset
|
|
|
|
Commit ef629504d4 ("smoketest: mirror: drop testcase") dropped the smoketests
entirely as they have been programmed in a wrong way leading to outages on live
development systems which used dummy interfaces for management traffic.
|
|
|
|
The mirror smoketest does more harm then good. It creates interfaces which are
not cleaned up afterwards resulting and strange behavior on live systems.
The entrie interface mirror smoketest must be redone.
|
|
This reverts commit 9541355433e202fade4692851bffa33ba9d48f44.
|
|
One should not use hardcoded interfaces in this base class, rather rely on
self._options like every other test. This implementation breaks on my development
system ans also deletes my OSPF dummy mgmt interface :(.
|
|
|
|
Since the dependency problem has not been solved before,
if the monitoring interface does not exist when the
mirror rule is created, the execution will be abnormal
|
|
|
|
|
|
Commit 5f5b2808c0a ("ethernet: T3048: drop static smp-affinity for dynamic
performance tuning") but the CLI nodes have not been removed.
|
|
|
|
We had two places were the is_ip, is_ipv4 and is_ipv6 helpers had been defined.
All places now have been converged into vyos.template as they are used both
in the Jinja2 templates and also in our scripts.
|
|
|
|
Renamed using snippet below:
----------------------------
for file in $(find . -name "*.py")
do
sed -i "s/vyos_dict_search/dict_search/" $file
done
|