-
Notifications
You must be signed in to change notification settings - Fork 24
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
add resource consumption testing result #67
Conversation
Welcome @PerforMance308! It looks like this is your first PR to kmesh-net/website 🎉 |
✅ Deploy Preview for kmesh-net ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Shiqi Wang <[email protected]>
b0d7b67
to
a7cf8d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This tests need some improvements
- First lack of how the test is run, assume this is ads mode
- The cpu usage monitor is very corse, should use pod monitor
- About the memory test, I cannot get how you get the usage. Should supplement the procedure,
For the cpu test, should distinguish different conccurrent connections, qps
Signed-off-by: Shiqi Wang <[email protected]>
the purpose to conduct this test is to get the Kmesh maximum CPU and momery usage, and see if there are any limitations of that. So the connections, qps metrics are not considered in this test |
Signed-off-by: Shiqi Wang <[email protected]>
Signed-off-by: Shiqi Wang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The graph does not show up can you reference other article
This is the preview https://deploy-preview-67--kmesh-net.netlify.app/en/docs/performance/resource_consumption/
Signed-off-by: shiqi wang <[email protected]>
Signed-off-by: shiqi wang <[email protected]>
Signed-off-by: Shiqi Wang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hzxuzhonghu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Collected CPU usage metrics in two scenarios:
Collected Memory consumption