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

osd: avoid FORCE updating digest been overwritten by MAYBE when comparing scrub map #7051

Merged
merged 2 commits into from Feb 24, 2016

Conversation

wonzhq
Copy link
Contributor

@wonzhq wonzhq commented Dec 24, 2015

Should set update to FORCE after MAYBE, to avoid it been overwritten.

@wonzhq wonzhq changed the title osd: avoid FORCE updating digest been overwritten by MAYBE osd: avoid FORCE updating digest been overwritten by MAYBE when comparing scrub map Dec 24, 2015
Should set update to FORCE after MAYBE, to avoid it been overwritten.

Signed-off-by: Zhiqiang Wang <wonzhq@hotmail.com>
@tchaikov
Copy link
Contributor

@wonzhq f302e05 looks good to me. but i have a different opinion regarding to b4ad817.

@tchaikov tchaikov assigned wonzhq and unassigned tchaikov Feb 15, 2016
Signed-off-by: Zhiqiang Wang <wonzhq@hotmail.com>
@wonzhq
Copy link
Contributor Author

wonzhq commented Feb 15, 2016

@tchaikov updated, pls take another look, thx!

@tchaikov
Copy link
Contributor

lgtm with a rados qa run.

liewegas added a commit that referenced this pull request Feb 24, 2016
osd: avoid FORCE updating digest been overwritten by MAYBE when comparing scrub map

Reviewed-by: Kefu Chai <kchai@redhat.com>
@liewegas liewegas merged commit f053e3e into ceph:master Feb 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants