Skip to content

Conversation

@YiteGu
Copy link
Member

@YiteGu YiteGu commented Aug 14, 2025

rbdVol already support qos base on capacity, in reality, performance doesn't always scale linearly with capacity. To address this, we've added max configurations to allow users to set a maximum QoS limit for a volume. Once the maximum limit is reached, further capacity increases will no longer improve performance. These configurations as below:

  • maxIops
  • maxReadIops
  • maxWriteIops
  • maxBps
  • maxReadBps
  • maxWriteBps

Describe what this PR does

Provide some context for the reviewer

Is there anything that requires special attention

Do you have any questions?

Is the change backward compatible?

Are there concerns around backward compatibility?

Provide any external context for the change, if any.

For example:

  • Kubernetes links that explain why the change is required
  • CSI spec related changes/catch-up that necessitates this patch
  • golang related practices that necessitates this change

Related issues

Mention any github issues relevant to this PR. Adding below line
will help to auto close the issue once the PR is merged.

Fixes: #issue_number

Future concerns

List items that are not part of the PR and do not impact it's
functionality, but are work items that can be taken up subsequently.

Checklist:

  • Commit Message Formatting: Commit titles and messages follow
    guidelines in the developer
    guide
    .
  • Reviewed the developer guide on Submitting a Pull
    Request
  • Pending release
    notes

    updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

@mergify mergify bot added the component/rbd Issues related to RBD label Aug 14, 2025
@YiteGu YiteGu force-pushed the add-qos-max-limit branch 3 times, most recently from 42f1a7d to b5d86ed Compare August 14, 2025 08:12
Copy link
Collaborator

@Madhu-1 Madhu-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YiteGu as we are adding lot of changes to it in think we need to do the basic E2E testing, can you please update e2e test for nbd driver to test IOPS and make sure everything works

@YiteGu
Copy link
Member Author

YiteGu commented Aug 14, 2025

@YiteGu as we are adding lot of changes to it in think we need to do the basic E2E testing, can you please update e2e test for nbd driver to test IOPS and make sure everything works

Get

@YiteGu YiteGu force-pushed the add-qos-max-limit branch from b5d86ed to 8f08fd8 Compare August 14, 2025 13:48
rbdVol already support qos base on capacity, in reality,
performance doesn't always scale linearly with capacity.
To address this, we've added max configurations to allow
users to set a maximum QoS limit for a volume. Once the
maximum limit is reached, further capacity increases will
no longer improve performance. These configurations as below:
- maxIops
- maxReadIops
- maxWriteIops
- maxBps
- maxReadBps
- maxWriteBps

Signed-off-by: Yite Gu <[email protected]>
@YiteGu YiteGu force-pushed the add-qos-max-limit branch from 8f08fd8 to 92ac16f Compare August 14, 2025 14:04
@YiteGu
Copy link
Member Author

YiteGu commented Aug 14, 2025

/test ci/centos/mini-e2e/k8s-1.31

@YiteGu YiteGu requested a review from Madhu-1 August 15, 2025 02:30
Copy link
Member

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@nixpanic nixpanic requested a review from a team August 15, 2025 09:46
@iPraveenParihar
Copy link
Contributor

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Aug 18, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at f0a905f

mergify bot added a commit that referenced this pull request Aug 18, 2025
@mergify mergify bot merged commit f0a905f into ceph:devel Aug 18, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/rbd Issues related to RBD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants