Skip to content

Add OCI labels - #749

Closed
lucacome wants to merge 1 commit into
nginx:masterfrom
lucacome:add-labels-oci
Closed

lucacome wants to merge 1 commit into
nginx:masterfrom
lucacome:add-labels-oci

Conversation

@lucacome

@lucacome lucacome commented Feb 1, 2023

Copy link
Copy Markdown
Contributor

@lucacome
lucacome requested a review from thresheek February 1, 2023 03:11
@thresheek

thresheek commented Feb 1, 2023 •

Copy link
Copy Markdown
Member

Hello @lucacome !

Given the fact that we also mirror those images to other registries, e.g. AWS ECR Public, how should we deal with org.opencontainers.image.base.name?

@lucacome

lucacome commented Feb 1, 2023

Copy link
Copy Markdown
Contributor Author

@thresheek the base images are always coming from docker.io right?

@thresheek

Copy link
Copy Markdown
Member

debian:slim and alpine are indeed coming from docker.io, correct.

@tianon

tianon commented Feb 1, 2023

Copy link
Copy Markdown
Contributor

@lucacome

Copy link
Copy Markdown
Contributor Author

@tianon so is the suggestion to not add any labels? But if we want to add them OCI is the way to go?

@tianon

tianon commented Feb 18, 2023

Copy link
Copy Markdown
Contributor

Yeah, I personally avoid LABEL entirely, but if you do want to use them, they should be compliant to the recommendations for annotations in the OCI's image-spec.

@lucacome lucacome closed this Oct 17, 2023
@lucacome
lucacome deleted the add-labels-oci branch October 17, 2023 12:11
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.

3 participants