-
Notifications
You must be signed in to change notification settings - Fork 1
Set up Single Layer OCI Artifacts #75
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
Comments
#### What this PR does / why we need it With this PR an in-cluster instance of `zot` OCI registry will be deployed aside to the controllers: * `make deploy-dev` makes `zot` accessible via http. * `make deploy` makes `zot` accessible via https. #### Which issue(s) this PR fixes This PR is in the context of #75 and should be merged before #98
Initially started to create a snapshot CR and a respective controller. But later adjusted the architecture to omit the intermediate CR. |
See further details in the ADR #90 |
Since everything here seems done, can we merge / move this to main? |
There is one missing change from @ikhandamirov that we will review today. Afterwards we will check for any leftovers and then open the related PR for review |
What would you like to be added:
Implementation for setting up single layer artifact containing the blobs (or, in other words, the content of what are currently called Artifacts).
Why is this needed:
So, in both cases described in the epic, we need an implementation to set up oci artifacts containing the corresponding blob content and store them in an oci registry. The common approach for this (which is also chosen by helm for setting up helm charts as oci artifacts) is to create single layer oci artifacts.
See #76
artifact
v1.0.0+dev
has to be mapped tov1.0.0.build-dev
to be reachable with OCMsnapshot
controller[ ] Populatesnapshot
controller testssnapshot
CR and controller, and extend status ofcomponent
,resource
andconfiguredResource
with respective fieldsresource
,controller to make sure the storage migration is working.localisation
, andconfiguration
The text was updated successfully, but these errors were encountered: