-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Ensure New Layers Match Image Media Type #2700
Ensure New Layers Match Image Media Type #2700
Conversation
test: add to ensure media type being saved
90b78ef
to
feea05e
Compare
Thanks for the PR here @loganprice! Just tested this using the repro identified in #1836 and looks like it’s working great! Adding my testing results below for others who might be looking at this: Build an OCI base image named
Results in the following
Then an image is built
Where
Using kaniko @ HEAD (w/o this PR) the results incorrectly have
Using kaniko w/ this PR the results corretly have a
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the PR here @loganprice!
Fixes #1836 in case of a bug fix, this should point to a bug and any other related issue(s)
Description
Before adding a layer to an image check the images manifest media type and then add the layer with the corresponding media type.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide for more details.
Reviewer Notes
Release Notes
Describe any changes here so maintainer can include it in the release notes, or delete this block.