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

[POC] Improve routeorch to stop process routes when high priority notification coming. #3278

Closed

Conversation

liuh-80
Copy link
Contributor

@liuh-80 liuh-80 commented Sep 4, 2024

Please ignore this POC

@liuh-80
Copy link
Contributor Author

liuh-80 commented Sep 5, 2024

/azpw run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-swss

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liuh-80 liuh-80 changed the title [POC] limit routeorch to only process 128 routes per doTask [POC] Improve routeorch to stop process routes and handle high priority notification. Sep 5, 2024
@liuh-80 liuh-80 changed the title [POC] Improve routeorch to stop process routes and handle high priority notification. [POC] Improve routeorch to stop process routes when high priority notification coming. Sep 5, 2024
@saksarav-nokia
Copy link
Contributor

@liuh-80 , Is this PR ready to try in our test beds?

@liuh-80
Copy link
Contributor Author

liuh-80 commented Sep 9, 2024

@liuh-80 , Is this PR ready to try in our test beds?

@saksarav-nokia , no, there still many sonic-mgmt test case failed:
sonic-net/sonic-buildimage#20143

I will investigation the reason and fix it this week

@liuh-80
Copy link
Contributor Author

liuh-80 commented Sep 9, 2024

Working on verify sonic-mgmt test case failure with this PR: sonic-net/sonic-buildimage#20143

@liuh-80 liuh-80 closed this Sep 11, 2024
@liuh-80
Copy link
Contributor Author

liuh-80 commented Sep 11, 2024

This approach can pass all sonic-swss test case but can't pass sonic-mgmt test case because performance issue.
Will try another approach: https://github.com/sonic-net/sonic-swss/pull/3287/files

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.

3 participants