Skip to content

rbd: enable WholeObject & IncludeParent in snap diff#6279

Merged
mergify[bot] merged 1 commit into
ceph:develfrom
Rakshith-R:diff-whole-object
May 19, 2026
Merged

rbd: enable WholeObject & IncludeParent in snap diff#6279
mergify[bot] merged 1 commit into
ceph:develfrom
Rakshith-R:diff-whole-object

Conversation

@Rakshith-R

Copy link
Copy Markdown
Contributor

Enable WholeObject and IncludeParent flags in DiffIterateByIDConfig
used by ProcessMetadata, matching the existing DiffIterate config
in getUsedBytes. WholeObject improves performance by reporting
whole-object granularity diffs instead of byte-level diffs.
IncludeParent ensures diffs against parent snapshots are included.

Assisted-by: Claude noreply@anthropic.com

@mergify mergify Bot added the component/rbd Issues related to RBD label May 18, 2026
@mergify

mergify Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏

@Rakshith-R Rakshith-R force-pushed the diff-whole-object branch from a806782 to b5a7f5a Compare May 18, 2026 10:09
@iPraveenParihar iPraveenParihar added the ci/skip/e2e skip running e2e CI jobs label May 19, 2026
@iPraveenParihar

Copy link
Copy Markdown
Contributor

Adding skip/e2e label since we don't have e2e for this Today.

@mergify

mergify Bot commented May 19, 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 19, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • Entered queue2026-05-19 05:52 UTC · Rule: default
  • Checks passed · in-place
  • Merged2026-05-19 11:42 UTC · at b999c6097410e6396a871915d222b04c3eecb394 · rebase

This pull request spent 5 hours 49 minutes 42 seconds in the queue, including 5 hours 48 minutes 59 seconds running CI.

Required conditions to merge

Enable WholeObject and IncludeParent flags in DiffIterateByIDConfig
used by ProcessMetadata, matching the existing DiffIterate config
in getUsedBytes. WholeObject improves performance by reporting
whole-object granularity diffs instead of byte-level diffs.
IncludeParent ensures diffs against parent snapshots are included.

Assisted-by: Claude <noreply@anthropic.com>
Signed-off-by: Rakshith R <rar@redhat.com>
@mergify mergify Bot added the queued label May 19, 2026
@mergify mergify Bot merged commit 6ab2551 into ceph:devel May 19, 2026
19 of 20 checks passed
@mergify mergify Bot removed the queued label May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs component/rbd Issues related to RBD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants