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

octopus: librbd/object_map: rbd diff between two snapshots lists entire image content #43806

Merged
merged 2 commits into from Jan 21, 2022

Conversation

sunnyku
Copy link
Contributor

@sunnyku sunnyku commented Nov 4, 2021

backport tracker: https://tracker.ceph.com/issues/53027


backport of #41657
parent tracker: https://tracker.ceph.com/issues/50787

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

…content

Problem:
rbd diff between two snapshots lists entire image content with `whole-object` switch.

Solution:
While computing resize diff during object map do not mark state as new data
unless diff state is valid.

Fixes: https://tracker.ceph.com/issues/50787
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
(cherry picked from commit 399a45e)
…f enable

This patch adds a test condition to test fast diff between two snapshots
with whole-object switch.

`TestMockObjectMapDiffRequest.EndDelta` expects diff state
`DIFF_STATE_DATA_UPDATED` which is an incorrect expectation when diff is calculated
between two snapshots.

Fixes: https://tracker.ceph.com/issues/50787

Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
(cherry picked from commit 8d62c4a)
@sunnyku sunnyku added this to the octopus milestone Nov 4, 2021
@sunnyku sunnyku added the rbd label Nov 4, 2021
@sunnyku sunnyku requested a review from trociny November 4, 2021 14:19
@yuriw
Copy link
Contributor

yuriw commented Jan 4, 2022

Pls add needs-qa when all is fixed ref: https://trello.com/c/41G4eFjp

@idryomov
Copy link
Contributor

@yuriw I looked at that card and I don't understand why this PR was deemed to be the problem.

@yuriw yuriw merged commit 8795262 into ceph:octopus Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants