Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updating pytest for sflow #1095

Merged
merged 1 commit into from
Oct 31, 2019
Merged

Updating pytest for sflow #1095

merged 1 commit into from
Oct 31, 2019

Conversation

dgsudharsan
Copy link
Collaborator

@dgsudharsan dgsudharsan commented Oct 12, 2019 •
edited
Loading

What I did
Update pytests for sflow

Why I did it
The updated pytests test the flow from cfg_db to sflowmgr to sfloworch to asic_db. Thus end to end flow is tested

How I verified it
Execute the tests

Details if related
DO NOT MERGE UNTIL THE BELOW PULL REQUEST IS MERGED
sonic-net/sonic-buildimage#3595

@dgsudharsan
Copy link
Collaborator Author

sudo pytest -s -v --dvsname=vs test_sflow.py
========================================================= test session starts =========================================================
platform linux2 -- Python 2.7.15rc1, pytest-3.3.0, py-1.8.0, pluggy-0.6.0 -- /usr/bin/python
cachedir: .cache
rootdir: /home/sudharsan/pytest, inifile:
collected 6 items

test_sflow.py::TestSflow::test_defaultGlobal remove extra link dummy
PASSED [ 16%]
test_sflow.py::TestSflow::test_globalAll PASSED [ 33%]
test_sflow.py::TestSflow::test_InterfaceSet PASSED [ 50%]
test_sflow.py::TestSflow::test_defaultRate PASSED [ 66%]
test_sflow.py::TestSflow::test_ConfigDel PASSED [ 83%]
test_sflow.py::TestSflow::test_Teardown PASSED [100%]
====================================================== 6 passed in 71.33 seconds ======================================================

@prsunny
Copy link
Collaborator

prsunny commented Oct 28, 2019

@dgsudharsan , can you fix the conflict?

@dgsudharsan
Copy link
Collaborator Author

retest vs please

@dgsudharsan
Copy link
Collaborator Author

retest this please

2 similar comments
@dgsudharsan
Copy link
Collaborator Author

retest this please

@dgsudharsan
Copy link
Collaborator Author

retest this please

@prsunny prsunny merged commit 5ab3f6b into sonic-net:master Oct 31, 2019
wendani
wendani reviewed Oct 7, 2020
View reviewed changes
tests/test_sflow.py

def test_ConfigDel(self, dvs, testlog):
self.setup_sflow(dvs)
ptbl = swsscommon.ProducerStateTable(self.pdb, "SFLOW_SESSION_TABLE")
ctbl = swsscommon.Table(self.cdb, "SFLOW_SESSION_TABLE")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

schema for config_db is SFLOW_SESSION not SFLOW_SESSION_TABLE

oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
@smaheshm
[submodule update] SAI submodule update to v1.6.3 (sonic-net#628)
ef72159 
- Update SAI VoQ support (sonic-net#1107) …
- Voq system (sonic-net#1081) …
- [meta] Add support for ignored enum values (sonic-net#1099)
- TPID SAI proposal (sonic-net#1089) …
- ACL GRE key match (sonic-net#1076) …
- Add IPv6 NS and NA Traps (sonic-net#1092) …
- MACsec flow list attribute added in MACsec object (sonic-net#1095) …
- Add Enterprise Number for IPFIX Report Type (sonic-net#1072) …
- Provide TTL and QoS treatment during MPLS encap and decap (sonic-net#1079)
- Create and Set for Tunnel Attributes (sonic-net#1086) …
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
@liushilongbuaa
Transfer organization from Azure to sonic-net (sonic-net#1095)
90ba09a 
Transfer organization from Azure to sonic-net
@dgsudharsan dgsudharsan deleted the sflow_tests branch March 9, 2023 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wendani wendani wendani left review comments

@prsunny prsunny prsunny approved these changes

Assignees

@prsunny prsunny

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dgsudharsan @prsunny @wendani