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

blueprint downloader #375

Merged

Conversation

fabianburth
Copy link
Contributor

What this PR does / why we need it:
Add a downloader for landscaper blueprints to the library.

The standard access methods simply return the raw blob data. Therefore, if the artifact (in this case, a blueprint) is stored as an OCI Artifact, the access method will return the whole OCI Artifact bytes. Correspondingly, if the artifact is stored as a local artifact with its Component Descriptor as a compressed archive (e.g. .tar.gzip), the compressed archive bytes are returned.
Thus, to access the actual artifact (in this case, the blueprint), further processing (e.g. decompressing and unpacking) is required. This blueprint downloader capsules this functionality for blueprints.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release Notes:

added blueprint downloader

@fabianburth fabianburth force-pushed the blueprint-downloader branch from 761f258 to 9fd1347 Compare June 7, 2023 09:34
@fabianburth fabianburth requested a review from mandelsoft June 9, 2023 06:21
@fabianburth fabianburth force-pushed the blueprint-downloader branch from 0cae4cc to 2f97ef3 Compare June 22, 2023 11:09
@fabianburth fabianburth merged commit b581bb4 into open-component-model:main Jun 30, 2023
@fabianburth fabianburth deleted the blueprint-downloader branch June 30, 2023 09:17
robertwol pushed a commit that referenced this pull request Sep 25, 2023
* initial implementation of a blueprint downloader

* add tests for download and registration

* utilize printer within downloader implementation

* fix linter issues

* fix linter issues

---------

Co-authored-by: Gerald Morrison <67469729+morri-son@users.noreply.github.com>
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.

2 participants