Skip to content

Adding sflowtool to ptf docker to for sflow tests#4045

Merged
lguohan merged 1 commit intosonic-net:masterfrom
chitra-raghavan:sflowtool
Jan 23, 2020
Merged

Adding sflowtool to ptf docker to for sflow tests#4045
lguohan merged 1 commit intosonic-net:masterfrom
chitra-raghavan:sflowtool

Conversation

@chitra-raghavan
Copy link
Copy Markdown
Contributor

Adding sflowtool to ptf docker for running sflow tests in sonic-mgmt repository

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Jan 21, 2020

retest this please

@chitra-raghavan
Copy link
Copy Markdown
Contributor Author

retest vsimage please

2 similar comments
@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Jan 22, 2020

retest vsimage please

@chitra-raghavan
Copy link
Copy Markdown
Contributor Author

retest vsimage please

@lguohan lguohan merged commit c6fee49 into sonic-net:master Jan 23, 2020
lguohan pushed a commit to sonic-net/sonic-mgmt that referenced this pull request Feb 4, 2020
How did you do it?
- The test will run on the t0 testbed. The script is validated in sonic-vs image with vms-kvm-t0 topology loaded.
- 1 port from each port channel is used for sending traffic. Sflow is enabled on these 4 ports
- 2 ports from Vlan1000 are removed and assigned with ip address and used to reach sflow collector in ptf docker .
- Traffic is sent from ptf docker with different packet sizes to the port channel interfaces which are enabled with sflow .
- Collection is implemented using the sflowtool. Counter sampling output and flow sampling output are directed to a text file. The test script parses the text file and validates the data according to the polling/sampling rate configured and the interfaces enabled.

For this test to run sflowtool to be installed in ptf docker sonic-net/sonic-buildimage#4045

Test plan Document : sonic-net/SONiC#554
@chitra-raghavan chitra-raghavan deleted the sflowtool branch February 7, 2020 10:59
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