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.

Sign up for GitHub

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

Jump to bottom

[BACKPORT][v1.5.6][IMPROVEMENT] Investigate performance bottleneck in v1 data path #8510

Closed
github-actions bot opened this issue May 6, 2024 · 5 comments
Closed

[BACKPORT][v1.5.6][IMPROVEMENT] Investigate performance bottleneck in v1 data path #8510

github-actions bot opened this issue May 6, 2024 · 5 comments
Assignees
@PhanLe1010 @roger-ryao
Labels
area/benchmark Performance Benchmark related kind/backport Backport request kind/improvement Request for improvement of existing function priority/0 Must be fixed in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Milestone
v1.5.6

Comments

@github-actions
Copy link

github-actions bot commented May 6, 2024

backport #8436
@github-actions github-actions bot added area/benchmark Performance Benchmark related kind/backport Backport request kind/improvement Request for improvement of existing function priority/0 Must be fixed in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated labels May 6, 2024
@github-actions github-actions bot added this to the v1.5.6 milestone May 6, 2024
@PhanLe1010
Copy link
Contributor

Ref PR: longhorn/longhorn-engine#1105

@roger-ryao
Copy link

longhorn/longhorn-engine#1105 has not been merged yet, and I change the status to implement.

@PhanLe1010
Copy link
Contributor

Sorry, there is a buid error on that PR. Will check it later

@PhanLe1010
Copy link
Contributor

PR was merged

@roger-ryao roger-ryao self-assigned this May 9, 2024
@roger-ryao
Copy link

Verified on v1.5.x-head 20240509

The test steps
#8436 (comment)

Result Passed

  1. In the AWS EC2 c5d.2xlarge environment, the summary tablet as follows
IOPS (Random Read/Write) IOPS (Sequential Read/Write)
v1.5.5-1st 26,287 / 13,118 42,366 / 22,478
v1.5.5-2nd 26,318 / 13,002 42,919 / 22,483
15x-head-1st 26,641 / 16,269 42,131 / 25,603
15x-head-2nd 26,849 / 16,139 42,993 / 26,378
Bandwidth KiB/sec (Random Read/Write) Bandwidth KiB/sec (Sequential Read/Write)
v1.5.5-1st 967,165 / 161,368 630,371 / 163,677
v1.5.5-2nd 1,012,821 / 161,977 685,921 / 164,216
15x-head-1st 1,031,153 / 161,721 579,175 / 164,007
15x-head-2nd 1,038,343 / 161,674 672,242 / 163,987
Latency ns (Random Read/Write) Latency ns (Sequential Read/Write)
v1.5.5-1st 414,989 / 589,467 417,373 / 602,674
v1.5.5-2nd 413,160 / 464,340 416,674 / 462,306
15x-head-1st 416,907 / 495,902 419,262 / 490,225
15x-head-2nd 416,831 / 475,264 413,764 / 474,407

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PhanLe1010 PhanLe1010

@roger-ryao roger-ryao

Labels
area/benchmark Performance Benchmark related kind/backport Backport request kind/improvement Request for improvement of existing function priority/0 Must be fixed in this release (managed by PO) require/backport Require backport. Only used when the specific versions to backport have not been definied. require/doc Require updating the longhorn.io documentation require/manual-test-plan Require adding/updating manual test cases if they can't be automated
Projects
None yet
Milestone
v1.5.6
Development

No branches or pull requests
2 participants
@PhanLe1010 @roger-ryao