librbd: S3-based migration source #38068
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
dillaman
force-pushed
the
wip-librbd-migration-4
branch
from
dc1d367
to
37757e9
Compare
dillaman
force-pushed
the
wip-librbd-migration-4
branch
from
37757e9
to
03d2de9
Compare
added 5 commits
November 23, 2020 08:45
This will allow the logic to be re-used between the standard HTTP stream and the future S3 stream. Signed-off-by: Jason Dillaman <[email protected]>
The S3 authentication scheme requires adding additional headers to the outbound request as well as computing a secure signature for the request using the secret key. Signed-off-by: Jason Dillaman <[email protected]>
Move the processing of common HTTP result codes (not found, permission denied, etc) higher up the stack so it can be re-used between all request paths. Signed-off-by: Jason Dillaman <[email protected]>
The new S3 stream requires a fully qualified URL to a bucket (i.e. no virtual host names) along with the access and secret keys to the bucket. Signed-off-by: Jason Dillaman <[email protected]>
Signed-off-by: Jason Dillaman <[email protected]>
dillaman
force-pushed
the
wip-librbd-migration-4
branch
from
03d2de9
to
8ee92b8
Compare
|
(rebased to include Clang fix) |
|
Is documentation on this feature available (or in a separate commit) ? |
Documentation will be added before Pacific (and integration tests pending resolution of various Sepia lab issues) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume tox