Replace redis-cli with sonic-db-cli for the test cases could be run for image with or without ENABLE_MULTIDB=y#15084
Replace redis-cli with sonic-db-cli for the test cases could be run for image with or without ENABLE_MULTIDB=y#15084Pan-XT wants to merge 3 commits intosonic-net:masterfrom
Conversation
…or image with or without ENABLE_MULTIDB=y
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
|
The pre-commit check detected issues in the files touched by this pull request. Detailed pre-commit check results: To run the pre-commit checks locally, you can follow below steps:
|
Description of PR
Modify /tests/iface_namingmode/test_iface_namingmode.py and /tests/generic_config_updater/test_mmu_dynamic_threshold_config_update.py
Replace redis-cli with sonic-db-cli for the test cases could be run for image with or without ENABLE_MULTIDB=y
Summary:
Fixes # (issue)
Type of change
Approach
What is the motivation for this PR?
sonic-mgmt tests for multi DB
How did you do it?
Replace redis-cli with sonic-db-cli
How did you verify/test it?
For test_iface_namingmode.py, run
./run_tests.sh -n vms-kvm-t0 -d vlab-01 -c iface_namingmode/test_iface_namingmode.py -f vtestbed.yaml -i ../ansible/veos_vtb -e "--neighbor_type=sonic"
in image with or without ENABLE_MULTIDB=y, results are both
38 passed, 22 skipped, 1 warning
For test_mmu_dynamic_threshold_config_update.py, run
./run_tests.sh -n vms-kvm-t0 -d vlab-01 -c generic_config_updater/test_mmu_dynamic_threshold_config_update.py -f vtestbed.yaml -i ../ansible/veos_vtb -e "--neighbor_type=sonic"
in image with or without ENABLE_MULTIDB=y, results are both
generic_config_updater/test_mmu_dynamic_threshold_config_update.py::test_dynamic_th_config_updates[replace] PASSED [100%]
instead of SKIPPED