-
Notifications
You must be signed in to change notification settings - Fork 6k
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
hammer: osd: bug with cache/tiering and snapshot reads #6589
Conversation
reset ssc->exsits in finish_ctx() if the ctx->cache_evict is true, and the head is removed. Fixes: ceph#12748 Signed-off-by: Kefu Chai <kchai@redhat.com> (cherry picked from commit b61f3e4)
Since the upstream PR (#6261) needed an additional test case in the qa suite, for the backport PR, it may be necessary to add the specific test to the ceph-qa-suite as well before merging this one. |
…merged, needs a test in ceph-qa-suite] Reviewed-by: Loic Dachary <ldachary@redhat.com>
…merged, needs a test in ceph-qa-suite] Reviewed-by: Loic Dachary <ldachary@redhat.com>
…merged, needs a test in ceph-qa-suite] Reviewed-by: Loic Dachary <ldachary@redhat.com>
…merged, needs a test in ceph-qa-suite] Reviewed-by: Loic Dachary <ldachary@redhat.com>
It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/13356#note-28 for details ). teuthology-openstack --verbose --key-name myself --key-filename ~/Downloads/myself --verbose --suite rados --ceph-qa-suite-git-url https://github.com/dachary/ceph-qa-suite.git --suite-branch wip-12748-hammer --ceph-git-url https://github.com/ceph/ceph.git --ceph hammer-backports --filter="export-after-evict.yaml"
Using a backport of the test ceph/ceph-qa-suite#720 |
@tchaikov it passes a rados suite and a backport of the matching test (see #6589 (comment)). Does it look good to merge ? |
lgtm! |
osd: bug with cache/tiering and snapshot reads Reviewed-by: Kefu Chai <kchai@redhat.com>
http://tracker.ceph.com/issues/13693