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

Test CI #1

Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open

Test CI #1

wants to merge 44 commits into from

Conversation

husni-faiz
Copy link
Owner

No description provided.

The OCI spec defines an Image Index concept to handle
multiple manifests in an OCI image. This adds the interface
for Image Index.

Signed-off-by: Husni Faiz <[email protected]>
remote.Image uses descriptor to get the image. Getting the descriptor
first and deriving the image from the descriptor is efficient instead
of getting the descriptor twice.

Signed-off-by: Husni Faiz <[email protected]>
AppendManifest and AnnotateManifest functions are now refactored
to be ImageIndex functions.

Signed-off-by: Husni Faiz <[email protected]>
When creating a new local index, if the the referenced images are
not in the same registry as the registry we intend to push the index
we copy the images to the same registry as the index.

NOTE: In future we plan to store the image reference names locally
and only copy the images when push the index to registry.

Signed-off-by: Husni Faiz <[email protected]>
This fixes the following errors,
- use strings.ReplaceAll method instead of strings.Replace
- `infromation` is a misspelling of `information`
- var-naming: don't use underscores in Go names
- unnecessary conversion (unconvert)
- ineffectual assignment to err
- unnecessary trailing newline
- unnecessary leading newline

Signed-off-by: Husni Faiz <[email protected]>
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.

1 participant