kytos-ng / kytos-end-to-end-tests

Kytos End-to-End Tests
0 stars 10 forks source link

[Fix] Fixed reconnect switches side effect and decreased `STATS_INTERVAL` #190

Closed viniarck closed 1 year ago

viniarck commented 1 year ago

Fixes #188 Fixes #189

I locally explored with the test test_040_replace_action_flow that @italovalcy reported:

+ sed -i 's/STATS_INTERVAL = 60/STATS_INTERVAL = 7/g' /var/lib/kytos/napps/kytos/of_core/settings.py
+ sed -i 's/CONSISTENCY_MIN_VERDICT_INTERVAL =.*/CONSISTENCY_MIN_VERDICT_INTERVAL = 60/g' /var/lib/kytos/napps/kytos/flow_manager/settings.py
+ sed -i 's/LINK_UP_TIMER = 10/LINK_UP_TIMER = 1/g' /var/lib/kytos/napps/kytos/topology/settings.py
+ sed -i 's/DEPLOY_EVCS_INTERVAL = 60/DEPLOY_EVCS_INTERVAL = 5/g' /var/lib/kytos/napps/kytos/mef_eline/settings.py
+ sed -i 's/LLDP_LOOP_ACTIONS = \["log"\]/LLDP_LOOP_ACTIONS = \["disable","log"\]/' /var/lib/kytos/napps/kytos/of_lldp/settings.py
+ sed -i 's/LLDP_IGNORED_LOOPS = {}/LLDP_IGNORED_LOOPS = {"00:00:00:00:00:00:00:01": \[\[4, 5\]\]}/' /var/lib/kytos/napps/kytos/of_lldp/settings.py
+ sed -i 's/CONSISTENCY_COOKIE_IGNORED_RANGE =.*/CONSISTENCY_COOKIE_IGNORED_RANGE = [(0xdd00000000000000, 0xdd00000000000009)]/g' /var/lib/kytos/napps/kytos/flow_manager/settings.py
+ sed -i 's/LIVENESS_DEAD_MULTIPLIER =.*/LIVENESS_DEAD_MULTIPLIER = 3/g' /var/lib/kytos/napps/kytos/of_lldp/settings.py

========== 1 passed, 198 deselected, 35 warnings in 93.78s (0:01:33) ===========
iteration 5
+ for i in {1..5}
+ echo 'iteration 5'
+ python3 -m pytest tests/ --timeout=300 -k test_040_replace_action_flow
============================= test session starts ==============================
platform linux -- Python 3.9.2, pytest-7.2.0, pluggy-1.0.0
rootdir: /tests
plugins: rerunfailures-10.2, timeout-2.1.0
timeout: 300.0s
timeout method: signal
timeout func_only: False
collected 199 items / 198 deselected / 1 selected

tests/test_e2e_20_flow_manager.py .                                      [100%]

=============================== warnings summary ===============================

I'll dispatch an e2e full test execution with this branch. I'll post the results here later

viniarck commented 1 year ago

LGTM

Thank for the reviews and suggestions, Italo. The e2e execution is running on GitLab, once it passes I'll merge this PR

20221202_120742

viniarck commented 1 year ago

Tests have passed:

end-to-end repo branch fix/reconnect_switches
$ git log -1
commit b5a7d84d04de194621b52b4603670c1b85560f4e
Author: Vinicius Arcanjo <viniarck@gmail.com>
Date:   Fri Dec 2 11:49:09 2022 -0300
    Decreased STATS_INTERVAL to 7
$ python3 ./scripts/gitlab_rs_init.py
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
------------------------------- start/stop times -------------------------------
= 172 passed, 2 skipped, 18 xfailed, 7 xpassed, 793 warnings in 10342.03s (2:52:22) =