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

Remove blobstore from manifest builder #3896

Merged
merged 2 commits into from May 19, 2023

Conversation

davidspek
Copy link
Collaborator

Rebase from #2782

This PR aims to close #2150 by removing blobstore from schema2 manifest builder.

The builder now take configuration Descriptor as parameter instead of a blobstore and media type.

@milosgajdos friendly ping since you were the one to review the original PR.

@davidspek
Copy link
Collaborator Author

Friendly ping @thaJeztah @squizzi

Copy link
Collaborator

@squizzi squizzi left a comment

Choose a reason for hiding this comment

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

lgtm

testutil/manifests.go Outdated Show resolved Hide resolved
Copy link
Member

@milosgajdos milosgajdos left a comment

Choose a reason for hiding this comment

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

LGTM. Would be nice to address the nits, though.

@davidspek
Copy link
Collaborator Author

@milosgajdos I’ve addressed the nit.

glefloch and others added 2 commits May 19, 2023 13:47
Signed-off-by: glefloch <glfloch@gmail.com>
Signed-off-by: David van der Spek <vanderspek.david@gmail.com>
Co-authored-by: Kyle Squizzato <ksquizz@gmail.com>
Signed-off-by: David van der Spek <vanderspek.david@gmail.com>
@milosgajdos milosgajdos merged commit 983358f into distribution:main May 19, 2023
12 checks passed
@davidspek davidspek deleted the clean-blobstore-rebase branch June 28, 2023 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove blobstore for schema2 builder
4 participants