Skip to content

rgw: add replication status header to s3 GetObj response#47743

Merged
ivancich merged 1 commit intoceph:mainfrom
cbodley:wip-rgw-replication-status
Jan 22, 2023
Merged

rgw: add replication status header to s3 GetObj response#47743
ivancich merged 1 commit intoceph:mainfrom
cbodley:wip-rgw-replication-status

Conversation

@cbodley
Copy link
Copy Markdown
Contributor

@cbodley cbodley commented Aug 22, 2022

implements the easy part of amazon's x-amz-replication-status feature: https://docs.aws.amazon.com/AmazonS3/latest/userguide/replication-status.html#replication-status-overview

also adds rgw header x-rgw-replicated-from for each zone in the replication trace

Fixes: https://tracker.ceph.com/issues/58565

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@cbodley cbodley requested a review from mattbenjamin August 22, 2022 20:59
@cbodley cbodley force-pushed the wip-rgw-replication-status branch from 6fbe2cb to 39fd491 Compare August 22, 2022 21:16
Copy link
Copy Markdown
Contributor

@mattbenjamin mattbenjamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@cbodley cbodley force-pushed the wip-rgw-replication-status branch from 39fd491 to 3e7b251 Compare September 29, 2022 20:58
@cbodley cbodley marked this pull request as ready for review September 29, 2022 20:59
implements the easy part of amazon's x-amz-replication-status feature:

https://docs.aws.amazon.com/AmazonS3/latest/userguide/replication-status.html#replication-status-overview

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley cbodley force-pushed the wip-rgw-replication-status branch from 3e7b251 to ca762e6 Compare January 17, 2023 16:44
@cbodley cbodley changed the title rgw: add replication status headers to s3 GetObj response rgw: add replication status header to s3 GetObj response Jan 17, 2023
@cbodley
Copy link
Copy Markdown
Contributor Author

cbodley commented Jan 17, 2023

also adds rgw header x-rgw-replicated-from for each zone in the replication trace

this part wasn't working, so i split it out to #49767

@mattbenjamin
Copy link
Copy Markdown
Contributor

woot

@ivancich ivancich added the wip-eric-testing-1 for ivancich testing label Jan 19, 2023
@ivancich
Copy link
Copy Markdown
Member

jenkins test make check

@ivancich ivancich merged commit 4130679 into ceph:main Jan 22, 2023
@mattbenjamin
Copy link
Copy Markdown
Contributor

@ivancich thank you!

@ivancich
Copy link
Copy Markdown
Member

@cbodley and @mattbenjamin : merged; I'm thinking this should have backports....

@ivancich ivancich removed wip-eric-testing-1 for ivancich testing needs-qa labels Jan 22, 2023
@mattbenjamin
Copy link
Copy Markdown
Contributor

It would be helpful in Quincy, I think

@ivancich
Copy link
Copy Markdown
Member

Thanks, @mattbenjamin. I've added a tracker for this PR, listed quincy as a desired backport, and set status to Pending Backport.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants