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

add resource consumption testing result #67

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

PerforMance308
Copy link

Collected CPU usage metrics in two scenarios:

  • Users' pod has CPU limit
  • Users' pod doesn't have CPU limit

Collected Memory consumption

@kmesh-bot
Copy link
Collaborator

Welcome @PerforMance308! It looks like this is your first PR to kmesh-net/website 🎉

Copy link

netlify bot commented Aug 29, 2024

Deploy Preview for kmesh-net ready!

Name Link
🔨 Latest commit 3f481d4
🔍 Latest deploy log https://app.netlify.com/sites/kmesh-net/deploys/671b9a39d09a6000089fb067
😎 Deploy Preview https://deploy-preview-67--kmesh-net.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@hzxuzhonghu hzxuzhonghu left a 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

  1. First lack of how the test is run, assume this is ads mode
  2. The cpu usage monitor is very corse, should use pod monitor
  3. 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]>
@PerforMance308
Copy link
Author

This tests need some improvements

  1. First lack of how the test is run, assume this is ads mode
  2. The cpu usage monitor is very corse, should use pod monitor
  3. 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

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

Copy link
Member

@hzxuzhonghu hzxuzhonghu left a 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]>
@kmesh-bot
Copy link
Collaborator

[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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Shiqi Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants