Skip to content

[vstest]: fix test_speed.py test#780

Merged
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:testspeed
Feb 1, 2019
Merged

[vstest]: fix test_speed.py test#780
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:testspeed

Conversation

@lguohan
Copy link
Contributor

@lguohan lguohan commented Feb 1, 2019

now, Force10-S6000 vs env has port speed set in port_config.ini.
Thus, dynamic pg profile is added to the initial list. Extend
the test to support this new scenario.

Signed-off-by: Guohan Lu [email protected]

What I did

Why I did it

How I verified it

Details if related

now, Force10-S6000 vs env has port speed set in port_config.ini.
Thus, dynamic pg profile is added to the initial list. Extend
the test to support this new scenario.

Signed-off-by: Guohan Lu <[email protected]>
@lguohan
Copy link
Contributor Author

lguohan commented Feb 1, 2019

pass my local test. depends on sonic-net/sonic-buildimage#2515

lgh@gulv-vm1:/data/sonic/sonic-swss/tests$ sudo py.test test_speed.py::TestSpeedSet::test_SpeedAndBufferSet --dvsname=eag
er_agnesi
================================================== test session starts ==================================================
platform linux2 -- Python 2.7.12, pytest-3.3.1, py-1.5.2, pluggy-0.6.0
rootdir: /data/sonic/sonic-swss/tests, inifile:
collected 1 item                                                                                                        

test_speed.py .                                                                                                   [100%]

=============================================== 1 passed in 32.57 seconds ===============================================

@lguohan
Copy link
Contributor Author

lguohan commented Feb 1, 2019

retest this please

1 similar comment
@lguohan
Copy link
Contributor Author

lguohan commented Feb 1, 2019

retest this please

@lguohan lguohan merged commit a2b987b into sonic-net:master Feb 1, 2019
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
now, Force10-S6000 vs env has port speed set in port_config.ini.
Thus, dynamic pg profile is added to the initial list. Extend
the test to support this new scenario.

Signed-off-by: Guohan Lu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants