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

jewel: rgw: object expirer's hints might be trimmed without processing in some circumstances #10763

Merged
merged 3 commits into from Aug 26, 2016
Merged

Conversation

ghost
Copy link

@ghost ghost commented Aug 17, 2016

Fixes: http://tracker.ceph.com/issues/16684
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 83cd785)
Wei Qiaomiao has found that expired objects, which should be removed by
the object expirer of RadosGW, might be left unprocessed till next
restart of a RadosGW's instance. This happens when process_single_shard
method of RGWObjectExpirer class exhaust a time slot for a single round
and finishes without informing caller about the situation.

Fixes: http://tracker.ceph.com/issues/16705
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 99f7d6e)
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit bc1ecdf)
@ghost ghost self-assigned this Aug 17, 2016
@ghost ghost added this to the jewel milestone Aug 17, 2016
@ghost ghost added bug-fix rgw labels Aug 17, 2016
ghost pushed a commit that referenced this pull request Aug 17, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 22, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 22, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 23, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 24, 2016
…e trimmed without processing in some circumstances

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 25, 2016
…e trimmed without processing in some circumstances

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

ghost commented Aug 25, 2016

@oritwas does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/16344#note-38 (the failure was due to #10832 which has been removed from the branch so that the failed test can be run).

@ghost ghost assigned oritwas Aug 25, 2016
@oritwas oritwas merged commit 12b374f into ceph:jewel Aug 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants