Skip to content

doc: update the qos design doc#6264

Merged
mergify[bot] merged 1 commit into
ceph:develfrom
Madhu-1:update-qos-design
May 8, 2026
Merged

doc: update the qos design doc#6264
mergify[bot] merged 1 commit into
ceph:develfrom
Madhu-1:update-qos-design

Conversation

@Madhu-1

@Madhu-1 Madhu-1 commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Updating the RBD qos design doc to consider only VAC for the implementation.

@Madhu-1 Madhu-1 added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures labels May 7, 2026
@Madhu-1 Madhu-1 requested review from a team and Rakshith-R May 7, 2026 04:30
@mergify mergify Bot added the component/docs Issues and PRs related to documentation label May 7, 2026
Rakshith-R
Rakshith-R previously approved these changes May 7, 2026
View reviewed changes

@Rakshith-R Rakshith-R left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To identify the right cgroup file, we need pod UUID and container UUID from the pod yaml output

Let's add details on how CephCSI driver will get this information during nodePublish too ?
I guess that will complete the whole picture.

@Madhu-1
doc: update the qos design doc
912801a 
Updating the RBD qos design doc to
consider only VAC for the implementation.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1 Madhu-1 force-pushed the update-qos-design branch from 3bc2269 to 912801a Compare May 8, 2026 05:34
@Madhu-1

Madhu-1 commented May 8, 2026

Copy link
Copy Markdown
Collaborator Author

To identify the right cgroup file, we need pod UUID and container UUID from the pod yaml output

Updated the details

@Madhu-1 Madhu-1 requested review from a team and Rakshith-R May 8, 2026 05:34
@mergify mergify Bot dismissed Rakshith-R’s stale review May 8, 2026 05:35

Pull request has been modified.

Rakshith-R
Rakshith-R approved these changes May 8, 2026
View reviewed changes

@Rakshith-R Rakshith-R left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks !

@mergify

mergify Bot commented May 8, 2026

Copy link
Copy Markdown
Contributor

Deprecation notice: This pull request comes from a fork and was queued with update_method=rebase and update_bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the merge queue will no longer be able to rebase fork pull requests with this configuration. To avoid disruption, switch to update_method=merge in your queue rule.

@mergify

mergify Bot commented May 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-05-08 11:19 UTC · Rule: default
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-05-08 11:20 UTC · at 912801a1e7b53d71a1792829401ebfdb3f5b3e3e · rebase

This pull request spent 15 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit 97ec3b4 into ceph:devel May 8, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@black-dragon74 black-dragon74 black-dragon74 approved these changes

@Rakshith-R Rakshith-R Rakshith-R approved these changes

Assignees

No one assigned

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Madhu-1 @black-dragon74 @Rakshith-R