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: ERROR: got unexpected error when trying to read object: -2 #11472

Merged
1 commit merged into from Oct 27, 2016
Merged

jewel: rgw: ERROR: got unexpected error when trying to read object: -2 #11472

1 commit merged into from Oct 27, 2016

Conversation

ghost
Copy link

@ghost ghost commented Oct 13, 2016

    When accessing a copied destination object, its source object's instance ID
    information is needed, however it's missing now in the destination object's
    manifest.

    In order to fix this problem, we can record source object's version_id/instance
    into dest object's manifest(a new filed 'tail_instance' is added). When creating
    a new object(not copy), 'tail_instance' should be equal to its instance value.
    When copy/get a object, 'tail_instance' should always be used to get the right
    tail objects.

    Fixes: http://tracker.ceph.com/issues/17111
    Signed-off-by: Yang Honggang <joseph.yang@xtaotech.com>

(cherry picked from commit d43b69e)
@ghost ghost self-assigned this Oct 13, 2016
@ghost ghost added this to the jewel milestone Oct 13, 2016
@ghost ghost added bug-fix rgw labels Oct 13, 2016
ghost pushed a commit that referenced this pull request Oct 17, 2016
…ying to read object: -2

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 18, 2016
…ying to read object: -2

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned yehudasa Oct 20, 2016
@ghost
Copy link
Author

ghost commented Oct 20, 2016

@yehudasa does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/17487#note-18

@yehudasa
Copy link
Member

@dachary lgtm

@ghost ghost merged commit 3c9fe54 into ceph:jewel Oct 27, 2016
@theanalyst theanalyst changed the title jewel: ERROR: got unexpected error when trying to read object: -2 jewel: rgw: got unexpected error when trying to read object: -2 Nov 17, 2016
@theanalyst theanalyst changed the title jewel: rgw: got unexpected error when trying to read object: -2 "jewel: rgw: ERROR: got unexpected error when trying to read object: -2" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: rgw: ERROR: got unexpected error when trying to read object: -2" jewel: rgw: ERROR: got unexpected error when trying to read object: -2 Nov 17, 2016
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
2 participants