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: cache agent is idle although one object is left in the cache #5765

Merged
2 commits merged into from Sep 7, 2015
Merged

osd: cache agent is idle although one object is left in the cache #5765

2 commits merged into from Sep 7, 2015

Conversation

ghost
Copy link

@ghost ghost commented Sep 2, 2015

@ghost ghost self-assigned this Sep 2, 2015
@ghost ghost added this to the hammer milestone Sep 2, 2015
@ghost ghost added bug-fix core labels Sep 2, 2015
ghost pushed a commit that referenced this pull request Sep 2, 2015
…left in the cache

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Sep 6, 2015

@tchaikov this hammer backport passed a rados suite (see http://tracker.ceph.com/issues/12701#note-62 for the details). Do you think it is ready to be merged ?

When a proxy read happens, the object promotion is done in parallel. The
agent_choose_mode function must be called to reconsider the situation
to protect against the following scenario:

  * proxy read
  * agent_choose_mode finds no object exists and the agent
    goes idle
  * object promotion happens
  * the agent does not reconsider and eviction does not happen
    although it should

http://tracker.ceph.com/issues/12673 Fixes: #12673

Signed-off-by: Loic Dachary <ldachary@redhat.com>
(cherry picked from commit e1f58fe)
Verify that an object promoted to a cache tier because of a proxy read
is evicted as expected.

http://tracker.ceph.com/issues/12673 Refs: #12673

Signed-off-by: Loic Dachary <ldachary@redhat.com>
(cherry picked from commit 7924231)
@ghost ghost assigned tchaikov and unassigned ghost Sep 6, 2015
@tchaikov
Copy link
Contributor

tchaikov commented Sep 7, 2015

👍

@tchaikov tchaikov assigned ghost and unassigned tchaikov Sep 7, 2015
ghost pushed a commit that referenced this pull request Sep 7, 2015
cache agent is idle although one object is left in the cache

Reviewed-by: Kefu Chai <kchai@redhat.com>
@ghost ghost merged commit e61fdc6 into ceph:hammer Sep 7, 2015
@ghost ghost changed the title cache agent is idle although one object is left in the cache osd: cache agent is idle although one object is left in the cache Sep 13, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants