-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat(CAS): AWS S3 blob storage support #390
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -84,7 +84,8 @@ func resourceName(digest string) string { | |||
// Exists check that the artifact is already present in the repository | |||
func (b *Backend) Exists(ctx context.Context, digest string) (bool, error) { | |||
_, err := b.Describe(ctx, digest) | |||
if err != nil && errors.As(err, &backend.ErrNotFound{}) { | |||
notFoundErr := &backend.ErrNotFound{} | |||
if err != nil && errors.As(err, ¬FoundErr) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please provide more info why this change here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The context is that this wasn't working before, as the error wasn't cast properly. I noticed it because of the tests in AWS which hit a similar code.
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>
Add support for AWS S3 as Content Addressable Storage.
NOTE: The implementation is very similar to the Azure one #357. The main enhancement in this case in comparison to Azure is that we can rely on sha256 integrity verification on their side easily.
Register storage backend in your Control plane
and metadata set
Closes #269