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.

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

update to scale numbers #369

Merged
merged 1 commit into from May 24, 2023

Conversation

mgheorghe
Copy link
Collaborator

  • background tcp flows 15M (was 1M)
  • background udp flows 15M (was 1M)
  • background traffic packet size reduced to 500B from 1500B
  • total flow capacity reduced by half (down to 32M from 64M) -
    Signed-off-by: Mircea Dan mircea-dan.gheorghe@keysight.com

- background tcp flows 15M (was 1M)
- background udp flows 15M (was 1M)
- background traffic packet size reduced to 500B from 1500B
- total flow capacity reduced by half (down to 32M from 64M)
-
Signed-off-by: Mircea Dan <mircea-dan.gheorghe@keysight.com>
@KrisNey-MSFT
Copy link
Collaborator

1M and 1M of Background traffic is not taxing the table even1%. Trying to keep the table full of connections that are sending a packet every second; the goal is to have 32M connections that are all Active. Discussed w/ @GeraldatMicrosoft May 24, 2023. The flow capacity (of 64M tested) was w/64GB of memory, not 32GB - hence the decrease in flow capacity of 32M total connections (Active and Backup) w/32GB of memory.

@KrisNey-MSFT KrisNey-MSFT merged commit 8862076 into sonic-net:main May 24, 2023
2 checks passed
@lguohan
Copy link
Contributor

lguohan commented May 24, 2023

@mgheorghe , i am reverting this pr. please reformat the pr, it is unclear what exactly change this pr contains.

lguohan added a commit that referenced this pull request May 24, 2023
@mgheorghe
Copy link
Collaborator Author

@mgheorghe , i am reverting this pr. please reformat the pr, it is unclear what exactly change this pr contains.

@lguohan in github if you click on rich diff it shows the changes very clear

image

image

also it looks good in other diff tools (not sure why standard github is not able to show properly the diffs

image

@lguohan
Copy link
Contributor

lguohan commented May 30, 2023

@mgheorghe , why do i need to do rich diff? why normal text diff won't work?

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.

None yet

3 participants