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

rgw: ListMultipartUploads returns the real upload Owners #43715

Merged
merged 4 commits into from Nov 3, 2021

Commits on Oct 28, 2021

  1. rgw/rest: dump_time() takes real_time by value

    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    cbodley committed Oct 28, 2021
    Configuration menu
    Copy the full SHA
    ace857f View commit details
    Browse the repository at this point in the history
  2. rgw/rest: dump_owner() takes name by const ref

    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    cbodley committed Oct 28, 2021
    Configuration menu
    Copy the full SHA
    200f9a0 View commit details
    Browse the repository at this point in the history
  3. rgw/sal: MultipartUpload::get_mtime() is const, returns by value

    also added override specifiers in RadosMultipartUpload
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    cbodley committed Oct 28, 2021
    Configuration menu
    Copy the full SHA
    b05e92c View commit details
    Browse the repository at this point in the history
  4. rgw/sal: add MultipartUpload::get_owner() for ListMultipartUploads

    the ListMultipartUploads response includes an Owner and Initiator for
    each multipart upload in the list, but we were just returning the
    requesting user for every upload instead
    
    we get the real owner field out of rgw_bucket_dir_entry when listing the
    uploads, so need to store that with MultipartUpload for later use in
    RGWListBucketMultiparts_ObjStore_S3::send_response()
    
    Fixes: https://tracker.ceph.com/issues/52716
    
    Signed-off-by: Casey Bodley <cbodley@redhat.com>
    cbodley committed Oct 28, 2021
    Configuration menu
    Copy the full SHA
    9ee191d View commit details
    Browse the repository at this point in the history