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: add 'proxy' cache mode #8210

Merged
merged 5 commits into from Mar 30, 2016
Merged

osd: add 'proxy' cache mode #8210

merged 5 commits into from Mar 30, 2016

Conversation

liewegas
Copy link
Member

The 'forward' mode is flawed for the same reason that redirects
are flawed: they allow the reordering of requests. Add a proxy
mode that can be used instead of forward. (Later we can consider
dropping forward entirely?)

Fixes: #12814
Signed-off-by: Sage Weil sage@redhat.com

@liewegas
Copy link
Member Author

ceph/ceph-qa-suite#906

@athanatos
Copy link
Contributor

lgtm

@liewegas liewegas changed the title DNM: osd: add 'proxy' cache mode osd: add 'proxy' cache mode Mar 28, 2016
The 'forward' mode is flawed for the same reason that redirects
are flawed: they allow the reordering of requests.  Add a proxy
mode that can be used instead of forward.  (Later we can consider
dropping forward entirely?)

Fixes: ceph#12814
Signed-off-by: Sage Weil <sage@redhat.com>
Only none, writeback, proxy, and readproxy are robust and
supported.  Require a force flag to use any of the others.

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit 3255208 into ceph:master Mar 30, 2016
@liewegas liewegas deleted the wip-12814 branch March 30, 2016 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants